-
Notifications
You must be signed in to change notification settings - Fork 0
Update configuration files to support ARM64 architecture #306
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Added 'hosts_arm64' entries in builder_tests_config.json and desktop_tests_config.json for ARM64 compatibility. - Updated 'chat_id_file' in both configuration files to specify default values. - Enhanced template definitions in templates.json to include ARM64 package paths. - Modified test data classes to utilize ARM64 host configurations based on architecture detection. These changes improve support for ARM64 environments in the testing framework.
…dule - Introduced properties for home_dir and path_module to improve lazy loading and encapsulation. - Removed direct initialization of path_module and home_dir in the constructor. - Enhanced compatibility with different operating systems by utilizing HostInfo for path resolution.
…onality - Simplified the RemotePaths class by removing the path_module property and directly using the join function for path joining. - Enhanced VboxUtilsWindows by introducing constants for command-line executables and streamlining the directory creation process. - Updated the _create_dir method to handle path formatting based on the shell type, improving compatibility with Windows command-line tools.
…rides - Updated the VmConfig class to allow loading and merging of VM-specific configurations. - Introduced new Pydantic models for partial network and VM-specific configurations. - Refactored the configuration loading process to support a complete configuration structure. - Updated the JSON configuration file to include default and VM-specific settings for various Windows versions.
- Removed hardcoded memory allocation for macOS, allowing for more flexible VM memory configuration based on the vm_config settings. - Updated the _get_memory_num method to streamline memory retrieval for virtual machines.
…nd improve readability.
…alue for improved configuration clarity.
…ase version - Changed start_hour from 2 to 0 and end_hour from 12 to 23 to allow for a full 24-hour scheduling capability. - Updated base_version from 9.1.0 to 9.3.0 to reflect the latest version.
…ta classes - Added source URL for the sdist of the 'aenum' package in uv.lock. - Updated the git source reference for 'host-tools' in uv.lock. - Refactored architecture detection in BuilderTestData and DesktopTestData classes to use HostInfo().is_arm for improved clarity and consistency.
…ass based on architecture - Added a check for ARM architecture using HostInfo to conditionally apply speculative execution control during VM configuration, enhancing compatibility for ARM hosts.
…indows and Debian - Added 'debian12_arm64' to the Debian OS family. - Included 'windows11_arm64' in the Windows OS family to enhance compatibility for ARM64 environments.
- Reformatted the JSON structure in both builder_tests and desktop_tests to enhance clarity by aligning array elements on separate lines. - Maintained existing entries while improving the overall organization of the configuration files.
…rity - Modified error messages to include the package name in the output when it is not found in packages_config.json, enhancing the debugging experience.
- Modified the host filtering logic to return all hosts without architecture-based filtering. - Updated the _get_hosts method to dynamically select host lists based on architecture, enhancing compatibility with different environments.
…ty and maintainability - Simplified the merging of network configurations by removing unnecessary comments and streamlining the logic. - Enhanced the handling of VM-specific configurations, clarifying the process for updating or creating configurations based on the presence of vm_name. - Cleaned up comments to focus on essential operations, improving overall readability of the code.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These changes improve support for ARM64 environments in the testing framework.