joewitt commented on PR #8708: URL: https://github.com/apache/nifi/pull/8708#issuecomment-2081686050
@exceptionfactory This changeset should do it. Hilariously reducing the number of log impls changes the behavior of aws-processors test. I had to change that in NIFI-12998 but now that it is back to one the assertions are different. I didn't look into why but that is...funny. The root of the adjustment here is we took logback-classic and logback-core out out of the dependency set (still in dep mgmt) of nifi-bom. This makes sense given we dont want things actually having compile time deps on them anyway. I did mark them as 'runtime' scope instead of provided. Meanwhile the nifi-py4j-bridge DOES have a direct code based requirement for logback-classic (and thus core). This seems undesirable but I didn't look into the full story there and instead just explained this and marked it as provided scope for that one module. Log output is dramatically reduced now thanks to the default behavior of test scoped slf4j-simple. Anyway this should be in much better shape. The integration tests seem fairly flakey in multiple local runs but they are able to pass most of the time. And now we should see why it fails instead of being buried in noisy logs. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
