Comment 2 for bug 2016187

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to stx-puppet (master)

Reviewed: https://review.opendev.org/c/starlingx/stx-puppet/+/880392
Committed: https://opendev.org/starlingx/stx-puppet/commit/27a980f8f4399e84e36d4c7e40bf067c1350794f
Submitter: "Zuul (22348)"
Branch: master

commit 27a980f8f4399e84e36d4c7e40bf067c1350794f
Author: Jerry Sun <email address hidden>
Date: Thu Apr 13 17:44:02 2023 -0400

    Fix puppet ignoring insecure registries

    Puppet upversioning changed the behavior of our puppet code to
    configure insecure registries. The new puppet always comes up with
    an empty list. This commit fixes this issue so insecure registries
    can be configured again.

    Testing

    PASS: Bootstrap a system with insecure registries. ensure that
          /etc/docker/demon.json is updated correctly with the insecure
          registries. Ensure /etc/containerd/config.toml is also updated.

    PASS: Test functionality by manually converting registry.local:9001
          to an insecure registry. Ensure pulls fails.
          Add insecure registry entries to the containerd and docker
          config files above and restart services. Ensure pulls
          are successful through both docker and containerd.

    Change-Id: Iac2c47a75c473a85faf8f2b9e10435e6e106a407
    Closes-bug: 2016187
    Signed-off-by: Jerry Sun <email address hidden>