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

Fix format string passed to pytest-multihost #1826

Closed
wants to merge 1 commit into from

Conversation

Tiboris
Copy link
Member

@Tiboris Tiboris commented Apr 17, 2018

Integration trust test suit failed with error trying to
start chronyd because of bad formating of passed string

See: https://pagure.io/python-pytest-multihost/issue/15
Resolves: https://pagure.io/freeipa/issue/7487

@Tiboris Tiboris added trivial needs review Pull Request is waiting for a review labels Apr 17, 2018
@Tiboris Tiboris requested a review from tiran April 17, 2018 12:17
@Tiboris Tiboris changed the title Fix format sting passed to pytest-multihost Fix format string passed to pytest-multihost Apr 17, 2018
Integration trust test suit failed with error trying to
start chronyd because of bad formating of passed string

See: https://pagure.io/python-pytest-multihost/issue/15
Resolves: https://pagure.io/freeipa/issue/7487
@gkaihorodova
Copy link
Contributor

Tested changes by running integration trust test - fix works

@tiran tiran added ack Pull Request approved, can be merged and removed needs review Pull Request is waiting for a review labels Apr 17, 2018
@Tiboris Tiboris added the re-run Trigger a new run of PR-CI label Apr 17, 2018
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Apr 17, 2018
@tiran tiran added the pushed Pull Request has already been pushed label Apr 17, 2018
@tiran
Copy link
Member

tiran commented Apr 17, 2018

master:

  • 692a993 Fix format string passed to pytest-multihost

@tiran tiran closed this Apr 17, 2018
@Tiboris Tiboris deleted the pytest-string-passing branch July 2, 2019 13:02
@Tiboris Tiboris restored the pytest-string-passing branch July 2, 2019 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed trivial
Projects
None yet
4 participants