Description of problem: Patches to EAP version 6.1.0 or less are not prevented, and thus fail due to patching not being introduced into EAP until version 6.2.0 Version-Release number of selected component (if applicable): JON 3.3 How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Patch deploy fails when applying to EAP 6.1.0 or less. Expected results: Patches deploys should be prevented when EAP version is <= 6.1.0 Additional info:
I don't exactly know how we should prevent the deployment to older versions of EAP. There is no mechanism on the server side to discriminate members of compatible groups based on further criteria. (I.e. I have a group of EAP servers, but I cannot say that only 2 of 3 members can be deployed to because they are of supported versions). On the agent side, the only thing we can do is to try the deployment and maybe fail with a nicer error message than the one coming out of EAP itself. I'd say that adding such feature is out of scope for 3.3 and I am not sure it is worth adding such capability to RHQ/JON at all at this time until we figure out the scope/functionality of provisioning in JON.Next.
To be more precise: * I am against adding a server-side facility to discriminate between group members based on further criteria when determining the targets of deployment (at least in JON 3.3 payload) * We can surely provide a nicer error message coming from the agent when trying to deploy to an unsupported version of EAP. For the record, the failure message coming from EAP < 6.2.0, when attempting the patch deployment, has the following form: "Unexpected command 'patch apply --path=...'. Type 'help --commands' for the list of supported commands." This is visible in JON's UI when examining the status of the failed deployment.
Fix proposed: https://github.com/rhq-project/rhq/pull/121
Merged in release/jon3.3.x commit 4cedda3a24228c93f0621e3c0aa8dd6043043e32 Author: Thomas Segismont <tsegismo> Date: Wed Sep 10 16:57:51 2014 +0200 BZs 1136996 1135034 1135107 - Check if patching is supported before trying to perform it on EAP based resource From patch file created with "git diff a3a59ff 5c39e3a"
Moving to ON_QA as available for test with the following brew build: https://brewweb.devel.redhat.com//buildinfo?buildID=385149