Skip to content

Conversation

@lcawl
Copy link
Contributor

@lcawl lcawl commented Sep 8, 2018

@lcawl lcawl added >docs General docs changes :Security/Security Security issues without another label v6.3.3 labels Sep 8, 2018
@lcawl lcawl requested a review from jaymode September 8, 2018 01:34
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security

@lcawl lcawl force-pushed the lcawley-breaking-conf branch from 5b506be to ac40622 Compare September 17, 2018 16:59
Copy link
Member

Choose a reason for hiding this comment

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

After digging into this, we've found that the breaking change is a bug. So maybe this should be a known issue instead? The fix is in #33688. The issue is the following:

  • Pre 6.3 with x-pack installed put the default configuration files for security in ES_PATH_CONF/x-pack
  • 6.3+ puts the default configuration files in ES_PATH_CONF/x-pack
  • A BWC layer was added to look in the ES_PATH_CONF/x-pack if the file did not exist in ES_PATH_CONF/x-pack. This essentially means that the old files are ignored since the new default files are there. The workaround is to move the old files up a level (out of the x-pack directory)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the information @jaymode. Will #33688 be backported to the 6.3 branch too? If not, I think this issue gets more visibility by staying in the breaking changes page.

Copy link
Member

Choose a reason for hiding this comment

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

No it will not be backported to 6.3

@lcawl lcawl force-pushed the lcawley-breaking-conf branch from c6a29c9 to e4407d6 Compare October 1, 2018 16:15
@lcawl lcawl merged commit 734593c into elastic:6.3 Oct 1, 2018
@lcawl lcawl deleted the lcawley-breaking-conf branch October 1, 2018 17:21
lcawl added a commit to lcawl/elasticsearch that referenced this pull request Oct 1, 2018
lcawl added a commit that referenced this pull request Oct 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>docs General docs changes :Security/Security Security issues without another label v6.3.3 v6.4.3 v6.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants