Skip to content

Commit

Permalink
allow per-app overrides of storageClass
Browse files Browse the repository at this point in the history
  • Loading branch information
AdamNowotny committed Feb 7, 2025
1 parent eec640b commit 52dfc07
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 5 deletions.
7 changes: 7 additions & 0 deletions Pulumi.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,13 @@ config:
# Below is list of all settings with default values assuming a single node cluster.
# Do NOT modify this file but add overrides in your Pulumi.<stack>.yaml.

# Standard settings supported by most apps:
# - enabled (enable/disable app)
# - hostname (hostname for HTTPS endpoint)
# - storageOnly (disable app but retain storage)
# - storageSize (expand default storage size if needed)
# - storageClass (force storage class used by app)

### SYSTEM ###

# tailscale:apiKey: TAILSCALE_API_KEY
Expand Down
6 changes: 1 addition & 5 deletions components/application.ts
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,6 @@ interface ContainerSpec {
*
* The `add*` methods use "fluent interface" to allow provisioning resources through "method chaining".
*
* Some standard configuration settings are supported:
* - storageOnly
* - storageSize
* - hostname
*
* Limitations:
* - only one Deployment supported
* - one endpoint required for Deployment
Expand Down Expand Up @@ -85,6 +80,7 @@ export class Application {
namespace: this.namespace.metadata.name,
size: args?.size ?? this.config.require('storageSize'),
type: args?.type ?? PersistentStorageType.Default,
storageClass: this.config.get('storageClass'),
},
{ parent: this.scope },
);
Expand Down

0 comments on commit 52dfc07

Please sign in to comment.