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

setting install_repo to false breaks module #233

Closed
bjwschaap opened this issue Aug 27, 2014 · 2 comments
Closed

setting install_repo to false breaks module #233

bjwschaap opened this issue Aug 27, 2014 · 2 comments

Comments

@bjwschaap
Copy link

Apply:
class { 'sensu': install_repo => false, }

Result:
Error: Could not retrieve catalog from remote server: Error 400 on SERVER: Invalid relationship: Package[sensu] { require => Yumrepo[sensu] }, because Yumrepo[sensu] doesn't seem to be in the catalog

Class sensu::package doesn't take install_repo switch into consideration when setting the package dependency.
@bjwschaap
Copy link
Author

Created pull request #234 for this issue.

jamtur01 added a commit that referenced this issue Aug 28, 2014
@jamtur01
Merge pull request #234 from bjwschaap/issue_233
dc74135 
Fix for issue #233: accomodating for install_repo, with specs
@jamtur01
Copy link
Contributor

Fixed by #234

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jamtur01 @bjwschaap