)]}'
{"id":"openstack%2Fpython-glanceclient~166595","triplet_id":"openstack%2Fpython-glanceclient~master~Ic3452ce4560d3cf90fa7f59f98e5ff42e804f8c9","project":"openstack/python-glanceclient","branch":"master","topic":"bug/1434381","hashtags":[],"change_id":"Ic3452ce4560d3cf90fa7f59f98e5ff42e804f8c9","subject":"glance image-show now have --human-readable option","status":"MERGED","created":"2015-03-22 11:26:38.000000000","updated":"2015-04-09 03:36:56.000000000","submitted":"2015-04-08 13:26:09.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"9c72cc293f8755244f30abd54715ce5f0f2cb5a8","_number":166595,"virtual_id_number":166595,"owner":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},{"value":2,"date":"2015-04-08 13:26:09.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2015-03-23 09:00:55.000000000","_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":0,"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},{"value":0,"_account_id":616,"name":"Mark Washenberger","email":"mark.washenberger@markwash.net","username":"markwash"},{"value":0,"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},{"value":0,"date":"2015-03-24 11:37:57.000000000","_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},{"value":0,"_account_id":4463,"name":"iccha-sethi","email":"iccha.sethi@rackspace.com","username":"iccha-sethi"},{"value":0,"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},{"value":0,"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"value":0,"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"value":0,"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},{"value":0,"_account_id":6549,"name":"Zhi Yan Liu","email":"lzy.dev@gmail.com","username":"lzy-dev"},{"value":0,"_account_id":8759,"name":"Arnaud Legendre","email":"arnaudleg@gmail.com","username":"arnaud"},{"value":0,"_account_id":792,"name":"OpenStack Hudson","email":"hudson@openstack.org","username":"hudson-openstack"},{"value":0,"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},{"value":0,"_account_id":8158,"name":"Hemanth Makkapati","email":"makstacker+osgerrit@gmail.com","username":"hemanth.makkapati"},{"value":0,"_account_id":11391,"name":"Mikhail Fedosin","email":"mfedosin@redhat.com","username":"fedosinme"},{"value":0,"_account_id":8127,"name":"Alexander Tivelkov","email":"locher@yandex.ru","username":"ativelkov"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"all":[{"value":0,"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":2,"date":"2015-04-08 10:54:54.000000000","_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},{"value":0,"_account_id":616,"name":"Mark Washenberger","email":"mark.washenberger@markwash.net","username":"markwash"},{"value":0,"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},{"value":1,"date":"2015-03-25 20:12:36.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},{"value":0,"_account_id":4463,"name":"iccha-sethi","email":"iccha.sethi@rackspace.com","username":"iccha-sethi"},{"value":2,"date":"2015-04-08 12:27:32.000000000","_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},{"value":0,"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"value":0,"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"value":0,"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},{"value":0,"_account_id":6549,"name":"Zhi Yan Liu","email":"lzy.dev@gmail.com","username":"lzy-dev"},{"value":0,"_account_id":8759,"name":"Arnaud Legendre","email":"arnaudleg@gmail.com","username":"arnaud"},{"value":0,"_account_id":792,"name":"OpenStack Hudson","email":"hudson@openstack.org","username":"hudson-openstack"},{"value":1,"date":"2015-04-08 10:46:08.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},{"value":0,"_account_id":8158,"name":"Hemanth Makkapati","email":"makstacker+osgerrit@gmail.com","username":"hemanth.makkapati"},{"value":0,"_account_id":11391,"name":"Mikhail Fedosin","email":"mfedosin@redhat.com","username":"fedosinme"},{"value":0,"_account_id":8127,"name":"Alexander Tivelkov","email":"locher@yandex.ru","username":"ativelkov"}],"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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"all":[{"value":0,"date":"2015-03-23 03:25:40.000000000","_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":0,"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},{"value":0,"date":"2015-03-23 03:25:40.000000000","_account_id":616,"name":"Mark Washenberger","email":"mark.washenberger@markwash.net","username":"markwash"},{"value":0,"date":"2015-03-23 03:25:40.000000000","_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},{"value":0,"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},{"value":0,"date":"2015-03-23 03:25:40.000000000","_account_id":4463,"name":"iccha-sethi","email":"iccha.sethi@rackspace.com","username":"iccha-sethi"},{"value":1,"date":"2015-04-08 12:27:32.000000000","_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},{"value":0,"date":"2015-03-23 03:25:40.000000000","_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"value":0,"date":"2015-03-23 03:25:40.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"value":0,"date":"2015-03-23 03:25:40.000000000","_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},{"value":0,"date":"2015-03-23 03:25:40.000000000","_account_id":6549,"name":"Zhi Yan Liu","email":"lzy.dev@gmail.com","username":"lzy-dev"},{"value":0,"date":"2015-03-23 03:25:40.000000000","_account_id":8759,"name":"Arnaud Legendre","email":"arnaudleg@gmail.com","username":"arnaud"},{"value":0,"date":"2015-03-23 03:25:40.000000000","_account_id":792,"name":"OpenStack Hudson","email":"hudson@openstack.org","username":"hudson-openstack"},{"value":0,"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},{"value":0,"date":"2015-03-23 03:25:40.000000000","_account_id":8158,"name":"Hemanth Makkapati","email":"makstacker+osgerrit@gmail.com","username":"hemanth.makkapati"},{"value":0,"date":"2015-03-23 03:25:40.000000000","_account_id":11391,"name":"Mikhail Fedosin","email":"mfedosin@redhat.com","username":"fedosinme"},{"value":0,"date":"2015-03-23 03:25:40.000000000","_account_id":8127,"name":"Alexander Tivelkov","email":"locher@yandex.ru","username":"ativelkov"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":0,"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},{"value":0,"_account_id":616,"name":"Mark Washenberger","email":"mark.washenberger@markwash.net","username":"markwash"},{"value":0,"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},{"value":0,"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},{"value":0,"_account_id":4463,"name":"iccha-sethi","email":"iccha.sethi@rackspace.com","username":"iccha-sethi"},{"value":0,"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},{"value":0,"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"value":0,"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"value":0,"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},{"value":0,"_account_id":6549,"name":"Zhi Yan Liu","email":"lzy.dev@gmail.com","username":"lzy-dev"},{"value":0,"_account_id":8759,"name":"Arnaud Legendre","email":"arnaudleg@gmail.com","username":"arnaud"},{"value":0,"_account_id":792,"name":"OpenStack Hudson","email":"hudson@openstack.org","username":"hudson-openstack"},{"value":0,"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},{"value":0,"_account_id":8158,"name":"Hemanth Makkapati","email":"makstacker+osgerrit@gmail.com","username":"hemanth.makkapati"},{"value":0,"_account_id":11391,"name":"Mikhail Fedosin","email":"mfedosin@redhat.com","username":"fedosinme"},{"value":0,"_account_id":8127,"name":"Alexander Tivelkov","email":"locher@yandex.ru","username":"ativelkov"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Top Priority / Holds Gate"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},{"_account_id":616,"name":"Mark Washenberger","email":"mark.washenberger@markwash.net","username":"markwash"},{"_account_id":792,"name":"OpenStack Hudson","email":"hudson@openstack.org","username":"hudson-openstack"},{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},{"_account_id":4463,"name":"iccha-sethi","email":"iccha.sethi@rackspace.com","username":"iccha-sethi"},{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},{"_account_id":6549,"name":"Zhi Yan Liu","email":"lzy.dev@gmail.com","username":"lzy-dev"},{"_account_id":8127,"name":"Alexander Tivelkov","email":"locher@yandex.ru","username":"ativelkov"},{"_account_id":8158,"name":"Hemanth Makkapati","email":"makstacker+osgerrit@gmail.com","username":"hemanth.makkapati"},{"_account_id":8759,"name":"Arnaud Legendre","email":"arnaudleg@gmail.com","username":"arnaud"},{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},{"_account_id":11391,"name":"Mikhail Fedosin","email":"mfedosin@redhat.com","username":"fedosinme"},{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},{"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-03-23 03:25:40.000000000","updated_by":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"reviewer":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"state":"REVIEWER"},{"updated":"2015-03-23 03:25:40.000000000","updated_by":{"_account_id":616,"name":"Mark Washenberger","email":"mark.washenberger@markwash.net","username":"markwash"},"reviewer":{"_account_id":616,"name":"Mark Washenberger","email":"mark.washenberger@markwash.net","username":"markwash"},"state":"REVIEWER"},{"updated":"2015-03-23 03:25:40.000000000","updated_by":{"_account_id":792,"name":"OpenStack Hudson","email":"hudson@openstack.org","username":"hudson-openstack"},"reviewer":{"_account_id":792,"name":"OpenStack Hudson","email":"hudson@openstack.org","username":"hudson-openstack"},"state":"REVIEWER"},{"updated":"2015-03-23 03:25:40.000000000","updated_by":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"reviewer":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"state":"REVIEWER"},{"updated":"2015-03-23 03:25:40.000000000","updated_by":{"_account_id":4463,"name":"iccha-sethi","email":"iccha.sethi@rackspace.com","username":"iccha-sethi"},"reviewer":{"_account_id":4463,"name":"iccha-sethi","email":"iccha.sethi@rackspace.com","username":"iccha-sethi"},"state":"REVIEWER"},{"updated":"2015-03-23 03:25:40.000000000","updated_by":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"reviewer":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"state":"REVIEWER"},{"updated":"2015-03-23 03:25:40.000000000","updated_by":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"reviewer":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"state":"REVIEWER"},{"updated":"2015-03-23 03:25:40.000000000","updated_by":{"_account_id":6549,"name":"Zhi Yan Liu","email":"lzy.dev@gmail.com","username":"lzy-dev"},"reviewer":{"_account_id":6549,"name":"Zhi Yan Liu","email":"lzy.dev@gmail.com","username":"lzy-dev"},"state":"REVIEWER"},{"updated":"2015-03-23 03:25:40.000000000","updated_by":{"_account_id":8127,"name":"Alexander Tivelkov","email":"locher@yandex.ru","username":"ativelkov"},"reviewer":{"_account_id":8127,"name":"Alexander Tivelkov","email":"locher@yandex.ru","username":"ativelkov"},"state":"REVIEWER"},{"updated":"2015-03-23 03:25:40.000000000","updated_by":{"_account_id":8158,"name":"Hemanth Makkapati","email":"makstacker+osgerrit@gmail.com","username":"hemanth.makkapati"},"reviewer":{"_account_id":8158,"name":"Hemanth Makkapati","email":"makstacker+osgerrit@gmail.com","username":"hemanth.makkapati"},"state":"REVIEWER"},{"updated":"2015-03-23 03:25:40.000000000","updated_by":{"_account_id":8759,"name":"Arnaud Legendre","email":"arnaudleg@gmail.com","username":"arnaud"},"reviewer":{"_account_id":8759,"name":"Arnaud Legendre","email":"arnaudleg@gmail.com","username":"arnaud"},"state":"REVIEWER"},{"updated":"2015-03-23 03:25:40.000000000","updated_by":{"_account_id":11391,"name":"Mikhail Fedosin","email":"mfedosin@redhat.com","username":"fedosinme"},"reviewer":{"_account_id":11391,"name":"Mikhail Fedosin","email":"mfedosin@redhat.com","username":"fedosinme"},"state":"REVIEWER"},{"updated":"2015-03-23 03:25:40.000000000","updated_by":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"reviewer":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"state":"REVIEWER"},{"updated":"2015-03-24 11:37:57.000000000","updated_by":{"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},"reviewer":{"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},"state":"REVIEWER"},{"updated":"2015-04-08 10:46:08.000000000","updated_by":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"reviewer":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"state":"REVIEWER"},{"updated":"2015-04-08 10:54:54.000000000","updated_by":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"reviewer":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"state":"REVIEWER"},{"updated":"2015-04-08 12:27:32.000000000","updated_by":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"reviewer":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"state":"REVIEWER"},{"updated":"2015-04-08 13:26:09.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"767fa593276adcdeffbc8ab1c10ad8e137d2ec79","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2015-03-22 11:26:38.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"fe150286deb7b0e3e4ff702824e6fe17a9a9067d","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2015-03-22 11:57:27.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"c6a1c136c5cb7034941cede6e177f9aad705ea7b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-22 12:53:31.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-glanceclient-python26 http://logs.openstack.org/95/166595/2/check/gate-python-glanceclient-python26/24c440d/ : SUCCESS in 1m 20s\n- gate-python-glanceclient-pep8 http://logs.openstack.org/95/166595/2/check/gate-python-glanceclient-pep8/f47d3c9/ : SUCCESS in 1m 03s\n- gate-python-glanceclient-docs http://docs-draft.openstack.org/95/166595/2/check/gate-python-glanceclient-docs/255fd95//doc/build/html/ : SUCCESS in 1m 10s\n- gate-python-glanceclient-python27 http://logs.openstack.org/95/166595/2/check/gate-python-glanceclient-python27/9ac3fda/ : SUCCESS in 1m 13s\n- gate-python-glanceclient-python34 http://logs.openstack.org/95/166595/2/check/gate-python-glanceclient-python34/5ccb9e1/ : SUCCESS in 2m 32s\n- gate-tempest-dsvm-neutron-src-python-glanceclient-juno http://logs.openstack.org/95/166595/2/check/gate-tempest-dsvm-neutron-src-python-glanceclient-juno/a4eb4b8/ : SUCCESS in 56m 01s\n- gate-python-glanceclient-pypy http://logs.openstack.org/95/166595/2/check/gate-python-glanceclient-pypy/56fc106/ : SUCCESS in 1m 34s\n- gate-tempest-dsvm-neutron-src-python-glanceclient http://logs.openstack.org/95/166595/2/check/gate-tempest-dsvm-neutron-src-python-glanceclient/7748ad4/ : SUCCESS in 49m 15s","accounts_in_message":[],"_revision_number":2},{"id":"9ecb5fb5ec2c83e581c689ee757ccef873f564ca","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2015-03-23 07:50:13.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"482d3a14bd6c457dd388c3680ae15f43e47d34ed","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2015-03-23 09:00:55.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"146e837aad7e2b25796812c6d308296f224cf153","author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"date":"2015-03-23 11:03:10.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"02d8b16d8a3c845e74c51f9620762b7f8dcc6ac9","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2015-03-23 13:42:53.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"7942ca142de01e0613ae8b2dbc88018ce2cc94ab","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2015-03-24 09:25:56.000000000","message":"Patch Set 2:\n\nNote that I was not suggesting to make the whole change in this patch. However, if we think it makes sense to make this the default behavior, we should start doing that in this patch. \n\nThat is, add a deprecation warning (or note this in the change log) so that people will know the behavior will change.\n\n@Stuart, Many patches have been blocked lately because \"there are scripts using the client as is\". TBH, I think we should stop using that as a blocker for changing behaviors we think should be changed. If a good deprecation path is provided, scripts will be able to be updated properly and without breaking.","accounts_in_message":[],"_revision_number":2},{"id":"3ab6e1123f321ab86105b0fa87735d76634e8230","author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"date":"2015-03-24 10:51:34.000000000","message":"Patch Set 2:\n\n@Flavio\n\nI understand where you\u0027re coming from, but my preference is, as much as possible, to provide a stable, predictable experience for users.\n\nFolks might have entire in-house test suites or scripts for publishing images that check that the uploaded image size is what is expected. I believe that they should fairly expect not to have to rewrite them.\n\n(Similarly, I don\u0027t think that a patch to change \u0027swift stat\u0027 to give human readable size output by default would get approved.)","accounts_in_message":[],"_revision_number":2},{"id":"5dce28c1968bec73eac8693c9f791b928caca01a","author":{"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},"date":"2015-03-24 11:37:57.000000000","message":"Patch Set 2:\n\nAgree with Stuart that we shouldn\u0027t change the default behavior and that human-readable should be explicitly called by the user to make such output. In that case the backward compatibility will be preserved.","accounts_in_message":[],"_revision_number":2},{"id":"9e17ffbbf64de24a08c29e25281105946e4734ad","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2015-03-25 08:56:33.000000000","message":"Patch Set 2:\n\nHi Flavio,\n\nI too agree with @stuart and @Kamil, human-readable should be provided as additional rather than default for backward compatibility.","accounts_in_message":[],"_revision_number":2},{"id":"e6b45671559b2c9f1f37ca8dfcff8bc04660757d","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2015-03-25 13:12:17.000000000","message":"Patch Set 2:\n\nAgain, I\u0027m not proposing to remove the current behavior in this patch. What I\u0027m trying to do is to find a consensus on what we want to do in the long run.\n\nDo we want to make human-readable the default behavior in the future? If yes (my preference) then we should keep the patch as is and add a warning that says the current behavior will change. If not, then there\u0027s nothing to do.\n\nIn my first comment, I believe I was very clear on what my question was. I understand Stuart\u0027s preference is to keep it as is because migrating scripts will apparently be hard (which I don\u0027t believe). However, it\u0027d be nice to have other fokls opinions too. I\u0027m just trying to build a better experience for the user consuming the library. I don\u0027t really care that much in this particular case. Nonetheless, if we\u0027re adding a new CLI parameter, it probably means there\u0027s a use case for it.","accounts_in_message":[],"_revision_number":2},{"id":"8e1d0798c7cd32bbdd5397f466f58ed916ed28cb","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2015-03-25 14:57:06.000000000","message":"Patch Set 2:\n\nHi Flavio,\nJust to add, current behavior is not changed in this patch, rather a new option is added. Definitely, there is a use case for it. Also, @Stuart and @Kamil points are also valid.\nLets have opinion from others, and then decide what to do with the patch and client library. Can you tell where the discussion will take place, i mean which forum. I will be ready to work on whatever is suggested.","accounts_in_message":[],"_revision_number":2},{"id":"593f668fa294bb4186ba6e221d5b34632cb70a2e","author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"date":"2015-03-25 18:23:00.000000000","message":"Patch Set 2:\n\n@Flavio\n\nI\u0027m not saying \"migrating scripts will apparently be hard\". I\u0027m saying that users do not want to migrate scripts and that we risk not being taken seriously if we don\u0027t provide a stable, predictable experience. I\u0027m virtually certain Swift wouldn\u0027t make the equivalent change.","accounts_in_message":[],"_revision_number":2},{"id":"e18305728671ee2e1e00b8df36725d8f9e121422","author":{"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},"date":"2015-03-25 20:12:36.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"8e609781434806d433b4a064edd8a9140efa6b4c","author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"date":"2015-04-08 10:46:08.000000000","message":"Patch Set 2: Code-Review+1\n\nThis is a nice addition to improve parity with the v1 shell. I don\u0027t believe it should be on by default, though.\n\nAre you planning on also adding --human-readable to image-list, image-create and image-update?","accounts_in_message":[],"_revision_number":2},{"id":"ae1f990ea557095859804e2f779415e34eb20e4a","author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"date":"2015-04-08 10:49:14.000000000","message":"Patch Set 2:\n\nAlso, testing locally with\n\n    $ glance --os-image-api-version 2 image-show --human-readable 28124859-1bf4-4946-aa7d-7663c31567e9\n\nlooks pretty:\n\n...\n| size             | 24MB                                 |\n...","accounts_in_message":[],"_revision_number":2},{"id":"2beacb265037e6f67417ace6de3735e2b468cd5c","author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"date":"2015-04-08 10:54:54.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"d3b4b3ba58a7d807be8279cd602074c44c255a04","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2015-04-08 12:27:32.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\nLets move this forward and think of a way to move to whatever we think is a good default. Either way, we\u0027ll need a human readable value.","accounts_in_message":[],"_revision_number":2},{"id":"58bea6b729ea3f047117dfc90f5af00bb77d203c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-08 12:27:39.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"58bfe65e42d3dd935fd4bb57f773a8ac80d9ba60","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-08 13:26:09.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-glanceclient-python26 http://logs.openstack.org/95/166595/2/gate/gate-python-glanceclient-python26/bd1ea44/ : SUCCESS in 3m 08s\n- gate-python-glanceclient-docs http://docs-draft.openstack.org/95/166595/2/gate/gate-python-glanceclient-docs/ca1454f//doc/build/html/ : SUCCESS in 1m 20s\n- gate-python-glanceclient-pep8 http://logs.openstack.org/95/166595/2/gate/gate-python-glanceclient-pep8/ef2e21f/ : SUCCESS in 1m 14s\n- gate-python-glanceclient-python27 http://logs.openstack.org/95/166595/2/gate/gate-python-glanceclient-python27/9c0844c/ : SUCCESS in 1m 48s\n- gate-python-glanceclient-python34 http://logs.openstack.org/95/166595/2/gate/gate-python-glanceclient-python34/d25a86a/ : SUCCESS in 3m 21s\n- gate-python-glanceclient-pypy http://logs.openstack.org/95/166595/2/gate/gate-python-glanceclient-pypy/1a548e0/ : SUCCESS in 2m 05s\n- gate-tempest-dsvm-neutron-src-python-glanceclient http://logs.openstack.org/95/166595/2/gate/gate-tempest-dsvm-neutron-src-python-glanceclient/ed7a905/ : SUCCESS in 58m 23s","accounts_in_message":[],"_revision_number":2},{"id":"4a4d60aa0419143932d57bcf759472c09af0b135","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-08 13:26:10.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2},{"id":"d92f53a35d1d5d8e64d40b59f41cf5c1ca56fb76","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2015-04-09 03:36:56.000000000","message":"Patch Set 2:\n\n@Louis, i have already filed a bug with bug id: #1433430 for v2 glance image-list, i will be working on this now, and for image-create and image-update, i have to check further.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"c149a94ee1c8c5a04b984063cadf1dbc934eeb8b","revisions":{"0462153c99ebf3edf67459ac41b066207645db82":{"kind":"REWORK","_number":1,"created":"2015-03-22 11:26:38.000000000","uploader":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"ref":"refs/changes/95/166595/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-glanceclient","ref":"refs/changes/95/166595/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-glanceclient refs/changes/95/166595/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-glanceclient refs/changes/95/166595/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-glanceclient refs/changes/95/166595/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-glanceclient refs/changes/95/166595/1"}}},"commit":{"parents":[{"commit":"26280ed58b5d16feed4ab27be063d305ef48a252","subject":"Add release notes for 0.17.0","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-glanceclient/commit/26280ed58b5d16feed4ab27be063d305ef48a252"}]}],"author":{"name":"yatin karel","email":"yatin.karel@nectechnologies.in","date":"2015-03-22 11:22:34.000000000","tz":330},"committer":{"name":"yatin karel","email":"yatin.karel@nectechnologies.in","date":"2015-03-22 11:26:25.000000000","tz":330},"subject":"glance image-show now have --human-readable option","message":"glance image-show now have --human-readable option\n\nAdded option \u0027--human-readable\u0027 to image-show cli which allows users\nto display image size in human-readable format.\n\nChange-Id: Ic3452ce4560d3cf90fa7f59f98e5ff42e804f8c9\nCloses-Bug: #1434381\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-glanceclient/commit/0462153c99ebf3edf67459ac41b066207645db82"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-glanceclient/commit/0462153c99ebf3edf67459ac41b066207645db82"}]},"branch":"refs/heads/master"},"c149a94ee1c8c5a04b984063cadf1dbc934eeb8b":{"kind":"REWORK","_number":2,"created":"2015-03-22 11:57:27.000000000","uploader":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"ref":"refs/changes/95/166595/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-glanceclient","ref":"refs/changes/95/166595/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-glanceclient refs/changes/95/166595/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-glanceclient refs/changes/95/166595/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-glanceclient refs/changes/95/166595/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-glanceclient refs/changes/95/166595/2"}}},"commit":{"parents":[{"commit":"26280ed58b5d16feed4ab27be063d305ef48a252","subject":"Add release notes for 0.17.0","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-glanceclient/commit/26280ed58b5d16feed4ab27be063d305ef48a252"}]}],"author":{"name":"yatin karel","email":"yatin.karel@nectechnologies.in","date":"2015-03-22 11:22:34.000000000","tz":330},"committer":{"name":"yatin karel","email":"yatin.karel@nectechnologies.in","date":"2015-03-22 11:56:55.000000000","tz":330},"subject":"glance image-show now have --human-readable option","message":"glance image-show now have --human-readable option\n\nAdded option \u0027--human-readable\u0027 to image-show cli which allows users\nto display image size in human-readable format.\n\nChange-Id: Ic3452ce4560d3cf90fa7f59f98e5ff42e804f8c9\nCloses-Bug: #1434381\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-glanceclient/commit/c149a94ee1c8c5a04b984063cadf1dbc934eeb8b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-glanceclient/commit/c149a94ee1c8c5a04b984063cadf1dbc934eeb8b"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}