Skip to content
This repository was archived by the owner on May 25, 2023. It is now read-only.

Specify contact-point-discovery.port-name#185

Merged
eed3si9n merged 2 commits intolightbend:masterfrom
eed3si9n:wip/endpoint
Jan 8, 2019
Merged

Specify contact-point-discovery.port-name#185
eed3si9n merged 2 commits intolightbend:masterfrom
eed3si9n:wip/endpoint

Conversation

@eed3si9n
Copy link
Contributor

@eed3si9n eed3si9n commented Jan 8, 2019

This is on top of #184

Fixes #177

This specifies the management endpoint name explicity to -Dakka.management.cluster.bootstrap.contact-point-discovery.port-name.
By default the management endpoint name will fallback to "akka-mgmt-http" for backward compatibility.

In the future, sbt-reactive-app can declare the management endpoint name using "management-endpoint" as a label on the Docker image. See lightbend/sbt-reactive-app#164.

These changes combined together provides softlanding of the switch to "management" in akka/akka-management#274.

/cc @lightbend/play-lagom

@eed3si9n eed3si9n requested a review from octonato January 8, 2019 06:05
@ghost ghost assigned eed3si9n Jan 8, 2019
@ghost ghost added the review label Jan 8, 2019
Fixes lightbend#177

This specifies the management endpoint name explicity to `-Dakka.management.cluster.bootstrap.contact-point-discovery.port-name`.
By default the management endpoint name will fallback to `"akka-mgmt-http"` for backward compatibility.

In the future, sbt-reactive-app can declare the management endpoint name using "management-endpoint" as a label on the Docker image. See lightbend/sbt-reactive-app#164.

These changes combined together provides softlanding of the switch to `"management"` in akka/akka-management#274.
@eed3si9n eed3si9n merged commit 47d7c4f into lightbend:master Jan 8, 2019
@ghost ghost removed the review label Jan 8, 2019
@eed3si9n eed3si9n deleted the wip/endpoint branch January 8, 2019 19:01
@eed3si9n eed3si9n added this to the 1.6.0 milestone Jan 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants