Skip to content

Conversation

@manovotn
Copy link
Member

No description provided.

@manovotn manovotn force-pushed the centralPortalMigration branch from 1ed9924 to 40e9912 Compare May 13, 2025 13:44
@manovotn manovotn force-pushed the centralPortalMigration branch from 40e9912 to 4aad4cb Compare May 13, 2025 13:49
@manovotn
Copy link
Member Author

I am not sure what are the failures about.
There were no code changes causing this and it works locally and looking at older runs in weld/core, the CI env setup was identical too. Possibly a transient failure?

@manovotn
Copy link
Member Author

I've gone through some additional steps trying to identify this:

  • More details in the env setup of the CI
    • All of which appear to be the same except image Runner Image Provisioner but I don't see how that would cause an issue
  • Tried different PR that would run on older Ubuntu version just in case
    • Didn't work either
  • Looked into how we use this JMX testing to see if we actually need it since the container is ancient
    • Apart from some of our tests using it, it has the ability to isolate class path and most importantly its API is udes by CDI SE TCK so thre is no way around it
  • Checked that the the JMX registration process passes
    • Which it does; even these failing jobs have logs such as INFO: JMXTestRunner initialized using [jdk.internal.loader.ClassLoaders$AppClassLoader@1dbd16a6]
  • I am also looking into these tests with FINE level logging and comparing that between the CI runs and local runs

@manovotn
Copy link
Member Author

manovotn commented May 14, 2025

I am also looking into these tests with FINE level logging and comparing that between the CI runs and local runs

Well, looks like the other CI run (weld/core#3163) is suddenly passing and now this one as well? 👀

@manovotn manovotn merged commit e22743d into weld:master May 15, 2025
24 of 54 checks passed
@manovotn manovotn deleted the centralPortalMigration branch May 15, 2025 16:16
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.

1 participant