Skip to content

Add back the Device Manager from x/ref/services/device/#78

Merged
razvanm merged 5 commits intovanadium:masterfrom
razvanm:device
Jan 7, 2019
Merged

Add back the Device Manager from x/ref/services/device/#78
razvanm merged 5 commits intovanadium:masterfrom
razvanm:device

Conversation

@razvanm
Copy link
Collaborator

@razvanm razvanm commented Dec 29, 2018

I did not include the x/services/device/dmrun because it has some
depencies on jiri that I need to look into. This also doesn't include
any of the other daemons that the device manager is using.

@cosnicolaou
Copy link
Contributor

Ok, so I refactored how factories/profiles work and to get these tests to pass you need to make a slight change to the tests. See, for example, ./x/ref/cmd/principal/principal_v23_test.go:

func init() {
// Allow v23.Init to be called multiple times.
library.AllowMultipleInitializations = true
}

@razvanm razvanm force-pushed the device branch 2 times, most recently from 4fcf69c to 03f11ad Compare January 4, 2019 22:58
@cosnicolaou cosnicolaou self-requested a review January 6, 2019 21:10
Copy link
Contributor

@cosnicolaou cosnicolaou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, let's add this back in and then figure out how to simplify it.

I did not include the x/services/device/dmrun because it has some
depencies on jiri that I need to look into. This also doesn't include
any of the other daemons that the device managed
This script works fine. I did not initially included because I was not
sure it's relevant.
Both of these are necessary to for starting an application using
deviced/device.
@razvanm
Copy link
Collaborator Author

razvanm commented Jan 7, 2019

I managed to use this to run a mounttabled and fortuned on my machine. I'll start a doc to track my experience.

Quick note: I also brought back the wiki page Device Management that talks about this.

@razvanm razvanm merged commit fa6bd9e into vanadium:master Jan 7, 2019
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

Successfully merging this pull request may close these issues.

2 participants