-
Notifications
You must be signed in to change notification settings - Fork 666
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
[Resource] Add Volumes Bulk Attach and Detach #5669
base: master
Are you sure you want to change the base?
Conversation
Reran acceptance tests |
6cb03cc
to
32ed835
Compare
8e63b84
32ed835
to
8e63b84
Compare
Please look at my comment on the similar feature of bulk volume delete. #5672 (comment) IMO attach and detach should be part of the same resource. The volumes attached should be maintained in the state when the resource is created and removed when the resource is deleted. Having two separate resources would make things worse with errors on the next runs. Imagine you would not want to write attach and detach code in the same flow, that does not make sense OR you would not want to keep attaching or detaching only. |
Output from acceptance testing: