SNMP: comment-only change to cisco-catalyst profile (CI baseline, do not merge)#23766
SNMP: comment-only change to cisco-catalyst profile (CI baseline, do not merge)#23766ian28223 wants to merge 3 commits into
Conversation
Adds a single comment line to verify the e2e test failures seen on PR #23498 are environmental rather than caused by any profile content change. No behavior change; not for merge.
🛑 Gate Violations
|
Adds the same one-line comment to _cisco-catalyst.yaml and cisco-sb.yaml so the CI test selector picks the same e2e tests as PR #23498. Lets us compare like-for-like on the e2e_core_profiles failures. No behavior change; not for merge.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files🚀 New features to boost your workflow:
|
Validation Report
Run Passed validations (20)
|
What does this PR do?
Adds comment-only lines to three profile YAML files in
snmp/datadog_checks/snmp/data/default_profiles/:cisco-catalyst.yaml_cisco-catalyst.yamlcisco-sb.yamlNo behavior change. No content change. Just one comment line in each file so the CI test selector schedules the same e2e tests that PR #23498 triggers.
Motivation
Diagnostic baseline for AGENT-15682 and PR #23498. PR #23498 saw failures in
test_e2e_snmp_listener,test_e2e_core.py::*,test_e2e_core_metadata.py::test_e2e_core_metadata_cisco_catalyst, and all oftest_e2e_core_profiles/. Many of those tests cover profiles entirely unrelated to the cisco-catalyst / cisco-sb changes in #23498, suggesting environmental flake.The initial single-file version of this PR confirmed
test_e2e_snmp_listeneris environmental (failed identically on both PRs), but thetest_e2e_core_profiles/tests were skipped on baseline because the file scope didn't trigger their selection. This expanded baseline touches the same three files as #23498, so the same tests should be scheduled and we get a like-for-like comparison.Not for merge. Close after CI signal is observed.
Review checklist (to be filled by reviewers)