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

libcni, skel: implement STATUS #1030

Merged
merged 1 commit into from
Dec 4, 2023
Merged

Conversation

squeed
Copy link
Member

@squeed squeed commented Oct 23, 2023

This adds an implementation of STATUS to libcni and skel, along with tests.

@squeed squeed requested a review from dcbw October 23, 2023 11:24
@squeed squeed force-pushed the status-impl branch 2 times, most recently from f2fd2a5 to ea6bad2 Compare November 6, 2023 16:25
@coveralls
Copy link

coveralls commented Nov 6, 2023

Coverage Status

coverage: 70.742% (-0.8%) from 71.573%
when pulling 4caff13 on squeed:status-impl
into 9c19f13 on containernetworking:main.

@squeed squeed added this to the CNI v1.1 milestone Nov 13, 2023
@dcbw
Copy link
Member

dcbw commented Nov 13, 2023

LGTM

This adds an implementation of STATUS to libcni and skel, along with
tests.

Signed-off-by: Casey Callendrello <c1@caseyc.net>
@squeed squeed merged commit 2cec989 into containernetworking:main Dec 4, 2023
5 checks passed
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.

3 participants