Skip to content

Commit

Permalink
Merge pull request #29 from theohbrothers/refactor/ci-use-a-job-group…
Browse files Browse the repository at this point in the history
…-key-to-group-jobs

Refactor (ci): Use a `job_group_key` to group jobs
  • Loading branch information
leojonathanoh authored Sep 25, 2023
2 parents b9bb424 + c07d03a commit f8b5476
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 9 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci-master-pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
test-nogitdiff:
runs-on: ubuntu-latest
container:
image: mcr.microsoft.com/powershell:7.2.2-alpine-3.14-20220318
image: mcr.microsoft.com/powershell:lts-7.2-alpine-3.17
steps:
- run: |
apk add --no-cache git
Expand Down
5 changes: 1 addition & 4 deletions .vscode/tasks.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,7 @@
"-c",
"Generate-DockerImageVariants ."
],
"group": {
"kind": "build",
"isDefault": true
}
"group": "build"
},
{
"label": "Install DockerImageVariants module",
Expand Down
3 changes: 1 addition & 2 deletions generate/definitions/VARIANTS.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -38,12 +38,11 @@ $VARIANTS = @(
package = $package
package_version = $package_version
platforms = 'linux/amd64'
job_group_key = $package_version
}
tag = "v$package_version-$package"
# The latest cloudflare image will be tagged :latest. E.g. v1.12.0-cloudflare
tag_as_latest = if ($package -eq 'cloudflare' -and $package_version -eq $local:PACKAGE_VERSIONS[0]) { $true } else { $false }


buildContextFiles = @{
templates = @{
'Dockerfile' = @{
Expand Down
4 changes: 2 additions & 2 deletions generate/templates/.github/workflows/ci-master-pr.yml.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ jobs:
test-nogitdiff:
runs-on: ubuntu-latest
container:
image: mcr.microsoft.com/powershell:7.2.2-alpine-3.14-20220318
image: mcr.microsoft.com/powershell:lts-7.2-alpine-3.17
steps:
- run: |
apk add --no-cache git
Expand All @@ -36,7 +36,7 @@ jobs:
'@

# Group variants by the package version
$groups = $VARIANTS | Group-Object -Property { $_['_metadata']['package_version'] } | Sort-Object -Property { [version]$_.Name } -Descending
$groups = $VARIANTS | Group-Object -Property { $_['_metadata']['job_group_key'] } | Sort-Object { [version]$_.Name.Split('-')[0] } -Descending
$WORKFLOW_JOB_NAMES = $groups | % { "build-$( $_.Name.Replace('.', '-') )" }
foreach ($g in $groups) {
@"
Expand Down

0 comments on commit f8b5476

Please sign in to comment.