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

Multihub needs practical testing (but works in theory) #7

Open
jinnatar opened this issue Sep 24, 2017 · 2 comments
Open

Multihub needs practical testing (but works in theory) #7

jinnatar opened this issue Sep 24, 2017 · 2 comments

Comments

@jinnatar
Copy link
Owner

.. most likely it doesn't work. The library makes every attempt to be usable with multiple hubs on the same account simultaneously. However since I only have one hub most of this is untested and there's at least one known point where it won't work:

  • When using a hub locally we don't yet have any sane way to map hub id to hub ip on authentication. It's just a matter of hammering it down but it's tedious without being able to test.
@jinnatar jinnatar added this to the Multi Hub milestone Sep 24, 2017
@jinnatar
Copy link
Owner Author

jinnatar commented Mar 4, 2018

Much of the initial multihub issues are gone already earlier with the change to storing hubs per id instead of name. This solved the chicken-egg issues initially mentioned.

v0.2.12 now brings cloud remote operations into multihub compatibility, at least in theory. Since I only have one real hub testing has been theoretical. Now hubs can exist in a mixed state of remoteness though so it's much more practical to have multiple hubs.

@jinnatar jinnatar changed the title Multihub may or may not work Multihub needs practical testing (but works in theory) Mar 4, 2018
@villesavolainen
Copy link

Current code saved hub information little bit wrong way. There was sections for each hub but both contained name and ip of local hub which was little bit confusing and default one was always wrong one for me. I tried to improve little bit how authentication works with multiple hubs.

#25

Needs still some clean up and need to change code little bit in case there is only one hub which is not local, then it is stored as default one. Any comments?

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

No branches or pull requests

2 participants