Debian Bug report logs - #709773
Wrong handling of debconf

Toggle useless messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Thomas Goirand <zigo@debian.org>

To: Debian Bug Tracking System <submit@bugs.debian.org>

Subject: Wrong handling of debconf

Date: Sat, 25 May 2013 18:31:35 +0800

Package: cloud-init
Version: 0.7.1-3
Severity: serious

Cloud-init has a debconf thing to handle which type of API should be
supported. Though it is done wrong:

1/ Preseeding doesn't work
2/ If /etc/cloud/cloud.cfg.d/90_dpkg.cfg exists, then the currently
configured avlue doesn't seem to be read in the config script.

Cheers,

Thomas Goirand (zigo)

Message #10 received at 709773@bugs.debian.org (full text, mbox, reply):

From: Charles Plessy <plessy@debian.org>

To: 709773@bugs.debian.org, debian-cloud@lists.debian.org

Subject: Re: Bug#709773: Wrong handling of debconf

Date: Sat, 25 May 2013 22:41:39 +0900

Le Sat, May 25, 2013 at 06:31:35PM +0800, Thomas Goirand a écrit :
> Package: cloud-init
> Version: 0.7.1-3
> Severity: serious
> 
> Cloud-init has a debconf thing to handle which type of API should be
> supported. Though it is done wrong:
> 
> 1/ Preseeding doesn't work
> 2/ If /etc/cloud/cloud.cfg.d/90_dpkg.cfg exists, then the currently
> configured avlue doesn't seem to be read in the config script.

Hi Thomas,

this might (or might not) be solved already in Ubuntu's package, which is ahead
(0.7.2~bzr809).  Now that upstream version 0.7.2 is out, we can probably
attempt to merge changes related to debconf from Ubuntu.

I have done the boring part (updated debian/copyright after finding three new
copyright notices and one new license), so if you or somebody else want to work
on an update to 0.7.2 that would solve this bug and #709173, please go ahead,
the road is clear.

(But I need to add that on my system, the package builds only with sudo, not
with fakeroot)

Have a nice week-end,

-- 
Charles Plessy
Tsurumi, Kanagawa, Japan

Message #15 received at 709773@bugs.debian.org (full text, mbox, reply):

From: Thomas Goirand <zigo@debian.org>

To: 709773@bugs.debian.org

Subject: Re: Bug#709773: Wrong handling of debconf

Date: Sat, 25 May 2013 22:33:40 +0800

On 05/25/2013 09:41 PM, Charles Plessy wrote:
> Le Sat, May 25, 2013 at 06:31:35PM +0800, Thomas Goirand a écrit :
>> Package: cloud-init
>> Version: 0.7.1-3
>> Severity: serious
>>
>> Cloud-init has a debconf thing to handle which type of API should be
>> supported. Though it is done wrong:
>>
>> 1/ Preseeding doesn't work
>> 2/ If /etc/cloud/cloud.cfg.d/90_dpkg.cfg exists, then the currently
>> configured avlue doesn't seem to be read in the config script.
> 
> Hi Thomas,
> 
> this might (or might not) be solved already in Ubuntu's package, which is ahead
> (0.7.2~bzr809). Now that upstream version 0.7.2 is out, we can probably
> attempt to merge changes related to debconf from Ubuntu.

I have just checked, and the config and postinst script from the Ubuntu
package didn't change between 0.7.1 and 0.7.2~bzr809. So no, it's not fixed.

By the way, while I'm sure preseeding didn't work, I'm not sure about
point 2/ above. I just found the debian/cloud-init.config script very
suspicious. Normally, in such script, we would:

1/ Check if the config file is there. If it is, then:
2/ get the configured value, and set it with db_set.
3/ Do the db_input + db_go

But in this script, I can read (simplified the logic):

db_get cloud-init/datasources
case " ${RET}," in
  *\ Ec2,*) :;;
  *) hasEc2Md &&
    db_set cloud-init/datasources "${def:+${def}, }Ec2";;

This doesn't seem right, and looks like a "using debconf as registry"
type of bug to me.

Your thoughts here?

> I have done the boring part (updated debian/copyright after finding three new
> copyright notices and one new license), so if you or somebody else want to work
> on an update to 0.7.2 that would solve this bug and #709173, please go ahead,
> the road is clear.

Thanks for this.

Also, I'm hitting another problem. I haven't reported it (yet) because
I'm not sure if the problem is on my setup or otherwise.

While the GROWPART in the cloud-initramfs-growroot works (after I
somehow fixed it), the resize2fs resize from cloud-init doesn't work for
me using OpenStack. Here's cloud-init output:

util.py[WARNING]: Failed to resize filesystem (cmd=['resize2fs',
'/run/cloudinit.resizefs.nEQ1UK'])
util.py[WARNING]: Running resizefs (<module
'cloudinit.config.cc_resizefs' from
/usr/lib/python2.7/dist-packages/cloudinit/config/cc_resizefs.pyc'>) failed

This is quite annoying me, because that's the last bit which I need to
fix in order to finish my work on the Debian OpenStack cloud image
(which otherwise work quite well!). Any help would be really appreciated.

Also, I wonder, how does cloud-init do for resizing the root filesystem,
since when cloud-init start, the rootfs is already mounted read/write?
Shouldn't this happen in the initrd like with GROWPART rather than in
the boot process?

Cheers,

Thomas Goirand (zigo)

Message #22 received at 709773-close@bugs.debian.org (full text, mbox, reply):

From: Thomas Goirand <zigo@debian.org>

To: 709773-close@bugs.debian.org

Subject: Bug#709773: fixed in cloud-init 0.7.2-3

Date: Mon, 22 Jul 2013 16:33:04 +0000

Source: cloud-init
Source-Version: 0.7.2-3

We believe that the bug you reported is fixed in the latest version of
cloud-init, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 709773@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Thomas Goirand <zigo@debian.org> (supplier of updated cloud-init package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Tue, 09 Jul 2013 23:10:34 +0800
Source: cloud-init
Binary: cloud-init
Architecture: source all
Version: 0.7.2-3
Distribution: unstable
Urgency: low
Maintainer: Python Applications Packaging Team <python-apps-team@lists.alioth.debian.org>
Changed-By: Thomas Goirand <zigo@debian.org>
Description: 
 cloud-init - initialization system for infrastructure cloud instances
Closes: 709173 709773 710444 711797 712985 714093 714356 714608 714646 714738 714785 715168 715241 715390 715500 715799
Changes: 
 cloud-init (0.7.2-3) unstable; urgency=low
 .
   [ Charles Plessy ]
   * Added python-requests to build dependencies (Closes: #711797).
   * Conforms with Policy 3.9.4.
   * Depend on ${python:Depends}.
   * Description and Debconf messages reviewed by the Smith project.
     Closes: #710444, thanks to Justin B Rye and Christian Perrier.
   * German translation (Closes: #714738, thanks to Chris Leick).
   * Swedish translation (Closes: #714646, thanks to Martin Bagge).
   * French translation (Closes: #714608, thanks to Julien Patriarca).
   * Russian translation (Closes: #714093, #715168, thanks to Yuri Kozlov).
   * Danish translation (Closes: #714785, thanks to Joe Dalton).
   * Czech tranlsation (Closes: #714356, thanks to Michal Šimůnek).
   * Portugese tranlsation (Closes: #715241, thanks to Américo Monteiro).
   * Japanese translation (Closes: #715500, thanks to victory).
   * Italian translation (Closes: #715390, thanks to Beatrice Torracca).
   * Polish translation (Closes: #715799, thanks to Michał Kułach).
 .
   [ Thomas Goirand ]
   * Added missing init scripts cloud-init-local, cloud-init-final and
     cloud-config, to run cloud-init as below (Closes: #709173):
     - modules --mode config (cloud-config)
     - modules --mode final (cloud-init-final)
     - init --local (cloud-init-local)
   * Reads the currently set value in /etc/cloud/cloud.cfg.d/90_dpkg.cfg to
     db_set the value of cloud-init/datasources (Closes: #709773).
   * Check if upstart is init in all init scripts (Closes: #712985). Thanks to
     Dmitrijs Ledkovs for reporting and his patch in the BTS.
Checksums-Sha1: 
 1cc3dc823f530ee73e5c963c6d599366add520cb 1620 cloud-init_0.7.2-3.dsc
 52c3a7cbba7c2b514d5e6b84f4c302b8366e22f0 30338 cloud-init_0.7.2-3.debian.tar.gz
 e9192f65c429e8d8ff2985f31ec3de3fe5390f8f 212640 cloud-init_0.7.2-3_all.deb
Checksums-Sha256: 
 cc7584526eaa019902aeb27d3d277b0bc048f6de4ed0be6f2c5d5b2e0e1add1d 1620 cloud-init_0.7.2-3.dsc
 60fa8203b811bb667c16ec9b80b682ed42d0d0ad755b5957b2aafa3716f4a9bc 30338 cloud-init_0.7.2-3.debian.tar.gz
 bc2817f50b694e8670b407f9e5b1680cf142b65421549123b7dcbc5cf5fe693c 212640 cloud-init_0.7.2-3_all.deb
Files: 
 e113e975b50f101c31369687f12aea94 1620 admin optional cloud-init_0.7.2-3.dsc
 f3a2b42d5e04be89fc3d13fc9d2567e6 30338 admin optional cloud-init_0.7.2-3.debian.tar.gz
 08ced503542f64a5c5927beaf20af642 212640 admin optional cloud-init_0.7.2-3_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)

iEYEARECAAYFAlHtW3oACgkQl4M9yZjvmkk0TwCfefMOdyxWMO8vmUxU/CMQ+U/U
sO4An1Vp6VLjH5w5+/MqGRK1mdK0ARIv
=hwZZ
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.