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

cf64cd9c76 add drone ci config with clippy for amd64

a76ec08da6 move drone config to repo root and add triggers

9ad580b86f install clippy in pipeline

27e9a8295c update probes and lockfile to fix time build error

defb8f5f09 try to fix probes build issue again

d31825f688 add tests to ci and fail on clippy warnings

271aa14322 add build status badge to root readme

da8d8f0ec3 add rustfmt and cargo check to ci pipeline

b158fba147 separate ci pipeline into discreet steps

c792aea2f6 add build step and arm64 pipeline

abde4ce1b4 only specify triggers once

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

4662b15ba3 remove unnecessary to_owned

e4078bd1ba specify triggers twice

126609a605 try arm64 ci only

00d33c2c69 remove arm64 pipeline [CI SKIP]

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

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

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

bug

documentation

duplicate

enhancement

help wanted

invalid

maintenance

peach-lib

peach-network

peach-oled

peach-stats

peach-web

question

refactor

wontfix

Milestone

Clear milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: PeachCloud/peach-workspace#101

No description provided.