)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"53fb3507d473722ee35afdcaf64d25b62f34f090","unresolved":false,"context_lines":[{"line_number":37,"context_line":"29036  swift-container-auditor /etc/swift/container-server/1.conf"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"So we are looking at saving at most ~350Mo of RAM (could be less"},{"line_number":40,"context_line":"because RSS doesn\u0027t account for shared memory)."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Change-Id: I8a0d03ac0296a74e38efd185beb8513866eaf0c4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ffe62b97_f00573c3","line":40,"updated":"2017-03-27 19:10:58.000000000","message":"350MB of memory doesn\u0027t seem like much savings. How much is it using altogether? If we\u0027re within 350MB of OOM on the box, seems like more significant steps may need to be taken.","commit_id":"c659e90cfae06cb533f27ebe387ed0f4c79f5dff"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"78690f724b6cac45419f89f445225086ed8c23aa","unresolved":false,"context_lines":[{"line_number":37,"context_line":"29036  swift-container-auditor /etc/swift/container-server/1.conf"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"So we are looking at saving at most ~350Mo of RAM (could be less"},{"line_number":40,"context_line":"because RSS doesn\u0027t account for shared memory)."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Change-Id: I8a0d03ac0296a74e38efd185beb8513866eaf0c4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ffe62b97_d0e02fcf","line":40,"in_reply_to":"ffe62b97_f00573c3","updated":"2017-03-27 19:20:31.000000000","message":"We are close to OOM on the box: https://bugs.launchpad.net/neutron/+bug/1656386\n\nThere\u0027s a whole effort to try to bring down memory consumption: https://review.openstack.org/#/c/446741/ https://review.openstack.org/#/c/447119/\n\nSo 350 Mo, it\u0027s something.","commit_id":"c659e90cfae06cb533f27ebe387ed0f4c79f5dff"}],"lib/swift":[{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"53fb3507d473722ee35afdcaf64d25b62f34f090","unresolved":false,"context_lines":[{"line_number":803,"context_line":""},{"line_number":804,"context_line":"        for type in ${todo}; do"},{"line_number":805,"context_line":"            run_process s-${type} \"swift-${type}-server ${SWIFT_CONF_DIR}/${type}-server/1.conf -v\""},{"line_number":806,"context_line":"        done"},{"line_number":807,"context_line":"        # The container-sync daemon is needed run the container sync Tempest tests."},{"line_number":808,"context_line":"        swift-init --run-dir\u003d${SWIFT_DATA_DIR}/run container-sync start"},{"line_number":809,"context_line":"    else"}],"source_content_type":"text/x-swift","patch_set":1,"id":"ffe62b97_b0092b2d","line":806,"updated":"2017-03-27 19:10:58.000000000","message":"This will work.\n\nBut I\u0027d suggest either using `swift-init main start` (for proxy, account, container, object) or `swift-init \u003ctype\u003e start` for each thing individually. However, this may affect how you\u0027re capturing logs (ie check syslog instead of stdout).","commit_id":"c659e90cfae06cb533f27ebe387ed0f4c79f5dff"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"78690f724b6cac45419f89f445225086ed8c23aa","unresolved":false,"context_lines":[{"line_number":803,"context_line":""},{"line_number":804,"context_line":"        for type in ${todo}; do"},{"line_number":805,"context_line":"            run_process s-${type} \"swift-${type}-server ${SWIFT_CONF_DIR}/${type}-server/1.conf -v\""},{"line_number":806,"context_line":"        done"},{"line_number":807,"context_line":"        # The container-sync daemon is needed run the container sync Tempest tests."},{"line_number":808,"context_line":"        swift-init --run-dir\u003d${SWIFT_DATA_DIR}/run container-sync start"},{"line_number":809,"context_line":"    else"}],"source_content_type":"text/x-swift","patch_set":1,"id":"ffe62b97_703f2325","line":806,"in_reply_to":"ffe62b97_b0092b2d","updated":"2017-03-27 19:20:31.000000000","message":"yeah but, afaik, swift-init launches the processes in the background (so the logs end up in /var/log/messages). run_process starts the process in a screen, with logs going into a per-service log file.","commit_id":"c659e90cfae06cb533f27ebe387ed0f4c79f5dff"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"53fb3507d473722ee35afdcaf64d25b62f34f090","unresolved":false,"context_lines":[{"line_number":815,"context_line":"        local proxy_port\u003d${SWIFT_DEFAULT_BIND_PORT}"},{"line_number":816,"context_line":"        start_tls_proxy swift \u0027*\u0027 $proxy_port $SERVICE_HOST $SWIFT_DEFAULT_BIND_PORT_INT"},{"line_number":817,"context_line":"    fi"},{"line_number":818,"context_line":"    run_process s-proxy \"swift-proxy-server ${SWIFT_CONF_DIR}/proxy-server.conf -v\""},{"line_number":819,"context_line":""},{"line_number":820,"context_line":"    if [[ \"$SWIFT_ENABLE_TEMPURLS\" \u003d\u003d \"True\" ]]; then"},{"line_number":821,"context_line":"        swift_configure_tempurls"}],"source_content_type":"text/x-swift","patch_set":1,"id":"ffe62b97_50377fe4","line":818,"updated":"2017-03-27 19:10:58.000000000","message":"Like above, I\u0027d suggest using `swift-init` instead of directly starting it this way.","commit_id":"c659e90cfae06cb533f27ebe387ed0f4c79f5dff"}]}
