Skip to content
This repository was archived by the owner on Oct 23, 2023. It is now read-only.

Rename ProcessOptions to separate the functionality #32

Closed
farshidtz opened this issue Mar 24, 2022 · 2 comments · Fixed by #33
Closed

Rename ProcessOptions to separate the functionality #32

farshidtz opened this issue Mar 24, 2022 · 2 comments · Fixed by #33
Assignees

Comments

@farshidtz
Copy link
Member

func ProcessOptions(services ...string) error {

Rename to options.ProcessAppConfig such that it only handles apps.<app>.config.* and config.*.

@farshidtz
Copy link
Member Author

farshidtz commented Mar 24, 2022

In the configure hook of edgex-go, we need to handle all options, except for the runtime ones.

Runtime option only exist for the proxy and are applied as a post-stop command: snap/local/runtime-helpers/bin/security-proxy-post-setup.sh

@farshidtz
Copy link
Member Author

Other functions can be, e.g.:

options.ProcessAppAutostart
options.ProcessAppRuntime

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

Successfully merging a pull request may close this issue.

2 participants