Support Migration Notice: To update migrated JIRA cases click here to open a new case use www.vmware.com/go/sr | vFabric Hyperic 5.7.0 is Now Available

Hyperic HQ

RuntimeReportFilter should move to hq-pdk.jar

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 4.5.1
  • Fix Version/s: 4.6
  • Component/s: Inventory
  • Case Links:
    none
  • Regression:
    Yes
  • Tags:

Description

Hyperic HQ 4.4 contains RuntimeReportFilter.class in both lib/handlers/AgentAutoinventoryServer.jar and pdk/lib/hq-product.jar. However, in 4.5, it's missing from hq-product.jar. This causes a classloader error when loading a new jar containing an implementation of this interface.

Activity

Hide
Greg Turnquist added a comment -

On Feb 10, 2011, at 6:35 PM, Jennifer Hickey wrote:

It actually needs to be in hq-pdk (which is a replacement for hq-product. hq-product.jar is only there to support a legacy backwards compat case and is pretty much empty). Since maven is pretty conventional, all you will need to do is move the class into the hq-pdk project. We don't want to do anything strange like hacking maven to put the class into 2 separate jar files since, if I understand correctly, hq-pdk is visible in the parent and will be accessible by the handler jar anyway.

Show
Greg Turnquist added a comment - On Feb 10, 2011, at 6:35 PM, Jennifer Hickey wrote: It actually needs to be in hq-pdk (which is a replacement for hq-product. hq-product.jar is only there to support a legacy backwards compat case and is pretty much empty). Since maven is pretty conventional, all you will need to do is move the class into the hq-pdk project. We don't want to do anything strange like hacking maven to put the class into 2 separate jar files since, if I understand correctly, hq-pdk is visible in the parent and will be accessible by the handler jar anyway.
Hide
Greg Turnquist added a comment -

I tried this on master and it worked.

Show
Greg Turnquist added a comment - I tried this on master and it worked.
Hide
Greg Turnquist added a comment -

Changes have been committed to master.

Show
Greg Turnquist added a comment - Changes have been committed to master.
Hide
Yoav Epelman added a comment -

Bulk change to new components

Show
Yoav Epelman added a comment - Bulk change to new components

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:
    Last comment:
    1 year, 9 weeks, 3 days ago