NobodyCam | devananda: you still around? | 00:05 |
---|---|---|
devananda | NobodyCam: ya | 00:08 |
devananda | dunno why :p | 00:08 |
NobodyCam | hehehe | 00:08 |
NobodyCam | ok your going to love this one.. I think | 00:08 |
devananda | trying to debug my changes to periodic tasks | 00:08 |
devananda | ?? | 00:08 |
NobodyCam | its vif logic question. | 00:08 |
NobodyCam | one sec need to get all the reff ready | 00:09 |
NobodyCam | https://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#L242 | 00:11 |
NobodyCam | which first clears anny vif in the db for each port | 00:12 |
*** martyntaylor has quit IRC | 00:12 | |
NobodyCam | then calls https://github.com/openstack/nova/blob/master/nova/virt/baremetal/vif_driver.py#L37 | 00:12 |
devananda | ya | 00:13 |
NobodyCam | which checks each port if no vif_id is set it wrights its vif_uuid to the database | 00:13 |
NobodyCam | what if there are more then one vif in network info | 00:14 |
devananda | and if there are too many vif and not enough ports, it raises an exception, ya? | 00:14 |
devananda | i may have misinterpreted/misremembered that code | 00:15 |
NobodyCam | is there a mapping vif to pif some where | 00:16 |
NobodyCam | https://github.com/openstack/nova/blob/master/nova/virt/baremetal/driver.py#L490 | 00:16 |
NobodyCam | seems to say there can be more then one vif | 00:16 |
NobodyCam | but the code wrights the first one to every bm port | 00:17 |
devananda | NobodyCam: https://github.com/openstack/nova/blob/master/nova/virt/baremetal/vif_driver.py#L53 | 00:23 |
devananda | NobodyCam: it looks like it only updates the first not-alraedy-set pif | 00:23 |
NobodyCam | doh | 00:23 |
devananda | :) | 00:23 |
devananda | it's late. we should both stop soon | 00:23 |
NobodyCam | so I can assume vif 1 to pif 1 vif2 to pif2 ect ... | 00:24 |
NobodyCam | :) ya | 00:24 |
devananda | ya | 00:24 |
NobodyCam | :) | 00:25 |
*** matsuhashi has joined #openstack-ironic | 00:25 | |
*** rongze has joined #openstack-ironic | 00:36 | |
*** rongze has quit IRC | 00:41 | |
openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add prepare and cleanup methods to deploy interface https://review.openstack.org/58894 | 00:57 |
openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add conductor_heartbeat_interval option https://review.openstack.org/58851 | 00:57 |
openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Implement sync_power_state periodic task https://review.openstack.org/56530 | 00:57 |
*** jbjohnso has quit IRC | 00:59 | |
devananda | right. I've made a mess of my patches. and I'm heading out in search of food | 01:01 |
*** sjing has joined #openstack-ironic | 01:31 | |
sjing | Morning, Ironic | 01:34 |
*** nosnos has joined #openstack-ironic | 01:40 | |
*** epim has quit IRC | 01:56 | |
*** rongze has joined #openstack-ironic | 01:59 | |
Haomeng | morning:) | 02:00 |
*** rongze has quit IRC | 02:19 | |
*** rloo has joined #openstack-ironic | 02:24 | |
*** yuiko_takada has joined #openstack-ironic | 02:32 | |
*** vkozhukalov has joined #openstack-ironic | 02:33 | |
*** rongze has joined #openstack-ironic | 02:36 | |
*** michchap has quit IRC | 02:52 | |
*** michchap has joined #openstack-ironic | 02:52 | |
*** matsuhashi has quit IRC | 03:26 | |
*** matsuhashi has joined #openstack-ironic | 04:09 | |
*** matsuhashi has quit IRC | 04:41 | |
*** rloo has quit IRC | 04:43 | |
*** rongze has quit IRC | 04:47 | |
*** matsuhashi has joined #openstack-ironic | 04:50 | |
*** prekarat has joined #openstack-ironic | 04:51 | |
*** jcooley_ has joined #openstack-ironic | 05:38 | |
*** rongze has joined #openstack-ironic | 05:47 | |
*** rongze has quit IRC | 05:52 | |
openstackgerrit | Jenkins proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/58438 | 06:00 |
*** vkozhukalov has quit IRC | 06:14 | |
*** jcooley_ has quit IRC | 06:23 | |
*** jcooley_ has joined #openstack-ironic | 06:23 | |
*** matsuhashi has quit IRC | 06:24 | |
*** matsuhashi has joined #openstack-ironic | 06:25 | |
*** sjing has quit IRC | 06:34 | |
*** sjing has joined #openstack-ironic | 06:37 | |
*** mihgen has joined #openstack-ironic | 06:41 | |
*** rongze has joined #openstack-ironic | 06:48 | |
*** rongze has quit IRC | 06:52 | |
*** mihgen has quit IRC | 07:03 | |
*** sjing has quit IRC | 07:15 | |
*** prekarat1 has joined #openstack-ironic | 07:20 | |
*** prekarat has quit IRC | 07:20 | |
*** sjing has joined #openstack-ironic | 07:28 | |
*** jcooley_ has quit IRC | 07:46 | |
*** rongze has joined #openstack-ironic | 07:48 | |
*** romcheg has joined #openstack-ironic | 07:49 | |
*** romcheg has left #openstack-ironic | 07:49 | |
*** jcooley_ has joined #openstack-ironic | 07:50 | |
*** vkozhukalov has joined #openstack-ironic | 07:53 | |
*** rongze has quit IRC | 07:53 | |
*** prekarat has joined #openstack-ironic | 08:25 | |
*** prekarat1 has quit IRC | 08:27 | |
*** prekarat has quit IRC | 08:33 | |
*** jheiss has quit IRC | 08:43 | |
*** sjing has quit IRC | 08:44 | |
*** jheiss has joined #openstack-ironic | 08:44 | |
*** rongze has joined #openstack-ironic | 08:49 | |
*** rongze has quit IRC | 08:55 | |
*** nosnos_ has joined #openstack-ironic | 09:00 | |
*** jistr has joined #openstack-ironic | 09:01 | |
*** nosnos has quit IRC | 09:03 | |
*** derekh has joined #openstack-ironic | 09:18 | |
*** lucasagomes has joined #openstack-ironic | 09:19 | |
*** ndipanov_gone is now known as ndipanov | 09:36 | |
*** yuiko_takada has quit IRC | 09:36 | |
*** Haomeng has quit IRC | 09:38 | |
*** rongze has joined #openstack-ironic | 09:51 | |
*** nosnos has joined #openstack-ironic | 09:54 | |
*** romcheg1 has joined #openstack-ironic | 09:55 | |
*** rongze has quit IRC | 09:56 | |
*** nosnos_ has quit IRC | 09:57 | |
*** martyntaylor has joined #openstack-ironic | 09:59 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add driver interfaces validation https://review.openstack.org/54593 | 10:06 |
*** romcheg1 is now known as romcheg | 10:08 | |
*** romcheg has left #openstack-ironic | 10:09 | |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Add port address unique constraint for sqlite https://review.openstack.org/57722 | 10:10 |
*** mihgen has joined #openstack-ironic | 10:26 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/python-ironicclient: Add missing i18n support https://review.openstack.org/58318 | 10:26 |
openstackgerrit | A change was merged to openstack/ironic: Add port address unique constraint for sqlite https://review.openstack.org/57722 | 10:32 |
*** rongze has joined #openstack-ironic | 10:52 | |
*** martyntaylor1 has joined #openstack-ironic | 10:54 | |
*** martyntaylor has quit IRC | 10:56 | |
*** rongze has quit IRC | 10:56 | |
*** martyntaylor1 has quit IRC | 10:59 | |
*** martyntaylor has joined #openstack-ironic | 10:59 | |
*** martyntaylor has quit IRC | 11:00 | |
*** martyntaylor1 has joined #openstack-ironic | 11:01 | |
*** max_lobur_afk is now known as max_lobur | 11:02 | |
*** michchap has quit IRC | 11:07 | |
*** martyntaylor1 has quit IRC | 11:08 | |
*** michchap has joined #openstack-ironic | 11:10 | |
*** martyntaylor has joined #openstack-ironic | 11:29 | |
*** max_lobur has quit IRC | 11:31 | |
*** max_lobur has joined #openstack-ironic | 11:32 | |
*** ben_duyujie has joined #openstack-ironic | 12:32 | |
*** ifarkas has joined #openstack-ironic | 12:35 | |
openstackgerrit | Max Lobur proposed a change to openstack/ironic: Do not allow API to expose error stacktrace https://review.openstack.org/58558 | 12:43 |
openstackgerrit | Max Lobur proposed a change to openstack/ironic: Allow Pecan to use 'debuginfo' response field https://review.openstack.org/58807 | 12:53 |
*** rongze has joined #openstack-ironic | 12:57 | |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Add power control to PXE driver https://review.openstack.org/50409 | 13:06 |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Add power control to PXE driver https://review.openstack.org/50409 | 13:11 |
*** rongze has quit IRC | 13:24 | |
*** jistr has quit IRC | 13:27 | |
*** jistr has joined #openstack-ironic | 13:30 | |
*** nosnos has quit IRC | 13:35 | |
*** Haomeng has joined #openstack-ironic | 13:35 | |
*** rongze has joined #openstack-ironic | 13:36 | |
*** SpamapS has quit IRC | 13:40 | |
*** SpamapS has joined #openstack-ironic | 13:40 | |
*** SpamapS has quit IRC | 13:40 | |
*** SpamapS has joined #openstack-ironic | 13:40 | |
*** ben_duyujie has quit IRC | 13:49 | |
*** Haomeng|2 has joined #openstack-ironic | 13:54 | |
*** Haomeng has quit IRC | 13:57 | |
*** martyntaylor has quit IRC | 14:08 | |
*** martyntaylor has joined #openstack-ironic | 14:23 | |
*** mihgen has quit IRC | 14:23 | |
*** mihgen has joined #openstack-ironic | 14:25 | |
*** rongze has quit IRC | 14:47 | |
*** rongze has joined #openstack-ironic | 14:50 | |
*** matsuhashi has quit IRC | 14:59 | |
max_lobur | moring Ironic! | 15:03 |
max_lobur | lucasagomes, will you please take a look https://review.openstack.org/#/c/59038/ ? this is CLI part of 'debuginfo' fix | 15:04 |
lucasagomes | max_lobur, morning :) | 15:04 |
lucasagomes | yes | 15:04 |
lucasagomes | The page you requested was not found, or you do not have permission to view this page | 15:04 |
max_lobur | hmm | 15:04 |
max_lobur | I uploaded this as a draft | 15:05 |
max_lobur | do I need to make some actions to make it public? | 15:05 |
lucasagomes | hmm actually idk | 15:06 |
max_lobur | https://review.openstack.org/#/c/59038/1 change to review | 15:08 |
max_lobur | I guess I need to read more about using drafts :) | 15:09 |
*** rongze has quit IRC | 15:10 | |
max_lobur | *changed | 15:10 |
lucasagomes | :) u can mark as wip :D | 15:11 |
max_lobur | hehe | 15:11 |
max_lobur | done | 15:11 |
*** jcooley_ has quit IRC | 15:15 | |
*** jcooley_ has joined #openstack-ironic | 15:15 | |
*** rongze has joined #openstack-ironic | 15:19 | |
openstackgerrit | Max Lobur proposed a change to openstack/ironic: Allow Pecan to use 'debuginfo' response field https://review.openstack.org/58807 | 15:31 |
lucasagomes | max_lobur, why # noqa on the comments? | 15:33 |
max_lobur | lucasagomes, they're longer than 79 chars | 15:48 |
lucasagomes | ahh | 15:51 |
*** jbjohnso has joined #openstack-ironic | 15:52 | |
*** vkozhukalov has quit IRC | 15:55 | |
NobodyCam | happy thanks giving Ironic | 16:06 |
max_lobur | u too NobodyCam :) | 16:06 |
*** mihgen has quit IRC | 16:09 | |
*** romcheg has joined #openstack-ironic | 16:14 | |
lucasagomes | NobodyCam, hey ya :) | 16:34 |
shadower | any plans to do an Ironic talk at FOSDEM? | 16:45 |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Save PKI token in a file for PXE deploy ramdisk https://review.openstack.org/59066 | 16:53 |
openstackgerrit | Max Lobur proposed a change to openstack/ironic: Add missing link for drivers resource https://review.openstack.org/59081 | 17:25 |
*** martyntaylor has quit IRC | 17:26 | |
max_lobur | lucasagomes, 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 patch | 17:30 |
*** jistr has quit IRC | 17:32 | |
max_lobur | just not sure if autogeneration will look OK, so let's just have everything sync'd for the beginning :) | 17:34 |
lucasagomes | max_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 critical | 17:38 |
max_lobur | yep, 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 bug | 17:40 |
lucasagomes | fair | 17:41 |
*** rongze has quit IRC | 17:46 | |
*** vkozhukalov has joined #openstack-ironic | 17:57 | |
*** ifarkas has quit IRC | 18:00 | |
*** romcheg has quit IRC | 18:08 | |
*** derekh has quit IRC | 18:10 | |
lucasagomes | good night everybody :) | 18:11 |
lucasagomes | Im heading off | 18:12 |
*** lucasagomes has quit IRC | 18:12 | |
*** romcheg has joined #openstack-ironic | 18:22 | |
lifeless | NobodyCam: https://review.openstack.org/#/c/58346/ has been earmarked for you btw | 18:28 |
*** max_lobur is now known as max_lobur_afk | 18:29 | |
max_lobur_afk | g'night everyone | 18:30 |
openstackgerrit | A change was merged to openstack/ironic: Do not allow API to expose error stacktrace https://review.openstack.org/58558 | 18:36 |
openstackgerrit | A change was merged to openstack/ironic: Sync some db changes from Oslo https://review.openstack.org/58817 | 18:37 |
*** rongze has joined #openstack-ironic | 18:44 | |
NobodyCam | lifeless: Thank you :) | 19:43 |
*** rongze_ has joined #openstack-ironic | 19:43 | |
*** rongze has quit IRC | 19:46 | |
*** rongze has joined #openstack-ironic | 19:51 | |
*** rongze_ has quit IRC | 19:52 | |
openstackgerrit | A change was merged to openstack/ironic: Use oslo's execute() and ssh_execute() methods https://review.openstack.org/58346 | 19:55 |
*** rongze has quit IRC | 20:14 | |
*** jcooley_ has quit IRC | 20:45 | |
*** jbjohnso has quit IRC | 20:55 | |
*** romcheg has left #openstack-ironic | 20:56 | |
*** michchap_ has joined #openstack-ironic | 21:28 | |
*** michchap has quit IRC | 21:30 | |
*** vkozhukalov has quit IRC | 21:32 | |
lifeless | devananda: I have commended on https://review.openstack.org/#/c/58236/ if you want to reconsider | 21:45 |
*** michchap has joined #openstack-ironic | 22:26 | |
*** michchap_ has quit IRC | 22:28 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!