Skip to content
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

fix: 🐛 Correctif playbook de désinstallation. #98

Merged
merged 1 commit into from
Nov 21, 2023

Conversation

cedric-montagne
Copy link
Collaborator

Issues liées

Issues numéro:


Quel est le comportement actuel ?

Le playbook uninstall.yaml désinstalle bien la chaîne configurée avec la dsc par défaut, mais pas une autre chaîne qui serait configurée avec une autre dsc.
Si lors du lancement du playbook de désinstallation nous précisons une dsc alternative, avec l'extra variable prévue à cet effet, c'est la chaîne installée à l'aide de la dsc conf-dso qui est désinstallée au lieu de celle qui correspond à notre dsc alternative.
En résumé : le playbook ne tient pas compte d'une dsc spécifiée par extra variable pendant la désinstallation et il désinstalle alors la mauvaise chaîne qui est celle d'origine.

Quel est le nouveau comportement ?

Le playbook uninstall.yaml fonctionne maintenant comme attendu.
Il désinstalle toujours la chaîne configurée avec la dsc conf-dso si nous ne lui précisons pas d'extra variable.
Il désinstalle la chaîne configurée avec une dsc alternative si nous lui précisons cette dsc comme extra variable.

Cette PR introduit-elle un breaking change ?

Non.

Autres informations

Le correctif s'inspire directement de la façon de déclarer la dsc dans le role socle-config.

@cedric-montagne cedric-montagne merged commit 4a11f67 into develop Nov 21, 2023
2 checks passed
@cedric-montagne cedric-montagne deleted the fix/uninstall branch November 21, 2023 15:49
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