gemmellr commented on a change in pull request #3924:
URL: https://github.com/apache/activemq-artemis/pull/3924#discussion_r790774612
##########
File path: tests/integration-tests/src/test/resources/broker.properties
##########
@@ -15,4 +15,6 @@
# limitations under the License.
#
-name=ConfiguredViaProperties
\ No newline at end of file
+# use a very innocuous bit of config b/c it will be picked up by all embedded
servers in the integration tests
Review comment:
The original change makes it look like the run command knows about it
and checksthe default name, passing it on if it exists. The broker having and
loading a file for _all_ the tests, that it is _not_ loading by default
typically (as it doesnt exist) for normal users, means it isnt the normal path
to me. It means the tests dont take the normal path.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
