-
Notifications
You must be signed in to change notification settings - Fork 305
WIP - Re-enable Ubuntu build in CI #284
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
Conversation
Re-enable Ubuntu build in CI Follow-up of #283
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: tomkerkhove If they are not already assigned, you can assign the PR to them by writing 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:
Approvers can indicate their approval by writing |
/assign @brendandburns |
Looks like the build is hanging because of minikube which is the same issue from original CI. An additional improvement would be not to wait for the cluster if Minikube has crashed: |
I've added logs for tracing what version for kubectl & minikube is being installed, but they don't always show up somehow. That said, I've changed minikube commands to use Turns out we have a couple of issues:
|
@tomkerkhove That's odd, perhaps you can add a Azure DevOps regularly upgrades the software installed on the agents, which can cause sudden build breaks like this. (Travis is much more conservative in that respect) |
I'll add the docker version to the output and make sure we install a specific version so that we run the correct version if we can. Looks like they've changed it recently: microsoft/azure-pipelines-image-generation@8e6db92#diff-f87405c9dd43f633a04dd078666d61af |
Yeah, so that means we'll probably need some packaging voodoo to uninstall the version of Docker that comes pre-installed, and replace it with a version which is supported by Kubernetes. |
Not really, they actually have it out of the box - https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/tool/docker-installer?view=azure-devops Will update the PR in a few hours |
Use Docker 17.09.0-ce which actually exists and allows older version to be verified as well
I'll abandon this PR for now and circle back once I get this working. Don't want to burn too much build time/spam people |
Nevermind, this is just the CLI |
Re-enable Ubuntu build in CI
Follow-up of #283