-
Notifications
You must be signed in to change notification settings - Fork 461
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
Revert kubernetes namespace field breaking change #2209
Revert kubernetes namespace field breaking change #2209
Conversation
After reverting the breaking change I think I should also revert the Kibana constraint to |
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.
The reverted change modified 54 files (#1792), but this one modifies 53, is this expected?
@@ -10,7 +10,7 @@ categories: | |||
- kubernetes | |||
release: ga | |||
conditions: | |||
kibana.version: "^8.0.0" | |||
kibana.version: "^7.16.0 || ^8.0.0" |
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.
👍
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
🤖 GitHub commentsTo re-run your PR in the CI, just comment with:
|
* Revert kubernetes namespace field breaking change * Update changelog.yml * Update the kibana constraint
What does this PR do?
After agreement on #2165 and in continuation of merging elastic/beats#29069
this PR reverts the
kubernetes.namespace
field update in kubernetes integration as well.Checklist
changelog.yml
file.manifest.yml
file to point to the latest Elastic stack release (e.g.^7.13.0
).