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
Feature/add deployment field2metadata #544
Feature/add deployment field2metadata #544
Conversation
7f92797
to
ac52e8d
Compare
charmtools/charms.py
Outdated
'daemonset': bool, | ||
'min-version': str, | ||
} | ||
for k, v in deployment.items(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we also validate the allowed service types?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, changed the whole implementation and ensures this case now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's add a validation check for the allowed service types (and a test).
* Add 'deployment' field support in metadata.yaml * Add tests; * Change to use colander.MappingSchema to do validation on deployment; * Add tests for unsupported deployment.service, deployment.type;;
Description of change
Checklist
make test
?