Merge lp:~bac/lpsetup/mv-initialize-lxc into lp:lpsetup
Status: | Merged |
---|---|
Approved by: | Brad Crittenden |
Approved revision: | 57 |
Merged at revision: | 49 |
Proposed branch: | lp:~bac/lpsetup/mv-initialize-lxc |
Merge into: | lp:lpsetup |
Diff against target: | 0 lines |
To merge this branch: | bzr merge lp:~bac/lpsetup/mv-initialize-lxc |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Benji York (community) | code | Approve | |
Review via email: mp+114239@code.launchpad.net |
Commit message
Move most of initialize_lxc to init-host. Rename lxc-install to install-lxc. Add commands.rst. Updated the docstrings, and thus the help strings, for the subcommands. NOTE: finish-init-host and install-lxc are broken for now.
Description of the change
Move most of initialize_lxc to init-host.
Rename lxc-install to install-lxc.
Add commands.rst, which lays out what each subcommand does, its relationships to other commands, the environment in which it is to be run, and whether it performs actions as root or the user.
Updated the docstrings, and thus the help strings, for the subcommands. Also updated the module-level docstrings for each subcommand.
NOTE:
finish-init-host and install-lxc are broken as a result of these changes and need to be refactored. Their tests have been disabled.
Since I modified setup.cfg for nosetest options to ignore 'disabled_' tests, I decided to move with-doctest to that file from the pre-commit.sh. For some reason this is now triggering a warning about the version of distribute being used.
Since commands.rst is reST, we can reST-style definition lists for the docutils. sourceforge. net/docs/ ref/rst/ restructuredtex t.html# definition- lists).
"Terminology" and "install-lxc" sections (see
http://
The line length in commands.rst seems inconsistent. The longest line
looks to be 76 characters wide, which would suggest a 79 character max.
In that case there are several lines that seem to be prematurely
wrapped. I suggest 79 character lines to match our source files.
The steps list in lpsetup/ subcommands/ inithost. py (line 381 of the diff)
should end with a comma after the last item in the list and the closing
parenthesis on its own line in order to be consistent with the other
subcommands.
The docstring for call_initialize_lxc can't decide if "lxc" should be
capitalized (and the subject and verb agreement is off, too).
You have a XXX comment on line 592 of the diff.
In running_ in_container, if neither of the running- in-container or
lxc-is-container commands exist it will just return false. It seems to
me that if that happens an exception should be raised instead.