Skip to content

Commit c780145

Browse files
authored
Switched package release tag to use the groupId plus the packageName (#47512)
* Fetched package properties by groupId and pkgName * Updated release tag for patch release and bom release * Falled back to fetch the old tag format
1 parent f30c0c9 commit c780145

File tree

4 files changed

+34
-15
lines changed

4 files changed

+34
-15
lines changed

eng/scripts/Generate-Patch.ps1

Lines changed: 16 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -82,8 +82,8 @@ function GetRemoteName($MainRemoteUrl) {
8282
return $null
8383
}
8484

85-
function ResetSourcesToReleaseTag($ArtifactName, $ServiceDirectoryName, $ReleaseVersion, $RepoRoot, $RemoteName) {
86-
$ReleaseTag = "${ArtifactName}_${ReleaseVersion}"
85+
function ResetSourcesToReleaseTag($ArtifactName, $ServiceDirectoryName, $ReleaseVersion, $RepoRoot, $RemoteName, $GroupId = "com.azure") {
86+
$ReleaseTag = "${GroupId}+${ArtifactName}_${ReleaseVersion}"
8787
Write-Information "Resetting the $ArtifactName sources to the release $ReleaseTag."
8888

8989
$SdkDirPath = Join-Path $RepoRoot "sdk"
@@ -92,7 +92,7 @@ function ResetSourcesToReleaseTag($ArtifactName, $ServiceDirectoryName, $Release
9292
$ArtifactDirPath = Join-Path $ServiceDirPath $ArtifactName
9393
TestPathThrow -Path $ArtifactDirPath -PathName 'ArtifactDirPath'
9494

95-
$pkgProperties = Get-PkgProperties -PackageName $ArtifactName -ServiceDirectory $ServiceDirectoryName
95+
$pkgProperties = Get-PkgProperties -PackageName $ArtifactName -ServiceDirectory $ServiceDirectoryName -GroupId $GroupId
9696
$currentPackageVersion = $pkgProperties.Version
9797
if($currentPackageVersion -eq $ReleaseVersion) {
9898
Write-Information "We do not have to reset the sources."
@@ -108,8 +108,17 @@ function ResetSourcesToReleaseTag($ArtifactName, $ServiceDirectoryName, $Release
108108
Write-Information "Fetching all the tags from $RemoteName"
109109
$CmdOutput = git fetch $RemoteName $ReleaseTag
110110
if($LASTEXITCODE -ne 0) {
111-
LogError "Could not restore the tags for release tag $ReleaseTag"
112-
exit 1
111+
# Fall back to old tag format: <artifactName>_<version>
112+
$OldReleaseTag = "${ArtifactName}_${ReleaseVersion}"
113+
Write-Information "Failed to fetch new tag format. Trying old tag format: $OldReleaseTag"
114+
$CmdOutput = git fetch $RemoteName $OldReleaseTag
115+
116+
if($LASTEXITCODE -ne 0) {
117+
LogError "Could not restore the tags for release tag $ReleaseTag or $OldReleaseTag"
118+
exit 1
119+
}
120+
121+
$ReleaseTag = $OldReleaseTag
113122
}
114123

115124
$cmdOutput = git restore --source $ReleaseTag -W -S $ArtifactDirPath
@@ -162,7 +171,7 @@ function CreatePatchRelease($ArtifactName, $ServiceDirectoryName, $PatchVersion,
162171
$EngVersioningDir = Join-Path $EngDir "versioning"
163172
$SetVersionFilePath = Join-Path $EngVersioningDir "set_versions.py"
164173
$UpdateVersionFilePath = Join-Path $EngVersioningDir "update_versions.py"
165-
$pkgProperties = Get-PkgProperties -PackageName $ArtifactName -ServiceDirectory $ServiceDirectoryName
174+
$pkgProperties = Get-PkgProperties -PackageName $ArtifactName -ServiceDirectory $ServiceDirectoryName -GroupId $GroupId
166175
$ChangelogPath = $pkgProperties.ChangeLogPath
167176
$PomFilePath = Join-Path $pkgProperties.DirectoryPath "pom.xml"
168177

@@ -264,7 +273,7 @@ try {
264273

265274
## Hard resetting it to the contents of the release tag.
266275
## Fetching all the tags from the remote branch
267-
ResetSourcesToReleaseTag -ArtifactName $ArtifactName -ServiceDirectoryName $ServiceDirectoryName -ReleaseVersion $ReleaseVersion -RepoRoot $RepoRoot -RemoteName $RemoteName
276+
ResetSourcesToReleaseTag -ArtifactName $ArtifactName -ServiceDirectoryName $ServiceDirectoryName -ReleaseVersion $ReleaseVersion -RepoRoot $RepoRoot -RemoteName $RemoteName -GroupId $GroupId
268277
CreatePatchRelease -ArtifactName $ArtifactName -ServiceDirectoryName $ServiceDirectoryName -PatchVersion $PatchVersion -RepoRoot $RepoRoot
269278
$cmdOutput = git add $RepoRoot
270279
if($LASTEXITCODE -ne 0) {

eng/scripts/Language-Settings.ps1

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -350,7 +350,7 @@ function Get-java-PackageInfoFromPackageFile ($pkg, $workingDirectory)
350350
PackageId = $pkgId
351351
GroupId = $groupId
352352
PackageVersion = $pkgVersion
353-
ReleaseTag = "$($pkgId)_$($pkgVersion)"
353+
ReleaseTag = "$($groupId)+$($pkgId)_$($pkgVersion)"
354354
Deployable = $forceCreate -or !(IsMavenPackageVersionPublished -pkgId $pkgId -pkgVersion $pkgVersion -groupId $groupId.Replace(".", "/"))
355355
ReleaseNotes = $releaseNotes
356356
ReadmeContent = $readmeContent
@@ -551,7 +551,7 @@ function SetPackageVersion ($PackageName, $Version, $ServiceDirectory, $ReleaseD
551551
# -ll option says "only update README and CHANGELOG entries for libraries that are on the list"
552552
python "$EngDir/versioning/update_versions.py" --library-list $fullLibraryName
553553
& "$EngCommonScriptsDir/Update-ChangeLog.ps1" -Version $Version -ServiceDirectory $ServiceDirectory -PackageName $PackageName `
554-
-Unreleased $False -ReplaceLatestEntryTitle $ReplaceLatestEntryTitle -ReleaseDate $ReleaseDate
554+
-Unreleased $False -ReplaceLatestEntryTitle $ReplaceLatestEntryTitle -ReleaseDate $ReleaseDate -GroupId $GroupId
555555
}
556556

557557
function GetExistingPackageVersions ($PackageName, $GroupId=$null)

eng/scripts/bomhelpers.ps1

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -303,9 +303,9 @@ function GeneratePatch($PatchInfo, [string]$BranchName, [string]$RemoteName, [st
303303
Write-Output "PatchVersion is: $patchVersion"
304304
}
305305

306-
$releaseTag = "$($artifactId)_$($releaseVersion)"
306+
$releaseTag = "$($GroupId)+$($artifactId)_$($releaseVersion)"
307307
if (!$currentPomFileVersion -or !$artifactDirPath -or !$changelogPath) {
308-
$pkgProperties = [PackageProps](Get-PkgProperties -PackageName $artifactId -ServiceDirectory $serviceDirectoryName)
308+
$pkgProperties = [PackageProps](Get-PkgProperties -PackageName $artifactId -ServiceDirectory $serviceDirectoryName -GroupId $GroupId)
309309
$artifactDirPath = $pkgProperties.DirectoryPath
310310
$currentPomFileVersion = $pkgProperties.Version
311311
$changelogPath = $pkgProperties.ChangeLogPath
@@ -323,8 +323,18 @@ function GeneratePatch($PatchInfo, [string]$BranchName, [string]$RemoteName, [st
323323
$cmdOutput = git fetch $RemoteName $releaseTag
324324

325325
if ($LASTEXITCODE -ne 0) {
326-
LogError "Could not restore the tags for release tag $releaseTag"
327-
exit $LASTEXITCODE
326+
# Fall back to old tag format: <artifactId>_<version>
327+
$oldReleaseTag = "$($artifactId)_$($releaseVersion)"
328+
Write-Output "Failed to fetch new tag format. Trying old tag format: $oldReleaseTag"
329+
Write-Host "git fetch $RemoteName $oldReleaseTag"
330+
$cmdOutput = git fetch $RemoteName $oldReleaseTag
331+
332+
if ($LASTEXITCODE -ne 0) {
333+
LogError "Could not restore the tags for release tag $releaseTag or $oldReleaseTag"
334+
exit $LASTEXITCODE
335+
}
336+
337+
$releaseTag = $oldReleaseTag
328338
}
329339

330340
Write-Host "git restore --source $releaseTag -W -S $artifactDirPath"

eng/scripts/patchhelpers.ps1

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ function UpdateCIInformation($ArtifactInfos) {
8989
$serviceDirectory = $arInfo.ServiceDirectoryName
9090

9191
if (!$serviceDirectory) {
92-
$pkgProperties = [PackageProps](Get-PkgProperties -PackageName $artifactId -ServiceDirectory $serviceDirectory)
92+
$pkgProperties = [PackageProps](Get-PkgProperties -PackageName $artifactId -ServiceDirectory $serviceDirectory -GroupId $arInfo.GroupId)
9393
$arInfo.ServiceDirectoryName = $pkgProperties.ServiceDirectory
9494
$arInfo.ArtifactDirPath = $pkgProperties.DirectoryPath
9595
$arInfo.CurrentPomFileVersion = $pkgProperties.Version
@@ -188,7 +188,7 @@ function UpdateDependenciesInVersionClient([hashtable]$ArtifactInfos) {
188188

189189
# Get the release version for the next bom artifact.
190190
function GetNextBomVersion() {
191-
$pkgProperties = [PackageProps](Get-PkgProperties -PackageName "azure-sdk-bom")
191+
$pkgProperties = [PackageProps](Get-PkgProperties -PackageName "azure-sdk-bom" -GroupId "com.azure")
192192
$currentVersion = $pkgProperties.Version
193193

194194
$patchVersion = GetPatchVersion -ReleaseVersion $currentVersion

0 commit comments

Comments
 (0)