-
Notifications
You must be signed in to change notification settings - Fork 36.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
v23.0 testing #24501
Comments
Signet miner no longer runs for me on a custom signet.
When reverting PR: #22514 it works again |
Compiled v23.0rc1 with default config settings in Qubes 4.0 in a Debian-11 VM with standard networking. Passed all functional, util, and unit tests. |
#22514 config. Compiled v23.0 |
v23.0rc2 on Windows 11: Do we really want to make a release in March with a certificate which is valid until May only? See:
cc @achow101 |
@hebasto As long as the key is not revoked, there won't be any issues with the installer after it expires. Those previous issues were because the key was revoked. |
v23.0rc2 on macOS Monterey x86_64:
|
The dmg needs to be unpacked. The application bundle contained inside the dmg is signed, not the dmg itself. |
You're right:
|
Compiled v23.0rc1 FreeBSD 13.0-RELEASE Passed all tests. |
…tch) 12cc020 contrib: fix signet miner (sighash mismatch) (Sebastian Falbesoner) Pull request description: gruve-p reported that the signet miner doesn't work anymore (see bitcoin/bitcoin#24501 (comment)), failing with the following error of the `walletprocesspsbt` RPC: ``` error code: -22 error message: Specified sighash value does not match value stored in PSBT ..... subprocess.CalledProcessError: Command '['bitcoin-cli', '-signet', '-stdin', 'walletprocesspsbt']' returned non-zero exit status 22 ``` PSBT signing was changed to use SIGHASH_DEFAULT by default in #22514. The signet miner script sets the sighash type of the created PSBT to SIGHASH_ALL (3 is the per-input type PSBT_IN_SIGHASH_TYPE, following a little-endian 32 unsigned integer of the sighash type): https://github.com/bitcoin/bitcoin/blob/e04720ec3336e3df7fce522e3b1da972aa65ff62/contrib/signet/miner#L169-L170 hence this leads to a sighash mismatch when the `walletprocesspsbt` RPC is called. Fix this by explicitly passing the correct sighash type. The same change was needed in one of our functional tests, see commit d399266. Note that instead of feeding the PSBT via `-stdin` it is directly passed as parameter, as I couldn't figure out a way to pass multiple parameters otherwise (separating by newline also didn't work). ACKs for top commit: kallewoof: ACK 12cc020 ajtowns: ACK 12cc020 ; code review only Tree-SHA512: 8509e768e96f85e28c0ca0dc2d35874aa29623febddc46bf90472ec38f38cb3a1b5407c563fd9101d07088775d0fdb18e9137cc38955e847885b83c16591c736
12cc020 contrib: fix signet miner (sighash mismatch) (Sebastian Falbesoner) Pull request description: gruve-p reported that the signet miner doesn't work anymore (see bitcoin#24501 (comment)), failing with the following error of the `walletprocesspsbt` RPC: ``` error code: -22 error message: Specified sighash value does not match value stored in PSBT ..... subprocess.CalledProcessError: Command '['bitcoin-cli', '-signet', '-stdin', 'walletprocesspsbt']' returned non-zero exit status 22 ``` PSBT signing was changed to use SIGHASH_DEFAULT by default in bitcoin#22514. The signet miner script sets the sighash type of the created PSBT to SIGHASH_ALL (3 is the per-input type PSBT_IN_SIGHASH_TYPE, following a little-endian 32 unsigned integer of the sighash type): https://github.com/bitcoin/bitcoin/blob/e04720ec3336e3df7fce522e3b1da972aa65ff62/contrib/signet/miner#L169-L170 hence this leads to a sighash mismatch when the `walletprocesspsbt` RPC is called. Fix this by explicitly passing the correct sighash type. The same change was needed in one of our functional tests, see commit d399266. Note that instead of feeding the PSBT via `-stdin` it is directly passed as parameter, as I couldn't figure out a way to pass multiple parameters otherwise (separating by newline also didn't work). ACKs for top commit: kallewoof: ACK 12cc020 ajtowns: ACK 12cc020 ; code review only Tree-SHA512: 8509e768e96f85e28c0ca0dc2d35874aa29623febddc46bf90472ec38f38cb3a1b5407c563fd9101d07088775d0fdb18e9137cc38955e847885b83c16591c736
Cross-posting a bug in the GUI: bitcoin-core/gui/issues/567. |
I was able to fully resync the chain without issue |
A fix has been suggested in |
Downloaded and syncing w/ GUI on macOS 12.3 M1 with no problems so far. Great stuff! |
👍😉👍😎👍😊👍🤩👍😘👍 |
Tested the tracepoints in v23rc2 with the tests from #24358.
|
Tested https://bitcoincore.org/bin/bitcoin-core-23.0/test.rc2/bitcoin-23.0rc2-win64.zip on Windows 11 Pro 21H2.
|
compiled without errors but got countless warnings (hundreds) on C++17 Mac OS X.
all the warnings are related to either the use of macros or overriding. All the tests passed:
The make deploy also works after command |
You can use the |
Updated https://jonatack.github.io/articles/how-to-compile-bitcoin-core-and-run-the-tests to add the |
#22041 :) |
The testing guide is now available on https://github.com/bitcoin-core/bitcoin-devwiki/wiki/23.0-Release-Candidate-Testing-Guide , @laanwj could you please add that link to OP? If anyone has any feedback or suggested changes re the guide, please let me know! |
Thanks for a great guide! ❤️ For Apple M1 testers like me jumping straight at the binary, a section about how to configure for signet could be good. Otherwise, new testers might end up waiting for the IBD on mainnet. Also, if you're unfamiliar with working at the terminal, it might be intimidating at first. Given the .conf extension is not mapped to something obscure (like Tunnelblick in my case) these steps should work:
|
@hebasto |
macOs 10.15.6/compiled from source |
My test feedback is available here https://gist.github.com/eriknylund/768787a123ed981d4bf824995bdda733 if you prefer I can submit the full feedback in this thread as well. The error dialogs when opening a wallet resurfaced (see #24501 (comment) above) and I'm a bit concerned about it and what it would mean for an unexperienced user. Could you give me some insight on what this means and is it perhaps more likely to occur during the IBD? I guess most users will start off with a pruned node, with it being the default setting, and are more likely to run into this? Please see the last section https://gist.github.com/eriknylund/768787a123ed981d4bf824995bdda733#ux-considerations for some ideas on UI/UX improvements. |
Fellow testers who is able/willing to test ARM binaries! Please test #24115. On supported platforms the |
Tested on: Does not match expected output in
|
@eriknylund what does this give for output on your system:
|
On
v23.0.0rc2 says:
v23.0.0rc3 says:
|
Tested on: Matches expected output in debug.log:
|
Hm, maybe macOS 11.x reports Let's wait for what @eriknylund will report back about Edit: Yes, that's it! See dotnet/runtime#62832 (comment) - macOS 11.x does not report I think we are fine. If the OS does not report the feature, we should not be using it; plus this is an opt-in optimization. Finally, most of people with M1 hardware will be running macOS 12.x anyway (because that's what being preinstalled since October 2021). |
Fellow testers who run Bitcoin Core on macOS Monterey. Please postpone the Monterey 12.3.1 update until 23.0rc4 to test whether #23134 has been really fixed. |
@prusnak unfortunately I won't reach this computer for some time but hopefully you found your answer already. I'll make sure to verify the above when I'm able to. |
Agree, this is the best kind of problem, it only marginally affects performance and will solve itself in time simply by updating the underlying OS. Nothing to do here. |
I've managed to sign v23.0rc3 (using this patch), and did apply the Monterey 12.3.1 update both on Intel based and Apple Silicon based systems. The previously installed Bitcoin Core v23.0rc3 has survived. #23134 is indeed fixed. |
Tested on: iMac (Retina 5K, 27-inch, 2017) Findings:
|
Running latest Bitcoin from |
Compiled v23.0rc4 on Debian x86. Ran all tests. Was able to set up CJDNS, Tor and i2p connections. |
Ran the rc4 linux binary bitcoin-23.0rc4-x86_64-linux-gnu.tar.gz on mainnet for an hour and some CLI/RPC commands. Verified that the "lock" logging category is disabled in rc4. Ran the GUI at the same time on signet, checked various tabs and did various operations. |
Cross-posting a confirmed GUI bug on Windows: bitcoin-core/gui#582. |
^ same here, 23.0rc4 kept working fine after updating from 12.3 to 12.3.1 (M1 Pro). |
I'm experiencing performance issues with v23 on macos, but I'm running it on an Intel macbook pro mid 2015 with Big Sur. I described the issue on stack exchange. |
Umbrella issue for 23.0 testing. Please help testing on a wide variety of supported platforms, as well as interaction with different software.
Let us know which version you tested on which operating system.
If you find an issue, please search Github for known issues first and then open a new Github issue. This meta issue should not be used to report bugs, as a single thread makes it impossible to track more than one topic.
What to test
For an idea what to test, see @stickies-v's 23.0 release candidate testing guide, or
Known issues (as of rc5)
Other known issues:
hw.optional.arm.*
. Please upgrade to MacOS 12.x if this is important to you.Known issues fixed (as of rc5)
Due to a naming conflict after adding support for the new MacOS architecture, we had to rename the following dmgs manually frombitcoin-23.0rc2-osx-signed.dmg
during upload:This may cause issues during verification because in theSHA256SUMS.asc
they still have the old name.MacOS: please postpone the Monterey 12.3.1 update until 23.0rc4 to test whether macOS Monterey deleting bitcoin core after updates (M1) #23134 has been really fixed(it's really fixed, see v23.0 testing #24501 (comment))Cross-compiledbitcoind -signet
silently fails on Windows #24726 bitcoind/-qt may silently fail on Windows when using AVX2 SHA256 implementationWindows GUI crashes while rendering bandwidth graph: 23.0rc4 crashes when trying to open the "Network Traffic" tab bitcoin-core/gui#582The text was updated successfully, but these errors were encountered: