Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

scripts: Add testing of etcd in local image in release workflow. #15505

Merged
merged 4 commits into from
Mar 24, 2023

Conversation

pchan
Copy link
Contributor

@pchan pchan commented Mar 18, 2023

Please see the previous PR #15350 for details.
Fixes: #15139

@ahrtr @chaochn47

@pchan
Copy link
Contributor Author

pchan commented Mar 18, 2023

I have added version checks and carried forward the review comments from before. Apologies for the git mess-up.

scripts/test_images.sh Outdated Show resolved Hide resolved
scripts/test_images.sh Outdated Show resolved Hide resolved
@pchan pchan force-pushed the add_test_image_2 branch from cb897e1 to 2ec31f1 Compare March 23, 2023 09:00
scripts/test_images.sh Outdated Show resolved Hide resolved
Copy link
Member

@ahrtr ahrtr left a comment

Choose a reason for hiding this comment

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

LGTM with a minor comment.

cc @ptabor @serathius @spzala to take a look.

scripts/test_images.sh Outdated Show resolved Hide resolved
scripts/test_images.sh Outdated Show resolved Hide resolved
scripts/test_images.sh Outdated Show resolved Hide resolved
@serathius
Copy link
Member

serathius commented Mar 23, 2023

Running detached container with server and running etcdctl put&get seems like overkill for bash.
It's ok for now, however I would suggest to rewrite the tests in Go if we want to complicate them further.

@pchan pchan force-pushed the add_test_image_2 branch from d05dfaf to 6e99afa Compare March 24, 2023 07:49
@pchan
Copy link
Contributor Author

pchan commented Mar 24, 2023

@serathius @ahrtr Thank you for your feedback. I have incoporated them and have updated the PR. Please review.

scripts/test_images.sh Outdated Show resolved Hide resolved
Signed-off-by: Prasad Chandrasekaran <prasadc@vmware.com>
@pchan pchan force-pushed the add_test_image_2 branch from 6e99afa to d5e5f1b Compare March 24, 2023 08:03
scripts/test_images.sh Outdated Show resolved Hide resolved
scripts/test_images.sh Outdated Show resolved Hide resolved
Signed-off-by: Prasad Chandrasekaran <prasadc@vmware.com>
scripts/test_images.sh Outdated Show resolved Hide resolved
pchan added 2 commits March 24, 2023 15:27
Signed-off-by: Prasad Chandrasekaran <prasadc@vmware.com>
Signed-off-by: Prasad Chandrasekaran <prasadc@vmware.com>
@ahrtr
Copy link
Member

ahrtr commented Mar 31, 2023

@pchan could you backport this PR to both 3.5 and 3.4? thx

@pchan
Copy link
Contributor Author

pchan commented Apr 1, 2023

@pchan could you backport this PR to both 3.5 and 3.4? thx

Created #15608 and #15609 for 3.5 and 3.4 respectively.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Test etcd container in K8s cluster
3 participants