New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
omit PgVersion diff on sync #860
Conversation
5e486b7
to
9668d27
Compare
@@ -716,6 +716,55 @@ func makeResources(cpuRequest, memoryRequest, cpuLimit, memoryLimit string) acid | |||
} | |||
} | |||
|
|||
func extractPgVersionFromBinPath(binPath string, template string) (string, error) { | |||
var pgVersion float32 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't it be just an integer?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when there can be 9.6
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, forgot that for a couple of years it still would be a thing.
👍 |
1 similar comment
👍 |
Changing the Postgres version can currently break the cluster. It is caught on
Update
but not onSync
.Retrieving the current PG version on sync is only possible through the
bin_dir
setting in theSPILO_CONFIGURATION
env variable of thepostgres
container. Extracting the version is done withSscanf
to make it work even when the bin path template is configurable, as suggested in #554.This PR also adds
PostgresParam
wherePgVersion
field is read