alastairp: Hi! I was trying to prepare the CB for consul upgrade. I think cron jobs for CB are run in the same container as the website. Is that right?
alastairp
hi _lucifer. that might be the case, I'm not sure
however it makes sense that we split it out into a separate container like LB and AB
ruaok
pristine___: spark-request-consumer updated in case it was necessary. :)
TFW you mark a sentry issue as fixed and it instantly marks it as a regression.
Guys, just found this - https://observablehq.com/@d3/versor-dragging, would it be better than the present data visualization presentation? The library too looks good, we can make use of other animations too.
_lucifer
adhi001, you'll need to provide more context
adhi001
I thought this can be used in listenbrainz artist origins data representation
_lucifer
ishaanshah and iliekcomputers worked on that. they'll be able to answer that.
alastairp: i have made the changes for consul upgrade of CB. does AB also need something similiar?
i saw that most of the changes for CB and LB are already present in AB.
iliekcomputers
adhi001: i think the map is more maintainable in the long term for us + personally I prefer the map where I can click whatever country I want without needing to drag as I'd have to in the globe.
alastairp
_lucifer: we have the split into separate services, but not the new base image version
feel free to make that change too!
other projects that need this - sir, artwork-redirect, and mb2wikidatabot
Mr_Monkey: js tests fail because eslint has errors and returns an error status code. If we want the test execution to pass but then report it as a degraded status in jenkins we should add `|| true` to the end of the test command
Mr_Monkey
OK, thanks
I'm actually looking into why the lint command fails, but I was thinking about what you propose
you should also add this to the docker cp commands, because if the test command fails and doesn't create the file, the cp will also fail causing the test to completely fail and meaning that there are no stats for that job run
ruaok: Mr_Monkey: ishaanshah: shivam-kapila: iliekcomputers: I'm merging https://github.com/metabrainz/listenbrainz-serv..., which combines the postgres and timescale databases in development which will mean that if you have local user data or messybrainz data it will disappear the next time you update. If this is a problem let me know and we can work out how to migrate data
any timescale/listen data that you have stored will stay there
Mr_Monkey: interestingly, jenkins says "JSLint Warnings: 55 warnings from one analysis", but eslint says "✖ 55 problems (10 errors, 45 warnings)"
so it's possible that jenkins isn't correctly separating them by category (or the jslint output format doesn't support it, or the jenkins output parser doesn't support it)