beginner: add coverage

[mattklein123]

Going to leave open a persistent beginner ticket for adding coverage: https://s3.amazonaws.com/lyft-envoy/coverage/report-master/coverage.html

There are always a few missing things here and there especially regarding random error handling. If you are a beginner this is a great way to learn and do something valuable.

Rough steps:

• Look at the coverage report and find something (often a single line) that looks like it should be covered.
  • Find the relevant unit test that should cover it (often in the test tree with the same file name)
  • Write the test
  • Confirm in PR coverage build that line is covered. (Coverage report is posted as a build artifact).

[minhlongdo]

@mattklein123 Hi, could I pick this one up?

[mattklein123]

@minhlongdo this is a "perma issue." Feel free to just go ahead and add some coverage!

[williambanfield]

@mattklein123 I went to try my hand at one of these and noticed that the s3 bucket gives an AccessDenied error. Any idea why that may be?

[mattklein123]

Sorry the coverage report has moved here: https://storage.googleapis.com/envoy-coverage/report-master/index.html

[saiskee]

@mattklein123 Where can I find the coverage report? Above link gives key does not exist error

Edit: Found it at https://storage.googleapis.com/envoy-postsubmit/main/coverage/index.html

[phlax]

@saiskee clicking that link work for me - not sure what the error you saw was but should work

[saiskee]

@phlax

[phlax]

apologies - i see now - i clicked the link that you posted - the one that works

[JayLog22]

Hi! I would love to help a bit with coverage, but getting access denied on the link above for coverage report. Is help still needed?

[mattklein123]

https://storage.googleapis.com/envoy-postsubmit/main/coverage/index.html should be the working URL and I think it's wide open.