<div dir="ltr">Greetings,<div><br></div><div>This email is meant to get feedback on a fix to an issue was raised against the broker project regarding the <a href="https://github.com/openshift/ansible-service-broker/issues/912">run_latest_build.sh version check</a>. The issue, as Jesus had predicted <a href="https://github.com/openshift/ansible-service-broker/pull/891#discussion_r182487585">two weeks ago</a>, is that our version check is not very strong. While we could correct the version check to properly handle semantic versioning, I believe we should take this as an opportunity to bring the run_latest_build script and run_latest_k8s_build script into one.</div><div><br></div><div>You may ask "What would this look like?". Only the run_latest_build.sh would be left standing and it would:</div><div><ol><li>Expect a cluster (Kubernetes|OpenShift) to already exist with service-catalog installed</li><li>Create the broker namespace</li><li>Create a service account for the <a href="https://github.com/automationbroker/automation-broker-apb">automation-broker-apb</a></li><li>Create a automation-broker-apb clusterrolebinding "cluster-admin" tied to the automation-broker-apb service account</li><li>Run the automation-broker-apb as a pod</li><li>Delete the automation-broker-apb pod</li><li>Delete the automation-broker-apb service account</li><li>Delete the automation-broker-apb clusterrolebinding</li></ol><div>Since the automation-broker-apb will detect the cluster type and do the right thing (always, no bugs here). We can simply launch the pod and watch.</div></div><div><div><br></div><div>Removing the cluster startup from run_latest_build.sh was brought up during the ansible-service-broker IRC meeting which would make it virtually equivalent to the run_latest_k8s_build.sh. Having a single run_latest_build script is one step in a long process of simplifying our onboarding process with the benefit of reducing ways broker installs could fail.</div></div><div><br></div><div>Your feedback is appreciated.</div><div><br></div><div>Regards,</div><div>Zager</div></div>