Project

General

Profile

Bug #22647

Katello doesn't update sync notification URL on sync

Added by Stephen Benjamin over 3 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Repositories
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1545364

Summary 2: Cloned Satellites will still make API callbacks into the Source Satellite via the sync_complete API. This causes task errors due to lock contention and should be prevented at all costs.

this gets properly updated during the clone process.

The issue is that this change is never properly updated at the Mongo/Pulp level and the old hostname will persist in the event_notifier database entry, note the URL is not that of the new cloned Satellite:

  1. echo "db.event_listeners.find()" | mongo pulp_database

{ "_id" : ObjectId("20B3G0ppQ49r9xR00a"), "notifier_config" : { "url" : "https://production-satellite.example.com/katello/api/v2/repositories/sync_complete?token=R27Jp95A1drsJ7oER27Jp95A1drsJ7oE" }, "_ns" : "event_listeners", "event_types" : [ "repo.sync.finish" ], "id" : "3HkpvGemRtq2ISm83411", "notifier_type_id" : "http" }

  • Second Issue: The cloned Satellite will often have 2 event_listener rows in the database. This causes multiple API calls to be made to either the production source Satellite or if the hostname was correct, the clone itself.

This can be found by running:

  1. echo "db.event_listeners.find().count()" | mongo pulp_database
    MongoDB shell version: 2.6.11
    connecting to: pulp_database
    2

If the count is > 1, we have an error condition where there are too many event_listeners

Associated revisions

Revision 75c44228 (diff)
Added by Stephen Benjamin over 3 years ago

fixes #22647 - ensure sync notification on repo sync

History

#1 Updated by The Foreman Bot over 3 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/7209 added

#2 Updated by The Foreman Bot over 3 years ago

  • Pull request https://github.com/Katello/katello/pull/7210 added

#3 Updated by Justin Sherrill over 3 years ago

  • Legacy Backlogs Release (now unused) set to 338
  • Category set to Repositories
  • Subject changed from Katello doesn't update sync notification URL on sync to Katello doesn't update sync notification URL on sync
  • Pull request deleted (https://github.com/Katello/katello/pull/7209)

#4 Updated by Stephen Benjamin over 3 years ago

  • % Done changed from 0 to 100
  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF