Skip to content
This repository was archived by the owner on Jun 2, 2021. It is now read-only.

Commit aebbe82

Browse files
committed
kpack is no longer optional under kubernetes
This has been the case for some time, but was not reflected in the config schema [#173630293] Authored-by: Matt Royal <mroyal@pivotal.io>
1 parent 9604a97 commit aebbe82

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

lib/cloud_controller/config_schemas/api_schema.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -372,7 +372,7 @@ class ApiSchema < VCAP::Config
372372
},
373373
ca_file: String,
374374
workloads_namespace: String,
375-
optional(:kpack) => { # TODO: this isn't optional in practice
375+
kpack: {
376376
builder_namespace: String,
377377
registry_service_account_name: String,
378378
registry_tag_base: String,

lib/cloud_controller/config_schemas/worker_schema.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ class WorkerSchema < VCAP::Config
179179
},
180180
ca_file: String,
181181
workloads_namespace: String,
182-
optional(:kpack) => {
182+
kpack: {
183183
builder_namespace: String,
184184
registry_service_account_name: String,
185185
registry_tag_base: String,

0 commit comments

Comments
 (0)