Skip to content
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

Next Citadel update won't support the Umbrel migration script anymore #2642

Closed
AaronDewes opened this issue Oct 19, 2021 · 12 comments
Closed
Milestone

Comments

@AaronDewes
Copy link
Contributor

I'm currently focused on buiilding Citadel and can't implement a fix for this, but the next Citadel update will change the data layout by renaming the umbrel dir to citadel. This means the exisiting migration logic needs changes to work with Citadel.

@rootzoll
Copy link
Collaborator

Thanks for the heads up - will mark to adapt for next release. If possible can you please simply post links into your code where you make the changes?

@rootzoll rootzoll added this to the 1.7.2 Release milestone Oct 21, 2021
@AaronDewes
Copy link
Contributor Author

Basically, I just did

mv /mnt/hdd/umbrel /mnt/hdd/citadel

Everything else is the same.

@rootzoll
Copy link
Collaborator

@AaronDewes just to ask are you also using the default password moneyprintergobrrr for the lnd wallet like umbrel since 0.3.9 or did you implemented changes to that part?

rootzoll added a commit that referenced this issue Nov 30, 2021
@rootzoll
Copy link
Collaborator

Added code changes for citadel for v1.7.2 .. to be tested with fresh sd card build.

@rootzoll rootzoll added the final testing was fixed - needs testing label Nov 30, 2021
@AaronDewes
Copy link
Contributor Author

just to ask are you also using the default password moneyprintergobrrr for the lnd wallet like umbrel since 0.3.9 or did you implemented changes to that part?

I did implement some changes to the unlocking logic so lnd.conf is a bit different from Umbrel's default, but you seem to be overwriting it anyway, so it shouldn't matter.

The password stays the same though (currently). With the security changes we're discussing right now, I don't think a separate password for LND would still make sense in Citadel.

@rootzoll
Copy link
Collaborator

OK cool ... if you change something on that LND wallet password side give me a heads up so that I can adjust the migration procedure.

@AaronDewes
Copy link
Contributor Author

Sure! Anything else would be vendor lock-in in my opinion by breaking migration logic of other systems.

rootzoll pushed a commit that referenced this issue Feb 17, 2022
@rootzoll rootzoll removed the final testing was fixed - needs testing label Feb 17, 2022
@rootzoll
Copy link
Collaborator

Migrationtest from Citadel to RaspiBlitz looked good in testing ... closing issue for v1.7.2 release.

@cezars
Copy link

cezars commented Jul 1, 2023

what is the default user and password wen u connect to the ssh ?

@AaronDewes
Copy link
Contributor Author

what is the default user and password wen u connect to the ssh ?

Please ask this somewhere else, not here. GitHub issues are mostly for tracking bugs, not questions. Also, it's not possible to know from your question if you're talking about RaspiBlitz, Umbrel or Citadel. Please ask at the appropriate place for the project you want to know this.

@cezars
Copy link

cezars commented Jul 1, 2023

Citadel

sry for posting here but i try to find all over the social media, its about Citadel because i use migrate function from umbrel to citadel and its downloding all the node data again and i need to delete the umbrel node data. sry again for posting here but i really need help to delete the data and i cant connect via ssh to do so (first i made a mistake flashing nirvati-os and after this i flashed again citadel-os-v0.2.3)

@AaronDewes
Copy link
Contributor Author

Please contact me at t.me/runcitadel or DM @runcitadel on Twitter (German or English).

Citadel does not have a migration system from Umbrel, you can not keep Umbrel data when moving to Citadel right now, so your data will be erased when setting up Citadel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants