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

IS-IS: Unknown command when removing domain-password and area-password #17722

Open
2 tasks done
c-po opened this issue Dec 26, 2024 · 4 comments · May be fixed by #17725
Open
2 tasks done

IS-IS: Unknown command when removing domain-password and area-password #17722

c-po opened this issue Dec 26, 2024 · 4 comments · May be fixed by #17725

Comments

@c-po
Copy link
Contributor

c-po commented Dec 26, 2024

Description

When removing area-password or domain-password from ISIS using frr-reload it will error out

Version

stable/10.2
9e3f285

How to reproduce

Configure ISIS

!
router isis VyOS
 net 49.0001.1921.6800.1002.00
 area-password clear foo
 domain-password clear foo
exit
!

unconfigure area-password using frr-reload

cpo@LR1.wue3:~$ cat  no-area-pwd-isis.txt
!
router isis VyOS
 net 49.0001.1921.6800.1002.00
 domain-password clear foo
exit
!
sudo /usr/lib/frr/frr-reload.py --reload --debug --stdout no-area-pwd-isis.txt
...
2024-12-26 14:23:09,094   ERROR: Failed to execute router isis VyOS  no area-password clear foo exit
2024-12-26 14:23:09,094   ERROR: "router isis VyOS --  no area-password clear foo -- exit" we failed to remove this command
2024-12-26 14:23:09,094   ERROR: % Unknown command:  no area-password clear foo

Same also applies to domain-password

cpo@LR1.wue3:~$ cat no-domain-pwd-isis.txt
!
router isis VyOS
 net 49.0001.1921.6800.1002.00
 area-password clear foo
exit
!
cpo@LR1.wue3:~$ sudo /usr/lib/frr/frr-reload.py --reload --debug --stdout no-domain-pwd-isis.txt
...
2024-12-26 14:24:10,940   ERROR: Failed to execute router isis VyOS  no domain-password clear foo exit
2024-12-26 14:24:10,940   ERROR: "router isis VyOS --  no domain-password clear foo -- exit" we failed to remove this command
2024-12-26 14:24:10,940   ERROR: % Unknown command:  no domain-password clear foo
...

Expected behavior

No error

Actual behavior

% Unknown command: no  area-password clear
% Unknown command: no  area-password clear foo

Additional context

Same error can be observed from vtysh

LR1.wue3(config)# do sh run
Building configuration...

Current configuration:
!
frr version 10.2
frr defaults traditional
hostname LR1.wue3
service integrated-vtysh-config
!
router isis VyOS
 net 49.0001.1921.6800.1002.00
 area-password clear foo
 domain-password clear foo
exit
!
end
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)#

Checklist

  • I have searched the open issues for this bug.
  • I have not included sensitive information in this report.
@c-po c-po added the triage Needs further investigation label Dec 26, 2024
@c-po c-po changed the title is-is and frr-reload produce % Unknown command for domain-password and area-password IS-IS: Unknown command when removing domain-password and area-password Dec 26, 2024
@ton31337 ton31337 added isis and removed triage Needs further investigation labels Dec 26, 2024
ykholod pushed a commit to ykholod/frr that referenced this issue Dec 26, 2024
When removing router isis area-password and domain-password using
frr-reload.py system drops errors.
This is a fix for the issue FRRouting#17722:
FRRouting#17722

Signed-off-by: Yaroslav Kholod <y.kholod@vyos.io>
ton31337 added a commit to opensourcerouting/frr that referenced this issue Dec 26, 2024
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>
@ton31337
Copy link
Member

Could you try this fix #17725?

@c-po
Copy link
Contributor Author

c-po commented Dec 29, 2024

Could you try this fix #17725?

Thank you @ton31337 for the propesed fix. Unfortunately it only works half way:

LR1.wue3(config)# router isis VyOS
LR1.wue3(config-router)# no  area-password clear
% Command incomplete: no  area-password clear
LR1.wue3(config-router)# no  area-password clear foo
LR1.wue3(config-router)# no domain-password clear
% Command incomplete: no domain-password clear
LR1.wue3(config-router)# no domain-password clear bar
LR1.wue3(config-router)#

@ton31337
Copy link
Member

But in what case do you need a partial no form for these commands? E.g. domain-password clear can't be without an argument.

@c-po
Copy link
Contributor Author

c-po commented Dec 29, 2024

But in what case do you need a partial no form for these commands? E.g. domain-password clear can't be without an argument.

You're right. I also verified frr-reload behavior. It's now working as expected.

Thanks for the quick fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants