Skip to content
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

[VPC]: Fix security group assignment for opentelekomcloud_networking_port_v2 #2310

Merged
merged 3 commits into from
Sep 14, 2023

Conversation

artem-lifshits
Copy link
Member

Summary of the Pull Request

Security group assignment fixed in cases when it's provided

PR Checklist

  • Refers to: #OTC Community
  • Tests added/passed.
  • Release notes added.

Acceptance Steps Performed

=== RUN   TestAccNetworkingV2Port_basic
=== PAUSE TestAccNetworkingV2Port_basic
=== CONT  TestAccNetworkingV2Port_basic
--- PASS: TestAccNetworkingV2Port_basic (70.19s)
=== RUN   TestAccNetworkingV2Port_importBasic
=== PAUSE TestAccNetworkingV2Port_importBasic
=== CONT  TestAccNetworkingV2Port_importBasic
--- PASS: TestAccNetworkingV2Port_importBasic (76.19s)
=== RUN   TestAccNetworkingV2Port_noip
=== PAUSE TestAccNetworkingV2Port_noip
=== CONT  TestAccNetworkingV2Port_noip
--- PASS: TestAccNetworkingV2Port_noip (69.66s)
=== RUN   TestAccNetworkingV2Port_allowedAddressPairs
=== PAUSE TestAccNetworkingV2Port_allowedAddressPairs
=== CONT  TestAccNetworkingV2Port_allowedAddressPairs
--- PASS: TestAccNetworkingV2Port_allowedAddressPairs (87.75s)
=== RUN   TestAccNetworkingV2Port_portSecurity_enabled
=== PAUSE TestAccNetworkingV2Port_portSecurity_enabled
=== CONT  TestAccNetworkingV2Port_portSecurity_enabled
--- PASS: TestAccNetworkingV2Port_portSecurity_enabled (88.15s)
=== RUN   TestAccNetworkingV2Port_noPortSecurityNoSecurityGroups
=== PAUSE TestAccNetworkingV2Port_noPortSecurityNoSecurityGroups
=== CONT  TestAccNetworkingV2Port_noPortSecurityNoSecurityGroups
--- PASS: TestAccNetworkingV2Port_noPortSecurityNoSecurityGroups (68.98s)
=== RUN   TestAccNetworkingV2Port_timeout
=== PAUSE TestAccNetworkingV2Port_timeout
=== CONT  TestAccNetworkingV2Port_timeout
--- PASS: TestAccNetworkingV2Port_timeout (70.31s)
PASS

Process finished with the exit code 0

@artem-lifshits artem-lifshits self-assigned this Sep 14, 2023
@artem-lifshits artem-lifshits added the gate Merge PR label Sep 14, 2023
@otc-zuul
Copy link

otc-zuul bot commented Sep 14, 2023

Build succeeded.
https://zuul.otc-service.com/t/eco/buildset/42827b261b8c469698de67d98ff31cef

✔️ build-otc-releasenotes SUCCESS in 4m 59s
✔️ otc-golangci-lint SUCCESS in 3m 19s
✔️ golang-make-test SUCCESS in 4m 39s
✔️ golang-make-vet SUCCESS in 3m 36s
✔️ tflint SUCCESS in 3m 56s
✔️ goreleaser-build SUCCESS in 7m 01s

@otc-zuul otc-zuul bot merged commit d2c738f into devel Sep 14, 2023
@otc-zuul otc-zuul bot deleted the compute_instance_v2_fix branch September 14, 2023 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gate Merge PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants