[Merge] lp:~thedac/cinder/kilo into lp:~ubuntu-server-dev/cinder/kilo
Corey Bryant
corey.bryant at canonical.com
Mon Aug 3 19:23:17 UTC 2015
Thanks David. Looking pretty good. A couple of comments below.
Diff comments:
> === modified file 'debian/changelog'
> --- debian/changelog 2015-05-03 17:40:59 +0000
> +++ debian/changelog 2015-08-03 19:03:24 +0000
> @@ -1,3 +1,39 @@
> +cinder (1:2015.1.1-0ubuntu1) vivid; urgency=medium
You can keep this as UNRELEASED instead of vivid, and I'll change it when I upload. I'm not sure why tbh but that's what my predecessors had me do before I had upload rights. :) I assume it prevents some sort of mistake.
> +
> + [ David Ames ]
You can drop your name from here. If 2+ people contribute changes to a version then we'll add their names before their changes, otherwise we can just go off the signature.
> + * New upstream point release for Openstack Kilo (1cca93c) (LP: #1481008)
I know it's nit picky but let's stick with "Resynchronize with stable/kilo" so we have consistency across the packages. Also swap out 1cca93c in favor of 651bab0, using the last commit number instead.
> + - [1cca93c] Updated from global requirements
You can drop the commits that aren't of interest to users, like this one.
> + - [b130f00] Fix NetApp cDOT driver use of Glance locations
> + - [3388fff] Volume manager should set filter_function and goodness_function
> + - [e1143ae] Updated from global requirements
and this one
> + - [08ff005] Update expected error message from lvs
> + - [27b5e75] Revert "Graceful shutdown WSGI/RPC server"
> + - [4f86ae7] Fix 'no actual-pathname' NetApp API error
> + - [70c6a35] Graceful shutdown WSGI/RPC server
> + - [6c05594] Add exception catch in report_state for DBError
> + - [50db0fc] Fix broken export commands on block_device driver
> + - [9634b76] Disallow backing files when uploading volumes to image
> + - [223b258] Fix cinder concurrency issues on rtstool
> + - [ea9e2e0] Create iSCSI lio portals with right IPs and port
> + - [f8149fc] Dispose DB connections between backend proc starts
> + - [28bb8d2] Allow provisioning to reach max oversubscription
> + - [f3203aa] Updated from global requirements
and this one
> + - [0490f1c] Create initiator id for VMAX iSCSI driver
> + - [28df54c] Fix capacity filter to allow oversubscription
> + - [c56bbb7] VMware: Fix missing target resource pool
> + - [03562c3] Check volume_backend in retype
> + - [5ff9497] Use nfs_oversub_ratio when reporting pool capacity
> + - [9affeaf] NFS Backup: Correcting backup_sha_block_size_bytes help message
> + - [c0f12f2] LVM: Pass volume size in MiB to copy_volume() during volume migration
> + - [e316d6f] Dell: Added support for update_migrated_volume
> + - [f0a4447] Allow rexports for targets with Datera
> + - [dabbaed] HDS HNAS Driver fails when FS unhealthy
> + - [a5908fd] Revert state if attachment already exists
> + - [e384ef0] Bump pre-release to 2015.1.1
and this one
> + - [651bab0] VolMgr: reschedule only when filter_properties has retry
Also since you modified a patch please add a changelog for that, such as:
* d/p/fix-requirements.patch: Rebased.
> +
> + -- David Ames <david.ames at canonical.com> Mon, 03 Aug 2015 10:43:21 -0700
> +
> cinder (1:2015.1.0-0ubuntu1) vivid; urgency=medium
>
> [ Chuck Short ]
--
https://code.launchpad.net/~thedac/cinder/kilo/+merge/266779
Your team Ubuntu Server Developers is requested to review the proposed merge of lp:~thedac/cinder/kilo into lp:~ubuntu-server-dev/cinder/kilo.
More information about the Ubuntu-reviews
mailing list