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

Should validate changeset 'source' is in config hierarchy #27

Closed
macgregor opened this issue Jan 11, 2016 · 2 comments
Closed

Should validate changeset 'source' is in config hierarchy #27

macgregor opened this issue Jan 11, 2016 · 2 comments

Comments

@macgregor
Copy link

You can specify a source in your changeset that is not in the hierarchy. I dont think it is trying to apply the changeset, but at the very least it is still rewriting your hieradata output in alphabetical order.

Should probably verify that the source is declared in your hierarchy and fail loudly.

@alechenninger
Copy link
Owner

Should probably verify that the source is declared in your hierarchy and fail loudly.

Agreed. Thank you!

@alechenninger
Copy link
Owner

Hrm can you give a more detailed outline of the scenario you experienced? Right now we're validating the target is in the hierarchy, and looking for any changes for anything in the hierarchy. No part of the process explicitly requires that all of the sources in a changeset exist in the hierarchy, only that the target exists in the hierarchy. There may or may not be changes for sources in the hierarchy which is intentional of course.

Are you trying to catch misconfigured hierarchy or misconfigured change or either?

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

No branches or pull requests

2 participants