add ci config #10

MrBearing · 2024-08-31T04:37:54Z

No description provided.

MrBearing · 2024-08-31T04:38:11Z

作業中です。

MrBearing · 2024-08-31T05:18:23Z

Actionsの設定有効化しました。

MrBearing · 2024-09-02T05:38:16Z

@takasehideki
本PRですが、一度GithubActionを有効化するために、mainにマージしても良いでしょうか？
(まだ作業途中ですが現段階でGithub Actionで実行できるか検証したく。)

takasehideki · 2024-09-02T05:41:02Z

さほど差し支えはありませんが，ここを編集したら任意のブランチで実行できませんか？おそらく試行錯誤がありえるかと思いますので，別のブランチで十分に検証できてから main に適用がよいように思いました．
https://github.com/mROS-base/mros2-posix/pull/10/files#diff-3115ebcaaf2d8712a47490997ad972249736f61ea4c0092a930d23dbce656a5eR6

MrBearing · 2024-09-02T06:01:36Z

このPRのブランチをpush配下に追加したらいけました。
取り急ぎは、この対応でCIの設定進められそうです。

mainにマージしてしまうとまずいので、直前に当該箇所を削除します。

MrBearing · 2024-09-09T13:16:17Z

@takasehideki jinja2をaptで追加する必要があったので、READMEに追加した方が良いかもです。
とりあえず、ビルドまでは動作させることができました。

MrBearing · 2024-09-09T13:20:59Z

IPのアドレスの取得方法別途シェルスクリプト書いて、書き換える
テスト方法は検討の必要あり。
- 対向のNative ROSは ~~dockerで立てる~~CIスクリプト内でビルド
- mROS2 - mROS2 のテストはテスト対象をcmake_build下をビルド後にリネーム

MrBearing · 2024-09-11T01:20:41Z

@takasehideki
テスト対象のプログラムやテストスートのご指定ありますか？

特になければ、workspace下に適当に作ってテスト書こうと思っています。

takasehideki · 2024-09-12T03:13:59Z

@MrBearing 細かい指定はありません．

mros2/native ROS2 のそれぞれpub/subで計４セット（native/nativeはあまり意味はないがあっても良い）
primitive type (String) と custom type (Twistでよいか？メッセージ生成込み）

の計８セットがあると良いのかなぁ．

MrBearing · 2024-09-12T03:32:38Z

@takasehideki
承知しました。

以下の理解で良いでしょうか。

テスト時の通信形態の組み合わせは以下の4セット

pub mros2 / sub mros2
pub mros2 / sub native ROS 2
pub native ROS 2 / sub mros2
(pub native ROS 2 / sub native ROS 2) あれば尚良し

テスト対象の型は以下の2セット

primitive type std_msgs::msg::String
custom type Twist

4セット(3セット) × 2 セットで合計8セット(6セット)

takasehideki · 2024-09-12T03:33:38Z

いえすです 💯

MrBearing · 2024-09-12T03:40:56Z

@takasehideki

2点、質問というか相談です。

1．テスト通過条件について

mros2 - native ROS 2間は通信を手元でテストしているのですが、最初の1～2メッセージくらいがパケロスするような挙動をします。ロストする数がちょっと不定でテストの冪等性に不安があります。

テストのが通過条件は、一旦、複数回pubして1回でも良いから到達すればOKとしても良いでしょうか。

2．mros2 - mros 2 間の通信

mros2 - mros 2間が上手く通信できてないです。。
後ほどログを共有しますので、お知恵をお借りできればと

MrBearing · 2024-09-14T08:39:05Z

@takasehideki

テストプログラム書いてるのですが、ちょっとトラブっています。
プッシュ済みのworkspace/test_echoreply_stringを実行すると、
subscriberのコールバックが呼ばれない現象が発生しています。

ros2 topic echoとros2 topic pub で/to_linuxのNativeROSとのpub/subは動作確認できていますが、

mROS 2間での通信は同一IPアドレスだと通信できない等の制限が有るのでしょうか。
同様の現象はworkspace/echoreply_stringなどでも確認できています。

以下動作ログです。

$ bash build.bash all test_echoreply_string
$ ./cmake_build/mros2-posix 
ipaddr=0xc4d317ac
mask=0x00feffff
LOG_NOTICE : 00000000.000 : osKernelStart
mros2-posix start!
app name: echoback_string
LOG_DEBUG : 00000000.000 : mROS 2 initialization is completed

LOG_DEBUG : 00000000.000 : [MROS2LIB] create_node
LOG_DEBUG : 00000000.000 : [MROS2LIB] start creating participant
LOG_DEBUG : 00000000.000 : [MROS2LIB] mros2_init task start
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_init() 28: enter
LOG_DEBUG : 00000000.000 : [MROS2LIB] Initilizing lwIP complete
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_new() 306: enter
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp_multicast_manager.c udp_mc_info_alloc() 24: enter
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp_multicast_manager.c udp_mc_info_alloc() 42: mcp=0x7fd74c000db0 exit
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_new() 326: exit
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_bind() 141: ipaddr=0x0 port=7401 enter
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_bind() 154: mcp exist 0x7fd74c000db0
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_bind() 224: exit
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_recv() 239: enter
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_recv() 250: exit
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_new() 306: enter
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp_multicast_manager.c udp_mc_info_alloc() 24: enter
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp_multicast_manager.c udp_mc_info_alloc() 42: mcp=0x7fd74c000dd0 exit
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_new() 326: exit
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_bind() 141: ipaddr=0x0 port=7400 enter
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_bind() 154: mcp exist 0x7fd74c000dd0
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_bind() 224: exit
LOG_DEBUG : 00000000.000 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_recv() 239: enter
LOG_NOTICE : 00000000.000 : thread_udp_recv:UP: mcp=0x7fd74c000db0
LOG_DEBUG : 00000000.001 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_recv() 250: exit
LOG_DEBUG : 00000000.001 : /home/takumi/mros2-posix/lwip-posix/src/core/udp_multicast_manager.c udp_mc_joingroup() 78: enter ifaddr=0xc4d317ac groupaddr=0x100ffef
LOG_NOTICE : 00000000.001 : udp_mc_joingroup(): mcp=0x7fd74c000db0 ifaddr=0xc4d317ac groupaddr=0x100ffef
LOG_NOTICE : 00000000.001 : udp_mc_joingroup(): mcp=0x7fd74c000dd0 ifaddr=0xc4d317ac groupaddr=0x100ffef
LOG_DEBUG : 00000000.001 : /home/takumi/mros2-posix/lwip-posix/src/core/udp_multicast_manager.c udp_mc_joingroup() 93: exit
LOG_NOTICE : 00000000.001 : thread_udp_recv:UP: mcp=0x7fd74c000dd0
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_new() 306: enter
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp_multicast_manager.c udp_mc_info_alloc() 24: enter
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp_multicast_manager.c udp_mc_info_alloc() 42: mcp=0x7fd74c000df0 exit
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_new() 326: exit
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_bind() 141: ipaddr=0x0 port=7411 enter
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_bind() 154: mcp exist 0x7fd74c000df0
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_bind() 224: exit
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_recv() 239: enter
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_recv() 250: exit
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_new() 306: enter
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp_multicast_manager.c udp_mc_info_alloc() 24: enter
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp_multicast_manager.c udp_mc_info_alloc() 42: mcp=0x7fd74c000e10 exit
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_new() 326: exit
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_bind() 141: ipaddr=0x0 port=7410 enter
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_bind() 154: mcp exist 0x7fd74c000e10
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_bind() 224: exit
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_recv() 239: enter
LOG_DEBUG : 00000000.100 : /home/takumi/mros2-posix/lwip-posix/src/core/udp.c udp_recv() 250: exit
LOG_NOTICE : 00000000.100 : thread_udp_recv:UP: mcp=0x7fd74c000df0
LOG_NOTICE : 00000000.100 : thread_udp_recv:UP: mcp=0x7fd74c000e10
LOG_DEBUG : 00000000.101 : [MROS2LIB] successfully created participant
LOG_DEBUG : 00000000.101 : [MROS2LIB] create_publisher complete.
LOG_DEBUG : 00000000.102 : [MROS2LIB] Initilizing Domain complete
LOG_DEBUG : 00000001.101 : [MROS2LIB] create_subscription complete.
LOG_NOTICE : 00000001.101 : ready to pub/sub message

publishing msg: 'TestMessage0'
publishing msg: 'TestMessage1'
publishing msg: 'TestMessage2'
(中略)
publishing msg: 'TestMessage14'
publishing msg: 'TestMessage15'

MrBearing · 2024-09-15T10:32:24Z

@takasehideki

取り急ぎ、Native ROSとmROSのテストを追加しましたが、テストが通りません。
手元では動作確認しているのですが、起動タイミング等によっては失敗するので、原因が把握できておりません。

以下のログ参照いただき、ご意見いただきたいです。
https://github.com/mROS-base/mros2-posix/actions/runs/10869981965/job/30161980310?pr=10#step:11:70

s-hosoai · 2024-09-17T09:28:25Z

LOGでudp_bind() bind()でErrorが出てるように見えます。
err=98は”Address already in use”とのことですが、同じコンテナ内で取り合ってたり、解放し忘れてたりとかないですかね？

MrBearing · 2024-09-24T03:54:08Z

@s-hosoai ( @takasehideki )
ありがとうございます。

どうやら、NativeROS起動ー＞mros起動の順序で実行すると、件のupd_bind() bind() err=98 のエラーが発生する様です。
mros起動 -> NativeROS起動の順序で実行するとPub/Subできています。

https://github.com/mROS-base/mros2-posix/actions/runs/11006608360/job/30561156281

本来なら、こういうのもテストで検出できるようにすべきだとは思うのですが、
(READMEは mROS起動->NativeROS起動の順序になってますね)

一旦mros起動 -> NativeROS起動の順序で動作するようにCI構成していきます。

MrBearing · 2024-09-27T17:00:25Z

@takasehideki
このファイルで、docker-composeでテスト環境をセットアップするアイディアを一応は試してみたのですが、上手くいきませんでした。
tcpレベルでの疎通も一応は確認できている様ですが、デバッグに時間が掛かりそうです。

一旦テストとしては以前上げた8セットを網羅できていますので、先日お話した通り、テストの通らない箇所についてはissueとして上げてさせて頂き、このPRはクローズさせていただきます。

takasehideki · 2024-09-28T03:22:42Z

まずはたいへんおつかれさまでした．
クローズ is ready to review と思っています．ちょっとちゃんと見ます．
関係するIssueはタイトルだけ英語でお願いしたいです．本文は日本語で構いません．
コード中のコメントも英語推奨ですが，これは対応なしでも差し支えありません．

README.md

takasehideki

いったんコードレベルで気になるところにコメントを入れました

.github/workflows/humble_docker_test.yaml

.github/workflows/humble_test.yaml

.github/workflows/humble_docker_test.yaml

takasehideki · 2024-09-29T01:59:38Z

@MrBearing ディレクトリ構造に違和感があります．今更感がありますし，これはすぐに難しければ対応ナシでも構いません．

workspace/test_* を workspace/test/mros2/* に移動させたい
- ただし build.bash がネックになると認識していて，たぶんこれがいちばんネックになると思っています．
- これは変えたくないし hard copy で二重管理にしたくない．
docker-compose.yml と update_ip.sh は workspace/test/mros2/* に置きたい
- トップにあると，通常利用でもなんらか意味のあるものに見えてしまう．
なんらか README.md を書いて欲しい
- 1点目がすぐに解消可能であれば workspace/test/README.md に ./mros2/ の意図を追記する
- 解消が難しければ，トップの README.md で構わないので，どのファイルがテスト用であるか，どのようなテストが走っているかをセクション設けて書いて欲しい．メモレベルで構いません．

MrBearing · 2024-09-29T05:22:58Z

@takasehideki

docker-compose.yml と update_ip.sh は workspace/test/mros2/* に置きたい
トップにあると，通常利用でもなんらか意味のあるものに見えてしまう．

docker-compose.yml の方は同意します。
updata_ipに関してはユーザーがユーティリティ的に使えるので、直下に置いても良いのかなと思うのですがいかがでしょうか。

MrBearing · 2024-09-29T05:25:30Z

@takasehideki

workspace/test_* を workspace/test/mros2/* に移動させたい
ただし build.bash がネックになると認識していて，たぶんこれがいちばんネックになると思っています．
これは変えたくないし hard copy で二重管理にしたくない．

手元で試しにディレクトリ移動させて、以下の様にビルドができたので、後ほど反映します。

bash build.bash all test/mros2/test_echoback_string

(追記　このやり方でCIも通りました)

takasehideki · 2024-09-29T05:27:13Z

updata_ipに関してはユーザーがユーティリティ的に使えるので、直下に置いても良いのかなと思うのですがいかがでしょうか。

なるほどそれはそうかもしれませんね．将来的には build.bash に組み込んでもいいかも（util/ を作るほどでもないか）．
代わりに README.md に意図や使い方を記載してもらえるでしょうか．中身のソースを見ないとなにをするものなのか分からないは避けたいです．

README.md

workspace/test/README.md

takasehideki

好みレベルですが suggestion しました

workspace/test/README.md

README.md

MrBearing · 2024-09-30T05:43:32Z

@takasehideki
一応、現在頂いていたコメントに関してはすべて反映し終えたと認識しています。
追加で修正点なければPRのマージお願いいたします。

MrBearing · 2024-09-30T05:46:31Z

あとREADMEに追加するか迷ったので、ここにstatus_badgeを貼っておきます。

takasehideki · 2024-10-01T09:18:19Z

@MrBearing 改めましてありがとうございました！
とりあえず approved ではあります．でも最初から Failing なのは格好が付かないので，ちょっとこちらでもやってみて探ってみます．ということで merge はまだお待ちいただきたく，ご理解ください．

MrBearing self-assigned this Aug 31, 2024

MrBearing added the TESTING label Aug 31, 2024

MrBearing force-pushed the feat/add_ci_config branch from 601a895 to 1418eac Compare September 9, 2024 13:17

MrBearing added TESTING and removed TESTING labels Sep 10, 2024

MrBearing added TESTING and removed TESTING labels Sep 14, 2024

MrBearing force-pushed the feat/add_ci_config branch from f3c7f94 to 9d21b95 Compare September 15, 2024 09:44

MrBearing mentioned this pull request Sep 24, 2024

Add echoreply_twist mROS-base/mros2-host-examples#15

Merged

MrBearing force-pushed the feat/add_ci_config branch 5 times, most recently from c34a8b1 to a904a79 Compare September 24, 2024 08:02

MrBearing force-pushed the feat/add_ci_config branch 2 times, most recently from 2be52f6 to 09b7038 Compare September 27, 2024 17:06

add CI and Test

8eca2c7

MrBearing force-pushed the feat/add_ci_config branch from 09b7038 to 8eca2c7 Compare September 27, 2024 17:09

MrBearing commented Sep 28, 2024

View reviewed changes

README.md Outdated Show resolved Hide resolved

takasehideki reviewed Sep 29, 2024

View reviewed changes

MrBearing force-pushed the feat/add_ci_config branch from 94352a9 to bc50df8 Compare September 29, 2024 05:08

MrBearing commented Sep 29, 2024

View reviewed changes

README.md Outdated Show resolved Hide resolved

MrBearing commented Sep 29, 2024

View reviewed changes

workspace/test/README.md Outdated Show resolved Hide resolved

MrBearing force-pushed the feat/add_ci_config branch 4 times, most recently from a1fc7a5 to 8eca2c7 Compare September 29, 2024 14:11

takasehideki reviewed Sep 30, 2024

View reviewed changes

workspace/test/README.md Outdated Show resolved Hide resolved

README.md Outdated Show resolved Hide resolved

MrBearing force-pushed the feat/add_ci_config branch from 4683044 to e522fbe Compare September 30, 2024 01:35

MrBearing removed the TESTING label Sep 30, 2024

takasehideki reviewed Sep 30, 2024

View reviewed changes

README.md Outdated Show resolved Hide resolved

MrBearing force-pushed the feat/add_ci_config branch 2 times, most recently from 54f04e2 to e2eec14 Compare September 30, 2024 05:36

Addressed the comment.

97feb0b

MrBearing force-pushed the feat/add_ci_config branch from e2eec14 to 97feb0b Compare September 30, 2024 05:39

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

add ci config #10

add ci config #10

MrBearing commented Aug 31, 2024

MrBearing commented Aug 31, 2024

MrBearing commented Aug 31, 2024 •

edited

Loading

MrBearing commented Sep 2, 2024 •

edited

Loading

takasehideki commented Sep 2, 2024

MrBearing commented Sep 2, 2024 •

edited

Loading

MrBearing commented Sep 9, 2024 •

edited

Loading

MrBearing commented Sep 9, 2024 •

edited

Loading

MrBearing commented Sep 11, 2024 •

edited

Loading

takasehideki commented Sep 12, 2024

MrBearing commented Sep 12, 2024 •

edited

Loading

takasehideki commented Sep 12, 2024

MrBearing commented Sep 12, 2024 •

edited

Loading

MrBearing commented Sep 14, 2024 •

edited

Loading

MrBearing commented Sep 15, 2024

s-hosoai commented Sep 17, 2024

MrBearing commented Sep 24, 2024 •

edited

Loading

MrBearing commented Sep 27, 2024

takasehideki commented Sep 28, 2024

takasehideki left a comment

takasehideki commented Sep 29, 2024

MrBearing commented Sep 29, 2024 •

edited

Loading

MrBearing commented Sep 29, 2024 •

edited

Loading

takasehideki commented Sep 29, 2024

takasehideki left a comment

MrBearing commented Sep 30, 2024 •

edited

Loading

MrBearing commented Sep 30, 2024 •

edited

Loading

takasehideki commented Oct 1, 2024

add ci config #10

Are you sure you want to change the base?

add ci config #10

Conversation

MrBearing commented Aug 31, 2024

MrBearing commented Aug 31, 2024

MrBearing commented Aug 31, 2024 • edited Loading

MrBearing commented Sep 2, 2024 • edited Loading

takasehideki commented Sep 2, 2024

MrBearing commented Sep 2, 2024 • edited Loading

MrBearing commented Sep 9, 2024 • edited Loading

MrBearing commented Sep 9, 2024 • edited Loading

MrBearing commented Sep 11, 2024 • edited Loading

takasehideki commented Sep 12, 2024

MrBearing commented Sep 12, 2024 • edited Loading

takasehideki commented Sep 12, 2024

MrBearing commented Sep 12, 2024 • edited Loading

1． テスト通過条件について

2．mros2 - mros 2 間の通信

MrBearing commented Sep 14, 2024 • edited Loading

MrBearing commented Sep 15, 2024

s-hosoai commented Sep 17, 2024

MrBearing commented Sep 24, 2024 • edited Loading

MrBearing commented Sep 27, 2024

takasehideki commented Sep 28, 2024

takasehideki left a comment

Choose a reason for hiding this comment

takasehideki commented Sep 29, 2024

MrBearing commented Sep 29, 2024 • edited Loading

MrBearing commented Sep 29, 2024 • edited Loading

takasehideki commented Sep 29, 2024

takasehideki left a comment

Choose a reason for hiding this comment

MrBearing commented Sep 30, 2024 • edited Loading

MrBearing commented Sep 30, 2024 • edited Loading

takasehideki commented Oct 1, 2024

MrBearing commented Aug 31, 2024 •

edited

Loading

MrBearing commented Sep 2, 2024 •

edited

Loading

MrBearing commented Sep 2, 2024 •

edited

Loading

MrBearing commented Sep 9, 2024 •

edited

Loading

MrBearing commented Sep 9, 2024 •

edited

Loading

MrBearing commented Sep 11, 2024 •

edited

Loading

MrBearing commented Sep 12, 2024 •

edited

Loading

MrBearing commented Sep 12, 2024 •

edited

Loading

1．テスト通過条件について

MrBearing commented Sep 14, 2024 •

edited

Loading

MrBearing commented Sep 24, 2024 •

edited

Loading

MrBearing commented Sep 29, 2024 •

edited

Loading

MrBearing commented Sep 29, 2024 •

edited

Loading

MrBearing commented Sep 30, 2024 •

edited

Loading

MrBearing commented Sep 30, 2024 •

edited

Loading