Issue #3952: Using the ErrataRelatedModelSaver can result in creating incomplete content. - RPM Support - Pulp

Agile board
Agile charts

Agile boards

Pulp 2 QE Agile Board
Pulp 2 QE Agile Board - Priority View
Pulp 3 QE Agile Board
Pulp 3 RPM Tests Agile Board
Sprint 112 Agile Board

Custom queries

[All Projects] All Test Trackers
2.21.4 - Next Bugfix Release
2.21.4 MODIFIED
2.21.5
3.14.3
Easy Fix
FIPS open
Functional Tests
Galaxy NG
Groomed
Issues for Triage
Katello Integration
Next Docs Day
POST/ASSIGNED - no Sprint - Pulp2
POST/ASSIGNED - no Sprint - Pulp3
Prioritized Bugs
Pulp 3 Docs issues
Pulp 3 Tests
Pulp RPM 3.x blockers
Pulp RPM Priority items
Pulp2 issues available for next release
SELinux related
Sprint 111
Sprint 112
Sprint Candidates (also groomed)
Un-Triaged Bugs
Verification Required

Actions

Send by e-mail Copy link

Issue #3952

closed

Using the ErrataRelatedModelSaver can result in creating incomplete content.

Added by jortel@redhat.com over 5 years ago. Updated over 4 years ago.

Status:

CLOSED - CURRENTRELEASE

Priority:

Normal

Assignee:

bmbouter

Sprint/Milestone:

Pulp 3 RPM MVP

Start date:

Due date:

Estimated time:

Severity:

2. Medium

Version:

Platform Release:

OS:

Triaged:

Yes

Groomed:

Sprint Candidate:

Tags:

Sprint:

Sprint 42

Quarter:

Description

The insert of errata and all of its related tables need to be committed atomically. A fatal exception or worker crash between stages can result in storing an incomplete Errata.

Related issues

Blocked by Pulp - Story #3953: ContentUnitSaver should support creating content with related models inside its transaction

CLOSED - CURRENTRELEASE

bmbouter

Actions

Issue # Delay: days Cancel

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by daviddavis over 5 years ago

Sprint/Milestone set to Pulp 3 RPM MVP

Actions

Copy link

Updated by daviddavis over 5 years ago

Blocked by Story #3953: ContentUnitSaver should support creating content with related models inside its transaction added

Actions

Copy link

Updated by bmbouter over 5 years ago

Tracker changed from Issue to Story
% Done set to 0

Converting to story per triage convo. Thanks for filing @jortel.

Actions

Copy link

Updated by bmbouter over 5 years ago

Tracker changed from Story to Issue
Severity set to 2. Medium
Triaged set to No

The story work is happening in core, so I'm moving this to be a bugfix, which for pulp_rpm users it is.

Actions

Copy link

Updated by bmbouter over 5 years ago

Status changed from NEW to ASSIGNED
Assignee set to bmbouter
Sprint set to Sprint 42

Adding to sprint as part of rpm sync work.

Actions

Copy link

Updated by bmbouter over 5 years ago

Status changed from ASSIGNED to POST

PR available at: https://github.com/pulp/pulp_rpm/pull/1178

Added by bmbouter over 5 years ago

Revision 22d250ff | View on GitHub

Use ContentUnitSaver _post_save hook

The ErrataRelatedModelSaver stage is instead repurposed as a subclass of the ContentUnitSaver stage. This is currently the best way for related objects to be saved within the same transaction.

Required PR: https://github.com/pulp/pulp/pull/3621

https://pulp.plan.io/issues/3952 closes #3952

asdf

Actions

Copy link