cloud-init-dev team mailing list archive

[Ryan Harper <ryan.harper@xxxxxxxxxxxxx>]

On Tue, Oct 18, 2016 at 3:03 PM, Ryan Harper <ryan.harper@xxxxxxxxxxxxx>
wrote:

>
>
> On Tue, Oct 18, 2016 at 2:49 PM, Scott Moser <smoser@xxxxxxxxxx> wrote:
>
>> $ cat go.sh
>> #!/bin/sh
>> deps="
>> apt cloud-guest-utils coreutils dash debconf e2fsprogs gnupg2 hostname
>> ifupdown iproute2 locales lsb-release mount net-tools passwd procps sed
>> ssh-import-id systemd udev
>> "
>> for dep in $deps; do
>>    apt-cache show $dep | grep -qi "^Essential:" && r=essential || r=not
>>    printf "%-25s %s\n" "$dep" "$r"
>> done
>>
>> $ sh ./go.sh
>> apt                       not
>> cloud-guest-utils         not
>> coreutils                 essential
>> dash                      essential
>> debconf                   not
>> e2fsprogs                 essential
>> gnupg2                    not
>> hostname                  essential
>> ifupdown                  not
>> iproute2                  not
>> locales                   not
>> lsb-release               not
>> mount                     essential
>> net-tools                 not
>> passwd                    not
>> procps                    not
>> sed                       essential
>> ssh-import-id             not
>> systemd                   not
>> udev                      not
>>
>> So, todo here is:
>> a.) anything 'essential' does not have to be listed as a depends. Its
>> assumed part of a Ubuntu system.
>>
>
> ACK
>
>
>> b.) That leaves the others...
>> Some that I think we should add for sure:
>>  - net-tools (and then subsequently remove it by dropping use of ifconfig
>> in favor of ip)
>>
>
> Do we want to require the dropping of ifconfig to this PR?  If so, that's
> fine but I want to be clear on the TODO.
>

Note that net-tools is compatible with freebsd ... so, dropping net-tools
requires replacement of uses of route, ifconfig and others
and adding runtime checks to pick the right tool and additional parsers;
the old parser for ifconfig/route/netstat output and a
new one for ip.

I'd like to defer dropping this I think; rather we need to have some sort
of distro object lookup for the right "tools"

Ryan

-- 
https://code.launchpad.net/~raharper/cloud-init/+git/cloud-init/+merge/308555
Your team cloud init development team is requested to review the proposed merge of ~raharper/cloud-init:fix-lp1619423 into cloud-init:master.