Skip to content

Wavefront application tags differ from those used in a Spring Boot 2.x application#32844

Closed
oppegard wants to merge 5 commits into
spring-projects:mainfrom
oppegard:wf-app-tags
Closed

Wavefront application tags differ from those used in a Spring Boot 2.x application#32844
oppegard wants to merge 5 commits into
spring-projects:mainfrom
oppegard:wf-app-tags

Conversation

@oppegard

@oppegard oppegard commented Oct 21, 2022

Copy link
Copy Markdown
Contributor

This PR fleshes out the use of Wavefront Application Tags, used in Wavefront to organize a service within a larger distributed application.

  • application (required): this is the name of a distributed application comprised of multiple microservices (not to be confused with an individual service, below).
  • service (required): this is the name of the current service that is part of the larger application. It will use spring.application.name if available.
  • cluster (optional)
  • shard (optional)

This PR add two new properties:

  • management.wavefront.tracing.cluster-name
  • management.wavefront.tracing.shard-name

In wavefront-spring-boot these properties were wavefront.application.cluster and wavefront.application.shard. I'm not sure if there's a way to migrate users of the old properties (same goes for wavefront.application.name and wavefront.application.service).

This PR also adds Application Tags to metrics exported by instances of WavefrontMeterRegistry, replacing functionality from wavefront-spring-boot.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Oct 21, 2022
@philwebb philwebb added for: team-meeting An issue we'd like to discuss as a team to make progress type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged for: team-meeting An issue we'd like to discuss as a team to make progress labels Oct 25, 2022
@philwebb philwebb added this to the 3.1.x milestone Oct 26, 2022
@philwebb philwebb added type: bug A general bug and removed type: enhancement A general enhancement labels Nov 16, 2022
@philwebb philwebb modified the milestones: 3.1.x, 3.0.x Nov 16, 2022
@philwebb philwebb changed the title Enhance use of Wavefront Application Tags Wavefront application tags differ from those used in a Spring Boot 2.x application Nov 16, 2022
philwebb pushed a commit that referenced this pull request Nov 17, 2022
Update `application-name` and `service-name` `management.wavefront`
properties to align with Spring Boot 2.x support. The properties now
use the wavefront terminology where `application` refers to an a
collection of (micro)`services`.

This commit also adds `cluster-name` and `shard-name` properties that
can be used to add additional items to the `ApplicationTags`.

See gh-32844
@philwebb philwebb closed this in 7525c0f Nov 17, 2022
@philwebb philwebb modified the milestones: 3.0.x, 3.0.0 Nov 17, 2022
@oppegard oppegard deleted the wf-app-tags branch November 17, 2022 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug A general bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants