-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Flaky test #131192] HTTP SSL Redirects #163726
Conversation
Pinging @elastic/kibana-core (Team:Core) |
|
||
// Skips because the current version of supertest cannot follow redirects | ||
// Can be unskipped once https://github.com/elastic/kibana/pull/163716 is merged | ||
it.skip('does not boot-loop (2nd redirect points to the landing page)', async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Being pedantically explicit, this 2nd check didn't test what the previous claimed (redirecting to the https port). That's why I split it to a different test.
Then, I found out the issue with the old library of supertest
, and I had to skip it.
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: cc @afharo |
💔 Some backports could not be created
Note: Successful backport PRs will be merged automatically after passing CI. Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
(cherry picked from commit 2611863)
# Backport This will backport the following commits from `main` to `8.9`: - [[Flaky test #131192] HTTP SSL Redirects (#163726)](#163726) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Alejandro Fernández Haro","email":"alejandro.haro@elastic.co"},"sourceCommit":{"committedDate":"2023-08-14T16:48:45Z","message":"[Flaky test #131192] HTTP SSL Redirects (#163726)","sha":"261186313e93661c93e214b7dc9a26ba0aa5b749","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Core","technical debt","release_note:skip","backport:all-open","v8.10.0"],"number":163726,"url":"https://github.com/elastic/kibana/pull/163726","mergeCommit":{"message":"[Flaky test #131192] HTTP SSL Redirects (#163726)","sha":"261186313e93661c93e214b7dc9a26ba0aa5b749"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/163726","number":163726,"mergeCommit":{"message":"[Flaky test #131192] HTTP SSL Redirects (#163726)","sha":"261186313e93661c93e214b7dc9a26ba0aa5b749"}}]}] BACKPORT--> Co-authored-by: Alejandro Fernández Haro <alejandro.haro@elastic.co>
Summary
Resolves #131192.
✅ Flaky test runner (400x): https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2860
For maintainers