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

[CBR] - Possibillity to assign only a new ECS(Server), Disk (EVS), SFS (3.0 for future) to an existing vault. #2273

Open
uli-max opened this issue Aug 21, 2023 · 2 comments

Comments

@uli-max
Copy link

uli-max commented Aug 21, 2023

Description of the enhancement

In the actual implementation i can only add/remove, ECS(Server), Disks( EVS), SFS (Turbo maybe not so interesting for this, but upcoming SFS 3.0), to the main object of the vault.

https://registry.terraform.io/providers/opentelekomcloud/opentelekomcloud/latest/docs/resources/cbr_vault_v3

But it ist not possible to do just an simple, "assignment, associate", which we offer for a lot of services only to ECS(Server), Disks( EVS) (and whatever CBR offer in future).

Having therefore an simple assingment / associate - provider task would be helpful.

Examples: (from network)
https://registry.terraform.io/providers/opentelekomcloud/opentelekomcloud/latest/docs/resources/vpc_bandwidth_associate_v2

(from IAM / assigement)
https://registry.terraform.io/providers/opentelekomcloud/opentelekomcloud/latest/docs/resources/identity_role_assignment_v3

From API persipective it should be possible.

https://docs.otc.t-systems.com/cloud-backup-recovery/api-ref/cbr_apis/vaults/associating_resources.html

@artem-lifshits
Copy link
Member

Hello @uli-max could you describe a reasoning for this enhancement?
If we accept this then 2 new resources will have to be created (one for vault and another one for association) with same functionality that we already have and that wouldn't bring anything new to the table.
Moreover it might be unclear for customers which CBR vault resource should be used and why there are 2 similar resources with same API version.

Looking forward for your reply.

@uli-max
Copy link
Author

uli-max commented Aug 24, 2023

Just to make adding/remove EVS/Server to/from a vault easier, without touching the complete code/definition of vault.
For the initial definition of a vault the actual way is ok, but if you add Disks/Servers you need need to know also to change the definition of vault. Having a way todo it similar like the provided samples makes it easier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants