Skip to content

Conversation

artembilan
Copy link
Member

Spring Integration can instrument its component with an Observation according to the @EnableIntegrationManagement.observationPatterns value

  • Expose spring.integration.management.observation-patterns configuration property
  • Propagate this property into an @EnableIntegrationManagement in the IntegrationAutoConfiguration.IntegrationManagementConfiguration
  • Verify that property has an effect via IntegrationAutoConfigurationTests.integrationManagementInstrumentedWithObservation()
Spring Integration can instrument its component with an `Observation` according to the `@EnableIntegrationManagement.observationPatterns` value * Expose `spring.integration.management.observation-patterns` configuration property * Propagate this property into an `@EnableIntegrationManagement` in the `IntegrationAutoConfiguration.IntegrationManagementConfiguration` * Verify that property has an effect via `IntegrationAutoConfigurationTests.integrationManagementInstrumentedWithObservation()`
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Nov 10, 2022
@philwebb philwebb added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Nov 10, 2022
@philwebb philwebb added this to the 3.0.x milestone Nov 10, 2022
@philwebb philwebb added the status: blocked An issue that's blocked on an external project change label Nov 10, 2022
@philwebb
Copy link
Member

Considering this one a bug of omission, but we can't add it yet because we can't move to SNAPSHOTs before our next RC.

@artembilan
Copy link
Member Author

No. This is not a bug or omission.
An observation instrumentation is a new feature in Spring Integration 6.0.
Yes, we can squeeze this for the next release for Spring Boot in two week.
Or you can just move it as a theme of next Spring Boot 3.1 (or 3.0.1 if any).

Thanks

@philwebb
Copy link
Member

By "bug of omission" I meant that it's a feature we should ideally have support for in the 3.0 release since the feature was added in Spring Integration 6. I guess you're right though, it's really an enhancement. I still think it might be worth adding to 3.0 if we can. It seems low risk.

@philwebb philwebb added type: enhancement A general enhancement and removed type: bug A general bug labels Nov 10, 2022
@artembilan
Copy link
Member Author

Sounds good, Phil!
Sorry that I came up with this property a bit late.
Let's see if it can make it into 3.0!

@wilkinsona wilkinsona removed the status: blocked An issue that's blocked on an external project change label Nov 14, 2022
@wilkinsona wilkinsona changed the title Add Spring Integration observationPatterns prop Provide a configuration property for the observation patterns of Spring Integration components Nov 14, 2022
wilkinsona pushed a commit that referenced this pull request Nov 14, 2022
Spring Integration has introduced a new observationPatterns attribute on EnableIntegrationManagement. Spring Boot auto-configures EnableIntegrationManagement so this commit adds a property that allows users to configure the patterns without declaring the annotation themselves. See gh-33099
@philwebb philwebb modified the milestones: 3.0.x, 3.0.0 Nov 14, 2022
ndwlocatieservices added a commit to ndwnu/nls-accessibility-map that referenced this pull request Aug 19, 2024
…o v3 This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [nu.ndw.nls.geometry:nls-geometry](https://spring.io/projects/spring-boot) ([source](https://github.com/spring-projects/spring-boot)) | compile | major | `2.2.0` -> `3.0.0` | --- ### Release Notes <details> <summary>spring-projects/spring-boot (nu.ndw.nls.geometry:nls-geometry)</summary> ### [`v3.0.0`](https://github.com/spring-projects/spring-boot/releases/tag/v3.0.0) [Compare Source](spring-projects/spring-boot@v2.2.1.RELEASE...v3.0.0) See the [Release notes for 3.0](https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-3.0-Release-Notes) for upgrade instructions and details of new features. #### ⭐ New Features - Provide a configuration property for the observation patterns of Spring Integration components [#&#8203;33099](spring-projects/spring-boot#33099) #### 🐞 Bug Fixes - io.micrometer.tracing.Tracer on the classpath breaks AOT processing for tests [#&#8203;33298](spring-projects/spring-boot#33298) - Tracer library HTTP instrumentation is auto-configured unnecessarily [#&#8203;33287](spring-projects/spring-boot#33287) - Auto-configuration ignores user-provided ObservationConventions [#&#8203;33285](spring-projects/spring-boot#33285) - `ScheduledBeanLazyInitializationExcludeFilter` is auto-configured even when annotation-based scheduled has not been enabled [#&#8203;33284](spring-projects/spring-boot#33284) - SpringBootContextLoader prints banner twice when using a `@ContextHierarchy` [#&#8203;33263](spring-projects/spring-boot#33263) - Properties migrator causes an application to fail to start if it tries to map a property whose metadata data entry contains an invalid configuration property name [#&#8203;33250](spring-projects/spring-boot#33250) - Wavefront MeterRegistryCustomizer is not applying application tags from application.properties [#&#8203;33244](spring-projects/spring-boot#33244) - Actuator responses no longer format timestamps as ISO-8601 [#&#8203;33236](spring-projects/spring-boot#33236) - Configuration property is not bound in a native image when property has get, set, and is methods [#&#8203;33232](spring-projects/spring-boot#33232) - Configuration property binding does not deal with bridge methods [#&#8203;33212](spring-projects/spring-boot#33212) - Contribute missing resource hints for GraphQL schema files and GraphiQL HTML page [#&#8203;33208](spring-projects/spring-boot#33208) - Hints for ClientHttpRequestFactory should only be generated for matching methods [#&#8203;33203](spring-projects/spring-boot#33203) -...
ndwlocatieservices added a commit to ndwnu/nls-routing-map-matcher that referenced this pull request Aug 21, 2024
…o v3 This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [nu.ndw.nls.geometry:nls-geometry](https://spring.io/projects/spring-boot) ([source](https://github.com/spring-projects/spring-boot)) | compile | major | `2.1.4` -> `3.0.0` | --- ### Release Notes <details> <summary>spring-projects/spring-boot (nu.ndw.nls.geometry:nls-geometry)</summary> ### [`v3.0.0`](https://github.com/spring-projects/spring-boot/releases/tag/v3.0.0) [Compare Source](spring-projects/spring-boot@v2.2.1.RELEASE...v3.0.0) See the [Release notes for 3.0](https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-3.0-Release-Notes) for upgrade instructions and details of new features. #### ⭐ New Features - Provide a configuration property for the observation patterns of Spring Integration components [#&#8203;33099](spring-projects/spring-boot#33099) #### 🐞 Bug Fixes - io.micrometer.tracing.Tracer on the classpath breaks AOT processing for tests [#&#8203;33298](spring-projects/spring-boot#33298) - Tracer library HTTP instrumentation is auto-configured unnecessarily [#&#8203;33287](spring-projects/spring-boot#33287) - Auto-configuration ignores user-provided ObservationConventions [#&#8203;33285](spring-projects/spring-boot#33285) - `ScheduledBeanLazyInitializationExcludeFilter` is auto-configured even when annotation-based scheduled has not been enabled [#&#8203;33284](spring-projects/spring-boot#33284) - SpringBootContextLoader prints banner twice when using a `@ContextHierarchy` [#&#8203;33263](spring-projects/spring-boot#33263) - Properties migrator causes an application to fail to start if it tries to map a property whose metadata data entry contains an invalid configuration property name [#&#8203;33250](spring-projects/spring-boot#33250) - Wavefront MeterRegistryCustomizer is not applying application tags from application.properties [#&#8203;33244](spring-projects/spring-boot#33244) - Actuator responses no longer format timestamps as ISO-8601 [#&#8203;33236](spring-projects/spring-boot#33236) - Configuration property is not bound in a native image when property has get, set, and is methods [#&#8203;33232](spring-projects/spring-boot#33232) - Configuration property binding does not deal with bridge methods [#&#8203;33212](spring-projects/spring-boot#33212) - Contribute missing resource hints for GraphQL schema files and GraphiQL HTML page [#&#8203;33208](spring-projects/spring-boot#33208) - Hints for ClientHttpRequestFactory should only be generated for matching methods [#&#8203;33203](spring-projects/spring-boot#33203) -...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement A general enhancement

4 participants