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

conn-tests: move away from jenkins url #506

Closed
wants to merge 1 commit into from

Conversation

ldelossa
Copy link
Contributor

as reported here:
cilium/cilium#16938 (comment) the
usage of the jenkins url causes our tests to flake.

in this patch we move away from jenkins to the "http://docs.cilium.io"
URL which should have more reliable uptime.

Signed-off-by: Louis DeLosSantos [email protected]

@ldelossa ldelossa requested a review from a team as a code owner August 31, 2021 19:13
@ldelossa ldelossa temporarily deployed to ci August 31, 2021 19:13 Inactive
@ldelossa ldelossa requested a review from twpayne August 31, 2021 19:13
as reported here:
cilium/cilium#16938 (comment) the
usage of the jenkins url causes our tests to flake.

in this patch we move away from jenkins to the "http://docs.cilium.io"
URL which should have more reliable uptime.

Signed-off-by: Louis DeLosSantos <[email protected]>
@ldelossa ldelossa force-pushed the louis/remove-jenkins-http-test branch from cf40b6c to 244d942 Compare August 31, 2021 19:15
@ldelossa ldelossa temporarily deployed to ci August 31, 2021 19:15 Inactive
@ldelossa ldelossa marked this pull request as draft August 31, 2021 23:19
@ldelossa
Copy link
Contributor Author

Making this a draft, the flakes appear to be happening on other URLs as well, as shown by the latest failures on this PR.

@tklauser
Copy link
Member

tklauser commented Sep 3, 2021

Superseded by #511

@tklauser tklauser closed this Sep 3, 2021
@tklauser tklauser deleted the louis/remove-jenkins-http-test branch September 3, 2021 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants