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

facts broken on all systems #184

Closed
stefanandres opened this issue Apr 1, 2015 · 3 comments
Closed

facts broken on all systems #184

stefanandres opened this issue Apr 1, 2015 · 3 comments

Comments

@stefanandres
Copy link
Contributor

Since c6ca80f the facts do not do what they are supposed to do:

The pkg resources always gets the information from the python-pip package.
Beyond that, on operatingsystems != Debian/Ubuntu every puppet run will throw an ugly error, because the packages have different names than python-pip.

Could not retrieve fact='virtualenv_version', resolution='<anonymous>': No package found with the specified name [<unspecified category>/python-pip]
Could not retrieve fact='system_python_version', resolution='<anonymous>': No package found with the specified name [<unspecified category>/python-pip]

Suggeststion::
We throw that entire pkg = Code away and only keep the weight 100 facts.

I can send a PR if wanted.

@stefanandres
Copy link
Contributor Author

I removed the broken code like this:
master...syseleven:fix-facts

@ghoneycutt
Copy link
Member

Merged in PR #210 and released in v1.9.5

Thank you!!

@poolski
Copy link

poolski commented Dec 18, 2015

This is still an issue in 1.10.0, unfortunately...

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

No branches or pull requests

3 participants