6 Commits (master)

Author	SHA1	Message	Date
Alexander Lutay	6dabc7a42c	TT#154105 Improve error output for 'ngcpcfg get' ... The previous error message was really confusing: > root@spce:~# ngcpcfg get /asdf > 2021-12-06 18:48:12 sp1: Error: Cannot process template '/tmp/tmp.RBRJg02PEz': > > file error - parse error - input file handle line 1: unexpected token (/asdf) > [% /asdf %] > 2021-12-06 18:48:12 sp1: Error: Generating /tmp/tmp.NKfB8BtJXv based on /tmp/tmp.RBRJg02PEz: FAILED It happens due to the way ngcpcfg using TT2 template which is trying to include the file it the string looks like a PATH. The new error message is more user friendly (and generates proper/non-zero exit code): > root@spce:~# ngcpcfg get /asdf > 2021-12-07 12:42:50 spce: Error: cannot process request for '/asdf'! > root@spce:~# echo $? > 1 > root@spce:~# Change-Id: Ifaf56dec07adf8600bd9d7cf2e733769d6d82a4f	3 years ago
ngcp-config	bee2a1b028	TT#139455 test: some dirs must be a git repository ... - NGCPCTL_BASE - OUTPUT_DIRECTORY{CONFIG_POOL} * add empty-git-repo.tar.gz * fs.py: define check_output() helper * set DEBUG by default * ngcpcfgcli, return env and cfg instead of just outdir * cleanup on tests: - don't compile the regex, it's useless in a test contex - remove unused imports and fixtures - use check_output() from fs Change-Id: I0eb859b6f7aad4cd3dd8e15f4e211395c5131ce7	3 years ago
ngcp-config	682ddd0293	TT#139455 tests: OUTPUT_DIRECTORY by default ... ngcpcfg service expects that the produced files are inside a git repository * ngcpcfgcli: - set RUN_DIR by default - set STATE_FILES_DIR by default - use default-git-repository.tar.gz as base for outdir - return outdir value too - print stdout and stderr so on failure we can debug * t/test_*: - remove unused imports - don't create OUTPUT_DIRECTORY unless really necessary - remove print stdout/stderr - black indent Change-Id: I61e475cc9c775c30c3700a8c0e560e7c5b22f0fc	4 years ago
Victor Seva	744458aa96	TT#139455 test: use universal_newlines=True so no replace is needed ... * remove replace() * fix asserts now that we have output per line * use gitrepo when necessary Change-Id: I7e89998f630b8323d5463e218a55ea9cd9c86d57	4 years ago
Guillem Jover	354e7578c5	TT#81050 Add HA_CONFIG and PAIR_CONFIG support ... This makes it possible to use more specific config.yml files either for a pair of nodes, for settings that will always affect both, or for the HA names, for settings that might affect all A or B nodes. This matches the support in the template files. We need to move sourcing the ha_features and carrier_features before sourcing init, as now the ngcpcfg.cfg file can use these variables if set. Change-Id: Ic6b32acceb8f020f48c3afa09d9e84252b3531e7	5 years ago
Guillem Jover	6b9e569c91	TT#71952 Integrate ngcpcfg functional tests into t testsuite ... There were functional tests for the get and values actions, but had never been hooked into any testing framework. And some obsolete build action tests already covered by the python tests. Rewrite the get tests in python so that they get executed within docker with the expected permissions, and remove the old unhooked tests that are just confusing when adding new functionality. Change-Id: I7570e35428c1fa0a0f26de75721f84365d4daf9a	5 years ago