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

[WFCORE-7076] Check required installation metadata is present before registering installation-manager subsystem #6266

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

spyrkob
Copy link
Contributor

@spyrkob spyrkob commented Nov 29, 2024

@spyrkob spyrkob force-pushed the WFCORE-7076_instaman_check_required_files branch from 857f3a3 to c13f1ee Compare November 29, 2024 12:52
@spyrkob
Copy link
Contributor Author

spyrkob commented Nov 29, 2024

@yersan I tried to implement your suggestion about changing ModelControllerServiceInitialization to pass the PathManager to the initializer. It ended up with a lot of other changes - could you take a look at the PR if that's roughly what you had in mind?

@wildfly-ci
Copy link

Core -> Full Integration Build 14377 outcome was FAILURE using a merge of c13f1ee
Summary: Exit code 1 (Step: Build core (Maven)) (new) Build time: 00:02:27

@wildfly-ci
Copy link

Core -> Full Integration Build 14080 outcome was FAILURE using a merge of c13f1ee
Summary: Exit code 1 (Step: Build core (Maven)) (new) Build time: 00:02:53

@wildfly-ci
Copy link

Core -> WildFly Preview Integration Build 14157 outcome was FAILURE using a merge of c13f1ee
Summary: Exit code 1 (Step: Build core (Maven)) (new) Build time: 00:02:05

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.

2 participants