Skip to content

ci: use Alpine 3.23#34442

Closed
fanquake wants to merge 2 commits intobitcoin:masterfrom
fanquake:ci_alpine_23
Closed

ci: use Alpine 3.23#34442
fanquake wants to merge 2 commits intobitcoin:masterfrom
fanquake:ci_alpine_23

Conversation

@fanquake
Copy link
Member

@fanquake fanquake commented Jan 29, 2026

Use the latest version of Alpine (3.23) in the CI job.
This should fail for now (bitcoin-core/secp256k1#1813), given the use of GCC 15.x.

@DrahtBot DrahtBot added the Tests label Jan 29, 2026
@fanquake fanquake mentioned this pull request Jan 29, 2026
Merged
@DrahtBot
Copy link
Contributor

DrahtBot commented Jan 29, 2026 •
edited
Loading

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

Type Reviewers
Concept ACK hebasto

If your review is incorrectly listed, please copy-paste <!--meta-tag:bot-skip--> into the comment that the bot should ignore.

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #31349 (ci: detect outbound internet traffic generated while running tests by vasild)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@fanquake
ci: detect apk usage generally
ef4ef03 
Rather than listing image names, which makes it less easy to
run custom CI jobs.
@DrahtBot
Copy link
Contributor

DrahtBot commented Jan 29, 2026

🚧 At least one of the CI tasks failed.
Task Alpine (musl): https://github.com/bitcoin/bitcoin/actions/runs/21475736614/job/61862623026
LLM reason (✨ experimental): Compiler error: trailing whitespace in secp256k1.h treated as error (-Werror) stops the build.

Hints

Try to run the tests locally, according to the documentation. However, a CI failure may still
happen due to a number of reasons, for example:

  • Possibly due to a silent merge conflict (the changes in this pull request being
    incompatible with the current code in the target branch). If so, make sure to rebase on the latest
    commit of the target branch.

  • A sanitizer issue, which can only be found by compiling with the sanitizer and running the
    affected test.

  • An intermittent issue.

Leave a comment here, if you need help tracking down a confusing failure.

hebasto
hebasto approved these changes Jan 29, 2026
View reviewed changes
Copy link
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Concept ACK.

@maflcko
Copy link
Member

maflcko commented Jan 29, 2026

If you want, you can also sneak in the apk change (#34421 (comment)), but up to you

@fanquake
Copy link
Member Author

fanquake commented Jan 29, 2026

Pushed up some bash

@DrahtBot DrahtBot mentioned this pull request Jan 31, 2026
Open
@maflcko maflcko mentioned this pull request Feb 2, 2026
Merged
@fanquake
Copy link
Member Author

fanquake commented Feb 2, 2026

Closing for now, as this is currently part of #34481.

@fanquake fanquake closed this Feb 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hebasto hebasto hebasto approved these changes

Assignees

No one assigned

Labels

CI failed Tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@fanquake @DrahtBot @maflcko @hebasto