Skip to content

feat: import user with new password - #1761

Open
mhenselin wants to merge 4 commits into
stackitcloud:mainfrom
mhenselin:fix/import_user_with_new_password
Open

mhenselin wants to merge 4 commits into
stackitcloud:mainfrom
mhenselin:fix/import_user_with_new_password

Conversation

@mhenselin

Copy link
Copy Markdown

Description

Currently there is no possibility to import an existing PostgresFlex User with a password. This change gives the option to have the user imported with a new password. This enables terraform to reference the password in further steps.

Checklist

  • Issue was linked above
  • Code format was applied: make fmt
  • Examples were added / adjusted (see examples/ directory)
  • Docs are up-to-date: make generate-docs (will be checked by CI)
  • Unit tests got implemented or updated
  • Acceptance tests got implemented or updated (see e.g. here)
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

@mhenselin
mhenselin requested a review from a team as a code owner September 10, 2026 14:24
@rubenhoenle

Copy link
Copy Markdown
Member

Sorry, but I don't think this is the solution we aim for.

The actual root cause here is that you currently can't rotate the user password via Terraform. We should allow this somehow.

For the import you would import the user first and then rigger a rotation of the password via Terraform.

Thanks for the PR, but I consider this technical debt and nothing we should even consider 😅

@github-actions

Copy link
Copy Markdown

This PR was marked as stale after 7 days of inactivity and will be closed after another 7 days of further inactivity. If this PR should be kept open, just add a comment, remove the stale label or push new commits to it.

@github-actions github-actions Bot added the Stale PR is marked as stale due to inactivity. label Sep 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Stale PR is marked as stale due to inactivity.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants