Bug#853744: cloud-init needs net-tools

[Sam Hartman]

> "Marco" == Marco d'Itri  writes:

Marco> On Jan 31, Sam Hartman  wrote:
>> Why?  I can understand "it would be nice if cloud-init used ip
>> instead", but you seem to have a preference stronger than that.
Marco> To save space on the images.

I think having it work is more important than space.
I think applying the patch and then opening a wishlist bug to use
iproute2 would be a good path forward.

Bug#853744: cloud-init needs net-tools

[Marcin Kulisz]

On 2017-01-31 17:14:04, Marco d'Itri wrote:
> On Jan 31, Sam Hartman  wrote:
> 
> > Why?
> > I can understand "it would be nice if cloud-init used ip instead", but
> > you seem to have a preference stronger than that.
> To save space on the images.

Lol nice one :-)
Installed-Size: 963 kB

IMO we should ship it in Stretch with net-tools and we can consider switching
in the new cycle.
-- 

|_|0|_|  |
|_|_|0|  "Panta rei" |
|0|0|0|  kuLa    |

gpg --keyserver pgp.mit.edu --recv-keys 0x686930DD58C338B3
3DF1  A4DF  C732  4688  38BC  F121  6869  30DD  58C3  38B3


signature.asc
Description: PGP signature

Bug#853744: cloud-init needs net-tools

[Marco d'Itri]

On Jan 31, Sam Hartman  wrote:

> Why?
> I can understand "it would be nice if cloud-init used ip instead", but
> you seem to have a preference stronger than that.
To save space on the images.

-- 
ciao,
Marco


signature.asc
Description: PGP signature

Bug#853744: cloud-init needs net-tools

[Ross Vandegrift]

On Tue, Jan 31, 2017 at 04:48:24PM +0100, Marco d'Itri wrote:
> On Jan 31, Ross Vandegrift  wrote:
> 
> > Recently, net-tools was made optional.  Since cloud-init does not depend
> > on net-tools, this causes breakage:
> Please do not apply this patch! Fix cloud-init to use ip(8) instead.

Isn't it too late in the stretch freeze?  If not, and it can be fixed,
great!  Otherwise please consider, as cloud-init will be broken in
stretch.

If it's more acceptable, the FAI built cloud images could include
net-tools even though it's optional.

Ross


signature.asc
Description: PGP signature

Bug#853744: cloud-init needs net-tools

[Sam Hartman]

> "Marco" == Marco d'Itri  writes:

Marco> On Jan 31, Ross Vandegrift  wrote:
>> Recently, net-tools was made optional.  Since cloud-init does not
>> depend on net-tools, this causes breakage:
Marco> Please do not apply this patch! Fix cloud-init to use ip(8)
Marco> instead.

Why?
I can understand "it would be nice if cloud-init used ip instead", but
you seem to have a preference stronger than that.
Also, does cloud-init work on anything non-Linux?

Bug#853744: cloud-init needs net-tools

[Marco d'Itri]

On Jan 31, Ross Vandegrift  wrote:

> Recently, net-tools was made optional.  Since cloud-init does not depend
> on net-tools, this causes breakage:
Please do not apply this patch! Fix cloud-init to use ip(8) instead.

-- 
ciao,
Marco


signature.asc
Description: PGP signature

Bug#853744: cloud-init needs net-tools

[Ross Vandegrift]

Package: cloud-init
Version: 0.7.9-1
Severity: important
Tags: patch

Recently, net-tools was made optional.  Since cloud-init does not depend
on net-tools, this causes breakage:

$ sudo cloud-init init
Cloud-init v. 0.7.9 running 'init' at Tue, 31 Jan 2017 14:27:17 +. Up 
1432.75 seconds.
2017-01-31 14:27:17,650 - util.py[WARNING]: Route info failed: Unexpected error 
while running command.
Command: ['netstat', '-rn']
Exit code: -
Reason: [Errno 2] No such file or directory: 'netstat'
Stdout: -
Stderr: -
ci-info: !Net device info 
failed!
ci-info: !!!Route info 
failed

Trivial fix proposal is attached.

Ross


-- System Information:
Debian Release: 9.0
  APT prefers testing
  APT policy: (500, 'testing'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 4.9.0-1-amd64 (SMP w/1 CPU core)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages cloud-init depends on:
ii  debconf [debconf-2.0]  1.5.59
ii  gdisk  1.0.1-1
ii  ifupdown   0.8.18
ii  init-system-helpers1.46
ii  lsb-release9.20161125
ii  procps 2:3.3.12-3
ii  python33.5.1-4
ii  python3-configobj  5.0.6-2
ii  python3-jinja2 2.8-1
ii  python3-jsonpatch  1.19-4
ii  python3-oauthlib   2.0.1-1
ii  python3-prettytable0.7.2-3
ii  python3-requests   2.12.4-1
ii  python3-six1.10.0-3
ii  python3-yaml   3.12-1
pn  python3:any

cloud-init recommends no packages.

cloud-init suggests no packages.

-- debconf information:
* cloud-init/datasources:Ec2
diff --git a/debian/control b/debian/control
index c0c855a7..8136aaea 100644
--- a/debian/control
+++ b/debian/control
@@ -45,6 +45,7 @@ Depends:
   lsb-release,
   ifupdown,
   procps,
+  net-tools,
   ${misc:Depends},
   ${python3:Depends}
 Description: initialization system for infrastructure cloud instances