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.

Sign up for GitHub

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

Jump to bottom

migrate bgp control plane v2 before upgrading cilium to 1.16 #2110

Merged
merged 1 commit into from
Aug 22, 2024
Merged

migrate bgp control plane v2 before upgrading cilium to 1.16 #2110

merged 1 commit into from
Aug 22, 2024

Conversation

outductor
Copy link
Contributor

公式ドキュメント によれば、CiliumBGPPeeringPolicy と CiliumBGPClusterConfig が同時に存在し、Cilium エージェントが両方の nodeSelector にマッチする場合、 CiliumBGPPeeringPolicy が優先されるそうです。

ほか、Upgrade Guide によると、

  1. CiliumLoadBalancerIPPool.spec.allowFirstLastIPs のデフォルト値が yes に変更されました。
    つまり、明示的に設定を変更しない限り、IP プールの最初と最後の IP アドレスも割り当て可能になります。
    以前の動作（最初と最後の IP アドレスは割り当て不可）に依存している場合は、アップグレード前に IP プールの設定で allowFirstLastIPs: no を明示的に設定する必要があります。

  2. CiliumLoadBalancerIPPool.spec.cidrs フィールドは v1.15 で非推奨となり、 CiliumLoadBalancerIPPool.spec.blocks が代わりに推奨されるようになりました。
    v1.15 の時点では両方のフィールドは同じ動作をしますが、cidrs フィールドは v1.16 で削除されます。
    アップグレード前に、IP プールの設定を cidrs から blocks に更新してください。

@outductor outductor requested a review from unchama August 16, 2024 14:01
unchama
unchama approved these changes Aug 22, 2024
View reviewed changes
Copy link
Member

@unchama unchama left a comment

Choose a reason for hiding this comment

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

とりあえず

@unchama unchama merged commit 21a7054 into main Aug 22, 2024
2 checks passed
@unchama unchama deleted the migrate_cilium_bgp_config branch August 22, 2024 11:12
unchama added a commit that referenced this pull request Aug 22, 2024
@unchama
Revert "Merge pull request #2110 from GiganticMinecraft/migrate_ciliu…
418276d 
…m_bgp_config"

This reverts commit 21a7054, reversing
changes made to 9865ce6.
@unchama unchama mentioned this pull request Aug 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@unchama unchama unchama approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@outductor @unchama