Skip to content

Conversation

@serdarozerr
Copy link
Contributor

Context

The GCS client currently supports a limited set of operations (e.g., get, delete, exists, sign). However, in order to integrate it with the Cloud Controller, several additional capabilities are required, including recursive delete, listing objects, retrieving properties, copying objects, and ensuring bucket existence.

Solution

The missing operations have now been implemented. In addition, integration tests have been added to validate these new features and ensure end-to-end correctness.

- ensure-storage-exists
- list
- delete-recursive
- listing with prefix
- deleting with prefix
- copying an object
- ensuring a bucket is exist
- retriving attributes of object
These tests are added. Also logging in new functionality is added.
- ensure-bucket-exist refactored. ProjectId moved inside that function since it was failing while creating a new read only client.
- semaphore based concurency limit added. Currently we set number of concurent request to 10
@jochenehret jochenehret self-requested a review January 5, 2026 13:13
jochenehret
jochenehret previously approved these changes Jan 5, 2026
@github-project-automation github-project-automation bot moved this from Inbox to Pending Merge | Prioritized in Foundational Infrastructure Working Group Jan 5, 2026
@stephanme
Copy link
Member

We plan to merge the PR end of the week.
Integration tests are red because the PR comes from a fork -> missing creds.

- log.Printf statements are added into all operations
- logs in gcs_encryption_tests are suppressed to keep output clean
@rkoster rkoster moved this from Pending Merge | Prioritized to Waiting for Changes | Open for Contribution in Foundational Infrastructure Working Group Jan 15, 2026
@github-project-automation github-project-automation bot moved this from Waiting for Changes | Open for Contribution to Pending Merge | Prioritized in Foundational Infrastructure Working Group Jan 16, 2026
@stephanme stephanme merged commit 45325d2 into cloudfoundry:main Jan 16, 2026
3 of 5 checks passed
@github-project-automation github-project-automation bot moved this from Pending Merge | Prioritized to Done in Foundational Infrastructure Working Group Jan 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants