<div dir="ltr">+1 to removing it <br></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Dec 4, 2018 at 8:34 AM Austin Macdonald <<a href="mailto:amacdona@redhat.com">amacdona@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="auto">+1, nice catch Daniel. </div><br><div class="gmail_quote"><div dir="ltr">On Tue, Dec 4, 2018, 7:14 AM David Davis <<a href="mailto:daviddavis@redhat.com" rel="noreferrer" 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">Big +1. This seems like something we could add later on at any time when we need it. Hopefully we can get Katello and our users involved in sussing out the requirements for how this field should work too.<br clear="all"><div><div dir="ltr" class="m_8391685313005884251m_5034688039171882167m_-1574823636173518570gmail_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 Mon, Dec 3, 2018 at 11:34 PM Daniel Alley <<a href="mailto:dalley@redhat.com" rel="noreferrer noreferrer" target="_blank">dalley@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><b>Background:</b></div><div><br></div><div>"Notes" are a generic key value store where data can be attached to repositories and content and publications and so forth.  The eventual plan is to use this to enable adding tags to those sorts of objects, which is important for Katello.</div><div><br></div><div>Most of the code for this is located in pulp/app/models/generic.py<br></div><div><br></div><div><b>Motivation:</b></div><div><br></div><div>"Notes" have been in Pulp 3 for a very very long time and are completely unchanged for the last 12 months (the git history doesn't go back further because the file was moved).  The data model behind it is extremely complex and while we have a few unit tests around it, we have no functional tests for it whatsoever, and (to my knowledge) we haven't been using/exercising this functionality manually in a meaningful way (if at all).  I could be wrong here, but I haven't seen it discussed  or any issues related to it filed in quite some time.<br></div><b></b><br><div><b>Proposal:</b><br></div><div><br></div><div>We should pull out all of the "notes" code (models/generic.py + the fields on the aforementioned models) until we've had a chance to properly evaluate our needs and whether the current design fits them.</div></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" rel="noreferrer noreferrer" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer noreferrer noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" rel="noreferrer noreferrer" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer noreferrer noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</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>