jonpspri commented on issue #3407: WIP: No longer need to build 'whisk/*'
runtimes for builtins.
Agreed this is pretty massive. I'll see what I can do to break it up as it
leaves WIP; some of the changes can be added now and still test clean.
The reason for deleting the actionRuntime directories at the same time as
changing the manifest behavior was to ensure during that the new manifest
behavior didn't have some leftover dependency on the 'whisk/*' images and their
being pulled in './gradlew distDocker'. If we're nervous, we can put them
back, but as other pieces move, they become unneeded and confusing. I'd
definitely advise following up with a second PR not too soon after.
Thanks for pointing out the artifacts from those directories that are worth
saving. They can be moved into 'tools' and 'docs' with a little work. We'll
leave this PR and notes as WIP to guide the staged implementation as well as an
opportunity to run Travis over the whole enchilada. I'd suggest three PR's:
1. Additional configuration variables and ansible changes that can work
with the existing codebase.
2. The invoker changes to use the 'openwhisk/' prefix for runtimes and
accept directly-pulled images, including any additional ansible changes needed
3. Move the useful artifacts from the 'actionRuntimes' and delete the
actionRuntimes directory tree (and Travis changes to remove from the ignores
Once all that is done, I can move on to my original goal of converting from
'distDocker' to 'gradle-docker-plugin' to make multi-architecture builds more
accessible and less of a maintenance headache.
Sound good @rabbah @markusthoemmes ? (Meanwhile, back to figuring out why
Travis is failing, then I'll probably still want a PG to see what I broke in
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
Apache Git Services