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

Cogctl should not fall back to installing from warehouse when a config file exists but fails validation #1137

Closed
vanstee opened this issue Nov 9, 2016 · 0 comments · Fixed by operable/cogctl#108
Assignees
Milestone

Comments

@vanstee
Copy link
Member

vanstee commented Nov 9, 2016

If you try to install a bundle from a local config file like bundle install config.yaml and that config file has a validation error, cogctl will return ERROR: nil. The main problem is that if schema validation fails, even if the config exists, we still fallback to check warehouse for a bundle named config.yaml, let's fix that bug so that it fails at schema validation and properly returns the schema related errors.

@vanstee vanstee added this to the Cog 0.17 milestone Nov 9, 2016
@mpeck mpeck self-assigned this Nov 15, 2016
@mpeck mpeck added review and removed in-progress labels Nov 16, 2016
@mpeck mpeck removed the review label Nov 18, 2016
@kevsmith kevsmith modified the milestones: Cog 0.17, Cog 0.16.2 Nov 18, 2016
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

Successfully merging a pull request may close this issue.

3 participants