Project

General

Profile

Actions

Refactor #38953

closed

Refactor manifest import so Katello no longer consumes Candlepin events

Added by Jeremy Lenz 7 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Category:
Subscriptions
Target version:
-

Description

After switching to an SCA-only world, the only Candlepin events consumed by Katello are pool.created and pool.deleted. This happens only during manifest import, since subscriptions are attached only to manifests and no longer attached to hosts or activation keys.

With upcoming SCA-aware manifests, you won't need to attach any subscriptions to your manifest either. So depending on the architecture choices, Katello may not need to consume any events after that work.

However, on https://github.com/candlepin/candlepin/pull/5307#discussion_r2605133424, there's some discussion about how Katello can remove event handling (and dependencies on Artemis) even independently of SCA-aware manifests. The proposed refactoring will look like this:

Before:
1. Katello starts a Candlepin manifest import task, and starts polling the task using a Foreman task
2. Katello consumes pool.created and pool.deleted events, and updates the Katello DB based on those events
3. Katello tracks when the Candlepin task completes, and the Foreman task completes

After:
1. Katello queries Candlepin via API for a list of existing pools
2. Katello starts a Candlepin manifest import task, and starts polling the task using a Foreman task
3. Katello tracks when the Candlepin task completes
4. Katello queries Candlepin via API for the new, resulting list of pools
5. Katello compares the two lists of Candlepin pools and uses the diff to update the Katello DB
6. The Foreman task completes.

Actions #1

Updated by Sam Bible 7 months ago

  • Triaged changed from No to Yes
  • Red Hat JIRA set to SAT-41277
Actions #2

Updated by The Foreman Bot 5 months ago

  • Status changed from New to Ready For Testing
  • Assignee set to Jonathon Turel
  • Pull request https://github.com/Katello/katello/pull/11645 added
Actions #3

Updated by The Foreman Bot 5 months ago

  • Pull request https://github.com/Katello/katello/pull/11647 added
Actions #4

Updated by The Foreman Bot 5 months ago

  • Pull request https://github.com/Katello/katello/pull/11652 added
Actions #5

Updated by The Foreman Bot 5 months ago

  • Pull request https://github.com/Katello/hammer-cli-katello/pull/1025 added
Actions #6

Updated by The Foreman Bot 4 months ago

  • Fixed in Releases Katello 4.21.0 added
Actions #7

Updated by Jonathon Turel 4 months ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF