Skip to content

Fix PTY serial console#29

Merged
hertrste merged 1 commit intocyberus-technology:gardenlinuxfrom
hertrste:fix-console
Oct 27, 2025
Merged

Fix PTY serial console#29
hertrste merged 1 commit intocyberus-technology:gardenlinuxfrom
hertrste:fix-console

Conversation

@hertrste
Copy link
Collaborator

As we use the write_distributor only if TCP is used, we only add it to the serial out in that case. Not doing so results in e.g. the console not working in the PTY case, because we would overwrite the out member of the serial that was already set before.

As we use the write_distributor only if TCP is used, we only add it to
the serial out in that case. Not doing so results in e.g. the console
not working in the PTY case, because we would overwrite the out member
of the serial that was already set before.

On-behalf-of: SAP stefan.kober@sap.com
Signed-off-by: Stefan Kober <stefan.kober@cyberus-technology.de>
@hertrste
Copy link
Collaborator Author

See corresponding test: cyberus-technology/libvirt-tests#42

Copy link
Member

@phip1611 phip1611 left a comment

Choose a reason for hiding this comment

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

LGTM

@hertrste hertrste merged commit c343b19 into cyberus-technology:gardenlinux Oct 27, 2025
20 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants