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

GCP utils #16

Merged
merged 1 commit into from
Jan 11, 2025
Merged

GCP utils #16

merged 1 commit into from
Jan 11, 2025

Conversation

JerryPan2718
Copy link
Contributor

@JerryPan2718 JerryPan2718 commented Jan 11, 2025

What changes were proposed in this pull request?

Introduces a new GCPUtils class that provides a unified interface for interacting with various Google Cloud Platform services:

  • Secret Manager operations for managing secrets
  • Cloud Storage methods for file upload/download
  • Cloud Scheduler functionality for job creation
  • Cloud Run service deployment capabilities
  • Cloud SQL database management
  • Helper methods for resource verification

Why are the changes needed?

To centralize and standardize GCP service interactions within the application, reducing code duplication and providing a consistent interface for common cloud operations. This utility class will make it easier to manage cloud resources and perform common GCP operations throughout the application.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Each GCP service method includes error handling and follows GCP best practices. Integration tests should be added to verify interactions with each cloud service in isolation.

@JerryPan2718 JerryPan2718 marked this pull request as ready for review January 11, 2025 10:21
Copy link
Contributor Author

JerryPan2718 commented Jan 11, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@JerryPan2718 JerryPan2718 merged commit dd74e3a into main Jan 11, 2025
2 checks passed
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.

1 participant