Skip to content

TestExternallyManagedServiceAccount is flaky #6670

@jbw976

Description

@jbw976

We've seen the TestExternallyManagedServiceAccount e2e test fail a lot recently. It's starting to become easily ignored with how frequently it fails, so we should fix it 😇

According to buildpulse, it was pretty reliable until just a couple weeks ago:

It failed in two different test suites for me in the same PR:

A common failure looks like:

                  +e2e |     pkg_test.go:235: Waiting 3m0s for Function.pkg.crossplane.io function-dummy-pkg-external-sa to become HealthyPackageRevision, ActivePackageRevision...
                  +e2e |     feature.go:419: - CONDITION: Function.pkg.crossplane.io function-dummy-pkg-external-sa[@1]: Healthy=False[@1] Reason=UnhealthyPackageRevision: Package revision health is "Unknown" (2025-07-30 22:44:41 +0000 UTC)
                  +e2e | I0730 22:47:44.743575     252 warnings.go:110] "Warning: v1 Endpoints is deprecated in v1.33+; use discovery.k8s.io/v1 EndpointSlice"
                  +e2e | I0730 22:47:45.349647     252 warnings.go:110] "Warning: v1 ComponentStatus is deprecated in v1.19+"
                  +e2e | I0730 22:47:54.943886     252 warnings.go:110] "Warning: apiextensions.crossplane.io Usage is deprecated; migrate to protection.crossplane.io Usage or ClusterUsage"
                  +e2e |     pkg_test.go:235: resource did not have desired conditions: HealthyPackageRevision, ActivePackageRevision: context deadline exceeded:
                  +e2e |         
                  +e2e |         apiVersion: pkg.crossplane.io/v1beta1
                  +e2e |         kind: Function
                  +e2e |         metadata:
                  +e2e |           creationTimestamp: "2025-07-30T22:44:40Z"
                  +e2e |           generation: 1
                  +e2e |           name: function-dummy-pkg-external-sa
                  +e2e |           resourceVersion: "1830"
                  +e2e |           uid: e4bb57a9-64a3-4993-b8af-ba83a72de7dc
                  +e2e |         spec:
                  +e2e |           ignoreCrossplaneConstraints: false
                  +e2e |           package: xpkg.crossplane.io/crossplane-contrib/function-dummy:v0.4.1
                  +e2e |           packagePullPolicy: IfNotPresent
                  +e2e |           revisionActivationPolicy: Automatic
                  +e2e |           revisionHistoryLimit: 1
                  +e2e |           runtimeConfigRef:
                  +e2e |             apiVersion: pkg.crossplane.io/v1beta1
                  +e2e |             kind: DeploymentRuntimeConfig
                  +e2e |             name: default
                  +e2e |           skipDependencyResolution: false
                  +e2e |         status:
                  +e2e |           conditions:
                  +e2e |           - lastTransitionTime: "2025-07-30T22:44:41Z"
                  +e2e |             message: Package revision health is "Unknown"
                  +e2e |             observedGeneration: 1
                  +e2e |             reason: UnhealthyPackageRevision
                  +e2e |             status: "False"
                  +e2e |             type: Healthy
                  +e2e |           - lastTransitionTime: "2025-07-30T22:44:41Z"
                  +e2e |             observedGeneration: 1
                  +e2e |             reason: ActivePackageRevision
                  +e2e |             status: "True"
                  +e2e |             type: Installed
                  +e2e |           currentIdentifier: xpkg.crossplane.io/crossplane-contrib/function-dummy:v0.4.1
                  +e2e |           currentRevision: function-dummy-pkg-external-sa-b8a8590f61fb
                  +e2e |           resolvedPackage: xpkg.crossplane.io/crossplane-contrib/function-dummy:v0.4.1
                  +e2e |         
                  +e2e |         
                  +e2e |         Related objects:
                  +e2e |         
                  +e2e |         
                  +e2e |     --- FAIL: TestExternallyManagedServiceAccount/TestExternallyManagedServiceAccount (198.74s)

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    Status

    Backlog

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions