Merge lp:~andreserl/debian-cd/maas into lp:~ubuntu-cdimage/debian-cd/ubun3
Proposed by
Andres Rodriguez
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 1935 | ||||
Proposed branch: | lp:~andreserl/debian-cd/maas | ||||
Merge into: | lp:~ubuntu-cdimage/debian-cd/ubun3 | ||||
Diff against target: |
37 lines (+18/-7) 1 file modified
tools/boot/xenial/boot-amd64 (+18/-7) |
||||
To merge this branch: | bzr merge lp:~andreserl/debian-cd/maas | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mathieu Trudel-Lapierre (community) | Approve | ||
VCS imports | Pending | ||
Review via email: mp+291285@code.launchpad.net |
This proposal supersedes a proposal from 2016-04-07.
Commit message
Add CD Options for Installing MAAS Region Controller and Rack Controller.
Description of the change
This branch adds the too options and replaces the old way of installing MAAS from the CD. This is dependent in the following branch been released in Ubuntu:
https:/
To post a comment you must log in.
It looks reasonable, but why add the menu items in two passes at writing to txt.cfg or grub.cfg, if you can do it in just one?
You also can't use the 'cloud' label twice, that label will be used to start the right menu option if all you have is a command-line, so it can be useful on, for instance, powerpc.
Please fix the labels and make it all a write in one go ;)