Skip to content

Commit 9e13557

Browse files
Stopped trigger_release script from releasing to int and fixed it for multi phase approvals
1 parent f2d4d69 commit 9e13557

1 file changed

Lines changed: 8 additions & 9 deletions

File tree

scripts/trigger_release.py

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,7 @@ def __init__(self, token: str, owner: str, repo: str, logger: logging.Logger):
3232
self.version_tag: Optional[str] = None
3333
self.jobs_requiring_approval = [
3434
"release_ref",
35-
"release_qa",
36-
"release_int"
35+
"release_qa"
3736
]
3837
self.approved_jobs = set()
3938
self.completed_jobs = set()
@@ -207,28 +206,28 @@ def monitor_and_approve(self) -> bool:
207206
env_name = deployment["environment"]["name"]
208207
env_id = deployment["environment"]["id"]
209208

210-
# Check if this is release_prod
211-
if env_name == "prod":
209+
# Check if this is release_int
210+
if env_name == "int":
212211
self.logger.info(
213-
f"\n🛑 Reached production deployment for "
212+
f"\n🛑 Reached INT deployment for "
214213
f"environment '{env_name}'"
215214
)
216215
self.print_summary(
217-
"Stopped at production deployment"
216+
"Stopped at INT deployment"
218217
)
219218
return True
220219

221220
# Auto-approve other environments
222221
job_name = f"release_{env_name}"
223-
if (job_name in self.jobs_requiring_approval and
224-
job_name not in self.approved_jobs):
222+
if (job_name in self.jobs_requiring_approval):
225223
self.logger.info(
226224
f"✓ Approving deployment to environment "
227225
f"'{env_name}'..."
228226
)
229227
if self.approve_deployment(self.run_id, [env_id]):
230-
self.approved_jobs.add(job_name)
231228
self.logger.info(f"✅ Approved: {job_name}")
229+
if job_name not in self.approved_jobs:
230+
self.approved_jobs.add(job_name)
232231
else:
233232
self.logger.warning(f"⚠️ Failed to approve: {job_name}")
234233

0 commit comments

Comments
 (0)