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

improvements for xdist mode #16827

Merged

Conversation

choppsv1
Copy link
Contributor

@choppsv1 choppsv1 commented Sep 14, 2024

  • fix spawning shells/vtysh on error in xdist mode

  • munet 0.14.14

    • Another improvement for remote CLI in xdist mode
  • munet 0.14.13

    • Improve remote CLI operation [improves xdist mode]
  • munet 0.14.12

    • Fix --stdout and --stderr munet CLI args
    • Adds retry_sleep (i.e., interval) parameter to native @Retry decorator

@frrbot frrbot bot added bugfix tests Topotests, make check, etc labels Sep 14, 2024
- Adds retry_sleep (i.e., interval) parameter to native @Retry decorator
- Fix --stdout and --stderr munet CLI args

Signed-off-by: Christian Hopps <[email protected]>
- Improve remote CLI operation [improves xdist mode]

Signed-off-by: Christian Hopps <[email protected]>
@choppsv1 choppsv1 force-pushed the chopps/fix-on-error-with-xdist branch 2 times, most recently from 8d12a0d to 85841e0 Compare September 14, 2024 03:33
@github-actions github-actions bot added size/L and removed size/M labels Sep 14, 2024
- improvement to remote CLI
- 0.14.15 fixed a small bug in 0.14.14

Signed-off-by: Christian Hopps <[email protected]>
- Also fix the above and CLI when running in munet native mode

Signed-off-by: Christian Hopps <[email protected]>
@choppsv1 choppsv1 force-pushed the chopps/fix-on-error-with-xdist branch from 85841e0 to 98aaeab Compare September 14, 2024 14:05
@Jafaral Jafaral merged commit 84c5035 into FRRouting:master Sep 14, 2024
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix master size/L tests Topotests, make check, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants