Skip to content

Log security groups correctly#646

Merged
mogren merged 1 commit intoaws:masterfrom
mogren:small-fix
Oct 16, 2019
Merged

Log security groups correctly#646
mogren merged 1 commit intoaws:masterfrom
mogren:small-fix

Conversation

@mogren
Copy link
Copy Markdown
Contributor

@mogren mogren commented Oct 8, 2019

Description of changes:
Before:

[INFO] createENI: use primary interface's config, [0xc00003b680 0xc00003b6b0], subnet-0391a9d4a4d2cb999

After:

[INFO] Using same config as the primary interface for the new ENI
[INFO] Creating ENI with security groups: [sg-08f1036b507b30999 sg-0aa43d9d01804c999] in subnet: subnet-02d0c4ea9bd25d999

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Copy link
Copy Markdown
Contributor

@tiffanyfay tiffanyfay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me.

@mogren mogren merged commit 3aacadc into aws:master Oct 16, 2019
@mogren mogren deleted the small-fix branch November 5, 2019 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants