)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"3d3e61ce7f70342c9bc58a3be215038709908423","unresolved":false,"context_lines":[{"line_number":19,"context_line":"which list paginations we want to check with the same option."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Blueprint snapshots-and-bakcup-support-pagination-query"},{"line_number":22,"context_line":"Change-Id: Ie854ff58540a8e7d88d41a2c4074ca49b8fe67df"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ba3cc151_31040f14","line":22,"updated":"2015-06-29 09:45:29.000000000","message":"You should add a \nDepends-On: I21edac6d29abf7fc2f446111a7d6888db9ab6eba \nline","commit_id":"a403db9549c0ed9b7e42ecefe6ce90bbc3b56fb2"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c7cba28c0af00b8a6a1d9b036e93c64923bef7fb","unresolved":false,"context_lines":[{"line_number":19,"context_line":"which list paginations we want to check with the same option."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Blueprint snapshots-and-bakcup-support-pagination-query"},{"line_number":22,"context_line":"Change-Id: Ie854ff58540a8e7d88d41a2c4074ca49b8fe67df"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9a41bdd9_5c01fead","line":22,"in_reply_to":"ba3cc151_31040f14","updated":"2015-07-15 11:57:46.000000000","message":"Done","commit_id":"a403db9549c0ed9b7e42ecefe6ce90bbc3b56fb2"}],"etc/tempest.conf.sample":[{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"0c485f0d62ef039158da3f2a21e1153f6c62c056","unresolved":false,"context_lines":[{"line_number":1192,"context_line":""},{"line_number":1193,"context_line":"# List of resources for which to check v2 list pagination. Currently"},{"line_number":1194,"context_line":"# only 1 value is acceptable, \u0027snapshots\u0027. Example:  available"},{"line_number":1195,"context_line":"# list_pagination \u003d [\u0027snapshots\u0027] (list value)"},{"line_number":1196,"context_line":"#list_pagination \u003d"},{"line_number":1197,"context_line":""},{"line_number":1198,"context_line":"# A list of enabled volume extensions with a special entry all which"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa32b979_148eaf78","line":1195,"updated":"2015-06-25 16:21:00.000000000","message":"Hi Gorka, \nCould you set any empy value to list_pagination , its a list type.\nmaybe list_pagination \u003d []  \nBecause if its an empy string its \u003d \u0027\u0027\n\nWhat do you think ?\n\nBenny","commit_id":"a403db9549c0ed9b7e42ecefe6ce90bbc3b56fb2"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c2f78ddb796c82f443bcd86e5d39fbe52334ae8c","unresolved":false,"context_lines":[{"line_number":1192,"context_line":""},{"line_number":1193,"context_line":"# List of resources for which to check v2 list pagination. Currently"},{"line_number":1194,"context_line":"# only 1 value is acceptable, \u0027snapshots\u0027. Example:  available"},{"line_number":1195,"context_line":"# list_pagination \u003d [\u0027snapshots\u0027] (list value)"},{"line_number":1196,"context_line":"#list_pagination \u003d"},{"line_number":1197,"context_line":""},{"line_number":1198,"context_line":"# A list of enabled volume extensions with a special entry all which"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa32b979_eb2801f1","line":1195,"in_reply_to":"fa32b979_148eaf78","updated":"2015-06-25 19:57:50.000000000","message":"I had the same thought, unfortunately that is not an option.\n\nSince this is only my second Tempest patch, and the first that uses configuration options, I initially did try  with `#list_pagination \u003d [\u0027snapshots\u0027]` and then with `#list_pagination \u003d []`  but if you do that, `tox -epep8` fails at the gate (and locally), as you can see here: http://logs.openstack.org/72/195072/2/check/gate-tempest-pep8/e4b0554/console.html#_2015-06-24_13_48_14_822\n\nSo this file needs to be autogenerated for pep8 tox test to pass.","commit_id":"a403db9549c0ed9b7e42ecefe6ce90bbc3b56fb2"}],"tempest/config.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"3d3e61ce7f70342c9bc58a3be215038709908423","unresolved":false,"context_lines":[{"line_number":669,"context_line":"                default\u003d[],"},{"line_number":670,"context_line":"                help\u003d\"List of resources for which to check v2 list pagination.\""},{"line_number":671,"context_line":"                     \" Currently only 1 value is acceptable, \u0027snapshots\u0027. \""},{"line_number":672,"context_line":"                     \"Example:  available list_pagination \u003d [\u0027snapshots\u0027]\"),"},{"line_number":673,"context_line":"    cfg.ListOpt(\u0027api_extensions\u0027,"},{"line_number":674,"context_line":"                default\u003d[\u0027all\u0027],"},{"line_number":675,"context_line":"                help\u003d\u0027A list of enabled volume extensions with a special \u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba3cc151_960cf143","line":672,"updated":"2015-06-29 09:45:29.000000000","message":"I think you should remove the square bracket. In Tempest.conf\nlist_pagination \u003d [\u0027snapshots\u0027] will be incorrect but list_pagination \u003d \u0027snapshots\u0027 is correct.","commit_id":"a403db9549c0ed9b7e42ecefe6ce90bbc3b56fb2"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c7cba28c0af00b8a6a1d9b036e93c64923bef7fb","unresolved":false,"context_lines":[{"line_number":669,"context_line":"                default\u003d[],"},{"line_number":670,"context_line":"                help\u003d\"List of resources for which to check v2 list pagination.\""},{"line_number":671,"context_line":"                     \" Currently only 1 value is acceptable, \u0027snapshots\u0027. \""},{"line_number":672,"context_line":"                     \"Example:  available list_pagination \u003d [\u0027snapshots\u0027]\"),"},{"line_number":673,"context_line":"    cfg.ListOpt(\u0027api_extensions\u0027,"},{"line_number":674,"context_line":"                default\u003d[\u0027all\u0027],"},{"line_number":675,"context_line":"                help\u003d\u0027A list of enabled volume extensions with a special \u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a41bdd9_3c899a75","line":672,"in_reply_to":"ba3cc151_960cf143","updated":"2015-07-15 11:57:46.000000000","message":"I don\u0027t understand what you mean, this is a list option and default should be an empty list.\n\nLater on, when we have backup pagination, a new value will be available, \u0027backups\u0027, and we may want to test both paginations.","commit_id":"a403db9549c0ed9b7e42ecefe6ce90bbc3b56fb2"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"000bff286ef26ecf3541e4aebf799c65eacd0edf","unresolved":false,"context_lines":[{"line_number":672,"context_line":"                default\u003d[],"},{"line_number":673,"context_line":"                help\u003d\"List of resources for which to check v2 list pagination.\""},{"line_number":674,"context_line":"                     \" Currently only 1 value is acceptable, \u0027snapshots\u0027. \""},{"line_number":675,"context_line":"                     \"Example:  available list_pagination \u003d [\u0027snapshots\u0027]\"),"},{"line_number":676,"context_line":"    cfg.ListOpt(\u0027api_extensions\u0027,"},{"line_number":677,"context_line":"                default\u003d[\u0027all\u0027],"},{"line_number":678,"context_line":"                help\u003d\u0027A list of enabled volume extensions with a special \u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a41bdd9_a7f4b25a","line":675,"updated":"2015-07-15 13:44:20.000000000","message":"What I mean is the example you give here is wrong. If a edit my tempest.conf file and set list_pagination \u003d [\u0027snapshots\u0027] it will not work. The correct syntax is list_pagination \u003d \u0027snapshots\u0027","commit_id":"0229b9569ab2af649c201ff2dea7b57f6c117ed7"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"b3278b8b4c2dae0c0000efec9b703c0d523064ec","unresolved":false,"context_lines":[{"line_number":672,"context_line":"                default\u003d[],"},{"line_number":673,"context_line":"                help\u003d\"List of resources for which to check v2 list pagination.\""},{"line_number":674,"context_line":"                     \" Currently only 1 value is acceptable, \u0027snapshots\u0027. \""},{"line_number":675,"context_line":"                     \"Example:  available list_pagination \u003d [\u0027snapshots\u0027]\"),"},{"line_number":676,"context_line":"    cfg.ListOpt(\u0027api_extensions\u0027,"},{"line_number":677,"context_line":"                default\u003d[\u0027all\u0027],"},{"line_number":678,"context_line":"                help\u003d\u0027A list of enabled volume extensions with a special \u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"3a50d1a3_f1dd75e0","line":675,"in_reply_to":"9a41bdd9_a7f4b25a","updated":"2015-07-29 11:28:29.000000000","message":"You are totally right!!!  Sorry for not understanding you earlier.","commit_id":"0229b9569ab2af649c201ff2dea7b57f6c117ed7"}]}
