Skip to content

[GCP] Add GCP Billing Metricset - #2141

Merged
endorama merged 3 commits into
elastic:gcp-metricsfrom
legoguy1000:490-gcp-billing
Nov 30, 2021
Merged

[GCP] Add GCP Billing Metricset#2141
endorama merged 3 commits into
elastic:gcp-metricsfrom
legoguy1000:490-gcp-billing

Conversation

@legoguy1000

Copy link
Copy Markdown
Contributor

What does this PR do?

Add GCP Billing Metricset

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.
  • If I'm introducing a new feature, I have modified the Kibana version constraint in my package's manifest.yml file to point to the latest Elastic stack release (e.g. ^7.13.0).

Author's Checklist

How to test this PR locally

Related issues

Screenshots

@legoguy1000

legoguy1000 commented Nov 8, 2021

Copy link
Copy Markdown
Contributor Author

@endorama Split from #2066

@elasticmachine

elasticmachine commented Nov 8, 2021

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: 2021-11-16T17:17:42.501+0000

  • Duration: 18 min 47 sec

  • Commit: a39a970

Test stats 🧪

Test Results
Failed 0
Passed 32
Skipped 0
Total 32

🤖 GitHub comments

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

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

Copy link
Copy Markdown
Contributor

/test

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

Copy link
Copy Markdown
Contributor Author

@endorama Per #2066 (comment), I figured but didn't want to assume. I can update this branch.

@endorama

Copy link
Copy Markdown
Contributor

@legoguy1000 yes please! Thanks

@legoguy1000

Copy link
Copy Markdown
Contributor Author

@endorama updated. Ready for review/test.

@endorama

Copy link
Copy Markdown
Contributor

/test

@legoguy1000

legoguy1000 commented Nov 25, 2021

Copy link
Copy Markdown
Contributor Author

@endorama Are you good to merge this? Then I can start on the others.

@endorama

Copy link
Copy Markdown
Contributor

Hello @legoguy1000 I've been working on adding tests but is taking longer than expected. The PR works and tests are green, so I'm going to merge and continue working in the background.

Thank you for the contribution!

@endorama
endorama merged commit 6d69fec into elastic:gcp-metrics Nov 30, 2021
@legoguy1000
legoguy1000 deleted the 490-gcp-billing branch November 30, 2021 15:31
@legoguy1000

Copy link
Copy Markdown
Contributor Author

@endorama do u want me to start building the other PRs for the other metric data sets?

@endorama

Copy link
Copy Markdown
Contributor

Yes please.

endorama pushed a commit that referenced this pull request Feb 1, 2022
endorama pushed a commit that referenced this pull request Mar 17, 2022
endorama pushed a commit that referenced this pull request Mar 22, 2022
This commit has been changed to remove changes to README included in
 #2141 to allow documentation changes from #2842

Changes to  packages/gcp/data_stream/audit/manifest.yml has been removed
too.
endorama pushed a commit that referenced this pull request Mar 22, 2022
This commit has been changed to remove changes to README included in
 #2141 to allow documentation changes from #2842
ghost pushed a commit that referenced this pull request Mar 30, 2022
* gcp: stub logs and metrics doc files

* [GCP] Add GCP Billing Metricset (#2141)

* [GCP] Add GCP Compute Metricset (#2301)

* [GCP] Add GCP Billing Metricset (#2141)

* [GCP] Add GCP Compute Metricset (#2301)

* Add firestore data_stream

* Fix formatting issue

* Add gcp tests env vars in Jenkinsfile

* Add missing env fields

* Update terraform deployer

* Update system test

* Fix credentials json field

* Format & add period

* Update docs

* Update default period for firestoer

* Update terraform deployer

* Remove period from system test

* Update manifest version

* Update manifest file

* Add default project id

* Fix owner

* Update audit docs

* Fix terraform deployer

* Remove redundant alternative host

* Fix project variable

* Remove default project string

* Remove firestore datastream tests

* Revert "Add gcp tests env vars in Jenkinsfile"

This reverts commit ceea735.

Co-authored-by: Edoardo Tenani <edoardo.tenani@elastic.co>
Co-authored-by: Alex Resnick <adr8292@gmail.com>
endorama pushed a commit that referenced this pull request Apr 5, 2022
This commit has been changed to remove changes to README included in
 #2141 to allow documentation changes from #2842

Changes to  packages/gcp/data_stream/audit/manifest.yml has been removed
too.
endorama pushed a commit that referenced this pull request Apr 5, 2022
This commit has been changed to remove changes to README included in
 #2141 to allow documentation changes from #2842
endorama pushed a commit that referenced this pull request Jun 9, 2022
This commit has been changed to remove changes to README included in
 #2141 to allow documentation changes from #2842

Changes to  packages/gcp/data_stream/audit/manifest.yml has been removed
too.
endorama pushed a commit that referenced this pull request Jun 9, 2022
This commit has been changed to remove changes to README included in
 #2141 to allow documentation changes from #2842
endorama pushed a commit that referenced this pull request Jul 12, 2022
This commit has been changed to remove changes to README included in
 #2141 to allow documentation changes from #2842

Changes to  packages/gcp/data_stream/audit/manifest.yml has been removed
too.
endorama pushed a commit that referenced this pull request Jul 12, 2022
This commit has been changed to remove changes to README included in
 #2141 to allow documentation changes from #2842
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants