Feat/#61 remote exec

[comavius]

関連Issue

• close #61

概要

telnetからsshに変更し実装

変更内容

• tenlet moduleを削除
• remote_exec moduleを作成
• RemoteExec traitを実装
• RemoteExecの実装としてSshConnectionを実装

チェックリスト

• [x] テストが通っている
• [x] 下記のいずれかを満たしている
  • • [x] 変更点についてテストを追加/修正した
  • • [ ] テストを追加するissueを立てた
  • • [ ] テストが不要な変更である
• [x] レビュワーを指定した
• [x] タグをつけた

補足

[comavius]

テストをするときにsystemctlでdockerデーモンを起動しているせいでcargo testするたびにroot権限を求められて面倒くさい

[comavius]

actionsの環境では最初からdockerデーモンが立ち上がっていそう

[comavius]

ソケットがrunningではない場合systemctl status dockerが異常終了していそうなので解決できそう

[comavius]

正常系の失敗ケースは追加したいかも

[comavius]

あと、ジャッジサーバーの責任で起こるエラーと作問者の責任で起こるエラーがありそれらを適切にハンドリングする必要があるため、anyhow::Resultを使うべきではないかもしれない

[comavius]

timeoutが動いていない

[comavius]

あと、ジャッジサーバーの責任で起こるエラーと作問者の責任で起こるエラーがありそれらを適切にハンドリングする必要があるため、anyhow::Resultを使うべきではないかもしれない

解決した

[kenken714]

手元でcargo clippyをしてからPRをしてください:pray:

[comavius]

cargo clippy --fixした

[comavius]

@kenken714 clippyとcargo fmtで結果が変わる部分がありそう

[kenken714]

• [x] trait実装を確認
• [x] testが通ることを確認

[kenken714]

テストをするときにsystemctlでdockerデーモンを起動しているせいでcargo testするたびにroot権限を求められて面倒くさい

:koreni_natteru: (手元だけ)

[comavius]

(dockerデーモンが起動済み or root権限あり)が満たされないなら自動で落とすようにしたからよくはなっているはず