Skip to content

Commit

Permalink
*: add "KUBEINKS_E2E_IMAGE"
Browse files Browse the repository at this point in the history
Signed-off-by: Gyuho Lee <[email protected]>
  • Loading branch information
gyuho committed Oct 15, 2018
1 parent 8ec2018 commit 7396106
Show file tree
Hide file tree
Showing 5 changed files with 15 additions and 5 deletions.
3 changes: 2 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,8 @@ GINKGO_TIMEOUT=10h \
AWSTESTER_EKS_EMBEDDED=true \
AWSTESTER_EKS_WAIT_BEFORE_DOWN=10m \
AWSTESTER_EKS_DOWN=true \
AWSTESTER_EKS_AWSTESTER_IMAGE=[YOUR_AWS_ACCOUNT_ID].dkr.ecr.us-west-2.amazonaws.com/awstester:[BUILT_TAG] \
AWSTESTER_EKS_KUBEKINS_E2E_IMAGE=[ACCOUNT_ID].dkr.ecr.us-west-2.amazonaws.com/kubekins-e2e:[TAG] \
AWSTESTER_EKS_AWSTESTER_IMAGE=[ACCOUNT_ID].dkr.ecr.us-west-2.amazonaws.com/awstester:[TAG] \
AWSTESTER_EKS_WORKER_NODE_INSTANCE_TYPE=c5.xlarge \
AWSTESTER_EKS_WORKER_NODE_ASG_MIN=5 \
AWSTESTER_EKS_WORKER_NODE_ASG_MAX=5 \
Expand Down
5 changes: 3 additions & 2 deletions cmd/awstester/eks/prow.go
Original file line number Diff line number Diff line change
Expand Up @@ -110,9 +110,10 @@ func prowALBFunc(cmd *cobra.Command, args []string) {
AWSTESTER_EKS_WAIT_BEFORE_DOWN: fmt.Sprintf("%v", cfg.WaitBeforeDown),
AWSTESTER_EKS_DOWN: fmt.Sprintf("%v", cfg.Down),

AWSTESTER_EKS_AWSTESTER_IMAGE: fmt.Sprintf("%v", cfg.AWSTesterImage),
AWSTESTER_EKS_KUBEKINS_E2E_IMAGE: fmt.Sprintf("%s", cfg.KubekinsE2E),
AWSTESTER_EKS_AWSTESTER_IMAGE: fmt.Sprintf("%s", cfg.AWSTesterImage),

AWSTESTER_EKS_WORKER_NODE_INSTANCE_TYPE: fmt.Sprintf("%v", cfg.WorkerNodeInstanceType),
AWSTESTER_EKS_WORKER_NODE_INSTANCE_TYPE: fmt.Sprintf("%s", cfg.WorkerNodeInstanceType),
AWSTESTER_EKS_WORKER_NODE_ASG_MIN: fmt.Sprintf("%d", cfg.WorkderNodeASGMin),
AWSTESTER_EKS_WORKER_NODE_ASG_MAX: fmt.Sprintf("%d", cfg.WorkderNodeASGMax),

Expand Down
2 changes: 2 additions & 0 deletions eksconfig/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@ import (

// Config defines EKS testing configuration.
type Config struct {
// KubekinsE2E is the custom-built Kubernetes e2e test container image.
KubekinsE2E string `json:"kubekins-e2e-image,omitempty"`
// AWSTesterImage is the awstester container image.
// Required for "awstester ingress server" for ALB Ingress Controller tests.
AWSTesterImage string `json:"awstester-image,omitempty"`
Expand Down
9 changes: 7 additions & 2 deletions internal/eks/alb/alb_prow.go
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@ type ConfigProwJobYAML struct {
AWSTESTER_EKS_WAIT_BEFORE_DOWN string
AWSTESTER_EKS_DOWN string

AWSTESTER_EKS_AWSTESTER_IMAGE string
AWSTESTER_EKS_KUBEKINS_E2E_IMAGE string
AWSTESTER_EKS_AWSTESTER_IMAGE string

AWSTESTER_EKS_WORKER_NODE_INSTANCE_TYPE string
AWSTESTER_EKS_WORKER_NODE_ASG_MIN string
Expand Down Expand Up @@ -101,7 +102,11 @@ func CreateProwJobYAML(cfg ConfigProwJobYAML) (string, error) {
Spec: &v1.PodSpec{
Containers: []v1.Container{
{
Image: "gcr.io/k8s-testimages/kubekins-e2e:v20181005-fd9cfb8b0-master",
// Image: "gcr.io/k8s-testimages/kubekins-e2e:v20181005-fd9cfb8b0-master",

// use custom built image to include "wrk", "awstester", etc.
// e.g. 607362164682.dkr.ecr.us-west-2.amazonaws.com/kubekins-e2e:ade682b5fc04
Image: cfg.AWSTESTER_EKS_KUBEKINS_E2E_IMAGE,

Args: []string{
"--repo=github.com/gyuho/$(REPO_NAME)=$(PULL_REFS)",
Expand Down
1 change: 1 addition & 0 deletions internal/eks/alb/alb_prow_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ func TestALBProw(t *testing.T) {
GINKGO_VERBOSE: "true",
AWSTESTER_EKS_WAIT_BEFORE_DOWN: "1m",
AWSTESTER_EKS_DOWN: "true",
AWSTESTER_EKS_KUBEKINS_E2E_IMAGE: "ACCOUNT-ID.dkr.ecr.us-west-2.amazonaws.com/kubekins-e2e:latest",
AWSTESTER_EKS_AWSTESTER_IMAGE: "ACCOUNT-ID.dkr.ecr.us-west-2.amazonaws.com/awstester:latest",
AWSTESTER_EKS_WORKER_NODE_INSTANCE_TYPE: "m3.xlarge",
AWSTESTER_EKS_WORKER_NODE_ASG_MIN: "1",
Expand Down

0 comments on commit 7396106

Please sign in to comment.