<div dir="ltr"><div>tl;dr I'm +1 to making this switch.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Nov 4, 2019 at 3:51 PM David Davis <<a href="mailto:daviddavis@redhat.com">daviddavis@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>Currently in pulp, syncs always create repository versions regardless of whether or not any content changed. One of the tasks[0] for 3.0 GA is to document this behavior. However, I've heard several complaints about this from users so I wonder if it's worth reconsidering. <br></div></div></blockquote><div>I love making users happy, but the complaints didn't resonate as much with me because another user with a different subjective preferences could walk up and complain after we switch it. I try to listen for user complaints that come with objective claims of usability.<br></div><div><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><br></div><div>Here are some reasons against always creating repo versions:</div><div>- They were meant to serve as a historical record but this information is available by looking at the tasks api</div><div>- It creates additional, unnecessary versions and bumps the latest version number of the repo</div><div>- If we ever have a feature to retain only the latest X repo versions, it'll be less useful since some repo versions may not have any changes</div></div></blockquote><div>This last bullet I see an objective reason to make no-content-change repo versions not increment. Users concerned about their cron jobs not running can check the task records. Users get RepositoryVersions that always include change and are therefore more meaningful (perhaps that was Bin Li's objective claim). Also future users could get a repo-version retention option which would be difficult to create if we don't switch this.<br></div><div><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><br></div><div>Any thoughts? I'd like to get this on the sprint by Wednesday so it can be changed before the dev freeze date of Nov 12.</div></div></blockquote><div>+1 to making this change <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><br></div>[0] <a href="https://pulp.plan.io/issues/3308" target="_blank">https://pulp.plan.io/issues/3308</a><br clear="all"><div><div dir="ltr"><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></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></div>