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

Revert "bgpd: fix default instance when leaving the hidden state (backport 10.3)" #18162 #18255

Merged
merged 6 commits into from
Feb 26, 2025

Conversation

Jafaral
Copy link
Member

@Jafaral Jafaral commented Feb 24, 2025

PR #18162 is a backport of #18119. The backport was merged assuming the original PR (#18119 ) has been merged. But that is not the case. I still see activity and fixes going in the original PR. Furthermore, dev/10.3 now fails CI.

@ton31337
Copy link
Member

Why is this a revert?

@Jafaral
Copy link
Member Author

Jafaral commented Feb 25, 2025

Why is this a revert?

@ton31337 I added a comment above.

@louis-6wind
Copy link
Contributor

hold on. I am providing a fix

Copy link
Contributor

@louis-6wind louis-6wind left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not OK to merge the revert. Here is the fix #18257

This reverts commit 85c5598.

Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
This reverts commit bb79a65.

Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
This reverts commit 8e04277.

Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
This reverts commit d9d74d3.

Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
This reverts commit a2c1956.

Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
This reverts commit 2066493.

Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
@louis-6wind
Copy link
Contributor

louis-6wind commented Feb 25, 2025

#18119 is now merged.

Continuing in #18257

@donaldsharp donaldsharp reopened this Feb 25, 2025
@donaldsharp donaldsharp merged commit 308daf8 into FRRouting:dev/10.3 Feb 26, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants