Thanks Tyson. Both look good. The alpine based image is > 2x smaller than the 
Ubuntu image. 

Didn’t you do the swagger update in another pr? Not sure why it’s in 3063.

-r

> On Dec 13, 2017, at 2:16 AM, Tyson Norris <[email protected]> wrote:
> 
> Hi -
> I have opened a few PRs recently related to deployment that I’d like feedback 
> on, or at least some indication if you think these changes will/won't impact 
> others:
> - https://github.com/apache/incubator-openwhisk/pull/3076 enhance 
> transformEnvironment.sh so that CONFIG_* values can dereference other env 
> vars, e.g. CONFIG_some_value=$SOME_OTHER_ENV
> - https://github.com/apache/incubator-openwhisk/pull/3063 use alpine for base 
> image on controller/invoker images (I just merged latest from master to 
> address conflicts)
> 
> Thanks
> Tyson

Reply via email to