Issue #828
closed
pulp eats helpful validation errors from plugins and returns unhelpful PulpDataException
Status:
CLOSED - CURRENTRELEASE
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 to Issue #291: Pulp allows creation of docker repos that have invalid names added
- Status changed from ASSIGNED to POST
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
- Project changed from Docker Support to Pulp
- Platform Release set to 2.6.2
- Related to deleted (Issue #291: Pulp allows creation of docker repos that have invalid names)
- Blocks Issue #291: Pulp allows creation of docker repos that have invalid names added
- Status changed from MODIFIED to 5
- 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.
- Status changed from 6 to CLOSED - CURRENTRELEASE
Also available in: Atom
PDF
Allow pulp CLI to print sub_errors
closes #828