Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

Add description to service config in admin command #172

Merged
merged 1 commit into from
May 4, 2017
Merged

Conversation

thuningxu
Copy link
Contributor

@thuningxu thuningxu commented Apr 26, 2017

$ ./cherami-admin cfg --help
NAME:
   cherami serviceconfig - Manage service configs. Supported service names, config keys and usage are:
"cherami-controllerhost", "numPublisherExtentsByPath",      comma-separated list of "prefix=extent_count"
"cherami-controllerhost", "numConsumerExtentsByPath",       comma-separated list of "prefix=extent_count"
"cherami-controllerhost", "numRemoteConsumerExtentsByPath", comma-separated list of "prefix=extent_count"
"cherami-controllerhost", "activeZone",                     the active zone for multi-zone consumers
"cherami-controllerhost", "failoverMode",                   "enabled" or "disabled"
"cherami-storehost",      "adminStatus",                    if set to anything other than "enabled", will prevent placement of new extent on this store
"cherami-storehost",      "minFreeDiskSpaceBytes",          integer, minimum required free disk space in bytes to place a new extent
"cherami-outputhost",     "messagecachesize",               comma-separated list of "destination/CG_name=value" for message cache size
      

USAGE:
   cherami serviceconfig command [command options] [arguments...]

@thuningxu thuningxu requested a review from venkat1109 April 26, 2017 17:05
@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 68.651% when pulling 259928e on admin into 6442730 on master.

"cherami-controllerhost", "numPublisherExtentsByPath", comma-separated list of "prefix=extent_count"
"cherami-controllerhost", "numConsumerExtentsByPath", comma-separated list of "prefix=extent_count"
"cherami-controllerhost", "numRemoteConsumerExtentsByPath", comma-separated list of "prefix=extent_count"
"cherami-controllerhost", "activeZone", comma-separated list of active zones
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

actually there should be only one activeZone

"cherami-controllerhost", "numConsumerExtentsByPath", comma-separated list of "prefix=extent_count"
"cherami-controllerhost", "numRemoteConsumerExtentsByPath", comma-separated list of "prefix=extent_count"
"cherami-controllerhost", "activeZone", comma-separated list of active zones
"cherami-controllerhost", "failoverMode", true or false
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

enabled or disabled

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 68.397% when pulling 02de19f on admin into d81c1fa on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 68.332% when pulling a4a06f5 on admin into 76665c4 on master.

@thuningxu thuningxu merged commit 3f513f9 into master May 4, 2017
@thuningxu thuningxu deleted the admin branch May 4, 2017 22:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants