Project

General

Profile

Refactor #13177

Extend ProxyStatus to have children classes

Added by Shlomi Zadok almost 4 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Shlomi Zadok
Category:
Smart Proxy
Target version:
1.11.0
Difficulty:
Triaged:
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/3099, https://github.com/theforeman/foreman/pull/3069
Fixed in Releases:
Found in Releases:

Description

Currently ProxyStatus is serving version (and tftp), but other plugins require same functionality so it should be extended to be by resource.
Also, in this way we will not delegate its methods to smart_proxy model

Related issues

Has duplicate Foreman - Feature #13233: Changes in smart_proxy/show so Pulp status can be added by KatelloRejected2016-01-15

Associated revisions

Revision 37edf55c (diff)
Added by Shlomi Zadok almost 4 years ago

Fixes #13177, #13255 - Extend ProxyStatus to a base class, handle ProxyAPI errors

Revision 73b1a6b9 (diff)
Added by Dominic Cleal almost 4 years ago

refs #13177 - fix usage of WrappedException constructor

History

#1 Updated by The Foreman Bot almost 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3069 added

#2 Updated by Ondřej Pražák almost 4 years ago

  • Blocks Feature #13233: Changes in smart_proxy/show so Pulp status can be added by Katello added

#3 Updated by Ondřej Pražák almost 4 years ago

  • Blocks deleted (Feature #13233: Changes in smart_proxy/show so Pulp status can be added by Katello)

#4 Updated by Dominic Cleal almost 4 years ago

  • Has duplicate Feature #13233: Changes in smart_proxy/show so Pulp status can be added by Katello added

#5 Updated by Dominic Cleal almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 71

If this is intended as an extension point for plugins, please add documentation to http://projects.theforeman.org/projects/foreman/wiki/How_to_Create_a_Plugin. It should probably have also had an extension to the plugin registration.

#6 Updated by Shlomi Zadok almost 4 years ago

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

#7 Updated by The Foreman Bot almost 4 years ago

  • Pull request https://github.com/theforeman/foreman/pull/3099 added

#8 Updated by Dominic Cleal almost 4 years ago

  • Category set to Smart Proxy

Also available in: Atom PDF