Skip to content

Commit

Permalink
Reducing string restrictions on k8s service limits/requests (#889)
Browse files Browse the repository at this point in the history
So that they can actually reference variables
  • Loading branch information
juandiegopalomino authored May 19, 2022
1 parent 74ecb21 commit ae71d45
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 12 deletions.
6 changes: 3 additions & 3 deletions modules/aws_k8s_service/aws-k8s-service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -216,10 +216,10 @@ inputs:
default: 25
extra_validators:
limits:
memory: any(int(min=1), regex('^[1-9][0-9]{0,4}$'), required=True)
memory: any(int(min=1), str(), required=True)
resources:
cpu: any(int(min=1), regex('^[1-9][0-9]{0,4}$'), required=True)
memory: any(int(min=1), regex('^[1-9][0-9]{0,4}$'), required=True)
cpu: any(int(min=1), str(), required=True)
memory: any(int(min=1), str(), required=True)
cron_job:
commands: list(required=True)
schedule: str(required=True)
Expand Down
6 changes: 3 additions & 3 deletions modules/azure_k8s_service/azure-k8s-service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -187,10 +187,10 @@ inputs: # (what users see)
default: 25
extra_validators:
limits:
memory: any(int(min=1), regex('^[1-9][0-9]{0,4}$'), required=True)
memory: any(int(min=1), str(), required=True)
resources:
cpu: any(int(min=1), regex('^[1-9][0-9]{0,4}$'), required=True)
memory: any(int(min=1), regex('^[1-9][0-9]{0,4}$'), required=True)
cpu: any(int(min=1), str(), required=True)
memory: any(int(min=1), str(), required=True)
persistent_storage:
size: int(required=True)
path: str(required=True)
Expand Down
6 changes: 3 additions & 3 deletions modules/gcp_k8s_service/gcp-k8s-service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -207,10 +207,10 @@ inputs:
default: 25
extra_validators:
limits:
memory: any(int(min=1), regex('^[1-9][0-9]{0,4}$'), required=True)
memory: any(int(min=1), str(), required=True)
resources:
cpu: any(int(min=1), regex('^[1-9][0-9]{0,4}$'), required=True)
memory: any(int(min=1), regex('^[1-9][0-9]{0,4}$'), required=True)
cpu: any(int(min=1), str(), required=True)
memory: any(int(min=1), str(), required=True)
cron_job:
commands: list(required=True)
schedule: str(required=True)
Expand Down
6 changes: 3 additions & 3 deletions modules/local_k8s_service/local-k8s-service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -176,10 +176,10 @@ inputs:
default: 25
extra_validators:
limits:
memory: any(int(min=1), regex('^[1-9][0-9]{0,4}$'), required=True)
memory: any(int(min=1), str(), required=True)
resources:
cpu: any(int(min=1), regex('^[1-9][0-9]{0,4}$'), required=True)
memory: any(int(min=1), regex('^[1-9][0-9]{0,4}$'), required=True)
cpu: any(int(min=1), str(), required=True)
memory: any(int(min=1), str(), required=True)
cron_job:
commands: list(required=True)
schedule: str(required=True)
Expand Down

0 comments on commit ae71d45

Please sign in to comment.