Refactor #10812
Remove require 'fog'
Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Plugin (foreman_docker)
Target version:
-
Difficulty:
Triaged:
No
Description
In #8516 (Foreman 1.9.0) we removed the require 'fog' from core and now only load parts of Fog that we need, e.g. 'fog/libvirt'.
foreman_docker has a require 'fog' call here, which can be removed to give a performance boost: https://github.com/theforeman/foreman-docker/blob/master/lib/foreman_docker/engine.rb#L3
Related issues
Associated revisions
fixes #10812 - remove require fog from plugin's engine
History
#1
Updated by Dominic Cleal almost 8 years ago
Updated by - Related to Refactor #8516: Remove require 'fog' and only load specific providers added
#2
Updated by The Foreman Bot almost 8 years ago
Updated by - Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman-docker/pull/119 added
- Pull request deleted (
)
#3
Updated by Shlomi Zadok over 7 years ago
Updated by - Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset foreman-docker|e6b93aecaa0231b64e50ddfd0ea3cb15019180e7.
fixes #10812 - remove require fog from plugin's engine