From 5959c21d934159d722e15b5c88291e46435fe1c3 Mon Sep 17 00:00:00 2001 From: wayne warren Date: Tue, 5 Nov 2024 12:35:19 -0700 Subject: [PATCH 1/5] fix(clustered): add Known Bugs section to release --- .../reference/release-notes/clustered.md | 43 +++++++++++++++++++ 1 file changed, 43 insertions(+) diff --git a/content/influxdb/clustered/reference/release-notes/clustered.md b/content/influxdb/clustered/reference/release-notes/clustered.md index 9721b31da..17ea9ef11 100644 --- a/content/influxdb/clustered/reference/release-notes/clustered.md +++ b/content/influxdb/clustered/reference/release-notes/clustered.md @@ -36,6 +36,49 @@ spec: image: us-docker.pkg.dev/influxdb2-artifacts/clustered/influxdb:20241022-1346953 ``` +### Known Bugs + +### `core` service DSN parsing errors + +This release has a known bug in the `core` pods with respect to handling of +options in Postgres DSNs. This bug can be seen in the `core-MMMMMMMMMM-NNNNN` +logs that look like the following: + +``` +2024-11-04T01:00:00.000Z | 3: error returned from database: database "influxdb&options=-c%20search_path=" does not exist +2024-11-04T01:00:19.000Z | 4: database "influxdb&options=-c%20search_path=" does not exist +``` + +The issue here is that entire `influxdb&options=-c%20search_path=` string is +being interpreted as the database name, owing to incorrect parsing of the +`POSTGRES_DSN` environment variable. + +The workaround to deal with this before a fix makes its way into the next +release is to use an image override in your AppInstance that looks like: + +``` +apiVersion: kubecfg.dev/v1alpha1 +kind: AppInstance +metadata: + name: influxdb + namespace: influxdb +spec: + package: + image: us-docker.pkg.dev/influxdb2-artifacts/clustered/influxdb:20241024-1354148 + apiVersion: influxdata.com/v1alpha1 + spec: + images: + overrides: + - name: 'influxdb2-artifacts/granite/granite' + newFQIN: 'us-docker.pkg.dev/influxdb2-artifacts/granite/granite:7acf9ca6e1ad15db80b22cd0bc071acdb561eb51' +# ...additional configuration left out for brevity +``` + +Your AppInstance may have additional configuration, but the important part here +is the `spec.package.spec.images.overrides` section where we override the +built-in image used by the `core` pods with one that has a bugfix for the DSN +parsing error. + ### Highlights #### AppInstance image override bug fix From 93a59917193c586e4f6a14ba2bce493e3ed22c32 Mon Sep 17 00:00:00 2001 From: wayne Date: Tue, 5 Nov 2024 15:45:04 -0700 Subject: [PATCH 2/5] Update content/influxdb/clustered/reference/release-notes/clustered.md Co-authored-by: Jason Stirnaman --- .../influxdb/clustered/reference/release-notes/clustered.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/content/influxdb/clustered/reference/release-notes/clustered.md b/content/influxdb/clustered/reference/release-notes/clustered.md index 17ea9ef11..2ddf06632 100644 --- a/content/influxdb/clustered/reference/release-notes/clustered.md +++ b/content/influxdb/clustered/reference/release-notes/clustered.md @@ -49,9 +49,9 @@ logs that look like the following: 2024-11-04T01:00:19.000Z | 4: database "influxdb&options=-c%20search_path=" does not exist ``` -The issue here is that entire `influxdb&options=-c%20search_path=` string is -being interpreted as the database name, owing to incorrect parsing of the -`POSTGRES_DSN` environment variable. +Due to incorrect parsing of the +`POSTGRES_DSN` environment variable, the `influxdb&options=-c%20search_path=` string is +interpreted as the database name. The workaround to deal with this before a fix makes its way into the next release is to use an image override in your AppInstance that looks like: From da19d4efe547268643089ed95864b9e99d894afb Mon Sep 17 00:00:00 2001 From: wayne Date: Tue, 5 Nov 2024 15:45:21 -0700 Subject: [PATCH 3/5] Update content/influxdb/clustered/reference/release-notes/clustered.md Co-authored-by: Jason Stirnaman --- .../influxdb/clustered/reference/release-notes/clustered.md | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/content/influxdb/clustered/reference/release-notes/clustered.md b/content/influxdb/clustered/reference/release-notes/clustered.md index 2ddf06632..fa52b6a94 100644 --- a/content/influxdb/clustered/reference/release-notes/clustered.md +++ b/content/influxdb/clustered/reference/release-notes/clustered.md @@ -53,8 +53,10 @@ Due to incorrect parsing of the `POSTGRES_DSN` environment variable, the `influxdb&options=-c%20search_path=` string is interpreted as the database name. -The workaround to deal with this before a fix makes its way into the next -release is to use an image override in your AppInstance that looks like: +To work around this bug, in your AppInstance, +include a `spec.package.spec.images.overrides` section to override the +`core` pods built-in image with an image that has the bugfix for the DSN +parsing error--for example: ``` apiVersion: kubecfg.dev/v1alpha1 From dc78e42fa0a7025f313ac904c3bbe4a93a56c062 Mon Sep 17 00:00:00 2001 From: wayne Date: Tue, 5 Nov 2024 15:45:36 -0700 Subject: [PATCH 4/5] Update content/influxdb/clustered/reference/release-notes/clustered.md Co-authored-by: Jason Stirnaman --- content/influxdb/clustered/reference/release-notes/clustered.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/content/influxdb/clustered/reference/release-notes/clustered.md b/content/influxdb/clustered/reference/release-notes/clustered.md index fa52b6a94..079306ec2 100644 --- a/content/influxdb/clustered/reference/release-notes/clustered.md +++ b/content/influxdb/clustered/reference/release-notes/clustered.md @@ -73,7 +73,7 @@ spec: overrides: - name: 'influxdb2-artifacts/granite/granite' newFQIN: 'us-docker.pkg.dev/influxdb2-artifacts/granite/granite:7acf9ca6e1ad15db80b22cd0bc071acdb561eb51' -# ...additional configuration left out for brevity +# ...[remaining configuration] ``` Your AppInstance may have additional configuration, but the important part here From 9e609cb606c57fb4b805a6c5997f3204c0f94348 Mon Sep 17 00:00:00 2001 From: wayne Date: Tue, 5 Nov 2024 15:45:56 -0700 Subject: [PATCH 5/5] Update content/influxdb/clustered/reference/release-notes/clustered.md Co-authored-by: Jason Stirnaman --- .../influxdb/clustered/reference/release-notes/clustered.md | 5 ----- 1 file changed, 5 deletions(-) diff --git a/content/influxdb/clustered/reference/release-notes/clustered.md b/content/influxdb/clustered/reference/release-notes/clustered.md index 079306ec2..7474bf116 100644 --- a/content/influxdb/clustered/reference/release-notes/clustered.md +++ b/content/influxdb/clustered/reference/release-notes/clustered.md @@ -76,11 +76,6 @@ spec: # ...[remaining configuration] ``` -Your AppInstance may have additional configuration, but the important part here -is the `spec.package.spec.images.overrides` section where we override the -built-in image used by the `core` pods with one that has a bugfix for the DSN -parsing error. - ### Highlights #### AppInstance image override bug fix