Skip to content

test: Download correct cilium-istioctl for the executing OS.#12109

Merged
aanm merged 1 commit intomasterfrom
pr/jrajahalme/test-istioctl-download-os
Jun 17, 2020
Merged

test: Download correct cilium-istioctl for the executing OS.#12109
aanm merged 1 commit intomasterfrom
pr/jrajahalme/test-istioctl-download-os

Conversation

@jrajahalme
Copy link
Copy Markdown
Member

Only use the Ginkgo runtime OS for determining which cilium-istioctl
binary to download is the command executor is local, otherwise default
to "linux". This supports Ginkgo running in OSX both with local and
SSH Executors.

Fixes: #11905
Signed-off-by: Jarno Rajahalme jarno@covalent.io

Only use the Ginkgo runtime OS for determining which cilium-istioctl
binary to download is the command executor is local, otherwise default
to "linux". This supports Ginkgo running in OSX both with local and
SSH Executors.

Fixes: #11905
Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
@jrajahalme jrajahalme added kind/bug/CI This is a bug in the testing code. area/CI Continuous Integration testing issue or flake needs-backport/1.8 labels Jun 16, 2020
@jrajahalme jrajahalme requested a review from a team as a code owner June 16, 2020 17:28
@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

3 similar comments
@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

@jrajahalme
Copy link
Copy Markdown
Member Author

test-me-please

@jrajahalme jrajahalme added the release-note/ci This PR makes changes to the CI. label Jun 16, 2020
@coveralls
Copy link
Copy Markdown

coveralls commented Jun 16, 2020

Coverage Status

Coverage decreased (-0.01%) to 37.062% when pulling a840535 on pr/jrajahalme/test-istioctl-download-os into 331aab4 on master.

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

Labels

area/CI Continuous Integration testing issue or flake kind/bug/CI This is a bug in the testing code. release-note/ci This PR makes changes to the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants