Move PORTS after ADDRESS in tabular status. #5870

Merged
merged 1 commit into from Jul 26, 2016

Conversation

Projects
None yet
3 participants
Owner

jameinel commented Jul 26, 2016

The idea is that standard syntax is 'ip:port' so it makes more sense for PORTS
to come after ADDRESS instead of before.

Testing:

  $ juju bootstrap test-lxd lxd
  $ juju deploy mysql
  $ juju status

And see something like:

MODEL    CONTROLLER  CLOUD/REGION   VERSION
default  test-lxd    lxd/localhost  2.0-beta14.1

APP    VERSION  STATUS   EXPOSED  ORIGIN      CHARM  REV  OS
mysql           unknown  false    jujucharms  mysql  55   ubuntu

RELATION  PROVIDES  CONSUMES  TYPE
cluster   mysql     mysql     peer

UNIT     WORKLOAD  AGENT  MACHINE  PUBLIC-ADDRESS  PORTS     MESSAGE
mysql/0  unknown   idle   0        10.246.146.20   3306/tcp  

MACHINE  STATE    DNS            INS-ID         SERIES  AZ
0        started  10.246.146.20  juju-7c9629-0  trusty  

which shows that PORTS is coming after the PUBLIC-ADDRESS

Move PORTS after ADDRESS in tabular status.
The idea is that standard syntax is 'ip:port' so it makes more sense for PORTS
to come after ADDRESS instead of before.
Member

anastasiamac commented Jul 26, 2016

LGTM

Owner

jameinel commented Jul 26, 2016

$$merge$$

Contributor

jujubot commented Jul 26, 2016

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

Contributor

jujubot commented Jul 26, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8550

Owner

jameinel commented Jul 26, 2016

$$merge$$

Contributor

jujubot commented Jul 26, 2016

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

@jujubot jujubot merged commit 33ecbb8 into juju:master Jul 26, 2016

@jameinel jameinel deleted the jameinel:status-ports-addresses branch Dec 13, 2016

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