Skip to content

Commit

Permalink
patch(integration_test_charm.yaml): Add juju version prefix (#92)
Browse files Browse the repository at this point in the history
In nested CI calls (e.g. release.yaml calls ci.yaml calls this workflow)
the job name in ci.yaml will not show up on the GitHub Actions sidebar.

If this workflow is called with a matrix (e.g. to test multiple juju
versions), the ci.yaml job name containing the Juju version will be
lost.

So, we add the Juju version to one of the first jobs in this workflow.

(In the UI, when this workflow is called with a matrix, GitHub will
separate each matrix combination and preserve job ordering within a
matrix combination.)
  • Loading branch information
carlcsaposs-canonical authored Nov 15, 2023
1 parent 317190b commit c9bf1f5
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion .github/workflows/integration_test_charm.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,14 @@ jobs:
file-name: integration_test_charm.yaml

collect-integration-tests:
name: Collect integration test groups
# In nested CI calls (e.g. release.yaml calls ci.yaml calls this workflow) the job name in
# ci.yaml will not show up on the GitHub Actions sidebar.
# If this workflow is called with a matrix (e.g. to test multiple juju versions), the ci.yaml
# job name containing the Juju version will be lost.
# So, we add the Juju version to one of the first jobs in this workflow.
# (In the UI, when this workflow is called with a matrix, GitHub will separate each matrix
# combination and preserve job ordering within a matrix combination.)
name: ${{ inputs.juju-agent-version || inputs.juju-snap-channel }} | Collect integration test groups
runs-on: ubuntu-latest
steps:
- name: Checkout
Expand Down

0 comments on commit c9bf1f5

Please sign in to comment.