Project

Profile

Help

Issue #2727

Updating importer with ssl_* options along with other options fails "Cannot update 'config.remove_missing' and 'config' at the same time"

Added by jsherril@redhat.com over 4 years ago. Updated over 2 years ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
3. High
Version:
2.12.1
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

Trying to update a repository's importer:

put "https://dev.example.com/pulp/api/v2/repositories/e7f93d88-a382-437f-a4d1-b48e6160c916/importers/yum_importer//"

and specifying some options such as remove_missing along with ssl_client_cert, ssl_client_key, and ssl_ca_cert will fail with:

OperationError: Could not save document (Cannot update 'config.remove_missing' and 'config' at the same time) 

To reproduce:

put "https://dev.example.com/pulp/api/v2/repositories/e7f93d88-a382-437f-a4d1-b48e6160c916/importers/yum_importer/"

with the body that I have attached. If you update the 3 ssl_* options separately, and then update everything else in a 2nd call, it works just fine.

This was seen in 2.12.1

request-750920.txt (125 KB) request-750920.txt jsherril@redhat.com, 04/25/2017 05:52 PM

History

#2 Updated by ttereshc over 4 years ago

I suspect this is something to do with simultaneous update of a MongoDB document itself and another MongoDB document embedded in it. Such combination is not allowed, AFAIK.
The solution for the similar issue before was to introduce additional save. Save document with embedded updates only and then save one more time the document as a whole.

#3 Updated by ttereshc over 4 years ago

  • Severity changed from 2. Medium to 3. High
  • Triaged changed from No to Yes

#5 Updated by mhrivnak over 4 years ago

I understand from a meeting just now that Katello has a workaround in place, so this is not a blocker. Please correct that in case it's wrong.

#6 Updated by bmbouter over 2 years ago

  • Status changed from NEW to CLOSED - WONTFIX

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.

#7 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF