Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bugfix:forget to set metrics & log ch as well #155

Merged
merged 3 commits into from
Dec 5, 2023

Conversation

icodeasy
Copy link
Contributor

@icodeasy icodeasy commented Dec 4, 2023

for the setting SW_AGENT_REPORTER_GRPC_MAX_SEND_QUEUE in the https://github.com/apache/skywalking-go/blob/99c2c38ff78507db17aa14839047219a2a6c99b8/tools/go-agent/config/agent.default.yaml

found out that its not working on metricsSendCh & logSendCh chans

@wu-sheng wu-sheng requested a review from mrproliu December 4, 2023 13:42
@wu-sheng wu-sheng added this to the 0.4.0 milestone Dec 4, 2023
@wu-sheng wu-sheng added the bug Something isn't working label Dec 4, 2023
@mrproliu
Copy link
Contributor

mrproliu commented Dec 4, 2023

Could you please update the CHANGES.md?

@icodeasy
Copy link
Contributor Author

icodeasy commented Dec 5, 2023

Could you please update the CHANGES.md?

done, thank you for quick response

mrproliu
mrproliu previously approved these changes Dec 5, 2023
Copy link
Contributor

@mrproliu mrproliu left a comment

Choose a reason for hiding this comment

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

LGTM

CHANGES.md Show resolved Hide resolved
Co-authored-by: 吴晟 Wu Sheng <[email protected]>
@mrproliu mrproliu merged commit f9929c6 into apache:main Dec 5, 2023
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants