Skip to content
This repository has been archived by the owner on Mar 31, 2022. It is now read-only.

Commit

Permalink
meta-ibm: remove nginx from IBM platforms.
Browse files Browse the repository at this point in the history
Nginx on OpenBMC has a number of issues that matter to openbmc.

1. It increases the binary size.  This is an issue given that OpenBMC
targets a relatively minimal flash footprint.
2. It increases the runtime overhead.  Running nginx as a reverse proxy
to the application servers causes a runtime overhead, and context switch
for every single page load, as well as an extra socket.
3. nginx doesn't implement any kind of authentication, so auth needs to
be implemented in every application server.  This removes a lot of the
advantages of the reverse proxy, and duplicates a lot of code amongst
multiple application servers
4. A number of nginx parameters run from the nginx config file.  Some of
these parameters (like cipher suite support) are desired to be changed
at runtime, rather than fixed at compile time.

Related to commit here to move system to bmcweb:
https://gerrit.openbmc-project.xyz/#/c/openbmc/meta-phosphor/+/12933/

Change-Id: I988fce8dae565808bd0eeacd8b7a71f3cc06d98f
Signed-off-by: Ed Tanous <ed.tanous@intel.com>
(cherry picked from commit 699e296)
  • Loading branch information
edtanous authored and bradbishop committed Feb 1, 2019
1 parent adfde90 commit b6639a2
Show file tree
Hide file tree
Showing 8 changed files with 2 additions and 223 deletions.
9 changes: 0 additions & 9 deletions recipes-httpd/nginx/files/gen-cert.sh

This file was deleted.

128 changes: 0 additions & 128 deletions recipes-httpd/nginx/files/nginx.conf

This file was deleted.

20 changes: 0 additions & 20 deletions recipes-httpd/nginx/files/nginx.service

This file was deleted.

8 changes: 0 additions & 8 deletions recipes-httpd/nginx/files/nginx.socket

This file was deleted.

38 changes: 0 additions & 38 deletions recipes-httpd/nginx/nginx_%.bbappend

This file was deleted.

@@ -1,2 +1,3 @@
RDEPENDS_${PN}-logging += "ibm-logging"
RDEPENDS_${PN}-extras += "nginx bmcweb"
RDEPENDS_${PN}-extras += " bmcweb"
RDEPENDS_${PN}-remove += " phosphor-rest phosphor-gevent"

This file was deleted.

This file was deleted.

1 comment on commit b6639a2

@joseph-reynolds
Copy link

Choose a reason for hiding this comment

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

See related openbmc/openbmc#3564

Please sign in to comment.