Skip to content

fix: misc fixes for codefresh, native build image and helm deploys#33

Merged
vigneshrajsb merged 7 commits intomainfrom
codefresh-git-org-steps
Aug 14, 2025
Merged

fix: misc fixes for codefresh, native build image and helm deploys#33
vigneshrajsb merged 7 commits intomainfrom
codefresh-git-org-steps

Conversation

@vigneshrajsb
Copy link
Copy Markdown
Contributor

@vigneshrajsb vigneshrajsb commented Aug 11, 2025

What

  1. Currently there is no way to configure custom codefresh steps if needed. for now use the github org value from app_setup.org that will be populated when setting up the app in a github org. this is assuming that the custom clone and kube-context steps have the same org namespace

assumption:

github org: vigneshrajsb
codefresh custom steps with namespace: vigneshrajsb/clone
  1. service name suffix is not needed for the image_name to keep external ECR repos in sync
  2. normalize git-branch annotation value to handle for special characters
  3. append --version to helm install commands for public and org charts since globalConfig or lifecycle config can have pinned versions for a chart
  4. fix the duration column for native build and deploy logs page

_PS:

  • for now using the config from app_setup.org. this still can be empty with current setup
  • when we build the onboarding flow, maybe make this required so this config value can never be empty_

- for now using the config from app_setup.org. this still can be empty
with current setup
- when we build the onboarding flow, maybe make this required so this
config value can never be empty!
@vigneshrajsb vigneshrajsb requested a review from a team as a code owner August 11, 2025 18:39
@vigneshrajsb vigneshrajsb changed the title fix: use git org from app_setup config for codefresh steps fix: misc fixes for codefresh, native build image and helm deploys Aug 14, 2025
@vigneshrajsb vigneshrajsb merged commit b7c3a09 into main Aug 14, 2025
3 checks passed
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.

2 participants