-
Notifications
You must be signed in to change notification settings - Fork 461
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
Include apt in apt::backports #891
Conversation
@zivis Thanks for submitting the PR. Could you please take a look into the travis failures. Thank you. |
Codecov Report
@@ Coverage Diff @@
## master #891 +/- ##
=========================================
Coverage ? 73.07%
=========================================
Files ? 5
Lines ? 260
Branches ? 0
=========================================
Hits ? 190
Misses ? 70
Partials ? 0 Continue to review full report at Codecov.
|
@sheenaajay it works with the changes from #894 |
after #894 is merged, i can rebase this branch again. |
Otherwise there can be issues with undef values. like: Evaluation Error: Operator '[]' is not applicable to an Undef Value. (file: /etc/puppetlabs/code/environments/production/modules/apt/manifests/backports.pp, line: 73, column: 18) on node …
thanks for notifying, now travis builds without any errors :) |
we had issues with the order apt and apt::backports were evaluated.
Even if we have both in catalog.
like: