search

IBM / cloud-native-starter

Cloud Native Starter for Java/Jakarta EE based Microservices on Kubernetes and Istio
https://cloud-native-starter.mybluemix.net/
Apache License 2.0
518 stars 273 forks source link

Impact of "IBM Cloud Container Registry is Deprecating Container Builds" #51

Open thomassuedbroecker opened 4 years ago

thomassuedbroecker commented 4 years ago

We need to verify the impact of "IBM Cloud Container Registry is Deprecating Container Builds". This could have impact for our bash scripts and other automations we made for the workshops.

https://www.ibm.com/cloud/blog/announcements/ibm-cloud-container-registry-deprecating-container-builds?_ga=2.151918137.1436322187.1603873424-1354875151.1603726778

Harald-U commented 4 years ago

ibmcloud cr build is used in these scripts:

and also referenced in the documentation here:

Seems like we need to replace it with local docker build, docker tag, and docker push. This in turn requires local installation of Docker. Alternative: look into Code Engines build options.

thomassuedbroecker commented 4 years ago

@Harald-U Thanks for the list. Let's see how to convert the existing commands and scripts in the workshops.

Harald-U commented 4 years ago

@thomassuedbroecker I have updated the deploy scripts in iks-scripts, tested, works. Next I'll do the reactive scripts :-) Local Docker is a requirement anyways, ic cr build requires local docker installed.

Harald-U commented 4 years ago

There are scripts for reactive but the instructions (https://github.com/IBM/cloud-native-starter/blob/master/reactive/README.md#reactive-java-microservices) point to a toolchain: "Server-side Setup in IBM Cloud Kubernetes Service"(https://github.com/nheidloff/cloud-native-starter-reactive-toolchain). The toolchain definition will probably need rework once the deployment method for IKS changes. I wouldn't modify the scripts in reactive/iks-scripts, I would rather remove them. Your decision :-D

thomassuedbroecker commented 4 years ago

@Harald-U Thanks for your help :-)