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

Resolve SERVICE_UNAVAILABLE in IdP IntegTest #54622

Merged
merged 1 commit into from
Apr 3, 2020

Conversation

tvernum
Copy link
Contributor

@tvernum tvernum commented Apr 2, 2020

The SamlIdentityProviderTests IntegTests would sometimes encounter a
service unavailable exception when registering a new service provider.

This change ensure that there is a data node, and that the cluster
state is recovered before registering providers

Resolves: #54555

The SamlIdentityProviderTests IntegTests would sometimes encounter a
service unavailable exception when registering a new service provider.

This change ensure that there is a data node, and that the cluster
state is recovered before registering providers
@tvernum tvernum added >test Issues or PRs that are addressing/adding tests :Security/Security Security issues without another label v8.0.0 v7.8.0 v7.7.1 labels Apr 2, 2020
@tvernum tvernum requested a review from jkakavas April 2, 2020 03:57
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security (:Security/Security)

Copy link
Member

@jkakavas jkakavas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tvernum tvernum merged commit 9ad03b7 into elastic:master Apr 3, 2020
tvernum added a commit to tvernum/elasticsearch that referenced this pull request Apr 3, 2020
The SamlIdentityProviderTests IntegTests would sometimes encounter a
service unavailable exception when registering a new service provider.

This change ensure that there is a data node, and that the cluster
state is recovered before registering providers

Backport of: elastic#54622
tvernum added a commit that referenced this pull request Apr 6, 2020
The SamlIdentityProviderTests IntegTests would sometimes encounter a
service unavailable exception when registering a new service provider.

This change ensure that there is a data node, and that the cluster
state is recovered before registering providers

Backport of: #54622
tvernum added a commit to tvernum/elasticsearch that referenced this pull request Apr 6, 2020
The SamlIdentityProviderTests IntegTests would sometimes encounter a
service unavailable exception when registering a new service provider.

This change ensure that there is a data node, and that the cluster
state is recovered before registering providers

Backport of: elastic#54622
Backport of: elastic#54700
tvernum added a commit that referenced this pull request Apr 6, 2020
The SamlIdentityProviderTests IntegTests would sometimes encounter a
service unavailable exception when registering a new service provider.

This change ensure that there is a data node, and that the cluster
state is recovered before registering providers

Backport of: #54622, #54700
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Security/Security Security issues without another label >test Issues or PRs that are addressing/adding tests v7.7.1 v7.8.0 v8.0.0-alpha1
Projects
None yet
4 participants