Virtual Machines tab on NAD/UDN/CUDN detail pages#442
Conversation
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Parthivk100 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Caution Review failedAn error occurred during the review process. Please try again later. WalkthroughAdds NAD and UDN resource detail pages with metadata, YAML, and associated VirtualMachine tabs. It also introduces NAD-aware Multus patch helpers for attaching, reconnecting, hot-unplugging, and removing VMs, plus searchable DataView management and localized UI strings. ChangesNetwork resource detail pages
VM association management
NAD-aware VM network operations
Shared UI and localization
Estimated code review effort: 4 (Complex) | ~60 minutes Sequence Diagram(s)sequenceDiagram
participant NADDetailsPage
participant useK8sWatchResource
participant useNADVirtualMachines
participant HorizontalNav
NADDetailsPage->>useK8sWatchResource: watch the NAD resource
NADDetailsPage->>useNADVirtualMachines: derive related VMs and count
NADDetailsPage->>HorizontalNav: render details, YAML, and VM tabs
sequenceDiagram
participant AddVirtualMachinesModal
participant addVMToNAD
participant VirtualMachineAPI
participant removeVMFromNAD
AddVirtualMachinesModal->>addVMToNAD: submit selected VMs
addVMToNAD->>VirtualMachineAPI: apply NAD-aware network patches
removeVMFromNAD->>VirtualMachineAPI: remove or mark the VM connection absent
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning Tools execution failed with the following error: Failed to run tools: 13 INTERNAL: Received RST_STREAM with code 2 (Internal server error) Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@Parthivk100: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Virtual Machines tab on NAD/UDN/CUDN detail pages
Summary by CodeRabbit