Closed Rajalakshmi-Girish closed 6 months ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mkumatag, Rajalakshmi-Girish
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@Rajalakshmi-Girish: Updated the following 2 configmaps:
config
configmap in namespace prow
at cluster default
using the following files:
config.yaml
using file config/prow/config.yaml
job-config
configmap in namespace prow
at cluster default
using the following files:
test-etcd-periodics.yaml
using file config/jobs/periodic/etcd/test-etcd-periodics.yaml
build-golang-periodics.yaml
using file config/jobs/periodic/golang/build-golang-periodics.yaml
test-kubernetes-periodics.yaml
using file config/jobs/periodic/kubernetes/test-kubernetes-periodics.yaml
etcd-postsubmit.yaml
using file config/jobs/ppc64le-cloud/builds/etcd-postsubmit.yaml
kubernetes-postsubmit.yaml
using file config/jobs/ppc64le-cloud/builds/kubernetes-postsubmit.yaml
openshift-postsubmit.yaml
using file config/jobs/ppc64le-cloud/builds/openshift-postsubmit.yaml
Have added a preset label to enable the environment variable
GOPPC64=power9
for all the go binary building jobs.Also have removed the export of the PATH variable to add
/usr/local/go/bin
and$GOPATH/bin
to the PATH value. As theall-in-one:0.6
image has these values already added to PATH.@mkumatag I have verified if the value is being picked up while binary building by running a few jobs by doing
strings <binary_name> | grep GOPPC
Also have checked in a couple of runs if the removal of PATH export is not creating any failure. Any idea why we had to include these PATH exports inspite of having these values set in
all-in-one
image from the beginning?