Bug 1191592

Summary: drop spring-asm dependency
Product: [oVirt] ovirt-engine Reporter: Sandro Bonazzola <sbonazzo>
Component: GeneralAssignee: Martin Perina <mperina>
Status: CLOSED CURRENTRELEASE QA Contact: Pavol Brilla <pbrilla>
Severity: medium Docs Contact:
Priority: unspecified    
Version: ---CC: amureini, bugs, gklein, juan.hernandez, lsurette, lsvaty, mperina, oourfali, rbalakri, rgolan, srevivo, ykaul
Target Milestone: ovirt-4.0.0-alphaFlags: rule-engine: ovirt-4.0.0+
rule-engine: planning_ack+
oourfali: devel_ack+
pstehlik: testing_ack+
Target Release: 4.0.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: ovirt 4.0.0 alpha1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-07-26 11:00:32 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Infra RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1168605    

Description Sandro Bonazzola 2015-02-11 14:50:41 UTC
ovirt-engine is requiring spring-asm in ./backend/manager/dependencies/common/pom.xml

It was removed by upstream in version 3.2 M2 [1]:
* inlined ASM 4.0 into spring-core, removed spring-asm subproject and jar (SPR-9669)

We should
 - not to depend on spring-asm artifact (changelog entry indicates that similar functionality could be now available in spring-core)
 - update spring dependencies in your project to version 3.2+

[1]: http://docs.spring.io/spring/docs/3.2.x/changelog.txt

Comment 1 Red Hat Bugzilla Rules Engine 2015-10-19 10:54:10 UTC
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 2 Allon Mureinik 2016-02-02 08:56:37 UTC
Oved, IIUC, the patch in https://gerrit.ovirt.org/#/c/52850/ solves this issue

Comment 3 Martin Perina 2016-02-02 09:45:13 UTC
(In reply to Allon Mureinik from comment #2)
> Oved, IIUC, the patch in https://gerrit.ovirt.org/#/c/52850/ solves this
> issue

Yes, we need to deliver Spring Framework 4 inside oVirt packages as it's not packaged on any platform we use (even Fedora rawhide doesn't have Spring 4), so all Spring RPM dependencies are removed from our packages

Comment 4 Lukas Svaty 2016-07-26 11:00:32 UTC
This bug was fixed and is slated to be in the upcoming version. As we
are focusing our testing at this phase on severe bugs, this bug was
closed without going through its verification step. If you think this
bug should be verified by QE, please set its severity to high and move
it back to ON_QA