Skip to content

Fix velero command examples#259

Open
fxierh wants to merge 1 commit intovelero-io:mainfrom
fxierh:fix-velero-install-example
Open

Fix velero command examples#259
fxierh wants to merge 1 commit intovelero-io:mainfrom
fxierh:fix-velero-install-example

Conversation

@fxierh
Copy link
Copy Markdown

@fxierh fxierh commented Nov 5, 2024

The storageAccount key passed to the velero install --backup-location-config argument should contain the storage account name, not the storage account ID.

Signed-off-by: Feilian Xie <fxie@redhat.com>
@anshulahuja98
Copy link
Copy Markdown
Collaborator

I see 5 more occurrences of AZURE_STORAGE_ACCOUNT_ID in the doc, can you also change them to NAME

Comment thread README.md
--plugins velero/velero-plugin-for-microsoft-azure:v1.11.0 \
--bucket $BLOB_CONTAINER \
--secret-file ./credentials-velero \
--backup-location-config useAAD="true",resourceGroup=$AZURE_BACKUP_RESOURCE_GROUP,storageAccount=$AZURE_STORAGE_ACCOUNT_ID[,subscriptionId=$AZURE_BACKUP_SUBSCRIPTION_ID] \
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are few more places where it is still AZURE_STORAGE_ACCOUNT_ID
Around line 119 and other places. please change those also

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants