/snapd

daemon: maintain a daemon-wide overlord instance #621

Merged
merged 1 commit into from Mar 10, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@zyga @chipaca @mvo5
@zyga
Contributor

zyga commented Mar 9, 2016

This patch begins the path of overlord-izing the daemon and making
particular API requests happen through interactions with particular
managers.

Signed-off-by: Zygmunt Krynicki zygmunt.krynicki@canonical.com
@zyga
daemon: maintain a daemon-wide overlord instance 
This patch begins the path of overlord-izing the daemon and making
particular API requests happen through interactions with particular
managers.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>

Some checks were not successful

1 failing and 1 successful checks
@snappy-m-o
Integration tests — 22 tests run, 0 skipped, 7 failed.
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
3c0a906
@chipaca
Member

chipaca commented Mar 9, 2016

This seems fine to me. It might be worth renaming the variable that's being called orverlord in a couple of places, but it's in a separate file and we'll be removing it soon anyway, so it's fine as is.
@mvo5
Collaborator

mvo5 commented Mar 10, 2016

👍

mvo5 added a commit that referenced this pull request Mar 10, 2016

@mvo5
Merge pull request #621 from zyga/iface-manager 
daemon: maintain a daemon-wide overlord instance

All checks have passed

2 successful checks
continuous-integration/travis-ci/push — The Travis CI build passed
Details
coverage/coveralls — First build on master at 71.918%
Details
a009557

@mvo5 mvo5 merged commit a009557 into snapcore:master Mar 10, 2016
1 of 2 checks passed

1 of 2 checks passed

Integration tests 22 tests run, 0 skipped, 7 failed.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment