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

Add tests for handling disk devices with long names #172

Merged

Conversation

hamistao
Copy link
Contributor

@hamistao hamistao commented May 17, 2024

Devices with long names can cause some problems due to socket paths or device tags that include those names being too long, resulting in errors from QEMU. See #13320 and #15516.

@hamistao hamistao force-pushed the test_handling_vms_with_long_names branch 2 times, most recently from 5785f74 to 9dc5284 Compare May 23, 2024 15:44
@hamistao hamistao force-pushed the test_handling_vms_with_long_names branch 2 times, most recently from 7f6719f to e2af9b8 Compare June 2, 2024 17:42
@hamistao hamistao changed the title Add tests for handling vms with long names Add tests for handling disk devices with long names Jun 2, 2024
@hamistao
Copy link
Contributor Author

hamistao commented Jun 2, 2024

@tomponline I changed the approach of this PR from using long VM names to using long device names because that way we can check for regressions for both #13320 and #15516.
As a consequence, the branch name is now unrelated to the PR's changes, if this is a problem please let me know so I can close this PR and open another one with another branch name.

tests/storage-disks-vm Outdated Show resolved Hide resolved
@hamistao hamistao force-pushed the test_handling_vms_with_long_names branch from e2af9b8 to 81c4623 Compare June 3, 2024 12:26
tests/storage-disks-vm Show resolved Hide resolved
tests/storage-disks-vm Show resolved Hide resolved
tests/storage-disks-vm Outdated Show resolved Hide resolved
tests/storage-disks-vm Outdated Show resolved Hide resolved
@hamistao hamistao force-pushed the test_handling_vms_with_long_names branch 2 times, most recently from 71c08cf to 52ac028 Compare June 3, 2024 19:25
tests/storage-disks-vm Outdated Show resolved Hide resolved
Signed-off-by: hamistao <pedro.ribeiro@canonical.com>
@hamistao hamistao force-pushed the test_handling_vms_with_long_names branch from 52ac028 to 351b7f3 Compare June 3, 2024 19:31
@tomponline tomponline merged commit 7667abf into canonical:main Jun 10, 2024
56 of 66 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants