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

Base kicbase directly on ubuntu, without kindbase #9330

Merged
merged 6 commits into from
Sep 29, 2020

Conversation

afbjorklund
Copy link
Collaborator

Here is the all-in-one version, that includes all the changes from PR #9135 and PR #9225 - but without the interim steps.

Closes #7788

Remove local things from kind, replace with packages.

Use the "clean-install" script for a nicer Dockerfile.
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Sep 27, 2020
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: afbjorklund

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Sep 27, 2020
This is for reference, to be able to combine them
Also fix the 19.10 comment, it hadn't been updated
Base kicbase directly on ubuntu, without kindbase

Move all the files that are still copied and used

Avoid installing software only to be deleted later

The entrypoint had already been forked since earlier
@codecov-commenter
Copy link

Codecov Report

Merging #9330 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #9330   +/-   ##
=======================================
  Coverage   29.38%   29.38%           
=======================================
  Files         170      170           
  Lines       10334    10334           
=======================================
  Hits         3037     3037           
  Misses       6875     6875           
  Partials      422      422           

deploy/kicbase/Dockerfile Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

move off kindbase image
4 participants