Skip to content
This repository has been archived by the owner on Jan 8, 2024. It is now read-only.

Delete deployment (CLI changes, GCR support) #2

Merged
merged 6 commits into from
Jun 3, 2020
Merged

Conversation

pearkes
Copy link
Contributor

@pearkes pearkes commented May 20, 2020

This modifies the CLI to:

  • support passing id to deployment destroy
  • only allow destroy of successful deployment

For cloudrun it now supports the DestroyFunc, which deletes the underlying revision.

It also adds a GetDeployment method to the server to support the individual deletion
workflow.

Finally, used a new version of protoc-gen-go so there are some major changes there isolated in a commit. Everything seems to work so far.

@pearkes pearkes requested a review from mitchellh May 20, 2020 21:43
pearkes added 6 commits June 3, 2020 10:59
This also updates the command to only support one deployment. I think
this may actually be desired – awk workflows from `deployment list` and
xargs seems like the modern approach here.
@pearkes pearkes force-pushed the delete-deployment branch from 863e8f8 to df14998 Compare June 3, 2020 18:01
@evanphx evanphx self-requested a review June 3, 2020 18:09
Copy link
Contributor

@evanphx evanphx left a comment

Choose a reason for hiding this comment

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

Looks good to me

@pearkes pearkes merged commit e9f6cd6 into master Jun 3, 2020
@pearkes pearkes deleted the delete-deployment branch June 3, 2020 18:14
alexcarpenter added a commit that referenced this pull request Oct 13, 2021
* Init <GradientBox /> component

* SVG alignment

* Add TS interface
alexcarpenter added a commit that referenced this pull request Oct 13, 2021
* Init <GradientBox /> component

* SVG alignment

* Add TS interface
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants