Skip to content

DLPX-99530 containerized-masking: drop SBOM_DEEP_SCAN, it builds no .deb - #419

Merged
justsanjeev merged 3 commits into
developfrom
dlpx/pr/justsanjeev/f373f398-065a-4601-8309-0417af063f1b
Oct 6, 2026
Merged

justsanjeev merged 3 commits into
developfrom
dlpx/pr/justsanjeev/f373f398-065a-4601-8309-0417af063f1b

Conversation

@justsanjeev

@justsanjeev justsanjeev commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Problem

https://ops-jenkins.eng-tools-prd.aws.delphixcloud.com/job/linux-pkg/job/develop/job/build-package/job/containerized-masking/job/post-push failing because containerized-masking doesn't build a .deb. Its build() runs :tools:docker:packageMaskingKubernetes and publishes only a zip. Its last post-push folder (#1387) contains only:

% aws s3 ls s3://snapshot-de-images/builds/jenkins-ops/linux-pkg/develop/build-package/containerized-masking/post-push/1387/
2026-10-02 01:43:55        126 BUILD_INFO
2026-10-02 01:43:55         41 GIT_HASH
2026-10-02 01:43:55         67 PACKAGE_MIRROR_URL_MAIN
2026-10-02 01:43:55         65 PACKAGE_MIRROR_URL_SECONDARY
2026-10-02 01:43:55        437 SHA256SUMS
2026-10-02 01:43:55 1228511500 masking-kubernetes-2026.6.0.0.zip

Solution

The fix requires to remove SBOM_DEEP_SCAN="false" in packages/containerized-masking/config.sh file.

Testing Done

https://ops-jenkins.eng-tools-prd.aws.delphixcloud.com/job/linux-pkg/job/develop/job/build-package/job/containerized-masking/job/post-push/1400/ 🏃‍♂️

@justsanjeev
justsanjeev force-pushed the dlpx/pr/justsanjeev/f373f398-065a-4601-8309-0417af063f1b branch from 7905d5a to 72cf3de Compare October 6, 2026 11:17
justsanjeev and others added 2 commits October 6, 2026 17:03
… unset

Dropping the assignment left the flag unclassified, failing the
verify-sbom-scan-flag check added in #418, which requires every
package to explicitly opt in or out.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…sking

The doc described containerized-masking's SBOM_DEEP_SCAN as removed/
unset, but 325d309 set it explicitly to "false" instead, so the false
count was off by one and the explanatory text no longer matched.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@justsanjeev
justsanjeev marked this pull request as ready for review October 6, 2026 11:38
@justsanjeev
justsanjeev merged commit 05db8fb into develop Oct 6, 2026
14 checks passed
@justsanjeev
justsanjeev deleted the dlpx/pr/justsanjeev/f373f398-065a-4601-8309-0417af063f1b branch October 6, 2026 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants