Skip to content

Conversation

brianmori
Copy link

@brianmori brianmori commented Aug 21, 2020

I have redeveloped the module to take advantage of the 0.12 sintax.

The major benefit is to declare multiple users in a module rather than calling the module each time a new user must be created.

It does also print the secret keys for users who use terraform with a different state than local.

@brianmori brianmori changed the title Redevelop module Redevelop module for TF 0.12 and multiple users Aug 21, 2020
@brianmori
Copy link
Author

Also let me know if the module should still support one single user creation for maximum flexibility in case the configuration with the maps is more complex

Copy link
Contributor

@xiaozhu36 xiaozhu36 left a comment

Choose a reason for hiding this comment

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

Please update the readme

################################
# RAM user
################################
variable "create" {
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the create should be kept and it can used to control whether create users.

Copy link
Author

Choose a reason for hiding this comment

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

if the user does not want to create users, commenting the variable "users" will control whether to create users

@brianmori brianmori requested a review from xiaozhu36 August 24, 2020 14:17
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.

2 participants