Update logging to use generated logs

[wabalubdub]

This PR uses the generated logs feature to simplify and reduce the log declarations.

closes #2529

Also my first PR so feedback would be lovely.

[linux-foundation-easycla[bot]]

The committers listed above are authorized under a signed CLA.

• :white_check_mark: login: wabalubdub / name: Noam Greenberg (b68076b42526cf32438572586f4e99b1ef316dfd, c54c6841e7592e2e7645ced49e23d6986404aeb8, d0c17385baf708d0f92061037fa50723352195c5)

[JamesNK]

Hi, thanks for the PR.

FYI this is just one file of many with logs. See https://github.com/search?q=repo%3Agrpc%2Fgrpc-dotnet+LoggerMessage.Define&type=code

You're welcome to update more if you'd like or stick with this one. I'll close the issue once all are updated.

[wabalubdub]

Hey, of course! I ordered the methods according to Event ID. I didn't notice there were more, ill work on the rest of them. would you rather i update the rest of the in this PR? I can probably get to all of them tomorrow.

[JamesNK]

One pr is fine

[wabalubdub]

Hey I updated the PR to include changes to all the files that have the LoggerMessage.Define syntax.

A couple notes:

• I opted to keep the interface as identical as possible, therefore when the call for logging included any formatting beyond a call to the default ToString() I kept the original function signature and made a call to a private generated log after the formatting. The LoggerMessage Attribute doesn't seem to have a way to access member variables of objects that are passed to it so I couldn't generate the formatting automatically as well.
• I did the same thing when there were conditional statements for optimization like logger.isTraceEnabled()
• Some of the logging classes are nested in larger classes, the generated logs requires the class to be partial so i made the surrounding class partial as well.

[JamesNK]

Thanks! Looks good to me. There are a lot of changes so I haven't excuasively checked every log, but the ones I did look at were correct.