Skip to content

Sonic Chassis http-server HLD - #1289

Closed
anamehra wants to merge 6 commits into
sonic-net:masterfrom
anamehra:anamehra/http-server
Closed

anamehra wants to merge 6 commits into
sonic-net:masterfrom
anamehra:anamehra/http-server

Conversation

@anamehra

@anamehra anamehra commented Mar 16, 2023 •

Copy link
Copy Markdown
Contributor

Signed-off-by: anamehra anamehra@cisco.com

Sonic Chassis network http-server HLD

Repo PR title State
sonic-buildimage database-chassis python http-server implementation PR13345  

Signed-off-by: anamehra <anamehra@cisco.com>
@anamehra

Copy link
Copy Markdown
Contributor Author

Hi @rlhui , @abdosi , Please review. Thanks

@anamehra

Copy link
Copy Markdown
Contributor Author

Hi @abdosi , @qiluo-msft , please review.

http_server_dir=/var/www/tftp
```

The service will be added as a new process in docker database-chassis. dockers/docker-database/supervisord.conf.j2 will be modified to support the new service. This service should run only on database-chassis server, which also hosts redis_chassis.server. Other database dockers should not run the http-server.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

not sure why re-use database docker not starting its own docker for this http_server?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hi @lguohan , this feature was discussed in Chassis community group last year and adding http-server docker was one of the options. But as this is a light weight python http-server, community was more in favor of adding a host side http-server feature. A PR was raised for that but that was an issue enabling the http-server on host side as a feature as it required the service to run a docker.

Later in the community discussion it was decided as its needed in 202205 and only for chassis, to add it as a light weight service in database-chassis which only runs for chassis.
This service is only required for Chassis, binds only to midplane/internal network and uses redis_chassis.server IP to bind to host the server.

@abdosi , please add if I missed anything.

Let me know if we need to have a quick meeting to close on this.
Thanks


### Architecture Design

The new http-server shall use the current Sonic architecture and shall not require any architecture design change. The server shall run only on the supervisor and inside the existing database-chassis docker, which runs only on supervisor.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why put this service in database container? I do not see any dependency.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's in database-chassis container which runs only on Supervisor. Please see above comment for more details. Thanks

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.

3 participants