Seagate / halon

High availability solution
Apache License 2.0
1 stars 0 forks source link

HALON-887: Tag Resource Coordinator in `hctl mero status` output #1549

Closed rajanikantchirmade closed 5 years ago

rajanikantchirmade commented 5 years ago

Resource Coordinator is tagged with " (RC)" suffix:

$ hctl mero status
Cluster disposition: ONLINE
…
Hosts:
  [   online] 0x6e00000000000001:0x9    client1
  [   online] 0x7200000000000001:0xa      172.28.128.6@tcp:12345:34:101 halon
…
  [   online] 0x6e00000000000001:0x15   cmu
  [   online] 0x7200000000000001:0x16     172.28.128.3@tcp:12345:34:101 halon (RC)
…
  [   online] 0x6e00000000000001:0x1c   ssu1
  [   online] 0x7200000000000001:0x2a     172.28.128.4@tcp:12345:34:101 halon (TS)
…
  [   online] 0x7200000000000001:0x2d     172.28.128.4@tcp:12345:44:101 confd
  [   online] 0x7300000000000001:0x2e       CST_CONFD
  [   online] 0x7300000000000001:0x2f       CST_RMS (principal)
…
  [   online] 0x6e00000000000001:0x38   ssu2
  [   online] 0x7200000000000001:0x46     172.28.128.5@tcp:12345:34:101 halon (TS)
…
vvv commented 5 years ago

merged

vvv commented 5 years ago

I've removed this line.

vvv commented 5 years ago

Commented me and pushed the code that doesn't even compile. Twice. :flushed:

vvv commented 5 years ago

enabled an automatic merge when the pipeline for 87747f9c40de359a4574d59fe7e3c23ccc4db952 succeeds

vvv commented 5 years ago

added 1 commit

Compare with previous version

vvv commented 5 years ago

changed this line in version 6 of the diff

vvv commented 5 years ago

enabled an automatic merge when the pipeline for 61fe5f4d27f95751a4ad87d06c54655b49eef900 succeeds

vvv commented 5 years ago

added 3 commits

Compare with previous version

vvv commented 5 years ago

added 1 commit

Compare with previous version

vvv commented 5 years ago

I've used do block and split into separate lines for better readability.

vvv commented 5 years ago

@rajanikant.chirmade FYI.

vvv commented 5 years ago

Parentheses are not needed.

vvv commented 5 years ago

This is equivalent to isJust mp.

vvv commented 5 years ago

enabled an automatic merge when the pipeline for 9d47b9c767536a94bea7534631691e5ebe0224b5 succeeds

vvv commented 5 years ago

added 1 commit

Compare with previous version

vvv commented 5 years ago

changed this line in version 3 of the diff

vvv commented 5 years ago

changed this line in version 3 of the diff

vvv commented 5 years ago

enabled an automatic merge when the pipeline for 900b7e99e5562b6ede6615f30417321b9291241d succeeds

vvv commented 5 years ago

Haddock format expected. s/Is/^ Is/ please.

vvv commented 5 years ago

@rajanikant.chirmade It looks like a field is forgotten here?

vvv commented 5 years ago

@rajanikant.chirmade

whitespace_errors_scored += 2

:trophy:

vvv commented 5 years ago

changed the description

vvv commented 5 years ago

added 1 commit

Compare with previous version

rajanikantchirmade commented 5 years ago

assigned to @vvv