sbp commented on issue #500: URL: https://github.com/apache/tooling-trusted-releases/issues/500#issuecomment-3739571144
Unlike our Bootstrap and Playwright containers, our e2e tests did not use a custom context in the corresponding compose file. I fixed that in c21981fc0c3a0714ea94dd6a8430b71b891b19cf. The consequence is that the root `.dockerignore` file now only applies to `Dockerfile.alpine`, which enabled me to use the proposed allowlist strategy. I found that we were requiring some directories only for the purpose of docs generation, so I made a new docs generator `Makefile` target that doesn't perform the generation. (I may need to update our linter or workflows.) I did this, and migrated the `.dockerfile` to the new strategy, in commit eb1934ec20f5cccfb4e842ec8e7caec4a2fc965f. I've restarted our deployed containers with the new code and they appear to still work. Thanks very much for the suggestion. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
