)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":18838,"name":"Paul Dardeau","email":"paul.dardeau@intel.com","username":"pdardeau"},"change_message_id":"5942ae630a129958f8a2edd2084c3af40db60740","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-05-15 14:51:22 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Running swift-init with -h, --help, or no arguments"},{"line_number":8,"context_line":"displays help for the command. The help does not"},{"line_number":9,"context_line":"document the \u0027main\u0027, \u0027all\u0027, and \u0027rest\u0027 options."},{"line_number":10,"context_line":"These are documented in the man page."},{"line_number":11,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dab17558_c76942c0","line":8,"updated":"2016-05-15 21:36:43.000000000","message":"The first line should be a brief summary of the change and then be followed by a blank line.  See https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"0181e2ac8e315e53e3e5ae1ba961db9de8f376cd"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"ec4d9806d97b78c70d128cb5623a88e47f819887","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-05-15 14:51:22 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Running swift-init with -h, --help, or no arguments"},{"line_number":8,"context_line":"displays help for the command. The help does not"},{"line_number":9,"context_line":"document the \u0027main\u0027, \u0027all\u0027, and \u0027rest\u0027 options."},{"line_number":10,"context_line":"These are documented in the man page."},{"line_number":11,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dab17558_e729e69c","line":8,"updated":"2016-05-15 23:19:57.000000000","message":"Yeah what Paul said, also I find this link useful when writting commit messages: https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"0181e2ac8e315e53e3e5ae1ba961db9de8f376cd"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"11bf83c7b3453201ef5e4d88145c76cf894b4075","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-05-15 14:51:22 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Running swift-init with -h, --help, or no arguments"},{"line_number":8,"context_line":"displays help for the command. The help does not"},{"line_number":9,"context_line":"document the \u0027main\u0027, \u0027all\u0027, and \u0027rest\u0027 options."},{"line_number":10,"context_line":"These are documented in the man page."},{"line_number":11,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dab17558_bb24170d","line":8,"in_reply_to":"dab17558_c76942c0","updated":"2016-05-16 04:21:10.000000000","message":"Done","commit_id":"0181e2ac8e315e53e3e5ae1ba961db9de8f376cd"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"11bf83c7b3453201ef5e4d88145c76cf894b4075","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-05-15 14:51:22 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Running swift-init with -h, --help, or no arguments"},{"line_number":8,"context_line":"displays help for the command. The help does not"},{"line_number":9,"context_line":"document the \u0027main\u0027, \u0027all\u0027, and \u0027rest\u0027 options."},{"line_number":10,"context_line":"These are documented in the man page."},{"line_number":11,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dab17558_db275b10","line":8,"in_reply_to":"dab17558_e729e69c","updated":"2016-05-16 04:21:10.000000000","message":"Done","commit_id":"0181e2ac8e315e53e3e5ae1ba961db9de8f376cd"},{"author":{"_account_id":18838,"name":"Paul Dardeau","email":"paul.dardeau@intel.com","username":"pdardeau"},"change_message_id":"d6d1891cba9ea465a3de50a954e624b1660ec076","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Mohit Motiani \u003cmohit.motiani@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-05-16 04:14:45 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Updating --help of swift-init:"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Running swift-init with -h, --help, or no arguments"},{"line_number":10,"context_line":"displays help for the command. The help does not"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"dab17558_5294ce89","line":7,"updated":"2016-05-16 15:22:20.000000000","message":"Please make the first line more descriptive of change.  Maybe something like:\n\nadd description of main, all, and rest options for swift-init help","commit_id":"57f6ccd9c096d7b60a5635e5c0dbea637c3099b1"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"5b4d69637d394ec9927797a62063659d37ba4e53","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Mohit Motiani \u003cmohit.motiani@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-05-16 04:14:45 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Updating --help of swift-init:"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Running swift-init with -h, --help, or no arguments"},{"line_number":10,"context_line":"displays help for the command. The help does not"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"dab17558_29c27c85","line":7,"in_reply_to":"dab17558_5294ce89","updated":"2016-05-16 15:45:18.000000000","message":"Done","commit_id":"57f6ccd9c096d7b60a5635e5c0dbea637c3099b1"},{"author":{"_account_id":18838,"name":"Paul Dardeau","email":"paul.dardeau@intel.com","username":"pdardeau"},"change_message_id":"75830bed0d675ff066cfb461c9e1d0b3e77b8383","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Mohit Motiani \u003cmohit.motiani@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-05-16 15:42:43 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add description of server options (all, main, rest) for swift-init help:"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Running swift-init with -h, --help, or no arguments"},{"line_number":10,"context_line":"displays help for the command. The help does not"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dab17558_4940a0a5","line":7,"updated":"2016-05-16 15:51:34.000000000","message":"please remove \":\" at end of line","commit_id":"a07cff2794f53845d5c622cf7fc4a7667c1cda49"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"c0864c98bedfac79688926bcb7e7721e2f1c7081","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Mohit Motiani \u003cmohit.motiani@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-05-16 15:42:43 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add description of server options (all, main, rest) for swift-init help:"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Running swift-init with -h, --help, or no arguments"},{"line_number":10,"context_line":"displays help for the command. The help does not"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dab17558_3c0d9c3e","line":7,"in_reply_to":"dab17558_4940a0a5","updated":"2016-05-16 16:11:05.000000000","message":"Done","commit_id":"a07cff2794f53845d5c622cf7fc4a7667c1cda49"}],"bin/swift-init":[{"author":{"_account_id":18838,"name":"Paul Dardeau","email":"paul.dardeau@intel.com","username":"pdardeau"},"change_message_id":"5942ae630a129958f8a2edd2084c3af40db60740","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    \u003cserver\u003e  is the name of a swift service e.g. proxy-server."},{"line_number":28,"context_line":"              The \u0027-server\u0027 part of the name may be omitted."},{"line_number":29,"context_line":"              \u0027main\u0027, \u0027all\u0027 and \u0027rest\u0027 are the important server options"},{"line_number":30,"context_line":"              used to start/stop deamons."},{"line_number":31,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":32,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"},{"line_number":33,"context_line":"              refer to a directory containing the configuration file, e.g.:"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"dab17558_e75c461b","line":30,"updated":"2016-05-15 21:36:43.000000000","message":"Should add a brief explanation of what each of these options does (like in man page).  Also, change \u0027deamons\u0027 to \u0027daemons\u0027.","commit_id":"0181e2ac8e315e53e3e5ae1ba961db9de8f376cd"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"11bf83c7b3453201ef5e4d88145c76cf894b4075","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    \u003cserver\u003e  is the name of a swift service e.g. proxy-server."},{"line_number":28,"context_line":"              The \u0027-server\u0027 part of the name may be omitted."},{"line_number":29,"context_line":"              \u0027main\u0027, \u0027all\u0027 and \u0027rest\u0027 are the important server options"},{"line_number":30,"context_line":"              used to start/stop deamons."},{"line_number":31,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":32,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"},{"line_number":33,"context_line":"              refer to a directory containing the configuration file, e.g.:"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"dab17558_9b095387","line":30,"in_reply_to":"dab17558_e75c461b","updated":"2016-05-16 04:21:10.000000000","message":"Done","commit_id":"0181e2ac8e315e53e3e5ae1ba961db9de8f376cd"},{"author":{"_account_id":18838,"name":"Paul Dardeau","email":"paul.dardeau@intel.com","username":"pdardeau"},"change_message_id":"d6d1891cba9ea465a3de50a954e624b1660ec076","unresolved":false,"context_lines":[{"line_number":29,"context_line":"              \u0027all\u0027, \u0027main\u0027 and \u0027rest\u0027 are the important server options"},{"line_number":30,"context_line":"              used to start/stop daemons."},{"line_number":31,"context_line":"              all: Initializes all swift daemons."},{"line_number":32,"context_line":"              main: Initializes main swift daemons."},{"line_number":33,"context_line":"              rest: Initializes all other background  daemons."},{"line_number":34,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":35,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"dab17558_521e8e0c","line":32,"updated":"2016-05-16 15:22:20.000000000","message":"maybe indicate which daemons are the main ones (like the man page does)","commit_id":"57f6ccd9c096d7b60a5635e5c0dbea637c3099b1"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"5b4d69637d394ec9927797a62063659d37ba4e53","unresolved":false,"context_lines":[{"line_number":29,"context_line":"              \u0027all\u0027, \u0027main\u0027 and \u0027rest\u0027 are the important server options"},{"line_number":30,"context_line":"              used to start/stop daemons."},{"line_number":31,"context_line":"              all: Initializes all swift daemons."},{"line_number":32,"context_line":"              main: Initializes main swift daemons."},{"line_number":33,"context_line":"              rest: Initializes all other background  daemons."},{"line_number":34,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":35,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"dab17558_0955f8c0","line":32,"in_reply_to":"dab17558_521e8e0c","updated":"2016-05-16 15:45:18.000000000","message":"Done","commit_id":"57f6ccd9c096d7b60a5635e5c0dbea637c3099b1"},{"author":{"_account_id":18838,"name":"Paul Dardeau","email":"paul.dardeau@intel.com","username":"pdardeau"},"change_message_id":"d6d1891cba9ea465a3de50a954e624b1660ec076","unresolved":false,"context_lines":[{"line_number":30,"context_line":"              used to start/stop daemons."},{"line_number":31,"context_line":"              all: Initializes all swift daemons."},{"line_number":32,"context_line":"              main: Initializes main swift daemons."},{"line_number":33,"context_line":"              rest: Initializes all other background  daemons."},{"line_number":34,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":35,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"},{"line_number":36,"context_line":"              refer to a directory containing the configuration file, e.g.:"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"dab17558_d209de96","line":33,"updated":"2016-05-16 15:22:20.000000000","message":"indicate which ones (like man page does).\n\nNit: extra space between \"background\" and \"daemons\"","commit_id":"57f6ccd9c096d7b60a5635e5c0dbea637c3099b1"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"5b4d69637d394ec9927797a62063659d37ba4e53","unresolved":false,"context_lines":[{"line_number":30,"context_line":"              used to start/stop daemons."},{"line_number":31,"context_line":"              all: Initializes all swift daemons."},{"line_number":32,"context_line":"              main: Initializes main swift daemons."},{"line_number":33,"context_line":"              rest: Initializes all other background  daemons."},{"line_number":34,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":35,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"},{"line_number":36,"context_line":"              refer to a directory containing the configuration file, e.g.:"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"dab17558_49738080","line":33,"in_reply_to":"dab17558_d209de96","updated":"2016-05-16 15:45:18.000000000","message":"Done","commit_id":"57f6ccd9c096d7b60a5635e5c0dbea637c3099b1"},{"author":{"_account_id":18838,"name":"Paul Dardeau","email":"paul.dardeau@intel.com","username":"pdardeau"},"change_message_id":"75830bed0d675ff066cfb461c9e1d0b3e77b8383","unresolved":false,"context_lines":[{"line_number":30,"context_line":"              used to start/stop daemons."},{"line_number":31,"context_line":"              all: Initializes all swift daemons."},{"line_number":32,"context_line":"              main: Initializes main swift daemons."},{"line_number":33,"context_line":"                    (example: proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Initializes all other background daemons."},{"line_number":35,"context_line":"                    (example: updater, replicator, auditor, reaper)"},{"line_number":36,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"dab17558_49f9c07f","line":33,"updated":"2016-05-16 15:51:34.000000000","message":"please remove \"example: \"","commit_id":"a07cff2794f53845d5c622cf7fc4a7667c1cda49"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"c0864c98bedfac79688926bcb7e7721e2f1c7081","unresolved":false,"context_lines":[{"line_number":30,"context_line":"              used to start/stop daemons."},{"line_number":31,"context_line":"              all: Initializes all swift daemons."},{"line_number":32,"context_line":"              main: Initializes main swift daemons."},{"line_number":33,"context_line":"                    (example: proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Initializes all other background daemons."},{"line_number":35,"context_line":"                    (example: updater, replicator, auditor, reaper)"},{"line_number":36,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"dab17558_5c006002","line":33,"in_reply_to":"dab17558_49f9c07f","updated":"2016-05-16 16:11:05.000000000","message":"Done","commit_id":"a07cff2794f53845d5c622cf7fc4a7667c1cda49"},{"author":{"_account_id":18838,"name":"Paul Dardeau","email":"paul.dardeau@intel.com","username":"pdardeau"},"change_message_id":"75830bed0d675ff066cfb461c9e1d0b3e77b8383","unresolved":false,"context_lines":[{"line_number":32,"context_line":"              main: Initializes main swift daemons."},{"line_number":33,"context_line":"                    (example: proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Initializes all other background daemons."},{"line_number":35,"context_line":"                    (example: updater, replicator, auditor, reaper)"},{"line_number":36,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":37,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"},{"line_number":38,"context_line":"              refer to a directory containing the configuration file, e.g.:"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"dab17558_c92bf0e9","line":35,"updated":"2016-05-16 15:51:34.000000000","message":"please remove \"example: \"","commit_id":"a07cff2794f53845d5c622cf7fc4a7667c1cda49"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"c0864c98bedfac79688926bcb7e7721e2f1c7081","unresolved":false,"context_lines":[{"line_number":32,"context_line":"              main: Initializes main swift daemons."},{"line_number":33,"context_line":"                    (example: proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Initializes all other background daemons."},{"line_number":35,"context_line":"                    (example: updater, replicator, auditor, reaper)"},{"line_number":36,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":37,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"},{"line_number":38,"context_line":"              refer to a directory containing the configuration file, e.g.:"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"dab17558_ff05c60f","line":35,"in_reply_to":"dab17558_c92bf0e9","updated":"2016-05-16 16:11:05.000000000","message":"Done","commit_id":"a07cff2794f53845d5c622cf7fc4a7667c1cda49"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"c3d41bb330c40566197295623d0124862db45db6","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    \u003cserver\u003e  is the name of a swift service e.g. proxy-server."},{"line_number":28,"context_line":"              The \u0027-server\u0027 part of the name may be omitted."},{"line_number":29,"context_line":"              \u0027all\u0027, \u0027main\u0027 and \u0027rest\u0027 are the important server options"},{"line_number":30,"context_line":"              used to start/stop daemons."},{"line_number":31,"context_line":"              all: Initializes all swift daemons."},{"line_number":32,"context_line":"              main: Initializes main swift daemons."},{"line_number":33,"context_line":"                    (proxy, container, account, object)"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"bab6814e_bca486b9","line":30,"updated":"2016-05-24 03:49:47.000000000","message":"Not 100% sure about the wording. I wouldn\u0027t say they are used to start/stop (what about statis or once) or are important server options (as they aren\u0027t really options but valid values). They are more special group names, maybe it should read something like:\n  \u0027all\u0027, \u0027main\u0027 and \u0027rest\u0027 are reserved words that represent a group of services.\n  all: matches/expands to all swift daemons\n  main: matches the main swift daemons (proxy, account, container, object).\n  rest: matches all other backgroud daemons (updaters, replicators, auditors, reaper, etc).","commit_id":"c673aae275610546275aeddf061ebd96ec947133"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"9140f37c3af29c71a80a5430f55a052f7e5bef5b","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    \u003cserver\u003e  is the name of a swift service e.g. proxy-server."},{"line_number":28,"context_line":"              The \u0027-server\u0027 part of the name may be omitted."},{"line_number":29,"context_line":"              \u0027all\u0027, \u0027main\u0027 and \u0027rest\u0027 are the important server options"},{"line_number":30,"context_line":"              used to start/stop daemons."},{"line_number":31,"context_line":"              all: Initializes all swift daemons."},{"line_number":32,"context_line":"              main: Initializes main swift daemons."},{"line_number":33,"context_line":"                    (proxy, container, account, object)"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"bab6814e_fcd25933","line":30,"in_reply_to":"bab6814e_bca486b9","updated":"2016-05-24 21:36:23.000000000","message":"Done","commit_id":"c673aae275610546275aeddf061ebd96ec947133"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"04bbb2f55df89dfd612e15a3819ab085ef0e3608","unresolved":false,"context_lines":[{"line_number":32,"context_line":"              main: Initializes main swift daemons."},{"line_number":33,"context_line":"                    (proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Initializes all other background daemons."},{"line_number":35,"context_line":"                    (updater, replicator, auditor, reaper)"},{"line_number":36,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":37,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"},{"line_number":38,"context_line":"              refer to a directory containing the configuration file, e.g.:"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"bab6814e_f3248a61","line":35,"updated":"2016-05-23 19:38:52.000000000","message":"Maybe also worth calling out?\n- object-reconstructor\n- object-expirer\n- container-sync\n\nThey seem comparable in importance to account-reaper. Or maybe just say\n\n (updater, replicator, auditor, etc.)","commit_id":"c673aae275610546275aeddf061ebd96ec947133"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"9140f37c3af29c71a80a5430f55a052f7e5bef5b","unresolved":false,"context_lines":[{"line_number":32,"context_line":"              main: Initializes main swift daemons."},{"line_number":33,"context_line":"                    (proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Initializes all other background daemons."},{"line_number":35,"context_line":"                    (updater, replicator, auditor, reaper)"},{"line_number":36,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":37,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"},{"line_number":38,"context_line":"              refer to a directory containing the configuration file, e.g.:"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"bab6814e_dcd5d51b","line":35,"in_reply_to":"bab6814e_f3248a61","updated":"2016-05-24 21:36:23.000000000","message":"Done","commit_id":"c673aae275610546275aeddf061ebd96ec947133"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"77b1a01aa36da62f4cf71e1d4f5c89f6703b4b21","unresolved":false,"context_lines":[{"line_number":28,"context_line":"              The \u0027-server\u0027 part of the name may be omitted."},{"line_number":29,"context_line":"              \u0027all\u0027, \u0027main\u0027 and \u0027rest\u0027 are the reserve words that represent a"},{"line_number":30,"context_line":"              group of services."},{"line_number":31,"context_line":"              all: Matches/Expands all swift daemons."},{"line_number":32,"context_line":"              main: Matches main swift daemons."},{"line_number":33,"context_line":"                    (proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Matches all other background daemons."}],"source_content_type":"application/octet-stream","patch_set":6,"id":"bab6814e_2e6819ba","line":31,"updated":"2016-05-25 07:09:28.000000000","message":"I think we need to use Matches or Expands, I couldn\u0027t decide so I put both versions.\n\n  all: Matches all swift daemons\n  main: Matches main swift daemons...\n\nor\n\n  all: Expands to all swift daemons.\n  main: Expands to all main swift daemons...\n\nI\u0027ll let you pick one or something different. Whatever you think sounds better :)","commit_id":"39bcc021ba2981eb259e2ac1740bd46a42689a01"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"9e42511f6adc6eb136c0f0045ad4b0d271f90420","unresolved":false,"context_lines":[{"line_number":28,"context_line":"              The \u0027-server\u0027 part of the name may be omitted."},{"line_number":29,"context_line":"              \u0027all\u0027, \u0027main\u0027 and \u0027rest\u0027 are the reserve words that represent a"},{"line_number":30,"context_line":"              group of services."},{"line_number":31,"context_line":"              all: Matches/Expands all swift daemons."},{"line_number":32,"context_line":"              main: Matches main swift daemons."},{"line_number":33,"context_line":"                    (proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Matches all other background daemons."}],"source_content_type":"application/octet-stream","patch_set":6,"id":"bab6814e_7894408a","line":31,"in_reply_to":"bab6814e_2e6819ba","updated":"2016-05-26 17:51:42.000000000","message":"Done","commit_id":"39bcc021ba2981eb259e2ac1740bd46a42689a01"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"7397cff30e7bf64e2ad060bd522ad4b801916b91","unresolved":false,"context_lines":[{"line_number":31,"context_line":"              all: Expands all swift daemons."},{"line_number":32,"context_line":"              main: Expands main swift daemons."},{"line_number":33,"context_line":"                    (proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Expands all other background daemons."},{"line_number":35,"context_line":"                    (updater, replicator, auditor, etc)"},{"line_number":36,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":37,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"}],"source_content_type":"application/octet-stream","patch_set":7,"id":"9abb7d3a_604ac527","line":34,"range":{"start_line":34,"start_character":19,"end_line":34,"end_character":57},"updated":"2016-06-01 08:36:35.000000000","message":"To be more obvious, is \u0027Expand all other background daemons except \"main\"\u0027 better, right?","commit_id":"31d8cb5df0b732d852e1afac5bf27fe0cd460c1a"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"fa14c5dda1f79742b183ec30a0ec43aec8c139ea","unresolved":false,"context_lines":[{"line_number":31,"context_line":"              all: Expands all swift daemons."},{"line_number":32,"context_line":"              main: Expands main swift daemons."},{"line_number":33,"context_line":"                    (proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Expands all other background daemons."},{"line_number":35,"context_line":"                    (updater, replicator, auditor, etc)"},{"line_number":36,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":37,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"}],"source_content_type":"application/octet-stream","patch_set":7,"id":"9abb7d3a_be7af3ba","line":34,"range":{"start_line":34,"start_character":19,"end_line":34,"end_character":57},"in_reply_to":"9abb7d3a_604ac527","updated":"2016-06-01 14:57:11.000000000","message":"Done","commit_id":"31d8cb5df0b732d852e1afac5bf27fe0cd460c1a"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"50467d148ba9b3ca9eafa2c32b12a53552bc9e3a","unresolved":false,"context_lines":[{"line_number":31,"context_line":"              all: Expands all swift daemons."},{"line_number":32,"context_line":"              main: Expands main swift daemons."},{"line_number":33,"context_line":"                    (proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Expands all other background daemons."},{"line_number":35,"context_line":"                    (updater, replicator, auditor, etc)"},{"line_number":36,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":37,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"}],"source_content_type":"application/octet-stream","patch_set":7,"id":"9abb7d3a_9e70cf73","line":34,"range":{"start_line":34,"start_character":19,"end_line":34,"end_character":57},"in_reply_to":"9abb7d3a_604ac527","updated":"2016-06-01 14:56:48.000000000","message":"Yeah, sounds good. Will update it in a while.\nThanks for the review.","commit_id":"31d8cb5df0b732d852e1afac5bf27fe0cd460c1a"},{"author":{"_account_id":18838,"name":"Paul Dardeau","email":"paul.dardeau@intel.com","username":"pdardeau"},"change_message_id":"b76e27b6072f4f118bb41475cd058ca5a24a6d7f","unresolved":false,"context_lines":[{"line_number":31,"context_line":"              all: Expands all swift daemons."},{"line_number":32,"context_line":"              main: Expands main swift daemons."},{"line_number":33,"context_line":"                    (proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Expands all other background daemons except \"main\"."},{"line_number":35,"context_line":"                    (updater, replicator, auditor, etc)"},{"line_number":36,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":37,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"9abb7d3a_b2406708","line":34,"updated":"2016-06-01 15:01:49.000000000","message":"I know the intent, but I think it could be worded slightly different to be more clear.  Maybe \"Expands all remaining background daemons (beyond \"main\").\"","commit_id":"f124a6c69d72b0cff09237cf747debd1ffb20718"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"b5bb754c710812c528f8f8d79aaca213060a7d93","unresolved":false,"context_lines":[{"line_number":31,"context_line":"              all: Expands all swift daemons."},{"line_number":32,"context_line":"              main: Expands main swift daemons."},{"line_number":33,"context_line":"                    (proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Expands all other background daemons except \"main\"."},{"line_number":35,"context_line":"                    (updater, replicator, auditor, etc)"},{"line_number":36,"context_line":"    \u003cconfig\u003e  is an explicit configuration filename without the"},{"line_number":37,"context_line":"              .conf extension. If \u003cconfig\u003e is specified then \u003cserver\u003e should"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"9abb7d3a_158b1d69","line":34,"in_reply_to":"9abb7d3a_b2406708","updated":"2016-06-01 15:16:23.000000000","message":"Done","commit_id":"f124a6c69d72b0cff09237cf747debd1ffb20718"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"7d2653c1dc1bf94abf39c092ee9b0ca5e2a64f50","unresolved":false,"context_lines":[{"line_number":26,"context_line":"where:"},{"line_number":27,"context_line":"    \u003cserver\u003e  is the name of a swift service e.g. proxy-server."},{"line_number":28,"context_line":"              The \u0027-server\u0027 part of the name may be omitted."},{"line_number":29,"context_line":"              \u0027all\u0027, \u0027main\u0027 and \u0027rest\u0027 are the reserve words that represent a"},{"line_number":30,"context_line":"              group of services."},{"line_number":31,"context_line":"              all: Expands all swift daemons."},{"line_number":32,"context_line":"              main: Expands main swift daemons."}],"source_content_type":"application/octet-stream","patch_set":9,"id":"7aa08908_dfb12be3","line":29,"updated":"2016-06-07 09:31:06.000000000","message":"s/the reserve/reserved/","commit_id":"83df4b47999db8eef443bd5eb5f533014b3470ee"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"882580c3699d1c2a5a728f615c46c759369ea7f3","unresolved":false,"context_lines":[{"line_number":26,"context_line":"where:"},{"line_number":27,"context_line":"    \u003cserver\u003e  is the name of a swift service e.g. proxy-server."},{"line_number":28,"context_line":"              The \u0027-server\u0027 part of the name may be omitted."},{"line_number":29,"context_line":"              \u0027all\u0027, \u0027main\u0027 and \u0027rest\u0027 are the reserve words that represent a"},{"line_number":30,"context_line":"              group of services."},{"line_number":31,"context_line":"              all: Expands all swift daemons."},{"line_number":32,"context_line":"              main: Expands main swift daemons."}],"source_content_type":"application/octet-stream","patch_set":9,"id":"1aa78d24_613414de","line":29,"in_reply_to":"7aa08908_dfb12be3","updated":"2016-07-08 16:25:02.000000000","message":"Done","commit_id":"83df4b47999db8eef443bd5eb5f533014b3470ee"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"7d2653c1dc1bf94abf39c092ee9b0ca5e2a64f50","unresolved":false,"context_lines":[{"line_number":28,"context_line":"              The \u0027-server\u0027 part of the name may be omitted."},{"line_number":29,"context_line":"              \u0027all\u0027, \u0027main\u0027 and \u0027rest\u0027 are the reserve words that represent a"},{"line_number":30,"context_line":"              group of services."},{"line_number":31,"context_line":"              all: Expands all swift daemons."},{"line_number":32,"context_line":"              main: Expands main swift daemons."},{"line_number":33,"context_line":"                    (proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Expands all remaining background daemons (beyond \"main\")."}],"source_content_type":"application/octet-stream","patch_set":9,"id":"7aa08908_d1abdfd4","line":31,"updated":"2016-06-07 09:31:06.000000000","message":"I like the use of \u0027Expands\u0027 but I think it should read \u0027Expands to ...\u0027 in each case all/main/rest. If someone has a different understanding of the grammar then please correct me.\n\nSee for example the usage of \u0027expands to\u0027 on this page https://www.gnu.org/software/bash/manual/html_node/Shell-Parameter-Expansion.html.","commit_id":"83df4b47999db8eef443bd5eb5f533014b3470ee"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"882580c3699d1c2a5a728f615c46c759369ea7f3","unresolved":false,"context_lines":[{"line_number":28,"context_line":"              The \u0027-server\u0027 part of the name may be omitted."},{"line_number":29,"context_line":"              \u0027all\u0027, \u0027main\u0027 and \u0027rest\u0027 are the reserve words that represent a"},{"line_number":30,"context_line":"              group of services."},{"line_number":31,"context_line":"              all: Expands all swift daemons."},{"line_number":32,"context_line":"              main: Expands main swift daemons."},{"line_number":33,"context_line":"                    (proxy, container, account, object)"},{"line_number":34,"context_line":"              rest: Expands all remaining background daemons (beyond \"main\")."}],"source_content_type":"application/octet-stream","patch_set":9,"id":"1aa78d24_413118ec","line":31,"in_reply_to":"7aa08908_d1abdfd4","updated":"2016-07-08 16:25:02.000000000","message":"Done","commit_id":"83df4b47999db8eef443bd5eb5f533014b3470ee"}]}