Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

Validate removal of cartridges cartridgegroups#187

Merged
asfgit merged 5 commits intoapache:masterfrom
shirolk:validate-removal-cartridges-cartridgegroups
Jan 26, 2015
Merged

Validate removal of cartridges cartridgegroups#187
asfgit merged 5 commits intoapache:masterfrom
shirolk:validate-removal-cartridges-cartridgegroups

Conversation

@shirolk
Copy link
Copy Markdown

@shirolk shirolk commented Jan 26, 2015

The purpose of the pull request is to incorporate validation when either a cartridge or cartridge group is removed - essentially these can only be remove if they are not included in either cartridge groups or applications.
This fixes STRATOS-1092

Performed the following testing on this PR.

  1. add cartridge - remove cartridge = OK
  2. add cartridge - add application with cartridge - remove cartridge = OK
  3. add cartridge - add service group - remove cartridge = OK
  4. add cartridge - add service group - add application with service group and cartridge - remove service group - remove cartirdge = OK

@asfgit asfgit merged commit 46ac684 into apache:master Jan 26, 2015
asfgit pushed a commit that referenced this pull request Jan 26, 2015
@imesh
Copy link
Copy Markdown
Contributor

imesh commented Jan 26, 2015

Thanks Shiro for your contribution! Appreciate it!

@shirolk
Copy link
Copy Markdown
Author

shirolk commented Jan 26, 2015

Thank you for merging !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants