Aliyun ecs: Add support for deleting security group #816

Closed
wants to merge 9 commits into
from

Conversation

Projects
None yet
2 participants
@net613
Contributor

net613 commented Jun 16, 2016

Aliyun ecs: Add support for deleting security group

Description

Currently for Aliyun ECS, you cannot delete a security group through ecs deriver.

Status

Replace this: describe the PR status. Examples:

  • work in progress
  • done, ready for review

Checklist (tick everything that applies)

@tonybaloney

This comment has been minimized.

Show comment
Hide comment
@tonybaloney

tonybaloney Jun 19, 2016

Contributor

thanks for the contribution @net613, changes LGTM

Contributor

tonybaloney commented Jun 19, 2016

thanks for the contribution @net613, changes LGTM

@net613

This comment has been minimized.

Show comment
Hide comment
@net613

net613 Jun 21, 2016

Contributor

@tonybaloney , what's the means about "Add more commits by pushing to the aliyun_ecs_support_deleting_security_group branch on net613/libcloud"?

Contributor

net613 commented Jun 21, 2016

@tonybaloney , what's the means about "Add more commits by pushing to the aliyun_ecs_support_deleting_security_group branch on net613/libcloud"?

@asfgit asfgit closed this in 4005559 Jun 24, 2016

asfgit pushed a commit that referenced this pull request Jun 24, 2016

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