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

sg: cloud ephemeral check if deployment exists already#62456

Merged
burmudar merged 10 commits into
mainfrom
wb/sg/check-existing-deployment
May 8, 2024
Merged

sg: cloud ephemeral check if deployment exists already#62456
burmudar merged 10 commits into
mainfrom
wb/sg/check-existing-deployment

Conversation

@burmudar

@burmudar burmudar commented May 6, 2024

Copy link
Copy Markdown
Contributor

Before trying to create new deployment we check whether the deployment exists already. If it does

  1. Notify user that deployment will not proceed
  2. Give alternative methods for creating a deployment

Screenshot 2024-05-07 at 15 50 03

Test plan

Tested locally

@burmudar burmudar self-assigned this May 6, 2024
@cla-bot cla-bot Bot added the cla-signed label May 6, 2024
@burmudar burmudar force-pushed the wb/sg/check-existing-deployment branch 2 times, most recently from e205d2e to 0c63f26 Compare May 7, 2024 08:40
@burmudar burmudar changed the title Wb/sg/check existing deployment sg: cloud ephemeral check if deployment exists already May 7, 2024
@burmudar burmudar force-pushed the wb/sg/check-existing-deployment branch from 7c68096 to f86df58 Compare May 7, 2024 09:20
@burmudar burmudar marked this pull request as ready for review May 7, 2024 09:33
@burmudar burmudar requested review from a team and filiphaftek May 7, 2024 13:51
Comment thread dev/sg/internal/cloud/deploy_command.go Outdated

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have to do this since escaping the back ticks doesn't work and it's annoying otherwise

Comment thread dev/sg/internal/cloud/deploy_command.go Outdated
Comment thread dev/sg/internal/bk/bk.go Outdated

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Going to keep this - just makes the buildkite client a little more complete

@burmudar burmudar force-pushed the wb/sg/check-existing-deployment branch from 13dd854 to 76542ab Compare May 7, 2024 14:34
* fix list instances

* create local dev const and use it with secrets

* fetch ephemeral license key from secrets

* Update dev/sg/internal/cloud/client.go
@burmudar burmudar merged commit 826db2c into main May 8, 2024
@burmudar burmudar deleted the wb/sg/check-existing-deployment branch May 8, 2024 09:56
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