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

ospfd: mpls ldp-sync flag is not cleared if ldp-sync is disabled #16375

Closed
2 tasks done
c-po opened this issue Jul 13, 2024 · 0 comments · Fixed by #16376
Closed
2 tasks done

ospfd: mpls ldp-sync flag is not cleared if ldp-sync is disabled #16375

c-po opened this issue Jul 13, 2024 · 0 comments · Fixed by #16376
Labels
triage Needs further investigation

Comments

@c-po
Copy link
Contributor

c-po commented Jul 13, 2024

Description

When enabling mpls ldp-sync under router ospf FRR ospfd configures SET_FLAG(ldp_sync_info->flags, LDP_SYNC_FLAG_IF_CONFIG); so internally knowing "hey - LDP was enabled", but the flag persists beeing enabled even if ldp-sync is turned off.

At first glance this might look like a cosmetic issue, but the internal state flags should be in sync with the actual CLI configuration, even that the comment in the source states this:

frr/ospfd/ospf_ldp_sync.c

Lines 900 to 904 in 5203914

/* disable LDP-SYNC on an interface
* stop holddown timer if running
* restore ospf cost
*/
SET_FLAG(ldp_sync_info->flags, LDP_SYNC_FLAG_IF_CONFIG);

Version

* stable/9.1
* master

How to reproduce

Start MPLS ldp-sync

!
interface eth0.203
 ip ospf mpls ldp-sync
exit
!
router ospf
 ospf router-id 172.18.254.203
 log-adjacency-changes
 mpls ldp-sync
 mpls ldp-sync holddown 60
exit
!

Disable MPLS ldp-sync

!
interface eth0.203
 no ip ospf mpls ldp-sync
exit
!
router ospf
 no mpls ldp-sync
exit
!

Expected behavior

no ip ospf mpls ldp-sync is not show in the interface context

vtysh -c "show run"

!
interface eth0.203
exit
!
router ospf
 ospf router-id 172.18.254.203
 log-adjacency-changes
exit
!

Actual behavior

Interface keeps showing ldp-sync is disabled on vtysh, but the entire ldp-sync section is gone, this feels "wrong"

vtysh -c "show run"

!
interface eth0.203
 no ip ospf mpls ldp-sync
exit
!
router ospf
 ospf router-id 172.18.254.203
 log-adjacency-changes
exit
!

Additional context

No response

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 Jul 13, 2024
c-po added a commit to c-po/frr that referenced this issue Jul 13, 2024
When enabling "mpls ldp-sync" under "router ospf" ospfd configures
SET_FLAG(ldp_sync_info->flags, LDP_SYNC_FLAG_IF_CONFIG) so internally knowing
that the ldp-sync feature is enabled. However the flag is not cleared when
turning of the feature using "nompls ldp-sync"!

FRRouting#16375

Signed-off-by: Christian Breunig <[email protected]>
c-po added a commit to c-po/frr that referenced this issue Jul 16, 2024
When enabling "mpls ldp-sync" under "router ospf" ospfd configures
SET_FLAG(ldp_sync_info->flags, LDP_SYNC_FLAG_IF_CONFIG) so internally knowing
that the ldp-sync feature is enabled. However the flag is not cleared when
turning of the feature using "nompls ldp-sync"!

FRRouting#16375

Signed-off-by: Christian Breunig <[email protected]>
c-po added a commit to c-po/frr that referenced this issue Jul 17, 2024
When enabling "mpls ldp-sync" under "router ospf" ospfd configures
SET_FLAG(ldp_sync_info->flags, LDP_SYNC_FLAG_IF_CONFIG) so internally knowing
that the ldp-sync feature is enabled. However the flag is not cleared when
turning of the feature using "nompls ldp-sync"!

FRRouting#16375

Signed-off-by: Christian Breunig <[email protected]>
mergify bot pushed a commit that referenced this issue Jul 23, 2024
When enabling "mpls ldp-sync" under "router ospf" ospfd configures
SET_FLAG(ldp_sync_info->flags, LDP_SYNC_FLAG_IF_CONFIG) so internally knowing
that the ldp-sync feature is enabled. However the flag is not cleared when
turning of the feature using "nompls ldp-sync"!

#16375

Signed-off-by: Christian Breunig <[email protected]>
(cherry picked from commit 5a70378)
mergify bot pushed a commit that referenced this issue Jul 23, 2024
When enabling "mpls ldp-sync" under "router ospf" ospfd configures
SET_FLAG(ldp_sync_info->flags, LDP_SYNC_FLAG_IF_CONFIG) so internally knowing
that the ldp-sync feature is enabled. However the flag is not cleared when
turning of the feature using "nompls ldp-sync"!

#16375

Signed-off-by: Christian Breunig <[email protected]>
(cherry picked from commit 5a70378)
mergify bot pushed a commit that referenced this issue Jul 23, 2024
When enabling "mpls ldp-sync" under "router ospf" ospfd configures
SET_FLAG(ldp_sync_info->flags, LDP_SYNC_FLAG_IF_CONFIG) so internally knowing
that the ldp-sync feature is enabled. However the flag is not cleared when
turning of the feature using "nompls ldp-sync"!

#16375

Signed-off-by: Christian Breunig <[email protected]>
(cherry picked from commit 5a70378)
mergify bot pushed a commit that referenced this issue Jul 23, 2024
When enabling "mpls ldp-sync" under "router ospf" ospfd configures
SET_FLAG(ldp_sync_info->flags, LDP_SYNC_FLAG_IF_CONFIG) so internally knowing
that the ldp-sync feature is enabled. However the flag is not cleared when
turning of the feature using "nompls ldp-sync"!

#16375

Signed-off-by: Christian Breunig <[email protected]>
(cherry picked from commit 5a70378)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage Needs further investigation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant