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

[Quota Manager] Fix k8s Deployment Issues #391

Merged
merged 182 commits into from
Sep 24, 2020

Conversation

kevin-zhonghao
Copy link
Contributor

@kevin-zhonghao kevin-zhonghao commented Sep 24, 2020

This PR proposes to fix k8s deployment issues for quota manager:

  1. update quota_manager.yaml and api_gateway.yaml
  2. update dockerfile of quota manager

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 changed the title [Quota-Manager] Fix k8s Deployment Issues [Quota Manager] Fix k8s Deployment Issues Sep 24, 2020
@codecov-commenter
Copy link

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #391   +/-   ##
=========================================
  Coverage     35.09%   35.10%           
  Complexity      992      992           
=========================================
  Files           422      422           
  Lines         10415    10415           
  Branches       1256     1256           
=========================================
+ Hits           3655     3656    +1     
+ Misses         6296     6295    -1     
  Partials        464      464           
Impacted Files Coverage Δ Complexity Δ
...alcor/elasticipmanager/dao/ElasticIpAllocator.java 63.81% <0.00%> (+0.27%) 48.00% <0.00%> (ø%)

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 78d66d6...a6cfa95. Read the comment docs.

@xieus xieus merged commit 740b604 into futurewei-cloud:master Sep 24, 2020
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.

4 participants