Skip to content

Start stop nodes#3055

Open
FloSch62 wants to merge 32 commits intomainfrom
start_stop_nodes
Open

Start stop nodes#3055
FloSch62 wants to merge 32 commits intomainfrom
start_stop_nodes

Conversation

@FloSch62
Copy link
Copy Markdown
Member

@FloSch62 FloSch62 commented Feb 11, 2026

The start/stop/restart command starts/stops/restarts one or more nodes in a deployed lab and restores parked dataplane
interfaces

Tested:

  • nokia_srl
  • nokia_srsim (standalone and distributed)
  • nokia_sros
  • juniper_vmx

Copy link
Copy Markdown
Collaborator

@steiler steiler left a comment

Choose a reason for hiding this comment

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

just a couple of comments

Comment thread core/stop.go Outdated
Comment thread links/endpoint.go Outdated
Comment thread links/endpoint.go Outdated
Comment thread links/endpoint.go Outdated
@FloSch62
Copy link
Copy Markdown
Member Author

vxlan tests fail, because of #3056

@FloSch62
Copy link
Copy Markdown
Member Author

FloSch62 commented Feb 13, 2026

For srsim distributed, I may would need some help from you @kaelemc and @sacckth , but imho that can also be added later.

Comment thread nodes/default_node.go Outdated
Comment thread nodes/default_node.go Outdated
Comment thread nodes/default_node.go Outdated
Comment thread nodes/default_node.go Outdated
Comment thread nodes/default_node.go Outdated
Comment thread nodes/default_node.go Outdated
@kaelemc kaelemc self-assigned this Feb 18, 2026
@kaelemc kaelemc marked this pull request as draft February 18, 2026 15:03
Comment thread links/generic_link_node.go Outdated
Comment thread links/parking_node.go Outdated
Comment thread links/parking_node.go
@FloSch62
Copy link
Copy Markdown
Member Author

FloSch62 commented Mar 3, 2026

@kaelemc and @steiler I tried to drive this further. Feel free to revert the last commit if you had other plans or if its not satisfying to you

@kaelemc kaelemc marked this pull request as ready for review March 5, 2026 00:57
@kaelemc kaelemc requested a review from steiler March 5, 2026 00:57
Comment thread nodes/vr_node.go
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 18, 2026

Codecov Report

❌ Patch coverage is 48.33703% with 233 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.42%. Comparing base (fb0bcf1) to head (f1be230).
⚠️ Report is 33 commits behind head on main.

Files with missing lines Patch % Lines
nodes/vr_node.go 0.00% 70 Missing ⚠️
nodes/default_node.go 40.62% 28 Missing and 10 partials ⚠️
nodes/sros/sros.go 26.92% 19 Missing ⚠️
mocks/mocknodes/node.go 0.00% 16 Missing ⚠️
links/parking_node.go 70.83% 8 Missing and 6 partials ⚠️
runtime/podman/podman.go 0.00% 12 Missing ⚠️
utils/netlink.go 67.56% 10 Missing and 2 partials ⚠️
core/restart.go 33.33% 4 Missing and 4 partials ⚠️
mocks/mocknodes/default_node.go 0.00% 8 Missing ⚠️
core/lifecycle.go 68.42% 3 Missing and 3 partials ⚠️
... and 7 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3055      +/-   ##
==========================================
- Coverage   56.92%   56.42%   -0.50%     
==========================================
  Files         216      224       +8     
  Lines       21469    22019     +550     
==========================================
+ Hits        12221    12425     +204     
- Misses       8003     8304     +301     
- Partials     1245     1290      +45     
Files with missing lines Coverage Δ
cmd/options.go 91.23% <100.00%> (+0.04%) ⬆️
cmd/root.go 81.06% <100.00%> (+2.44%) ⬆️
core/destroy.go 59.21% <100.00%> (+0.72%) ⬆️
nodes/ceos/ceos.go 64.10% <100.00%> (+0.18%) ⬆️
nodes/linux/linux.go 79.54% <100.00%> (+0.47%) ⬆️
nodes/node.go 21.42% <ø> (ø)
nodes/srl/srl.go 62.17% <100.00%> (+0.22%) ⬆️
runtime/runtime.go 63.82% <ø> (ø)
types/types.go 50.00% <ø> (ø)
runtime/docker/docker.go 66.94% <89.47%> (+0.93%) ⬆️
... and 16 more

... and 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sacckth
Copy link
Copy Markdown
Contributor

sacckth commented Mar 25, 2026

stalled or what? @steiler @kaelemc @FloSch62

@kaelemc
Copy link
Copy Markdown
Member

kaelemc commented Mar 25, 2026 via email

@FloSch62
Copy link
Copy Markdown
Member Author

I tried to rework this so parking and endpoint ownership are handled in the node lifecycle, with the parking netns treated like a regular node and the same MoveTo flow used in both directions. So far I only tested this with srl; before I spend more time, are there any concerns with this direction @steiler @kaelemc ?

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.

4 participants