Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[client] Enable userspace forwarder conditionally #3309

Merged
merged 5 commits into from
Feb 12, 2025

Conversation

lixmal
Copy link
Contributor

@lixmal lixmal commented Feb 11, 2025

Describe your changes

Issue ticket number and link

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • Extended the README / documentation, if necessary

@lixmal lixmal force-pushed the enable-forwarder-conditionally branch from e0a910c to cef794a Compare February 11, 2025 18:34
@lixmal lixmal force-pushed the enable-forwarder-conditionally branch from cef794a to 641dd51 Compare February 11, 2025 18:37
@lixmal lixmal force-pushed the enable-forwarder-conditionally branch from f4c1740 to 7a7e456 Compare February 11, 2025 19:16
@lixmal lixmal marked this pull request as ready for review February 11, 2025 19:26
Copy link

sonarqubecloud bot commented Feb 11, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
2 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@mlsmaycon mlsmaycon requested a review from Copilot February 11, 2025 19:46

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 7 changed files in this pull request and generated no comments.

Files not reviewed (2)
  • client/firewall/iptables/manager_linux.go: Evaluated as low risk
  • client/firewall/manager/firewall.go: Evaluated as low risk
@mlsmaycon mlsmaycon merged commit b41de7f into main Feb 12, 2025
41 checks passed
@mlsmaycon mlsmaycon deleted the enable-forwarder-conditionally branch February 12, 2025 10:10
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants