llvm-lnt icon indicating copy to clipboard operation
llvm-lnt copied to clipboard

Drop support for MySQL

Open ldionne opened this issue 4 months ago • 6 comments

According to Chris Matthews, this had been added exclusively for the (now defunct) lnt.llvm.org instance, which was running on Google's internal infrastructure and hence had constraints on the database software it could use.

Since that is not relevant anymore, simplify the codebase and the support matrix of LNT by dropping support for MySQL. According to Chris, LNT was originally always intended to run on a Postgres database.

This allows us to actually test what we claim to support, making LNT more robust despite its slightly shrunken support matrix. In practice, I expect that nobody will be impacted by this change since nobody is running a MySQL instance.

ldionne avatar Oct 22 '25 20:10 ldionne

CC @luke957 since you're running the igalia instance @cmatthew in case you happen to see this

What do you think about this patch?

ldionne avatar Oct 22 '25 20:10 ldionne

@antmox what about Linaro's instance(s)?

DavidSpickett avatar Oct 23 '25 09:10 DavidSpickett

@antmox what about Linaro's instance(s)?

we're still using mysql. we should move to postgres anyway

antmox avatar Oct 23 '25 09:10 antmox

@antmox what about Linaro's instance(s)?

we're still using mysql. we should move to postgres anyway

Aw, what a bummer. I didn't know Linaro was running an instance -- I'm currently building context on this. @antmox Do you think this is something that can be done in the short/mid term? It's not critically needed, but it would definitely be nice to straighten up our support matrix.

ldionne avatar Oct 23 '25 21:10 ldionne

@ldionne Yes, switching to postgres doesn't seem too complicated. I'll look at this soon. No problem merging this patch right now.

antmox avatar Oct 24 '25 08:10 antmox

@antmox That's nice, thanks! I'll still wait a bit before I merge this, in case you run into something unexpected.

ldionne avatar Oct 24 '25 16:10 ldionne

@antmox Any news?

ldionne avatar Dec 01 '25 23:12 ldionne