Fix the issue delete volume based VM twice #127
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There is a issue that while build image on volume based VM, the VM will be delete twice
![图片](https://private-user-images.githubusercontent.com/14901481/335019229-18d64f46-4767-4a53-a69f-a775c4947a72.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk2Mjc1NjYsIm5iZiI6MTcxOTYyNzI2NiwicGF0aCI6Ii8xNDkwMTQ4MS8zMzUwMTkyMjktMThkNjRmNDYtNDc2Ny00YTUzLWE2OWYtYTc3NWM0OTQ3YTcyLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA2MjklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNjI5VDAyMTQyNlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTkyMTUzMDdiNTI1Y2NhNThhNjE4YzIwMmNiZjI4Mzk1ODQ2OWM2NDkzNjQyZjFiMmFhY2FjYmY5NWU1OGQ0MWMmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.fs_R9wmua77q-K7k7B1lHJh6IMwISR9BSdKq5Y4yEpA)
The first "Terminating the source server" comes from
packer-plugin-openstack/builder/openstack/step_delete_server.go
Line 24 in 2e7e219
packer-plugin-openstack/builder/openstack/step_run_source_server.go
Line 148 in 2e7e219
Before volume based VM, it is good to store the instance information into StepRunSourceServer, but for now, StepDeleteServer also does something on it, so I think it is not good to store it as a parameter of StepRunSourceServer right now.
I know my solution is not good, but I want good suggestion on it.