Skip to content
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

#27 Add Spring Boot Admin server #36

Merged
merged 12 commits into from
Jan 15, 2017
Merged

Conversation

arey
Copy link
Member

@arey arey commented Jan 14, 2017

@mszarlinski I let you test and review it.

The spring-boot-maven-plugin configuration has been centralized into the parent pom in order to display build-related information to all applications:
springbootadmin

I've got some errors with docker-compose :

admin-server         | 2017-01-14 11:16:38.868  INFO 1 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Getting all instance registry info from the eureka server
admin-server         | 2017-01-14 11:16:38.887 ERROR 1 --- [tbeatExecutor-0] c.n.d.s.t.d.RedirectingEurekaHttpClient  : Request execution error
admin-server         | 
admin-server         | com.sun.jersey.api.client.ClientHandlerException: java.net.ConnectException: Connection refused (Connection refused)
admin-server         | 2017-01-14 11:16:38.890 ERROR 1 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : DiscoveryClient_ADMIN-SERVER/8178f9a55f9c:ADMIN-SERVER:0 - was unable to refresh its cache! status = Cannot execute request on any known server
admin-server         | 

@@ -180,40 +141,4 @@
</plugins>
</build>

<profiles>
Copy link
Collaborator

Choose a reason for hiding this comment

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

buildDocker profile has been removed from api-gateway after merge

Copy link
Member Author

Choose a reason for hiding this comment

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

It's a mistake. I've just pushed the fix.

@mszarlinski
Copy link
Collaborator

admin-server seems to work properly using docker-compose startup (after fixing pom.xml from api-gateway):
image

@mszarlinski mszarlinski merged commit 59d24f4 into spring-petclinic:master Jan 15, 2017
@arey arey deleted the admin branch January 15, 2017 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants