yvanzo[m]: That we don't need to worry about 9.8 at all at the moment and can keep working to release what we have now
2025-01-21 02117, 2025
yvanzo[m]
we should just be cautious about using API v2 because there can be breaking changes in Solr 9.x releases.
2025-01-21 02152, 2025
bitmap[m]
<yvanzo[m]> "Did anyone test the mb-solr PR?s" <- I've built it now and that was successful at least (including creating the collections at the start) -- trying it with some data
I forgot that I can no longer run sir on my laptop since upgrading to PG v17 locally, as the version of psycopg2 used in the 2to3 branch doesn't support it (and I had issues trying to upgrade it).
2025-01-21 02114, 2025
yvanzo[m]
Using Docker allows to work around dependency issues on your host system.
2025-01-21 02109, 2025
BobSwift[m]
<reosarevok[m]> "wileyfoxyx, Bob Swift: does..." <- It's supposed to happen on a merge / push, but something must have broken when I transferred the repo to MetaBrainz. For now, I've been triggering a rebuild manually on RTD.
2025-01-21 02151, 2025
yvanzo[m]
bitmap, reosarevok : The Table Row Counts dashboard doesn’t seem to work for me: No data. Even though I created the database more than 5 minutes ago.
2025-01-21 02113, 2025
reosarevok[m]
Hmm. Does it show a specific error?
2025-01-21 02134, 2025
yvanzo[m]
No data.
2025-01-21 02135, 2025
bitmap[m]
yvanzo[m]: The sir repo does contain a Dockerfile, but it appears to be intended for production use and requires consul. I could probably override the entrypoint, but the Selenium test scripts can't work use it currently either
2025-01-21 02150, 2025
bitmap[m]
s/work//
2025-01-21 02108, 2025
yvanzo[m]
bitmap: I mean musicbrainz-docker
2025-01-21 02150, 2025
reosarevok[m]
yvanzo[m]: Can you see the logs for sql-exporter?
2025-01-21 02124, 2025
yvanzo[m]
How?
2025-01-21 02114, 2025
reosarevok[m]
It's a separate container, isn't it? Just checking the container logs should work
2025-01-21 02131, 2025
reosarevok[m]
(well, I didn't need to for sql-exporter but it worked when I needed the grafana ones)
2025-01-21 02134, 2025
bitmap[m]
yvanzo: did you already have a musicbrainz-docker branch that points to the current mb-solr branch solr-9.7.0?
Apparently it gave up before I created the database.
2025-01-21 02110, 2025
yvanzo[m]
I just restarted it.
2025-01-21 02134, 2025
yvanzo[m]
level=ERROR source=postgres_exporter.go:681 msg="error scraping dsn" err="Error opening connection to database (postgres://musicbrainz:PASSWORD_REMOVED@db:5432/musicbrainz_db?sslmode=disable): pq: database \"musicbrainz_db\" does not exist" dsn="postgres://musicbrainz:musicbrainz@db:5432/musicbrainz_db?sslmode=disable"
2025-01-21 02139, 2025
reosarevok[m]
I do mean sql-exporter - that's separate :)
2025-01-21 02143, 2025
yvanzo[m]
(that was before)
2025-01-21 02146, 2025
reosarevok[m]
But that suggests there's an issue, yeah :D
2025-01-21 02153, 2025
yvanzo[m]
I don't have sql-exporter
2025-01-21 02114, 2025
reosarevok[m]
Huh. then it cannot show the data
2025-01-21 02117, 2025
yvanzo[m]
I started it.
2025-01-21 02139, 2025
yvanzo[m]
Why didn’t it start then?
2025-01-21 02157, 2025
reosarevok[m]
I dunno, it has always started for me
2025-01-21 02147, 2025
yvanzo[m]
You didn’t make grafana container depend on it.
2025-01-21 02105, 2025
yvanzo[m]
But that can be intended.
2025-01-21 02120, 2025
yvanzo[m]
Please document how to start it.
2025-01-21 02140, 2025
reosarevok[m]
I literally have only ever started it by doing docker compose up -d
2025-01-21 02154, 2025
reosarevok[m]
Which is how we start everything else :D
2025-01-21 02101, 2025
yvanzo[m]
I just up grafana.
2025-01-21 02123, 2025
BrainzGit
[listenbrainz-server] 14MonkeyDo merged pull request #3125 (03master…LB-1528): LB-1528: Added shortcut to today's date in Fresh Releases using a Calendar Icon https://github.com/metabrainz/listenbrainz-server…
2025-01-21 02136, 2025
yvanzo[m]
when working on a specific container, I don't up the rest
2025-01-21 02144, 2025
reosarevok[m]
I mean, that's okay but that's not what the documentation says for starting the server :) So it's not surprising things don't work
2025-01-21 02118, 2025
yvanzo[m]
but again, doesn't matter if it is expected to up everything or some things only, just need to be documented.
2025-01-21 02109, 2025
monkey[m]
mayhem, lucifer: LB stats are currently borked, is it on your radar? I saw some discussion about failed jobs earlier so i was wondering if that's what it was
2025-01-21 02150, 2025
mayhem[m]
I assume this is collateral damage from the fight that lucifer is fighting with the dumps
2025-01-21 02122, 2025
bitmap[m]
reosarevok[m]: would it make sense to add the dependency between grafana and sql-exporter?
2025-01-21 02122, 2025
yvanzo[m]
reosarevok: Can the history of the count be shown instead of just the current count?
2025-01-21 02147, 2025
reosarevok[m]
I mean, I'm fine with adding the dependency if that seems useful, that's not a problem
2025-01-21 02134, 2025
reosarevok[m]
It can, it's trivial, but what's the point? It's not like they change fast
2025-01-21 02155, 2025
reosarevok[m]
That's why I specifically made it a gauge
2025-01-21 02111, 2025
reosarevok[m]
Because having a graph seemed useless
2025-01-21 02121, 2025
reosarevok[m]
But if you think it's not I can change it 🤷
2025-01-21 02138, 2025
yvanzo[m]
If you have to investigate what happened at a certain time, it would be useful to have it.
2025-01-21 02153, 2025
yvanzo[m]
Which is mostly what we are looking for.
2025-01-21 02104, 2025
yvanzo[m]
Just like the other dashboard actually.
2025-01-21 02133, 2025
yvanzo[m]
It can be a separate dashboard too, if they have different goals.
2025-01-21 02158, 2025
reosarevok[m]
Ok, that's the default anyway, so sure - it just seemed like a waste of space to have 20 graphs, but I will change it back
2025-01-21 02104, 2025
reosarevok[m]
Hmm
2025-01-21 02107, 2025
reosarevok[m]
Actually
2025-01-21 02121, 2025
reosarevok[m]
When you click the three dot menu on the gauge, then explore, do you get a graph?
2025-01-21 02136, 2025
reosarevok[m]
It kinda seems it might be built in as a "go deeper" option anyway
holycow23[m] uploaded an image: (21KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/matrix.org/aoYARSnRzLFWWikVwneSQqYo/image.png >
2025-01-21 02147, 2025
holycow23[m]
* For the ticket LB-1722, and discussion we had does the below attached image look good to go?
2025-01-21 02129, 2025
s1b1 joined the channel
2025-01-21 02126, 2025
BobSwift[m]
reosarevok, wileyfoxyx: I've fixed the RTD setup so that the proof of concept site now updates automatically again when there is a merge / push in the GitHub repo.
2025-01-21 02141, 2025
wileyfoxyx[m]
rdswift: on, that's good then. It was updating automatically with my repo, though.
2025-01-21 02138, 2025
monkey[m]
holycow23: Looks good. Perhaps the phrasing "Link unmatched listens" or "Link unknown listens" would sound less redundant.
Might be good to check with the user who created the ticket, to get confirmation
2025-01-21 02145, 2025
holycow23[m]
Sure thing, will ask the user about the same
2025-01-21 02146, 2025
pite_ has quit
2025-01-21 02122, 2025
BobSwift[m]
mayhem, reosarevok, aerozol, wileyfoxyx: I started a page listing some questions that I think we might need to consider. There are about a bazillion more, but this is a start. Feel free to add or comment (I think I've opened it up so you can make edits). The list is at https://github.com/rdswift/musicbrainz-docs/wiki/…
2025-01-21 02135, 2025
wileyfoxyx[m]
rdswift: just added my notes. I think it'll be okay that I haven't put "wileyfoxyx agrees with that" to every note of yours in it.
2025-01-21 02152, 2025
q3lont has quit
2025-01-21 02101, 2025
q3lont joined the channel
2025-01-21 02120, 2025
vardhan_ joined the channel
2025-01-21 02104, 2025
vardhan has quit
2025-01-21 02142, 2025
q3lont has quit
2025-01-21 02120, 2025
bitmap[m]
yvanzo: I was able to successfully reindex the sample database after setting `MB_SOLR_VERSION` and copying over the necessary changes to docker-compose.yml too (for the volume location and startup command)
2025-01-21 02157, 2025
bitmap[m]
the only issue I had was the sir container crashing initially (and having to be manually started later) due to the search service not being available immediately
