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

Add swiftv2 middleware support for standalone service fabric scenario (windows) #2634

Merged
merged 34 commits into from
Jul 10, 2024

Conversation

kmurudi
Copy link
Contributor

@kmurudi kmurudi commented Mar 6, 2024

Reason for Change:
L1VH/Singularity & Swiftv2 need to program secondary nics for creating pod via containerd. Changes have been supported for AKS pod scenario, but this PR adds support for windows Service fabric/standalone scenario as it gets the data from DNC goal state.

Requirements:

Notes:

@kmurudi kmurudi requested a review from a team as a code owner March 6, 2024 10:13
@kmurudi kmurudi requested a review from jaer-tsun March 6, 2024 10:13
cns/middlewares/k8sSwiftV2.go Outdated Show resolved Hide resolved
cns/restserver/ipam.go Outdated Show resolved Hide resolved
cns/restserver/ipam.go Outdated Show resolved Hide resolved
cns/api.go Outdated Show resolved Hide resolved
cns/wireserver/net.go Outdated Show resolved Hide resolved
cns/restserver/ipam.go Outdated Show resolved Hide resolved
cns/restserver/ipam.go Outdated Show resolved Hide resolved
cns/wireserver/net.go Outdated Show resolved Hide resolved
@kmurudi kmurudi force-pushed the Singularity_SwiftV2SFMiddleware branch from 18fb031 to 4b58ac6 Compare March 18, 2024 22:51
cns/wireserver/net.go Outdated Show resolved Hide resolved
cns/middlewares/SFSwiftV2.go Outdated Show resolved Hide resolved
cns/restserver/ipam.go Outdated Show resolved Hide resolved
cns/restserver/ipam.go Outdated Show resolved Hide resolved
cns/restserver/ipam.go Outdated Show resolved Hide resolved
cns/restserver/util.go Outdated Show resolved Hide resolved
cns/restserver/ipam.go Outdated Show resolved Hide resolved
cns/restserver/ipam.go Outdated Show resolved Hide resolved
cns/restserver/ipam.go Outdated Show resolved Hide resolved
cns/restserver/ipam.go Outdated Show resolved Hide resolved
Copy link

This pull request is stale because it has been open for 2 weeks with no activity. Remove stale label or comment or this will be closed in 7 days

@github-actions github-actions bot added the stale Stale due to inactivity. label Apr 10, 2024
@kmurudi kmurudi removed the stale Stale due to inactivity. label Apr 10, 2024
cns/middlewares/SFSwiftV2.go Outdated Show resolved Hide resolved
cns/wireserver/net.go Outdated Show resolved Hide resolved
@nddq nddq added the cns Related to CNS. label May 6, 2024
cns/middlewares/SFSwiftV2.go Outdated Show resolved Hide resolved
cns/wireserver/net.go Outdated Show resolved Hide resolved
cns/restserver/ipam.go Outdated Show resolved Hide resolved
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@paulyufan2
Copy link
Contributor

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@paulyufan2
Copy link
Contributor

/azp run Azure Container Networking PR

Copy link

Pipelines were unable to run due to time out waiting for the pull request to finish merging.

@nddq
Copy link
Contributor

nddq commented Jul 9, 2024

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@paulyufan2 paulyufan2 enabled auto-merge July 9, 2024 22:27
@paulyufan2
Copy link
Contributor

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@paulyufan2 paulyufan2 added this pull request to the merge queue Jul 10, 2024
Merged via the queue into master with commit b9b8627 Jul 10, 2024
11 checks passed
@paulyufan2 paulyufan2 deleted the Singularity_SwiftV2SFMiddleware branch July 10, 2024 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cns Related to CNS.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants