Skip to content

Add tests for Loader Driver interaction conformance for Sysman VF Management API's#360

Merged
nrspruit merged 4 commits into
oneapi-src:masterfrom
vishnu-khanth:ldr_driver_interaction
Jul 30, 2025
Merged

Add tests for Loader Driver interaction conformance for Sysman VF Management API's#360
nrspruit merged 4 commits into
oneapi-src:masterfrom
vishnu-khanth:ldr_driver_interaction

Conversation

@vishnu-khanth

@vishnu-khanth vishnu-khanth commented Jul 17, 2025

Copy link
Copy Markdown
Contributor

Related-To: NEO-15334

@vishnu-khanth vishnu-khanth marked this pull request as ready for review July 18, 2025 05:05
@vishnu-khanth vishnu-khanth force-pushed the ldr_driver_interaction branch from e2cdb8e to ec50c9c Compare July 21, 2025 12:34
nrspruit
nrspruit previously approved these changes Jul 21, 2025
View reviewed changes

@nrspruit nrspruit left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@vishnu-khanth

vishnu-khanth commented Jul 21, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Now with this null driver update, will add tests for all L0 API's (Core, Sysman & Tools) in a separate PR

@vishnu-khanth

vishnu-khanth commented Jul 21, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Just did a minor refactor. Thanks!

@vishnu-khanth vishnu-khanth marked this pull request as draft July 22, 2025 06:38
@vishnu-khanth

vishnu-khanth commented Jul 24, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Right now keeping it in draft bcos exploring if any other better method exists which can be extensible for other validation like structs in/out.
E.g: Instead of env, writing to a file etc...

Will finalize before merging this & going ahead for updating all API's

@vishnu-khanth vishnu-khanth marked this pull request as ready for review July 30, 2025 13:50
@vishnu-khanth
Add tests for Loader Driver interaction conformance for Sysman VF Man…
f20421e 
…agement API's

Related-To: NEO-15334

Signed-off-by: Vishnu Khanth <vishnu.khanth.b@intel.com>
@vishnu-khanth
Add tests for Loader Driver interaction conformance for Sysman VF Man…
1868495 
…agement API's

Related-To: NEO-15334

Signed-off-by: Vishnu Khanth <vishnu.khanth.b@intel.com>
@vishnu-khanth
Add tests for Loader Driver interaction conformance for Sysman VF Man…
9169323 
…agement API's

Related-To: NEO-15334

Signed-off-by: Vishnu Khanth <vishnu.khanth.b@intel.com>
@vishnu-khanth
Add tests for Loader Driver interaction conformance for Sysman VF Man…
67d1197 
…agement API's

Related-To: NEO-15334

Signed-off-by: Vishnu Khanth <vishnu.khanth.b@intel.com>
@vishnu-khanth vishnu-khanth force-pushed the ldr_driver_interaction branch from 35de06d to 67d1197 Compare July 30, 2025 13:51
@vishnu-khanth

vishnu-khanth commented Jul 30, 2025

Copy link
Copy Markdown
Contributor Author

Right now keeping it in draft bcos exploring if any other better method exists which can be extensible for other validation like structs in/out. E.g: Instead of env, writing to a file etc...

Will finalize before merging this & going ahead for updating all API's

Finalized the approach. This approach itself looks good to me

@vishnu-khanth

vishnu-khanth commented Jul 30, 2025

Copy link
Copy Markdown
Contributor Author

Rebased the PR to get the latest changes.
@nrspruit PR is ready to merge once the checks are passed. Thanks!

@nrspruit nrspruit merged commit e121961 into oneapi-src:master Jul 30, 2025
50 checks passed
@vishnu-khanth vishnu-khanth deleted the ldr_driver_interaction branch July 30, 2025 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nrspruit nrspruit nrspruit approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vishnu-khanth @nrspruit