[Entitlements] Fixing reindex policy #123599
Conversation
ldematte
added
>non-issue
auto-backport
|
Pinging @elastic/es-core-infra (Team:Core/Infra) |
| - create_class_loader # needed for Painless to generate runtime classes | ||
| - files: | ||
| - relative_path: "" | ||
| - relative_path_setting: "reindex.ssl.certificate" |
There was a problem hiding this comment.
@rjernst I have no idea if I'm using this correctly, as this is the first time we use it I think.
Will this work with exclusive access?
Can I use a glob (reindex.ssl.*) instead?
certificate_authorities is a list; will that work with your file settings implementation?
There was a problem hiding this comment.
It should work if the setting is a list, we resolve all paths, whether for a single setting, or multiple settings. It should also work with exclusive. Also note the properties are changing per our earlier discussion, see #123649.
But it's curious that we didn't need this before? I guess it would have only worked if it was relative to the config dir.
There was a problem hiding this comment.
Also note the properties are changing per our earlier discussion, see #123649.
Thanks! So reindex.ssl.* should work? I'll try it/look at logs/debug what it resolves to. Or maybe I'll write a test for it.
I'll update this to use the new properties after that is merged.
|
Obsolete |
3 participants
Relates to ES-10031