-
Notifications
You must be signed in to change notification settings - Fork 944
azure-pipelines-rpi: add cloudsmith upload support for rpi artifacts #3053
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -118,4 +118,25 @@ artifacts_swdownloads() { | |
| -i ${KEY_FILE} -r rpi_archives_properties.txt ${DEST_SERVER}/${BUILD_SOURCEBRANCHNAME} | ||
| } | ||
|
|
||
| #upload artifacts to Cloudsmith | ||
| artifacts_cloudsmith() { | ||
| artifacts_structure | ||
| cd ${SOURCE_DIRECTORY}/${timestamp} | ||
| mv ./32bit/rpi_modules_32bit.tar.gz ./ | ||
| mv ./64bit/rpi_modules_64bit.tar.gz ./ | ||
| tar -C ${PWD}/32bit -czvf rpi_latest_boot_32bit.tar.gz . | ||
| tar -C ${PWD}/64bit -czvf rpi_latest_boot_64bit.tar.gz . | ||
| rm -r ./32bit | ||
| rm -r ./64bit | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. nit: new line here.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Regarding the duplicated work, we want to keep the azure pipeline until we ensure that the github action ones export also the current structure that we use in the boot partition for Kuiper. This applies also for the main linux build, not just rpi one.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Alright, can you still update the PT based on comments? |
||
|
|
||
| python3 ${BUILD_SOURCESDIRECTORY}/wiki-scripts/utils/cloudsmith_utils/upload_to_cloudsmith.py \ | ||
| --repo="sdg-linux-rpi" \ | ||
| --version="linux_rpi/${BUILD_SOURCEBRANCHNAME}/${timestamp}/" \ | ||
| --local_path="${SOURCE_DIRECTORY}/${timestamp}" \ | ||
| --tags="git_sha-${GIT_SHA};timestamp_${timestamp}" \ | ||
| --token="${CLOUDSMITH_API_KEY}" \ | ||
| --log_file="upload_to_cloudsmith.log" \ | ||
| --no_rel_path | ||
| } | ||
|
|
||
| artifacts_${1} | ||
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.
maybe consider adding
set -e?