| *** rama_y has quit IRC | 00:01 | |
| *** Nisha1 has joined #openstack-ironic | 00:02 | |
| *** causten has joined #openstack-ironic | 00:04 | |
| *** aNuposic has quit IRC | 00:04 | |
| *** Nisha1 has quit IRC | 00:06 | |
| *** harlowja has joined #openstack-ironic | 00:11 | |
| *** rpioso has quit IRC | 00:12 | |
| *** shobhar__ has quit IRC | 00:14 | |
| *** Sukhdev has quit IRC | 00:24 | |
| *** links has joined #openstack-ironic | 00:27 | |
| *** hamzy has joined #openstack-ironic | 00:30 | |
| *** zhugaoxiao has quit IRC | 00:30 | |
| *** zhugaoxiao has joined #openstack-ironic | 00:30 | |
| *** amotoki has joined #openstack-ironic | 00:41 | |
| *** hoangcx has joined #openstack-ironic | 00:44 | |
| *** hoangcx_ has joined #openstack-ironic | 00:45 | |
| *** hoangcx_ has quit IRC | 00:45 | |
| *** hoangcx has quit IRC | 00:45 | |
| *** hoangcx has joined #openstack-ironic | 00:46 | |
| *** causten has quit IRC | 00:51 | |
| *** cdearborn has quit IRC | 01:00 | |
| *** phuongnh has joined #openstack-ironic | 01:03 | |
| *** jkilpatr has quit IRC | 01:07 | |
| *** tuanluong has joined #openstack-ironic | 01:10 | |
| *** moshele has joined #openstack-ironic | 01:13 | |
| *** moshele has quit IRC | 01:23 | |
| *** jerrygb has joined #openstack-ironic | 01:26 | |
| *** jerrygb_ has quit IRC | 01:27 | |
| *** jerrygb_ has joined #openstack-ironic | 01:32 | |
| *** jerrygb has quit IRC | 01:35 | |
| *** jerrygb_ has quit IRC | 01:39 | |
| *** jerrygb has joined #openstack-ironic | 01:41 | |
| *** syed_ has quit IRC | 01:45 | |
| *** jerrygb has quit IRC | 01:49 | |
| *** baoli has joined #openstack-ironic | 01:49 | |
| *** jerrygb has joined #openstack-ironic | 01:50 | |
| *** yibo has joined #openstack-ironic | 01:58 | |
| *** baoli has quit IRC | 02:05 | |
| *** chlong has joined #openstack-ironic | 02:06 | |
| *** jerrygb has quit IRC | 02:12 | |
| *** jerrygb has joined #openstack-ironic | 02:13 | |
| *** shuquan has joined #openstack-ironic | 02:17 | |
| *** jerrygb has quit IRC | 02:29 | |
| *** jerrygb has joined #openstack-ironic | 02:30 | |
| *** jerrygb_ has joined #openstack-ironic | 02:42 | |
| *** jerrygb has quit IRC | 02:44 | |
| gcb | dtantsur, please revisit https://review.openstack.org/#/c/389513/ when you're free :-) | 02:44 |
|---|---|---|
| patchbot | patch 389513 - ironic - Use function is_valid_mac from oslo.utils | 02:44 |
| *** jerrygb has joined #openstack-ironic | 02:47 | |
| *** jerrygb_ has quit IRC | 02:50 | |
| *** rbudden has joined #openstack-ironic | 02:52 | |
| *** Sukhdev has joined #openstack-ironic | 02:57 | |
| *** khamtamtun has joined #openstack-ironic | 02:59 | |
| *** rbudden_ has joined #openstack-ironic | 03:10 | |
| *** rbudden has quit IRC | 03:10 | |
| *** rbudden_ is now known as rbudden | 03:10 | |
| *** shuquan has quit IRC | 03:19 | |
| *** aparnav has joined #openstack-ironic | 03:21 | |
| *** shuquan has joined #openstack-ironic | 03:31 | |
| *** Nisha_away has joined #openstack-ironic | 03:51 | |
| *** khamtamtun has quit IRC | 03:58 | |
| *** rbudden has quit IRC | 03:58 | |
| *** khamtamtun has joined #openstack-ironic | 03:59 | |
| *** shuquan has quit IRC | 04:00 | |
| *** khamtamtun has quit IRC | 04:00 | |
| *** shuquan has joined #openstack-ironic | 04:02 | |
| *** vinsh has quit IRC | 04:09 | |
| *** baoli has joined #openstack-ironic | 04:17 | |
| *** shobhar has joined #openstack-ironic | 04:20 | |
| *** baoli has quit IRC | 04:22 | |
| *** rbrud has quit IRC | 04:24 | |
| *** jerrygb has quit IRC | 04:24 | |
| *** shuquan has quit IRC | 04:27 | |
| *** khamtamtun has joined #openstack-ironic | 04:36 | |
| *** khamtamtun has quit IRC | 04:45 | |
| *** khamtamtun has joined #openstack-ironic | 04:50 | |
| *** livelace has joined #openstack-ironic | 04:57 | |
| *** Sukhdev has quit IRC | 05:00 | |
| *** khamtam25 has joined #openstack-ironic | 05:14 | |
| *** khamtamtun has quit IRC | 05:14 | |
| *** hshiina has joined #openstack-ironic | 05:18 | |
| hshiina | morning ironic | 05:18 |
| *** shuquan has joined #openstack-ironic | 05:23 | |
| *** g3ek has quit IRC | 05:35 | |
| *** khamtam25 has quit IRC | 05:40 | |
| *** g3ek has joined #openstack-ironic | 05:42 | |
| *** shobhar has quit IRC | 05:46 | |
| *** khamtamtun has joined #openstack-ironic | 05:52 | |
| *** shuquan has quit IRC | 05:53 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/bifrost: Updated from global requirements https://review.openstack.org/389876 | 05:53 |
| *** shuquan has joined #openstack-ironic | 05:54 | |
| *** rcernin has joined #openstack-ironic | 05:55 | |
| *** khamtamtun has quit IRC | 06:00 | |
| *** khamtamtun has joined #openstack-ironic | 06:05 | |
| *** vinsh has joined #openstack-ironic | 06:09 | |
| *** khamtamtun has quit IRC | 06:12 | |
| *** vinsh has quit IRC | 06:14 | |
| *** phuongnh has quit IRC | 06:25 | |
| *** phuongnh has joined #openstack-ironic | 06:26 | |
| *** ChubYann has quit IRC | 06:27 | |
| *** baoli has joined #openstack-ironic | 06:30 | |
| *** sabeen has quit IRC | 06:31 | |
| *** baoli has quit IRC | 06:34 | |
| *** fxpester has joined #openstack-ironic | 06:56 | |
| *** tesseract has joined #openstack-ironic | 07:04 | |
| *** tesseract is now known as Guest13194 | 07:04 | |
| *** mjura has joined #openstack-ironic | 07:06 | |
| Nisha_away | morning Ironic!!! | 07:08 |
| Nisha_away | hshiina, morning | 07:11 |
| hshiina | Nisha_away, morning | 07:12 |
| *** phuongnh has quit IRC | 07:14 | |
| *** mjura has quit IRC | 07:15 | |
| *** mjura has joined #openstack-ironic | 07:15 | |
| pas-ha | morning Ironic. TGIF :) | 07:24 |
| *** shuquan has quit IRC | 07:27 | |
| tuanluong | morning ironic | 07:27 |
| *** shuquan has joined #openstack-ironic | 07:28 | |
| *** pcaruana has joined #openstack-ironic | 07:33 | |
| *** chlong has quit IRC | 07:33 | |
| *** shuquan has quit IRC | 07:34 | |
| *** shuquan has joined #openstack-ironic | 07:34 | |
| *** khamtamtun has joined #openstack-ironic | 07:41 | |
| *** khamtamtun has quit IRC | 07:43 | |
| *** raghusab has quit IRC | 07:50 | |
| *** ccamacho has quit IRC | 07:53 | |
| *** ohamada has joined #openstack-ironic | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** ccamacho has joined #openstack-ironic | 08:05 | |
| *** livelace has quit IRC | 08:07 | |
| *** d0ugal has joined #openstack-ironic | 08:07 | |
| *** jaosorior has joined #openstack-ironic | 08:07 | |
| *** fragatina has joined #openstack-ironic | 08:09 | |
| *** shuquan_ has joined #openstack-ironic | 08:09 | |
| *** fragatina has quit IRC | 08:10 | |
| *** raghusab has joined #openstack-ironic | 08:10 | |
| *** fragatina has joined #openstack-ironic | 08:10 | |
| *** shuquan has quit IRC | 08:11 | |
| *** Nisha_away has quit IRC | 08:16 | |
| *** khamtamtun has joined #openstack-ironic | 08:17 | |
| *** athomas has joined #openstack-ironic | 08:19 | |
| *** aarefiev_afk is now known as aarefiev | 08:21 | |
| aarefiev | morning! | 08:21 |
| *** khamtamtun has quit IRC | 08:21 | |
| *** d0ugal has quit IRC | 08:28 | |
| *** d0ugal has joined #openstack-ironic | 08:32 | |
| *** d0ugal has quit IRC | 08:32 | |
| *** khamtamtun has joined #openstack-ironic | 08:33 | |
| *** vsaienk0 has quit IRC | 08:35 | |
| *** khamtamtun has quit IRC | 08:35 | |
| *** khamtamtun has joined #openstack-ironic | 08:37 | |
| *** xek has joined #openstack-ironic | 08:38 | |
| xek | morning ironic :) | 08:39 |
| *** vsaienk0 has joined #openstack-ironic | 08:41 | |
| *** khamtamtun has quit IRC | 08:44 | |
| *** d0ugal has joined #openstack-ironic | 08:55 | |
| *** d0ugal_ has joined #openstack-ironic | 08:57 | |
| *** sborkows has joined #openstack-ironic | 08:58 | |
| sborkows | morning, all | 08:58 |
| *** d0ugal has quit IRC | 08:59 | |
| *** jpich has joined #openstack-ironic | 08:59 | |
| *** strigazi_AFK is now known as strigazi | 09:04 | |
| *** hshiina has quit IRC | 09:04 | |
| *** lucas-afk is now known as lucasagomes | 09:12 | |
| lucasagomes | morning all | 09:15 |
| *** e0ne has joined #openstack-ironic | 09:19 | |
| *** dtantsur|afk is now known as dtantsur | 09:32 | |
| dtantsur | Morning Ironic | 09:32 |
| sborkows | morning lucasagomes, dtantsur | 09:33 |
| *** Nisha_away has joined #openstack-ironic | 09:34 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-specs: Update ironic-ml2-integration spec https://review.openstack.org/393660 | 09:34 |
| vsaienk0 | morning lucasagomes, dtantsur, sborkows and All! | 09:35 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification https://review.openstack.org/348437 | 09:36 |
| *** e0ne has quit IRC | 09:38 | |
| tuanluong | morning ironic | 09:38 |
| *** milan has joined #openstack-ironic | 09:44 | |
| milan | morning Ironic! :) | 09:44 |
| sborkows | dtantsur: I have a question about custom hardware managers - in my patch https://review.openstack.org/#/c/391836/ I added such manager to hardware.py file. However, I got an idea from Joanna, that the better solution will be to move it to separate file in separate 'hardware_managers' directory. What do you think about it? | 09:44 |
| patchbot | patch 391836 - ironic-python-agent - Add a new Hardware Manager for X710 network card | 09:44 |
| lucasagomes | folks, I've just opened a RFE ( https://bugs.launchpad.net/ironic/+bug/1639187 ) for the petitboot change ( https://review.openstack.org/#/c/185987 ) as requested in the comments. Someone mind taking a look and approving if it's good ? (It's tiny) | 09:46 |
| openstack | Launchpad bug 1639187 in Ironic "[RFE] Extend PXE to support petitboot" [Wishlist,New] | 09:46 |
| patchbot | patch 185987 - ironic - Enable PXE for systems using petitboot | 09:46 |
| dtantsur | sborkows, ++ for a separate file, makes sense | 09:46 |
| sborkows | dtantsur, what about a separate directory for future hardware managers? | 09:47 |
| dtantsur | sborkows, yeah, why not | 09:48 |
| sborkows | dtantsur, ok, so 1 more thing - what about the hardware.py file? Should we leave it untouched or move along with the other hardware managers? | 09:48 |
| dtantsur | sborkows, please leave it as it is, as it contains generic code | 09:49 |
| sborkows | dtantsur, ok thanks for advice. | 09:50 |
| *** d0ugal_ is now known as d0ugal | 09:51 | |
| *** d0ugal has joined #openstack-ironic | 09:51 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work https://review.openstack.org/376322 | 09:53 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work https://review.openstack.org/376322 | 09:54 |
| *** karthi has joined #openstack-ironic | 09:56 | |
| *** shuquan_ has quit IRC | 09:57 | |
| *** shuquan has joined #openstack-ironic | 09:58 | |
| *** shuquan has quit IRC | 10:03 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work https://review.openstack.org/376322 | 10:06 |
| *** yufei_ has quit IRC | 10:09 | |
| *** yufei_ has joined #openstack-ironic | 10:11 | |
| *** vinsh has joined #openstack-ironic | 10:11 | |
| *** yufei_ has quit IRC | 10:13 | |
| jroll | lucasagomes: does this RFE not cover it? https://bugs.launchpad.net/ironic/+bug/1526265 | 10:13 |
| openstack | Launchpad bug 1526265 in Ironic "[RFE] OpenPower Petitboot Boot Driver" [Wishlist,Triaged] - Assigned to chenglch (chenglch) | 10:13 |
| *** yufei_ has joined #openstack-ironic | 10:13 | |
| jroll | lucasagomes: which had a spec: http://specs.openstack.org/openstack/ironic-specs/specs/approved/petitboot-boot-driver.html | 10:13 |
| *** yufei_ has quit IRC | 10:13 | |
| lucasagomes | jroll, hi there, lemme see | 10:13 |
| jroll | oh, and good morning everybody :) | 10:14 |
| jroll | lucasagomes: it's a bit odd because the spec proposes new drivers, which mostly just add that pxe option | 10:14 |
| lucasagomes | jroll, it looks like different work. What I get from turkes' patch is that you don't need a new driver | 10:14 |
| *** yufei_ has joined #openstack-ironic | 10:15 | |
| lucasagomes | you just need a DHCP option | 10:15 |
| lucasagomes | right | 10:15 |
| *** athomas has quit IRC | 10:15 | |
| *** yufei_ has quit IRC | 10:15 | |
| jroll | lucasagomes: right, so we can get rid of the existing rfe/spec, yeah? | 10:15 |
| lucasagomes | I will ask in the review, I believe we could get rid of the spec (and it's RFE) yes | 10:15 |
| jroll | yeah, cool | 10:16 |
| jroll | approving your new RFE now | 10:16 |
| *** vinsh has quit IRC | 10:16 | |
| *** ppiela has quit IRC | 10:16 | |
| jroll | I think I'm finally feeling good enough to actually work today \o/ | 10:17 |
| *** ppiela has joined #openstack-ironic | 10:17 | |
| lucasagomes | jroll, thanks, I left a comment in the review about the spec too | 10:17 |
| *** yufei_ has joined #openstack-ironic | 10:17 | |
| lucasagomes | jroll, o/ good | 10:17 |
| jroll | :) | 10:18 |
| *** kamtamtun has joined #openstack-ironic | 10:18 | |
| *** e0ne has joined #openstack-ironic | 10:19 | |
| dtantsur | morning jroll | 10:19 |
| dtantsur | good to hear you feel better! | 10:19 |
| jroll | morning dtantsur :) | 10:19 |
| jroll | not 100% but I think I have a brain now | 10:20 |
| dtantsur | jroll, let's celebrate that with approving a backport :) https://review.openstack.org/393188 | 10:20 |
| patchbot | patch 393188 - ironic (stable/newton) - API: lookup() ignore malformed MAC addresses | 10:20 |
| jroll | ah, fun | 10:20 |
| jroll | dtantsur: done | 10:21 |
| dtantsur | thnx | 10:21 |
| *** athomas has joined #openstack-ironic | 10:21 | |
| *** sambetts|afk is now known as sambetts | 10:25 | |
| sambetts | Morning all | 10:25 |
| sambetts | o/ jroll, dtantsur | 10:25 |
| jroll | morning sambetts :) | 10:25 |
| dtantsur | hey sambetts | 10:26 |
| milan | morning jroll, sambetts! :) | 10:30 |
| jroll | morning milan | 10:30 |
| *** yufei_ has quit IRC | 10:34 | |
| *** tuanluong has quit IRC | 10:34 | |
| *** hoangcx has quit IRC | 10:35 | |
| *** yufei_ has joined #openstack-ironic | 10:38 | |
| yufei_ | test ping | 10:39 |
| *** NobodyCam has quit IRC | 10:42 | |
| *** evgenyl has quit IRC | 10:42 | |
| *** Ng has quit IRC | 10:42 | |
| *** betherly has quit IRC | 10:42 | |
| jroll | hi yufei_ | 10:42 |
| *** romcheg_ has quit IRC | 10:43 | |
| *** sergek_ has quit IRC | 10:43 | |
| *** ildikov has quit IRC | 10:43 | |
| *** kun_huang has quit IRC | 10:43 | |
| *** izaakk has quit IRC | 10:43 | |
| *** vdrok has quit IRC | 10:44 | |
| *** fungi has quit IRC | 10:44 | |
| *** zhenguo has quit IRC | 10:44 | |
| *** mgould|afk is now known as mgould | 10:44 | |
| mgould | morning all | 10:45 |
| mgould | morning lucasagomes jroll milan yufei_ sambetts dtantsur | 10:45 |
| milan | morning mgould! :) | 10:45 |
| *** sergek_ has joined #openstack-ironic | 10:45 | |
| mgould | morning milan :-) | 10:45 |
| dtantsur | hey mgould, milan | 10:45 |
| *** Ng has joined #openstack-ironic | 10:45 | |
| *** ildikov has joined #openstack-ironic | 10:46 | |
| milan | morning dtantsur! :) | 10:46 |
| *** betherly has joined #openstack-ironic | 10:46 | |
| *** kun_huang has joined #openstack-ironic | 10:46 | |
| *** vdrok has joined #openstack-ironic | 10:47 | |
| *** zhenguo has joined #openstack-ironic | 10:47 | |
| *** openstackgerrit has quit IRC | 10:47 | |
| *** openstackgerrit has joined #openstack-ironic | 10:48 | |
| Nisha_away | morning jroll, dtantsur milan mgould lucasagomes sambetts n all others | 10:51 |
| milan | morning Nisha_away! :) | 10:51 |
| jroll | morning Nisha_away, mgould :) | 10:51 |
| *** Nisha_away is now known as Nisha1 | 10:51 | |
| Nisha1 | jroll, do we have the priority list for Octata captured somewhere? I am still catching up with the mails | 10:54 |
| jroll | Nisha1: not in git yet (I've been sick this week) but the list from the summit is at the bottom of https://etherpad.openstack.org/p/ironic-ocata-summit-priorities | 10:54 |
| Nisha1 | jroll, thanks...i also just joined back from my long leave yesterday only | 10:55 |
| jroll | welcome back :) | 10:55 |
| *** NobodyCam has joined #openstack-ironic | 10:56 | |
| *** romcheg_ has joined #openstack-ironic | 10:56 | |
| Nisha1 | thank you | 10:56 |
| *** fungi has joined #openstack-ironic | 10:58 | |
| *** ralonsoh has joined #openstack-ironic | 10:59 | |
| *** evgenyl has joined #openstack-ironic | 10:59 | |
| *** izaakk has joined #openstack-ironic | 10:59 | |
| Nisha1 | jroll, what is the likeliness of the specs and code patches related to inspection to be merged in Octata | 11:04 |
| Nisha1 | they are already in review from 2 releases | 11:04 |
| jroll | Nisha1: remind me which patches? | 11:05 |
| Nisha1 | jroll,the spec patches https://review.openstack.org/305864, https://review.openstack.org/201904, https://review.openstack.org/338138 | 11:06 |
| patchbot | patch 305864 - ironic-specs - Boot management for in-band inspection | 11:06 |
| patchbot | patch 201904 - ironic-specs - Enhance ilo drivers to do inband inspection | 11:06 |
| patchbot | patch 338138 - ironic-specs - Add new out-of-band properties to ilo drivers | 11:06 |
| Nisha1 | there are 2 more small RFE proposed related to inspection last release | 11:07 |
| Nisha1 | https://bugs.launchpad.net/ironic/+bug/1623769-, https://bugs.launchpad.net/ironic/+bug/1609622 | 11:07 |
| openstack | Launchpad bug 1623769 in Ironic "[RFE] Allow iLO drivers to add new capabilities that are to be used by Nova scheduler without any changes to Ironic" [Wishlist,In progress] - Assigned to Shivanand Tendulker (stendulker) | 11:07 |
| openstack | Launchpad bug 1609622 in Ironic "[RFE] Enhance iLO drivers to use snmp to do OOB inspection for local_gb" [Wishlist,In progress] - Assigned to Nisha Agarwal (agarwalnisha1980) | 11:07 |
| *** jkilpatr has joined #openstack-ironic | 11:07 | |
| Nisha1 | since i dont see them in priority list at all, would like to know the likeliness for Octata release for them | 11:08 |
| jroll | Nisha1: well, single-vendor work is never on the priority list - I'd like to say we can make progress but I need to get more organized before I can promise anything | 11:10 |
| jroll | I haven't had a full day back from summit yet, still need to catch up on everything | 11:10 |
| jroll | dtantsur: ^^ any thoughts? | 11:10 |
| milan | sambetts, dtantsur let's merge https://review.openstack.org/#/c/392943/ ? | 11:11 |
| patchbot | patch 392943 - ironic-inspector - Allow to configure cache clean up period | 11:11 |
| milan | the infra patch was merged already | 11:11 |
| dtantsur | jroll, Nisha1, "add new capabilities" I want a generic spec on capabilities we expose. otherwise we end up with all drivers incompatible with each other and ironic-inspector | 11:11 |
| dtantsur | milan, will do | 11:11 |
| jroll | dtantsur: ++ | 11:12 |
| dtantsur | jroll, Nisha1, otherwise I agree, we do take vendor work, but it rarely gets on the priority list. it does not mean we won't land it. | 11:12 |
| jroll | I also find it hard to believe we should be scheduling based on firmware versions and such | 11:12 |
| dtantsur | as to "Boot management for in-band inspection", it is my RFE, so I won't comment on its prioritization :) | 11:13 |
| jroll | dtantsur: oh, I'm asking for comments on 'can we get it done', not prioritization :) | 11:13 |
| milan | dtantsur, thx | 11:13 |
| dtantsur | "Boot management for in-band inspection" is a small thing, partly contained in ironic-inspector. I'd love to get it done any time, as soon as the spec gets more attention. | 11:14 |
| dtantsur | but it's also not as urgent as other things I've committed to | 11:14 |
| sambetts | jroll: do we have a template for RFEs? | 11:16 |
| jroll | sambetts: don't think so | 11:16 |
| openstackgerrit | Merged openstack/ironic: API: lookup() ignore malformed MAC addresses https://review.openstack.org/393188 | 11:16 |
| * dtantsur lols at https://bugs.launchpad.net/cirros/+bug/1638807 | 11:20 | |
| openstack | Launchpad bug 1638807 in CirrOS "Improper password to root user" [Undecided,New] | 11:20 |
| Nisha1 | jroll, dtantsur i agree | 11:23 |
| *** vinsh has joined #openstack-ironic | 11:23 | |
| mat128 | good morning | 11:23 |
| dtantsur | morning mat128 | 11:24 |
| Nisha1 | the spec is already there for new capabilities | 11:26 |
| Nisha1 | dtantsur, jroll https://review.openstack.org/338138 | 11:26 |
| patchbot | patch 338138 - ironic-specs - Add new out-of-band properties to ilo drivers | 11:26 |
| *** aparnav has quit IRC | 11:27 | |
| *** karthi has quit IRC | 11:27 | |
| mgould | morning Nisha1 mat128 | 11:29 |
| dtantsur | Nisha1, if you remove "ilo" from it, add reasoning behind every of them (preferably with examples) and how to detect them in-band (if possible), then it will be the spec I'm looking for | 11:29 |
| *** hw_wutianwei has quit IRC | 11:29 | |
| cdaniel | morning all | 11:30 |
| dtantsur | morning cdaniel | 11:30 |
| Nisha1 | dtantsur, i can remove ilo, but i havent explored all thru inband | 11:30 |
| Nisha1 | i know hw to get them oob | 11:31 |
| dtantsur | I understand, but need to agree on common direction for capabilities | 11:31 |
| Nisha1 | dtantsur, we can have the implementation oob and inband both | 11:31 |
| dtantsur | the ilo driver is unique enough already. we have to make sure we converge more drivers to exposing the same capabilities | 11:31 |
| Nisha1 | oob - for ilo drivers | 11:31 |
| dtantsur | in particular, I'm leaned towards being able to get the majority of the things in-band | 11:32 |
| Nisha1 | inband- for inpector | 11:32 |
| Nisha1 | dtantsur, OOB is always faster | 11:32 |
| dtantsur | I understand, I don't see how it's related to the discussion | 11:33 |
| dtantsur | does you spec have enough information for non-ILO drivers to implement it? | 11:33 |
| Nisha1 | dtantsur, no i havent explored much how to get those info inband | 11:34 |
| dtantsur | well, so now you see the problem. I don't want the ilo driver to diverge even more feature-wise. | 11:35 |
| dtantsur | (boot_mode situation is more than enough) | 11:35 |
| *** dprince has joined #openstack-ironic | 11:36 | |
| *** d0ugal has quit IRC | 11:40 | |
| jroll | dtantsur: that cirros bug is already fixed :D https://git.launchpad.net/cirros/commit/?id=9a7c371ef329cf78f256d0a5a8f475d9c57f5477 | 11:41 |
| aarefiev | we are still broken :( | 11:43 |
| *** d0ugal has joined #openstack-ironic | 11:44 | |
| dtantsur | jroll, lol | 11:44 |
| dtantsur | aarefiev, ugh :( | 11:44 |
| aarefiev | looks like same bug was in ironic, changing configuration option doesn't have affect | 11:45 |
| aarefiev | yeah | 11:45 |
| Nisha1 | dtantsur, if the spec covers the way how to discover inband, will ilo drivers be allowed to implement oob without specifically doing for refernce drivers. | 11:45 |
| Nisha1 | ? | 11:45 |
| *** portdirect_away is now known as portdirect | 11:46 | |
| dtantsur | Nisha1, as soon as we agree on which capabilities we discover and why, I'm fine with ilo being the first to implement them | 11:46 |
| dtantsur | jroll, ^^^ agree? | 11:46 |
| jroll | dtantsur: yeah, sure, as long as someone is going to work on the in-band implementation | 11:47 |
| jroll | dtantsur: what can I do today to best help cores out - priorities in spec repo + trello setup? | 11:48 |
| dtantsur | jroll, yes, these two | 11:48 |
| Nisha1 | dtantsur,so ilo oob implementation can land first n doesnt has to wait fr reference implementation to land? | 11:48 |
| jroll | dtantsur: thanks, I'll get those done today | 11:48 |
| dtantsur | jroll, and then form the subteams the same way we did for newton | 11:48 |
| jroll | yep | 11:48 |
| dtantsur | Nisha1, for me it's fine, other folks may have different opinion | 11:49 |
| *** ralonsoh_ has joined #openstack-ironic | 11:49 | |
| *** ralonsoh has quit IRC | 11:53 | |
| Nisha1 | jroll ^^^ | 11:53 |
| *** bfournie has quit IRC | 11:55 | |
| dtantsur | folks, do we have a CI with plain PXE? I'm afraid to +2 https://review.openstack.org/185987 without it.. | 11:56 |
| patchbot | patch 185987 - ironic - Enable PXE for systems using petitboot | 11:56 |
| dtantsur | lucasagomes, ^^^? | 11:56 |
| jroll | Nisha1: like I said a few minutes ago, "as long as someone is going to work on the in-band implementation" | 11:56 |
| jroll | dtantsur: I believe so, let me check | 11:56 |
| lucasagomes | dtantsur, I think IBM does use it for powerpc | 11:56 |
| * lucasagomes checks | 11:56 | |
| lucasagomes | we probably should mention it in the RFE if that's the case tho | 11:57 |
| jroll | dtantsur: pxe_ssh | 11:57 |
| dtantsur | jroll, which of them? I see ipxe_enabled=True http://logs.openstack.org/87/185987/12/check/gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa/b848902/logs/screen-ir-cond.txt.gz#_2016-11-03_14_20_32_282 | 11:57 |
| dtantsur | lucasagomes, 3rd party CI usually use plain PXE for some reason. but they're so unstable... | 11:58 |
| jroll | dtantsur: this one https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L549 | 11:58 |
| dtantsur | jroll, I'm not sure if we run this job any more.. maybe on some stable branches | 11:58 |
| dtantsur | jroll, this is for old DIB ramdisk, no? | 11:59 |
| *** Nisha1 has quit IRC | 11:59 | |
| lucasagomes | jroll, that runs standard pxe but not necessarily pettitboot right ? | 11:59 |
| lucasagomes | (btw, it's broken on xenial, I put a patch up yesterday fixing it) | 11:59 |
| dtantsur | lucasagomes, ignore petitboot for now, I want to make sure we don't break PXE at all | 11:59 |
| lucasagomes | oh right | 11:59 |
| dtantsur | oh so it IS broken? | 11:59 |
| lucasagomes | yes | 11:59 |
| jroll | dtantsur: yeah, you might be right | 12:00 |
| lucasagomes | on xenial | 12:00 |
| dtantsur | lucasagomes, do we have a single job on master running plain PXE? | 12:00 |
| lucasagomes | dtantsur, https://review.openstack.org/#/c/393353/ | 12:00 |
| patchbot | patch 393353 - ironic - DevStack: Fix standard PXE on Ubuntu Xenial | 12:00 |
| dtantsur | I suspect the answer is "no", which makes uncomfortable both generally and with approving this patch | 12:00 |
| lucasagomes | dtantsur, I think no at the moment | 12:00 |
| aarefiev | looks like pas-ha didn't find pxe job https://etherpad.openstack.org/p/ironic-gate-jobs-described | 12:01 |
| *** baoli has joined #openstack-ironic | 12:01 | |
| lucasagomes | dtantsur, maybe we should add a experimental job in the gate for it ? | 12:01 |
| lucasagomes | then, we flip the trusty to xenial once pxe is fixed | 12:01 |
| lucasagomes | or we just flip and fix it | 12:01 |
| dtantsur | lucasagomes, we used to run the postgres job with plain PXE. I suggest we do it again. | 12:02 |
| dtantsur | it's non-voting, so we won't be affected by PXE instability too much | 12:02 |
| dtantsur | (i.e. we can ignore it for patches not touching PXE code) | 12:02 |
| *** ralonsoh__ has joined #openstack-ironic | 12:03 | |
| lucasagomes | dtantsur, heh fair, it consolidates too not-very-used technologies (in openstack realms) in one job :-) | 12:03 |
| dtantsur | wanna propose a patch? | 12:04 |
| lucasagomes | I guess, project-config here I go again | 12:04 |
| dtantsur | it's fun, isn't it? ;) | 12:05 |
| lucasagomes | yeah, I feel like I'm paying for my sis | 12:05 |
| lucasagomes | win-win | 12:05 |
| *** baoli has quit IRC | 12:05 | |
| lucasagomes | sins* | 12:05 |
| *** ralonsoh_ has quit IRC | 12:06 | |
| openstackgerrit | Stig Telfer proposed openstack/ironic-specs: Torrent-based provisioning https://review.openstack.org/311091 | 12:07 |
| nicodemos | morning. | 12:08 |
| *** baoli has joined #openstack-ironic | 12:11 | |
| *** baoli_ has joined #openstack-ironic | 12:12 | |
| *** chlong has joined #openstack-ironic | 12:13 | |
| *** baoli has quit IRC | 12:16 | |
| *** xek has quit IRC | 12:17 | |
| *** cdearborn has joined #openstack-ironic | 12:21 | |
| *** jerrygb has joined #openstack-ironic | 12:23 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow nginx port on selinux https://review.openstack.org/393745 | 12:24 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: IPMI command should depend on console type https://review.openstack.org/393746 | 12:26 |
| dtantsur | looks like a reasonable backport too ^^^ | 12:27 |
| *** bfournie has joined #openstack-ironic | 12:29 | |
| *** betherly has quit IRC | 12:31 | |
| *** jerrygb has quit IRC | 12:31 | |
| *** e0ne has quit IRC | 12:33 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Add ocata priorities https://review.openstack.org/393753 | 12:38 |
| jroll | wheeeeeeeeeeeee | 12:38 |
| jroll | cores please review ^^ | 12:39 |
| *** jerrygb has joined #openstack-ironic | 12:42 | |
| *** rbrud has joined #openstack-ironic | 12:44 | |
| *** vinsh has quit IRC | 12:45 | |
| *** chlong has quit IRC | 12:46 | |
| openstackgerrit | Merged openstack/ironic: Ignore required_services for multinode topology https://review.openstack.org/352793 | 12:51 |
| openstackgerrit | Merged openstack/ironic: Skip db configuration on subnodes https://review.openstack.org/353303 | 12:51 |
| *** lucasagomes is now known as lucas-hungry | 12:52 | |
| *** gcb has quit IRC | 12:52 | |
| *** rbudden has joined #openstack-ironic | 12:53 | |
| TheJulia | jroll: comments posted, generally looks good to me, although the CI refactoring bit likely needs to be a higher priority :\ | 12:55 |
| *** jheroux has joined #openstack-ironic | 12:55 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow nginx port on selinux https://review.openstack.org/393745 | 12:58 |
| *** jcoufal has joined #openstack-ironic | 12:58 | |
| *** shuquan has joined #openstack-ironic | 12:59 | |
| jroll | TheJulia: yeah, agree, thanks | 12:59 |
| *** ralonsoh_ has joined #openstack-ironic | 13:01 | |
| *** jerrygb has quit IRC | 13:01 | |
| *** betherly has joined #openstack-ironic | 13:03 | |
| *** shuquan has quit IRC | 13:04 | |
| *** ralonsoh__ has quit IRC | 13:04 | |
| mgould | does anyone here understand magnet links? I'm trying to understand L115 of https://review.openstack.org/#/c/311091/4..5/specs/approved/torrent-provisioning.rst | 13:09 |
| patchbot | patch 311091 - ironic-specs - Torrent-based provisioning | 13:09 |
| aarefiev | mgould: just ask | 13:10 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow nginx port on selinux https://review.openstack.org/393745 | 13:11 |
| jroll | mgould: magnet links are a way to join a swarm without a central tracker | 13:11 |
| mgould | aarefiev: I'm trying to form a good questiojn :-) | 13:11 |
| mgould | yep | 13:11 |
| jroll | used in combination with DHT | 13:11 |
| aarefiev | jroll: it could be used with tracker too | 13:11 |
| jroll | aarefiev: oh, yeah, you're right | 13:11 |
| *** d0ugal has quit IRC | 13:12 | |
| mgould | so, the magnet link contains a hash of the .torrent file, and optionally a URL to the .torrent file? | 13:12 |
| jroll | mgould: seen http://www.bittorrent.org/beps/bep_0009.html ? | 13:13 |
| mgould | then the client fetches the .torrent from the DHT or from the URL, then downloads the files of interest using the information in the .torrent? | 13:13 |
| mgould | jroll: aha, thanks | 13:13 |
| jroll | hope it helps :) | 13:13 |
| * jroll also reads it | 13:13 | |
| *** links has quit IRC | 13:13 | |
| *** mgoddard_ has joined #openstack-ironic | 13:14 | |
| aarefiev | yeah and all other stuff ^ | 13:14 |
| aarefiev | you need to get .torrent somehow first and then download file if seeds available | 13:16 |
| mgould | yep | 13:16 |
| mariojv | it looks like optionally you can still use a tracker? instead of DHT | 13:17 |
| aarefiev | yes | 13:17 |
| mgould | and we're seeding the image file from Glance, and the question is where we seed the .torrent file from? | 13:17 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow nginx port on selinux https://review.openstack.org/393745 | 13:17 |
| *** mgoddard has quit IRC | 13:17 | |
| aarefiev | problem, how to get this torrent file, actually glance doesn't seed image | 13:18 |
| aarefiev | we should get it via http first and then deployed nodes will seed it | 13:19 |
| mgould | sure | 13:19 |
| *** d0ugal has joined #openstack-ironic | 13:20 | |
| mgould | but to get an HTTP URL of an image in Glance we need to use Glare? | 13:21 |
| aarefiev | jroll, when you will finish reading torrent specification, maybe you could take a look on ironic spec :) | 13:22 |
| jroll | aarefiev: I'm trying to catch up on post-summit stuff today, sorry | 13:22 |
| *** mgoddard has joined #openstack-ironic | 13:22 | |
| aarefiev | np | 13:22 |
| jroll | also, isn't glare work stopping? | 13:22 |
| *** mgoddard_ has quit IRC | 13:22 | |
| aarefiev | nope | 13:23 |
| jroll | hm | 13:23 |
| aarefiev | mgould: Glare is artifact repository , it was part of Glance | 13:23 |
| aarefiev | and now separate service | 13:24 |
| aarefiev | jroll: I've disscused Glare with mfedosin on summit | 13:25 |
| *** [1]cdearborn has joined #openstack-ironic | 13:25 | |
| jroll | aarefiev: ok | 13:25 |
| aarefiev | community will support it | 13:25 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow nginx port on selinux https://review.openstack.org/393745 | 13:26 |
| mariojv | one alternative would be to use swift right? | 13:26 |
| mgould | aarefiev: so Glare solves the problem "here is a file; please give me a URL where other clients can download that file"? | 13:26 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow nginx port on selinux https://review.openstack.org/393745 | 13:27 |
| * mgould thought that was what Swift did :-( | 13:27 | |
| aarefiev | mgould: we will add new file type torrent, and there are would be no needs in storing torrent for example in glance as "image" | 13:29 |
| aarefiev | swift could be backend to glare | 13:30 |
| mgould | ah, OK | 13:30 |
| aarefiev | it uses same glance_store | 13:30 |
| *** daemontool has joined #openstack-ironic | 13:32 | |
| *** ccamacho is now known as ccamacho|lunch | 13:32 | |
| *** dansmith is now known as superdan | 13:34 | |
| *** d0ugal has quit IRC | 13:35 | |
| *** d0ugal has joined #openstack-ironic | 13:37 | |
| *** jerrygb has joined #openstack-ironic | 13:39 | |
| *** jerrygb has quit IRC | 13:39 | |
| *** cdearborn has quit IRC | 13:40 | |
| *** jroll is now known as jrollinhatin | 13:40 | |
| *** jerrygb has joined #openstack-ironic | 13:46 | |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Ignore KeyError on delete of missing sensor https://review.openstack.org/393778 | 13:46 |
| mgould | aarefiev: left some comments/questions, hopefully not too stupid | 13:48 |
| *** jerrygb has quit IRC | 13:48 | |
| aarefiev | mgould: thanks, I'll take a look | 13:48 |
| *** causten has joined #openstack-ironic | 13:49 | |
| mgould | np | 13:50 |
| *** e0ne has joined #openstack-ironic | 13:50 | |
| *** d0ugal_ has joined #openstack-ironic | 13:51 | |
| milan | aarefiev, seems the patch to update the cleanup in inspector fails to update the config file :-/ http://logs.openstack.org/43/392943/6/check/gate-tempest-dsvm-ironic-inspector-discovery/eb78693/logs/etc/ironic-inspector/inspector.conf.txt.gz although the var is present in reproduce: http://logs.openstack.org/43/392943/6/check/gate-tempest-dsvm-ironic-inspector-discovery/eb78693/logs/reproduce.sh | 13:53 |
| *** d0ugal has quit IRC | 13:53 | |
| * milan wonders what to do about it :-/ | 13:53 | |
| *** kamtamtun has quit IRC | 13:54 | |
| dtantsur | milan, check devstack logs, maybe just some typo? | 13:54 |
| aarefiev | milan: yeah, we are initialize that value before read config file | 13:55 |
| aarefiev | it uses default | 13:55 |
| aarefiev | I'm trying to figure out how to fix better | 13:56 |
| *** lucas-hungry is now known as lucasagomes | 13:56 | |
| aarefiev | 2 fixes in 1 patch ( | 13:57 |
| * milan reads devstacklog | 13:57 | |
| mgould | brb | 13:57 |
| *** Guest13194 has quit IRC | 13:58 | |
| dtantsur | aarefiev, oh, this is a bug for sure | 13:58 |
| aarefiev | dtantsur: are you fine if we will declare periodic task in init method for now | 13:59 |
| milan | aarefiev, you think it's because the plugin is downloaded from http and somehow overrides the job template? | 13:59 |
| aarefiev | inspector starts with config value 5, so devstack part is fine | 13:59 |
| dtantsur | aarefiev, I don't think we should move the task itself, rather the default value | 13:59 |
| * dtantsur looks at the code | 14:00 | |
| *** khamtamtun has joined #openstack-ironic | 14:00 | |
| *** yibo has quit IRC | 14:00 | |
| aarefiev | it goes to futurist | 14:00 |
| milan | aarefiev, /me rechecks etc | 14:00 |
| *** anush has joined #openstack-ironic | 14:00 | |
| dtantsur | wait, I think I'm confused | 14:00 |
| dtantsur | do we set configuration value after starting inspector? this is doomed to failure | 14:01 |
| milan | aarefiev, ack, the value is DEFAULT.clean_up_period = 5 | 14:01 |
| milan | in etc/inspector.conf | 14:01 |
| dtantsur | ah, I think I get it | 14:01 |
| milan | dtantsur, share please, I'm still figuring out :) | 14:02 |
| aarefiev | no we set value from config before start | 14:02 |
| dtantsur | milan, we use CONF values before we load ironic.conf | 14:02 |
| aarefiev | and this is default one | 14:02 |
| milan | dtantsur, in the code you mean? | 14:02 |
| dtantsur | milan, yep | 14:02 |
| milan | Oops! :D | 14:02 |
| dtantsur | aarefiev, yeah, I get we have to move periodic tasks to run().. which is bad, but not as bad as what we've got | 14:03 |
| aarefiev | here https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/main.py#L336 | 14:03 |
| aarefiev | dtantsur: I guess if we split services we could refactor this | 14:03 |
| aarefiev | also 2 fixes in one patch, I don't like this ( | 14:04 |
| * dtantsur too | 14:04 | |
| dtantsur | but I guess it can't be helped | 14:04 |
| *** ralonsoh_ has quit IRC | 14:04 | |
| *** ralonsoh_ has joined #openstack-ironic | 14:04 | |
| milan | aarefiev, +1 the decorator indeed is "compile-time" :-/ | 14:06 |
| milan | what about if we load the config compile time too? | 14:06 |
| dtantsur | dunno what you're referring to as "compile time".. it just executes earlier | 14:08 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/393791 | 14:08 |
| milan | dtantsur, outside of the service initialisation; btw where exactly is the conf loaded? | 14:09 |
| jrollinhatin | "import time" :) | 14:09 |
| *** Goneri has joined #openstack-ironic | 14:09 | |
| jrollinhatin | this is where oslo.config gets painful | 14:09 |
| dtantsur | milan, so, the default value is calculated at function definition (a weird part of Python), but we load CONF in main() | 14:10 |
| dtantsur | jrollinhatin++ (also cool nick) | 14:10 |
| jrollinhatin | heh, casual nick friday \o/ | 14:10 |
| milan | dtantsur, yeah, is it this line? | 14:10 |
| milan | https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/main.py#L390 | 14:11 |
| *** baoli_ has quit IRC | 14:11 | |
| *** baoli has joined #openstack-ironic | 14:11 | |
| dtantsur | milan, yep | 14:11 |
| dtantsur | oh, casual nick friday! | 14:11 |
| *** dtantsur is now known as creepy_owlet | 14:11 | |
| *** khamtamtun has quit IRC | 14:12 | |
| aarefiev | yeah, we could try to load config in conf.py and import it in main | 14:12 |
| milan | aarefiev, creepy_owlet :) how about we bump the call outside of the service? | 14:12 |
| milan | aarefiev, exactly | 14:12 |
| *** khamtamtun has joined #openstack-ironic | 14:12 | |
| creepy_owlet | milan, then you'll have to move it to the top of the file, which is ugly as well | 14:12 |
| milan | creepy_owlet, would you -2 it right away? :D | 14:12 |
| milan | yeah but might help :D | 14:12 |
| creepy_owlet | well... I don't like code which depends on the order of execution | 14:12 |
| aarefiev | how many new people here :) | 14:13 |
| creepy_owlet | imagine someone wants to refator it later | 14:13 |
| *** e0ne has quit IRC | 14:13 | |
| creepy_owlet | haha, new people and animals | 14:13 |
| milan | I'd also prefer Haskell :D | 14:13 |
| aarefiev | :) | 14:13 |
| lucasagomes | hah | 14:13 |
| *** e0ne has joined #openstack-ironic | 14:13 | |
| *** Guest13194 has joined #openstack-ironic | 14:13 | |
| milan | so what to do? | 14:13 |
| milan | I'm +2 on bumping it out of main 0:-) | 14:14 |
| milan | sambetts, ^ ? | 14:14 |
| aarefiev | move decorator to init ? | 14:14 |
| milan | https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/main.py#L390 to conf.py | 14:14 |
| milan | aarefiev, not sure I get you | 14:14 |
| aarefiev | here https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/main.py#L441 | 14:15 |
| sambetts | milan: sorry, i've been focused on a different window, can you TL;DR? | 14:15 |
| aarefiev | call periodic.periiodic | 14:15 |
| milan | sambetts, TL;DR we decorate periodics with conf which isn't parsed until service.run in main.py | 14:15 |
| milan | sambetts, the reason why the gate is still broken | 14:15 |
| aarefiev | sambetts: same issue like in ironic was, maybe you remember | 14:16 |
| *** mtanino has joined #openstack-ironic | 14:16 | |
| sambetts | milan: ah sigh... this is the same thing we had in ironic too with one of the decorator variables, lucasagomes helped fix it | 14:16 |
| aarefiev | yep | 14:16 |
| milan | aarefiev, so you suggest to remove the decorator and configure in init? | 14:17 |
| openstackgerrit | Merged openstack/pyghmi: Ignore KeyError on delete of missing sensor https://review.openstack.org/393778 | 14:17 |
| * milan feels that's too pretty&normal to do ;) | 14:17 | |
| * lucasagomes which we could pass a callback to the decorator | 14:17 | |
| creepy_owlet | lucasagomes++ we need to fix futurist to allow that | 14:17 |
| lucasagomes | but yeah, you have to make sure the imports are in a certain order so the methods doesn't get built prior to the config options being evaluated | 14:18 |
| *** khamtamtun has quit IRC | 14:18 | |
| lucasagomes | creepy_owlet, yeah | 14:18 |
| aarefiev | creepy_owlet doesn't like to rely on import order | 14:18 |
| milan | lucasagomes, creepy_owlet that's more obfuscated imo than calling the CONF() explicitly in conf.py | 14:19 |
| jrollinhatin | what if (both here and in ironic) we just read config files before even importing things | 14:19 |
| lucasagomes | that's what we try to do actually, in the service module | 14:19 |
| creepy_owlet | having code before imports is kind of non-pep8 | 14:19 |
| lucasagomes | oh | 14:19 |
| jrollinhatin | I care more about working code than pep8 | 14:20 |
| creepy_owlet | I mean, flake8 doesn't fail on it, but still | 14:20 |
| lucasagomes | now I understand, right, yeah... could work but looks odd | 14:20 |
| milan | jrollinhatin, I'd prefer that, if it's after the conf.py registering of opts etc | 14:20 |
| jrollinhatin | we have comments for a reason :) | 14:20 |
| lucasagomes | milan, you can reload() a module too | 14:20 |
| milan | lucasagomes, that's superobfuscated :D | 14:20 |
| lucasagomes | lol | 14:20 |
| creepy_owlet | no reload() please >_< I may be fine with running CONF at the very top in the end | 14:20 |
| jrollinhatin | creepy_owlet: +1 | 14:20 |
| milan | you can reload even upon periodics execution and rebuild the func on demand with fresh conf every time I guess :D | 14:21 |
| aarefiev | my idea was do this in conf.py , it one of the first import | 14:21 |
| *** causten has quit IRC | 14:22 | |
| aarefiev | and leave comment | 14:22 |
| milan | aarefiev, +1 on that, same jrollinhatin suggests | 14:22 |
| milan | I'd vote for that too | 14:22 |
| lucasagomes | yeah I think that would be the nicer way for now | 14:23 |
| *** causten has joined #openstack-ironic | 14:23 | |
| milan | https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/conf.py#L27 seems a cosy place for that CONF call ;) | 14:23 |
| *** shobhar has joined #openstack-ironic | 14:24 | |
| milan | or does it have to be after all the registering? | 14:24 |
| creepy_owlet | milan, rather https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/conf.py#L225 then | 14:24 |
| aarefiev | it should be after | 14:24 |
| milan | creepy_owlet, yeah | 14:24 |
| milan | OK | 14:24 |
| creepy_owlet | milan, I wonder how it's going to work with passing args to main() though | 14:25 |
| aarefiev | creepy_owlet: +1 | 14:25 |
| milan | ouch | 14:25 |
| aarefiev | yeah | 14:25 |
| milan | sys.args? O:-) | 14:26 |
| *** rajinir has quit IRC | 14:26 | |
| sambetts | can we add our own decorator that wraps the futurist one and can wait until the config is parsed? | 14:26 |
| creepy_owlet | milan, I think we use it in functional tests | 14:26 |
| * milan imagines the cycle of dependencies | 14:27 | |
| creepy_owlet | sambetts, ... or just take the value every time the periodic is invoked | 14:27 |
| *** amotoki has quit IRC | 14:27 | |
| creepy_owlet | yeah, makes sense | 14:27 |
| milan | build the function each time? | 14:27 |
| *** khamtamtun has joined #openstack-ironic | 14:27 | |
| *** vinsh has joined #openstack-ironic | 14:27 | |
| creepy_owlet | hmm, yeah, too tricky.. | 14:28 |
| sambetts | are we talking about variables pushed into the periodic function or passed into the futurist decorator? | 14:28 |
| *** khamtamtun has quit IRC | 14:28 | |
| *** ccamacho|lunch is now known as ccamacho | 14:28 | |
| *** jistr is now known as jistr|call | 14:29 | |
| *** jaosorior is now known as jaosorior_mtg | 14:29 | |
| aarefiev | to futurist decorator | 14:29 |
| * milan needs to check futurist decorator | 14:29 | |
| aarefiev | https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/main.py#L336 | 14:29 |
| *** khamtamtun has joined #openstack-ironic | 14:29 | |
| sambetts | creepy_owlet: can we do something like this http://paste.openstack.org/show/587902/ | 14:31 |
| *** jaosorior_mtg has quit IRC | 14:31 | |
| sambetts | creepy_owlet: obviously that is very psudo code, but I'm better at python than english ;) | 14:32 |
| creepy_owlet | sambetts, this will run once and do nothing | 14:32 |
| *** jaosorior_mtg has joined #openstack-ironic | 14:32 | |
| creepy_owlet | otherwise it's nice to somehow delay registration.. | 14:32 |
| creepy_owlet | sambetts, oh wait! | 14:32 |
| *** mjura has quit IRC | 14:32 | |
| creepy_owlet | I think futurist allows adding periodics later on | 14:32 |
| creepy_owlet | mmm, but they probably still have to be decorated, hmm | 14:32 |
| sambetts | creepy_owlet: a decorator is just a function, so you can call the futurist decorator like a function, which is what the futurist.periodic_function line is supposed to represent | 14:33 |
| creepy_owlet | sambetts, yeah, but you snipped will not run twice.. | 14:34 |
| sambetts | why do we need it to run twice? | 14:34 |
| sambetts | I might have missed something | 14:34 |
| milan | sambetts, first, you need to return the decorated func | 14:35 |
| creepy_owlet | sambetts, ah, I've confused "while" with "if" | 14:35 |
| milan | second, that would block the interpreter trying to "compile" the code | 14:35 |
| sambetts | milan: oh, thats my bad for not adding a return | 14:35 |
| * creepy_owlet should probably get rid of StackPlague first... | 14:35 | |
| *** khamtamtun has quit IRC | 14:36 | |
| sambetts | milan: the futurist.periodic_function line should have a return | 14:36 |
| milan | yeah but how a decorator can be kept running while you're loading a module that it's generating function in? | 14:37 |
| milan | s/while/after | 14:37 |
| milan | and not blocking everything else | 14:37 |
| * milan tries | 14:37 | |
| milan | :D | 14:37 |
| sambetts | milan: yeah, that I'm not sure about | 14:37 |
| creepy_owlet | note: we're not using real threading, so we need eventlet.sleep at best | 14:37 |
| sambetts | creepy_owlet: yeah | 14:38 |
| milan | one can yield then | 14:38 |
| milan | but | 14:38 |
| milan | we need to yield with a tmp function or what | 14:38 |
| milan | and then monkey-patch the module later | 14:38 |
| creepy_owlet | anyway, the code which will call that will block | 14:38 |
| milan | how about the monkey patch? | 14:38 |
| milan | we could return a dummy | 14:39 |
| milan | and after the thing is configured, monkey patch the dummy | 14:39 |
| milan | ? | 14:39 |
| milan | crazy? | 14:39 |
| * milan feels that way... lack of coffee :P | 14:39 | |
| *** d0ugal_ has quit IRC | 14:39 | |
| *** d0ugal has joined #openstack-ironic | 14:39 | |
| *** d0ugal has quit IRC | 14:39 | |
| *** d0ugal has joined #openstack-ironic | 14:39 | |
| jrollinhatin | any time you say monkey patch, trust it is crazy :) | 14:39 |
| milan | jrollinhatin, heh :D | 14:40 |
| milan | jrollinhatin, imho rebuilding the function every time is actually a sort of a hidden monkey patch :D | 14:40 |
| milan | so that should be crazier ;) | 14:41 |
| jrollinhatin | heh | 14:41 |
| * milan coffee | 14:41 | |
| aarefiev | wow, just went out for a moment :) | 14:41 |
| * creepy_owlet is already fine with moving periodic task creation wherever is needd | 14:41 | |
| milan | need to break the train of crazy ideas in my head :D | 14:41 |
| aarefiev | heh | 14:41 |
| *** rama_y has joined #openstack-ironic | 14:42 | |
| *** fragatina has quit IRC | 14:42 | |
| *** fragatina has joined #openstack-ironic | 14:42 | |
| aarefiev | creepy_owlet : and what about dtantsur | 14:42 |
| sambetts | how about something like this http://paste.openstack.org/show/587903/ | 14:43 |
| *** rajinir has joined #openstack-ironic | 14:43 | |
| creepy_owlet | aarefiev, he's picky, but he seems offline now, so let's quickly approve that | 14:43 |
| aarefiev | heh | 14:44 |
| milan | lol | 14:44 |
| creepy_owlet | sambetts, not so bad, we can try it | 14:44 |
| milan | sambetts, where would the config be read? | 14:45 |
| sambetts | milan: the config should be passed in the __new__ | 14:46 |
| sambetts | milan: when the futurist decorator is called | 14:46 |
| *** mgoddard_ has joined #openstack-ironic | 14:47 | |
| milan | __new__ is called once ThingWithPeriodics is subclassed? | 14:47 |
| milan | ah sry | 14:47 |
| sambetts | milan: no, __new__ is called before __init__ | 14:47 |
| milan | new is there to manipulate the instantiation, so basically hack the __dict__ | 14:48 |
| milan | right? | 14:48 |
| * lucasagomes thinks the patch switching the ipmitool jobs to xenial is fine now, https://review.openstack.org/392890 (if someone wants to take a peek) | 14:48 | |
| patchbot | patch 392890 - openstack-infra/project-config - Switch some ironic-ipa jobs to Ubuntu Xenial | 14:48 |
| sambetts | basically, we should probably have a flag in there to ensure that we only create the periodics the first time the class is used | 14:49 |
| sambetts | milan: ^ | 14:49 |
| aarefiev | could we just create wrapped methods before creating PeriodicWorker | 14:49 |
| *** mgoddard has quit IRC | 14:50 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector-specs: Add LLDP processing hook and new CLI commands https://review.openstack.org/374381 | 14:50 |
| milan | sambetts, and the class would be used as the decorator? | 14:51 |
| * milan lost | 14:51 | |
| milan | or the class is the service in this case? | 14:52 |
| *** jerrygb_ has joined #openstack-ironic | 14:52 | |
| *** wajdi has joined #openstack-ironic | 14:54 | |
| sambetts | milan: the class is the thing that current has the decorators on its methods | 14:55 |
| milan | I just can't see how it helps :-/ | 14:55 |
| sambetts | milan: it means the functions are only decorating on the first use of the class, not module load | 14:56 |
| *** mgoddard_ has quit IRC | 14:56 | |
| *** mgoddard has joined #openstack-ironic | 14:56 | |
| sambetts | milan: which we can gurantee is after the config file is loaded | 14:56 |
| milan | sambetts, so Service would instantiate that before registering with the worker? | 14:57 |
| sambetts | it will run at the first time someone does Service() | 14:58 |
| sambetts | so we want to load the config file before then | 14:58 |
| *** jerrygb has joined #openstack-ironic | 15:01 | |
| *** rbrud has quit IRC | 15:02 | |
| *** betherly has quit IRC | 15:03 | |
| *** jistr|call is now known as jistr | 15:03 | |
| *** jerrygb_ has quit IRC | 15:04 | |
| milan | sambetts, sounds like a self-monkeypatching class but why not | 15:05 |
| * milan still thinks about it | 15:05 | |
| *** sandriichenko has quit IRC | 15:05 | |
| *** ralonsoh_ is now known as ralonsoh | 15:09 | |
| sambetts | milan: its essentially creating a python metaclass, which is a "normal" thing in python | 15:10 |
| *** jistr is now known as jistr|biab | 15:10 | |
| milan | but you need the funcs in advance | 15:10 |
| milan | and somehow you need to stetattr with some reasonable name | 15:10 |
| *** causten has quit IRC | 15:11 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session glance and swift https://review.openstack.org/352289 | 15:11 |
| sambetts | the subclass of the metaclass that supports periodics just overrides the list_of_periodics, or we can do autodiscovery of the functions to turn into periodics using a decorator similar to the way we use the decorators for vendor_passthru in ironic | 15:12 |
| * mgould reads the last hour of scrollback | 15:13 | |
| mgould | I am now very confused | 15:13 |
| NobodyCam | Good Morning Ironic'ers | 15:13 |
| sambetts | o/ NobodyCam TGIF | 15:13 |
| mgould | also, is creepy_owlet the Mr Hyde to dtantsur's Dr Jekyll? | 15:13 |
| NobodyCam | And ofc. | 15:13 |
| mgould | morning NobodyCam | 15:13 |
| NobodyCam | :) | 15:13 |
| milan | sambetts, but the pastebin wasn't a metaclass | 15:13 |
| NobodyCam | LOL ... Morning sambetts mgould milan | 15:13 |
| milan | sambetts, it's exactly what I'm thinking about --- how to discover the func | 15:13 |
| creepy_owlet | morning NobodyCam | 15:13 |
| milan | morning NobodyCam! :) | 15:14 |
| NobodyCam | morning creepy_owlet :) | 15:14 |
| * creepy_owlet puts what mgould said to google translate | 15:14 | |
| NobodyCam | lol :) | 15:14 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Update description of stable states and fix state machine diagram https://review.openstack.org/390666 | 15:14 |
| creepy_owlet | double translations gives me "Owlet is a terrible Mr. Hyde to dtantsur in Dr. Jekyll" | 15:14 |
| *** jistr|biab is now known as jistr | 15:14 | |
| sambetts | milan: https://github.com/openstack/ironic/blob/master/ironic/drivers/base.py#L729 | 15:15 |
| creepy_owlet | now it has a part I understand: Owlet is a terrible (true, I am) | 15:15 |
| * milan feels like we're complicating it but unsure how to make it simple | 15:15 | |
| *** wajdi_ has joined #openstack-ironic | 15:15 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 15:15 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 15:15 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic: [WIP]Add REST API for volume connector and volume target operation https://review.openstack.org/214586 | 15:15 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic: Introduce new RPCs to support volume target operations https://review.openstack.org/285220 | 15:15 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic: Introduce VolumeTarget object https://review.openstack.org/285219 | 15:15 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic: Add volume_targets table to save target information https://review.openstack.org/285218 | 15:15 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 15:15 |
| sambetts | creepy_owlet: dr jekyll and my hyde is a novel about a scientist that has a split personality, one dr jekyll is a good guy, and mr hyde is evil | 15:16 |
| JayF | didn't he cause his own split with science? | 15:16 |
| creepy_owlet | hah, makes sense now! | 15:16 |
| JayF | wouldn't that mean Dr Jekyll wasn't a nice guy at all?! | 15:17 |
| *** wajdi__ has joined #openstack-ironic | 15:17 | |
| JayF | lol | 15:17 |
| sambetts | JayF: yup something like that ) | 15:17 |
| creepy_owlet | also morning JayF | 15:17 |
| sambetts | ;) | 15:17 |
| JayF | o/ | 15:17 |
| creepy_owlet | JayF, we might need your hw manager vision on https://review.openstack.org/391836 | 15:17 |
| patchbot | patch 391836 - ironic-python-agent - Add a new Hardware Manager for X710 network card | 15:17 |
| JayF | fun | 15:17 |
| openstackgerrit | Sergii Nozhka proposed openstack/ironic-inspector: Add a test for introspection abort action verification. https://review.openstack.org/393824 | 15:18 |
| *** causten has joined #openstack-ironic | 15:18 | |
| *** wajdi has quit IRC | 15:18 | |
| *** wajdi_ has quit IRC | 15:20 | |
| milan | sambetts, but we don't need that | 15:21 |
| milan | we could register the funcs while decorating them in the service after having read the conf | 15:22 |
| *** causten has quit IRC | 15:22 | |
| JayF | creepy_owlet: I reviewed it, pretty good first shot :D | 15:22 |
| milan | i.e moving the decorator away from the module | 15:22 |
| creepy_owlet | cool! I recommended this direction, wanted to make sure it's not in vain :) | 15:22 |
| sambetts | milan: thats what my paste does | 15:22 |
| milan | https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/main.py#L442 | 15:23 |
| JayF | creepy_owlet: this is like the A+++ example for an in-tree specific hardware manager | 15:23 |
| creepy_owlet | cool! | 15:23 |
| milan | sambetts, your paste does more :) | 15:24 |
| sambetts | milan: does it? do mean just do it in the __init__ instead? | 15:24 |
| sambetts | do you mean * | 15:24 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-specs: Update ironic-ml2-integration spec https://review.openstack.org/393660 | 15:24 |
| milan | sambetts, lemme pastebin | 15:25 |
| *** rcernin has quit IRC | 15:25 | |
| *** betherly has joined #openstack-ironic | 15:25 | |
| mgould | milan, sambetts: what problem are you trying to solve? Decorating a function, but only once CONF has been read? | 15:25 |
| *** rcernin has joined #openstack-ironic | 15:27 | |
| milan | mgould, ack | 15:28 |
| milan | sambetts, since the service doesn't do any magic w/ discovery of functions but instead keeps hardwired "links" we could just postpone the decoration to that moment: http://paste.openstack.org/show/587907/ | 15:29 |
| mgould | milan: could you always decorate it, check the CONF value in the decorator, error out if it's not set, and suck up the tiny runtime performance hit? | 15:29 |
| milan | mgould, it's rather default vs conf value | 15:30 |
| mgould | aaah | 15:30 |
| milan | but yeah we could do that | 15:30 |
| milan | too | 15:30 |
| mgould | so it might be legitimately called in a situation where that conf value isn't set? | 15:30 |
| sambetts | mgould: decorators are processed on module load, so the config file might not be read yet | 15:31 |
| milan | not really, so it's decorated nicely in the module while still being able to change the parameter of the decoration via conf file or sys.argv | 15:31 |
| mgould | OK | 15:31 |
| *** absubram has joined #openstack-ironic | 15:32 | |
| mgould | "decorate anyway, check the parameter in the body of the decorator" sounds more robust than careful fiddling with evaluation order, but I could be wrong | 15:32 |
| *** causten has joined #openstack-ironic | 15:32 | |
| milan | aarefiev, I think you suggested to kill the decorator in the module and decorate in the service in the very beginning of this discussion, correct? | 15:33 |
| milan | aarefiev, http://paste.openstack.org/show/587907/ < is that similar to what you suggested? | 15:33 |
| *** Nisha1 has joined #openstack-ironic | 15:35 | |
| *** khamtamtun has joined #openstack-ironic | 15:35 | |
| *** sabeen has joined #openstack-ironic | 15:36 | |
| *** wajdi__ has quit IRC | 15:36 | |
| *** wajdi has joined #openstack-ironic | 15:37 | |
| aarefiev | milan: something like that, but you need to make 2 call decorator(a=2)(method) | 15:38 |
| *** Guest13194 is now known as tesseract- | 15:38 | |
| *** khamtamtun has quit IRC | 15:38 | |
| milan | yeah | 15:39 |
| milan | but that call is up to the worker to perform | 15:39 |
| milan | but I might have missed some parts :) | 15:39 |
| milan | sambetts, creepy_owlet went guys? | 15:39 |
| milan | *wdyt | 15:40 |
| * creepy_owlet is out of context, sorry | 15:40 | |
| mgould | creepy_owlet: I think milan is trying to write Haskell in Python :-) | 15:40 |
| creepy_owlet | ah, cool | 15:40 |
| milan | creepy_owlet, http://paste.openstack.org/show/587907/ instead of registering the funcs before decorating them in a metaclass | 15:41 |
| milan | mgould, yeah, better 2 lines of Haskell than 50 of a metaclass :P | 15:41 |
| mgould | looks relatively sane to me... | 15:41 |
| creepy_owlet | milan, you don't need self.CONF, everything else looks ok | 15:41 |
| milan | creepy_owlet, yeah true | 15:41 |
| sambetts | I say lets give it a go and see how the patch looks, are there any other places in inspector where we decorate periodics | 15:42 |
| milan | would kill the decorators though in the place of those functions definition | 15:42 |
| *** wajdi_ has joined #openstack-ironic | 15:42 | |
| milan | sambetts, ack | 15:42 |
| mgould | +1 | 15:43 |
| aarefiev | sambetts: looks like 2 periodic tasks only | 15:45 |
| *** wajdi has quit IRC | 15:45 | |
| aarefiev | milan: decorator returns new wrapped method, which accept method, we are trying to decorate, as argument | 15:47 |
| aarefiev | or I'm missing something | 15:47 |
| milan | aarefiev, yeah, to call a decorated would be decorator(decorated, 1, 2, 3)(*decorated_args, **decorated_kwargs) | 15:48 |
| milan | but to have the worker process the decorated function the second call is omitted; just the worker might need to pass the *decorated_args, **decorated_kwargs | 15:49 |
| aarefiev | milan: periodics.periodic(spacing=CONF.clean_up_period)(periodic_clean_up) | 15:50 |
| milan | but as sambetts says, let's review it in a patch | 15:50 |
| milan | aarefiev, aaah | 15:50 |
| milan | aarefiev, yeah | 15:50 |
| milan | aarefiev, +1000 | 15:50 |
| milan | :D | 15:50 |
| * milan files a bug | 15:50 | |
| *** athomas has quit IRC | 15:55 | |
| *** rcernin has quit IRC | 15:56 | |
| *** dprince has quit IRC | 15:58 | |
| *** pcaruana has quit IRC | 15:58 | |
| *** tesseract- has quit IRC | 15:59 | |
| *** athomas has joined #openstack-ironic | 16:01 | |
| mgould | aarefiev: hurrah for currying | 16:01 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroup to api-ref https://review.openstack.org/393834 | 16:04 |
| milan | aarefiev, creepy_owlet, mgould, sambetts: https://bugs.launchpad.net/ironic-inspector/+bug/1639286 | 16:04 |
| openstack | Launchpad bug 1639286 in Ironic Inspector "Periodic tasks configuration not propagated from inspector.conf" [Undecided,New] | 16:04 |
| aarefiev | mgould: curry? :) | 16:05 |
| sambetts | aarefiev: https://en.wikipedia.org/wiki/Currying | 16:06 |
| *** 18VAABSMN has quit IRC | 16:06 | |
| aarefiev | sambetts: aha, thanks, I didn't know this term | 16:07 |
| * milan forgotten it already now he has nightmares coming back :P | 16:07 | |
| milan | thank you mgould :P | 16:07 |
| aarefiev | learn something new | 16:07 |
| aarefiev | milan: will you fix it ? | 16:08 |
| milan | aarefiev, ack | 16:08 |
| vsaienk0 | folks, ironic-multinode stuff is no longer blocked by devstack-gate (yesterday 3 devstack-gate patches were merged). Bash experts please help to review ironic part (6 patches ) https://review.openstack.org/#/q/topic:bug/1622653 | 16:08 |
| *** krtaylor has quit IRC | 16:08 | |
| milan | I'll try to have it ready till EOD | 16:08 |
| *** raginbaj- has joined #openstack-ironic | 16:09 | |
| TheJulia | vsaienk0: awesome, thank you for the update! | 16:10 |
| aarefiev | milan: thanks, note: gate is still broken, you need my patch :) | 16:11 |
| *** jaosorior_mtg is now known as jaosorior | 16:11 | |
| aarefiev | awesome vsaienk0 | 16:11 |
| milan | aarefiev, so I should rebase it on top of yours? | 16:11 |
| aarefiev | milan: it should be one patch | 16:12 |
| milan | aarefiev, OK | 16:12 |
| milan | aarefiev, so that one wasn't merged yet so I can just post new revision, right? | 16:13 |
| *** mjturek has quit IRC | 16:14 | |
| aarefiev | milan: yeah, we can't land it due this bug | 16:15 |
| milan | aarefiev, ack | 16:15 |
| * milan hacks | 16:15 | |
| *** dprince has joined #openstack-ironic | 16:21 | |
| aslezil | sambetts: as I am trying to write a spec for adding collector for ironic-inspector. The idea is to collect the data from biosdename utility, but biosdevname doesn't have the package for tiny core OS. Is it a good idea to create a package for biosdevname in tiny core OS | 16:21 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroup to api-ref https://review.openstack.org/393834 | 16:22 |
| sambetts | aslezil: there are a couple of packages missing from the tinycore linux repo, its quite difficult to get them added too, so for the missing ones we build them from source as part of the tinyIPA build script | 16:23 |
| JayF | sambetts: You are OK with the utility being added though if the build can get done? | 16:24 |
| JayF | I suggested Annie ask you before she went through all the effort to get it built :) | 16:24 |
| *** sabeen has quit IRC | 16:25 | |
| aslezil | sambetts: yeah I saw that few there in build-tinyipa.sh as JayF said is it ok to add there | 16:25 |
| sambetts | JayF: I'd be interested to see how big it is, a few months back someone else tried to add a new utility and it included some v.large lookup tables for vendor IDs to vendor names and products, which added ~20mb to tinyipa, which is currently only ~38mb | 16:27 |
| creepy_owlet | pci_devices, I guess? | 16:27 |
| sambetts | yeah | 16:27 |
| sambetts | so we worked around it by moving the ID processing to server side in inspector | 16:27 |
| JayF | I'm seeing how large it is installed on fedora | 16:28 |
| JayF | that should give some kind of idea I'd hope | 16:28 |
| *** bnemec is now known as beekneemech | 16:28 | |
| JayF | 58k for biosdevname, 72k for the lib it installed | 16:28 |
| JayF | so unless there's some giant dependency my fedora already had installed, that seems pretty small | 16:29 |
| sambetts | JayF: sounds reasonable then | 16:29 |
| *** jcoufal has quit IRC | 16:29 | |
| aslezil | Sambetts and Jay in that case can I give a try. | 16:30 |
| *** jcoufal has joined #openstack-ironic | 16:30 | |
| JayF | awesome :) | 16:30 |
| sambetts | great :D | 16:30 |
| JayF | will this be the first time you've installed any software from source? | 16:30 |
| JayF | that was an "oh wow" moment for me back in the day when I started working on linux :P | 16:30 |
| aslezil | yeah JayF: I know I will face lot of issues | 16:31 |
| JayF | I think you'll find it's easier than you expect | 16:31 |
| aslezil | Do you want to report a bug and work on it or try locally first | 16:31 |
| JayF | I mean, this work will all fall under the spec you're writing for the new collector | 16:32 |
| sambetts | hehe, the most confusing part of the build is likely to be building it into tinycore linux package, hopefully the examples of how to do it in the build-tinyipa script will help | 16:32 |
| JayF | so I'd still draft that spec first, and while waiting for review, you can science on the building of the package | 16:32 |
| aslezil | JayF: done and sambetts: thank you :) | 16:33 |
| sambetts | :) | 16:33 |
| * mgould checks the biosdevname build process | 16:34 | |
| mgould | looks like a standard "./configure; make; make install" | 16:34 |
| sambetts | the weird bit comes when you are having to do it in a chroot for building the ramdisk | 16:35 |
| mgould | yeah | 16:35 |
| JayF | that's what cheating is for | 16:35 |
| mgould | you might need to pass some flags to ./configure, maybe? | 16:35 |
| JayF | ^c^v the existing code, poke it with a stick until it works for biosdevname | 16:35 |
| JayF | lol | 16:36 |
| sambetts | hahaha yup, best way | 16:36 |
| aslezil | he he | 16:36 |
| mgould | https://www.gitbook.com/book/tra38/essential-copying-and-pasting-from-stack-overflow/details | 16:36 |
| JayF | I only steal code from known trusted developers | 16:36 |
| JayF | lol | 16:36 |
| *** pester has joined #openstack-ironic | 16:37 | |
| *** shobhar has quit IRC | 16:37 | |
| *** aNuposic has joined #openstack-ironic | 16:37 | |
| sambetts | ;) | 16:38 |
| *** pester has quit IRC | 16:39 | |
| *** fxpester has quit IRC | 16:40 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Allow to configure cache clean up period https://review.openstack.org/392943 | 16:41 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/python-ironicclient: Avoid string interpolation in logging calls https://review.openstack.org/376343 | 16:41 |
| * lucasagomes just updated the commit message | 16:42 | |
| milan | aarefiev, sambetts ^^ | 16:42 |
| milan | let's see what jenkins thinks about it | 16:43 |
| *** jaosorior has quit IRC | 16:43 | |
| aarefiev | yeah, do we want to keep decorators ? | 16:45 |
| milan | aarefiev, Oops | 16:45 |
| * milan removes | 16:45 | |
| aarefiev | maybe move to Service | 16:46 |
| milan | the funcs? | 16:46 |
| aarefiev | yep | 16:46 |
| milan | hmmm... but those functions don't require any "state" managed by Service, or? | 16:47 |
| *** sborkows has quit IRC | 16:48 | |
| aarefiev | This is just a thought aloud on Fr evening | 16:48 |
| aarefiev | :) | 16:48 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Simplify base interfaces in ironic.drivers.base https://review.openstack.org/393848 | 16:49 |
| milan | aarefiev, :) | 16:49 |
| creepy_owlet | first weird patch of the driver comp ^^^ | 16:49 |
| mgould | number 3 will astound us? | 16:49 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Allow to configure cache clean up period https://review.openstack.org/392943 | 16:49 |
| openstackgerrit | Sergii Nozhka proposed openstack/ironic-inspector: Add a test for introspection abort action verification. https://review.openstack.org/393824 | 16:50 |
| milan | mgould, heh, yeah, I'm worried creepy_owlet taking over the world, initiated by that patch ;) | 16:51 |
| creepy_owlet | \o/ | 16:51 |
| creepy_owlet | power to owls! | 16:51 |
| milan | :D | 16:51 |
| *** mjturek has joined #openstack-ironic | 16:51 | |
| JayF | who? power to who? who? | 16:52 |
| *** krtaylor has joined #openstack-ironic | 16:53 | |
| mgould | groan | 16:53 |
| creepy_owlet | :D | 17:02 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Create noop interfaces for use with hardware types https://review.openstack.org/393867 | 17:02 |
| *** ralonsoh has quit IRC | 17:04 | |
| *** e0ne has quit IRC | 17:07 | |
| *** e0ne_ has joined #openstack-ironic | 17:07 | |
| *** e0ne_ has quit IRC | 17:11 | |
| *** fragatina has quit IRC | 17:11 | |
| *** jpich has quit IRC | 17:13 | |
| jcook | jrollinhatin how do I bump microversion: https://review.openstack.org/#/c/350831/3/ironic/api/controllers/v1/node.py ? | 17:17 |
| patchbot | patch 350831 - ironic - Add API and conductor methods for [un]rescue | 17:17 |
| jcook | api version here: https://review.openstack.org/#/c/350831/3/ironic/conductor/manager.py | 17:17 |
| patchbot | patch 350831 - ironic - Add API and conductor methods for [un]rescue | 17:17 |
| sambetts | haha I like "unrescue" I have visions of people breaking instance deliberately using openstack server unrescue :-P | 17:19 |
| JayF | jcook: there's a good patch up doing it now | 17:19 |
| JayF | jcook: lemme find it | 17:19 |
| *** e0ne has joined #openstack-ironic | 17:19 | |
| jcook | JayF thanks | 17:19 |
| *** ipukha has quit IRC | 17:20 | |
| JayF | actually pretty sure that patch is doing it wrong, so going to find another patch | 17:20 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Simplify base interfaces in ironic.drivers.base https://review.openstack.org/393848 | 17:20 |
| JayF | lol | 17:20 |
| JayF | jcook: https://github.com/openstack/ironic/commit/a7310377e27bbce454b8a7a6b943cc5b7a45bbad | 17:21 |
| *** ChubYann has joined #openstack-ironic | 17:21 | |
| *** ohamada has quit IRC | 17:23 | |
| jcook | JayF so it's just a rst update? | 17:23 |
| *** e0ne has quit IRC | 17:23 | |
| JayF | jcook: https://github.com/openstack/ironic/commit/a7310377e27bbce454b8a7a6b943cc5b7a45bbad#diff-fcb18843d1a9ec5117a71319e3d89151R306 | 17:24 |
| JayF | see those guard methods? making sure if you're not a new enough microversion you can't see the new api endpoint | 17:24 |
| jcook | ah | 17:25 |
| JayF | https://github.com/openstack/ironic/commit/a7310377e27bbce454b8a7a6b943cc5b7a45bbad#diff-c8b579c95b86075825625f02118fe4d0R416 being the guard method itself | 17:25 |
| jcook | cool thanks | 17:26 |
| JayF | no problem, helped me to find an issue in the other review that also is adding a microversion | 17:29 |
| lucasagomes | folks, calling it a day! | 17:31 |
| lucasagomes | have a great evening all, enjoy the weekend | 17:31 |
| sambetts | o/ lucasagomes | 17:32 |
| *** lucasagomes is now known as lucas-afk | 17:32 | |
| lucas-afk | o/ | 17:32 |
| JayF | o/ | 17:35 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Create noop interfaces for use with hardware types https://review.openstack.org/393867 | 17:36 |
| *** d0ugal has quit IRC | 17:39 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 17:39 |
| JayF | sambetts: that patch sounds interesting | 17:40 |
| JayF | sambetts: I suspect you have something in mind though, like what's the next step? | 17:40 |
| sambetts | JayF: basically it remodels the way we simulate the network with the BM/VMs so that it mimics and interactes with neutron like more a real world physical network | 17:42 |
| JayF | sambetts: I'm just saying, this is something that enables folks to do cool things | 17:42 |
| JayF | sambetts: which made me assume you had a cool thing in mind :D | 17:42 |
| *** jkilpatr has quit IRC | 17:42 | |
| *** jkilpatr has joined #openstack-ironic | 17:43 | |
| sambetts | JayF: ;) right now we're working pretty hard to get the rest of the multitenancy stuff done like bonding and trunks etc so it makes it much nicer to work with | 17:44 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Support defining and loading hardware types https://review.openstack.org/336626 | 17:44 |
| creepy_owlet | ok, enough for today, see you on Monday :) | 17:44 |
| JayF | ah | 17:44 |
| sambetts | JayF: also it should reduce the hackyness of our grenade plugin too | 17:44 |
| JayF | so like actaully being able to bond interfaces in vms | 17:44 |
| JayF | for instance | 17:44 |
| JayF | because it's just a linux bridge you could emulate stuff like that | 17:44 |
| JayF | (maybe? Can you? I'm now curious...) | 17:44 |
| *** creepy_owlet is now known as dtantsur|afk | 17:44 | |
| sambetts | JayF: OVS lets to do it so we'll be using that for the virtual switch, but we'll be using a separate OVS bridge from the ones given to neutron to prevent stomping on toes | 17:45 |
| JayF | ah, okay | 17:45 |
| JayF | I might take reviewing that patch as an opportunity to understand how that works better | 17:46 |
| *** mjturek has quit IRC | 17:46 | |
| JayF | right now I treat it mostly as magic which is a copout given I actually know a lot about linux networking | 17:46 |
| *** srobert has joined #openstack-ironic | 17:46 | |
| openstackgerrit | Merged openstack/python-ironicclient: Avoid string interpolation in logging calls https://review.openstack.org/376343 | 17:46 |
| sambetts | its a little bit of a mad patch because its replacing a bunch of hacky stuff :) but I personally hope it actually results in a simpler devstack script and a less hacky simulation | 17:47 |
| sambetts | I also hope to make it easier for people like myself to hook up real hardware networks so we don't have to do too much extra plumbing in our third party CI job scripts | 17:48 |
| mgould | sambetts: sounds cool | 17:50 |
| mgould | having read the commit message, I'd like to double down on that statement :-) | 17:56 |
| *** Sukhdev has joined #openstack-ironic | 17:58 | |
| sambetts | mgould: thanks :) I hope it lives up to expectations ;) | 17:59 |
| *** harlowja has quit IRC | 18:00 | |
| *** Nisha1 has quit IRC | 18:01 | |
| sambetts | mgould: I've tested it locally but I'm waiting to see what jenkins says ;) | 18:02 |
| *** mjturek has joined #openstack-ironic | 18:03 | |
| mgould | I have an ironic node, with values set in properties/cpus, properties/local_gb and properties/memory_mb | 18:03 |
| *** harlowja has joined #openstack-ironic | 18:03 | |
| mgould | however, nova hypervisor-show [uuid] thinks it has 0 cpus, 0 ram and 0 disk | 18:04 |
| mgould | how can I fix that? | 18:04 |
| sambetts | mgould: it can take a couple of minutes for that information to sync | 18:04 |
| mgould | ah, OK | 18:04 |
| * mgould cracks another beer while he's waiting | 18:04 | |
| sambetts | mgould: how long has it been sync you setup the ironic node? | 18:04 |
| sambetts | s/sync/since* | 18:05 |
| mgould | not sure, but several minutes | 18:05 |
| *** dougs1 has quit IRC | 18:05 | |
| sambetts | mgould: hmm ok, normal time to sync is ~2mins, the other thing to check for is if the node validates, you can check by doing ironic node-validate <uuid/name> | 18:06 |
| mgould | ah, lots of validation failures | 18:06 |
| mgould | missing ramdisk, kernel and image_source parameters in instance_info | 18:07 |
| sambetts | thats expected, becuase nova will add those | 18:07 |
| mgould | OK | 18:07 |
| sambetts | you'll looking for things like missing credentialls or ramdisk kernel/ramdisk etc | 18:07 |
| sambetts | your* | 18:07 |
| mgould | http://paste.openstack.org/show/587922/ | 18:08 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroup to api-ref https://review.openstack.org/393834 | 18:08 |
| sambetts | I was about to say, and your paste has just confirm this for me, the next thing to check is the nodes state, it needs to be in avaiable, and not in maintenance to advertise resources to nova, your node is in managable | 18:09 |
| *** links has joined #openstack-ironic | 18:10 | |
| mgould | aaargh | 18:10 |
| mgould | thanks! | 18:10 |
| sambetts | :D | 18:10 |
| *** links has quit IRC | 18:10 | |
| mgould | OK, so now I wait a couple of minutes for the info to propagate to nova, and then I can nova boot? | 18:11 |
| sambetts | yup | 18:11 |
| mgould | cool | 18:11 |
| openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 18:11 |
| sambetts | knowing the steps to work through all comes with experience debugging :) thats why we've started writing some of them up in the docs | 18:11 |
| mgould | ah, I see that's point #3 in http://docs.openstack.org/developer/ironic/deploy/troubleshooting.html, which my eyes had skipped over :-/ | 18:12 |
| sambetts | :-P | 18:12 |
| * mgould runs "nova boot" | 18:13 | |
| sambetts | Sukhdev: QQ, on the security groups patch, what happens if a user sets port_security_enabled=False on the provisioning network, but still configures a security group in the config file?? | 18:14 |
| sambetts | Sukhdev: I hadn't had a chance to reply yet, but thats what I meant by my question on the patch | 18:14 |
| Sukhdev | sambetts : where is port_security_enabled set? | 18:16 |
| jcook | JayF stupid question, how do I run the Ironic unit tests. I know I've ran them local before but either something in the tree changed or I'm have a neural failure and it's not popping out at me | 18:16 |
| sambetts | Sukhdev: on the neutron network, its one of its properties | 18:17 |
| JayF | jcook: tox -epy27 | 18:17 |
| jcook | JayF that should totes be in the README | 18:17 |
| jcook | JayF thanks | 18:17 |
| Sukhdev | sambetts : oh I see - now I understand the question. | 18:17 |
| Sukhdev | sambetts : that flag supercedes | 18:18 |
| *** harlowja has quit IRC | 18:18 | |
| sambetts | Sukhdev: if that is set to false, and we try to set a security group on it, will it error? | 18:18 |
| sambetts | Sukhdev: do we need to catch that in ironic? | 18:19 |
| Sukhdev | sambetts : hang on there are two flags - we are talking about two flags | 18:19 |
| *** daemontool has quit IRC | 18:19 | |
| Sukhdev | sambetts : I check for one flag in the ML2 and that superceeds - I think you are talking about yet another flag | 18:20 |
| sambetts | Yes there is a flag on the network: https://wiki.openstack.org/wiki/Neutron/ML2PortSecurityExtensionDriver | 18:20 |
| sambetts | and there is a flag on the port | 18:20 |
| sambetts | I was wondering if we need to check for the flag on the network in Ironic, and if it'll error if we try to use a security group when security groups are disabled | 18:22 |
| sambetts | I think it might error, because it used to be a bug in Nova, that even if security groups were disabled it would try to force one on, resulting in a failure to create an instance | 18:23 |
| Sukhdev | sambetts : In ML2 driver, if the vnic_type is baremetal, I ignore this flag | 18:23 |
| sambetts | Sukhdev: its not at that stage that it needs the check, its at the port update/create | 18:24 |
| Sukhdev | sambetts : right - if one disables that flag and then specifies --security-group as a parameter to nova boot - it will error out - | 18:25 |
| sambetts | Sukhdev: and i think the same thing will happen in ironic, if the flag is disabled, but secuirty groups are configured in ironic | 18:26 |
| sambetts | do we treat that as a configuration error and error out all the deployments, or check the flag and throw a warning instead | 18:26 |
| Sukhdev | sambetts : so, there are two things here - | 18:27 |
| Sukhdev | sambetts : 1) security groups for tenant networks and | 18:27 |
| Sukhdev | 2) for admin networks (provisioning and cleaning) - | 18:28 |
| sambetts | right, I'm talking about provisioning and cleaning networks | 18:28 |
| Sukhdev | for 1) we are good - and do not need to anything additional | 18:28 |
| *** betherly has quit IRC | 18:28 | |
| *** johnthetubaguy has quit IRC | 18:28 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/393791 | 18:28 |
| *** johnthetubaguy_ has joined #openstack-ironic | 18:28 | |
| Sukhdev | for 2, the network as well as security groups are created by the admin - | 18:29 |
| Sukhdev | why would they create security groups and then set them to disable - | 18:29 |
| *** johnthetubaguy_ is now known as johnthetubaguy | 18:30 | |
| Sukhdev | however, if they do, in my present implementation, I honor the configured security groups (if they are configured) | 18:30 |
| sambetts | Sukhdev: 2 different operators that don't talk to each other/disables it at a later date but doesn't want to shutdown ironic to remove the configured security groups | 18:30 |
| Sukhdev | sambetts: I see your point | 18:30 |
| *** harlowja has joined #openstack-ironic | 18:30 | |
| sambetts | Sukhdev: neutron won't let the port create happen if that flag is disabled and security groups is in the port body | 18:31 |
| sambetts | Sukhdev: thats the error we need to catch or avoid | 18:31 |
| sambetts | so it'll error in the configure_provision_network function | 18:31 |
| Sukhdev | sambetts: right | 18:32 |
| milan | sambetts, mind a second w+2+ https://review.openstack.org/#/c/392943/ | 18:32 |
| patchbot | patch 392943 - ironic-inspector - Allow to configure cache clean up period | 18:32 |
| milan | ? | 18:32 |
| milan | seems we might have a fix for Inspector gate :D | 18:32 |
| sambetts | milan: :D | 18:32 |
| milan | thaks to dtantsur|afk | 18:32 |
| milan | for first +2 | 18:32 |
| mgould | \o/ | 18:33 |
| milan | aarefiev, \o/ :) | 18:33 |
| sambetts | Sukhdev: /me -> home now, but we should continue this conversion on Monday in the meeting | 18:33 |
| sambetts | Good night all | 18:34 |
| Sukhdev | sambetts : wait a minute - Is this flag available to operators to set? | 18:34 |
| mgould | sambetts: good night, have a good weekend! | 18:34 |
| Sukhdev | sambetts : good night | 18:34 |
| milan | night sambetts! :) | 18:34 |
| sambetts | Sukhdev: hmm I think so | 18:34 |
| sambetts | Sukhdev: I have set it before for stuff I think | 18:34 |
| *** sambetts is now known as sambetts|afk | 18:35 | |
| sambetts|afk | o/ Sukhdev, milan, mgould | 18:35 |
| Sukhdev | sambetts|afk : I will check - you have a good night | 18:35 |
| * mgould gets another No Valid Host error, but I'm pretty sure this one's legit and means "you need to write some code now" :-) | 18:36 | |
| mgould | gnite all, have a good weekend! | 18:37 |
| *** mgould is now known as mgould|afk | 18:37 | |
| milan | good night mgould|afk! :) | 18:37 |
| * milan out too | 18:37 | |
| milan | good night Ironic! :) | 18:37 |
| *** milan has quit IRC | 18:38 | |
| *** sabeen has joined #openstack-ironic | 18:42 | |
| *** jerrygb has quit IRC | 18:45 | |
| *** jerrygb has joined #openstack-ironic | 18:45 | |
| *** jkilpatr has quit IRC | 18:50 | |
| *** causten has quit IRC | 18:59 | |
| jcook | hmm, ironic.tests.unit.api.v1.test_nodes is using the latest microversion | 19:01 |
| rajinir | Need help with using non-default image for the thirdparty builds, other than cirros. Any idea what the variables ? Any help or pointers? krtaylor: Sambetts: | 19:01 |
| rajinir | I have these set but they didn't quiet work as I expected | 19:02 |
| rajinir | https://www.irccloud.com/pastebin/9xFK1zCr/ironic-image | 19:02 |
| jrollinhatin | jcook: you can pass different versions e.g. https://github.com/openstack/ironic/blob/master/ironic/tests/unit/api/v1/test_nodes.py#L150 | 19:03 |
| *** marlinc has quit IRC | 19:03 | |
| *** jkilpatr has joined #openstack-ironic | 19:04 | |
| jcook | jrollinhatin oh, IC. I was confused by RPC version be 1.35 and api version being 24 | 19:06 |
| jrollinhatin | mmm | 19:06 |
| jcook | so I should just have to pass 1.24 | 19:06 |
| jrollinhatin | yep | 19:07 |
| jcook | jrollinhatin the same test file is passing versions of 1.4 and such, but max version is 1.24 | 19:08 |
| jrollinhatin | jcook: yes, we test various versions here | 19:09 |
| jrollinhatin | for various reasons | 19:09 |
| JayF | for instance you might wanna add a test confirming that resceu *doesn't* work for an older microversion | 19:10 |
| JayF | right? | 19:10 |
| jcook | but 1.4 is > 1.24 the max version | 19:10 |
| jrollinhatin | is it? | 19:10 |
| JayF | not 1.2.4 | 19:10 |
| JayF | 1.4 | 19:10 |
| jrollinhatin | is 4 > 24? | 19:10 |
| JayF | 4 < 24 | 19:11 |
| jrollinhatin | 1.4 != 1.40 :) | 19:11 |
| jcook | oh...thanks lol | 19:11 |
| jrollinhatin | jcook: think of it as (1, 4) < (1, 24) | 19:11 |
| jrollinhatin | that's the actual comparison we use in python | 19:11 |
| jcook | yeah, I got it heh | 19:11 |
| jcook | (－‸ლ) | 19:12 |
| jrollinhatin | :D | 19:12 |
| jrollinhatin | that's interesting though, I wonder how many people hit that wall | 19:12 |
| jcook | counting is hard | 19:12 |
| jcook | so yay, that fixed it | 19:13 |
| *** jerrygb has quit IRC | 19:14 | |
| *** jerrygb has joined #openstack-ironic | 19:16 | |
| *** shobhar has joined #openstack-ironic | 19:19 | |
| openstackgerrit | Jesse J. Cook proposed openstack/ironic: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 19:20 |
| *** jvgrant__ has joined #openstack-ironic | 19:24 | |
| jrollinhatin | okay, I think I mostly have trello sorted: https://trello.com/b/ROTxmGIc/ironic-newton-priorities | 19:26 |
| jrollinhatin | rather, https://trello.com/b/ROTxmGIc/ironic-ocata-priorities | 19:27 |
| jrollinhatin | (both links will work) | 19:27 |
| *** jvgrant_ has quit IRC | 19:27 | |
| jrollinhatin | needs some assignees, tags, etc, but everything should be there and most should be up to date | 19:27 |
| *** jcoufal has quit IRC | 19:28 | |
| openstackgerrit | Merged openstack/ironic-inspector: Allow to configure cache clean up period https://review.openstack.org/392943 | 19:30 |
| *** fragatina has joined #openstack-ironic | 19:44 | |
| *** marlinc has joined #openstack-ironic | 19:45 | |
| openstackgerrit | Oleksandr Berezovskyi proposed openstack/ironic-python-agent: Add possibility to inspect systems with multiple CPUs https://review.openstack.org/391290 | 19:52 |
| *** srobert_ has joined #openstack-ironic | 20:00 | |
| *** srobert has quit IRC | 20:01 | |
| *** jerrygb_ has joined #openstack-ironic | 20:01 | |
| *** fragatina has quit IRC | 20:03 | |
| *** jerrygb has quit IRC | 20:05 | |
| *** e0ne has joined #openstack-ironic | 20:05 | |
| *** jerrygb_ has quit IRC | 20:06 | |
| *** vinsh has quit IRC | 20:06 | |
| *** vinsh has joined #openstack-ironic | 20:06 | |
| *** dprince has quit IRC | 20:12 | |
| *** rbudden_ has joined #openstack-ironic | 20:22 | |
| *** kvcobb has joined #openstack-ironic | 20:22 | |
| *** rbudden has quit IRC | 20:23 | |
| *** rbudden_ is now known as rbudden | 20:23 | |
| *** jkilpatr has quit IRC | 20:25 | |
| *** athomas has quit IRC | 20:25 | |
| *** krtaylor has quit IRC | 20:27 | |
| kvcobb | Good afternoon, Folks. I'm having an issue with spinning up an Ironic instance. Wondering if someone has seen this before, so I spin up an instance with my BM flavor selected. The server powers down, gets set for PXE, powers back up, sees PXE and gets the deploy image and ramdisk, shortly after it fails with these errors in the conductor logs: http://paste.openstack.org/show/587940/ Anybody see that before? This a problem with the | 20:27 |
| kvcobb | images I created (I build my image with the partition image instructions rather than whole disk))? | 20:27 |
| *** derekh has joined #openstack-ironic | 20:27 | |
| *** shobhar has quit IRC | 20:28 | |
| *** shobhar has joined #openstack-ironic | 20:28 | |
| *** oberezovskyi has joined #openstack-ironic | 20:33 | |
| *** jjohnson2 has quit IRC | 20:36 | |
| *** oberezovskyi has quit IRC | 20:42 | |
| *** zzzeek_ has joined #openstack-ironic | 20:46 | |
| *** jkilpatr has joined #openstack-ironic | 20:46 | |
| *** zzzeek has quit IRC | 20:48 | |
| *** jheroux has quit IRC | 20:57 | |
| *** ijw has joined #openstack-ironic | 21:01 | |
| *** srobert_ has quit IRC | 21:04 | |
| *** sabeen has quit IRC | 21:11 | |
| *** absubram has quit IRC | 21:13 | |
| *** shuquan has joined #openstack-ironic | 21:18 | |
| *** shuquan has quit IRC | 21:23 | |
| *** mattcburns has joined #openstack-ironic | 21:24 | |
| *** jrollinhatin is now known as jroll | 21:36 | |
| *** snozhka has quit IRC | 21:39 | |
| *** sdake has joined #openstack-ironic | 21:42 | |
| *** jerrygb has joined #openstack-ironic | 21:51 | |
| *** mxburns has joined #openstack-ironic | 21:52 | |
| *** jerrygb has quit IRC | 21:56 | |
| *** wajdi has joined #openstack-ironic | 21:57 | |
| *** derekh has quit IRC | 21:58 | |
| *** wajdi_ has quit IRC | 22:00 | |
| *** wajdi has quit IRC | 22:01 | |
| *** derekh has joined #openstack-ironic | 22:09 | |
| *** snozhka has joined #openstack-ironic | 22:11 | |
| openstackgerrit | Félix Bouliane proposed openstack/ironic: WIP- Add virtualpdu to ironic devstack plugin https://review.openstack.org/388154 | 22:14 |
| *** e0ne_ has joined #openstack-ironic | 22:17 | |
| *** e0ne has quit IRC | 22:21 | |
| *** bfournie has quit IRC | 22:29 | |
| *** rajinir has quit IRC | 22:36 | |
| *** sdake has quit IRC | 22:37 | |
| *** sdake has joined #openstack-ironic | 22:40 | |
| *** aNuposic has quit IRC | 22:50 | |
| *** jerrygb has joined #openstack-ironic | 22:52 | |
| *** jerrygb has quit IRC | 22:58 | |
| *** ppiela has quit IRC | 23:06 | |
| *** bfournie has joined #openstack-ironic | 23:29 | |
| *** mattcburns has quit IRC | 23:30 | |
| *** rama_y has quit IRC | 23:35 | |
| *** mtanino has quit IRC | 23:35 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/392731 | 23:50 |
| *** sabeen has joined #openstack-ironic | 23:56 | |
| *** Nisha1 has joined #openstack-ironic | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!