Skip to content

Conversation

@svghadi
Copy link
Member

@svghadi svghadi commented Dec 4, 2024

What type of PR is this?
/kind ci

What does this PR do / why we need it:

Enables CI build on konflux. The PR job builds on amd only and the PUSH job builds on all supported arch i.e amd64, arm64, ppc64le & s390x.

Signed-off-by: Siddhesh Ghadi <sghadi1203@gmail.com>
@openshift-ci
Copy link

openshift-ci bot commented Dec 4, 2024

@svghadi: The label(s) kind/ci cannot be applied, because the repository doesn't have them.

In response to this:

What type of PR is this?
/kind ci

What does this PR do / why we need it:

Enables CI build on konflux. The PR job builds on amd only and the PUSH job builds on all supported arch i.e amd64, arm64, ppc64le & s390x.

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-sigs/prow repository.

@svghadi
Copy link
Member Author

svghadi commented Dec 4, 2024

The image build is successful on Konflux

$ docker run -it --rm --entrypoint /bin/bash quay.io/redhat-user-workloads/rh-openshift-gitops-tenant/gitops-must-gather:on-pr-9f06669e7b22b1d187489e066fcb4c29b116a88c-linux-x86-64
WARNING: image platform (linux/amd64) does not match the expected platform (linux/arm64)
[root@685cec0225f5 /]# cat /usr/bin/gather
#!/usr/bin/env bash

set -eu -o pipefail

LOGS_DIR="/must-gather"

mkdir -p ${LOGS_DIR}

GITOPS_CURRENT_CSV=$(oc get subscription.operators.coreos.com --ignore-not-found -A -o json | jq '.items[] | select(.metadata.name=="openshift-gitops-operator") | .status.currentCSV' -r)

@svghadi
Copy link
Member Author

svghadi commented Dec 4, 2024

The PR is ready for review. Will fix the failing static check job in another PR.

@anandrkskd
Copy link

LGTM

@svghadi svghadi merged commit 4f91ccf into redhat-developer:main Dec 4, 2024
3 of 4 checks passed
@svghadi svghadi deleted the konflux-build branch December 4, 2024 06:30
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