Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

CI: build a test artifact #527

Merged
merged 14 commits into from
Jul 23, 2021
Merged

CI: build a test artifact #527

merged 14 commits into from
Jul 23, 2021

Conversation

TriplEight
Copy link
Contributor

@TriplEight TriplEight commented Jul 6, 2021

build --release --bin polkadot-collator takes 5 minutes after a test, test --release --all gets built in ~8 minutes without the cache.
So in the case of building after the test it saves a parallel/concurrent task, but certainly it doesn't make sense to run it every time on PRs, not even on every master pipeline.

I can run the build as it is, in the same job, but only when the pipeline's executed by schedule/trigger, @lovelaced WDYT?

Closes: https://github.com/paritytech/ci_cd/issues/141

@TriplEight TriplEight added A3-inprogress B0-silent Changes should not be mentioned in any release notes labels Jul 6, 2021
@TriplEight TriplEight self-assigned this Jul 6, 2021
@TriplEight TriplEight changed the title WIP: CI: build a test artifact CI: build a test artifact Jul 12, 2021
scripts/ci/pre_cache.sh Show resolved Hide resolved
scripts/ci/pre_cache.sh Show resolved Hide resolved
@TriplEight TriplEight requested a review from apopiak July 23, 2021 10:34
@TriplEight
Copy link
Contributor Author

TriplEight commented Jul 23, 2021

@bkchr @apopiak @lovelaced done, it's published automatically on tags to s3://releases.parity.io/cumulus/x86_64-debian:stretch/ [tag name]/polkadot-collator AND /latest/polkadot-collator by default.
Also can set up a schedule if needed. It's just https://gitlab.parity.io/parity/cumulus/-/pipeline_schedules , choose cron, branch, add a non-empty ARE_WE_RELEASING_YET variable, after that binary will be found in s3/.../branch/polkadot-collator and /latest/polkadot-collator.

@TriplEight
Copy link
Contributor Author

Please raise a new issue with details on publishing the containerized version.

@apopiak
Copy link
Contributor

apopiak commented Jul 23, 2021

@bkchr @apopiak @lovelaced done, it's published automatically on tags to s3://releases.parity.io/cumulus/x86_64-debian:stretch/ [tag name]/polkadot-collator AND /latest/polkadot-collator by default.
Also can set up a schedule if needed. It's just https://gitlab.parity.io/parity/cumulus/-/pipeline_schedules , choose cron, branch, add a non-empty ARE_WE_RELEASING_YET variable, after that binary will be found in s3/.../branch/polkadot-collator and /latest/polkadot-collator.

DQ: Does that mean it's not published to github?

@TriplEight
Copy link
Contributor Author

@apopiak we didn't talk about publishing to GitHub yet, only testing release.

@TriplEight TriplEight merged commit 3883f73 into master Jul 23, 2021
@TriplEight TriplEight deleted the 3x8_build_bin branch July 23, 2021 15:06
slumber pushed a commit that referenced this pull request Sep 1, 2021
* CI: build a test artifact

* CI: try building before testing

* CI: try proper bin name

* CI: preempty cache to benchmark properly

* CI: run build after test

* CI: change target source

* CI: try build before test once again

* CI: chore

* CI: prepopulate target cache

* CI: conditional build; aws sync

* CI: mock building to test publishing

* CI: lacks permissions in the other job

* CI: undebug
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants