Feat: Per Tenant Logging

[shaangill025]

• resolve Test Per Tenant Log Access and expand out #2359
• Derived from work done in Per Tenant Logging - Complete Implementation #2425
• Needed to close that PR as the pre-release images in there somehow included legacy code which was removed subsequently.

[shaangill025]

@WadeBarnes Can you please create a pre-release image for this PR.

[WadeBarnes]

@shaangill025, A prerelease image has been built; https://github.com/hyperledger/aries-cloudagent-python/pkgs/container/aries-cloudagent-python/138271403?tag=py3.9-0.11.0-pr2550-pre0

[WadeBarnes]

@shaangill025, based on the testing, it looks like you'll need to update the logging to default to ${HOME}/log (or ./log), for example /home/aries/log/ on the askar only images. On OCP the non-privilege user can't write to ${HOME}.

[WadeBarnes]

New pre-release image; https://github.com/hyperledger/aries-cloudagent-python/pkgs/container/aries-cloudagent-python/141058880?tag=py3.9-0.11.0-pr2550-pre1

[dbluhm]

Any word on when we can look at getting the failing tests resolved?

[esune]

I think changes look good, I am a bit confused about having same settings (e.g.: the log file path) in both default_per_tenant_logging.ini and default_per_tenant_logging.yaml, but I might be missing something.

@dbluhm if you are happy with the latest tweaks we can probably merge.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[shaangill025]

I am a bit confused about having same settings (e.g.: the log file path) in both default_per_tenant_logging.ini and default_per_tenant_logging.yaml, but I might be missing something.

This is about supporting both ini (logging FileConfig) and yml (logging DictConfig) config files. Could use either one of that format.

[swcurran]

Awesome — thanks, @shaangill025 . Thanks for reviewing and merging @esune @dbluhm .

One more to go...