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

demo: Fixing incorrect log line - should be generalized to 'web server' #4067

Merged
merged 1 commit into from
Sep 1, 2021

Conversation

draychev
Copy link
Contributor

@draychev draychev commented Sep 1, 2021

This PR fixes a small log issue, where the bookbuyer pod claims it is bookstore, which is not true.

The fix generalized the message to announce that a 'Web server' is running, not specifically bookbuyer or a booskstore.

Example:
image

@draychev draychev requested a review from a team as a code owner September 1, 2021 01:21
@codecov-commenter
Copy link

Codecov Report

Merging #4067 (9604189) into main (2d7f62b) will increase coverage by 0.11%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4067      +/-   ##
==========================================
+ Coverage   68.32%   68.43%   +0.11%     
==========================================
  Files         206      206              
  Lines       11671    11671              
==========================================
+ Hits         7974     7987      +13     
+ Misses       3647     3634      -13     
  Partials       50       50              
Flag Coverage Δ
unittests 68.43% <ø> (+0.11%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/certificate/rotor/rotor.go 84.37% <0.00%> (-3.13%) ⬇️
pkg/crdconversion/crdconversion.go 75.00% <0.00%> (+15.21%) ⬆️

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 2d7f62b...9604189. Read the comment docs.

@draychev draychev merged commit 2cec5d7 into openservicemesh:main Sep 1, 2021
@draychev draychev deleted the demo-fix branch September 1, 2021 18:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants