[NDMII-3168] Fix Citrix Profile

[dplepage-dd]

What does this PR do?

Fix a typo in one of our out-of-the-box SNMP profiles.

Metric tags using the old syntax (where OID and name are top-level instead part of Symbol) don't support the format specifier, and it gets silently ignored by the YAML loader.

From a user perspective, the only thing that will change is that anyone using citrix netscaler SDX devices will suddenly see a few tags that are IP addresses be formatted as such instead of as raw bytes.

Motivation

DataDog/datadog-agent#31054 revealed this issue.

Review checklist (to be filled by reviewers)

• Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
• Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
• If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.07%. Comparing base (3fa69c8) to head (fd3ab43).
Report is 2 commits behind head on master.

Additional details and impacted files

Flag	Coverage Δ
activemq	?
cassandra	?
hive	?
hivemq	?
hudi	?
ignite	?
jboss_wildfly	?
kafka	?
presto	?
snmp	95.71% <ø> (-0.02%)	⬇️
solr	?

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

[NouhaManai96]

Can you add a release note/changelog ?

[dplepage-dd]

My fix was tripping up the python validator and I don't have time to fix that right now, so I replaced it with a fix that just comments out the format block (which is being ignored anyway) and adds a todo to upgrade this to the newer structure and fix the format then. This shouldn't need a changelog now, because it shouldn't affect the behavior at all (the field was already being ignored by YAML when the struct was being loaded).