https://pulp.plan.io/https://pulp.plan.io/favicon.ico2018-02-23T18:01:41ZPulpPulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=247412018-02-23T18:01:41Zdaviddavis
<ul><li><strong>Tags</strong> <i>Pulp 3, Pulp 3 MVP</i> added</li></ul> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=247422018-02-23T18:28:56Zamacdona@redhat.comaustin@redhat.com
<ul></ul><p>Importers can be for more than just syncing repositories. For instance, plugins might want to use an importer for a rich copy, but that importer may not need a feed_url.</p> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=247432018-02-23T18:30:05Zdaviddavis
<ul><li><strong>Project</strong> changed from <i>Pulp</i> to <i>File Support</i></li></ul> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=247652018-02-26T10:17:44Zipanova@redhat.comipanova@redhat.com
<ul></ul><p>This could be solved by validating the presence of the field feed_url before triggering the sync.</p> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=247972018-02-27T14:57:53Zamacdona@redhat.comaustin@redhat.com
<ul></ul><p><a href="mailto:ipanova@redhat.com" class="email">ipanova@redhat.com</a> wrote:</p>
<blockquote>
<p>This could be solved by validating the presence of the field feed_url before triggering the sync.</p>
</blockquote>
<p>That is currently how it is done. <a href="https://github.com/pulp/pulp_file/blob/master/pulp_file/app/tasks/synchronizing.py#L52" class="external">https://github.com/pulp/pulp_file/blob/master/pulp_file/app/tasks/synchronizing.py#L52</a></p>
<p>Since this is no longer a Pulp issue, and is a pulp_file issue, the file plugin has no use for importers that don't have a feed_url, so I am +1 on accepting this.</p> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=247992018-02-27T15:44:03Zdalleydalley@redhat.com
<ul><li><strong>Triaged</strong> changed from <i>No</i> to <i>Yes</i></li></ul> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=248052018-02-27T16:15:19Zdaviddavis
<ul><li><strong>Subject</strong> changed from <i>Consider requiring feed_url on importers</i> to <i>Consider requiring feed_url on file importers</i></li></ul> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=272262018-04-04T18:59:01Zamacdona@redhat.comaustin@redhat.com
<ul><li><strong>Project</strong> changed from <i>File Support</i> to <i>Pulp</i></li><li><strong>Subject</strong> changed from <i>Consider requiring feed_url on file importers</i> to <i>Require `url` on Remotes</i></li><li><strong>Groomed</strong> changed from <i>No</i> to <i>Yes</i></li><li><strong>Sprint Candidate</strong> changed from <i>No</i> to <i>Yes</i></li></ul> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=272332018-04-04T20:03:01Zmilan
<ul></ul><p>maybe s/importers/remotes/ but still LGTM</p> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=272452018-04-04T21:06:20Zdaviddavis
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/27245/diff?detail_id=27837">diff</a>)</li><li><strong>Tags</strong> <i>Easy Fix</i> added</li></ul><p>Changed importers to remotes.</p> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=273392018-04-09T15:25:24Zamacdona@redhat.comaustin@redhat.com
<ul><li><strong>Sprint</strong> set to <i>Sprint 35</i></li></ul><p>Please file issues for pulp_python and pulp_file to remove the validation at sync task creation time.</p> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=273422018-04-09T16:41:59Zdalleydalley@redhat.com
<ul></ul><p>Wait, why was this converted back to a pulp issue from a pulp_file issue? This isn't something we're planning to do in core.</p>
<p>A pulp_python issue should still be filed but this should remain a pulp_file issue.</p> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=273432018-04-09T16:58:00Zamacdona@redhat.comaustin@redhat.com
<ul></ul><p>With the change from "importer" to "remote" we were thinking that a remote is nothing without a url.</p> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=273612018-04-10T16:20:59Zdaviddavis
<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 - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=273642018-04-10T17:20:51Zdaviddavis
<ul><li><strong>Status</strong> changed from <i>ASSIGNED</i> to <i>POST</i></li></ul><p><a href="https://github.com/pulp/pulp/pull/3426" class="external">https://github.com/pulp/pulp/pull/3426</a><br>
<a href="https://github.com/pulp/pulp_file/pull/65" class="external">https://github.com/pulp/pulp_file/pull/65</a><br>
<a href="https://github.com/pulp/pulp_python/pull/140" class="external">https://github.com/pulp/pulp_python/pull/140</a></p> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=273692018-04-10T18:29:20Zdaviddavis
<ul><li><strong>Status</strong> changed from <i>POST</i> to <i>MODIFIED</i></li></ul><p>Applied in changeset <a class="changeset" title="Require url on Remotes fixes #3395 https://pulp.plan.io/issues/3395" href="https://pulp.plan.io/projects/pulp/repository/pulp/revisions/82b2a80c7d49233969968950b0946bf9b65c17a3">pulp|82b2a80c7d49233969968950b0946bf9b65c17a3</a>.</p> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=276362018-04-19T15:12:31Zdkliban@redhat.com
<ul><li><strong>Sprint/Milestone</strong> set to <i>3.0.0</i></li></ul> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=426572019-04-26T20:36:34Zbmbouterbmbouter@redhat.com
<ul><li><strong>Tags</strong> deleted (<del><i>Pulp 3, Pulp 3 MVP</i></del>)</li></ul> Pulp - Issue #3395: Require `url` on Remoteshttps://pulp.plan.io/issues/3395?journal_id=510252019-12-13T17:30:03Zbmbouterbmbouter@redhat.com
<ul><li><strong>Status</strong> changed from <i>MODIFIED</i> to <i>CLOSED - CURRENTRELEASE</i></li></ul>