[Bug 851279] Review Request: eucalyptus - Elastic Utility Computing Architecture

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=851279

--- Comment #10 from Andy Grimm <agrimm@xxxxxxxxx> ---
(In reply to comment #7)
> In file httpd-cc.conf, you have /usr/lib64 hardcoded, that's likely not
> working on 32 bits.

Ah, true; I don't ever use 32-bit, so I had not tested that yet.

> I also find weird to have :
> ServerRoot "/etc/httpd"
>
> even if i am likely too cuatious about potential side effects.

What's the suggestion here?  I used that so that specifying relative paths for
modules would work.  I'm open to other ideas here. 

> PidFile /var/run/eucalyptus/httpd.pid
> 
> would be cleaner to use the suffix -cc there too, as this would be more
> coherent with the -nc web service.

Indeed, this is a bug.

> Another potential improvement would be to have a common file that would be
> included, and maybe removing a bit the various modules ( so 1) this would
> take less memory, start faster and easier to read ).
> 
> For example :
> LoadModule userdir_module modules/mod_userdir.so
> 
> seems weird there. setenvif_module is not used in the configuration so could
> be dropped, etc, etc.
> 
> Also, since it use mpm_event, maybe the other part of the config file could
> be dropped : ( mpm_prefork_module, etc ) so the file is easier to read and
> maintain ?

I'm actually not yet sure which is our preferred mpm.  And as for the other
modules, I started with a very minimal list and got frustrated trying to figure
out which modules are now required for a functional apache instance, so I added
all of those which are enabled by default in Fedora's apache config.  I would
love to prune that list.  Suggestions on a true minimally viable set would be
great.  And yes, a common file definitely makes sense.  I'll work on that.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]