Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stop using a forked Rally #63

Open
mestery opened this issue Jul 1, 2016 · 8 comments
Open

Stop using a forked Rally #63

mestery opened this issue Jul 1, 2016 · 8 comments

Comments

@mestery
Copy link
Collaborator

mestery commented Jul 1, 2016

Our local Rally changes should be pushed upstream and we should not rely on a fork [1].

[1] http://ovn-scale-test.readthedocs.io/en/latest/install.html

@mestery
Copy link
Collaborator Author

mestery commented Jul 1, 2016

@l8huang @huikang @hzhou8 FYI on this one, I have opened this to track the merge back of our forked rally upstream.

@hzhou8
Copy link
Collaborator

hzhou8 commented Jul 1, 2016

@mestery Thanks for tracking this. The ongoing work of unbounding from openstack in Rally seems not yet completed. I saw this spec [1] merged, but some related works has been pending for a long time, e.g. [2], [3].

[1] https://review.openstack.org/#/c/244353/17
[2] https://review.openstack.org/#/c/294521/4
[3] https://review.openstack.org/#/c/294520/6

@huikang
Copy link
Collaborator

huikang commented Aug 7, 2016

@hzhou8 I remember months ago you or @l8huang mentioned that rally has been undergoing radical change to support more workload than openstack. This feature accommodates integrating rally-ovs to upstream rally. Is there any update on this? Thanks.

@huikang
Copy link
Collaborator

huikang commented Aug 7, 2016

@l8huang
Copy link
Collaborator

l8huang commented Aug 8, 2016

As I know, the work of unbound rally from openstack is still going on. If we want to stop to use https://github.com/l8huang/rally.git as early as possible, we may make changes on ovn-scale-test to make it depending on upstream rally.

@huikang
Copy link
Collaborator

huikang commented Aug 8, 2016

@l8huang Does that mean we need to rebase our change against the latest rally codebase? Thanks.

@l8huang
Copy link
Collaborator

l8huang commented Aug 8, 2016

We'd better rebase to a release, e.g 5.0, because the master stream may not stable. Any suggests?

@huikang
Copy link
Collaborator

huikang commented Aug 8, 2016

@l8huang not yet; it seems the only way to get rid of this issue. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants