-
Notifications
You must be signed in to change notification settings - Fork 12
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
ovn-tester: introduce ovn-ic #169
Conversation
4db21d0
to
931c96d
Compare
53b69bc
to
9571f55
Compare
dc42821
to
31ab73f
Compare
36f9ce2
to
2932cf3
Compare
6340665
to
4727b5b
Compare
4727b5b
to
bd4433a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a few minor comments, with those addressed the rest looks good for now (we still need to add support for multiple central nodes but we can do that as a follow up).
We should also not forget to remove the commit that uses the private ovn-fake-multinode branch:
ovn-heater: rely on private ovn-fake-multinode for the moment
0755d07
to
2b965ca
Compare
This is a preliminary patch to add ovn-ic support to ovn-tester. Signed-off-by: Lorenzo Bianconi <[email protected]>
…ing multiple az Signed-off-by: Lorenzo Bianconi <[email protected]>
Signed-off-by: Lorenzo Bianconi <[email protected]>
Signed-off-by: Lorenzo Bianconi <[email protected]>
Signed-off-by: Lorenzo Bianconi <[email protected]>
Signed-off-by: Lorenzo Bianconi <[email protected]>
Signed-off-by: Lorenzo Bianconi <[email protected]>
Signed-off-by: Lorenzo Bianconi <[email protected]>
Signed-off-by: Lorenzo Bianconi <[email protected]>
Signed-off-by: Lorenzo Bianconi <[email protected]>
Signed-off-by: Lorenzo Bianconi <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I force pushed once in order to:
- remove the commit that used the custom ovn-fake-multinode branch (not needed anymore as the PR has been merged there too)
- remove the accidentally added tags file.
The rest looked OK to me, thanks!
Add ovn-ic support to ovn-tester.