MAAS

Merge ~cgrabowski/maas:power_management_workflow into maas:master

  1. Git
  2. lp:~cgrabowski/maas
  3. power_management_workflow
  4. Merge into master
Proposed by Christian Grabowski
Status: Needs review
Proposed branch: ~cgrabowski/maas:power_management_workflow
Merge into: maas:master
Diff against target: 961 lines (+390/-131)
11 files modified
src/maasagent/cmd/maas-agent/main.go (+6/-9)
src/maasagent/internal/workflow/ephemeral.go (+2/-2)
src/maasagent/internal/workflow/power.go (+129/-45)
src/maasagent/internal/workflow/worker/pool.go (+45/-0)
src/maasserver/models/node.py (+44/-16)
src/maasserver/websockets/base.py (+3/-16)
src/maasserver/workflow/__init__.py (+90/-15)
src/maasserver/workflow/configure.py (+19/-2)
src/maasserver/workflow/power.py (+38/-23)
src/maasserver/workflow/worker/__init__.py (+12/-2)
src/maasserver/workflow/worker/worker.py (+2/-1)
Reviewer Review Type Date Requested Status
Anton Troyanov Approve
MAAS Lander Needs Fixing
Review via email: mp+453353@code.staging.launchpad.net

Commit message

feat: move execution of power workflow to Node model and add full power management workflow

To post a comment you must log in.
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b power_management_workflow lp:~cgrabowski/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/3750/console
COMMIT: 76e294e7a4e253b2719db91e74848c969bbe25bb

review: Needs Fixing
Reply
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b power_management_workflow lp:~cgrabowski/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/3753/console
COMMIT: a260e0b69ef154a42e8086db254ad8403fc1911f

review: Needs Fixing
Reply
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b power_management_workflow lp:~cgrabowski/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/3766/console
COMMIT: 547584c488ec5852afa7c02f13b9e61f357fc8c7

review: Needs Fixing
Reply
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b power_management_workflow lp:~cgrabowski/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/3767/console
COMMIT: 26590893ea23d9ec8520c9d6b17c85c928198b45

review: Needs Fixing
Reply
Revision history for this message
Anton Troyanov (troyanov) wrote :

Nice work!

Some nitpick comments and question about master pool registration.

review: Needs Information
Reply
Revision history for this message
Christian Grabowski (cgrabowski) :
Reply
Revision history for this message
Anton Troyanov (troyanov) :
review: Approve
Reply

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches