Merge with upstream - #5
Open
sprasad-microsoft wants to merge 46 commits into
Open
Conversation
* Initial plan for issue * Add private network configuration to azure.yml runbook Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Fix use_public_address default value to true Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Add create_public_address variable to azure.yml runbook Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Add fallback to default values for empty subnet and vnet names Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com>
Add hook `azure_update_vm_capabilities` to allow editing of Azure VM SKU capabilities before being saved to cache. This will allow capability information to be inserted by extensions for internal SKUs or the processing of additional SKU information such as disk and network performance. Alternatively, SKUs can be removed from the results to restrict the SKUs that LISA will deploy.
…soft#3839) * Initial plan for issue * Add source_address_prefixes parameter to azure.yml runbook with empty default Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Support string input for source_address_prefixes parameter Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com> * Fix flake8 errors: remove trailing whitespace and use double quotes Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Update documentation for source_address_prefixes parameter Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Improve source_address_prefixes parsing and simplify documentation - Enhanced _get_ip_addresses method with ast.literal_eval for robust string parsing - Added proper error handling for malformed list strings - Simplified documentation examples to show string and list formats only - Removed variable file example as requested Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com> Co-authored-by: LiliDeng <lildeng@microsoft.com>
If the capabilities dictionary is empty for any reason, the hook implementation would have no way to know which region is being queried.
If Modprobe fails to remove a module, run some additional commands to debug why it failed
make detection accurate for root not allowed in /bin/sh
Add testcase to verify crash functionality having crash generated from mshv. Separate the util functions used for kdump and Re-use kdump setup of LISA to run the test and verify if crashdump is generated. Signed-off-by: Smit Gardhariya <sgardhariya@microsoft.com>
* Added openSSL tool and testsuites to lay ground work for encryption and decryption tests * Fixing tuple type error in actions * made helper method private and added more to test description * Removed duplicated line * fixed comment
…kages (microsoft#3851) * tool: add createrepo tool Signed-off-by: Thien Trung Vuong <tvuong@microsoft.com> * test/cvm_boot: optionally add package repository before upgrading packages Signed-off-by: Thien Trung Vuong <tvuong@microsoft.com> --------- Signed-off-by: Thien Trung Vuong <tvuong@microsoft.com>
With older baremetal server, lscpu wont expose virtualisation feature details (VT-x or AMD-V) This will cause CH test to be skipped when runs on Ubuntu-KVM with older generation windows machine. Fix it by checking if /dev/kvm exist Signed-off-by: Smit Gardhariya <sgardhariya@microsoft.com>
* reboot stress test * indentations * flake * median, info changed to debug * flake - long line * flake * reboot stress test * indentations * flake * median, info changed to debug * flake - long line * flake * Review comment resolve
…crosoft#3872) * Initial plan for issue * Create .github/copilot-instructions.md with optimized PR test suggestions Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com> * Address feedback on copilot instructions - clarify test case selection and fix LISA Features terminology Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com> * Update copilot instructions to use "ensure adequate test coverage" instead of "test plan" Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com> * Provide comprehensive explanation of LISA Feature class concept Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com>
…reation and clarify UT exclusion
…ft#3863) * Initial plan for issue * Add unified performance message support to ntttcp tool Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Follow PR microsoft#3862 pattern for unified performance message support Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Add MetricRelativity support to ntttcp unified performance messages Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Refactor ntttcp unified perf messages to reduce code duplication Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Move unified perf message calls into create methods Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com> * Set protocol_type for unified performance messages Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Set default protocol_type to None in PerfMessage Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com> * Fix flake8 formatting errors - remove whitespace and wrap long lines Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Add MetricUnit support to ntttcp unified performance metrics Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Fix metric units: change buffer_size from bytes to KB Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Remove units from rx_packets, tx_packets, and pkts_interrupts metrics Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Set protocol_type as a metric instead of PerfMessage field Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Revert protocol_type as metric - keep it in PerfMessage base class Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Add protocol_type field to TCP performance message Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Set protocol type for different perf messages * add default protocol_type for NetworkLatencyPerformanceMessage * Remove unused TransportProtocol import from iperf3.py Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com> Co-authored-by: Lili Deng <lildeng@microsoft.com>
* Added openSSL tool and testsuites to lay ground work for encryption and decryption tests Fixing tuple type error in actions made helper method private and added more to test description Removed duplicated line Added Go test added Openssl speed test removed the speed test to seprate PRs * fixed install package operating systems error * renamed test method * Fixed casting and namming, merged co methods * changed path to purepath and removed unused logging
Add a test to check the size of logfile generated by mshvlog service. Signed-off-by: Praveen K Paladugu <prapal@linux.microsoft.com>
…on fields Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com>
A testcase to verify the enablement of essential non Hyper-V kernel modules.
Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com>
* Fix verify_ifcfg_eth0 for fedora 39 & above This testcase failes for the latest fedora 42. For Fedora 39 and above, fedora uses NetworkManager for the network management. The legacy /etc/sysconfig/network-scripts/ directory and ifcg-* files are deprecated. Tested with azure and the testcase is passing. Signed-off-by: Bala Konda Reddy M <bala12352@gmail.com> * Fix the flake8 errors --------- Signed-off-by: Bala Konda Reddy M <bala12352@gmail.com>
…ure_deploy transformer (microsoft#3855) * Initial plan for issue * Add support for source_address_prefixes and resource_group_name in azure_deploy transformer Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Update source_address_prefixes type to match Azure platform schema Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Remove unnecessary test file per feedback Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Fix CI linting errors: format long line in azure transformers Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Add documentation for source_address_prefixes field in azure_deploy transformer Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Add deploy field to azure_deploy transformer usage example in documentation Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Add clarifying comments for resource_group_name and source_address_prefixes in documentation Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com> * Add clarification that resource_group_name only works for new deployments Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com>
Fix features to build cloud-hypervisor. Latest CH uses sev_snp feature. Signed-off-by: Smit Gardhariya <sgardhariya@microsoft.com>
* Add option to filter for kernel logs in journalctl tool Introduces Journalctl.logs_for_kernel() which retrieves kernel logs with journalctl -k. Signed-off-by: Cameron Baird <cameronbaird@microsoft.com> * Implement verify_mshv_secure_boot_succeeds Introduce a test which consumes mshv components "mshv_rpm_tarball" to stand up a Dom0 image. Checks: 1. We come up with secure boot successfully 2. The mshv driver is running Signed-off-by: Cameron Baird <cameronbaird@microsoft.com> * Invert Hyper-V SB check to address lack of secure boot signed components * Update microsoft/testsuites/mshv/mshv_secure_boot.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Signed-off-by: Cameron Baird <cameronbaird@microsoft.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…3876) * Add VMM/MSHV version info in azure and baremetal platform * Add common utils * remove information * Fix black test failure * refactor: use loop-based approach for environment information hooks in BareMetalPlatform * move constants to platform_utils * fix check error
Subtest failure of `verify_rust_vmm_mshv_tests` results in an assertion error for the entire test case. The cargo tool implementation for executing the test itself asserts a non zero exit code & results in cargo test failure which hides the subtest failure. To control whether the test should assert on command exit code, make the exit code params optional. This ensures that the subtests are individually reported as failed instead of throwing an assert error on the entire test case. Signed-off-by: AASTHA RAWAT <aastharawat@microsoft.com> Co-authored-by: AASTHA RAWAT <aastharawat@microsoft.com>
…include it when combinators are used; otherwise, use the base test case name. (microsoft#3884) * Initial plan for issue * Add configurable append_message_id option to JUnit notifier Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com> * Fix flake8 linting errors: remove trailing whitespace Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com> * Add JUnit notifier documentation to runbook.rst Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: squirrelsc <27178119+squirrelsc@users.noreply.github.com> Co-authored-by: LiliDeng <10083705+LiliDeng@users.noreply.github.com>
…oft#3892) * Fix verify_network_file_configuration for fedora 39 and above For fedora 39 and above network script file is present in /etc/NetworkManager/system-connections/ and uses nmcli, Made sure it works for previous versions Signed-off-by: Bala Konda Reddy M <bala12352@gmail.com> * Fix verify_network_file_configuration for fedora 39 and above For fedora 39 and above network script file is present in /etc/NetworkManager/system-connections/ and uses nmcli, Made sure it works for previous versions Signed-off-by: Bala Konda Reddy M <bala12352@gmail.com> --------- Signed-off-by: Bala Konda Reddy M <bala12352@gmail.com>
Add mechanism to forcefully install CH/Libvirt/QEMU using transformer. Signed-off-by: Smit Gardhariya <sgardhariya@microsoft.com>
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
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.
Upstream now has a few changes that we require