Skip to content

Commit

Permalink
Fix the document about the flags
Browse files Browse the repository at this point in the history
There shouldn't have the = character

Longhorn-3714

Signed-off-by: Phan Le <phan.le@suse.com>
  • Loading branch information
PhanLe1010 authored and innobead committed Sep 2, 2022
1 parent cccabbf commit cc7a937
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions scripts/longhorn_rancher_chart_migration.sh
Original file line number Diff line number Diff line change
@@ -12,9 +12,9 @@ set -e
usage() {
echo ""
echo "The migration includes:"
echo "1. Running the script with --type=migrate to migrate the labels and annotations for Longhorn resources"
echo "1. Running the script with --type migrate to migrate the labels and annotations for Longhorn resources"
echo "2. Manually installing Longhorn chart in app&marketplace UI"
echo "3. Running script with --type=cleanup to remove the old Longhorn chart from old catalog UI"
echo "3. Running script with --type cleanup to remove the old Longhorn chart from old catalog UI"
echo ""
echo "usage:"
echo "$0 [options]"
@@ -119,7 +119,7 @@ if [ "$TYPE" == "cleanup" ] ; then

MANAGER=$(kubectl --kubeconfig ${DOWNSTREAM_KUBECONFIG} -n ${RELEASE_NAMESPACE} get ds longhorn-manager -ojsonpath="{.metadata.labels['app\.kubernetes\.io/managed-by']}")
if [ $MANAGER != "Helm" ] ; then
echo "Labels have not been migrated. Did you run the part 1 by specifying the flag --type=migrate ?"
echo "Labels have not been migrated. Did you run the part 1 by specifying the flag --type migrate ?"
exit 1
fi

@@ -198,4 +198,4 @@ echo " 3. Select the chart version corresponding the Longhorn version ${RANCHER
echo " 4. Install the chart with the correct helm values. Here are the helm values of your old charts: "
printf '%s\n' "${RANCHERAPPVALUES:-$RANCHERAPPANSWERS}"
echo " 5. Verify that the migrated charts are working ok"
echo " 6. Run this script again with the flag --type=cleanup to remove the old chart from the legacy UI"
echo " 6. Run this script again with the flag --type cleanup to remove the old chart from the legacy UI"

0 comments on commit cc7a937

Please sign in to comment.