Skip to content

Commit

Permalink
Remove FloatingTagSuffix infrastructure and usage (#1175)
Browse files Browse the repository at this point in the history
  • Loading branch information
MichaelSimons authored Aug 13, 2024
1 parent e6b81c5 commit 67c539b
Show file tree
Hide file tree
Showing 14 changed files with 151 additions and 157 deletions.
2 changes: 1 addition & 1 deletion build.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ param(
Set-StrictMode -Version Latest
$ErrorActionPreference = 'Stop'
pushd $PSScriptRoot
$ImageBuilderCustomArgs = "$ImageBuilderCustomArgs --var UniqueId=$(Get-Date -Format yyyyMMddHHmmss) --var FloatingTagSuffix=local"
$ImageBuilderCustomArgs = "$ImageBuilderCustomArgs --var UniqueId=$(Get-Date -Format yyyyMMddHHmmss)"
try {
./eng/common/Invoke-ImageBuilder.ps1 "build --path '$DockerfilePath' $ImageBuilderCustomArgs"
}
Expand Down
8 changes: 1 addition & 7 deletions eng/pipelines/variables/common.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,14 +15,8 @@ variables:
value: ''
- name: publicGitRepoUri
value: https://github.com/dotnet/dotnet-buildtools-prereqs-docker
- ${{ if eq(variables['Build.SourceBranch'], 'refs/heads/main') }}:
- name: floatingTagSuffix
value: ""
- ${{ else }}:
- name: floatingTagSuffix
value: $(Build.SourceBranchName)
- name: manifestVariables
value: --var UniqueId=$(sourceBuildId) --var FloatingTagSuffix=$(floatingTagSuffix)
value: --var UniqueId=$(sourceBuildId)
- name: publicSourceBranch
value: main
- name: ingestKustoImageInfo
Expand Down
4 changes: 2 additions & 2 deletions src/almalinux/manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
"osVersion": "almalinux8",
"tags": {
"almalinux-8-source-build-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"almalinux-8-source-build$(FloatingTagSuffix)": {}
"almalinux-8-source-build": {}
}
},
{
Expand All @@ -20,7 +20,7 @@
"osVersion": "almalinux8",
"tags": {
"almalinux-8-helix-amd64-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"almalinux-8-helix-amd64$(FloatingTagSuffix)": {}
"almalinux-8-helix-amd64": {}
}
}
]
Expand Down
32 changes: 16 additions & 16 deletions src/alpine/manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
"osVersion": "alpine3.17",
"tags": {
"alpine-3.17-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.17$(FloatingTagSuffix)": {},
"alpine-3.17": {},
"alpine-3.17-local": {
"isLocal": true
}
Expand All @@ -28,7 +28,7 @@
"osVersion": "alpine3.17",
"tags": {
"alpine-3.17-helix-amd64-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.17-helix-amd64$(FloatingTagSuffix)": {}
"alpine-3.17-helix-amd64": {}
}
}
]
Expand All @@ -42,7 +42,7 @@
"osVersion": "alpine3.17",
"tags": {
"alpine-3.17-helix-arm32v7-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.17-helix-arm32v7$(FloatingTagSuffix)": {}
"alpine-3.17-helix-arm32v7": {}
},
"variant": "v7"
}
Expand All @@ -57,7 +57,7 @@
"osVersion": "alpine3.17",
"tags": {
"alpine-3.17-helix-arm64v8-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.17-helix-arm64v8$(FloatingTagSuffix)": {}
"alpine-3.17-helix-arm64v8": {}
},
"variant": "v8"
}
Expand All @@ -72,7 +72,7 @@
"osVersion": "alpine3.18",
"tags": {
"alpine-3.18-helix-amd64-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.18-helix-amd64$(FloatingTagSuffix)": {}
"alpine-3.18-helix-amd64": {}
}
}
]
Expand All @@ -86,7 +86,7 @@
"osVersion": "alpine3.18",
"tags": {
"alpine-3.18-helix-arm32v7-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.18-helix-arm32v7$(FloatingTagSuffix)": {}
"alpine-3.18-helix-arm32v7": {}
},
"variant": "v7"
}
Expand All @@ -101,7 +101,7 @@
"osVersion": "alpine3.18",
"tags": {
"alpine-3.18-helix-arm64v8-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.18-helix-arm64v8$(FloatingTagSuffix)": {}
"alpine-3.18-helix-arm64v8": {}
},
"variant": "v8"
}
Expand All @@ -115,7 +115,7 @@
"osVersion": "alpine3.18",
"tags": {
"alpine-3.18-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.18$(FloatingTagSuffix)": {},
"alpine-3.18": {},
"alpine-3.18-local": {
"isLocal": true
}
Expand All @@ -131,7 +131,7 @@
"osVersion": "alpine3.18",
"tags": {
"alpine-3.18-WithNode-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.18-WithNode$(FloatingTagSuffix)": {}
"alpine-3.18-WithNode": {}
}
}
]
Expand All @@ -144,7 +144,7 @@
"osVersion": "alpine3.19",
"tags": {
"alpine-3.19-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.19$(FloatingTagSuffix)": {},
"alpine-3.19": {},
"alpine-3.19-local": {
"isLocal": true
}
Expand All @@ -160,7 +160,7 @@
"osVersion": "alpine3.19",
"tags": {
"alpine-3.19-WithNode-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.19-WithNode$(FloatingTagSuffix)": {}
"alpine-3.19-WithNode": {}
}
}
]
Expand All @@ -174,7 +174,7 @@
"osVersion": "alpine3.20",
"tags": {
"alpine-3.20-helix-amd64-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.20-helix-amd64$(FloatingTagSuffix)": {}
"alpine-3.20-helix-amd64": {}
}
}
]
Expand All @@ -188,7 +188,7 @@
"osVersion": "alpine3.20",
"tags": {
"alpine-3.20-helix-arm32v7-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.20-helix-arm32v7$(FloatingTagSuffix)": {}
"alpine-3.20-helix-arm32v7": {}
},
"variant": "v7"
}
Expand All @@ -203,7 +203,7 @@
"osVersion": "alpine3.20",
"tags": {
"alpine-3.20-helix-arm64v8-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.20-helix-arm64v8$(FloatingTagSuffix)": {}
"alpine-3.20-helix-arm64v8": {}
},
"variant": "v8"
}
Expand All @@ -217,7 +217,7 @@
"osVersion": "alpine3.20",
"tags": {
"alpine-3.20-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.20$(FloatingTagSuffix)": {},
"alpine-3.20": {},
"alpine-3.20-local": {
"isLocal": true
}
Expand All @@ -233,7 +233,7 @@
"osVersion": "alpine3.20",
"tags": {
"alpine-3.20-withnode-$(System:TimeStamp)-$(System:DockerfileGitCommitSha)": {},
"alpine-3.20-withnode$(FloatingTagSuffix)": {}
"alpine-3.20-withnode": {}
}
}
]
Expand Down
Loading

0 comments on commit 67c539b

Please sign in to comment.