-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Elastic Agent] Rename *ConfigChange to PolicyChange #20779
[Elastic Agent] Rename *ConfigChange to PolicyChange #20779
Conversation
This rename the ConfigChagne to PolicyChange for: - The internal type for the actions. - The handler - The introspections This depends on change on Kibana to return the new Type and the new "Policy" key.
Pinging @elastic/ingest-management (Team:Ingest Management) |
💔 Build FailedExpand to view the summary
Build stats
Test stats 🧪
Steps errorsExpand to view the steps failures
Log outputExpand to view the last 100 lines of log output
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks ok, it either needs to be synced with fleet changes or made in a non-breaking two step process when we accept both types with same result and then we remove CONFIG_CHANGE
later
I've talked to @michalpristas and he will take it over from me. |
Hi @michalpristas, I have Kibana changes here: elastic/kibana#78221 The Kibana changes includes BWC for <= 7.9 agent. I tested changing policies with a 7.9.1 snapshot build of agent and it worked well. Could you verify BWC as well as with the agent changes in this PR? Thanks! |
@jen-huang tested with agent from this branch. it retrieves action of type |
- Prevent closing closed reader {pull}20214[20214] | ||
- Fix Windows service installation script {pull}20203[20203] | ||
- Fix timeout issue stopping service applications {pull}20256[20256] | ||
- Fix rename *ConfigChange to *PolicyChange to align on changes in the UI. {pull}20779[20779] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops changelog issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wou
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changelog vs changelog.next
[Elastic Agent] Rename *ConfigChange to PolicyChange (elastic#20779) Co-authored-by: Michal Pristas <michal.pristas@gmail.com>
…ci-build-label-support * upstream/master: [JJBB] Set shallow cloning to 10 (elastic#21409) docs: add link to release notes for 7.9.2 (elastic#21405) (elastic#21419) docs: Prepare Changelog for 7.9.2 (elastic#21229) (elastic#21403) fix: mark flaky tests (elastic#21300) fix: use a fixed version of setuptools (elastic#21393) Move Kubernetes events metricset to its own block in reference config (elastic#21407) [libbeat] Enable WriteAheadLimit in the disk queue (elastic#21391) docs: fix apt/yum formatting (elastic#21362) Fix shutdown tracking in s3 input (elastic#21380) [libbeat] Fix position writing in the disk queue Add UBI 8 image to the dependencies report (elastic#21374) Fix debug message to show actual SQS message ID (elastic#20614) [Elastic Agent] Rename *ConfigChange to PolicyChange (elastic#20779) [Elastic Agent] Add install/uninstall sub-command (elastic#21206) [Filebeat][httpjson] Make httpjson use cursor input when using date cursor (elastic#20751) feat: prepare release pipelines (elastic#21238) Add IP validation to Security module (elastic#21325)
This rename the ConfigChagne to PolicyChange for:
This depends on change on Kibana to return the new Type and the new
"Policy" key.
The kibana API need to be adjusted to send the right action type and the right fields.
What does this PR do?
Why is it important?
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
Logs