Skip to content

Commit

Permalink
feat: disable ALTER SYSTEM by default
Browse files Browse the repository at this point in the history
Closes #3544
  • Loading branch information
mnencia committed Dec 13, 2023
1 parent 220393d commit 835f826
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 5 deletions.
3 changes: 1 addition & 2 deletions api/v1/cluster_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -1181,8 +1181,7 @@ type PostgresConfiguration struct {
// If this parameter is true, the user will be able to invoke `ALTER SYSTEM`
// on this CloudNativePG Cluster.
// This should only be used for debugging and troubleshooting.
// Defaults to true.
// +kubebuilder:default:=true
// Defaults to false.
// +optional
EnableAlterSystem bool `json:"enableAlterSystem,omitempty"`
}
Expand Down
3 changes: 1 addition & 2 deletions config/crd/bases/postgresql.cnpg.io_clusters.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2715,11 +2715,10 @@ spec:
description: Configuration of the PostgreSQL server
properties:
enableAlterSystem:
default: true
description: If this parameter is true, the user will be able
to invoke `ALTER SYSTEM` on this CloudNativePG Cluster. This
should only be used for debugging and troubleshooting. Defaults
to true.
to false.
type: boolean
ldap:
description: Options to specify LDAP configuration
Expand Down
2 changes: 1 addition & 1 deletion docs/src/cloudnative-pg.v1.md
Original file line number Diff line number Diff line change
Expand Up @@ -3554,7 +3554,7 @@ big enough to simulate an infinite timeout</p>
<p>If this parameter is true, the user will be able to invoke <code>ALTER SYSTEM</code>
on this CloudNativePG Cluster.
This should only be used for debugging and troubleshooting.
Defaults to true.</p>
Defaults to false.</p>
</td>
</tr>
</tbody>
Expand Down

0 comments on commit 835f826

Please sign in to comment.