Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use bionic repo for all supported ubuntu images when installing azure… #2871

Merged
merged 1 commit into from
Aug 8, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 27 additions & 15 deletions lisa/operating_system.py
Original file line number Diff line number Diff line change
Expand Up @@ -464,7 +464,9 @@ def get_package_information(
def get_repositories(self) -> List[RepositoryInfo]:
raise NotImplementedError("get_repositories is not implemented")

def add_azure_core_repo(self, repo_name: Optional[AzureCoreRepo] = None) -> None:
def add_azure_core_repo(
self, repo_name: Optional[AzureCoreRepo] = None, code_name: Optional[str] = None
) -> None:
raise NotImplementedError("add_azure_core_repo is not implemented")

def _process_extra_package_args(self, extra_args: Optional[List[str]]) -> str:
Expand Down Expand Up @@ -798,7 +800,9 @@ def _get_package_information(self, package_name: str) -> VersionInfo:
)
return self._cache_and_return_version_info(package_name, version_info)

def add_azure_core_repo(self, repo_name: Optional[AzureCoreRepo] = None) -> None:
def add_azure_core_repo(
self, repo_name: Optional[AzureCoreRepo] = None, code_name: Optional[str] = None
) -> None:
arch = self.get_kernel_information().hardware_platform
arch_name = "arm64" if arch == "aarch64" else "amd64"

Expand All @@ -815,21 +819,22 @@ def add_azure_core_repo(self, repo_name: Optional[AzureCoreRepo] = None) -> None
# azure-compatscanner. Add azurecore from Ubuntu bionic instead
# 2. azurezcore-debian only supports buster. For other versions,
# use azurecore instead
codename = "bionic"
code_name = "bionic"
repo_name = AzureCoreRepo.AzureCore
# If it's architecture is aarch64, azurecore-multiarch repo is also needed
if arch == "aarch64":
repo_url = f"http://packages.microsoft.com/repos/{AzureCoreRepo.AzureCoreMultiarch}/" # noqa: E501
repo_url = "http://packages.microsoft.com/repos/azurecore-multiarch/"
self.add_repository(
repo=(f"deb [arch={arch_name}] {repo_url} {codename} main"),
repo=(f"deb [arch={arch_name}] {repo_url} {code_name} main"),
keys_location=keys,
)
else:
codename = self.information.codename
code_name = self.information.codename
repo_name = AzureCoreRepo.AzureCoreDebian

repo_url = f"http://packages.microsoft.com/repos/{repo_name}/"
repo_url = f"http://packages.microsoft.com/repos/{repo_name.value}/"
self.add_repository(
repo=(f"deb [arch={arch_name}] {repo_url} {codename} main"),
repo=(f"deb [arch={arch_name}] {repo_url} {code_name} main"),
keys_location=keys,
)

Expand Down Expand Up @@ -1226,24 +1231,27 @@ def _get_information(self) -> OsInformation:

return information

def add_azure_core_repo(self, repo_name: Optional[AzureCoreRepo] = None) -> None:
def add_azure_core_repo(
self, repo_name: Optional[AzureCoreRepo] = None, code_name: Optional[str] = None
) -> None:
arch = self.get_kernel_information().hardware_platform
arch_name = "arm64" if arch == "aarch64" else "amd64"
codename = self.information.codename
if not code_name:
code_name = self.information.codename
repo_url = "http://packages.microsoft.com/repos/azurecore/"
self.add_repository(
repo=(f"deb [arch={arch_name}] {repo_url} {codename} main"),
repo=(f"deb [arch={arch_name}] {repo_url} {code_name} main"),
keys_location=[
"https://packages.microsoft.com/keys/microsoft.asc",
"https://packages.microsoft.com/keys/msopentech.asc",
],
)
# If its architecture is aarch64 for bionic and xenial,
# azurecore-multiarch repo is also needed
if arch == "aarch64" and (codename == "bionic" or codename == "xenial"):
if arch == "aarch64" and (code_name == "bionic" or code_name == "xenial"):
repo_url = "http://packages.microsoft.com/repos/azurecore-multiarch/"
self.add_repository(
repo=(f"deb [arch={arch_name}] {repo_url} {codename} main"),
repo=(f"deb [arch={arch_name}] {repo_url} {code_name} main"),
keys_location=[
"https://packages.microsoft.com/keys/microsoft.asc",
"https://packages.microsoft.com/keys/msopentech.asc",
Expand Down Expand Up @@ -1372,7 +1380,9 @@ def add_repository(
) -> None:
self._node.tools[YumConfigManager].add_repository(repo, no_gpgcheck)

def add_azure_core_repo(self, repo_name: Optional[AzureCoreRepo] = None) -> None:
def add_azure_core_repo(
self, repo_name: Optional[AzureCoreRepo] = None, code_name: Optional[str] = None
) -> None:
self.add_repository("https://packages.microsoft.com/yumrepos/azurecore/")

def _get_package_information(self, package_name: str) -> VersionInfo:
Expand Down Expand Up @@ -1869,7 +1879,9 @@ def add_repository(
else:
self._log.debug(f"repo {repo_name} already exist")

def add_azure_core_repo(self, repo_name: Optional[AzureCoreRepo] = None) -> None:
def add_azure_core_repo(
self, repo_name: Optional[AzureCoreRepo] = None, code_name: Optional[str] = None
) -> None:
self.add_repository(
repo="https://packages.microsoft.com/yumrepos/azurecore/",
repo_name="packages-microsoft-com-azurecore",
Expand Down
8 changes: 6 additions & 2 deletions microsoft/testsuites/tvm/tvmsuite.py
Original file line number Diff line number Diff line change
Expand Up @@ -71,10 +71,14 @@ def verify_secureboot_compatibility(self, node: Node) -> None:
def verify_measuredboot_compatibility(self, node: Node) -> None:
self._is_supported(node)
posix_os: Posix = cast(Posix, node.os)
if isinstance(posix_os, Debian):
if isinstance(posix_os, Ubuntu):
# focal and jammy don't have azure-compatscanner package in azurecore repo
# use bionic for temp solution
posix_os.add_azure_core_repo(code_name="bionic")
elif isinstance(posix_os, Debian):
# azurecore-debian doesn't have azure-compatscanner package
# use azurecore instead
posix_os.add_azure_core_repo(AzureCoreRepo.AzureCore)
posix_os.add_azure_core_repo(repo_name=AzureCoreRepo.AzureCore)
else:
posix_os.add_azure_core_repo()
posix_os.install_packages("azure-compatscanner", signed=False)
Expand Down
Loading