[listenbrainz-server] 14alastair merged pull request #2340 (03master…canonical-dump-connection): Add option to select the database connection to dump canonical data from https://github.com/metabrainz/listenbrainz-server…
2023-01-31 03107, 2023
alastairp
finally!
2023-01-31 03141, 2023
alastairp
lucifer: I'll make a release with these changes if that sounds good
ill keep the repos which i think could be used in future
2023-01-31 03130, 2023
jasje_
most of the stuff ill just remove
2023-01-31 03137, 2023
alastairp
ok, at the moment I'll just use this release for cron and testing the dumps, let me coordinate with lucifer and we can do a .1 release for the website
2023-01-31 03147, 2023
mayhem
ah, ok.
2023-01-31 03126, 2023
alastairp
bitmap: (or yvanzo, reosarevok?): is there a way if I can tell if sshd-musicbrainz-fullexport on aretha is currently receiving an rsync dump? `ps` shows nothing running in the container, so I suspect it's fine
2023-01-31 03148, 2023
alastairp
lucifer: quick PR in docker-server-configs related to this too
2023-01-31 03122, 2023
yvanzo
good question
2023-01-31 03155, 2023
alastairp
I need to restart it to install a new key
2023-01-31 03112, 2023
yvanzo
let me check
2023-01-31 03138, 2023
alastairp
or I guess if we don't want to restart it I could copy the key contents in manually, but that doesn't let me verify that the PR that I opened works correctly
apologies, akshaaatt. i'll be contributing to bookbrainz
2023-01-31 03126, 2023
akshaaatt
Sounds great ArjunM ! You should definitely join the #bookbrainz channel and discuss things in detail there. The best way to start is by understanding the codebase and kicking things off with a PR or two :) There’s tons of things you could explore in the project and have a look at the bug tracker for issues to fix or discuss.
2023-01-31 03127, 2023
yvanzo
alastairp: don't have any better to suggest than 'ps aux' atm
2023-01-31 03100, 2023
alastairp
thanks yvanzo, restarting the service now
2023-01-31 03135, 2023
ArjunM
will head right there, akshaaatt!
2023-01-31 03137, 2023
lucifer
mayhem: i think there's a bug in ds hoster code.
2023-01-31 03151, 2023
lucifer
i'll put a temp fix in the patch and labs api endpoint for now.
2023-01-31 03106, 2023
mayhem
k.
2023-01-31 03113, 2023
mayhem
DSH needs some love in any case.
2023-01-31 03151, 2023
akshaaatt
ArjunM: monkey should be you PoC. But go through the docs first and try to exhaust all the available information before making conclusions. That being said, genuine doubts are welcome 24*7 !
2023-01-31 03113, 2023
akshaaatt
Your*
2023-01-31 03137, 2023
ArjunM
yes, got it
2023-01-31 03158, 2023
yvanzo
lucifer: CB security alerts will need a lighthouse keeper for the next months, would you agree to replace alastairp?
2023-01-31 03112, 2023
lucifer
sure yvanzo
2023-01-31 03120, 2023
alastairp
🙏 thank you
2023-01-31 03124, 2023
yvanzo
alastairp: should I reassign the current CB security alerts too?
2023-01-31 03132, 2023
alastairp
yes please
2023-01-31 03137, 2023
yvanzo
Done, thank you both!
2023-01-31 03100, 2023
lucifer
mayhem: dataset fetcher element code in troi has some special casing for artist_mbid which is creating troubles
alastairp: if there's no rrsync process it should be fine
2023-01-31 03133, 2023
alastairp
yvanzo: yes, I think that's reasonable
2023-01-31 03139, 2023
alastairp
thanks bitmap
2023-01-31 03109, 2023
alastairp
bitmap: related, not sure if you saw earlier discussion about the proliferation of ssh keys in that container - it seems that it's because we use `command` in a pubkey in authorized_keys to run `rrsync` which limits the connecting client to a particular directory
2023-01-31 03143, 2023
alastairp
we're already up to 4 keys for ListenBrainz, not sure if you know the original motivation behind that, but it could be something to clean up and improve
2023-01-31 03151, 2023
jasje_ has quit
2023-01-31 03135, 2023
lucifer
mayhem: its the popular recordings element that is erroring! not the labs api one
2023-01-31 03153, 2023
mayhem
d'oh
2023-01-31 03117, 2023
mayhem
was that always the case? or did you fix something and the error moved on?
2023-01-31 03130, 2023
lucifer
i am not sure, its the case as of now.
2023-01-31 03144, 2023
mayhem
ok, I'll get back on it.
2023-01-31 03145, 2023
bitmap
alastairp: right, the goal for MB at least was to restrict keys to a particular directory to prevent accidental clobbering or deletion. may not be as relevant to other projects in which case one or two keys might be fine
2023-01-31 03148, 2023
mayhem
no changes to troi code?
2023-01-31 03148, 2023
yvanzo
alastairp: Done.
2023-01-31 03153, 2023
lucifer
none
2023-01-31 03102, 2023
mayhem
k
2023-01-31 03141, 2023
alastairp
bitmap: right. We have 4 types of dumps, each going into different subdirectories. Perhaps we could use a single key to limit to the LB dir, but then use the same one for each subdir
2023-01-31 03111, 2023
bitmap
that sounds reasonable to me
2023-01-31 03149, 2023
yvanzo
Jira apps update at :45, tickets might be less available for a very short time.
this is what i get currently without nay changes to the branch
2023-01-31 03147, 2023
mayhem
uh, that is extra special.
2023-01-31 03106, 2023
lucifer
cool, sounds good. i'll try to fix it in meantime and let you know how it goes.
2023-01-31 03117, 2023
mayhem
ok, thanks.
2023-01-31 03112, 2023
aerozol
Freso: thanks! I can indeed now edit other people's posts from ages ago, crazy
2023-01-31 03102, 2023
aerozol
reosarevok: are you the only forum admin for the year now? Let me know if you want a hand (and if there's any moderator docs/guides, though I assume it's just the CoC)
