From yzaspits at redhat.com Sun Aug 17 06:44:01 2014 Content-Type: multipart/mixed; boundary="===============5604952005813401158==" MIME-Version: 1.0 From: Yevgeny Zaspitsky To: devel at ovirt.org Subject: Re: [ovirt-devel] Management network as a role - design proposal Date: Sun, 17 Aug 2014 13:43:59 +0300 Message-ID: <53F0876F.6030801@redhat.com> In-Reply-To: 20140815095523.GA6805@redhat.com --===============5604952005813401158== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable This is a multi-part message in MIME format. --------------050009080408030600070300 Content-Type: text/plain; charset=3DISO-8859-1; format=3Dflowed Content-Transfer-Encoding: 7bit On 15/08/14 12:55, Dan Kenigsberg wrote: > On Thu, Aug 14, 2014 at 11:52:41AM -0400, Yevgeny Zaspitsky wrote: >> Hi All, >> >> The proposed feature will allow defining an arbitrary network in the DC = as the management network for the cluster, which in its turn will allow ass= igning different VLANs for the management networks in the same DC. >> >> Feature page can be found here - http://www.ovirt.org/Features/Managemen= t_Network_As_A_Role . >> >> Please take a look into the page especially into "Open issues" section. = I'd like to have your opinions on that. > May I ask why you change the default management network from ovirtmgmt > to "Management"? (And why the upercase M?) We'd like to get rid of that difference between oVirt and REVM. IMHO = there's no good reason for having product name in the network/bridge name. If you do not like capital letters in the network name I'm OK with = changing it to the lower one. > > Regarding your open question: "Creating new cluster would have to > receive the new parameter (management network)" This new parameter > should be kept optional, with a default value of ovirtmgmt. This way, a > user that is unaware of the new feature, would see no change in > functionality. Using a specific network name seems isn't possible, as that network = might be not existent at the time of issuing the command. Doing so could reduce the number cases where backward compatibility is = broken, but can not eliminate it totally. In those broken cases we might = return an error to a RESTful API user. > The feature page should discuss the possibility of chaning the > management role. Is it supported after there are hosts in the cluster? > If we allow that, there's a bit of complexity. The management network's > gateway is used as the default route of each host. If you change the > role, all hosts should be notified (with a new setupNetwork command). > > I think that the cleanest solution would be to allow editing, but report > the hosts as out-of-sync. This approach requires a Vdsm-side change - it > would need to report which of its network is the default route. Thank you for turning my attention to this scenario, I'll update the = wiki page. > > Dan. --------------050009080408030600070300 Content-Type: text/html; charset=3DISO-8859-1 Content-Transfer-Encoding: 7bit
On 15/08/14 12:55, Dan Kenigsberg wrote:
On Thu, Aug 14, 2014 at 11:52:41AM -0400, Yevgeny Zasp=
itsky wrote:

Hi All, =


The proposed feature will allow defining an arbitrary network in the DC as =
the management network for the cluster, which in its turn will allow assign=
ing different VLANs for the management networks in the same DC. =


Feature page can be found here - http://www.ovi=
rt.org/Features/Management_Network_As_A_Role . =


Please take a look into the page especially into "Open issues" section. I'd=
 like to have your opinions on that. =


May I ask why you change the default management network from ovirtmgmt
to "Management"? (And why the upercase M?)
We'd like to get rid of that difference between oVirt and REVM. IMHO there's no good reason for having product name in the network/bridge name.
If you do not like capital letters in the network name I'm OK with changing it to the lower one.

Regarding your open question: "Creating new cluster would have to
receive the new parameter (management network)" This new parameter
should be kept optional, with a default value of ovirtmgmt. This way, a
user that is unaware of the new feature, would see no change in
functionality.
Using a specific network name seems isn't possible, as that network might be not existent at the time of issuing the command.
Doing so could reduce the number cases where backward compatibility is broken, but can not eliminate it totally. In those broken cases we might return an error to a RESTful API user.
The feature page should discuss the possibility of chaning the
management role. Is it supported after there are hosts in the cluster?
If we allow that, there's a bit of complexity. The management network's
gateway is used as the default route of each host. If you change the
role, all hosts should be notified (with a new setupNetwork command).

I think that the cleanest solution would be to allow editing, but report
the hosts as out-of-sync. This approach requires a Vdsm-side change - it
would need to report which of its network is the default route.
Thank you for turning my attention to this scenario, I'll update the wiki page.

Dan.

--------------050009080408030600070300-- --===============5604952005813401158== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" VGhpcyBpcyBhIG11bHRpLXBhcnQgbWVzc2FnZSBpbiBNSU1FIGZvcm1hdC4KLS0tLS0tLS0tLS0t LS0wNTAwMDkwODA0MDgwMzA2MDAwNzAzMDAKQ29udGVudC1UeXBlOiB0ZXh0L3BsYWluOyBjaGFy c2V0PUlTTy04ODU5LTE7IGZvcm1hdD1mbG93ZWQKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzog N2JpdAoKCk9uIDE1LzA4LzE0IDEyOjU1LCBEYW4gS2VuaWdzYmVyZyB3cm90ZToKPiBPbiBUaHUs IEF1ZyAxNCwgMjAxNCBhdCAxMTo1Mjo0MUFNIC0wNDAwLCBZZXZnZW55IFphc3BpdHNreSB3cm90 ZToKPj4gSGkgQWxsLAo+Pgo+PiBUaGUgcHJvcG9zZWQgZmVhdHVyZSB3aWxsIGFsbG93IGRlZmlu aW5nIGFuIGFyYml0cmFyeSBuZXR3b3JrIGluIHRoZSBEQyBhcyB0aGUgbWFuYWdlbWVudCBuZXR3 b3JrIGZvciB0aGUgY2x1c3Rlciwgd2hpY2ggaW4gaXRzIHR1cm4gd2lsbCBhbGxvdyBhc3NpZ25p bmcgZGlmZmVyZW50IFZMQU5zIGZvciB0aGUgbWFuYWdlbWVudCBuZXR3b3JrcyBpbiB0aGUgc2Ft ZSBEQy4KPj4KPj4gRmVhdHVyZSBwYWdlIGNhbiBiZSBmb3VuZCBoZXJlIC0gaHR0cDovL3d3dy5v dmlydC5vcmcvRmVhdHVyZXMvTWFuYWdlbWVudF9OZXR3b3JrX0FzX0FfUm9sZSAuCj4+Cj4+IFBs ZWFzZSB0YWtlIGEgbG9vayBpbnRvIHRoZSBwYWdlIGVzcGVjaWFsbHkgaW50byAiT3BlbiBpc3N1 ZXMiIHNlY3Rpb24uIEknZCBsaWtlIHRvIGhhdmUgeW91ciBvcGluaW9ucyBvbiB0aGF0Lgo+IE1h eSBJIGFzayB3aHkgeW91IGNoYW5nZSB0aGUgZGVmYXVsdCBtYW5hZ2VtZW50IG5ldHdvcmsgZnJv bSBvdmlydG1nbXQKPiB0byAiTWFuYWdlbWVudCI/IChBbmQgd2h5IHRoZSB1cGVyY2FzZSBNPykK V2UnZCBsaWtlIHRvIGdldCByaWQgb2YgdGhhdCBkaWZmZXJlbmNlIGJldHdlZW4gb1ZpcnQgYW5k IFJFVk0uIElNSE8gCnRoZXJlJ3Mgbm8gZ29vZCByZWFzb24gZm9yIGhhdmluZyBwcm9kdWN0IG5h bWUgaW4gdGhlIG5ldHdvcmsvYnJpZGdlIG5hbWUuCklmIHlvdSBkbyBub3QgbGlrZSBjYXBpdGFs IGxldHRlcnMgaW4gdGhlIG5ldHdvcmsgbmFtZSBJJ20gT0sgd2l0aCAKY2hhbmdpbmcgaXQgdG8g dGhlIGxvd2VyIG9uZS4KPgo+IFJlZ2FyZGluZyB5b3VyIG9wZW4gcXVlc3Rpb246ICJDcmVhdGlu ZyBuZXcgY2x1c3RlciB3b3VsZCBoYXZlIHRvCj4gcmVjZWl2ZSB0aGUgbmV3IHBhcmFtZXRlciAo bWFuYWdlbWVudCBuZXR3b3JrKSIgVGhpcyBuZXcgcGFyYW1ldGVyCj4gc2hvdWxkIGJlIGtlcHQg b3B0aW9uYWwsIHdpdGggYSBkZWZhdWx0IHZhbHVlIG9mIG92aXJ0bWdtdC4gVGhpcyB3YXksIGEK PiB1c2VyIHRoYXQgaXMgdW5hd2FyZSBvZiB0aGUgbmV3IGZlYXR1cmUsIHdvdWxkIHNlZSBubyBj aGFuZ2UgaW4KPiBmdW5jdGlvbmFsaXR5LgpVc2luZyBhIHNwZWNpZmljIG5ldHdvcmsgbmFtZSBz ZWVtcyBpc24ndCBwb3NzaWJsZSwgYXMgdGhhdCBuZXR3b3JrIAptaWdodCBiZSBub3QgZXhpc3Rl bnQgYXQgdGhlIHRpbWUgb2YgaXNzdWluZyB0aGUgY29tbWFuZC4KRG9pbmcgc28gY291bGQgcmVk dWNlIHRoZSBudW1iZXIgY2FzZXMgd2hlcmUgYmFja3dhcmQgY29tcGF0aWJpbGl0eSBpcyAKYnJv a2VuLCBidXQgY2FuIG5vdCBlbGltaW5hdGUgaXQgdG90YWxseS4gSW4gdGhvc2UgYnJva2VuIGNh c2VzIHdlIG1pZ2h0IApyZXR1cm4gYW4gZXJyb3IgdG8gYSBSRVNUZnVsIEFQSSB1c2VyLgo+IFRo ZSBmZWF0dXJlIHBhZ2Ugc2hvdWxkIGRpc2N1c3MgdGhlIHBvc3NpYmlsaXR5IG9mIGNoYW5pbmcg dGhlCj4gbWFuYWdlbWVudCByb2xlLiBJcyBpdCBzdXBwb3J0ZWQgYWZ0ZXIgdGhlcmUgYXJlIGhv c3RzIGluIHRoZSBjbHVzdGVyPwo+IElmIHdlIGFsbG93IHRoYXQsIHRoZXJlJ3MgYSBiaXQgb2Yg Y29tcGxleGl0eS4gVGhlIG1hbmFnZW1lbnQgbmV0d29yaydzCj4gZ2F0ZXdheSBpcyB1c2VkIGFz IHRoZSBkZWZhdWx0IHJvdXRlIG9mIGVhY2ggaG9zdC4gSWYgeW91IGNoYW5nZSB0aGUKPiByb2xl LCBhbGwgaG9zdHMgc2hvdWxkIGJlIG5vdGlmaWVkICh3aXRoIGEgbmV3IHNldHVwTmV0d29yayBj b21tYW5kKS4KPgo+IEkgdGhpbmsgdGhhdCB0aGUgY2xlYW5lc3Qgc29sdXRpb24gd291bGQgYmUg dG8gYWxsb3cgZWRpdGluZywgYnV0IHJlcG9ydAo+IHRoZSBob3N0cyBhcyBvdXQtb2Ytc3luYy4g VGhpcyBhcHByb2FjaCByZXF1aXJlcyBhIFZkc20tc2lkZSBjaGFuZ2UgLSBpdAo+IHdvdWxkIG5l ZWQgdG8gcmVwb3J0IHdoaWNoIG9mIGl0cyBuZXR3b3JrIGlzIHRoZSBkZWZhdWx0IHJvdXRlLgpU aGFuayB5b3UgZm9yIHR1cm5pbmcgbXkgYXR0ZW50aW9uIHRvIHRoaXMgc2NlbmFyaW8sIEknbGwg dXBkYXRlIHRoZSAKd2lraSBwYWdlLgo+Cj4gRGFuLgoKCi0tLS0tLS0tLS0tLS0tMDUwMDA5MDgw NDA4MDMwNjAwMDcwMzAwCkNvbnRlbnQtVHlwZTogdGV4dC9odG1sOyBjaGFyc2V0PUlTTy04ODU5 LTEKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzogN2JpdAoKPGh0bWw+CiAgPGhlYWQ+CiAgICA8 bWV0YSBjb250ZW50PSJ0ZXh0L2h0bWw7IGNoYXJzZXQ9SVNPLTg4NTktMSIKICAgICAgaHR0cC1l cXVpdj0iQ29udGVudC1UeXBlIj4KICAgIDxsaW5rIGhyZWY9ImNocm9tZTovL3RyYW5zbGF0b3Iv c2tpbi9mbG9hdGluZ1BhbmVsLmNzcyIKICAgICAgdHlwZT0idGV4dC9jc3MiIHJlbD0ic3R5bGVz aGVldCI+CiAgPC9oZWFkPgogIDxib2R5IHRleHQ9IiMwMDAwMDAiIGJnY29sb3I9IiNGRkZGRkYi PgogICAgPGJyPgogICAgPGRpdiBjbGFzcz0ibW96LWNpdGUtcHJlZml4Ij5PbiAxNS8wOC8xNCAx Mjo1NSwgRGFuIEtlbmlnc2JlcmcKICAgICAgd3JvdGU6PGJyPgogICAgPC9kaXY+CiAgICA8Ymxv Y2txdW90ZSBjaXRlPSJtaWQ6MjAxNDA4MTUwOTU1MjMuR0E2ODA1QHJlZGhhdC5jb20iIHR5cGU9 ImNpdGUiPgogICAgICA8cHJlIHdyYXA9IiI+T24gVGh1LCBBdWcgMTQsIDIwMTQgYXQgMTE6NTI6 NDFBTSAtMDQwMCwgWWV2Z2VueSBaYXNwaXRza3kgd3JvdGU6CjwvcHJlPgogICAgICA8YmxvY2tx dW90ZSB0eXBlPSJjaXRlIj4KICAgICAgICA8cHJlIHdyYXA9IiI+SGkgQWxsLCAKClRoZSBwcm9w b3NlZCBmZWF0dXJlIHdpbGwgYWxsb3cgZGVmaW5pbmcgYW4gYXJiaXRyYXJ5IG5ldHdvcmsgaW4g dGhlIERDIGFzIHRoZSBtYW5hZ2VtZW50IG5ldHdvcmsgZm9yIHRoZSBjbHVzdGVyLCB3aGljaCBp biBpdHMgdHVybiB3aWxsIGFsbG93IGFzc2lnbmluZyBkaWZmZXJlbnQgVkxBTnMgZm9yIHRoZSBt YW5hZ2VtZW50IG5ldHdvcmtzIGluIHRoZSBzYW1lIERDLiAKCkZlYXR1cmUgcGFnZSBjYW4gYmUg Zm91bmQgaGVyZSAtIDxhIGNsYXNzPSJtb3otdHh0LWxpbmstZnJlZXRleHQiIGhyZWY9Imh0dHA6 Ly93d3cub3ZpcnQub3JnL0ZlYXR1cmVzL01hbmFnZW1lbnRfTmV0d29ya19Bc19BX1JvbGUiPmh0 dHA6Ly93d3cub3ZpcnQub3JnL0ZlYXR1cmVzL01hbmFnZW1lbnRfTmV0d29ya19Bc19BX1JvbGU8 L2E+IC4gCgpQbGVhc2UgdGFrZSBhIGxvb2sgaW50byB0aGUgcGFnZSBlc3BlY2lhbGx5IGludG8g Ik9wZW4gaXNzdWVzIiBzZWN0aW9uLiBJJ2QgbGlrZSB0byBoYXZlIHlvdXIgb3BpbmlvbnMgb24g dGhhdC4gCjwvcHJlPgogICAgICA8L2Jsb2NrcXVvdGU+CiAgICAgIDxwcmUgd3JhcD0iIj4KTWF5 IEkgYXNrIHdoeSB5b3UgY2hhbmdlIHRoZSBkZWZhdWx0IG1hbmFnZW1lbnQgbmV0d29yayBmcm9t IG92aXJ0bWdtdAp0byAiTWFuYWdlbWVudCI/IChBbmQgd2h5IHRoZSB1cGVyY2FzZSBNPyk8L3By ZT4KICAgIDwvYmxvY2txdW90ZT4KICAgIFdlJ2QgbGlrZSB0byBnZXQgcmlkIG9mIHRoYXQgZGlm ZmVyZW5jZSBiZXR3ZWVuIG9WaXJ0IGFuZCBSRVZNLiBJTUhPCiAgICB0aGVyZSdzIG5vIGdvb2Qg cmVhc29uIGZvciBoYXZpbmcgcHJvZHVjdCBuYW1lIGluIHRoZSBuZXR3b3JrL2JyaWRnZQogICAg bmFtZS48YnI+CiAgICBJZiB5b3UgZG8gbm90IGxpa2UgY2FwaXRhbCBsZXR0ZXJzIGluIHRoZSBu ZXR3b3JrIG5hbWUgSSdtIE9LIHdpdGgKICAgIGNoYW5naW5nIGl0IHRvIHRoZSBsb3dlciBvbmUu PGJyPgogICAgPGJsb2NrcXVvdGUgY2l0ZT0ibWlkOjIwMTQwODE1MDk1NTIzLkdBNjgwNUByZWRo YXQuY29tIiB0eXBlPSJjaXRlIj4KICAgICAgPHByZSB3cmFwPSIiPgoKUmVnYXJkaW5nIHlvdXIg b3BlbiBxdWVzdGlvbjogIkNyZWF0aW5nIG5ldyBjbHVzdGVyIHdvdWxkIGhhdmUgdG8KcmVjZWl2 ZSB0aGUgbmV3IHBhcmFtZXRlciAobWFuYWdlbWVudCBuZXR3b3JrKSIgVGhpcyBuZXcgcGFyYW1l dGVyCnNob3VsZCBiZSBrZXB0IG9wdGlvbmFsLCB3aXRoIGEgZGVmYXVsdCB2YWx1ZSBvZiBvdmly dG1nbXQuIFRoaXMgd2F5LCBhCnVzZXIgdGhhdCBpcyB1bmF3YXJlIG9mIHRoZSBuZXcgZmVhdHVy ZSwgd291bGQgc2VlIG5vIGNoYW5nZSBpbgpmdW5jdGlvbmFsaXR5Lgo8L3ByZT4KICAgIDwvYmxv Y2txdW90ZT4KICAgIFVzaW5nIGEgc3BlY2lmaWMgbmV0d29yayBuYW1lIHNlZW1zIGlzbid0IHBv c3NpYmxlLCBhcyB0aGF0IG5ldHdvcmsKICAgIG1pZ2h0IGJlIG5vdCBleGlzdGVudCBhdCB0aGUg dGltZSBvZiBpc3N1aW5nIHRoZSBjb21tYW5kLjxicj4KICAgIERvaW5nIHNvIGNvdWxkIHJlZHVj ZSB0aGUgbnVtYmVyIGNhc2VzIHdoZXJlIGJhY2t3YXJkIGNvbXBhdGliaWxpdHkKICAgIGlzIGJy b2tlbiwgYnV0IGNhbiBub3QgZWxpbWluYXRlIGl0IHRvdGFsbHkuIEluIHRob3NlIGJyb2tlbiBj YXNlcwogICAgd2UgbWlnaHQgcmV0dXJuIGFuIGVycm9yIHRvIGEgUkVTVGZ1bCBBUEkgdXNlci48 YnI+CiAgICA8YmxvY2txdW90ZSBjaXRlPSJtaWQ6MjAxNDA4MTUwOTU1MjMuR0E2ODA1QHJlZGhh dC5jb20iIHR5cGU9ImNpdGUiPgogICAgICA8cHJlIHdyYXA9IiI+ClRoZSBmZWF0dXJlIHBhZ2Ug c2hvdWxkIGRpc2N1c3MgdGhlIHBvc3NpYmlsaXR5IG9mIGNoYW5pbmcgdGhlCm1hbmFnZW1lbnQg cm9sZS4gSXMgaXQgc3VwcG9ydGVkIGFmdGVyIHRoZXJlIGFyZSBob3N0cyBpbiB0aGUgY2x1c3Rl cj8KSWYgd2UgYWxsb3cgdGhhdCwgdGhlcmUncyBhIGJpdCBvZiBjb21wbGV4aXR5LiBUaGUgbWFu YWdlbWVudCBuZXR3b3JrJ3MKZ2F0ZXdheSBpcyB1c2VkIGFzIHRoZSBkZWZhdWx0IHJvdXRlIG9m IGVhY2ggaG9zdC4gSWYgeW91IGNoYW5nZSB0aGUKcm9sZSwgYWxsIGhvc3RzIHNob3VsZCBiZSBu b3RpZmllZCAod2l0aCBhIG5ldyBzZXR1cE5ldHdvcmsgY29tbWFuZCkuCgpJIHRoaW5rIHRoYXQg dGhlIGNsZWFuZXN0IHNvbHV0aW9uIHdvdWxkIGJlIHRvIGFsbG93IGVkaXRpbmcsIGJ1dCByZXBv cnQKdGhlIGhvc3RzIGFzIG91dC1vZi1zeW5jLiBUaGlzIGFwcHJvYWNoIHJlcXVpcmVzIGEgVmRz bS1zaWRlIGNoYW5nZSAtIGl0CndvdWxkIG5lZWQgdG8gcmVwb3J0IHdoaWNoIG9mIGl0cyBuZXR3 b3JrIGlzIHRoZSBkZWZhdWx0IHJvdXRlLjwvcHJlPgogICAgPC9ibG9ja3F1b3RlPgogICAgVGhh bmsgeW91IGZvciB0dXJuaW5nIG15IGF0dGVudGlvbiB0byB0aGlzIHNjZW5hcmlvLCBJJ2xsIHVw ZGF0ZSB0aGUKICAgIHdpa2kgcGFnZS48YnI+CiAgICA8YmxvY2txdW90ZSBjaXRlPSJtaWQ6MjAx NDA4MTUwOTU1MjMuR0E2ODA1QHJlZGhhdC5jb20iIHR5cGU9ImNpdGUiPgogICAgICA8cHJlIHdy YXA9IiI+CgpEYW4uCjwvcHJlPgogICAgPC9ibG9ja3F1b3RlPgogICAgPGJyPgogICAgPGRpdiBz dHlsZT0iYm90dG9tOiBhdXRvOyBsZWZ0OiAzMTFweDsgcmlnaHQ6IGF1dG87IHRvcDogMzcxcHg7 CiAgICAgIGRpc3BsYXk6IG5vbmU7IiBjbGFzcz0idHJhbnNsYXRvci10aGVtZS1kZWZhdWx0Igog ICAgICBpZD0idHJhbnNsYXRvci1mbG9hdGluZy1wYW5lbCI+CiAgICAgIDxkaXYgdGl0bGU9IkNs aWNrIHRvIHRyYW5zbGF0ZSIKICAgICAgICBpZD0idHJhbnNsYXRvci1mbG9hdGluZy1wYW5lbC1i dXR0b24iPjwvZGl2PgogICAgPC9kaXY+CiAgPC9ib2R5Pgo8L2h0bWw+CgotLS0tLS0tLS0tLS0t LTA1MDAwOTA4MDQwODAzMDYwMDA3MDMwMC0tCg== --===============5604952005813401158==--