CLI Tests excluded from functional tests workflow #7279
Assignees
Labels
bug
Something is broken or not working as expected
Comments
|
👋 @kachawla Thanks for filing this bug report. A project maintainer will review this report and get back to you soon. If you'd like immediate help troubleshooting, please visit our Discord server. For more information on our triage process please visit our triage overview |
vinayada1
added a commit
that referenced
this issue
Mar 6, 2024
# Description cli tests were moved out of the shared folder. The functional test workflow was missing the cli tests in the matrix and it does not use "make test-functional-all" unlike the long running tests. As a result, these tests were skipped. This PR adds the cli tests to the matrix. ## Type of change <!-- Please select **one** of the following options that describes your change and delete the others. Clearly identifying the type of change you are making will help us review your PR faster, and is used in authoring release notes. If you are making a bug fix or functionality change to Radius and do not have an associated issue link please create one now. --> - This pull request fixes a bug in Radius and has an approved issue (#7279 ). <!-- Please update the following to link the associated issue. This is required for some kinds of changes (see above). --> Fixes: #7279 Signed-off-by: vinayada1 <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Steps to reproduce
This commit 0c2406b moved CLI tests out of the shared folder, resulting into them being skipped from functional test runs.
Observed behavior
CLI tests aren't executing as a part of functional test run workflow, for example: https://github.com/radius-project/radius/actions/runs/8166973316/job/22326708720
Desired behavior
All functional tests should be included in the functional test workflow.
Workaround
Rely on long running workflow failure reporting, since make target for long running workflow is different, explicitly targeting all https://github.com/radius-project/radius/blob/main/.github/workflows/long-running-azure.yaml#L440
rad Version
N/A
Operating system
N/A
Additional context
No response
Would you like to support us?
AB#11403
The text was updated successfully, but these errors were encountered: