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
omit b8bab69059 basic change to allow a broker distribution to <just> start
and serve clients
omit 0fcc5f22ce unsaved change missed from previous commit
omit 6c8f3a257c some more fixups following review of delta on main prior to
last rebase, from e49a9521aaf6b706d692d797c32100a205704186 to
d0d2a9a16bb6664d1ebeb977f7213a9b8fa640da
omit 1c36d932da fixups for rebase from
e49a9521aaf6b706d692d797c32100a205704186 to
d0d2a9a16bb6664d1ebeb977f7213a9b8fa640da
omit ab1ed6c26f add TODOs and disable certain tests that need more
extensive fixups, gets overall build 'completing' for next steps
omit 0a71eb0532 update various tests to fix compilation
omit 132a445290 fixup various compilations issues in more modules, and
poking to get artemis-features build to complete
omit 7a2b481a31 add TODO about potentially expensive preparation of
probably-not-logged details
omit 233445878a get artemis-cli compiling
omit f31c69337a get artemis-mqtt-protocol compiling
omit 492c79ad56 get artemis-jms-server compiling
omit bf44d5bd14 add some TODO notes
omit 80e6fe4a4d get artemis-amqp-protocol compiling [again]
omit 804ae3c953 some more rebase fixups missed in prior commit
omit 67ba9eadc5 Fixing clashes after rebasing
omit 1f961de74b remove log config reloader that depends on jboss logmanager
omit 9036de4dd4 WIP - some osgi changes probably needed, and use snapshot
artemis-native to remove JBL dep that should never have been passed on
omit 4be34b586d fix some missed conversions, dep updates from previous
changes
omit d3bf8f985a jms-tests, joram tests
omit e1f95ff856 Rest conv
omit f0a44b5adf Jakarta RA pom
omit d5eb7530d9 RA changes
omit 964b1c1170 some progress... next on the list now is RA
omit 66e69c5433 tweak variables for simplicity, make debug env variable
align better with processor name/purpose
omit bbf9e6c2b7 add missing license header
omit 49c32f25d3 Placing getlogger back in place
omit b6e3e0e485 moving System.out loggers on LogProcessor
omit df25cc0352 removing system.out
omit 5dbaa2a6ac some javadoc fixup
omit 3f727bbb4a Some System.out.println logging on the log processor
omit ca242c9da1 removing some non used logging
omit c86b3c2c10 removing non used class
omit 5fbca4cc15 update artemis-cli
omit 1fe3a540c5 update artemis-cdi-client
omit 12ad8271c8 update artemis-mqtt-protocol
omit 5fcdcbf4ff restore some levels to original DEBUG, for now at least (as
they dont seem like INFOs)
omit bb5a9b098b fix some placeholder references not previously converted
omit 54288d45de fix some handling of out-of-order placeholder references
previously missed
omit 98b8913a23 restore original log level for a couple of entries in
previous change
omit b11abd2c20 converting jms-server
omit 4338fe0af1 fix placeholders missed in previous updates
omit ddde4e7bcc typo on comment
omit d774249df6 adding some known objects to LogProcessor, adding an
additional test
omit 05b4707e59 service-extensions conversion
omit 7bd4bfe0da openwire conversion
omit b628a27685 convert logging for artemis-jms-client
omit 22f09396c5 convert logging for artemis-stomp-protocol
omit a20a28011f convert logging for artemis-amqp-protocol
omit 87aa7fcb02 some fixups or notes, for prior changes or existing code
noticed reviewing them
omit 2920cc7dfe small tweak on log processor
omit 263dbe2b3d making progress: Artemis-server is compiling fine at this
point
omit f6f8214240 more progress
omit 6bf89f31c2 progress on server
omit 34b9084063 Making progress with artemis-server
omit 55eb7ed731 removing example created during prototype
omit c5ce66d0b3 small tweak on processor to use a more unique name
omit 07f3a5a17a Tweaks on log processor to support Exception, add DEBUG and
TRACE (temporarily)
omit 5726a27c54 restore a @Cause annotation to args that had it removed
previously, where method still exists
omit ebecf9822a small test tweak
omit 0ac71db124 Adding support for exception on LogProcessor
omit e819a5ee10 some more converstion on artemis-server
omit e5481d72a0 disable push builds on apache repo; still enabled for
forks, along with PR + manual builds
omit 37a738f709 make start on artemis-server, note some issues with these
and previous changes
omit bbb821b7b3 remove actioned TODO comment
omit 2bb0006a49 fix some arg/format issues with prior commit
omit 99772f0967 Addressing variable arguments on AuditLogger
omit 24bb838a7d Addressing TODOs: we should never have any wrong class
names, no more usage of LOGGER.error, always remove those
omit d56041b487 some TODOs around logger names etc
omit 915e0eeca4 update more of artemis-jdbc-store, but not all in order to
note specific issues
omit f61c43eaa7 fixing formatting with {} some real implementation on
Processor
omit 2f8af615b2 make start on artemis-jdbc-store to highlight some needed
conversions
omit d969ba854e add scopes, mostly test and some runtime for now, for SLF4J
Log4J binding dep
omit f38b24b23a use SLF4J -> Log4J 2 impl
omit 2042e95a8b fix AuditLogger line spacing
omit c6b664835c remove comment note already actioned
omit ba6d550667 fix junit licence note again
omit 1ab2743203 progress - artemis-journal is compiling
omit 83019db1af progress
omit dc46dd1be7 WIP: some notes
omit ca0c64a411 progress
omit 87b00081e8 Progress
omit 26423be44c Fixes
omit 75ff495081 progress
omit a23199ab72 progress
omit f66167eb37 moving some stuff
omit 1895474a05 removing jboss logging
omit c3ff24eba7 fail on duplicate
omit 3daa3dd603 removing logger level
omit 74905447db prototype Log Processor
add 196e604778 NO-JIRA: suppress significant teardown stacktraces after
compatibility test that is a no-op...make test itself skip
add 9dd026118e ARTEMIS-3862 Short lived subscriptiong makes address size
inconsistent
add c1fd16d66a ARTEMIS-3864 StompTransactions leaking on
ActiveMQServer.getSessions()
add fd345f0a03 ARTEMIS-3858 fix navigation from consumer/producer columns
of a session to those consumers/producers
add 78587f0a82 NO-JIRA: remove unused methods and related dependencies
left from initial copy plus some other cleanup
add 8d29742d40 ARTEMIS-3862 Adding Topic Durable Subscription to
RemoveSubscriptionRaceTest
add 56e9d9525d ARTEMIS-3864 Removing debug statement
add 6d926719f4 ARTEMIS-3863 - allow Role configuration via properties
add d199bf3c8c ARTEMIS-3870: mark -all client deps optional in
distribution pom, avoid passing on clashing/duplicate deps
add 2123de415b ARTEMIS-3873 AMQP Broker Conn Encrypted Attrs
add 2ab0e85db4 ARTEMIS-3874: move/update various bits to allow removing
many test-jar creations and dependencies
new 1d050d1d3b squashed logging changes from Clebert + me
new 41785fe2aa fixups for changes on main between
d0d2a9a16bb6664d1ebeb977f7213a9b8fa640da and
2ab0e85db48964fdb45b47b7c7da3ea504943c56
new c658de2cbb more fixups from chainges on main to
2ab0e85db48964fdb45b47b7c7da3ea504943c56, convert modules missed originally due
to the test interdependencies
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 (b8bab69059)
\
N -- N -- N refs/heads/new-logging (c658de2cbb)
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 3 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-cli/pom.xml | 20 +--
.../ActiveMQDefaultConfigurationTestAccessor.java | 0
artemis-commons/pom.xml | 18 +-
artemis-core-client/pom.xml | 20 +--
.../activemq/artemis/core/security/Role.java | 70 ++++++--
.../apache/activemq/artemis/util/XMLUtilTest.java | 2 +-
artemis-distribution/pom.xml | 27 +--
.../main/webapp/plugin/js/components/consumers.js | 4 +-
.../main/webapp/plugin/js/components/producers.js | 4 +-
artemis-jakarta-client/pom.xml | 7 -
artemis-jdbc-store/pom.xml | 13 +-
artemis-jms-client/pom.xml | 7 -
.../core/protocol/stomp/StompProtocolManager.java | 66 ++++---
.../artemis/core/protocol/stomp/StompSession.java | 14 ++
artemis-quorum-ri/pom.xml | 25 +--
artemis-rest/pom.xml | 14 +-
artemis-server/pom.xml | 20 +--
.../core/config/impl/ConfigurationImpl.java | 22 ++-
.../activemq/artemis/core/config/impl/RoleSet.java | 28 ++-
.../deployers/impl/FileConfigurationParser.java | 6 +
.../artemis/core/server/impl/QueueImpl.java | 46 +++++
.../resources/schema/artemis-configuration.xsd | 1 +
.../core/config/impl/ConfigurationImplTest.java | 78 ++++++++
...ConfigurationBrokerConnectionEncryptedTest.java | 86 +++++++++
.../artemis/core/filter/impl/FilterTest.java | 2 +-
.../core/server/impl/AdressInfoJSonTest.java | 2 +-
.../utils/SimpleServerTestPasswordCodec.java | 36 ++--
.../utils/critical/CriticalAnalyzerAccessor.java | 0
...tionTest-broker-connection-encrypted-config.xml | 37 ++++
artemis-server/src/test/resources/login.config | 2 +-
.../pom.xml | 16 +-
.../artemis/utils/CleanupSystemPropertiesRule.java | 2 +-
.../activemq/artemis/utils/PortCheckRule.java | 0
.../apache/activemq/artemis/utils/RetryMethod.java | 0
.../apache/activemq/artemis/utils/RetryRule.java | 0
.../activemq/artemis/utils}/SilentTestCase.java | 2 +-
.../artemis/utils/ThreadLeakCheckRule.java | 2 -
.../org/apache/activemq/artemis/utils/Wait.java | 44 ++++-
artemis-web/pom.xml | 13 +-
.../artemis/utils/SimpleWebTestPasswordCodec.java | 34 ++--
.../src/test/resources/bootstrap_web_codec.xml | 2 +-
.../test/resources/bootstrap_web_old_config.xml | 2 +-
pom.xml | 14 +-
tests/activemq5-unit-tests/pom.xml | 20 +--
tests/artemis-test-support/pom.xml | 60 ++++---
.../apache/activemq/artemis/tests/util/CFUtil.java | 0
.../apache/activemq/artemis/tests/util}/Jmx.java | 10 +-
.../apache/activemq/artemis/utils/ExecuteUtil.java | 0
.../activemq/artemis/utils/network/NetUtil.java | 0
.../artemis/utils/network/NetUtilResource.java | 0
.../activemq/transport/amqp/AmqpSupport.java | 85 ---------
.../transport/amqp/client/AmqpConnection.java | 3 +-
...lt.java => AmqpOperationTimedOutException.java} | 26 ++-
.../transport/amqp/client/AmqpReceiver.java | 3 +-
tests/compatibility-tests/pom.xml | 44 +----
.../impl/nullpm/NullStorageAccessor.java | 0
.../tests/compatibility/ExportImportTest.java | 17 +-
tests/e2e-tests/pom.xml | 55 ++----
.../e2e/brokerConnection/ChainedMirrorTest.java | 3 -
.../DualMirrorFailoverWithContainerTest.java | 5 +-
.../e2e/brokerConnection/QpidDispatchPeerTest.java | 9 +-
.../e2e/brokerConnection/SplitMirrorTest.java | 3 -
.../GCPauseSimulationTests.java | 2 +-
tests/extra-tests/pom.xml | 45 ++---
tests/integration-tests/pom.xml | 27 +--
.../client/RemoveSubscriptionRaceTest.java | 197 +++++++++++++++++++++
.../tests/integration/paging/PagingTest.java | 2 +-
.../artemis/tests/integration/stomp/StompTest.java | 33 ++++
.../activemq/artemis/tests/util/RandomUtil.java | 0
tests/jms-tests/pom.xml | 38 ++--
tests/joram-tests/pom.xml | 11 --
tests/performance-tests/pom.xml | 20 +--
tests/smoke-tests/pom.xml | 63 +++----
.../bridgeTransfer/BridgeTransferingTest.java | 34 +---
.../DualMirrorNoContainerTest.java | 3 -
.../brokerConnection/PagedMirrorSmokeTest.java | 5 +-
.../tests/smoke/dnsswitch/DNSSwitchTest.java | 5 +-
.../infinite/InfiniteRedeliverySmokeTest.java | 11 +-
.../artemis/tests/smoke/jms/ManifestTest.java | 2 -
.../ReplicatedMultipleFailbackTest.java | 27 +--
.../tests/smoke/logging/AuditLoggerTest.java | 3 +-
.../tests/smoke/logging/AuditLoggerTestBase.java | 19 --
.../tests/smoke/mmfactory/MMSFactoryTest.java | 48 ++---
.../smoke/paging/FloodServerWithAsyncSendTest.java | 11 +-
.../quorum/PluggableQuorumSinglePairTest.java | 63 +++----
.../quorum/ZookeeperPluggableQuorumPeerTest.java | 31 ++--
.../ZookeeperPluggableQuorumSinglePairTest.java | 7 +-
.../smoke/replicationflow/SoakPagingTest.java | 17 +-
.../artemis/tests/smoke/transfer/TransferTest.java | 22 +--
tests/soak-tests/pom.xml | 12 +-
tests/stress-tests/pom.xml | 26 +--
tests/timing-tests/pom.xml | 26 ++-
tests/unit-tests/pom.xml | 27 +--
.../artemis/tests/unit/AllClassesTest.java | 2 +-
.../remoting/impl/netty/NettyAcceptorTest.java | 2 +-
.../tests/unit/logging/AssertionLoggerTest.java | 2 +-
.../artemis/tests/unit/util/LinkedListTest.java | 2 +-
97 files changed, 1107 insertions(+), 889 deletions(-)
rename {artemis-core-client =>
artemis-cli}/src/test/java/org/apache/activemq/artemis/api/config/ActiveMQDefaultConfigurationTestAccessor.java
(100%)
copy
artemis-rest/src/main/java/org/apache/activemq/artemis/rest/queue/QueueDeployment.java
=>
artemis-server/src/main/java/org/apache/activemq/artemis/core/config/impl/RoleSet.java
(66%)
create mode 100644
artemis-server/src/test/java/org/apache/activemq/artemis/core/config/impl/FileConfigurationBrokerConnectionEncryptedTest.java
copy
artemis-core-client/src/main/java/org/apache/activemq/artemis/utils/SimpleIDGenerator.java
=>
artemis-server/src/test/java/org/apache/activemq/artemis/utils/SimpleServerTestPasswordCodec.java
(56%)
rename {artemis-commons =>
artemis-server}/src/test/java/org/apache/activemq/artemis/utils/critical/CriticalAnalyzerAccessor.java
(100%)
create mode 100644
artemis-server/src/test/resources/ConfigurationTest-broker-connection-encrypted-config.xml
copy {artemis-boot => artemis-unit-test-support}/pom.xml (83%)
rename {artemis-commons/src/test =>
artemis-unit-test-support/src/main}/java/org/apache/activemq/artemis/utils/CleanupSystemPropertiesRule.java
(98%)
rename {artemis-commons/src/test =>
artemis-unit-test-support/src/main}/java/org/apache/activemq/artemis/utils/PortCheckRule.java
(100%)
rename {artemis-commons/src/test =>
artemis-unit-test-support/src/main}/java/org/apache/activemq/artemis/utils/RetryMethod.java
(100%)
rename {artemis-commons/src/test =>
artemis-unit-test-support/src/main}/java/org/apache/activemq/artemis/utils/RetryRule.java
(100%)
rename
{artemis-core-client/src/test/java/org/apache/activemq/artemis/tests/util =>
artemis-unit-test-support/src/main/java/org/apache/activemq/artemis/utils}/SilentTestCase.java
(97%)
rename {artemis-commons/src/test =>
artemis-unit-test-support/src/main}/java/org/apache/activemq/artemis/utils/ThreadLeakCheckRule.java
(99%)
rename {artemis-commons/src/test =>
artemis-unit-test-support/src/main}/java/org/apache/activemq/artemis/utils/Wait.java
(82%)
copy
artemis-core-client/src/main/java/org/apache/activemq/artemis/utils/SimpleIDGenerator.java
=>
artemis-web/src/test/java/org/apache/activemq/artemis/utils/SimpleWebTestPasswordCodec.java
(56%)
rename tests/{integration-tests/src/test =>
artemis-test-support/src/main}/java/org/apache/activemq/artemis/tests/util/CFUtil.java
(100%)
rename
tests/{smoke-tests/src/test/java/org/apache/activemq/artemis/tests/smoke/utils
=>
artemis-test-support/src/main/java/org/apache/activemq/artemis/tests/util}/Jmx.java
(96%)
rename {artemis-commons/src/test =>
tests/artemis-test-support/src/main}/java/org/apache/activemq/artemis/utils/ExecuteUtil.java
(100%)
rename {artemis-commons/src/test =>
tests/artemis-test-support/src/main}/java/org/apache/activemq/artemis/utils/network/NetUtil.java
(100%)
rename {artemis-commons/src/test =>
tests/artemis-test-support/src/main}/java/org/apache/activemq/artemis/utils/network/NetUtilResource.java
(100%)
copy
tests/artemis-test-support/src/main/java/org/apache/activemq/transport/amqp/client/{util/NoOpAsyncResult.java
=> AmqpOperationTimedOutException.java} (61%)
rename {artemis-server =>
tests/compatibility-tests}/src/test/java/org/apache/activemq/artemis/core/persistence/impl/nullpm/NullStorageAccessor.java
(100%)
create mode 100644
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/client/RemoveSubscriptionRaceTest.java
rename {artemis-core-client =>
tests/integration-tests}/src/test/java/org/apache/activemq/artemis/tests/util/RandomUtil.java
(100%)
