Skip to content

Conversation

@vakalapa
Copy link
Contributor

@vakalapa vakalapa commented Feb 2, 2021

  • Ignoring hostnetwork pods from being added into Ipsets

  • generalizing the check on hostnetwork pod

  • Adding tests for add, update and delete hostnetwork pods

* Ignoring hostnetwork pods from being added into Ipsets

* generalizing the check on hostnetwork pod

* Adding tests for add, update and delete hostnetwork pods
@vakalapa vakalapa requested review from csfmomo and matmerr February 2, 2021 22:48
@codecov
Copy link

codecov bot commented Feb 2, 2021

Codecov Report

Merging #777 (40cc3f8) into v1.2.2_hotfix (cc3f970) will increase coverage by 2.16%.
The diff coverage is 100.00%.

@@                Coverage Diff                @@
##           v1.2.2_hotfix     #777      +/-   ##
=================================================
+ Coverage          41.41%   43.57%   +2.16%     
=================================================
  Files                142      142              
  Lines              13517    14141     +624     
=================================================
+ Hits                5598     6162     +564     
- Misses              7214     7239      +25     
- Partials             705      740      +35     

@vakalapa vakalapa changed the base branch from master to v1.2.2_hotfix February 3, 2021 01:08
@vakalapa
Copy link
Contributor Author

vakalapa commented Feb 3, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@matmerr
Copy link
Member

matmerr commented Feb 3, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vakalapa vakalapa merged commit 82768ac into v1.2.2_hotfix Feb 3, 2021
@vakalapa vakalapa deleted the vakr/hotnet_hostfix branch February 12, 2021 19:02
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.

4 participants