On 10/7/22 03:43, Bryan Davis wrote:
For the fonts themselves, should we:
- Revert the change and tell svgtranslate to move back to the grid?
- Propagate the change outward by making the same/similar change to
all php images?
- Propagate the change outward by making the same/similar change to
all base images?
- Let it be.
First of all: I don't see anything specific on the PHP images, so if we're going to keep the fonts, I think we should move them to the base images.
For the bigger picture of breaking our long held stance on "bloat", I would like to hear suggestions from y'all. If the font decision is to revert then maybe there is nothing to talk about here. If the fonts stay then I think there is a need to either document this as a rogue action that has been allowed to stand which should not set a precedent for the future or to come up with a rubric for what is allowed and why.
My concern with the current form of Docker images has been for a while that once something has been added to it, it's pretty much impossible to remove that without breaking previously-working volunteer maintained projects (for example: Python2/3 used by webservice-runner). So I'd propose to revert the change while we work out what to do with it on the medium/long term.
I do think that fonts needed for rendering images falls into the category of "specialty features for a small number of use cases" that you mention in your cloud@ post. Long-term this feels like a great use case of a build pack based image, but as I mentioned already I don't have up to date information on the current vision and roadmap for that project.
I am also open to hearing from anyone on or off list who feels that I need to make additional amends to the Toolforge admins, the Toolforge user community, or any particular individuals. I really didn't mean to make a mess, but I did and I would like to work towards correcting that as much as possible.
Thank you. I don't have anything specific in mind, but in general I don't like things happening based on private Slack threads.
Taavi