Skip to content

[O11y][AWS] Migrate AWS Redshift dashboard input controls - #6865

Merged
rajvi-patel-22 merged 4 commits into
elastic:mainfrom
kush-elastic:lens_migration-aws-redshift
Jul 19, 2023
Merged

[O11y][AWS] Migrate AWS Redshift dashboard input controls#6865
rajvi-patel-22 merged 4 commits into
elastic:mainfrom
kush-elastic:lens_migration-aws-redshift

Conversation

@kush-elastic

@kush-elastic kush-elastic commented Jul 7, 2023

Copy link
Copy Markdown
Contributor
  • Enhancement

What does this PR do?

  • Manually migrate AWS Redshift dashboard visualizations to the lens in the current Kibana version 8.8.1 itself.

  • Statistics for AWS Redshift dashboard Lens migration:

AWS Redshift Dashboards Before Migration       After Migration      
  Control Panel Search Lens Visualization New Controls Search Lens Visualization
[Metrics AWS] Redshift metrics overview 1 0 16 0 1 0 16 0

Checklist

  • I have added an entry to my package's changelog.yml file.
  • I have verified that panels are populated with data.
  • I have verified that panels are not distorted after being migrated to the lens.
  • I have updated screenshots of the dashboard.
  • I have verified that the data counts are matching and panel aggregations are the same as before.

Author's Checklist

  • Migrated panels should be removed from visualization folder.
  • Migrated visualizations are populating in current Kibana version 8.8.1 itself.

Related issues
@kush-elastic
kush-elastic requested review from a team as code owners July 7, 2023 11:16
@elasticmachine

elasticmachine commented Jul 7, 2023 •
edited
Loading

Copy link
Copy Markdown

💚 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: 2023-07-19T07:39:41.068+0000

  • Duration: 51 min 45 sec

Test stats 🧪

Test Results
Failed 0
Passed 194
Skipped 4
Total 198

🤖 GitHub comments

Expand to view the GitHub comments

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

  • /test : Re-trigger the build.
@elasticmachine

elasticmachine commented Jul 7, 2023 •
edited
Loading

Copy link
Copy Markdown

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (15/15) 💚
Files 93.75% (15/16) 👎 -6.25
Classes 93.75% (15/16) 👎 -6.25
Methods 85.714% (240/280) 👍 12.987
Lines 85.925% (7387/8597) 👎 -14.075
Conditionals 100.0% (0/0) 💚
@kush-elastic kush-elastic self-assigned this Jul 7, 2023
@kush-elastic kush-elastic added the enhancement New feature or request label Jul 7, 2023
rajvi-patel-22
rajvi-patel-22 approved these changes Jul 10, 2023
View reviewed changes

@rajvi-patel-22 rajvi-patel-22 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!
@rajvi-patel-22 rajvi-patel-22 mentioned this pull request Jul 10, 2023
Closed
5 tasks
@drewdaemon

drewdaemon commented Jul 10, 2023

Copy link
Copy Markdown
Contributor

Nice work. Can we swap out those legacy metric visualizations for the normal Lens metric?
@rajvi-patel-22 rajvi-patel-22 mentioned this pull request Jul 11, 2023
Closed
24 tasks
@kush-elastic

kush-elastic commented Jul 11, 2023

Copy link
Copy Markdown
Contributor Author

Nice work. Can we swap out those legacy metric visualizations for the normal Lens metric?

@drewdaemon, I have created a separate backlog issue (#6909) to track this and will raise separate PR for converting those visualizations to new metric.
mbondyra
mbondyra approved these changes Jul 11, 2023
View reviewed changes

@mbondyra mbondyra left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good to me, seeing that the metrics will be swapped separately 🙆🏼‍♀️
@kush-elastic

kush-elastic commented Jul 19, 2023

Copy link
Copy Markdown
Contributor Author

The dashboards are exported from 8.7.0 due to export issue from 8.8.x (elastic/elastic-package#1354)
@kush-elastic
Merge branch 'main' of https://github.com/kush-elastic/integrations i…
2782fe3 
…nto lens_migration-aws-redshift

Conflicts:
	packages/aws/changelog.yml
	packages/aws/manifest.yml
@rajvi-patel-22
rajvi-patel-22 merged commit 9d4bcaa into elastic:main Jul 19, 2023
@elasticmachine

elasticmachine commented Jul 19, 2023

Copy link
Copy Markdown

Package aws - 1.47.1 containing this change is available at https://epr.elastic.co/search?package=aws
orestisfl pushed a commit to orestisfl/integrations that referenced this pull request May 15, 2026
@kush-elastic
[O11y][AWS] Migrate AWS Redshift dashboard input controls (elastic#6865)
f024436 
* Migrate AWS Redshift dashboard input controls

* Update changelog

* Export dashboards from 8.7.0
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

7 participants

@kush-elastic @elasticmachine @drewdaemon @mbondyra @kaiyan-sheng @rajvi-patel-22 @andrewkroh