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

test: pipeline changes for cilium nodesubnet #3083

Merged
merged 8 commits into from
Nov 5, 2024
Merged

test: pipeline changes for cilium nodesubnet #3083

merged 8 commits into from
Nov 5, 2024

Conversation

santhoshmprabhu
Copy link
Contributor

Reason for Change:
This PR introduces the pipeline changes to test Cilium+Nodesubnet E2E. The code changes are in #3073. This PR includes:

  1. Inclusion of Cilium+Nodesubnet E2E scenario in the PR pipeline
  2. Updates to IP masq agent config to skip SNAT for cluster IPs
  3. Better Go script to update IP configs in pipeline-createdCilium nodesubnet clusters
  4. Minor changes to hack/Makefile to allow use of standard LB

Issue Fixed:
Pipeline changes for Cilium nodesubnet

Requirements:

Notes:
This PR should not be mreged before #3073

@santhoshmprabhu santhoshmprabhu added do-not-merge cns Related to CNS. ci Infra or tooling. labels Oct 23, 2024
@santhoshmprabhu santhoshmprabhu self-assigned this Oct 23, 2024
@santhoshmprabhu santhoshmprabhu requested a review from a team as a code owner October 23, 2024 19:09
@santhoshmprabhu santhoshmprabhu mentioned this pull request Oct 23, 2024
Merged
4 tasks
@santhoshmprabhu
fix:make linter happy
95d47dc
@santhoshmprabhu
Merge branch 'sanprabhu/cilium-node-subnet-pipeline-changes' of githu…
901441f 
…b.com:Azure/azure-container-networking into sanprabhu/cilium-node-subnet-pipeline-changes
@santhoshmprabhu
Copy link
Contributor Author

/azp run Azure Container Networking PR

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jpayne3506 jpayne3506 mentioned this pull request Oct 28, 2024
Merged
4 tasks
@jpayne3506
Copy link
Contributor

/azp list

@azure-pipelines Azure Pipelines
Copy link

CI/CD Pipelines for this repository:

@santhoshmprabhu
Copy link
Contributor Author

/azp run Azure Container Networking PR

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@santhoshmprabhu santhoshmprabhu added this pull request to the merge queue Nov 5, 2024
Merged via the queue into master with commit b5a7f14 Nov 5, 2024
14 checks passed
@santhoshmprabhu santhoshmprabhu deleted the sanprabhu/cilium-node-subnet-pipeline-changes branch November 5, 2024 03:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jpayne3506 jpayne3506 jpayne3506 approved these changes

@vipul-21 vipul-21 vipul-21 approved these changes

@thomasricci thomasricci Awaiting requested review from thomasricci thomasricci is a code owner automatically assigned from Azure/azure-sdn-members

@behzad-mir behzad-mir Awaiting requested review from behzad-mir

@QxBytes QxBytes Awaiting requested review from QxBytes

@rbtr rbtr Awaiting requested review from rbtr

@tamilmani1989 tamilmani1989 Awaiting requested review from tamilmani1989

Assignees

@santhoshmprabhu santhoshmprabhu

Labels
ci Infra or tooling. cns Related to CNS.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@santhoshmprabhu @jpayne3506 @vipul-21