Form conversion: Compute/Physical infrastructure/ Providers change password

[himdel]

Convert form to use DDF/React and API - parent issue #6825

Location: Compute/Physical infrastrucute/ Providers
Form: change password

current type: Angular
API state:
Need info

PM Input: 2 - Medium

imported status: NOT ASSIGNED

[skateman]

@agrare does this form even make sense? I think it could be deleted and we can just edit the provider and change the password there.

@miq-bot add_label question

[chessbyte]

I wonder if this goes out and resets the password on the target. A standard Change Password form is probably good to have in our arsenal that we can re-use all over. /cc @Fryguy

[skateman]

@chessbyte this one expects the current password, meanwhile the edit provider doesn't. Theres also validation on the edit provider and here it just asks for the password twice, so it's a nice place to mess up the provider credentials ...

[agrare]

@skateman this is an ems_operation that changes the password of the target provider, not just the password in our database.

[Fryguy]

A standard Change Password form is probably good to have in our arsenal that we can re-use all over.

Agreed on a component that is common.

this is an ems_operation that changes the password of the target provider, not just the password in our database.

We don't do this for any other provider...should we do it here?

[agrare]

We don't do this for any other provider...should we do it here?

Not sure what you mean, should we support this operation for the Lenovo provider? They added it as a feature so I think so yes

[Fryguy]

Yeah, I'm wondering if we should remove that feature from the Lenovo plugin since they are the only group that implemented it? Or should we add that as a future-feature to all of the other plugins?

[skateman]

Any update on this?

[chessbyte]

@agrare Do you know if users have an alternative way of changing password on the target provider? If so, I would suggest we drop this feature. cc @Fryguy

[agrare]

Yes they can still change the password natively on the xclarity management UI

[skateman]

There's also a wrongly named RBAC feature (ems_infra_change_password) and an API action for this, should those go as well?