Project

General

Profile

Actions

Refactor #8516

closed

Remove require 'fog' and only load specific providers

Added by Dominic Cleal over 9 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Performance
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

geemus suggested that we can selectively load providers from fog, cutting down the amount of code we load.

https://github.com/fog/fog/issues/3295#issuecomment-64659051

app/models/concerns/fog_extensions.rb mostly does this already, but does a require 'fog' at the top, as does ssh_provision.rb, and bundler will require the gem by default. These could probably all be removed.

We should ensure though that any plugins using Fog (xen, one, docker(?) etc) all have the appropriate requires in place to ensure their providers are accessible.


Related issues 5 (0 open5 closed)

Related to DigitalOcean - Refactor #10795: Remove require 'fog'Closed06/12/2015Actions
Related to Xen - Bug #10796: Remove require 'fog'Closed06/12/2015Actions
Related to OpenNebula - Bug #10797: Remove require 'fog'ClosedShlomi Zadok06/12/2015Actions
Related to Docker - Refactor #10812: Remove require 'fog'Closed06/14/2015Actions
Blocked by Foreman - Refactor #8641: Update fog version to 1.27.xClosedDaniel Lobato Garcia12/10/2014Actions
Actions #1

Updated by The Foreman Bot over 9 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2017 added
  • Pull request deleted ()
Actions #2

Updated by Dominic Cleal over 9 years ago

  • Assignee set to Tom Caspy
Actions #3

Updated by Dominic Cleal over 9 years ago

Actions #4

Updated by Dominic Cleal over 9 years ago

  • Translation missing: en.field_release set to 35

Please ensure foreman_digitalocean, one and xen get updates for this.

Actions #5

Updated by Tom Caspy over 9 years ago

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

Updated by Dominic Cleal about 9 years ago

Actions #7

Updated by Dominic Cleal about 9 years ago

  • Related to Bug #10796: Remove require 'fog' added
Actions #8

Updated by Dominic Cleal about 9 years ago

  • Related to Bug #10797: Remove require 'fog' added
Actions #9

Updated by Dominic Cleal about 9 years ago

Actions

Also available in: Atom PDF