Project

Profile

Help

Issue #1590

Criteria object needs to be updated for use with pymongo 3

Added by dkliban@redhat.com almost 5 years ago. Updated almost 2 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

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 644b6075 View on GitHub
Added by rbarlow almost 5 years ago

Update Pulp to work with the pymongo 3.0.0 API.

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

fixes #1528

History

#1 Updated by semyers almost 5 years ago

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

#2 Updated by jortel@redhat.com almost 5 years ago

  • Triaged changed from No to Yes

#4 Updated by pcreech almost 5 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

#5 Updated by dkliban@redhat.com almost 5 years ago

  • Status changed from MODIFIED to 5

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

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#7 Updated by bmbouter almost 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF