<div dir="ltr"><div dir="ltr">I commented on the jwt one that I think it can be closed and why:  <a href="https://pulp.plan.io/issues/3248#note-6">https://pulp.plan.io/issues/3248#note-6</a><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 5, 2018 at 8:54 AM David Davis <<a href="mailto:daviddavis@redhat.com">daviddavis@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">Awesome, thanks! <br clear="all"><div><div dir="ltr" class="m_-7433154378387610223gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>David<br></div></div></div></div></div></div></div></div><br></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 5, 2018 at 8:44 AM Austin Macdonald <<a href="mailto:austin@redhat.com" target="_blank">austin@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 dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>For those with ambiguity, I added the RC blocker to force discussion and [acceptance | closing]. <br></div><div><br></div><div>Added RC Blocker:</div><div><ul><li>Add task names:<a href="https://pulp.plan.io/issues/2889" target="_blank"> https://pulp.plan.io/issues/2889</a></li><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669gmail-m_-7331540305823691868gmail-">Determine mutable fields: <a href="https://pulp.plan.io/issues/2635" target="_blank">https://pulp.plan.io/issues/2635</a></span></li><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669gmail-author-g-g19ilqfjh9s421z122zt m_-7433154378387610223m_3577236378744123312m_-5531564129566308669gmail-url">pulp-manager migrate order: <a href="https://pulp.plan.io/issues/3062" target="_blank">https://pulp.plan.io/issues/3062</a></span></li><ul class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669gmail-list-bullet2"><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669gmail-author-g-g19ilqfjh9s421z122zt m_-7433154378387610223m_3577236378744123312m_-5531564129566308669gmail-url">@david - <a href="https://pulp.plan.io/issues/4067#note-5" target="_blank">https://pulp.plan.io/issues/4067#note-5</a></span></li></ul><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669gmail-author-g-g19ilqfjh9s421z122zt">Asynchronous Distribution update/delete: </span><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669gmail-author-g-g19ilqfjh9s421z122zt m_-7433154378387610223m_3577236378744123312m_-5531564129566308669gmail-url"><a href="https://pulp.plan.io/issues/3044" target="_blank">https://pulp.plan.io/issues/3044</a></span></li><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669gmail-author-g-g19ilqfjh9s421z122zt m_-7433154378387610223m_3577236378744123312m_-5531564129566308669gmail-url">Distribution base_path model validation: <a href="https://pulp.plan.io/issues/3051" target="_blank">https://pulp.plan.io/issues/3051</a><br></span></li></ul></div><div>Closed: <br></div><div><ul><li>Viewable status endpoint w/out database running: <a href="https://pulp.plan.io/issues/2850" target="_blank">https://pulp.plan.io/issues/2850</a></li><li>Port Dependencies to Python3: <a href="https://pulp.plan.io/issues/2247" target="_blank">https://pulp.plan.io/issues/2247</a></li><li>Plugins can specify plugin API version: <a href="https://pulp.plan.io/issues/2656" target="_blank">https://pulp.plan.io/issues/2656</a></li></ul><div>No action:</div><div><ul><li>jwt: <a href="https://pulp.plan.io/issues/3248" target="_blank">https://pulp.plan.io/issues/3248</a></li><li>Add Publication.created (MODIFIED, david++): <a href="https://pulp.plan.io/issues/2989" target="_blank">https://pulp.plan.io/issues/2989</a></li></ul></div></div></div></div></div></div></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Dec 3, 2018 at 3:21 PM David Davis <<a href="mailto:daviddavis@redhat.com" target="_blank">daviddavis@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 dir="ltr"><div>Thanks for digging through older issues to find potential RC blockers.</div><div><br></div>2889 - +1 to making it an RC blocker<div>2635 - +1 here as well</div><div>2850 - I spent some time working on this and didn’t get far. I think we should just require the db to be running. I vote to close it out.</div><div>2989 - +1 to RC blocker</div><div>3044 - I guess we should revisit 3051 and decide on a design before the RC which will determine if the distribution endpoints need to be async?</div><div>2247 - Agreed on closing. Seems like we open issues on an as-needed basis</div><div>2656 - Seems like this is done or am I missing something?</div><div>3062 - Will checking in migrations to source control not solve this problem?</div><div>3248 - I haven’t heard anyone asking for jwt so I would say we don’t need it. We can just leave the issue open I think.</div><div><br></div><div><div dir="ltr" class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>David<br></div></div></div></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Dec 3, 2018 at 2:41 PM Austin Macdonald <<a href="mailto:austin@redhat.com" target="_blank">austin@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 dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><div>To be on the safe side, I'd like to highlight issues that *might* need to be RC blockers. Please reply directly onto the issue, I'll update this thread periodically if necessary.<br></div><div><br></div><div>REST API, backwards incompatible changes:<br></div><ul><li>Add Task Names:</li><ul><li> <a href="https://pulp.plan.io/issues/2889" target="_blank">https://pulp.plan.io/issues/2889</a></li><li>IMO: We should make this an RC Blocker, because this will be an additional requirement for every task in every plugin.</li></ul><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">Determine mutable fields</span></li><ul><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><a href="https://pulp.plan.io/issues/2635" target="_blank">https://pulp.plan.io/issues/2635</a></span></li><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">IMO: someone (or a group) should take this as assigned and audit the mutability of fields. If we find one that needs to change, it will be a backwards incompatible change to the REST API, so this should have the RC blocker tack.</span></span></li></ul><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">Status API without db connection</span></span></li><ul><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><a href="https://pulp.plan.io/issues/2850" target="_blank">https://pulp.plan.io/issues/2850</a></span></span></li><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">IMO: RC blocker or close. As it is the db connection field is not useful, and later removal would be backwards incompatible.</span></span></li></ul><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">Add new field, Publication.created</span></span></li><ul><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><a href="https://pulp.plan.io/issues/2989" target="_blank">https://pulp.plan.io/issues/2989</a></span></span></li><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">IMO: RC blocker or close, this would be a backwards incompatible change.</span></span></span></li></ul><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">Asynchronous Distribution update/delete</span></span></span></li><ul><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><a href="https://pulp.plan.io/issues/3044" target="_blank">https://pulp.plan.io/issues/3044</a></span></span></span></li><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">IMO: RC blocker or close, this would be a backwards incompatible change.</span></span></span></li></ul></ul><div>Packaging<br></div><ul><li>Port dependencies to Python 3</li><ul><li><a href="https://pulp.plan.io/issues/2247" target="_blank">https://pulp.plan.io/issues/2247</a></li><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">IMO:
 It seems like if this weren't done, we'd be having problems. Anyone 
mind if I close this one? If we do need to keep it open, should it be an
 RC blocker?</span></li></ul><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">Plugins can declare PluginAPI version</span></span></li><ul><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><a href="https://pulp.plan.io/issues/2656" target="_blank">https://pulp.plan.io/issues/2656</a></span></span></li><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">IMO: Are we happy with what we've got now? If we want to change it, now is the time.<br></span></span></li></ul></ul>Misc<div><ul><li>pulp-manager migrate order</li><ul><li><a href="https://pulp.plan.io/issues/3062" target="_blank">https://pulp.plan.io/issues/3062</a></li><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">IMO: RC Blocker. This is how users should migrate, so it should be correct before RC</span></li></ul><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">jwt</span></li><ul><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-"><a href="https://pulp.plan.io/issues/3248" target="_blank">https://pulp.plan.io/issues/3248</a></span></li><li><span class="m_-7433154378387610223m_3577236378744123312m_-5531564129566308669m_8023065864074573615m_8660533282841615791gmail-">This was removed from Beta (MVP) but do we need this for RC/GA?<br></span></li></ul></ul></div></div></div></div></div></div></div></div></div></div></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>
</blockquote></div>
</blockquote></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>