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

dryrun mode skip some config errors? #17264

Closed
ak503 opened this issue Oct 28, 2024 · 1 comment
Closed

dryrun mode skip some config errors? #17264

ak503 opened this issue Oct 28, 2024 · 1 comment

Comments

@ak503
Copy link
Contributor

ak503 commented Oct 28, 2024

u1-VirtualBox(config)# do show interface enp0s3 
Interface enp0s3 is up, line protocol is up
  Link ups:       0    last: (never)
  Link downs:     0    last: (never)
  vrf: default
  index 2 metric 0 mtu 1500 speed 1000 txqlen 1000
  flags: <UP,LOWER_UP,BROADCAST,RUNNING,MULTICAST>
  Type: Ethernet
  HWaddr: 08:00:27:74:86:7c
  inet 10.0.0.31/24
  inet6 fe80::373b:c15e:2bcd:7c1b/64
  Interface Type Other
  Interface Slave Type None
  protodown: off 
u1-VirtualBox(config)# router bgp 1
u1-VirtualBox(config-router)# neighbor 10.0.0.31 remote-as 2
% Can not configure the local system as neighbor     <--------- Its OK
u1-VirtualBox(config-router)# do show running-config 
Building configuration...

Current configuration:
!
frr version 10.1.1
frr defaults traditional
hostname u1-VirtualBox
no ip forwarding
no ipv6 forwarding
!
router bgp 1
exit

but if i want check bgp config via bgpd -C -f <path> then its ok

conf file:

router bgp 1
 neighbor 10.0.0.31 remote-as 2
exit

result:

# bgpd -C -f /usr/local/etc/frr/frr.conf
# echo "$?"
0
@eqvinox
Copy link
Contributor

eqvinox commented Oct 29, 2024

config dry-run is unfortunately only a syntax check, not a semantic check for the config. At some point in the future, we will be able to do better config checks through YANG and the validation stage there, but this needs a lot more work (even after basic YANG works, validation must be implemented) and won't happen anytime soon. Sorry about that, but right now there is no way to catch this before actually configuring.

@eqvinox eqvinox closed this as completed Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants