diff --git a/pom.xml b/pom.xml index 1958e248768cf502a2b84bb34f7e1a554e1672f9..3627c00deff6697f2bed2f7505e74270df667a9d 100644 --- a/pom.xml +++ b/pom.xml @@ -33,7 +33,6 @@ <spring-boot.version>2.0.4.RELEASE</spring-boot.version> <spring-cloud.version>Finchley.SR2</spring-cloud.version> <sleuth.version>2.0.0.RC2</sleuth.version> - <micrometer.version>1.0.5</micrometer.version> <maven-surefire-plugin.version>2.22.0</maven-surefire-plugin.version> @@ -79,18 +78,6 @@ <scope>test</scope> </dependency> - <!-- Micrometer core dependency --> - <dependency> - <groupId>io.micrometer</groupId> - <artifactId>micrometer-core</artifactId> - <version>${micrometer.version}</version> - </dependency> - <!-- Micrometer Prometheus registry --> - <dependency> - <groupId>io.micrometer</groupId> - <artifactId>micrometer-registry-prometheus</artifactId> - <version>${micrometer.version}</version> - </dependency> </dependencies> </dependencyManagement> diff --git a/spring-petclinic-api-gateway/pom.xml b/spring-petclinic-api-gateway/pom.xml index ae40e34eb04dc67f455d2d70230d02ab627cf1ea..790845120eb49db6ac2ab97087eab572d478338c 100644 --- a/spring-petclinic-api-gateway/pom.xml +++ b/spring-petclinic-api-gateway/pom.xml @@ -87,10 +87,6 @@ <groupId>org.projectlombok</groupId> <artifactId>lombok</artifactId> </dependency> - <dependency> - <groupId>io.micrometer</groupId> - <artifactId>micrometer-core</artifactId> - </dependency> <dependency> <groupId>io.micrometer</groupId> <artifactId>micrometer-registry-prometheus</artifactId> diff --git a/spring-petclinic-customers-service/pom.xml b/spring-petclinic-customers-service/pom.xml index dc5928e477bcc4cda5d72006c66700d5d07c1ac7..9c411deb62087ea1fee872ed9515896b7c5e025d 100644 --- a/spring-petclinic-customers-service/pom.xml +++ b/spring-petclinic-customers-service/pom.xml @@ -77,10 +77,6 @@ <groupId>org.projectlombok</groupId> <artifactId>lombok</artifactId> </dependency> - <dependency> - <groupId>io.micrometer</groupId> - <artifactId>micrometer-core</artifactId> - </dependency> <dependency> <groupId>io.micrometer</groupId> <artifactId>micrometer-registry-prometheus</artifactId> diff --git a/spring-petclinic-customers-service/src/main/java/org/springframework/samples/petclinic/customers/web/OwnerResource.java b/spring-petclinic-customers-service/src/main/java/org/springframework/samples/petclinic/customers/web/OwnerResource.java index 80916f3efd0e60341bacff931fe56be06edfa759..1791043496f4cfa8e68d612109ac56ea572f7ecb 100644 --- a/spring-petclinic-customers-service/src/main/java/org/springframework/samples/petclinic/customers/web/OwnerResource.java +++ b/spring-petclinic-customers-service/src/main/java/org/springframework/samples/petclinic/customers/web/OwnerResource.java @@ -15,7 +15,7 @@ */ package org.springframework.samples.petclinic.customers.web; -import io.micrometer.core.instrument.MeterRegistry; +import io.micrometer.core.annotation.Timed; import lombok.RequiredArgsConstructor; import lombok.extern.slf4j.Slf4j; import org.springframework.http.HttpStatus; @@ -36,12 +36,12 @@ import java.util.Optional; */ @RequestMapping("/owners") @RestController +@Timed("petclinic.owner") @RequiredArgsConstructor @Slf4j class OwnerResource { private final OwnerRepository ownerRepository; - private final MeterRegistry registry; /** * Create Owner @@ -49,7 +49,6 @@ class OwnerResource { @PostMapping @ResponseStatus(HttpStatus.CREATED) public void createOwner(@Valid @RequestBody Owner owner) { - registry.counter("create.owner").increment(); ownerRepository.save(owner); } @@ -84,7 +83,6 @@ class OwnerResource { ownerModel.setAddress(ownerRequest.getAddress()); ownerModel.setTelephone(ownerRequest.getTelephone()); log.info("Saving owner {}", ownerModel); - registry.counter("update.owner").increment(); return ownerRepository.save(ownerModel); } } diff --git a/spring-petclinic-customers-service/src/main/java/org/springframework/samples/petclinic/customers/web/PetResource.java b/spring-petclinic-customers-service/src/main/java/org/springframework/samples/petclinic/customers/web/PetResource.java index 34c8c70d6da7487791803f8fc8b6638ce6c3fdc7..3bb7f7bf5bcd3d18d055a7a8aad2d0fffe082b6f 100644 --- a/spring-petclinic-customers-service/src/main/java/org/springframework/samples/petclinic/customers/web/PetResource.java +++ b/spring-petclinic-customers-service/src/main/java/org/springframework/samples/petclinic/customers/web/PetResource.java @@ -15,7 +15,7 @@ */ package org.springframework.samples.petclinic.customers.web; -import io.micrometer.core.instrument.MeterRegistry; +import io.micrometer.core.annotation.Timed; import lombok.RequiredArgsConstructor; import lombok.extern.slf4j.Slf4j; import org.springframework.http.HttpStatus; @@ -32,13 +32,13 @@ import java.util.Optional; * @author Maciej Szarlinski */ @RestController +@Timed("petclinic.pet") @RequiredArgsConstructor @Slf4j class PetResource { private final PetRepository petRepository; private final OwnerRepository ownerRepository; - private final MeterRegistry registry; @GetMapping("/petTypes") @@ -57,7 +57,6 @@ class PetResource { Owner owner = optionalOwner.orElseThrow(() -> new ResourceNotFoundException("Owner "+ownerId+" not found")); owner.addPet(pet); - registry.counter("create.pet").increment(); save(pet, petRequest); } @@ -66,7 +65,6 @@ class PetResource { public void processUpdateForm(@RequestBody PetRequest petRequest) { int petId = petRequest.getId(); Pet pet = findPetById(petId); - registry.counter("update.pet").increment(); save(pet, petRequest); } diff --git a/spring-petclinic-customers-service/src/test/java/org/springframework/samples/petclinic/customers/web/PetResourceTest.java b/spring-petclinic-customers-service/src/test/java/org/springframework/samples/petclinic/customers/web/PetResourceTest.java index 5b2b3303c556b0efd3f1e1fd17a9d50a56cc3379..04a6e0079df23370789c31ad55d65ffd092ee994 100644 --- a/spring-petclinic-customers-service/src/test/java/org/springframework/samples/petclinic/customers/web/PetResourceTest.java +++ b/spring-petclinic-customers-service/src/test/java/org/springframework/samples/petclinic/customers/web/PetResourceTest.java @@ -2,7 +2,6 @@ package org.springframework.samples.petclinic.customers.web; import java.util.Optional; -import io.micrometer.core.instrument.MeterRegistry; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.extension.ExtendWith; import org.springframework.beans.factory.annotation.Autowired; @@ -42,9 +41,6 @@ class PetResourceTest { @MockBean OwnerRepository ownerRepository; - @MockBean - MeterRegistry registry; - @Test void shouldGetAPetInJSonFormat() throws Exception { diff --git a/spring-petclinic-vets-service/pom.xml b/spring-petclinic-vets-service/pom.xml index 99b95770388e3001f753f5c1a1349d137fb523e7..9f3c58cd7f1bbeb78151a69dae1d2ae98396761f 100644 --- a/spring-petclinic-vets-service/pom.xml +++ b/spring-petclinic-vets-service/pom.xml @@ -93,10 +93,6 @@ <artifactId>mysql-connector-java</artifactId> <scope>runtime</scope> </dependency> - <dependency> - <groupId>io.micrometer</groupId> - <artifactId>micrometer-core</artifactId> - </dependency> <dependency> <groupId>io.micrometer</groupId> <artifactId>micrometer-registry-prometheus</artifactId> diff --git a/spring-petclinic-visits-service/pom.xml b/spring-petclinic-visits-service/pom.xml index c60737406048892caf24887750a0a576bc6bbc50..2bf6cc5d07af1dc9d18e0913f6ee2d8a42e1440a 100644 --- a/spring-petclinic-visits-service/pom.xml +++ b/spring-petclinic-visits-service/pom.xml @@ -76,10 +76,6 @@ <artifactId>mysql-connector-java</artifactId> <scope>runtime</scope> </dependency> - <dependency> - <groupId>io.micrometer</groupId> - <artifactId>micrometer-core</artifactId> - </dependency> <dependency> <groupId>io.micrometer</groupId> <artifactId>micrometer-registry-prometheus</artifactId> diff --git a/spring-petclinic-visits-service/src/main/java/org/springframework/samples/petclinic/visits/web/VisitResource.java b/spring-petclinic-visits-service/src/main/java/org/springframework/samples/petclinic/visits/web/VisitResource.java index 5905f3715392fac0d3d7b59047ad84817caffd34..8f6bd5ca1ecd6ee721930817987c7fe716d8b283 100644 --- a/spring-petclinic-visits-service/src/main/java/org/springframework/samples/petclinic/visits/web/VisitResource.java +++ b/spring-petclinic-visits-service/src/main/java/org/springframework/samples/petclinic/visits/web/VisitResource.java @@ -18,7 +18,7 @@ package org.springframework.samples.petclinic.visits.web; import java.util.List; import javax.validation.Valid; -import io.micrometer.core.instrument.MeterRegistry; +import io.micrometer.core.annotation.Timed; import lombok.RequiredArgsConstructor; import lombok.Value; import lombok.extern.slf4j.Slf4j; @@ -43,10 +43,10 @@ import org.springframework.web.bind.annotation.RestController; @RestController @RequiredArgsConstructor @Slf4j +@Timed("petclinic.visit") class VisitResource { private final VisitRepository visitRepository; - private final MeterRegistry registry; @PostMapping("owners/*/pets/{petId}/visits") @ResponseStatus(HttpStatus.NO_CONTENT) @@ -56,7 +56,6 @@ class VisitResource { visit.setPetId(petId); log.info("Saving visit {}", visit); - registry.counter("create.visit").increment(); visitRepository.save(visit); } diff --git a/spring-petclinic-visits-service/src/test/java/org/springframework/samples/petclinic/visits/web/VisitResourceTest.java b/spring-petclinic-visits-service/src/test/java/org/springframework/samples/petclinic/visits/web/VisitResourceTest.java index 3409b30abd3b848b888586e422a7cb84bc294481..25d33f24ba7bba4bbd962752b6f05a98acd786d3 100644 --- a/spring-petclinic-visits-service/src/test/java/org/springframework/samples/petclinic/visits/web/VisitResourceTest.java +++ b/spring-petclinic-visits-service/src/test/java/org/springframework/samples/petclinic/visits/web/VisitResourceTest.java @@ -1,6 +1,5 @@ package org.springframework.samples.petclinic.visits.web; -import io.micrometer.core.instrument.MeterRegistry; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.extension.ExtendWith; import org.springframework.beans.factory.annotation.Autowired; @@ -30,9 +29,6 @@ class VisitResourceTest { @MockBean VisitRepository visitRepository; - @MockBean - MeterRegistry registry; - @Test void shouldFetchVisits() throws Exception { given(visitRepository.findByPetIdIn(asList(111, 222)))