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

[Fix] fix update when source_port not set #2715

Merged
merged 4 commits into from
Nov 11, 2024

Conversation

anton-sidelnikov
Copy link
Member

@anton-sidelnikov anton-sidelnikov commented Nov 11, 2024

Summary of the Pull Request

PR Checklist

Acceptance Steps Performed

=== RUN   TestAccFWRuleV2_basic
--- PASS: TestAccFWRuleV2_basic (57.94s)
=== RUN   TestAccFWRuleV2_anyProtocol
--- PASS: TestAccFWRuleV2_anyProtocol (46.55s)
=== RUN   TestAccFWRuleV2_TCPtoICMP
--- PASS: TestAccFWRuleV2_TCPtoICMP (76.22s)
=== RUN   TestAccFWRuleV2_TCPUpdate
--- PASS: TestAccFWRuleV2_TCPUpdate (40.18s)
=== RUN   TestAccFWRuleV2_emptySourcePort
--- PASS: TestAccFWRuleV2_emptySourcePort (88.37s)
PASS

Process finished with the exit code 0

artem-lifshits
artem-lifshits previously approved these changes Nov 11, 2024
@anton-sidelnikov anton-sidelnikov added the gate Merge PR label Nov 11, 2024
Copy link

otc-zuul bot commented Nov 11, 2024

Build succeeded.
https://zuul.otc-service.com/t/eco/buildset/117dffa40938472c87a212e805a8a17f

✔️ build-otc-releasenotes SUCCESS in 4m 07s
✔️ noop SUCCESS in 0s
✔️ otc-golangci-lint SUCCESS in 3m 04s
✔️ golang-make-test SUCCESS in 3m 49s
✔️ golang-make-vet SUCCESS in 2m 43s
✔️ tflint SUCCESS in 1m 50s
✔️ goreleaser-build SUCCESS in 7m 41s

@otc-zuul otc-zuul bot merged commit ffd440e into devel Nov 11, 2024
3 checks passed
@otc-zuul otc-zuul bot deleted the fw-rule-source-port-update-fix branch November 11, 2024 12:32
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