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

Relax timeout in waitForActiveLicense #113673

Conversation

DaveCTurner
Copy link
Contributor

No sense in failing these tests just because the cluster took a bit too
long to start up.

Closes #113343

No sense in failing these tests just because the cluster took a bit too
long to start up.

Closes elastic#113343
@DaveCTurner DaveCTurner added >test Issues or PRs that are addressing/adding tests :Delivery/Build Build or test infrastructure v8.16.0 v9.0.0 labels Sep 27, 2024
@elasticsearchmachine elasticsearchmachine added the Team:Delivery Meta label for Delivery team label Sep 27, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@DaveCTurner DaveCTurner added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) auto-backport-and-merge labels Oct 3, 2024
@elasticsearchmachine elasticsearchmachine merged commit 52fd836 into elastic:main Oct 3, 2024
15 of 16 checks passed
@DaveCTurner DaveCTurner deleted the 2024/09/27/waitForActiveLicense-longer branch October 3, 2024 18:35
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Oct 3, 2024
No sense in failing these tests just because the cluster took a bit too
long to start up.

Closes elastic#113343
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.x

elasticsearchmachine pushed a commit that referenced this pull request Oct 3, 2024
No sense in failing these tests just because the cluster took a bit too
long to start up.

Closes #113343
matthewabbott pushed a commit to matthewabbott/elasticsearch that referenced this pull request Oct 4, 2024
No sense in failing these tests just because the cluster took a bit too
long to start up.

Closes elastic#113343
elasticsearchmachine pushed a commit that referenced this pull request Nov 5, 2024
Test was fixed by #113673 but wasn't unmuted at the time.

Closes #113343
elasticsearchmachine pushed a commit that referenced this pull request Nov 5, 2024
jozala pushed a commit that referenced this pull request Nov 13, 2024
Test was fixed by #113673 but wasn't unmuted at the time.

Closes #113343
alexey-ivanov-es pushed a commit to alexey-ivanov-es/elasticsearch that referenced this pull request Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team >test Issues or PRs that are addressing/adding tests v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] DocsClientYamlTestSuiteIT test {yaml=reference/ccr/apis/follow/post-resume-follow/line_84} failing
3 participants