Drone CI build pipeline #101

Merged

glyph merged 16 commits from drone_ci_test into main 2022-04-15 09:05:10 +00:00

Conversation 2 Commits 16 Files Changed 4 +270 -217

glyph commented 2022-04-06 11:44:28 +00:00

Owner

Copy Link

Here's a PR to address issue #98.

Drone has been quite easy to setup and navigate (https://build.coopcloud.tech/).

So far I'm using the Rust Docker image for Debian Buster and running the following steps:

• rustfmt
• clippy
• cargo check
• cargo test
• cargo build

The clippy step is run with the -D flag which means the run will fail if a clippy warning is encountered.

The steps are run for amd64 and arm64 architectures. The cargo check is currently failing due to a lint warning in peach-lib (I'll fix that once this PR is merged and then fine-tune the CI config if necessary).

Once thing I noticed after adding the arm64 part to the config is that the amd64 pipeline keeps running after it fails the cargo check step; it doesn't move onto the next steps but it doesn't stop either...it keeps ticking along and needs to be manually cancelled. I'd like to figure out how to prevent that happening.

Here's a PR to address [issue #98](https://git.coopcloud.tech/PeachCloud/peach-workspace/issues/98). Drone has been quite easy to setup and navigate (https://build.coopcloud.tech/). So far I'm using the Rust Docker image for Debian Buster and running the following steps: - `rustfmt` - `clippy` - `cargo check` - `cargo test` - `cargo build` The `clippy` step is run with the `-D` flag which means the run will fail if a `clippy` warning is encountered. The steps are run for `amd64` and `arm64` architectures. The `cargo check` is currently failing due to a lint warning in `peach-lib` (I'll fix that once this PR is merged and then fine-tune the CI config if necessary). Once thing I noticed after adding the `arm64` part to the config is that the `amd64` pipeline keeps running after it fails the `cargo check` step; it doesn't move onto the next steps but it doesn't stop either...it keeps ticking along and needs to be manually cancelled. I'd like to figure out how to prevent that happening.

glyph added 11 commits 2022-04-06 11:44:29 +00:00

add drone ci config with clippy for amd64 cf64cd9c76

move drone config to repo root and add triggers a76ec08da6

install clippy in pipeline 9ad580b86f

update probes and lockfile to fix time build error 27e9a8295c

try to fix probes build issue again defb8f5f09

add tests to ci and fail on clippy warnings d31825f688

add build status badge to root readme 271aa14322

add rustfmt and cargo check to ci pipeline da8d8f0ec3

separate ci pipeline into discreet steps b158fba147

add build step and arm64 pipeline c792aea2f6

only specify triggers once abde4ce1b4

glyph requested review from notplants 2022-04-06 11:44:41 +00:00

glyph added the

maintenance

label 2022-04-06 11:44:50 +00:00

notplants commented 2022-04-14 14:24:26 +00:00

Owner

Copy Link

wow so cool that a 77 line yml file can do all that

wow so cool that a 77 line yml file can do all that

glyph commented 2022-04-14 17:50:55 +00:00

Author

Owner

Copy Link

Yeah it's pretty powerful! Maybe tomorrow I'll spend some time trying to understand why it hangs after the amd64 run (without moving onto the arm64 section).

Yeah it's pretty powerful! Maybe tomorrow I'll spend some time trying to understand why it hangs after the amd64 run (without moving onto the arm64 section).

glyph added 4 commits 2022-04-15 08:25:33 +00:00

remove unnecessary to_owned 4662b15ba3

specify triggers twice e4078bd1ba

try arm64 ci only 126609a605

remove arm64 pipeline [CI SKIP] 00d33c2c69

glyph added 1 commit 2022-04-15 08:47:04 +00:00

only run ci on pull request and add docs to readme [CI SKIP] 5a95ade8b9

glyph merged commit 92c7d7daa9 into main 2022-04-15 09:05:10 +00:00

glyph deleted branch drone_ci_test 2022-04-15 09:05:11 +00:00

glyph referenced this issue from a commit 2022-04-15 09:05:12 +00:00

Merge pull request 'Drone CI build pipeline' (#101) from drone_ci_test into main

glyph referenced this pull request 2022-04-15 09:09:40 +00:00

Setup CI Flows #98

Sign in to join this conversation.

Reviewers

No Reviewers

notplants

Labels

Clear labels

bug

Something is not working

documentation

duplicate

This issue or pull request already exists

enhancement

New feature

help wanted

Need some help

invalid

Something is wrong

maintenance

Pruning and weeding

peach-lib

peach-network

peach-oled

peach-stats

peach-web

question

More information is needed

refactor

Code rewrite

wontfix

This won't be fixed

No Label

maintenance

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

cblgh glyph notplants

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: PeachCloud/peach-workspace#101

No description provided.