Skip to content

/go

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

x/build: TryBots broken for x/mobile subrepo #30785

Closed
bcmills opened this issue Mar 12, 2019 · 7 comments

Comments

Assignees
No one assigned
Labels
Builders NeedsInvestigation mobile
Projects
None yet
Milestone
  Unreleased
4 participants
@bcmills @bradfitz @eliasnaur @gopherbot
@bcmills
Copy link
Member

commented Mar 12, 2019

I thought the TryBot failures on https://golang.org/cl/167179 seemed suspicious, so I created https://golang.org/cl/167182 to test the TryBots on a silly no-op change.

Both of the configured TryBots failed for that change too:
https://storage.googleapis.com/go-build-log/a0836481/android-amd64-emu_5da43c59.log
https://storage.googleapis.com/go-build-log/a0836481/linux-amd64-androidemu_3d8ec5df.log

(CC @bradfitz @dmitshur @eliasnaur)

@gopherbot gopherbot added this to the Unreleased milestone Mar 12, 2019

@gopherbot gopherbot added the Builders label Mar 12, 2019

@bcmills bcmills added mobile NeedsInvestigation labels Mar 12, 2019

@bradfitz

This comment has been minimized.

Sign in to view
Copy link
Member

commented Mar 13, 2019

Your silly no-no change was quite silly indeed: it was based on a Feb 22nd git parent, while @eliasnaur was still furiously working on getting x/mobile working.

I rebased your test CL and it's running again now against the current x/mobile HEAD.
@bradfitz

This comment has been minimized.

Sign in to view
Copy link
Member

commented Mar 13, 2019

It now passed android-amd64-emu and got an entirely different error (perhaps a flake?) on linux-amd64-androidemu.

@eliasnaur, can you take a look?

Failed on linux-amd64-androidemu: https://storage.googleapis.com/go-build-log/bea58ef3/linux-amd64-androidemu_92ef0445.log

ok  	golang.org/x/mobile/app	0.008s
?   	golang.org/x/mobile/app/internal/apptest	[no test files]
?   	golang.org/x/mobile/app/internal/testapp	[no test files]
?   	golang.org/x/mobile/asset	[no test files]
ok  	golang.org/x/mobile/bind	30.154s
--- FAIL: TestJavaSeqTest (22.56s)
    seq_test.go:124: tmpdir = /workdir/tmp/bind-java-seq-test-030688498
    seq_test.go:181: > Task :preBuild UP-TO-DATE
        > Task :preDebugBuild UP-TO-DATE
        > Task :compileDebugAidl
        > Task :compileDebugRenderscript
        > Task :checkDebugManifest
        > Task :generateDebugBuildConfig
        > Task :generateDebugResValues
        > Task :generateDebugResources
        > Task :packageDebugResources
        > Task :processDebugManifest
        > Task :prepareLintJar
        > Task :javaPreCompileDebug
        > Task :mergeDebugConsumerProguardFiles
        > Task :mergeDebugShaders
        > Task :platformAttrExtractor
        > Task :generateDebugRFile
        > Task :generateDebugSources
        > Task :compileDebugJavaWithJavac
        > Task :extractDebugAnnotations
        > Task :compileDebugShaders
        > Task :generateDebugAssets
        > Task :packageDebugAssets
        > Task :packageDebugRenderscript NO-SOURCE
        > Task :processDebugJavaRes NO-SOURCE
        > Task :transformResourcesWithMergeJavaResForDebug
        > Task :transformClassesAndResourcesWithSyncLibJarsForDebug
        > Task :compileDebugNdk NO-SOURCE
        > Task :mergeDebugJniLibFolders
        > Task :transformNativeLibsWithMergeJniLibsForDebug
        > Task :transformNativeLibsWithStripDebugSymbolForDebug
        > Task :transformNativeLibsWithSyncJniLibsForDebug
        > Task :bundleDebug
        > Task :compileDebugSources
        > Task :assembleDebug
        > Task :preDebugAndroidTestBuild UP-TO-DATE
        > Task :compileDebugAndroidTestAidl
        > Task :processDebugAndroidTestManifest
        > Task :compileDebugAndroidTestRenderscript
        > Task :generateDebugAndroidTestBuildConfig
        > Task :mainApkListPersistenceDebugAndroidTest
        > Task :generateDebugAndroidTestResValues
        > Task :generateDebugAndroidTestResources
        > Task :mergeDebugAndroidTestResources
        > Task :processDebugAndroidTestResources
        > Task :generateDebugAndroidTestSources
        > Task :transformClassesAndResourcesWithPrepareIntermediateJarsForDebug
        > Task :javaPreCompileDebugAndroidTest
        > Task :compileDebugAndroidTestJavaWithJavac
        > Task :compileDebugAndroidTestNdk NO-SOURCE
        > Task :compileDebugAndroidTestSources
        > Task :mergeDebugAndroidTestShaders
        > Task :compileDebugAndroidTestShaders
        > Task :generateDebugAndroidTestAssets
        > Task :mergeDebugAndroidTestAssets
        > Task :transformClassesWithDexBuilderForDebugAndroidTest
        > Task :transformDexArchiveWithExternalLibsDexMergerForDebugAndroidTest
        > Task :transformDexArchiveWithDexMergerForDebugAndroidTest
        > Task :mergeDebugAndroidTestJniLibFolders
        > Task :transformNativeLibsWithIntermediateJniLibsForDebug
        > Task :transformNativeLibsWithMergeJniLibsForDebugAndroidTest
        > Task :processDebugAndroidTestJavaRes NO-SOURCE
        > Task :transformResourcesWithMergeJavaResForDebugAndroidTest
        > Task :validateSigningDebugAndroidTest
        > Task :packageDebugAndroidTest
        > Task :assembleDebugAndroidTest
        
        > Task :connectedDebugAndroidTest
        Starting 42 tests on android-avd(AVD) - 8.0.0
        
        go.SeqTest > testJavaRefKeep[android-avd(AVD) - 8.0.0] �[31mFAILED �[0m
        	junit.framework.AssertionFailedError: want obj not to be kept by Go
        	at go.SeqTest.testJavaRefKeep(SeqTest.java:346)
        
        > Task :connectedDebugAndroidTest FAILED
        
        FAILURE: Build failed with an exception.
        
        * What went wrong:
        Execution failed for task ':connectedDebugAndroidTest'.
        > There were failing tests. See the report at: file:///workdir/tmp/bind-java-seq-test-030688498/build/reports/androidTests/connected/index.html
        
        * Try:
        Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
        
        * Get more help at https://help.gradle.org
        
        BUILD FAILED in 7s
        48 actionable tasks: 48 executed
    seq_test.go:182: failed to run gradle test: exit status 1
FAIL
FAIL	golang.org/x/mobile/bind/java	152.286s
ok  	golang.org/x/mobile/bind/objc	0.035s
ok  	golang.org/x/mobile/bind/seq	0.017s
ok  	golang.org/x/mobile/cmd/gobind	25.942s
ok  	golang.org/x/mobile/cmd/gomobile	0.554s
?   	golang.org/x/mobile/event/key	[no test files]
?   	golang.org/x/mobile/event/lifecycle	[no test files]
?   	golang.org/x/mobile/event/mouse	[no test files]
?   	golang.org/x/mobile/event/paint	[no test files]
?   	golang.org/x/mobile/event/size	[no test files]
?   	golang.org/x/mobile/event/touch	[no test files]
?   	golang.org/x/mobile/example/basic	[no test files]
?   	golang.org/x/mobile/example/bind/hello	[no test files]
?   	golang.org/x/mobile/example/flappy	[no test files]
?   	golang.org/x/mobile/example/network	[no test files]
?   	golang.org/x/mobile/exp/app/debug	[no test files]
?   	golang.org/x/mobile/exp/audio/al	[no test files]
ok  	golang.org/x/mobile/exp/f32	0.007s
ok  	golang.org/x/mobile/exp/font	0.014s
ok  	golang.org/x/mobile/exp/gl/glutil	0.005s [no tests to run]
?   	golang.org/x/mobile/exp/sensor	[no test files]
?   	golang.org/x/mobile/exp/sprite	[no test files]
ok  	golang.org/x/mobile/exp/sprite/clock	0.002s
?   	golang.org/x/mobile/exp/sprite/glsprite	[no test files]
ok  	golang.org/x/mobile/exp/sprite/portable	0.008s
?   	golang.org/x/mobile/geom	[no test files]
?   	golang.org/x/mobile/gl	[no test files]
ok  	golang.org/x/mobile/internal/binres	0.089s
ok  	golang.org/x/mobile/internal/importers	0.030s
ok  	golang.org/x/mobile/internal/importers/java	1.928s
ok  	golang.org/x/mobile/internal/importers/objc	0.013s
?   	golang.org/x/mobile/internal/mobileinit	[no test files]
@bradfitz

This comment has been minimized.

Sign in to view
Copy link
Member

commented Mar 13, 2019

Actually I'll close this and let @eliasnaur open a new bug for x/mobile.

@bradfitz bradfitz closed this Mar 13, 2019

@eliasnaur

This comment has been minimized.

Sign in to view
Copy link
Contributor

commented Mar 13, 2019

Yes, that was a flake. Thanks for the investigation.

@bcmills bcmills referenced this issue Mar 13, 2019

Closed

x/build: warn in TryBot CL comment if the parent commit is old #30807

@bradfitz

This comment has been minimized.

Sign in to view
Copy link
Member

commented Mar 14, 2019

@eliasnaur, can you file a bug about the flake and de-flake it? To get these as trybots, we need to get them as un-flaky as possible.
@eliasnaur

This comment has been minimized.

Sign in to view
Copy link
Contributor

commented Mar 14, 2019

I would have fixed it yesterday but traveling came in the way. I'll deflake this and a related test soon, perhaps today.
@gopherbot

This comment has been minimized.

Sign in to view
Copy link

commented Mar 14, 2019

Change https://golang.org/cl/167657 mentions this issue: bind/java: deflake testJavaRefKeep

gopherbot pushed a commit to golang/mobile that referenced this issue Mar 14, 2019

@eliasnaur
bind/java: deflake testJavaRefKeep 
Retry a few times to ensure the garbage collector collects what
we expect. Also merge and remove testJavaRefGC which was identical
except for one line.

Updates golang/go#30785

Change-Id: Id6c541b41fe483633c40eeea712c1f43b647e4f4
Reviewed-on: https://go-review.googlesource.com/c/mobile/+/167657
Run-TryBot: Elias Naur <mail@eliasnaur.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
f2ef6f4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.