ngcpcfg

Commit Graph

Author	SHA1	Message	Date
Guillem Jover	1a3304646c	MT#62508 services: Remove any systemd broken service symlinks Before running «systemctl preset-all» we remove all .wants dependency directories, because otherwise the preset-all command leaves obsolete files behind. But «systemctl preset-all» fails if there's any broken service symlink around, which means we end up with no dependency information whatsoever implying a severely broken system, that in addition is then unbootable. Remove any such broken symlinks before calling «systemctl preset-all». Change-Id: I58fe650cb04ddb140922563169a7fedb09a8ceac (cherry picked from commit `a09d8f42e2`) (cherry picked from commit `46d3592934`)	9 months ago
Guillem Jover	c8a0469db9	MT#56568 status: Fix argument passing We need to pass the host lists to the check remote code, and we should not be passing any host lists to the check shared storage code, as that takes an action instead. Change-Id: I7292cf56f110df81f5a3b83da911742bae2af1b2 (cherry picked from commit `4a33099dd2`)	3 years ago
Guillem Jover	e37f377a14	TT#168102 check: Pass --network-file to network.yml validators These commands require the network.yml to be passed as an argument to the --network-file option. Change-Id: I3b1eb78eb1f810a1ea03a09576f8e1b90443953b (cherry picked from commit `7554d5e98f`)	3 years ago
dzenichev	d7be98ed72	TT#168102 Create script for instances connections validation Introduce a list of checks related to connections: * check that we don't have instance names duplicates * prove an existence of a preferred for instnace host * an instance to which instance gets connected exists * a host to which instance gets connected exists * interface via which instance gets connected is defined Change-Id: I24f43b0fb24e308f571a88f30a72a9b6dd04b94d (cherry picked from commit `6cb4416611`)	3 years ago
Guillem Jover	dd287165d6	TT#168102 check: Add a YAML schema and validation for instances.yml Change-Id: I5134d10b7b438e2d6ce60d838034df68e4bc71cb (cherry picked from commit `c27e56ce86`)	3 years ago
Guillem Jover	22fb20d0ad	TT#177550 check: Add a YAML schema and validation for sites.yml We add support for YAML schemas, and hook it into the check command to validate the sites.yml file. Change-Id: Ic6ed43e36361d395b485230af4ec3cf7e77758bb (cherry picked from commit `0f20e0ffe5`)	3 years ago
ngcp-config	955e76df44	TT#177550 clean: don't remove sites/current Change-Id: Ib8492d6f6ae87fddf349282e6edbc70de787405e (cherry picked from commit `6100f2002b`)	3 years ago
ngcp-config	29935251fd	TT#177550 check: support MGMTNODE config we can define what is the expected origin via /etc/ngcp-ngcpcfg/ngcpcfg.d/mgmnode.cfg for instance Change-Id: Ied83502ddc022a1c7bb948f43bee7ccc10efcd88 (cherry picked from commit `240b32ef0b`)	3 years ago
Guillem Jover	043df757f7	TT#177550 check: Refactor config YAML file validation into a function This way we can reuse to validate other YAML files. Change-Id: I2e6587e16a3176cba349f6260df1769d4312c619 (cherry picked from commit `09e83fc5a9`)	3 years ago
Guillem Jover	cc16eed64f	TT#177550 check: Add code to check the multi-site setup This off-loads the check to the ngcp-site script, and prints any error message from it if the script fails. Only run the check if ngcp-site is present, so that we avoid a hard-dependency on system-tools. Change-Id: I640f85383d424980d81a207dee7d586a88422862 (cherry picked from commit `ac1744b253`)	3 years ago
Guillem Jover	c4ae823c5e	TT#177550 initialise: Add sites/current to the default .gitignore This is the symlink pointing to the current site for this site, which cannot be committed to git, or we'd get constant conflicts when multiple sites commit their own. Change-Id: I054afc15bb3a52d0ca2357db5ea4a152cfff035b (cherry picked from commit `b89b447e64`)	3 years ago
Guillem Jover	5e2037cf82	TT#177600 Remove redundant <template>.tt2.<nodename> suffix handling This special handling was from a time when we did not have any other generic suffix support to override based on hostname, nodename, role, etc. It also depends on the now obsolete systems.cfg file which is redundant and already covered by the information contained in network.yml. Our tt2-process tool already handles these suffixes, so there is no point in special-casing them from within the build script. This way we simplify our templating and overriding mechanism, and reduce duplication of information. Change-Id: Iabf597dbaddf755a7f85504975afc8666a58d272	4 years ago
Michael Prokop	e51151ec35	TT#173500 Create tmpfiles with template name We want to be able to track down any left-behind tmp files, so ensure we're creating them with according file names. Change-Id: I1aa0ccdb8834e8aef919f1f028028c1bb14cbd01	4 years ago
Guillem Jover	732bc1705b	TT#161150 Unify and switch to NGCP_NODENAME and NGCP_PAIRNAME The HA_NODE, HA_FILE and HA_CONFIG variables and nomenclature are deprecated in favor of the NGCP_NODENAME, NODE_FILE and NODE_CONFIG. Switch the old uses to the new ones. Unify PAIRNAME to NGCP_PAIRNAME. Leave the old HA_ variables for backwards compatibility Change-Id: I11f09f1620c2e456f10fc17229b61ec5a35a732b	4 years ago
ngcp-config	063b139dd2	TT#160855 patch: don't use static match for PRO/CARRIER * use ngcpcfg environment variables to match files * same approach as tt2-process Change-Id: I396dcc71a09cd6b9d51ddfcb7e3e84461c6e1667	4 years ago
Guillem Jover	07f9aef4ca	TT#149202 check: Concatenate all configuration files when schema validating We should validate all configuration files by concatenating them, so that any local addition or override gets schema validated, otherwise these modifications will pass without notice, until it is probably too late. Change-Id: I9c7774eefdeadddb399dd68668779dffcd173696	4 years ago
Guillem Jover	6c45358677	TT#149202 cat: Add new action to concatenate configuration files This new action concatenates the various configuration files and outputs the result as YAML. This is useful because they have a determined ordering, and include optional host specific files that might not be obvious how to merge. This will also be the foundation for verifying all the current host settings against the cfg-schema. Change-Id: I72a61193f74caf3b2f7a58a47eefad2ed46c973a	4 years ago
Alexander Lutay	59f691b673	TT#158650 Improve error reporting for failing action 'build' Otherwise 'ngcpcfg apply' silently aborted on stage 'build' and error message is not visible due to long/parallel output. Change-Id: I34b2c14fecad33526d89939e62d85dbafab31535	4 years ago
ngcp-config	686a3d765a	TT#154852 services: fix detection of changes on instance builds * We were missing the scenario when only the build files for an instance had changes. The service for that instance was not restarted Change-Id: I85ba86bc57c7fc8b6df1ac2ef1144c2d3952a4da	4 years ago
Guillem Jover	a4f3d9d2de	TT#156156 Switch to «-» as the word separator in CLI options The prevalent convention in the project is to use «-» instead of «_» to split words in options. Switch the options to that, and keep backwards compatibility option alias that emits a deprecation warning while people transition to the new names. The alias will be removed on the next LTS. Change-Id: I8bb7f26f8a766f7ea76ba2d9e49b793e5cb8d11e	4 years ago
Michael Prokop	241040ec7f	TT#102912 Provide invocation name of ngcpcfg to subprocesses Instead of having to identify the parent process name (ngcpcfg vs. ngcp-config) inside subprocesses (like "status"), let's provide the invocation name through the main ngcpcfg binary via variable NGCPCFG_NAME. Thanks to Guillem Jover for the suggestion. Change-Id: Iafbac535bedb9cc9dc73f3d95861a5bc735cc739	4 years ago
ngcp-config	664ad7ddfe	TT#154852 build: inject instance_info to templates will provide a new instance_info variable with the all the content defined at network.yml for the instance removed intance_name since instance_info.name has the same value Change-Id: I349d0b430b59280bc383d46d5ba6a61f5d1f9046	4 years ago
Michael Prokop	3abb63eaac	TT#102912 status: report the name of the invoked binary When invoking `ngcp-config status`, we reported "ngcpcfg": \| [sipwise-lab-trunk] sipwise@sp1:~$ sudo ngcpcfg status \| 2021-12-09 17:30:08 sp1: Checking state of ngcpcfg: \| 2021-12-09 17:30:08 sp1: OK: has been initialised already \| [...] While what we should report instead is "ngcpcfg-config", like: \| [sipwise-lab-trunk] sipwise@sp1:~$ sudo ngcp-config status \| 2021-12-09 17:31:31 sp1: Checking state of ngcp-config: \| 2021-12-09 17:31:31 sp1: OK: has been initialised already \| [...] Change-Id: If8fc8076fdf1d214baea3c7388ab208947d66a05	4 years ago
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	4 years ago
ngcp-config	36c2cd5742	TT#139455 script/services: fix generate_list_to_process * expects values from CONFIG_POOL * append OUTPUT_DIRECTORY just when changing directory added at `83003649e3` Change-Id: I9c22bff70b58d4cfb49c3dab45330bccb898011e	4 years ago
ngcp-config	6ad43764ea	TT#139455 etckeeper: obey NGCPCTL_BASE Change-Id: I82fdde1f03a25409c330c83abe769dbd64149f04	4 years ago
Guillem Jover	e875197950	TT#142950 scripts: Fix options parsing Switch all code to use a loop to parse options so that these can parse multiple options (currently some scripts would simply parse one or another but not several), and to make them easier to extend. Change-Id: I5e902f2d7d8db78e66e48db1237924fa7d8fdda1	4 years ago
Victor Seva	83003649e3	TT#139455 support instances build we need to produce different output for each instance defined from the same source * helper/instances-info, produces the needed output to feed helper/tt2-process with the proper arguments to build instances files. We inject ``instance_name`` with the proper value to templates and INSTANCE_NAME to hooks environment * ${NGCPCTL_MAIN}/ngcpcfg.instances is where we define the supported instance templates and where the output path is * helper/tt2-process, add -r --replace <path:path> option in order to allow generation of different output from the same source Change-Id: I067266b5226485ea0d6c3bc3436275c5553a8177	4 years ago
ngcp-config	f7d9d812a4	TT#139455 don't chown configs if NGCP_TESTSUITE is set * create chown_config function and remove duplicate code Change-Id: I4996c1931cb53ec8120fa715ab0067b71ab8d5a8	4 years ago
Michael Prokop	3e8814205a	TT#82051 Shellcheck v0.8 fixes shellcheck v0.8 has new check, known as SC2295 ("Warn about "${x#$y}" treating $y as a pattern when not quoted"). Fixes: \| In scripts/build line 117: \| build_file="${build_file%%.${host}}" \| ^-----^ SC2295 (info): Expansions inside ${..} need to be quoted separately, otherwise they match as patterns. \| In scripts/encrypt line 24: \| x=${file##${NGCPCTL_MAIN}/templates/} # drop leading /etc/ngcp-config \| ^-------------^ SC2295 (info): Expansions inside ${..} need to be quoted separately, otherwise they match as patterns. ... as reported by our Github actions at https://github.com/sipwise/ngcpcfg Change-Id: I3371388ecf445905cca0bc4681276151aceb43ae	4 years ago
Alexander Lutay	d550799c1c	TT#139455 Improve patchtt error for missing tt2 template The previous error message was confusing: > root@web01a:~# ngcpcfg patch > 2021-10-20 14:21:29 web01a: Validating patch '/etc/ngcp-config/templates/etc/kamailio/lb_A/proxy/kamailio.conf.patchtt.tt2' > 2021-10-20 14:21:29 web01a: Error: Missing template for patch '/etc/ngcp-config/templates/etc/kamailio/lb_A/proxy/kamailio.conf.patchtt.tt2' > 2021-10-20 14:21:29 web01a: Error: Some operations above finished with an error for the file(s): > /etc/ngcp-config/templates/etc/kamailio/lb_A/proxy/kamailio.conf.patchtt.tt2 > root@web01a:~# The new version is: > root@web01a:~# ngcpcfg patch > 2021-10-20 14:26:45 web01a: Validating patch '/etc/ngcp-config/templates/etc/kamailio/lb_A/proxy/kamailio.conf.patchtt.tt2' > 2021-10-20 14:26:45 web01a: Error: Missing template '/etc/ngcp-config/templates/etc/kamailio/lb_A/proxy/kamailio.conf.tt2' > 2021-10-20 14:26:45 web01a: Error: for patch '/etc/ngcp-config/templates/etc/kamailio/lb_A/proxy/kamailio.conf.patchtt.tt2' > 2021-10-20 14:26:45 web01a: Error: Some operations above finished with an error for the file(s): > /etc/ngcp-config/templates/etc/kamailio/lb_A/proxy/kamailio.conf.patchtt.tt2 > root@web01a:~# Change-Id: I8979dc9dc545f8129ec69de71eef8cceab5fc337	4 years ago
Guillem Jover	6997e5a612	TT#146104 services: Always flush queues even when not running .services We need to flush the queues unconditionally regardless on whether we have executed .services files, otherwise the enqueued services during package upgrades might be left unattended for long periods of time in case no .services files got triggered due to modified configuration files. We flush all queues in case some enqueued service was left behind in the default queue. Change-Id: Iea8ab11fcb7e580cb8e160b8674e2dff3c9c85a0	4 years ago
Guillem Jover	ff5b523483	TT#142950 ngcpcfg: Make commit action update config file permissions We need to make sure the permissions for any config file are set correctly, otherwise on the next build action these will be updated, which might cause spurious commits just for these updates, that can end up generating rebase conflicts. Change-Id: I7c0f4cf13593da0d696ba25097b202e0b7a33674	4 years ago
Guillem Jover	93e84e054f	TT#142950 Centralize YAML file metadata initialization Instead of checking for the user, group and mode configuration knobs in various scripts (currently «build» and «set»), just make sure the variables have sane defaults when loading the «ngcpcfg.cfg» file. This also makes it possible to simplify the chown/chgrp calls into single chown calls. Change-Id: I1fcc06a7d311e4ebc4a96cc84ffae333be243c53	4 years ago
Guillem Jover	79ed4aa594	TT#142950 ngcpcfg: Use proper file metadata when creating config with set When we are creating a new file with set, for a known config.yml variation, we need to apply the correct owner/group and mode or the next build will modify them and that will change spurious commits that have no relation to the action being taken at the time. Change-Id: I93017cdcdcf22fa1c5ef8e103d215aa7bc110b9a	4 years ago
Mykola Malkov	589dedb674	TT#139151 Switch to use ngcp_hostname Use ngcp-hostname instead of using real hostname. Change-Id: Ia818f50e300e0f66762370eab331c7b7f1288d26	4 years ago
Victor Seva	cb2fe7406a	TT#126250 apply: fix error when called with no arguments introduced at commit `eff9d4ae07` > ngcp-service: info: Action restart skipped, service kamailio-proxy is not for this role > 2021-08-17 14:59:18 lb01a: Synchronizing current with expected services state > /usr/share/ngcp-ngcpcfg/scripts//apply: line 56: $1: unbound variable Change-Id: I75a551327ecb42b7ff6a14a5206b85fb35e5f608	5 years ago
Guillem Jover	2e8c2833e3	TT#79100 services: Special case corosync/pacemaker handling Like we were doing with heartbeat-2 we need to special case corosync/pacemaker and force their services first thing, so that the HA system is ready by the time the rest of the services are handled. Change-Id: I3215aed2172436e61b6b33c4a4159c1c3f89bc24 Reported-by: Alex Lutay <alutay@sipwise.com>	5 years ago
Victor Seva	eff9d4ae07	TT#126250 Bypass 'apply' options into 'build' script It is necessary to allow bypassing '--ignore-shared-storage-check': > ngcpcfg apply --ignore-shared-storage-check Previously 'apply' options were not passing to 'build' script. * docs: - add all supported options in apply - move '--force-all-services' description to services and add it in the list of options Change-Id: I091142d1f74b7b9e09ce94b963ff4bf9dfaa8db7	5 years ago
Alexander Lutay	1b339d2717	Revert "TT#126250 Bypass 'apply' options into 'build' script" It brakes trunk, we have to reconsider implementation here. This reverts commit `41c722a183`. Change-Id: I319c9a9568e250d939864a4c2b9c2a3cdec145e7	5 years ago
Alexander Lutay	41c722a183	TT#126250 Bypass 'apply' options into 'build' script It is necessary to allow bypassing '--ignore-shared-storage-check': > ngcpcfg apply --ignore-shared-storage-check Previously 'apply' options were not passing to 'build' script. Change-Id: Ia01558b05920e6aec10c8206a3747c828b237fc9	5 years ago
Mykola Malkov	eb4eca2e2c	TT#123200 Simplify check of remote origin of ngcp-config Previously we had quite complicated schema of origin urls on Pro/Carrier mgmt/Carrier non-mgmt nodes: Pro, both sp1 and sp2 - local /mnt/glusterfs/ngcpcfg-share directory. Carrier mgmt nodes, web01a/web01b - local /mnt/glusterfs/ngcpcfg-share directory. Carrier other nodes - web01:/mnt/glusterfs/ngcpcfg-share. There is no sense in such a complication so it was changed to use ${mgmt}:/mnt/glusterfs/ngcpcfg-share for all installation types and nodes. Change-Id: If84f70da51739a89763fb54091faaab9e15ffc9c	5 years ago
Victor Seva	24098da128	TT#83358 status: add test * bin/ngcpcfg: SKIP_RESTORE_PERMS so we can skip that on tests * scripts/status: use NGCPCTL_BASE and NGCPCTL_MAIN so it would work in test environment Change-Id: I8ba80bbb2ea3adf1b8cc2a4229568a87f9508cbd	5 years ago
Michael Prokop	0ce3b7b110	TT#120901 Support ShellCheck v7.0.2 with new SC2260 check SC2259/SC2260 were added in ShellCheck v7.0.2 ("Warn when redirections override pipes") Fixes: \| In scripts/commit line 91: \| ngcp-sync-constants >/dev/null \| sed "s/^/$timestamp_replacementchars/" \| ^-- SC2260: This redirection overrides the output pipe. Use 'tee' to output to both. While at it, fix incomplete log_debug regarding no-db-sync, thanks Alexander Lutay <alutay@sipwise.com> for spotting. Change-Id: I92e5ce73dc000a52fa7c5c79f710d87cec292908	5 years ago
Alexander Lutay	567318c578	TT#106202 Add tty check for ngcpcfg 'read' calls On a Carrier cluster setup the command 'ngcpcfg clean --all' can be executed on several nodes in parallel (e.g. using ngcp-parallel-ssh). Which can get stuck on the yes/no question if the operator forgot to use '--force'. In practice it causes 100% CPU usage after several days due to logs rotation (reason is not 100% clear). Strace for such case: > read(0, "", 1) = 0 > select(1, [0], NULL, [0], {tv_sec=0, tv_usec=0}) = 1 (in [0], left {tv_sec=0, tv_usec=0}) > rt_sigprocmask(SIG_BLOCK, [CHLD], [], 8) = 0 > rt_sigprocmask(SIG_SETMASK, [], NULL, 8) = 0 > ioctl(0, TCGETS, 0x7fff317e39a0) = -1 ENOTTY (Inappropriate ioctl for device) > lseek(0, 0, SEEK_CUR) = -1 ESPIPE (Illegal seek) > read(0, "", 1) = 0 > select(1, [0], NULL, [0], {tv_sec=0, tv_usec=0}) = 1 (in [0], left {tv_sec=0, tv_usec=0}) > rt_sigprocmask(SIG_BLOCK, [CHLD], [], 8) = 0 > rt_sigprocmask(SIG_SETMASK, [], NULL, 8) = 0 > ioctl(0, TCGETS, 0x7fff317e39a0) = -1 ENOTTY (Inappropriate ioctl for device) > lseek(0, 0, SEEK_CUR) = -1 ESPIPE (Illegal seek) > read(0, "", 1) = 0 > select(1, [0], NULL, [0], {tv_sec=0, tv_usec=0}) = 1 (in [0], left {tv_sec=0, tv_usec=0}) > rt_sigprocmask(SIG_BLOCK, [CHLD], [], 8) = 0 > rt_sigprocmask(SIG_SETMASK, [], NULL, 8) = 0 > ioctl(0, TCGETS, 0x7fff317e39a0) = -1 ENOTTY (Inappropriate ioctl for device) > lseek(0, 0, SEEK_CUR) = -1 ESPIPE (Illegal seek) > read(0, "", 1) = 0 > select(1, [0], NULL, [0], {tv_sec=0, tv_usec=0}) = 1 (in [0], left {tv_sec=0, tv_usec=0}) > rt_sigprocmask(SIG_BLOCK, [CHLD], [], 8) = 0 > rt_sigprocmask(SIG_SETMASK, [], NULL, 8) = 0 > ioctl(0, TCGETS, 0x7fff317e39a0) = -1 ENOTTY (Inappropriate ioctl for device) > lseek(0, 0, SEEK_CUR) = -1 ESPIPE (Illegal seek) > read(0, "", 1) = 0 > select(1, [0], NULL, [0], {tv_sec=0, tv_usec=0}) = 1 (in [0], left {tv_sec=0, tv_usec=0}) > rt_sigprocmask(SIG_BLOCK, [CHLD], [], 8) = 0 > rt_sigprocmask(SIG_SETMASK, [], NULL, 8) = 0 > ioctl(0, TCGETS, 0x7fff317e39a0) = -1 ENOTTY (Inappropriate ioctl for device) > lseek(0, 0, SEEK_CUR) = -1 ESPIPE (Illegal seek) > read(0, "", 1) = 0 > select(1, [0], NULL, [0], {tv_sec=0, tv_usec=0}) = 1 (in [0], left {tv_sec=0, tv_usec=0}) > rt_sigprocmask(SIG_BLOCK, [CHLD], [], 8) = 0 > rt_sigprocmask(SIG_SETMASK, [], NULL, 8) = 0 > ioctl(0, TCGETS, 0x7fff317e39a0) = -1 ENOTTY (Inappropriate ioctl for device) > lseek(0, 0, SEEK_CUR) = -1 ESPIPE (Illegal seek) > read(0, "", 1) = 0 The idea here is to check tty invocation and skip questions if no tty found. JFYI, there is no tty when running scripts under ngcp-parallel-ssh. Change-Id: Ia8ded3ada4d48ecb374f36859161645776381c31	5 years ago
Alexander Lutay	45de0f4e8a	TT#120752 Fix b821e00: allow 'ngcpcfg edit /path/real_file.yml' The commit addresses those issues: > root@spce:~# ngcpcfg edit /etc/ngcp-config/config.yml > Error: Unknown choice '/etc/ngcp-config/config.yml'. Aborting. > > root@spce:~# ngcpcfg edit /etc/ngcp-config/config.yml.factory_default > Error: Unknown choice '/etc/ngcp-config/config.yml.factory_default'. Aborting. > > root@spce:~# Also fixed typo in error message. Change-Id: I2dbf1667ace5eb481b47cf63224a53b5586b4bf0	5 years ago
Alexander Lutay	6a733861b0	TT#120752 Fix b821e00: move main function load close to validation It produces no functional changes but align code better with Sipwise/ngcpcfg style. We normally check the file availability and load the file the next line. Change-Id: Ifebddc452f7701fd02a85a8d3273edc89b7fd21a	5 years ago
Alexander Lutay	b821e0052b	TT#120752 Add 'ngcpcfg edit' to speedup YML editing Normally it is necessary to press TAB multiple times, e.g: > vim /etc/ngcp-conf<TAB>/con<TAB>f<TAB>... For new NGCP users it is even longer. In the same time we have a nice usability experience for ngcp-ppa, where users are just pressing "ngcp-ppa<ENTER>1<ENTER>1<ENTER>". Let's add the same concept here as the order of the main files is static: > 1) /etc/ngcp-config/config.yml > 2) /etc/ngcp-config/network.yml > 3) /etc/ngcp-config/constants.yml if I need to edit network.yml I will type: > ngcpc<TAB> e<TAB><ENTER>2<ENTER> or even: > ngcpc<TAB> e<TAB>2<ENTER> Change-Id: Id20fec8c931c15c73a73689a3f8b552a72ef11f7	5 years ago
Michael Prokop	fbbe998e98	TT#119602 Properly handle trap also in case of errors in functions Quoting from "man bash" about `-E` (AKA errtrace): \| If set, any trap on ERR is inherited by shell functions, command \| substitutions, and commands executed in a subshell environment. \| The ERR trap is normally not inherited in such cases. To demonstrate the problem see this short shell script: \| % cat foo \| set -eu -o pipefail \| \| bailout() { \| echo "Bailing out because of error" >&2 \| exit 1 \| } \| trap bailout 1 2 3 6 9 14 15 ERR \| \| foo() { \| echo "Executing magic" \| magic \| } \| \| foo \| echo end If "magic" can't be executed, then this fails as follows: \| % bash ./foo \| Executing magic \| ./foo: line 11: magic: command not found But it doesn't invoke the bailout function via trap. When using `set -eE` (AKA errexit + errtrace), instead of only `set -e` (errexit), then it behaves as expected though: \| % bash ./foo \| Executing magic \| ./foo: line 11: magic: command not found \| Bailing out because of error Change-Id: I9f05355716551cd8fc8f22bd2fa16bbc9d0b7247	5 years ago
Manuel Montecelo	5bc669a61e	TT#80164 Remove extra '}' at the end of debug message Change-Id: I0c09d5ea02220d352629842df92ea13cca6a73b1	5 years ago

1 2 3 4

178 Commits (a134907dc7292a832540da8f239b6f629e00cecf)