Feature #17145
openCandlepin 2.0: Add in rollback support for failed manifest imports and refreshes
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.
Updated by David Davis over 8 years ago
I started working on this but there's a snag. The existing code checks for a length of 1 for imports:
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