-
Notifications
You must be signed in to change notification settings - Fork 1.4k
isisd: Allow full no
form for domain-password
and area-password
#17725
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
isisd: Allow full no
form for domain-password
and area-password
#17725
Conversation
Before: ``` LR1.wue3(config)# router isis VyOS LR1.wue3(config-router)# no area-password clear % Unknown command: no area-password clear LR1.wue3(config-router)# no area-password clear foo % Unknown command: no area-password clear foo LR1.wue3(config-router)# ``` Closes FRRouting#17722 Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
It looks tremendously similar to #17724 |
@Mergifyio backport stable/10.2 stable/10.1 stable/10.0 stable/9.1 stable/9.0 |
✅ Backports have been created
|
Fix working as expected - IS-IS configuration properly removed via frr-reload when applying this change to stable/10.2 |
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
isisd: Allow full `no` form for `domain-password` and `area-password` (backport #17725)
isisd: Allow full `no` form for `domain-password` and `area-password` (backport #17725)
isisd: Allow full `no` form for `domain-password` and `area-password` (backport #17725)
isisd: Allow full `no` form for `domain-password` and `area-password` (backport #17725)
isisd: Allow full `no` form for `domain-password` and `area-password` (backport #17725)
Closes #17722