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

openshift: Latest stable Kubevirt version respects semver comparision #9638

Closed
wants to merge 1 commit into from

Conversation

mareklibra
Copy link
Contributor

@mareklibra mareklibra commented Jul 11, 2018

Prior this change, the latest kubevirt version was retrieved from [1]
causing mismatch if minor version of former release went out after
latest major (like 0.6.2 after 0.7.0).

The latest Kubeviret version is determined by sorting within installation
script.

[1] https://api.github.com/repos/kubevirt/kubevirt/releases/latest

  • FAIL: image-refresh openshift

@mareklibra
Copy link
Contributor Author

Leading towards kubevirt upgrade to latest v0.7.0

@mareklibra
Copy link
Contributor Author

Kubevirt API endpoints will need to be updated due to their rename (VM, OVM ==> VMI, VM)
/Cc @suomiy

@cockpituous
Copy link
Contributor

image-refresh in progress on cockpit-tasks-8f0b3.
Log: http://fedorapeople.org/groups/cockpit/logs/image-refresh-9638-20180711-075151/

@cockpituous cockpituous changed the title openshift: Latest stable Kubevirt version respects semver comparision WIP: cockpit-tasks-8f0b3: openshift: Latest stable Kubevirt version respects semver comparision Jul 11, 2018
@cockpituous
Copy link
Contributor

@cockpituous cockpituous changed the title WIP: cockpit-tasks-8f0b3: openshift: Latest stable Kubevirt version respects semver comparision openshift: Latest stable Kubevirt version respects semver comparision Jul 11, 2018
@cockpituous
Copy link
Contributor

image-refresh in progress on cockpit-tasks-zp9rr.
Log: http://fedorapeople.org/groups/cockpit/logs/image-refresh-9638-20180711-130202/

@cockpituous cockpituous changed the title openshift: Latest stable Kubevirt version respects semver comparision WIP: cockpit-tasks-zp9rr: openshift: Latest stable Kubevirt version respects semver comparision Jul 11, 2018
@cockpituous
Copy link
Contributor

@cockpituous cockpituous changed the title WIP: cockpit-tasks-zp9rr: openshift: Latest stable Kubevirt version respects semver comparision openshift: Latest stable Kubevirt version respects semver comparision Jul 11, 2018
@martinpitt
Copy link
Member

@mareklibra: This failed with

+ cp kubevirt.src/cluster/vm-template-fedora.yaml ./
cp: cannot stat 'kubevirt.src/cluster/vm-template-fedora.yaml': No such file or directory
image-create: setup failed with code 1

Seems you are running into much of the same issues I already ran into in #9479. I seem to have lost my attempted changes from that, but the issue still has a bunch of things.

Prior this change, the latest kubevirt version was retrieved from [1]
causing mismatch if minor version of former release went out after
latest major (like 0.6.2 after 0.7.0).

The latest Kubeviret version is determined by sorting within installation
script.

[1] https://api.github.com/repos/kubevirt/kubevirt/releases/latest
@mareklibra
Copy link
Contributor Author

Fixed multiple issues. Recently stucked on devices provided by node:

  Warning  FailedScheduling  2m (x57 over 18m)  default-scheduler  0/1 nodes are available: 1 Insufficient devices.kubevirt.io/kvm, 1 Insufficient devices.kubevirt.io/tun.

Required by vm-launcher pod.

@atiratree
Copy link
Contributor

vms cannot be run until kubevirt/kubevirt#1316 is fixed, hopefully in 0.7.1

@atiratree
Copy link
Contributor

Kubevirt API endpoints will need to be updated due to their rename (VM, OVM ==> VMI, VM)

implemented in #8948

@mareklibra
Copy link
Contributor Author

vms cannot be run until kubevirt/kubevirt#1316 is fixed, hopefully in 0.7.1

Since we can't get tests working, setting this as blocked until kubevirt/kubevirt#1316 is merged/released.

croissanne added a commit to croissanne/cockpit that referenced this pull request Jul 26, 2018
Current latest release (v0.7.0) is broken,
see cockpit-project#9638.

Fixes cockpit-project#9479
martinpitt added a commit to martinpitt/cockpit that referenced this pull request Sep 4, 2018
 - Drop kubevirt for now. It has been broken for many months now (cockpit-project#9479,
   cockpit-project#9638), and we must rebuild openshift to finally unbreak our cockpit/ws
   container (cockpit-project#9941). kubevirt tests continue to run on the
   openshift-prerelease image, which is sufficient for nw.

 - Move image to fedora-28, so that we can move cockpit/ws and cockpit/base to
   fedora-28.

 - Drop workaround for [certificate bug](openshift/origin#18715),
   the fix is in latest OpenShift 3.9.0 now.

Fixes cockpit-project#9479
martinpitt added a commit to martinpitt/cockpit that referenced this pull request Sep 4, 2018
 - Drop kubevirt for now. It has been broken for many months now (cockpit-project#9479,
   cockpit-project#9638), and we must rebuild openshift to finally unbreak our cockpit/ws
   container (cockpit-project#9941). kubevirt tests continue to run on the
   openshift-prerelease image.

 - Move image to fedora-28, so that we can move cockpit/ws and
   cockpit/base to fedora-28.

 - Update workaround for openshift/origin#18715
   to install "openssl", which is not in the default install any more on
   Fedora 28.

Fixes cockpit-project#9479
Closes cockpit-project#9975
martinpitt added a commit to croissanne/cockpit that referenced this pull request Sep 4, 2018
 - Drop kubevirt for now. It has been broken for many months now (cockpit-project#9479,
   cockpit-project#9638), and we must rebuild openshift to finally unbreak our cockpit/ws
   container (cockpit-project#9941). kubevirt tests continue to run on the
   openshift-prerelease image.

 - Move image to fedora-28, so that we can move cockpit/ws and
   cockpit/base to fedora-28.

 - Drop workaround for openshift/origin#18715,
   the fix is in latest OpenShift 3.9.0 now.

Fixes cockpit-project#9479
Closes cockpit-project#9975
martinpitt added a commit that referenced this pull request Sep 4, 2018
 - Drop kubevirt for now. It has been broken for many months now (#9479,
   #9638), and we must rebuild openshift to finally unbreak our cockpit/ws
   container (#9941). kubevirt tests continue to run on the
   openshift-prerelease image.

 - Move image to fedora-28, so that we can move cockpit/ws and
   cockpit/base to fedora-28.

 - Update workaround for openshift/origin#18715
   to install "openssl", which is not in the default install any more on
   Fedora 28.

Fixes #9479
Closes #9975
@martinpitt
Copy link
Member

Due to insufficient maintenance, and kubevirt UI moving to Tectonic/Quay, we are removing the kubevirt bits from Cockpit now. Details:

@martinpitt martinpitt closed this Sep 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants