Skip to content

Conversation

@ldionne
Copy link
Member

@ldionne ldionne commented Oct 22, 2025

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.

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
Copy link
Member Author

ldionne commented Oct 22, 2025

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

What do you think about this patch?

@DavidSpickett
Copy link
Contributor

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

@antmox
Copy link
Contributor

antmox commented Oct 23, 2025

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

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

@ldionne
Copy link
Member Author

ldionne commented Oct 23, 2025

@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.

@antmox
Copy link
Contributor

antmox commented Oct 24, 2025

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

@ldionne
Copy link
Member Author

ldionne commented Oct 24, 2025

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

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

Successfully merging this pull request may close these issues.

3 participants