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

Revert "testnode: install QEMU packages on Fedora, RHEL, and CentOS" #168

Closed
wants to merge 1 commit into from

Conversation

vasukulkarni
Copy link
Contributor

@dillaman @andrewschoen

This has dependency on librados2 and librbd1 and endsup installing older version either on rhel or on fedora, Since @ceph provides the right version of librados and rbd, it should be the main source. I think it might be better to handle this at the later stage.

Check here for logs: http://fpaste.org/291685/18351144/

Reverts #153

@vasukulkarni
Copy link
Contributor Author

@dillaman is this package only required for rbd/qemu suite? we can add the ansible task there to install qemu-kvm after the install task, with this installed before install task it will have older version which is not overwritten for some distro.

@dillaman
Copy link

@vasukulkarni I don't think a straight revert is the correct answer since Ubuntu does install qemu at this stage (which also will pull in librbd / librados). You should keep the OSes consistent.

@vasukulkarni
Copy link
Contributor Author

@dillaman agree, I think it should be removed for ubuntu as well and add it after Ceph install, if you want to do it differently that is fine as well.

@dillaman
Copy link

@vasukulkarni I'm fine with that approach -- but in that case, this PR should include a commit to remove the Ubuntu QEMU install and another commit for a new ansible task to install QEMU on all OSes which can be invoked via the teuthology task.

@vasukulkarni
Copy link
Contributor Author

@dillaman sure, I will include the ubuntu commit, will check with @zmc and @andrewschoen on ansible task that can used to install dependency packages for tests which can be all committed together.

@wusui
Copy link

wusui commented Dec 8, 2015

I believe that the problems in http://tracker.ceph.com/issues/13881 may be happening due to this change not being implemented.

@wusui
Copy link

wusui commented Dec 9, 2015

Note: The problem reported in my previous comment happens on trusty, so qemu-system-x86 should probably also be removed from at least the ubuntu_15.yml file as well (and probably others too).

@vasukulkarni vasukulkarni deleted the revert-153-wip-13308 branch December 5, 2016 23:19
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.

None yet

3 participants