Bug 1033119 - Incorrect bundle destination is chosen in deploy-to-and-restart-JBAS.js sample file
Summary: Incorrect bundle destination is chosen in deploy-to-and-restart-JBAS.js samp...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Operations Network
Classification: JBoss
Component: CLI
Version: JON 3.2
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: CR01
: JON 3.2.0
Assignee: RHQ Project Maintainer
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-11-21 15:19 UTC by Filip Brychta
Modified: 2014-01-02 20:34 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-01-02 20:34:47 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Filip Brychta 2013-11-21 15:19:08 UTC
Description of problem:
Here is the problem:
<code>
    var destinationId = jbasDestination;
    if (typeof(jbasDestination) == 'object') {
        destinationId = jbasDestination.id;
    }

    var destCrit = new BundleDestinationCriteria
    destCrit.fetchGroup(true)

    var destinations = BundleManager.findBundleDestinationsByCriteria(destCrit);
</code>

Used destCrit doesn't have FilterId set. So the findBundleDestinationsByCriteria function then finds first destination which could be incorrect.

Correct version:
<code>
    var destinationId = jbasDestination;
    if (typeof(jbasDestination) == 'object') {
        destinationId = jbasDestination.id;
    }

    var destCrit = new BundleDestinationCriteria
    destCrit.fetchGroup(true)
    destCrit.addFilterId(destinationId);

    var destinations = BundleManager.findBundleDestinationsByCriteria(destCrit);
</code>

Version-Release number of selected component (if applicable):
Jon3.2.ER5 and master as well:
Version :	
4.10.0-SNAPSHOT
Build Number :	
cb5f098

Comment 1 Filip Brychta 2013-11-21 15:46:39 UTC
The same problem is in jbossas module

Comment 2 Lukas Krejci 2013-11-21 20:49:40 UTC
Note that this was fixed POST ER7.

commit 3217afa463913656a1e0af3b58b27a782f55d472
Author: Lukas Krejci <lkrejci>
Date:   Thu Nov 21 21:41:06 2013 +0100

    [BZ 1033119] Find the asked for destination, not just a random one in
    deploy-to-and-restrart-JBAS.js#updateAppAndRestartJBAS(),
    require("modules:/jbossas").updateAppAndRestart()

Comment 3 Simeon Pinder 2013-12-03 23:19:37 UTC
Moving to ON_QA for testing in latest(CR1) brew build.

Comment 4 Filip Brychta 2013-12-04 08:55:27 UTC
Verified on:
Version :	
3.2.0.CR1
Build Number :	
6ecd678:d0dc0b6


Note You need to log in before you can comment on or make changes to this bug.