Skip to content
This repository was archived by the owner on Mar 31, 2023. It is now read-only.

Conversation

@pkommoju
Copy link
Contributor

Add gateway manager delete operations in k8s

@codecov-commenter
Copy link

Codecov Report

Merging #622 (05bc14c) into master (35c879f) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #622      +/-   ##
============================================
+ Coverage     31.91%   31.93%   +0.02%     
- Complexity     1247     1249       +2     
============================================
  Files           525      525              
  Lines         13462    13462              
  Branches       1666     1666              
============================================
+ Hits           4296     4299       +3     
+ Misses         8579     8576       -3     
  Partials        587      587              
Impacted Files Coverage Δ
...alcor/elasticipmanager/dao/ElasticIpAllocator.java 64.08% <0.00%> (+0.27%) ⬆️
...alcor/portmanager/util/RestParameterValidator.java 46.15% <0.00%> (+3.07%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 35c879f...05bc14c. Read the comment docs.

@pkommoju pkommoju requested review from cj-chung and xieus May 29, 2021 02:02
@xieus xieus added this to the Version 0.17.2021.07.30 milestone May 29, 2021
@xieus xieus changed the title Node upload yaml fix 03 [Deployment] Add Gateway manager cleanup in K8s deployment script May 29, 2021
Copy link
Contributor

@xieus xieus left a comment

Choose a reason for hiding this comment

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

LGTM

@xieus xieus merged commit c4c3189 into futurewei-cloud:master May 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants