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

[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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants