Skip to content

Create default BGPConfig at startup#12045

Merged
mazdakn merged 8 commits intoprojectcalico:masterfrom
mazdakn:default-bgpconfig
Mar 11, 2026
Merged

Create default BGPConfig at startup#12045
mazdakn merged 8 commits intoprojectcalico:masterfrom
mazdakn:default-bgpconfig

Conversation

@mazdakn
Copy link
Member

@mazdakn mazdakn commented Mar 9, 2026

Create default BGPConfiguration similar to FelixConfiguration at startup to simplify any operation on BGPConfig.

Operator PR: tigera/operator#4520

Needed by tigera/operator#4511

Release note:

TBD

Copilot AI review requested due to automatic review settings March 9, 2026 19:14
@mazdakn mazdakn requested a review from a team as a code owner March 9, 2026 19:14
@mazdakn mazdakn added docs-not-required Docs not required for this change release-note-not-required Change has no user-facing impact labels Mar 9, 2026
@marvin-tigera marvin-tigera added this to the Calico v3.32.0 milestone Mar 9, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Ensures calico/node startup populates a default BGPConfiguration resource (default) in the datastore, aligning startup-time global configuration initialization with expectations from other components/tests.

Changes:

  • Add a startup-time check/create for the global BGPConfiguration("default").
  • Refactor ensureDefaultConfig signature formatting and update copyright year.
  • Introduce a constant for the global BGP config name.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated 9 comments.

@mazdakn mazdakn requested a review from a team as a code owner March 11, 2026 17:44
@mazdakn mazdakn merged commit 0dcf161 into projectcalico:master Mar 11, 2026
1 of 2 checks passed
@mazdakn mazdakn deleted the default-bgpconfig branch March 11, 2026 21:06
lwr20 pushed a commit to lwr20/calico that referenced this pull request Mar 12, 2026
* first commit

* markup

* Delete node/:w

* update

* revert operator

* fix e2e runs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-not-required Docs not required for this change release-note-not-required Change has no user-facing impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants