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

Upgrade guide v5 -> v6 #539

Closed
ayoh1 opened this issue Feb 12, 2024 · 6 comments · Fixed by #548
Closed

Upgrade guide v5 -> v6 #539

ayoh1 opened this issue Feb 12, 2024 · 6 comments · Fixed by #548

Comments

@ayoh1
Copy link

ayoh1 commented Feb 12, 2024

Is your request related to a problem? Please describe.

We are upgrading some MSSQL instances currently in version 5. When moving to 6.0.0 we are not sure about how to handle the removed argument "create_random_password" and its impact in the instance when applying terraform.

We did understand we had to remove that parameter and replace with "manage_master_user_password", but we don't know if doing this it will modify the instance master password with a new one from the secret the module will be creating. Terraform plan is showing to delete "random_password" resource and "password" from the instance, so we thought it could be left in a state without master password or something weird.

# module.database.module.mssql-db.random_password.master_password[0] will be destroyed
    # (because random_password.master_password is not in configuration)
    - resource "random_password" "master_password" {
        - bcrypt_hash = (sensitive value) -> null
        - id          = "none" -> null
        - length      = 10 -> null
        - lower       = true -> null
        - min_lower   = 0 -> null
        - min_numeric = 0 -> null
        - min_special = 0 -> null
        - min_upper   = 0 -> null
        - number      = true -> null
        - numeric     = true -> null
        - result      = (sensitive value) -> null
        - special     = false -> null
        - upper       = true -> null
      }

    # module.database.module.mssql-db.module.db_instance.aws_db_instance.this[0] will be updated in-place
    ~ resource "aws_db_instance" "this" {
          id                                    = "db-REDACTED"
        + manage_master_user_password           = true
        - password                              = (sensitive value) -> null
          # (55 unchanged attributes hidden)
          # (1 unchanged block hidden)
      }

Since there is a upgrade guide for 4.0 we thought we could ask for guidelines on best way to do the upgrade to 6.0 and handle this breaking change without breaking master password.

Thanks!!

Describe the solution you'd like.

An UPGRADE-6.0.md file providing some guidance on how to handle this breaking change since could affect an instance master password and cause problems if terraform changes that password without the user previous knowledge.

Describe alternatives you've considered.

We have thought on doing some trial/error in nonprod to see what happens, but thought that would also be helpful for others having a quick guide for this breaking in the repo.

Additional context

Thank your for your work here!

@mukeshfulewale-h1
Copy link

Hi,
Is there any update on this? Thanks!

Copy link

This issue has been automatically marked as stale because it has been open 30 days
with no activity. Remove stale label or comment or this issue will be closed in 10 days

@github-actions github-actions bot added the stale label Mar 24, 2024
Copy link

github-actions bot commented Apr 3, 2024

This issue was automatically closed because of stale in 10 days

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Apr 3, 2024
@antonbabenko
Copy link
Member

This issue has been resolved in version 6.5.5 🎉

@ayoh1
Copy link
Author

ayoh1 commented Apr 22, 2024

thanks

Copy link

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants