[jira] [Commented] (ARTEMIS-4531) Limit logfile output by default

2023-12-20 Thread Geert Schuring (Jira)


[ 
https://issues.apache.org/jira/browse/ARTEMIS-4531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17798893#comment-17798893
 ] 

Geert Schuring commented on ARTEMIS-4531:
-

I'm seeing strange behaviour duing some more tests and I'm no longer sure this 
works as intended. Lets put this on hold until I've done more tests.

> Limit logfile output by default
> ---
>
> Key: ARTEMIS-4531
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4531
> Project: ActiveMQ Artemis
>  Issue Type: Improvement
>  Components: Broker
>Affects Versions: 2.31.2
>Reporter: Geert Schuring
>Priority: Minor
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> The logging config does not limit the amount of rolled-over logfiles that are 
> stored, causing a minor resource leak. The system will (eventually) run out 
> of disk space without manual intervention.
> The relevant config in etc/log4j2.properties currently looks like this:
> {code:java}
> # Log file appender
> appender.log_file.type = RollingFile
> appender.log_file.name = log_file
> appender.log_file.fileName = ${sys:artemis.instance}/log/artemis.log
> appender.log_file.filePattern = 
> ${sys:artemis.instance}/log/artemis.log.%d{-MM-dd}
> appender.log_file.layout.type = PatternLayout
> appender.log_file.layout.pattern = %d %-5level [%logger] %msg%n
> appender.log_file.policies.type = Policies
> appender.log_file.policies.cron.type = CronTriggeringPolicy
> appender.log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.log_file.policies.cron.evaluateOnStartup = true
> # Audit log file appender
> appender.audit_log_file.type = RollingFile
> appender.audit_log_file.name = audit_log_file
> appender.audit_log_file.fileName = ${sys:artemis.instance}/log/audit.log
> appender.audit_log_file.filePattern = 
> ${sys:artemis.instance}/log/audit.log.%d{-MM-dd}
> appender.audit_log_file.layout.type = PatternLayout
> appender.audit_log_file.layout.pattern = %d [AUDIT](%t) %msg%n
> appender.audit_log_file.policies.type = Policies
> appender.audit_log_file.policies.cron.type = CronTriggeringPolicy
> appender.audit_log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.audit_log_file.policies.cron.evaluateOnStartup = true{code}
> This results in a new log file every day if there is any logging output. 
> These files are never deleted and keep accumulating.
> After setting a limit of 5 files it looks like this:
> {code:java}
> # Log file appender
> appender.log_file.type = RollingFile
> appender.log_file.name = log_file
> appender.log_file.fileName = ${sys:artemis.instance}/log/artemis.log
> appender.log_file.filePattern = 
> ${sys:artemis.instance}/log/artemis.log.%d{-MM-dd}
> appender.log_file.layout.type = PatternLayout
> appender.log_file.layout.pattern = %d %-5level [%logger] %msg%n
> appender.log_file.policies.type = Policies
> appender.log_file.policies.cron.type = CronTriggeringPolicy
> appender.log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.log_file.policies.cron.evaluateOnStartup = true
> appender.log_file.strategy.type = DefaultRolloverStrategy
> appender.log_file.strategy.max = 5
> # Audit log file appender
> appender.audit_log_file.type = RollingFile
> appender.audit_log_file.name = audit_log_file
> appender.audit_log_file.fileName = ${sys:artemis.instance}/log/audit.log
> appender.audit_log_file.filePattern = 
> ${sys:artemis.instance}/log/audit.log.%d{-MM-dd}
> appender.audit_log_file.layout.type = PatternLayout
> appender.audit_log_file.layout.pattern = %d [AUDIT](%t) %msg%n
> appender.audit_log_file.policies.type = Policies
> appender.audit_log_file.policies.cron.type = CronTriggeringPolicy
> appender.audit_log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.audit_log_file.policies.cron.evaluateOnStartup = true
> appender.audit_log_file.strategy.type = DefaultRolloverStrategy
> appender.audit_log_file.strategy.max = 5 {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (ARTEMIS-4545) Allow node ID to be configured

2023-12-20 Thread Gary Tully (Jira)


[ 
https://issues.apache.org/jira/browse/ARTEMIS-4545?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17798943#comment-17798943
 ] 

Gary Tully commented on ARTEMIS-4545:
-

the peer activation allows the node id to be set, it is named as  the 
correlation id but is actually the node id.

see: 
[https://github.com/apache/activemq-artemis/blob/main/tests/smoke-tests/src/main/resources/servers/zkReplicationPrimaryPeerA/broker.xml#L42]

not sure if that helps your use case, but a peer activation with a ha file 
lock, say on a shared volume with tight tolerance, could be a good approach.

at the moment, I don't think the file lock can be on a different volume from 
the journal, but if it could, we could have a mostly reliable file lock on a 
shared directory, with only the locking taking the sync and reliable read/lock 
hit.

 

> Allow node ID to be configured
> --
>
> Key: ARTEMIS-4545
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4545
> Project: ActiveMQ Artemis
>  Issue Type: New Feature
>Reporter: Justin Bertram
>Assignee: Justin Bertram
>Priority: Major
>
> In certain situations it would be beneficial to configure the node ID rather 
> than having it automatically generated. 
> For example, when using replication + failback if the primary server fails 
> the backup will take over. Then when the primary is restarted it will 
> initiate failback. However, if the primary broker's journal is damaged or 
> lost during the initial failure then it won't be able to initiate failback 
> because it won't have the same node ID as the backup. This kind of situation 
> is not uncommon in cloud environments where there is no persistent, attached 
> storage.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Work logged] (ARTEMIS-4531) Limit logfile output by default

2023-12-20 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/ARTEMIS-4531?focusedWorklogId=896538&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-896538
 ]

ASF GitHub Bot logged work on ARTEMIS-4531:
---

Author: ASF GitHub Bot
Created on: 20/Dec/23 15:20
Start Date: 20/Dec/23 15:20
Worklog Time Spent: 10m 
  Work Description: geertschuring commented on PR #4718:
URL: 
https://github.com/apache/activemq-artemis/pull/4718#issuecomment-1864656817

   @gemmellr, @jbertram After some more digging around it turns out the date in 
the filePattern (log/artemis.log.%d{-MM-dd}) keeps the 
DefaultRolloverStrategy from cleaning up old log files. To enable it to work 
the filePattern would need to be changed as well to something like 
(log/artemis.log.%i)
   I already changed my mind after reading your previous comments, but now I'm 
sure it would be better to delete this PR and close the Issue without 
implementing it. 




Issue Time Tracking
---

Worklog Id: (was: 896538)
Time Spent: 40m  (was: 0.5h)

> Limit logfile output by default
> ---
>
> Key: ARTEMIS-4531
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4531
> Project: ActiveMQ Artemis
>  Issue Type: Improvement
>  Components: Broker
>Affects Versions: 2.31.2
>Reporter: Geert Schuring
>Priority: Minor
>  Time Spent: 40m
>  Remaining Estimate: 0h
>
> The logging config does not limit the amount of rolled-over logfiles that are 
> stored, causing a minor resource leak. The system will (eventually) run out 
> of disk space without manual intervention.
> The relevant config in etc/log4j2.properties currently looks like this:
> {code:java}
> # Log file appender
> appender.log_file.type = RollingFile
> appender.log_file.name = log_file
> appender.log_file.fileName = ${sys:artemis.instance}/log/artemis.log
> appender.log_file.filePattern = 
> ${sys:artemis.instance}/log/artemis.log.%d{-MM-dd}
> appender.log_file.layout.type = PatternLayout
> appender.log_file.layout.pattern = %d %-5level [%logger] %msg%n
> appender.log_file.policies.type = Policies
> appender.log_file.policies.cron.type = CronTriggeringPolicy
> appender.log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.log_file.policies.cron.evaluateOnStartup = true
> # Audit log file appender
> appender.audit_log_file.type = RollingFile
> appender.audit_log_file.name = audit_log_file
> appender.audit_log_file.fileName = ${sys:artemis.instance}/log/audit.log
> appender.audit_log_file.filePattern = 
> ${sys:artemis.instance}/log/audit.log.%d{-MM-dd}
> appender.audit_log_file.layout.type = PatternLayout
> appender.audit_log_file.layout.pattern = %d [AUDIT](%t) %msg%n
> appender.audit_log_file.policies.type = Policies
> appender.audit_log_file.policies.cron.type = CronTriggeringPolicy
> appender.audit_log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.audit_log_file.policies.cron.evaluateOnStartup = true{code}
> This results in a new log file every day if there is any logging output. 
> These files are never deleted and keep accumulating.
> After setting a limit of 5 files it looks like this:
> {code:java}
> # Log file appender
> appender.log_file.type = RollingFile
> appender.log_file.name = log_file
> appender.log_file.fileName = ${sys:artemis.instance}/log/artemis.log
> appender.log_file.filePattern = 
> ${sys:artemis.instance}/log/artemis.log.%d{-MM-dd}
> appender.log_file.layout.type = PatternLayout
> appender.log_file.layout.pattern = %d %-5level [%logger] %msg%n
> appender.log_file.policies.type = Policies
> appender.log_file.policies.cron.type = CronTriggeringPolicy
> appender.log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.log_file.policies.cron.evaluateOnStartup = true
> appender.log_file.strategy.type = DefaultRolloverStrategy
> appender.log_file.strategy.max = 5
> # Audit log file appender
> appender.audit_log_file.type = RollingFile
> appender.audit_log_file.name = audit_log_file
> appender.audit_log_file.fileName = ${sys:artemis.instance}/log/audit.log
> appender.audit_log_file.filePattern = 
> ${sys:artemis.instance}/log/audit.log.%d{-MM-dd}
> appender.audit_log_file.layout.type = PatternLayout
> appender.audit_log_file.layout.pattern = %d [AUDIT](%t) %msg%n
> appender.audit_log_file.policies.type = Policies
> appender.audit_log_file.policies.cron.type = CronTriggeringPolicy
> appender.audit_log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.audit_log_file.policies.cron.evaluateOnStartup = true
> appender.audit_log_file.strategy.type = DefaultRolloverStrategy
> appender.audit_log_file.strategy.max = 5 {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (ARTEMIS-4531) Limit logfile output by default

2023-12-20 Thread Geert Schuring (Jira)


[ 
https://issues.apache.org/jira/browse/ARTEMIS-4531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17799063#comment-17799063
 ] 

Geert Schuring commented on ARTEMIS-4531:
-

TLDR: Doesn't work as intended. Lets delete the PR and close this issue.

I've commented on the PR: 
[https://github.com/apache/activemq-artemis/pull/4718#issuecomment-1864656817]

 

> Limit logfile output by default
> ---
>
> Key: ARTEMIS-4531
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4531
> Project: ActiveMQ Artemis
>  Issue Type: Improvement
>  Components: Broker
>Affects Versions: 2.31.2
>Reporter: Geert Schuring
>Priority: Minor
>  Time Spent: 40m
>  Remaining Estimate: 0h
>
> The logging config does not limit the amount of rolled-over logfiles that are 
> stored, causing a minor resource leak. The system will (eventually) run out 
> of disk space without manual intervention.
> The relevant config in etc/log4j2.properties currently looks like this:
> {code:java}
> # Log file appender
> appender.log_file.type = RollingFile
> appender.log_file.name = log_file
> appender.log_file.fileName = ${sys:artemis.instance}/log/artemis.log
> appender.log_file.filePattern = 
> ${sys:artemis.instance}/log/artemis.log.%d{-MM-dd}
> appender.log_file.layout.type = PatternLayout
> appender.log_file.layout.pattern = %d %-5level [%logger] %msg%n
> appender.log_file.policies.type = Policies
> appender.log_file.policies.cron.type = CronTriggeringPolicy
> appender.log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.log_file.policies.cron.evaluateOnStartup = true
> # Audit log file appender
> appender.audit_log_file.type = RollingFile
> appender.audit_log_file.name = audit_log_file
> appender.audit_log_file.fileName = ${sys:artemis.instance}/log/audit.log
> appender.audit_log_file.filePattern = 
> ${sys:artemis.instance}/log/audit.log.%d{-MM-dd}
> appender.audit_log_file.layout.type = PatternLayout
> appender.audit_log_file.layout.pattern = %d [AUDIT](%t) %msg%n
> appender.audit_log_file.policies.type = Policies
> appender.audit_log_file.policies.cron.type = CronTriggeringPolicy
> appender.audit_log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.audit_log_file.policies.cron.evaluateOnStartup = true{code}
> This results in a new log file every day if there is any logging output. 
> These files are never deleted and keep accumulating.
> After setting a limit of 5 files it looks like this:
> {code:java}
> # Log file appender
> appender.log_file.type = RollingFile
> appender.log_file.name = log_file
> appender.log_file.fileName = ${sys:artemis.instance}/log/artemis.log
> appender.log_file.filePattern = 
> ${sys:artemis.instance}/log/artemis.log.%d{-MM-dd}
> appender.log_file.layout.type = PatternLayout
> appender.log_file.layout.pattern = %d %-5level [%logger] %msg%n
> appender.log_file.policies.type = Policies
> appender.log_file.policies.cron.type = CronTriggeringPolicy
> appender.log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.log_file.policies.cron.evaluateOnStartup = true
> appender.log_file.strategy.type = DefaultRolloverStrategy
> appender.log_file.strategy.max = 5
> # Audit log file appender
> appender.audit_log_file.type = RollingFile
> appender.audit_log_file.name = audit_log_file
> appender.audit_log_file.fileName = ${sys:artemis.instance}/log/audit.log
> appender.audit_log_file.filePattern = 
> ${sys:artemis.instance}/log/audit.log.%d{-MM-dd}
> appender.audit_log_file.layout.type = PatternLayout
> appender.audit_log_file.layout.pattern = %d [AUDIT](%t) %msg%n
> appender.audit_log_file.policies.type = Policies
> appender.audit_log_file.policies.cron.type = CronTriggeringPolicy
> appender.audit_log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.audit_log_file.policies.cron.evaluateOnStartup = true
> appender.audit_log_file.strategy.type = DefaultRolloverStrategy
> appender.audit_log_file.strategy.max = 5 {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Comment Edited] (ARTEMIS-4531) Limit logfile output by default

2023-12-20 Thread Geert Schuring (Jira)


[ 
https://issues.apache.org/jira/browse/ARTEMIS-4531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17799063#comment-17799063
 ] 

Geert Schuring edited comment on ARTEMIS-4531 at 12/20/23 3:24 PM:
---

TLDR: Doesn't work as intended. Lets close the PR and close this issue.

I've commented on the PR: 
[https://github.com/apache/activemq-artemis/pull/4718#issuecomment-1864656817]

 


was (Author: cathodion):
TLDR: Doesn't work as intended. Lets delete the PR and close this issue.

I've commented on the PR: 
[https://github.com/apache/activemq-artemis/pull/4718#issuecomment-1864656817]

 

> Limit logfile output by default
> ---
>
> Key: ARTEMIS-4531
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4531
> Project: ActiveMQ Artemis
>  Issue Type: Improvement
>  Components: Broker
>Affects Versions: 2.31.2
>Reporter: Geert Schuring
>Priority: Minor
>  Time Spent: 40m
>  Remaining Estimate: 0h
>
> The logging config does not limit the amount of rolled-over logfiles that are 
> stored, causing a minor resource leak. The system will (eventually) run out 
> of disk space without manual intervention.
> The relevant config in etc/log4j2.properties currently looks like this:
> {code:java}
> # Log file appender
> appender.log_file.type = RollingFile
> appender.log_file.name = log_file
> appender.log_file.fileName = ${sys:artemis.instance}/log/artemis.log
> appender.log_file.filePattern = 
> ${sys:artemis.instance}/log/artemis.log.%d{-MM-dd}
> appender.log_file.layout.type = PatternLayout
> appender.log_file.layout.pattern = %d %-5level [%logger] %msg%n
> appender.log_file.policies.type = Policies
> appender.log_file.policies.cron.type = CronTriggeringPolicy
> appender.log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.log_file.policies.cron.evaluateOnStartup = true
> # Audit log file appender
> appender.audit_log_file.type = RollingFile
> appender.audit_log_file.name = audit_log_file
> appender.audit_log_file.fileName = ${sys:artemis.instance}/log/audit.log
> appender.audit_log_file.filePattern = 
> ${sys:artemis.instance}/log/audit.log.%d{-MM-dd}
> appender.audit_log_file.layout.type = PatternLayout
> appender.audit_log_file.layout.pattern = %d [AUDIT](%t) %msg%n
> appender.audit_log_file.policies.type = Policies
> appender.audit_log_file.policies.cron.type = CronTriggeringPolicy
> appender.audit_log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.audit_log_file.policies.cron.evaluateOnStartup = true{code}
> This results in a new log file every day if there is any logging output. 
> These files are never deleted and keep accumulating.
> After setting a limit of 5 files it looks like this:
> {code:java}
> # Log file appender
> appender.log_file.type = RollingFile
> appender.log_file.name = log_file
> appender.log_file.fileName = ${sys:artemis.instance}/log/artemis.log
> appender.log_file.filePattern = 
> ${sys:artemis.instance}/log/artemis.log.%d{-MM-dd}
> appender.log_file.layout.type = PatternLayout
> appender.log_file.layout.pattern = %d %-5level [%logger] %msg%n
> appender.log_file.policies.type = Policies
> appender.log_file.policies.cron.type = CronTriggeringPolicy
> appender.log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.log_file.policies.cron.evaluateOnStartup = true
> appender.log_file.strategy.type = DefaultRolloverStrategy
> appender.log_file.strategy.max = 5
> # Audit log file appender
> appender.audit_log_file.type = RollingFile
> appender.audit_log_file.name = audit_log_file
> appender.audit_log_file.fileName = ${sys:artemis.instance}/log/audit.log
> appender.audit_log_file.filePattern = 
> ${sys:artemis.instance}/log/audit.log.%d{-MM-dd}
> appender.audit_log_file.layout.type = PatternLayout
> appender.audit_log_file.layout.pattern = %d [AUDIT](%t) %msg%n
> appender.audit_log_file.policies.type = Policies
> appender.audit_log_file.policies.cron.type = CronTriggeringPolicy
> appender.audit_log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.audit_log_file.policies.cron.evaluateOnStartup = true
> appender.audit_log_file.strategy.type = DefaultRolloverStrategy
> appender.audit_log_file.strategy.max = 5 {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Work logged] (ARTEMIS-4531) Limit logfile output by default

2023-12-20 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/ARTEMIS-4531?focusedWorklogId=896539&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-896539
 ]

ASF GitHub Bot logged work on ARTEMIS-4531:
---

Author: ASF GitHub Bot
Created on: 20/Dec/23 15:26
Start Date: 20/Dec/23 15:26
Worklog Time Spent: 10m 
  Work Description: geertschuring closed pull request #4718: ARTEMIS-4531: 
Limit logfile output by default to 5
URL: https://github.com/apache/activemq-artemis/pull/4718




Issue Time Tracking
---

Worklog Id: (was: 896539)
Time Spent: 50m  (was: 40m)

> Limit logfile output by default
> ---
>
> Key: ARTEMIS-4531
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4531
> Project: ActiveMQ Artemis
>  Issue Type: Improvement
>  Components: Broker
>Affects Versions: 2.31.2
>Reporter: Geert Schuring
>Priority: Minor
>  Time Spent: 50m
>  Remaining Estimate: 0h
>
> The logging config does not limit the amount of rolled-over logfiles that are 
> stored, causing a minor resource leak. The system will (eventually) run out 
> of disk space without manual intervention.
> The relevant config in etc/log4j2.properties currently looks like this:
> {code:java}
> # Log file appender
> appender.log_file.type = RollingFile
> appender.log_file.name = log_file
> appender.log_file.fileName = ${sys:artemis.instance}/log/artemis.log
> appender.log_file.filePattern = 
> ${sys:artemis.instance}/log/artemis.log.%d{-MM-dd}
> appender.log_file.layout.type = PatternLayout
> appender.log_file.layout.pattern = %d %-5level [%logger] %msg%n
> appender.log_file.policies.type = Policies
> appender.log_file.policies.cron.type = CronTriggeringPolicy
> appender.log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.log_file.policies.cron.evaluateOnStartup = true
> # Audit log file appender
> appender.audit_log_file.type = RollingFile
> appender.audit_log_file.name = audit_log_file
> appender.audit_log_file.fileName = ${sys:artemis.instance}/log/audit.log
> appender.audit_log_file.filePattern = 
> ${sys:artemis.instance}/log/audit.log.%d{-MM-dd}
> appender.audit_log_file.layout.type = PatternLayout
> appender.audit_log_file.layout.pattern = %d [AUDIT](%t) %msg%n
> appender.audit_log_file.policies.type = Policies
> appender.audit_log_file.policies.cron.type = CronTriggeringPolicy
> appender.audit_log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.audit_log_file.policies.cron.evaluateOnStartup = true{code}
> This results in a new log file every day if there is any logging output. 
> These files are never deleted and keep accumulating.
> After setting a limit of 5 files it looks like this:
> {code:java}
> # Log file appender
> appender.log_file.type = RollingFile
> appender.log_file.name = log_file
> appender.log_file.fileName = ${sys:artemis.instance}/log/artemis.log
> appender.log_file.filePattern = 
> ${sys:artemis.instance}/log/artemis.log.%d{-MM-dd}
> appender.log_file.layout.type = PatternLayout
> appender.log_file.layout.pattern = %d %-5level [%logger] %msg%n
> appender.log_file.policies.type = Policies
> appender.log_file.policies.cron.type = CronTriggeringPolicy
> appender.log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.log_file.policies.cron.evaluateOnStartup = true
> appender.log_file.strategy.type = DefaultRolloverStrategy
> appender.log_file.strategy.max = 5
> # Audit log file appender
> appender.audit_log_file.type = RollingFile
> appender.audit_log_file.name = audit_log_file
> appender.audit_log_file.fileName = ${sys:artemis.instance}/log/audit.log
> appender.audit_log_file.filePattern = 
> ${sys:artemis.instance}/log/audit.log.%d{-MM-dd}
> appender.audit_log_file.layout.type = PatternLayout
> appender.audit_log_file.layout.pattern = %d [AUDIT](%t) %msg%n
> appender.audit_log_file.policies.type = Policies
> appender.audit_log_file.policies.cron.type = CronTriggeringPolicy
> appender.audit_log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.audit_log_file.policies.cron.evaluateOnStartup = true
> appender.audit_log_file.strategy.type = DefaultRolloverStrategy
> appender.audit_log_file.strategy.max = 5 {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Closed] (ARTEMIS-4531) Limit logfile output by default

2023-12-20 Thread Geert Schuring (Jira)


 [ 
https://issues.apache.org/jira/browse/ARTEMIS-4531?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Geert Schuring closed ARTEMIS-4531.
---
Resolution: Won't Fix

Suggested solution does not work as intended, and we don't want to introduce 
more changes to log file handling in a minor release.

> Limit logfile output by default
> ---
>
> Key: ARTEMIS-4531
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4531
> Project: ActiveMQ Artemis
>  Issue Type: Improvement
>  Components: Broker
>Affects Versions: 2.31.2
>Reporter: Geert Schuring
>Priority: Minor
>  Time Spent: 50m
>  Remaining Estimate: 0h
>
> The logging config does not limit the amount of rolled-over logfiles that are 
> stored, causing a minor resource leak. The system will (eventually) run out 
> of disk space without manual intervention.
> The relevant config in etc/log4j2.properties currently looks like this:
> {code:java}
> # Log file appender
> appender.log_file.type = RollingFile
> appender.log_file.name = log_file
> appender.log_file.fileName = ${sys:artemis.instance}/log/artemis.log
> appender.log_file.filePattern = 
> ${sys:artemis.instance}/log/artemis.log.%d{-MM-dd}
> appender.log_file.layout.type = PatternLayout
> appender.log_file.layout.pattern = %d %-5level [%logger] %msg%n
> appender.log_file.policies.type = Policies
> appender.log_file.policies.cron.type = CronTriggeringPolicy
> appender.log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.log_file.policies.cron.evaluateOnStartup = true
> # Audit log file appender
> appender.audit_log_file.type = RollingFile
> appender.audit_log_file.name = audit_log_file
> appender.audit_log_file.fileName = ${sys:artemis.instance}/log/audit.log
> appender.audit_log_file.filePattern = 
> ${sys:artemis.instance}/log/audit.log.%d{-MM-dd}
> appender.audit_log_file.layout.type = PatternLayout
> appender.audit_log_file.layout.pattern = %d [AUDIT](%t) %msg%n
> appender.audit_log_file.policies.type = Policies
> appender.audit_log_file.policies.cron.type = CronTriggeringPolicy
> appender.audit_log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.audit_log_file.policies.cron.evaluateOnStartup = true{code}
> This results in a new log file every day if there is any logging output. 
> These files are never deleted and keep accumulating.
> After setting a limit of 5 files it looks like this:
> {code:java}
> # Log file appender
> appender.log_file.type = RollingFile
> appender.log_file.name = log_file
> appender.log_file.fileName = ${sys:artemis.instance}/log/artemis.log
> appender.log_file.filePattern = 
> ${sys:artemis.instance}/log/artemis.log.%d{-MM-dd}
> appender.log_file.layout.type = PatternLayout
> appender.log_file.layout.pattern = %d %-5level [%logger] %msg%n
> appender.log_file.policies.type = Policies
> appender.log_file.policies.cron.type = CronTriggeringPolicy
> appender.log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.log_file.policies.cron.evaluateOnStartup = true
> appender.log_file.strategy.type = DefaultRolloverStrategy
> appender.log_file.strategy.max = 5
> # Audit log file appender
> appender.audit_log_file.type = RollingFile
> appender.audit_log_file.name = audit_log_file
> appender.audit_log_file.fileName = ${sys:artemis.instance}/log/audit.log
> appender.audit_log_file.filePattern = 
> ${sys:artemis.instance}/log/audit.log.%d{-MM-dd}
> appender.audit_log_file.layout.type = PatternLayout
> appender.audit_log_file.layout.pattern = %d [AUDIT](%t) %msg%n
> appender.audit_log_file.policies.type = Policies
> appender.audit_log_file.policies.cron.type = CronTriggeringPolicy
> appender.audit_log_file.policies.cron.schedule = 0 0 0 * * ?
> appender.audit_log_file.policies.cron.evaluateOnStartup = true
> appender.audit_log_file.strategy.type = DefaultRolloverStrategy
> appender.audit_log_file.strategy.max = 5 {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (ARTEMIS-4539) simplify various release process steps

2023-12-20 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/ARTEMIS-4539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17799068#comment-17799068
 ] 

ASF subversion and git services commented on ARTEMIS-4539:
--

Commit 28a917f4e747aaf2622a2585e1f50347f384122d in activemq-artemis's branch 
refs/heads/main from Robbie Gemmell
[ https://gitbox.apache.org/repos/asf?p=activemq-artemis.git;h=28a917f4e7 ]

ARTEMIS-4539: update website steps for adding release, use new aggregating 
script in website repo

- Initial script added in website repo commit 
https://github.com/apache/activemq-website/commit/b0f987e3c1308e483fc8a770d8c7b0b24ed36279


> simplify various release process steps
> --
>
> Key: ARTEMIS-4539
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4539
> Project: ActiveMQ Artemis
>  Issue Type: Task
>Reporter: Robbie Gemmell
>Assignee: Robbie Gemmell
>Priority: Major
> Fix For: 2.32.0
>
>
> There are currently a lot of manual steps during the release process that 
> involve running various commands from the RELEASING.md document, with various 
> c&p generally involved. They aren't difficult steps, but it does make the 
> process more cumbersome, fragile, and significantly slower than it need 
> actually be. We can script various aspects of the updates so just some simple 
> easily-remembered/discovered commands can be run to do most of the work, 
> simplifying the release process and making it much less of a chore to 
> actually do releases.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Resolved] (ARTEMIS-4539) simplify various release process steps

2023-12-20 Thread Robbie Gemmell (Jira)


 [ 
https://issues.apache.org/jira/browse/ARTEMIS-4539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robbie Gemmell resolved ARTEMIS-4539.
-
Resolution: Fixed

> simplify various release process steps
> --
>
> Key: ARTEMIS-4539
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4539
> Project: ActiveMQ Artemis
>  Issue Type: Task
>Reporter: Robbie Gemmell
>Assignee: Robbie Gemmell
>Priority: Major
> Fix For: 2.32.0
>
>
> There are currently a lot of manual steps during the release process that 
> involve running various commands from the RELEASING.md document, with various 
> c&p generally involved. They aren't difficult steps, but it does make the 
> process more cumbersome, fragile, and significantly slower than it need 
> actually be. We can script various aspects of the updates so just some simple 
> easily-remembered/discovered commands can be run to do most of the work, 
> simplifying the release process and making it much less of a chore to 
> actually do releases.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (ARTEMIS-4539) simplify various release process steps

2023-12-20 Thread Robbie Gemmell (Jira)


[ 
https://issues.apache.org/jira/browse/ARTEMIS-4539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17799073#comment-17799073
 ] 

Robbie Gemmell commented on ARTEMIS-4539:
-

The first 4 commits add specific scripts to do various component steps of the 
website updates (creating git report, release-notes, release collection file, 
and copying docs+javadocs from the artemis release build). These can all be 
called individually (git-report usually being created pre-vote), however the 
last commit above adds another script that calls all the others (skips 
git-report if found to be present already,as it should be) such that the bulk 
of the post-vote website updates can all now be done with that single script 
call.

> simplify various release process steps
> --
>
> Key: ARTEMIS-4539
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4539
> Project: ActiveMQ Artemis
>  Issue Type: Task
>Reporter: Robbie Gemmell
>Assignee: Robbie Gemmell
>Priority: Major
> Fix For: 2.32.0
>
>
> There are currently a lot of manual steps during the release process that 
> involve running various commands from the RELEASING.md document, with various 
> c&p generally involved. They aren't difficult steps, but it does make the 
> process more cumbersome, fragile, and significantly slower than it need 
> actually be. We can script various aspects of the updates so just some simple 
> easily-remembered/discovered commands can be run to do most of the work, 
> simplifying the release process and making it much less of a chore to 
> actually do releases.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (ARTEMIS-4546) Modernize Validators

2023-12-20 Thread Justin Bertram (Jira)
Justin Bertram created ARTEMIS-4546:
---

 Summary: Modernize Validators
 Key: ARTEMIS-4546
 URL: https://issues.apache.org/jira/browse/ARTEMIS-4546
 Project: ActiveMQ Artemis
  Issue Type: Task
Reporter: Justin Bertram
Assignee: Justin Bertram


Modernize {{org.apache.activemq.artemis.core.config.impl.Validators}} to use 
lambdas and update callers.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Work logged] (ARTEMIS-4546) Modernize Validators

2023-12-20 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/ARTEMIS-4546?focusedWorklogId=896591&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-896591
 ]

ASF GitHub Bot logged work on ARTEMIS-4546:
---

Author: ASF GitHub Bot
Created on: 20/Dec/23 22:41
Start Date: 20/Dec/23 22:41
Worklog Time Spent: 10m 
  Work Description: jbertram opened a new pull request, #4722:
URL: https://github.com/apache/activemq-artemis/pull/4722

   No semantic changes here. Just some code gardening.




Issue Time Tracking
---

Worklog Id: (was: 896591)
Remaining Estimate: 0h
Time Spent: 10m

> Modernize Validators
> 
>
> Key: ARTEMIS-4546
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4546
> Project: ActiveMQ Artemis
>  Issue Type: Task
>Reporter: Justin Bertram
>Assignee: Justin Bertram
>Priority: Major
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Modernize {{org.apache.activemq.artemis.core.config.impl.Validators}} to use 
> lambdas and update callers.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)