Skip to content

Remove fog-local and fog-openstack blobstore support#5197

Merged
johha merged 1 commit into
cloudfoundry:mainfrom
sap-contributions:Remove-fog-local-and-openstack
Jun 26, 2026
Merged

Remove fog-local and fog-openstack blobstore support#5197
johha merged 1 commit into
cloudfoundry:mainfrom
sap-contributions:Remove-fog-local-and-openstack

Conversation

@WeiQuan0605

Copy link
Copy Markdown
Contributor

Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

  • A short explanation of the proposed change:
    Remove fog-local and fog-openstack gem dependencies and all related code paths.

  • An explanation of the use cases your change solves
    fog-local and fog-openstack are legacy blobstore providers that are no longer used in production Cloud Foundry deployments. They have no migration path in storage-cli, which is the recommended replacement for all fog-based blobstores. Removing them reduces the gem surface area, eliminates dead code branches in FogClient, and removes a real-filesystem dependency from the test suite that caused unnecessary I/O and cleanup complexity.

  • Links to any other associated PRs

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

@WeiQuan0605 WeiQuan0605 force-pushed the Remove-fog-local-and-openstack branch from 0ee8209 to 568022d Compare June 16, 2026 18:10
@WeiQuan0605 WeiQuan0605 force-pushed the Remove-fog-local-and-openstack branch 3 times, most recently from 3cdb785 to a3e5093 Compare June 26, 2026 10:12
Drop the fog-local and fog-openstack gems, remove all related code paths
in FogClient (local? method, Local storage branch in files_for, OpenStack
limit workaround, local endpoint override), and migrate all test specs from
real-filesystem fog-local mocks to in-memory fog-aws mocks.
@WeiQuan0605 WeiQuan0605 force-pushed the Remove-fog-local-and-openstack branch from a3e5093 to 5f5e039 Compare June 26, 2026 10:51

@johha johha left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@johha johha merged commit bdaef63 into cloudfoundry:main Jun 26, 2026
11 checks passed
@johha johha deleted the Remove-fog-local-and-openstack branch June 26, 2026 12:46
ari-wg-gitbot added a commit to cloudfoundry/capi-release that referenced this pull request Jun 26, 2026
Changes in cloud_controller_ng:

- Remove fog-local and fog-openstack blobstore support
    PR: cloudfoundry/cloud_controller_ng#5197
    Author: Wei Quan <64415962+WeiQuan0605@users.noreply.github.com>
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.

2 participants