Project

Profile

Help

Task #4714

closed

Remove the 'validate' field from Master Remote object

Added by amacdona@redhat.com almost 5 years ago. Updated over 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Sprint:
Sprint 55
Quarter:

Description

The 'validate' field should be removed.

Our Master Remote includes fields that it shouldn't. In our original planning, the fields that were included seem to be "all fields that many plugins will probably need". This sends a mixed message to plugin authors, who may not be familiar with how or if these fields are used at all by pulpcore.

`validate. The pulpcore provided downloaders do some validation, but are unaware of the Remote.validate value. If this field is used, it must be used by the plugin. Therefore, the plugin should be the one to add it if they want it.

Actions #1

Updated by amacdona@redhat.com almost 5 years ago

To expand on this, some of these fields could be semi-implemented by pulpcore. For instance, policy should not be included on the remote, but policy choices should be defined by pulpcore and set on the plugin remote if they decide to have a policy field.

Actions #2

Updated by amacdona@redhat.com almost 5 years ago

  • Description updated (diff)
Actions #3

Updated by dkliban@redhat.com almost 5 years ago

  • Groomed changed from No to Yes
  • Sprint Candidate changed from No to Yes
Actions #4

Updated by amacdona@redhat.com almost 5 years ago

This should be considered a "backwards incompatible" change and should be included in a pulpcore-plugin release with with a version bump (not just an rc bump).

Actions #5

Updated by ipanova@redhat.com almost 5 years ago

  • Sprint set to Sprint 52
Actions #6

Updated by rchan almost 5 years ago

  • Sprint changed from Sprint 52 to Sprint 53
Actions #7

Updated by amacdona@redhat.com almost 5 years ago

  • Sprint changed from Sprint 53 to Sprint 54
Actions #8

Updated by ttereshc almost 5 years ago

  • Sprint changed from Sprint 54 to Sprint 55
Actions #9

Updated by bmbouter almost 5 years ago

  • Subject changed from Remove extra fields from Master Remote object to Remove the 'validate' field from Master Remote object
  • Description updated (diff)

Rewriting to remove mention of 'policy' which is being fixed with https://pulp.plan.io/issues/4990 instead.

Actions #10

Updated by amacdona@redhat.com almost 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to amacdona@redhat.com
Actions #11

Updated by amacdona@redhat.com almost 5 years ago

  • Status changed from ASSIGNED to POST

Added by amacdona@redhat.com almost 5 years ago

Revision 9e6e5d36 | View on GitHub

Remove unused field Remote.validate

This field was intended for plugin use, but so far plugins validate unconditionally. For now, we remove the field from core but plugins can add it on their own if they have a use for it.

https://pulp.plan.io/issues/4714 fixes #4714

Actions #12

Updated by amacdona@redhat.com almost 5 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #13

Updated by bmbouter over 4 years ago

  • Sprint/Milestone set to 3.0.0
Actions #14

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF