Skip to content

Conversation

@zhiying-lin
Copy link
Contributor

What type of PR is this?

/kind feature

What this PR does / why we need it:

validate whether the atm profile and populate the backend status

Which issue(s) this PR fixes:

Fixes #

Requirements:

How has this code been tested

Special notes for your reviewer

@zhiying-lin zhiying-lin force-pushed the trafficmanager-backend-update-2 branch from e335a36 to c3e23f6 Compare November 5, 2024 03:38
@codecov
Copy link

codecov bot commented Nov 5, 2024

Codecov Report

Attention: Patch coverage is 91.66667% with 3 lines in your changes missing coverage. Please review.

Project coverage is 78.83%. Comparing base (728dd0d) to head (c96852a).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...ontrollers/hub/trafficmanagerbackend/controller.go 91.66% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #208      +/-   ##
==========================================
+ Coverage   78.42%   78.83%   +0.41%     
==========================================
  Files          27       27              
  Lines        3504     3539      +35     
==========================================
+ Hits         2748     2790      +42     
+ Misses        604      600       -4     
+ Partials      152      149       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@zhiying-lin zhiying-lin merged commit 5fb2495 into Azure:main Nov 6, 2024
10 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.

2 participants