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 Minikube ISO as part of integration tests. #1532

Closed
LalatenduMohanty opened this issue Oct 10, 2017 · 10 comments
Closed

Add Minikube ISO as part of integration tests. #1532

LalatenduMohanty opened this issue Oct 10, 2017 · 10 comments

Comments

@LalatenduMohanty
Copy link
Member

LalatenduMohanty commented Oct 10, 2017

Recently we have added support for Minikube ISO i.e. #1058 to Minishift. However we do not run any tests of Minishift on it. We should add Minikube ISO as another ISO in our integration tests.

PS: Updated the description as discussed in #1552

@amitkrout
Copy link
Contributor

@LalatenduMohanty i got the abstract of the issue however this confuses me when Minikube ISO comes to picture. In minishift repo as we have ISO like centos, b2d. The question is where we can get latest release minikube iso? are we maintaining any repo for this or referring to some other location? How i attach it to minishift start ?

@praveenkumar
Copy link
Contributor

The question is where we can get latest release minikube iso?

you can get released iso from https://storage.googleapis.com/minikube/iso/minikube-v<iso_version>.iso

are we maintaining any repo for this or referring to some other location?

We are not maintaining any repo for this ourself and it's complete on minikube side.

How i attach it to minishift start ?

minishift start --iso-url minikube

agajdosi pushed a commit to agajdosi/minishift that referenced this issue Oct 10, 2017
@amitkrout
Copy link
Contributor

amitkrout commented Oct 11, 2017

I use minishift start --iso-url https://storage.googleapis.com/minikube/iso/minikube-v0.23.5.iso instead of minishift start and run integration test with all the features one by one (including coolstore with my pr). I dont see any major functionlity issue, though there is some failure. For example

Command 'ssh -- cat /var/lib/boot2docker/profile' did not execute successfully. cmdExit: 1, cmdErr: cat: /var/lib/boot2docker/profile: No such file or directory
Command failed: exit status 1
Output did not match. Expected: 'Disk \/dev\/sda: 2[4-6]\.?[0-9]{0,2} GB', Actual: 'Disk /dev/sda: 24.4 GiB,26214400000 bytes, 51200000 sectors
Disklabel type: dos
Disk identifier: 0x9c76f717

Enviornment : OS X + xhyve + minishift 1.7.0

I would suggest to run the tests on the ISO as a separate job in ci.centos.org. However I am open to other suggestions too.

+1. If we make it parallel execution that will cost us less

@agajdosi
Copy link

@amitkrout We need to fix these errors, second one should be easy, just failing on GB vs GiB. First one will need some code changes as /var/lib/boot2docker/profile is not a valid location for minikube.iso.

@amitkrout
Copy link
Contributor

@agajdosi right

@coolbrg
Copy link
Contributor

coolbrg commented Oct 12, 2017

@amitkrout We need to fix these errors, second one should be easy, just failing on GB vs GiB. First one will need some code changes as /var/lib/boot2docker/profile is not a valid location for minikube.iso.

@amitkrout @agajdosi anyone working on it? Since, it is CI task so I had assigned myself and then found we need to fix few features file to adapt with minikube ISO.

@coolbrg coolbrg removed their assignment Oct 12, 2017
@agajdosi
Copy link

@budhrg Will start on it today as discussed on IRC.

@agajdosi agajdosi self-assigned this Oct 12, 2017
@amitkrout
Copy link
Contributor

Sorry i was away for some time. @agajdosi if you are overburden with this task please feel free to assign to my name. cc_ @budhrg

agajdosi pushed a commit to agajdosi/minishift that referenced this issue Oct 12, 2017
@agajdosi
Copy link

@budhrg Adding the support for Minikube iso in opened PR #1552.

agajdosi pushed a commit to agajdosi/minishift that referenced this issue Oct 12, 2017
agajdosi pushed a commit to agajdosi/minishift that referenced this issue Oct 13, 2017
agajdosi pushed a commit to agajdosi/minishift that referenced this issue Oct 13, 2017
agajdosi pushed a commit to agajdosi/minishift that referenced this issue Oct 13, 2017
agajdosi pushed a commit to agajdosi/minishift that referenced this issue Oct 13, 2017
agajdosi pushed a commit to agajdosi/minishift that referenced this issue Oct 13, 2017
agajdosi pushed a commit to agajdosi/minishift that referenced this issue Oct 16, 2017
agajdosi pushed a commit to agajdosi/minishift that referenced this issue Oct 16, 2017
@LalatenduMohanty LalatenduMohanty changed the title Test Minishift with Minikube ISO as part of CI Add Minikube ISO as part of integration tests. Oct 16, 2017
anjannath pushed a commit to anjannath/minishift that referenced this issue Nov 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants