Skip to content

fix(workflows): remove unrecognized containerImageName Bicep parameter - #67

Merged
emmanuelknafo merged 1 commit into
mainfrom
feature/66-fix-containerimagename-bicep-param
Jul 22, 2026
Merged

fix(workflows): remove unrecognized containerImageName Bicep parameter#67
emmanuelknafo merged 1 commit into
mainfrom
feature/66-fix-containerimagename-bicep-param

Conversation

@emmanuelknafo

Copy link
Copy Markdown
Contributor

Summary

Fixes the CI/CD deploy failure where az deployment sub create rejected containerImageName (unrecognized template parameter), introduced by the small sync with ADO commit 049e139.

Closes #66

Root cause

The sync re-added a containerImageName variable + --parameters containerImageName line to .github/workflows/cicd.yml, but blueprints/gh-aspnet-webapp/bicep/main.bicep does not define that parameter.

Fix

Removed both lines, realigning the GitHub workflow with the ADO pipeline and the Bicep template's allowed parameters.

Tri-provider safety

  • Azure DevOps.azuredevops/pipelines/cicd.yml never passed containerImageName; unchanged.
  • GitLab — does not reference it; unchanged.
  • containerImageName was referenced only in the GitHub workflow, so this removal cannot affect the other providers.

The ADO sync re-added containerImageName to the az deployment sub create call, but main.bicep does not define it, causing 'unrecognized template parameter'. Removes it to realign with the ADO pipeline (which never passed it) and GitLab. containerImageName is referenced only in the GitHub workflow, so other providers are unaffected.

Fixes #66
@emmanuelknafo
emmanuelknafo requested a review from CalinL as a code owner July 22, 2026 17:25
@emmanuelknafo
emmanuelknafo merged commit 958fb81 into main Jul 22, 2026
14 checks passed
@emmanuelknafo
emmanuelknafo deleted the feature/66-fix-containerimagename-bicep-param branch July 22, 2026 17:25
@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 acdd66e.
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

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 CI/CD deploy failure: unrecognized Bicep parameter containerImageName

1 participant