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

feat: Replace metallb with cilium l announcement #379

Merged

yuta1979 merged 1 commit into cndw2024 from feat/cilium_l2_announcement

Oct 7, 2024

Contributor

ystkfujii commented Oct 6, 2024 •

edited by yuta1979

Loading

Summary

ciliumのl2 announceネタを追加しました
- ref: [cilium] コンテンツネタ: L2 Annoucement #368
MetalLBは不要になるんですが、結局dockerのネットワークを設定しないといけないことに変わりないので、微妙な気もしますが...
- cnd-handson/chapter_cilium/manifest/l2announcement.yaml
  
  Lines 23 to 24 in a0201da
  
  - start: 172.18.0.200
  
  stop: 172.18.0.250
- MetalLBが消えるのでコンポーネントとしては減ってるので、分かりやすくなっているはず... !

Review

よさそう


          feat: Replace metallb with cilium l announcement

a0201da

ystkfujii requested a review from yuta1979

October 6, 2024 02:49

ystkfujii self-assigned this

Contributor

yuta1979 commented Oct 7, 2024

ありがとうございます！
内容確認しました。LGTMですのでマージいたします！

yuta1979 merged commit 43dc19f into cndw2024

yuta1979 deleted the feat/cilium_l2_announcement branch

October 7, 2024 00:46

Himalayan-wildcat linked an issue

that may be closed by this pull request

[cilium] コンテンツネタ: L2 Annoucement #368

Closed

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

Labels

None yet