Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ConnectionResetError in _wait_for_sync() #2850

Closed
bowlofeggs opened this issue Dec 14, 2018 · 2 comments · Fixed by #2855
Closed

ConnectionResetError in _wait_for_sync() #2850

bowlofeggs opened this issue Dec 14, 2018 · 2 comments · Fixed by #2855
Labels
Composer Issues related to the composer Crash Issues related to an unhandled crash Critical We can't go on living in this sqalor, drop everything and fix it! python3 Issues related to porting Bodhi to Python 3 reliability Issues pertaining to Bodhi's reliability

Comments

@bowlofeggs
Copy link
Contributor

bowlofeggs commented Dec 14, 2018

Workaround

This exception is raised in the code that checks to see if the compose has been synchronized with the mirrors yet, so it is safe to simply resume the compose if this happens.

The problem

Today several of the Bodhi composes failed in _wait_for_sync():

Traceback (most recent call last):
  File "/usr/lib/python3.7/site-packages/bodhi/server/consumers/masher.py", line 344, in run
    self.work()
  File "/usr/lib/python3.7/site-packages/bodhi/server/consumers/masher.py", line 415, in work
    self._compose_updates()
  File "/usr/lib/python3.7/site-packages/bodhi/server/consumers/masher.py", line 964, in _compose_updates
    self._wait_for_sync()
  File "/usr/lib/python3.7/site-packages/bodhi/server/consumers/masher.py", line 1327, in _wait_for_sync
    masterrepomd = urllib2.urlopen(master_repomd_url)
  File "/usr/lib64/python3.7/urllib/request.py", line 222, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/lib64/python3.7/urllib/request.py", line 525, in open
    response = self._open(req, data)
  File "/usr/lib64/python3.7/urllib/request.py", line 543, in _open
    '_open', req)
  File "/usr/lib64/python3.7/urllib/request.py", line 503, in _call_chain
    result = func(*args)
  File "/usr/lib64/python3.7/urllib/request.py", line 1345, in http_open
    return self.do_open(http.client.HTTPConnection, req)
  File "/usr/lib64/python3.7/urllib/request.py", line 1320, in do_open
    r = h.getresponse()
  File "/usr/lib64/python3.7/http/client.py", line 1321, in getresponse
    response.begin()
  File "/usr/lib64/python3.7/http/client.py", line 296, in begin
    version, status, reason = self._read_status()
  File "/usr/lib64/python3.7/http/client.py", line 257, in _read_status
    line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1")
  File "/usr/lib64/python3.7/socket.py", line 589, in readinto
    return self._sock.recv_into(b)
ConnectionResetError: [Errno 104] Connection reset by peer

This seems like just another new Exception type that can be raised in Python 3 that wasn't raised in Python 2, similar to #2758.

@bowlofeggs bowlofeggs added Critical We can't go on living in this sqalor, drop everything and fix it! Composer Issues related to the composer Crash Issues related to an unhandled crash reliability Issues pertaining to Bodhi's reliability python3 Issues related to porting Bodhi to Python 3 labels Dec 14, 2018
sebwoj added a commit to sebwoj/bodhi that referenced this issue Dec 15, 2018
Fixes fedora-infra#2850

Signed-off-by: Sebastian Wojciechowski <s.wojciechowski89@gmail.com>
sebwoj added a commit to sebwoj/bodhi that referenced this issue Jan 15, 2019
Fixes fedora-infra#2850

Signed-off-by: Sebastian Wojciechowski <s.wojciechowski89@gmail.com>
bowlofeggs pushed a commit to sebwoj/bodhi that referenced this issue Feb 4, 2019
Fixes fedora-infra#2850

Signed-off-by: Sebastian Wojciechowski <s.wojciechowski89@gmail.com>
@mergify mergify bot closed this as completed in #2855 Feb 4, 2019
mergify bot pushed a commit that referenced this issue Feb 4, 2019
Fixes #2850

Signed-off-by: Sebastian Wojciechowski <s.wojciechowski89@gmail.com>
@bowlofeggs
Copy link
Contributor Author

A fix for this issue is planned for inclusion in the upcoming 4.0.0 release: #3221

@bowlofeggs
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Composer Issues related to the composer Crash Issues related to an unhandled crash Critical We can't go on living in this sqalor, drop everything and fix it! python3 Issues related to porting Bodhi to Python 3 reliability Issues pertaining to Bodhi's reliability
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant