set viper delimiter to :: to support . in config keys

[pvditt]

Tracking issue

closes: #6166

Why are the changes needed?

parsing configs breaks when "." is part of a key

What changes were proposed in this pull request?

Update key delimiter to "::" when parsing configs

Note both "." and "::" in keys are valid yaml, but "." seems more likely to be utilized than "::"

How was this patch tested?

• updated unit test

Labels

Please add one or more of the following labels to categorize your PR:

• added: For new features.
• changed: For changes in existing functionality.
• deprecated: For soon-to-be-removed features.
• removed: For features being removed.
• fixed: For any bug fixed.
• security: In case of vulnerabilities

This is important to improve the readability of release notes.

Setup process

Screenshots

Check all the applicable boxes

• I updated the documentation accordingly.
• All new and existing tests passed.
• All commits are signed-off.

Related PRs

Docs link

[flyte-bot]

Code Review Agent Run Status

• Limitations and other issues: ❌ Failure - The AI Code Review Agent skipped reviewing this change because it is configured to exclude certain pull requests based on the source/target branch or the pull request status. You can change the settings here, or contact the agent instance creator at [email protected].

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.

Project coverage is 37.06%. Comparing base (729e71a) to head (56a609c).

Files with missing lines	Patch %	Lines
flytestdlib/config/viper/viper.go	0.00%	5 Missing ⚠️
flytestdlib/config/viper/collection.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #6178   +/-   ##
=======================================
  Coverage   37.05%   37.06%           
=======================================
  Files        1318     1318           
  Lines      132638   132638           
=======================================
+ Hits        49151    49158    +7     
+ Misses      79237    79230    -7     
  Partials     4250     4250           

Flag	Coverage Δ
unittests-datacatalog	51.58% <ø> (ø)
unittests-flyteadmin	54.33% <ø> (+0.02%)	⬆️
unittests-flytecopilot	30.99% <ø> (ø)
unittests-flytectl	62.29% <ø> (ø)
unittests-flyteidl	7.23% <ø> (ø)
unittests-flyteplugins	53.85% <ø> (ø)
unittests-flytepropeller	42.69% <ø> (ø)
unittests-flytestdlib	55.29% <0.00%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[flyte-bot]

Code Review Agent Run Status

• Limitations and other issues: ❌ Failure - The AI Code Review Agent skipped reviewing this change because it is configured to exclude certain pull requests based on the source/target branch or the pull request status. You can change the settings here, or contact the agent instance creator at [email protected].