Skip to content

Make sure daily release tag does not change when retrying failures#15286

Merged
TravisEz13 merged 6 commits into
PowerShell:masterfrom
TravisEz13:fix-dailytest-releasetag
Apr 22, 2021
Merged

Make sure daily release tag does not change when retrying failures#15286
TravisEz13 merged 6 commits into
PowerShell:masterfrom
TravisEz13:fix-dailytest-releasetag

Conversation

@TravisEz13

@TravisEz13 TravisEz13 commented Apr 20, 2021

Copy link
Copy Markdown
Member

PR Summary

Make sure daily release tag does not change when retrying failures

PR Context

If the seed that is used for a counter changes for a rerun, the variables are recalculated on rerun.

This PR persists the release tag as an artifact, preventing any change.

PR Checklist

@ghost ghost assigned iSazonov Apr 20, 2021

- powershell: |
$createJson = ("${{ parameters.ReleaseTagVarName }}" -ne "no")
$createJson = ("${{ parameters.CreateJson }}" -ne "no")

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was a pre-existing issue I noticed when verifying the fix.

@TravisEz13 TravisEz13 added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Apr 20, 2021
@TravisEz13 TravisEz13 marked this pull request as ready for review April 20, 2021 23:05
Comment thread tools/releaseBuild/setReleaseTag.ps1 Outdated
Co-authored-by: Aditya Patwardhan <adityap@microsoft.com>
@TravisEz13 TravisEz13 assigned TravisEz13 and unassigned iSazonov Apr 21, 2021
@TravisEz13 TravisEz13 merged commit aae705a into PowerShell:master Apr 22, 2021
@TravisEz13 TravisEz13 deleted the fix-dailytest-releasetag branch April 22, 2021 23:30
rkeithhill pushed a commit to rkeithhill/PowerShell that referenced this pull request May 3, 2021
@ghost

ghost commented May 27, 2021

Copy link
Copy Markdown

🎉v7.2.0-preview.6 has been released which incorporates this pull request.:tada:

Handy links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants