Skip to content

fix(docker): remove unused ci-final stage breaking container builds - #65

Merged
emmanuelknafo merged 1 commit into
mainfrom
feature/64-fix-ci-final-dockerfile-stage
Jul 22, 2026
Merged

fix(docker): remove unused ci-final stage breaking container builds#65
emmanuelknafo merged 1 commit into
mainfrom
feature/64-fix-ci-final-dockerfile-stage

Conversation

@emmanuelknafo

Copy link
Copy Markdown
Contributor

Summary

Restores container builds across all three CI providers by removing the broken ci-final Dockerfile stage introduced in 601ebff.

Closes #64

Root cause

The added third provider: GitLab push appended ci-final as the last stage of src/webapp01/Dockerfile. Its COPY publish/ . needs a pre-published publish/ dir absent from the build context. Plain docker build / docker/build-push-action (no --target) default to the last stage and fail with \/publish\: not found.

Failing workflows fixed

  • CI/CD for Azure Web App
  • SCA - Anchore Syft SBOM Scan
  • CIS - Trivy Container Image Scanning
  • CIS - Anchore Grype Vulnerability Scan

Tri-provider safety

  • GitLab — builds with az acr build ... --target final explicitly; unaffected.
  • Azure DevOps — plain docker build returns to default final; fixed.
  • GitHub — returns to default final; fixed.

No pipeline anywhere references ci-final.

The ci-final stage was added as the last Dockerfile stage but its COPY publish/ . requires a pre-published directory absent from the build context. Plain docker build (GitHub + Azure DevOps) defaults to the last stage and fails. GitLab explicitly targets --target final and is unaffected. Nothing references ci-final.

Fixes #64
@emmanuelknafo
emmanuelknafo requested a review from CalinL as a code owner July 22, 2026 14:35
@emmanuelknafo
emmanuelknafo merged commit 21746d0 into main Jul 22, 2026
14 checks passed
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 1402b13.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@emmanuelknafo
emmanuelknafo deleted the feature/64-fix-ci-final-dockerfile-stage branch July 22, 2026 14:35
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.

Fix container build failures from unused ci-final Dockerfile stage

1 participant