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

Fixing multipleNamespaceTest bug - Missing expect statement in test #3983

Merged
merged 3 commits into from
Jul 27, 2021

Conversation

jaidevmane
Copy link
Contributor

@jaidevmane jaidevmane commented Jul 26, 2021

Signed-off-by: Jai Subash Devmane [email protected]

Thank you for contributing to Velero!

Please add a summary of your change

  • Bug Fix for multiple namespace test where Expect statement was not present

Does your change fix a particular issue?

Fixes #(issue)

Please indicate you've done the following:

@github-actions github-actions bot requested review from jenting and zubron July 26, 2021 01:46
@jaidevmane jaidevmane changed the title Fixing multipleNamespaceTest bug - No expect statement in test Fixing multipleNamespaceTest bug - Missing expect statement in test Jul 26, 2021
Signed-off-by: Jai Subash Devmane <[email protected]>
@jenting jenting requested a review from sseago July 26, 2021 02:12
@sseago
Copy link
Collaborator

sseago commented Jul 26, 2021

#3987 seems to be a duplicate of this fix -- the only difference is the failure message. Both PRs are failing CI tests on kind with this change in the same way, although talking to @jaidevmane just now, the tests are succeeding locally.

@ywk253100
Copy link
Contributor

@jaidevmane The case random failure is a known issue #3970 and I'll raise PR to fix it.
As #3989 get merged, could you try to rebase your PR? and this should workaround the CI failure

@jaidevmane
Copy link
Contributor Author

@ywk253100 Thank you for your guidance. I have incorporated your suggestions as mentioned.

@jaidevmane
Copy link
Contributor Author

All checks passed. Ready to review.

@jenting jenting merged commit 36b1aaa into vmware-tanzu:main Jul 27, 2021
dsu-igeek pushed a commit to dsu-igeek/velero that referenced this pull request Jul 29, 2021
…mware-tanzu#3983)

* Fixing multipleNamespaceTest bug - No expect statement in test
* adding changelog

Signed-off-by: Jai Subash Devmane <[email protected]>
Signed-off-by: Dave Smith-Uchida <[email protected]>
gyaozhou pushed a commit to gyaozhou/velero-read that referenced this pull request May 14, 2022
…mware-tanzu#3983)

* Fixing multipleNamespaceTest bug - No expect statement in test
* adding changelog

Signed-off-by: Jai Subash Devmane <[email protected]>
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.

4 participants