Thursday, 2015-12-31

*** links has joined #openstack-ironic00:23
jrolldevananda: on that joinedload() review, what happens if other attributes down the road are not loaded via joinedload() but rather subquery() or something else entirely? another parameter?00:28
devanandajroll: hm, fair point00:31
devanandaif other attributes are loaded via other means, will they present the same object interface?00:31
devananda*same interface through o.vo ?00:32
devanandajroll: btw, i got the neutron integration patch stack working locally today in devstack00:32
devanandatrying to figure out how to validate the network isolation right now00:32
jrollyes, it would still be expected_attrs00:34
jrollin the objects layer00:34
devanandaso the difference would be only in the db/api layer?00:34
jrolland then I'd have _build_port_query do something like if col == node: joinedload() elif col == foo: subquery()00:35
devanandayea, so in that case, I could see using a differently named parameter00:35
jrollI could, but OTOH I wish the objects layer didn't have to know how the db layer builds a query00:36
* devananda rereads the review00:36
devanandayea, i see your point00:37
devanandastill it's not loading a column00:37
jrollyeah00:37
jrollrelationships_to_load is long but makes the most sense imo00:37
jrollI was also thinking extra_attrs or something00:37
devanandathis approach only works for N:[0,1] relationships00:38
devanandaor rather, think of how weird the interface would be if it was get_node_by_id(node_id, columns_to_join=['port'])00:39
devanandathen you'd have node.port be a list of result objects00:39
jrollwell, we'd make it 'ports'00:39
jrollthese are defined on the model00:39
devanandaright - this is useful, but awkward00:40
jrollthings_to_conjur00:40
devanandaports.uuid is a scalar. ports.node is an object. nodes.uuid is a scalar. nodes.ports is a list of objects00:40
devanandaeh - it's super useful. I'm talking myself into liking it :)00:41
jrollheh00:41
devanandaI like extra_attrs00:41
jrollI still like things_to_conjur00:41
devanandathat, at least, says "hey this isn't actually part of the THING, it's extra"00:42
jrollbut I'll run with extra_attrs00:42
devanandawell, I also like things_to_conjur :)00:42
jroll:P00:42
jrollthanks00:42
devanandanp00:42
devanandafor the neutron integration, have you done any testing with the OVS/devstack support?00:43
jrollI haven't, I've been meaning to00:44
jrollI started playing with the claims stuff, needed that magic registry thing to do what I wanted, saw nastiness in ports api, and I've been shaving that yak for a couple days now00:44
devanandaI hear yak butter has a strong flavor00:47
jrollheh00:48
* jroll tries milking rather than shaving00:48
jrolloh, but as to your isolation question, I think the devstack code is supposed to handle that00:49
devanandaat first glance, it looks like it is separating provisioning from tenant networks00:49
devanandabut not tenants from each other00:50
* devananda may also be doing something wrong00:50
jrolloh00:50
jrollI could see that00:50
devanandalike, maybe I (as admin) need to create multiple private nets and assign each one to a tenant?00:51
jrolldevananda: looks like that's intended? https://review.openstack.org/#/c/256366/6/devstack/tools/ironic/scripts/create-node.sh00:51
devanandaright now, there's only one private net, one public net, and one "ironic-provision" net00:52
jrolloh no, there's a bridge for each vm00:52
jrollhrm00:52
devanandayea00:52
jrollso the first two are probably provider networks00:53
devanandai've got 3 tenants, but they're all getting IPs from the same "private" network00:53
jrollcan you make a network with one tenant and nova boot --net whatever?00:53
jrolland see if it's isolated from the other00:53
*** _zouyee has joined #openstack-ironic01:07
*** shuquan has joined #openstack-ironic01:17
devanandajroll: hm, so, that seems to also require me to use ovs on the host to attach to that network01:21
devanandawhich sort of makes sense -- but I had expected neutron to do that01:21
devanandawhen I run neutron net-create ...; neutron subnet-create ....; as far as I can tell, it isn't linking those to the host01:22
devanandaI see the dhcpreqest from the VM, and subsequent dhcpoffer for the IP on the new subnet - but that's the end of it01:30
*** yolanda has quit IRC01:48
*** jaypipes has quit IRC01:53
*** shuquan has quit IRC01:59
*** dims has quit IRC02:08
*** dims has joined #openstack-ironic02:39
*** dims has quit IRC02:39
*** jaybeale has quit IRC03:02
*** yarkot has quit IRC03:02
*** harshs has quit IRC03:14
*** MattMan has quit IRC03:17
*** MattMan has joined #openstack-ironic03:17
*** baoli has quit IRC03:33
*** baoli has joined #openstack-ironic03:33
*** shuquan has joined #openstack-ironic03:34
*** yarkot has joined #openstack-ironic03:47
*** shuquan_ has joined #openstack-ironic03:56
*** shuquan has quit IRC03:56
*** shuquan_ is now known as shuquan03:56
*** shuquan has quit IRC04:01
*** shuquan has joined #openstack-ironic05:35
*** shuquan has quit IRC05:44
*** baoli has quit IRC05:49
*** baoli has joined #openstack-ironic05:53
*** coolsvap|away is now known as coolsvap06:02
*** jaybeale has joined #openstack-ironic06:14
*** jaybeale has quit IRC06:50
*** jaybeale has joined #openstack-ironic06:52
*** jaybeale has quit IRC06:53
*** baoli has quit IRC07:13
*** shuquan has joined #openstack-ironic07:14
*** baoli has joined #openstack-ironic07:29
*** baoli has quit IRC07:33
*** ukalifon has joined #openstack-ironic07:34
*** baoli has joined #openstack-ironic07:41
*** shuquan has quit IRC07:41
*** coolsvap is now known as coolsvap|away07:45
*** baoli has quit IRC07:45
*** ndipanov has joined #openstack-ironic07:47
*** shuquan has joined #openstack-ironic07:48
*** david-lyle has quit IRC07:50
*** rcernin has joined #openstack-ironic07:52
*** baoli has joined #openstack-ironic07:52
*** shuquan has quit IRC07:53
*** baoli has quit IRC07:57
*** shuquan has joined #openstack-ironic07:57
*** moshele has joined #openstack-ironic08:00
*** shuquan_ has joined #openstack-ironic08:02
*** shuquan has quit IRC08:02
*** shuquan_ is now known as shuquan08:02
*** shuquan has quit IRC08:08
*** baoli has joined #openstack-ironic08:11
*** yolanda has joined #openstack-ironic08:15
*** baoli has quit IRC08:16
*** pas-ha has joined #openstack-ironic08:17
pas-hamorning Ironic :) and Happy New Year to everyone!08:17
*** baoli has joined #openstack-ironic08:23
*** boris-42 has joined #openstack-ironic08:23
*** baoli has quit IRC08:28
*** ChubYann has quit IRC08:33
*** baoli has joined #openstack-ironic08:34
*** baoli has quit IRC08:39
*** baoli has joined #openstack-ironic08:46
*** david-lyle has joined #openstack-ironic08:50
*** baoli has quit IRC08:51
*** baoli has joined #openstack-ironic08:58
zhenguoHaomeng:  hello, are you around?08:58
Haomengzhenguo: yes, hello ZhengGuo08:59
Haomengpas-ha: happy  new year08:59
zhenguoHaomeng: I'm interested in https://blueprints.launchpad.net/ironic/+spec/backup-disk-to-image, are you still working on that?08:59
Haomengzhenguo: yes,08:59
Haomengzhenguo: will propose bp these days09:00
zhenguoHaomeng: cool09:00
zhenguoHaomeng: we can poweroff the node and copy the image out09:00
Haomengzhenguo: want to leverage cinder as storage, and will restore via 'boot from volume'09:01
Haomengzhenguo: yes, it is offline snapshot09:01
*** Marga_ has joined #openstack-ironic09:01
zhenguoHaomeng: is that like nova snapshot?09:01
*** _zouyee has quit IRC09:02
Haomengzhenguo: yes, but does not support multi-lays fs such as qcow209:02
*** baoli has quit IRC09:03
zhenguoHaomeng: and does we need to upload the image to glance?09:03
*** moshele has quit IRC09:09
*** baoli has joined #openstack-ironic09:10
Haomengzhenguo: before 'boot from cinder volume' is ready, it need to be uploaded to glance09:11
zhenguoHaomeng: got it, thanks.09:13
*** baoli has quit IRC09:14
*** Marga_ has quit IRC09:20
*** baoli has joined #openstack-ironic09:21
*** baoli has quit IRC09:26
*** baoli has joined #openstack-ironic09:32
*** baoli has quit IRC09:37
vdrokmorning ironic09:40
openstackgerritZhenguo Niu proposed openstack/ironic: tox: make it possible to run pep8 on current patch only  https://review.openstack.org/26101109:42
zhenguomorning vdrok09:43
vdrokgood morning zhenguo09:43
*** baoli has joined #openstack-ironic09:44
*** moshele has joined #openstack-ironic10:00
*** zhenguo has quit IRC10:03
*** e0ne has joined #openstack-ironic10:18
*** yolanda has quit IRC10:55
*** yolanda has joined #openstack-ironic10:55
*** mbound has joined #openstack-ironic11:11
*** Marga_ has joined #openstack-ironic11:21
*** Marga_ has quit IRC11:26
*** links has quit IRC11:32
*** boris-42 has quit IRC11:33
*** vsaienko has joined #openstack-ironic11:52
sergekHappy New Year, Ironicers!12:15
*** _degorenko is now known as degorenko12:26
*** ndipanov has quit IRC12:40
*** Marga_ has joined #openstack-ironic12:44
*** Marga_ has quit IRC12:49
*** ndipanov has joined #openstack-ironic12:59
openstackgerritMerged openstack/ironic: Change assertTrue(isinstance()) by optimal assert  https://review.openstack.org/26252713:01
*** ndipanov has quit IRC13:35
*** baoli has quit IRC13:38
*** e0ne has quit IRC13:40
*** mbound has quit IRC13:42
*** baoli has joined #openstack-ironic13:46
*** baoli has quit IRC13:51
*** moshele has quit IRC13:56
*** baoli has joined #openstack-ironic13:58
*** baoli has quit IRC14:02
*** vdrok has quit IRC14:06
*** vdrok has joined #openstack-ironic14:08
vdrokhappy new year everyone, see you next year! :)14:09
*** baoli has joined #openstack-ironic14:11
*** degorenko has quit IRC14:13
*** baoli has quit IRC14:16
*** pas-ha has quit IRC14:16
*** mbound has joined #openstack-ironic14:19
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command  https://review.openstack.org/25145814:21
*** baoli has joined #openstack-ironic14:22
*** baoli has quit IRC14:27
*** Marga_ has joined #openstack-ironic14:35
*** Marga_ has quit IRC14:36
*** Marga_ has joined #openstack-ironic14:37
*** Marga_ has quit IRC14:45
*** Marga_ has joined #openstack-ironic14:45
*** e0ne has joined #openstack-ironic14:47
*** e0ne has quit IRC14:53
*** mbound has quit IRC14:54
*** mbound has joined #openstack-ironic14:55
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Tests for CLI help messages  https://review.openstack.org/26227314:59
*** mbound has quit IRC15:08
*** intr1nsic has quit IRC15:39
*** intr1nsic has joined #openstack-ironic15:46
*** e0ne has joined #openstack-ironic15:47
*** rcernin has quit IRC16:16
*** david-lyle has quit IRC16:25
*** jaybeale has joined #openstack-ironic16:29
*** Marga_ has quit IRC16:32
*** e0ne has quit IRC16:40
*** rcernin has joined #openstack-ironic16:54
*** e0ne has joined #openstack-ironic17:01
*** harshs has joined #openstack-ironic17:09
*** e0ne has quit IRC17:18
*** david-lyle has joined #openstack-ironic17:25
*** ukalifon has quit IRC17:34
*** david-lyle has quit IRC17:42
*** david-lyle has joined #openstack-ironic17:42
*** ChubYann has joined #openstack-ironic17:54
*** jaybeale has quit IRC18:00
*** rcernin has quit IRC18:09
*** harshs has quit IRC18:18
*** harshs has joined #openstack-ironic18:36
*** Marga_ has joined #openstack-ironic18:41
openstackgerritJay Faulkner proposed openstack/ironic: Support reboot_requested bool on agent clean_steps  https://review.openstack.org/25913518:47
JayFvdrok: jroll: ^ added reno and tests as requested18:47
JayFmariojv: ^ the tests I was asking you about in the other channel are in that review, if you wanna take a look18:49
mariojvcool, i'll look in a few mins18:50
*** yolanda has quit IRC19:00
*** Marga_ has quit IRC19:49
*** pas-ha has joined #openstack-ironic19:51
*** Marga_ has joined #openstack-ironic19:55
*** pas-ha has quit IRC19:57
mariojvJayF: looks fine but the reno tests are failing19:57
*** harshs has quit IRC20:10
*** e0ne has joined #openstack-ironic20:24
*** Marga_ has quit IRC20:31
*** openstackgerrit has quit IRC20:32
*** openstackgerrit has joined #openstack-ironic20:32
JayF^ I can't seem to figure out the reno stuff, after working on it locally for a bit20:54
JayFare there any documented guidelines?20:54
openstackgerritJay Faulkner proposed openstack/ironic: Support reboot_requested bool on agent clean_steps  https://review.openstack.org/25913521:04
openstackgerritJay Faulkner proposed openstack/ironic: Support reboot_requested bool on agent clean_steps  https://review.openstack.org/25913521:10
JayFmariojv: ^ reno is happy now21:10
JayFwell, now ^ lol21:10
*** Marga_ has joined #openstack-ironic22:42
*** e0ne has quit IRC22:45
*** harshs has joined #openstack-ironic22:54
*** e0ne has joined #openstack-ironic23:04
*** harshs has quit IRC23:09
*** harshs has joined #openstack-ironic23:24
*** Marga_ has quit IRC23:24
*** e0ne has quit IRC23:46

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!