Thursday, 2013-11-28

NobodyCamdevananda: you still around?00:05
devanandaNobodyCam: ya00:08
devanandadunno why :p00:08
NobodyCamhehehe00:08
NobodyCamok your going to love this one.. I think00:08
devanandatrying to debug my changes to periodic tasks00:08
devananda??00:08
NobodyCamits vif logic question.00:08
NobodyCamone sec need to get all the reff ready00:09
NobodyCamhttps://github.com/openstack/nova/blob/master/nova/virt/baremetal/driver.py#L242 calls https://github.com/openstack/nova/blob/master/nova/virt/baremetal/driver.py#L24200:11
NobodyCamwhich first clears anny vif in the db for each port00:12
*** martyntaylor has quit IRC00:12
NobodyCamthen calls https://github.com/openstack/nova/blob/master/nova/virt/baremetal/vif_driver.py#L3700:12
devanandaya00:13
NobodyCamwhich checks each port if no vif_id is set it wrights its vif_uuid to the database00:13
NobodyCamwhat if there are more then one vif in network info00:14
devanandaand if there are too many vif and not enough ports, it raises an exception, ya?00:14
devanandai may have misinterpreted/misremembered that code00:15
NobodyCamis there a mapping vif to pif some where00:16
NobodyCamhttps://github.com/openstack/nova/blob/master/nova/virt/baremetal/driver.py#L49000:16
NobodyCamseems to say there can be more then one vif00:16
NobodyCambut the code wrights the first one to every bm port00:17
devanandaNobodyCam: https://github.com/openstack/nova/blob/master/nova/virt/baremetal/vif_driver.py#L5300:23
devanandaNobodyCam: it looks like it only updates the first not-alraedy-set pif00:23
NobodyCamdoh00:23
devananda:)00:23
devanandait's late. we should both stop soon00:23
NobodyCamso I can assume vif 1 to pif 1 vif2 to pif2 ect ...00:24
NobodyCam:) ya00:24
devanandaya00:24
NobodyCam:)00:25
*** matsuhashi has joined #openstack-ironic00:25
*** rongze has joined #openstack-ironic00:36
*** rongze has quit IRC00:41
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Add prepare and cleanup methods to deploy interface  https://review.openstack.org/5889400:57
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Add conductor_heartbeat_interval option  https://review.openstack.org/5885100:57
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Implement sync_power_state periodic task  https://review.openstack.org/5653000:57
*** jbjohnso has quit IRC00:59
devanandaright. I've made a mess of my patches. and I'm heading out in search of food01:01
*** sjing has joined #openstack-ironic01:31
sjingMorning, Ironic01:34
*** nosnos has joined #openstack-ironic01:40
*** epim has quit IRC01:56
*** rongze has joined #openstack-ironic01:59
Haomengmorning:)02:00
*** rongze has quit IRC02:19
*** rloo has joined #openstack-ironic02:24
*** yuiko_takada has joined #openstack-ironic02:32
*** vkozhukalov has joined #openstack-ironic02:33
*** rongze has joined #openstack-ironic02:36
*** michchap has quit IRC02:52
*** michchap has joined #openstack-ironic02:52
*** matsuhashi has quit IRC03:26
*** matsuhashi has joined #openstack-ironic04:09
*** matsuhashi has quit IRC04:41
*** rloo has quit IRC04:43
*** rongze has quit IRC04:47
*** matsuhashi has joined #openstack-ironic04:50
*** prekarat has joined #openstack-ironic04:51
*** jcooley_ has joined #openstack-ironic05:38
*** rongze has joined #openstack-ironic05:47
*** rongze has quit IRC05:52
openstackgerritJenkins proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/5843806:00
*** vkozhukalov has quit IRC06:14
*** jcooley_ has quit IRC06:23
*** jcooley_ has joined #openstack-ironic06:23
*** matsuhashi has quit IRC06:24
*** matsuhashi has joined #openstack-ironic06:25
*** sjing has quit IRC06:34
*** sjing has joined #openstack-ironic06:37
*** mihgen has joined #openstack-ironic06:41
*** rongze has joined #openstack-ironic06:48
*** rongze has quit IRC06:52
*** mihgen has quit IRC07:03
*** sjing has quit IRC07:15
*** prekarat1 has joined #openstack-ironic07:20
*** prekarat has quit IRC07:20
*** sjing has joined #openstack-ironic07:28
*** jcooley_ has quit IRC07:46
*** rongze has joined #openstack-ironic07:48
*** romcheg has joined #openstack-ironic07:49
*** romcheg has left #openstack-ironic07:49
*** jcooley_ has joined #openstack-ironic07:50
*** vkozhukalov has joined #openstack-ironic07:53
*** rongze has quit IRC07:53
*** prekarat has joined #openstack-ironic08:25
*** prekarat1 has quit IRC08:27
*** prekarat has quit IRC08:33
*** jheiss has quit IRC08:43
*** sjing has quit IRC08:44
*** jheiss has joined #openstack-ironic08:44
*** rongze has joined #openstack-ironic08:49
*** rongze has quit IRC08:55
*** nosnos_ has joined #openstack-ironic09:00
*** jistr has joined #openstack-ironic09:01
*** nosnos has quit IRC09:03
*** derekh has joined #openstack-ironic09:18
*** lucasagomes has joined #openstack-ironic09:19
*** ndipanov_gone is now known as ndipanov09:36
*** yuiko_takada has quit IRC09:36
*** Haomeng has quit IRC09:38
*** rongze has joined #openstack-ironic09:51
*** nosnos has joined #openstack-ironic09:54
*** romcheg1 has joined #openstack-ironic09:55
*** rongze has quit IRC09:56
*** nosnos_ has quit IRC09:57
*** martyntaylor has joined #openstack-ironic09:59
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add driver interfaces validation  https://review.openstack.org/5459310:06
*** romcheg1 is now known as romcheg10:08
*** romcheg has left #openstack-ironic10:09
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Add port address unique constraint for sqlite  https://review.openstack.org/5772210:10
*** mihgen has joined #openstack-ironic10:26
openstackgerritLucas Alvares Gomes proposed a change to openstack/python-ironicclient: Add missing i18n support  https://review.openstack.org/5831810:26
openstackgerritA change was merged to openstack/ironic: Add port address unique constraint for sqlite  https://review.openstack.org/5772210:32
*** rongze has joined #openstack-ironic10:52
*** martyntaylor1 has joined #openstack-ironic10:54
*** martyntaylor has quit IRC10:56
*** rongze has quit IRC10:56
*** martyntaylor1 has quit IRC10:59
*** martyntaylor has joined #openstack-ironic10:59
*** martyntaylor has quit IRC11:00
*** martyntaylor1 has joined #openstack-ironic11:01
*** max_lobur_afk is now known as max_lobur11:02
*** michchap has quit IRC11:07
*** martyntaylor1 has quit IRC11:08
*** michchap has joined #openstack-ironic11:10
*** martyntaylor has joined #openstack-ironic11:29
*** max_lobur has quit IRC11:31
*** max_lobur has joined #openstack-ironic11:32
*** ben_duyujie has joined #openstack-ironic12:32
*** ifarkas has joined #openstack-ironic12:35
openstackgerritMax Lobur proposed a change to openstack/ironic: Do not allow API to expose error stacktrace  https://review.openstack.org/5855812:43
openstackgerritMax Lobur proposed a change to openstack/ironic: Allow Pecan to use 'debuginfo' response field  https://review.openstack.org/5880712:53
*** rongze has joined #openstack-ironic12:57
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Add power control to PXE driver  https://review.openstack.org/5040913:06
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Add power control to PXE driver  https://review.openstack.org/5040913:11
*** rongze has quit IRC13:24
*** jistr has quit IRC13:27
*** jistr has joined #openstack-ironic13:30
*** nosnos has quit IRC13:35
*** Haomeng has joined #openstack-ironic13:35
*** rongze has joined #openstack-ironic13:36
*** SpamapS has quit IRC13:40
*** SpamapS has joined #openstack-ironic13:40
*** SpamapS has quit IRC13:40
*** SpamapS has joined #openstack-ironic13:40
*** ben_duyujie has quit IRC13:49
*** Haomeng|2 has joined #openstack-ironic13:54
*** Haomeng has quit IRC13:57
*** martyntaylor has quit IRC14:08
*** martyntaylor has joined #openstack-ironic14:23
*** mihgen has quit IRC14:23
*** mihgen has joined #openstack-ironic14:25
*** rongze has quit IRC14:47
*** rongze has joined #openstack-ironic14:50
*** matsuhashi has quit IRC14:59
max_loburmoring Ironic!15:03
max_loburlucasagomes, will you please take a look https://review.openstack.org/#/c/59038/ ? this is CLI part of 'debuginfo' fix15:04
lucasagomesmax_lobur, morning :)15:04
lucasagomesyes15:04
lucasagomesThe page you requested was not found, or you do not have permission to view this page15:04
max_loburhmm15:04
max_loburI uploaded this as a draft15:05
max_loburdo I need to make some actions to make it public?15:05
lucasagomeshmm actually idk15:06
max_loburhttps://review.openstack.org/#/c/59038/1 change to review15:08
max_loburI guess I need to read more about using drafts :)15:09
*** rongze has quit IRC15:10
max_lobur*changed15:10
lucasagomes:) u can mark as wip :D15:11
max_loburhehe15:11
max_loburdone15:11
*** jcooley_ has quit IRC15:15
*** jcooley_ has joined #openstack-ironic15:15
*** rongze has joined #openstack-ironic15:19
openstackgerritMax Lobur proposed a change to openstack/ironic: Allow Pecan to use 'debuginfo' response field  https://review.openstack.org/5880715:31
lucasagomesmax_lobur, why # noqa on the comments?15:33
max_loburlucasagomes, they're longer than 79 chars15:48
lucasagomesahh15:51
*** jbjohnso has joined #openstack-ironic15:52
*** vkozhukalov has quit IRC15:55
NobodyCamhappy thanks giving Ironic16:06
max_loburu too NobodyCam :)16:06
*** mihgen has quit IRC16:09
*** romcheg has joined #openstack-ironic16:14
lucasagomesNobodyCam, hey ya :)16:34
shadowerany plans to do an Ironic talk at FOSDEM?16:45
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Save PKI token in a file for PXE deploy ramdisk  https://review.openstack.org/5906616:53
openstackgerritMax Lobur proposed a change to openstack/ironic: Add missing link for drivers resource  https://review.openstack.org/5908117:25
*** martyntaylor has quit IRC17:26
max_loburlucasagomes, about our yesterday's conversation regarding API links - I've made initial patch that just adds missing link (https://review.openstack.org/#/c/59081/1). I also improved the test a little, but it still doesn't guarantee that each subcontroller will have a link.. Going to try to make autogeneration for the links and make a smart test for this in a separate patch17:30
*** jistr has quit IRC17:32
max_loburjust not sure if autogeneration will look OK, so let's just have everything sync'd for the beginning :)17:34
lucasagomesmax_lobur, right :) makes sense. The autogeneration would be somehting nice to have, but I don't think that we are going to add many (if any) new resources to the API, so it's not that critical17:38
max_loburyep, just played with it a little, so far it might add more complexity than convenience. If I won't get something cool I'll just postpone those bug17:40
lucasagomesfair17:41
*** rongze has quit IRC17:46
*** vkozhukalov has joined #openstack-ironic17:57
*** ifarkas has quit IRC18:00
*** romcheg has quit IRC18:08
*** derekh has quit IRC18:10
lucasagomesgood night everybody :)18:11
lucasagomesIm heading off18:12
*** lucasagomes has quit IRC18:12
*** romcheg has joined #openstack-ironic18:22
lifelessNobodyCam: https://review.openstack.org/#/c/58346/ has been earmarked for you btw18:28
*** max_lobur is now known as max_lobur_afk18:29
max_lobur_afkg'night everyone18:30
openstackgerritA change was merged to openstack/ironic: Do not allow API to expose error stacktrace  https://review.openstack.org/5855818:36
openstackgerritA change was merged to openstack/ironic: Sync some db changes from Oslo  https://review.openstack.org/5881718:37
*** rongze has joined #openstack-ironic18:44
NobodyCamlifeless: Thank you :)19:43
*** rongze_ has joined #openstack-ironic19:43
*** rongze has quit IRC19:46
*** rongze has joined #openstack-ironic19:51
*** rongze_ has quit IRC19:52
openstackgerritA change was merged to openstack/ironic: Use oslo's execute() and ssh_execute() methods  https://review.openstack.org/5834619:55
*** rongze has quit IRC20:14
*** jcooley_ has quit IRC20:45
*** jbjohnso has quit IRC20:55
*** romcheg has left #openstack-ironic20:56
*** michchap_ has joined #openstack-ironic21:28
*** michchap has quit IRC21:30
*** vkozhukalov has quit IRC21:32
lifelessdevananda: I have commended on https://review.openstack.org/#/c/58236/ if you want to reconsider21:45
*** michchap has joined #openstack-ironic22:26
*** michchap_ has quit IRC22:28

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