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

Change cloud.provider from googlecloud to gcp in billing metricset #22287

Merged
merged 1 commit into from
Oct 30, 2020
Merged

Change cloud.provider from googlecloud to gcp in billing metricset #22287

merged 1 commit into from
Oct 30, 2020

Conversation

kaiyan-sheng
Copy link
Contributor

@kaiyan-sheng kaiyan-sheng commented Oct 29, 2020

This PR is to change cloud.provider from googlecloud to gcp in billing metricset to match the rest of the metricsets under googlecloud module. This is to match the definition of this field in ECS.

The rest of the metricsets under googlecloud module had cloud.provider changed to gcp in #21775.

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Oct 29, 2020
@kaiyan-sheng kaiyan-sheng self-assigned this Oct 29, 2020
@kaiyan-sheng kaiyan-sheng added needs_backport PR is waiting to be backported to other branches. review Team:Platforms Label for the Integrations - Platforms team and removed needs_team Indicates that the issue/PR needs a Team:* label labels Oct 29, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations-platforms (Team:Platforms)

@elasticmachine
Copy link
Collaborator

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 606
Skipped 48
Total 654

@elasticmachine
Copy link
Collaborator

💚 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

Expand to view the summary

Build stats

  • Build Cause: [Pull request #22287 opened]

  • Start Time: 2020-10-29T22:28:36.311+0000

  • Duration: 49 min 49 sec

Test stats 🧪

Test Results
Failed 0
Passed 606
Skipped 48
Total 654

@kaiyan-sheng kaiyan-sheng merged commit f23f05a into elastic:master Oct 30, 2020
@kaiyan-sheng kaiyan-sheng deleted the billing_gcp_name branch October 30, 2020 13:26
@kaiyan-sheng kaiyan-sheng added v7.11.0 and removed needs_backport PR is waiting to be backported to other branches. labels Oct 30, 2020
kaiyan-sheng added a commit that referenced this pull request Oct 30, 2020
kaiyan-sheng added a commit that referenced this pull request Oct 30, 2020
v1v added a commit to v1v/beats that referenced this pull request Oct 30, 2020
* upstream/master:
  [JJBB] Add 6.8+ branches (elastic#22321)
  [CI] Support Windows-8 (elastic#22307)
  Change cloud.provider from googlecloud to gcp in billing metricset (elastic#22287)
  [packaging][beats-tester] use commit id binaries (elastic#22027)
  [CI] Report error in the catch section (elastic#22297)
v1v added a commit to v1v/beats that referenced this pull request Oct 30, 2020
* upstream/master:
  [JJBB] Add 6.8+ branches (elastic#22321)
  [CI] Support Windows-8 (elastic#22307)
  Change cloud.provider from googlecloud to gcp in billing metricset (elastic#22287)
  [packaging][beats-tester] use commit id binaries (elastic#22027)
  [CI] Report error in the catch section (elastic#22297)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review Team:Platforms Label for the Integrations - Platforms team v7.10.0 v7.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants