Lior Vernia has posted comments on this change. Change subject: core: Mark management network as display and migration ......................................................................
Patch Set 3: Ugh, you're right. It's no problem to mark the management network as display and migration in new clusters, but the REST thing is an issue. Suggestions? It is weird that when introducing the display/migration network features we preferred to leave fallback code around and allow no network to be marked as display/migration, rather than just not make it possible to not have any network defined as such. It's of course solvable in the GUI, but in my opinion it would be more correct to fix it at the engine core. -- To view, visit http://gerrit.ovirt.org/22382 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I73bf3c583ae28bf32d7c602ec9f4b0fb5c7f391d Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Lior Vernia <[email protected]> Gerrit-Reviewer: Mike Kolesnik <[email protected]> Gerrit-Reviewer: Moti Asayag <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
