bfe icon indicating copy to clipboard operation
bfe copied to clipboard

Feature configuration test

Open kwanhur opened this issue 2 years ago • 2 comments

support bfe -t to test configuration valid or not

test failed when invalid bfe.conf

[vagrant@lvs bfe]$ ./bfe -t
2021/10/17 17:28:23 maxprocs: Leaving GOMAXPROCS=2: CPU quota undefined
bfe: configuration file conf/bfe.conf test failed
[10/17/21 17:28:23] [EROR] main(): in BfeConfigLoad():HttpPort[-8080] should be in [1, 65535]

test failed when invalid bfe module's conf, like mod_block invalid IPBlocklistPath config item

[vagrant@lvs bfe]$ ./bfe -t
2021/10/17 17:13:45 maxprocs: Leaving GOMAXPROCS=2: CPU quota undefined
[10/17/21 17:13:45] [EROR] Err in module.init() for mod_block [mod_block: loadGlobalIPTable() err err in GlobalIPTableLoad(conf/mod_block/ip_blocklist.data.0):load dict: loadFile(): conf/mod_block/ip_blocklist.data.0 open(): conf/mod_block/ip_blocklist.data.0, open conf/mod_block/ip_blocklist.data.0: no such file or directory]
[10/17/21 17:13:45] [EROR] StartUp(): bfeServer.InitModules():mod_block: loadGlobalIPTable() err err in GlobalIPTableLoad(conf/mod_block/ip_blocklist.data.0):load dict: loadFile(): conf/mod_block/ip_blocklist.data.0 open(): conf/mod_block/ip_blocklist.data.0, open conf/mod_block/ip_blocklist.data.0: no such file or directory
[10/17/21 17:13:45] [EROR] main(): bfe_server.StartUp(): mod_block: loadGlobalIPTable() err err in GlobalIPTableLoad(conf/mod_block/ip_blocklist.data.0):load dict: loadFile(): conf/mod_block/ip_blocklist.data.0 open(): conf/mod_block/ip_blocklist.data.0, open conf/mod_block/ip_blocklist.data.0: no such file or directory
bfe: configuration file conf/bfe.conf test failed

test successfully

[vagrant@lvs bfe]$ ./bfe -t
2021/10/17 17:29:43 maxprocs: Leaving GOMAXPROCS=2: CPU quota undefined
bfe: configuration file conf/bfe.conf test is successful

kwanhur avatar Sep 25 '21 08:09 kwanhur

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Oct 08 '21 03:10 CLAassistant

Codecov Report

Merging #846 (d240d10) into develop (68b570b) will decrease coverage by 0.03%. The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #846      +/-   ##
===========================================
- Coverage    54.29%   54.25%   -0.04%     
===========================================
  Files          301      301              
  Lines        26550    26556       +6     
===========================================
- Hits         14415    14409       -6     
- Misses       10477    10487      +10     
- Partials      1658     1660       +2     
Impacted Files Coverage Δ
bfe_server/bfe_server_init.go 0.00% <0.00%> (ø)
bfe_util/exit.go 0.00% <0.00%> (ø)
bfe_spdy/server_write_sched.go 78.46% <0.00%> (-1.54%) :arrow_down:
bfe_spdy/frame_write.go 49.49% <0.00%> (-1.02%) :arrow_down:
bfe_http2/server.go 72.36% <0.00%> (-0.17%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 68b570b...d240d10. Read the comment docs.

codecov-commenter avatar Oct 17 '21 04:10 codecov-commenter