Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion .rubocop_todo.yml
Original file line number Diff line number Diff line change
Expand Up @@ -581,7 +581,6 @@ Style/HashSlice:
- 'app/controllers/services/service_instances_controller.rb'
- 'app/controllers/services/service_plans_controller.rb'
- 'app/presenters/system_environment/service_binding_presenter.rb'
- 'lib/cloud_controller/blobstore/fog/fog_blob.rb'
- 'lib/cloud_controller/blobstore/storage_cli/storage_cli_blob.rb'
- 'lib/cloud_controller/blobstore/webdav/dav_blob.rb'
- 'lib/cloud_controller/telemetry_logger.rb'
Expand Down
3 changes: 0 additions & 3 deletions Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,6 @@ gem 'actionview', '~> 8.1.3'
gem 'activemodel', '~> 8.1.2'
gem 'railties', '~> 8.1.1'

gem 'fog-aws'
gem 'fog-core', '~> 2.6.0'

gem 'cf-uaa-lib', '~> 4.0.10'

group :db do
Expand Down
18 changes: 0 additions & 18 deletions Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -90,22 +90,6 @@ GEM
fluent-logger (0.10.0)
logger (~> 1.6)
msgpack (>= 1.0.0, < 2)
fog-aws (3.33.2)
base64 (>= 0.2, < 0.4)
fog-core (~> 2.6)
fog-json (~> 1.1)
fog-xml (~> 0.1)
fog-core (2.6.0)
builder
excon (~> 1.0)
formatador (>= 0.2, < 2.0)
mime-types
fog-json (1.3.0)
fog-core
multi_json (~> 1.10)
fog-xml (0.1.5)
fog-core
nokogiri (>= 1.5.11, < 2.0.0)
formatador (1.2.3)
reline
google-protobuf (4.35.1)
Expand Down Expand Up @@ -481,8 +465,6 @@ DEPENDENCIES
digest-xxhash
factory_bot (~> 6.5)
fluent-logger
fog-aws
fog-core (~> 2.6.0)
googleapis-common-protos (>= 1.8.0)
hashdiff
httpclient
Expand Down
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@ When deployed via capi-release only:
| Provider | `blobstore_type` | Backends | Notes |
|----------|------------------|-----------------------------------------------|-------|
| Storage CLI | `storage-cli` | S3, S3-compatible, GCS, Azure, Alibaba Cloud | |
| Fog | `fog` | AWS, GCS, Alibaba Cloud, OpenStack, Local/NFS | **Default.** Local/NFS not recommended for production |
| WebDAV | `webdav` | WebDAV servers | |
| Local | `local`, `local-temp-storage` | Filesystem, NFS | Development and testing only |

Expand Down
4 changes: 0 additions & 4 deletions config/bosh-lite.yml
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,6 @@ resource_pool:
uri:
key_pair_id:
private_key: ""
fog_connection: {}

packages:
blobstore_type: webdav
Expand All @@ -153,7 +152,6 @@ packages:
uri:
key_pair_id:
private_key: ""
fog_connection: {}

droplets:
blobstore_type: webdav
Expand All @@ -168,7 +166,6 @@ droplets:
uri:
key_pair_id:
private_key: ""
fog_connection: {}
max_staged_droplets_stored: 5

buildpacks:
Expand All @@ -184,7 +181,6 @@ buildpacks:
uri:
key_pair_id:
private_key: ""
fog_connection: {}

db_encryption_key: db-encryption-key

Expand Down
8 changes: 0 additions & 8 deletions config/cloud_controller.yml
Original file line number Diff line number Diff line change
Expand Up @@ -182,29 +182,21 @@ resource_pool:
minimum_size: 1
resource_directory_key: "spec-cc-resources"
blobstore_type: local-temp-storage
fog_connection: {}
fog_aws_storage_options: {}

packages:
app_package_directory_key: "cc-packages"
max_package_size: 42
max_valid_packages_stored: 42
blobstore_type: local-temp-storage
fog_connection: {}
fog_aws_storage_options: {}

droplets:
droplet_directory_key: cc-droplets
max_staged_droplets_stored: 42
blobstore_type: local-temp-storage
fog_connection: {}
fog_aws_storage_options: {}

buildpacks:
buildpack_directory_key: cc-buildpacks
blobstore_type: local-temp-storage
fog_connection: {}
fog_aws_storage_options: {}

db_encryption_key: "asdfasdfasdf"
database_encryption:
Expand Down
12 changes: 12 additions & 0 deletions lib/cloud_controller/blobstore/blob.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,18 @@ module CloudController
module Blobstore
class Blob
CACHE_ATTRIBUTES = %i[etag last_modified created_at content_length].freeze

def internal_download_url
raise NotImplementedError
end

def public_download_url
raise NotImplementedError
end

def local_path
raise NotImplementedError
end
end
end
end
31 changes: 0 additions & 31 deletions lib/cloud_controller/blobstore/client_provider.rb
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
require 'cloud_controller/blobstore/client'
require 'cloud_controller/blobstore/retryable_client'
require 'cloud_controller/blobstore/fog/fog_client'
require 'cloud_controller/blobstore/error_handling_client'
require 'cloud_controller/blobstore/webdav/dav_client'
require 'cloud_controller/blobstore/local/local_client'
Expand All @@ -18,8 +17,6 @@ def self.provide(options:, directory_key:, root_dir: nil, resource_type: nil)
provide_local(options, directory_key, root_dir, use_temp_storage: true)
when 'storage-cli'
provide_storage_cli(options, directory_key, root_dir, resource_type)
when 'fog', nil, ''
provide_fog(options, directory_key, root_dir)
else
provide_webdav(options, directory_key, root_dir)
end
Expand All @@ -28,34 +25,6 @@ def self.provide(options:, directory_key:, root_dir: nil, resource_type: nil)
class << self
private

def provide_fog(options, directory_key, root_dir)
cdn_uri = HashUtils.dig(options[:cdn], :uri)
cdn = CloudController::Blobstore::Cdn.make(cdn_uri)

client = FogClient.new(
connection_config: options.fetch(:fog_connection),
directory_key: directory_key,
cdn: cdn,
root_dir: root_dir,
min_size: options[:minimum_size],
max_size: options[:maximum_size],
aws_storage_options: options[:fog_aws_storage_options]
)

logger = Steno.logger('cc.blobstore')

# work around https://github.com/fog/fog/issues/3137
# and Fog raising an EOFError SocketError intermittently
# and https://github.com/fog/fog-aws/issues/264
# and https://github.com/fog/fog-aws/issues/265
# and intermittent GCS blobstore download errors
errors = [Excon::Errors::BadRequest, Excon::Errors::SocketError, SystemCallError,
Excon::Errors::InternalServerError, Excon::Errors::ServiceUnavailable, OpenSSL::OpenSSLError]
retryable_client = RetryableClient.new(client:, errors:, logger:)

Client.new(ErrorHandlingClient.new(SafeDeleteClient.new(retryable_client, root_dir)))
end

def provide_local(options, directory_key, root_dir, use_temp_storage:)
client = LocalClient.new(
directory_key: directory_key,
Expand Down
42 changes: 0 additions & 42 deletions lib/cloud_controller/blobstore/fog/cdn.rb

This file was deleted.

40 changes: 0 additions & 40 deletions lib/cloud_controller/blobstore/fog/fog_blob.rb

This file was deleted.

Loading
Loading