Skip to content

[Vertex AI] Review the integration page #12433

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

Merged
merged 0 commits into from
Jan 30, 2025
Merged

[Vertex AI] Review the integration page #12433

merged 0 commits into from
Jan 30, 2025

Conversation

alaudazzi
Copy link
Contributor

@alaudazzi alaudazzi commented Jan 23, 2025

This PR improves the GCP Vertex AI documentation.

  • Align the structure to the template
  • Edit the text

Fixes https://github.com/elastic/obs-integration-team/issues/164
Related: https://github.com/elastic/obs-integration-team/issues/159

@alaudazzi alaudazzi added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] labels Jan 23, 2025
@alaudazzi alaudazzi requested a review from ishleenk17 January 23, 2025 10:36
@alaudazzi alaudazzi requested a review from a team as a code owner January 23, 2025 10:36
@andrewkroh andrewkroh added the Integration:gcp_vertexai GCP Vertex AI label Jan 23, 2025
@ishleenk17
Copy link
Member

@daniela-elastic : Please check the documentation for GCP Vertex AI.
This will be used for Vertex AI GA.

@ishleenk17
Copy link
Member

@daniela-elastic : Please check the documentation for GCP Vertex AI. This will be used for Vertex AI GA.

@daniela-elastic : I plan to get this merged today, please let me know if you have any further comments.

@muthu-mps
Copy link
Contributor

@alaudazzi - Can you include the changelog and manifest entry for this change?

@elasticmachine
Copy link

💚 Build Succeeded

History

  • 💚 Build #20863 succeeded 4d16741f55f3465921c52c4ea275134bf3c8ca78
  • 💚 Build #20845 succeeded be3e20733adf3249b14c0e2113e2f1bd348165b6

Copy link

Copy link

@daniela-elastic daniela-elastic left a comment

Choose a reason for hiding this comment

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

Left a couple of comments on the URL to Requirements. Please address if the concern is valid, else feel free to merge.

@ishleenk17 ishleenk17 merged this pull request into main Jan 30, 2025
5 checks passed
@ishleenk17 ishleenk17 deleted the edit-gcp-vertexai branch January 30, 2025 04:50
@elastic-vault-github-plugin-prod

Package gcp_vertexai - 0.4.1 containing this change is available at https://epr.elastic.co/package/gcp_vertexai/0.4.1/

harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 4, 2025
* Align the page structure with the template

* Editing

* Update changelog and manifest

* Fix the link format

* Update changelog and manifest
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 5, 2025
* Align the page structure with the template

* Editing

* Update changelog and manifest

* Fix the link format

* Update changelog and manifest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:gcp_vertexai GCP Vertex AI Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants