Skip to content

Commit

Permalink
Review remarks
Browse files Browse the repository at this point in the history
  • Loading branch information
janboll committed Nov 21, 2024
1 parent 64cc4f7 commit b44f96e
Show file tree
Hide file tree
Showing 6 changed files with 9 additions and 8 deletions.
7 changes: 4 additions & 3 deletions config/config.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -226,15 +226,16 @@
"name": {
"type": "string"
},
"minimumTlsVersionAllowed": {
"type": "string"
"minTLSVersion": {
"type": "string",
"enum": ["1.2"]
}
},
"additionalProperties": false,
"required": [
"maxClientSessionsPerAuthName",
"name",
"minimumTlsVersionAllowed"
"minTLSVersion"
]
},
"imageBase": {
Expand Down
2 changes: 1 addition & 1 deletion config/config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ defaults:
eventGrid:
name: arohcp-maestro-{{ .ctx.regionShort }}
maxClientSessionsPerAuthName: '4'
minimumTlsVersionAllowed: '1.2'
minTLSVersion: '1.2'
certDomain: 'selfsigned.maestro.keyvault.azure.com'
postgres:
name: arohcp-maestro-{{ .ctx.regionShort }}
Expand Down
2 changes: 1 addition & 1 deletion config/public-cloud-cs-pr.json
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@
"consumerName": "hcp-underlay-cspr-mgmt-1",
"eventGrid": {
"maxClientSessionsPerAuthName": "4",
"minimumTlsVersionAllowed": "1.2",
"minTLSVersion": "1.2",
"name": "arohcp-maestro-cspr"
},
"imageBase": "quay.io/redhat-user-workloads/maestro-rhtap-tenant/maestro/maestro",
Expand Down
2 changes: 1 addition & 1 deletion config/public-cloud-dev.json
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@
"consumerName": "hcp-underlay-dev-mgmt-1",
"eventGrid": {
"maxClientSessionsPerAuthName": "4",
"minimumTlsVersionAllowed": "1.2",
"minTLSVersion": "1.2",
"name": "arohcp-maestro-dev"
},
"imageBase": "quay.io/redhat-user-workloads/maestro-rhtap-tenant/maestro/maestro",
Expand Down
2 changes: 1 addition & 1 deletion config/public-cloud-personal-dev.json
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@
"consumerName": "hcp-underlay-usw3tst-mgmt-1",
"eventGrid": {
"maxClientSessionsPerAuthName": "4",
"minimumTlsVersionAllowed": "1.2",
"minTLSVersion": "1.2",
"name": "arohcp-maestro-usw3tst"
},
"imageBase": "quay.io/redhat-user-workloads/maestro-rhtap-tenant/maestro/maestro",
Expand Down
2 changes: 1 addition & 1 deletion dev-infrastructure/configurations/region.tmpl.bicepparam
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,4 @@ param regionalDNSSubdomain = '{{ .regionalDNSSubdomain }}'
param maestroKeyVaultName = '{{ .maestro.keyVaultName }}'
param maestroEventGridNamespacesName = '{{ .maestro.eventGrid.name }}'
param maestroEventGridMaxClientSessionsPerAuthName = {{ .maestro.eventGrid.maxClientSessionsPerAuthName }}
param maestroEventGridMinimumTlsVersionAllowed = '{{ .maestro.eventGrid.minimumTlsVersionAllowed }}'
param maestroEventGridMinimumTlsVersionAllowed = '{{ .maestro.eventGrid.minTLSVersion }}'

0 comments on commit b44f96e

Please sign in to comment.