-
Notifications
You must be signed in to change notification settings - Fork 75
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tests are failing with modified reflection API (reflect -> goccy/go-reflect ), on Aarch64 #1130
Comments
This on behalf of https://groups.google.com/g/golang-nuts/c/S2gBW3BV4QU/m/I4gWtrPxBwAJ |
@advancedwebdeveloper How about opening a PR with your change and describe your intention more here? |
@advancedwebdeveloper Running the "goTest" (gradle command) has never been valid in local configurations (especially for integration tests which requires a full OpenWhisk backend plus all providers installed and working with backing services). As the README states, you should run only "unit" tests locally with "go test" (actual go command); "go test" integration tests will only work in Travis as the needed credentials are provided in that environment. FWIW, The direction of the client tools (wskdeploy and CLI) is one where gogradle (which has not been updated since 2017 and is hardcoded to the deprecated govendor) is replaced with a newer plugin or removed altogether. |
OK, understood.
And how would I run Travis in my local environment?
OK, I am just following the current instructions. |
Travis is not run locally, but in the Travis CI servers that are paid for by the Apache Software Foundation. If you submit a pull request, a Travis PR job will be run, and our Travis scripts will automate the install of OpenWhisk and its services and run all unit and integration tests. Of course, you can change the build.gradle to attempt to build for a new architecture (if it is supported on the compiler) by adding the architecure's acronym to the build matrix. However, If it is your intent to run the resultant binary (executable) on a new architecture that would likely be a manual process (at the moment) by taking the matching binary and actually running it on a machine of that architecture (unit tests only unless you want to figure out how to install a full production OW platform locally). There may be possibilities that 3rd party services might support Travis jobs/testing that could be enabled within Travis to test on specific platforms (e.g., we have tried for Windows, but it was very involved to enable this with Apache infra. team). TBH, I have no knowledge if these 3rd party exist (especially for a new architecture) and am doubtful they may for your specific case. One question though... it was my belief that AARCH64 and ARM64 were the same (merged) and not sure why supporting this architecture (by name/acronym) is important? i.e., https://www.phoronix.com/scan.php?page=news_item&px=MTY5ODk |
and a recent blog post confirms our linux-arm64 would work on aarch64 servers… https://opensource.com/article/21/1/go-cross-compiling |
and reading this (although they neglect to say they removed support for Darwin 32-bit in Go 1.15) indicates 1.16 beta will support new M1 chips (better?) https://blog.golang.org/ports (is this why I see a PR for 1.16.2? #1132) |
Consult the following discussions, for farther details: https://groups.google.com/g/golang-nuts/c/S2gBW3BV4QU/m/I4gWtrPxBwAJ apache#1130
Consider inspecting the following discussions: https://groups.google.com/g/golang-nuts/c/S2gBW3BV4QU/m/I4gWtrPxBwAJ apache#1130
Consult the following, in case if you have any questions: https://groups.google.com/g/golang-nuts/c/S2gBW3BV4QU/m/I4gWtrPxBwAJ apache#1130
So,
while I was able to build
with such an applied patch:
https://gist.github.com/advancedwebdeveloper/9ee1d333d0101b06823735cf871055b1
.
Now, test are failing:
test_report.zip
The text was updated successfully, but these errors were encountered: