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

Fails to get obmc-console object from srvcfg-manager #27

Open
jonathan-doman opened this issue Oct 12, 2023 · 2 comments
Open

Fails to get obmc-console object from srvcfg-manager #27

jonathan-doman opened this issue Oct 12, 2023 · 2 comments

Comments

@jonathan-doman
Copy link
Contributor

jonathan-doman commented Oct 12, 2023

auto servicePath = ipmi::getDbusObject(
bus, "xyz.openbmc_project.Control.Service.Attributes",
"/xyz/openbmc_project/control/service", "_6fbmc_2dconsole");

This line always fails, leading to a constant stream of error messages when SOL session is active.

Oct 12 17:20:05 netipmid[299]: Failed to find object which matches
Oct 12 17:20:05 netipmid[299]: The operation failed internally.
Oct 12 17:20:05 netipmid[299]: Failed to get service path in registerSOLServiceChangeCallback: xyz.openbmc_project.Common.Error.InternalFailure: The operation failed internally.

srvcfg-manager does not publish this object, but rather only the instantiated service names, e.g. obmc_2dconsole_40ttyS2. AFAIK netipmid does not know which instance to use so there's no simple one-line fix for this.

@mdmillerii
Copy link

The serial console server exports connection information over dbus https://github.com/openbmc/obmc-console/blob/955d140e3caafb8a082a094d4fb6ebd2e3d7782a/console-dbus.c#L31

https://github.com/openbmc/phosphor-dbus-interfaces/tree/master/yaml/xyz/openbmc_project/Console

But I don't understand what ipmi is doing with the service manager. It looks like some kind of hook invoked when the sol session parameters is changed but what is it trying to accomplish?

From 15 minutes of study it's restarting the whole console service instead of killing and restarting it's session when configuration such as required encryption is changed. I don't believe it shouldn't be trying restart or start/stop the multiple interfaces of the console service. Also if the administrator had disabled the service though redfish I doubt ipmi would be expected to enable it.

@Kostr
Copy link
Contributor

Kostr commented Oct 13, 2023

I think that is the relevant commit that describes what is the purpose of the srvcfg-manager in this case f6e7230

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

No branches or pull requests

3 participants