-
Notifications
You must be signed in to change notification settings - Fork 522
[integrations/linux] - update kibana constraint to 9.x #12368
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
[integrations/linux] - update kibana constraint to 9.x #12368
Conversation
|
Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane) |
ac7f228 to
9669f5d
Compare
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.
Changes LGTM but CI is failing.
|
@jlind23 I've converted it to v3 and fixed all the dashboards (finally 😌). Can you please take another look at screenshots? |
6527822 to
0cb5033
Compare
|
@leehinman @jsoriano if you can also take a look at these changes, as they are major changes. I'd appreciate your review! |
jsoriano
left a comment
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 good, only didn't check changes in dashboards in detail.
A changelog entry would be needed.
packages/linux/kibana/dashboard/linux-system-c431f410-f9ac-11e9-90e8-1fb18e796788.json
Outdated
Show resolved
Hide resolved
leehinman
left a comment
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.
LGTM. One question about the check exclusion but it isn't a blocker.
|
@jsoriano I've added changelog and updated minor version. Can you please take another look? |
💚 Build Succeeded
History
|
|
packages/linux/changelog.yml
Outdated
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.
Wrong PR number. Should be 12368.
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.
Sorry 🤦
Here's the fix PR. PTAL!
* linux integration * add object_type * long * additional property * release and license * chore: convert to lens * update minor version and changelog * remove validations
* linux integration * add object_type * long * additional property * release and license * chore: convert to lens * update minor version and changelog * remove validations




Closes #12143
Screenshots:

