11:37 a.m.
I agree with option 3 considering that the idea here is to solve this
"Kimchi feature being coded in WoK" problem and this is easier
than moving everything to WoK.
Daniel
On 07/04/2016 12:27 PM, Aline Manera wrote:
Hi all,
Kimchi provides an feature called federation to discover other Wok
servers running in the same network.
This feature is not enabled by default. To enable it, the user needs
to follow the instructions in
https://github.com/kimchi-project/kimchi/blob/master/docs/README-federati...
The problem is: this feature is exposed by the user as a drop down
menu with a list of servers found. But it is displayed in the top
header, ie, on Wok header.
That way, we have some code on Wok related to a Kimchi feature.
To fix it, we should change the way we expose that feature on UI
moving it as a content tab.
As it is not strictly related to virtualization I don't see it fits in
any current Kimchi tabs and I don't think it justifies a new tab.
But from the other hand, there is a plan to integrate the federation
feature with guest migration, so user don't need to input the server
details to do the migration - he/she only needs to select one from the
list. Said that, the federation feature is still required for Kimchi.
So I have 3 proposals:
1. Move federation feature to WOK
The peers will continue to be displayed as part of the top header but
all code (backend/API) will be moved to Wok.
2. Move federation feature to Ginger Base
The Dashboard tab will display a new section with the peers
information when federation is enabled.
3. Keep federation feature on Kimchi and make Kimchi extend the Ginger
Base UI (the same way Ginger s390x does with Ginger tabs)
The Dashboard tab will be extended by Kimchi to display a new section
with the peers information when federation is enabled.
*I prefer the option 3* as it solves the original issue (having UI
code related to a Kimchi feature on Wok) and do not affect any other
plugin or Wok behavior.
Once we get an agreement on it, I will proper involve Ginger community
if needed.
What are you thoughts on it?
Regards,
Aline Manera
_______________________________________________
Kimchi-devel mailing list
Kimchi-devel(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/kimchi-devel