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

Failed to import plugin plugins.ginger.Ginger #39

Closed
potula-chandra opened this issue Nov 18, 2015 · 7 comments
Closed

Failed to import plugin plugins.ginger.Ginger #39

potula-chandra opened this issue Nov 18, 2015 · 7 comments
Assignees

Comments

@potula-chandra
Copy link
Member

Failed to import plugin plugins.ginger.Ginger

because model/utils.py is referring to
from wok.plugins.kimchi.disks import _get_dev_major_min
from wok.plugins.kimchi.disks import _get_dev_node_path

/usr/lib/python2.7/site-packages/wok/plugins/ginger/models/model.py(35)()
-> from users import UsersModel, UserModel
(Pdb) n
/usr/lib/python2.7/site-packages/wok/plugins/ginger/models/model.py(36)()
-> from swaps import SwapsModel, SwapModel
(Pdb) n
ImportError: 'No module named kimchi.disks'
/usr/lib/python2.7/site-packages/wok/plugins/ginger/models/model.py(36)()
-> from swaps import SwapsModel, SwapModel
(Pdb) c

@danielhb
Copy link
Contributor

Nice catch. Only happened because you've ran Ginger without Kimchi I believe.

We'll need to either move this to WoK or import it to Ginger. I guess the first option is better because it avoids duplicating code. Any comments, @alinefm ?

@danielhb
Copy link
Contributor

Moving to ginger-base is another option as well

@alinefm
Copy link
Member

alinefm commented Nov 18, 2015

Hi @danielhb! Hi @chandrureddy!

Wok should only contain what is needed to start up the web server. So it is not an option for this.
I suggest move the common code to ginger-base and adjust the imports in Kimchi and Ginger.

@danielhb
Copy link
Contributor

Thanks for the input @alinefm !

For anyone willing to solve this issue: be careful with adjusting Kimchi imports. They'll not appear only as 'wok.plugins.kimchi.disks' but also as 'import disks' in all the source files inside /plugins/kimchi/model/ dir.

I can work in this after fixing #37 .

@pkulkark
Copy link
Member

Hi @danielhb

I will be taking care of this in my next patch.

@danielhb
Copy link
Contributor

Thanks @pkulkark

@danielhb
Copy link
Contributor

danielhb commented Dec 3, 2015

This bug was referring specifically to errors in the the 'disks' module. These errors were fixed upstream a while ago.

@danielhb danielhb closed this as completed Dec 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants