WARN: environment contains invalid option 'backup-enabled'

[mindrunner]

I see a lot of those warnings in my logs:

Sourcing /home/postgres/.pgbackrest_environment                                                                                                                                                                                                                                                                                                                                         
WARN: environment contains invalid option 'backup-enabled'                                                                                                                                                                                                                                                                                                                              
Sourcing /home/postgres/.pgbackrest_environment                                                                                                                                                                                                                                                                                                                                         
WARN: environment contains invalid option 'backup-enabled'                                                                                                                                                                                                                                                                                                                              
Sourcing /home/postgres/.pgbackrest_environment                                                                                                                                                                                                                                                                                                                                         
WARN: environment contains invalid option 'backup-enabled'                                                                                                                                                                                                                                                                                                                              
Sourcing /home/postgres/.pgbackrest_environment                                                                                                                                                                                                                                                                                                                                         
WARN: environment contains invalid option 'backup-enabled'                                                                                                                                                                                                                                                                                                                              
Sourcing /home/postgres/.pgbackrest_environment                                                                                                                                                                                                                                                                                                                                         
WARN: environment contains invalid option 'backup-enabled'     

It seems like helm chart is setting some environment whicht pgbackrest tries to parse. Similar to https://github.com/pgbackrest/pgbackrest/issues/799

Is that intended? Do I miss some configuration?

[davidandreoletti]

@mindrunner This is a non issue, caused by PGBACKREST_BACKUP_ENABLED set by the chart and pgbackrest having no support for a 'backup-enabled' setting.

[mindrunner]

Well, it's pretty verbose in my logs. Feels really not necessary to use a variable name which is sort of reserved by another application in the same stack. I mean, warnings are there for a reason, aren't they. Would a PR changing this variable be accepted?

[davidandreoletti]

Your suggestion seems like a reasonable fix it. Best to confirm with maintainers.

[jleni]

I would like to fix this. @nhudson @paulfantom Is this something I can start with?

[nhudson]

@jleni please if you can, create a PR and I will review when I can.

[mshivanna]

is there any update on this ticket? thank you