[mithileshreddy04]

I'll check the actual working-tree state so the file list is accurate.

Only one file differs from HEAD, so the other changes may already be committed. I'll check the recent history.

The history shows a fold commit f7debbe9f, which includes the role file. The role file is now identical to its pre-redirect version, so I'll check the net diff against the base commit to get the real file count.

Net of the base commit 8acb63820, the PR changes two files. I'll write the description for those two only. The artifacts sweep is a third, uncommitted change, so I'm leaving it out.

Description of the Solution

The IPoIB input FVT validates each case against a scratch project named omnia_fvt_ipoib_<hex>. The validator writes its log under <OMNIA_DATA_PATH>/orchestrator/log/<scratch>/, and the teardown removed only the scratch input and output. Every run therefore left a new log/omnia_fvt_ipoib_<hex>/ directory that blocks omnia cleanup --all.

This PR changes only the test code. After each validation, the FVT copies the scratch log into the active project's single validation log, <OMNIA_DATA_PATH>/orchestrator/log/<project>/orchestrator_validation_<project>.log. The first case of a session rewrites that file, and later cases append to it, each under a header naming the scratch project. The teardown removes the scratch input, output, and log directories. The aggregate file is kept, and the next run rewrites it.

Related Issue

• Fixes #

Changes

IPoIB input FVT (test/orchestrator/library/functions/ipoib_inputs_precheck_func.py)

• _read_live returns a log_root key, derived from the resolved input directory as <shared>/log.
• New _append_remote writes base64 chunks, truncating on the first write and appending after.
• New _fold_scratch_log copies log/<scratch>/orchestrator_validation_<scratch>.log into log/<project>/orchestrator_validation_<project>.log. It creates the project directory with mkdir -p if needed, and returns False when the scratch log is absent.
• New module flag _AGGREGATE_LOG_STARTED decides whether the next write truncates or appends. It is set only after a log has been written.
• _run_validate runs the clone and validation as before, folds the scratch log, and removes the scratch input, output, and log directories in the finally block.

IPoIB FVT constants (test/orchestrator/library/vars/ipoib_vars.py)

• Adds the make_parent_dirs command (mkdir -p -- %s).

Files Changed

File	Change Type	Description
test/orchestrator/library/functions/ipoib_inputs_precheck_func.py	Modified	Folds each scratch validation log into the active project's log file, and removes the scratch directories at teardown.
test/orchestrator/library/vars/ipoib_vars.py	Modified	Adds the make_parent_dirs command.

Testing

• Not run: Python and Ansible are not available in the development shell, so the change was checked by reading the code only.
• Required before merge, on a test OIM:
  • Run ./run_validation.sh fvt_orchestrator precheck verify --suite ipoib_inputs.
  • Confirm that log/<project>/orchestrator_validation_<project>.log contains a header and output for each case.
  • Confirm that no omnia_fvt_ipoib_* directory remains under /opt/omnia/orchestrator/log/.
  • Confirm that log/<project>/ is rewritten, not appended to, at the start of a new run.

Backward Compatibility

• No product code changes. The validator and role are unchanged, so deploy, precheck, and validate behave as before.
• The aggregate file sits at the active project's standard log path. A later real validation of that project overwrites it, which is expected.
• If an interrupted run never reaches teardown, its scratch directory stays under orchestrator/log/ and blocks cleanup --all until removed.
• Existing omnia_fvt_ipoib_* directories from earlier runs are not removed by this PR and need manual cleanup.