This is an archive of the discontinued Mercurial Phabricator instance.

Differential D6287

automation: wait longer for WinRM connection
ClosedPublic

Authored by indygreg on Apr 19 2019, 10:41 AM.

Details

Reviewers
martinvonz
Group Reviewers
hg-reviewers
Commits
rHGf30184484dd1: automation: wait longer for WinRM connection
Summary

I got a few timeouts waiting for only 120s for the WinRM connection
to become available. Increasing to 180s seems to fix. I guess
AWS isn't as consistent as I would like :(

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

indygreg created this revision.Apr 19 2019, 10:41 AM
martinvonz accepted this revision.May 15 2019, 11:43 AM
martinvonz added a subscriber: martinvonz.

I think I asked on #mercurial a while ago if anyone knew enough about Windows packaging to review this. No one said anything. So I can't see any reason *not* to queue this series.

This revision is now accepted and ready to land.May 15 2019, 11:43 AM
Closed by commit rHGf30184484dd1: automation: wait longer for WinRM connection (authored by indygreg). · Explain WhyMay 15 2019, 11:46 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 14866

contrib/automation/hgautomation/winrm.py

RunspacePool, RunspacePool,
) )
import requests.exceptions import requests.exceptions
logger = logging.getLogger(__name__) logger = logging.getLogger(__name__)
def wait_for_winrm(host, username, password, timeout=120, ssl=False): def wait_for_winrm(host, username, password, timeout=180, ssl=False):
"""Wait for the Windows Remoting (WinRM) service to become available. """Wait for the Windows Remoting (WinRM) service to become available.
Returns a ``psrpclient.Client`` instance. Returns a ``psrpclient.Client`` instance.
""" """
end_time = time.time() + timeout end_time = time.time() + timeout
while True: while True: