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

fix: use better log for no gc #1426

Merged
merged 1 commit into from
May 17, 2023
Merged

fix: use better log for no gc #1426

merged 1 commit into from
May 17, 2023

Conversation

Xunzhuo
Copy link
Member

@Xunzhuo Xunzhuo commented May 16, 2023

What type of PR is this?

fix: use better log for no gc

What this PR does / why we need it:

failed always means something wrong happened, like an error, but after deploying eg, and I did not create gatewayClass, I see this log which makes me feel weird.

@Xunzhuo Xunzhuo requested a review from a team as a code owner May 16, 2023 12:04
Signed-off-by: bitliu <bitliu@tencent.com>
@codecov
Copy link

codecov bot commented May 16, 2023

Codecov Report

Merging #1426 (9ea8774) into main (eaf7ad8) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1426   +/-   ##
=======================================
  Coverage   62.28%   62.29%           
=======================================
  Files          79       79           
  Lines       11260    11260           
=======================================
+ Hits         7013     7014    +1     
+ Misses       3790     3789    -1     
  Partials      457      457           
Impacted Files Coverage Δ
internal/provider/kubernetes/controller.go 50.27% <100.00%> (-0.33%) ⬇️

... and 1 file with indirect coverage changes

@Xunzhuo
Copy link
Member Author

Xunzhuo commented May 17, 2023

cc @qicz

@Xunzhuo Xunzhuo requested a review from qicz May 17, 2023 02:14
@zirain zirain merged commit 89b8b39 into envoyproxy:main May 17, 2023
18 checks passed
tanujd11 pushed a commit to tanujd11/gateway that referenced this pull request May 20, 2023
Signed-off-by: bitliu <bitliu@tencent.com>
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.

None yet

3 participants