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
Backport 2.9: Handle cases where normal commit operation throws a prompt (#62132) #62191
Backport 2.9: Handle cases where normal commit operation throws a prompt (#62132) #62191
Conversation
The test
|
164ba7e
to
e6224a3
Compare
Build failed (third-party-check pipeline) integration testing with
|
recheck |
Build failed (third-party-check pipeline) integration testing with
|
@pabelanger we're repeatedly seeing connection issues in this PR's tests. Basically, the |
recheck |
Build failed (third-party-check pipeline) integration testing with
|
recheck |
We have made iosxr jobs non-voting now, until we have a new version of the appliance online. This means, status check will pass, if iosxr fails. But we should still manually check results to confirm new issue isn't exposed. |
Build succeeded (third-party-check pipeline).
|
# throws a prompt and we need to handle it before | ||
# proceeding further | ||
cmd_obj['prompt'] = '(C|c)onfirm' | ||
cmd_obj['answer'] = 'y' |
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.
This API is odd... it seems to be, if I encounter this prompt, give it the answer. But it doesn't require the prompt to occur. Since that's what is desired here, there's no bug and I'll merge it.... Just commenting because that's not the standard way to write an API and it took me a bit to analyze the code to be sure that it was actually doing this.
Merged for the 2.9.0rc1 release. |
Signed-off-by: NilashishC nilashishchakraborty8@gmail.com
(cherry picked from commit f176645)
Add changelog for iosxr prompt fix
Signed-off-by: NilashishC nilashishchakraborty8@gmail.com
SUMMARY
ISSUE TYPE
COMPONENT NAME
cliconf/iosxr.py