Skip to content
This repository has been archived by the owner on Feb 5, 2020. It is now read-only.

Ignore AMI updates on etcd/master nodes to prevent destroying nodes accidently #367

Merged
merged 1 commit into from
Apr 26, 2017

Commits on Apr 26, 2017

  1. Ignore AMI updates on etcd/master nodes

    Currently if a new version of CoreOS comes out on a user's configured
    channel after they've already deployed, and they re-run terraform apply,
    Terraform will detect the new AMI, and when reconciling desired state,
    it will attempt to destroy existing nodes to update the AMI.
    
    In order to avoid this, I added a ignore_changes to both the etcd node
    resource, and the master launch configuration resource, to avoid
    updating masters/etcd nodes if the AMI changes.
    
    Longer term, I would expect the Container Linux Update Operator, or a
    terraform operator could resolve this, but this seems like a reasonable
    change to prevent accidental destruction of a cluster.
    
    Previously this change also set prevent_destroy, but that cannot be
    parameterized with vars, so it was removed.
    Chance Zibolski committed Apr 26, 2017
    Configuration menu
    Copy the full SHA
    abb1aef View commit details
    Browse the repository at this point in the history