search

cloudfoundry-attic / warden

Cloud Foundry - the open platform as a service project
Apache License 2.0
283 stars 108 forks source link

Allow destroy to work regardless of links #106

Closed fraenkel closed 9 years ago

fraenkel commented 9 years ago

Signed-off-by: Matthew Sykes matthew.sykes@gmail.com

cfdreddbot commented 9 years ago

Hey fraenkel!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you've already signed the CLA.

cf-gitbot commented 9 years ago

We have created an issue in Pivotal Tracker to manage this. You can view the current status of your issue at: https://www.pivotaltracker.com/story/show/108082340.

Amit-PivotalLabs commented 9 years ago

Thanks @fraenkel

I'm not very familiar with the warden test coverage, would it be reasonable to add a test for this case?

fraenkel commented 9 years ago

Nope... the problem occurs in the warden_ctl script which drives destroy prior to actually starting warden.

fraenkel commented 9 years ago

I guess you could argue that all destroys should work regardless and so we could run all the destroys when containers do not exist.