Skip to content

[INFRA-317] move k8s deploys to playform events#196

Open
dmartschenko wants to merge 3 commits into
masterfrom
infra-317
Open

[INFRA-317] move k8s deploys to playform events#196
dmartschenko wants to merge 3 commits into
masterfrom
infra-317

Conversation

@dmartschenko
Copy link
Copy Markdown
Contributor

@dmartschenko dmartschenko commented Jun 3, 2026

Clever Coding Standards Agreement

JIRA

https://linear.app/clever/issue/INFRA-317/update-circleci-to-create-deployment-events

About

Checklist

  • Increment the version number in VERSION
  • Add release notes

Testing

  1. Updated do-nothing test branch to use this branch of goci in the CircleCi pipeline: https://app.circleci.com/pipelines/github/Clever/do-nothing/345/workflows/d6472e12-8e2a-4e48-8a64-2e4f35b1971f/jobs/1269
  2. Set up sns subscription and deploy.created queue in dianam workspace
  3. Ran Slingshot locally so it would consume from my queue
ark start -l -E DEPLOY_CREATED_QUEUE_URL="https://sqs.us-west-2.amazonaws.com/405617742722/dianam--slingshot-deploy-created" -s infra-315
  1. Verified the event from CircleCi made it to my local Slingshot and was processed: https://github.com/Clever/k8-deploys/commit/c686ceb2b9e3314dfcb7b4a97ac703e3adda14fc
}
Wed 15:31:47.864 slingshot/local> {
        source.title=slingshot-sqs-consumer.batch-received
        count:1
        detail-type:"deploy.created"
        pod-account:"585008086734"
        pod-id:"local"
        pod-region:"us-west-2"
        pod-shortname:"local"
        queue-url:"https://sqs.us-west-2.amazonaws.com/405617742722/dianam--slingshot-deploy-created"
        team:"eng-infra"
}

Rollout

  • The Slingshot PR must be merged first so the queue consumer is set up: https://github.com/Clever/slingshot/pull/53
  • Perform testing for do-nothing dev + prod to verify end-to-end connectivity
  • In case of testing failure rollback this PR

Separately since this is no longer directly calling slingshot the deploy_apps orb can be updated to remove the twingate setup and it can be run on a docker machine instead of a linux machine

@dmartschenko dmartschenko requested a review from a team as a code owner June 3, 2026 23:12
@dmartschenko dmartschenko requested review from andruwm and removed request for a team June 3, 2026 23:12
Comment thread internal/platformevents/schemas/deploycreated/auto_scaling_config.go Outdated
@@ -0,0 +1,14 @@
package deploycreated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should also update the codegen to add the standard generated code header so it gets ignored by linting tools.

Comment thread internal/platformevents/deploy_publisher.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants