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/obs-infraobs-integrations: Address skipped validation SVR00004 #8423
Conversation
🌐 Coverage report
|
Please read the full PR description before reviewing. |
CC: @harnish-elastic @aliabbas-elastic to review the dashboard changes as part of the pass by value is correctly being added. |
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 for the changes. Please refer here for testing details
Hi @shmsr, please update your branch with the latest contents from main branch. There was an important PR merged updating the CI pipelines. Thanks! |
What is pending on this as the validation is complete? |
I have to resolve the merge conflicts. I'll do it ASAP. |
76edce9
to
e596616
Compare
@harnish-elastic @aliabbas-elastic I have resolved the merge conflicts. I had to inline viz in springboot from the checkout of main brain as dashboard changes had been made in the last couple of months. Otherwise, for other packages only manifest and changelog yaml's are touched. So it'd be great if could take a look at springboot again. |
Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as |
e596616
to
1da12b1
Compare
Fixed merge conflicts. |
🚀 Benchmarks reportTo see the full report comment with |
💚 Build Succeeded
cc @shmsr |
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
As latest changes in oracle_weblogic includes saved searches; inlining them could only be done for 8.10 or above stack. So leaving out oracle_weblogic. cc: @harnish-elastic |
Package apache_spark - 1.0.2 containing this change is available at https://epr.elastic.co/search?package=apache_spark |
Package azure_functions - 0.2.1 containing this change is available at https://epr.elastic.co/search?package=azure_functions |
Package hadoop - 1.5.2 containing this change is available at https://epr.elastic.co/search?package=hadoop |
Package iis - 1.17.4 containing this change is available at https://epr.elastic.co/search?package=iis |
Package spring_boot - 1.2.2 containing this change is available at https://epr.elastic.co/search?package=spring_boot |
Package traefik - 1.11.1 containing this change is available at https://epr.elastic.co/search?package=traefik |
Package websphere_application_server - 1.1.1 containing this change is available at https://epr.elastic.co/search?package=websphere_application_server |
Tip: How to use the inliner script? Follow this. Also read the following:
Proposed commit message
Used https://github.com/elastic/visualizations_integrations_tools/blob/master/inline.js to inline "by reference" viz.
Couldn't migrate other packages 11 packages out of 19 packages with SVR00004 as inliner script does not support inlining saved searches because the script was developed before users could have by-value (inline) saved searches in Kibana dashboards. So those packages have references to "search" and hence SVR00004 couldn't be addressed.
Thanks to Drew for creating this related issue to update the inliner script: elastic/visualizations_integrations_tools#2
Checklist
changelog.yml
file.Author's Checklist
Here is a list of packages that have SVR00004 with the kibana.version:
websphere_application_server
(8.3.0).How to test this PR locally
Review should include loading the dashboard into an instance with data to make sure everything is displayed properly.
Related issues
obs-infraobs-integrations
owned packages toformat_version: 3.0.0
#8028