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

feat: add IPConfig state logs/metrics to Request/Release handlers #3050

Merged
merged 2 commits into from
Nov 6, 2024

Conversation

rbtr
Copy link
Contributor

@rbtr rbtr commented Oct 4, 2024

Reason for Change:

Logs and records IPAM metrics on Request/Release and MarkPendingRelease calls.

Issue Fixed:

Requirements:

Notes:

@rbtr rbtr requested review from a team as code owners October 4, 2024 22:34
@rbtr rbtr added the cns Related to CNS. label Oct 4, 2024
@rbtr rbtr self-assigned this Oct 4, 2024
cns/restserver/metrics.go Outdated Show resolved Hide resolved
cns/restserver/metrics.go Show resolved Hide resolved
go.mod Outdated Show resolved Hide resolved
cns/restserver/metrics.go Outdated Show resolved Hide resolved
@rbtr rbtr force-pushed the feat/log-ip-states branch 2 times, most recently from ca644f0 to a07cd62 Compare October 14, 2024 20:17
@rbtr rbtr requested a review from timraymond October 15, 2024 01:04
@rbtr rbtr force-pushed the feat/log-ip-states branch 2 times, most recently from 08dfec5 to d00a6af Compare October 15, 2024 19:26
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 Oct 30, 2024
@thatmattlong thatmattlong removed the stale Stale due to inactivity. label Oct 30, 2024
@rbtr rbtr requested a review from timraymond November 5, 2024 16:44
rbtr added 2 commits November 5, 2024 10:48
Signed-off-by: Evan BaKer <rbtr@users.noreply.github.com>
Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
@rbtr rbtr force-pushed the feat/log-ip-states branch from d00a6af to 1d6c4b4 Compare November 5, 2024 16:48
@rbtr
Copy link
Contributor Author

rbtr commented Nov 5, 2024

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rbtr rbtr enabled auto-merge November 5, 2024 16:48
@rbtr rbtr added this pull request to the merge queue Nov 5, 2024
Merged via the queue into master with commit 3c4641c Nov 6, 2024
14 checks passed
@rbtr rbtr deleted the feat/log-ip-states branch November 6, 2024 01:37
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.

3 participants