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.

Sign up for GitHub

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

Jump to bottom

Fog::Mock.reset raises NameError: uninitialized constant Fog::Rackspace::Storage #34

Open
smathy opened this issue Mar 9, 2020 · 1 comment · May be fixed by #35
Open

Fog::Mock.reset raises NameError: uninitialized constant Fog::Rackspace::Storage #34

smathy opened this issue Mar 9, 2020 · 1 comment · May be fixed by #35

Comments

@smathy
Copy link

smathy commented Mar 9, 2020 •
edited
Loading

This is basically the same problem as in fog/fog-openstack#59 (which I think should be closed, from fog/fog-openstack#65) ie. that in lib/fog/rackspace.rb you using/expecting Fog::Rackspace::Storage but in lib/fog/rackspace/storage.rb you have defined it the other way around: Fog::Storage::Rackspace.

My gut feel would be to correct the definition to match the filesystem, but that's not what they did in fog/fog-openstack#65 - so I suspect there's some other pieces of the "fog" world that expect the Fog::Storage::Rackspace hierarchy.

Happy to basically reproduce what was done in fog-openstack in a PR, just would like someone to say "Yep, do that" before I proceed.
smathy added a commit to amco/fog-rackspace that referenced this issue Mar 12, 2020
@smathy
Add legacy constant names explicitly, fixes fog#34
087aad3
@smathy smathy linked a pull request Mar 12, 2020 that will close this issue
Open
@smathy
Copy link
Author

smathy commented Mar 12, 2020

Ok, I ended up quite happy with my workaround, so there's a PR :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add legacy constant names explicitly amco/fog-rackspace
1 participant
@smathy