Skip to content

Wait for sudo enablement in GCP#406

Open
BillAnastasiadis wants to merge 1 commit intoSUSE:mainfrom
BillAnastasiadis:sudogcp
Open

Wait for sudo enablement in GCP#406
BillAnastasiadis wants to merge 1 commit intoSUSE:mainfrom
BillAnastasiadis:sudogcp

Conversation

@BillAnastasiadis
Copy link
Collaborator

This pr adds a task to wait for paswordless sudo to be available before moving forward with the rest of the ansible execution. It also disables automatic fact-gathering, and moves it aftair the above task.

Copy link
Collaborator

@lpalovsky lpalovsky left a comment

Choose a reason for hiding this comment

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

LGTM

delay: 5
until: sudo_test.rc == 0
changed_when: false
failed_when: sudo_test.rc != 0
Copy link
Collaborator

Choose a reason for hiding this comment

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

Sound sequential more than declarative.
Can we restrict it to GCP only?
I do not feel comfortable to disable gathering only to manually re-enable it later on, what if anything went wrong?

Copy link
Collaborator

Choose a reason for hiding this comment

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

+1

@alvarocarvajald alvarocarvajald changed the title Wait for sudo enabement in GCP Wait for sudo enablement in GCP Nov 4, 2025
Copy link
Collaborator

@mpagot mpagot left a comment

Choose a reason for hiding this comment

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

LGTM but no more needed

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.

4 participants