Project

Profile

Help

Issue #828

closed

pulp eats helpful validation errors from plugins and returns unhelpful PulpDataException

Added by amacdona@redhat.com over 7 years ago. Updated over 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
2.6.2
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

This issue was found after fixing a docker validation bug, pulp_docker now does the right thing and returns a helpful exception, but pulp eats it and loses all of the helpful data.

This issue has been partially fixed here:
https://github.com/pulp/pulp/pull/1487

Given Comment#9 here:
https://pulp.plan.io/issues/291

It seems like update goes through a different code path and still eats the exception.


Related issues

Blocks Docker Support - Issue #291: Pulp allows creation of docker repos that have invalid namesCLOSED - CURRENTRELEASEamacdona@redhat.comActions
Actions #1

Updated by amacdona@redhat.com over 7 years ago

  • Related to Issue #291: Pulp allows creation of docker repos that have invalid names added
Actions #3

Updated by amacdona@redhat.com over 7 years ago

  • Status changed from ASSIGNED to POST

Added by Austin Macdonald over 7 years ago

Revision 17d40165

Allow pulp CLI to print sub_errors

closes #828

Added by Austin Macdonald over 7 years ago

Revision 17d40165

Allow pulp CLI to print sub_errors

closes #828

Actions #4

Updated by Anonymous over 7 years ago

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

Updated by dkliban@redhat.com over 7 years ago

  • Project changed from Docker Support to Pulp
  • Platform Release set to 2.6.2
Actions #6

Updated by dkliban@redhat.com over 7 years ago

  • Related to deleted (Issue #291: Pulp allows creation of docker repos that have invalid names)
Actions #7

Updated by dkliban@redhat.com over 7 years ago

  • Blocks Issue #291: Pulp allows creation of docker repos that have invalid names added
Actions #8

Updated by dkliban@redhat.com over 7 years ago

  • Status changed from MODIFIED to 5
Actions #9

Updated by igulina@redhat.com over 7 years ago

  • Status changed from 5 to 6

Verified. PulpDataException doesn't appear anymore.

>> rpm -qa pulp-server
pulp-server-2.6.2-0.2.beta.el7.noarch

>> pulp-admin docker repo update --repo-id=busybox --repo-registry-id=pulpdemo/Busybox
This command may be exited via ctrl+c without affecting the request.

[\]
Running...
Updating distributor: docker_web_distributor_name_cli

Task Failed

A validation error occurred.

The value specified for repo-registry-id: 'pulpdemo/Busybox' is invalid.
Registry id must contain only lower case letters, integers, hyphens, periods,
and may include a single slash.
Actions #10

Updated by dkliban@redhat.com about 7 years ago

  • Status changed from 6 to CLOSED - CURRENTRELEASE
Actions #12

Updated by bmbouter over 3 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF