feat(orchestrator): implement tag-based cleanup framework with NFS se… - #5142
Open
Nagachandan-P wants to merge 1 commit into
Open
feat(orchestrator): implement tag-based cleanup framework with NFS se…#5142Nagachandan-P wants to merge 1 commit into
Nagachandan-P wants to merge 1 commit into
Conversation
…rver cleanup Signed-off-by: Nagachandan-P <Nagachandan.p@dell.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of the Solution
Summary: Implements a comprehensive tag-based cleanup framework for the Omnia orchestrator with selective component cleanup, opt-in credential cleanup,
and NFS server data deletion capabilities. The framework ensures cleanup only runs with explicit tags, preventing accidental execution during normal orches
trator operations.
Changes
Cleanup Framework Core
cleanup_orchestrator.ymlwith component-level granularitywhencondition inorchestrator.ymlto gate cleanup execution, preventing runs without explicit cleanup tagsalltag to avoid Ansible conflictsnevertags to legacy cleanup playbooks (cleanup_openchami.yml,cleanup_openldap.yml) to prevent execution without tagsCredentials Cleanup Component
cleanup_credentialstagcredentials/tasks/cleanup.ymlandcredentials/vars/component_spec.ymlorchestrator_credentials.ymlto skip credential collection when cleanup tags are presentenabled: falseby defaultNFS Server Cleanup
cleanup_nfs_server: trueto Slurm and K8s component specs (default behavior)Tag Validation and Configuration
orchestrator_setup/vars/main.ymlto remove conflicting tag combinations for cleanupcleanup_credentialsto supported tags listdefault_cleanup.ymlwith component configurations and removed redundant settingscleanup_config.ymloverride fileDocumentation
cleanup/README.mdwith usage examples, available tags, and troubleshootingFiles Changed
src/orchestrator/playbooks/cleanup/README.mdsrc/orchestrator/playbooks/cleanup/cleanup_all.ymlsrc/orchestrator/playbooks/cleanup/cleanup_openchami.ymlsrc/orchestrator/playbooks/cleanup/cleanup_openldap.ymlsrc/orchestrator/playbooks/cleanup/cleanup_orchestrator.ymlsrc/orchestrator/playbooks/credentials/orchestrator_credentials.ymlsrc/orchestrator/playbooks/orchestrator.ymlsrc/orchestrator/roles/cleanup/components/credentials/tasks/cleanup.ymlsrc/orchestrator/roles/cleanup/components/credentials/vars/component_spec.ymlsrc/orchestrator/roles/cleanup/components/k8s/tasks/cleanup.ymlsrc/orchestrator/roles/cleanup/components/k8s/vars/component_spec.ymlsrc/orchestrator/roles/cleanup/components/slurm/tasks/cleanup.ymlsrc/orchestrator/roles/cleanup/components/slurm/vars/component_spec.ymlsrc/orchestrator/roles/cleanup/config/default_cleanup.ymlsrc/orchestrator/roles/orchestrator_setup/vars/main.ymlTesting
orchestrator.ymlis called without tags--tags cleanup(6 components, preserves credentials)--tags cleanup_credentials--tags cleanup,cleanup_credentials(all 7 components)Backward Compatibility
cleanup_all.ymlplaybook removed (replaced by new framework)--tags cleanup,--tags cleanup_credentials, or component-specific tags)