Code review comment for ~pappacena/launchpad-buildd:security-manifest

Revision history for this message
Thiago F. Pappacena (pappacena) wrote :

twom, I agree we should discuss this. The current approach (implemented in follow up MPs) is enclosing the manifest file content generation in try/except blocks, and always generating at least a minimum version of the file.

The steps to copy the file into the image are not protected, but there is a bit less uncertainty in those steps. For that reason, I thought it would be better to fail the build if something goes wrong in the file injection phase. But of course, if we agree I can add the guard there too.

« Back to merge proposal