Skip to content

Commit

Permalink
Set USE_MS_GUEST_KERNEL for CLH testcase only if YES (#3074)
Browse files Browse the repository at this point in the history
We will set the env var USR_MS_GUEST_KERNEL only
if it is set to YES as new CLH dev_cli script had some
changes which expect this env var to be set only
when required. Setting default value to NO would not
work with it anymore.

Signed-off-by: Smit Gardhariya <[email protected]>
  • Loading branch information
smit-gardhariya authored Nov 28, 2023
1 parent d48daea commit 781d7b9
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion microsoft/testsuites/cloud_hypervisor/ch_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,8 @@ def _set_ms_clh_param(self, variables: Dict[str, Any]) -> None:
CloudHypervisorTests.ms_clh_repo = ms_clh_repo
CloudHypervisorTests.ms_igvm_parser_repo = ms_igvm_parser_repo
CloudHypervisorTests.clh_guest_vm_type = clh_guest_vm_type
CloudHypervisorTests.use_ms_guest_kernel = use_ms_guest_kernel
if use_ms_guest_kernel == "YES":
CloudHypervisorTests.use_ms_guest_kernel = use_ms_guest_kernel


def get_test_list(variables: Dict[str, Any], var1: str, var2: str) -> Any:
Expand Down
3 changes: 2 additions & 1 deletion microsoft/testsuites/cloud_hypervisor/ch_tests_tool.py
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,8 @@ def _install(self) -> bool:
auth_token=self.ms_access_token,
)
self.env_vars["GUEST_VM_TYPE"] = self.clh_guest_vm_type
self.env_vars["USE_MS_GUEST_KERNEL"] = self.use_ms_guest_kernel
if self.use_ms_guest_kernel:
self.env_vars["USE_MS_GUEST_KERNEL"] = self.use_ms_guest_kernel
else:
git.clone(self.upstream_repo, clone_path)

Expand Down

0 comments on commit 781d7b9

Please sign in to comment.