You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ro...@apache.org on 2022/08/22 10:25:44 UTC

[activemq-artemis] branch new-logging updated (e9bdbd0854 -> dca9127cbd)

This is an automated email from the ASF dual-hosted git repository.

robbie pushed a change to branch new-logging
in repository https://gitbox.apache.org/repos/asf/activemq-artemis.git


 discard e9bdbd0854 fix some file references and update notes
 discard 5f975277c3 convert some more test loggers
 discard 32586abc34 remove slf4j->jbl bridge from example
 discard f80ce402a1 replace a couple of examples logging.properties that enable audit logger output
 discard dc57abd97f add a note
 discard 787afaad17 remove exclusion that is no longer needed
 discard 16536ddc91 switch another logger missed previously
 discard 8ae5e8797a fixups after rebase to main at 0ca44c0e6972991cf80e5feda34f6d02cf94dea0
 discard f3603a084f remove uses of JBL from compatibility tests, add artemis-unit-test-support to snapshots CP to ensure test appender class is found and avoids stacktracing
 discard 6741264882 add another TODO note
 discard c8547c50c8 move another old log config file out to root, for any later comparisons needed
 discard d210e75f05 replace logging.properties files for e2e-tests server configs
 discard f3b087f6f6 remove commons-logging slf4j bridge now the karaf bits dont cause beanutils log spam
 discard a574123705 move karaf server feature test to its own module, its logging deps mess with the commons-config handling
 discard 2456ca3351 delete old Log4J 1.x log4j.properties files that shouldnt be doing anything, build isnt using log4j 1.x / reload4j anymore
 discard 2b1f65fb55 update deps to match actual SLF4J base usage and simplify
 discard 2999d99369 use latest log4j version
 discard 29ec97e592 remove now-unused import
 discard dfd7f56a2d update notes
 discard c583ea9681 remove rather than reinstate a couple of unnecesarry level manipulations
 discard 8dbae6a051 ensure method or class based cleanups for AssertionLoggerHandler exist or will be effective, where startCapture() has been called previously
 discard 5d243844ad reinstate another test manipulating levels
 discard 7aac36f3e7 restore one of the remaining tests using JBL, manipulating levels
 discard ed693eb704 Revert "add commons-logging -> SLF4J bridge and exclude commons-logging, suppresses unconfigured log spam from beanutils etc"
 discard b0deddac27 Revert "Moving dependencies as test"
 discard a62681d2b8 Revert "Remove slf4j bridge depedencies from libs", towards getting build back to consistent deps state and to more easily revert original addition leading to issue.
 discard ecbfeaadcb fixing build after rebase
 discard 3e0479104b Rebased upstream/1662bc740eb4435f5236a70e310825c1daef2e08
 discard 597bfddfda fixing test
 discard cd786ada61 Moving dependencies as test
 discard 9a801920f9 Remove slf4j bridge depedencies from libs
 discard bebf88a722 Removing log facade package
 discard 4e445f677d make start on updating remaining integration tests, manipulating levels etc
 discard 8f6b8850e9 remove various mentions of jblm and wildfly-common from scripts etc, update service config logging details..now to see what it breaks
 discard 11a2636505 also move the old main tests logging.config file
 discard f2aaf0bb75 move old base logging.properties out of artemis-cli, but leave it in root for comparison to see needed config updates as others replaced
 discard 722f185ac8 add some TODOs for area cleanup still needed
 discard 7b43f29e41 change remaining integration tests except those with TODOs
 discard bbe9e2724b Adding safeguard against {?} on LogProcessor
 discard 628199186d fix compile issues IDE somehow decided not to show
 discard 47754ad947 more test changes
 discard 92751e615f more test updates
 discard 62451d9d32 some more integration test updates
 discard 45f34eaa77 make start on updating integration tests
 discard 56a2b2af28 add note about e2e-test fixups needed
 discard 7027029f87 add artemis-unit-test-support dep to various modules that have tests, avoid stacktrace on not finding appender from shared logging config
 discard a712ef436f Verifying JMX Security and setting up javax.management.builder.initial before Log4J starts its own JMX
 discard 6406adb1a5 add artemis-unit-test-support dep to various modules that have tests, and will stacktrace if the appender isnt found which is referenced in their shared logging test config
 discard 9f20820f10 fix dep scope
 discard 2febad9879 add note about many integration-tests etc still using JBL
 discard c1d300ba4a fix other files similar to commit titled 'Fixing duplicate properties'
 discard 31da1baeae Fixing duplicate properties
 discard 45525b6b13 Setting version as 3.0.0
 discard 0096670dc2 updating version after rebasing
 discard 1af97a6d42 add log config to get NettyNativeTest working
 discard 5285aab49b simplify test logging config again, adjust assertion to be more specific and take into account change in output from 07329118c10d14bce018a580b0dadbcf081233a8
 discard 5347ef1dea add log config for AuditLoggerAMQPMutualSSLTest
 discard bc616444dd ARTEMIS-3902 Adding Reason to Security Exception
 discard a7d41ccb51 add some TODO notes from memory
 discard b2c443637d add some [in-flux] config to get more failing audit log smoke tests passing
 discard 3c3831d497 apply needed change only, and remove old unused logging.properties file
 discard 300087f1ed reset smoke test config to current base assembly config
 discard ea20c6cc96 some more helper additions to bring default assembly logging config into line with prior config
 discard 2ddff5be10 Adding a test with no argument on exception (just to be sure)
 discard e5ff7898b5 Removing @Cause annotation
 discard 620937cc76 fix up errorprone error and an IDE warning
 discard 86ec72e454 ensure exceptions/causes are the last arg passed, as the logging framework expects, and processor now enforces
 discard 21ccb9e9e6 fix checkstyle on processor
 discard dc5f2da3e8 small tweak on test
 discard 8bb2f9c703 Enforcing exception as the last argument
 discard d0bb050b4c log processing improvement with exceptions
 discard 57f910c679 flesh out assembly default logging config, give output like previous behaviour
 discard 6137392b44 Removing Reload4J
 discard 8d677667b2 fix some more placeholders missed previously
 discard ea504a0d47 make basic start on audit log file config, add [unfinished] config changes to get AuditLoggerResourceTest smoke test passing
 discard b73b7806a3 reorder args to match previously out-of-order placeholders, ensure new output matches previous
 discard 22a366e114 remove now-unnecessary quote escaping from log/message bundle format strings, giving same output as before switch to slf4j formatter
 discard e5dcfc0062 switch to released 2.0.0 artemis-native dep
 discard 427172c08f Fixing up some {0} versus {}
 discard 95522d5bcb re-enable GHA CI runs in the main repo, build now in state it can pass the PR test subset (except for d153efde4e04703875c92fdcb45f9f65b87a215b needing reverted which would make it fail)
 discard e1c63ade71 add TODO for another fun task
 discard 2f78510859 fix various test failures due to unexpected message string; also make some tests indicate why they failed
 discard f0b2754d81 temporarily remove test to allow concentrating on other issues
 discard dfbde76fda fix test missed in prior change
 discard d31b73f96b fix format of code prefix in messages, pointed out by failing test
 discard 5ddddc37b6 make a start on a basic loging config for distribution
 discard 1f1e6ef1b1 add some more needed deps; essentially every module with tests (or without specific log test config) will need this to avoid a stacktrace from logging setup issues
 discard 8c4416abbc avoid NPE in a unit test due to audit loggers still being enabled, unlike prior test logging config
 discard 6d385b4c8f Using Artemis Native 2.1.0 snapshot until 2.0.0 is released
 discard 49e596146f add some deps needed/used already, look into test failure
 discard 9d998c6986 fixup for release profile following prior rebases+squashes
 discard 643c43b893 add commons-logging -> SLF4J bridge and exclude commons-logging, suppresses unconfigured log spam from beanutils etc
 discard a99f7abd32 basic logging config and some more on AssertionLoggerHandler to get some tests using it working
 discard 9c343afba6 make a start on restoring AssertionLoggerHandler to get tests going
 discard 35da8ef084 Artemis native at 2.0 snapshot
 discard 4e50c840f0 squashed logging changes from Clebert + me, rebased onto main at 688b894c6205058c24d225b340aea1d8a5599902
     add 51825b18ba NO-JIRA: remove now-stale comments on tests that I missed when switching to cp files in 71b781d9193c64e34e9d986939af40745ea9740b
     add c96243698d NO-JIRA Fixing suggested script for test variables
     add 9acd036dcc ARTEMIS-3928 Fixing yield and shutdownNow
     add 86db53da9a ARTEMIS-3918 support FQQN + anycast + redistribution
     add f1c9e3df14 ARTEMIS-3947 fix merging for expiry address-settings
     add fb292fde35 This closes #4182
     new 619d7f8dcf squashed logging changes from Clebert + me, rebased onto main at 688b894c6205058c24d225b340aea1d8a5599902
     new 0073fc9252 Artemis native at 2.0 snapshot
     new 2ec9c1e5e9 make a start on restoring AssertionLoggerHandler to get tests going
     new ae1209ffb2 basic logging config and some more on AssertionLoggerHandler to get some tests using it working
     new 19a55b27a8 add commons-logging -> SLF4J bridge and exclude commons-logging, suppresses unconfigured log spam from beanutils etc
     new 31b46b98c2 fixup for release profile following prior rebases+squashes
     new 2acdf12480 add some deps needed/used already, look into test failure
     new 4c399512b5 Using Artemis Native 2.1.0 snapshot until 2.0.0 is released
     new 0768fc187e avoid NPE in a unit test due to audit loggers still being enabled, unlike prior test logging config
     new 9c6d08ea92 add some more needed deps; essentially every module with tests (or without specific log test config) will need this to avoid a stacktrace from logging setup issues
     new 8ee0f9d392 make a start on a basic loging config for distribution
     new 0dcac3ffd3 fix format of code prefix in messages, pointed out by failing test
     new f2e7b37ef7 fix test missed in prior change
     new 1ff0d19a5b temporarily remove test to allow concentrating on other issues
     new 52c7685742 fix various test failures due to unexpected message string; also make some tests indicate why they failed
     new 8e23ff25be add TODO for another fun task
     new c9dfcd1ace re-enable GHA CI runs in the main repo, build now in state it can pass the PR test subset (except for d153efde4e04703875c92fdcb45f9f65b87a215b needing reverted which would make it fail)
     new f18d68ad2f Fixing up some {0} versus {}
     new f6e04b85c7 switch to released 2.0.0 artemis-native dep
     new 28cf49861f remove now-unnecessary quote escaping from log/message bundle format strings, giving same output as before switch to slf4j formatter
     new 3d66be0c09 reorder args to match previously out-of-order placeholders, ensure new output matches previous
     new 2245603ad8 make basic start on audit log file config, add [unfinished] config changes to get AuditLoggerResourceTest smoke test passing
     new c9d7fd40ce fix some more placeholders missed previously
     new 3819ff98ec Removing Reload4J
     new b24055f947 flesh out assembly default logging config, give output like previous behaviour
     new 9bbb3a69b5 log processing improvement with exceptions
     new 5155f88a4f Enforcing exception as the last argument
     new c2c275b0cf small tweak on test
     new 7ab5bca92c fix checkstyle on processor
     new b6ecd1cd20 ensure exceptions/causes are the last arg passed, as the logging framework expects, and processor now enforces
     new ea562bb0e4 fix up errorprone error and an IDE warning
     new 972fc66d44 Removing @Cause annotation
     new d8b234af38 Adding a test with no argument on exception (just to be sure)
     new bfd3c85532 some more helper additions to bring default assembly logging config into line with prior config
     new 52e876e180 reset smoke test config to current base assembly config
     new 9456b33fd8 apply needed change only, and remove old unused logging.properties file
     new fb4977b4dd add some [in-flux] config to get more failing audit log smoke tests passing
     new 4110244ced add some TODO notes from memory
     new c20b2dcedc ARTEMIS-3902 Adding Reason to Security Exception
     new 339c90fb91 add log config for AuditLoggerAMQPMutualSSLTest
     new 0e01d3019d simplify test logging config again, adjust assertion to be more specific and take into account change in output from 07329118c10d14bce018a580b0dadbcf081233a8
     new d5e0222461 add log config to get NettyNativeTest working
     new d734e4d667 updating version after rebasing
     new 61e813b99e Setting version as 3.0.0
     new 01fdcba41e Fixing duplicate properties
     new a59c62707e fix other files similar to commit titled 'Fixing duplicate properties'
     new dc91ae3063 add note about many integration-tests etc still using JBL
     new 6733d22738 fix dep scope
     new eb0d2db6a9 add artemis-unit-test-support dep to various modules that have tests, and will stacktrace if the appender isnt found which is referenced in their shared logging test config
     new 9cfbb68068 Verifying JMX Security and setting up javax.management.builder.initial before Log4J starts its own JMX
     new b4ebd4ce2c add artemis-unit-test-support dep to various modules that have tests, avoid stacktrace on not finding appender from shared logging config
     new 8c8792f778 add note about e2e-test fixups needed
     new 33c58663c4 make start on updating integration tests
     new 68dc834633 some more integration test updates
     new be6e62a3f1 more test updates
     new c6efc0240e more test changes
     new 2ccca7c7b4 fix compile issues IDE somehow decided not to show
     new 5ae9b8af9a Adding safeguard against {?} on LogProcessor
     new 58422bcd95 change remaining integration tests except those with TODOs
     new eb59d2373a add some TODOs for area cleanup still needed
     new 7bd1d3c1d7 move old base logging.properties out of artemis-cli, but leave it in root for comparison to see needed config updates as others replaced
     new 02523f06c1 also move the old main tests logging.config file
     new a394609d6d remove various mentions of jblm and wildfly-common from scripts etc, update service config logging details..now to see what it breaks
     new 3abfd6fad5 make start on updating remaining integration tests, manipulating levels etc
     new c3874ee405 Removing log facade package
     new 87d69b3552 Remove slf4j bridge depedencies from libs
     new 33cc92d7df Moving dependencies as test
     new a6a2d5b33f fixing test
     new 45976bd903 Rebased upstream/1662bc740eb4435f5236a70e310825c1daef2e08
     new eeb52d8122 fixing build after rebase
     new 0e9ec84a64 Revert "Remove slf4j bridge depedencies from libs", towards getting build back to consistent deps state and to more easily revert original addition leading to issue.
     new af5bc2ba22 Revert "Moving dependencies as test"
     new 956a87459f Revert "add commons-logging -> SLF4J bridge and exclude commons-logging, suppresses unconfigured log spam from beanutils etc"
     new ca5efc02b1 restore one of the remaining tests using JBL, manipulating levels
     new 66e877005f reinstate another test manipulating levels
     new a12dad1a0a ensure method or class based cleanups for AssertionLoggerHandler exist or will be effective, where startCapture() has been called previously
     new ac25c21df5 remove rather than reinstate a couple of unnecesarry level manipulations
     new ef83b1142e update notes
     new 03e8e6e141 remove now-unused import
     new 1da884eb9a use latest log4j version
     new 6f0daf8f0d update deps to match actual SLF4J base usage and simplify
     new 89a9c0287b delete old Log4J 1.x log4j.properties files that shouldnt be doing anything, build isnt using log4j 1.x / reload4j anymore
     new 07ffa54499 move karaf server feature test to its own module, its logging deps mess with the commons-config handling
     new 4b46813327 remove commons-logging slf4j bridge now the karaf bits dont cause beanutils log spam
     new 1df5d62755 replace logging.properties files for e2e-tests server configs
     new 2bf49612aa move another old log config file out to root, for any later comparisons needed
     new 13f15eb1a3 add another TODO note
     new f34c8c83e6 remove uses of JBL from compatibility tests, add artemis-unit-test-support to snapshots CP to ensure test appender class is found and avoids stacktracing
     new e57dfbad67 fixups after rebase to main at 0ca44c0e6972991cf80e5feda34f6d02cf94dea0
     new 0a40553600 switch another logger missed previously
     new a7a5a8efcf remove exclusion that is no longer needed
     new ab7eaed9cf add a note
     new 852ed3583a replace a couple of examples logging.properties that enable audit logger output
     new d6405d8f0e remove slf4j->jbl bridge from example
     new f53f19eae3 convert some more test loggers
     new dca9127cbd fix some file references and update notes

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (e9bdbd0854)
            \
             N -- N -- N   refs/heads/new-logging (dca9127cbd)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

The 96 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../artemis/utils/actors/ProcessorBase.java        |  38 +++----
 .../artemis/core/postoffice/impl/BindingsImpl.java |   2 +-
 .../core/postoffice/impl/PostOfficeImpl.java       |   3 +-
 .../server/cluster/impl/ClusterConnectionImpl.java |   5 +-
 .../core/server/impl/ServerSessionImpl.java        |   2 +-
 .../core/settings/impl/AddressSettings.java        |  12 +--
 .../artemis/core/settings/AddressSettingsTest.java |   6 ++
 .../AddressPauseJournalCompatibilityTest.java      |  13 ---
 ...ctionFactoryConfigurationSerializationTest.java |  13 ---
 .../tests/compatibility/ExportImportTest.java      |  13 ---
 .../tests/compatibility/HQLargeMeshTest.java       |  13 ---
 .../tests/compatibility/HQSelectorTest.java        |  13 ---
 .../tests/compatibility/HornetQSoakTest.java       |  13 ---
 .../compatibility/JournalCompatibilityTest.java    |  13 ---
 .../artemis/tests/compatibility/Mesh2Test.java     |  13 ---
 .../artemis/tests/compatibility/MeshTest.java      |  13 ---
 .../tests/compatibility/OldOpenWireTest.java       |  13 ---
 .../QueuePauseJournalCompatibilityTest.java        |  13 ---
 .../artemis/tests/compatibility/ReplyToTest.java   |  13 ---
 .../artemis/tests/compatibility/SendAckTest.java   |  13 ---
 .../tests/compatibility/SerializationTest.java     |  13 ---
 .../artemis/tests/compatibility/package-info.java  |   3 +-
 .../distribution/MessageRedistributionTest.java    | 119 +++++++++++++++++++--
 .../src/test/scripts/parameters-paging-short.sh    |   3 -
 .../src/test/scripts/parameters-paging.sh          |   3 -
 25 files changed, 152 insertions(+), 226 deletions(-)