Skip to content

Conversation

@ashvindeodhar
Copy link
Member

@ashvindeodhar ashvindeodhar commented Nov 3, 2020

Remove GetNCVersionURLFmt from NodeInfoResponse api contract

This change removes GetNCVersionURLFmt from the NodeInfoResponse which CNS gets
as a part of SyncNodeStatus in case of managed mode. This data is available with CNS and
should not be needed from DNC.

Reason for Change:

Issue Fixed:

Requirements:

Notes:

@codecov
Copy link

codecov bot commented Nov 3, 2020

Codecov Report

Merging #718 into master will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #718   +/-   ##
=======================================
  Coverage   38.89%   38.89%           
=======================================
  Files          80       80           
  Lines       10625    10625           
=======================================
  Hits         4133     4133           
  Misses       5992     5992           
  Partials      500      500           

@ashvindeodhar ashvindeodhar merged commit 2a6ce61 into Azure:master Nov 3, 2020
@ashvindeodhar ashvindeodhar deleted the fork-getncversionurl-update branch November 3, 2020 20:19
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