@@ -370,7 +370,7 @@ jobs:
370370api-key :${{ secrets.DATADOG_API_KEY }}
371371
372372test-go-pg :
373- runs-on :${{ matrix.os == 'ubuntu-latest' && github.repository_owner == 'coder' && 'depot-ubuntu-22.04-4' || matrix.os == 'macos-latest' && github.repository_owner == 'coder' && 'macos-latest-xlarge' || matrix.os }}
373+ runs-on :${{ matrix.os == 'ubuntu-latest' && github.repository_owner == 'coder' && 'depot-ubuntu-22.04-4' || matrix.os == 'macos-latest' && github.repository_owner == 'coder' && 'macos-latest-xlarge' || matrix.os== 'windows-2022' && github.repository_owner == 'coder' && 'windows-latest-16-cores' || matrix.os }}
374374needs :changes
375375if :needs.changes.outputs.go == 'true' || needs.changes.outputs.ci == 'true' || github.ref == 'refs/heads/main'
376376# This timeout must be greater than the timeout set by `go test` in
@@ -383,6 +383,7 @@ jobs:
383383os :
384384 -ubuntu-latest
385385 -macos-latest
386+ -windows-2022
386387steps :
387388 -name :Harden Runner
388389uses :step-security/harden-runner@0080882f6c36860b6ba35c610c98ce87d4e2f26f # v2.10.2
@@ -438,56 +439,6 @@ jobs:
438439with :
439440api-key :${{ secrets.DATADOG_API_KEY }}
440441
441- # NOTE: this could instead be defined as a matrix strategy, but we want to
442- # temporarily allow windows tests to fail. Using a matrix strategy here makes
443- # the check in the `required` job rather complicated.
444- test-go-pg-windows :
445- runs-on :${{ github.repository_owner == 'coder' && 'windows-latest-16-cores' || 'windows-latest' }}
446- needs :changes
447- if :needs.changes.outputs.go == 'true' || needs.changes.outputs.ci == 'true' || github.ref == 'refs/heads/main'
448- # This timeout must be greater than the timeout set by `go test` in
449- # `make test-postgres` to ensure we receive a trace of running
450- # goroutines. Setting this to the timeout +5m should work quite well
451- # even if some of the preceding steps are slow.
452- timeout-minutes :25
453- steps :
454- -name :Harden Runner
455- uses :step-security/harden-runner@91182cccc01eb5e619899d80e4e971d6181294a7 # v2.10.1
456- with :
457- egress-policy :audit
458-
459- -name :Checkout
460- uses :actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1
461- with :
462- fetch-depth :1
463-
464- -name :Setup Go
465- uses :./.github/actions/setup-go
466-
467- -name :Setup Terraform
468- uses :./.github/actions/setup-tf
469-
470- -name :Test with PostgreSQL Database
471- env :
472- POSTGRES_VERSION :" 13"
473- TS_DEBUG_DISCO :" true"
474- shell :bash
475- run :|
476- # By default Go will use the number of logical CPUs, which
477- # is a fine default.
478- PARALLEL_FLAG=""
479-
480- go run scripts/embedded-pg/main.go
481- DB=ci gotestsum --format standard-quiet -- -v -short -count=1 ./...
482-
483- name :Upload test stats to Datadog
484- timeout-minutes :1
485- continue-on-error :true
486- uses :./.github/actions/upload-datadog
487- if :success() || failure()
488- with :
489- api-key :${{ secrets.DATADOG_API_KEY }}
490-
491442# NOTE: this could instead be defined as a matrix strategy, but we want to
492443# only block merging if tests on postgres 13 fail. Using a matrix strategy
493444# here makes the check in the above `required` job rather complicated.