*** shobhar_ has quit IRC | 00:17 | |
*** jerrygb has joined #openstack-ironic | 00:17 | |
*** jerrygb_ has joined #openstack-ironic | 00:20 | |
*** jerrygb has quit IRC | 00:23 | |
*** mtanino has quit IRC | 00:35 | |
*** awiddersheim has quit IRC | 00:45 | |
*** maratoid_ has quit IRC | 00:45 | |
*** baoli has joined #openstack-ironic | 00:47 | |
*** baoli has quit IRC | 00:48 | |
*** baoli has joined #openstack-ironic | 00:50 | |
*** baoli_ has joined #openstack-ironic | 00:51 | |
*** jerrygb_ has quit IRC | 00:52 | |
rloo | JayF: wrt 392114, don't think so. looks like it is fixing a bug. w/o that change, things will fail, right? i added a comment to the patch | 00:52 |
---|---|---|
*** rloo has quit IRC | 00:53 | |
*** baoli has quit IRC | 00:55 | |
*** jkilpatr has quit IRC | 00:55 | |
*** baoli_ has quit IRC | 00:56 | |
*** tuanluong has joined #openstack-ironic | 00:59 | |
*** hoangcx has joined #openstack-ironic | 01:01 | |
*** hw_wutianwei has joined #openstack-ironic | 01:07 | |
*** aignatov has quit IRC | 01:08 | |
*** aignatov has joined #openstack-ironic | 01:11 | |
*** [1]cdearborn has quit IRC | 01:17 | |
*** awiddersheim has joined #openstack-ironic | 01:27 | |
*** baoli has joined #openstack-ironic | 01:33 | |
*** sabeen has joined #openstack-ironic | 01:40 | |
*** mtanino has joined #openstack-ironic | 01:42 | |
*** zhangyufei has joined #openstack-ironic | 01:54 | |
*** baoli has quit IRC | 01:59 | |
*** baoli has joined #openstack-ironic | 01:59 | |
*** mtanino has quit IRC | 02:01 | |
*** baoli has quit IRC | 02:04 | |
*** syed_ has quit IRC | 02:05 | |
*** jerrygb has joined #openstack-ironic | 02:05 | |
*** jvgrant has quit IRC | 02:14 | |
*** jvgrant has joined #openstack-ironic | 02:14 | |
*** zhangyufei has quit IRC | 02:15 | |
*** zhangyufei has joined #openstack-ironic | 02:29 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off https://review.openstack.org/216738 | 02:53 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/389896 | 03:03 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/python-ironicclient: Use function import_versioned_module from oslo.utils https://review.openstack.org/392395 | 03:23 |
*** mtanino has joined #openstack-ironic | 03:33 | |
*** vikrant has joined #openstack-ironic | 03:35 | |
*** vinsh has quit IRC | 03:36 | |
*** sabeen has quit IRC | 03:36 | |
*** lifeless has quit IRC | 03:41 | |
*** lifeless has joined #openstack-ironic | 03:42 | |
*** phuongnh has joined #openstack-ironic | 03:44 | |
*** mcarden has joined #openstack-ironic | 03:48 | |
*** absubram has joined #openstack-ironic | 04:04 | |
*** absubram_ has joined #openstack-ironic | 04:05 | |
*** absubram has quit IRC | 04:08 | |
*** absubram_ is now known as absubram | 04:08 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work https://review.openstack.org/376322 | 04:10 |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work https://review.openstack.org/376322 | 04:11 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/python-ironicclient: Use function import_versioned_module from oslo.utils https://review.openstack.org/392395 | 04:11 |
*** yuikotakadamori has quit IRC | 04:13 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work https://review.openstack.org/376322 | 04:19 |
*** jerrygb has quit IRC | 04:19 | |
*** jerrygb has joined #openstack-ironic | 04:24 | |
*** yuanying has quit IRC | 04:24 | |
*** chlong has quit IRC | 04:24 | |
*** shobhar_ has joined #openstack-ironic | 04:27 | |
*** shobhar_ has quit IRC | 04:28 | |
*** Nisha_away has joined #openstack-ironic | 04:32 | |
*** yuanying has joined #openstack-ironic | 04:36 | |
*** absubram has quit IRC | 04:37 | |
*** mtanino has quit IRC | 04:48 | |
*** hoangcx has quit IRC | 04:54 | |
*** jerrygb has quit IRC | 05:03 | |
*** jerrygb has joined #openstack-ironic | 05:06 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/python-ironicclient: Use function import_versioned_module from oslo.utils https://review.openstack.org/392395 | 05:36 |
*** vinsh has joined #openstack-ironic | 05:37 | |
*** vinsh has quit IRC | 05:41 | |
*** vmud213 has joined #openstack-ironic | 05:47 | |
*** maratoid_ has joined #openstack-ironic | 05:47 | |
*** jerrygb has quit IRC | 05:53 | |
*** jerrygb has joined #openstack-ironic | 05:57 | |
*** hshiina has joined #openstack-ironic | 06:04 | |
*** Nisha_away has quit IRC | 06:12 | |
*** ChubYann has quit IRC | 06:18 | |
*** jerrygb has quit IRC | 06:22 | |
*** jerrygb has joined #openstack-ironic | 06:27 | |
*** Nisha_away has joined #openstack-ironic | 06:30 | |
*** Nisha_brb has joined #openstack-ironic | 06:31 | |
*** Nisha_away has quit IRC | 06:35 | |
*** moshele has joined #openstack-ironic | 06:40 | |
*** zhangyufei has quit IRC | 06:42 | |
*** zhangyufei has joined #openstack-ironic | 06:43 | |
*** yuanying has quit IRC | 06:46 | |
*** moshele has quit IRC | 06:46 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/python-ironicclient: Trivial fix: Unified formats of log_xxx function https://review.openstack.org/376343 | 06:49 |
*** yuanying has joined #openstack-ironic | 06:49 | |
*** rcernin has joined #openstack-ironic | 06:50 | |
*** mjura has joined #openstack-ironic | 07:04 | |
*** jerrygb has quit IRC | 07:06 | |
*** jerrygb has joined #openstack-ironic | 07:10 | |
*** mjura has quit IRC | 07:11 | |
*** ohamada has joined #openstack-ironic | 07:11 | |
*** mjura has joined #openstack-ironic | 07:11 | |
*** rbartal has joined #openstack-ironic | 07:13 | |
*** tesseract has joined #openstack-ironic | 07:17 | |
*** tesseract is now known as Guest48712 | 07:17 | |
*** Nisha_brb has quit IRC | 07:28 | |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Enable upper requirements on bifrost https://review.openstack.org/391828 | 07:29 |
*** fragatina has joined #openstack-ironic | 07:30 | |
*** fragatina has quit IRC | 07:39 | |
*** fragatina has joined #openstack-ironic | 07:39 | |
*** jerrygb has quit IRC | 07:42 | |
*** jerrygb has joined #openstack-ironic | 07:46 | |
*** openstackgerrit has quit IRC | 07:48 | |
*** openstackgerrit has joined #openstack-ironic | 07:48 | |
*** Nisha_brb has joined #openstack-ironic | 07:50 | |
*** jerrygb_ has joined #openstack-ironic | 07:50 | |
*** ccamacho has joined #openstack-ironic | 07:51 | |
*** snozhka has quit IRC | 07:51 | |
*** jerrygb has quit IRC | 07:52 | |
*** moshele has joined #openstack-ironic | 07:53 | |
*** snozhka has joined #openstack-ironic | 07:55 | |
*** jerrygb has joined #openstack-ironic | 07:56 | |
*** jerrygb_ has quit IRC | 07:58 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:01 | |
*** jerrygb has quit IRC | 08:08 | |
*** pcaruana has joined #openstack-ironic | 08:09 | |
openstackgerrit | Merged openstack/ironic-inspector: Switch func3 environment to Python 3.5 https://review.openstack.org/391818 | 08:12 |
*** jerrygb has joined #openstack-ironic | 08:13 | |
*** Nisha_brb has quit IRC | 08:14 | |
*** maratoid_ has quit IRC | 08:22 | |
*** Nisha_away has joined #openstack-ironic | 08:26 | |
*** jerrygb_ has joined #openstack-ironic | 08:27 | |
*** athomas has joined #openstack-ironic | 08:28 | |
*** xek__ is now known as xek | 08:28 | |
*** jerrygb has quit IRC | 08:29 | |
xek | morning :) | 08:31 |
*** mgoddard has joined #openstack-ironic | 08:37 | |
*** skazi has quit IRC | 08:42 | |
openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: OSC add capability to remove node/chassis_uuid https://review.openstack.org/391812 | 08:42 |
*** skazi has joined #openstack-ironic | 08:42 | |
*** jaosorior has joined #openstack-ironic | 08:45 | |
*** jpich has joined #openstack-ironic | 08:47 | |
*** lucas-afk is now known as lucasagomes | 08:51 | |
lucasagomes | morning all | 08:51 |
Nisha_away | lucasagomes, MORNING | 09:06 |
openstackgerrit | Zhou Ya proposed openstack/ironic-python-agent: Get numa_node info when collecting pci devices info https://review.openstack.org/369245 | 09:06 |
*** jerrygb_ has quit IRC | 09:08 | |
*** jerrygb has joined #openstack-ironic | 09:12 | |
*** jaosorior has quit IRC | 09:14 | |
*** jaosorior has joined #openstack-ironic | 09:14 | |
*** Nisha_away has quit IRC | 09:15 | |
*** Nisha_away has joined #openstack-ironic | 09:16 | |
*** dougs1 has joined #openstack-ironic | 09:17 | |
milan | morning lucasagomes, Nisha_away, xek! :) | 09:19 |
*** dtantsur|afk is now known as dtantsur | 09:21 | |
dtantsur | Morning Ironic! | 09:21 |
milan | morning dtantsur! :) | 09:23 |
* milan relocates | 09:25 | |
*** derekh has joined #openstack-ironic | 09:26 | |
*** milan has quit IRC | 09:26 | |
openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: OSC add capability to set chassis-uuid for node https://review.openstack.org/392495 | 09:32 |
*** galyna has joined #openstack-ironic | 09:34 | |
*** hshiina has quit IRC | 09:35 | |
*** vinsh has joined #openstack-ironic | 09:38 | |
*** vinsh has quit IRC | 09:43 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: API: lookup() ignore malformed MAC addresses https://review.openstack.org/392114 | 09:43 |
*** kirankv has quit IRC | 09:44 | |
*** strigazi_AFK is now known as strigazi | 09:44 | |
Nisha_away | morning dtantsur | 09:45 |
dtantsur | hey Nisha_away, mikal | 09:45 |
*** Nisha_away is now known as Nisha1 | 09:45 | |
dtantsur | ooops, that was for milan :) | 09:45 |
vsaienk0 | morning dtantsur, aarefiev, lucasagomes, sambetts, milan, mgould and all Ironic'ers! | 09:47 |
dtantsur | hey vsaienk0 | 09:47 |
*** jaosorior has quit IRC | 09:48 | |
*** ijw has joined #openstack-ironic | 09:49 | |
*** jaosorior has joined #openstack-ironic | 09:49 | |
*** ijw has quit IRC | 09:49 | |
*** ijw has joined #openstack-ironic | 09:50 | |
*** kirankv has joined #openstack-ironic | 09:52 | |
*** jerrygb has quit IRC | 09:54 | |
*** dprince has joined #openstack-ironic | 09:56 | |
*** jerrygb has joined #openstack-ironic | 09:58 | |
lucasagomes | vsaienk0, hi there! | 10:01 |
*** snecklifter has joined #openstack-ironic | 10:03 | |
*** vmud213 has quit IRC | 10:06 | |
*** vmud213 has joined #openstack-ironic | 10:07 | |
*** jaosorior is now known as jaosorior_lunch | 10:08 | |
*** phuongnh has quit IRC | 10:08 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: ETAG supporting to enhance API evolution https://review.openstack.org/392213 | 10:08 |
zhangyufei | Some one can help review this patch about deploy image doc? https://review.openstack.org/#/c/387082/ | 10:09 |
patchbot | patch 387082 - ironic - Add the way to get the deploy ram disks | 10:09 |
*** milan has joined #openstack-ironic | 10:10 | |
openstackgerrit | Aparna proposed openstack/ironic-specs: Generic way to configure clean step priorites in Automated cleaning https://review.openstack.org/362615 | 10:12 |
milan | morning Ironic! :) | 10:12 |
*** zhangyufei has quit IRC | 10:14 | |
*** Nisha1 has quit IRC | 10:14 | |
*** tuanluong has quit IRC | 10:16 | |
*** e0ne has joined #openstack-ironic | 10:16 | |
*** aparnav has joined #openstack-ironic | 10:17 | |
*** kirankv has quit IRC | 10:17 | |
*** ishant|away has quit IRC | 10:17 | |
*** lazy_prince has quit IRC | 10:17 | |
*** sambetts|afk is now known as sambetts | 10:18 | |
*** e0ne_ has joined #openstack-ironic | 10:18 | |
sambetts | Morning all | 10:18 |
milan | morning sambetts! :) | 10:18 |
dtantsur | morning sambetts, milan! | 10:19 |
sambetts | hey dtantsur, milan | 10:19 |
*** e0ne has quit IRC | 10:21 | |
vsaienk0 | morning sambetts! | 10:21 |
milan | morning vsaienk0! :) | 10:21 |
*** harbor_log is now known as potdirect | 10:22 | |
lucasagomes | sambetts, hey there | 10:23 |
*** ishant|away has joined #openstack-ironic | 10:24 | |
*** kirankv has joined #openstack-ironic | 10:24 | |
*** lazy_prince has joined #openstack-ironic | 10:25 | |
sambetts | o/ lucasagomes | 10:25 |
*** hw_wutianwei has quit IRC | 10:28 | |
*** portdirect has joined #openstack-ironic | 10:34 | |
*** mgould|afk is now known as mgould | 10:36 | |
mgould | morning Ironic! | 10:36 |
mgould | morning milan sambetts vsaienk0 lucasagomes | 10:36 |
lucasagomes | hi there | 10:36 |
*** potdirect has quit IRC | 10:37 | |
dtantsur | morning mgould | 10:37 |
milan | morning mgould! :) | 10:38 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Switch to Neutron Flat network. https://review.openstack.org/340695 | 10:40 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to use FLAT network setup https://review.openstack.org/334382 | 10:40 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add IRONIC_TEST_NETWORK_NAME devstack plugin var https://review.openstack.org/338117 | 10:40 |
*** portdirect has quit IRC | 10:48 | |
*** [1]cdearborn has joined #openstack-ironic | 10:49 | |
openstackgerrit | Merged openstack/ironic: Log currently known iSCSI devices when we retry waiting for iSCSI target https://review.openstack.org/383730 | 10:50 |
*** e0ne has joined #openstack-ironic | 10:54 | |
*** e0ne_ has quit IRC | 10:54 | |
sambetts | vsaienk0: Hey! Did you see my comment on the flat network patch? | 10:58 |
sambetts | vsaienk0: I'm having quite a bit of success getting the simulation to be more like a real neutron deployment | 10:59 |
vsaienk0 | sambetts: I'm sorry. I've missed that | 10:59 |
sambetts | vsaienk0: I have flat networks working with my new way, I'm hoping to try and get vlan networks working today | 11:00 |
sambetts | vsaienk0: it code I have now really simplifys what we have to do in devstack to setup the environment which is soo nice | 11:01 |
sambetts | the code* | 11:01 |
openstackgerrit | Merged openstack/ironic: DRAC get_bios_config() passthru causes exception https://review.openstack.org/391571 | 11:02 |
sambetts | I like the change you made to change the USE_FLAT variable to a string | 11:03 |
*** g3ek has quit IRC | 11:04 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: DRAC get_bios_config() passthru causes exception https://review.openstack.org/392548 | 11:07 |
*** g3ek has joined #openstack-ironic | 11:07 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Remove "dhcp" command from the iPXE template https://review.openstack.org/392552 | 11:09 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Add a missing error check in ipmitool driver's reboot https://review.openstack.org/392553 | 11:10 |
* dtantsur is proactively backporting stuff ^^^ | 11:10 | |
*** portdirect has joined #openstack-ironic | 11:11 | |
dtantsur | aarefiev, ugh, the discovery job seems broken: http://logs.openstack.org/31/389331/2/check/gate-tempest-dsvm-ironic-inspector-discovery/f52e5c2/ | 11:12 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Fix config drive writing script https://review.openstack.org/392557 | 11:16 |
dtantsur | vdrok, mind backporting https://review.openstack.org/#/c/387664/ to stable/newton please? looks like an important thing, I can't cleanly backport it due to merge conflicts | 11:19 |
patchbot | patch 387664 - ironic-lib - Wait for a config drive device to be available (MERGED) | 11:19 |
* dtantsur is done with proactvie backports | 11:20 | |
*** jaosorior_lunch is now known as jaosorior | 11:21 | |
*** hshiina has joined #openstack-ironic | 11:23 | |
* dtantsur -> reboot for updates brb | 11:25 | |
*** daemontool has joined #openstack-ironic | 11:25 | |
*** bfournie has quit IRC | 11:26 | |
*** bfournie has joined #openstack-ironic | 11:26 | |
*** jkilpatr has joined #openstack-ironic | 11:27 | |
vdrok | dtantsur: is it OK if I'll do it tomorrow? On pto today. And morning everyone :) | 11:28 |
lucasagomes | vdrok, totally, if you want I can backport it for ya in a sec | 11:30 |
lucasagomes | vdrok, morning | 11:30 |
vdrok | lucasagomes: thanks! | 11:33 |
*** Administrator__ has quit IRC | 11:34 | |
*** Administrator__ has joined #openstack-ironic | 11:35 | |
mgould | morning vdrok | 11:36 |
mgould | also, what are you doing here? Go and enjoy your day off :-) | 11:36 |
*** zhugaoxiao has joined #openstack-ironic | 11:37 | |
dtantsur | vdrok, morning! tomorrow or any other day. enjoy your pto! | 11:38 |
*** baoli has joined #openstack-ironic | 11:38 | |
vdrok | Morning mgould , that's all because of IRC client on my phone :) morning dtantsur | 11:38 |
*** Administrator__ has quit IRC | 11:40 | |
mat128 | morning everyone | 11:41 |
mgould | morning mat128 | 11:42 |
*** baoli_ has joined #openstack-ironic | 11:42 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Wait for a config drive device to be available https://review.openstack.org/392573 | 11:43 |
lucasagomes | vdrok, ^ (don't look at it today!) | 11:43 |
lucasagomes | there was a merge conflict btw | 11:43 |
*** baoli has quit IRC | 11:45 | |
openstackgerrit | Merged openstack/ironic-ui: Eliminate use of a now obsolete admin basePath variable https://review.openstack.org/388707 | 11:45 |
*** mtanino has joined #openstack-ironic | 11:46 | |
*** mtanino has quit IRC | 11:52 | |
jroll | morning y'all | 11:58 |
sambetts | o/ jroll | 11:58 |
*** jcoufal has joined #openstack-ironic | 11:59 | |
lucasagomes | jroll, morning | 12:02 |
jroll | dtantsur: all those backports are on newton yeah? I can roll through reviewing them | 12:02 |
dtantsur | jroll, yep, all newton. I think mitaka is critical-only right now. also good morning :) | 12:02 |
jroll | cool | 12:03 |
jroll | morning :) | 12:03 |
*** krtaylor has joined #openstack-ironic | 12:03 | |
*** moshele has quit IRC | 12:03 | |
lucasagomes | jroll, yup, I want to backport the infiniband/invalid mac fix to newton as well once it's merged upstream | 12:05 |
* lucasagomes goes get some food | 12:05 | |
*** lucasagomes is now known as lucas-hungry | 12:05 | |
openstackgerrit | Aline Bousquet proposed openstack/ironic: Put nodes in CLEANWAIT during a cleaning reboot https://review.openstack.org/392584 | 12:07 |
jroll | dtantsur: fyi, added to the whiteboard, but this should cover all stable reviews http://bit.ly/2fE0QTj | 12:07 |
dtantsur | cool! | 12:07 |
*** aparnav has quit IRC | 12:13 | |
*** moshele has joined #openstack-ironic | 12:14 | |
*** rbrud has joined #openstack-ironic | 12:15 | |
*** hshiina has quit IRC | 12:16 | |
jroll | dtantsur: +A'd some, one is failing unit tests, lucas' backport needs your +2 | 12:18 |
dtantsur | damn tests, let's make them non-voting | 12:19 |
jroll | heh | 12:19 |
dtantsur | :-P | 12:19 |
milan | morning jroll! :) | 12:19 |
milan | morning mat128, vdrok! :) | 12:19 |
jroll | hey milan | 12:20 |
mgould | morning jroll | 12:21 |
jroll | \o mgould | 12:22 |
*** cdearborn has joined #openstack-ironic | 12:28 | |
*** vikrant has quit IRC | 12:30 | |
*** ohamada has quit IRC | 12:32 | |
*** ohamada has joined #openstack-ironic | 12:33 | |
*** baoli_ has quit IRC | 12:34 | |
*** baoli has joined #openstack-ironic | 12:36 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Remove "dhcp" command from the iPXE template https://review.openstack.org/392552 | 12:39 |
*** jerrygb has quit IRC | 12:42 | |
*** e0ne has quit IRC | 12:42 | |
*** yolanda has quit IRC | 12:44 | |
*** yolanda has joined #openstack-ironic | 12:44 | |
*** [1]cdearborn has quit IRC | 12:44 | |
openstackgerrit | Merged openstack/ironic-python-agent: Fix config drive writing script https://review.openstack.org/392557 | 12:44 |
*** jerrygb has joined #openstack-ironic | 12:45 | |
*** jheroux has joined #openstack-ironic | 12:47 | |
*** hshiina has joined #openstack-ironic | 12:51 | |
*** dougs1 has quit IRC | 12:55 | |
*** dougs1 has joined #openstack-ironic | 12:56 | |
*** ccamacho is now known as ccamacho|lunch | 12:59 | |
*** vmud213 has quit IRC | 13:02 | |
*** snecklifter has quit IRC | 13:03 | |
*** snecklifter has joined #openstack-ironic | 13:03 | |
*** Goneri has joined #openstack-ironic | 13:07 | |
*** rloo has joined #openstack-ironic | 13:08 | |
*** mtanino has joined #openstack-ironic | 13:11 | |
openstackgerrit | Merged openstack/ironic: DRAC get_bios_config() passthru causes exception https://review.openstack.org/392548 | 13:12 |
*** [1]cdearborn has joined #openstack-ironic | 13:16 | |
openstackgerrit | Merged openstack/ironic: Add a missing error check in ipmitool driver's reboot https://review.openstack.org/392553 | 13:16 |
*** ralonsoh has joined #openstack-ironic | 13:20 | |
*** lucas-hungry is now known as lucasagomes | 13:22 | |
*** jkilpatr has quit IRC | 13:27 | |
*** hshiina has quit IRC | 13:28 | |
*** ijw has quit IRC | 13:28 | |
*** absubram has joined #openstack-ironic | 13:29 | |
*** cdearborn has quit IRC | 13:30 | |
pas-ha | do we have somewhere a matrix/table clearly describing what gate job tests what combination of deploy options? | 13:33 |
dtantsur | not really.. | 13:33 |
pas-ha | I'd probably start an etherpad for this | 13:33 |
rloo | pas-ha: ++ it should be a wiki or a doc somewhere, but etherpad is a good start | 13:35 |
rloo | hi and good morning ironic'ers :) | 13:35 |
*** baoli_ has joined #openstack-ironic | 13:36 | |
*** baoli has quit IRC | 13:36 | |
*** baoli has joined #openstack-ironic | 13:37 | |
dtantsur | morning rloo | 13:38 |
*** baoli__ has joined #openstack-ironic | 13:38 | |
dtantsur | I think etherpad is better, provided that we're going to change it soon | 13:38 |
*** jkilpatr has joined #openstack-ironic | 13:39 | |
lucasagomes | rloo, morning | 13:40 |
*** vinsh has joined #openstack-ironic | 13:40 | |
rloo | dtantsur: yes, that's why it is a good start. when (if?) it 'stabilizes', we should move it | 13:40 |
dtantsur | ++ | 13:40 |
rloo | hi lucasagomes, dtantsur! | 13:41 |
*** baoli_ has quit IRC | 13:41 | |
*** baoli has quit IRC | 13:42 | |
*** baoli__ has quit IRC | 13:43 | |
*** baoli has joined #openstack-ironic | 13:44 | |
*** vinsh has quit IRC | 13:44 | |
*** jkilpatr has quit IRC | 13:45 | |
*** ccamacho|lunch is now known as ccamacho | 13:48 | |
*** jaosorior has quit IRC | 13:53 | |
*** mjturek has joined #openstack-ironic | 13:53 | |
*** jkilpatr has joined #openstack-ironic | 13:57 | |
*** baoli has quit IRC | 13:58 | |
mjturek | hamzy: krtaylor: I'm thinking this could be the new table structure for MI so we can support other drivers http://imgur.com/zV9oMlY. thoughts? | 13:58 |
mjturek | dict represents fields in the table | 13:59 |
rloo | morning JayF! Just saw your comment on 392114. wrt Aline's patch 390558. good question. if it needs a microversion bump, then the 'bug' should be an RFE and it needs to be approved, no? | 13:59 |
patchbot | https://review.openstack.org/#/c/390558/ - ironic - Add possibility to remove chassis_uuid from a node | 13:59 |
*** baoli has joined #openstack-ironic | 13:59 | |
*** ppiela has joined #openstack-ironic | 13:59 | |
JayF | rloo: I guess so; tbh I didn't think alineb's patch should require a microversion on initial review | 13:59 |
JayF | rloo: if it's possib | 13:59 |
JayF | *if it's possible to create a node w/o chassis uuid, it should be possible to remove chassis uuid from a node | 14:00 |
rloo | JayF: i'm still coming out of the fog of not having heads down in ironic. not sure yet where i fall wrt needing microversion or not. | 14:00 |
rloo | JayF: what I'm wondering is why we made chassis uuid a mandatory field (this is based on the bug description, i didn't look at the code yet.) | 14:01 |
JayF | I'm not super familiar with our microversion rules, I should go read the spec, but the rule of thumb I use is "if I had an API client, would this change likely break me?" | 14:01 |
JayF | and in the case of both of those patches, I think the answer is clearly no | 14:01 |
dtantsur | this is not how people see microversions (even though I do agree with your definition) | 14:01 |
rloo | JayF: it isn't just that. IF we add a new 'feature', that also requires a microversion bump. | 14:01 |
rloo | JayF: new features don't necessarily break existing setups | 14:02 |
dtantsur | microversions are designed like they're designed for 1. visibility of changes, 2. avoiding breaking changes, 3. to force people to use microversions | 14:02 |
dtantsur | point #2 is clearly not applicable to both cases, #1 and #3 are questionable | 14:03 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: API: lookup() ignore malformed MAC addresses https://review.openstack.org/392114 | 14:03 |
dtantsur | for me the border is if the change actually fixes something broken or rather adds something missing | 14:03 |
milan | morning JayF, rloo! :) | 14:04 |
rloo | hi milan | 14:04 |
JayF | so playing the devils advocate for ^ that patch, wouldn't adding support for non-ethernet macs count as a feature? | 14:04 |
rloo | JayF: it doesn't add support for non-ethernet macs. | 14:04 |
dtantsur | JayF, for what definition of support? | 14:04 |
dtantsur | rloo++ we just stop crashing if they happen to be on the same machine | 14:05 |
rloo | dtantsur: exactly. | 14:05 |
lucasagomes | rloo, JayF I think the reason why chassis_uuid couldn't be removed is just bad design | 14:05 |
JayF | Well but I look back to what caused this error -- when we added support in IPA For fibre channel macs | 14:05 |
dtantsur | like, if we failed deploy for machines with AMD CPU (as opposed to Intel ones), I'd also call it a bug | 14:05 |
dtantsur | JayF, that's not correct. it's caused by lookup API introduction IIRC | 14:05 |
JayF | I don't actually think either change needs a microversion, I'm just trying to make the point that I think it's still pretty subjective :/ | 14:05 |
lucasagomes | that said, I also don't know if we should have a microversion on alines' patch. It sounds like somehting we would like to get fixed across multiple versions (cause it's a bug) | 14:06 |
rloo | JayF: i agree, the microversion thing wasn't that clear. did the api group issue some microversion spec that clarifies it? | 14:06 |
dtantsur | JayF, I've made a strict input check on MAC format, hence it started failing if a NIC is present with a different hw address format. it has not relation to the ongoing Infiniband effort | 14:06 |
* lucasagomes don't like microversion to be a "bug compat" thing | 14:06 | |
dtantsur | also note that if we start version bug fixes, they'll become not backportable | 14:07 |
rloo | lucasagomes: when you say the chassis uuid thing was due to a bad design, what do you mean by that? | 14:07 |
JayF | dtantsur: interesting, I thought it was a change we added to IPA to send back extra macs, but looking at the git log it's clear you're right | 14:07 |
lucasagomes | rloo, I mean that, when we designed the API we didn't understand the relation between the resources very well | 14:07 |
lucasagomes | rloo, the API in general was a great PoC in my view, we didn't know ReST, wsme, etc... at the time | 14:08 |
rloo | lucasagomes: so there isn't anything inherent in our underlying code that makes a chassis uuid mandatory for a node, we just coded it that we wanted it mandatory? | 14:08 |
dtantsur | rloo, JayF, another approach to this: you'll benefit from the MAC fix without any changes to your code; you won't benefit from chassis_uuid fix without actually starting deleting chassis_uuid. hence the discoverability aspec of microversioning kicks in here. | 14:08 |
lucasagomes | rloo, right. I don't think there's anything in the code that depends on chassis uuid being mandatory for nodes | 14:09 |
* dtantsur is not sure whether his comment clarifies things or makes them even more compex | 14:09 | |
JayF | dtantsur: at least it tells me exactly where we're splitting the hair, lol | 14:09 |
lucasagomes | I think that one idea of microversion is so that, the same version across different clouds (different vendors) will work exactly the same way. So, in the chassis_uuid thing (or even the MAC) that won't be true if we do not microversion it | 14:10 |
rloo | JayF: so when I look at the code (via Aline's patch) I see that it is coded explicitly that chassi_uuid is not removable | 14:10 |
rloo | JayF: which, in conjunction to what lucasagomes said, makes me think we/someone made a conscious decision for chassis_uuid to be mandatory | 14:11 |
rloo | JayF: which makes me think it requires a microversion. | 14:11 |
JayF | Yeah, with what dtantsur said I can see where the distinction is now | 14:11 |
rloo | JayF: cuz it was a 'feature', not a bug. | 14:11 |
lucasagomes | but I don't really agree with that goal, being compatible is cool, but being "bug compatible" is dummy | 14:11 |
JayF | I think I'm with lucasagomes in that I'd prefer the distinction be less strict, but I'm OK with whatever as long as I understand it :) | 14:11 |
* rloo worries that we'll get back into the discussion of why we added microversions and i don't want to upset/remind dtantsur of those discussions... | 14:12 | |
dtantsur | heh :) | 14:12 |
* dtantsur has no clue why, but he got used to them | 14:13 | |
lucasagomes | dtantsur, human nature :-) | 14:13 |
*** causten has joined #openstack-ironic | 14:13 | |
*** baoli has quit IRC | 14:15 | |
pas-ha | started etherpad describing tested combinations per gate job https://etherpad.openstack.org/p/ironic-gate-jobs-described (WiP) | 14:15 |
*** baoli has joined #openstack-ironic | 14:15 | |
*** baoli_ has joined #openstack-ironic | 14:16 | |
rloo | thx pas-ha. would you mind sending out email to the devlist about that etherpad? | 14:16 |
pas-ha | sure | 14:16 |
*** e0ne has joined #openstack-ironic | 14:19 | |
*** baoli has quit IRC | 14:20 | |
JayF | rloo: so can we swap over the bug for aline's patch to an rfe and approve it? Did you wanna dig deeper into it first? | 14:22 |
pas-ha | rloo: done | 14:23 |
rloo | JayF: give me about 5 more minutes; just looking at existing code now. | 14:23 |
rloo | pas-ha: thx! | 14:23 |
JayF | perfect, going to go finish making breakfast then :D | 14:23 |
*** daemontool has quit IRC | 14:31 | |
*** daemontool has joined #openstack-ironic | 14:32 | |
*** baoli_ has quit IRC | 14:32 | |
*** anush has quit IRC | 14:36 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clarify ironic governance requirements and process https://review.openstack.org/392685 | 14:36 |
jroll | ^^ happy fun time | 14:37 |
jroll | I need all cores on that at some point, also sent an email | 14:37 |
*** vinsh has joined #openstack-ironic | 14:39 | |
vsaienk0 | dtantsur: are you planning to work on https://review.openstack.org/#/c/357983? | 14:41 |
patchbot | patch 357983 - ironic - Support names for {cleaning,provisioning}_network (ABANDONED) | 14:41 |
dtantsur | vsaienk0, maybe later. feel free to overtake, if you want to | 14:41 |
*** jaosorior has joined #openstack-ironic | 14:41 | |
rloo | dtantsur, lucasagomes, JayF: wrt ability to delete node.chassis_uuid, are you all for it? I am updating the bug/RFE, just need to put down who is for it so I can approve it. | 14:41 |
JayF | +2 | 14:42 |
dtantsur | +2 | 14:42 |
lucasagomes | rloo, I'm good with being able to delete node.chassis_uuid. +1 | 14:42 |
*** dmellado is now known as dmellado|lunch | 14:42 | |
*** zackf has joined #openstack-ironic | 14:42 | |
jroll | +2 | 14:43 |
rloo | thx lucasagomes. oh, i didn't even see the other +2s. thx all! | 14:43 |
jroll | :D | 14:43 |
rloo | process-related question, jroll etc. if the RFE touches python-ironicclient, we add that project to the bug too, right? vs creating a new bug? | 14:44 |
jroll | rloo: yeah, that's good | 14:44 |
*** wajdi has joined #openstack-ironic | 14:46 | |
openstackgerrit | Félix Bouliane proposed openstack/ironic: DONOTMERGE-change IRONIC_DEPLOY_DRIVER to pxe_snmp https://review.openstack.org/388168 | 14:46 |
*** mgoddard_ has joined #openstack-ironic | 14:47 | |
*** wajdi_ has joined #openstack-ironic | 14:48 | |
*** mgoddard has quit IRC | 14:50 | |
*** wajdi has quit IRC | 14:51 | |
*** mtanino has quit IRC | 14:52 | |
*** mgoddard_ has quit IRC | 14:53 | |
*** jjohnson2 has quit IRC | 14:53 | |
*** mtanino has joined #openstack-ironic | 14:53 | |
*** mgoddard_ has joined #openstack-ironic | 14:53 | |
*** dougs1 has quit IRC | 14:53 | |
*** jjohnson2 has joined #openstack-ironic | 14:54 | |
*** sai has quit IRC | 14:55 | |
*** dougs1 has joined #openstack-ironic | 14:55 | |
*** syed_ has joined #openstack-ironic | 14:55 | |
vsaienk0 | rloo, dtantsur, sambetts, jroll: https://review.openstack.org/#/c/390558/ is in conflict with Portgroup API patch, can we block it until we land portgroup API patch? it is pretty big and already has two +2 | 14:56 |
patchbot | patch 390558 - ironic - Add possibility to remove chassis_uuid from a node | 14:56 |
openstackgerrit | Merged openstack/ironic-lib: Wait for a config drive device to be available https://review.openstack.org/392573 | 14:56 |
jroll | vsaienk0: ++ | 14:57 |
*** yufei has joined #openstack-ironic | 14:57 | |
jroll | vsaienk0: -1'd with a note | 14:58 |
*** daemontool has quit IRC | 14:58 | |
*** sai has joined #openstack-ironic | 14:58 | |
vsaienk0 | thanks jroll! | 14:59 |
rloo | vsaienk0: thx for reminder. portgroups is on my high priority list. i shouldn't have gotten distracted! :) | 14:59 |
*** jcoufal has quit IRC | 14:59 | |
*** jcoufal has joined #openstack-ironic | 15:00 | |
*** rbartal has quit IRC | 15:00 | |
rloo | vsaienk0: do you have a link to the portgroup patches? | 15:00 |
rloo | vsaienk0: i see this from trello, not sure it is correct: https://review.openstack.org/#/q/status:open+branch:master+topic:bug/1526403 | 15:01 |
vsaienk0 | rloo: first patch https://review.openstack.org/#/c/347549/ | 15:01 |
patchbot | patch 347549 - ironic - Add PortGroups API | 15:01 |
*** mgoddard_ has quit IRC | 15:02 | |
*** mgoddard has joined #openstack-ironic | 15:02 | |
rloo | vsaienk0: thx. that patch shows 5 other related changes, yet when i look at the topic, i see https://review.openstack.org/#/q/topic:bug/1618754 | 15:03 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: API: lookup() ignore malformed MAC addresses https://review.openstack.org/392114 | 15:04 |
rloo | vsaienk0: and do you have any idea wrt the link that is in trello? should I delete that? | 15:04 |
vsaienk0 | rloo: we decided to track portgroups by separate RFE 1618754. Filtering by topic:bug/1618754 should return a list of all related portgroup patches | 15:04 |
*** moshele has left #openstack-ironic | 15:05 | |
vsaienk0 | rloo: I don't have write access to trello, if you have please update it to https://review.openstack.org/#/q/topic:bug/1618754 | 15:05 |
*** moshele has joined #openstack-ironic | 15:05 | |
rloo | vsaienk0: ok, will do... | 15:05 |
hamzy | mjturek, are you proposing using four different tables? hrm. I am almost finished with a patch that just uses one field of user supplied data | 15:08 |
openstackgerrit | Merged openstack/ironic: Remove "dhcp" command from the iPXE template https://review.openstack.org/392552 | 15:09 |
JayF | So is there a reason we haven't landed portgroup API patch yet? Myself and Yuriy have +2 on it | 15:09 |
*** e0ne has quit IRC | 15:09 | |
JayF | maybe some folks could take a look today and we can land that? | 15:09 |
rloo | JayF: i'll look today. (or you could land it if you want) | 15:10 |
JayF | Honestly, for API changes I'm always a little bit happier if more folks get a look | 15:10 |
JayF | I don't like to be the deciding core vote on those :) | 15:10 |
mjturek | hamzy: well, a table per driver (not all at once, just as support is needed), and a join table so you can get the right info from the right table. It's more complex, but it's a better structure | 15:11 |
mjturek | hamzy: but I suppose having json blobs is fine for now | 15:11 |
hamzy | mjturek, What is the end goal of molteniron? Provide locking with data or complex database operations? | 15:12 |
mjturek | hamzy: locking with data... but that doesn't mean we shouldn't strive for a well structured DB | 15:13 |
*** snozhka has quit IRC | 15:13 | |
*** snozhka has joined #openstack-ironic | 15:14 | |
hamzy | mjturek, I don't have a lot of experience in databases. I can understand the concept of locking one row in one table. In your picture the Nodes table is the one being locked? | 15:15 |
hamzy | and all of the other tables are queryable? | 15:15 |
mgould | mjturek, hamzy: are you going to query on the contents of those JSON blobs? | 15:16 |
*** srobert has joined #openstack-ironic | 15:16 | |
mgould | IME storing JSON in strings in an RDBMS is a fertile source of pain | 15:16 |
hamzy | mgould, my initial rewrite looks like this http://paste.openstack.org/show/587653/ | 15:17 |
*** anush has joined #openstack-ironic | 15:17 | |
hamzy | the status field is the lock | 15:17 |
mjturek | hamzy: yeah it's the same flow | 15:17 |
rloo | hey dtantsur: wrt https://review.openstack.org/#/c/390558/3//COMMIT_MSG, i thought we used partial-bug for all patches related to an rfe/bug? or do we just do it per project per bug? (now i am wondering if in past rfes, we've added the client as a separate project) | 15:18 |
patchbot | patch 390558 - ironic - Add possibility to remove chassis_uuid from a node | 15:18 |
mjturek | mgould: I don't think we need to query the data in the JSON, we just need to get it | 15:18 |
mgould | OK, that's probably fine | 15:18 |
hamzy | mgould, is there problems with encoding and corruption when storing JSON in a database? | 15:18 |
mgould | that, and it makes your query logic awful and slow | 15:19 |
hamzy | mgould, is there a safe datastore for JSON that is also simple? | 15:20 |
mgould | queries that would be a nice efficient join turn into a mess of string parsing | 15:20 |
NobodyCam | Good Morning Ironic'ers | 15:20 |
mgould | morning NobodyCam | 15:20 |
NobodyCam | morning mgould :) | 15:21 |
*** e0ne has joined #openstack-ironic | 15:21 | |
mgould | hamzy: sadly, I don't have a good answer to that in our case | 15:21 |
hamzy | understood | 15:21 |
mgould | recent versions of Postgres (which we can't rely on in Ironic) support native JSON queries | 15:21 |
mgould | I don't know enough about the NoSQL space to answer your safety concerns | 15:22 |
milan | morning NobodyCam! :) | 15:22 |
hamzy | because that postgress would have to run and be supported on all the platforms, right? | 15:22 |
mgould | it probably does, but we can't rely on it being there | 15:22 |
mgould | recent versions of MySQL also support native JSON queries (in an incompatible way, of course) | 15:23 |
NobodyCam | morning milan | 15:23 |
mgould | when this came up a few months ago, the conclusion was that we couldn't rely on users having an up-to-date MySQL install | 15:23 |
mgould | I suspect at our scale (say a few tens of thousands of rows?) most NoSQL databases are safe enough :-) | 15:24 |
mgould | but yeah, CouchDB or ElasticSearch are designed for searching arbitrary JSON blobs efficiently | 15:28 |
*** sandriichenko has joined #openstack-ironic | 15:28 | |
*** jcoufal has quit IRC | 15:28 | |
mgould | and if you know the structure in advance, it's quite possibly worth expressing it as a relational schema | 15:29 |
mgould | [IME, as a developer and not an operator :-)] | 15:29 |
mariojv | rloo: do you or lintan want help with any particular piece of rolling upgrades? looking at https://review.openstack.org/#/c/306357 it seems like the basic framework is underway, maybe i can help with unit tests or docs? | 15:29 |
patchbot | patch 306357 - ironic - [PoC]Ironic rolling upgrade framework - Make servi... | 15:29 |
rloo | mariojv: so xek is going to take the lead on rolling upgrades. he might appreciate help on unit tests or docs, dunno. | 15:30 |
mariojv | alternatively maybe jlvill-travel / vdrok / vsaienk0 need help with multinode grenade for that? i don't know much about it but am willing to learn :) | 15:30 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/bifrost: Updated from global requirements https://review.openstack.org/389876 | 15:30 |
mariojv | ok, cool | 15:30 |
rloo | mariojv: i think we would welcome more folks knowing about grenade etc :) | 15:30 |
*** moshele has quit IRC | 15:32 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/392730 | 15:32 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/392731 | 15:32 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/389896 | 15:32 |
openstackgerrit | Szymon Borkowski proposed openstack/ironic-python-agent: Add a new Hardware Manager for X710 network card https://review.openstack.org/391836 | 15:34 |
*** jcoufal has joined #openstack-ironic | 15:34 | |
*** sabeen has joined #openstack-ironic | 15:36 | |
*** g3ek has quit IRC | 15:36 | |
*** harlowja_at_home has joined #openstack-ironic | 15:38 | |
*** mmedvede has quit IRC | 15:38 | |
*** g3ek has joined #openstack-ironic | 15:39 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/389938 | 15:40 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/392820 | 15:40 |
dtantsur | rloo, sorry for the delay. closes-bug closes only one project side of the bug | 15:40 |
dtantsur | rloo, so if you plan on only one patch for ironic repo, you should use closes-bug | 15:40 |
dtantsur | no matter how many projects are added to the bug | 15:40 |
dtantsur | otherwise the ironic part of the bug will stay opened, and I'll have to manually close it one day :) | 15:41 |
rloo | dtantsur: gotcha. | 15:42 |
*** rama_y has joined #openstack-ironic | 15:42 | |
* lucasagomes still can't get his head around the project-config yaml's | 15:44 | |
dtantsur | lucasagomes, haha, I can imagine. need some help? | 15:45 |
mgould | lucasagomes: anything I can help with? | 15:45 |
mgould | snap | 15:45 |
hamzy | mgould, thanks for the input! | 15:45 |
mgould | hamzy: np, sorry for the wall of text | 15:45 |
lucasagomes | dtantsur, mgould hah thanks. Lemme try to get the first version of the patch up and I may shout for help if it fails | 15:46 |
lucasagomes | I'm switching the jobs to use ipmitool instead of ssh btw | 15:46 |
dtantsur | aha, that's fun | 15:46 |
*** yufei has quit IRC | 15:47 | |
*** dmellado|lunch is now known as dmellado | 15:47 | |
dtantsur | lucasagomes, tl;dr: jenkins/jobs/ironic defines job templates, jenkins/jobs/projects creates actual jobs definitions from these templates, zuul/layout insert jobs into project's gate | 15:47 |
lucasagomes | dtantsur, thanks that helps! | 15:48 |
lucasagomes | sometimes I just feel like s/pxe_ssh/pxe_ipmitool | 15:48 |
lucasagomes | but that won't work I think | 15:48 |
lucasagomes | heh | 15:48 |
mgould | lucasagomes: try it and see what breaks :-) | 15:49 |
dtantsur | yeah, you need duplicate jobs to be able to run different ones on stable/{liberty,mitaka} and newer | 15:49 |
dtantsur | lucasagomes, make sure to move new jobs to xenial btw.. they'll request it anyway from you | 15:49 |
lucasagomes | dtantsur, we should do it separately, right ? | 15:49 |
lucasagomes | I've seem you changed the inspector ones in the git logs | 15:50 |
*** mmedvede has joined #openstack-ironic | 15:50 | |
dtantsur | lucasagomes, well... if you do it separately, you'll have to duplicate jobs for trusty and xenial first :) | 15:50 |
* dtantsur is not sure we haven't done it already though | 15:50 | |
lucasagomes | oh dear | 15:50 |
mgould | dtantsur: move to xenial, then move to pxe_ipmitool? | 15:50 |
dtantsur | mgould++ | 15:50 |
dtantsur | lucasagomes, I think we first move ipmitool jobs to xenial, if they're not there already | 15:51 |
dtantsur | lucasagomes, then we switch ipmitool to voting and shut down ssh jobs | 15:51 |
lucasagomes | dtantsur, mgould yeah, sounds more sane | 15:51 |
* lucasagomes does it | 15:51 | |
*** krtaylor has quit IRC | 15:54 | |
*** sabeen has quit IRC | 15:56 | |
*** snecklifter has quit IRC | 16:01 | |
*** [1]cdearborn has quit IRC | 16:02 | |
*** harlowja_at_home has quit IRC | 16:03 | |
*** ohamada has quit IRC | 16:04 | |
*** rbartal has joined #openstack-ironic | 16:04 | |
*** ohamada has joined #openstack-ironic | 16:05 | |
*** ohamada has quit IRC | 16:05 | |
*** pcaruana has quit IRC | 16:06 | |
*** sabeen has joined #openstack-ironic | 16:06 | |
*** rbrud has quit IRC | 16:06 | |
*** mgoddard_ has joined #openstack-ironic | 16:07 | |
*** rcernin has quit IRC | 16:09 | |
*** anush has quit IRC | 16:10 | |
*** mgoddard has quit IRC | 16:10 | |
*** sabeen has quit IRC | 16:10 | |
*** sabeen has joined #openstack-ironic | 16:11 | |
hwoarang | TheJulia: cinerama: any idea about the recent "No module named vine.five" ci failures? | 16:15 |
*** rpioso has joined #openstack-ironic | 16:16 | |
*** causten has quit IRC | 16:17 | |
hwoarang | plus, this error in the ci job is interesting: "2016-11-02 16:06:44.579114 | mkdir: cannot create directory ‘/opt/stack’: Permission denied" | 16:17 |
hwoarang | http://logs.openstack.org/79/389079/2/check/gate-bifrost-integration-tinyipa/5402386/console.html | 16:17 |
*** causten has joined #openstack-ironic | 16:18 | |
*** causten_ has joined #openstack-ironic | 16:19 | |
*** causten has quit IRC | 16:22 | |
*** baoli has joined #openstack-ironic | 16:22 | |
mat128 | hwoarang: https://bugs.launchpad.net/ceilometer/+bug/1638323 | 16:26 |
openstack | Launchpad bug 1638323 in Ceilometer "gate fails reporting 'No module named vine.five'" [Undecided,In progress] - Assigned to Hanxi Liu (hanxi-liu) | 16:26 |
*** jkilpatr has quit IRC | 16:26 | |
mat128 | hwoarang: tl;dr, pin kombu<4.0 | 16:26 |
hwoarang | oh thanks mat128 | 16:26 |
*** jaosorior has quit IRC | 16:28 | |
*** mjura has quit IRC | 16:33 | |
*** Guest48712 has quit IRC | 16:34 | |
*** mmedvede has quit IRC | 16:37 | |
*** rbrud has joined #openstack-ironic | 16:37 | |
*** jkilpatr has joined #openstack-ironic | 16:39 | |
*** e0ne has quit IRC | 16:40 | |
*** baoli has quit IRC | 16:42 | |
*** baoli has joined #openstack-ironic | 16:42 | |
*** aNuposic has joined #openstack-ironic | 16:44 | |
*** mmedvede has joined #openstack-ironic | 16:45 | |
jroll | so jlvill-travel is out and I don't see krtaylor, maybe we skip the qa meeting today? | 16:46 |
* dtantsur wants to skip too | 16:47 | |
dtantsur | (like if I ever attended, lol) | 16:47 |
*** e0ne has joined #openstack-ironic | 16:48 | |
jroll | heh | 16:50 |
jroll | "dtantsur will skip per usual" | 16:50 |
dtantsur | right | 16:51 |
dtantsur | well, after I committed to do some CI refactoring, I should probably start attending | 16:51 |
jroll | fair enough | 16:51 |
dtantsur | also it's now 6pm, not 7pm (ditto for v2 meeting) | 16:51 |
jroll | \o/ | 16:52 |
JayF | I am pretty sure that when DST hits in America (this weekend) | 16:52 |
JayF | I'm going to keep the same working hours in UTC | 16:52 |
JayF | My wife typically gets home from teaching around 3:30pm, and that would put me getting off around 4pm, so we get a longer afternoon and I have incentive to not waste so much time in the morning :) | 16:53 |
*** kvcobb has joined #openstack-ironic | 16:53 | |
dtantsur | hah, great | 16:53 |
dtantsur | I work random hours | 16:53 |
JayF | I work a very consistent 8-5 (1500-2400 UTC), sometimes leaving early if I start early or skip taking a lunch break | 16:54 |
JayF | My favorite thing about the job right now is how consistent the hours are | 16:54 |
dtantsur | 8am is deep night for me | 16:54 |
JayF | I used to be a stay-up-late-wake-up-late guy, but when I got married, and she kept a strict early bed/early rise schedule, I decided to not fight it :) | 16:55 |
dtantsur | hah, I see. we're both night owls, soooo | 16:55 |
*** shobhar_ has joined #openstack-ironic | 16:56 | |
JayF | Yeah; makes sense. I'm usually in bed by 9p, asleep by 10p | 16:56 |
JayF | I know for most folks that's just when they're getting started :D | 16:56 |
dtantsur | well.. I try not to do any work after 9pm :) | 16:57 |
*** milan has quit IRC | 16:57 | |
*** krtaylor has joined #openstack-ironic | 16:58 | |
* mgould tries to be in bed by 2330, but often fails | 17:01 | |
*** causten_ has quit IRC | 17:01 | |
*** mgoddard has joined #openstack-ironic | 17:02 | |
*** mgoddard_ has quit IRC | 17:02 | |
*** causten has joined #openstack-ironic | 17:02 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: ETAG supporting to enhance API evolution https://review.openstack.org/392213 | 17:03 |
dtantsur | leaving early today, see you tomorrow | 17:06 |
*** dtantsur is now known as dtantsur|afk | 17:06 | |
JayF | o/ | 17:07 |
*** causten has quit IRC | 17:10 | |
lucasagomes | not sure if it's enough but: https://review.openstack.org/#/c/392890/ | 17:11 |
patchbot | patch 392890 - openstack-infra/project-config - Switch ironic-ipa jobs to Xenial | 17:11 |
JayF | I'll review lucas | 17:11 |
lucasagomes | JayF, thanks, I mean it's my first attempt on it. Not sure what other jobs we should switch as well (if not all) | 17:12 |
JayF | that looks sane to me | 17:12 |
JayF | lucasagomes: the only thing I'm curious about is if that would impact stable jobs at all | 17:12 |
JayF | lucasagomes: aiui all our jobs should be running xenial at this point | 17:12 |
lucasagomes | yeah, good point on the stables... let's see if I can figure it out | 17:13 |
JayF | It's really hard to know exactly what you're impacting when changing those jobs | 17:13 |
JayF | zuul v3 will make it a lot more clear b/c we can actually make changes in-tree and see them take effect | 17:14 |
mgould | JayF: hurrah! | 17:14 |
*** rbudden has joined #openstack-ironic | 17:14 | |
lucasagomes | oh that would be perfect. That's exactly my fear because I see inspector there, I see granade I don't wanna touch those at the moment | 17:14 |
lucasagomes | (running tox locally worked for the templates with my changes) | 17:15 |
JayF | lucasagomes: xenial is what is going to be supported for pike though, so we gotta get all our jobs working there | 17:15 |
JayF | I'm wondering why we have the strange mix we have | 17:15 |
JayF | i.e. why py34/35 jobs aren't running on xenial yet | 17:16 |
JayF | lucasagomes: that's another thing; all the jobs running xenial have -xenial in the name; idk if we'd want to continue that | 17:16 |
lucasagomes | JayF, yeah or functional... or why postgres have xenial and trusty there | 17:16 |
JayF | ayup | 17:16 |
JayF | might be worth an email to the mailing list to get a grip on what is transitioning where | 17:17 |
lucasagomes | JayF, re: name... no idea | 17:17 |
JayF | and if there were known issues that are tribally known but not documented | 17:17 |
lucasagomes | vsaienk0, it was you that hit some problems with xenial ? | 17:17 |
mgould | dtantsur|afk spent a lot of time fighting with it, IIRC | 17:18 |
* lucasagomes remembers the MTU thingy | 17:18 | |
mgould | lucasagomes: https://review.openstack.org/384652 | 17:19 |
patchbot | patch 384652 - ironic - Fix broken xenial job (MERGED) | 17:19 |
*** harlowja has quit IRC | 17:21 | |
*** kvcobb has quit IRC | 17:25 | |
*** e0ne has quit IRC | 17:27 | |
*** e0ne has joined #openstack-ironic | 17:28 | |
*** e0ne has quit IRC | 17:28 | |
*** e0ne has joined #openstack-ironic | 17:29 | |
lucasagomes | right, I will hunt reviews/expert on that and working on changing the jobs to use ipmitool tomorrow | 17:31 |
lucasagomes | calling it a day all, have a great evening! | 17:31 |
JayF | o/ have a good evening | 17:31 |
mgould | lucasagomes: gnite! | 17:31 |
*** lucasagomes is now known as lucas-afk | 17:32 | |
anish | dtantsur|afk: can anyone else do the second review for all the python-dracclient patches ? | 17:34 |
mgould | anish: looking now | 17:36 |
*** [1]cdearborn has joined #openstack-ironic | 17:36 | |
*** milan has joined #openstack-ironic | 17:37 | |
*** baoli has quit IRC | 17:38 | |
*** e0ne has quit IRC | 17:39 | |
* mgould had totally forgotten he was now a core on python-dracclient | 17:39 | |
mgould | mwahahaha | 17:39 |
anish | heh, thanks mgould ! | 17:40 |
mgould | I took over from ifarkas, but the handover was quite abrupt | 17:40 |
*** fragatina has quit IRC | 17:40 | |
openstackgerrit | Oleksandr Berezovskyi proposed openstack/ironic-python-agent: CPU inspection now supports multiple processors https://review.openstack.org/391290 | 17:40 |
*** ccamacho is now known as ccamacho|out | 17:42 | |
*** harlowja has joined #openstack-ironic | 17:44 | |
mgould | anish: all +W'ed, thanks for the patches! | 17:46 |
anish | man, that's some velocity :) | 17:47 |
mgould | they were all pretty simple, and dtantsur|afk had already +2ed them :-) | 17:47 |
*** jpich has quit IRC | 17:47 | |
anish | related question : I am adding sensor + draccard profile support to python-dracclient | 17:47 |
anish | my code's ready and working | 17:47 |
mgould | cool | 17:48 |
anish | but I can't find the the schedule for ironic on spec/blueprint submissions | 17:48 |
mgould | we decided at the summit that python-dracclient is being dropped from the ironic project's governance | 17:49 |
mgould | so it doesn't actually matter what ironic does here | 17:49 |
anish | ohh, interesting | 17:49 |
anish | does that remove the spec requirement or just the schedule requirements | 17:49 |
mgould | er, dunno | 17:49 |
openstackgerrit | Oleksandr Berezovskyi proposed openstack/ironic-python-agent: CPU inspection now supports multiple processors https://review.openstack.org/391290 | 17:50 |
mgould | I think it means that the python-dracclient cores can now do whatever the hell we like? | 17:50 |
*** rbartal has quit IRC | 17:50 | |
anish | does it get replaced by python-redfish or something, or is it just being spun out of ironic ? | 17:50 |
TheJulia | Ironic does not have an official freeze, so their is no sxhwdule | 17:50 |
*** athomas has quit IRC | 17:50 | |
mgould | TheJulia: ah, thanks | 17:50 |
mgould | anish: yes to both, IIRC | 17:50 |
mgould | one moment... | 17:50 |
TheJulia | Additionally sub-projects could always have their own cores :) | 17:51 |
mgould | anish: https://etherpad.openstack.org/p/ironic-ocata-summit-contributor-meetup | 17:51 |
*** athomas has joined #openstack-ironic | 17:51 | |
TheJulia | anish: think of it this way. If the code in the ironic repo talking to python-dracclient requires additions or changes for features, then a spec would be required. | 17:52 |
mgould | anish: docs patch explaining this here: https://review.openstack.org/#/c/392685/ | 17:52 |
patchbot | patch 392685 - ironic - Clarify ironic governance requirements and process | 17:52 |
*** aNuposic has quit IRC | 17:52 | |
anish | TheJulia: thanks will submit a spec, that was my original plan anyways | 17:52 |
mgould | anish: cool, I look forward to reading it | 17:53 |
*** aNuposic has joined #openstack-ironic | 17:53 | |
mgould | but the python-dracclient side can be submitted without a spec, I think | 17:53 |
TheJulia | It can | 17:53 |
mgould | \o/ | 17:53 |
anish | oh I misread that | 17:53 |
TheJulia | Well, more than anything that is up to the cores over that repo | 17:53 |
anish | faster patches ! | 17:53 |
anish | so is there a future-redfish-plans doc somewhere ? The only reason I started looking at python-dracclient was dell redfish support was woeful | 17:54 |
mgould | heh | 17:54 |
anish | apparently that's changed with the 2.40 drac release | 17:54 |
anish | (or so they claim) | 17:54 |
* mgould hasn't looked | 17:55 | |
TheJulia | anish: there is a spec in review for redfish support | 17:55 |
TheJulia | bcornec has been championing that effort | 17:55 |
mgould | https://review.openstack.org/#/c/184653/ | 17:55 |
patchbot | patch 184653 - ironic-specs - Propose Redfish support into Ironic | 17:55 |
*** anush has joined #openstack-ironic | 17:55 | |
*** derekh has quit IRC | 17:55 | |
TheJulia | Thank you mgould | 17:56 |
anish | oh man, how did I miss this | 17:56 |
mgould | anish: AIUI it's something we'd very much like to have in Ironic, but neither the protocol itself nor the implementations are quite ready yet | 17:56 |
mgould | anish: there's a lotta stuff going on, you can't pay attention to it all :-) | 17:56 |
mgould | more reviews on that spec always welcome, of course | 17:57 |
anish | my experience with redfish was it's similar but not the same between vendors. | 17:57 |
mgould | yeah | 17:57 |
mgould | bcornec gave a talk at the summit in which he mentioned a few key bits which were still vendor-defined | 17:58 |
TheJulia | More than anything, we need a way to do CI testing for a redfish driver which is the true blocker, although I think we will see rapid progress on that | 17:58 |
mgould | yeah | 17:58 |
mgould | TheJulia: would it be enough to test against the reference virtual implementation? | 17:59 |
mgould | or do we need to test against actual redfish-supporting hardware? | 17:59 |
anish | iirc, that is incomplete no (assuming you mean simulator) | 17:59 |
JayF | mgould: the rule is that it has to have ci | 17:59 |
JayF | mgould: if it's emulated ci; that might be sufficient | 17:59 |
TheJulia | mgould: for power/boot control, the consensus that I encountered at the summit was yes. More advanced things like vendor specific things would need to be bolt-ons with appropriate CI though | 18:00 |
mgould | JayF: yeah, I think DMTF provide a docker container running a redfish server that emulates a BMC | 18:00 |
TheJulia | They do | 18:00 |
mgould | anish: https://www.openstack.org/summit/barcelona-2016/summit-schedule/events/16231/empowering-ironic-with-redfish-support | 18:00 |
anish | mgould: I think that was devananda | 18:01 |
mgould | anish: sorry, what was? | 18:01 |
anish | https://github.com/devananda/python-redfish | 18:01 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 18:01 |
anish | the simulator | 18:01 |
TheJulia | bcornec: perhaps a weekly meeting or call, or something to push it forward | 18:01 |
*** ralonsoh has quit IRC | 18:01 | |
mgould | anish: that looks like a client library to me, or does that contain the simulator as well? | 18:02 |
* TheJulia gets out laptop to type faster | 18:02 | |
anish | mgould: yeah, the layout is just confusing https://github.com/devananda/python-redfish/tree/master/dmtf | 18:02 |
mgould | OK, thanks | 18:03 |
anish | bcornec: very interested in any sort of regular meetings. we're doing python-dracclient and our own redfish work internally, I would rather do it for openstack | 18:03 |
mgould | anish: OK, looks like it includes a binary of the simulator and some code to unpack it into a docker container | 18:04 |
anish | ayup | 18:04 |
mgould | but anyway, that's the thing I meant :-) | 18:04 |
mgould | video of bcornec's talk: https://www.youtube.com/watch?v=KxRo5cRpj6k | 18:05 |
TheJulia | What I suggested, which is horrible, but hopefully minimally sufficent was to put up a rest api similar to virtualbmc's ipmi server, and only implement the endpoints required for testing the node. | 18:05 |
mgould | that's me asking annoying questions about vendor extensions at the end | 18:05 |
anish | haha | 18:05 |
anish | vendor extensions are the worst | 18:05 |
TheJulia | anish: fyi, bcornec is based out of UTC+1 | 18:05 |
TheJulia | tl;dr we're leaving him messages in his irc client that he will see tomorrow :) | 18:06 |
anish | spain I'm guessing ? | 18:06 |
mgould | France, I think | 18:06 |
TheJulia | Yup, France | 18:07 |
mgould | TheJulia: so write our own Redfish BMC emulator implementing the subset of the API we care about? | 18:07 |
TheJulia | mgould: bingo | 18:07 |
anish | or strip out the unwanted parts of the API ? | 18:08 |
anish | the simulator just reads a giant json iirc and serves it back | 18:08 |
TheJulia | We would actually want to wire the redfish api to control a local VM through virsh | 18:08 |
TheJulia | so /redfish/v1/servers/testvm1 could map to testvm1 in virsh | 18:08 |
* mgould wonders if we could add the functionality into virtualbmc | 18:08 | |
TheJulia | mgould: that ;) | 18:09 |
mgould | yeah, that sounds relatively doable | 18:09 |
* mgould hasn't looked deeply at the virtualbmc code, though | 18:09 | |
mgould | https://github.com/umago/virtualbmc, for those following along | 18:10 |
TheJulia | The library it leverages, if memory serves, makes it rather easy since I started prototyping something similar before lucas created virtualbmc | 18:10 |
* anish is following curiously | 18:11 | |
TheJulia | or https://github.com/openstack/virtualbmc | 18:11 |
mgould | ah, cool | 18:11 |
*** athomas has quit IRC | 18:11 | |
mgould | OK, so it subclasses pyghmi.ipmi.bmc and makes libvirt calls to simulate the effects of each IPMI command | 18:14 |
*** anush has quit IRC | 18:14 | |
*** jvgrant_ has joined #openstack-ironic | 18:15 | |
anish | hrm, the virtualbmc part is already part of the spec | 18:16 |
anish | someone was thinking ahead ! | 18:16 |
mgould | always nice when that happens :-) | 18:16 |
* TheJulia smiles | 18:16 | |
*** openstackgerrit has quit IRC | 18:18 | |
TheJulia | I really want to see redfish support get in. I also chatted with openbmc folk in BCN, I know they were going to talk with bcornec some because standardization == ++good | 18:18 |
anish | same here, at oscon | 18:18 |
*** openstackgerrit has joined #openstack-ironic | 18:18 | |
mgould | hell yes | 18:18 |
mgould | anish: FWIW, the python-dracclient cores are listed at https://review.openstack.org/#/admin/groups/1032,members | 18:18 |
anish | they seem to have stalled a bit though, they were almost there last time I talked to them | 18:18 |
*** jvgrant has quit IRC | 18:18 | |
anish | TheJulia: was it shenki or redbeard ? | 18:19 |
TheJulia | openbmc already has a virsh wrapper for their rest api though :) | 18:19 |
*** jkilpatr has quit IRC | 18:19 | |
TheJulia | anish: I'm horrible mapping names to faces, and then remembering names. It was one of the folks at IBM | 18:19 |
anish | TheJulia: I was just about to ask if someone has done the reverse for virsh | 18:19 |
anish | I think they rely heavily on qemu for their testing/development so the virsh wrapper makes total sense | 18:20 |
TheJulia | there was also chatter of possibly doing the same as redfish, putting up an a driver in ironic for openbmc, but it really seems like there is a desire to head in a common direction | 18:21 |
mgould | TheJulia: does OpenBMC define its own protocol? | 18:22 |
TheJulia | mgould: So, based on the ad-hoc discussion where I introduced people, my impression was that they did | 18:22 |
mgould | bah | 18:23 |
mgould | yeah, looks like it: https://github.com/facebook/openbmc/blob/master/common/recipes-rest/rest-api/rest-api_0.2.bb | 18:24 |
anish | wrong openbmc https://github.com/openbmc/openbmc | 18:25 |
mgould | gah | 18:25 |
anish | (not the same) | 18:25 |
anish | because, why make things less confusing | 18:25 |
anish | I understood the openbmc thing differently. they basically concluded they need to support redfish to not be obsolete | 18:26 |
mgould | aha: https://github.com/openbmc/docs/blob/master/rest-api.md | 18:26 |
*** anush has joined #openstack-ironic | 18:26 | |
mgould | right, /me -> home | 18:29 |
mgould | good night, everybody! | 18:29 |
mgould | anish: clearly whoever got there second should rebrand to LibreBMC | 18:29 |
TheJulia | goodnight mgould | 18:29 |
anish | night mgould | 18:30 |
mgould | goodnight TheJulia anish | 18:30 |
*** mgould is now known as mgould|afk | 18:30 | |
* TheJulia ponders coffee | 18:31 | |
* anish sips coffee stares at TheJulia | 18:31 | |
anish | TIL, /me swallows commas | 18:32 |
* TheJulia now really kind of wants coffee, but wonders about commas instead. | 18:32 | |
TheJulia | anish: I think it may be your client | 18:32 |
*** jkilpatr has joined #openstack-ironic | 18:32 | |
anish | possibly, it's an old version of irssi | 18:33 |
TheJulia | Weird... | 18:34 |
*** shobhar_ has quit IRC | 18:37 | |
*** pester has quit IRC | 18:39 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clarify ironic governance requirements and process https://review.openstack.org/392685 | 18:40 |
* jroll chugs coffee | 18:40 | |
*** shobhar_ has joined #openstack-ironic | 18:42 | |
JayF | fwiw facebook sponsors openbmc | 18:43 |
JayF | so I suspect those two projects might be the same thing, just at different points in time | 18:44 |
sambetts | is openBMC linked with the opencompute hardware stuff ? | 18:44 |
anish | from what I heard from the openbmc ppl last time I met them, not so | 18:44 |
* TheJulia awaits for rendered doc output from CI | 18:44 | |
JayF | sambetts: openbmc is used by some opencompute platforms, such as barreleye (the openpower stuff) | 18:45 |
JayF | sambetts: but not actually affiliated | 18:45 |
*** baoli has joined #openstack-ironic | 18:45 | |
sambetts | ah cool | 18:45 |
JayF | When I was at the OCP summit about a year ago or so | 18:45 |
sambetts | I looked into some of the opencompute chassis stuff at a summit or two back and it was pretty cool | 18:45 |
JayF | I talked to them about redfish and they barely knew what I was talking about | 18:45 |
JayF | so nice to hear they've come around | 18:45 |
JayF | sambetts: all onmetal servers are open compute :) | 18:45 |
JayF | sambetts: so I've worked with them in production :D | 18:46 |
jroll | since when is 'opencompute in production' a :D | 18:46 |
jroll | :P | 18:46 |
jroll | (I kid, they've been pretty good to us) | 18:46 |
JayF | jroll: destroy-proprietary-control (of the datacenteR) | 18:46 |
JayF | lol | 18:46 |
jroll | ++ | 18:46 |
sambetts | oh cool, like the ones with all the harddrives laying down in them? | 18:46 |
JayF | there are multiple different ocp designs | 18:47 |
anish | so many | 18:47 |
JayF | We run 3 different brands | 18:47 |
sambetts | oh :/ I have no idea which ones I saw then | 18:49 |
sambetts | somethink like this https://mguerri.web.cern.ch/mguerri/2015-01-21-OCP/img/wiwynn-jbod.png | 18:49 |
sambetts | something* | 18:49 |
*** portdirect is now known as portdirect_away | 18:51 | |
*** yuanying has quit IRC | 19:03 | |
*** milan has quit IRC | 19:10 | |
*** bfournie has quit IRC | 19:10 | |
*** Goneri has quit IRC | 19:10 | |
*** anush has quit IRC | 19:11 | |
*** moshele has joined #openstack-ironic | 19:13 | |
*** bfournie has joined #openstack-ironic | 19:13 | |
*** bfournie has quit IRC | 19:18 | |
*** jerrygb has quit IRC | 19:21 | |
*** jerrygb has joined #openstack-ironic | 19:22 | |
*** ChubYann has joined #openstack-ironic | 19:22 | |
*** sabeen has quit IRC | 19:24 | |
*** jerrygb has quit IRC | 19:26 | |
*** mmedvede has quit IRC | 19:28 | |
*** moshele has quit IRC | 19:30 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Test discovery job https://review.openstack.org/392943 | 19:36 |
*** jerrygb has joined #openstack-ironic | 19:42 | |
*** wajdi_ has quit IRC | 19:43 | |
*** anush has joined #openstack-ironic | 19:44 | |
*** bcornec1 has quit IRC | 19:45 | |
*** rcernin has joined #openstack-ironic | 19:45 | |
*** bcornec1 has joined #openstack-ironic | 19:45 | |
TheJulia | bcornec: FYI, I replied to the redfish spec again | 19:47 |
*** sabeen has joined #openstack-ironic | 19:47 | |
*** bfournie has joined #openstack-ironic | 19:52 | |
openstackgerrit | Merged openstack/ironic: API: lookup() ignore malformed MAC addresses https://review.openstack.org/392114 | 19:55 |
openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 19:57 |
*** openstack has joined #openstack-ironic | 19:59 | |
*** yolanda has quit IRC | 19:59 | |
*** shobhar_ has quit IRC | 20:01 | |
*** mmedvede has joined #openstack-ironic | 20:04 | |
*** Goneri has joined #openstack-ironic | 20:04 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Test discovery job https://review.openstack.org/392943 | 20:04 |
*** anush has quit IRC | 20:05 | |
*** dprince has quit IRC | 20:08 | |
*** devananda has quit IRC | 20:12 | |
rama_y | Hello, I have a question about /opt/stack/data/ironic-inspector/ramdisk-logs | 20:15 |
JayF | Go ahead and ask | 20:15 |
*** e0ne has joined #openstack-ironic | 20:16 | |
rama_y | JayF: when does a new entry get created for a particular node in ramdisk-logs? | 20:16 |
JayF | What do you mean by a new entry? | 20:16 |
rama_y | I have updated a node with new ramdisk and kernel images and I expected to see a new log entry | 20:16 |
*** jtomasek has quit IRC | 20:17 | |
rama_y | I see several tar.gz files per node uuid | 20:17 |
*** openstackgerrit has quit IRC | 20:18 | |
rama_y | I am trying to figure out if a new entry is created per node whenever its images are updated or when node is inspected. | 20:18 |
JayF | https://github.com/openstack/ironic-inspector/blob/de04f7c5a0d768f54d2b20e78005638d7dba5781/ironic_inspector/process.py#L46 is the code in ironic-inspector that writes those logs out | 20:18 |
JayF | it looks like they're named based on node uuid | 20:18 |
JayF | and it looks like old logs are getting truncated | 20:18 |
*** openstackgerrit has joined #openstack-ironic | 20:18 | |
JayF | so it would be the same logfile with different contents, unless I'm misreading this | 20:18 |
rama_y | I have several entries per node uuid with different time stamps. Oh I see, these are just continued in different files, basically. | 20:19 |
*** e0ne has quit IRC | 20:20 | |
JayF | yeah; I mean, it's pretty straightforward how it's handled in the code | 20:20 |
JayF | I'm not 100% sure what you're asking but I think it may be answered? | 20:20 |
*** e0ne has joined #openstack-ironic | 20:20 | |
rama_y | JayF: I am actually interested in the var/log/ironic-python-agent.log | 20:21 |
*** jtomasek has joined #openstack-ironic | 20:21 | |
*** e0ne has quit IRC | 20:21 | |
*** e0ne has joined #openstack-ironic | 20:21 | |
rama_y | I do not see new date stamp after I updated the node with new images and did a node inspection in the ironic-python-agent.log | 20:21 |
*** e0ne has quit IRC | 20:21 | |
rama_y | What am I missing? | 20:21 |
JayF | So you had a node | 20:22 |
rama_y | yes | 20:22 |
JayF | you updated the ramdisk and kernel to include (I'm assuming) some changes you made | 20:22 |
rama_y | yes | 20:22 |
JayF | re-inspected, and you didn't see a new set of logs get added in the ramdisk-logs dir? | 20:22 |
rama_y | yes | 20:22 |
JayF | Did you check the ironic-bm-logs/ logs for that node? The actual console output? | 20:22 |
JayF | and read the logs to see if anything unexpected or different happened compared to other inspections you've done | 20:23 |
rama_y | I checked the console output; it showed node going into introspection | 20:23 |
JayF | Can you paste those logs? | 20:25 |
JayF | into a pastebin or something and link them here | 20:25 |
rama_y | sure | 20:25 |
rama_y | paste.openstack.org/show/587690/ | 20:29 |
*** jkilpatr has quit IRC | 20:30 | |
JayF | So that pretty clearly looks like inspection didn't work on the node | 20:30 |
JayF | because there are no logs there | 20:30 |
rama_y | This shows no bootable device; | 20:30 |
JayF | does the node in ironic show the same way? | 20:30 |
JayF | like does it show as inspection failed? | 20:30 |
*** rbudden has quit IRC | 20:30 | |
rama_y | It shows inspect failed | 20:30 |
JayF | okay; and the only change you made was building your own ramdisk and putting the informatino about it in your node | 20:31 |
JayF | so most likely that somehow got messed up | 20:31 |
*** devananda has joined #openstack-ironic | 20:31 | |
JayF | do you wanna verify that those ramdisk and kernel options are mapped to the proper glance images? | 20:31 |
rama_y | Yes; prior to that, I set the node's maintenance mode to false. | 20:31 |
rama_y | I will stop and start ironic-inspector and see if it helps. | 20:32 |
JayF | that will not likely make any difference | 20:32 |
JayF | lets just do a basic verificatino of the environment -- on one of the nodes you haven't changed anything about the ramdisk at all on, lets try to inspect it and see if it succeeds | 20:33 |
rama_y | Ok, I am inspecting another node that is in manageable state. | 20:34 |
*** sabeen has quit IRC | 20:35 | |
*** moshele has joined #openstack-ironic | 20:36 | |
rama_y | JayF: all nodes are failing inspection; in node-cache there is database integrity error. | 20:38 |
JayF | rama_y: I've never heard of a database integrity error in node-cache? Where are you seeing that error? Can you paste the logs? | 20:38 |
openstackgerrit | Sam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 20:38 |
rama_y | sure | 20:39 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add ability to time out on individual clean steps https://review.openstack.org/391554 | 20:42 |
rama_y | paste.openstack.org/show/587692/ | 20:42 |
*** moshele has quit IRC | 20:42 | |
*** aNuposic has quit IRC | 20:43 | |
JayF | rama_y: interesting; I'm not sure if that's to be expected or not | 20:44 |
JayF | rama_y: it looks at a glance like it's just trying to re-insert the same info twice | 20:44 |
JayF | in the log it says bmc_address-127.0.0.1 rather than bmc_address=127.0.0.1 | 20:44 |
JayF | idk if that's supposed to be that way or indicates some kind of problem though | 20:45 |
*** srobert has quit IRC | 20:45 | |
rama_y | I don't know either; I may have sent several nodes into inspection in succession | 20:47 |
JayF | that should be a fine thing to do | 20:47 |
JayF | it seems like somehow your environment has gotten messed up; I'd unstack/restack and then make sure you can inspect a node without making changes | 20:47 |
rama_y | Sure, thanks JayF. | 20:48 |
*** jkilpatr has joined #openstack-ironic | 20:50 | |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Fix functional tests after recent API changes https://review.openstack.org/390018 | 20:52 |
*** sabeen has joined #openstack-ironic | 20:53 | |
*** shobhar_ has joined #openstack-ironic | 20:53 | |
*** krtaylor has quit IRC | 20:55 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Allow to configure cache clean up https://review.openstack.org/392943 | 20:55 |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Allow to configure cache clean up period https://review.openstack.org/392943 | 20:56 |
*** Sukhdev has joined #openstack-ironic | 20:57 | |
*** shobhar_ has quit IRC | 21:02 | |
JayF | So question about inspection | 21:03 |
JayF | apparently setting deploy_ramdisk and deploy_kernel only impacts cleaning/deployment and not inspection, at least from some testing aslezil and I did | 21:03 |
JayF | Are there separate settings fro inspection ramdisk? Where/what are they? I can't find anything about it in documentation | 21:04 |
sambetts | JayF: inspection has its own files in the tftp root | 21:04 |
JayF | sambetts: so if I want to modify the ramdisk a node uses in inspection | 21:04 |
*** jtomasek has quit IRC | 21:04 | |
JayF | sambetts: what's the right setting to modify on the node? | 21:04 |
JayF | sambetts: and where does whatever the default is come from? | 21:04 |
sambetts | JayF: there isn't a way at the moment we're working on making it work like that, currently the same ramdisk is used for all nodes and its stored in the tftp root, and the dhcp/kernel pxe params are all set in the inspector specfic dnsmasq/pxe settings | 21:06 |
JayF | sambetts: so basically you don't even have to upload ramdisks for inspection to glance at all? | 21:06 |
JayF | you just yolo copy them into the tftproot? | 21:06 |
sambetts | yep | 21:06 |
JayF | that's ... surprising | 21:06 |
JayF | aslezil: rama_y ^ that's relevant to what both of you are working on | 21:06 |
JayF | sambetts: Man, that's almost impossible to figure out as someone with ironic but not inspector knowledge :( I couldn't find it documented anywhere either | 21:07 |
aslezil | thanks JayF and sambetts :) | 21:07 |
openstackgerrit | Michael Turek proposed openstack/ironic-specs: Add spec for OpenBMC power module, management module, and driver. https://review.openstack.org/388218 | 21:07 |
sambetts | yeah we're working on making it possible for the inspection interface to use the prepare_ramdisk function on the boot/deploy interfaces | 21:07 |
JayF | perfect | 21:07 |
JayF | aslezil: good hunch on proving your code was actually running | 21:07 |
JayF | sambetts: you should meet aslezil -- annie is working on adding some inspector collectors, and right now is figuring out how it all works before writing her first spec | 21:08 |
sambetts | JayF: I think all we have is here: http://docs.openstack.org/developer/ironic-inspector/install.html#configuring-pxe | 21:09 |
aslezil | sambetts that would be great if you could help me. | 21:09 |
JayF | yeah, reading that made me come in here and ask | 21:09 |
JayF | Sam is probably going to help you the most by reviewing your specs and code :) | 21:09 |
JayF | so bribe him at all opportunities :P | 21:09 |
JayF | (the best way to bribe openstack contributors is with quality contributions and code review, lol) | 21:10 |
aslezil | he he | 21:10 |
*** e0ne has joined #openstack-ironic | 21:10 | |
*** e0ne has quit IRC | 21:10 | |
sambetts | Hi aslezil, whats your timezone?, its like 21:00 for me right now so I'm not normally on this late, I just happened to still have IRC open ;) I'm in the UK so tend to be around in UTC time | 21:10 |
*** e0ne has joined #openstack-ironic | 21:11 | |
rama_y | Thanks JayF, sambetts. I was looking at the tftpboot as well | 21:11 |
*** e0ne has quit IRC | 21:11 | |
*** e0ne has joined #openstack-ironic | 21:11 | |
*** e0ne has quit IRC | 21:12 | |
*** e0ne has joined #openstack-ironic | 21:12 | |
*** e0ne has quit IRC | 21:13 | |
*** e0ne has joined #openstack-ironic | 21:13 | |
aslezil | sambetts: I am in CDT(Texas). I am trying to debug and understand the hankshake between IPA and inspector to write my spec for adding a collector in Ironic Inspector. | 21:13 |
*** e0ne has quit IRC | 21:14 | |
*** e0ne has joined #openstack-ironic | 21:14 | |
*** e0ne has quit IRC | 21:14 | |
rama_y | sambetts: will tinyipa kernel, ramdisk images work in the tftproot? | 21:16 |
sambetts | yup :) | 21:16 |
rama_y | Thanks. | 21:16 |
rama_y | JayF: quick question about placing 'logs' collector at the end of all the collectors. | 21:17 |
JayF | rama_y: go ahead | 21:17 |
rama_y | Does this need to be checked in ironic-inspector/devstack/plugin.sh? | 21:18 |
rama_y | And/Or ironic.conf [pxe] pxe_append_params= as well? | 21:19 |
*** bfournie has quit IRC | 21:19 | |
sambetts | aslezil: so right now there isn't really a "handshake" as such, the way inspection works is that we have a default DHCP rule that PXE boots IPA onto anything not blocked by the inspector firewall rules, we pass the inspectors IP address into IPA via the kernel parameters, IPA runs its collectors and then sends all the collected data to the inspector data collection endpoint, at that point the | 21:19 |
sambetts | ramdisk is finished and inspector doesn't call back to it, all data processing then happens on the inspector server side | 21:19 |
sambetts | this is unlike the ironic use case for the ramdisk where there is handshake via the lookup and heartbeat endpoints | 21:20 |
JayF | rama_y: I think those collectors are set in the ironic-inspector devstack plugin, if you wanted to change the order of those collectors you'd do it there and restack | 21:21 |
rama_y | JayF: Thanks! | 21:21 |
sambetts | the collectors to run are configured via the PXE kernel parameters which are set by the devstack plugin | 21:21 |
aslezil | sambetts:thank you for detailed explaination :) | 21:22 |
rama_y | sambetts: the devstack plugin of Ironic Inspector, right? | 21:22 |
sambetts | rama_y: correct | 21:22 |
sambetts | aslezil: No problem :D if you have any other questions feel free to message me on here async and I'll pick them up in the morning | 21:23 |
* sambetts -> sleep | 21:23 | |
*** sambetts is now known as sambetts|afk | 21:23 | |
sambetts|afk | night aslezil, rama_y, JayF | 21:23 |
JayF | nn | 21:23 |
rama_y | Thanks sambetts}afk | 21:23 |
aslezil | good night sambetts | 21:23 |
mrda | Morning Ironic | 21:27 |
mariojv | morning mrda | 21:28 |
*** jrcloud has joined #openstack-ironic | 21:35 | |
Sukhdev | vsaienk0 : Ping | 21:35 |
*** zackf has quit IRC | 21:37 | |
Sukhdev | vsaienk0 : when you get on-line, please ping me | 21:38 |
*** jcoufal has quit IRC | 21:38 | |
openstackgerrit | Oleksandr Berezovskyi proposed openstack/ironic-inspector: Added support of new CPU data format https://review.openstack.org/392976 | 21:41 |
mrda | mariojv: o/ | 21:43 |
*** jrcloud has quit IRC | 21:45 | |
*** jheroux has quit IRC | 21:46 | |
*** jrcloud has joined #openstack-ironic | 21:50 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/392730 | 21:53 |
*** e0ne has joined #openstack-ironic | 21:54 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/389938 | 22:00 |
*** harlowja has quit IRC | 22:02 | |
*** rbrud has quit IRC | 22:14 | |
rama_y | Hi JayF, aslezil: so, to inspect a node with a specific ramdisk image, we need to have that image copied to tftpboot/master_images? | 22:36 |
JayF | rama_y: it's not possible to set the inspection image per ndoe, you have to replace it globally | 22:36 |
JayF | and I believe that's where it is | 22:36 |
JayF | aslezil should be able to confirm as I think she's testing it right now | 22:36 |
rama_y | Ok | 22:38 |
aslezil | JayF rama_y I am trying to find the path to add the images but not able to find it | 22:39 |
JayF | so lets backtrack it then, right? | 22:40 |
JayF | aslezil: you remember where the ipxe config is for ironic-inspector? That should have a url to the ramdisk/kernel you're downloading | 22:40 |
rama_y | aslezil, JayF: in plugin.sh, there are two parameters: IRONIC_AGENT_KERNEL_URL and IRONIC_AGENT_RAMDISK_URL | 22:40 |
JayF | rama_y: that's the URL that devstack will download the agent from if you don't build it. It's not relevant here since you're building your own agent. | 22:41 |
rama_y | Oh I see | 22:41 |
JayF | so basically I think the images are going to be in $IRONIC_HTTP_DIR/ironic-inspector.[kernel|initramfs] | 22:41 |
aslezil | that's what rama_y mentioned /opt/stack/data/ironic/tftpboot/master_image | 22:42 |
*** Goneri has quit IRC | 22:42 | |
JayF | are there no ironic-inspector.* files in there? | 22:42 |
JayF | https://github.com/openstack/ironic-inspector/blob/master/devstack/plugin.sh#L128 | 22:42 |
JayF | points to IRONIC_HTTP_SERVER | 22:42 |
JayF | let me see how that's configured; what dir it's using for webroot | 22:42 |
*** maratoid_ has joined #openstack-ironic | 22:42 | |
aslezil | JayF: I didn't find any | 22:42 |
JayF | https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L479 looks like here is where it configures apache | 22:43 |
JayF | using a template from the devstack files dir | 22:43 |
JayF | so going to trace backwards from there | 22:43 |
JayF | https://github.com/openstack/ironic/blob/master/devstack/files/apache-ironic.template is the template, but HTTPROOT is a variable, so what is it replaced with | 22:44 |
JayF | looks like $IRONIC_HTTP_DIR from the previous link | 22:44 |
JayF | https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L247 says IRONIC_HTTP_DIR=${IRONIC_HTTP_DIR:-$IRONIC_DATA_DIR/httpboot} | 22:44 |
*** syed_ has quit IRC | 22:45 | |
JayF | so instead of /opt/stack/data/ironic/tftpboot, try /opt/stack/data/ironic/httpboot/ | 22:45 |
JayF | aslezil: rama_y ^ follow how I got there? And I bet in there you'll see the ironic-inspector.* files | 22:45 |
JayF | as a warning, something I saw when reading this devstack code is that on unstack/restack that directory gets cleared, so make sure any ramdisks you put in there are backed up so you can have them even if you unstack/restack | 22:45 |
rama_y | JayF, aslezil: yes | 22:46 |
aslezil | JayF: these files are found ther | 22:46 |
JayF | so that's the ramdisk and kernel you have to replace with the custom ones you built | 22:46 |
JayF | to get new code testing in inspection | 22:46 |
JayF | I'd backup the originals just so you can flip it back to 'stock' in case you need to see if a problem is caused by your code or by something else | 22:46 |
aslezil | JayF you just copy the new images ther right? | 22:46 |
JayF | The file you would've uploaded to glance as the aki (kernel image) would be copied to ironic-inspector.kernel | 22:47 |
JayF | the ari (ramdisk image) would be copied to ironic-inspector.initramfs | 22:47 |
JayF | then if you inspect you should get the new code | 22:47 |
rama_y | So, we need to first delete the .initramfs and .kernel from httpboot first and then create new glance images? | 22:48 |
JayF | so ironic-inspector ramdisk images are not managed by glance | 22:48 |
JayF | that's what we learned today | 22:48 |
aslezil | JayF sorry I am wrong the tiny IPA images have te kernal as .vimlinux and ramdisk as .gz files right? | 22:48 |
JayF | so all you ahve to do is put your ramdisk and kernel over those files | 22:48 |
JayF | aslezil: basically; ignore the extensions; just do something like mv ironic-inspector.kernel ironic-inspector.kernel.orig && mv /path/to/my/tinyipa.vmlinuz ironic-inspector.kernel | 22:49 |
JayF | and then the same for ironic-inspector.ramdisk + tinyipa.gz | 22:49 |
*** ccamacho|out has quit IRC | 22:50 | |
aslezil | JayF the ramdisk and kernel files should be as .initramfs and .kernal | 22:50 |
JayF | aslezil: yes; because otherwise you break those urls in the .ipxe config file | 22:51 |
aslezil | JayF I got it | 22:51 |
rama_y | Thanks JayF | 22:51 |
openstackgerrit | Bruno Cornec proposed openstack/ironic-specs: Propose Redfish support into Ironic https://review.openstack.org/184653 | 22:51 |
JayF | np :) happy to help | 22:51 |
aslezil | JayF Thank you :) | 22:51 |
JayF | I am calling it a day in about 10 minutes though, so if there's anything else speak soon ;) | 22:51 |
openstackgerrit | Oleksandr Berezovskyi proposed openstack/ironic-inspector: Added support of new CPU data format https://review.openstack.org/392976 | 22:52 |
aslezil | JayF good night zzzzzzzzz | 22:52 |
JayF | hah; idk about sleep, but getting chores done and then watching game 7 of the world series THEN maybe sleep :P | 22:52 |
aslezil | he he | 22:53 |
rama_y | Good night, aslezil; thanks very much, JayF! | 22:53 |
JayF | no problem | 22:53 |
JayF | I'm getting to learn how inspector works which was a big gap in my knowledge | 22:53 |
JayF | it's always fun to learn new stuff | 22:53 |
aslezil | Good night rama_y | 22:53 |
*** shobhar has joined #openstack-ironic | 22:54 | |
*** g3ek has quit IRC | 22:57 | |
*** g3ek has joined #openstack-ironic | 22:58 | |
*** david-lyle has quit IRC | 23:04 | |
*** harlowja has joined #openstack-ironic | 23:08 | |
*** absubram has quit IRC | 23:09 | |
*** sabeen has quit IRC | 23:11 | |
*** ppiela has quit IRC | 23:12 | |
openstackgerrit | Oleksandr Berezovskyi proposed openstack/ironic-python-agent: Add possibility to inspect systems with multiple CPUs https://review.openstack.org/391290 | 23:16 |
openstackgerrit | Oleksandr Berezovskyi proposed openstack/ironic-inspector: Added support of multiple CPU inspection data https://review.openstack.org/392976 | 23:18 |
*** mtanino has quit IRC | 23:20 | |
*** e0ne has quit IRC | 23:24 | |
*** david-lyle has joined #openstack-ironic | 23:30 | |
*** rcernin has quit IRC | 23:31 | |
*** pmannidi_ has joined #openstack-ironic | 23:36 | |
*** awiddersheim has quit IRC | 23:38 | |
*** pmannidi has quit IRC | 23:38 | |
*** rpioso has quit IRC | 23:43 | |
*** awiddersheim has joined #openstack-ironic | 23:51 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!