Cargo

Set properties for JBoss configuration changes instead of using cargo-specific config files

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.0.2
  • Fix Version/s: 1.0.3
  • Component/s: JBoss
  • Labels:
    None
  • Environment:
    n/a
  • Complexity:
    Intermediate
  • Number of attachments :
    0

Description

While looking into using Cargo for setting up dev environments of JBoss, I see that Cargo has some altered config files that are used instead of the default JBoss ones (for a standalone configuration). This is because we use token replacements. One problem that I now run into is that these config files are slightly different from the Jboss default ones. As JBoss supports system property resolution for most configuration I'm wondering if we cannot use that?

One example is the log4j config file where we use token replacement to set the logging level. The JBoss way of doing that would be to set the jboss.server.log.threshold property instead when starting the container.

Issue Links

is related to

Bug - A problem which impairs or prevents the functions of the product. CARGO-585 Maven/Cargo/JBoss problem with verbose, unconfigurable logging

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Bug - A problem which impairs or prevents the functions of the product. CARGO-824 Incorrect log4j logging levels are used in JBoss

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Improvement - An improvement or enhancement to an existing feature or task. CARGO-839 Avoid having the cargo-specific XML files in the JBoss local containers

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Anders Hammar added a comment -

Committed rev 2440 on trunk, which replaces the cargo specific log4j file with standard JBoss system property for logging threshold.

Show
Anders Hammar added a comment - Committed rev 2440 on trunk, which replaces the cargo specific log4j file with standard JBoss system property for logging threshold.
Hide
Permalink
Savas Ali Tokmen added a comment -

Any other tokens you plan to add, Anders; or is this one finished?

Show
Savas Ali Tokmen added a comment - Any other tokens you plan to add, Anders; or is this one finished?
Hide
Permalink
Anders Hammar added a comment -

I was hoping on being able to get ride of all the cargo specific config files (for JBoss). However, the two remaining seems to be harder to solve. It might be possible, but would require me to really dig into the details regarding JBoss configs (all the mbeans etc).
I don't think I have the time for that right now. The log4j file was my primary concern. I'll keep this ticket open for one more week if I find the time to dig in. Otherwise I'll close it.

Show
Anders Hammar added a comment - I was hoping on being able to get ride of all the cargo specific config files (for JBoss). However, the two remaining seems to be harder to solve. It might be possible, but would require me to really dig into the details regarding JBoss configs (all the mbeans etc). I don't think I have the time for that right now. The log4j file was my primary concern. I'll keep this ticket open for one more week if I find the time to dig in. Otherwise I'll close it.
Hide
Permalink
Anders Hammar added a comment -

Closing as I can't find time right now to dig deeper regarding the mbeans to get rid of all the other cargo specific config files. If anyone knows how to do this, please create a new ticket with a patch.

Show
Anders Hammar added a comment - Closing as I can't find time right now to dig deeper regarding the mbeans to get rid of all the other cargo specific config files. If anyone knows how to do this, please create a new ticket with a patch.

People

  • Assignee:
    Anders Hammar
    Reporter:
    Anders Hammar
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: