sfc-aqua / qrsa

Quantum Repeater Software Architecture
1 stars 1 forks source link

Connection Setup Implementation #9

Closed Chibikuri closed 1 year ago

Chibikuri commented 1 year ago

This change is Reviewable

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 87.46% and project coverage change: +68.22% :tada:

Comparison is base (c6b05f1) 9.01% compared to head (9cebb47) 77.24%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #9 +/- ## ========================================== + Coverage 9.01% 77.24% +68.22% ========================================== Files 18 63 +45 Lines 122 936 +814 ========================================== + Hits 11 723 +712 - Misses 111 213 +102 ``` | [Files Changed](https://app.codecov.io/gh/sfc-aqua/qrsa/pull/9?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sfc-aqua) | Coverage Δ | | |---|---|---| | [...e/qnode/src/qnode/routing\_daemon/routing\_daemon.py](https://app.codecov.io/gh/sfc-aqua/qrsa/pull/9?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sfc-aqua#diff-cXJzYV9wcm90b3R5cGUvcW5vZGUvc3JjL3Fub2RlL3JvdXRpbmdfZGFlbW9uL3JvdXRpbmdfZGFlbW9uLnB5) | `40.74% <40.74%> (ø)` | | | [qrsa\_prototype/qnode/src/qnode/bootstrap.py](https://app.codecov.io/gh/sfc-aqua/qrsa/pull/9?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sfc-aqua#diff-cXJzYV9wcm90b3R5cGUvcW5vZGUvc3JjL3Fub2RlL2Jvb3RzdHJhcC5weQ==) | `46.66% <46.66%> (ø)` | | | [...qnode/real\_time\_controller/real\_time\_controller.py](https://app.codecov.io/gh/sfc-aqua/qrsa/pull/9?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sfc-aqua#diff-cXJzYV9wcm90b3R5cGUvcW5vZGUvc3JjL3Fub2RlL3JlYWxfdGltZV9jb250cm9sbGVyL3JlYWxfdGltZV9jb250cm9sbGVyLnB5) | `50.00% <50.00%> (ø)` | | | [qrsa\_prototype/tests/test\_rule\_engine.py](https://app.codecov.io/gh/sfc-aqua/qrsa/pull/9?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sfc-aqua#diff-cXJzYV9wcm90b3R5cGUvdGVzdHMvdGVzdF9ydWxlX2VuZ2luZS5weQ==) | `50.00% <50.00%> (ø)` | | | [...ototype/qnode/src/qnode/rule\_engine/rule\_engine.py](https://app.codecov.io/gh/sfc-aqua/qrsa/pull/9?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sfc-aqua#diff-cXJzYV9wcm90b3R5cGUvcW5vZGUvc3JjL3Fub2RlL3J1bGVfZW5naW5lL3J1bGVfZW5naW5lLnB5) | `57.14% <57.14%> (ø)` | | | [...de/src/qnode/connection\_manager/ruleset\_factory.py](https://app.codecov.io/gh/sfc-aqua/qrsa/pull/9?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sfc-aqua#diff-cXJzYV9wcm90b3R5cGUvcW5vZGUvc3JjL3Fub2RlL2Nvbm5lY3Rpb25fbWFuYWdlci9ydWxlc2V0X2ZhY3RvcnkucHk=) | `58.33% <58.33%> (ø)` | | | [...pe/qnode/src/qnode/connection\_manager/interface.py](https://app.codecov.io/gh/sfc-aqua/qrsa/pull/9?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sfc-aqua#diff-cXJzYV9wcm90b3R5cGUvcW5vZGUvc3JjL3Fub2RlL2Nvbm5lY3Rpb25fbWFuYWdlci9pbnRlcmZhY2UucHk=) | `62.50% <62.50%> (ø)` | | | [.../qnode/src/qnode/real\_time\_controller/interface.py](https://app.codecov.io/gh/sfc-aqua/qrsa/pull/9?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sfc-aqua#diff-cXJzYV9wcm90b3R5cGUvcW5vZGUvc3JjL3Fub2RlL3JlYWxfdGltZV9jb250cm9sbGVyL2ludGVyZmFjZS5weQ==) | `72.72% <72.72%> (ø)` | | | [...ode/src/qnode/hardware\_monitor/hardware\_monitor.py](https://app.codecov.io/gh/sfc-aqua/qrsa/pull/9?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sfc-aqua#diff-cXJzYV9wcm90b3R5cGUvcW5vZGUvc3JjL3Fub2RlL2hhcmR3YXJlX21vbml0b3IvaGFyZHdhcmVfbW9uaXRvci5weQ==) | `77.77% <77.77%> (ø)` | | | [...ype/qnode/src/qnode/rule\_engine/ruleset\_runtime.py](https://app.codecov.io/gh/sfc-aqua/qrsa/pull/9?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sfc-aqua#diff-cXJzYV9wcm90b3R5cGUvcW5vZGUvc3JjL3Fub2RlL3J1bGVfZW5naW5lL3J1bGVzZXRfcnVudGltZS5weQ==) | `81.81% <81.81%> (ø)` | | | ... and [35 more](https://app.codecov.io/gh/sfc-aqua/qrsa/pull/9?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sfc-aqua) | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Chibikuri commented 1 year ago

ryeの.venvがコンテナボリュームにマウントされており、複数のコンテナから触ってしまい起動時にエラーを起こすので、一旦ローカルで生成されたrequirements.lockをベースにpipインストールするように変更した。

コンテナ内でryeが必要な場合は.venvをマウントしない等解決するが、dockerを検証用と想定すればとりあえずこれでも良さそう。

zigen commented 1 year ago

rye installをbuild時にやってしまうのではだめなんだっけ

Chibikuri commented 1 year ago

rye installにpip install -rのようなオプションがなさそうだったので、一旦コンテナ内で新しいvenv立ててpipしている感じになってます。

zigen commented 1 year ago

でもそもそも.venvの中身ってコンテナ共通じゃだめなんだっけ。

Chibikuri commented 1 year ago

内容自体はコンテナ共通でも良いんですけど、共通にすると何故かrye syncしたときに成功するコンテナと、失敗するコンテナが出てきて、見た感じ片方のrye syncが先に実行されて.venvの内容を書き換えて、他のコンテナが参照した時モジュールがないみたいなことが起こってそうなんですよね... ただ僕の設定が良くないかもしれないので、もう一回見てみます!

zigen commented 1 year ago

rye syncを複数のコンテナでやると失敗するので、一箇所だけでやるようにしたい。なので、imageビルド時とあとは開発時に一つのコンテナだけでrye syncすればいい気がする。それで他のコンテナからも見えるとおもうんだよね。

Chibikuri commented 1 year ago

確かに、今調べたらdepends_onというcomposeのオプションがあるのに気がつきました、、 これで一回やってみます!

Chibikuri commented 1 year ago

build時にrye syncに必要なファイルを前もってコンテナ内にもっていくように変更して、build時にcontrollerで代表して実行されるように直しました。

zigen commented 1 year ago

まだimage消してビルドしなおすとこけるので、別PRでその辺直すわ