Skip to content

CLOUDSTACK-9005: Modifying tearDown function#1000

Merged
asfgit merged 1 commit into
apache:masterfrom
pritisarap12:CLOUDSTACK-9005-Modifying-tearDown-function
Jan 17, 2016
Merged

CLOUDSTACK-9005: Modifying tearDown function#1000
asfgit merged 1 commit into
apache:masterfrom
pritisarap12:CLOUDSTACK-9005-Modifying-tearDown-function

Conversation

@pritisarap12
Copy link
Copy Markdown

Modifying tearDown function to check if data volume is in detached state before deleting the volume

@asfbot
Copy link
Copy Markdown

asfbot commented Nov 3, 2015

Sanjeev N on dev@cloudstack.apache.org replies:
I looked at the changes Priti has done in this PR. It looks good to me.
LGTM!!

@nitt10prashant
Copy link
Copy Markdown
Contributor

LGTM Based on code understanding

One line change if attribute is there then proceed to cleanup if hasattr(self, "data_volume_created"):

@DaanHoogland
Copy link
Copy Markdown
Contributor

2 lgtm but no test report, will execute

@pritisarap12
Copy link
Copy Markdown
Author

Test Result:

Storage and Snapshot Limit ...

Ran 1 test in 970.317s

OK

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Nov 4, 2015

LGTM

@rajap9711
Copy link
Copy Markdown

@remibergsma can you please merge this in? thanks, Raja

@asfgit asfgit merged commit d793c7f into apache:master Jan 17, 2016
asfgit pushed a commit that referenced this pull request Jan 17, 2016
…tearDown-function

CLOUDSTACK-9005: Modifying tearDown functionModifying tearDown function to check if data volume is in detached state before deleting the volume

* pr/1000:
  CLOUDSTACK-9005: Modifying tearDown function

Signed-off-by: Remi Bergsma <github@remi.nl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants