ARTEMIS-5364 clean up scripts & profiles #5578
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jbertram
force-pushed
the
ARTEMIS-5364
branch
2 times, most recently
from
6259320 to
f15403f
Compare
jbertram
force-pushed
the
ARTEMIS-5364
branch
2 times, most recently
from
fac489d to
267c065
Compare
jbertram
changed the title
This commit cleans up the various scripts and profile files via the following changes: - Remove ARTEMIS_INSTANCE_URI from all profiles since it is no longer used after we moved to Log4j2 - Remove ARTEMIS_INSTANCE_ETC_URI from all profiles; it is not necessary because the jolokia-access.xml can be loaded directly from the classpath - Remove ARTEMIS_ETC_DIR from all profiles and instead rely on ARTEMIS_INSTANCE_ETC which is already being set in the scripts - Remove unnecessary code from Create command - Remove ARTEMIS_OOME_DUMP from the utility profiles since it's not used - Adjust Upgrade command & related tests - Fix Windows service file so it uses the same memory settings as the other configs - Update the Windows' profile JAVA_ARGS to match those from Linux
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This commit cleans up the various scripts and profile files via the following changes: