Unrecognized named-value: 'matrix' in job if conditional

[Joe-Kollin]

Describe the bug
A job generates a matrix as an output and I have a subsequent job that depends on the generation of that matrix. The first job does an API call to get all customers which each have a unique schema association. I only want to run this second job in the event that my input SCHEMA array contains the schema of a given return record. However, when I execute the build I get the following error: Unrecognized named-value: 'matrix'. Located at position 26 within expression: contains(inputs.SCHEMAS, matrix.customer.schema). The pattern works without the conditional, but does not fit my use case.

This is a rough idea of what the job looks like.

job-name:
  runs-on: ubuntu-latest
  needs: get-customers
  if: contains(inputs.SCHEMAS, matrix.customer.schema)
  strategy:
    matrix:
      customer: ${{fromJson(needs.get-customers.outputs.customers)}}
  steps:
     ....

Expected behavior
The job should only run when the matrix schema is contained within the input SCHEMA

Runner Version and Platform

ubuntu-latest

What's not working?

The use of matrix in a job level conditional

[alexandrebrg]

I had the same issue, it seems that if statement is evaluated before running a matrix of jobs. I found this topic on GitHub forum about this problem.

[funes79]

I do have the same issue, and not using any if statement. And the matrix value does not contain any spaces.

  list-dev-configs:
    runs-on: ubuntu-latest
    outputs:
      matrix: ${{ steps.set-matrix.outputs.matrix }}
    steps:
      - uses: actions/checkout@v3
      - id: set-matrix
        run: echo "::set-output name=matrix::[\"A\",\"B\"]"
  check:
    needs: list-dev-configs
    runs-on: ubuntu-latest
    strategy:
        matrix:
            dev_config: ${{ fromJSON(needs.list-dev-configs.outputs.matrix) }}
    steps:
      - uses: actions/checkout@v3   
      - run: |
          echo "${{ needs.list-dev-configs.outputs.matrix }}"
          echo "${{ fromJSON(needs.list-dev-configs.outputs.matrix) }}"
          echo "${{ martix.dev_config }}"

[funes79]

My bad, that was a typo in the script last line martix instead of matrix.

[korewaChino]

Also having this issue. the workflow works fine if there was no if statement.

  build:
    needs: manifest
    strategy:
      matrix: ${{ fromJson(needs.manifest.outputs.build_matrix) }}

    uses: andaman-common-pkgs/github-actions/.github/workflows/anda-rpm.yml@main
    if: ${{ matrix.changed_folders != '' }}
    with:
      name: "rust/${{ matrix.changed_folders }}/pkg"
      subatomicServer: "https://subatomic.fyralabs.com"
      subatomicRepo: ad37
      mockConfig: anda-37-x86_64
      mockConfigPackage: anda-mock-configs
      upload: true
    secrets:
      subatomicToken: ${{ secrets.SUBATOMIC_TOKEN }}

[carmocca]

I also have this problem in Lightning-AI/pytorch-lightning#15043

[jsoref]

You can probably work around this by having additional jobs that generate outputs and relying on needs and the outputs from those jobs.

You might need to demote your ifs from job level to step level.

[garciasdos]

Same problem here. Are there any plans to add the matrix context to the jobs.<job_id>.if condition?

[mrkesu]

I'm also encountering this issue.

Sadly my workflow had to be changed from this:

jobs:
  remediate:
    runs-on: ubuntu-latest
    strategy:
      matrix:
        env: ["dev", "test", "prod"]
    if: ${{ github.event.schedule || matrix.env == github.event.inputs.env }}
    environment: ${{ matrix.env }}
    steps:
      - name: Checkout
        uses: actions/checkout@v2
      - uses: cue-lang/setup-cue@v1.0.0-alpha.2
        with:
      [ etc... ]

to this:

jobs:
  remediate:
    runs-on: ubuntu-latest
    strategy:
      matrix:
        env: ["dev", "test", "prod"]
    environment: ${{ matrix.env }}
    steps:
      - name: Checkout
        if: ${{ github.event.schedule || matrix.env == github.event.inputs.env }}
        uses: actions/checkout@v2
      - uses: cue-lang/setup-cue@v1.0.0-alpha.2
        if: ${{ github.event.schedule || matrix.env == github.event.inputs.env }}
        with:
      [ etc... ]

Obviously with a few more steps than in the above truncated example.

It makes the whole workflow feel a bit messy, especially since the jobs are being marked as successfully run when in reality all the steps within it have been skipped.

(never mind my failed job, it's still in development😅)

[jsoref]

@mrkesu: if you're up for it, you should really try having a job that creates your matrix and then have it create an empty array when you don't want it to do work. #1985 (comment)

Here's a workflow which does something similar:
https://github.com/commercialhaskell/stack/blob/71a85a488ef458d5c7d315b647411056aafd3d0a/.github/workflows/integration-tests.yml#L104-L127