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

load kiln arguments from Kilnfile #473

Merged

Conversation

crhntr
Copy link
Contributor

@crhntr crhntr commented Jan 23, 2024

The problems

Shared build script

Some tiles don't follow default path configurations.
TAS Slingshots is trying to build shared build pipelines.
This will be easier if tile authors can configure their build arguments without over-complicating to the shared build scripts.

Build reproducibility

We want to be able to checkout a reference and build tiles the same way they were built at the time.

The Solution

Make most kiln bake configuration configurable in the Kilnfile.

The Story

Story: https://www.pivotaltracker.com/story/show/186802003

@cf-gitbot
Copy link
Member

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

@crhntr crhntr force-pushed the 186802003-bake-configuration-from-Kilnfile branch 3 times, most recently from 72b8be0 to 9b17fd6 Compare January 25, 2024 00:19
@crhntr crhntr added the tas-slingshots Created by https://github.com/orgs/pivotal-cf/teams/tas-strategic-initiatives-slingshot label Feb 21, 2024
@crhntr crhntr force-pushed the 186802003-bake-configuration-from-Kilnfile branch from 9b17fd6 to a27420f Compare February 23, 2024 21:16
@crhntr crhntr changed the base branch from main to slingshots-bake-changes February 23, 2024 21:45
@crhntr crhntr merged commit 22c31e4 into slingshots-bake-changes Feb 23, 2024
3 checks passed
@crhntr crhntr deleted the 186802003-bake-configuration-from-Kilnfile branch February 23, 2024 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tas-slingshots Created by https://github.com/orgs/pivotal-cf/teams/tas-strategic-initiatives-slingshot
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants