Remove unused Phylum/Postman CI jobs; document workflows in .github/README.md - #1075
Merged
Conversation
…EADME.md Phylum and Postman sync are no longer used, so drop the phylum-analyze job from ci.yml and delete the standalone phylum-daily-analysis.yaml and postman.yml workflows entirely. Nothing else depends on these jobs via needs:. Also adds .github/README.md documenting every workflow's trigger, release sequence, and external dependencies (secrets, AWS roles, self-hosted runners, GitHub environments).
The Snyk deps/licences and code scan jobs fail on every PR with ResourceNotFoundException fetching AWS_SECRET_NAME_SNYK - the secret path no longer exists, and there's no Snyk integration to fix it for anymore.
apt-get install fails with exit code 100 across three build stages because several exact-pinned bookworm package versions have been superseded and removed from the mirror: - library-build-stage-base: curl deb12u14 -> deb12u15, libssl-dev deb12u1 -> deb12u2 (this stage backs core-rust's buildRustForDocker / make build-core, i.e. the "Build debian package" and "Test core-rust docker build" CI jobs) - java-build-stage: libssl-dev deb12u1 -> deb12u2. Also pin openjdk-17-jre/-jdk-headless/-jre-headless alongside openjdk-17-jdk, since apt's resolver was picking the newer bookworm-security build for the unpinned transitive deps and then failing to satisfy the older jdk's exact-version Depends - app-container: curl deb12u14 -> deb12u15 Verified each stage builds cleanly against the live bookworm mirror.
|
Docker tags |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Phylum, Postman sync, and Snyk are no longer used/working in this repo, so this PR removes them, documents the remaining workflows, and fixes a real CI build breakage found while verifying this branch.
CI cleanup
phylum-analyzejob fromci.ymland deletedphylum-daily-analysis.yamlentirely (no Phylum access anymore).postman.ymlentirely (Postman collection sync is not used).snyk-scan-deps-licencesandsnyk-scan-codejobs fromci.yml— both failed on every PR withResourceNotFoundExceptionfetchingAWS_SECRET_NAME_SNYK(the secret no longer exists), and there's no Snyk integration to fix it for.grepthat no job inci.ymlordocker.ymldepends on any of the removed jobs vianeeds:..github/README.md: which workflows run automatically vs. need manualworkflow_dispatch, the release sequence, and a table of each job's external dependencies (secrets, AWS IAM roles/Secrets Manager paths, self-hosted runner requirements, GitHub environment gates, third-party services).Dockerfile fix
While validating this branch's CI, found
mainis currently broken:apt-get installfails with exit 100 across three build stages because several exact-pinned Debian bookworm package versions have been superseded and removed from the mirror. Fixed:library-build-stage-base(backscore-rust'sbuildRustForDocker/make build-core— i.e. the "Build debian package" and "Test core-rust docker build" CI jobs):curldeb12u14→deb12u15,libssl-devdeb12u1→deb12u2.java-build-stage:libssl-devdeb12u1→deb12u2. Also pinnedopenjdk-17-jre/-jdk-headless/-jre-headlessalongsideopenjdk-17-jdk— apt's resolver was picking the newerbookworm-securitybuild for the unpinned transitive deps and then failing to satisfy the olderjdk's exact-versionDepends.app-container:curldeb12u14→deb12u15.Testing
grepthat no job depends on any removed job vianeeds:— safe to remove without breaking dependency chains.docker build --target library-build-stage-baseanddocker build --target java-build-stageboth succeed; theapp-containerpackage set was verified by isolating and running its exactapt-get installline..github/README.mdis documentation-only; no workflow behavior changes there.Note: this does not fix the Rust-toolchain-too-old failure also present on
main(iter_repeat_n/is_multiple_ofcompile errors under the pinned1.81.0toolchain) — that's a separate, unrelated breakage tracked outside this PR.