Skip to content

[TRAVIS] Test enterprise modules#1707

Merged
matthiasblaesing merged 1 commit intoapache:masterfrom
hectorespert:test_enterprise_modules
Dec 6, 2019
Merged

[TRAVIS] Test enterprise modules#1707
matthiasblaesing merged 1 commit intoapache:masterfrom
hectorespert:test_enterprise_modules

Conversation

@hectorespert
Copy link
Member

Change Travis config to test enterprise modules

@hectorespert hectorespert marked this pull request as ready for review November 30, 2019 18:59
@mklaehn
Copy link
Contributor

mklaehn commented Nov 30, 2019

Build seems to have failed (https://travis-ci.org/apache/netbeans/jobs/618973448#L1046)
platform/api.htmlui test hadn't emitted any output for 10 minutes.
@matthiasblaesing maybe this is caused by your change of #1700 ?

@matthiasblaesing
Copy link
Contributor

@mklaehn it is not my change, I'm just the committer (not the author). And no the change is not the problem, the problem is either the travis infrastructure or the htmlui unittests. Given that this is a heisenbug, it is difficult to diagnose.

@hectorespert
Copy link
Member Author

@mklaehn @matthiasblaesing I created this pull request to revert the changes in api.htmlui tests: #1710
Seems that the problem are with api.htmlui tests that fails randomly.

@mklaehn
Copy link
Contributor

mklaehn commented Nov 30, 2019

@mklaehn it is not my change, I'm just the committer (not the author). And no the change is not the problem, the problem is either the travis infrastructure or the htmlui unittests. Given that this is a heisenbug, it is difficult to diagnose.

I saw your name at the top of the page wnd without reading further I made a bad assumption. Sorry for that.

@matthiasblaesing
Copy link
Contributor

@BLACKLEG is this intentional, that only a single module is tested?

@hectorespert hectorespert force-pushed the test_enterprise_modules branch from 2af6f83 to dd64ec3 Compare December 5, 2019 21:04
@hectorespert
Copy link
Member Author

It's intentional, I didn't time to test more modules when prepared the pull request. It's the base to add more enterprise modules in Travis config.

@hectorespert hectorespert force-pushed the test_enterprise_modules branch from dd64ec3 to 20955a4 Compare December 6, 2019 15:07
@matthiasblaesing
Copy link
Contributor

Thank you.

@matthiasblaesing matthiasblaesing merged commit 5175026 into apache:master Dec 6, 2019
@hectorespert hectorespert deleted the test_enterprise_modules branch December 6, 2019 18:35
@junichi11 junichi11 added this to the 11.3 milestone Dec 6, 2019
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

Successfully merging this pull request may close these issues.

4 participants