-
Notifications
You must be signed in to change notification settings - Fork 516
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
feat: support environment in ExtensionServiceConfig
#8273
Conversation
5b1bbec
to
34e4f2f
Compare
e71db82
to
78b4138
Compare
/promote integration-extensions |
pkg/machinery/config/types/extensionservicesconfig/extension_services_config.go
Outdated
Show resolved
Hide resolved
f0a933c
to
b8e2c18
Compare
ExtensionServicesConfig
ExtensionServiceConfig
b8e2c18
to
071e078
Compare
/promote integration-extensions |
// ConfigFiles implements config.ExtensionServiceConfig interface. | ||
func (e *ServiceConfigV1Alpha1) ConfigFiles() []config.ExtensionServiceConfigFile { | ||
return xslices.Map(e.ServiceConfigFiles, func(c ConfigFile) config.ExtensionServiceConfigFile { | ||
return &ConfigFile{ |
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.
that's a nit, but looks like we could either do return &c
here, or we could define methods Content()
and MountPath()
on non-pointer ConfigFile
so we can avoid this copying
we still need .Map()
to convert to the interface type though
071e078
to
fef3995
Compare
Support setting extension services environment variables in `ExtensionServiceConfig` document. Refactor `ExtensionServicesConfig` -> `ExtensionServiceConfig` and move extensions config under `runtime` pkg. Fixes: siderolabs#8271 Signed-off-by: Noel Georgi <[email protected]>
fef3995
to
15e8bca
Compare
Use `ExtensionServiceConfig` document for extensions using `.machine.files` or `environmentFile`. Ref: siderolabs/talos#8273 Signed-off-by: Noel Georgi <[email protected]>
/m |
Use `ExtensionServiceConfig` document for extensions using `.machine.files` or `environmentFile`. Ref: siderolabs/talos#8273 Signed-off-by: Noel Georgi <[email protected]>
Support setting extension services environment variables in
ExtensionServiceConfig
document.Fixes: #8271