-
Notifications
You must be signed in to change notification settings - Fork 20
Description
Bug Description
The workflow tests .github/workflows/wokflow_test.yaml refers to existing re-usable workflows. However, the re-usable workflows (e.g. integration_test.yaml) then calls the code in the /internal directory, pinned to the @main reference (i.e. canonical/operator-workflows/internal/plan@main).
This makes any changes to the /internal actions workflow call not be tested and be subject to human errors.
Impact
Medium (functionality degraded, workaround exists)
Impact Rationale
Currently, there is now workaround since GitHub does not allow dynamic reference to a re-usable workflow.
However, we might be able to improve this by having development/staging workflow operations. See: https://github.com/orgs/community/discussions/45342
Link to the failing workflow
There is no failing workflow because we're not testing it.
Additional context
When index maps are generated using npm run all, we can only assume that it would work or test it by calling that particular workflow from another repository. This is very inefficient and is subject to human errors. We should improve the process in which we contribute to the operator-workflows repository since it is a workflow used by many teams other than our own.