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

Validate whether 'runtimes.manifest' is still needed by tests #3493

Closed
jonpspri opened this issue Mar 26, 2018 · 1 comment
Closed

Validate whether 'runtimes.manifest' is still needed by tests #3493

jonpspri opened this issue Mar 26, 2018 · 1 comment

Comments

@jonpspri
Copy link
Contributor

jonpspri commented Mar 26, 2018

The 'runtimes.manifest' variable in 'whisk.properties' and, therefore, 'whisk.properties.j2' may no longer be needed. If it is still needed, it is needed by tests run for 'incubator-openwhisk', and such tests should be rewritten to read from an agreed configuration file directly, instead.

As that is resolved, several layers of indirection between runtimes specification and resolution could be resolved.

It should also be investigated whether the runtimes configuration for invoker (and other components) should be provided as a JSON string or as a file containing configuration. The handling of the configuration in ansible is a major consideration for this conversation. @csantanapr is a major user of YAML runtimes specification in creating/testing the turntimes for example and his requirements for flexibility in deployment must be addressed.

See PR #3412 for additional discussion.

@rabbah
Copy link
Member

rabbah commented Dec 18, 2019

It is.

@rabbah rabbah closed this as completed Dec 18, 2019
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

No branches or pull requests

2 participants