<div dir="ltr"><div>Excellent and good catch on the duplicate already on sprint. Yes let's use 5321 as you suggest. I added to that issue that this was announced and scheduled for Sept 3rd.</div><div><br></div><div>If it's still at NEW on the 3rd, I have a reminder to make a PR and merge on that day. Otherwise anyone can make a PR ahead of time and merge on the 3rd.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 27, 2019 at 2:19 PM Austin Macdonald <<a href="mailto:austin@redhat.com">austin@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">This all sounds good to me. FYI, I closed <a href="https://pulp.plan.io/issues/5361" target="_blank">https://pulp.plan.io/issues/5361</a> as a dupe since <a href="https://pulp.plan.io/issues/5321" target="_blank">https://pulp.plan.io/issues/5321</a> has already been added to the sprint. Feel free to use 5361 and close 5321 if you prefer.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 27, 2019 at 2:16 PM Brian Bouterse <<a href="mailto:bmbouter@redhat.com" target="_blank">bmbouter@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>tl;dr if we make this change on Sept 3rd, the installer won't auto-create migrations anymore. For every change needing a migration, please commit one.<br></div><div><br></div><div># Background</div><div>We enabled the installer to "auto-create" migrations as a solution to the problem of them changing a crazy amount early on in Pulp3's development. Now the migrations are expected to be checked in and I believe they are for all plugins. This is only a breaking change for a plugin that is missing migrations.<br></div><div><br></div><div># The Problem<br></div><div><a href="https://pulp.plan.io/issues/5361" target="_blank">https://pulp.plan.io/issues/5361</a> This was originally reported by Katello as a P2; I added details to it.<br></div><div><br></div><div># Feedback</div><div>If there is something better that we should do please suggest it on the issue. If this is a concern or not going to work for you or our users please bring that up anywhere.<br></div><div><br></div><div># Timeline</div><div>If there are no blocking concerns, I plan to make this change on Tuesday Sept 3rd.</div><div><br></div><div>Thank you!</div><div>Brian</div><div><br></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>