Implement juju resources --details flag #4374

Merged

jujubot merged 12 commits into juju:feature-resources from unknown repository

Feb 12, 2016

Contributor

natefinch commented Feb 10, 2016

This adds a --details flag that does a diff of the expected resources on each unit, and the actual resources.

(Review request: http://reviews.vapour.ws/r/3818/)

natefinch and others added some commits Feb 8, 2016


      rename show-service-resources to list-resources and remove the show-c…

…harm-resources command since it's being written in a different commit and conflicts with the alias for list-resources

f7f61bd


      add details flag

bb26864


      handle unit resources that haven't been retrieved with resource-get

2583cb9


      rename debug to details in-code, and start writing tests

824a30e


      fixes after rebase

861a8f3


      Merge pull request #4334 from ericsnowcurrently/resources-charm-command

Rename "juju resources" to "juju charm list-resources".

A key part of this is adding the "juju charm" command.

(Review request: http://reviews.vapour.ws/r/3778/)


      fix merge problem

9884bba


      update tests so they pass and add table headers to match spec

2b86c52


      make single unit details have header [Unit]

a820ca2


      put back test that shouldn't have been deleted

b269bfe


      unexport details flag var


      couple sanity check tests for formatting. Also tweaked the output for…

… juju resources <unit> --details, since it was useless printing out the unit number.

922cf5e

Contributor

natefinch commented Feb 12, 2016

$$merge$$

Contributor

jujubot commented Feb 12, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

natefinch commented Feb 12, 2016

$$merge$$

jujubot added a commit that referenced this pull request Feb 12, 2016


      Merge pull request #4374 from natefinch/list-resources-debug

Implement juju resources --details flag

This adds a --details flag that does a diff of the expected resources on each unit, and the actual resources.

(Review request: http://reviews.vapour.ws/r/3818/)

f0e1848

jujubot merged commit `f0e1848` into juju:feature-resources Feb 12, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

You can't perform that action at this time.