-
Notifications
You must be signed in to change notification settings - Fork 499
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[JUJU-1050] Output life in juju show-unit
and juju show-application
#14001
Conversation
2e7a916
to
06509b6
Compare
juju show-unit
juju show-unit
and juju show-application
88c7b8f
to
bf2574c
Compare
@jack-w-shaw - QA is good for same client/controller. New client with older controller yields
for show-unit, but not for show-application. A usability nit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good, a few tests need fixing, as does the empty string output when client/agents are mismatched.
bf2574c
to
7809a18
Compare
7809a18
to
e01c6f8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
#14017 Merge from 2.9 to bring forward: - #14007 from wallyworld/update-golang-deps - #14013 from hmlanigan/show-controller-panic - #14012 from hmlanigan/remove-unit-subordinate - #13998 from turrisxyz/setup-permissions - #14011 from ycliuhw/update-ck-overlay - #14010 from tlm/juju-jenkins-tests - #14004 from tlm/juju-jenkins-tests - #14003 from manadart/2.9-close-unit-loggers - #14001 from jack-w-shaw/JUJU-1050_show_unit_life - #13890 from juju/dependabot/github_actions/actions/upload-artifact-3 - #14000 from wallyworld/use-go1.18 - #13999 from tlm/juju-jenkins-tests - #13991 from ycliuhw/fix/lp1968643 - #13997 from tlm/cross-compile - #13931 from arnodel/juju-885-remove-modelaccess-connection-iface-method - #13992 from jack-w-shaw/JUJU-1034_reduce_over_verbose_logging_network - #13993 from hmlanigan/fix-intermittent-fail-testenqueuedoperation - #13990 from wallyworld/handle-azure-quota-errors - #13989 from benhoyt/start-worker-if-dead - #13987 from hpidcock/filter-test-packages Conflicts: - cmd/juju/controller/showcontroller.go - go.mod - go.sum - provider/azure/environ_test.go
Juju only inconsistently outputs the
life
status of entities when runningjuju show-
. This PR is the first step in making this consistentChecklist
[ ] Requires a pylibjuju change[ ] Added or updated doc.go related to packages changedQA steps
Deploy something big and complex
And wait for it to be fully deployed
Test show-unit with alive unit
Test show-application with alive application
This is not sufficient QA, however, since
alive
corresponds to the integer 0, so this value could just be uninitialised under the hoodDestroy the model, and quickly whilst units are dying, check
show-x
Unit
and application
Bug reference
https://bugs.launchpad.net/juju/+bug/1968092