Skip to content

Build artifacts upload#38086

Merged
Felixoid merged 2 commits intomasterfrom
build-artifacts-upload
Jun 20, 2022
Merged

Build artifacts upload#38086
Felixoid merged 2 commits intomasterfrom
build-artifacts-upload

Conversation

@Felixoid
Copy link
Copy Markdown
Member

@Felixoid Felixoid commented Jun 15, 2022

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Push static builds with lto=thin.
Improve rerun logic for deb and binary builds:

  • Upload the build log at the latest point
  • If log is not found in build urls, then we rerun the task

It fixes #37549

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Jun 15, 2022
@Felixoid
Copy link
Copy Markdown
Member Author

FYI @yakov-olkhovskiy and @devcrafter

@Felixoid
Copy link
Copy Markdown
Member Author

Felixoid commented Jun 15, 2022

It works as expected:

@Felixoid Felixoid self-assigned this Jun 20, 2022
@Felixoid
Copy link
Copy Markdown
Member Author

Merging as it will solve pretty annoying building issue

@Felixoid Felixoid merged commit e154705 into master Jun 20, 2022
@Felixoid Felixoid deleted the build-artifacts-upload branch June 20, 2022 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build in CI can be interrupted but results uploaded to S3

2 participants