)]}'
{"specs/add-service-tags.rst":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"11b84c796c3a17c069938fd460e49db49e793021","unresolved":false,"context_lines":[{"line_number":3,"context_line":" This work is licensed under a Creative Commons Attribution 3.0 Unported"},{"line_number":4,"context_line":" License."},{"line_number":5,"context_line":""},{"line_number":6,"context_line":" http://creativecommons.org/licenses/by/3.0/legalcode"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":".."},{"line_number":9,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"AAAAWn%2F%2F9hM%3D","line":6,"updated":"2014-04-08 08:30:40.000000000","message":"This change seems out of scope.","commit_id":"fc58550b08b959e5d042674c820bde82aa759af0"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"5c6793da571a10f1c8e880476137d43f529c2042","unresolved":false,"context_lines":[{"line_number":3,"context_line":" This work is licensed under a Creative Commons Attribution 3.0 Unported"},{"line_number":4,"context_line":" License."},{"line_number":5,"context_line":""},{"line_number":6,"context_line":" http://creativecommons.org/licenses/by/3.0/legalcode"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":".."},{"line_number":9,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"AAAAWn%2F%2F5M8%3D","line":6,"in_reply_to":"AAAAWn%2F%2F9hM%3D","updated":"2014-04-08 17:12:41.000000000","message":"Nice catch, I screwed up my rebase. Done","commit_id":"fc58550b08b959e5d042674c820bde82aa759af0"}],"specs/config-verification.rst":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"eb5e5881f6c22ed7d6aad01e99c97480b6b12346","unresolved":false,"context_lines":[{"line_number":38,"context_line":"This new tool will be added to the tools/ directory in the root of the tempest"},{"line_number":39,"context_line":"tree along with the other helper utilities in tempest."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"The usage statement for the new tool will be something like the following::"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    usage: verify_tempest_config.py [-h] [-u]"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"AAAAWH%2F9mvU%3D","line":41,"updated":"2014-04-03 12:10:08.000000000","message":"If detecting something wrong or needing to update config with -u option, the command return 1(not 0) as the exit code?","commit_id":"3d90921018813f251ad61a51c1a80e9fb7ba976b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"982e3a7df108fd78919c38d40cf4fb2df1c17084","unresolved":false,"context_lines":[{"line_number":38,"context_line":"This new tool will be added to the tools/ directory in the root of the tempest"},{"line_number":39,"context_line":"tree along with the other helper utilities in tempest."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"The usage statement for the new tool will be something like the following::"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    usage: verify_tempest_config.py [-h] [-u]"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"AAAAWH%2F9fnY%3D","line":41,"in_reply_to":"AAAAWH%2F9h2o%3D","updated":"2014-04-04 06:00:55.000000000","message":"I got it, the behavior also is nice. I wanted just to clarify it. thanks.","commit_id":"3d90921018813f251ad61a51c1a80e9fb7ba976b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"e74590ca215b64cf2b8898b20a6774832608150a","unresolved":false,"context_lines":[{"line_number":38,"context_line":"This new tool will be added to the tools/ directory in the root of the tempest"},{"line_number":39,"context_line":"tree along with the other helper utilities in tempest."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"The usage statement for the new tool will be something like the following::"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    usage: verify_tempest_config.py [-h] [-u]"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"AAAAWH%2F9h2o%3D","line":41,"in_reply_to":"AAAAWH%2F9mvU%3D","updated":"2014-04-03 21:53:51.000000000","message":"So the way the script currently operates is that it will return 0 unless there is an internal failure. (ie an api request fails or the return format isn\u0027t parsable, basically an exception is raised)\n\nI can change this behavior but my thought was that it\u0027s performing as expected even if it finds mismatches. The tool is simply informing you of how you config file differs, it might be intentional.","commit_id":"3d90921018813f251ad61a51c1a80e9fb7ba976b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"eb5e5881f6c22ed7d6aad01e99c97480b6b12346","unresolved":false,"context_lines":[{"line_number":49,"context_line":"                    the case of endpoint checks where it could either be the"},{"line_number":50,"context_line":"                    incorrect catalog type or the service available option the"},{"line_number":51,"context_line":"                    service available option is assumed to be incorrect and is"},{"line_number":52,"context_line":"                    thus changed."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Alternatives"},{"line_number":55,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"AAAAWH%2F9mwM%3D","line":52,"updated":"2014-04-03 12:10:08.000000000","message":"Can we get the config differences between before/after of this command operation?\nThat would provide good info for deployers.","commit_id":"3d90921018813f251ad61a51c1a80e9fb7ba976b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"982e3a7df108fd78919c38d40cf4fb2df1c17084","unresolved":false,"context_lines":[{"line_number":49,"context_line":"                    the case of endpoint checks where it could either be the"},{"line_number":50,"context_line":"                    incorrect catalog type or the service available option the"},{"line_number":51,"context_line":"                    service available option is assumed to be incorrect and is"},{"line_number":52,"context_line":"                    thus changed."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Alternatives"},{"line_number":55,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"AAAAWH%2F9fn8%3D","line":52,"in_reply_to":"AAAAWH%2F9h1k%3D","updated":"2014-04-04 06:00:55.000000000","message":"\u003e So with or without the -u the script will print out the differences it finds.\n\nNice behavior:)\n\n \u003e If that\u0027s insufficient I can add a \u0027-o\u0027 flag (or something similar) that\n \u003e will make an updated copy instead of overwriting the config file.\n\nor default behavior is creating original file(tempest.conf.orig etc) and some option overwrites it without any copy file like \"patch\" command.","commit_id":"3d90921018813f251ad61a51c1a80e9fb7ba976b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"e74590ca215b64cf2b8898b20a6774832608150a","unresolved":false,"context_lines":[{"line_number":49,"context_line":"                    the case of endpoint checks where it could either be the"},{"line_number":50,"context_line":"                    incorrect catalog type or the service available option the"},{"line_number":51,"context_line":"                    service available option is assumed to be incorrect and is"},{"line_number":52,"context_line":"                    thus changed."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Alternatives"},{"line_number":55,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"AAAAWH%2F9h1k%3D","line":52,"in_reply_to":"AAAAWH%2F9mwM%3D","updated":"2014-04-03 21:53:51.000000000","message":"So with or without the -u the script will print out the differences it finds. If that\u0027s insufficient I can add a \u0027-o\u0027 flag (or something similar) that will make an updated copy instead of overwriting the config file.","commit_id":"3d90921018813f251ad61a51c1a80e9fb7ba976b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"11b84c796c3a17c069938fd460e49db49e793021","unresolved":false,"context_lines":[{"line_number":52,"context_line":"                    thus changed. A copy of the original config file will be"},{"line_number":53,"context_line":"                    created with .orig appended to the filename."},{"line_number":54,"context_line":"      -n, --nocopy  Don\u0027t create a copy of original config file when running"},{"line_number":55,"context_line":"                    with the update option."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"To test the functionality of the tool unit tests will be added to verify that"},{"line_number":58,"context_line":"both the verification functionality and the config file updating work as"}],"source_content_type":"text/x-rst","patch_set":7,"id":"AAAAWn%2F%2F9gw%3D","line":55,"updated":"2014-04-08 08:30:40.000000000","message":"I like it, thanks.","commit_id":"fc58550b08b959e5d042674c820bde82aa759af0"}]}