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

Circular dependency in 0.2.3 #609

Closed
3flex opened this issue Apr 16, 2015 · 4 comments
Closed

Circular dependency in 0.2.3 #609

3flex opened this issue Apr 16, 2015 · 4 comments
Labels
bug Something isn't working needs-feedback Further information is requested

Comments

@3flex
Copy link
Contributor

3flex commented Apr 16, 2015

Copied from #244 (comment)

I am having the same problem (unrolled below for easier reading)...

err: Could not apply complete catalog: Found 1 dependency cycle:
( Anchor[nginx::end] 
  => Class[Nginx] 
  => Nginx::Resource::Vhost[vhost-searchhub] 
  => File[vhost-searchhub.conf symlink] 
  => Service[nginx]
  => Class[Nginx::Service] 
  => Anchor[nginx::end] )

Seems like nothing I do breaks this dependency cycle.

cc @shadone @bluecurio

@3flex
Copy link
Contributor Author

3flex commented Apr 16, 2015

@shadone thanks for investigating the versions affected, it helped to narrow down the problem @bluecurio is having. On comparing 0.2.2 and 0.2.3 I see two changes to dependencies, one that shouldn't have any impact (#557) and one that probably did (#553).

I'll need to investigate further and possibly untangle a few of the module's dependencies to simplify things and make it more robust.

@3flex
Copy link
Contributor Author

3flex commented Apr 17, 2015

@shadone @bluecurio could you both please add your manifests, the Puppet version you're using, and confirm the module version you're using?

I've done a bit of investigation but I can't see anything that would be causing this behaviour.

@bluecurio
Copy link

Cheers Matt,

Unfortunately I can't provide a working case of the bug because my team
ended up using a the passenger
https://github.com/phusion/passenger/wiki/Phusion-Passenger%3A-Node.js-tutorial package
(which provides nginx) instead of using your module. I ultimately needed an
environment for my nodejs app to run and Passenger was easier than
rolling-my-own. Hope you understand. I've never used anchors (in Puppet),
so your guess is as good as mine as to what's going on in the nginx
module...

--Daniel

-Daniel

On Fri, Apr 17, 2015 at 12:21 AM, Matthew Haughton <notifications@github.com

wrote:

@shadone https://github.com/shadone @bluecurio
https://github.com/bluecurio could you both please add your manifests,
the Puppet version you're using, and confirm the module version you're
using?

I've done a bit of investigation but I can't see anything that would be
causing this behaviour.


Reply to this email directly or view it on GitHub
#609 (comment)
.

@3flex 3flex added the needs-feedback Further information is requested label Apr 17, 2015
@3flex
Copy link
Contributor Author

3flex commented May 6, 2015

I'm closing this for now. @shadone please comment if you have any information to add and I'll reopen. Thanks!

@3flex 3flex closed this as completed May 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs-feedback Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants