Skip to content

ci: encrypted-sql-ops-pg の使ってないテストを削除 & E2Eテスト通らなくなってるので修正(途中)#639

Merged
osuketh merged 5 commits intomainfrom
ci/rm-esop-e2e
Jun 7, 2021
Merged

ci: encrypted-sql-ops-pg の使ってないテストを削除 & E2Eテスト通らなくなってるので修正(途中)#639
osuketh merged 5 commits intomainfrom
ci/rm-esop-e2e

Conversation

@laysakura
Copy link
Copy Markdown
Contributor

@laysakura laysakura commented Jun 6, 2021

Issueへのリンク

(なし)

やったこと

前回PRで追加した E2Eテスト、結局 scripts/encrypted-sql-ops-pg-test.sh で実行するようになったので、削除しておくべき差分でした...

mainのCIが落ちて気づきました。
現行の encrypted-sql-ops-pg の docker-compose 設定に従い、起動して postgres クライアントの入力待ちになり、60min経ってCIがタイムアウトしました。


ブランチ名に e2e 入って久しぶりに e2e テストが走り、通らなくなってるので修正を試みているところですが、まだ完了はしていません。

example-*.Dockerfile において COPY fixuid.bash . をしていなかったのは明らかなミスなので、それだけは修正済みです。
こいつが main にマージされて新しいイメージが push できたら、現在CI落ちの原因になっている

Creating s_state_runtime_1 ... done
+ sleep 10
+ cd /home/anonify-dev/ci-agent/_work/1/s
++ curl http://0.0.0.0:18550/api/v1/enclave_encryption_key -s -f -k -X GET -H 'Content-Type: application/json' -d ''
+ pubkey=
##[error]Bash exited with code '7'.

の原因を改めて調査しようと思います。

やらないこと

e2eテストパス(追ってやります)

動作検証

(なし)

参考

(なし)

@laysakura laysakura requested a review from osuketh June 6, 2021 00:42
@laysakura
Copy link
Copy Markdown
Contributor Author

うっ、 e2e ってブランチ名に入ってるから e2e テスト走ってそれが落ちた...

このブランチで e2e テストもついでに直します 😭

@laysakura laysakura removed the request for review from osuketh June 6, 2021 01:52
@laysakura laysakura marked this pull request as draft June 6, 2021 01:52
@laysakura laysakura changed the title ci: E2E test for encrypted-sql-ops-pg is no longer used ci: encrypted-sql-ops-pg の使ってないテストを削除 & E2Eテスト通らなくなってるので修正 Jun 6, 2021
@laysakura laysakura changed the title ci: encrypted-sql-ops-pg の使ってないテストを削除 & E2Eテスト通らなくなってるので修正 ci: encrypted-sql-ops-pg の使ってないテストを削除 & E2Eテスト通らなくなってるので修正(途中) Jun 6, 2021
@laysakura laysakura requested a review from osuketh June 6, 2021 23:30
@laysakura
Copy link
Copy Markdown
Contributor Author

@osuketh descriptionに書いた事情で、E2Eはfailのままこいつはマージしたいと考えています。
レビューお願いします 🙏

@laysakura laysakura marked this pull request as ready for review June 6, 2021 23:37
Copy link
Copy Markdown
Member

@osuketh osuketh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@osuketh
Copy link
Copy Markdown
Member

osuketh commented Jun 7, 2021

@laysakura こちらCI failしているので僕の方でmergeしますね

@osuketh osuketh merged commit da9871f into main Jun 7, 2021
@osuketh osuketh deleted the ci/rm-esop-e2e branch June 7, 2021 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants