-
Notifications
You must be signed in to change notification settings - Fork 327
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
kuma-dp: rename #164
kuma-dp: rename #164
Conversation
@@ -68,7 +68,7 @@ BINTRAY_USERNAME ?= | |||
BINTRAY_API_KEY ?= | |||
|
|||
KONVOY_CP_DOCKER_IMAGE ?= konvoy/konvoy-control-plane:latest | |||
KONVOY_DP_DOCKER_IMAGE ?= konvoy/konvoy-dataplane:latest | |||
KUMA_DP_DOCKER_IMAGE ?= konvoy/kuma-dataplane:latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think, it should be kuma/kuma-dataplane:latest
@@ -50,7 +50,7 @@ func newInstallControlPlaneCmd(pctx *kumactl_cmd.RootContext) *cobra.Command { | |||
InjectorServiceName: "konvoy-injector", | |||
InjectorTlsCert: "", | |||
InjectorTlsKey: "", | |||
DataplaneImage: "konvoy/konvoy-dataplane", | |||
DataplaneImage: "konvoy/kuma-dataplane", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think, it should be kuma/kuma-dataplane
@@ -101,7 +101,7 @@ var _ = Describe("kumactl install control-plane", func() { | |||
"--injector-service-name", "injector", | |||
"--injector-tls-cert", "InjectorCert", | |||
"--injector-tls-key", "InjectorKey", | |||
"--dataplane-image", "konvoy-ci/konvoy-dataplane", | |||
"--dataplane-image", "konvoy-ci/kuma-dataplane", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think, it should be kuma-ci/kuma-dataplane
@@ -2150,7 +2150,7 @@ spec: | |||
- name: KONVOY_INJECTOR_CONTROL_PLANE_API_SERVER_URL | |||
value: http://konvoy-control-plane.konvoy-system:5681 | |||
- name: KONVOY_INJECTOR_SIDECAR_CONTAINER_IMAGE | |||
value: konvoy/konvoy-dataplane:latest | |||
value: konvoy/kuma-dataplane:latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think, it should be kuma/kuma-dataplane:latest
@@ -2150,7 +2150,7 @@ spec: | |||
- name: KONVOY_INJECTOR_CONTROL_PLANE_API_SERVER_URL | |||
value: http://konvoy-control-plane.konvoy:5681 | |||
- name: KONVOY_INJECTOR_SIDECAR_CONTAINER_IMAGE | |||
value: konvoy-ci/konvoy-dataplane:greatest | |||
value: konvoy-ci/kuma-dataplane:greatest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma-ci/kuma-dataplane:greatest
@@ -30,7 +30,7 @@ func DefaultConfig() Config { | |||
}, | |||
}, | |||
SidecarContainer: SidecarContainer{ | |||
Image: "konvoy/konvoy-dataplane:latest", | |||
Image: "konvoy/kuma-dataplane:latest", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma/kuma-dataplane:latest
@@ -7,7 +7,7 @@ injector: | |||
bootstrapServer: | |||
url: http://konvoy-control-plane.konvoy-system:5682 | |||
sidecarContainer: | |||
image: konvoy/konvoy-dataplane:latest | |||
image: konvoy/kuma-dataplane:latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma/kuma-dataplane:latest
@@ -68,8 +68,8 @@ BINTRAY_USERNAME ?= | |||
BINTRAY_API_KEY ?= | |||
|
|||
KONVOY_CP_DOCKER_IMAGE ?= konvoy/konvoy-control-plane:latest | |||
KUMA_DP_DOCKER_IMAGE ?= konvoy/kuma-dataplane:latest | |||
KUMACTL_DOCKER_IMAGE ?= konvoy/kumactl:latest | |||
KUMA_DP_DOCKER_IMAGE ?= kuma/kuma-dataplane:latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be kuma/kuma-dp:latest
per @subnetmarco
@@ -50,7 +50,7 @@ func newInstallControlPlaneCmd(pctx *kumactl_cmd.RootContext) *cobra.Command { | |||
InjectorServiceName: "konvoy-injector", | |||
InjectorTlsCert: "", | |||
InjectorTlsKey: "", | |||
DataplaneImage: "konvoy/kuma-dataplane", | |||
DataplaneImage: "kuma/kuma-dataplane", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma/kuma-dp:latest
@@ -2150,7 +2150,7 @@ spec: | |||
- name: KONVOY_INJECTOR_CONTROL_PLANE_API_SERVER_URL | |||
value: http://konvoy-control-plane.konvoy-system:5681 | |||
- name: KONVOY_INJECTOR_SIDECAR_CONTAINER_IMAGE | |||
value: konvoy/kuma-dataplane:latest | |||
value: kuma/kuma-dataplane:latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma/kuma-dp:latest
@@ -30,7 +30,7 @@ func DefaultConfig() Config { | |||
}, | |||
}, | |||
SidecarContainer: SidecarContainer{ | |||
Image: "konvoy/kuma-dataplane:latest", | |||
Image: "kuma/kuma-dataplane:latest", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma/kuma-dp:latest
@@ -7,7 +7,7 @@ injector: | |||
bootstrapServer: | |||
url: http://konvoy-control-plane.konvoy-system:5682 | |||
sidecarContainer: | |||
image: konvoy/kuma-dataplane:latest | |||
image: kuma/kuma-dataplane:latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma/kuma-dp:latest
KUMA_CONTROL_PLANE_BOOTSTRAP_SERVER_URL=http://localhost:5682 \ | ||
KUMA_DATAPLANE_ID=$(EXAMPLE_ENVOY_ID) \ | ||
KUMA_DATAPLANE_ADMIN_PORT=$(ENVOY_ADMIN_PORT) \ | ||
${BUILD_ARTIFACTS_DIR}/kuma-dataplane/kuma-dataplane run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rename to ${BUILD_ARTIFACTS_DIR}/kuma-dp/kuma-dp
@@ -10,7 +10,7 @@ import ( | |||
) | |||
|
|||
var ( | |||
dataplaneLog = core.Log.WithName("konvoy-dataplane") | |||
dataplaneLog = core.Log.WithName("kuma-dataplane") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
core.Log.WithName("kuma-dp")
@@ -19,7 +19,7 @@ func newRootCmd() *cobra.Command { | |||
debug bool | |||
}{} | |||
cmd := &cobra.Command{ | |||
Use: "konvoy-dataplane", | |||
Use: "kuma-dataplane", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma-dp
@@ -6,7 +6,7 @@ import ( | |||
) | |||
|
|||
var _ = Describe("root", func() { | |||
It("should be possible to run `konvoy-dataplane` without a sub-command", func() { | |||
It("should be possible to run `kuma-dataplane` without a sub-command", func() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma-dp
@@ -83,18 +83,18 @@ var _ = Describe("run", func() { | |||
} | |||
}) | |||
|
|||
It("should be possible to start dataplane (Envoy) using `konvoy-dataplane run`", func(done Done) { | |||
It("should be possible to start dataplane (Envoy) using `kuma-dataplane run`", func(done Done) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma-dp run
@@ -101,7 +101,7 @@ var _ = Describe("kumactl install control-plane", func() { | |||
"--injector-service-name", "injector", | |||
"--injector-tls-cert", "InjectorCert", | |||
"--injector-tls-key", "InjectorKey", | |||
"--dataplane-image", "konvoy-ci/konvoy-dataplane", | |||
"--dataplane-image", "konvoy-ci/kuma-dp", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma-ci/kuma-dp
@@ -101,7 +101,7 @@ var _ = Describe("kumactl install control-plane", func() { | |||
"--injector-service-name", "injector", | |||
"--injector-tls-cert", "InjectorCert", | |||
"--injector-tls-key", "InjectorKey", | |||
"--dataplane-image", "konvoy-ci/konvoy-dataplane", | |||
"--dataplane-image", "konvoy-ci/kuma-dp", | |||
"--dataplane-init-image", "konvoy-ci/konvoy-init", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma-ci/kuma-init
@@ -2150,7 +2150,7 @@ spec: | |||
- name: KONVOY_INJECTOR_CONTROL_PLANE_API_SERVER_URL | |||
value: http://konvoy-control-plane.konvoy:5681 | |||
- name: KONVOY_INJECTOR_SIDECAR_CONTAINER_IMAGE | |||
value: konvoy-ci/konvoy-dataplane:greatest | |||
value: konvoy-ci/kuma-dp:greatest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma-ci/kuma-dp:greatest
@@ -2150,7 +2150,7 @@ spec: | |||
- name: KONVOY_INJECTOR_CONTROL_PLANE_API_SERVER_URL | |||
value: http://konvoy-control-plane.konvoy:5681 | |||
- name: KONVOY_INJECTOR_SIDECAR_CONTAINER_IMAGE | |||
value: konvoy-ci/konvoy-dataplane:greatest | |||
value: konvoy-ci/kuma-dp:greatest | |||
- name: KONVOY_INJECTOR_INIT_CONTAINER_IMAGE | |||
value: konvoy-ci/konvoy-init:dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kuma-ci/kuma-init:dev
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's revert the last commit and move it into a separate PR for renaming konvoy-injector
4575608
to
97277ec
Compare
No description provided.