Skip to content
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

Package pipeline #1629

Merged
merged 33 commits into from
Oct 26, 2020
Merged

Conversation

bricerisingalgorand
Copy link
Contributor

Summary

Implement mule packaging pipeline

Test Plan

This pipeline builds algorand, creates deb/rpm packages and uploads them to algorand-internal

@bricerisingalgorand bricerisingalgorand marked this pull request as ready for review October 20, 2020 18:30
We don't really need to hold onto those between stages
egieseke
egieseke previously approved these changes Oct 21, 2020
Copy link
Contributor

@egieseke egieseke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I tested this pipeline using the 2.2.0 release branch.

@@ -44,19 +49,21 @@ tasks:
jobs:
package:
tasks:
- docker.Make.build
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why does packaging rebuild?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the plan right now is to use the build artifacts produced by this pipeline rather than what travis uploads. That way we're only using travis to ensure that tests are passing.

egieseke
egieseke previously approved these changes Oct 26, 2020
Copy link
Contributor

@egieseke egieseke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Copy link
Contributor

@onetechnical onetechnical left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had some questions

Makefile Outdated
PATH=$(SRCPATH)/tmp/node_pkgs/$(OS_TYPE)/$(ARCH)/bin:$${PATH} \
scripts/upload_version.sh $(CHANNEL) $(SRCPATH)/tmp/node_pkgs $(S3_RELEASE_BUCKET)
FULLBUILDNUMBER=$(FULLBUILDNUMBER) \
aws s3 cp tmp/node_pkgs s3://algorand-internal/channel/${CHANNEL}/${FULLBUILDNUMBER} --recursive --exclude "*" --include "*${CHANNEL}*${FULLBUILDNUMBER}*"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting, just to be clear here, this is uploading to algorand-internal, not algorand-staging, and will do so to a subdirectory based on the buildnumber? That should be fine, because it won't overlap with Travis's build artifacts if that's the case (uploads to the root of that dir).

scripts/release/mule/package/rpm/package.sh Outdated Show resolved Hide resolved
test/muleCI/Jenkinsfile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
@algojohnlee algojohnlee merged commit 397583f into algorand:master Oct 26, 2020
tsachiherman pushed a commit to tsachiherman/go-algorand that referenced this pull request Jul 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants