-
Notifications
You must be signed in to change notification settings - Fork 453
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor CI artifacts for release improvements #10704
Conversation
2c2640c
to
9adf37f
Compare
zipping Symbols don't need JITTrace insertion. Do they? why we don't zip it anymore? |
The way zipping worked it is easier to not zip all instead of only some. We are going to have the release pipeline perform zipping. But I am separately looking into uploading symbols directly to the public symbol server, which will remove the need for this artifact entirely. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm; just need to fix the default bitness in the script
Co-authored-by: Lilian Kasem <likasem@microsoft.com>
* Consolidate artifacts * Disable sdl spotBugs * Remove PrivateSiteExtension * stage release_notes.md as artifact * Add script to generate private site extension * Copy release_notes.md in site extension * Flatten SiteExtension artifacts, custom hash task * Generate release.json during build * modifyOutputPath: false * Use SHA256 for the hash * Default private siteext to 64bit * Default private siteext to zipping * Remove binlog artifact to save space * Update SiteExtension readme * Update src/WebJobs.Script.SiteExtension/New-PrivateSiteExtension.ps1 Co-authored-by: Lilian Kasem <likasem@microsoft.com> --------- Co-authored-by: Lilian Kasem <likasem@microsoft.com>
* Refactor CI artifacts for release improvements (#10704) * Consolidate artifacts * Disable sdl spotBugs * Remove PrivateSiteExtension * stage release_notes.md as artifact * Add script to generate private site extension * Copy release_notes.md in site extension * Flatten SiteExtension artifacts, custom hash task * Generate release.json during build * modifyOutputPath: false * Use SHA256 for the hash * Default private siteext to 64bit * Default private siteext to zipping * Remove binlog artifact to save space * Update SiteExtension readme * Update src/WebJobs.Script.SiteExtension/New-PrivateSiteExtension.ps1 Co-authored-by: Lilian Kasem <likasem@microsoft.com> --------- Co-authored-by: Lilian Kasem <likasem@microsoft.com> * Fix MinorVersionPrefix * Fix nuget_drop_path * Fix modifyOutputPath: false * Publish site-ext to sub-folders per tfm --------- Co-authored-by: Lilian Kasem <likasem@microsoft.com>
Issue describing the changes in this PR
resolves #10762
Pull request checklist
IMPORTANT: Currently, changes must be backported to the
in-proc
branch to be included in Core Tools and non-Flex deployments.in-proc
branch is not requiredrelease_notes.md
Additional information
This PR refactors our build artifacts to support the upcoming release improvements. The changes are:
drop_windows
artifact.release.json
as a build artifact indrop_windows
. This is used by the release pipeline for creating the GitHub release.New-ReleaseJson.ps1
scriptdrop_linux
for consistency with windows drop.GetFileHas_Custom
msbuild task to work around an issue with file path length in the existing msbuild task.