diff options
author | Andrew Jeffery <andrew@aj.id.au> | 2023-05-11 11:12:13 +0300 |
---|---|---|
committer | Patrick Williams <patrick@stwcx.xyz> | 2023-05-12 05:37:54 +0300 |
commit | 8ee46d309af05e8af0ca9b861b5edf0948cfeecc (patch) | |
tree | bd48fa8668f7deb7af892c88df6f9abd3860a023 /meta-inspur | |
parent | 344b89444cb39a06e1de843e763aed6d2fac2bf2 (diff) | |
download | openbmc-8ee46d309af05e8af0ca9b861b5edf0948cfeecc.tar.xz |
Revert "Add socket-id for the first console"
This reverts commit ec7cab9378f548e94ee4af83946b2583d1fc7194.
ec7cab9378f5 ("Add socket-id for the first console") was only a partial
fixup of the obmc-console configurations found in the tree. For
historical reasons not all platforms that support multiple consoles
install a client configuration file for all consoles. Instead they relied
on some default behaviour that was removed in obmc-console's 4e7186918599
("Fixed broken dbus interface for multiple consoles"). Applying the
configuration changes in the manner of ec7cab9378f5 ("Add socket-id for
the first console") resulted in an asymmetric configuration between the
client and server, breaking some SSH SOL instances.
As of ae2460d0b8e8 ("obmc-console: Provide a default value for
`console-id`.") in obmc-console the requirement to specify
`socket-id`[^1] is lifted. Instead, the configuration can choose to
override a default value. This restores the original behaviour and
unbreaks SSH SOL.
[^1]: Now an alias of the `console-id` configuration key
Websocket and IPMI consoles remain broken. A fix is under development
that also takes a proper approach to supporting multiple consoles in
bmcweb, and the same solution pattern can be applied in ipmid.
More discussion of the problems involved can be found here:
https://amboar.github.io/notes/2023/05/08/happenings-in-obmc-console.html
Change-Id: I274284c791758f336da6d6301dc523bac2b5dd69
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Diffstat (limited to 'meta-inspur')
-rw-r--r-- | meta-inspur/meta-fp5280g2/recipes-phosphor/console/obmc-console/obmc-console.conf | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/meta-inspur/meta-fp5280g2/recipes-phosphor/console/obmc-console/obmc-console.conf b/meta-inspur/meta-fp5280g2/recipes-phosphor/console/obmc-console/obmc-console.conf index 5d81673f76..638ae7e5d9 100644 --- a/meta-inspur/meta-fp5280g2/recipes-phosphor/console/obmc-console/obmc-console.conf +++ b/meta-inspur/meta-fp5280g2/recipes-phosphor/console/obmc-console/obmc-console.conf @@ -3,4 +3,3 @@ sirq = 4 local-tty = ttyS0 local-tty-baud = 115200 logsize = 256k -socket-id = console0 |