Skip to content
This repository has been archived by the owner on Nov 18, 2024. It is now read-only.

Velero install variable #51

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Velero install variable #51

wants to merge 3 commits into from

Conversation

ElieCaratge
Copy link
Contributor

No description provided.

@marcantoinegodde marcantoinegodde force-pushed the velero-install-variable branch from 5472b08 to dda6863 Compare May 24, 2023 15:09
@marcantoinegodde
Copy link
Contributor

@ElieCaratge Il faut aussi rajouter install_velero = "" dans les fichiers terraform.tfvars.template.

Copy link
Contributor

@Quentique Quentique left a comment

Choose a reason for hiding this comment

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

  • il faut supprimer le fichier velero-credentials qui ne sert à rien
  • toutes les variables en velero_ doivent avoir une valeur par défaut, car sinon, même en mettant install_velero = false, Terraform promptera pour avoir les variables en velero_. On peut soit mettre des valeurs par défaut (ce que je préconiserais), soit créer des variables locales (mais ça me semble compliqué pour un rien)

@Quentique Quentique added the enhancement New feature or request label May 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants