-
Couldn't load subscription status.
- Fork 21.8k
Update helm upgrade command for GCZ deployment #127778
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
@diksha-hop : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 1434853: ✅ Validation status: passed
For more details, please refer to the build report. |
The above needs to be changed to match step 1 from the community https://community.opengroup.org/osdu/platform/consumption/geospatial/-/blob/master/devops/azure/charts/geospatial/README.md#create-the-gcz-helm-chart-values This part needs to be changed too https://github.com/MicrosoftDocs/azure-docs/blob/f161f56f45317780d3b42c79b6b58abb05f2037f/articles/energy-data-services/includes/how-to/how-to-deploy-gcz/deploy-gcz-on-aks.md#unix-shell-1. The custom values have changed in the community. |
| ```bash | ||
| helm upgrade -i $CHART . -n $NAMESPACE -f osdu_gcz_custom_values.yaml --set-file global.provider.configLoaderJs="../../../../gcz-provider/gcz-provider-core/config/configLoader.js" | ||
| helm upgrade -i "$RELEASE_CHART_NAME" . -n "$NAMESPACE" -f osdu_gcz_custom_values.yaml \ |
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.
|
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
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.
Pull Request Overview
This PR updates the Helm upgrade command in the GCZ deployment documentation to improve readability and consistency. The command is refactored to use quoted variables and split across multiple lines for better clarity.
Key Changes:
- Modified the Helm upgrade command to use quoted environment variables (
"$RELEASE_CHART_NAME"and"$NAMESPACE") instead of an unquoted variable reference - Split the command across two lines for improved readability
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ```bash | ||
| helm upgrade -i $CHART . -n $NAMESPACE -f osdu_gcz_custom_values.yaml --set-file global.provider.configLoaderJs="../../../../gcz-provider/gcz-provider-core/config/configLoader.js" | ||
| helm upgrade -i "$RELEASE_CHART_NAME" . -n "$NAMESPACE" -f osdu_gcz_custom_values.yaml \ |
Copilot
AI
Oct 24, 2025
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.
The variable $RELEASE_CHART_NAME is introduced in this command but was not used previously (the original command used $CHART). This documentation should either define $RELEASE_CHART_NAME before this step or continue using $CHART for consistency with the variable that was presumably defined earlier in the documentation.
| helm upgrade -i "$RELEASE_CHART_NAME" . -n "$NAMESPACE" -f osdu_gcz_custom_values.yaml \ | |
| helm upgrade -i "$CHART" . -n "$NAMESPACE" -f osdu_gcz_custom_values.yaml \ |


updated help upgrade command