Allow port update() to modify security groups. #602

kaecyra · 2015-05-19T23:33:51Z

Allows the \OpenCloud\Networking\Resource\Port::update() method to affect security groups, as per API and library documentation. Currently the 'securityGroups' option is silently ignored.

Additional keys are also omitted in the $updateKeys property, but they are beyond the scope of this pull-request:

adminStateUp
fixedIps
deviceOwner

jamiehannaford · 2015-05-20T08:47:42Z

👍 LGTM

Allow port update() to modify security groups.

Allow port update() to modify security groups.

6ee1f09

jamiehannaford pushed a commit that referenced this pull request May 20, 2015

Merge pull request #602 from kaecyra/fix-update-port-securitygroups

abe8163

Allow port update() to modify security groups.

jamiehannaford merged commit abe8163 into rackspace:working May 20, 2015

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Allow port update() to modify security groups. #602

Allow port update() to modify security groups. #602

kaecyra commented May 19, 2015

jamiehannaford commented May 20, 2015

Allow port update() to modify security groups. #602

Allow port update() to modify security groups. #602

Conversation

kaecyra commented May 19, 2015

jamiehannaford commented May 20, 2015