Project

General

Profile

Refactor #13177

Extend ProxyStatus to have children classes

Added by Shlomi Zadok over 3 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Shlomi Zadok
Category:
Smart proxies
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
Team Backlog:
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 over 3 years ago

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

Revision 73b1a6b9 (diff)
Added by Dominic Cleal over 3 years ago

refs #13177 - fix usage of WrappedException constructor

History

#1 Updated by The Foreman Bot over 3 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 over 3 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 over 3 years ago

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

#4 Updated by Dominic Cleal over 3 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 over 3 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 over 3 years ago

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

#7 Updated by The Foreman Bot over 3 years ago

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

#8 Updated by Dominic Cleal over 3 years ago

  • Category set to Smart proxies

Also available in: Atom PDF