Skip to content

CI: restructure release workflows#9346

Open
daywalker90 wants to merge 2 commits into
ElementsProject:masterfrom
daywalker90:release-atomic
Open

CI: restructure release workflows#9346
daywalker90 wants to merge 2 commits into
ElementsProject:masterfrom
daywalker90:release-atomic

Conversation

@daywalker90

Copy link
Copy Markdown
Collaborator

Binaries, docker images, and python packages are now separated into
build and publish flows. First everything gets built and if every build
step succeeds the publish step can continue.
Also fan out the docker build into a matrix over the targets, to speed up the workflow.

Patch version rc tags are now also valid.

During testing i discovered that the vls docker image build fails
because it is missing protoc, so i included a fix here.

> [vls-builder  9/10] RUN cargo build --release --target x86_64-unknown-linux-gnu:
177.2
177.2 Caused by:
177.2   process didn't exit successfully: `/opt/validating-lightning-signer/target/release/build/lightning-storage-server-85c73031cf3e0f51/build-script-build` (exit status: 1)
177.2   --- stdout
177.2   cargo:rerun-if-changed=proto/lss.proto
177.2   cargo:rerun-if-changed=proto
177.2
177.2   --- stderr
177.2   Error: Custom { kind: NotFound, error: "Could not find `protoc`. If `protoc` is installed, try setting the `PROTOC` environment variable to the path of the `protoc` binary. To install it on Debian, run `apt-get install protobuf-compiler`. It is also available at https://github.com/protocolbuffers/protobuf/releases  For more information: https://docs.rs/prost-build/#sourcing-protoc" }
177.2 warning: build failed, waiting for other jobs to finish...

After 608f695 the vls docker image build fails
because it is missing protoc.

```
> [vls-builder  9/10] RUN cargo build --release --target x86_64-unknown-linux-gnu:
177.2
177.2 Caused by:
177.2   process didn't exit successfully: `/opt/validating-lightning-signer/target/release/build/lightning-storage-server-85c73031cf3e0f51/build-script-build` (exit status: 1)
177.2   --- stdout
177.2   cargo:rerun-if-changed=proto/lss.proto
177.2   cargo:rerun-if-changed=proto
177.2
177.2   --- stderr
177.2   Error: Custom { kind: NotFound, error: "Could not find `protoc`. If `protoc` is installed, try setting the `PROTOC` environment variable to the path of the `protoc` binary. To install it on Debian, run `apt-get install protobuf-compiler`. It is also available at https://github.com/protocolbuffers/protobuf/releases  For more information: https://docs.rs/prost-build/#sourcing-protoc" }
177.2 warning: build failed, waiting for other jobs to finish...
```

vls 0.14 also has an external dependency that requires protoc, so vls vendoring it would not be enough.
@daywalker90 daywalker90 changed the title Release atomic CI: restructure release workflows Jul 23, 2026
@daywalker90
daywalker90 requested a review from cdecker July 23, 2026 12:15
Binaries, docker images, and python packages are now separated into
build and publish flows. First everything gets built and if every build
step succeeds the publish step can continue.
Also fan out the docker build into a matrix to speed up the workflow.

Changelog-None
@daywalker90 daywalker90 added the Status::Ready for Review The work has been completed and is now awaiting evaluation or approval. label Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status::Ready for Review The work has been completed and is now awaiting evaluation or approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant