Support Security Group Name

[GnatorX]

• sg name field in crd

• add implementation

• Move to getsgforVPC, move SG names into SG, unique items validation

---

Issue #, if available: #20

Description of changes:
Add support for Security group names
Continuation of #408

Credit: Most of the code was implemented by @abeer-stripe

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[sushrk]

Thanks @GnatorX, I'm taking a look at the PR.

Few initial thoughts:
Can we merge this into a feature branch so I can work on testing this internally before release? I created a new branch sg-name.

expiration cache you linked earlier looks like a good start. We can check if CreateNetworkInterface fails due to missing security group and update the cache, and retry the branch ENI creation.

Can you also add integration tests? This can be in a follow up PR.

[GnatorX]

PR to merge into feature branch #410

[sushrk]

Oh, it should be possible to just update the base branch in same PR, let me do that.

Edit: Cannot update it as there is already a new PR A pull request already exists for base branch 'sg-name' and head branch 'master-up'. We'll continue discussion on #410 , thanks.

[GnatorX]

Oh sorry. Didn't realize i could update it.

[sushrk]

Closing in favor of #410