MAAS

Merge ~ack/maas:1928623-storage-layout-fs-allocated into maas:master

  1. Git
  2. lp:~ack/maas
  3. 1928623-storage-layout-fs-allocated
  4. Merge into master
Proposed by Alberto Donato
Status: Work in progress
Proposed branch: ~ack/maas:1928623-storage-layout-fs-allocated
Merge into: maas:master
Diff against target: 216 lines (+57/-25)
4 files modified
src/maasserver/models/node.py (+1/-1)
src/maasserver/storage_layouts.py (+8/-3)
src/maasserver/tests/test_storage_layouts.py (+42/-14)
src/maasserver/utils/storage.py (+6/-7)
Reviewer Review Type Date Requested Status
Björn Tillenius Needs Information
MAAS Lander Needs Fixing
Review via email: mp+402834@code.staging.launchpad.net

Commit message

LP: #1928623 - set storage layout filesystems as acquired if the node is

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

UNIT TESTS
-b 1928623-storage-layout-fs-allocated lp:~ack/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas/job/branch-tester/10089/console
COMMIT: 82c16b015858494b028ef0bcf642861611a2b097

review: Needs Fixing
Reply
Revision history for this message
Björn Tillenius (bjornt) wrote :

I wonder if this is the right solution. What happens if you change the layout while the machine is Allocated, and then release it?

review: Needs Information
Reply
Revision history for this message
Alberto Donato (ack) wrote :

> I wonder if this is the right solution. What happens if you change the layout
> while the machine is Allocated, and then release it?

It goes back to the original layout, which is what is expected?

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