Merge lp://qastaging/~ericsnowcurrently/landscape-charm/juju2-collect-inner-model into lp://qastaging/~landscape/landscape-charm/tools

Proposed by Eric Snow
Status: Superseded
Proposed branch: lp://qastaging/~ericsnowcurrently/landscape-charm/juju2-collect-inner-model
Merge into: lp://qastaging/~landscape/landscape-charm/tools
Diff against target: 1230 lines (+1004/-79)
2 files modified
collect-logs (+187/-79)
test_collect-logs.py (+817/-0)
To merge this branch: bzr merge lp://qastaging/~ericsnowcurrently/landscape-charm/juju2-collect-inner-model
Reviewer Review Type Date Requested Status
Landscape Pending
Landscape Pending
Review via email: mp+303197@code.qastaging.launchpad.net

This proposal has been superseded by a proposal from 2016-08-17.

Description of the change

Make collect-logs work with Juju 2.

Testing instructions:

1. Deploy landscape
2. install juju-2.0 on the landscape-server/0 unit
3. enable the "juju-2.0" feature flag
4. autopilot
5. run collect-logs

To post a comment you must log in.
32. By Eric Snow

Fix the retry message.

33. By Eric Snow

More clearly identify "inner" log collection.

34. By Eric Snow

Merge from parent branch.

35. By Eric Snow

Merge from parent branch.

36. By Eric Snow

Use correct types.

37. By Eric Snow

Correctly build commands for the inner model.

38. By Eric Snow

Improve logging.

39. By Eric Snow

Drop special treatment for sudo.

40. By Eric Snow

Work around a permissions issue with the cookie file.

41. By Eric Snow

Clean up log messages.

42. By Eric Snow

Distinguish between Juju 1 and 2 better.

43. By Eric Snow

Add missing docstrings.

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: