Skip to content

ci: +questing -plucky and dry-run fix for localectl #2209

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

dbungert
Copy link
Member

Update CI to run on questing instead of plucky.

Doing so produced many failures in the answers integration tests, like so:

Exception ignored in: <function BaseSubprocessTransport.__del__ at 0x788856adf600>
Traceback (most recent call last):
  File "/usr/lib/python3.13/asyncio/base_subprocess.py", line 130, in __del__
    self.close()
  File "/usr/lib/python3.13/asyncio/base_subprocess.py", line 107, in close
    proto.pipe.close()
  File "/usr/lib/python3.13/asyncio/unix_events.py", line 603, in close
    self._close(None)
  File "/usr/lib/python3.13/asyncio/unix_events.py", line 627, in _close
    self._loop.call_soon(self._call_connection_lost, exc)
  File "/usr/lib/python3.13/asyncio/base_events.py", line 827, in call_soon
    self._check_closed()
  File "/usr/lib/python3.13/asyncio/base_events.py", line 550, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed

After modifying BaseSubprocessTransport to dump info on the subprocesses being cleaned up, I found that localectl is run in dry-run cases, so let's not do that.

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.

1 participant