Skip to content

Fix example plugin using config dir #122212

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.

Sign up for GitHub

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

Jump to bottom
Merged
rjernst merged 1 commit into from
Feb 11, 2025
Merged

Fix example plugin using config dir #122212

rjernst merged 1 commit into from
Feb 11, 2025

Conversation

@rjernst
Copy link
Member

@rjernst rjernst commented Feb 10, 2025

This commit fixes a use of accessing the config dir from the environment.

@rjernst
Fix example plugin using config dir
9911ce4 
This commit fixes a use of accessing the config dir from the
environment.
@rjernst rjernst added :Core/Infra/Settings Settings infrastructure and APIs auto-backport Automatically create backport pull requests when merged v8.18.1 v8.19.0 v9.0.1 v9.1.0 labels Feb 10, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Feb 10, 2025
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Feb 10, 2025

Pinging @elastic/es-core-infra (Team:Core/Infra)

ldematte
ldematte approved these changes Feb 11, 2025
View reviewed changes
Copy link
Contributor

@ldematte ldematte left a comment

Choose a reason for hiding this comment

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

LGTM

@rjernst rjernst merged commit 30a65fb into elastic:main Feb 11, 2025
18 checks passed
@rjernst rjernst deleted the plugins/example_config_dir_use branch February 11, 2025 19:06
@rjernst rjernst mentioned this pull request Feb 11, 2025
Merged
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Feb 11, 2025
@rjernst
Fix example plugin using config dir (elastic#122212)
685f279 
This commit fixes a use of accessing the config dir from the
environment.
@rjernst rjernst mentioned this pull request Feb 11, 2025
Merged
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Feb 11, 2025
@rjernst
Fix example plugin using config dir (elastic#122212)
9e917e1 
This commit fixes a use of accessing the config dir from the
environment.
@rjernst rjernst mentioned this pull request Feb 11, 2025
Merged
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Feb 11, 2025

💚 Backport successful

Status Branch Result
✅ 8.18
✅ 8.x
✅ 9.0

rjernst added a commit to rjernst/elasticsearch that referenced this pull request Feb 11, 2025
@rjernst
Fix example plugin using config dir (elastic#122212)
f6cbf54 
This commit fixes a use of accessing the config dir from the
environment.
elasticsearchmachine pushed a commit that referenced this pull request Feb 11, 2025
@rjernst
Fix example plugin using config dir (#122212) (#122284)
d0907db 
This commit fixes a use of accessing the config dir from the
environment.
rjernst added a commit that referenced this pull request Feb 13, 2025
@rjernst
Fix example plugin using config dir (#122212) (#122286)
d04a78d 
This commit fixes a use of accessing the config dir from the
environment.
rjernst added a commit that referenced this pull request Feb 13, 2025
@rjernst
Fix example plugin using config dir (#122212) (#122285)
a67d659 
This commit fixes a use of accessing the config dir from the
environment.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ldematte ldematte ldematte approved these changes

Assignees

No one assigned

Labels

auto-backport Automatically create backport pull requests when merged :Core/Infra/Settings Settings infrastructure and APIs >non-issue Team:Core/Infra Meta label for core/infra team v8.18.1 v8.19.0 v9.0.1 v9.1.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rjernst @elasticsearchmachine @ldematte