openstreetmap-tile-server icon indicating copy to clipboard operation
openstreetmap-tile-server copied to clipboard

renderd.sock

Open mpele opened this issue 2 years ago • 5 comments

I was just looking the code and noticed that the location of the renderd.sock is not consistent:

https://github.com/Overv/openstreetmap-tile-server/blob/25a661e3da27c1fc91d5b586eed09cb088f49ab9/openstreetmap-tiles-update-expire.sh#L194 https://github.com/Overv/openstreetmap-tile-server/blob/81c343305595fb8ae976dbbcf5a7fa2efd4093c1/apache.conf#L6

Is this a mistake?

mpele avatar Aug 23 '22 07:08 mpele

That's a great question, that would've happened when I migrated from compiled renderd to the one available in the Ubuntu 22.04 repos. I hadn't noticed an issue when making the PR and later use of this, so I don't know what the impacts are if any

galewis2 avatar Aug 23 '22 15:08 galewis2

The one in the apache.conf seems to be the new one.

If render_expired fails to communicate with renderd because of the wrong socket, then the tiles that were changed by the update might not get re-rendered and will instead keep their old content.

Any "Expiry failed" messages in the run log maybe?

Istador avatar Aug 23 '22 15:08 Istador

I'll have a look at this in the next couple days to see if there's any impacts that I missed.

galewis2 avatar Aug 25 '22 13:08 galewis2

I struggled to see any errors, but I didn't notice any tiles get re-rendered. So I will just fix this and I will have someone else check it out because I certainly agree it should cause issues

galewis2 avatar Aug 26 '22 17:08 galewis2

So /var/run is symlinked to /run, so it should be inconsequential. But I will still make the fix for consistency!

galewis2 avatar Aug 26 '22 17:08 galewis2

https://github.com/Overv/openstreetmap-tile-server/pull/314

nielsole avatar Mar 18 '23 20:03 nielsole