<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>