~chad.smith/cloud-init:feature/kvp-reporting

Branch merges

Propose for merging

Merged into cloud-init:master

Server Team CI bot: Approve (continuous-integration) on 2018-08-24

cloud-init Commiters: Pending requested 2018-08-24

Diff: 510 lines (+396/-7)

7 files modified

cloudinit/cloud.py (+2/-2)
cloudinit/cmd/main.py (+2/-2)
cloudinit/distros/__init__.py (+1/-1)
cloudinit/reporting/__init__.py (+1/-1)
cloudinit/reporting/handlers.py (+255/-0)
cloudinit/stages.py (+1/-1)
tests/unittests/test_reporting_hyperv.py (+134/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

0124a19... by Andy on 2018-08-24

1. resolve comments. 2. update some unit test.

af45f66... by Andy on 2018-08-24

resolve the code review comments.

3388592... by Andy on 2018-07-07

overwrite the event passed.

5d68f97... by Paul Meyer on 2017-05-17

Add Hyper-V KVP reporter

e8b9a57... by Paul Meyer on 2017-05-16

Fix inaccurate logging

3246f4c... by Paul Meyer on 2017-05-16

Add extra util.subp logging

9e38bf8... by Paul Meyer on 2017-05-16

typos

3146c96... by Mike Gerdts on 2018-07-26

update_metadata re-config on every boot comments and tests not quite right

The comment in update_metadata() that explains how a datasource should
enable network reconfig on every boot presumes that
EventType.BOOT_NEW_INSTANCE is a subset of EventType.BOOT. That's not
the case, and as such a datasource that needs to configure networking
when it is a new instance and every boot needs to include both event
types.

To make the situation above easier to debug, update_metadata() now
logs when it returns false.

To make it so that datasources do not need to test before appending to
the update_events['network'], it is changed from a list to a set.

test_update_metadata_only_acts_on_supported_update_events is updated
to allow datasources to support EventType.BOOT.

Author: Mike Gerdts <email address hidden>

6a9e576... by Scott Moser on 2018-07-23

tests: Collect build_info from system if available.

This adds a script to always get the /etc/cloud/build.info file
if it exists, and a hook when preparing the image to log the information
if it is available.

INFO - setting up ubuntu-cosmic (build_name=server serial=20180718)

This is just useful for debug and reproduce.

2a9d620... by Scott Moser on 2018-07-21

pylint: Fix pylint warnings reported in pylint 2.0.0.

Pylint 2.0.0 was recently released and complains more about
logging-not-lazy than it used to. I've fixed those warnings, here.

The changes in rh_subscription are more extensive. pylint may be
complaining incorrectly there, but the tests were not correctly un-doing
all of their mock/patching. This cleans those up and makes pylint happy.