Skip to content

Commit b8bacb2

Browse files
aidangarskeaidangarske
committed
ci: fix GHA expression string quoting (wolfssl_refs_json != '')
GitHub Actions expressions require single quotes for string literals; the matrix override used double quotes (!= ""), which fails workflow validation - nightly-osp startup_failed with 0 jobs. Replace != "" with != '' across all 40 OSP workflow matrix lines.
1 parent b120a82 commit b8bacb2

40 files changed

Lines changed: 78 additions & 78 deletions

‎.github/workflows/bind9.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
strategy:
2727
fail-fast: false
2828
matrix:
29-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
29+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
3030
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
3131
fips_ref: [ 'FIPS', 'non-FIPS' ]
3232
replace_default: [ true ]
@@ -44,7 +44,7 @@ jobs:
4444
fail-fast: false
4545
matrix:
4646
bind_ref: [ 'v9.18.28' ]
47-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
47+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
4848
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
4949
fips_ref: [ 'FIPS', 'non-FIPS' ]
5050
force_fail: ['WOLFPROV_FORCE_FAIL=1', '']

‎.github/workflows/cjose.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
strategy:
2727
fail-fast: false
2828
matrix:
29-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
29+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
3030
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
3131
fips_ref: [ 'FIPS', 'non-FIPS' ]
3232
replace_default: [ true ]
@@ -46,7 +46,7 @@ jobs:
4646
matrix:
4747
# Dont test osp master since it might be unstable
4848
cjose_ref: [ 'v0.6.2.1' ]
49-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
49+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
5050
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
5151
fips_ref: [ 'FIPS', 'non-FIPS' ]
5252
force_fail: ['WOLFPROV_FORCE_FAIL=1', '']

‎.github/workflows/curl.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
strategy:
2727
fail-fast: false
2828
matrix:
29-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
29+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
3030
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
3131
fips_ref: [ 'FIPS', 'non-FIPS' ]
3232
replace_default: [ true ]
@@ -43,7 +43,7 @@ jobs:
4343
fail-fast: false
4444
matrix:
4545
curl_ref: [ 'curl-8_4_0', 'curl-7_88_1' ]
46-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
46+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
4747
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
4848
fips_ref: [ 'FIPS', 'non-FIPS' ]
4949
force_fail: ['WOLFPROV_FORCE_FAIL=1', '']

‎.github/workflows/debian-package.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ jobs:
2525
strategy:
2626
fail-fast: false
2727
matrix:
28-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
28+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
2929
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
3030
fips_ref: [ 'FIPS', 'non-FIPS' ]
3131
replace_default: [ true, false ]
@@ -44,7 +44,7 @@ jobs:
4444
strategy:
4545
fail-fast: false
4646
matrix:
47-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
47+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
4848
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
4949
fips_ref: [ 'FIPS', 'non-FIPS' ]
5050
force_fail: ['WOLFPROV_FORCE_FAIL=1', '']

‎.github/workflows/git-ssh-dr.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
strategy:
2727
fail-fast: false
2828
matrix:
29-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
29+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
3030
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
3131
fips_ref: [ 'FIPS', 'non-FIPS' ]
3232
replace_default: [ true ]
@@ -43,7 +43,7 @@ jobs:
4343
strategy:
4444
fail-fast: false
4545
matrix:
46-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
46+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
4747
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
4848
fips_ref: [ 'FIPS', 'non-FIPS' ]
4949
replace_default: [ true ]

‎.github/workflows/grpc.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
strategy:
2727
fail-fast: false
2828
matrix:
29-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
29+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
3030
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
3131
fips_ref: [ 'FIPS', 'non-FIPS' ]
3232
replace_default: [ true ]
@@ -51,7 +51,7 @@ jobs:
5151
ssl_transport_security_test ssl_transport_security_utils_test
5252
test_core_security_ssl_credentials_test test_cpp_end2end_ssl_credentials_test
5353
h2_ssl_cert_test h2_ssl_session_reuse_test
54-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
54+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
5555
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
5656
fips_ref: [ 'FIPS', 'non-FIPS' ]
5757
force_fail: ['WOLFPROV_FORCE_FAIL=1', '']

‎.github/workflows/hostap.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
strategy:
2727
fail-fast: false
2828
matrix:
29-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
29+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
3030
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
3131
fips_ref: [ 'FIPS', 'non-FIPS' ]
3232
replace_default: [ true ]
@@ -46,7 +46,7 @@ jobs:
4646
fail-fast: false
4747
matrix:
4848
hostap_ref: [ 'hostap_2_11' ]
49-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
49+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
5050
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
5151
fips_ref: [ 'FIPS', 'non-FIPS' ]
5252
force_fail: ['WOLFPROV_FORCE_FAIL=1', '']

‎.github/workflows/iperf.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
strategy:
2727
fail-fast: false
2828
matrix:
29-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
29+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
3030
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
3131
fips_ref: [ 'FIPS', 'non-FIPS' ]
3232
replace_default: [ true ]
@@ -44,7 +44,7 @@ jobs:
4444
fail-fast: false
4545
matrix:
4646
iperf_ref: [ '3.12' ]
47-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
47+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
4848
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
4949
fips_ref: [ 'FIPS', 'non-FIPS' ]
5050
force_fail: ['WOLFPROV_FORCE_FAIL=1', '']

‎.github/workflows/krb5.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
strategy:
2727
fail-fast: false
2828
matrix:
29-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
29+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
3030
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
3131
fips_ref: [ 'FIPS', 'non-FIPS' ]
3232
replace_default: [ true ]
@@ -44,7 +44,7 @@ jobs:
4444
fail-fast: false
4545
matrix:
4646
krb5_ref: [ 'krb5-1.20.1-final' ]
47-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
47+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
4848
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
4949
fips_ref: [ 'FIPS', 'non-FIPS' ]
5050
force_fail: ['WOLFPROV_FORCE_FAIL=1', '']

‎.github/workflows/libcryptsetup.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
strategy:
2727
fail-fast: false
2828
matrix:
29-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
29+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
3030
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
3131
fips_ref: [ 'FIPS', 'non-FIPS' ]
3232
replace_default: [ true ]
@@ -44,7 +44,7 @@ jobs:
4444
fail-fast: false
4545
matrix:
4646
cryptsetup_ref: [ 'v2.6.1' ]
47-
wolfssl_ref: ${{ inputs.wolfssl_refs_json != "" && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
47+
wolfssl_ref: ${{ inputs.wolfssl_refs_json != '' && fromJson(inputs.wolfssl_refs_json) || fromJson(needs.discover_versions.outputs.wolfssl_ref_array) }}
4848
openssl_ref: ${{ fromJson(needs.discover_versions.outputs.openssl_ref_array) }}
4949
fips_ref: [ 'FIPS', 'non-FIPS' ]
5050
force_fail: ['WOLFPROV_FORCE_FAIL=1', '']

0 commit comments

Comments
 (0)