Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Support for canary releases with EXTERNAL deployments #231

Closed
wants to merge 36 commits into from

Conversation

jfranzoi
Copy link

almost done with what we started discussing in #228 (and sorry, it starts from another PR still waiting for approval, #227.
still WIP since it lacks some more documentations on README.

added support for detecting deployment controllers other than default ECS (see $DEPLOYMENT_CONTROLLER abstraction). implemented EXTERNAL, both on force new deployment and complete service deployment. CODE_DEPLOY is not supported (and I guess it would never be, since you'd interact with code-deploy itself).

few more arguments, to configure the canary deployment process:

  • --task-set-file to optionally use file for task set
  • --canary-percent to choose canary deployment size (by default, 25% of primary deployment's task set)
  • --canary-confirmation to choose a strategy to confirm (or abort) canary deployment, with few provided ones (interactively ask to user, wait for timeout, or simply proceed) and a custom one, intended as a bash script to be executed (and proceeding on 0 exit code)

@devon-sil
Copy link
Contributor

Given the age of this PR and that it has conflicts, I'm going to close it out. If you still want to pursue this, please submit a new PR based on the latest code.

@devon-sil devon-sil closed this Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants