Skip to content
This repository has been archived by the owner on Jan 20, 2023. It is now read-only.

Make '-race' option configurable for s390x #103

Merged
merged 1 commit into from
Jun 4, 2020

Conversation

shahidhs-ibm
Copy link
Contributor

What does this PR do?

'-race' flag is not supported on s390x architecture. This PR will remove '-race' option from go test command (which gets executed in make test).

What issues does this PR fix or reference?

This PR is part of this initiative.

@codecov
Copy link

codecov bot commented Apr 24, 2020

Codecov Report

Merging #103 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #103   +/-   ##
=======================================
  Coverage   74.26%   74.26%           
=======================================
  Files          13       13           
  Lines         645      645           
=======================================
  Hits          479      479           
  Misses        158      158           
  Partials        8        8           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0ba6290...b486753. Read the comment docs.

Signed-off-by: Shahid Shaikh <shahid@us.ibm.com>
Copy link
Contributor

@amisevsk amisevsk left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

as race seems to be only available on x86-64 platform I would do the opposite on the check
Only enable it on this platform (x86-64/amd64)

@benoitf
Copy link

benoitf commented Apr 27, 2020

seems the first article I read was incorrect.
https://golang.org/doc/articles/race_detector.html#Supported_Systems

@shahidhs-ibm
Copy link
Contributor Author

@benoitf Any update on merge?

@shahidhs-ibm
Copy link
Contributor Author

@nickboldt @benoitf When this PR can be merged?

@nickboldt nickboldt merged commit 57d5134 into eclipse-che:master Jun 4, 2020
@nickboldt
Copy link
Contributor

Not sure why this sat for 2 weeks without merge.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants