-
Notifications
You must be signed in to change notification settings - Fork 47
CORTX-29179: Update RGW endpoints #172
CORTX-29179: Update RGW endpoints #172
Conversation
Adjust RGW http/https ports to reflect new CORTX defaults. Signed-off-by: Keith Pine <[email protected]>
These changes are not really needed functionality-wise. Aside from hard-coding the ports in the Chart templates, everything is configured correctly. This just brings a new deployment up-to-date with the CORTX defaults, and cleans up the |
k8_cortx_cloud/cortx-cloud-helm-pkg/cortx-configmap/templates/_config.tpl
Show resolved
Hide resolved
k8_cortx_cloud/cortx-cloud-helm-pkg/cortx-configmap/templates/_config.tpl
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
k8_cortx_cloud/cortx-cloud-helm-pkg/cortx-server/templates/cortx-server-pod.yaml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One question on the updates to the individual Server Pod NodePort Services, but other than that it looks pretty good to me.
k8_cortx_cloud/cortx-cloud-helm-pkg/cortx-server/templates/cortx-server-svc.yaml
Outdated
Show resolved
Hide resolved
Signed-off-by: Keith Pine <[email protected]>
k8_cortx_cloud/cortx-cloud-helm-pkg/cortx-server/templates/cortx-server-pod.yaml
Show resolved
Hide resolved
Signed-off-by: Keith Pine <[email protected]>
Signed-off-by: Keith Pine <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Adjust RGW http/https ports to reflect updated CORTX defaults.
config.yaml
ConfigMap updated to match latest sampleconfig.yaml
removed as it's no longer usedSigned-off-by: Keith Pine [email protected]