Skip to content

Commit

Permalink
[crdb] Upgrade CockroachDB to 24.1.3
Browse files Browse the repository at this point in the history
  • Loading branch information
barroco committed Aug 27, 2024
1 parent 06a9756 commit ee038e1
Show file tree
Hide file tree
Showing 7 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ test-go-units:

.PHONY: test-go-units-crdb
test-go-units-crdb: cleanup-test-go-units-crdb
@docker run -d --name dss-crdb-for-testing -p 26257:26257 -p 8080:8080 cockroachdb/cockroach:v21.2.7 start-single-node --listen-addr=0.0.0.0 --insecure > /dev/null
@docker run -d --name dss-crdb-for-testing -p 26257:26257 -p 8080:8080 cockroachdb/cockroach:v24.1.3 start-single-node --listen-addr=0.0.0.0 --insecure > /dev/null
@until [ -n "`docker logs dss-crdb-for-testing | grep 'nodeID'`" ]; do echo "Waiting for CRDB to be ready"; sleep 3; done;
go run ./cmds/db-manager/main.go --schemas_dir ./build/db_schemas/rid --db_version latest --cockroach_host localhost
go test -count=1 -v ./pkg/rid/store/cockroach --cockroach_host localhost --cockroach_port 26257 --cockroach_ssl_mode disable --cockroach_user root --cockroach_db_name rid
Expand Down
2 changes: 1 addition & 1 deletion build/deploy/metadata_base.libsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
shouldInit: false, // Set this to true if you are starting a new cluster.
grpc_port: 26257,
http_port: 8080,
image: 'cockroachdb/cockroach:v21.2.7',
image: 'cockroachdb/cockroach:v24.1.3',
nodeIPs: error 'must supply the per-node ip addresses as an array', // For AWS, this array should contain the allocation id of the elastic ips.
JoinExisting: [],
storageClass: 'standard',
Expand Down
2 changes: 1 addition & 1 deletion build/dev/docker-compose_dss.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ version: '3.8'
services:

local-dss-crdb:
image: cockroachdb/cockroach:v21.2.7
image: cockroachdb/cockroach:v24.1.3
command: start-single-node --insecure
ports:
- "26257:26257"
Expand Down
2 changes: 1 addition & 1 deletion cmds/core-service/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ go run ./cmds/core-service \
To run correctly, core-service must be able to [access](../../pkg/cockroach/flags/flags.go) a CockroachDB cluster. Provision of this cluster is handled automatically for a local development environment if following [the instructions for a standalone instance](../../build/dev/standalone_instance.md). Or, a CockroachDB instance can be created manually with:

```bash
docker container run -p 26257:26257 -p 8080:8080 --rm cockroachdb/cockroach:v21.2.7 start-single-node --insecure
docker container run -p 26257:26257 -p 8080:8080 --rm cockroachdb/cockroach:v24.1.3 start-single-node --insecure
```

#### Database configuration
Expand Down
2 changes: 1 addition & 1 deletion deploy/operations/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
FROM ubuntu:22.04

ENV COCKROACH_VERSION 21.2.7
ENV COCKROACH_VERSION 24.1.3

RUN apt-get update \
&& apt-get install -y unzip curl gnupg lsb-release apt-transport-https ca-certificates
Expand Down
2 changes: 1 addition & 1 deletion deploy/services/helm-charts/dss/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ cockroachdb:
# See https://github.com/cockroachdb/helm-charts/blob/master/cockroachdb/values.yaml
image:
repository: cockroachdb/cockroach
tag: v21.2.7
tag: v24.1.3
tls:
certs:
provided: true
Expand Down
2 changes: 1 addition & 1 deletion test/migrations/clear_db.sh
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@ echo "Starting CRDB container"
docker run -d --rm --name dss-crdb-for-migration-testing \
-p 26257:26257 \
-p 8080:8080 \
cockroachdb/cockroach:v21.2.7 start-single-node \
cockroachdb/cockroach:v24.1.3 start-single-node \
--insecure > /dev/null

0 comments on commit ee038e1

Please sign in to comment.