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

build: install python-dbusmock via pip for a Docker container #18589

Merged
merged 2 commits into from Jun 3, 2019

Conversation

alexeykuzmin
Copy link
Contributor

@alexeykuzmin alexeykuzmin commented Jun 3, 2019

Description of Change

To prevent timeout errors with some other python-dbusmock versions.

Related to #18523.

  • Linux builds are timing out but passing tests. It turns out that dbusmock was hanging on dbus_stop. This PR changes the docker container to use a newer version of python-dbusmock which adds timeouts to dbus_stop.

Checklist

  • PR description included and stakeholders cc'd
  • npm test passes
  • PR title follows semantic commit guidelines

Release Notes

Notes: no-notes

To prevent timeout errors with some other python-dbusmock versions.
@codebytere
Copy link
Member

@alexeykuzmin what timeout errors were we seeing? do you have links to relevant tests that were dying, by chance?

@alexeykuzmin
Copy link
Contributor Author

@codebytere see the second bullet point in the #18523 linked in the description.

@alexeykuzmin
Copy link
Contributor Author

@jkleinsc I remember you saying that all containers for branches > 3 work fine,
but if we'll have to build a new container version, it will get python-dbusmock 0.17.0-1 with which I see that timeout issue.

Copy link
Contributor

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

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

@alexeykuzmin the change I was picking up for 3-1-x was the change to actually have a timeout on dbus_stop, which you can see was added in 0.17: https://github.com/martinpitt/python-dbusmock/blob/master/NEWS#L31

We still see the timeout message, but it works better than before where it would wait forever. That I can tell the timeout issue is still in 0.18.2 so I would recommend just changing the Docker file to pull in the latest so that once a new version is released if would pick up this commit:
martinpitt/python-dbusmock@f3b9d61

Dockerfile Outdated Show resolved Hide resolved
Co-Authored-By: John Kleinschmidt <jkleinsc@github.com>
@alexeykuzmin alexeykuzmin changed the title build: use python-dbusmock v0.18.1 on Linux build: install python-dbusmock via pip for a Docker container Jun 3, 2019
@jkleinsc jkleinsc merged commit 7f369c3 into master Jun 3, 2019
@release-clerk
Copy link

release-clerk bot commented Jun 3, 2019

No Release Notes

@trop
Copy link
Contributor

trop bot commented Jun 3, 2019

I have automatically backported this PR to "6-0-x", please check out #18597

@trop
Copy link
Contributor

trop bot commented Jun 3, 2019

I have automatically backported this PR to "4-2-x", please check out #18598

@trop trop bot added the in-flight/6-0-x label Jun 3, 2019
@trop
Copy link
Contributor

trop bot commented Jun 3, 2019

I have automatically backported this PR to "5-0-x", please check out #18599

@jkleinsc jkleinsc deleted the pip-install-python-dbusmock branch June 3, 2019 20:11
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.

None yet

3 participants