{"push_id":817753007,"size":245,"distinct_size":245,"ref":"refs/heads/master","head":"b4e5c3ef9a9b12bb1c47f45b1bc4e8afb3033304","before":"8a7a7e409720606f1e9006bbdd7c0ea6fd6187c3","commits":[{"sha":"d7f7e0b45f44ddf8ef215215394714ff00bff2e2","author":{"email":"6972bdd7dec75710593d62d13eab0eaf0ce26d83@suse.de","name":"Ralf Haferkamp"},"message":"Fix block_device driver to behave as documented\n\nAccording to the original blueprint\n(https://blueprints.launchpad.net/cinder/+spec/block-device-driver) the\nintended behavior of the \"BlockDeviceDriver\" was to directly attach the block\ndevice to the instances using virtio (without LVM and iSCSI involved) when the\nvolume's underlying device is located on the same host as the instance that it\nis going to be attached to. This is also how it worked through Juno. It got\nbroken during the Kilo cycle as part of this change:\nhttps://review.openstack.org/#/c/135139/\nIt seems that the change in behavior was not intended by that patch as it\naddressed something completely different. And it was neither mentioned in the\ncommit message nor anywhere in the review comments.\n\nThis patch restores the original behavior.\n\nChange-Id: Ic22f03860a43b767e9097ce6dc5b397cf9f3f076\nCloses-Bug: 1473001","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/d7f7e0b45f44ddf8ef215215394714ff00bff2e2"},{"sha":"eec3512f66e9771ede5443ec374a595281517108","author":{"email":"694f58a821bf423801150f81b19ffeab56e6818d@emc.com","name":"Xing Yang"},"message":"Create CG needs extra specs\n\nGet volume types no longer returns extra specs for non-admin users.\nThis breaks CG creation which needs extra specs. This patch fixes it\nby elevating the context when getting volume types for CG creation.\n\nChange-Id: I610149e8bf9a067f78a9ad52eb02b2da41dc64f6\nCloses-Bug: #1495758","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/eec3512f66e9771ede5443ec374a595281517108"},{"sha":"87b6072b8401528c5582f48ec3cc5b52049a3774","author":{"email":"694f58a821bf423801150f81b19ffeab56e6818d@emc.com","name":"Xing Yang"},"message":"Extra specs may not be in volume types\n\nGet volume types no longer returns extra specs for non-admin users.\nThis breaks volume creation which needs extra specs. This patch fixes it\nas follows:\n* In volume/api.py, get extra specs when checking for retype.\n* In scheduler/filter_scheduler.py, get extra specs if they are not\n  already in volume type.\n\nCloses-Bug: 1495764\nChange-Id: I1eca87d14cce84596628f0e8b3ea1602914bd883","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/87b6072b8401528c5582f48ec3cc5b52049a3774"},{"sha":"5b0a3196e726cbbab1828e99976d677866f4d5d5","author":{"email":"941760a5d79f418fbb85781c6e644b268f4c9136@redhat.com","name":"Eric Harney"},"message":"Small cleanups in BaseVD/VolumeDriver\n\nself._throttle is defined outside of __init__.\n\nVolumeDriver's initialize_connection signature doesn't\nmatch the method it overrides from BaseVD.\n\nChange-Id: Ie9f741ad331a79e3a85fd586f6393f8f80aa2932","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/5b0a3196e726cbbab1828e99976d677866f4d5d5"},{"sha":"ce575378da90ee3772438095b81a004ed7553102","author":{"email":"694f58a821bf423801150f81b19ffeab56e6818d@emc.com","name":"Xing Yang"},"message":"Check sio_storage_pools in check_for_setup_error\n\nThe ScaleIO driver fails to load if sio_storage_pools is not\nspecified in cinder.conf. This patch checks it in check_for_setup_error\nand also check it in __init__.\n\nChange-Id: I0fed70848ad2ecf3899b07058fabfa68df07edbd\nCloses-Bug: #1497413","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/ce575378da90ee3772438095b81a004ed7553102"},{"sha":"f6d531bf56f080fc7a6ea6ba63f08be249b50a51","author":{"email":"b7d82c9eb00b2557d3b70ebca40b6c66d180932a@intel.com","name":"Michal Dulko"},"message":"Add consistency group tests to test_volume_rpcapi\n\ntest_volume_rpcapi was missing tests for RPC calls related to\nconsistency groups. This commit adds them.\n\nChange-Id: I08a1337467742c8ce2b24c9382df537841bdbcb2","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/f6d531bf56f080fc7a6ea6ba63f08be249b50a51"},{"sha":"de14a81b7370f4af954bc3c533947f51686fe5f5","author":{"email":"9b03ac33999704426bdf2fa09f447e2093ebaafa@intel.com","name":"Szymon Wroblewski"},"message":"Python 3 incompatible expression fix\n\nIn Python 3 dict.keys() return iterable view which\ndoes not support indexing. This patch fixes the issue by using\nappropriate iterator functions.\n\nImplements: bp cinder-python3\nChange-Id: Ib31531e5890fb050ce3c03ea0a078afd2122abe4","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/de14a81b7370f4af954bc3c533947f51686fe5f5"},{"sha":"6831f27976ffbc1d9ec77c9b68b04559406f14b6","author":{"email":"ac84f775b4e581f8fe1f02661f66a69d7356479c@easystack.cn","name":"ChangBo Guo(gcb)"},"message":"Use version convert methods from oslo.utils\n\noslo.utils provides version convert methods in versionutils[1], so it is\nredundant to have them in Cinder.\n\n[1]https://github.com/openstack/oslo.utils/blob/master/oslo_utils/versionutils.py\n\nCloses-Bug: #1487273\n\nChange-Id: Ic8ba775bc067759319829a1bcbde2660ddeb921e","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/6831f27976ffbc1d9ec77c9b68b04559406f14b6"},{"sha":"9e494b1be69d177f43fddbc7cd7b99baec0f72bd","author":{"email":"941760a5d79f418fbb85781c6e644b268f4c9136@redhat.com","name":"Eric Harney"},"message":"Remove driver.set_execute()\n\nThis method doesn't appear to really be needed,\nand mostly exists to help unit tests.  Update a\nfew tests to mock out the required execute calls,\nand remove the method to make the driver interface\na bit simpler.\n\nChange-Id: I70d8a7e74761ce7bc1e51bf9f3c49eb0ce0986e0","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/9e494b1be69d177f43fddbc7cd7b99baec0f72bd"},{"sha":"d548495e9e1ec2ad995060112308fbf8da79c809","author":{"email":"59dfe7e004393a2f89ad78e3a810b4cb6d0cd637@redhat.com","name":"Victor Stinner"},"message":"Port volume transfer to Python 3\n\nEncode Unicode to UTF-8 for salt and authentication key when computing\nthe crypt hash.\n\nPartially implements: blueprint cinder-python3\nChange-Id: I19d337cf0d40d91378e1c42061bc51b6009970a2","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/d548495e9e1ec2ad995060112308fbf8da79c809"},{"sha":"d95bd7c37666c849e2e6283b2269ec0ef8d2fd10","author":{"email":"59dfe7e004393a2f89ad78e3a810b4cb6d0cd637@redhat.com","name":"Victor Stinner"},"message":"Port image_utils to Python 3\n\nPort cinder.image_utils.check_qemu_img_version() to Python 3.\n\nAdd the following tests to Python 3.4 in tox.ini:\n\n* cinder.tests.unit.test_image_utils\n* cinder.tests.unit.test_migrations\n* cinder.tests.unit.test_misc\n* cinder.tests.unit.test_service\n\nPartially implements: blueprint cinder-python3\nChange-Id: I69feda94c90206adc8771fbd5ddc6aae5bb6b64b","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/d95bd7c37666c849e2e6283b2269ec0ef8d2fd10"},{"sha":"1d1695838d4291e92043a31893d8cb103a26f627","author":{"email":"e771d771c72ed1cb908097fce7d8ff44e64e7b00@cn.ibm.com","name":"Vincent Hou"},"message":"Remove the unnecassary volume_api.get(context, volume_id)\n\nIn the method delete from volume_image_metadata, volume_api.get()\nmay be called twice. This first time is to get the metadata, and\nthe second is to get the volume. We can get both of them at the\nsame time.\n\nChange-Id: I0633c16db21e254ec43bad472748cbc0b7796a24\nCloses-Bug: #1489342","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/1d1695838d4291e92043a31893d8cb103a26f627"},{"sha":"34a28c302c6fbde5cf5307e16bb0f746d3ee03c3","author":{"email":"8394850d429a0e4e8473b32fa0e3517ffc7d0ee3@lab.ntt.co.jp","name":"Teruaki Ishizaki"},"message":"Sheepdog: Improve snapshot and clone operation\n\nThis patch changes SheepdogDriver Class methods to use\nSheepdogClient Class methods for detailed Error logs.\n\nWe change the following SheepdogDriver methods.\n\n- create_snapshot\n- delete_snapshot\n- create_volume_from_snapshot\n- create_cloned_volume\n- extend_volume\n\nSheepdogClient Class methods are implemented to enable\nfine grained Error handling.\n\nSo, the administrators are enable to do error recovery\nmore easily using those logs.\n\nChange-Id: Ib126088e4100739db013b21970051af1f29790d5","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/34a28c302c6fbde5cf5307e16bb0f746d3ee03c3"},{"sha":"075ff30d7d8bbeca1af634718f3cb19099bc44b3","author":{"email":"5bf32be10a6cecf10540ea7cc88e8b11d4db1769@oracle.com","name":"Abhiram Moturi"},"message":"ZFSSA driver to return project 'available' space\n\nThis fix allows the iSCSI driver to return the 'available' space\nproperty at project level instead of the pool level which is more\naccurate in cases when storage is not thin provisioned.\n\nChange-Id: I52dec5e527eab393fd464fbc7f4f910fafb67268\nCloses-Bug: #1479342","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/075ff30d7d8bbeca1af634718f3cb19099bc44b3"},{"sha":"e065e70a69b1b6de1bd383cb471dbab62b93e03a","author":{"email":"1914a63836d1b0b3dacd38e82e8facc9fa43c804@e0ne.info","name":"Ivan Kolodyazhny"},"message":"Remove empty rules from policies for API access\n\nEmpty policy rule means that API method is allowed by anybody. Default rule\nis used only if such rule is not defined in policy.json.\n\nThis patch changes empty rules to admin_api or admin_or_owner.\n\nCloses-Bug: #1477621\nCloses-Bug: #1491495\nChange-Id: I512e65e62da7dab5834a21ef9fd967ed6e9bb695","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/e065e70a69b1b6de1bd383cb471dbab62b93e03a"},{"sha":"5a2efeb6ab383e34d56a4f166c05fc3b6981dc4c","author":{"email":"fb460628c014c8c87225cf12a730c4f36aef46ff@huawei.com","name":"chenying"},"message":"Fix order of arguments in assertEqual\n\nSome tests used incorrect order assertEqual(observed, expected).\nThe correct order expected by testtools is\nassertEqual(expected, observed).\nThis patch does not cover v1 api tests and vendor unit tests.\n\nPartial-Bug: #1486906\nChange-Id: I4671cc75c999d22ea9c181c515c67d03b7d54c71","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/5a2efeb6ab383e34d56a4f166c05fc3b6981dc4c"},{"sha":"223fbd36b7ec433040f5b95dcbaacf46f6939b79","author":{"email":"941760a5d79f418fbb85781c6e644b268f4c9136@redhat.com","name":"Eric Harney"},"message":"Add migrate_volume to BaseVD\n\nmigrate_volume() only exists in the deprecated VolumeDriver()\nclass and not BaseVD.  This means that drivers inheriting from\n*VD hit attribute errors when trying to call driver.migrate_volume().\n\nI suspect this indicates that we should rework some of our\nabc class structure a little bit, but this seems like a\nreasonable fix for now.\n\nRelated-Bug: #1478987\nRelated-Bug: #1471807\nCloses-Bug: #1493286\n\nChange-Id: I3d555507416606db265b8ebe335b51a08fe32328","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/223fbd36b7ec433040f5b95dcbaacf46f6939b79"},{"sha":"7edc87ff0e92043f5a21d630bbb9b966f5e34535","author":{"email":"941760a5d79f418fbb85781c6e644b268f4c9136@redhat.com","name":"Eric Harney"},"message":"Add manage_existing and unmanage to BaseVD\n\nmanage_existing() and unmanage() only exist in the deprecated\nVolumeDriver() class and not BaseVD.  This means that drivers\ninheriting from *VD hit attribute errors when trying to call\neither of these methods.\n\nRelated-Bug: #1478987\nRelated-Bug: #1493286\nRelated-Bug: #1471807\nCloses-Bug: #1487329\n\nChange-Id: Ie8aba971b51e1f5eaac5a4e681ea06b358ebb7b8","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/7edc87ff0e92043f5a21d630bbb9b966f5e34535"},{"sha":"327df72aec6f9b03f513e74e0a22bb40ce590712","author":{"email":"6c8ef7b17188f0c8ec1713daf5ef7aca67f02fc4@purestorage.com","name":"Patrick East"},"message":"Correctly report multiattach in Pure drivers\n\nAs-is we end up ‘reporting’ to the scheduler that Pure backends do not\nsupport multiattach. This is confusing because volumes created on a\nPure FlashArray will always be capable of doing multiattach even if not\nspecified by Cinder.\n\nThis change syncs up the reporting to the scheduler and what is actually\nhappening on the array.\n\nChange-Id: I3a1e519efdd0c27f63b049aa686ff9045c260ab7\nCloses-Bug: #1493478","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/327df72aec6f9b03f513e74e0a22bb40ce590712"},{"sha":"bf22aeb185afe8de0db526c83e50c15113f84d88","author":{"email":"941760a5d79f418fbb85781c6e644b268f4c9136@redhat.com","name":"Eric Harney"},"message":"Add unmanage default implementation to VolumeDriver\n\nCurrently, only drivers that inherit from ManageableVD\nget the default unmanage() implementation.\n\nThis is not correct, because drivers can support unmanage\nwithout driver code.  Fix unmanage() in VolumeDriver to\nsupport the LVM driver's unmanage operation.\n\n(Regression from the refactoring done when introducing\nabc classes.)\n\nCloses-Bug: #1493544\n\nChange-Id: Iaacf10141559a2c88b5873722d1e237dc9223f27","distinct":true,"url":"https://api.github.com/repos/dims/cinder/commits/bf22aeb185afe8de0db526c83e50c15113f84d88"}]}