Skip to content
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

Inline all aws dashboards #3688

Merged
merged 9 commits into from
Aug 8, 2022
Merged

Inline all aws dashboards #3688

merged 9 commits into from
Aug 8, 2022

Conversation

flash1293
Copy link
Contributor

What does this PR do?

Inlines all visualizations into their respective dashboards and deletes the individual visualization saved objects using https://github.com/flash1293/legacy_vis_analyzer

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
    - [ ] I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Author's Checklist

  • All panels on all dashboards still work and show the right data

Related issues

Fixes #3454

@flash1293 flash1293 added the enhancement New feature or request label Jul 12, 2022
@flash1293 flash1293 requested review from a team as code owners July 12, 2022 15:40
@elasticmachine
Copy link

elasticmachine commented Jul 12, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-08-08T11:12:21.412+0000

  • Duration: 40 min 12 sec

Test stats 🧪

Test Results
Failed 0
Passed 151
Skipped 0
Total 151

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@@ -10,7 +10,7 @@ categories:
- cloud
release: ga
conditions:
kibana.version: "^7.15.0 || ^8.0.0"
kibana.version: "^8.1.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Before merging we should see if these changes work in 7.17.x

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As mentioned via email, as the aws dashboards are partially very old the inliner script only works for version 8.1 and up

Copy link
Contributor

Choose a reason for hiding this comment

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

I tried to install this integration(changing the kibana.version to include 7.17.0 and above) with running 7.17.5 stack version and here is the error I got:

Error installing aws 1.18.1: Document "aws-0eb5a6a0-694f-11ea-b0ac-95d4ecb1fecd" has property "dashboard" which belongs to a more recent version of Kibana [8.1.0]. The last known version is [7.17.3]

Copy link
Contributor Author

@flash1293 flash1293 Jul 21, 2022

Choose a reason for hiding this comment

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

Correct, they only work for 8.1 and above - 7.x or 8.0 is not supported - that's why I adjsuted the min version in the manifest like this

@elasticmachine
Copy link

elasticmachine commented Jul 13, 2022

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (11/11) 💚
Files 91.667% (11/12) 👎 -5.471
Classes 91.667% (11/12) 👎 -5.471
Methods 82.243% (176/214) 👎 -7.019
Lines 92.894% (2157/2322) 👍 2.253
Conditionals 100.0% (0/0) 💚

Copy link
Contributor

@kaiyan-sheng kaiyan-sheng left a comment

Choose a reason for hiding this comment

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

I tested some of the dashboards with this change and the dashboards do load. But some of them don't show data caused by the known bug in the dashboards in #3703.

Copy link
Contributor

@drewdaemon drewdaemon left a comment

Choose a reason for hiding this comment

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

Looked at a random sample of five dashboards and all panels were by-value. Great work!

@flash1293
Copy link
Contributor Author

@kaiyan-sheng could you take another look? Had to re-inline due to merge conflicts

@flash1293 flash1293 merged commit aa63e1f into main Aug 8, 2022
@kaiyan-sheng kaiyan-sheng deleted the aws-inline branch August 31, 2022 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Integration:aws AWS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate AWS dashboards to by value
6 participants