Skip to content

Provide a configuration property for the observation patterns of Spring Integration components#33099

Closed
artembilan wants to merge 1 commit into
spring-projects:mainfrom
artembilan:integration_observation
Closed

Provide a configuration property for the observation patterns of Spring Integration components#33099
artembilan wants to merge 1 commit into
spring-projects:mainfrom
artembilan:integration_observation

Conversation

@artembilan

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants