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

Improve application error logs about Syslog drain problems #579

Open
chombium opened this issue May 29, 2024 · 2 comments · May be fixed by #633
Open

Improve application error logs about Syslog drain problems #579

chombium opened this issue May 29, 2024 · 2 comments · May be fixed by #633
Assignees

Comments

@chombium
Copy link
Contributor

chombium commented May 29, 2024

At SAP, we were analyzing the problems which our customers have when creating Syslog drains and the whole user experience when creating and configuring Syslog drains.

We have noticed the following:

  • The Syslog Drains are created with as user provided service and there is no validation on creation time about connectivity and proper configuration in case if mTLS is configured;
  • Only the foundation operators are informed about the problems with Sylsog drains

We should add application related logs in the app log stream where ever it is possible, so that the app devs are also informed about problems with their syslog drains.

I've checked the code and we the app_guid in the binding poller. We could propagate the app_guid and add a go-loggregator client with which we will emit the logs in the app's log stream. There is already something coded in the emitLoggregatorErrorLog in the Syslog Connector.

We will provide a PR about this.

@chombium chombium changed the title Improve error logging for Syslog Agents Improve application error logs about Syslog drain problems May 29, 2024
@ctlong ctlong moved this from Inbox to Waiting for Changes | Open for Contribution in Application Runtime Platform Working Group Jun 6, 2024
@chombium chombium self-assigned this Sep 25, 2024
@chombium
Copy link
Contributor Author

chombium commented Sep 25, 2024

Someone from our team will do it ;)

@chombium chombium removed their assignment Sep 26, 2024
@chombium chombium assigned chombium and unassigned chombium Oct 22, 2024
@corporatemax
Copy link

@chombium I do it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Waiting for Changes | Open for Contribution
Development

Successfully merging a pull request may close this issue.

2 participants