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
Copy link
Copy Markdown
Member

@TravisEz13 TravisEz13 commented Apr 20, 2021

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
Copy link
Copy Markdown

ghost commented May 27, 2021

🎉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