yahoo-eng-team team mailing list archive

[Launchpad Bug Tracker <1724951@xxxxxxxxxxxxxxxxxx>]

This bug was fixed in the package cloud-init - 17.1-25-g17a15f9e-
0ubuntu1~17.10.1

---------------
cloud-init (17.1-25-g17a15f9e-0ubuntu1~17.10.1) artful-proposed; urgency=medium

  * New upstream snapshot.
    - resizefs: Fix regression when system booted with root=PARTUUID=
      (LP: #1725067)
    - tools: make yum package installation more reliable
    - citest: fix remaining warnings raised by integration tests.
    - citest: show the class actual class name in results.
    - ntp: fix config module schema to allow empty ntp config
      (LP: #1724951)
    - tools: disable fastestmirror if using proxy [Joshua Powers]
    - schema: Log debug instead of warning when jsonschema is not available.
      (LP: #1724354)

 -- Chad Smith <chad.smith@xxxxxxxxxxxxx>  Mon, 23 Oct 2017 15:07:35
-0600

** Changed in: cloud-init (Ubuntu)
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to cloud-init.
https://bugs.launchpad.net/bugs/1724951

Title:
  Ntp schema definition permits empty ntp cloud-config, but code
  disallows

Status in cloud-init:
  In Progress
Status in cloud-init package in Ubuntu:
  Fix Released
Status in cloud-init source package in Xenial:
  Fix Committed
Status in cloud-init source package in Zesty:
  Fix Committed
Status in cloud-init source package in Artful:
  Fix Committed

Bug description:
  http://pad.lv/1724951
  https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1724951

  === Begin SRU Template ===
  [Impact]
  Customers who provide the following cloud-config will get a Runtime error
  #cloud-config
  ntp:

  The expected behavior, per docs, is that an empty ntp configuration will result in
  "4 pools will be used in the format {0-3}.{distro}.pool.ntp.org.".

  [Test Case]

  if [ ! -f './lxc-proposed-snapshot' ]; then
    wget https://raw.githubusercontent.com/cloud-init/ubuntu-sru/master/bin/lxc-proposed-snapshot;
    chmod 755 lxc-proposed-snapshot;
  fi

  # 1. Provide a empty ntp configuration to cloud-init
  cat > install-ntp.conf <<EOF
  #cloud-config
  ntp:
  EOF

  # 2.
  for release in xenial zesty artful; do
          ref=$release-proposed;
          echo "$release START --------------";
          lxc-proposed-snapshot --proposed --publish $release $ref;
          lxc init $ref test-$release;
          lxc config set test-$release user.user-data - < install-ntp.conf;
          lxc start test-$release;
          sleep 10;
          # no runtime error
          lxc exec test-$release -- cat /run/cloud-init/result.json
          # ntp installed
          lxc exec test-$release -- dpkg-query --show ntp;
          # default pools
          lxc exec test-$release -- grep pool /etc/ntp.conf
  done

  [Regression Potential]
  Minimal, this fixes only a corner case where cloud-config ntp specified is completely empty.
  Any more complex ntp configuration are unaffected by code changes.

  [Other Info]
  Upstream commit at
    https://git.launchpad.net/cloud-init/commit/?id=6bc504e41666

  === End SRU Template ===

To manage notifications about this bug go to:
https://bugs.launchpad.net/cloud-init/+bug/1724951/+subscriptions