Project

Profile

Help

Issue #1590

closed

Criteria object needs to be updated for use with pymongo 3

Added by dkliban@redhat.com over 6 years ago. Updated over 3 years ago.

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

Description

Criteria object[0] is used to pass query parameters to various mongoengine managers. Here[1] is just one example. Pymongo 3 stopped using the parameter 'fields' and replaced it with 'projection'. We need to update criteria object and it's use to make sure it is in compliance with pymongo 3.

[0] https://github.com/pulp/pulp/blob/master/server/pulp/server/db/model/criteria.py#L13
[1] https://github.com/pulp/pulp/blob/master/server/pulp/server/managers/consumer/query.py#L72


Related issues

Related to Pulp - Task #1578: Incoming searches should be translated by the modelCLOSED - WONTFIX

Actions

Added by rbarlow over 6 years ago

Revision 644b6075

Update Pulp to work with the pymongo 3.0.0 API.

https://pulp.plan.io/issues/1528

fixes #1528

Actions #1

Updated by semyers over 6 years ago

  • Related to Task #1578: Incoming searches should be translated by the model added
Actions #2

Updated by jortel@redhat.com over 6 years ago

  • Triaged changed from No to Yes
Actions #4

Updated by pcreech over 6 years ago

  • Status changed from NEW to MODIFIED
  • Assignee set to pcreech

This appears to be resolved with the pull request [0].
All underlying queries have been converted to using 'projection' instead of 'fields'

The criteria object's fields have been left alone to maintain api compatibility

https://github.com/pulp/pulp/pull/2338

Actions #5

Updated by dkliban@redhat.com over 6 years ago

  • Status changed from MODIFIED to 5
Actions #6

Updated by dkliban@redhat.com over 6 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE
Actions #7

Updated by bmbouter over 3 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF