POWERMON-199: fix gather power-monitoring-operator info #353
Conversation
vimalk78
force-pushed
the
fix-must-gather-ns-1
branch
from
6e7dca2 to
6e82190
Compare
| get_catalogsource() { | ||
| log "getting catalogsource info for kepler-operator" | ||
| run oc -n "$KEPLER_OPERATOR_NS" get catalogsource kepler-operator-catalog -oyaml "$POWER_MONITORING_OPERATOR_INFO_DIR/kepler-operator-catalogsource.yaml" |
There was a problem hiding this comment.
Shouldn't we remove this as the operator would be from Red Hat certified Operators catalog source right?
There was a problem hiding this comment.
kept it for community operator
There was a problem hiding this comment.
but gather-power-monitoring-operator-info will never be called up for community operator
vimalk78
force-pushed
the
fix-must-gather-ns-1
branch
from
6e82190 to
015d934
Compare
|
It would be nice if we could address the previous issue with must-gather: #261 (comment) |
| KEPLER_OPERATOR_LABELS="operators.coreos.com/kepler-operator.openshift-operators" | ||
| POWER_MONITORING_OPERATOR_LABELS="operators.coreos.com/power-monitoring-operator.openshift-operators" |
There was a problem hiding this comment.
For the record I am okay with this but we need to address operator getting installed to ns other than openshift-operators (esp on 4.14+) in a different PR. Perhaps worth a bug @vprashar2929 ?
There was a problem hiding this comment.
if that is really needed, lets do it in another PR
There was a problem hiding this comment.
yes, as already mentioned :-)
There was a problem hiding this comment.
ok, can we merge this?
vimalk78
force-pushed
the
fix-must-gather-ns-1
branch
2 times, most recently
from
efa462c to
84caa6b
Compare
Signed-off-by: Vimal Kumar <vimal78@gmail.com>
vimalk78
force-pushed
the
fix-must-gather-ns-1
branch
from
84caa6b to
5820918
Compare
fix gather power-monitoring-operator info