Add sync for logging calls to ensure aggregation across devices
# What does this PR do ?

After speaking to @nithinraok a few places do not have sync calls when logging (some are taken care off by the underlying Metrics, or they are already synced across processes like global steps).

I've been a bit cautious and not touched the return objects from `validation_step` or `test_step`. There seems to be some parsing of logic in the ModelPT file so need to investigate further as to what to do with those metrics, and how they are logged.

# Changelog 
- Add `sync_dist` to logging metric calls to ensure aggregation across all devices.

# Before your PR is "Ready for review"
**Pre checks**:
- [x] Make sure you read and followed [Contributor guidelines](https://github.com/NVIDIA/NeMo/blob/main/CONTRIBUTING.md)
- [ ] Did you write any new necessary tests?
- [ ] Did you add or update any necessary documentation?
- [ ] Does the PR affect components that are optional to install? (Ex: Numba, Pynini, Apex etc)
  - [ ] Reviewer: Does the PR have correct import guards for all optional libraries?
  
**PR Type**:
- [ ] New Feature
- [x] Bugfix
- [ ] Documentation

## Who can review?

Anyone in the community is free to review the PR once the checks have passed. 
[Contributor guidelines](https://github.com/NVIDIA/NeMo/blob/main/CONTRIBUTING.md) contains specific people who can review PRs to various areas.