-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
tests: add support for ospf instances with unified configs #17331
Conversation
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
0142a18
to
b1346e9
Compare
b1346e9
to
4cec12a
Compare
5b71132
to
78a716b
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.
hmm - no description, and nothing in the commit comment, about what this is trying to do.
so ... was it not working to have the test script say "I want to run an ospf with this command-line arg, pointing at the unified config file?"
if we think we're going to drive daemon start and daemon command-lines from the config, then I think we need an explicit "daemon x" command for that purpose: there are too many things that may need to be expressed on the daemon command-line to try to do that implicitly.
I tried the tests/topotests/lib changes with 2 instances and this PR worked fine for me. |
@mjstapp , sorry, this was a"quick" pr following a discussion on slack after @aceelindem raised the issue the he coudn't get ospf instances to work. This PR doesn't do anyting more/less than what is stated in the title. 1- This PR does NOT add support for unified config in the tests, that capability exists already I'm mostly interested in using unified frr.conf, so: |
Also, I do agree that it is good to come up with a way to be able to pass commands arguments to a daemon when using unified configs. I like to solve that problem, but it is out of scope for this PR. |
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.
looks good
need to look at new core ... |
Signed-off-by: Jafar Al-Gharaibeh <[email protected]>
78a716b
to
92ecd0a
Compare
rerunning failed test |
@Mergifyio backport stable/10.2 |
✅ Backports have been created
|
No description provided.