Skip to content

[GCP] Add GCP Load Balancer Logs Datastream - #3493

Merged
endorama merged 14 commits into
elastic:mainfrom
legoguy1000:gcp-load-balancers
Aug 23, 2022
Merged

[GCP] Add GCP Load Balancer Logs Datastream#3493
endorama merged 14 commits into
elastic:mainfrom
legoguy1000:gcp-load-balancers

Conversation

@legoguy1000

@legoguy1000 legoguy1000 commented Jun 8, 2022

Copy link
Copy Markdown
Contributor

What does this PR do?

Create a new data stream for GCP Load Balancers logs

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

@legoguy1000
legoguy1000 marked this pull request as ready for review June 8, 2022 23:43
@legoguy1000
legoguy1000 requested review from a team as code owners June 8, 2022 23:43
@elasticmachine

elasticmachine commented Jun 8, 2022

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: 2022-08-16T08:50:42.366+0000

  • Duration: 22 min 11 sec

Test stats 🧪

Test Results
Failed 0
Passed 74
Skipped 0
Total 74

🤖 GitHub comments

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

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

Copy link
Copy Markdown
Contributor Author

@efd6 can u run tests if u get a chance.

@efd6

efd6 commented Jul 11, 2022

Copy link
Copy Markdown
Contributor

/test

Comment thread packages/gcp/data_stream/load_balancer/fields/fields.yml Outdated
@legoguy1000
legoguy1000 requested a review from efd6 July 12, 2022 00:34
@efd6

efd6 commented Jul 12, 2022

Copy link
Copy Markdown
Contributor

/test

@legoguy1000

Copy link
Copy Markdown
Contributor Author

ok, should be fixed

@efd6

efd6 commented Jul 12, 2022

Copy link
Copy Markdown
Contributor

/test

@elasticmachine

elasticmachine commented Jul 12, 2022

Copy link
Copy Markdown

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (5/5) 💚
Files 100.0% (5/5) 💚 2.879
Classes 100.0% (5/5) 💚 2.879
Methods 94.186% (81/86) 👍 4.128
Lines 95.752% (1375/1436) 👍 3.925
Conditionals 100.0% (0/0) 💚
Comment thread packages/gcp/_dev/build/docs/load_balancer.md Outdated
Comment thread packages/gcp/changelog.yml Outdated
Comment thread packages/gcp/manifest.yml Outdated
Comment thread packages/gcp/data_stream/load_balancer/manifest.yml Outdated
Comment thread packages/gcp/data_stream/load_balancer/manifest.yml Outdated
Comment thread packages/gcp/data_stream/load_balancer/elasticsearch/ingest_pipeline/default.yml Outdated
Comment thread packages/gcp/changelog.yml Outdated
@legoguy1000

Copy link
Copy Markdown
Contributor Author

I will update this PR once the LB Metrics datastream is merged

@legoguy1000 legoguy1000 changed the title [GCP] Add GCP Load Balancer Datastream Jul 20, 2022
@legoguy1000
legoguy1000 requested a review from endorama July 25, 2022 16:27
@endorama endorama added Integration:gcp Google Cloud Platform Team:Cloud-Monitoring Label for the Cloud Monitoring team labels Aug 1, 2022
@legoguy1000

Copy link
Copy Markdown
Contributor Author

@endorama Rebased and ready for ur review

@endorama

Copy link
Copy Markdown
Contributor

/test

Comment thread packages/gcp/data_stream/loadbalancing_logs/fields/base-fields.yml Outdated
Comment thread packages/gcp/data_stream/loadbalancing_logs/sample_event.json Outdated
@legoguy1000

Copy link
Copy Markdown
Contributor Author

I'll fix these.

@legoguy1000
legoguy1000 requested a review from endorama August 12, 2022 14:39
@legoguy1000

Copy link
Copy Markdown
Contributor Author

@endorama Ready when u are

@endorama

Copy link
Copy Markdown
Contributor

/test

Comment thread packages/gcp/_dev/build/docs/loadbalancing.md

@sayden sayden 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 so let's ask the Grandmaster GCP for his approval

@endorama ? 😬

@legoguy1000

Copy link
Copy Markdown
Contributor Author

@endorama any reason this can't be merged

@endorama

Copy link
Copy Markdown
Contributor

I'm sorry this slipped through. LGTM

@endorama
endorama merged commit 660e504 into elastic:main Aug 23, 2022
ghost pushed a commit that referenced this pull request Aug 24, 2022
* add storage data stream

* fix sample event

* fix storage docs

* target latest changelog version

* bump to 2.7.0

* Update manifest.yml

* Update changelog.yml

* Revert "Update manifest.yml"

This reverts commit efe5ae5.

Bump to 2.7.0 was correct so this PR can be merged after #3493

* Revert "Update changelog.yml"

This reverts commit 41c69fb.

Bump to 2.7.0 was correct so this PR can be merged after #3493

* fix broken merge

* Fix typo in metric name

`authz.acl_based_object_access_.ount` -> `authz.acl_based_object_access_.count`

* update docs with fixed metric name

* remove underscore in field

Co-authored-by: Edoardo Tenani <526307+endorama@users.noreply.github.com>
Co-authored-by: Edoardo Tenani <edoardo.tenani@elastic.co>
Co-authored-by: Tom Myers <106530686+tommyers-elastic@users.noreply.github.com>
orestisfl pushed a commit to orestisfl/integrations that referenced this pull request May 15, 2026
Co-authored-by: Edoardo Tenani <526307+endorama@users.noreply.github.com>
orestisfl pushed a commit to orestisfl/integrations that referenced this pull request May 15, 2026
* add storage data stream

* fix sample event

* fix storage docs

* target latest changelog version

* bump to 2.7.0

* Update manifest.yml

* Update changelog.yml

* Revert "Update manifest.yml"

This reverts commit 196fa22853ac0aa746fab853679c3adbe7721791.

Bump to 2.7.0 was correct so this PR can be merged after elastic#3493

* Revert "Update changelog.yml"

This reverts commit 67ce298719a616ce203d21728cd7f9ea31250449.

Bump to 2.7.0 was correct so this PR can be merged after elastic#3493

* fix broken merge

* Fix typo in metric name

`authz.acl_based_object_access_.ount` -> `authz.acl_based_object_access_.count`

* update docs with fixed metric name

* remove underscore in field

Co-authored-by: Edoardo Tenani <526307+endorama@users.noreply.github.com>
Co-authored-by: Edoardo Tenani <edoardo.tenani@elastic.co>
Co-authored-by: Tom Myers <106530686+tommyers-elastic@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration:gcp Google Cloud Platform Team:Cloud-Monitoring Label for the Cloud Monitoring team

5 participants