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

FISH-10146 FISH-9875 Deployment Group Nullpointer #7099

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

Conversation

Viii3
Copy link
Member

@Viii3 Viii3 commented Nov 28, 2024

Description

  • Resolves FISH-10146
    • Instances no longer strip data from the <deployment-groups> block of the domain xml.
    • Instances have improved checking for deciding whether or not to discard a config from their own domain xml.
      • Previously instances would discard any config for instances not in their first deployment group. They now check all of their deployment groups.
  • Resolves FISH-9875
    • Deployment groups that share instances no longer exhibit inconsistent behaviour.

Important Info

Blockers

Testing

New tests

Testing Performed

Tested using a modified reproducer for FISH-10146:

The following batch file was used for testing.

@ECHO OFF
CALL asadmin start-domain
echo.
CALL asadmin create-local-instance instance1
echo.
CALL asadmin start-local-instance instance1
echo.
CALL asadmin create-deployment-group group1
echo.
CALL asadmin add-instance-to-deployment-group --instance instance1 --deploymentgroup group1
echo.
CALL asadmin create-local-instance instance2
echo.
CALL asadmin start-local-instance instance2
echo.
CALL asadmin add-instance-to-deployment-group --instance instance2 --deploymentgroup group1
echo.
CALL asadmin stop-local-instance instance1
echo.
CALL asadmin stop-local-instance instance2
echo.
CALL asadmin stop-domain

Testing Environment

Maven version: 3.9.6
Java version: 11.0.23, vendor: Eclipse Adoptium
Default locale: en_GB, platform encoding: Cp1252
OS name: "windows 11", version: "10.0", arch: "amd64", family: "windows"

Documentation

Notes for Reviewers

@breakponchito
Copy link
Contributor

Jenkins test please

@Viii3
Copy link
Member Author

Viii3 commented Nov 29, 2024

There appears to be a Java versioning error in some of the tests, one of the test classes has been compiled with Java 17.

Caused by: org.apache.maven.plugin.PluginContainerException: An API incompatibility was encountered while executing org.apache.felix:maven-bundle-plugin:6.0.0:bundle: java.lang.UnsupportedClassVersionError: aQute/bnd/osgi/Analyzer has been compiled by a more recent version of the Java Runtime (class file version 61.0), this version of the Java Runtime only recognizes class file versions up to 55.0

@breakponchito
Copy link
Contributor

I already tested and the issue now is not presented

Copy link
Contributor

@breakponchito breakponchito left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Viii3 Viii3 changed the title FISH-10146 Deployment Group Nullpointer FISH-10146 FISH-9875 Deployment Group Nullpointer Dec 2, 2024
@breakponchito
Copy link
Contributor

Jenkins test please

@Viii3 Viii3 force-pushed the FISH-10146-Nullpointer-Instance branch from 86d290c to 375b756 Compare December 3, 2024 17:01
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.

3 participants