https://pulp.plan.io/https://pulp.plan.io/favicon.ico2016-01-08T16:25:26ZPulpPulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=77412016-01-08T16:25:26Zmhrivnakmhrivnak@redhat.com
<ul><li><strong>Priority</strong> changed from <i>High</i> to <i>Normal</i></li><li><strong>Severity</strong> changed from <i>3. High</i> to <i>2. Medium</i></li><li><strong>Platform Release</strong> deleted (<del><i>2.8.0</i></del>)</li><li><strong>Triaged</strong> changed from <i>No</i> to <i>Yes</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=77422016-01-08T16:25:47Zmhrivnakmhrivnak@redhat.com
<ul><li><strong>Tracker</strong> changed from <i>Issue</i> to <i>Task</i></li><li><strong>Groomed</strong> set to <i>Yes</i></li><li><strong>Sprint Candidate</strong> set to <i>Yes</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=112762016-05-04T13:21:16Zmhrivnakmhrivnak@redhat.com
<ul><li><strong>Sprint Candidate</strong> changed from <i>Yes</i> to <i>No</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=143122016-09-07T19:58:10Zamacdona@redhat.comaustin@redhat.com
<ul><li><strong>Sprint Candidate</strong> changed from <i>No</i> to <i>Yes</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=148202016-10-03T13:57:28Zmhrivnakmhrivnak@redhat.com
<ul><li><strong>Sprint/Milestone</strong> set to <i>27</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=152742016-10-24T14:28:50Zjortel@redhat.comjortel@redhat.com
<ul><li><strong>Sprint/Milestone</strong> changed from <i>27</i> to <i>28</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=156402016-11-09T19:40:03Zdaviddavis
<ul><li><strong>Assignee</strong> set to <i>daviddavis</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=157752016-11-14T18:00:28Zipanova@redhat.comipanova@redhat.com
<ul><li><strong>Assignee</strong> deleted (<del><i>daviddavis</i></del>)</li><li><strong>Sprint/Milestone</strong> changed from <i>28</i> to <i>29</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=158622016-11-17T13:10:25Zdaviddavis
<ul><li><strong>Status</strong> changed from <i>NEW</i> to <i>ASSIGNED</i></li><li><strong>Assignee</strong> set to <i>daviddavis</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=158922016-11-18T19:28:00Zdaviddavis
<ul></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=158932016-11-18T19:28:00Zdaviddavis
<ul></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=159212016-11-21T10:59:53Zdaviddavis
<ul></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=159612016-11-22T14:41:11Zdaviddavis
<ul></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=159622016-11-22T14:45:23Zdaviddavis
<ul></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=160282016-11-28T10:40:23Zdaviddavis
<ul></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=160292016-11-28T10:40:24Zdaviddavis
<ul></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=160312016-11-28T11:09:47Zdaviddavis
<ul></ul><p>What should I do about node calls that rely on code that's not specific to nodes? For example, when I call node sync on the parent, it looks like the server just creates a consumer update_content task[0]. I may be mistaken but it looks like this task isn't specific to nodes (at least for the parent—on the child, there is some node specific importer code).</p>
<p>I could either:</p>
<p>1. Add code to pulp core and check if the consumer is a node and then display a warning message<br>
2. Not worry about showing a deprecation warning when code not specific to nodes (but used by nodes) is called.</p>
<p>I'm kind of leaning toward option 2 as I had hoped to keep this change limited to nodes directory.</p>
<p>[0] <a href="https://git.io/v1IHM" class="external">https://git.io/v1IHM</a></p> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=160332016-11-28T12:53:55Zdaviddavis
<ul></ul><p>As mentioned in the bug description, it does indeed look like DeprecationWarning is silenced by default in 2.7. One option would be to run python with '-W all' or something similar in dev mode. I'm not sure exactly where/how to configure this flag though?</p>
<p>Also, we could update the logging code[0] by adding something like:</p>
<pre><code>warnings.simplefilter('always', DeprecationWarning)
</code></pre>
<p>I'm not sure how to tell though when we're in a dev setup. Maybe we could add a deprecation config variable?</p>
<p>[0] <a href="https://github.com/pulp/pulp/blob/91a1e28c9e7d3dee418d5c7680dbf25c3e7adc63/server/pulp/server/logs.py" class="external">https://github.com/pulp/pulp/blob/91a1e28c9e7d3dee418d5c7680dbf25c3e7adc63/server/pulp/server/logs.py</a></p> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=160642016-11-29T15:08:06Zbmbouterbmbouter@redhat.com
<ul></ul><p>I think we should not add the -W and instead use the <code>warnings.simplefilter</code> as you suggest. Adding that to the logging setup code would set that for all server side processes, but you'll also have to add it for <code>pulp-admin</code> and <code>pulp-consumer</code>. Those standalone, client-side binaries don't use the logging code.</p>
<p>In terms of the shared codepaths, I also prefer option 2 where we only add deprecation warnings to code that we know is node specific. Overall I believe for each server side processes (daemon processes) having at least 1 deprecation warning logged for each node operation (sync, publish) would be good. Also for interactive processes (pulp-admin, pulp-consumer) having at least 1 deprecation warning shown in the foreground for each node operation would be good.</p> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=161422016-12-01T15:19:50Zdaviddavis
<ul></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=161772016-12-02T08:42:13Zdaviddavis
<ul><li><strong>Blocks</strong> <i><a class="issue tracker-2 status-11 priority-6 priority-default closed" href="/issues/2460">Task #2460</a>: Write a blog post about nodes being deprecated and what functionality users can use to replace them</i> added</li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=161792016-12-02T08:42:48Zdaviddavis
<ul></ul><p>Went ahead and opened a separate issue to address creating a blog post after this issue is closed out:</p>
<p><a href="https://pulp.plan.io/issues/2460" class="external">https://pulp.plan.io/issues/2460</a></p> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=161802016-12-02T08:47:41Zdaviddavis
<ul></ul><p><a class="user active" href="https://pulp.plan.io/users/1">bmbouter</a>, you mentioned that we previously deprecated nodes in a release of pulp but looking through the docs/user-guide/release-notes directory of pulp, I am not seeing it. Do you remember the release?</p> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=161812016-12-02T09:08:22Zdaviddavis
<ul></ul><p>So turning on logging for deprecation warnings is spitting out tons of deprecation warnings in places we don't control like:</p>
<pre><code>Dec 02 09:06:35 dev.example.com pulp[18247]: py.warnings:WARNING: (18247-87552) /usr/lib/python2.7/site-packages/mongoengine/queryset/base.py:461: DeprecationWarning: update is deprecated. Use replace_one, update_one or update_many instead.
Dec 02 09:06:35 dev.example.com pulp[18247]: py.warnings:WARNING: (18247-87552) upsert=upsert, **write_concern)
</code></pre>
<p>This is from mongoengine calling update which pymongo deprecated: <a href="https://github.com/mongodb/mongo-python-driver/blob/master/pymongo/collection.py#L2482-L2483" class="external">https://github.com/mongodb/mongo-python-driver/blob/master/pymongo/collection.py#L2482-L2483</a></p>
<p>I'm not sure if we want or care about these messages? Another alternative is to define a new deprecation warning like NodeDeprecationWarning and just turn on those.</p> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=161912016-12-02T15:59:18Zbmbouterbmbouter@redhat.com
<ul></ul><p>@daviddavis, I thought we had deprecated nodes in an earlier release, but in searching for the details, I see that we have not declared it deprecated. It looks like the release note checklist item you are doing will be the official nodes deprecation. I've added another checklist item to add a big deprecation note on the nodes.rst page.</p> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=161922016-12-02T16:03:14Zbmbouterbmbouter@redhat.com
<ul></ul><p>Let's not enable all deprecation warnings. I really like the idea of declaring a NodeDeprecationWarning and just enabling that one.</p>
<p>Does it make any sense to write a blog post about nodes being deprecated with 2.12 and giving specific recommendations for how users can switch from nodes to a natural sync? I think that would be good. We could link to that blog post from the deprecation statement in the release notes and on nodes.rst pages. That should either become a checklist item or its own task. I think doing it all as 1 task would be good. Whatever you decide, would you be willing to track that work (as checklist item or its own task)?</p>
<p>FYI, here is a draft post on writing blog posts that @dkliban is working on: <a href="https://github.com/pulp/pulpproject.org/pull/29" class="external">https://github.com/pulp/pulpproject.org/pull/29</a></p>
<p>Thank you for doing all of this, it's a big job!</p> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=162162016-12-05T13:41:22Zdaviddavis
<ul></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=162312016-12-05T17:58:01Zmhrivnakmhrivnak@redhat.com
<ul><li><strong>Sprint/Milestone</strong> changed from <i>29</i> to <i>30</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=162502016-12-06T15:08:14Zdaviddavis
<ul></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=162732016-12-06T20:28:47Zdaviddavis
<ul></ul><p>Adding in deprecation warnings to the parent/child code is kind of tricky because there are often multiple places where I can add them. For example, when I bind a repo to a node, we could add a deprecation warning here:</p>
<p><a href="https://git.io/v10R2" class="external">https://git.io/v10R2</a></p>
<p>But it looks like this also gets called:</p>
<p><a href="https://git.io/v10RM" class="external">https://git.io/v10RM</a></p>
<p>Also it looks like there are places in the code that I am not sure how to reach. Here's an example:</p>
<p><a href="https://git.io/v10R9" class="external">https://git.io/v10R9</a></p> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=162742016-12-06T20:40:23Zbmbouterbmbouter@redhat.com
<ul></ul><p>I think the fact that [0] is called by [1] is fine because [1] is the publish repo inside of the pulp_node package so I expect [1] is only called for node calls.</p>
<p>Regarding [2], I believe the platform will call that when a orphan delete occurs. The platform would be running an orphan delete celery task through the node workflow. @jortel could confirm this I think.</p>
<p>[0]: <a href="https://git.io/v10R2" class="external">https://git.io/v10R2</a><br>
[1]: <a href="https://git.io/v10RM" class="external">https://git.io/v10RM</a><br>
[2]: <a href="https://git.io/v10R9" class="external">https://git.io/v10R9</a></p> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=163212016-12-08T14:03:42Zdaviddavis
<ul></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=163222016-12-08T14:03:43Zdaviddavis
<ul></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=163852016-12-09T21:59:51Zdaviddavis
<ul><li><strong>Status</strong> changed from <i>ASSIGNED</i> to <i>MODIFIED</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Applied in changeset <a class="changeset" title="Deprecate Pulp nodes Update the documentation and release notes to indicate that nodes are depre..." href="https://pulp.plan.io/projects/pulp/repository/pulp/revisions/490203cfe117568aefdca982c64971bc4ef4e814">pulp|490203cfe117568aefdca982c64971bc4ef4e814</a>.</p> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=167982017-01-09T15:40:37Zsemyerssean.myers@redhat.com
<ul><li><strong>Platform Release</strong> set to <i>2.12.0</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=170172017-01-17T23:42:11Zsemyerssean.myers@redhat.com
<ul><li><strong>Status</strong> changed from <i>MODIFIED</i> to <i>5</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=172742017-02-01T20:56:10Zsemyerssean.myers@redhat.com
<ul><li><strong>Status</strong> changed from <i>5</i> to <i>CLOSED - CURRENTRELEASE</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=256322018-03-08T19:18:45Zbmbouterbmbouter@redhat.com
<ul><li><strong>Sprint</strong> set to <i>Sprint 12</i></li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=256442018-03-08T19:19:04Zbmbouterbmbouter@redhat.com
<ul><li><strong>Sprint/Milestone</strong> deleted (<del><i>30</i></del>)</li></ul> Pulp - Task #1488: Deprecate nodeshttps://pulp.plan.io/issues/1488?journal_id=393412019-04-15T20:39:00Zbmbouterbmbouter@redhat.com
<ul><li><strong>Tags</strong> <i>Pulp 2</i> added</li></ul>