Skip to content

[GCP] [Billing] Improve documentation - #9931

Merged
15 commits merged into
mainfrom
unknown repository
Aug 12, 2024
Merged

[GCP] [Billing] Improve documentation#9931
15 commits merged into
mainfrom
unknown repository

Conversation

@ghost

@ghost ghost commented May 21, 2024

Copy link
Copy Markdown

Overview

Improves GCP Billing documentation to clarify what each config param means.

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

Related issues

Screenshots

@ghost ghost self-assigned this May 21, 2024
@elasticmachine

elasticmachine commented May 21, 2024

Copy link
Copy Markdown

🚀 Benchmarks report

Package gcp 👍(1) 💚(2) 💔(3)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
audit 2192.98 1692.05 -500.93 (-22.84%) 💔
dns 3030.3 2237.14 -793.16 (-26.17%) 💔
vpcflow 3984.06 3344.48 -639.58 (-16.05%) 💔

To see the full report comment with /test benchmark fullreport

@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

@ghost
ghost marked this pull request as ready for review May 21, 2024 20:25
@ghost
ghost requested review from a team as code owners May 21, 2024 20:25
Comment thread packages/gcp/_dev/build/docs/billing.md Outdated
Comment thread packages/gcp/_dev/build/docs/billing.md Outdated
Comment thread packages/gcp/_dev/build/docs/billing.md Outdated
Comment thread packages/gcp/_dev/build/docs/billing.md
Comment thread packages/gcp/_dev/build/docs/billing.md Outdated
Comment thread packages/gcp/_dev/build/docs/billing.md Outdated
Comment thread packages/gcp/docs/billing.md
@botelastic

botelastic Bot commented Jun 26, 2024

Copy link
Copy Markdown

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic Bot added the Stalled label Jun 26, 2024
Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>
@botelastic botelastic Bot removed the Stalled label Jul 18, 2024
Gabriel Pop and others added 5 commits July 18, 2024 11:45
Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>
Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>
Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>
@ghost
ghost requested review from ishleenk17 and shmsr July 18, 2024 09:55
@andrewkroh andrewkroh added the Integration:gcp Google Cloud Platform label Jul 19, 2024

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

Left a few minor editing suggestions, otherwise LGTM.

Comment thread packages/gcp/_dev/build/docs/billing.md Outdated
Comment thread packages/gcp/docs/billing.md Outdated
Gabriel Pop and others added 3 commits August 12, 2024 18:05
Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @gpop63

@ghost
ghost merged commit 4448989 into elastic:main Aug 12, 2024
@elasticmachine

Copy link
Copy Markdown

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

harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 4, 2025
* improve docs

* bump package version

* fix pr id

* add cost_type in docs

* Update packages/gcp/_dev/build/docs/billing.md

Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>

* Update packages/gcp/_dev/build/docs/billing.md

Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>

* Update packages/gcp/_dev/build/docs/billing.md

Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>

* Update packages/gcp/_dev/build/docs/billing.md

Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>

* address reviews

* Update packages/gcp/_dev/build/docs/billing.md

Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>

* Update packages/gcp/docs/billing.md

Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>

* remove newline

---------

Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>
Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 5, 2025
* improve docs

* bump package version

* fix pr id

* add cost_type in docs

* Update packages/gcp/_dev/build/docs/billing.md

Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>

* Update packages/gcp/_dev/build/docs/billing.md

Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>

* Update packages/gcp/_dev/build/docs/billing.md

Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>

* Update packages/gcp/_dev/build/docs/billing.md

Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>

* address reviews

* Update packages/gcp/_dev/build/docs/billing.md

Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>

* Update packages/gcp/docs/billing.md

Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>

* remove newline

---------

Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>
Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration:gcp Google Cloud Platform

7 participants