Skip to content

GHA: set concurrency: where missing#19215

Closed
vszakats wants to merge 1 commit intocurl:masterfrom
vszakats:zizmornew
Closed

GHA: set concurrency: where missing#19215
vszakats wants to merge 1 commit intocurl:masterfrom
vszakats:zizmornew

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Oct 24, 2025

To silence zizmor 1.16.0 warnings.

Also:

Expressions used (before and after this patch):

  • group: ${{ github.workflow }}-${{ github.event.sha }}-${{ github.event.target_url }}
    for GHA/appveyor-status.
  • group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}
    for all the rest.

Ref: https://github.com/curl/curl/actions/runs/18776245057/job/53571438139?pr=19209


I admit to not grasp how concurrency group expression works and if there
is supposed to be a special case for codeql, or appveyor-status.

Also:
- http3-linux: replace hard-coded workflow name with variable.
  Follow-up to a817417 curl#13841
- codeql: set `cancel-in-progress: true`.
- codeql: sync concurrency setting with the rest of the jobs.

I admit to not grasp how concurrency settings works and if there is
supposed to be a special case for codeql.

Expressions used (before and after this patch):
- group: ${{ github.workflow }}-${{ github.event.sha }}-${{ github.event.target_url }}
  for GHA/appveyor-status.
- group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}
  for all the rest.
@vszakats vszakats added the CI Continuous Integration label Oct 24, 2025
@vszakats vszakats changed the title GHA: add concurrency: where missing GHA: set concurrency: where missing Oct 24, 2025
@vszakats vszakats closed this in 82fa986 Oct 24, 2025
@vszakats vszakats deleted the zizmornew branch October 24, 2025 11:38
vszakats added a commit that referenced this pull request Oct 24, 2025
```
Fuzzer
Canceling since a deadlock was detected for concurrency group:
'Fuzzer-82fa9862dfa3083d4014d6dcfb721a7278e66f0b' between a top level workflow and 'Fuzzing'
```
https://github.com/curl/curl/actions/runs/18778617351

Follow-up to 82fa986 #19215
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous Integration

Development

Successfully merging this pull request may close these issues.

1 participant