Puppet

Up until 0.24 we had “class proxy” defined multiple times in our manifests. Infact one class actually included the other proxy so it would have created:

class proxy {

 include proxy

}

Somehow this was working (just dumb luck I guess) until 0.24. Perhaps there should be a check to make sure that classes aren’t defined multiple times or at least a better error message here:

err: Could not apply complete catalog: Found cycles in the following relationships: Class[proxy] => Class[proxy]

I’m not sure if this error was from having multiple proxy classes defined or if it was because a class included itself.