Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bugfix: fix secret created later than apisixtls #1715

Merged
merged 2 commits into from
Mar 17, 2023

Conversation

lingsamuel
Copy link
Member

Type of change:

  • Bugfix

What this PR does / why we need it:

Fixes #1190

Cache ApisixTls -> Secret relationship even if the translation failed. This can fix secret created later than ApisixTls case.

@lingsamuel lingsamuel requested review from tao12345666333 and AlinsRan and removed request for tao12345666333 March 8, 2023 01:08
Signed-off-by: Ling Samuel <lingsamuelgrace@gmail.com>
@tao12345666333 tao12345666333 added bug Something isn't working area/controller labels Mar 8, 2023
@tao12345666333 tao12345666333 added this to the v1.7.0 milestone Mar 8, 2023
@tao12345666333
Copy link
Member

please check failed jobs

Signed-off-by: Ling Samuel <lingsamuelgrace@gmail.com>
@codecov-commenter
Copy link

Codecov Report

Merging #1715 (878c350) into master (7d62b7e) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 878c350 differs from pull request most recent head b5d3ae6. Consider uploading reports for the commit b5d3ae6 to get more accurate results

@@           Coverage Diff           @@
##           master    #1715   +/-   ##
=======================================
  Coverage   41.48%   41.49%           
=======================================
  Files          89       89           
  Lines        7667     7668    +1     
=======================================
+ Hits         3181     3182    +1     
  Misses       4122     4122           
  Partials      364      364           
Impacted Files Coverage Δ
cmd/ingress/ingress.go 83.05% <100.00%> (+0.14%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

help request: ingress TLS certificate update
4 participants