Remove SQL Server upgrade scripts and code

[labkey-jeckels]

Rationale

We no longer support SQLServer as a primary DB

Changes

• Remove schema upgrade scripts
• Remove SQLServer conditional code
• Remove SupportedDatabases in module.properties
• Remove uses of PostgresOnlyTest
• Create Postgres version of covidseq schema

[labkey-jeckels]

@bbimber the interesting thing for this repo is the covidseq module. It had no Postgres upgrade scripts. I did a trivial port (to be verified by TeamCity) to prevent schema validation failures. If this is a module worth migrating, you'll want to backport these changes to 26.3 or 26.7 so there's something to migrate into.

[bbimber]

@labkey-jeckels - this is an example of what I'm writing about here for PRIMe/EHR: https://www.labkey.org/ONPRC/Support%20Tickets/issues-insert.view?issueDefName=siteissues

[bbimber]

@labkey-jeckels: thanks for this. two points come up:

1. covidseq is one of a few defunct modules that I would just as soon get rid of. I dont change modules much because it's gotten cumbersome to work with the distributions repo (i cant open PRs anymore). For our github actions-based workflows, I cloned the module lists (https://github.com/bimberlabinternal/DevOps/tree/master/githubActions/discvr-build/distributions/discvr), and the gradle framework works perfectly fine with the module hosting the module list, as opposed to needing one monolithic distributions repo.

That aside, we can eliminate covidseq and probably hivrc. I'm happy to address that separate from here.

2. I appreciate the addition to this table to catch/identify case-sensitivity issues. This is one of many, many places that might need attention. I'm able to look at non-PRIMe/EHR instances, but this is a significant potential for difficult to detect see breaks in expected behavior.

I will also think about this, but I would really like to see us implement lower-level solutions to either avoid or at least identify case-sensitivity issues

[labkey-jeckels]

@labkey-jeckels: thanks for this. two points come up:

1. covidseq is one of a few defunct modules that I would just as soon get rid of. I dont change modules much because it's gotten cumbersome to work with the distributions repo (i cant open PRs anymore). For our github actions-based workflows, I cloned the module lists (https://github.com/bimberlabinternal/DevOps/tree/master/githubActions/discvr-build/distributions/discvr), and the gradle framework works perfectly fine with the module hosting the module list, as opposed to needing one monolithic distributions repo.

That aside, we can eliminate covidseq and probably hivrc. I'm happy to address that separate from here.

2. I appreciate the addition to this table to catch/identify case-sensitivity issues. This is one of many, many places that might need attention. I'm able to look at non-PRIMe/EHR instances, but this is a significant potential for difficult to detect see breaks in expected behavior.

I will also think about this, but I would really like to see us implement lower-level solutions to either avoid or at least identify case-sensitivity issues

I will merge with the covidseq changes as-is then. Please open a ticket on removing whichever modules you want and we can help coordinate the distribution changes.