Description of problem: two permissions contain the same permit ID : <permit id="1104"><name>delete_disk</name><administrative>false</administrative></permit> <permit id="1104"><name>port_mirroring</name><administrative>true</administrative></permit> Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info:
oved/yair - maybe also add a unitest to check no duplicate permits in the enum? (i'd add this unitest to run on all enums actually!) this will require a release note, since we won't handle upgrade on this...
The problem is in the restapi side, and not backend side. There is a PermitType enum in the restapi-definition project, and the permit ID there for the port mirroring is wrong. Movig the bug to restapi component.
why would we maintain these numbers manually in the code?
leftover from 2.3, enum member's id should be inherited from the BE enum, and not being specified in the client side decorating enum, Oved, please remove id from the org.ovirt.engine.api.model.PermitType and inject it in the PermitMapper according to the ActionGroup, thanks.
Posted fix: http://gerrit.ovirt.org/#/c/8128/
Commit: c41d40f1483a861d5a007f0579c5d40fcf3c3ce0 http://gerrit.ovirt.org/gitweb?p=ovirt-engine.git;a=commit;h=c41d40f1483a861d5a007f0579c5d40fcf3c3ce0
<permit id="1104"><name>delete_disk</name><administrative>false</administrative></permit><permit id="1200"><name>port_mirroring</name><administrative>true</administrative></permit> The ID's are different now Verified si20