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

Enable the E2E test on Github Action #3912

Merged
merged 1 commit into from Jul 1, 2021

Conversation

ywk253100
Copy link
Contributor

@ywk253100 ywk253100 commented Jun 29, 2021

  1. Run the E2E test with kind(provision various versions of k8s cluster) and MinIO on Github Action
  2. Bug fix: the variable "stdoutBuf" is assigned to both "installPluginCmd.Stdout" and "installPluginCmd.Stderr", this causes 'if !strings.Contains(stderrBuf.String(), "Duplicate value")' takes no effect as the "stderrBuf.String()" is always empty
  3. Print the stdout and stderr for easy debugging

Signed-off-by: Wenkai Yin(尹文开) yinw@vmware.com

Thank you for contributing to Velero!

Please add a summary of your change

Does your change fix a particular issue?

Fixes #(issue)

Please indicate you've done the following:

@ywk253100 ywk253100 mentioned this pull request Jun 29, 2021
3 tasks
@github-actions github-actions bot added Area/Design Design Documents Dependencies Pull requests that update a dependency file Documentation has-unit-tests Website non-docs changes for the website labels Jun 29, 2021
@github-actions github-actions bot removed Documentation Dependencies Pull requests that update a dependency file Area/Design Design Documents has-unit-tests Website non-docs changes for the website labels Jun 29, 2021
@ywk253100 ywk253100 requested review from reasonerjt, danfengliu and dsu-igeek and removed request for carlisia and jenting June 29, 2021 15:05
Copy link
Contributor

@dsu-igeek dsu-igeek left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for getting this together. Just a couple of small changes.

.github/workflows/e2e-test-kind.yaml Show resolved Hide resolved
if: steps.image-cache.outputs.cache-hit != 'true'
run: |
IMAGE=velero VERSION=main make container
docker save velero:main -o ./velero.tar
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's call the image version "pr-test" or if possible, something unique. That will keep us from ever getting confused that main is under test.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated

reasonerjt
reasonerjt previously approved these changes Jun 30, 2021
Copy link
Contributor

@reasonerjt reasonerjt left a comment

Choose a reason for hiding this comment

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

lgtm

1. Run the E2E test with kind(provision various versions of k8s cluster) and MinIO on Github Action
2. Bug fix: the variable "stdoutBuf" is assigned to both "installPluginCmd.Stdout" and "installPluginCmd.Stderr", this causes 'if !strings.Contains(stderrBuf.String(), "Duplicate value")' takes no effect as the "stderrBuf.String()" is always empty
3. Print the stdout and stderr for easy debugging

Signed-off-by: Wenkai Yin(尹文开) <yinw@vmware.com>
Copy link
Contributor

@dsu-igeek dsu-igeek left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@reasonerjt reasonerjt merged commit f28a41d into vmware-tanzu:main Jul 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants