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

update travis config to use openjdk8 #179

Merged
merged 1 commit into from
Nov 26, 2019
Merged

update travis config to use openjdk8 #179

merged 1 commit into from
Nov 26, 2019

Conversation

ctomc
Copy link
Member

@ctomc ctomc commented Oct 29, 2019

No description provided.

@ctomc
Copy link
Member Author

ctomc commented Oct 29, 2019

with this change travis jobs run again but there are problems in tests itself.

@Ladicek
Copy link
Contributor

Ladicek commented Oct 29, 2019

Yup, thanks for the PR!

@Ladicek
Copy link
Contributor

Ladicek commented Oct 29, 2019

The test failure is clear, the test assumes that switching on journal store will automatically switch off JDBC store, et vice versa, which used to be true in some older versions of WildFly, but clearly isn't true in WildFly 12. It should be easy to rewrite the test, but I don't know who works on Creaper these days. @rsvoboda Do you know? :-)

@simkam
Copy link
Collaborator

simkam commented Oct 29, 2019

there is old PR with fix for JournalStoreVsJdbcStoreOnlineTest - #169

@Ladicek
Copy link
Contributor

Ladicek commented Oct 29, 2019

Ah, good point. Well, clearly someone needs to start taking care of Creaper again :-)

@rsvoboda
Copy link
Contributor

@ctomc @Ladicek I will bring this topic to the proper guys ;)

@rsvoboda
Copy link
Contributor

@msve please take a look

@Ladicek can you grant rights for creaper repo to @msve ?
He is "component owner" now on eap qe side.

@Ladicek
Copy link
Contributor

Ladicek commented Nov 25, 2019

Invite sent.

@msve msve merged commit 407f6d0 into master Nov 26, 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.

5 participants