Skip to content

Increase Ansible default timeout#398

Merged
BillAnastasiadis merged 1 commit intoSUSE:mainfrom
BillAnastasiadis:ansible_timeour
Oct 23, 2025
Merged

Increase Ansible default timeout#398
BillAnastasiadis merged 1 commit intoSUSE:mainfrom
BillAnastasiadis:ansible_timeour

Conversation

@BillAnastasiadis
Copy link
Copy Markdown
Collaborator

@BillAnastasiadis BillAnastasiadis commented Oct 21, 2025

Increases the default timeout for Ansible globally, to avoid timeouts in ssh connections after 12 seconds (10s default and +2 seconds always added by ansible).

Reading material:

Copy link
Copy Markdown
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

Comment thread scripts/qesap/lib/cmds.py
# for avoiding the ansible ssh connection failure introduced by
# https://github.com/ansible/ansible/pull/78826 in "ansible-core 2.15.0"
ssh_share += ' --ssh-extra-args="-l cloudadmin -o UpdateHostKeys=yes -o StrictHostKeyChecking=accept-new"'
ansible_cmd_seq.append({"cmd": ssh_share})
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@mpagot we are adding env=original_env in the ansible-playbook commands, but not in this initial plain ansible command. Do we want to add it here too, or doesn't really matter? All the problems I can think of usually happen in the playbook runs..

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

no, it is fine like this

Copy link
Copy Markdown
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

@BillAnastasiadis BillAnastasiadis force-pushed the ansible_timeour branch 3 times, most recently from f797d2d to 339d5a8 Compare October 21, 2025 16:21
Copy link
Copy Markdown
Collaborator

@alvarocarvajald alvarocarvajald left a comment

Choose a reason for hiding this comment

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

LGTM

@BillAnastasiadis BillAnastasiadis changed the title WIP: Increase Ansible default timeout Increase Ansible default timeout Oct 23, 2025
@BillAnastasiadis BillAnastasiadis merged commit f84e980 into SUSE:main Oct 23, 2025
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.

4 participants