Mike Kolesnik has posted comments on this change.
Change subject: engine: Add support for vNICs in OpenstackNetwork
......................................................................
Patch Set 17: (3 inline comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/network/openstack/OpenstackNetworkProviderProxy.java
Line 92:
Line 93: Port createdPort = getQuantumClient().execute(new
CreatePort(port));
Line 94: return createdPort.getId();
Line 95: } catch (RuntimeException e) {
Line 96: throw new VdcBLLException(VdcBllErrors.PROVIDER_FAILURE,
new RuntimeException(e));
Done
Line 97: }
Line 98: }
Line 99:
Line 100: @Override
Line 106: getQuantumClient().execute(new
DeletePort(port.getId()));
Line 107: }
Line 108: }
Line 109: } catch (RuntimeException e) {
Line 110: throw new VdcBLLException(VdcBllErrors.PROVIDER_FAILURE,
new RuntimeException(e));
Done
Line 111: }
Line 112: }
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/network/openstack/PortForCreate.java
Line 3: import org.codehaus.jackson.annotate.JsonProperty;
Line 4: import org.codehaus.jackson.map.annotate.JsonRootName;
Line 5:
Line 6: @JsonRootName("port")
Line 7: public class PortForCreate extends
org.openstack.quantum.model.PortForCreate {
I will try to push this to SDK upstream, but meanwhile it doesn't hurt us to
have it here.
Line 8:
Line 9: @JsonProperty("mac_address")
Line 10: private String macAddress;
Line 11:
--
To view, visit http://gerrit.ovirt.org/11110
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I465dee81c3348a3f49ff41a8983a8ba2c8d43fad
Gerrit-PatchSet: 17
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Mike Kolesnik <[email protected]>
Gerrit-Reviewer: Alona Kaplan <[email protected]>
Gerrit-Reviewer: Lior Vernia <[email protected]>
Gerrit-Reviewer: Livnat Peer <[email protected]>
Gerrit-Reviewer: Mike Kolesnik <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
