Project

Profile

Help

Story #8761

closed

Task #8740: [EPIC] Publication based plugins should use either `distribution.repository` or `distribution.publication` but not both

Add validation to Distribution serializer

Added by bmbouter over 3 years ago. Updated over 3 years ago.

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

100%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:

Description

BaseDistributionSerializer and RepositoryVersionSerializer had some validation preventing users from setting them up incorrectly which was not added to DistributionSerializer.

  • You should not be allowed to set Distribution.repository_version and Distribution.publication at the same time
  • You should not be allowed to set Distribution.repository and Distribution.repository_version at the same time
Actions #1

Updated by bmbouter over 3 years ago

  • Sprint/Milestone set to 3.14.0

This can't happen in 3.13 because pulp_file has a release which could set Distribution.repository and Distribution.publication at the same time. That release declares compatibility with <=3.13, so the earliest we can do this is 3.14.

Added by dalley over 3 years ago

Revision 1e262aa9 | View on GitHub

Bring back some of the Distribution validation around "repository"

closes: #8761 https://pulp.plan.io/issues/8761

Actions #2

Updated by dalley over 3 years ago

  • Status changed from NEW to MODIFIED
  • % Done changed from 0 to 100
Actions #3

Updated by dalley over 3 years ago

  • Subject changed from Add database constraints relating to Distributions to Add validation to Distribution serializer
  • Description updated (diff)
  • Sprint/Milestone changed from 3.14.0 to 3.13.0
Actions #4

Updated by pulpbot over 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF