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

[ Feature ] - Propagate credential change #18

Open
FranckSallet opened this issue Mar 29, 2024 · 17 comments
Open

[ Feature ] - Propagate credential change #18

FranckSallet opened this issue Mar 29, 2024 · 17 comments
Assignees
Labels
enhancement New feature or request

Comments

@FranckSallet
Copy link

Is your feature request related to a problem? Please describe.
I use a global account to manage password of active directory accounts.
The problem appears when I create a "wallix-bastion_domain_account_credential resource", I cannot propagate the password to the Active Directory account like I could do on the GUI. We therefore need manual action on the bastion for the account to be fully functional.

Describe the solution you'd like
add a boolean parameter like "propagate_credential_change" in the "wallix-bastion_domain_account_credential" resource

Regards

@FranckSallet FranckSallet added the enhancement New feature or request label Mar 29, 2024
@bsimonWallix
Copy link
Collaborator

After study, the endpoint exist on the API but not yet implemented on the Terraform provider.

image

I am going to see with the team how we can add it for the next release.

@FranckSallet
Copy link
Author

thank you for the reply
in this case, you must repair the destruction of the resource when changing the password. cf issue 17

@bsimonWallix
Copy link
Collaborator

It is tracked for fix and improvment for next release.

@moulip
Copy link
Collaborator

moulip commented May 17, 2024

Hi @FranckSallet,

I'm currently looking into your enhancement request. I need some details about your usecase.
Let me expand on what I have understood and correct me if I'm mistaken.

You have an already existing account in your AD.
You want to create it in your Bastion with TF, create a new password for that account, and propagate it to the AD.
Is that right ?

@FranckSallet
Copy link
Author

Hi @moulip

yes, that's exactly it.
We need to manage AD account passwords through the password manager with TF.
The password must also be able to be changed through the interface or via the password policy.

regards

@moulip
Copy link
Collaborator

moulip commented May 21, 2024

Hi @moulip

yes, that's exactly it. We need to manage AD account passwords through the password manager with TF. The password must also be able to be changed through the interface or via the password policy.

regards

I completely get your use-case. I just want to make sure that we agree on the fact that the account already exists in the AD with a password already set and as soon as you create it for the first time in the Bastion, you will create it with another password which will replace the previously set password in AD upon creation.

@FranckSallet
Copy link
Author

Hi @moulip

We are completely agree about my use case.
We have a already Active Directory account with a password.
After that we want to manage the password with the "Password Manager" and the "Password Policy".

Regards

@moulip
Copy link
Collaborator

moulip commented May 21, 2024

All right will dig into it now ;-)

@moulip
Copy link
Collaborator

moulip commented Aug 13, 2024

Hi @FranckSallet !

Long time no talk sorry for that. I have done the task and the provider is now able to propagate the credential change BUT, this has led to find a bug in the Bastion which prevents me from releasing a new version.
We are on the path of fixing this bug and as from there I'll be able to commit.
Just to keep you updated.

@FranckSallet
Copy link
Author

Hi @moulip

great to hear that
This next release of the provider will work with the bastion version 12 ?

@moulip
Copy link
Collaborator

moulip commented Aug 14, 2024

Will try to do so !

@FranckSallet
Copy link
Author

Hi @moulip

Any news or release date ?

Regards

@bsimonWallix
Copy link
Collaborator

We are just pushing the v0.14.0 - The fix of this is issue wasn't merged as it does need more testing.

We target a v0.14.1 next week for it.

@bsimonWallix
Copy link
Collaborator

Hello we just pushed the change from @moulip for this request.
We are still waiting on the fix on the bastion API. As for now the uuid of the password is lost on change and terraform have issue to keep the state.

@FranckSallet
Copy link
Author

Bonjour Simon
Merci du retour
ce n'est toujours pas fixé dans la version 12.0.5 du bastion ?
Merci, Franck

@FranckSallet
Copy link
Author

je ne vois pas l'option "Propagate credential change" dans la documentation de la version 14, c'est normal ?
Merci, Franck

@bsimonWallix
Copy link
Collaborator

C'est sur la branche develop pour le moment.
On devrait sortir une nouvelle release sous peu

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants