<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I'm okay with doing a release of bundle-lib to release any features 
already there but I don't think you need a release in order to make 
follow on prs. the changes to bundle-lib could be copied into the vendor
 directory of the broker to pick up any changes you are needing.<br>
<br>
Then prs can be filed separately for bundle-lib and broker.</blockquote><div><br></div><div>Would these changes not get wiped out if you ran dep ensure? Do we do this before a release or during a build? If running dep ensure would cause problems this approach seems a little scary particularly for reproducible builds as what is in vendor would not be reflected by what is in the lock file. I probably need to go check it out as I am not 100% on this.<br><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>-1 to putting the lib in mainline (it's already in vendor)</blockquote><div><br></div><div>Vendor is kinda special. The approach I am highlighting would remove it from vendor and put it under the main src code. In k8s this is /staging/src/<a href="http://k8s.io/client-go">k8s.io/client-go</a> as an example. Then something automated would sync the latest for that directory to a separate repo for general consumption. <br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 9, 2018 at 2:46 PM, jesus m. rodriguez <span dir="ltr"><<a href="mailto:jesusr@redhat.com" target="_blank">jesusr@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'm okay with doing a release of bundle-lib to release any features already there but I don't think you need a release in order to make follow on prs. the changes to bundle-lib could be copied into the vendor directory of the broker to pick up any changes you are needing.<br>
<br>
Then prs can be filed separately for bundle-lib and broker.<br>
<br>
-1 to putting the lib in mainline (it's already in vendor)<br>
<br>
Sincerely,<br>
Jesus<br>
<br>
<br>
⁣Sent from Blue ​<br>
<div><div class="h5"><br>
On Apr 9, 2018, 9:41 AM, at 9:41 AM, Craig Brookes <<a href="mailto:cbrookes@redhat.com">cbrookes@redhat.com</a>> wrote:<br>
>While looking at this bug<br>
><a href="https://bugzilla.redhat.com/show_bug.cgi?id=1564542" rel="noreferrer" target="_blank">https://bugzilla.redhat.com/<wbr>show_bug.cgi?id=1564542</a>, I realized that I<br>
>needed to create a PR against the bundle-lib before I could fix it in<br>
>the<br>
>broker.<br>
><br>
>However I will need a new release in order to pick it up. Wondering<br>
>what<br>
>folks think to doing a release of the lib?<br>
><br>
>It raises an interesting conundrum for contributors. As most of the<br>
>logic<br>
>is in this lib, it seems you will often need a release of the lib<br>
>before<br>
>you can create the follow on PR in the broker.<br>
><br>
>Something that the Kubernetes repo has done is have separate repos for<br>
>things like the client-go but have the actual source back in the main<br>
>repo<br>
>and sync to the external repo regularly. Wondering if there was any<br>
>interest in an approach like this.<br>
><br>
>Making it as simple as possible to expand the broker while still<br>
>providing<br>
>a valuable library seems like an ideal sweet spot to me.<br>
><br>
>--<br>
>Craig Brookes<br>
>RHMAP<br>
>@maleck13 Github<br>
><br>
><br>
</div></div>>-----------------------------<wbr>------------------------------<wbr>-------------<br>
><br>
>_____________________________<wbr>__________________<br>
>Ansible-service-broker mailing list<br>
><a href="mailto:Ansible-service-broker@redhat.com">Ansible-service-broker@<wbr>redhat.com</a><br>
><a href="https://www.redhat.com/mailman/listinfo/ansible-service-broker" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/ansible-<wbr>service-broker</a><br>
<br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Craig Brookes<div>RHMAP </div><div>@maleck13 Github</div></div></div>
</div>