Skip to content

Conversation

@anon-pradip
Copy link
Contributor

@anon-pradip anon-pradip commented Nov 17, 2025

Description

Enables Unicode text preview support for the thumbnails service in Kubernetes and ensures consistent thumbnail generation between test builds and production images.

Problem

  1. K8s deployments: Thumbnails service couldn't render Unicode characters (e.g., Devanagari script) due to missing fonts
  2. Inconsistent test builds: Test builds used standard Go image/jpeg encoder while production Docker images (built with ENABLE_VIPS=true) use libvips encoder, causing different JPEG output and test failures

Changes

  • K8s font support: Added ConfigMaps for NotoSans font and font configuration, mounted into thumbnails deployment
  • Consistent test builds: Added ENABLE_VIPS=true to test builds to match production images (which already have VIPS enabled)
  • Fixtures: Updated 5 preview fixtures to match VIPS output (JPEG with EXIF metadata):
    • unicode-fixture.png: 620 → 739 bytes
    • fit.png: 713 → 885 bytes
    • fill.png: 874 → 1029 bytes
    • resize.png: 829 → 984 bytes
    • thumbnail.png: 874 → 1029 bytes

Note

Production Docker images have been built with ENABLE_VIPS=true. This PR ensures test builds use the same encoder.

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@anon-pradip anon-pradip self-assigned this Nov 17, 2025
@anon-pradip anon-pradip force-pushed the test/include-coreApi8-suite-in-k8s branch from 3d0724e to 528df24 Compare November 18, 2025 06:25
@anon-pradip anon-pradip force-pushed the test/include-coreApi8-suite-in-k8s branch from 528df24 to 71ec233 Compare November 18, 2025 06:43
@anon-pradip anon-pradip force-pushed the test/include-coreApi8-suite-in-k8s branch from 2dcd0ac to 1f82ae9 Compare November 19, 2025 05:56
@anon-pradip anon-pradip force-pushed the test/include-coreApi8-suite-in-k8s branch from de853fc to 6e5a7ff Compare November 19, 2025 06:43
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants