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.

Sign up for GitHub

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

Jump to bottom

Fix panic in networking_secgroup_v2 data source #1176

Merged
merged 1 commit into from
Jun 29, 2021
Merged

Fix panic in networking_secgroup_v2 data source #1176

merged 1 commit into from
Jun 29, 2021

Conversation

outcatcher
Copy link
Contributor

Summary of the Pull Request

Fix #1175

PR Checklist

Acceptance Steps Performed

=== RUN   TestAccOpenTelekomCloudNetworkingSecGroupV2DataSource_basic
--- PASS: TestAccOpenTelekomCloudNetworkingSecGroupV2DataSource_basic (28.28s)
=== RUN   TestAccOpenTelekomCloudNetworkingSecGroupV2DataSource_secGroupID
--- PASS: TestAccOpenTelekomCloudNetworkingSecGroupV2DataSource_secGroupID (27.95s)
PASS

Process finished with the exit code 0

@outcatcher outcatcher requested review from a team, lego963 and anton-sidelnikov and removed request for a team June 29, 2021 09:39
@outcatcher
Copy link
Contributor Author

This blocks testing of #1173

@outcatcher outcatcher added the gate Merge PR label Jun 29, 2021
@otc-zuul
Copy link

otc-zuul bot commented Jun 29, 2021

Build succeeded.

@otc-zuul otc-zuul bot merged commit 1396cfb into devel Jun 29, 2021
@otc-zuul otc-zuul bot deleted the fix/nw-secgroup branch June 29, 2021 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lego963 lego963 lego963 approved these changes

@otc-zuul otc-zuul otc-zuul approved these changes

@anton-sidelnikov anton-sidelnikov anton-sidelnikov approved these changes

Assignees
No one assigned
Labels
gate Merge PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

dataSourceNetworkingSecGroupV2Read crashes with panic
3 participants
@outcatcher @lego963 @anton-sidelnikov