Skip to content
New issue

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

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

Already on GitHub? # to your account

test/connectivity: Run connectivity test in non-default namespace #1118

Merged
merged 1 commit into from
Oct 3, 2022

Conversation

sayboras
Copy link
Member

@sayboras sayboras commented Oct 3, 2022

This is to make sure that we have the coverage of running tests in a namespace other than cilium-test.

Relates: #1109 (comment)
Signed-off-by: Tam Mach tam.mach@cilium.io

This is to make sure that we have the coverage of running tests in a
namespace other than cilium-test.

Relates: cilium#1109 (comment)
Signed-off-by: Tam Mach <tam.mach@cilium.io>
@sayboras sayboras temporarily deployed to ci October 3, 2022 12:54 Inactive
@sayboras sayboras marked this pull request as ready for review October 3, 2022 12:54
@sayboras sayboras requested review from a team as code owners October 3, 2022 12:54
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Oct 3, 2022
@tklauser tklauser merged commit 3fa865d into cilium:master Oct 3, 2022
@sayboras sayboras deleted the tam/non-default-ns branch October 3, 2022 20:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants