Skip to content

Add tags and processors on GCP Compute, Firestore, PostgreSQL - #9508

Merged
ali786XI merged 1 commit into
elastic:mainfrom
Lucas-Feat:feat/gcp-processors
Jun 26, 2024
Merged

Add tags and processors on GCP Compute, Firestore, PostgreSQL#9508
ali786XI merged 1 commit into
elastic:mainfrom
Lucas-Feat:feat/gcp-processors

Conversation

@Lucas-Feat

@Lucas-Feat Lucas-Feat commented Apr 3, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

Proposed commit message

Add custom processors and tags to gcp Compute, Firestore and PostgreSQL integrations

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.
@Lucas-Feat
Lucas-Feat requested review from a team as code owners April 3, 2024 13:26
@cla-checker-service

cla-checker-service Bot commented Apr 3, 2024 •
edited
Loading

Copy link
Copy Markdown

💚 CLA has been signed
@Lucas-Feat
Lucas-Feat force-pushed the feat/gcp-processors branch 2 times, most recently from eb50814 to 27f261b Compare April 3, 2024 13:34
@Lucas-Feat

Lucas-Feat commented Apr 9, 2024

Copy link
Copy Markdown
Contributor Author

Do we have to reopen a new PR after we sign the Contributor Agreement ?

Thkx
@ishleenk17

ishleenk17 commented Apr 9, 2024

Copy link
Copy Markdown
Member

Do we have to reopen a new PR after we sign the Contributor Agreement ?

Thkx

No, the same one should be fine
ali786XI
ali786XI suggested changes Apr 15, 2024
View reviewed changes
Comment thread packages/gcp/data_stream/cloudsql_postgresql/agent/stream/stream.yml.hbs Outdated
Comment thread packages/gcp/data_stream/cloudsql_postgresql/agent/stream/stream.yml.hbs Outdated
@ali786XI

ali786XI commented Apr 15, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

@Lucas-Feat Can you also update the same at rest places as per above suggestions :)
Also please sign the contributor agreement else the CI won't be green
@Lucas-Feat
Lucas-Feat force-pushed the feat/gcp-processors branch from 35dae2f to 8d696ef Compare April 15, 2024 14:01
@Lucas-Feat

Lucas-Feat commented Apr 15, 2024

Copy link
Copy Markdown
Contributor Author

@Lucas-Feat Can you also update the same at rest places as per above suggestions :) Also please sign the contributor agreement else the CI won't be green

I signed it twice, i don't know what i am doing wrong.
@Lucas-Feat
Lucas-Feat force-pushed the feat/gcp-processors branch from 8d696ef to 45a24eb Compare May 14, 2024 13:20
@Lucas-Feat
Lucas-Feat force-pushed the feat/gcp-processors branch from 45a24eb to b24f409 Compare May 29, 2024 13:01
@RJiraya

RJiraya commented May 29, 2024

Copy link
Copy Markdown

Hello, what's missing to merge this ? Thanks !
@Lucas-Feat
Lucas-Feat requested a review from ali786XI May 29, 2024 13:03
@Lucas-Feat
Lucas-Feat force-pushed the feat/gcp-processors branch from b24f409 to 577265c Compare May 29, 2024 13:21
@ali786XI

ali786XI commented May 29, 2024

Copy link
Copy Markdown
Contributor

/test
ali786XI
ali786XI approved these changes May 29, 2024
View reviewed changes

@ali786XI ali786XI 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
@RJiraya

RJiraya commented Jun 12, 2024

Copy link
Copy Markdown

Hello, what's the next step to merge, and when it can be available ? Thank's !
ishleenk17
ishleenk17 reviewed Jun 12, 2024
View reviewed changes

@ishleenk17 ishleenk17 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good. Just one nit.
Comment thread packages/gcp/manifest.yml Outdated
@Lucas-Feat
Lucas-Feat force-pushed the feat/gcp-processors branch from 577265c to 102fd36 Compare June 12, 2024 13:43
@RJiraya

RJiraya commented Jun 19, 2024

Copy link
Copy Markdown

Hello, is it good now ?
@ali786XI

ali786XI commented Jun 21, 2024

Copy link
Copy Markdown
Contributor

/test
@elasticmachine

elasticmachine commented Jun 21, 2024

Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport
@elasticmachine

elasticmachine commented Jun 21, 2024

Copy link
Copy Markdown

💚 Build Succeeded

History

  • 💚 Build #12071 succeeded 577265cec01e2f87e9216f8fc10cf631eeab770f
@elastic-sonarqube

elastic-sonarqube Bot commented Jun 21, 2024

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
0.0% Duplication on New Code

See analysis details on SonarQube
@RJiraya

RJiraya commented Jun 24, 2024

Copy link
Copy Markdown

any update ?
ishleenk17
ishleenk17 approved these changes Jun 26, 2024
View reviewed changes

@ishleenk17 ishleenk17 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!
ShourieG
ShourieG approved these changes Jun 26, 2024
View reviewed changes

@ShourieG ShourieG left a comment •
edited
Loading

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
@ali786XI
ali786XI merged commit 44fb7a6 into elastic:main Jun 26, 2024
@elasticmachine

elasticmachine commented Jun 26, 2024

Copy link
Copy Markdown

Package gcp - 2.34.0 containing this change is available at https://epr.elastic.co/search?package=gcp
@andrewkroh andrewkroh added the Integration:gcp Google Cloud Platform label Jul 22, 2024
@ali786XI ali786XI mentioned this pull request Jul 27, 2024
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration:gcp Google Cloud Platform

8 participants

@Lucas-Feat @ishleenk17 @ali786XI @RJiraya @elasticmachine @MichaelKatsoulis @ShourieG @andrewkroh