Skip to content

remove unused env var ROX_RECORDER_DURATION#3559

Merged
RTann merged 1 commit intomasterfrom
ross/rm-recorder-env-var
Oct 26, 2022
Merged

remove unused env var ROX_RECORDER_DURATION#3559
RTann merged 1 commit intomasterfrom
ross/rm-recorder-env-var

Conversation

@RTann
Copy link
Contributor

@RTann RTann commented Oct 25, 2022

Description

This is no longer used, so why not delete it?

Checklist

  • Investigated and inspected CI test results
  • [] Unit test and regression tests added
  • [ ] Evaluated and added CHANGELOG entry if required
  • [ ] Determined and documented upgrade steps
  • [ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

Testing Performed

CI

@RTann RTann changed the title remove unused env var remove unused env var ROX_RECORDER_DURATION Oct 25, 2022
@RTann RTann mentioned this pull request Oct 25, 2022
1 task
@ghost
Copy link

ghost commented Oct 25, 2022

Images are ready for the commit at b650a82.

To use with deploy scripts, first export MAIN_IMAGE_TAG=3.72.x-401-gb650a82d21.

@openshift-ci
Copy link

openshift-ci bot commented Oct 26, 2022

@RTann: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/gke-postgres-qa-e2e-tests b650a82 link false /test gke-postgres-qa-e2e-tests

Full PR test history. Your PR dashboard.

Details

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/test-infra repository. I understand the commands that are listed here.

@RTann RTann requested a review from misberner October 26, 2022 04:39
@RTann RTann merged commit 74c227a into master Oct 26, 2022
@RTann RTann deleted the ross/rm-recorder-env-var branch October 26, 2022 04:49
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