)]}'
{"id":"openstack%2Fglance~138183","triplet_id":"openstack%2Fglance~master~I75da1e9309e0f7ef8839dea3ec9c99c58edc5d63","project":"openstack/glance","branch":"master","topic":"bug/1398314","hashtags":[],"change_id":"I75da1e9309e0f7ef8839dea3ec9c99c58edc5d63","subject":"Allow some fields to be None","status":"MERGED","created":"2014-12-01 21:18:25.000000000","updated":"2014-12-03 04:29:48.000000000","submitted":"2014-12-03 04:29:47.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"3b343efea1af446a6f9a5e07cf8e2e19764a5fb6","_number":138183,"virtual_id_number":138183,"owner":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"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":"2014-12-03 04:29:47.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},{"value":0,"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},{"value":0,"date":"2014-12-02 21:09:20.000000000","_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},{"value":0,"date":"2014-12-02 20:27:22.000000000","_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"}],"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":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"recommended":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"all":[{"value":1,"date":"2014-12-02 17:39:04.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2014-12-02 21:56:23.000000000","_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},{"value":2,"date":"2014-12-02 17:29:25.000000000","_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},{"value":0,"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":1,"date":"2014-12-02 17:30:29.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},{"value":0,"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"}],"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":{"approved":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"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":1,"date":"2014-12-03 01:25:58.000000000","_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},{"value":0,"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},{"value":0,"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},{"value":0,"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"}],"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":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},{"value":0,"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},{"value":0,"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},{"value":0,"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"}],"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":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},{"_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":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-12-02 17:29:25.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":"2014-12-02 17:30:29.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":"2014-12-02 17:39:04.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":"2014-12-02 21:09:20.000000000","updated_by":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"reviewer":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"state":"REVIEWER"},{"updated":"2014-12-03 01:25:58.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":"2014-12-03 04:29:47.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"1afc15cf58f3daa4006d32e473eb6ac941e9ef88","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-01 21:18:25.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"21aae35282852095ed717daab22cbe7da96deb00","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-01 22:02:12.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-glance-pep8 http://logs.openstack.org/83/138183/1/check/gate-glance-pep8/6937505 : SUCCESS in 2m 05s\n- gate-glance-docs http://docs-draft.openstack.org/83/138183/1/check/gate-glance-docs/e4649ce/doc/build/html/ : SUCCESS in 2m 01s\n- gate-glance-python27 http://logs.openstack.org/83/138183/1/check/gate-glance-python27/f730d36 : SUCCESS in 6m 05s\n- check-tempest-dsvm-full http://logs.openstack.org/83/138183/1/check/check-tempest-dsvm-full/c06d778 : SUCCESS in 27m 38s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/83/138183/1/check/check-tempest-dsvm-postgres-full/0d74f1b : SUCCESS in 37m 08s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/83/138183/1/check/check-tempest-dsvm-neutron-full/ddaccf1 : SUCCESS in 43m 44s\n- check-grenade-dsvm http://logs.openstack.org/83/138183/1/check/check-grenade-dsvm/a738ecd : SUCCESS in 28m 24s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/83/138183/1/check/gate-tempest-dsvm-large-ops/618e458 : SUCCESS in 17m 56s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/83/138183/1/check/gate-tempest-dsvm-neutron-large-ops/1b016a1 : SUCCESS in 17m 23s\n- check-devstack-dsvm-cells http://logs.openstack.org/83/138183/1/check/check-devstack-dsvm-cells/a8088df : SUCCESS in 15m 01s\n- gate-rally-dsvm-glance http://logs.openstack.org/83/138183/1/check/gate-rally-dsvm-glance/61e5905 : SUCCESS in 19m 56s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"15ffbf92836072626ac0d6e9a4344f96eb05fc68","author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"date":"2014-12-01 22:08:36.000000000","message":"Patch Set 1: Code-Review-1\n\nIs there any bug cased by this? I\u0027m just wondering what\u0027s the benefit. Cheers.","accounts_in_message":[],"_revision_number":1},{"id":"2671f9edcb2751ccfceb3af1ebf31e65d0055922","author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"date":"2014-12-01 22:26:56.000000000","message":"Patch Set 1: Code-Review+1\n\nFei Long: this is to do with the discrepancies between the v1 and v2 client. This makes sure each field has some value.","accounts_in_message":[],"_revision_number":1},{"id":"48b1546bfde832c6eaf1fa20a9a25ea82d9fac84","author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"date":"2014-12-01 22:53:44.000000000","message":"Patch Set 1:\n\n@Louis, thanks for the clarification. So there is no blueprint or bug to track this work, is it?","accounts_in_message":[],"_revision_number":1},{"id":"e0652bd106dffe5bbb0753c5d05d4327883a5ca4","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 08:40:52.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"aeffbac4f9bcead9d0f5f188be84f53e64abd01d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-02 09:24:22.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-glance-pep8 http://logs.openstack.org/83/138183/2/check/gate-glance-pep8/a6b038a : SUCCESS in 1m 48s\n- gate-glance-docs http://docs-draft.openstack.org/83/138183/2/check/gate-glance-docs/23c1f79/doc/build/html/ : SUCCESS in 1m 34s\n- gate-glance-python27 http://logs.openstack.org/83/138183/2/check/gate-glance-python27/f417262 : SUCCESS in 4m 46s\n- check-tempest-dsvm-full http://logs.openstack.org/83/138183/2/check/check-tempest-dsvm-full/57e4030 : SUCCESS in 28m 31s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/83/138183/2/check/check-tempest-dsvm-postgres-full/22aee37 : SUCCESS in 31m 10s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/83/138183/2/check/check-tempest-dsvm-neutron-full/bc67ba2 : SUCCESS in 43m 21s\n- check-grenade-dsvm http://logs.openstack.org/83/138183/2/check/check-grenade-dsvm/3043c93 : SUCCESS in 26m 46s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/83/138183/2/check/gate-tempest-dsvm-large-ops/80aa0c7 : SUCCESS in 17m 46s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/83/138183/2/check/gate-tempest-dsvm-neutron-large-ops/5581dd6 : SUCCESS in 17m 33s\n- check-devstack-dsvm-cells http://logs.openstack.org/83/138183/2/check/check-devstack-dsvm-cells/04176ed : SUCCESS in 12m 31s\n- gate-rally-dsvm-glance http://logs.openstack.org/83/138183/2/check/gate-rally-dsvm-glance/b5039a5 : SUCCESS in 19m 15s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"7f09825ae0ef59e9c41c92486dcae7466b9c1cfb","author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"date":"2014-12-02 09:26:36.000000000","message":"Patch Set 2: Code-Review+2\n\nLGTM, thanks, Flavio.","accounts_in_message":[],"_revision_number":2},{"id":"30e25e7e4f6cba95cf4d01ad1e3747255cc1a2e9","author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"date":"2014-12-02 09:52:43.000000000","message":"Patch Set 2:\n\nIs \u0027None\u0027 a python implementation detail that users are typically aware of?\n\nDo we usually return \u0027None\u0027 for unset variables? (Or something else such as \u0027null\u0027?)","accounts_in_message":[],"_revision_number":2},{"id":"5be35d5e4ce0d6ae027549dadaa0f6a2972b7681","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 10:18:19.000000000","message":"Patch Set 2: Workflow-1\n\nWe usually return None and I believe that\u0027s something the user is aware of.\n\n-Workflow because I found other fields that are also nullable.","accounts_in_message":[],"_revision_number":2},{"id":"a789e9fb0dd4ed20b8430ddf2273367991ffa308","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 10:33:51.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"04276a6650357aeac1fda4f2319efbd73899c2b8","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 10:37:04.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"0c7a22707e0b59edcc02dd8efabc291a88f06e9e","author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"date":"2014-12-02 10:38:28.000000000","message":"Patch Set 4: Code-Review+2","accounts_in_message":[],"_revision_number":4},{"id":"8018eac83db7618ba2c425ac215b741360329603","author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"date":"2014-12-02 10:50:19.000000000","message":"Patch Set 4:\n\n\u003e We usually return None and I believe that\u0027s something the user is aware of.\n\nI think I\u0027m being slow today ... when does the API return None? Thanks!\n\nHere\u0027s the registry:\n\n{\"image\": {\"status\": \"queued\", \"created_at\": \"2014-12-02T10:42:13\", \"properties\": {}, \"virtual_size\": null, \"name\": null, \"deleted\": false, \"container_format\": null, \"min_ram\": 0, \"disk_format\": null, \"location_data\": [], \"updated_at\": \"2014-12-02T10:42:13\", \"owner\": \"f68be3a5c2b14721a9e0ed2fcb750481\", \"protected\": false, \"location\": null, \"checksum\": null, \"min_disk\": 0, \"is_public\": false, \"deleted_at\": null, \"id\": \"60a720cd-a2d8-4440-9f72-fc0bc09784b0\", \"size\": null}}","accounts_in_message":[],"_revision_number":4},{"id":"7c67a3ac69ea0b71206143e08b5268f7940fafae","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 10:57:59.000000000","message":"Patch Set 4:\n\nIt doesn\u0027t, that\u0027s the problem.\n\nglance-api filters fields with value `None` out and it doesn\u0027t send them back to the client. This is the first fix of 2 fixes that will allow the glance-api service to send all fields in the response.\n\nFields are filtered out in this call[0], which does this[1]\n\n[0] https://github.com/openstack/glance/blob/master/glance/api/v2/images.py#L649\n\n[1] https://github.com/openstack/glance/blob/master/glance/schema.py#L48","accounts_in_message":[],"_revision_number":4},{"id":"9347ba763246737be58892ee69c01d39dcb4cece","author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"date":"2014-12-02 11:17:42.000000000","message":"Patch Set 4:\n\nThanks Flavio.\n\nI\u0027m trying to understand why eg disk-format is different to \u0027name\u0027.\nCurrently it looks like you can set \u0027name\u0027 using the v2 api -- even if it was originally \u0027null\u0027.\n\nCurrently if you create an image without specifying a name and then do a show using the v2 api you get this (which contains no name):\n\n $ glance --os-image-api-version 2 image-create --disk-format raw --container-format bare\n +------------------+--------------------------------------+\n | Property         | Value                                |\n +------------------+--------------------------------------+\n | container_format | bare                                 |\n | created_at       | 2014-12-02T11:05:17Z                 |\n | disk_format      | raw                                  |\n | id               | 2baf3808-f2f2-4b72-a050-0527bbeb7109 |\n | min_disk         | 0                                    |\n | min_ram          | 0                                    |\n | owner            | f68be3a5c2b14721a9e0ed2fcb750481     |\n | protected        | False                                |\n | status           | queued                               |\n | tags             | []                                   |\n | updated_at       | 2014-12-02T11:05:17Z                 |\n | visibility       | private                              |\n +------------------+--------------------------------------+\n\nif you show the image\n\n $ glance --os-image-api-version 2 image-show 2baf3808-f2f2-4b72-a050-0527bbeb7109\n +------------------+--------------------------------------+\n | Property         | Value                                |\n +------------------+--------------------------------------+\n | container_format | bare                                 |\n | created_at       | 2014-12-02T11:05:17Z                 |\n | disk_format      | raw                                  |\n | id               | 2baf3808-f2f2-4b72-a050-0527bbeb7109 |\n | min_disk         | 0                                    |\n | min_ram          | 0                                    |\n | owner            | f68be3a5c2b14721a9e0ed2fcb750481     |\n | protected        | False                                |\n | status           | queued                               |\n | tags             | []                                   |\n | updated_at       | 2014-12-02T11:05:17Z                 |\n | visibility       | private                              |\n +------------------+--------------------------------------+\n\nthis is what goes over the wire (no name):\n\n {\"status\": \"queued\", \"tags\": [], \"container_format\": \"bare\", \"created_at\": \"2014-12-02T11:05:17Z\", \"disk_format\": \"raw\", \"updated_at\": \"2014-12-02T11:05:17Z\",  \"visibility\": \"private\", \"self\": \"/v2/images/2baf3808-f2f2-4b72-a050-0527bbeb7109\", \"min_disk\": 0, \"protected\": false, \"id\": \"2baf3808-f2f2-4b72-a050-0527bbeb7109\", \"file\": \"/v2/images/2baf3808-f2f2-4b72-a050-0527bbeb7109/file\", \"owner\": \"f68be3a5c2b14721a9e0ed2fcb750481\", \"min_ram\": 0, \"schema\": \"/v2/schemas/image\"}\n\nyet it\u0027s possible to set the name using the v2 api:\n\n $ glance --os-image-api-version 2 image-update --name name1 2baf3808-f2f2-4b72-a050-0527bbeb7109\n +------------------+--------------------------------------+\n | Property         | Value                                |\n +------------------+--------------------------------------+\n | container_format | bare                                 |\n | created_at       | 2014-12-02T11:05:17Z                 |\n | disk_format      | raw                                  |\n | id               | 2baf3808-f2f2-4b72-a050-0527bbeb7109 |\n | min_disk         | 0                                    |\n | min_ram          | 0                                    |\n | name             | name1                                |\n | owner            | f68be3a5c2b14721a9e0ed2fcb750481     |\n | protected        | False                                |\n | status           | queued                               |\n | tags             | []                                   |\n | updated_at       | 2014-12-02T11:08:18Z                 |\n | visibility       | private                              |\n +------------------+--------------------------------------+\n\nI\u0027m not 100% sure how it works -- do you know? Thanks!","accounts_in_message":[],"_revision_number":4},{"id":"83272533f290f873633d5bf26aabc3173f6960b9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-02 11:22:16.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-glance-pep8 http://logs.openstack.org/83/138183/4/check/gate-glance-pep8/fb142ec : SUCCESS in 2m 08s\n- gate-glance-docs http://docs-draft.openstack.org/83/138183/4/check/gate-glance-docs/fab4267/doc/build/html/ : SUCCESS in 2m 11s\n- gate-glance-python27 http://logs.openstack.org/83/138183/4/check/gate-glance-python27/5e62cb7 : SUCCESS in 4m 01s\n- check-tempest-dsvm-full http://logs.openstack.org/83/138183/4/check/check-tempest-dsvm-full/6780d29 : SUCCESS in 29m 09s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/83/138183/4/check/check-tempest-dsvm-postgres-full/5a4b8a5 : SUCCESS in 31m 17s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/83/138183/4/check/check-tempest-dsvm-neutron-full/4b5f886 : SUCCESS in 40m 31s\n- check-grenade-dsvm http://logs.openstack.org/83/138183/4/check/check-grenade-dsvm/364d8f1 : SUCCESS in 23m 58s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/83/138183/4/check/gate-tempest-dsvm-large-ops/7d3eb57 : SUCCESS in 21m 00s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/83/138183/4/check/gate-tempest-dsvm-neutron-large-ops/db36b73 : SUCCESS in 19m 37s\n- check-devstack-dsvm-cells http://logs.openstack.org/83/138183/4/check/check-devstack-dsvm-cells/d7dd70c : SUCCESS in 13m 43s\n- gate-rally-dsvm-glance http://logs.openstack.org/83/138183/4/check/gate-rally-dsvm-glance/22e1ac0 : SUCCESS in 22m 42s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"24e1e5b677cab53220b9230d88bf93ba50ff722d","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 11:27:34.000000000","message":"Patch Set 4:\n\nJust to be clear, the problem I\u0027m trying to fix here is on reads not writes. That is, the patch allows fields with value `None` to be valid json schemas (mostly used by the client). The patch that follows it, removes the `None` filter in the schema module and allow those fields to be returned. \n\nThat said, you can set name because it\u0027s not a read-only property[0]. However, if you noticed, the field is not returned/shown by the client until you set it. That\u0027s because the API is not returning it.\n\nI think we\u0027ll need a patch for the client to hide fields that are `None` from the shell (assuming we want to keep hiding them).\n\n[0] https://github.com/openstack/glance/blob/master/glance/api/v2/images.py#L702","accounts_in_message":[],"_revision_number":4},{"id":"4ca97ef739f622f12e679df9cf18e6629f180848","author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"date":"2014-12-02 11:46:16.000000000","message":"Patch Set 4:\n\n\u003e Just to be clear, the problem I\u0027m trying to fix here is on reads not writes. \n\nYou guessed right -- I thought the motivation here was for writes. Thanks for clarifying :-)\n\nI re-read the IRC chat, and I think I\u0027ve a better idea of where you\u0027re coming from now.\n\nBut I\u0027m still curious as to why we need to send the disk-format over the network.\nEg in v1 we don\u0027t send the disk-format over the wire if it is null.\n\nCould we just do an image.get(\u0027disk-format\u0027, None) in the relevant bit of nova code to handle this (rather than changing the API)?\n\nOr does warlock make that impossible? Thanks!","accounts_in_message":[],"_revision_number":4},{"id":"86c97c508541520bc6793df4e6275875db19c10d","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 12:28:56.000000000","message":"Patch Set 4:\n\nThat unfortunately makes warlock impossible to use but even if we weren\u0027t using warlock, I would advice returning all the fields always to keep responses consistent. It\u0027s kind of frustrating for clients to have to check whether a field has been returned or not. Furthermore, it\u0027s even more frustrating to have the client choose a sane default value for those fields.\n\nhope it makes sense","accounts_in_message":[],"_revision_number":4},{"id":"13c7838c83c80f3f3ab415b9c7a3d34a790d06e6","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 13:02:05.000000000","message":"Patch Set 4: Workflow-1\n\nI\u0027m running some manual tests","accounts_in_message":[],"_revision_number":4},{"id":"4e60e3c027e2724d7e2aef4b4b6d5118b28c4c08","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 13:09:07.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"94904a7ebc2a3a2ab3276d6deb8de149a48624f5","author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"date":"2014-12-02 13:38:57.000000000","message":"Patch Set 5: Code-Review+1","accounts_in_message":[],"_revision_number":5},{"id":"57532a381e2c267b0cfbac04507750f61f74e663","author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"date":"2014-12-02 13:42:08.000000000","message":"Patch Set 5: Code-Review+1\n\nTested this series of patches locally.","accounts_in_message":[],"_revision_number":5},{"id":"0c7fe9a309a27e1761df65d1e70004b9d1d8f14f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-02 14:03:00.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-glance-pep8 http://logs.openstack.org/83/138183/5/check/gate-glance-pep8/1598672 : SUCCESS in 3m 28s\n- gate-glance-docs http://docs-draft.openstack.org/83/138183/5/check/gate-glance-docs/54a570d/doc/build/html/ : SUCCESS in 2m 33s\n- gate-glance-python27 http://logs.openstack.org/83/138183/5/check/gate-glance-python27/0ae8fe7 : SUCCESS in 5m 53s\n- check-tempest-dsvm-full http://logs.openstack.org/83/138183/5/check/check-tempest-dsvm-full/d601ee1 : SUCCESS in 35m 57s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/83/138183/5/check/check-tempest-dsvm-postgres-full/6d1eff7 : SUCCESS in 35m 32s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/83/138183/5/check/check-tempest-dsvm-neutron-full/9e8ee89 : SUCCESS in 42m 27s\n- check-grenade-dsvm http://logs.openstack.org/83/138183/5/check/check-grenade-dsvm/e701444 : SUCCESS in 29m 13s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/83/138183/5/check/gate-tempest-dsvm-large-ops/5279afd : SUCCESS in 22m 17s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/83/138183/5/check/gate-tempest-dsvm-neutron-large-ops/74059f2 : SUCCESS in 20m 33s\n- check-devstack-dsvm-cells http://logs.openstack.org/83/138183/5/check/check-devstack-dsvm-cells/259d040 : SUCCESS in 15m 08s\n- gate-rally-dsvm-glance http://logs.openstack.org/83/138183/5/check/gate-rally-dsvm-glance/42ed4c4 : SUCCESS in 23m 46s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"4aa2c680427c2ea2fb1d325b1bb75bb756ddab08","author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"date":"2014-12-02 14:51:16.000000000","message":"Patch Set 5:\n\n\u003e It\u0027s kind of frustrating for clients to have to check whether a field has been returned or not.\n\nAgreed. But the v2 server has been out for about two years now.\n\nSo I think the client application will need to perform this check anyway.\n\nIt could assume that it\u0027s running against a version of the glance v2 server which includes disk_format \u0027None\u0027 -- but it would then not work against Havana, Icehouse or Juno glance servers.\n\nI think using .get() will work for both v1 and v2, and will mean that v1 and v2 will continue to be consistent in what they send over the wire for unset variables.\nUsing .get() is generally more robust.\n\nMy .02 cents :-)","accounts_in_message":[],"_revision_number":5},{"id":"ee6bdf7b4f075d5fe48478924ab2b033e734cc93","author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"date":"2014-12-02 15:07:54.000000000","message":"Patch Set 5: Code-Review-1\n\nBased on compatibility I\u0027m going to mark this -1 for now.\n\nI think to encourage adoption of the v2 api that Nova Kilo should be able to work against a Juno (and potentially older) Glance server.\n\nHaving Nova use .get() to access image attributes is, I think, required for that.\n\nIf this is fixed on the client (ie nova) side I think this server side change may not be warranted.\n\nAs always, I\u0027m open to correction :-)","accounts_in_message":[],"_revision_number":5},{"id":"493e5da21ca57c5165061758e9ed4a9b117fd66c","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 15:16:41.000000000","message":"Patch Set 5:\n\nI don\u0027t think this change breaks compatibility. It\u0027s just allowing those fields to be None *in the schema*. The real incompatible change is Ieaddd8a686cf7361f18cb1ee83b7887cdca22bd6 - which I\u0027ve blocked myself - because it allows null fields to be returned.\n\nThe reason that change is incompatible is not because of Nova but because it will break users that check for the field existence to know if it\u0027s been set.","accounts_in_message":[],"_revision_number":5},{"id":"f525deca937fd3c1bc20704690ec7a788d4bcc4d","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 15:18:40.000000000","message":"Patch Set 5:\n\nAlso, all that said, if we\u0027re going to require an API version bump for this, we\u0027d rather do this change in that bump and not here. Lets discuss this a bit more.","accounts_in_message":[],"_revision_number":5},{"id":"92888cc5a45b2321c1d0c2ae97b2d6bd95caeed4","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 15:51:15.000000000","message":"Patch Set 5:\n\nI\u0027ve removed the -Workflow on the other patch. After discussing with Nikhil and other members, it seems that bumping the minor makes sense after all.","accounts_in_message":[],"_revision_number":5},{"id":"180e41d1087896cde95f41553185ceaadf177a1b","author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"date":"2014-12-02 16:50:04.000000000","message":"Patch Set 5: -Code-Review\n\nRemoving my -1.\n\nI think as long as we implement Nova changes so that Nova will work against older v2 servers which don\u0027t return \u0027disk_format\u0027/\u0027name\u0027/\u0027container_format\u0027 there\u0027s may be no real reason to object here.\n\nCan we do that?","accounts_in_message":[],"_revision_number":5},{"id":"80e4bf4fc4ca7c007188694bed1dec01d7ce1d2b","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 17:15:38.000000000","message":"Patch Set 5:\n\n@Stuart: Yup, that\u0027s the goal. I\u0027ve discussed this with some nova folks as well. They provided great feedback on the version thing","accounts_in_message":[],"_revision_number":5},{"id":"ba25e236424da8395483aec6ea95d3943f9ae4c7","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 17:22:54.000000000","message":"Patch Set 5:\n\n(2 comments)","accounts_in_message":[],"_revision_number":5},{"id":"661abf6d285081bab2775105aa31ac7a5f24dadb","author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"date":"2014-12-02 17:25:26.000000000","message":"Patch Set 5: Code-Review+2\n\nWe\u0027re ok on the compatibilty side, so no objections.\n\n@Flavio thanks for answering all my queries.","accounts_in_message":[],"_revision_number":5},{"id":"59fd7e6a7de45289dc765504a2849540e0072d13","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 17:26:35.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"02bd851c54f9dc8d33f832d3a96b710d998273f1","author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"date":"2014-12-02 17:29:25.000000000","message":"Patch Set 6: Code-Review+2\n\nThanks.\n\nNB: \u0027null\u0027 is what goes over the wire.","accounts_in_message":[],"_revision_number":6},{"id":"6e5f731af66c70355e8a7affc2f0d5220d07db5c","author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"date":"2014-12-02 17:30:29.000000000","message":"Patch Set 6: Code-Review+1\n\nLatest ps looks good.","accounts_in_message":[],"_revision_number":6},{"id":"c6176129e22a576cc98c86d9fcc13387626b83b6","author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"date":"2014-12-02 17:39:04.000000000","message":"Patch Set 6: Code-Review+1","accounts_in_message":[],"_revision_number":6},{"id":"62954d675352c88eb81b45def5a218b93d22066a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-02 18:07:07.000000000","message":"Patch Set 6: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-glance-pep8 http://logs.openstack.org/83/138183/6/check/gate-glance-pep8/11cc0f7 : FAILURE in 33s\n- gate-glance-docs http://logs.openstack.org/83/138183/6/check/gate-glance-docs/29b62f0 : FAILURE in 37s\n- gate-glance-python27 http://logs.openstack.org/83/138183/6/check/gate-glance-python27/1271d59 : FAILURE in 20s\n- check-tempest-dsvm-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-full/1f93491 : SUCCESS in 35m 36s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-postgres-full/9d68d99 : SUCCESS in 31m 59s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-neutron-full/2aef59d : SUCCESS in 39m 06s\n- check-grenade-dsvm http://logs.openstack.org/83/138183/6/check/check-grenade-dsvm/7b5a1ed : FAILURE in 18m 59s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/83/138183/6/check/gate-tempest-dsvm-large-ops/4c992f4 : SUCCESS in 19m 03s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/83/138183/6/check/gate-tempest-dsvm-neutron-large-ops/f648d1e : SUCCESS in 18m 19s\n- check-devstack-dsvm-cells http://logs.openstack.org/83/138183/6/check/check-devstack-dsvm-cells/9240df9 : SUCCESS in 14m 54s\n- gate-rally-dsvm-glance http://logs.openstack.org/83/138183/6/check/gate-rally-dsvm-glance/0fbe2d6 : SUCCESS in 23m 29s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"dd3de5e705a3c81766171d659371c7081ed6264b","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 20:27:22.000000000","message":"Patch Set 6:\n\nrecheck no bug","accounts_in_message":[],"_revision_number":6},{"id":"0929d771a2431af75356345bcf26fece0023b972","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-02 21:09:02.000000000","message":"Patch Set 6:\n\nBuild failed (check pipeline).  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-glance-pep8 http://logs.openstack.org/83/138183/6/check/gate-glance-pep8/6d0f3e4 : SUCCESS in 2m 27s\n- gate-glance-docs http://logs.openstack.org/83/138183/6/check/gate-glance-docs/261d76a : FAILURE in 1m 15s\n- gate-glance-python27 http://logs.openstack.org/83/138183/6/check/gate-glance-python27/4b3103f : SUCCESS in 6m 26s\n- check-tempest-dsvm-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-full/b7622ec : SUCCESS in 35m 49s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-postgres-full/b17e399 : SUCCESS in 30m 42s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-neutron-full/d2ea155 : SUCCESS in 41m 08s\n- check-grenade-dsvm http://logs.openstack.org/83/138183/6/check/check-grenade-dsvm/effd166 : FAILURE in 24m 23s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/83/138183/6/check/gate-tempest-dsvm-large-ops/2126847 : SUCCESS in 24m 34s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/83/138183/6/check/gate-tempest-dsvm-neutron-large-ops/89ab00d : SUCCESS in 21m 53s\n- check-devstack-dsvm-cells http://logs.openstack.org/83/138183/6/check/check-devstack-dsvm-cells/b4e3ca8 : SUCCESS in 15m 14s\n- gate-rally-dsvm-glance http://logs.openstack.org/83/138183/6/check/gate-rally-dsvm-glance/d2a2f4c : SUCCESS in 24m 59s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"9198b4b1f3fb5dfdfc72bacfabbb160d0dac2b41","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2014-12-02 21:09:20.000000000","message":"Patch Set 6:\n\nI noticed jenkins failed, I think you hit bug(s):\n\n- check-grenade-dsvm: unrecognized error\n- gate-glance-docs: https://bugs.launchpad.net/bugs/1398500\n\nSome of the tests failed in a way that we did not understand. Please help us classify these issues so that they can be part of Elastic Recheck http://status.openstack.org/elastic-recheck/\nFor more details on this and other bugs, please see http://status.openstack.org/elastic-recheck/","accounts_in_message":[],"_revision_number":6},{"id":"16098f1b145765d94ddc682c68b91760a1488372","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2014-12-02 21:51:53.000000000","message":"Patch Set 6:\n\nrecheck bug 1398500","accounts_in_message":[],"_revision_number":6},{"id":"745e3ce07bf34c8ee6d876a3d58f127e39e101ab","author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"date":"2014-12-02 21:56:23.000000000","message":"Patch Set 6: Code-Review+2\n\nI\u0027m doing this since there is a pusher :)","accounts_in_message":[],"_revision_number":6},{"id":"caaf7dfc93206c4101834e47ecd4a82063200adc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-02 22:51:04.000000000","message":"Patch Set 6:\n\nBuild failed (check pipeline).  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-glance-pep8 http://logs.openstack.org/83/138183/6/check/gate-glance-pep8/d33eaa2 : SUCCESS in 2m 20s\n- gate-glance-docs http://docs-draft.openstack.org/83/138183/6/check/gate-glance-docs/06bc1b0/doc/build/html/ : SUCCESS in 2m 11s\n- gate-glance-python27 http://logs.openstack.org/83/138183/6/check/gate-glance-python27/82bc77c : SUCCESS in 5m 49s\n- check-tempest-dsvm-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-full/10ed80e : SUCCESS in 33m 08s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-postgres-full/b65c764 : SUCCESS in 28m 01s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-neutron-full/71d8020 : SUCCESS in 46m 04s\n- check-grenade-dsvm http://logs.openstack.org/83/138183/6/check/check-grenade-dsvm/522e723 : FAILURE in 21m 22s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/83/138183/6/check/gate-tempest-dsvm-large-ops/a198f1d : SUCCESS in 18m 00s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/83/138183/6/check/gate-tempest-dsvm-neutron-large-ops/b959d2e : SUCCESS in 17m 42s\n- check-devstack-dsvm-cells http://logs.openstack.org/83/138183/6/check/check-devstack-dsvm-cells/35afca3 : SUCCESS in 14m 42s\n- gate-rally-dsvm-glance http://logs.openstack.org/83/138183/6/check/gate-rally-dsvm-glance/56a1d63 : SUCCESS in 23m 12s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"82d18dd7f741629136bc8e2849bb461e67a8143f","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2014-12-02 22:51:17.000000000","message":"Patch Set 6:\n\nI noticed jenkins failed, I think you hit bug(s):\n\n- check-grenade-dsvm: https://bugs.launchpad.net/bugs/1398561\n\nIf you believe we\u0027ve correctly identified the failure, feel free to leave a \u0027recheck\u0027 comment to run the tests again.\nFor more details on this and other bugs, please see http://status.openstack.org/elastic-recheck/","accounts_in_message":[],"_revision_number":6},{"id":"5c01033d39b6b1cb480be8bfe8dff0a758e7add6","author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"date":"2014-12-03 00:13:30.000000000","message":"Patch Set 6:\n\nrecheck bug 1398561","accounts_in_message":[],"_revision_number":6},{"id":"565828eda258f7f234ecf267b1ea7c2c4a0cf0d3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-03 00:58:37.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-glance-pep8 http://logs.openstack.org/83/138183/6/check/gate-glance-pep8/f6a5190 : SUCCESS in 2m 19s\n- gate-glance-docs http://docs-draft.openstack.org/83/138183/6/check/gate-glance-docs/face854/doc/build/html/ : SUCCESS in 2m 05s\n- gate-glance-python27 http://logs.openstack.org/83/138183/6/check/gate-glance-python27/0175ec8 : SUCCESS in 5m 06s\n- check-tempest-dsvm-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-full/db32826 : SUCCESS in 28m 46s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-postgres-full/7e2b827 : SUCCESS in 40m 11s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-neutron-full/e31f01b : SUCCESS in 45m 04s\n- check-grenade-dsvm http://logs.openstack.org/83/138183/6/check/check-grenade-dsvm/a761d85 : SUCCESS in 25m 32s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/83/138183/6/check/gate-tempest-dsvm-large-ops/064e940 : SUCCESS in 19m 06s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/83/138183/6/check/gate-tempest-dsvm-neutron-large-ops/7174afa : SUCCESS in 18m 09s\n- check-devstack-dsvm-cells http://logs.openstack.org/83/138183/6/check/check-devstack-dsvm-cells/e6eac2b : SUCCESS in 13m 29s\n- gate-rally-dsvm-glance http://logs.openstack.org/83/138183/6/check/gate-rally-dsvm-glance/af3f481 : SUCCESS in 22m 49s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"9454048649a63018ac1c9f07cb15b7b4d04977c2","author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"date":"2014-12-03 01:25:58.000000000","message":"Patch Set 6: Workflow+1","accounts_in_message":[],"_revision_number":6},{"id":"fb9420480b9116f4938f7669f1f1f3595461ac3b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-03 01:25:59.000000000","message":"Patch Set 6: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":6},{"id":"74c81bd994fbe0f65d4d16c93df300e969bf3bde","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-03 02:47:16.000000000","message":"Patch Set 6: Verified-2\n\nBuild failed (gate pipeline).  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-glance-docs http://logs.openstack.org/83/138183/6/gate/gate-glance-docs/6b02553 : FAILURE in 38s\n- gate-glance-pep8 http://logs.openstack.org/83/138183/6/gate/gate-glance-pep8/0ecc4cc : SUCCESS in 1m 54s\n- gate-glance-python27 http://logs.openstack.org/83/138183/6/gate/gate-glance-python27/d6942ea : SUCCESS in 4m 53s\n- gate-tempest-dsvm-full http://logs.openstack.org/83/138183/6/gate/gate-tempest-dsvm-full/d8374c7 : SUCCESS in 34m 43s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/83/138183/6/gate/gate-tempest-dsvm-postgres-full/f3af1b1 : SUCCESS in 32m 44s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/83/138183/6/gate/gate-tempest-dsvm-neutron-full/a57f730 : SUCCESS in 37m 25s\n- gate-grenade-dsvm http://logs.openstack.org/83/138183/6/gate/gate-grenade-dsvm/8a5c879 : SUCCESS in 28m 03s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/83/138183/6/gate/gate-tempest-dsvm-large-ops/c0dc8fd : SUCCESS in 19m 18s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/83/138183/6/gate/gate-tempest-dsvm-neutron-large-ops/2272b36 : SUCCESS in 19m 57s\n- gate-devstack-dsvm-cells http://logs.openstack.org/83/138183/6/gate/gate-devstack-dsvm-cells/23d3313 : SUCCESS in 14m 15s","accounts_in_message":[],"_revision_number":6},{"id":"3d81cb5146a7cb3a077e0d1a5d11740e4bebe373","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2014-12-03 02:47:30.000000000","message":"Patch Set 6:\n\nI noticed jenkins failed, I think you hit bug(s):\n\n- gate-glance-docs: https://bugs.launchpad.net/bugs/1398500\n\nIf you believe we\u0027ve correctly identified the failure, feel free to leave a \u0027recheck\u0027 comment to run the tests again.\nFor more details on this and other bugs, please see http://status.openstack.org/elastic-recheck/","accounts_in_message":[],"_revision_number":6},{"id":"46453193eb27131dbc9f34979834e9378d960580","author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"date":"2014-12-03 02:52:26.000000000","message":"Patch Set 6:\n\nrecheck bug 1398500","accounts_in_message":[],"_revision_number":6},{"id":"46c2ea1b5bd38479c92e8d5724572c4baf350fae","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-03 03:31:01.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-glance-pep8 http://logs.openstack.org/83/138183/6/check/gate-glance-pep8/fec7e07 : SUCCESS in 2m 31s\n- gate-glance-docs http://docs-draft.openstack.org/83/138183/6/check/gate-glance-docs/d0c36d7/doc/build/html/ : SUCCESS in 1m 42s\n- gate-glance-python27 http://logs.openstack.org/83/138183/6/check/gate-glance-python27/25d2e2a : SUCCESS in 6m 30s\n- check-tempest-dsvm-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-full/4bf324a : SUCCESS in 29m 34s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-postgres-full/e1ffa6e : SUCCESS in 37m 02s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/83/138183/6/check/check-tempest-dsvm-neutron-full/0a2c348 : SUCCESS in 38m 29s\n- check-grenade-dsvm http://logs.openstack.org/83/138183/6/check/check-grenade-dsvm/2f4dedb : SUCCESS in 23m 59s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/83/138183/6/check/gate-tempest-dsvm-large-ops/9b70522 : SUCCESS in 23m 36s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/83/138183/6/check/gate-tempest-dsvm-neutron-large-ops/5af073b : SUCCESS in 18m 03s\n- check-devstack-dsvm-cells http://logs.openstack.org/83/138183/6/check/check-devstack-dsvm-cells/e84d493 : SUCCESS in 15m 47s\n- gate-rally-dsvm-glance http://logs.openstack.org/83/138183/6/check/gate-rally-dsvm-glance/c16775b : SUCCESS in 19m 26s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"e2c3ffa19ad5693c3edae7dd650e79c3db417997","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-03 03:31:02.000000000","message":"Patch Set 6: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":6},{"id":"aba7544a1a8fd4f09e5e9cf5bf56c97977caf36b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-03 04:29:47.000000000","message":"Patch Set 6: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-glance-docs http://docs-draft.openstack.org/83/138183/6/gate/gate-glance-docs/a5ed15f/doc/build/html/ : SUCCESS in 1m 39s\n- gate-glance-pep8 http://logs.openstack.org/83/138183/6/gate/gate-glance-pep8/e3f59d5 : SUCCESS in 1m 51s\n- gate-glance-python27 http://logs.openstack.org/83/138183/6/gate/gate-glance-python27/70d3200 : SUCCESS in 4m 29s\n- gate-tempest-dsvm-full http://logs.openstack.org/83/138183/6/gate/gate-tempest-dsvm-full/7c767a1 : SUCCESS in 29m 32s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/83/138183/6/gate/gate-tempest-dsvm-postgres-full/2949ee5 : SUCCESS in 35m 07s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/83/138183/6/gate/gate-tempest-dsvm-neutron-full/d0d56b0 : SUCCESS in 46m 30s\n- gate-grenade-dsvm http://logs.openstack.org/83/138183/6/gate/gate-grenade-dsvm/4d3da8a : SUCCESS in 29m 01s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/83/138183/6/gate/gate-tempest-dsvm-large-ops/b60f436 : SUCCESS in 22m 42s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/83/138183/6/gate/gate-tempest-dsvm-neutron-large-ops/1f62b60 : SUCCESS in 18m 40s\n- gate-devstack-dsvm-cells http://logs.openstack.org/83/138183/6/gate/gate-devstack-dsvm-cells/656e1e1 : SUCCESS in 14m 54s","accounts_in_message":[],"_revision_number":6},{"id":"4648cef7dcd33fabba65711b98013f74529c69e9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-03 04:29:48.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"8b3ca087b7d3bbc1fb88daf6e780efbc1f0dcae8","revisions":{"b8f4b1a059a068904e2e4c8f0f9ae1b7d7427d59":{"kind":"REWORK","_number":1,"created":"2014-12-01 21:18:25.000000000","uploader":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"ref":"refs/changes/83/138183/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/83/138183/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/83/138183/1"}}},"commit":{"parents":[{"commit":"f82574fa686fd3aae2e1284233c40ff05ad8abf2","subject":"Merge \"Disable osprofiler by default\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/f82574fa686fd3aae2e1284233c40ff05ad8abf2"}]}],"author":{"name":"Flavio Percoco","email":"flaper87@gmail.com","date":"2014-12-01 13:45:44.000000000","tz":60},"committer":{"name":"Flavio Percoco","email":"flaper87@gmail.com","date":"2014-12-01 21:06:18.000000000","tz":60},"subject":"Allow (disk|container)_format to be None","message":"Allow (disk|container)_format to be None\n\nContainer and disk format are not required parameters, this patch allows\nthem to be None, hence having them assigned in the client side.\n\nChange-Id: I75da1e9309e0f7ef8839dea3ec9c99c58edc5d63\nNOTE: A follow-up patch will enable None values to be returned by the API.\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/b8f4b1a059a068904e2e4c8f0f9ae1b7d7427d59"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/b8f4b1a059a068904e2e4c8f0f9ae1b7d7427d59"}]},"branch":"refs/heads/master"},"033f8c0812c57c02335f6204a12cca275d98df37":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2014-12-02 08:40:52.000000000","uploader":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"ref":"refs/changes/83/138183/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/83/138183/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/83/138183/2"}}},"commit":{"parents":[{"commit":"f82574fa686fd3aae2e1284233c40ff05ad8abf2","subject":"Merge \"Disable osprofiler by default\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/f82574fa686fd3aae2e1284233c40ff05ad8abf2"}]}],"author":{"name":"Flavio Percoco","email":"flaper87@gmail.com","date":"2014-12-01 13:45:44.000000000","tz":60},"committer":{"name":"Flavio Percoco","email":"flaper87@gmail.com","date":"2014-12-02 08:33:49.000000000","tz":60},"subject":"Allow (disk|container)_format to be None","message":"Allow (disk|container)_format to be None\n\nContainer and disk format are not required parameters, this patch allows\nthem to be None, hence having them assigned in the client side.\n\nNOTE: A follow-up patch will enable None values to be returned by the API.\n\nCloses-bug: #1398313\n\nChange-Id: I75da1e9309e0f7ef8839dea3ec9c99c58edc5d63\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/033f8c0812c57c02335f6204a12cca275d98df37"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/033f8c0812c57c02335f6204a12cca275d98df37"}]},"branch":"refs/heads/master"},"a70083d8f2939665d8ca1caadb858830205698af":{"kind":"REWORK","_number":3,"created":"2014-12-02 10:33:51.000000000","uploader":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"ref":"refs/changes/83/138183/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/83/138183/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/83/138183/3"}}},"commit":{"parents":[{"commit":"f8d812054f533c2d467e62d6e588a1832cd17e37","subject":"Merge \"Correct GlanceStoreException to provide valid message - Glance\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/f8d812054f533c2d467e62d6e588a1832cd17e37"}]}],"author":{"name":"Flavio Percoco","email":"flaper87@gmail.com","date":"2014-12-01 13:45:44.000000000","tz":60},"committer":{"name":"Flavio Percoco","email":"flaper87@gmail.com","date":"2014-12-02 10:33:35.000000000","tz":60},"subject":"Allow (disk|container)_format to be None","message":"Allow (disk|container)_format to be None\n\nContainer and disk format are not required parameters, this patch allows\nthem to be None, hence having them assigned in the client side.\n\nNOTE: A follow-up patch will enable None values to be returned by the API.\n\nCloses-bug: #1398313\n\nChange-Id: I75da1e9309e0f7ef8839dea3ec9c99c58edc5d63\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/a70083d8f2939665d8ca1caadb858830205698af"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/a70083d8f2939665d8ca1caadb858830205698af"}]},"branch":"refs/heads/master"},"f95d4796f41287d1539b7f32313f0fb8bd47757d":{"kind":"NO_CODE_CHANGE","_number":4,"created":"2014-12-02 10:37:04.000000000","uploader":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"ref":"refs/changes/83/138183/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/83/138183/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/83/138183/4"}}},"commit":{"parents":[{"commit":"f8d812054f533c2d467e62d6e588a1832cd17e37","subject":"Merge \"Correct GlanceStoreException to provide valid message - Glance\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/f8d812054f533c2d467e62d6e588a1832cd17e37"}]}],"author":{"name":"Flavio Percoco","email":"flaper87@gmail.com","date":"2014-12-01 13:45:44.000000000","tz":60},"committer":{"name":"Flavio Percoco","email":"flaper87@gmail.com","date":"2014-12-02 10:35:15.000000000","tz":60},"subject":"Allow some fields to be None","message":"Allow some fields to be None\n\nContainer, disk format and checksum - among other fields - are not\nrequired parameters, therfore this patch allows them to be None.\n\nThis will allow the client to have attributes with value `None` and\nit\u0027ll make it consistent across responses and with regards to the API.\n\nNOTE: A follow-up patch will enable None values to be returned by the API.\n\nCloses-bug: #1398313\n\nChange-Id: I75da1e9309e0f7ef8839dea3ec9c99c58edc5d63\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/f95d4796f41287d1539b7f32313f0fb8bd47757d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/f95d4796f41287d1539b7f32313f0fb8bd47757d"}]},"branch":"refs/heads/master"},"e346f0771712f60cc269b011bf1f682e09783c7c":{"kind":"REWORK","_number":5,"created":"2014-12-02 13:09:07.000000000","uploader":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"ref":"refs/changes/83/138183/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/83/138183/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/83/138183/5"}}},"commit":{"parents":[{"commit":"f8d812054f533c2d467e62d6e588a1832cd17e37","subject":"Merge \"Correct GlanceStoreException to provide valid message - Glance\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/f8d812054f533c2d467e62d6e588a1832cd17e37"}]}],"author":{"name":"Flavio Percoco","email":"flaper87@gmail.com","date":"2014-12-01 13:45:44.000000000","tz":60},"committer":{"name":"Flavio Percoco","email":"flaper87@gmail.com","date":"2014-12-02 13:08:27.000000000","tz":60},"subject":"Allow some fields to be None","message":"Allow some fields to be None\n\nContainer, disk format and checksum - among other fields - are not\nrequired parameters, therfore this patch allows them to be None.\n\nThis will allow the client to have attributes with value `None` and\nit\u0027ll make it consistent across responses and with regards to the API.\n\nNOTE: A follow-up patch will enable None values to be returned by the API.\n\nCloses-bug: #1398313\n\nChange-Id: I75da1e9309e0f7ef8839dea3ec9c99c58edc5d63\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/e346f0771712f60cc269b011bf1f682e09783c7c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/e346f0771712f60cc269b011bf1f682e09783c7c"}]},"branch":"refs/heads/master"},"8b3ca087b7d3bbc1fb88daf6e780efbc1f0dcae8":{"kind":"REWORK","_number":6,"created":"2014-12-02 17:26:35.000000000","uploader":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"ref":"refs/changes/83/138183/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/83/138183/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/83/138183/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/83/138183/6"}}},"commit":{"parents":[{"commit":"f8d812054f533c2d467e62d6e588a1832cd17e37","subject":"Merge \"Correct GlanceStoreException to provide valid message - Glance\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/f8d812054f533c2d467e62d6e588a1832cd17e37"}]}],"author":{"name":"Flavio Percoco","email":"flaper87@gmail.com","date":"2014-12-01 13:45:44.000000000","tz":60},"committer":{"name":"Flavio Percoco","email":"flaper87@gmail.com","date":"2014-12-02 17:25:14.000000000","tz":60},"subject":"Allow some fields to be None","message":"Allow some fields to be None\n\nContainer, disk format and checksum - among other fields - are not\nrequired parameters, therfore this patch allows them to be None.\n\nThis will allow the client to have attributes with value `None` and\nit\u0027ll make it consistent across responses and with regards to the API.\n\nNOTE: A follow-up patch will enable None values to be returned by the API.\n\nCloses-bug: #1398313\n\nChange-Id: I75da1e9309e0f7ef8839dea3ec9c99c58edc5d63\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/8b3ca087b7d3bbc1fb88daf6e780efbc1f0dcae8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/8b3ca087b7d3bbc1fb88daf6e780efbc1f0dcae8"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}