[#RHQ-664] Applying alert template to existing inventory blows up

RHQ Project

Applying alert template to existing inventory blows up

Created: 11/Jul/08 10:41 AM Updated: Friday 02:45 PM

Component/s:

None

Affects Version/s:

1.1pre

Fix Version/s:

1.1

Time Tracking:

Not Specified

Environment:

I'm not sure this is an issue in 1.0.1, but it is in 1.1.0 r-1086

Resolution Date:	16/Jul/08 05:33 PM
Date of First Response:	16/Jul/08 05:33 PM
Tester:	Jeff Weiss
VCS Revision:	1,086

Description

« Hide

In some cases applying a new template to existing inventory blows up with "no transaction" type problems.

To Recreate:

1) Import a postgres server into inventory
2) Create an alert template for Postgres Server -> Database Service and check the apply to existing inventory option
3) Click OK

Stack Trace with root cause:

... Transaction is not active: tx=TransactionImple < ac, BasicAction: a0b0d52:effd:487772cf:ad3 status: ActionStatus.ABORT_ONLY >

All

Comments

Work Log

Change History

Sort Order:

Ascending order - Click to sort in descending order

[ Permlink | « Hide ]

Jay Shaughnessy - 11/Jul/08 10:43 AM

I ran into this while executing test steps for ~~RHQ-573~~ so I'll take it as part of that work, although it's independent of the changes for that fix as it was reproduced in trunk.

[ Permlink | « Hide ]

Jay Shaughnessy - 15/Jul/08 03:42 PM

This is due to the fact that at the moment the resource merge is done in layers but the schedule and template synch is done top down as soon as that first layer merge completes (after the initial blocking of performServiceScan in syncInventoryStatusRecursively). In other words, we're invoking sync for the children of the added roots prior to them being merged into inventory. It needs to happen after child services are discovered and merged.

This requires work in InventoryManager, which is in the midst of a refactor.

[ Permlink | « Hide ]

Jay Shaughnessy - 15/Jul/08 03:43 PM

Ian, asigning this to you as this fix is part of your work in InventoryManager. You can assign back to me for verification.

[ Permlink | « Hide ]

Ian Springer - 16/Jul/08 05:33 PM

Schedules are now synced for auto-imported Resources after a runtime report is merged and PC inventory is synced w/ Server inventory - r1106.

[ Permlink | « Hide ]

Jeff Weiss - 23/Jul/08 02:53 PM

Works on 6152 qa build, linux.