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

(maint) make apt testing more stable, cleanup #764

Merged
merged 1 commit into from
Jun 20, 2018

Conversation

tphoney
Copy link

@tphoney tphoney commented Jun 19, 2018

No description provided.

@tphoney tphoney added the bugfix label Jun 19, 2018
@tphoney tphoney force-pushed the stabilise_acceptance_tests branch 19 times, most recently from 02f5fb4 to c9b3f0f Compare June 20, 2018 10:27
@tphoney
Copy link
Author

tphoney commented Jun 20, 2018

This seems to fix most of our jenkins / ci issues, as well as clean up the code and tests.
There is an outstanding issue of debian 8 in travis CI failing, this only happened yesterday, proven by #765
I will tackle that issue separately

@tphoney tphoney force-pushed the stabilise_acceptance_tests branch 3 times, most recently from 0cc1ae8 to 04e8299 Compare June 20, 2018 14:55
@bastelfreak
Copy link
Collaborator

Hi @tphoney @HelenCampbell. Is there already a plan for a new release? We would like to see one since this PR fixes some of our issues with ubuntu 18.04: voxpupuli/puppet-zabbix#500 (comment)

@baurmatt
Copy link

baurmatt commented Jul 4, 2018

We would also appreciate a new releases which contains this fix! :)

@tphoney
Copy link
Author

tphoney commented Jul 5, 2018

Its on our back log https://tickets.puppetlabs.com/browse/MODULES-7337

@tphoney
Copy link
Author

tphoney commented Jul 5, 2018

:)

@@ -29,17 +28,9 @@
$_release = $release
}

# Some releases do not support https transport with default installation
Copy link
Collaborator

Choose a reason for hiding this comment

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

Removal of this is a regression. It's a feature we depend on.

Copy link
Author

Choose a reason for hiding this comment

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

https://tickets.puppetlabs.com/browse/MODULES-7540 has been created, this was my fault. I have a fix incoming, and will start a release soon after.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks, just ran into it but hadn't had time to create an issue myself. Luckily for me I noticed it on a CI job and not in production.

Copy link
Author

Choose a reason for hiding this comment

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

fix and tests in #775

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

Successfully merging this pull request may close these issues.

5 participants