Skip to content

Conversation

@marcel-wollschlaeger
Copy link
Contributor

This PR changes the stackit-postgres-db module in the following ways:

  • allows the creation of multiple databases with different names than the instance
  • allows the creation of a root user with a different name than the instance
  • allows the creation of multiple additional users with Login Permission
  • creates a local_file resource for the manifest if secrets are managed
  • Test and Readme adjustments

@marcel-wollschlaeger marcel-wollschlaeger requested a review from a team as a code owner December 17, 2025 07:55
Copy link
Member

@HartmannVolker HartmannVolker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After a detailed review I noticed that we need to add some more validation. If we use the local_file provider, terraform will keep the control over the generated file. This mean any manual change to the file will result in a diff in terraform.

@marcel-wollschlaeger
Copy link
Contributor Author

@HartmannVolker I've added a config-map output and applied all the fixes we talked about.

@marcel-wollschlaeger marcel-wollschlaeger merged commit bd7416a into main Dec 17, 2025
9 checks passed
@marcel-wollschlaeger marcel-wollschlaeger deleted the create-additional-users-and-databases branch December 17, 2025 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants