@blowmage I tried setting the logger as you suggested in #1907, but I got this warning:

irb(main):006:0> GRPC::DefaultLogger::LOGGER = my_logger
(irb):6: warning: already initialized constant GRPC::DefaultLogger::LOGGER

So instead I adapted the suggested example in the gRPC spec_helper.rb. It works, but can anyone suggest a simplification?

I agree it should not be as difficult as it is to set a custom Logger. Ideally it should be simply:

GRPC.logger = my_logger

@dazuma Updated, PTAL.

Thanks for approving, I will copy this change to the READMEs (and "guide" docs where appropriate) for all gRPC packages, then ping for a second look.

@jbolinger PTAL at the commit for generated packages. Does this addition look ok? If so, can it also be added in the generator? Should I open an issue in googleapis/gapic-generator?

@blowmage @dazuma Updated for all gRPC packages, both READMEs and guides, if you want to take a quick look. Text is unchanged from what you approved.

@quartzmo looks good! Feel free to open an issue there for this or any other changes you think might be useful. This one looks like it's just static text for each client. Adding it to the generator should be no problem.

@jbolinger I opened googleapis/gapic-generator#2022, "Ruby: Add logging configuration instructions to README and guide docs".

@jbolinger PTAL, I just added two commits, for changes we discussed in googleapis/gapic-generator#2022 :

• Move logging docs below Next Steps in READMEs and guides.
• Add Supported Ruby Versions above Next Steps in READMEs if missing.

@quartzmo why did you move the version section above the next steps? I thought we said below and at ##, like this: https://gist.github.com/jbolinger/0bcc9b62d4ee84a59fd454c8cc347f6f

Ah, sorry, I didn't understand. I did it because in a number of the generated READMEs, such as in dataproc for example it is above Next Steps. So, I thought it must be need to be there. However, I'm happy to move it down as in your gist. Will update.

@jbolinger Updated, PTAL. I will squash commits before merging.

@quartzmo looks good!