Project

Profile

Help

Issue #828

pulp eats helpful validation errors from plugins and returns unhelpful PulpDataException

Added by amacdona@redhat.com over 6 years ago. Updated over 2 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 - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 17d40165 View on GitHub
Added by Austin Macdonald over 6 years ago

Allow pulp CLI to print sub_errors

closes #828

Revision 17d40165 View on GitHub
Added by Austin Macdonald over 6 years ago

Allow pulp CLI to print sub_errors

closes #828

History

#1 Updated by amacdona@redhat.com over 6 years ago

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

#3 Updated by amacdona@redhat.com over 6 years ago

  • Status changed from ASSIGNED to POST

#4 Updated by Anonymous over 6 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#5 Updated by dkliban@redhat.com over 6 years ago

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

#6 Updated by dkliban@redhat.com over 6 years ago

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

#7 Updated by dkliban@redhat.com over 6 years ago

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

#8 Updated by dkliban@redhat.com over 6 years ago

  • Status changed from MODIFIED to 5

#9 Updated by igulina@redhat.com over 6 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.

#10 Updated by dkliban@redhat.com about 6 years ago

  • Status changed from 6 to CLOSED - CURRENTRELEASE

#12 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF