Skip to content

UCS/CONFIG: let ucx support configuration file#100

Open
ybwang1cbs wants to merge 3 commits intoyosefe:integration3from
ybwang1cbs:topic/add_config_file
Open

UCS/CONFIG: let ucx support configuration file#100
ybwang1cbs wants to merge 3 commits intoyosefe:integration3from
ybwang1cbs:topic/add_config_file

Conversation

@ybwang1cbs
Copy link

What

let ucx support configuration file

Why

Now we can use environment variable to set the configuration of UCX. Environment variable maybe not meet expectation if in a complex environmet. So I provide another way to set UCX configuration.

How

I use inih to read configuration file, which is BSD-3.
the prioirty of configuration: environment variable > config file > default.

} else {
/* Read and parse environment variable */
strncpy(buf + prefix_len, field->name, sizeof(buf) - prefix_len - 1);
/* environmental variavle prior to file config */

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: s/variavle/variable

@yosefe yosefe added the On hold label Feb 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants