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.

Sign up for GitHub

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

Jump to bottom

Adds check to params.pp if lab-release is not installed #329

Merged
merged 1 commit into from Jul 29, 2014
Merged

Adds check to params.pp if lab-release is not installed #329

merged 1 commit into from Jul 29, 2014

Commits on Jul 10, 2014

  1. Adds check to params.pp if lab-release is not installed 

    Adds spec test

If lab-release is not installed, then the end user sees a confusing/ vague message
Error: Unsupported lsbdistid () at /modules/apt/manifests/params.pp:52
It is common for docker containers to not include this package by default

After fix, the user sees a friendlier message if lab-release is not installed
Error: Unable to determine lsbdistid, is lsb-release installed? at /modules/apt/manifests/params.pp:52
    @spuder
    spuder committed Jul 10, 2014
    Copy the full SHA
    64b8eb1 View commit details
    Browse the repository at this point in the history