)]}'
{"id":"openstack%2Fpython-cinderclient~62593","triplet_id":"openstack%2Fpython-cinderclient~master~I7c389fbe8a5b0fcc83dc4567da425c5e0e64f777","project":"openstack/python-cinderclient","branch":"master","topic":"bug/1252665","hashtags":[],"change_id":"I7c389fbe8a5b0fcc83dc4567da425c5e0e64f777","subject":"Fix \u0027search_opts\u0027 unexpected keyword argument error","status":"ABANDONED","created":"2013-12-17 10:11:46.000000000","updated":"2014-02-12 08:54:37.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"1dccc0c6daa991ca90709b7a4f4a86b3a28ab3ee","_number":62593,"virtual_id_number":62593,"owner":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":7138,"name":"David Jia","email":"jiataotj@cn.ibm.com","username":"jiataotj"},{"value":-1,"date":"2014-02-11 06:41:54.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7764,"name":"Peter Hamilton","email":"peter.hamilton@jhuapl.edu","username":"Peter"},{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"recommended":{"_account_id":7138,"name":"David Jia","email":"jiataotj@cn.ibm.com","username":"jiataotj"},"all":[{"value":1,"date":"2014-02-11 06:41:14.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7138,"name":"David Jia","email":"jiataotj@cn.ibm.com","username":"jiataotj"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7764,"name":"Peter Hamilton","email":"peter.hamilton@jhuapl.edu","username":"Peter"},{"value":0,"date":"2013-12-19 07:47:58.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":1,"default_value":0,"optional":true},"Workflow":{"all":[{"date":"2014-01-20 03:01:43.000000000","_account_id":7138,"name":"David Jia","email":"jiataotj@cn.ibm.com","username":"jiataotj"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7764,"name":"Peter Hamilton","email":"peter.hamilton@jhuapl.edu","username":"Peter"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"_account_id":7138,"name":"David Jia","email":"jiataotj@cn.ibm.com","username":"jiataotj"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7764,"name":"Peter Hamilton","email":"peter.hamilton@jhuapl.edu","username":"Peter"},{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"_account_id":7138,"name":"David Jia","email":"jiataotj@cn.ibm.com","username":"jiataotj"},{"_account_id":7764,"name":"Peter Hamilton","email":"peter.hamilton@jhuapl.edu","username":"Peter"},{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-12-17 12:25:15.000000000","updated_by":{"_account_id":7764,"name":"Peter Hamilton","email":"peter.hamilton@jhuapl.edu","username":"Peter"},"reviewer":{"_account_id":7764,"name":"Peter Hamilton","email":"peter.hamilton@jhuapl.edu","username":"Peter"},"state":"REVIEWER"},{"updated":"2013-12-18 17:00:36.000000000","updated_by":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"reviewer":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"state":"REVIEWER"},{"updated":"2014-01-20 03:01:43.000000000","updated_by":{"_account_id":7138,"name":"David Jia","email":"jiataotj@cn.ibm.com","username":"jiataotj"},"reviewer":{"_account_id":7138,"name":"David Jia","email":"jiataotj@cn.ibm.com","username":"jiataotj"},"state":"REVIEWER"},{"updated":"2014-02-11 06:41:54.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"770ceb48b8c8a6e5f6233fba0add7488d63f14bb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-17 10:47:45.000000000","message":"Patch Set 1: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/93/62593/1/check/gate-python-cinderclient-pep8/8de2a5c : SUCCESS in 44s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/93/62593/1/check/gate-python-cinderclient-docs/7709550/doc/build/html/ : SUCCESS in 45s\n- gate-python-cinderclient-python26 http://logs.openstack.org/93/62593/1/check/gate-python-cinderclient-python26/f23a689 : SUCCESS in 54s\n- gate-python-cinderclient-python27 http://logs.openstack.org/93/62593/1/check/gate-python-cinderclient-python27/c5650c9 : SUCCESS in 55s\n- gate-python-cinderclient-python33 http://logs.openstack.org/93/62593/1/check/gate-python-cinderclient-python33/9a4db47 : SUCCESS in 4m 02s\n- gate-python-cinderclient-pypy http://logs.openstack.org/93/62593/1/check/gate-python-cinderclient-pypy/d6bd305 : SUCCESS in 1m 40s\n- check-tempest-dsvm-full http://logs.openstack.org/93/62593/1/check/check-tempest-dsvm-full/15f0a23 : FAILURE in 35m 43s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/93/62593/1/check/check-tempest-dsvm-postgres-full/ccb0137 : SUCCESS in 34m 38s\n- check-tempest-dsvm-neutron http://logs.openstack.org/93/62593/1/check/check-tempest-dsvm-neutron/2439b67 : SUCCESS in 32m 46s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/93/62593/1/check/gate-tempest-dsvm-large-ops/a33fd33 : SUCCESS in 14m 08s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/93/62593/1/check/gate-tempest-dsvm-neutron-large-ops/6b617e2 : SUCCESS in 15m 40s\n- check-grenade-dsvm http://logs.openstack.org/93/62593/1/check/check-grenade-dsvm/2b84d90 : SUCCESS in 28m 23s\n","accounts_in_message":[],"_revision_number":1},{"id":"7c28556f9fc3c50a7af479d62634c0e3a0aa6ca7","author":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"date":"2013-12-18 01:38:08.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"f39c157de1a45eb7a29ccf18416a44c1d1a8e7ab","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-12-18 01:38:14.000000000","message":"Patch Set 2:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":2},{"id":"9a403fd181647434588e3804a2d73a332c336ba1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-18 02:18:18.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/93/62593/2/check/gate-python-cinderclient-pep8/c1466e7 : SUCCESS in 41s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/93/62593/2/check/gate-python-cinderclient-docs/b4c0bf7/doc/build/html/ : SUCCESS in 47s\n- gate-python-cinderclient-python26 http://logs.openstack.org/93/62593/2/check/gate-python-cinderclient-python26/77fd783 : SUCCESS in 54s\n- gate-python-cinderclient-python27 http://logs.openstack.org/93/62593/2/check/gate-python-cinderclient-python27/d2dcc66 : SUCCESS in 51s\n- gate-python-cinderclient-python33 http://logs.openstack.org/93/62593/2/check/gate-python-cinderclient-python33/b8b35be : SUCCESS in 4m 01s\n- gate-python-cinderclient-pypy http://logs.openstack.org/93/62593/2/check/gate-python-cinderclient-pypy/f5125ba : SUCCESS in 1m 40s\n- check-tempest-dsvm-full http://logs.openstack.org/93/62593/2/check/check-tempest-dsvm-full/3562fb4 : SUCCESS in 39m 27s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/93/62593/2/check/check-tempest-dsvm-postgres-full/f039419 : SUCCESS in 39m 11s\n- check-tempest-dsvm-neutron http://logs.openstack.org/93/62593/2/check/check-tempest-dsvm-neutron/429fe84 : SUCCESS in 40m 05s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/93/62593/2/check/gate-tempest-dsvm-large-ops/a370164 : SUCCESS in 13m 56s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/93/62593/2/check/gate-tempest-dsvm-neutron-large-ops/adae712 : SUCCESS in 15m 36s\n- check-grenade-dsvm http://logs.openstack.org/93/62593/2/check/check-grenade-dsvm/654f401 : SUCCESS in 29m 35s\n","accounts_in_message":[],"_revision_number":2},{"id":"13c4de9376ef277d385a63348181330675fb813b","author":{"_account_id":6802,"name":"Joel Coffman","email":"jmc7tp@gmail.com","username":"joel-coffman"},"date":"2013-12-18 12:54:46.000000000","message":"Patch Set 2:\n\nI think that this patch does address the symptoms of the bug, but I\u0027m not sure that it\u0027s the best solution. Another possibility is changing the signature of all existing managers so that list(...) accepts kwargs, which seems clearer and easier to maintain in my opinion.","accounts_in_message":[],"_revision_number":2},{"id":"814bfd52916a242654c76249c2e6c0c4dbc5597a","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2013-12-18 17:00:36.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\nI would lean toward fixing this in VolumeEncryptionTimeManager rather than in the base class.\n\nSee bug 1252665 and associated https://review.openstack.org/#/c/57404/ .\n\n(It would also be nice to add Related-Bug: #1252665 in the commit message.)","accounts_in_message":[],"_revision_number":2},{"id":"4d5b817fac8b9f761f2dc2408d09eb71ddc2c65c","author":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"date":"2013-12-19 01:34:57.000000000","message":"Patch Set 2:\n\nHi, Eric Harney\n    I had seen the patch in https://review.openstack.org/#/c/57404/. I think we do not need add unuseful argument to the Managers which will use findall() in base class, and is more reasonable to check whether the subclass function has the argument, I know novaclient use this method to avoid argument issue too, if we do not change in base class and add new Manager in future, we need fix the same issue too, I will cleanup code in https://review.openstack.org/#/c/57404/ if this patch is merged. Thanks.","accounts_in_message":[],"_revision_number":2},{"id":"b5b97de30afef0f7a15dea7955b16bbf4935fd68","author":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"date":"2013-12-19 01:36:06.000000000","message":"Patch Set 2:\n\nHi, Eric Harney\nI had seen the patch in https://review.openstack.org/#/c/57404/.I think we do not need add unuseful argument to \nthe Managers which will use findall() in base class, and is \nmore reasonable to check whether the subclass function has the\nargument, I know novaclient use this method to avoid argument \nissue too, if we do not change in base class and add new \nManager in future, we need fix the same issue too, I will \ncleanup code in https://review.openstack.org/#/c/57404/ if \nthis patch is merged. Thanks.","accounts_in_message":[],"_revision_number":2},{"id":"a69bd118571db409da30f348d80f015b896f276b","author":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"date":"2013-12-19 02:50:13.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"9172f0653731300832ef56923891adf6dfd253dc","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-12-19 02:50:20.000000000","message":"Patch Set 3:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":3},{"id":"17dbea5b5433298e1677516f2dca5b93b769e23e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-19 03:40:44.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/93/62593/3/check/gate-python-cinderclient-pep8/a212c45 : SUCCESS in 1m 11s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/93/62593/3/check/gate-python-cinderclient-docs/b9d05de/doc/build/html/ : SUCCESS in 55s\n- gate-python-cinderclient-python26 http://logs.openstack.org/93/62593/3/check/gate-python-cinderclient-python26/954deb0 : SUCCESS in 57s\n- gate-python-cinderclient-python27 http://logs.openstack.org/93/62593/3/check/gate-python-cinderclient-python27/27bbf74 : SUCCESS in 44s\n- gate-python-cinderclient-python33 http://logs.openstack.org/93/62593/3/check/gate-python-cinderclient-python33/d2e32c2 : SUCCESS in 4m 02s\n- gate-python-cinderclient-pypy http://logs.openstack.org/93/62593/3/check/gate-python-cinderclient-pypy/9d3cc6c : SUCCESS in 1m 41s\n- check-tempest-dsvm-full http://logs.openstack.org/93/62593/3/check/check-tempest-dsvm-full/64d1db9 : SUCCESS in 50m 25s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/93/62593/3/check/check-tempest-dsvm-postgres-full/fb9fa2d : SUCCESS in 38m 24s\n- check-tempest-dsvm-neutron http://logs.openstack.org/93/62593/3/check/check-tempest-dsvm-neutron/f356b7b : SUCCESS in 31m 23s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/93/62593/3/check/gate-tempest-dsvm-large-ops/2d4f217 : SUCCESS in 14m 01s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/93/62593/3/check/gate-tempest-dsvm-neutron-large-ops/0244be6 : SUCCESS in 15m 00s\n- check-grenade-dsvm http://logs.openstack.org/93/62593/3/check/check-grenade-dsvm/f45a46f : SUCCESS in 31m 14s\n","accounts_in_message":[],"_revision_number":3},{"id":"ad194d40a681054be221f8d30aef38038e57798b","author":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"date":"2013-12-19 05:49:16.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"cf8551c41bc666f4f0bf9e96c6d904c24aeb724e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-19 06:22:02.000000000","message":"Patch Set 4: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/93/62593/4/check/gate-python-cinderclient-pep8/c223c3c : SUCCESS in 43s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/93/62593/4/check/gate-python-cinderclient-docs/87d76c8/doc/build/html/ : SUCCESS in 44s\n- gate-python-cinderclient-python26 http://logs.openstack.org/93/62593/4/check/gate-python-cinderclient-python26/c886b48 : SUCCESS in 52s\n- gate-python-cinderclient-python27 http://logs.openstack.org/93/62593/4/check/gate-python-cinderclient-python27/dcc5b3c : SUCCESS in 46s\n- gate-python-cinderclient-python33 http://logs.openstack.org/93/62593/4/check/gate-python-cinderclient-python33/76d3309 : SUCCESS in 4m 04s\n- gate-python-cinderclient-pypy http://logs.openstack.org/93/62593/4/check/gate-python-cinderclient-pypy/36cf02d : SUCCESS in 1m 44s\n- check-tempest-dsvm-full http://logs.openstack.org/93/62593/4/check/check-tempest-dsvm-full/289a117 : FAILURE in 24m 04s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/93/62593/4/check/check-tempest-dsvm-postgres-full/5459474 : FAILURE in 22m 28s\n- check-tempest-dsvm-neutron http://logs.openstack.org/93/62593/4/check/check-tempest-dsvm-neutron/b534a78 : SUCCESS in 32m 24s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/93/62593/4/check/gate-tempest-dsvm-large-ops/fc488f5 : SUCCESS in 14m 12s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/93/62593/4/check/gate-tempest-dsvm-neutron-large-ops/baae459 : SUCCESS in 16m 39s\n- check-grenade-dsvm http://logs.openstack.org/93/62593/4/check/check-grenade-dsvm/862856f : FAILURE in 21m 20s\n","accounts_in_message":[],"_revision_number":4},{"id":"e2d19d8d32b790e18a4477864c115cfa85a2bc22","author":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"date":"2013-12-19 07:03:50.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"570ef322fb507e641713d9de871c07081fee5f86","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-19 07:43:18.000000000","message":"Patch Set 5: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/93/62593/5/check/gate-python-cinderclient-pep8/e241573 : SUCCESS in 42s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/93/62593/5/check/gate-python-cinderclient-docs/ceb44b7/doc/build/html/ : SUCCESS in 41s\n- gate-python-cinderclient-python26 http://logs.openstack.org/93/62593/5/check/gate-python-cinderclient-python26/41de82c : SUCCESS in 53s\n- gate-python-cinderclient-python27 http://logs.openstack.org/93/62593/5/check/gate-python-cinderclient-python27/494a23d : SUCCESS in 47s\n- gate-python-cinderclient-python33 http://logs.openstack.org/93/62593/5/check/gate-python-cinderclient-python33/71e7461 : SUCCESS in 4m 01s\n- gate-python-cinderclient-pypy http://logs.openstack.org/93/62593/5/check/gate-python-cinderclient-pypy/7f2efad : SUCCESS in 1m 41s\n- check-tempest-dsvm-full http://logs.openstack.org/93/62593/5/check/check-tempest-dsvm-full/16e7098 : SUCCESS in 39m 22s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/93/62593/5/check/check-tempest-dsvm-postgres-full/3b2535a : FAILURE in 38m 24s\n- check-tempest-dsvm-neutron http://logs.openstack.org/93/62593/5/check/check-tempest-dsvm-neutron/ed803d4 : SUCCESS in 35m 16s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/93/62593/5/check/gate-tempest-dsvm-large-ops/e42ac1a : SUCCESS in 14m 07s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/93/62593/5/check/gate-tempest-dsvm-neutron-large-ops/a11c707 : SUCCESS in 21m 07s\n- check-grenade-dsvm http://logs.openstack.org/93/62593/5/check/check-grenade-dsvm/246e6a7 : SUCCESS in 31m 06s\n","accounts_in_message":[],"_revision_number":5},{"id":"c3ee948f8419f7eb0af692e2ba408b9bc0aff528","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2013-12-19 07:43:41.000000000","message":"Patch Set 5:\n\nI noticed tempest failed, I think you hit bug(s):\n\n- https://bugs.launchpad.net/bugs/1261433\n\nWe don\u0027t automatically recheck or reverify, so please consider\ndoing that manually if someone hasn\u0027t already. For a code review\nwhich is not yet approved, you can recheck by leaving a code\nreview comment with just the text:\n\n    recheck bug 1261433\n\nFor a code review which has been approved but failed to merge,\nyou can reverify by leaving a comment like this:\n\n    reverify bug 1261433","accounts_in_message":[],"_revision_number":5},{"id":"f4160592300a1ee0b3c5ad15270deee721be48e7","author":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"date":"2013-12-19 07:47:58.000000000","message":"Patch Set 5:\n\nrecheck bug 1261433","accounts_in_message":[],"_revision_number":5},{"id":"d1f487aed9b1807c6330b333281e9de3c42f25b9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-19 08:34:47.000000000","message":"Patch Set 5: Works for me\n\nBuild succeeded.\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/93/62593/5/check/gate-python-cinderclient-pep8/724918b : SUCCESS in 51s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/93/62593/5/check/gate-python-cinderclient-docs/1e89b89/doc/build/html/ : SUCCESS in 45s\n- gate-python-cinderclient-python26 http://logs.openstack.org/93/62593/5/check/gate-python-cinderclient-python26/7ca15c8 : SUCCESS in 52s\n- gate-python-cinderclient-python27 http://logs.openstack.org/93/62593/5/check/gate-python-cinderclient-python27/fef3754 : SUCCESS in 38s\n- gate-python-cinderclient-python33 http://logs.openstack.org/93/62593/5/check/gate-python-cinderclient-python33/f237b3a : SUCCESS in 3m 59s\n- gate-python-cinderclient-pypy http://logs.openstack.org/93/62593/5/check/gate-python-cinderclient-pypy/6631242 : SUCCESS in 1m 39s\n- check-tempest-dsvm-full http://logs.openstack.org/93/62593/5/check/check-tempest-dsvm-full/33bf450 : SUCCESS in 46m 41s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/93/62593/5/check/check-tempest-dsvm-postgres-full/8cdbfcb : SUCCESS in 41m 54s\n- check-tempest-dsvm-neutron http://logs.openstack.org/93/62593/5/check/check-tempest-dsvm-neutron/6b9d105 : SUCCESS in 31m 07s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/93/62593/5/check/gate-tempest-dsvm-large-ops/c6298c0 : SUCCESS in 13m 38s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/93/62593/5/check/gate-tempest-dsvm-neutron-large-ops/9f58694 : SUCCESS in 15m 22s\n- check-grenade-dsvm http://logs.openstack.org/93/62593/5/check/check-grenade-dsvm/c0298a3 : SUCCESS in 31m 13s\n","accounts_in_message":[],"_revision_number":5},{"id":"81f213494c66280f1fa6047062a4c12571ae416d","author":{"_account_id":7138,"name":"David Jia","email":"jiataotj@cn.ibm.com","username":"jiataotj"},"date":"2014-02-11 06:41:14.000000000","message":"Patch Set 5: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":5},{"id":"91050cdfddce7277e592fc518d428a79bad8f553","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-11 06:41:47.000000000","message":"Patch Set 5: No score\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":5},{"id":"a9349bcae266f4c368a95fcc9c2b1bc6b2d2c69f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-11 06:41:54.000000000","message":"Patch Set 5: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\nThis change was unable to be automatically merged with the current state of the repository. Please rebase your change and upload a new patchset.","accounts_in_message":[],"_revision_number":5},{"id":"2c67b874485ae092de7deb62afe1008a7c7b149d","author":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"date":"2014-02-12 08:54:37.000000000","message":"Patch Set 5: Abandoned\n\nhttps://review.openstack.org/#/c/67780/ can resolve the bug. now it is been merged.","accounts_in_message":[],"_revision_number":5}],"current_revision_number":5,"current_revision":"07bc57187c578e195e0391822065f6880977fb8c","revisions":{"c80d026c47bac8b26355f5dd844b832610848e97":{"kind":"REWORK","_number":1,"created":"2013-12-17 10:11:46.000000000","uploader":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"ref":"refs/changes/93/62593/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/93/62593/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/1"}}},"commit":{"parents":[{"commit":"a2b9bdd36c318ad6a5481f4fae789e5a7f8222dd","subject":"Merge \"Discrepancy between README.rst and cinder help\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/a2b9bdd36c318ad6a5481f4fae789e5a7f8222dd"}]}],"author":{"name":"Xiao Chen","email":"chenxiao@cn.ibm.com","date":"2013-12-17 09:56:13.000000000","tz":480},"committer":{"name":"Xiao Chen","email":"chenxiao@cn.ibm.com","date":"2013-12-17 10:11:48.000000000","tz":480},"subject":"Fix \u0027search_opts\u0027 unexpected keyword argument error","message":"Fix \u0027search_opts\u0027 unexpected keyword argument error\n\nWhen executing \"type-key set\" command using resource name,\nit will use the findall() in base.py, then error will occur\nbecause list() in findall() used \u0027search_opts\u0027 argument which\nlist() in VolumeTypeManager did not have.\n\nThis patch will check whether the list() function has\n\u0027search_opts\u0027 argument to avoid the bug.\n\nChange-Id: I7c389fbe8a5b0fcc83dc4567da425c5e0e64f777\nCloses-Bug: #1248519\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/c80d026c47bac8b26355f5dd844b832610848e97"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/c80d026c47bac8b26355f5dd844b832610848e97"}]},"branch":"refs/heads/master"},"e8d9351c2fab0af15f9e57ca5c48792ef6025e2d":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2013-12-18 01:38:08.000000000","uploader":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"ref":"refs/changes/93/62593/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/93/62593/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/2"}}},"commit":{"parents":[{"commit":"a2b9bdd36c318ad6a5481f4fae789e5a7f8222dd","subject":"Merge \"Discrepancy between README.rst and cinder help\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/a2b9bdd36c318ad6a5481f4fae789e5a7f8222dd"}]}],"author":{"name":"Xiao Chen","email":"chenxiao@cn.ibm.com","date":"2013-12-17 09:56:13.000000000","tz":480},"committer":{"name":"Xiao Chen","email":"chenxiao@cn.ibm.com","date":"2013-12-18 01:26:21.000000000","tz":480},"subject":"Fix \u0027search_opts\u0027 unexpected keyword argument error","message":"Fix \u0027search_opts\u0027 unexpected keyword argument error\n\nWhen executing \u0027encryption-type-create\u0027 command using resource\nname, it will use the findall() in base.py, then error will\noccur because list() in findall() used \u0027search_opts\u0027 argument\nwhich list() in VolumeEncryptionTypeManager did not have.\n\nThis patch will check whether the list() function has\n\u0027search_opts\u0027 argument to avoid the bug.\n\nChange-Id: I7c389fbe8a5b0fcc83dc4567da425c5e0e64f777\nCloses-Bug: #1248519\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/e8d9351c2fab0af15f9e57ca5c48792ef6025e2d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/e8d9351c2fab0af15f9e57ca5c48792ef6025e2d"}]},"branch":"refs/heads/master"},"34f7ec2f7b8e5d4048b18967847e1ad7915970cf":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2013-12-19 02:50:13.000000000","uploader":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"ref":"refs/changes/93/62593/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/93/62593/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/3"}}},"commit":{"parents":[{"commit":"a2b9bdd36c318ad6a5481f4fae789e5a7f8222dd","subject":"Merge \"Discrepancy between README.rst and cinder help\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/a2b9bdd36c318ad6a5481f4fae789e5a7f8222dd"}]}],"author":{"name":"Xiao Chen","email":"chenxiao@cn.ibm.com","date":"2013-12-17 09:56:13.000000000","tz":480},"committer":{"name":"Xiao Chen","email":"chenxiao@cn.ibm.com","date":"2013-12-19 02:49:40.000000000","tz":480},"subject":"Fix \u0027search_opts\u0027 unexpected keyword argument error","message":"Fix \u0027search_opts\u0027 unexpected keyword argument error\n\nWhen executing \u0027encryption-type-create\u0027 command using resource\nname, it will use the findall() in base.py, then error will\noccur because list() in findall() used \u0027search_opts\u0027 argument\nwhich list() in VolumeEncryptionTypeManager did not have.\n\nThis patch will check whether the list() function in child\nclass has \u0027search_opts\u0027 argument to avoid the bug if there are\nseveral managers that lack the list() \u0027search_opts\u0027 agrument.\nthis patch can fix bug 1252665 without VolumeTypeManager\ncode change too.\n\nChange-Id: I7c389fbe8a5b0fcc83dc4567da425c5e0e64f777\nCloses-Bug: #1248519\nRelated-Bug: #1252665\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/34f7ec2f7b8e5d4048b18967847e1ad7915970cf"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/34f7ec2f7b8e5d4048b18967847e1ad7915970cf"}]},"branch":"refs/heads/master"},"e9d75113de63ca6b54efff918add1412e7bdd2cd":{"kind":"REWORK","_number":4,"created":"2013-12-19 05:49:16.000000000","uploader":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"ref":"refs/changes/93/62593/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/93/62593/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/4"}}},"commit":{"parents":[{"commit":"a2b9bdd36c318ad6a5481f4fae789e5a7f8222dd","subject":"Merge \"Discrepancy between README.rst and cinder help\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/a2b9bdd36c318ad6a5481f4fae789e5a7f8222dd"}]}],"author":{"name":"Xiao Chen","email":"chenxiao@cn.ibm.com","date":"2013-12-17 09:56:13.000000000","tz":480},"committer":{"name":"Xiao Chen","email":"chenxiao@cn.ibm.com","date":"2013-12-19 05:48:46.000000000","tz":480},"subject":"Fix \u0027search_opts\u0027 unexpected keyword argument error","message":"Fix \u0027search_opts\u0027 unexpected keyword argument error\n\nWhen executing \u0027encryption-type-create\u0027 command using resource\nname, it will use the findall() in base.py, then error will\noccur because list() in findall() used \u0027search_opts\u0027 argument\nwhich list() in VolumeEncryptionTypeManager did not have.\n\nThis patch will check whether the list() function in child\nclass has \u0027search_opts\u0027 argument to avoid the bug if there are\nseveral managers that lack the list() \u0027search_opts\u0027 agrument.\nthis patch can fix bug 1252665 without VolumeTypeManager\ncode change too.\n\nChange-Id: I7c389fbe8a5b0fcc83dc4567da425c5e0e64f777\nCloses-Bug: #1248519\nRelated-Bug: #1252665\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/e9d75113de63ca6b54efff918add1412e7bdd2cd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/e9d75113de63ca6b54efff918add1412e7bdd2cd"}]},"branch":"refs/heads/master"},"07bc57187c578e195e0391822065f6880977fb8c":{"kind":"REWORK","_number":5,"created":"2013-12-19 07:03:50.000000000","uploader":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"ref":"refs/changes/93/62593/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/93/62593/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/93/62593/5"}}},"commit":{"parents":[{"commit":"a2b9bdd36c318ad6a5481f4fae789e5a7f8222dd","subject":"Merge \"Discrepancy between README.rst and cinder help\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/a2b9bdd36c318ad6a5481f4fae789e5a7f8222dd"}]}],"author":{"name":"Xiao Chen","email":"chenxiao@cn.ibm.com","date":"2013-12-17 09:56:13.000000000","tz":480},"committer":{"name":"Xiao Chen","email":"chenxiao@cn.ibm.com","date":"2013-12-19 06:54:32.000000000","tz":480},"subject":"Fix \u0027search_opts\u0027 unexpected keyword argument error","message":"Fix \u0027search_opts\u0027 unexpected keyword argument error\n\nWhen executing \u0027encryption-type-create\u0027 command using resource\nname, it will use the findall() in base.py, then error will\noccur because list() in findall() used \u0027search_opts\u0027 argument\nwhich list() in VolumeEncryptionTypeManager did not have. Some\nother command like \u0027transfer-delete\u0027 met same issue.\n\nThis patch will check whether the list() function in child\nclass has \u0027search_opts\u0027 argument to avoid the bug if there are\nseveral managers that lack the list() \u0027search_opts\u0027 agrument.\nthis patch can fix bug 1252665 without VolumeTypeManager\ncode change too.\n\nChange-Id: I7c389fbe8a5b0fcc83dc4567da425c5e0e64f777\nCloses-Bug: #1248519\nRelated-Bug: #1252665\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/07bc57187c578e195e0391822065f6880977fb8c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/07bc57187c578e195e0391822065f6880977fb8c"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}