Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

sg: deny cloud ephemeral builds from main#63127

Merged
burmudar merged 3 commits into
mainfrom
wb/sg/eph-no-main
Jun 6, 2024
Merged

sg: deny cloud ephemeral builds from main#63127
burmudar merged 3 commits into
mainfrom
wb/sg/eph-no-main

Conversation

@burmudar

@burmudar burmudar commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

Triggering cloud ephemeral builds on main is problematic since there are a bunch of specific conditions that get triggered because the branch is main

Test plan

Tested locally
sg cloud eph build

./sg cloud eph build
⚠️ Triggering Cloud Ephemeral builds from "main" is not supported.

  Alternatively, if you still want to deploy "main" you can do:

  1. create a new branch off main by running  git switch <branch-name>
  2. push the branch to the remote by running  git push -u origin <branch-name>
  3. trigger the build by running  sg cloud ephemeral build


❌ failed to trigger epehemeral build for branch: cannot trigger a Cloud Ephemeral build for main branch

./sg cloud eph deploy

⚠️ Triggering Cloud Ephemeral builds from "main" is not supported.

  Alternatively, if you still want to deploy "main" you can do:

  1. create a new branch off main by running  git switch <branch-name>
  2. push the branch to the remote by running  git push -u origin <branch-name>
  3. trigger the build by running  sg cloud ephemeral build


❌ cloud ephemeral deployment failure: cannot trigger a Cloud Ephemeral build for main branch

Changelog

  • deny cloud ephemeral deployments triggered from 'main'

@cla-bot cla-bot Bot added the cla-signed label Jun 6, 2024
@burmudar burmudar requested review from a team, filiphaftek and michaellzc June 6, 2024 15:19
@burmudar burmudar enabled auto-merge (squash) June 6, 2024 15:24
@burmudar burmudar self-assigned this Jun 6, 2024
@burmudar burmudar merged commit c458e14 into main Jun 6, 2024
@burmudar burmudar deleted the wb/sg/eph-no-main branch June 6, 2024 15:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants