Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated the base version from bullseye 3.9-slim-bullseye to bookworm 3.9.18-slim-bookworm #775

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

robbrazil
Copy link
Contributor

Modified dependencies versions

robbrazil and others added 2 commits November 6, 2023 16:10
The new version of Ghostscript (upgraded in the previous commit) has
changed the PDF interpreter
(https://www.ghostscript.com/blog/pdfi.html). This breaks the code by
causing fonts not to be embedded.

This change passes a command line flag to Ghostcript to tell it to use
the original interpreter. While we will need to upgrade the interpreter
too, this is best done as its own piece of work when we're not also
changing the base image and other dependencies.
@robbrazil
Copy link
Contributor Author

Latest local test run:
================================== test session starts ===================================
platform linux -- Python 3.9.18, pytest-7.1.3, pluggy-1.3.0
rootdir: /home/vcap/app, configfile: pytest.ini, testpaths: tests
plugins: mock-3.8.2, forked-1.6.0, requests-mock-1.10.0, xdist-2.5.0, env-0.6.2
gw0 [210] / gw1 [210] / gw2 [210] / gw3 [210] / gw4 [210] / gw5 [210]
.................................................................................. [ 39%]
.................................................................................. [ 78%]
.............................................. [100%]
================================== 210 passed in 16.27s ==================================

@robbrazil robbrazil marked this pull request as ready for review November 7, 2023 11:25
@idavidmcdonald
Copy link
Contributor

Nice one on getting the image building and the unit tests passing!

There is some guidance in the template preview repo for how to do some basic visual regression testing. Would you be able to do have a go at that please? We don't do it often so may be worth finding a pair

https://github.com/alphagov/notifications-template-preview/blob/main/docs/visual-testing.md

There is also some useful guidance in that docs directory for how to deploy it as well to reduce the chance of problems

@robbrazil
Copy link
Contributor Author

That visual testing has a git cherry-pick -n 41f6e4605c405c37d64aa4a3160f604f948f8536 that is causing conflicts. @klssmith is reviewing that first and we will retake this test once the issue is sorted

@klssmith
Copy link
Contributor

klssmith commented Nov 7, 2023

I've updated the commit which was being used to test (#777) so we can take another look now

@samuelhwilliams
Copy link
Contributor

@robbrazil how are we getting on with this?

@idavidmcdonald
Copy link
Contributor

We didn't make much progress and were struggling to figure out testing the changes and figuring out what differences were acceptable. For that reason we moved it into blocked on our trello board and we need to figure out the next steps...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants