Skip to content

[INFRANG-7812] implement deploy-apps command to support slingshot deploys#191

Open
dmartschenko wants to merge 3 commits intomasterfrom
INFRANG-7812
Open

[INFRANG-7812] implement deploy-apps command to support slingshot deploys#191
dmartschenko wants to merge 3 commits intomasterfrom
INFRANG-7812

Conversation

@dmartschenko
Copy link
Copy Markdown

Clever Coding Standards Agreement

JIRA

https://clever.atlassian.net/browse/INFRANG-7812

About

Add a new deploy-apps command to support slingshot deploys.

To call Slingshot directly from CircleCI using Twingate, the job needs to run on a linux machine executor not a docker executor. CircleCi docker executors do not support tunneling. However, machine executors are more expensive than docker executors, therefore rather than running the entire build/publish/deploy job on the machine executors I opted to create a new deploy-app job that can be configured to only run if the branch is master. The goal is to keep the additional cost minimal.

The vast majority of the circle-ci runs are for non-master branches so the additional cost will be applicable to a limited number of runs.

Checklist

  • Increment the version number in VERSION
  • Add release notes

Testing

@dmartschenko dmartschenko requested a review from a team as a code owner April 7, 2026 23:36
@dmartschenko dmartschenko requested review from andruwm and removed request for a team April 7, 2026 23:36
Comment on lines +11 to +12
appStackConfigPath = "config/%s/stack.yaml"
appValuesConfigPath = "config/%s/values.yaml"
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.

Should these be regex strings since values can have multiple variants?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

These are templates and the app is injected into the string. From what I understand we're expecting a common structure for the config directory

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.

Yes, common config directory, but there can be multiple values.yaml. i.e., development-values.yaml

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