chore: add cockroachdb 26.1 to the matrix and modify version parsing

[tstirrat15]

Description

Cockroach 26.1 has been released and we want to make sure that SpiceDB plays nice with it. This moves the matrix so that we test against 26.1.

We found that Cockroach now restricts access to its internal table, so we can't grab the version object directly. This changes the logic so that we rely on string parsing. This is less safe/direct, but it also works with v26.1.

Changes

• Move matrix so that we test against 26.1

Testing

Review. See that tests pass.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.27%. Comparing base (f3e150e) to head (4e0383b).

❌ Your project check has failed because the head coverage (73.27%) is below the target coverage (75.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2907      +/-   ##
==========================================
- Coverage   74.54%   73.27%   -1.27%     
==========================================
  Files         484      484              
  Lines       59228    59220       -8     
==========================================
- Hits        44146    43387     -759     
- Misses      11990    12789     +799     
+ Partials     3092     3044      -48     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[miparnisari]

why wrap VersionReadingTest at all with createDatastoreTest? just write it as a normal test func TestVersionReading(t *testing.T) { .. }

[tstirrat15]

Because the createDatastoreTest is parametrized over CRDB versions and takes care of setting up the datastore. I want to be sure that we're correctly reading the version for all CRDB versions that we're testing against.

[miparnisari]

to what DB? where was the DB created?

[tstirrat15]

The DB instance in the test? Or are you wanting a more descriptive comment?

[miparnisari]

the DB is created in createDatastoreTest, and the url that we hardcoded in line 986 just... works? is that it?

i wonder if we could structure this better. e.g. if the DB created points to port 5433, this test will break.

[tstirrat15]

I think so - I copied this test from the structure of a preceding test. I haven't dug into how they work internally. I kinda wonder if this is something with a pgx test connection or a detail of how we're using test containers.