search

kubevirt / kubevirt-tutorial

Demo that guides users through an end to end KubeVirt experience.
https://www.kubevirt.io/
Apache License 2.0
31 stars 28 forks source link

Fix: changed pip by pip3 in order to use python to execute ansible #111

Closed jparrill closed 4 years ago

jparrill commented 5 years ago

Fixing #109

jparrill commented 5 years ago

Humm as we could see here: https://prow.apps.ovirt.org/log?id=1164565008722432000&job=kubevirt-tutorial-presubmit-lab-testing-k8s-1.13.3

and concretelly:

The following packages have unmet dependencies:
 python3-pip : Depends: python3-distutils but it is not installable
               Depends: python-pip-whl (= 18.1-5) but 9.0.1-2+deb9u1 is to be installed
               Recommends: python3-dev (>= 3.2) but it is not installable
               Recommends: python3-setuptools but it is not installable
               Recommends: python3-wheel but it is not installable

Python3 has so many deps not installed, the thing is if we install all the deps by hand we will lose much time for every job execution, and the image used is not managed by us.

The other option is use apt strategy but it fails on execution: https://prow.apps.ovirt.org/log?id=1164570046417080320&job=kubevirt-tutorial-presubmit-lab-testing-k8s-1.13.3

Preparing to unpack .../libc-bin_2.28-10_amd64.deb ...
Unpacking libc-bin (2.28-10) over (2.24-11+deb9u4) ...
dpkg: error processing archive /var/cache/apt/archives/libc-bin_2.28-10_amd64.deb (--unpack):
 unable to clean up mess surrounding './etc/default/nss' before installing another version: Read-only file system
dpkg-deb: error: subprocess paste was killed by signal (Broken pipe)
Errors were encountered while processing:
 /var/cache/apt/archives/libc-bin_2.28-10_amd64.deb
E: Sub-process /usr/bin/dpkg returned an error code (1)
jparrill commented 5 years ago

/hold

iranzo commented 4 years ago

/retest

iranzo commented 4 years ago

/retest

jparrill commented 4 years ago

/lgtm

kubevirt-bot commented 4 years ago

@jparrill: you cannot LGTM your own PR.

In response to [this](https://github.com/kubevirt/kubevirt-tutorial/pull/111#issuecomment-543066278): >/lgtm Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
iranzo commented 4 years ago

/lgtm

/lgtm /approve

kubevirt-bot commented 4 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: iranzo

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/kubevirt/kubevirt-tutorial/blob/master/OWNERS)~~ [iranzo] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
iranzo commented 4 years ago

/hold cancel