-
Notifications
You must be signed in to change notification settings - Fork 342
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
How do you persist the keys and thus identity of remote harvester only? #216
Comments
Hi @CharlieTemplar could you clarify more about your setup? But to provide some quick points of clarification, remote harvesters have never needed your 24 word mnemonic keys, they need the CA from your farmer. From the logs you provided, it looks like you are copying your CA over from |
This might be good to read for more information in case you haven't seen it https://docs.chia.net/farming-on-many-machines It's not specific to chia-docker but explains how it would work on a baremetal setup, which helps me conceptually with how it should work in containers. |
To clarify, I'm not talking about mnemonic keys, only the key pairs generated for the harverster to remotely communicate with the farmer. The farmer's /ca keys are indeed used to generate the ssl keys for the harvester to successfully communicate with the farmer. Yes this works as it should. |
Ah I see, the confusion stems from the keys verbiage (in this context) typically referring to the mnemonic. Anyway, with that added information and now that we're on the same page, thanks for the report. You're correct that people are constantly iterating on this project and things do/will change :) In this case, the keys environment variable used to have an option that ran the command for chia to initialize a new CA, which didn't make any sense and needed to be changed because the main purpose of the variable was to determine whether or not to generate a new mnemonic on container startup. So there are probably a couple of solutions to this. One that you can do right now, and one that somebody needs to work on and add to the script :) If you remove the |
without the |
Hey Charlie, you're absolutely correct about that, my apologies. I was looking for a quick workaround for you, and the if statement that blocked you with that message was further down in a different portion of the script. Anyway, in #218 we added a check to verify the current private harvester cert against the provided CA, and if they match, it skips the extra |
Great, just tried this and indeed seems to be working again, finally I can retain the identity of my harvesters. |
Thanks for confirming the fix! We absolutely do our best to retain functionality like this when contributing. In this instance we refactored out some technical debt and this fairly innocuous regression was missed. But we're always open to hearing that we could do better, so we will. Have a great weekend :) |
This used to work by just setting keys to "persistent" but somewhere along the way various devs have "messed about" with this and now despite my keys being persisted in a harvester unique host bound volume, the keys get delete, then later the check for persistent says of, we don't need to touch these, they are persisted, but it's already TOO LATE and the harvester has a new identity.
Please fix properly or document properly, then leave it the heck alone, please.
The text was updated successfully, but these errors were encountered: