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

Fixes to test pipeline #540

Closed
wants to merge 3 commits into from
Closed

Conversation

agriffit79
Copy link

Pull Request (PR) description

Fix Docker tests by bumping Puppet version to 7
Re-add default Debian facts removed in #529
Remove erroneous require added in #499

This Pull Request (PR) fixes the following issues

Fixes #536

@agriffit79
Copy link
Author

Don't know what the issue is with the CentOS7 pipeline. It works fine on my machine but we can't ssh to the container here

  Warning: Try 1 -- Host 127.0.0.1 unreachable: Errno::ECONNRESET - Connection reset by peer
  Warning: Trying again in 3 seconds

@zilchms
Copy link
Contributor

zilchms commented Feb 20, 2024

Hey @agriffit79 , thanks for your contribution! This should have been superseeded by the now fixed #520
If you still think the CI needs changes feel free to reopen this PR or open a new one :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unit tests don't run
2 participants