New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement refresh() method for all resources and make securitygroup.name updatable. #70

Closed
nuwang opened this Issue Sep 7, 2017 · 3 comments

Comments

Projects
None yet
3 participants
@nuwang
Contributor

nuwang commented Sep 7, 2017

It seems logical to have a refresh method for most, if not all resources. For example, securitygroups do not have a refresh() method. Also, the security group's name property is not updatable.

@nuwang nuwang added the enhancement label Sep 22, 2017

@nuwang nuwang added this to the release 1.0.0 milestone Oct 1, 2017

@nuwang nuwang added backlog and removed backlog labels Oct 4, 2017

@almahmoud almahmoud self-assigned this Jul 24, 2018

@afgane

This comment has been minimized.

Show comment
Hide comment
@afgane

afgane Aug 2, 2018

Contributor

Completed with object refresh in 98959fc

Contributor

afgane commented Aug 2, 2018

Completed with object refresh in 98959fc

@afgane afgane closed this Aug 2, 2018

@nuwang nuwang reopened this Aug 2, 2018

@nuwang

This comment has been minimized.

Show comment
Hide comment
@nuwang

nuwang Aug 2, 2018

Contributor

Name property is not updatable?

Contributor

nuwang commented Aug 2, 2018

Name property is not updatable?

@almahmoud

This comment has been minimized.

Show comment
Hide comment
@almahmoud

almahmoud Aug 2, 2018

Collaborator

OS name setter in e88276a

AWS security groups have a 'Group Name' which can't be changed. A name tag could be used if we want to make it updatable.

Collaborator

almahmoud commented Aug 2, 2018

OS name setter in e88276a

AWS security groups have a 'Group Name' which can't be changed. A name tag could be used if we want to make it updatable.

@afgane afgane closed this in f8d0af0 Aug 3, 2018

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