Skip to content
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

Upload of the daily snapshot to gh pages doesn't work #308

Closed
iloveeclipse opened this issue Aug 8, 2017 · 6 comments
Closed

Upload of the daily snapshot to gh pages doesn't work #308

iloveeclipse opened this issue Aug 8, 2017 · 6 comments
Assignees

Comments

@iloveeclipse
Copy link
Member

See #302 and #304, log file: https://travis-ci.org/spotbugs/spotbugs/builds/262106745

We fail with:

!!! GitHub Pages support is experimental !!!

travis_fold:end:dpl.1
/home/travis/.rvm/rubies/ruby-2.2.6/lib/ruby/2.2.0/fileutils.rb:1386:in `copy': unknown file type: /home/travis/build/spotbugs/spotbugs/eclipsePlugin/build/site/eclipse-daily/. (RuntimeError)
	from /home/travis/.rvm/rubies/ruby-2.2.6/lib/ruby/2.2.0/fileutils.rb:471:in `block in copy_entry'
	from /home/travis/.rvm/rubies/ruby-2.2.6/lib/ruby/2.2.0/fileutils.rb:1497:in `call'
	from /home/travis/.rvm/rubies/ruby-2.2.6/lib/ruby/2.2.0/fileutils.rb:1497:in `wrap_traverse'
	from /home/travis/.rvm/rubies/ruby-2.2.6/lib/ruby/2.2.0/fileutils.rb:468:in `copy_entry'
	from /home/travis/.rvm/rubies/ruby-2.2.6/lib/ruby/2.2.0/fileutils.rb:443:in `block in cp_r'
	from /home/travis/.rvm/rubies/ruby-2.2.6/lib/ruby/2.2.0/fileutils.rb:1570:in `block in fu_each_src_dest'
	from /home/travis/.rvm/rubies/ruby-2.2.6/lib/ruby/2.2.0/fileutils.rb:1584:in `fu_each_src_dest0'
	from /home/travis/.rvm/rubies/ruby-2.2.6/lib/ruby/2.2.0/fileutils.rb:1568:in `fu_each_src_dest'
	from /home/travis/.rvm/rubies/ruby-2.2.6/lib/ruby/2.2.0/fileutils.rb:442:in `cp_r'
	from /home/travis/.rvm/gems/ruby-2.2.6/gems/dpl-1.8.40/lib/dpl/provider/pages.rb:68:in `block in push_app'
	from /home/travis/.rvm/rubies/ruby-2.2.6/lib/ruby/2.2.0/tmpdir.rb:88:in `mktmpdir'
	from /home/travis/.rvm/gems/ruby-2.2.6/gems/dpl-1.8.40/lib/dpl/provider/pages.rb:67:in `push_app'
	from /home/travis/.rvm/gems/ruby-2.2.6/gems/dpl-1.8.40/lib/dpl/provider.rb:154:in `block in deploy'
	from /home/travis/.rvm/gems/ruby-2.2.6/gems/dpl-1.8.40/lib/dpl/cli.rb:41:in `fold'
	from /home/travis/.rvm/gems/ruby-2.2.6/gems/dpl-1.8.40/lib/dpl/provider.rb:154:in `deploy'
	from /home/travis/.rvm/gems/ruby-2.2.6/gems/dpl-1.8.40/lib/dpl/cli.rb:32:in `run'
	from /home/travis/.rvm/gems/ruby-2.2.6/gems/dpl-1.8.40/lib/dpl/cli.rb:7:in `run'
	from /home/travis/.rvm/gems/ruby-2.2.6/gems/dpl-1.8.40/bin/dpl:5:in `<top (required)>'
	from /home/travis/.rvm/gems/ruby-2.2.6/bin/dpl:22:in `load'
	from /home/travis/.rvm/gems/ruby-2.2.6/bin/dpl:22:in `<main>'
iloveeclipse added a commit that referenced this issue Aug 8, 2017
@iloveeclipse
Copy link
Member Author

ls $TRAVIS_BUILD_DIR/eclipsePlugin/build/site/eclipse-daily ls: cannot access /home/travis/build/spotbugs/spotbugs/eclipsePlugin/build/site/eclipse-daily: No such file or directory

iloveeclipse added a commit that referenced this issue Aug 8, 2017
@iloveeclipse
Copy link
Member Author

Ok, looks like someone (travis?) deletes the files between script: and after_success steps, see log below: https://travis-ci.org/spotbugs/spotbugs/builds/262136943?utm_source=github_status&utm_medium=notification.

Looks like gradlew coveralls deletes generated files?

iloveeclipse added a commit that referenced this issue Aug 8, 2017
iloveeclipse added a commit that referenced this issue Aug 8, 2017
@iloveeclipse
Copy link
Member Author

At least the coveralls task fails with the error

Caused by: java.io.IOException: Error while analyzing /home/travis/build/spotbugs/spotbugs/spotbugsTestCases/build/classes/java/main/A.class.
	at org.jacoco.core.analysis.Analyzer.analyzerError(Analyzer.java:155)
	at org.jacoco.core.analysis.Analyzer.analyzeClass(Analyzer.java:130)
	at org.jacoco.core.analysis.Analyzer.analyzeClass(Analyzer.java:147)
	at org.jacoco.core.analysis.Analyzer.analyzeAll(Analyzer.java:185)
	at org.jacoco.core.analysis.Analyzer.analyzeAll(Analyzer.java:218)
	at org.jacoco.core.analysis.Analyzer.analyzeAll(Analyzer.java:213)
	at org.jacoco.ant.ReportTask.createBundle(ReportTask.java:567)
	at org.jacoco.ant.ReportTask.createReport(ReportTask.java:542)
	at org.jacoco.ant.ReportTask.execute(ReportTask.java:495)
	... 129 more
Caused by: java.lang.IllegalStateException: Can't add different class with same name: A
	at org.jacoco.core.analysis.CoverageBuilder.visitCoverage(CoverageBuilder.java:107)
	at org.jacoco.core.analysis.Analyzer$1.visitEnd(Analyzer.java:96)
	at org.objectweb.asm.ClassVisitor.visitEnd(ClassVisitor.java:317)
	at org.jacoco.core.internal.flow.ClassProbesAdapter.visitEnd(ClassProbesAdapter.java:98)
	at org.objectweb.asm.ClassReader.accept(ClassReader.java:712)
	at org.objectweb.asm.ClassReader.accept(ClassReader.java:521)
	at org.jacoco.core.analysis.Analyzer.analyzeClass(Analyzer.java:111)
	at org.jacoco.core.analysis.Analyzer.analyzeClass(Analyzer.java:127)
	... 136 more

iloveeclipse added a commit that referenced this issue Aug 8, 2017
iloveeclipse added a commit that referenced this issue Aug 8, 2017
@iloveeclipse
Copy link
Member Author

OK, after few attempts I've fixed gradlew coveralls task but the problem still remains, that after this task all generated site data is lost. WTF???

iloveeclipse added a commit that referenced this issue Aug 8, 2017
iloveeclipse added a commit that referenced this issue Aug 8, 2017
The coverage was executed also on test code, and tests contained
duplicated classes (which is OK), but this shoked jacoco build. We
should not run coverage on test code, it does not make any sense.
iloveeclipse added a commit that referenced this issue Aug 8, 2017
The coveralls task deletes eclipsePlugin/build/site directory contents
(I don't know why). The workaround is just to retrigger site generation
task for the daily update site.
@iloveeclipse
Copy link
Member Author

See PR #310. I have no idea why coveralls deletes data, so I did a workaround.

KengoTODA pushed a commit that referenced this issue Aug 9, 2017
The coverage was executed also on test code, and tests contained
duplicated classes (which is OK), but this shoked jacoco build. We
should not run coverage on test code, it does not make any sense.
KengoTODA pushed a commit that referenced this issue Aug 9, 2017
The coveralls task deletes eclipsePlugin/build/site directory contents
(I don't know why). The workaround is just to retrigger site generation
task for the daily update site.
@iloveeclipse
Copy link
Member Author

Finally with PR #311 we fixed deployment and the build!

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

No branches or pull requests

2 participants