Skip to content

skip approval update for omnia playbooks#4758

Merged
snarthan merged 1 commit into
dell:pub/q2_upgradefrom
abhishek-sa1:cleanup_skip
Jun 16, 2026
Merged

skip approval update for omnia playbooks#4758
snarthan merged 1 commit into
dell:pub/q2_upgradefrom
abhishek-sa1:cleanup_skip

Conversation

@abhishek-sa1

@abhishek-sa1 abhishek-sa1 commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

Variable: skip_approval

Usage: -e skip_approval=true — same variable used across the entire omnia codebase.

Pattern (matches existing upgrade.yml convention):

when: not (skip_approval | default(false) | bool)

Files Modified

OIM Cleanup (primary request)

File Prompts Behavior when skip_approval=true
utils/roles/oim_cleanup/pre_requisite/tasks/pre_requisite.yml 2 timed pauses Skips 30s/10s warning delays
utils/roles/oim_cleanup/oim_container_cleanup/tasks/cleanup_k8s.yml 1 confirmation (type YES) Auto-proceeds
utils/roles/oim_cleanup/oim_container_cleanup/tasks/cleanup_slurm.yml 1 confirmation (type YES) Auto-proceeds

Utils

File Prompts Behavior when skip_approval=true
utils/delete_migrated_pulp_rpm_repos.yml 1 confirmation skip_approval added as alias for force

Local Repo

File Prompts Behavior when skip_approval=true
local_repo/pulp_cleanup.yml 1 confirmation skip_approval added as alias for force

Upgrade/Rollback

File Prompts Behavior when skip_approval=true
upgrade/upgrade.yml 2 operator approval Already implemented
upgrade/roles/upgrade_telemetry/tasks/detect_victoria_state.yml 1 unhealthy pods Auto-proceeds

NOT Modified (by design)

  • Functional waits (seconds: N / minutes: N) in rollback/upgrade — these wait for services to stabilize, not for user approval
  • Credential prompts in credential_utility/ — these require actual data input; credentials should be provided via vault files

Example: Non-interactive OIM cleanup

ansible-playbook utils/oim_cleanup.yml -e skip_approval=true

Signed-off-by: Abhishek S A <abhishek.sa3@dell.com>
@abhishek-sa1 abhishek-sa1 marked this pull request as ready for review June 16, 2026 08:59
@snarthan snarthan merged commit e96490c into dell:pub/q2_upgrade Jun 16, 2026
4 checks passed
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.

3 participants