Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jul 29, 2025

This change enables the L7 proxy functionality for the Cilium nightly pipeline by updating the enable-l7-proxy configuration value from "false" to "true" in the cilium nightly config map.

What changed

  • Updated enable-l7-proxy: "false" to enable-l7-proxy: "true" in test/integration/manifests/cilium/cilium-nightly-config.yaml

Why this change is needed

The L7 proxy feature in Cilium provides application-layer (Layer 7) network policy enforcement and observability capabilities. Enabling this feature for the nightly pipeline will:

  • Allow testing of L7 network policies and FQDN-based policies
  • Enable advanced traffic management and security features
  • Ensure the nightly pipeline validates L7 proxy functionality

Testing

This change aligns with existing test expectations in test/integration/lrp/lrp_fqdn_test.go, which specifically validates that the enable-l7-proxy flag is set to "true" for FQDN-based Cilium Network Policy testing.

The YAML syntax has been validated and the change is minimal, affecting only the specific configuration value requested.

Fixes #3867.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copilot AI changed the title [WIP] ci: Enable l7 proxy for cilium nightly pipeline ci: Enable L7 proxy for cilium nightly pipeline Jul 29, 2025
Copilot AI requested a review from vipul-21 July 29, 2025 20:07
Copilot finished work on behalf of vipul-21 July 29, 2025 20:07
@vipul-21 vipul-21 marked this pull request as ready for review July 30, 2025 17:26
Copilot AI review requested due to automatic review settings July 30, 2025 17:26
@vipul-21 vipul-21 requested a review from a team as a code owner July 30, 2025 17:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enables the L7 proxy functionality for Cilium nightly pipeline testing by updating a single configuration value. This change allows the nightly pipeline to test Layer 7 network policies and FQDN-based policies, ensuring comprehensive validation of Cilium's advanced traffic management features.

Key changes:

  • Enabled L7 proxy functionality in Cilium nightly configuration

@vipul-21 vipul-21 enabled auto-merge July 30, 2025 17:29
@vipul-21 vipul-21 disabled auto-merge August 5, 2025 20:39
@vipul-21 vipul-21 enabled auto-merge August 5, 2025 20:41
@paulyufan2
Copy link
Contributor

/azp run Azure Container Networking PR

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vipul-21 vipul-21 added this pull request to the merge queue Aug 7, 2025
Merged via the queue into master with commit 8d9b4c5 Aug 7, 2025
27 of 33 checks passed
@vipul-21 vipul-21 deleted the copilot/fix-3867 branch August 7, 2025 17:17
NihaNallappagari pushed a commit to NihaNallappagari/azure-container-networking that referenced this pull request Sep 4, 2025
* Initial plan

* ci: Enable l7 proxy for cilium nightly pipeline

Co-authored-by: vipul-21 <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: vipul-21 <[email protected]>
sivakami-projects pushed a commit that referenced this pull request Oct 23, 2025
* Initial plan

* ci: Enable l7 proxy for cilium nightly pipeline

Co-authored-by: vipul-21 <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: vipul-21 <[email protected]>
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.

ci: Enable l7 proxy for cilium nightly pipeline

5 participants