Project

General

Profile

Actions

Feature #17145

open

Candlepin 2.0: Add in rollback support for failed manifest imports and refreshes

Added by David Davis over 7 years ago. Updated over 5 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

Before removing the old orchestration code, we had a way to rollback failed imports. See my PR here:

https://github.com/Katello/katello/pull/6414/files#diff-17c58a600ab07af9fd58f9c4ccb66be4L217

Dynflow didn't have this yet but there is a PR open to add support for rolling back actions:

https://github.com/Dynflow/dynflow/pull/194

Have the Owner::Import action rollback and call del_owner_import in Provider.


Related issues 1 (0 open1 closed)

Related to Katello - Feature #17027: CP 2.0: Phase 1: Enhance manifest import/refresh to support new candlepin async APIsRejectedJustin SherrillActions
Actions #1

Updated by David Davis over 7 years ago

  • Related to Feature #17027: CP 2.0: Phase 1: Enhance manifest import/refresh to support new candlepin async APIs added
Actions #2

Updated by Eric Helms over 7 years ago

  • translation missing: en.field_release set to 114
Actions #3

Updated by David Davis over 7 years ago

I started working on this but there's a snag. The existing code checks for a length of 1 for imports:

https://git.io/vXRfa

But here's what I see for imports. It seems to contain all attempts so it's rarely going to have a length of 1:

https://gist.github.com/daviddavis/4650f5de122c45d65f54031c34613c10

Here's my code which is a WIP. It reproduces the existing [broken] behavior in Dynflow:

https://github.com/daviddavis/katello/commit/fd4d08cb3fba6c121ac6e9d66db9b018c599487e

Actions #4

Updated by Brad Buckingham over 7 years ago

  • Target version changed from 133 to 140
Actions #5

Updated by Brad Buckingham over 7 years ago

  • Target version changed from 140 to 147
Actions #6

Updated by Brad Buckingham over 7 years ago

  • Target version changed from 147 to 151
Actions #7

Updated by David Davis over 7 years ago

  • Assignee deleted (David Davis)
Actions #8

Updated by Brad Buckingham about 7 years ago

  • Target version changed from 151 to 157
Actions #9

Updated by Brad Buckingham about 7 years ago

  • Target version changed from 157 to 160
Actions

Also available in: Atom PDF