Project

Profile

Help

Story #178

[RFE] Consider support for validating unit profiles

Added by jason.dobies@redhat.com almost 5 years ago. Updated 10 months ago.

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

0%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:

Description

++ This bug was initially created as a clone of Bugzilla Bug #947144 ++

Description of problem:

We may need to add something to the Profiler API that verifies a unit profile before Pulp saves it on the consumer. This originally came from another bug filed by Katello (that bug had two separate issues, this is tracking the validation portion):

In addition, no validation appears to be performed on the profile, so i could pass something like as the unit profile:

{
"_id": {
"$oid": "51360d559c60ed183200005e"
},
"_ns": "consumer_unit_profiles",
"consumer_id": "22eaa87c-76a1-4d5a-af34-f6a167e3bc44",
"content_type": "rpm",
"id": "51360d559c60ed183200005e",
"profile": [ {"version": "1.0", "name": "zsh"} ]
}

which is accepted perfectly fine (even though its invalid). However very bad things happen, as you cannot delete the consumer after you do this, or retrieve the unit profile.

For example, here's a partial traceback I get when trying to delete:

File "/usr/lib64/python2.6/site-packages/pymongo/database.py", line 663, in dereference
return self[dbref.collection].find_one({"_id": dbref.id})
File "/usr/lib64/python2.6/site-packages/pymongo/collection.py", line 520, in find_one
for result in self.find(spec_or_id, *args, **kwargs).limit(-1):
File "/usr/lib64/python2.6/site-packages/pymongo/cursor.py", line 703, in next
if len(self.__data) or self._refresh():
File "/usr/lib64/python2.6/site-packages/pymongo/cursor.py", line 666, in refresh
self.
_uuid_subtype))
File "/usr/lib64/python2.6/site-packages/pymongo/cursor.py", line 628, in _send_message
self.
_tz_aware)
File "/usr/lib64/python2.6/site-packages/pymongo/helpers.py", line 101, in _unpack_response
error_object["$err"])
pymongo.errors.OperationFailure: database error: invalid operator: $oid

History

#1 Updated by bmbouter 10 months ago

  • Status changed from NEW to CLOSED - WONTFIX

#2 Updated by bmbouter 10 months ago

Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.

#3 Updated by bmbouter 10 months ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF