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

Container to Host Communications using D-Bus #541

Merged
merged 9 commits into from Jun 3, 2020

Conversation

mikelazar
Copy link
Contributor

No description provided.

nirenjan and others added 4 commits December 17, 2019 21:22
Signed-off-by: Mike Lazar <mihai_lazar@dell.com>
(cherry picked from commit e3fe0cf)
…-doc

Dbus container  to host doc
(cherry picked from commit fdea75d)
HLD - D-Bus Container to Host Communications
(cherry picked from commit d17d2a4)
Signed-off-by: Mike Lazar <mihai_lazar@dell.com>
Add extra D-Bus security info
Copy link
Contributor Author

@mikelazar mikelazar left a comment

Choose a reason for hiding this comment

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

Replying to previous comments from "dev-aziz"

  1. why D-Bus used specific listed services/activities alone, why can't it use for all the services? Can you add some guidelines to the document?

We have listed some possible examples. As mentioned in the community meeting, I will remove ZTD from this list. We can have a separate discussion about ZTD.

I'll also add guidelines about when to use the D-Bus mechanism.

  1. How to secure the users shouldn't misuse the critical host access to commands like reboot? Can you provide some guidelines around it?

This is already mentioned in the security section ("Security of D-Bus Communications"). Access to all DBus commands is protected through ACL rules in the DBus configuration file - only certain user or groups can access specific set of command. However, I want to make it very clear that RBAC is implemented in the SONiC Mgmt Framework and not DBus, thus RBAC is completely outside the scope of this document.

  1. How does host communicate to ex: reboot/warm reboot execution status to clients (containers issued commands to host)? How does D-Bus helps here?

DBus allows us to return the output of any command executed on the host in the DBus method reply. Since DBus methods are remote procedure calls, they return values , e.g. the string value returned by the reboot command. Same for all the other commands implemented using DBus (config save/restore, image upgrade etc.).
Reboot case: Note that as long as the device does not stop the SONiC mgmt framework before the latter had the chance to display the string returned by the reboot command (e.g. "COLD reboot initiated"), via DBus, that command will be displayed to the user. The behavior is exactly the same if we used another communication mechanism vs. DBus.
This is already described in section 3.1.

"D-Bus provides a reliable communication channel between client (SONiC management container) and service (native host OS) – all actions are acknowledged and can provide return values. It should be noted that acknowledgements are important for operations such as “image upgrade” or “config-save”. In addition, D-Bus methods can return values of many types – not just ACKs. For instance, they can return strings, useful to return the output of a command."

  1. Will the design provide some D-Bus service infrastructure level audit mechanism who does what w.r.t host commands like reboot/warm reboot etc

We can log the user name and group of the application which makes the request. However, in the case of SONiC management, a full command audit (for all commands, not just those requested via DBus) will be implemented by the REST server, in future releases.

doc/mgmt/Docker to Host communication.md Outdated Show resolved Hide resolved
doc/mgmt/Docker to Host communication.md Outdated Show resolved Hide resolved
@renukamanavalan renukamanavalan merged commit 36a70b9 into sonic-net:master Jun 3, 2020
kartik-arista pushed a commit to kartik-arista/SONiC that referenced this pull request Jul 16, 2020
This document describes the high level design of container to host OS communications using D-Bus, and describes the infrastructure provided by the feature, with usage examples
@toravir
Copy link

toravir commented Jan 23, 2023

@mikelazar does these guidelines apply when a process running on the host wants to initiate communication with a docker. I am working on a process and considering running it on the host and having it directly communicate with the redis container to access db.

@mikelazar
Copy link
Contributor Author

@mikelazar does these guidelines apply when a process running on the host wants to initiate communication with a docker. I am working on a process and considering running it on the host and having it directly communicate with the redis container to access db.

@toravir
This mechanism is meant to communicate from container TO host, not vice-versa, host TO container. SONiC provides another means to access the Redis DB. In addition, please note that running processes on the host is generally not a good idea in SONiC - processes/services need to run as containers.

@toravir
Copy link

toravir commented Jan 24, 2023

@mikelazar what is the another means to access Redis DB ? I am considering running PTP on Sonic and the recommendation is to run it on the host - https://github.com/sonic-net/SONiC/blob/master/doc/ptp/ptp.md

@mikelazar
Copy link
Contributor Author

@mikelazar what is the another means to access Redis DB ? I am considering running PTP on Sonic and the recommendation is to run it on the host - https://github.com/sonic-net/SONiC/blob/master/doc/ptp/ptp.md

@toravir In general, please ask this type of questions using the SONiC mailing lists. If I recall correctly, Redis DB/SONiC is configured to use a Unix domain socket for client (application) to server communications. That socket is available for all containers, and for the host as well. So you can just use the Redis DB client library (Python or Hiredis). There might be a SONiC Redis DB wrapper - but I do not recall exactly. However, please use that for SONiC you need to define a DB schema using YANG.

Also, not sure who made the recommendation to run PTP on the host. This might be needed ONLY if you need to communicate with a non-standard Linux kernel module/driver directly (and even that's debatable in the case of SONiC). However, I do not see that dependency in the design document (the PHC driver seems to be standard in kernels >= 5.8). The only thing required for the host OS is to load the PHC driver, when PTP is enabled.

Anyway, this is something to discuss in the SONiC community meetings (and possibly using the SONiC mailing lists).

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.

None yet

6 participants