Initial version of httpd plan. #266
Conversation
|
This PR has passed 'Verify' and is ready for review and approval! |
| # documents. By default, all requests are taken from this directory, but | ||
| # symbolic links and aliases may be used to point to other locations. | ||
| # | ||
| DocumentRoot "/opt/bldr/srvc/httpd/htdocs" |
There was a problem hiding this comment.
FWIW in nginx, we use data:
root /opt/bldr/srvc/nginx/data
There was a problem hiding this comment.
what do you do for cgi-bin and things like that?
There was a problem hiding this comment.
It appears the convention comes from the pkg_srvc* variables in bldr-build.
$pkg_srvc: Where the running service is;/opt/bldr/srvc/$pkg_name$pkg_srvc_data: Service data;/opt/bldr/srvc/$pkg_name/data$pkg_srvc_var: Variable state;/opt/bldr/srvc/$pkg_name/var$pkg_srvc_config: Configuration;/opt/bldr/srvc/$pkg_name/config$pkg_srvc_static: Static data;/opt/bldr/srvc/$pkg_name/static
So perhaps data/htdocs, data/cgi-bin?
There was a problem hiding this comment.
Ah yeah that would make sense. Thanks
|
This PR has passed 'Verify' and is ready for review and approval! |
| @@ -0,0 +1,4 @@ | |||
| #!/bin/sh | |||
| mkdir -p /opt/bldr/srvc/httpd/htdocs | |||
| mkdir -p /opt/bldr/srvc/httpd/logs | |||
There was a problem hiding this comment.
I'm not sure about logs being in an /opt location, seems it should be /var/opt. But I'm not sure how much we care about "FHS" conventions here.
There was a problem hiding this comment.
As I commented above, we have
$pkg_srvc_var: Variable state;/opt/bldr/srvc/$pkg_name/var
from bldr-build. I think given this, we would have /opt/bldr/srvc/httpd/var/logs for the log configuration in this plan.
There was a problem hiding this comment.
Yeah, I'm pretty sure we're on the FFHS path here 😀
There was a problem hiding this comment.
Which is fine, but the biggest issue we're probably going to face is "where the hell are the logs?!"
Of course, documentation is key with that.
|
works! @delivery approve |
|
Failed to unknown change: 54f1c58e-8a41-403d-9a4d-0115ab8c7f03 |
jtimberman
force-pushed
the
jcd/apache2-plan
branch
from
deb8894 to
cf83e44
Compare
|
This PR has passed 'Verify' and is ready for review and approval! |
|
@jtimberman did you have more modifications to this plan or are we gtg? |
|
I have no other modifications. @delivery approve |
Merged change 54f1c58e-8a41-403d-9a4d-0115ab8c7f03 From review branch jcd/apache2-plan into master Signed-off-by: jtimberman <[email protected]>
No description provided.