fix: bump matrix version#509
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7c47d405ad
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| persist-credentials: false | ||
| - id: matrix | ||
| uses: splunk/addonfactory-test-matrix-action@v3.1 | ||
| uses: splunk/addonfactory-test-matrix-action@v3.2 |
There was a problem hiding this comment.
Pin the matrix action to the 10.4 tag
In workflow runs that rely on needs.meta.outputs.matrix_Splunk, this ref still resolves to upstream v3.2/action.yml, which runs the Docker image ...:v3.2.0; that tag's splunk_matrix.conf still has LATEST = 10.2 and no [10.4] entry, while v3.2.1 is the release containing Splunk 10.4. As a result, the bump does not actually add 10.4 coverage, so scheduled/latest-Splunk test matrices continue testing 10.2 until this is pinned to the tag/image that contains the 10.4 matrix.
Useful? React with 👍 / 👎.
Description
This PR bumps matrix version to 3.2 including Splunk 10.4
Checklist
README.mdhas been updated or is not requiredTesting done
(for each selected checkbox, the corresponding test results link should be listed here)