Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Services registered incorrectly #1

Closed
tokengeek opened this issue Dec 14, 2014 · 4 comments
Closed

Services registered incorrectly #1

tokengeek opened this issue Dec 14, 2014 · 4 comments
Labels

Comments

@tokengeek
Copy link
Member

All services are registered incorrectly:
https://github.com/fog/fog-storm_on_demand/blob/master/lib/fog/storm_on_demand.rb#L11-L19

The correct form of the argument is:

service(:compute, "Compute")

This should then map :compute to Fog::StormOnDemand::Compute

At the moment it's mapping many storm_on_demand and a models services.

@tokengeek tokengeek added the bug label Dec 14, 2014
@plribeiro3000
Copy link
Member

I will work on it asap. Thank you for clarifying it @tokengeek!

@tokengeek
Copy link
Member Author

No worries. It's creating some strange things as I'm picking through the registration code!

[fog][DEPRECATION] Registered softlayer/network - Fog::Softlayer::Network
[fog][DEPRECATION] Registered softlayer/storage - Fog::Softlayer::Storage
[fog][DEPRECATION] Registered stormondemand/storm_on_demand - Fog::StormOnDemand::Compute
[fog][DEPRECATION] Registered stormondemand/storm_on_demand - Fog::StormOnDemand::Network
[fog][DEPRECATION] Registered stormondemand/storm_on_demand - Fog::StormOnDemand::Storage
[fog][DEPRECATION] Registered stormondemand/storm_on_demand - Fog::StormOnDemand::DNS
[fog][DEPRECATION] Registered stormondemand/storm_on_demand - Fog::StormOnDemand::Billing
[fog][DEPRECATION] Registered stormondemand/storm_on_demand - Fog::StormOnDemand::Monitoring
[fog][DEPRECATION] Registered stormondemand/storm_on_demand - Fog::StormOnDemand::Support
[fog][DEPRECATION] Registered stormondemand/models - Fog::StormOnDemand::Account
[fog][DEPRECATION] Registered stormondemand/storm_on_demand - Fog::StormOnDemand::VPN
[fog][DEPRECATION] Registered vcloud/compute - Fog::Vcloud::Compute
[fog][DEPRECATION] Registered vclouddirector/compute - Fog::VcloudDirector::Compute

@plribeiro3000
Copy link
Member

@tokengeek Can you take a look at 2a9d6c6 ?
If it is fine, i will do a new release.

@plribeiro3000
Copy link
Member

I just released it as 0.1.0. Let me know if you need any other changes. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants