Skip to content

Commit

Permalink
Fix services registering. fixes #1
Browse files Browse the repository at this point in the history
  • Loading branch information
plribeiro3000 committed Dec 14, 2014
1 parent 2b69c01 commit 2a9d6c6
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions lib/fog/storm_on_demand.rb
Original file line number Diff line number Diff line change
Expand Up @@ -8,15 +8,15 @@ module StormOnDemand

extend Fog::Provider

service(:storm_on_demand, 'Compute')
service(:storm_on_demand, 'Network')
service(:storm_on_demand, 'Storage')
service(:storm_on_demand, 'DNS')
service(:storm_on_demand, 'Billing')
service(:storm_on_demand, 'Monitoring')
service(:storm_on_demand, 'Support')
service(:compute, 'Compute')
service(:network, 'Network')
service(:storage, 'Storage')
service(:dns, 'DNS')
service(:billing, 'Billing')
service(:monitoring, 'Monitoring')
service(:support, 'Support')
service(:models, 'Account')
service(:storm_on_demand, 'VPN')
service(:vpn, 'VPN')
end

module Account
Expand Down

3 comments on commit 2a9d6c6

@tokengeek
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@plribeiro3000 - Only just got back to my computer.

I just noticed the "bin" is out of date.

https://github.com/fog/fog-storm_on_demand/blob/master/lib/fog/bin/storm_on_demand.rb#L5

It is linked to storm_on_demand and only maps to Compute.

I missed it the other day but it's the next error in the chain. Since, I'm working on removing that file altogether it's not a big issue. I suspect this provider has been non-functional for a while.

@plribeiro3000
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tokengeek I believe that most of the providers i extracted might have this problem since all of them wasn't touched in the last 8 months.

Is there something i could do to help you out?

@tokengeek
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@plribeiro3000 I think I need to get fog/fog-core#105 ready. Some deprecation warnings I added here highlighted this.

When that is in place I'm sure we'll see other points where you can help fix it.

Thanks for fixing.

Please sign in to comment.