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

Add migration E2E test #5124

Merged
merged 1 commit into from
Aug 10, 2022

Conversation

danfengliu
Copy link
Contributor

@danfengliu danfengliu commented Jul 15, 2022

Signed-off-by: danfengl [email protected]

Thank you for contributing to Velero!

Please add a summary of your change

Does your change fix a particular issue?

Fixes #4791

Please indicate you've done the following:

  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Created a changelog file or added /kind changelog-not-required as a comment on this pull request.
  • Updated the corresponding documentation in site/content/docs/main.

@danfengliu danfengliu added kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes E2E Tests End to end test labels Jul 15, 2022
@github-actions github-actions bot added Dependencies Pull requests that update a dependency file has-e2e-tests has-unit-tests labels Jul 15, 2022
@danfengliu danfengliu force-pushed the add-migration-e2e-test branch 13 times, most recently from 9e2dcbe to 6be34fa Compare July 18, 2022 06:19
@github-actions github-actions bot removed the Dependencies Pull requests that update a dependency file label Jul 18, 2022
@danfengliu danfengliu force-pushed the add-migration-e2e-test branch 2 times, most recently from cba508e to f25bb8c Compare July 18, 2022 08:16
@github-actions github-actions bot added the Dependencies Pull requests that update a dependency file label Jul 18, 2022
@danfengliu danfengliu force-pushed the add-migration-e2e-test branch 2 times, most recently from b635a59 to f670c06 Compare July 18, 2022 09:05
@github-actions github-actions bot removed the Dependencies Pull requests that update a dependency file label Jul 18, 2022
@danfengliu danfengliu force-pushed the add-migration-e2e-test branch from f670c06 to 245b72e Compare July 18, 2022 09:06
@blackpiglet blackpiglet removed the request for review from dsu-igeek July 19, 2022 03:22
@danfengliu danfengliu force-pushed the add-migration-e2e-test branch 14 times, most recently from 41de9d5 to 1f26b73 Compare August 8, 2022 02:12
@qiuming-best
Copy link
Contributor

coverage.out is it needed to push into our repo?

StandbyCluster string
ClientToBeInstall TestClient
DefaultClient TestClient
StandbyClient TestClient
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe it is better to use pointer type (*TestClient)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right, good point!

@danfengliu
Copy link
Contributor Author

coverage.out is it needed to push into our repo?

It's not needed, I will remove it.

@danfengliu danfengliu force-pushed the add-migration-e2e-test branch 4 times, most recently from 8b19141 to 1f2c48b Compare August 9, 2022 06:15
@danfengliu danfengliu force-pushed the add-migration-e2e-test branch from 1f2c48b to b8b1961 Compare August 9, 2022 08:15
Signed-off-by: danfengl <[email protected]>
@danfengliu danfengliu force-pushed the add-migration-e2e-test branch from b8b1961 to fb445b3 Compare August 9, 2022 08:31
@danfengliu danfengliu merged commit a71237c into vmware-tanzu:main Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E2E Tests End to end test has-e2e-tests has-unit-tests kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add migration test in nightly
4 participants