[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#719632: pu: package nova/2012.1.1-18



Hi Thomas,

here's my take on this proposed update.

On Tue, Aug 13, 2013 at 20:46:34 +0200, Thomas Goirand wrote:

> From: Thomas Goirand <thomas@goirand.fr>
> Date: Sat, 13 Jul 2013 22:37:23 +0800
> Subject: [PATCH 2/6] Do not use file from /usr/share/doc/nova-compute-xen in
>  postinst (Closes: #710507)
> 
Not appropriate for stable IMO.  The likelyhood of somebody wanting to
both configure dpkg to strip /usr/share/doc, and installing openstack,
seems about as close to 0 as it gets.

> From: Thomas Goirand <thomas@goirand.fr>
> Date: Sat, 13 Jul 2013 23:17:23 +0800
> Subject: [PATCH 6/6] Applies https://review.openstack.org/#/c/10168/:
>  resolves issue where querying /v1.1/$tenant/os-hosts
>  returns an empty list (Closes: #689318).
> 
ok, I guess.

> From: Thomas Goirand <thomas@goirand.fr>
> Date: Sat, 13 Jul 2013 22:48:06 +0800
> Subject: [PATCH 3/6] Updates debian/gbp.conf to use the debian/wheezy branch
>  for building.
> 
meh.  ok.

> From: Thomas Goirand <thomas@goirand.fr>
> Date: Sat, 13 Jul 2013 22:32:52 +0800
> Subject: [PATCH 1/6]   * CVE-2013-2096: [OSSA 2013-012] Check QCOW2 image
>  size during root disk     creation (Closes: #710157).  
>  * Refreshes various patches (removes offsets when
>  applying).
> 
the cve fix is probably ok.  the refresh not so much.

> From: Thomas Goirand <thomas@goirand.fr>
> Date: Sat, 13 Jul 2013 23:06:28 +0800
> Subject: [PATCH 5/6]   * Add optional postgresql dependency to a number of
>  init script to ensure     proper startup ordering if
>  nova is configured to use a local postgresql    
>  backend. (applied patch from jcristau).
> 
I'm not sure this warrants a stable update.  The default config uses
sqlite which doesn't have this ordering requirement, and anybody who
decides to use postgres instead can probably figure this one out (plus
conffile changes cause prompts which are annoying IMO in a stable
update).

Cheers,
Julien

Attachment: signature.asc
Description: Digital signature


Reply to: