February 2024 - Zebra-dev - zebra.ginzado.ne.jp

Slack Digest: 1709049721-1709136121
by zebra-dev＠zebra.ginzado.ne.jp 29 Feb '24

29 Feb '24

チャンネル ========== --- __m-asama__ *2024-02-28 14:09:15 UTC* BGP で route-map を更新した際に自動的に UPDATE が走るのをデフォルトで無効にする修正スレスレッド ======== --- __m-asama__ *2024-02-28 14:09:15 UTC* BGP で route-map を更新した際に自動的に UPDATE が走るのをデフォルトで無効にする修正スレ ... __m-asama__ *2024-02-28 14:09:23 UTC* <https://github.com/yasuhironet/zebra-re/pull/1/commits/75694194f20c86cde0bd…> __m-asama__ *2024-02-28 14:14:36 UTC* <@U06CYG1HGKZ> <@U06DRMESJ9X> いろいろ実験してみたんですが、どうも ``` bgp route-map delay-timer (0-600) ``` に 0 を設定すると自動で UPDATE が走らないようになっているようです。 `bgpd/bgpd.h` のコメントには `/* disabled by default */` と書かれているのに、なぜか実際にはデフォルト 5 秒で自動で UPDATE が走るようになっていました。これを 0 にしてしまうのが最も簡単なようです。コメントが「デフォルトで無効」となっているのにそうなっていないのは当初はデフォルトで無効にするつもりだったのに後からデフォルト 5 秒で自動で UPDATE が走るように変更したのかもしれません。。 __yasu1976__ *2024-02-28 14:46:26 UTC* なるほど。仕事早いです！そこ0にして、コマンドをインストールしない、とかも手かも知れませんね。 __yasu1976__ *2024-02-28 14:47:24 UTC* なんか勝手にcommitlintてのが走ってるみたいですが、時間見つけてマージしておきます〜

1 0

Slack Digest: 1708876921-1708963321
by zebra-dev＠zebra.ginzado.ne.jp 27 Feb '24

27 Feb '24

チャンネル ========== --- __m-asama__ *2024-02-26 11:36:27 UTC* ML の購読の仕方ですが、こちら↓ <https://zebra.ginzado.ne.jp/postorius/lists/zebra-dev.zebra.ginzado.ne.jp/> から Subscribe したんでした。。 __yasu1976__ *2024-02-26 13:55:08 UTC* 登録できました！スレッド ========

1 0

Slack Digest: 1708782016-1708870293
by zebra-dev＠zebra.ginzado.ne.jp 26 Feb '24

26 Feb '24

チャンネル ========== --- __yasu1976__ *2024-02-25 13:40:04 UTC* format 変更して、bgpd まで一応ざっと見ましたので、これで push しちゃいます。 __yasu1976__ *2024-02-25 13:40:41 UTC* まだコンパイル通らない可能性あります。その時は対応しますので教えてください。 __yasu1976__ *2024-02-25 13:41:19 UTC* rfapi ってなんですかね。bgpd と同じくらい大きそうな感じでしたが。。。 __yasu1976__ *2024-02-25 13:55:35 UTC* <https://github.com/yasuhironet/zebra-re> __yasu1976__ *2024-02-25 13:56:56 UTC* zebra-re の方です。 __yasu1976__ *2024-02-25 13:57:35 UTC* コンパイル通すのは、帰国してから私も参加しますが、好きにやっててもらっても構いません > <@U06D6CW4QMU> __m-asama__ *2024-02-25 13:58:13 UTC* お、環境構築できまして、とりあえず 528527e の時点で Ubuntu 22.04 でパッケージできるところまで確認しました！ __yasu1976__ *2024-02-25 13:59:12 UTC* おお、素晴らしい。早い！w __m-asama__ *2024-02-25 13:59:56 UTC* いま git pull して 61a1ef6 でビルドしてみています。。 __yasu1976__ *2024-02-25 14:00:08 UTC* frr-9.0.2 のタグ打ってあるので、diff すればおかしくなってるところも戻せると思います __yasu1976__ *2024-02-25 14:01:15 UTC* コンパイルまで通したら、 develop を切りましょうかね。今 main に直接です。 __yasu1976__ *2024-02-25 14:02:01 UTC* bgpd の要らない部分を全部切って、根本的に変更してマルチスレッドにしたい。。。 __yasu1976__ *2024-02-25 14:05:50 UTC* 前回（commit log引き継いでない時）に GNU style にしてコンパイル通すところまでやったので、 __yasu1976__ *2024-02-25 14:05:59 UTC* コンパイル通らなかったら気軽に聞いてください __m-asama__ *2024-02-25 14:11:32 UTC* ビルド通りました！ __yasu1976__ *2024-02-24 13:40:17 UTC* このrevertですが、ちょっと難しそうなので、最新から機能を元に戻すような変更にした方が良さそうです。 ↓たくさんConflictが出ました。 ``` yasu@yasuhironets-MBP zebra % git status On branch main Your branch is up to date with 'origin/main'. You are currently reverting commit 518f0eb18. (fix conflicts and run "git revert --continue") (use "git revert --skip" to skip this patch) (use "git revert --abort" to cancel the revert operation) Unmerged paths: (use "git restore --staged <file>..." to unstage) (use "git add/rm <file>..." as appropriate to mark resolution) both modified: bgpd/bgp_clist.c both modified: bgpd/bgp_filter.c both modified: bgpd/bgp_filter.h both modified: bgpd/bgp_main.c both modified: bgpd/bgp_routemap.c both modified: bgpd/bgp_vty.c both modified: bgpd/bgp_zebra.c both modified: bgpd/bgp_zebra.h both modified: bgpd/bgpd.c both modified: bgpd/bgpd.h both modified: lib/filter.c deleted by us: lib/memtypes.c both modified: lib/plist.c both modified: lib/routemap.c both modified: lib/routemap.h deleted by us: zebra/Makefile.am both modified: zebra/zebra_rib.c both modified: zebra/zebra_routemap.c both modified: zebra/zebra_vty.c both modified: zebra/zserv.h no changes added to commit (use "git add" and/or "git commit -a") ``` スレッド ======== --- __kunihiroishiguro__ *2024-02-08 05:15:18 UTC* <@U06D6CW4QMU> 強制Route Refreshするようにしたんですね。これはいかん気が。。。。<@U06CYG1HGKZ> この修正はrevertしたほうがいいと思いますね。 ... __yasu1976__ *2024-02-24 13:40:17 UTC* このrevertですが、ちょっと難しそうなので、最新から機能を元に戻すような変更にした方が良さそうです。 ↓たくさんConflictが出ました。 ``` yasu@yasuhironets-MBP zebra % git status On branch main Your branch is up to date with 'origin/main'. You are currently reverting commit 518f0eb18. (fix conflicts and run "git revert --continue") (use "git revert --skip" to skip this patch) (use "git revert --abort" to cancel the revert operation) Unmerged paths: (use "git restore --staged <file>..." to unstage) (use "git add/rm <file>..." as appropriate to mark resolution) both modified: bgpd/bgp_clist.c both modified: bgpd/bgp_filter.c both modified: bgpd/bgp_filter.h both modified: bgpd/bgp_main.c both modified: bgpd/bgp_routemap.c both modified: bgpd/bgp_vty.c both modified: bgpd/bgp_zebra.c both modified: bgpd/bgp_zebra.h both modified: bgpd/bgpd.c both modified: bgpd/bgpd.h both modified: lib/filter.c deleted by us: lib/memtypes.c both modified: lib/plist.c both modified: lib/routemap.c both modified: lib/routemap.h deleted by us: zebra/Makefile.am both modified: zebra/zebra_rib.c both modified: zebra/zebra_routemap.c both modified: zebra/zebra_vty.c both modified: zebra/zserv.h no changes added to commit (use "git add" and/or "git commit -a") ``` __m-asama__ *2024-02-25 01:05:17 UTC* これ、、私やってみてもいいでしょうか。。言い出しっぺですし。。 __yasu1976__ *2024-02-25 01:34:14 UTC* もちろんです。 __yasu1976__ *2024-02-25 01:39:25 UTC* しかし、route-mapだけでも、commit/rollbackを入れる、というのも手かな、とも思っています。戻すというのはちょっと後ろ向きで、私としても、route-map変更の時間によって結果が変わるのは何か正しくない気もしています。commit/rollbackを実装するのが理想であれば、そっちに向かいたい気もします。 <@U06DRMESJ9X> さんも意見ください。 __yasu1976__ *2024-02-25 01:43:03 UTC* （そこだけcommit/rollback入れる、というのがどれだけ大変かあまり考えずに言っちゃってますけど。） __m-asama__ *2024-02-25 14:02:50 UTC* Legacy IOS(？) のアーキテクチャで commit/rollback 入れるのはしんどそうな気が、、します。。(自分はいい方法を思いつかない。。) commit/rollback モデルは新しい石黒さんの方に任せてしまった方が良いような気が。。 __yasu1976__ *2024-02-25 14:03:27 UTC* そうですね、そうしますか。 __yasu1976__ *2024-02-25 14:04:09 UTC* 好きに変えてもらって構いませんので！:man-bowing: __m-asama__ *2024-02-25 14:07:58 UTC* とりあえず現時点でどういう挙動するのか観察してみようかな、、と思ってます。。

1 0

Slack Digest: 1708531321-1708617721
by zebra-dev＠zebra.ginzado.ne.jp 23 Feb '24

23 Feb '24

チャンネル ========== --- __yasu1976__ *2024-02-22 03:46:28 UTC* すみません、どうすれば <mailto:zebra-dev@zebra.ginzado.ne.jp|zebra-dev@zebra.ginzado.ne.jp> 入れるのでしたっけ。今更すみません。 <https://zebra.ginzado.ne.jp/hyperkitty/> から購読ボタンとか押してみたんですが、反応ないみたいで。 __yasu1976__ *2024-02-22 03:47:44 UTC* ユーザ登録しないとダメなのかな。 __yasu1976__ *2024-02-22 03:50:58 UTC* zebra のレポジトリ、作り直そうかな。。。（気軽にそういうことやってはいけないのは理解してるのですが。） FRRのコミット履歴は今引き継いでない（tar.gz からインポートしちゃった）のですが、コミット履歴は引き継いだ方がやっぱり良いですよね。。。 revert の時とかも楽そうだし。 __m-asama__ *2024-02-22 05:14:37 UTC* あ、、あれ？？自分はここから登録できたと思うんですけど、、 __m-asama__ *2024-02-22 05:14:52 UTC* すみません、、ちょっと調べますのでお時間ください。。 __m-asama__ *2024-02-22 05:15:10 UTC* リポジトリ作り直し、今ならやっても問題ないんじゃないですかね。。 __m-asama__ *2024-02-22 05:15:32 UTC* FRR のコミット履歴が見れた方が良い、というのは確かにその通りな気がします。。 __yasu1976__ *2024-02-22 05:18:17 UTC* MLは、すみません、急ぎませんので、今度ゆるゆるお願いします。 __tomo__ *2024-02-22 06:47:12 UTC* <@U06L2JZT7NE>さんがチャンネルに参加しましたスレッド ========

1 0

Slack Digest: 1707321721-1707408121
by zebra-dev＠zebra.ginzado.ne.jp 09 Feb '24

09 Feb '24

チャンネル ========== --- __m-asama__ *2024-02-08 04:28:25 UTC* チャンネル名を「zebra」から「zebra-dev」に変更しました __m-asama__ *2024-02-08 05:01:23 UTC* <https://github.com/FRRouting/frr/commit/518f0eb1886d7f84194e31bc54ab9224f6e…> 前から気になってたんですけどこれ↑どうなんですかね。。 zebra のような running config 直接いじる実装でこの挙動、、問題にならないんですかね。。 __m-asama__ *2024-02-08 05:02:10 UTC* ``` Currently, modifications to route maps do not affect already processed routes; they only affect new route updates. This patch addresses this limitation. ``` と書かれてるんですけど、、「いや、そういうふうに作ってるんだが。。」という話じゃないのかな。。 __m-asama__ *2024-02-08 05:05:40 UTC* がこのチャンネルの説明を「このチャンネルでは #zebra-dev についてどんなことでも話し合えます。ミーティングの開催、資料の共有、チーム一体での意思決定ができます。」に設定しました __yasu1976__ *2024-02-08 05:10:15 UTC* どういう風にするのが理想だと思います？それが決まれば、そういう風に治しちゃいましょうかw __kunihiroishiguro__ *2024-02-08 05:15:18 UTC* <@U06D6CW4QMU> 強制Route Refreshするようにしたんですね。これはいかん気が。。。。<@U06CYG1HGKZ> この修正はrevertしたほうがいいと思いますね。 __m-asama__ *2024-02-08 05:15:51 UTC* いや、なんでこういうふうに変えたのかが気になったんですよね。。`clear bgp ipv4 xxx soft` するじゃダメだった理由が何かあったのかな、、というのが気になってるんですが、、あるのか？？そんな理由？？ __kunihiroishiguro__ *2024-02-08 05:15:57 UTC* configがcommit/rollbackベースならまだしもrunning configの変更時にこれやられると事故ると思いますね。 __m-asama__ *2024-02-08 05:17:08 UTC* でもいま master みたらこのままなんですよ、、もしかして自分の感覚がおかしいのかな、、と不安になってましたw __yasu1976__ *2024-02-08 05:33:45 UTC* 2015からの変更もたくさんありそうですし、そのパッチだけrevertするの難しいかもですね。 __yasu1976__ *2024-02-08 05:34:51 UTC* route-map変更後、新経路についてはどうすべきなんでしょうね。古いroute-map持っておいて、そちらを使い続ける、（つまり、パーシャルなcommit/rollbackをroute-mapだけ実装する）とかしたほうが良さそうな気もします __yasu1976__ *2024-02-08 05:35:43 UTC* route-mapってpeerごとにも使い回しそうですよね。そうすると、コピって各自持っておく、のかな？ __kunihiroishiguro__ *2024-02-08 05:35:45 UTC* 手動soft resetがいい気がしますね。 __yasu1976__ *2024-02-08 05:37:13 UTC* route_map_dupみたいなの作って、それぞれに別インスタンスを持たせるようにしましょうか。そうなってましたっけ？ __yasu1976__ *2024-02-08 05:38:00 UTC* そんな余計なことする必要ないですかね？ __kunihiroishiguro__ *2024-02-08 05:43:37 UTC* そうですね、特に特別なことはしなくていい気がします。スレッド ========

1 0

Slack Digest: 1707280992-1707367392
by zebra-dev＠zebra.ginzado.ne.jp 08 Feb '24

08 Feb '24

チャンネル ========== --- __m-asama__ *2024-02-08 04:28:25 UTC* チャンネル名を「zebra」から「zebra-dev」に変更しました __m-asama__ *2024-02-08 04:31:30 UTC* テストスレッド ========

1 0