[Geoserver-devel] System variable for log file location

GeoServer GeoServer Developer
1

Hi,
in a production system we're setting up we have a cluster of GeoServer all
sharing the same NFS mounted data directory.
Generally it works fine but the log file is shared among all the instances,
which is not exactly the best solution (as the writes will happen concurrently).

I think for this case the solution would be to allow the log file location to be
specified by system variable, env variable or servlet context param, so that
each instance can have its own separate log file.

I've just posted a patch that appears to be working fine for review here:
http://jira.codehaus.org/browse/GEOS-4334

I'd like to commit this on 2.1.x. The patch is simple but hitting a core
code path so I'm looking for another pair of eyes to look at it :slight_smile:

Cheers
Andrea

--
Ing. Andrea Aime
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy

phone: +39 0584962313
fax: +39 0584962313

http://www.geo-solutions.it
http://geo-solutions.blogspot.com/
http://www.linkedin.com/in/andreaaime
http://twitter.com/geowolf

-----------------------------------------------------

2

Patch looks good. Added my +1.

On Thu, Jan 27, 2011 at 5:18 AM, Andrea Aime <andrea.aime@anonymised.com> wrote:

Hi,
in a production system we’re setting up we have a cluster of GeoServer all
sharing the same NFS mounted data directory.
Generally it works fine but the log file is shared among all the instances,
which is not exactly the best solution (as the writes will happen concurrently).

I think for this case the solution would be to allow the log file location to be
specified by system variable, env variable or servlet context param, so that
each instance can have its own separate log file.

I’ve just posted a patch that appears to be working fine for review here:
http://jira.codehaus.org/browse/GEOS-4334

I’d like to commit this on 2.1.x. The patch is simple but hitting a core
code path so I’m looking for another pair of eyes to look at it :slight_smile:

Cheers
Andrea

–
Ing. Andrea Aime
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy

phone: +39 0584962313
fax: +39 0584962313

http://www.geo-solutions.it
http://geo-solutions.blogspot.com/
http://www.linkedin.com/in/andreaaime
http://twitter.com/geowolf

Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)!
Finally, a world-class log management solution at an even better price-free!
Download using promo code Free_Logger_4_Dev2Dev. Offer expires
February 28th, so secure your free ArcSight Logger TODAY!
http://p.sf.net/sfu/arcsight-sfd2d

Geoserver-devel mailing list
Geoserver-devel@anonymised.comsts.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

–
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.

3

On Thu, Jan 27, 2011 at 3:39 PM, Justin Deoliveira <jdeolive@anonymised.com> wrote:

Patch looks good. Added my +1.

Thanks for the review (Gabriel too).
I've committed the changes

Cheers
Andrea

--
Ing. Andrea Aime
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy

phone: +39 0584962313
fax: +39 0584962313

http://www.geo-solutions.it
http://geo-solutions.blogspot.com/
http://www.linkedin.com/in/andreaaime
http://twitter.com/geowolf

-----------------------------------------------------

4

I think this is a great idea.

For archive purposes, and for those who didn't read the patch, and until it shows up in the docs (you may be way ahead of me), I thought we should explicity state here that the variable name is GEOSERVER_LOG_LOCATION. (I hope I read the patch correctly!)

Thanks,
Mike Pumphrey
OpenGeo - http://opengeo.org

On 1/27/2011 7:36 AM, Andrea Aime wrote:

On Thu, Jan 27, 2011 at 3:39 PM, Justin Deoliveira<jdeolive@anonymised.com> wrote:

Patch looks good. Added my +1.

Thanks for the review (Gabriel too).
I've committed the changes

Cheers
Andrea

5

On Sat, Jan 29, 2011 at 12:20 AM, Mike Pumphrey <mike@anonymised.com> wrote:

I think this is a great idea.

For archive purposes, and for those who didn't read the patch, and until it
shows up in the docs (you may be way ahead of me), I thought we should
explicity state here that the variable name is GEOSERVER_LOG_LOCATION. (I
hope I read the patch correctly!)

Whoops, you got me :slight_smile:
I just added a new advanced page talking about about this and other
system variables
that do control logging.
Here it is:

http://docs.geoserver.org/stable/en/user/advanced/logging.html

Cheers
Andrea

--
Ing. Andrea Aime
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy

phone: +39 0584962313
fax: +39 0584962313

http://www.geo-solutions.it
http://geo-solutions.blogspot.com/
http://www.linkedin.com/in/andreaaime
http://twitter.com/geowolf

-----------------------------------------------------