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

Add documentation for GCS unsupported storage classes #26897

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nnajwaa
Copy link
Contributor

@nnajwaa nnajwaa commented Dec 24, 2024

What does this PR do? What is the motivation?

  • For GCS (Google Cloud Storage) Logs Rehydrations we only support rehydrating from the following storage Classes (Standard, Coldline and Nearline)
  • We should document this in the public docs.

Merge readiness:

  • Ready for merge

Merge queue is enabled in this repo. To have it automatically merged after it receives the required reviews, create the PR (from a branch that follows the <yourname>/description naming convention) and then add the following PR comment:

/merge

Additional notes

Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@nnajwaa nnajwaa force-pushed the najwa.elouennass/gcs_storage_class_docs branch from 6687fa6 to e97aff1 Compare December 24, 2024 14:19
@nnajwaa nnajwaa force-pushed the najwa.elouennass/gcs_storage_class_docs branch from e97aff1 to 62efa38 Compare December 24, 2024 15:00
@nnajwaa nnajwaa marked this pull request as ready for review December 26, 2024 16:32
@nnajwaa nnajwaa requested a review from a team as a code owner December 26, 2024 16:32
Copy link
Contributor

@estherk15 estherk15 left a comment

Choose a reason for hiding this comment

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

Looks good! When this is ready to publish, add the /merge comment to this PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants