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-7087] Support resolution of extension expressions during prep… #6275

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

bstansberry
Copy link
Contributor

@bstansberry bstansberry commented Dec 12, 2024

…aration of a domain server launch command

https://issues.redhat.com/browse/WFCORE-7087

This PR also includes a fix for another problem that prevented the test added for WFCORE-7087 passing when the full testsuite/domain is run:

https://issues.redhat.com/browse/WFCORE-7138

It also includes a DomainTestSuite enhancement that I used to find the WFCORE-7138 problem:

https://issues.redhat.com/browse/WFCORE-7137

@bstansberry bstansberry reopened this Dec 13, 2024
@wildfly wildfly deleted a comment from wildfly-ci Dec 15, 2024
@wildfly wildfly deleted a comment from wildfly-ci Dec 15, 2024
@wildfly wildfly deleted a comment from wildfly-ci Dec 15, 2024
@wildfly wildfly deleted a comment from wildfly-ci Dec 15, 2024
@wildfly wildfly deleted a comment from wildfly-ci Jan 12, 2025
@bstansberry bstansberry marked this pull request as ready for review January 12, 2025 23:50
@bstansberry bstansberry requested review from fjuma and yersan January 12, 2025 23:51
@wildfly wildfly deleted a comment from wildfly-ci Jan 14, 2025
@bstansberry bstansberry added the hold Do not merge this PR label Jan 14, 2025
@bstansberry
Copy link
Contributor Author

The new test fails because some other test is leaving the secondary HC in restart-required, so when the new test checks that it's properly reloaded into 'running' that fails.

I'm looking into adding a utility to DomainTestSupport to check for such a thing and fail the stop of the naughty test.

@wildfly-ci

This comment was marked as outdated.

@bstansberry bstansberry removed the hold Do not merge this PR label Jan 17, 2025
@bstansberry bstansberry added the hold Do not merge this PR label Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hold Do not merge this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants