Skip to content

fix(ci): align build and release workflow with deploy workflow#1167

Merged
dadiorchen merged 1 commit intoGreenstand:hotfix/new-masterfrom
ahmadpiran:fix/build-release-workflow
Nov 1, 2025
Merged

fix(ci): align build and release workflow with deploy workflow#1167
dadiorchen merged 1 commit intoGreenstand:hotfix/new-masterfrom
ahmadpiran:fix/build-release-workflow

Conversation

@ahmadpiran
Copy link
Copy Markdown
Collaborator

@ahmadpiran ahmadpiran commented Oct 31, 2025

Description

This PR updates the prepare job in build_and_release.yaml workflow to use the same environment resolution logic as deploy.yml workflow. It ensures that the resources are selected based on both the release channel and target environment (dev, test, prod). This eliminates mismatches where Build & Release previously deployed to the wrong environment.

Issue(s) addressed

What kind of change(s) does this PR introduce?

  • Enhancement
  • Bug fix (non-breaking change which fixes an issue)
  • Refactor

Please check if the PR fulfils these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Issue

What is the current behavior?
The build_and_release.yaml workflow resolves deployment targets using only the release channel (e.g., master, freetown) and ignores the environment (dev, test, prod). This causes deployments to go to incorrect S3 buckets and CDN distributions.

What is the new behavior?
The workflow now resolves secrets using a combined channel-env key (e.g., freetown-prod), matching the logic used in deploy.yml. This ensures consistent and correct deployment targets across both workflows.

Breaking change

Does this PR introduce a breaking change?
No, this change is backward-compatible and improves correctness without altering existing interfaces or behavior for consumers.

Other useful information

@ahmadpiran ahmadpiran requested a review from a team as a code owner October 31, 2025 21:54
@dadiorchen dadiorchen merged commit 76a4471 into Greenstand:hotfix/new-master Nov 1, 2025
1 check passed
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.

Our push GitHub action can not deploy app to dev env but dedicated deploy action can do

2 participants