Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix refresh for resource openstack_dns_recordset_v2 #1581

Merged
merged 2 commits into from
Jun 19, 2023

Commits on Jun 12, 2023

  1. fix refresh for resource openstack_dns_recordset_v2

    The resource was incorrectly failing to read in the actual record
    values of the recordset resource when Terraform runs a refresh.  I
    believe this change was made due to the designate system imposing its
    own internal sorting on the record values instead of using the order
    provided by the user when calling the API.  Due to this ordering,
    and the "records" attribute being a list instead of a set, this would
    cause the Terraform plan to be unnecessarily unstable until the user
    reordered their record list to match whatever ordering that designate
    wants to impose even though the order of the records is actually
    unimportant.
    
    This change converts the "records" attribute to a set since the
    order of the records doesn't matter and record values should be distinct
    within a given resource definition anyways. This attribute definition
    change does not require a new schema version since a list of strings
    and a set of strings are represented identically in the underlying
    .tfstate file.
    
    Also removed the support for stripping surrounding square brackets off
    of IPv6 addresses as the functionality could actually prevent other
    types of record values, such as TXT, being set correctly.
    jcarrothers-sap committed Jun 12, 2023
    Configuration menu
    Copy the full SHA
    adb866d View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    a08b3c4 View commit details
    Browse the repository at this point in the history