Skip to content

Conversation

@neaggarwMS
Copy link
Member

Reason for Change:

Issue Fixed:

Requirements:

Notes:

Copy link
Collaborator

@thatmattlong thatmattlong left a comment

Choose a reason for hiding this comment

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

To generate the new manifest file,

cd nodenetworkconfig
make manifests
git add manifests/

@codecov
Copy link

codecov bot commented Oct 26, 2020

Codecov Report

Merging #703 into master will increase coverage by 1.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #703      +/-   ##
==========================================
+ Coverage   38.62%   39.77%   +1.15%     
==========================================
  Files          79       80       +1     
  Lines       10471    11046     +575     
==========================================
+ Hits         4044     4394     +350     
- Misses       5938     6135     +197     
- Partials      489      517      +28     

Copy link
Collaborator

@thatmattlong thatmattlong left a comment

Choose a reason for hiding this comment

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

LGTM

@thatmattlong
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@neaggarwMS neaggarwMS merged commit e40551f into master Nov 3, 2020
@rbtr rbtr deleted the neaggarw_updateCRDDef branch November 30, 2021 20:14
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