Project

General

Profile

Actions
Copy link

Bug #24671

closed

[Sync-plans] Remove dead code

Added by Samir Jha over 6 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Justin Sherrill
Category:
Sync Plans
Target version:
Katello 3.9.0
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
https://github.com/Katello/katello/pull/7646
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

ACK from me, we need a follow up PR that removes:

https://github.com/Katello/katello/blob/master/app/controllers/katello/api/v2/repositories_controller.rb#L323-L340
https://github.com/Katello/katello/blob/master/app/lib/actions/katello/repository/scheduled_sync.rb
https://github.com/Katello/katello/blob/master/app/models/katello/glue/pulp/repo.rb#L30-L57

and an upgrade rake task that removes any http notifiers: https://github.com/Katello/katello/blob/master/app/models/katello/glue/pulp/repo.rb#L32

Related issues 1 (0 open — 1 closed)

Blocks Katello - Tracker #23927: Convert sync plans to Foreman TasksClosedJustin Sherrill

Actions
Actions
Copy link
 #1

Updated by Samir Jha over 6 years ago

Actions
Copy link
 #2

Updated by The Foreman Bot over 6 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Justin Sherrill
  • Pull request https://github.com/Katello/katello/pull/7646 added
Actions
Copy link
 #3

Updated by Justin Sherrill about 6 years ago

  • Status changed from Ready For Testing to Closed
Actions
Copy link

Also available in: Atom PDF