Skip to content

Conversation

@acornett21
Copy link
Contributor

Description of the change:

  • Adding -race flag to unit tests to catch race conditions in tests/code.

Motivation for the change:

Checklist

If the pull request includes user-facing changes, extra documentation is required:

Copy link
Member

@rashmigottipati rashmigottipati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 18, 2025
@acornett21 acornett21 merged commit f905c5b into operator-framework:master Nov 18, 2025
22 of 23 checks passed
acornett21 added a commit to acornett21/operator-sdk that referenced this pull request Nov 18, 2025
* updating unit test make cmd to add -race flag

Signed-off-by: Adam D. Cornett <[email protected]>

* fixing race condition in external go, ensure cmd has finished

Signed-off-by: Adam D. Cornett <[email protected]>

---------

Signed-off-by: Adam D. Cornett <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants