Skip to content

[INFRA-421] support repos with no launch.yaml#197

Open
dmartschenko wants to merge 2 commits into
masterfrom
infra-421
Open

[INFRA-421] support repos with no launch.yaml#197
dmartschenko wants to merge 2 commits into
masterfrom
infra-421

Conversation

@dmartschenko

Copy link
Copy Markdown
Contributor

Clever Coding Standards Agreement

JIRA

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

About

goci previously only built apps configured via Catapult launch configs. This PR adds support for apps configured for Kubernetes via stack.yaml, and handles repos with a mix of both.

In this PR:

  • Discovers and builds apps from both config/ (Kubernetes) and launch/ (Catapult), merging results with Kubernetes winning when the same app appears in both
  • Kubernetes apps skip the Catapult publish and deploy steps
  • Fails when a Kubernetes config exists for an app but is missing a build command that the legacy Catapult config had — forces migration to be completed explicitly
  • Only errors if both config directories are missing; a missing config/ or launch/ alone is silently ignored

Important

This PR proposes adding a build block to the stack.yaml files for kubernetes apps. It would be configured as such

build:
  command: make build
  dockerfile: Dockerfile
  artifactName: my-app
  dependencies:
    - src/**
    - go.mod

Checklist

  • Increment the version number in VERSION
  • Add release notes

Testing

  • Removed launch.yaml, migrated to new launch-gen, and verified cci builds for slingshot
  • Verified cci builds for groups which has catapult + kubernetes apps

@dmartschenko dmartschenko requested a review from a team as a code owner June 25, 2026 00:52
@dmartschenko dmartschenko requested review from amanda-orin and removed request for a team June 25, 2026 00:52
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.

1 participant