~smoser/cloud-init:fix/rh-subscription-add-null-check

Last commit made on 2017-10-10
Get this branch:
git clone -b fix/rh-subscription-add-null-check https://git.launchpad.net/~smoser/cloud-init
Only Scott Moser can upload to this branch. If you are Scott Moser please log in for upload directions.

Branch merges

Branch information

Name:
fix/rh-subscription-add-null-check
Repository:
lp:~smoser/cloud-init

Recent commits

534414d... by Scott Moser on 2017-10-10

add a test and always call update_repos from handle.

this just gets the disabled-repos=None path called in tests, so we can
verify it functional.

5d5297d... by Scott Moser on 2017-10-10

switch to passing a logger in and using LOG by default.

this is general "best practice" on how to use logging in cloud-init
now. the 'log' that is passed into the handle calls is generally
legacy.

make the SubscriptionManager take an optional log and set its
attribute. Then fix the calls to not trip pylint complaints.

85e01c5... by David Mulford on 2017-10-10

Fixed object identity check to 'is not' to pass the flake8 checks

1262f85... by David Mulford on 2017-10-09

Added null checks rh_subscription module to avoid needing to specify both enabled and disabled repos

45d361c... by Chad Smith on 2017-10-05

net: Handle bridge stp values of 0 and convert to boolean type

Update unit tests to pass a 0 instead of 'off' to validate that network
state is properly written.

6eb4dc2... by Scott Moser on 2017-10-05

tools: Give specific --abbrev=8 to "git describe"

The tools that use "git describe" were just assuming a consisent
number of characters in the hash. It seems ubuntu 16.04 would use 7
and later versions use 8. To avoid that discrepency in developer
environments, set it to 8.

57e2e01... by Ryan Harper on 2017-10-03

network: bridge_stp value not always correct

Update network_state to store the bridge_stp value as a boolean.
The various renderers then can map the boolean value to the correct
output as needed; eni uses 'on/off', sysconfig uses 'yes/no' and
netplan will use the boolean directly.

Update unittest values for sysconfig and netplan. Both contained the
network_state string value which resulted in not correctly enable/disable
STP in the target system.

Update network_state comment (fd -> forward-delay, add stp as boolean) on
bridge commands to match the expected format of a netplan bridge command.

LP: #1721157

aa024e3... by Joshua Powers on 2017-10-03

tests: re-enable tox with nocloud-kvm support

With the addition of the nocloud-kvm support a few other python modules
were pulled in as required and as a result this broke the tox run. The
fix was to add paramiko and simplestreams to re-enable testing.

7fd0425... by Robert Schweikert on 2017-09-18

systemd: remove limit on tasks created by cloud-init-final.service.

Depending on distribution the default number of tasks (threads) maybe
unexpectedly low or it may be the default systemd setting (512).
Setting TasksMax to "infinity" in cloud-init-final.service removes the
restriction on tasks created.

LP: #1717969

cc1475d... by Robert Schweikert on 2017-09-21

suse: Support addition of zypper repos via cloud-config.

This adds a config module so support for adding zypper repositories
via cloud-config.

LP: #1718675