Skip to content

feat: introduces builder images#590

Open
bartoszmajsak wants to merge 2 commits intomasterfrom
builder_image
Open

feat: introduces builder images#590
bartoszmajsak wants to merge 2 commits intomasterfrom
builder_image

Conversation

@bartoszmajsak
Copy link
Copy Markdown
Contributor

Short description of what this resolves:

Leverages docker builder images/ multistage build to build controller and test service.

Changes proposed in this pull request:

  • rewritten Dockerfiles
  • .dockerignore
  • adjustments in Makefile

Fixes #588

@bartoszmajsak bartoszmajsak added kind/enhancement Kind: New feature or improvement internal/infra Internal: related to project build / setup / release labels Dec 21, 2020
@bartoszmajsak
Copy link
Copy Markdown
Contributor Author

/test openshift

@bartoszmajsak
Copy link
Copy Markdown
Contributor Author

/retest

3 similar comments
@bartoszmajsak
Copy link
Copy Markdown
Contributor Author

/retest

@bartoszmajsak
Copy link
Copy Markdown
Contributor Author

/retest

@bartoszmajsak
Copy link
Copy Markdown
Contributor Author

/retest

@alien-ike
Copy link
Copy Markdown
Collaborator

@bartoszmajsak: The following test failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
e2e-tests-openshift ebab5b2 link /test openshift
Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Comment thread .circleci/config.yml
sudo circleci-install golang 1.14.7
docker:
- image: &golang-img cimg/go:1.15.5
- image: &golang-img cimg/go:1.14.7
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1.14.7?

Comment thread build/Dockerfile
@@ -1,3 +1,19 @@
FROM registry.access.redhat.com/ubi8/go-toolset:1.14.7 AS builder
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't we require 1.16.x now due to the fs thinggy?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal/infra Internal: related to project build / setup / release kind/enhancement Kind: New feature or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Leverage builder image when creating istio-workspace images

3 participants