-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Helm install is not completing but velero is installed #8059
Comments
This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days. If a Velero team member has requested log or more information, please provide the output of the shared commands. |
Using Helm install credentials:
secretContents:
cloud: ./credentials-velero.txt
backupStorageLocations:
- name: velerobackups
bucket: velerobackups # Make sure this matches your Azure Blob container name
provider: azure
config:
resourceGroup: rg-name
storageAccount: velerobackups
subscriptionId: ************
location: eastus
volumeSnapshotLocations:
- name: velerobackups
provider: azure
config:
resourceGroup: rg-name
storageAccount: velerobackups
subscriptionId: ************
location: eastus
Error: INSTALLATION FAILED: 2 errors occurred:
* BackupStorageLocation.velero.io "default" is invalid: [spec.objectStorage.bucket: Required value, spec.provider: Required value]
* VolumeSnapshotLocation.velero.io "default" is invalid: spec.provider: Required value |
@Lyndon-Li any ideas ? |
Still happens with latest Helm install
|
Hi, I had a similar problem and IIRC I needed to set defaultBackupStorageLocation and defaultVolumeSnapshotLocations with the name configured here. Maybe this helps. |
i'm getting the same error too trying today
helm values
|
i tried adding the config in as crds it worked correctly so to me it seems something is wrong with the helm chart
|
What steps did you take and what happened:
I recently updated my terraform helm install to upgrade to
7.1.0
version of Velero. After making modications for the new parameters, I can get it to install, but it will fail my terraform build regardless how I modify it. I then tried to reinstall Velero outside of terraform and experience a similar issue withhelm install
. Finally, after a semisuccessfulhelm install
, I created thebackupStorageLocation
andvolumeSnapshotLocation
as separate config yaml and applied them successfully withkubectl
. This successfully installs it albeit with manual intervention.Here's the configuration I used in Terraform to attempt the helm install:
Here's my the error I see from helm:
Here's my yaml file:
These are the two yaml's I successfully applied after unsuccessfully installing them using helm and notice how there is no difference:
What did you expect to happen:
I would have expected Velero to install like it had previously using terraform helm script. I would love to know if something is misconfigured in my terraform script above, but judging by the separate
helm install
I don't think that is the case unless the values I specified are wrong as well.The following information will help us better understand what's going on:
There is no issue with backups, only install.
If you are using earlier versions:
Please provide the output of the following commands (Pasting long output into a GitHub gist or other pastebin is fine.)
kubectl logs deployment/velero -n velero
The following commands are pertinent because the backups run fine after manual installation.
velero backup describe <backupname>
orkubectl get backup/<backupname> -n velero -o yaml
velero backup logs <backupname>
velero restore describe <restorename>
orkubectl get restore/<restorename> -n velero -o yaml
velero restore logs <restorename>
Anything else you would like to add:
Environment:
velero version
):velero client config get features
):kubectl version
):brew
/etc/os-release
): AWS Linux 2023Vote on this issue!
This is an invitation to the Velero community to vote on issues, you can see the project's top voted issues listed here.
Use the "reaction smiley face" up to the right of this comment to vote.
The text was updated successfully, but these errors were encountered: