fix: compatibility for cockroach v22.1 (#3754)

* fix: compatability for cockroach v22.1

* update cr version in docs and compose

* bind dev cockroach to 127.0.0.1

* Revert "bind dev cockroach to 127.0.0.1"

This reverts commit 1c7c6304ae6682801f8475a8eac91569442b6b8d.

* remove --listen-addr flag

* use v2.0.0-v2-alpha.16

* use v2.0.0-v2-alpha.17

Co-authored-by: Elio Bischof <eliobischof@gmail.com>
This commit is contained in:
Livio Amstutz 2022-06-01 11:41:01 +02:00 committed by GitHub
parent 66b6ef7f04
commit 32afa77b97
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
7 changed files with 19 additions and 17 deletions

View File

@ -89,9 +89,9 @@ RUN apt install openssl tzdata tar
# cockroach binary used to backup database
RUN mkdir /usr/local/lib/cockroach
RUN wget -qO- https://binaries.cockroachdb.com/cockroach-v21.2.9.linux-amd64.tgz \
| tar xvz && cp -i cockroach-v21.2.9.linux-amd64/cockroach /usr/local/bin/
RUN rm -r cockroach-v21.2.9.linux-amd64
RUN wget -qO- https://binaries.cockroachdb.com/cockroach-v22.1.0.linux-amd64.tgz \
| tar xvz && cp -i cockroach-v22.1.0.linux-amd64/cockroach /usr/local/bin/
RUN rm -r cockroach-v22.1.0.linux-amd64
# Migrations for cockroach-secure
RUN go install github.com/rakyll/statik \

View File

@ -1 +1,2 @@
CREATE USER $1 WITH PASSWORD $2
-- replace %[1]s with the name of the user
CREATE USER %[1]s WITH PASSWORD $1

View File

@ -3,6 +3,7 @@ package initialise
import (
"database/sql"
_ "embed"
"fmt"
"github.com/spf13/cobra"
"github.com/spf13/viper"
@ -43,7 +44,7 @@ func VerifyUser(username, password string) func(*sql.DB) error {
logging.WithFields("username", username).Info("verify user")
return verify(db,
exists(searchUser, username),
exec(createUserStmt, username, &sql.NullString{String: password, Valid: password != ""}),
exec(fmt.Sprintf(createUserStmt, username), &sql.NullString{String: password, Valid: password != ""}),
)
}
}

View File

@ -31,7 +31,7 @@ func Test_verifyUser(t *testing.T) {
args: args{
db: prepareDB(t,
expectExists("SELECT EXISTS(SELECT username FROM [show roles] WHERE username = $1)", false, "zitadel-user"),
expectExec("CREATE USER $1 WITH PASSWORD $2", sql.ErrTxDone, "zitadel-user", nil),
expectExec("CREATE USER zitadel-user WITH PASSWORD $1", sql.ErrTxDone, nil),
),
username: "zitadel-user",
password: "",
@ -43,7 +43,7 @@ func Test_verifyUser(t *testing.T) {
args: args{
db: prepareDB(t,
expectExists("SELECT EXISTS(SELECT username FROM [show roles] WHERE username = $1)", false, "zitadel-user"),
expectExec("CREATE USER $1 WITH PASSWORD $2", nil, "zitadel-user", nil),
expectExec("CREATE USER zitadel-user WITH PASSWORD $1", nil, nil),
),
username: "zitadel-user",
password: "",
@ -55,7 +55,7 @@ func Test_verifyUser(t *testing.T) {
args: args{
db: prepareDB(t,
expectExists("SELECT EXISTS(SELECT username FROM [show roles] WHERE username = $1)", false, "zitadel-user"),
expectExec("CREATE USER $1 WITH PASSWORD $2", nil, "zitadel-user", "password"),
expectExec("CREATE USER zitadel-user WITH PASSWORD $1", nil, "password"),
),
username: "zitadel-user",
password: "password",

View File

@ -5,7 +5,7 @@ services:
restart: always
networks:
- zitadel
image: ghcr.io/zitadel/zitadel:v2.0.0-v2-alpha.3-amd64
image: ghcr.io/zitadel/zitadel:v2.0.0-v2-alpha.17-amd64
command: admin start-from-init --masterkey "MasterkeyNeedsToHave32Characters"
environment:
- ZITADEL_DATABASE_HOST=db
@ -27,8 +27,8 @@ services:
restart: always
networks:
- zitadel
image: cockroachdb/cockroach:v21.2.5
command: start-single-node --insecure --listen-addr=0.0.0.0
image: cockroachdb/cockroach:v22.1.0
command: start-single-node --insecure
healthcheck:
test: ["CMD", "curl", "-f", "http://localhost:8080/health?ready=1"]
interval: 10s

View File

@ -1,7 +1,7 @@
## Download The CockroachDB binary
Download a `cockroach` binary as described [here](https://www.cockroachlabs.com/docs/v21.2/install-cockroachdb).
ZITADEL is tested against CockroachDB v21.2.8.
Download a `cockroach` binary as described [here](https://www.cockroachlabs.com/docs/v22.1/install-cockroachdb).
ZITADEL is tested against CockroachDB v22.1.0.
## Run CockroachDB and ZITADEL
@ -22,7 +22,7 @@ export ZITADEL_DEFAULTINSTANCE_CUSTOMDOMAIN=localhost
Download the zitadel binary
```bash
curl -s https://api.github.com/repos/zitadel/zitadel/releases/tags/v2.0.0-v2-alpha.6 | grep "browser_download_url.*zitadel_.*_linux_$(dpkg --print-architecture)" | cut -d '"' -f 4 | sudo wget -i - -O /usr/local/bin/zitadel && sudo chmod +x /usr/local/bin/zitadel && sudo chown $(id -u):$(id -g) /usr/local/bin/zitadel
curl -s https://api.github.com/repos/zitadel/zitadel/releases/tags/v2.0.0-v2-alpha.17 | grep "browser_download_url.*zitadel_.*_linux_$(dpkg --print-architecture)" | cut -d '"' -f 4 | sudo wget -i - -O /usr/local/bin/zitadel && sudo chmod +x /usr/local/bin/zitadel && sudo chown $(id -u):$(id -g) /usr/local/bin/zitadel
```
Run the database and application containers

View File

@ -1,7 +1,7 @@
## Download The CockroachDB binary
Download a `cockroach` binary as described [here](https://www.cockroachlabs.com/docs/v21.2/install-cockroachdb).
ZITADEL is tested against CockroachDB v21.2.8.
Download a `cockroach` binary as described [here](https://www.cockroachlabs.com/docs/v22.1/install-cockroachdb).
ZITADEL is tested against CockroachDB v22.1.0.
## Run CockroachDB and ZITADEL
@ -31,7 +31,7 @@ brew install zitadel/tap/zitadel
... or download the binary from GitHub
```bash
curl -s https://api.github.com/repos/zitadel/zitadel/releases/tags/v2.0.0-v2-alpha.6 | grep "browser_download_url.*zitadel_.*_darwin_${MY_ARCHITECTURE}" | cut -d '"' -f 4 | sudo wget -i - -O /usr/local/bin/zitadel && sudo chmod +x /usr/local/bin/zitadel && sudo chown $(id -u):$(id -g) /usr/local/bin/zitadel
curl -s https://api.github.com/repos/zitadel/zitadel/releases/tags/v2.0.0-v2-alpha.17 | grep "browser_download_url.*zitadel_.*_darwin_${MY_ARCHITECTURE}" | cut -d '"' -f 4 | sudo wget -i - -O /usr/local/bin/zitadel && sudo chmod +x /usr/local/bin/zitadel && sudo chown $(id -u):$(id -g) /usr/local/bin/zitadel
```
Run ZITADEL