Skip to content
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

Settings service: 2 envvars needed to be added to existin ones #10359

Closed
mmattel opened this issue Oct 21, 2024 · 1 comment · Fixed by #10362
Closed

Settings service: 2 envvars needed to be added to existin ones #10359

mmattel opened this issue Oct 21, 2024 · 1 comment · Fixed by #10362

Comments

@mmattel
Copy link
Contributor

mmattel commented Oct 21, 2024

Identified while reading: #9490 (set the configured protocol transport for service metadata)

The settings service has two envvars that do not start with SETTINGS_ but with STORAGE_. These envvars are unfindable because there is no storage service. The envvars in question are:

STORAGE_GRPC_ADDR and
STORAGE_GATEWAY_GRPC_ADDR

To fix this, we should ADD correct named envvars to the existing ones:

SETTINGS_STORAGE_GRPC_ADDR and
SETTINGS_STORAGE_GATEWAY_GRPC_ADDR

Of course deprecating the "wrong" named one is appreciated, technically there is no hard need for it.

@dragonchaser @kobergj @butonic

@butonic
Copy link
Member

butonic commented Oct 21, 2024

damn

// Metadata configures the metadata store to use
type Metadata struct {
	GatewayAddress string `yaml:"gateway_addr" env:"STORAGE_GATEWAY_GRPC_ADDR" desc:"GRPC address of the STORAGE-SYSTEM service." introductionVersion:"pre5.0"`
	StorageAddress string `yaml:"storage_addr" env:"STORAGE_GRPC_ADDR" desc:"GRPC address of the STORAGE-SYSTEM service." introductionVersion:"pre5.0"`

Yeah ... that should be

// Metadata configures the metadata store to use
type Metadata struct {
	GatewayAddress string `yaml:"gateway_addr" env:"SETTINGS_STORAGE_GATEWAY_GRPC_ADDR" desc:"GRPC address of the STORAGE-SYSTEM service." introductionVersion:"pre5.0"`
	StorageAddress string `yaml:"storage_addr" env:"SETTINGS_STORAGE_GRPC_ADDR" desc:"GRPC address of the STORAGE-SYSTEM service." introductionVersion:"pre5.0"`

introduced pre 5.0 ... we should at least add the SETTINGS_ prefixed env vars ... @mmattel care to prepare a PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants