Skip to content
This repository was archived by the owner on Nov 27, 2024. It is now read-only.

ci: build & publish kubesaw components using FORCED_TAG#94

Merged
sadlerap merged 5 commits intokonflux-workspaces:mainfrom
sadlerap:ci-build-push-images
May 2, 2024
Merged

ci: build & publish kubesaw components using FORCED_TAG#94
sadlerap merged 5 commits intokonflux-workspaces:mainfrom
sadlerap:ci-build-push-images

Conversation

@sadlerap
Copy link
Member

Build and publish our kubesaw dependencies using the publish-current-bundles-for-e2e make target from toolchain-e2e. The containers built here will be deployed in openshift-ci during pull request testing.

Build and publish our kubesaw dependencies using the
`publish-current-bundles-for-e2e` make target from toolchain-e2e.  The
containers built here will be deployed in openshift-ci during pull
request testing.

Signed-off-by: Andy Sadler <ansadler@redhat.com>
Signed-off-by: Andy Sadler <ansadler@redhat.com>
Signed-off-by: Andy Sadler <ansadler@redhat.com>
@sadlerap sadlerap marked this pull request as ready for review April 29, 2024 20:19
@sadlerap sadlerap requested a review from filariow April 29, 2024 20:19
@sadlerap
Copy link
Member Author

(this will fail to build until it's merged)

Copy link
Member

@filariow filariow left a comment

Choose a reason for hiding this comment

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

just a nit

Signed-off-by: Andy Sadler <ansadler@redhat.com>
Copy link
Contributor

@dperaza4dustbit dperaza4dustbit left a comment

Choose a reason for hiding this comment

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

Logic looks good, some comments on the current variable values

This is a more robust way of determining which branch we need to use
when we build kubesaw components.  Branches are now resolved like so:
1. First check the repo-specific branch environment variable.  If it's
   set, use it.
2. If it's not set, use the generic `${BRANCH}` environment variable.
3. If `${BRANCH}` is not set, fall back to a default (in this case,
   `master` for member-operator, `pubviewer-mvp` for all others).

Signed-off-by: Andy Sadler <ansadler@redhat.com>
Copy link
Contributor

@dperaza4dustbit dperaza4dustbit left a comment

Choose a reason for hiding this comment

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

lgtm

@sadlerap sadlerap merged commit 530ada2 into konflux-workspaces:main May 2, 2024
@sadlerap sadlerap deleted the ci-build-push-images branch May 2, 2024 15:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants