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.

Sign up for GitHub

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

Jump to bottom

attach_interface_with_model: don't attach incompatible controller #5879

Merged
merged 1 commit into from
Sep 9, 2024
Merged

attach_interface_with_model: don't attach incompatible controller #5879

merged 1 commit into from
Sep 9, 2024

Conversation

smitterl
Copy link
Contributor

@smitterl smitterl commented Sep 5, 2024

On s390x, pcie-to-pci-bridge is not available. Don't attach this.

@smitterl
attach_interface_with_model: don't attach incompatible controller
a217a59 
On s390x, pcie-to-pci-bridge is not available. Don't attach this.

Signed-off-by: Sebastian Mitterle <smitterl@redhat.com>
@smitterl
Copy link
Contributor Author

smitterl commented Sep 5, 2024 

JOB ID     : 1c3bfb6885d7d232d2c7ce957d6c404b644830e2
JOB LOG    : /var/log/avocado/job-results/job-2024-09-05T06.18-1c3bfb6/job.log
 (1/1) type_specific.io-github-autotest-libvirt.virtual_network.hotplug.attach_interface.model.virtio: PASS (37.81 s)
RESULTS    : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
JOB HTML   : /var/log/avocado/job-results/job-2024-09-05T06.18-1c3bfb6/results.html
JOB TIME   : 38.28 s

@smitterl
Copy link
Contributor Author

smitterl commented Sep 5, 2024

@Yingshun Please help review.

@Yingshun Yingshun merged commit d9f6455 into autotest:master Sep 9, 2024
5 checks passed
@smitterl smitterl deleted the s390_pci_controller branch September 12, 2024 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Yingshun Yingshun Yingshun approved these changes

Assignees
No one assigned
Labels
minorfix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@smitterl @Yingshun