ci: Build all PR commits independently #93
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This is a total mess but it is just for validation at this point
This reverts commit e2ce55f.
Flake lock file updates:
• Updated input 'flake-utils':
'github:numtide/flake-utils/b1d9ab70662946ef0850d488da1c9019f3a9752a?narHash=sha256-SZ5L6eA7HJ/nmkzGG7/ISclqe6oZdOZTNoesiInkXPQ%3D' (2024-03-11)
→ 'github:numtide/flake-utils/c1dfcf08411b08f6b8615f7d8971a2bfa81d5e8a?narHash=sha256-X6rJYSESBVr3hBoH0WbKE5KvhPU5bloyZ2L4K60/fPQ%3D' (2024-09-17)
• Updated input 'nixpkgs':
'github:NixOS/nixpkgs/8b4061fd60ccc3b3f44b73faa7c983eacf7a6f7b?narHash=sha256-gT%2BrUDbw%2BTQuszQEzMUJWTW7QYtccZ5xxWmKOSrPvEw%3D' (2024-09-03)
→ 'github:NixOS/nixpkgs/294eb5975def0caa718fca92dc5a9d656ae392a9?narHash=sha256-WO6Lmbn37PlamY2fDg3B187THkSKU/W01z8SxoIqJd0%3D' (2024-09-18)
Update flake.lock
The version spec for dependencies was sloppy (failed to specify minor and patch levels) Keep in mind that in cargo, dependency of type ```toml foo = "x.y.z" ``` means that `foo` must resolve with version `x'.y'.z'` where `x' == x`, `y' >= y`, and `z' >= z` and that the default dependency resolver will maximize `y'` and `z'`. That is, cargo assumes that projects are following semver and that you want the latest package which meets your desired API. If you really needed a specific version you could say ```toml foo = "=x.y.z" ``` which constrains the resolver
Flake lock file updates:
• Updated input 'nixpkgs':
'github:NixOS/nixpkgs/294eb5975def0caa718fca92dc5a9d656ae392a9?narHash=sha256-WO6Lmbn37PlamY2fDg3B187THkSKU/W01z8SxoIqJd0%3D' (2024-09-18)
→ 'github:NixOS/nixpkgs/a1d92660c6b3b7c26fb883500a80ea9d33321be2?narHash=sha256-V5LpfdHyQkUF7RfOaDPrZDP%2Boqz88lTJrMT1%2BstXNwo%3D' (2024-09-20)
Bumps the dev-dependencies group with 1 update in the / directory: [cc](https://github.com/rust-lang/cc-rs). Updates `cc` from 1.1.18 to 1.1.21 - [Release notes](https://github.com/rust-lang/cc-rs/releases) - [Changelog](https://github.com/rust-lang/cc-rs/blob/main/CHANGELOG.md) - [Commits](rust-lang/cc-rs@cc-v1.1.18...cc-v1.1.21) --- updated-dependencies: - dependency-name: cc dependency-type: direct:production update-type: version-update:semver-patch dependency-group: dev-dependencies ... Signed-off-by: dependabot[bot] <[email protected]>
Fix incorrect version spec
Make sure we only run the docs-related workflow on docs-related changes only, both for Pull Requests and when pushing new code. Also rename the workflow to make it explicit it builds and deploys _docs_. Link: https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#example-including-paths Signed-off-by: Quentin Monnet <[email protected]>
Avoid running the workflow when pushing on other branches. We already run it for Pull Requests. Signed-off-by: Quentin Monnet <[email protected]>
Cancel an ongoing docs build when the Pull Request (or the HEAD, when building following a push) is updated. No need to run concurrent builds for a same PR. Signed-off-by: Quentin Monnet <[email protected]>
This directory has not been used in a while. It was for a prior round of quick scratch work and was checked in accidentally in the first place.
Bumps the dev-dependencies group with 1 update in the / directory: [libc](https://github.com/rust-lang/libc). Updates `libc` from 0.2.158 to 0.2.159 - [Release notes](https://github.com/rust-lang/libc/releases) - [Changelog](https://github.com/rust-lang/libc/blob/0.2.159/CHANGELOG.md) - [Commits](rust-lang/libc@0.2.158...0.2.159) --- updated-dependencies: - dependency-name: libc dependency-type: direct:production update-type: version-update:semver-patch dependency-group: dev-dependencies ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps the dev-dependencies group with 1 update in the / directory: [cc](https://github.com/rust-lang/cc-rs). Updates `cc` from 1.1.21 to 1.1.22 - [Release notes](https://github.com/rust-lang/cc-rs/releases) - [Changelog](https://github.com/rust-lang/cc-rs/blob/main/CHANGELOG.md) - [Commits](rust-lang/cc-rs@cc-v1.1.21...cc-v1.1.22) --- updated-dependencies: - dependency-name: cc dependency-type: direct:production update-type: version-update:semver-patch dependency-group: dev-dependencies ... Signed-off-by: dependabot[bot] <[email protected]>
Flake lock file updates:
• Updated input 'nixpkgs':
'github:NixOS/nixpkgs/a1d92660c6b3b7c26fb883500a80ea9d33321be2?narHash=sha256-V5LpfdHyQkUF7RfOaDPrZDP%2Boqz88lTJrMT1%2BstXNwo%3D' (2024-09-20)
→ 'github:NixOS/nixpkgs/b5b2fecd0cadd82ef107c9583018f381ae70f222?narHash=sha256-k6YxGj08voz9NvuKExojiGXAVd69M8COtqWSKr6sQS4%3D' (2024-09-28)
When listing files affecting the workflow to build/publish documentation, we forgot to list flake.lock. Let's fix. Fixes: 4c15025 ("CI: Rename docs build workflow, restrict to docs changes") Signed-off-by: Quentin Monnet <[email protected]>
Bumps the dev-dependencies group with 1 update in the / directory: [cc](https://github.com/rust-lang/cc-rs). Updates `cc` from 1.1.22 to 1.1.23 - [Release notes](https://github.com/rust-lang/cc-rs/releases) - [Changelog](https://github.com/rust-lang/cc-rs/blob/main/CHANGELOG.md) - [Commits](rust-lang/cc-rs@cc-v1.1.22...cc-v1.1.23) --- updated-dependencies: - dependency-name: cc dependency-type: direct:production update-type: version-update:semver-patch dependency-group: dev-dependencies ... Signed-off-by: dependabot[bot] <[email protected]>
Make the README more welcoming to readers who don't know the project by adding a quick intro and linking to the docs. Also add a warning to state that the project is not usable, yet. Signed-off-by: Quentin Monnet <[email protected]>
Let Clippy run in pedantic mode, and fix some of the issue it reports in
the repository.
Fixes left aside:
- Expansion of dependencies in "use" directives
- Use of casts introducing new std::ptr (as I'm not sure we want to
rely more on the standard library)
- Use of backquotes around "EtherType" (not the variable) in a comment
Signed-off-by: Quentin Monnet <[email protected]>
Signed-off-by: Quentin Monnet <[email protected]>
In TX queue's "configure()", we do not otherwise consume the "config" object that we get as an argument. We can simply reuse it to craft the return TxQueue object rather than cloning the struct - just like we do for RxQueue. Found in Clippy's reports (Clippy suggested to pass "config" by reference). Signed-off-by: Quentin Monnet <[email protected]>
Make everything owned by a single team. The idea is to be able to have auto-assignment for reviews on Pull Requests. We can use finer-grained ownership later if necessary. Signed-off-by: Quentin Monnet <[email protected]>
Make workflow description files more consistent:
- Add a comment to describe the workflow at the top of the file, for
workflows that didn't have one yet
- Use consistent options for tmate options on "workflow_dispatch"
- Use a consistent order for trigger events ("on:")
- Use a consistent order for workflow permissions
- Use a blank line between top-level sections
- Remove trailing blank lines at the end of files
The objective is to make it easier to compare the different workflow
files visually.
No functional change.
Signed-off-by: Quentin Monnet <[email protected]>
Rather than running the dev.yml and sterile.yml workflows everytime developers push to any branch, run them for "pull_target" events. This makes it easier to integrate with Pull Requests (for example for setting required tests) and avoids running unnecessary workflows when branches are pushed with no immediate intention to open a PR. Keep running on "push", but for "main" branch only, to run the workflows on the new "main" when PRs get merged. Signed-off-by: Quentin Monnet <[email protected]>
In a previous commit I tried to clean up the README file, but I broke the link to rustup's website. Let's fix it. Fixes: aa8a163 ("README.md: Clean-up minor issues, add link to Fabric's docs") Signed-off-by: Quentin Monnet <[email protected]>
qmonnet
force-pushed
the
pr/qmonnet/ci-build-all-commits
branch
5 times, most recently
from
5e336b6 to
d4247af
Compare
In dev.yml workflow, instead of building only the commit at the top of a Pull Request branch, run a Cargo build for each commit in the Pull Request. The objective is to make sure that each commit can build in a standalone fashion, and to limit the risk of breaking bisects. Notes: - We do not check whether commit actually changes source code files. - We do not build all commits for each supported configuration. - We do not run tests for all commits. Signed-off-by: Quentin Monnet <[email protected]>
qmonnet
force-pushed
the
pr/qmonnet/ci-build-all-commits
branch
2 times, most recently
from
aa71a1b to
995fc0f
Compare
|
[Hold off until #92 is complete] |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In dev.yml workflow, instead of building only the commit at the top of a Pull Request branch, run a Cargo build for each commit in the Pull Request. The objective is to make sure that each commit can build in a standalone fashion, and to limit the risk of breaking bisects.
Notes: