<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
no objection<br>
<br>
<div class="moz-cite-prefix">On 12/3/18 10:32 PM, Daniel Alley
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:CAK1D4mD+4id-V1qa5Z-mZZXJXMV-h=nAB+id753Mrsn5v+gA7w@mail.gmail.com">
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<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>
<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>
<fieldset class="mimeAttachmentHeader"></fieldset>
<pre class="moz-quote-pre" wrap="">_______________________________________________
Pulp-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Pulp-dev@redhat.com">Pulp-dev@redhat.com</a>
<a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/pulp-dev">https://www.redhat.com/mailman/listinfo/pulp-dev</a>
</pre>
</blockquote>
<br>
</body>
</html>