search

openshift / origin-aggregated-logging

139 stars 230 forks source link

fix smoke test to use 5.1 #2135

Closed jcantrill closed 3 years ago

jcantrill commented 3 years ago

Description

This PR fixes the smoke test to use CLO, EO from 5.1

cc @ewolinetz @vimalk78

ewolinetz commented 3 years ago

/lgtm

ewolinetz commented 3 years ago

/hold @jcantrill can you also fix the following in hack/deploy-logging ? export IMAGE_ELASTICSEARCH_OPERATOR_REGISTRY=${IMAGE_ELASTICSEARCH_OPERATOR_REGISTRY:-registry.svc.ci.openshift.org ?

ewolinetz commented 3 years ago

/hold cancel /lgtm

openshift-ci[bot] commented 3 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ewolinetz, jcantrill, vimalk78

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/openshift/origin-aggregated-logging/blob/release-5.1/OWNERS)~~ [ewolinetz,jcantrill] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment