FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Support for tracing webflux-6.x and gateway-4.x by CzyerChen · Pull Request #661 · apache/skywalking-java · GitHub

Support for tracing webflux-6.x and gateway-4.x - #661

Merged
wu-sheng merged 3 commits into
apache:mainfrom
CzyerChen:feature/gateway4x
Dec 21, 2023
Merged

Support for tracing webflux-6.x and gateway-4.x#661
wu-sheng merged 3 commits into
apache:mainfrom
CzyerChen:feature/gateway4x

Conversation

Copy link
Copy Markdown
Contributor

Add support for spring-cloud-gateway 4.x

  • Add a unit test to verify that the fix works.
  • Explain briefly why the bug exists and how to fix it.

why:
spring-cloud-starter-gateway 4.0.0 depends on spring-boot-starter-webflux 3.0.0
spring-boot-starter-webflux 3.0.0 depends on spring-webflux 6.0.2
spring-webflux-5.x plugin can not trace the request
how:
add support for spring-webflux-6.x diff from 5.x by witnessmethod
upgrade spring-cloud-gateway to 4.x with spring-webflux-6.x diff from 3.x by witnessclass

Add an agent plugin to support spring-cloud-gateway 4.x

wu-sheng added this to the 9.2.0 milestone Dec 19, 2023
segments:
- segmentId: not null
spans:
- operationName: /provider/timeout/error

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Is this timeout built intentionally?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

in order to check error case.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

My question is, are the codes causing the timeout on testing purpose?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

according to gateway3.x, in order to check request timeout case, so add the testcasse, issue

segments:
- segmentId: not null
spans:
- operationName: /provider/timeout/error

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Why is a timeout for entry? How an entry span timeout?

Copy link
Copy Markdown
Member

Could you confirm this #661 (comment)? If the codes don't seem to trigger error, we should not have them in the expected data.

wu-sheng left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thanks for adding this.

wu-sheng changed the title Support for tracing spring-cloud-gateway 4.x in gateway-4.x-plugin Support for tracing webflux-6.x and gateway-4.x Dec 21, 2023
wu-sheng merged commit 7aec682 into apache:main Dec 21, 2023
CzyerChen deleted the feature/gateway4x branch May 11, 2024 06:13
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL