-
Notifications
You must be signed in to change notification settings - Fork 127
Sign and publish pipeline in Buildkite #1175
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
Conversation
9ec6b1c to
aed1851
Compare
| cleanup() { | ||
| echo "Deleting temporal files..." | ||
| cd ${WORKSPACE} | ||
| rm -rf tmp.elastic-package.* |
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.
Nit. Have a common base variable for the tmpdir folder pattern so we don't forget to update this if we change the template for mktemp?
| rm -rf tmp.elastic-package.* | |
| rm -rf ${TMP_FOLDER_TEMPLATE_BASE}.* |
|
|
||
| echo "Publishing package ${packageZip}" | ||
| publishPackage "${package}" | ||
| done |
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.
Will something like this pipeline work to publish packages from integrations repositories?
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.
Yes, something like this script would be the steps to be used/followed in integrations (iterating for each package defined) or other repositories like endpoint-package or apm-server.
Co-authored-by: Jaime Soriano Pastor <jaime.soriano@elastic.co>
| SIGNING_PACKAGES_GCS_CREDENTIALS_PATH=kv/ci-shared/platform-ingest/signing_packages_gcs_artifacts_credentials | ||
| PACKAGE_UPLOADER_GCS_CREDENTIALS_PATH=kv/ci-shared/platform-ingest/package_storage_uploader |
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.
Moved to shared path and renamed variables
💚 Build Succeeded
cc @mrodm |
This PR adds the missing steps to sign and publish the given test package:
pre-commandhook