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

[kbn-es] Only enable security for trial #20803

Merged
merged 2 commits into from
Jul 26, 2018

Conversation

tylersmalley
Copy link
Contributor

@tylersmalley tylersmalley commented Jul 14, 2018

Prior to this PR, you would be locked out of Kibana if you attempted to start a trial when using kbn-es. This is due to the fact we were enabling security when running with a basic or trial license. We should only be enabling security for a trial license. The reason security is disabled for trial by default, is to allow for starting a trial at will since security requires additional configuration to set up authentication.

To test:

  • Start ES with yarn es snapshot
  • Start Kibana
  • Management > License Management > Start Trial
  • Ensure the trial starts and security is not enabled, resulting in being signed out of Kibana.

@elastic/kibana-operations

Closes: #20799

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@tylersmalley
Copy link
Contributor Author

retest

Copy link
Member

@jbudz jbudz left a comment

Choose a reason for hiding this comment

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

I'm not sure how I missed this, sorry about that. LGTM.

Copy link
Member

@mistic mistic left a comment

Choose a reason for hiding this comment

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

LGTM too! 👍

@tylersmalley
Copy link
Contributor Author

retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@tylersmalley
Copy link
Contributor Author

retest

@tylersmalley
Copy link
Contributor Author

Fifth times a charm?

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@jbudz
Copy link
Member

jbudz commented Jul 26, 2018

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@tylersmalley tylersmalley merged commit 489e5bb into elastic:master Jul 26, 2018
tylersmalley added a commit that referenced this pull request Jul 26, 2018
@tylersmalley
Copy link
Contributor Author

6.5/6.x: 2ffae02

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review Team:Operations Team label for Operations Team v6.5.0 v7.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants