Skip to content

Commit 158f488

Browse files
committed
Merge branch 'yousef/provider-pod-annotations-stack' of github.com:cohere-ai/cloud-api-adaptor into yousef/configurable-overlay-mtu-stack
2 parents 3e82607 + 040ed16 commit 158f488

27 files changed

Lines changed: 166 additions & 147 deletions

src/cloud-api-adaptor/cmd/cloud-api-adaptor/main.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ func (cfg *daemonConfig) Setup() (cmd.Starter, error) {
114114
reg.BoolWithEnv(&cfg.serverConfig.EnableCloudConfigVerify, "cloud-config-verify", false, "CLOUD_CONFIG_VERIFY", "Enable cloud config verify - should use it for production")
115115
reg.IntWithEnv(&cfg.serverConfig.PeerPodsLimitPerNode, "peerpods-limit-per-node", 10, "PEERPODS_LIMIT_PER_NODE", "peer pods limit per node (default=10)")
116116
reg.BoolWithEnv(&cfg.serverConfig.EnableScratchSpace, "enable-scratch-space", false, "ENABLE_SCRATCH_SPACE", "Enable encrypted scratch space for pod VMs")
117-
reg.StringWithEnv(&cfg.serverConfig.AllowedCloudConfigAnnotations, "allowed-cloud-config-annotations", "", "ALLOWED_CLOUD_CONFIG_ANNOTATIONS", "Comma-separated allowlist of cloud-specific pod annotations that may override operator VM configuration")
117+
reg.StringWithEnv(&cfg.serverConfig.AllowedCloudConfigAnnotations, "allowed-cloud-config-annotations", "", "ALLOWED_CLOUD_CONFIG_ANNOTATIONS", "Comma-separated allowlist of GCP or Azure pod annotations that may override operator VM configuration")
118118
reg.BoolWithEnv(&cfg.networkConfig.ExternalNetViaPodVM, "ext-network-via-podvm", false, "EXTERNAL_NETWORK_VIA_PODVM", "[EXPERIMENTAL] Enable external networking via pod VM")
119119
reg.CustomTypeWithEnv(&cfg.networkConfig.PodSubnetCIDRs, "pod-subnet-cidrs", "", "POD_SUBNET_CIDRS", "[EXPERIMENTAL] Comma separated CIDRs for local pod subnets")
120120

@@ -139,7 +139,7 @@ func (cfg *daemonConfig) Setup() (cmd.Starter, error) {
139139
// This call will be removed in a future release.
140140
cloud.LoadEnv()
141141

142-
if err := util.ValidateAllowedCloudConfigAnnotations(cfg.serverConfig.AllowedCloudConfigAnnotations); err != nil {
142+
if err := util.ValidateAllowedCloudConfigAnnotations(cloudName, cfg.serverConfig.AllowedCloudConfigAnnotations); err != nil {
143143
return nil, err
144144
}
145145

src/cloud-api-adaptor/install/charts/peerpods/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,6 @@ Prepend `io.katacontainers.config.hypervisor.` to each suffix below.
193193
|---|---|---|---|
194194
| `gcp_zone` | GCP | Selects a zone in the configured subnet region | Capacity and data-residency change |
195195
| `gcp_disk_type` | GCP | Selects the boot disk type | Cost and performance change |
196-
| `gcp_confidential_type` | GCP | Selects the configured confidential VM technology | Attestation policy compatibility |
197196
| `gcp_root_volume_size` | GCP | Increases boot disk size | Cost increase |
198197
| `gcp_use_public_ip` | GCP | Attaches an external IP | Public network exposure |
199198
| `gcp_network_tags` | GCP | Adds firewall-targeting network tags | Expands firewall policy matches |
@@ -207,7 +206,8 @@ Prepend `io.katacontainers.config.hypervisor.` to each suffix below.
207206
| `use_spot` | GCP, Azure | Requests interruptible capacity | VM eviction and workload interruption |
208207
209208
The following security-sensitive keys cannot be allowlisted:
210-
`gcp_disable_cvm`, `azure_disable_cvm`, and `azure_enable_secure_boot`.
209+
`gcp_disable_cvm`, `gcp_confidential_type`, `azure_disable_cvm`, and
210+
`azure_enable_secure_boot`.
211211
Run a separate CAA instance if workloads require weaker confidential-compute
212212
or boot-integrity settings.
213213

src/cloud-api-adaptor/install/charts/peerpods/providers/alibabacloud.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ provider: alibabacloud
66

77
providerConfigs:
88
alibabacloud:
9-
# Comma-separated allowlist of cloud-specific pod annotations that may override operator VM configuration
9+
# Comma-separated allowlist of GCP or Azure pod annotations that may override operator VM configuration
1010
# (default: "")
1111
# ALLOWED_CLOUD_CONFIG_ANNOTATIONS: ""
1212

src/cloud-api-adaptor/install/charts/peerpods/providers/aws.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ provider: aws
66

77
providerConfigs:
88
aws:
9-
# Comma-separated allowlist of cloud-specific pod annotations that may override operator VM configuration
9+
# Comma-separated allowlist of GCP or Azure pod annotations that may override operator VM configuration
1010
# (default: "")
1111
# ALLOWED_CLOUD_CONFIG_ANNOTATIONS: ""
1212

src/cloud-api-adaptor/install/charts/peerpods/providers/azure.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ provider: azure
66

77
providerConfigs:
88
azure:
9-
# Comma-separated allowlist of cloud-specific pod annotations that may override operator VM configuration
9+
# Comma-separated allowlist of GCP or Azure pod annotations that may override operator VM configuration
1010
# (default: "")
1111
# ALLOWED_CLOUD_CONFIG_ANNOTATIONS: ""
1212

src/cloud-api-adaptor/install/charts/peerpods/providers/byom.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ provider: byom
66

77
providerConfigs:
88
byom:
9-
# Comma-separated allowlist of cloud-specific pod annotations that may override operator VM configuration
9+
# Comma-separated allowlist of GCP or Azure pod annotations that may override operator VM configuration
1010
# (default: "")
1111
# ALLOWED_CLOUD_CONFIG_ANNOTATIONS: ""
1212

src/cloud-api-adaptor/install/charts/peerpods/providers/docker.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ image:
1111

1212
providerConfigs:
1313
docker: {}
14-
# Comma-separated allowlist of cloud-specific pod annotations that may override operator VM configuration
14+
# Comma-separated allowlist of GCP or Azure pod annotations that may override operator VM configuration
1515
# (default: "")
1616
# ALLOWED_CLOUD_CONFIG_ANNOTATIONS: ""
1717

src/cloud-api-adaptor/install/charts/peerpods/providers/gcp.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ provider: gcp
66

77
providerConfigs:
88
gcp:
9-
# Comma-separated allowlist of cloud-specific pod annotations that may override operator VM configuration
9+
# Comma-separated allowlist of GCP or Azure pod annotations that may override operator VM configuration
1010
# (default: "")
1111
# ALLOWED_CLOUD_CONFIG_ANNOTATIONS: ""
1212

src/cloud-api-adaptor/install/charts/peerpods/providers/ibmcloud.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ provider: ibmcloud
66

77
providerConfigs:
88
ibmcloud:
9-
# Comma-separated allowlist of cloud-specific pod annotations that may override operator VM configuration
9+
# Comma-separated allowlist of GCP or Azure pod annotations that may override operator VM configuration
1010
# (default: "")
1111
# ALLOWED_CLOUD_CONFIG_ANNOTATIONS: ""
1212

src/cloud-api-adaptor/install/charts/peerpods/providers/ibmcloudpowervs.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ provider: ibmcloudpowervs
66

77
providerConfigs:
88
ibmcloudpowervs:
9-
# Comma-separated allowlist of cloud-specific pod annotations that may override operator VM configuration
9+
# Comma-separated allowlist of GCP or Azure pod annotations that may override operator VM configuration
1010
# (default: "")
1111
# ALLOWED_CLOUD_CONFIG_ANNOTATIONS: ""
1212

0 commit comments

Comments
 (0)