DAOS-10301 test: Wait for dmg completion (#9889)

[kjacque]

• In daos_test, ensure dmg completes before we read output from stdout.
• Add optional dmg command line parameter to log to a file. This isn't a config option -- we don't want to log unless the user decides to do so.
• Improve debug logging for tests_dmg_helpers.c functions. This includes output from stderr.
• Add missing pclose calls to other test helper functions.
• Ensure DB_TEST logging is included in client test log.

Signed-off-by: Kris Jacque [email protected]

[github-actions[bot]]

Bug-tracker data: Ticket title is 'daos_test/suite.py:DaosCoreTest.test_daos_pool - POOL14: pool connect access based on ACL' Status is 'Awaiting backport' Labels: 'ci_impact,daily_test,pr_test,triaged' Job should run at elevated priority (1) https://daosio.atlassian.net/browse/DAOS-10301

[kjacque]

The failure appears to be DAOS-11180, which had a fix backported to release/2.2 over the weekend.

[kjacque]

@daos-stack/daos-gatekeeper Requesting force landing, since the only failure was the already-fixed issue above.

[daosbuild1]

Test stage checkpatch completed with status FAILURE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-9965/4/execution/node/130/log

[daosbuild1]

Test stage checkpatch completed with status FAILURE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-9965/5/execution/node/130/log

[mjmac]

@kjacque, @phender: I think we should limit this PR to the original changes on master plus whatever is necessary to get it applied on 2.2. Other improvements should happen in follow-up PRs.

[kjacque]

Reverting back the test changes to simplify this backport. I've created a new ticket to address @phender's comments: https://daosio.atlassian.net/browse/DAOS-11394