Skip to content

Added GCS support guide to template files#233

Open
ammarhusain-EMR wants to merge 6 commits intoni:mainfrom
ammarhusain-EMR:users/ammar/gcs-support
Open

Added GCS support guide to template files#233
ammarhusain-EMR wants to merge 6 commits intoni:mainfrom
ammarhusain-EMR:users/ammar/gcs-support

Conversation

@ammarhusain-EMR
Copy link

What does this Pull Request accomplish?

Added comments to help the user configure GCS as object storage for SLE using the template values and secrets file.

Note: GCS support is not yet added to dataframe service. Because in dataframe service we are using S3 DeleteObjects API which is not supported by GCS S3 interoperable XML API, refer this for more information. The respective documentation for the dataframe service will be updated once we add this support.

Why should this Pull Request be merged?

Helps the user configure GCS as an object storage solution in SystemLink Enterprise except for Dataframe service.

What testing has been done?

NA

@ammarhusain-EMR ammarhusain-EMR changed the title Users/ammar/gcs support Added GCS support guide to template files Dec 2, 2024
@ammarhusain-EMR
Copy link
Author

Following our last discussion, I will keep this PR active.

Copy link
Collaborator

@prestwick prestwick left a comment

Choose a reason for hiding this comment

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

Overall looks great. This just needs some small updates to align with the decision to no longer redistribute MinIO and reference it in our documentation.

@SSSantosh18 SSSantosh18 requested a review from prestwick January 27, 2025 14:49
dataframeservice:
secrets:
## Access key information for S3 access.
## Access key information for S3/GCS access.
Copy link
Collaborator

Choose a reason for hiding this comment

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

We will also support Azure Block Storage soon. And the overall structure of these configuration options are changing. You should contact Paul Spangler and Csenge Gyorfi and standardize on terminology and layout.

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