<#6880 fix: resolve flytestdlib test failures> Pul...
# flyte-github
a
#6880 fix: resolve flytestdlib test failures Pull request opened by ankitkumarsingh1702 Summary This PR fixes two flaky/failing tests in flytestdlib on the v2 branch. Fixes Applied 1.
TestRegisterTracerProviderWithContext
(otelutils/factory.go)
Root Cause: Schema URL conflict between
resource.Default()
(which uses the OTel default schema) and
semconv
(which uses a different schema version) when calling
resource.Merge()
. Solution: Changed
resource.NewWithAttributes(semconv.SchemaURL, ...)
to
resource.NewSchemaless(...)
to avoid the schema version mismatch. 2.
TestAccessor_UpdateConfig/[Viper]_Change_handler_k8s_configmaps
(config/tests/accessor_test.go)
Root Cause: This test relies on fsnotify to detect symlink changes (mimicking K8s ConfigMap behavior), but fsnotify doesn't reliably detect symlink changes on macOS/Darwin systems. Solution: • Added
t.Skip()
for Darwin systems with an explanatory message • Reference: fsnotify/fsnotify#372 Testing All tests pass locally after these changes. Fixes #6856 •
main
• #6583 • #6880 👈 flyteorg/flyte