-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[WFLY-18562] re-evaluated test exclusions for ts.preview and ts.boota… #17330
Conversation
I need to test this on CI. Locally it fails on ModelControllerMBeanTestCase due to |
@istudens If that's the last thing blocking this (I saw some other CI failures but didn't investigate, nor have I done a code review), I suggest filing a WFLY against the Micrometer subsystem, and disabling the test with ts.preview with a comment noting the JIRA. I suspect this actually has nothing to do with Preview, except that we include Micrometer in the default config there, so this test picks up issues there. We support Micrometer in standard WF though, it's just not in the default config. I suspect this isn't a Micrometer bug per se, it's just it has an unusual thing, where an attribute of ModelType.LIST has a default value. That's a valid thing but it sounds like JMX might not like that. |
This needs https://issues.redhat.com/browse/WFCORE-6604 to fix the JMX issue. |
/retest |
@istudens The WFCORE-6604 fix is integrated into full WF now. |
e3037b2
to
db49746
Compare
…ble.preview profiles
Thanks @istudens! |
…ble.preview profiles
Issue: https://issues.redhat.com/browse/WFLY-18562