Skip to content

Conversation

@QxBytes
Copy link
Contributor

@QxBytes QxBytes commented Nov 12, 2024

Reason for Change:

Multitenancy swiftv1 transparent-vlan mode is only supported on linux, so the windows conflist is no longer needed and should not be included in the cni image. The azure-multitenancy-transparent-vlan.conflist in the cni image will now always be the linux azure-multitenancy-transparent-vlan conflist.

Issue Fixed:

See above

Requirements:

Notes:
Successful image build only: https://msazure.visualstudio.com/One/_build/results?buildId=107986351&view=results

@QxBytes QxBytes self-assigned this Nov 12, 2024
@QxBytes QxBytes added cni Related to CNI. swift Related to SWIFT networking. ci Infra or tooling. multitenancy labels Nov 12, 2024
@QxBytes QxBytes marked this pull request as ready for review November 12, 2024 20:12
@QxBytes QxBytes requested a review from a team as a code owner November 12, 2024 20:12
@QxBytes QxBytes requested a review from paulyufan2 November 12, 2024 20:12
@paulyufan2 paulyufan2 force-pushed the alew/remove-windows-transparent-vlan-conflist branch from 68047d7 to 50dd7f9 Compare November 12, 2024 20:14
@paulyufan2
Copy link
Contributor

/azp run Azure Container Networking PR

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@paulyufan2 paulyufan2 added this pull request to the merge queue Nov 12, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 13, 2024
@paulyufan2 paulyufan2 added this pull request to the merge queue Nov 13, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 13, 2024
@paulyufan2 paulyufan2 added this pull request to the merge queue Nov 13, 2024
Merged via the queue into master with commit 2a6e215 Nov 13, 2024
14 checks passed
@paulyufan2 paulyufan2 deleted the alew/remove-windows-transparent-vlan-conflist branch November 13, 2024 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Infra or tooling. cni Related to CNI. multitenancy swift Related to SWIFT networking.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants