-
Notifications
You must be signed in to change notification settings - Fork 11
Stratum v2 Template Provider common functionality #49
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
base: 2024/06/sv2_connection
Are you sure you want to change the base?
Conversation
3cf779b
to
b549fd6
Compare
6894e23
to
fc6ac1e
Compare
Updated to the latest interface changes proposed in bitcoin#30409 and #53. |
b549fd6
to
22d033c
Compare
491755c
to
16d2381
Compare
4f957ee
to
b69544c
Compare
ca73422
to
4fa25e0
Compare
Rebased for CMake |
One test still seems brittle:
|
b69544c
to
1ca68d2
Compare
1f720ff
to
1912743
Compare
1ca68d2
to
b578e0a
Compare
598f08a
to
436dc4d
Compare
Moved |
436dc4d
to
46fa8f1
Compare
b578e0a
to
489c9fb
Compare
7db898e
to
765af74
Compare
802ffdc
to
6a29c1d
Compare
Occasional failing on CentOS probably means the tests need to be made more robust. https://cirrus-ci.com/task/5262823086882816 |
185eb09
to
1a0f0f3
Compare
6a29c1d
to
b2f6576
Compare
5692ffa
to
7d5d5ad
Compare
b2f6576
to
134d8b3
Compare
Rebased after bitcoin#31785 landed, which completes the interface changes needed for the Template Provider. Dropped the use of |
7d5d5ad
to
de05710
Compare
134d8b3
to
59f376b
Compare
de05710
to
21324f0
Compare
59f376b
to
d4d57bc
Compare
CI fails because since bitcoin#29307 we have to |
b4abce4
to
6fda8e0
Compare
21324f0
to
b62f820
Compare
6fda8e0
to
5ff8c44
Compare
b62f820
to
5a6d2b3
Compare
The template provider will listen for a Job Declarator client. It can establish a connection and detect various protocol errors. Co-Authored-By: Christopher Coverdale <[email protected]> Co-Authored-By: Fi3
Co-authored-by: Christopher Coverdale <[email protected]>
Incrementally update the template.
Co-authored-by: Vasil Dimov <[email protected]>
On by default. Allow Stratum v2 miners to opt out, pending more discussion on the BIP.
5ff8c44
to
29a9c8c
Compare
Note to self for the next update, apply: diff --git a/src/sv2/template_provider.cpp b/src/sv2/template_provider.cpp
index a4331c6cb3..d83b86be69 100644
--- a/src/sv2/template_provider.cpp
+++ b/src/sv2/template_provider.cpp
@@ -66,7 +66,7 @@ Sv2TemplateProvider::Sv2TemplateProvider(interfaces::Mining& mining) : m_mining{
version_pubkey_bytes[1] = 0;
m_authority_pubkey = XOnlyPubKey(authority_key.GetPubKey());
std::copy(m_authority_pubkey.begin(), m_authority_pubkey.end(), version_pubkey_bytes.begin() + 2);
- LogInfo("Template Provider authority key: %s\n", EncodeBase58Check(version_pubkey_bytes));
+ LogPrintLevel(BCLog::SV2, BCLog::Level::Info, "Template Provider authority key: %s\n", EncodeBase58Check(version_pubkey_bytes));
LogTrace(BCLog::SV2, "Authority key: %s\n", HexStr(m_authority_pubkey)); |
Moved from bitcoin#30475.
Based on:
And the following interface changes:
As well as:
-coinbaselocktime=0
This contains all Template Provider functionality that can be used by both #68 and the IPC based sidecar alternative #48.