Skip to content
This repository has been archived by the owner on Feb 2, 2024. It is now read-only.

feat: Add BuildConfig option to build the image in cluster #7

Closed
wants to merge 1 commit into from

Conversation

tumido
Copy link
Member

@tumido tumido commented Oct 24, 2022

Description of the change

Allow users to build images in local cluster instead of using a prepared image. Options are described in the README, along with a values.yaml sample.

Existing or Associated Issue(s)

Requires: janus-idp/redhat-backstage-build#3 and backstage/charts#78

Additional Information

Checklist

  • Chart version bumped in Chart.yaml according to semver.
  • Variables are documented in the values.yaml and added to the README.md. The helm-docs utility can be used to generate the necessary content. Use helm-docs --dry-run to preview the content.
  • JSON Schema generated.
  • List tests pass for Chart using the Chart Testing tool and the ct lint command.

@tumido
Copy link
Member Author

tumido commented Oct 27, 2022

@sabre1041 PTAL

@tumido tumido force-pushed the in-cluster-build branch 2 times, most recently from 92c1fac to cef00f5 Compare November 17, 2022 10:48
Copy link
Contributor

@sabre1041 sabre1041 left a comment

Choose a reason for hiding this comment

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

A few changes requested

charts/backstage/templates/build/buildconfig.yaml Outdated Show resolved Hide resolved
charts/backstage/templates/deployment.yaml Outdated Show resolved Hide resolved
charts/backstage/templates/deployment.yaml Outdated Show resolved Hide resolved
@tumido tumido changed the title feat: Add BuildConfig option to build the image in cluster [wip] feat: Add BuildConfig option to build the image in cluster Apr 3, 2023
@tumido tumido force-pushed the in-cluster-build branch 3 times, most recently from 704ed65 to 784f62a Compare April 3, 2023 20:54
@tumido tumido requested a review from sabre1041 April 3, 2023 20:54
@tumido tumido changed the title [wip] feat: Add BuildConfig option to build the image in cluster feat: Add BuildConfig option to build the image in cluster Apr 3, 2023
@tumido tumido marked this pull request as draft June 1, 2023 17:07
@tumido tumido force-pushed the in-cluster-build branch 2 times, most recently from 9735d20 to 2fa9640 Compare June 1, 2023 17:08
Signed-off-by: Tomas Coufal <tcoufal@redhat.com>
@openshift-merge-robot
Copy link

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@tumido tumido closed this Jan 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants