<div dir="ltr"><div>PR Submitted: <a href="https://github.com/openshift/ansible-service-broker/pull/920">https://github.com/openshift/ansible-service-broker/pull/920</a> </div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, May 1, 2018 at 1:29 PM Ryan Hallisey <<a href="mailto:rhallise@redhat.com">rhallise@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, May 1, 2018 at 1:11 PM, Shawn Hurley <span dir="ltr"><<a href="mailto:shurley@redhat.com" target="_blank">shurley@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space">I think this is will simplify life as we make breaking changes. <div><div><div><span><br><blockquote type="cite"><div>On May 1, 2018, at 12:52 PM, David Zager <<a href="mailto:dzager@redhat.com" target="_blank">dzager@redhat.com</a>> wrote:</div><br class="m_-2263946391465536673m_-4068155674195796947Apple-interchange-newline"><div><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" target="_blank">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" target="_blank">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" target="_blank">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></div></div></blockquote></span></div></div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space"><div><div><div><span></span></div></div></div></div></blockquote></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>+1 for minimal amount of work done my run_latest_build.sh.   If we drop oc cluster up maybe we can combine the k8s and openshift setups?<br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space"><div><div><div><span><blockquote type="cite"><div><div dir="ltr"><div><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></blockquote></span><div>We already noticed that this can handle the issues that we have been having w/ the deployment template in catasb. Catasb uses master template but master tracks the canary image. If we also move catasb to using the APB then we can handle that situation with logic.</div><div>You can see an example of this with the latest CRD changes that Zager helped me with. </div><span><br><blockquote type="cite"><div><div dir="ltr"><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></div></blockquote></span><div>+1</div><blockquote type="cite"><div><span><div dir="ltr"><div><br></div><div>Your feedback is appreciated.</div><div><br></div><div>Regards,</div><div>Zager</div></div></span>
_______________________________________________<br>Ansible-service-broker mailing list<br><a href="mailto:Ansible-service-broker@redhat.com" target="_blank">Ansible-service-broker@redhat.com</a><br><a href="https://www.redhat.com/mailman/listinfo/ansible-service-broker" target="_blank">https://www.redhat.com/mailman/listinfo/ansible-service-broker</a><br></div></blockquote></div><br></div></div></div><br>_______________________________________________<br>
Ansible-service-broker mailing list<br>
<a href="mailto:Ansible-service-broker@redhat.com" target="_blank">Ansible-service-broker@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/ansible-service-broker" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/ansible-service-broker</a><br>
<br></blockquote></div></div></div></blockquote></div>