Merge ~chad.smith/cloud-init:collect-logs into cloud-init:master
Status: | Merged |
---|---|
Approved by: | Scott Moser on 2017-09-15 |
Approved revision: | ef77fe49051d67d7486e3ef7dd4dd662271abd1e |
Merged at revision: | e626966ee7d339b53d2c8b14a8f2ff8e3fe892ee |
Proposed branch: | ~chad.smith/cloud-init:collect-logs |
Merge into: | cloud-init:master |
Diff against target: |
439 lines (+354/-6) 7 files modified
cloudinit/apport.py (+105/-0) cloudinit/cmd/devel/logs.py (+101/-0) cloudinit/cmd/devel/tests/__init__.py (+0/-0) cloudinit/cmd/devel/tests/test_logs.py (+120/-0) cloudinit/cmd/main.py (+10/-1) packages/debian/rules.in (+1/-0) tests/unittests/test_cli.py (+17/-5) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Server Team CI bot | continuous-integration | Approve on 2017-09-15 | |
Scott Moser | 2017-09-12 | Approve on 2017-09-15 | |
Review via email:
|
Commit message
cmdline: add collect-logs subcommand.
Add a new collect-logs sub command to the cloud-init CLI. This script
will collect all logs pertinent to a cloud-init run and store them in a
compressed tar-gzipped file. This tarfile can be attached to any
cloud-init bug filed in order to aid in bug triage and resolution.
A cloudinit.apport module is also added that allows apport interaction.
Here is an example bug filed via ubuntu-bug cloud-init: LP: #1716975.
Once the apport launcher is packaged in cloud-init, bugs can be filed
against cloud-init with the following command:
ubuntu-bug cloud-init
LP: #1607345
Description of the change
cmdline: cloud-init collect-logs
Add a new collect-logs parameter to the cloud-init CLI. This script will
collect all logs pertinent to a cloud-init run and store them in a
compressed tar-gzipped file. this tarfile can be attached to any
cloud-init bug filed in order to aid in bug triage and resolution.
Added apport package-hooks script for cloud-init, here is an example bug filed via ubuntu-bug cloud-init: LP: #1716975.
Bugs can now be filed against cloud-init with the following command:
ubuntu-bug cloud-init
LP: #1607345
- 077f8e9... by Chad Smith on 2017-09-13
- 9428b5b... by Chad Smith on 2017-09-13
- c969087... by Chad Smith on 2017-09-13
- 7a2ae7b... by Chad Smith on 2017-09-13
- 1efed63... by Chad Smith on 2017-09-13
- 66dad59... by Chad Smith on 2017-09-13
PASSED: Continuous integration, rev:66dad59e4d2
https:/
Executed test runs:
SUCCESS: Checkout
SUCCESS: Unit & Style Tests
SUCCESS: Ubuntu LTS: Build
SUCCESS: Ubuntu LTS: Integration
SUCCESS: MAAS Compatability Testing
IN_PROGRESS: Declarative: Post Actions
Click here to trigger a rebuild:
https:/
Scott Moser (smoser) wrote : | # |
Others (rbasak) disagree, but if I run:
$ cloud-init collect-logs
And it shows me nothing. I wonder what it did, and if it did something.
I'd prefer:
$ cloud-init collect-logs
wrote cloud-init.tar.gz
other things:
* 'version' file has no carriage return.
some comments inline.
- e06880a... by Chad Smith on 2017-09-15
- 6d93c8c... by Chad Smith on 2017-09-15
- 327f3ee... by Chad Smith on 2017-09-15
FAILED: Continuous integration, rev:327f3ee9cf6
https:/
Executed test runs:
SUCCESS: Checkout
FAILED: Unit & Style Tests
Click here to trigger a rebuild:
https:/
- 04260ae... by Chad Smith on 2017-09-15
PASSED: Continuous integration, rev:04260ae927a
https:/
Executed test runs:
SUCCESS: Checkout
SUCCESS: Unit & Style Tests
SUCCESS: Ubuntu LTS: Build
SUCCESS: Ubuntu LTS: Integration
SUCCESS: MAAS Compatability Testing
IN_PROGRESS: Declarative: Post Actions
Click here to trigger a rebuild:
https:/
- c83e0ba... by Chad Smith on 2017-09-15
Chad Smith (chad.smith) wrote : | # |
For full apport support: here's the wrapper we need to deliver to /usr/share/
FAILED: Continuous integration, rev:c83e0bac7ed
https:/
Executed test runs:
SUCCESS: Checkout
FAILED: Unit & Style Tests
Click here to trigger a rebuild:
https:/
- dba1e05... by Chad Smith on 2017-09-15
FAILED: Continuous integration, rev:dba1e05097e
https:/
Executed test runs:
SUCCESS: Checkout
FAILED: Unit & Style Tests
Click here to trigger a rebuild:
https:/
Scott Moser (smoser) wrote : | # |
minor changes suggested.
make those and we're good.
thanks.
Scott Moser (smoser) wrote : | # |
I put this up
https:/
for packaging of the apport lauincher
- ef77fe4... by Chad Smith on 2017-09-15
PASSED: Continuous integration, rev:ef77fe49051
https:/
Executed test runs:
SUCCESS: Checkout
SUCCESS: Unit & Style Tests
SUCCESS: Ubuntu LTS: Build
SUCCESS: Ubuntu LTS: Integration
SUCCESS: MAAS Compatability Testing
IN_PROGRESS: Declarative: Post Actions
Click here to trigger a rebuild:
https:/
PASSED: Continuous integration, rev:077f8e937c1 81c94373a013309 7f2d5b904b3431 /jenkins. ubuntu. com/server/ job/cloud- init-ci/ 285/
https:/
Executed test runs:
SUCCESS: Checkout
SUCCESS: Unit & Style Tests
SUCCESS: Ubuntu LTS: Build
SUCCESS: Ubuntu LTS: Integration
SUCCESS: MAAS Compatability Testing
IN_PROGRESS: Declarative: Post Actions
Click here to trigger a rebuild: /jenkins. ubuntu. com/server/ job/cloud- init-ci/ 285/rebuild
https:/