Skip to content

GCP: Redis Field Type Fix - #10159

Merged
Linu-Elias merged 7 commits into
elastic:mainfrom
Linu-Elias:fix-gcp
Jun 28, 2024
Merged

GCP: Redis Field Type Fix#10159
Linu-Elias merged 7 commits into
elastic:mainfrom
Linu-Elias:fix-gcp

Conversation

@Linu-Elias

@Linu-Elias Linu-Elias commented Jun 17, 2024

Copy link
Copy Markdown
Contributor

Proposed commit message

GCP Redis field persistence.rdb.bgsave_in_progress had incorrect type.
This field was not used in the dashboards in the previous version.

Changed the GCP Redis field persistence.rdb.bgsave_in_progress type to boolean

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

  • [ ]

How to test this PR locally

I've tested with elastic-package, and run the tests for the respective package.

Related issues

Screenshots

@Linu-Elias
Linu-Elias requested review from a team as code owners June 17, 2024 06:20
@cla-checker-service

cla-checker-service Bot commented Jun 17, 2024

Copy link
Copy Markdown

💚 CLA has been signed

@ishleenk17

Copy link
Copy Markdown
Member

@Linu-Elias : Please sign the CLA.

@ghost ghost assigned Linu-Elias Jun 17, 2024
@ghost
ghost requested review from a user and lucian-ioan June 17, 2024 11:50
@ghost ghost added the Integration:gcp Google Cloud Platform label Jun 17, 2024
@ghost

ghost commented Jun 17, 2024

Copy link
Copy Markdown

/test

@elasticmachine

Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@ghost ghost 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.

LGTM. Please make the changelog entry description a bit more descriptive.

Comment thread packages/gcp/changelog.yml Outdated
@ghost

ghost commented Jun 18, 2024

Copy link
Copy Markdown

/test

@lucian-ioan lucian-ioan 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

@Linu-Elias

Copy link
Copy Markdown
Contributor Author

/test

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @Linu-Elias

@elastic-sonarqube

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@Linu-Elias
Linu-Elias merged commit a8a06ba into elastic:main Jun 28, 2024
@elasticmachine

Copy link
Copy Markdown

Package gcp - 2.34.1 containing this change is available at https://epr.elastic.co/search?package=gcp

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration:gcp Google Cloud Platform

4 participants