RedHat-Middleware-Workshops / modernize-apps-katacoda

Katacoda scenarios for learning how to modernize existing apps using JBoss and OpenShift
39 stars 47 forks source link

using outdated istio #13

Closed jamesfalkner closed 6 years ago

jamesfalkner commented 6 years ago

@BenHall does this workshop and the middleware workshops on learn.openshift.com share the same base image? Wondering if its safe to update to istio 0.6.0 here.

BenHall commented 6 years ago

No, I kept them independent so they can move at difference paces. I can upgrade if you wish?

jamesfalkner commented 6 years ago

OK. The lab was run in north america yesterday, and particpants reported that istio 0.6.0 was somehow present as a result of the istio-install.sh script (which forces ISTIO_VERSION=0.4.0 in this lab scenario).. I am double-checking with the staff to find out if it really was the case.

jamesfalkner commented 6 years ago

OK I don't know what happened, but in any case it works for me with 0.4.0. I'll issue a PR which can be your notice that it's time to upgrade to 0.6.0 in the base image. standby for PR! After I resolve the ordering issue from the learn-openshift repo.

jamesfalkner commented 6 years ago

I went ahead and merged 411b496b9194978b4995379d4de2bde4952482eb so please upgrade to istio 0.6.0 in the base image. Thanks @BenHall !

jamesfalkner commented 6 years ago

@BenHall can this be closed? has the image been updated to have 0.6.0 pre-installed/pulled?

BenHall commented 6 years ago

Yes, Istio 0.6.0 has been deployed.