lp:~larsks/cloud-init/fix-systemd-detection

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:~cloud-init-dev/cloud-init/trunk at revision 1112

Dan Watkins: Approve on 2015-06-10

Joshua Harlow: Pending requested 2015-06-05

Diff: 102 lines (+37/-12)

4 files modified

cloudinit/distros/__init__.py (+8/-0)
cloudinit/distros/rhel.py (+0/-8)
tests/unittests/helpers.py (+6/-4)
tests/unittests/test_distros/test_generic.py (+23/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1461201: Check for systemd (in distros/rhel.py) is fragile

Low

Expired

Link a bug report

Related blueprints

1113. By Lars Kellogg-Stedman on 2015-06-03

add tests for systemd detection

This adds the following tests in test_distros.test_generic:

- test_systemd_in_use

  Test the situation in which /run/systemd/system exists.

- test_systemd_not_in_use

  Test the situation in which /run/systemd/system does not exists.

- test_systemd_symlink

  This tests the situation in which /run/systemd/system exists but is a
  *symlink* to a directory, which according to sd_booted() should return
  false.

1112. By Lars Kellogg-Stedman on 2015-06-03

transform paths in functions taking more than a single argument

Patch FilesystemMockingTestcase.patchOS to support methods taking more
than a single path argument. This is required in order to properly mock
`os.symlink`, which takes two path arguments.

1111. By Lars Kellogg-Stedman on 2015-06-02

check for systemd using sd_booted() semantics

The existing cloud-init code determines if systemd is in use by looking at the
distribution name and version. This is prone to error because:

- RHEL derivatives other than CentOS (e.g., Scientific Linux) will fail this test, and
- Distributions that are not derived from RHEL also use systemd

This patch makes cloud-init use the same logic that is used in systemd's
sd_booted() method
(http://www.freedesktop.org/software/systemd/man/sd_booted.html)

1110. By Scott Moser on 2015-06-01

Doc: include information on vendor-data in OpenStack

1109. By Scott Moser on 2015-05-22

Azure: Redact on-disk user password in /var/lib/ovf-env.xml

The fabric provides the user password in plain text via the CDROM,
and cloud-init has previously wrote the ovf-env.xml in /var/lib/waagent
with the password in plain text. This change redacts the password.

1108. By Scott Moser on 2015-05-19

EC2: be aware of eu-central-1 availability zone

eu-central-1 means that 'central' is a direction to update the
regular expression to understand.

1107. By Scott Moser on 2015-05-19

packages/brpm: fix for oauth library name

brpm was broken when change from oauth to oauthlib.

1106. By Scott Moser on 2015-05-15

temporarily disasble test if no bin/cloud-init

the tests of bin/cloud-init would fail in a package build environment.
so, temporariliy skip them in that environment.

1105. By Scott Moser on 2015-05-15

pep8 fixes

1104. By Scott Moser on 2015-05-15

Azure: remove dependency on walinux-agent

This takes away our dependency on walinux-agent, by providing a builtin
path for doing cloud-init had delegated to it.

Currently the default is to still use the old path, but adding this code
in will allow us to move to the new code path with more confidence.