cisearch changes and add k8s ci search dns#561
cisearch changes and add k8s ci search dns#561Rajalakshmi-Girish wants to merge 1 commit intoppc64le-cloud:masterfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Rajalakshmi-Girish The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
mkumatag
left a comment
There was a problem hiding this comment.
will have to avoid building these multiple images and pass command line args.
| && grep -rl test-platform-results . | xargs sed -i 's/test-platform-results/ppc64le-kubernetes/g' \ | ||
| && grep -rl "OpenShift" . | xargs sed -i 's/OpenShift/ppc64le-cloud/g' \ | ||
| && grep -rl "ci\.openshift\.org" . | xargs sed -i 's/ci\.openshift\.org/ppc64le-cloud\.cis\.ibm\.net/g' \ | ||
| && GO_BUILD_PACKAGES=./cmd/search make GO_REQUIRED_MIN_VERSION:= build |
There was a problem hiding this comment.
I think we should get rid of these crazy seds, e.g: index-bucket can be used to override the bucket name, likewise lets find all the ways to override these variable to avoid hacks
| @@ -2,12 +2,12 @@ FROM golang | |||
| WORKDIR /go/src/github.com/openshift | |||
There was a problem hiding this comment.
we need to pin above golang version
| RUN mkdir /var/lib/ci-search && chown 1000:1000 /var/lib/ci-search && chmod 1777 /var/lib/ci-search | ||
| USER 1000:1000 | ||
| ENTRYPOINT ["search"] | ||
| CMD ["--path=/var/lib/ci-search/", "--deck-uri=https://prow.k8s.io"] |
There was a problem hiding this comment.
deck-url can be overridden during the runtime to avoid duplicate images
Fixed https://search.ppc64le-cloud.cis.ibm.net/ to point jobs from https://prow.ppc64le-cloud.cis.ibm.net/ which have logs uploaded to GCS bucket
ppc64le-kubernetesCreated new url https://k8search.ppc64le-cloud.cis.ibm.net/ for searching failures from https://prow.k8s.io/ which have logs uploaded to GCS bucket
kubernetes-ci-logs