Skip to content

feat(frontend): allow switching logs inside logs tab#2794

Merged
talos-bot merged 3 commits into
siderolabs:mainfrom
Slessi:service-logs-ux
May 12, 2026
Merged

feat(frontend): allow switching logs inside logs tab#2794
talos-bot merged 3 commits into
siderolabs:mainfrom
Slessi:service-logs-ux

Conversation

@Slessi
Copy link
Copy Markdown
Member

@Slessi Slessi commented May 6, 2026

Allow switching between machine and service logs from inside the logs tab via a dropdown of services. Also includes some refactoring to allow the same behaviour on both machine page and node page.

Closes #2712

image

@Unix4ever
Copy link
Copy Markdown
Member

I think dropdown height is 2 pixels bigger than the input. We can either do item-center align there, or fix the difference. Maybe 2nd option is prefferable.

@Unix4ever
Copy link
Copy Markdown
Member

Lets also manually add controller-runtime to the list of services.
It's not reported as the running service, but you can get the logs for it still.

@frezbo
Copy link
Copy Markdown
Member

frezbo commented May 6, 2026

Lets also manually add controller-runtime to the list of services. It's not reported as the running service, but you can get the logs for it still.

there;s also auditd and syslogd

@Slessi
Copy link
Copy Markdown
Member Author

Slessi commented May 6, 2026

@frezbo those ones are part of the service list already

@Unix4ever sure but should it be part of the service list? is fixing the backend API the correct fix or shall i just manually hardcode in frontend?

hardcoded for now, had to do some regex for parsing and to strip the ANSI colours

Slessi added 3 commits May 12, 2026 10:17
Extract logic to get list of machine services into a useMachineServices composable.

Signed-off-by: Edward Sammut Alessi <edward.sammutalessi@siderolabs.com>
Move NodeLogs logic into MachineLogsContainer. This deduplicates the code paths between machine vs service logs, and allows viewing service logs from the machines log tab as well.

Signed-off-by: Edward Sammut Alessi <edward.sammutalessi@siderolabs.com>
Allow switching between machine and service logs from inside the logs tab via a dropdown of services

Signed-off-by: Edward Sammut Alessi <edward.sammutalessi@siderolabs.com>
Copy link
Copy Markdown
Member

@utkuozdemir utkuozdemir left a comment

Choose a reason for hiding this comment

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

Don't count on my review on frontend-specific stuff, but logic-wise I don't see any issues.

@github-project-automation github-project-automation Bot moved this from In Review to Approved in Planning May 12, 2026
@Unix4ever
Copy link
Copy Markdown
Member

@frezbo those ones are part of the service list already

@Unix4ever sure but should it be part of the service list? is fixing the backend API the correct fix or shall i just manually hardcode in frontend?

hardcoded for now, had to do some regex for parsing and to strip the ANSI colours

We can't add it to the backend response as it's coming from Talos

@Slessi
Copy link
Copy Markdown
Member Author

Slessi commented May 12, 2026

/m

@talos-bot talos-bot merged commit cab0621 into siderolabs:main May 12, 2026
132 of 140 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Done in Planning May 12, 2026
@Slessi Slessi deleted the service-logs-ux branch May 12, 2026 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Make service logs UX more clear

5 participants