-
Notifications
You must be signed in to change notification settings - Fork 634
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
References new Kubernetes / OpenShift deployment guide #1785
Conversation
@@ -114,11 +114,11 @@ akka { | |||
|
|||
#### Docker images and deployment specs | |||
|
|||
With the removal of ConductR or Lightbend Orchestration, the docker images and deployment specs will have to be maintained manually. Therefore the recommended migration is to take ownership of the `Dockerfile`, deployment scripts and orchestration specs created by such tooling, such as by committing them to the source repository and then maintaining them. We also found that such maintenance can be made easier by making use of [kustomize](https://github.com/kubernetes-sigs/kustomize). | |||
With the removal of ConductR or Lightbend Orchestration, the docker images and deployment specs will have to be maintained manually. Therefore the recommended migration is to take ownership of the `Dockerfile`, deployment scripts and orchestration specs. |
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.
Can we have a small explanation about why do we consider this a better approach for users?
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 can add here the same text I used on a recent discussion at http://discuss.lightbend.com/
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.
Nothing to add to what @marcospereira already mentioned.
|
||
```scala |
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.
👍🏼
Co-Authored-By: renatocaval <[email protected]>
Co-Authored-By: renatocaval <[email protected]>
Co-Authored-By: renatocaval <[email protected]>
Co-Authored-By: renatocaval <[email protected]>
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.
LGTM.
Let's merge when green.
Cherrypicked |
Co-authored-by: Marcos Pereira <[email protected]>
No description provided.