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

docs: add doc for the configuration #2272

Merged
merged 3 commits into from
Sep 22, 2023
Merged

Conversation

Yisaer
Copy link
Collaborator

@Yisaer Yisaer commented Sep 21, 2023

No description provided.

Signed-off-by: yisaer <[email protected]>
@Yisaer Yisaer requested a review from ngjaying September 21, 2023 02:31
@@ -20,6 +20,14 @@ basic:
maxAge: 72
# Whether to ignore case in SQL processing. Note that, the name of customized function by plugins are case-sensitive.
ignoreCase: false
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we can add more descriptions. The cfgStorageType should be file and kv only. If kv is set, the real kv type is defined in the store section, which could be sqlite/redis and fdb in the future. Do not need to mention fdb for now.

Signed-off-by: yisaer <[email protected]>
@codecov
Copy link

codecov bot commented Sep 21, 2023

Codecov Report

Patch coverage: 25.00% and project coverage change: -0.03% ⚠️

Comparison is base (f7e6f3c) 62.83% compared to head (044a618) 62.80%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2272      +/-   ##
==========================================
- Coverage   62.83%   62.80%   -0.03%     
==========================================
  Files         292      292              
  Lines       34995    34995              
==========================================
- Hits        21988    21976      -12     
- Misses      10976    10987      +11     
- Partials     2031     2032       +1     
Files Changed Coverage Δ
internal/conf/yaml_config_storage.go 65.66% <0.00%> (ø)
internal/conf/yaml_config_ops.go 74.00% <33.33%> (ø)

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: yisaer <[email protected]>
@ngjaying ngjaying merged commit 025a1ce into lf-edge:master Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants