Message ID | 20220503220646.16925-1-mwilck@suse.com (mailing list archive) |
---|---|
Headers | show |
Series | multipath-tools: fix unit test breakage | expand |
On Wed, May 04, 2022 at 12:06:39AM +0200, mwilck@suse.com wrote: > From: Martin Wilck <mwilck@suse.com> > > Hi Christophe, hi Ben, > > The previous patches > > af15832 multipath-tools: make multipath_dir a compiled-in option > 1fc7c4d multipath-tools: make config_dir a compiled-in option > 21b3d6b multipath-tools: stop supporting getuid_callout > > have caused breakage in the unit tests. Not so much because they'd > actually break the tests, but because they broke the compilation and > the search paths for the configuration files (for the former two commits) > and because the hwtable test relied on being able to the retrieve > the now removed "getuid_callout" property from config files. > > Fix it. As always, comments welcome. For the set, Reviewed-by: Benjamin Marzinski <bmarzins@redhat.com> > > Regards > Martin > > Martin Wilck (7): > multipath-tools: enable local configdir and plugindir for unit tests > tests/mpathvalid: fix check for DEFAULT_CONFIG_FILE > multipath-tools tests: hwtable: adapt after removal of getuid > tests/hwtable: fix test_regex_2_strings_hwe_dir > libmultipath: fix valgrind_test > tests/Makefile: make TESTDIR configurable > github workflows: foreign.yaml: fix config dir in run environment > > .github/workflows/foreign.yaml | 7 +- > libmultipath/Makefile | 24 ++++-- > libmultipath/config.c | 3 + > tests/Makefile | 12 ++- > tests/hwtable.c | 151 ++++++++++++++++----------------- > tests/mpathvalid.c | 2 +- > tests/test-lib.c | 89 ++++++++++++++----- > tests/test-lib.h | 2 +- > 8 files changed, 180 insertions(+), 110 deletions(-) > > -- > 2.36.0 -- dm-devel mailing list dm-devel@redhat.com https://listman.redhat.com/mailman/listinfo/dm-devel
From: Martin Wilck <mwilck@suse.com> Hi Christophe, hi Ben, The previous patches af15832 multipath-tools: make multipath_dir a compiled-in option 1fc7c4d multipath-tools: make config_dir a compiled-in option 21b3d6b multipath-tools: stop supporting getuid_callout have caused breakage in the unit tests. Not so much because they'd actually break the tests, but because they broke the compilation and the search paths for the configuration files (for the former two commits) and because the hwtable test relied on being able to the retrieve the now removed "getuid_callout" property from config files. Fix it. As always, comments welcome. Regards Martin Martin Wilck (7): multipath-tools: enable local configdir and plugindir for unit tests tests/mpathvalid: fix check for DEFAULT_CONFIG_FILE multipath-tools tests: hwtable: adapt after removal of getuid tests/hwtable: fix test_regex_2_strings_hwe_dir libmultipath: fix valgrind_test tests/Makefile: make TESTDIR configurable github workflows: foreign.yaml: fix config dir in run environment .github/workflows/foreign.yaml | 7 +- libmultipath/Makefile | 24 ++++-- libmultipath/config.c | 3 + tests/Makefile | 12 ++- tests/hwtable.c | 151 ++++++++++++++++----------------- tests/mpathvalid.c | 2 +- tests/test-lib.c | 89 ++++++++++++++----- tests/test-lib.h | 2 +- 8 files changed, 180 insertions(+), 110 deletions(-)