*** causten has joined #openstack-ironic | 00:00 | |
*** penick has quit IRC | 00:02 | |
*** causten has quit IRC | 00:05 | |
*** rama_y has quit IRC | 00:05 | |
*** bfournie has joined #openstack-ironic | 00:09 | |
*** yangyapeng has quit IRC | 00:11 | |
*** bfournie has quit IRC | 00:12 | |
*** yangyapeng has joined #openstack-ironic | 00:12 | |
*** yufei has joined #openstack-ironic | 00:18 | |
*** yufei has quit IRC | 00:18 | |
*** [1]rpioso has quit IRC | 00:25 | |
*** hw_wutianwei1 has joined #openstack-ironic | 00:54 | |
*** phuongnh has joined #openstack-ironic | 01:03 | |
*** Sukhdev has quit IRC | 01:03 | |
*** litao__ has joined #openstack-ironic | 01:06 | |
*** stendulker has joined #openstack-ironic | 01:12 | |
*** leyal has quit IRC | 01:31 | |
*** lihi has quit IRC | 01:33 | |
*** leyal has joined #openstack-ironic | 01:33 | |
*** lihi has joined #openstack-ironic | 01:33 | |
*** ijw has quit IRC | 01:33 | |
*** causten has joined #openstack-ironic | 01:40 | |
*** crushil has joined #openstack-ironic | 01:46 | |
*** stendulker has quit IRC | 02:15 | |
*** srobert has joined #openstack-ironic | 02:20 | |
*** rama_y has joined #openstack-ironic | 02:38 | |
*** rama_y has quit IRC | 02:42 | |
*** ijw has joined #openstack-ironic | 03:12 | |
*** ijw has quit IRC | 03:17 | |
*** TuanLA has joined #openstack-ironic | 03:21 | |
*** yangyapeng has quit IRC | 03:32 | |
*** yangyapeng has joined #openstack-ironic | 03:32 | |
*** yangyapeng has quit IRC | 03:37 | |
*** mat128 has joined #openstack-ironic | 03:37 | |
*** stendulker has joined #openstack-ironic | 03:39 | |
*** mat128 has quit IRC | 03:41 | |
*** Sukhdev has joined #openstack-ironic | 03:47 | |
*** crushil has quit IRC | 03:57 | |
*** krtaylor has quit IRC | 04:00 | |
*** krtaylor has joined #openstack-ironic | 04:02 | |
*** krtaylor has quit IRC | 04:08 | |
*** ianychoi_ has joined #openstack-ironic | 04:20 | |
*** stendulker_ has joined #openstack-ironic | 04:22 | |
*** sasha21 has joined #openstack-ironic | 04:23 | |
*** Sukhdev has quit IRC | 04:29 | |
*** stendulker has quit IRC | 04:29 | |
*** sasha2 has quit IRC | 04:29 | |
*** yolanda has quit IRC | 04:29 | |
*** ianychoi has quit IRC | 04:29 | |
*** jrist has quit IRC | 04:29 | |
*** leifmadsen has quit IRC | 04:29 | |
*** jcook has quit IRC | 04:29 | |
*** sambetts|afk has quit IRC | 04:29 | |
*** lhinds has quit IRC | 04:29 | |
*** honza has quit IRC | 04:29 | |
*** honza has joined #openstack-ironic | 04:29 | |
*** honza is now known as Guest85888 | 04:30 | |
*** sambetts_ has joined #openstack-ironic | 04:30 | |
*** lhinds has joined #openstack-ironic | 04:30 | |
*** leifmadsen has joined #openstack-ironic | 04:32 | |
*** jcook has joined #openstack-ironic | 04:32 | |
*** radez has quit IRC | 04:33 | |
*** slagle has quit IRC | 04:34 | |
*** trown|outtypewww has quit IRC | 04:34 | |
*** crushil has joined #openstack-ironic | 04:34 | |
*** trown has joined #openstack-ironic | 04:35 | |
*** udesale has joined #openstack-ironic | 04:36 | |
*** jrist has joined #openstack-ironic | 04:36 | |
*** yolanda has joined #openstack-ironic | 04:37 | |
*** racedo has quit IRC | 04:37 | |
*** dpawar has joined #openstack-ironic | 04:39 | |
*** radez has joined #openstack-ironic | 04:39 | |
*** slagle has joined #openstack-ironic | 04:41 | |
*** ankkumar has joined #openstack-ironic | 04:41 | |
*** dpawar has quit IRC | 04:44 | |
*** dpawar has joined #openstack-ironic | 04:45 | |
*** jaianshu has joined #openstack-ironic | 04:54 | |
openstackgerrit | Merged openstack/ironic-specs master: Move completed specs to pike 9.0 & 9.1 https://review.openstack.org/505365 | 04:56 |
---|---|---|
openstackgerrit | Merged openstack/ironic master: Add documentation describing each Ironic state https://review.openstack.org/496719 | 04:59 |
*** marios has joined #openstack-ironic | 05:12 | |
*** causten has quit IRC | 05:13 | |
*** phuongnh has quit IRC | 05:19 | |
*** crushil has quit IRC | 05:21 | |
*** aparnav has joined #openstack-ironic | 05:30 | |
*** vsaienk0 has joined #openstack-ironic | 05:41 | |
openstackgerrit | zenghui.shi proposed openstack/ironic-specs master: Hardware interface for BIOS configuration https://review.openstack.org/496481 | 05:42 |
*** radez has quit IRC | 05:52 | |
*** ChubYann has quit IRC | 05:57 | |
*** mjura has joined #openstack-ironic | 06:04 | |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Switch to none auth for standalone mode https://review.openstack.org/359061 | 06:09 |
*** jtomasek has quit IRC | 06:18 | |
*** vsaienk0 has quit IRC | 06:18 | |
*** vsaienk0 has joined #openstack-ironic | 06:19 | |
*** fbouliane has quit IRC | 06:19 | |
*** zshi__ has joined #openstack-ironic | 06:20 | |
*** fbouliane has joined #openstack-ironic | 06:22 | |
*** dpawar has quit IRC | 06:34 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add uWSGI support https://review.openstack.org/507011 | 06:38 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [devstack] Switch ironic to uWSGI https://review.openstack.org/507067 | 06:38 |
*** rcernin has joined #openstack-ironic | 06:39 | |
*** e0ne has joined #openstack-ironic | 06:40 | |
*** hjensas has joined #openstack-ironic | 06:40 | |
*** hjensas has quit IRC | 06:40 | |
*** hjensas has joined #openstack-ironic | 06:40 | |
*** hjensas has quit IRC | 06:41 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add uWSGI support https://review.openstack.org/507011 | 06:43 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [devstack] Switch ironic to uWSGI https://review.openstack.org/507067 | 06:43 |
*** charcol has quit IRC | 06:50 | |
*** dpawar has joined #openstack-ironic | 06:51 | |
*** e0ne has quit IRC | 07:08 | |
*** jaianshu has quit IRC | 07:11 | |
*** causten has joined #openstack-ironic | 07:14 | |
*** Dinesh_Bhor has quit IRC | 07:17 | |
*** phuongnh has joined #openstack-ironic | 07:18 | |
*** causten has quit IRC | 07:18 | |
*** rbartal has joined #openstack-ironic | 07:19 | |
*** Dinesh_Bhor has joined #openstack-ironic | 07:21 | |
*** tesseract has joined #openstack-ironic | 07:21 | |
pas-ha | morning ironic :) | 07:28 |
*** hjensas has joined #openstack-ironic | 07:33 | |
*** hjensas has quit IRC | 07:33 | |
*** hjensas has joined #openstack-ironic | 07:33 | |
*** mat128 has joined #openstack-ironic | 07:38 | |
*** zshi has quit IRC | 07:39 | |
*** mrch has joined #openstack-ironic | 07:42 | |
*** mat128 has quit IRC | 07:44 | |
*** ijw has joined #openstack-ironic | 07:47 | |
etingof | morning pas-ha ;) | 07:54 |
*** jaganathan has joined #openstack-ironic | 07:57 | |
*** ccamacho has joined #openstack-ironic | 07:57 | |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Switch to none auth for standalone mode https://review.openstack.org/359061 | 08:01 |
*** ijw has quit IRC | 08:02 | |
*** ralonsoh has joined #openstack-ironic | 08:04 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic master: Secure boot support for irmc-virtual-media driver https://review.openstack.org/507780 | 08:08 |
*** hjensas has quit IRC | 08:12 | |
*** makowals has quit IRC | 08:17 | |
*** hjensas has joined #openstack-ironic | 08:21 | |
*** athomas has quit IRC | 08:23 | |
*** dpawar has quit IRC | 08:24 | |
*** athomas has joined #openstack-ironic | 08:27 | |
*** baoli has joined #openstack-ironic | 08:28 | |
*** makowals has joined #openstack-ironic | 08:32 | |
*** baoli has quit IRC | 08:32 | |
*** derekh has joined #openstack-ironic | 08:47 | |
*** chlong has quit IRC | 08:49 | |
*** zshi has joined #openstack-ironic | 08:50 | |
openstackgerrit | zenghui.shi proposed openstack/ironic master: WIP: Add BIOSInterface for BIOS configuration https://review.openstack.org/507793 | 08:50 |
*** etingof has quit IRC | 08:57 | |
*** udesale has quit IRC | 08:57 | |
*** udesale has joined #openstack-ironic | 08:57 | |
*** amarao has joined #openstack-ironic | 08:59 | |
*** jtomasek has joined #openstack-ironic | 09:01 | |
*** zshi__ has quit IRC | 09:01 | |
*** makowals has quit IRC | 09:07 | |
*** etingof has joined #openstack-ironic | 09:11 | |
*** e0ne has joined #openstack-ironic | 09:16 | |
*** makowals has joined #openstack-ironic | 09:16 | |
*** etingof has quit IRC | 09:19 | |
*** Guest85888 is now known as honza | 09:22 | |
*** dpawar has joined #openstack-ironic | 09:26 | |
*** dpawar has quit IRC | 09:27 | |
*** sambetts_ is now known as sambetts | 09:27 | |
sambetts | Morning all | 09:27 |
*** etingof has joined #openstack-ironic | 09:32 | |
*** jtomasek has quit IRC | 09:38 | |
*** jaganathan has quit IRC | 09:41 | |
*** jaianshu has joined #openstack-ironic | 09:46 | |
*** afazekas is now known as afazekas|seek4fo | 09:48 | |
*** dpawar has joined #openstack-ironic | 09:51 | |
*** d0ugal has joined #openstack-ironic | 10:03 | |
*** TuanLA has quit IRC | 10:06 | |
pas-ha | morning sambetts | 10:06 |
sambetts | hey pas-ha | 10:10 |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Switch to none auth for standalone mode https://review.openstack.org/359061 | 10:10 |
*** hw_wutianwei1 has quit IRC | 10:27 | |
*** jkilpatr has quit IRC | 10:34 | |
*** syqian has joined #openstack-ironic | 10:43 | |
*** ankkumar has quit IRC | 10:47 | |
*** d0ugal has quit IRC | 10:48 | |
syqian | Hi,everyone. is anyone know whether the ironic support to inject the root password for the baremetal server? | 10:50 |
*** stendulker has joined #openstack-ironic | 10:51 | |
*** udesale has quit IRC | 10:52 | |
*** racedo has joined #openstack-ironic | 10:52 | |
*** stendulker_ has quit IRC | 10:54 | |
*** rbudden has joined #openstack-ironic | 10:54 | |
*** milan has joined #openstack-ironic | 10:54 | |
*** amarao1 has joined #openstack-ironic | 10:55 | |
*** stendulker has quit IRC | 10:56 | |
*** syqian has quit IRC | 10:57 | |
*** d0ugal has joined #openstack-ironic | 11:02 | |
*** jkilpatr has joined #openstack-ironic | 11:02 | |
*** jkilpatr has quit IRC | 11:07 | |
*** vmud213 has joined #openstack-ironic | 11:08 | |
aparnav | hey vdrok, Can you take look at this patch https://review.openstack.org/#/c/422572/? | 11:11 |
patchbot | patch 422572 - ironic - Support SUM based firmware update as clean step fo... | 11:11 |
*** syqian has joined #openstack-ironic | 11:11 | |
sambetts | syqian: it is possble via the config drive and cloud-init https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux_OpenStack_Platform/4/html/End_User_Guide/config-drive.html | 11:13 |
syqian | sambetts, ok, i will search it, thank you | 11:18 |
*** phuongnh has quit IRC | 11:18 | |
*** jkilpatr has joined #openstack-ironic | 11:21 | |
*** strigazi has quit IRC | 11:23 | |
*** strigazi has joined #openstack-ironic | 11:24 | |
*** fellypefca-afk is now known as fellypefca | 11:25 | |
*** pcaruana has joined #openstack-ironic | 11:27 | |
*** udesale has joined #openstack-ironic | 11:28 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector master: WiP add policy enforcement engine https://review.openstack.org/507826 | 11:31 |
fellypefca | Good morning everyone :) | 11:32 |
fellypefca | rajinir hey, i will discuss with my team about the time slot of 3rd Party CI and answer you ASAP. But I Think that 16:00 UTC it's ok for us | 11:33 |
pas-ha | I kind of suspect 16UTC might not fit IRMC/Fujitsu team (AFAIU most of them are based in Japan) | 11:35 |
fellypefca | pas-ha Oh! yeah, you're right. 16:00UTC it's midnight in Japan I guess :/ | 11:39 |
*** mat128 has joined #openstack-ironic | 11:40 | |
*** jtomasek has joined #openstack-ironic | 11:41 | |
*** syqian_ has joined #openstack-ironic | 11:45 | |
*** mat128 has quit IRC | 11:45 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector master: WiP add policy enforcement engine https://review.openstack.org/507826 | 11:46 |
*** syqian has quit IRC | 11:47 | |
*** syqian_ has quit IRC | 11:49 | |
*** syqian_ has joined #openstack-ironic | 11:50 | |
*** dprince has joined #openstack-ironic | 11:56 | |
*** syqian__ has joined #openstack-ironic | 11:58 | |
*** syqian_ has quit IRC | 11:58 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ironic master: Add default configuration files to data_files https://review.openstack.org/506198 | 12:02 |
*** mat128 has joined #openstack-ironic | 12:03 | |
*** bfournie has joined #openstack-ironic | 12:03 | |
*** litao__ has quit IRC | 12:04 | |
*** trown is now known as trown|outtypewww | 12:05 | |
*** trown|outtypewww is now known as trown|afk | 12:05 | |
*** mat128 has quit IRC | 12:06 | |
*** mat128 has joined #openstack-ironic | 12:08 | |
*** mat128 has quit IRC | 12:08 | |
*** mat128 has joined #openstack-ironic | 12:08 | |
*** afazekas|seek4fo is now known as afazekas | 12:09 | |
*** dpawar has quit IRC | 12:10 | |
*** jlabarre has joined #openstack-ironic | 12:11 | |
*** amarao1 has quit IRC | 12:12 | |
openstackgerrit | Fellype Cavalcante proposed openstack/ironic stable/ocata: Fix multitenancy scenario on OneView Driver https://review.openstack.org/507632 | 12:12 |
*** dtantsur|afk is now known as dtantsur | 12:16 | |
dtantsur | morning ironic | 12:16 |
fellypefca | Good morning dtantsur :) | 12:17 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ironic master: Add default configuration files to data_files https://review.openstack.org/506198 | 12:18 |
*** amarao1 has joined #openstack-ironic | 12:23 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector master: WiP add policy enforcement engine https://review.openstack.org/507826 | 12:27 |
openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Update proliantutils version for master to "2.4.1" https://review.openstack.org/505506 | 12:30 |
fellypefca | dtantsur can you take a look? I need you +2 back :) https://review.openstack.org/#/c/502072/ | 12:31 |
patchbot | patch 502072 - ironic - Documentation for 'oneview' hardware type | 12:31 |
*** aparnav has quit IRC | 12:33 | |
*** yangyapeng has joined #openstack-ironic | 12:34 | |
*** yangyapeng has quit IRC | 12:38 | |
*** vmud213 has quit IRC | 12:38 | |
*** hw_wutianwei1 has joined #openstack-ironic | 12:42 | |
TheJulia | Good morning | 12:44 |
*** gcb has quit IRC | 12:45 | |
fellypefca | Good morning TheJulia and thak you :) | 12:47 |
fellypefca | thank* ops | 12:49 |
*** jcoufal has joined #openstack-ironic | 12:51 | |
*** mat128 has quit IRC | 12:53 | |
openstackgerrit | Merged openstack/ironic master: Documentation for 'oneview' hardware type https://review.openstack.org/502072 | 12:53 |
TheJulia | fellypefca: I took care of you :) | 12:53 |
fellypefca | TheJulia :D <3 | 12:54 |
dtantsur | sorry, was being coffeinated :) | 12:54 |
TheJulia | hehe | 12:55 |
TheJulia | I'm getting an early start | 12:55 |
TheJulia | and caffeinated at the same time | 12:55 |
dtantsur | we had a farewell party yesterday. I woke up at 12:30pm :D | 12:55 |
TheJulia | ouch :) | 12:56 |
fellypefca | dtantsur Is it your farewell? | 12:56 |
*** dpawar has joined #openstack-ironic | 12:56 | |
fellypefca | I heard you're moving to Germany | 12:57 |
dtantsur | fellypefca: yep :) | 12:57 |
dtantsur | so I was saying bye to local folks | 12:57 |
fellypefca | dtantsur nice! Oktoberfest o/ | 12:58 |
dtantsur | got people some free beers, etc :) | 12:58 |
TheJulia | dtantsur: before I dig into it, the current state of the refarch docs, is that representative of what we discussed at the ptg? | 12:59 |
dtantsur | TheJulia: I only have the introduction patch up, and it should be up-to-date | 12:59 |
dtantsur | the only thing, maybe we want to shrink it a bit, and move some bits to related install guide sections | 13:00 |
dtantsur | I'll leave it up for reviewers to tell me ;) | 13:00 |
*** rloo has joined #openstack-ironic | 13:00 | |
TheJulia | dtantsur: okay, I'm just hoping we get use cases landed soon-ish so I can go point to it in our documentation ;) | 13:01 |
TheJulia | good morning rloo | 13:01 |
dtantsur | yeah, just lemme relocate first :) | 13:01 |
jaianshu | Hello everyone, TheJulia | 13:01 |
TheJulia | Good morning | 13:01 |
jaianshu | i have a query w.r.t https://bugs.launchpad.net/ironic/+bug/1716329 | 13:01 |
openstack | Launchpad bug 1716329 in Ironic "[RFE] HPE ilo OOB RAID configuration" [Wishlist,New] - Assigned to Anshul Jain (theanshuljain) | 13:01 |
rloo | helloooooo and good morning TheJulia, dtantsur, jaianshu | 13:01 |
TheJulia | jaianshu: sure! | 13:01 |
jaianshu | actullay here i'm not implementing a new interface | 13:02 |
dtantsur | morning rloo, jaianshu | 13:02 |
jaianshu | i'm planning to use existing RAIDInterface of base class | 13:02 |
jaianshu | and writing a new olo raid interface | 13:03 |
jaianshu | will it require spec? | 13:03 |
dtantsur | usually no, but there one interesting caveat | 13:03 |
dtantsur | * there is | 13:03 |
jaianshu | *ilo raid interface | 13:03 |
TheJulia | jaianshu: Yes, but if your going to create a new class that will be used as an interface to for raid calls... That begins to enter that territory | 13:03 |
dtantsur | if your RAID interface only targets a subset of hardware that the iLO hardware type targets, you may need a new hardware type.. | 13:04 |
jaianshu | yes..it will be a new hardware type.. | 13:04 |
dtantsur | with this in mind, a spec probably won't hurt.. | 13:05 |
dtantsur | TheJulia: wdyt ^^ | 13:05 |
TheJulia | jaianshu: questions like this are why I indicated a spec, since the spec works you through a number of questions we would normally ask/look for during reviewing code. Asking them up front helps in the long run | 13:05 |
jaianshu | something like : class IloRAID(base.RAIDInterface): | 13:06 |
dtantsur | jaianshu: yes, but also a new hardware type (aka driver). class iLOGen5Hardware(iLOHardware) (or something) | 13:06 |
dtantsur | s/5/10/ but you got the idea | 13:06 |
jaianshu | yes | 13:06 |
dtantsur | because the ilo hardware type also supports gen8 and 9 | 13:07 |
jaianshu | sure dtantsur | 13:08 |
jaianshu | thanks TheJulia, dtantsur..will do that | 13:08 |
TheJulia | jaianshu: thanks! | 13:08 |
*** syqian__ has quit IRC | 13:10 | |
*** syqian__ has joined #openstack-ironic | 13:11 | |
dtantsur | thanks! | 13:11 |
*** jistr is now known as jistr|call | 13:12 | |
*** jaianshu has quit IRC | 13:12 | |
*** krtaylor has joined #openstack-ironic | 13:13 | |
*** syqian__ has quit IRC | 13:14 | |
*** syqian__ has joined #openstack-ironic | 13:15 | |
*** cdearborn_ has joined #openstack-ironic | 13:18 | |
*** trown|afk is now known as trown | 13:18 | |
*** hw_wutianwei1 has quit IRC | 13:18 | |
*** cdearborn__ has joined #openstack-ironic | 13:19 | |
*** cdearborn_ has quit IRC | 13:22 | |
*** baoli has joined #openstack-ironic | 13:26 | |
*** chlong has joined #openstack-ironic | 13:29 | |
*** chlong has quit IRC | 13:31 | |
*** causten has joined #openstack-ironic | 13:33 | |
TheJulia | Okay, I'm going to switch laptops and try and get this spec done | 13:34 |
dtantsur | which one? | 13:34 |
TheJulia | cleanhold which actually should only be like ~40 lines of code and state transitions | 13:35 |
TheJulia | oh, and tests | 13:35 |
dtantsur | sweet! | 13:35 |
*** syqian__ has quit IRC | 13:39 | |
sambetts | nice! | 13:41 |
openstackgerrit | Merged openstack/ironic-specs master: Proposal for Queens cycle priorities https://review.openstack.org/505173 | 13:42 |
rloo | btw TheJulia, did you get around to talking to Interop about deprecation of ironic CLI? Wanted to know if we are good to go ahead with it. | 13:42 |
openstackgerrit | Ilya Etingof proposed openstack/ironic-python-agent master: Report /dev/disk/by-path on inspection followup https://review.openstack.org/507858 | 13:42 |
TheJulia | I ended up double checking the opinion of hogepodge who was leading the discussions at the PTG and he is all for it since it improves consistency. | 13:43 |
TheJulia | rloo: tl;dr deprecate it! ;) | 13:43 |
dtantsur | \o/ | 13:44 |
dtantsur | also \o/ at merged priorities | 13:44 |
TheJulia | woot | 13:44 |
rloo | TheJulia: awesome, thx! | 13:44 |
rloo | dtantsur: i know you want to do a python-ironicclient release soon. was thinking the deprecation could be part of it, but not sure. when were you thinking of doing the release? | 13:45 |
dtantsur | rloo: after we get vdrok's patch in | 13:45 |
rloo | dtantsur: i believe you wanted vdrok's none auth patch (on my list to do today) | 13:45 |
dtantsur | which does give us some time | 13:45 |
rloo | dtantsur: oh. so i shouldn't review vdrok's patch today then, i should work on deprecation? :) | 13:46 |
dtantsur | rloo: both, ideally ;) | 13:46 |
*** lifeless has quit IRC | 13:46 | |
dtantsur | I won't be able to request a release until next Monday | 13:46 |
rloo | dtantsur: heh, ok, will see how far i get. | 13:46 |
rloo | dtantsur: ah, ok, then i have some time :) | 13:47 |
dtantsur | yep | 13:47 |
hogepodge | TheJulia: +1 | 13:48 |
*** dpawar has quit IRC | 13:52 | |
*** dpawar has joined #openstack-ironic | 13:52 | |
*** crushil has joined #openstack-ironic | 13:53 | |
dtantsur | great, thanks hogepodge | 13:54 |
*** lifeless has joined #openstack-ironic | 13:54 | |
*** udesale__ has joined #openstack-ironic | 13:55 | |
*** dpawar_ has joined #openstack-ironic | 13:55 | |
*** udesale has quit IRC | 13:56 | |
*** dpawar has quit IRC | 13:57 | |
*** baoli has quit IRC | 13:58 | |
*** baoli has joined #openstack-ironic | 13:59 | |
*** dpawar_ has quit IRC | 14:00 | |
*** cdearborn__ is now known as cdearborn | 14:01 | |
*** mat128 has joined #openstack-ironic | 14:12 | |
*** Goneri has joined #openstack-ironic | 14:15 | |
*** vsaienk0 has quit IRC | 14:16 | |
*** alex_xu has quit IRC | 14:18 | |
*** srobert has quit IRC | 14:19 | |
*** alex_xu has joined #openstack-ironic | 14:21 | |
*** mat128 has quit IRC | 14:23 | |
*** jistr|call is now known as jistr | 14:27 | |
openstackgerrit | Ilya Etingof proposed openstack/ironic-python-agent master: Report /dev/disk/by-path on inspection followup https://review.openstack.org/507858 | 14:29 |
dtantsur | everyone, the priorities on the whiteboard were updated to match what we agreed on | 14:29 |
dtantsur | please provide your status updates | 14:29 |
dtantsur | I tried to capture what is happening now to my best knowledge | 14:29 |
sambetts | awesome :D | 14:29 |
*** vsaienk0 has joined #openstack-ironic | 14:33 | |
*** livelace2 has joined #openstack-ironic | 14:33 | |
*** rbartal has quit IRC | 14:35 | |
*** udesale__ has quit IRC | 14:38 | |
openstackgerrit | Ilya Etingof proposed openstack/ironic-python-agent master: Report /dev/disk/by-path on inspection followup https://review.openstack.org/507858 | 14:39 |
*** makowals has quit IRC | 14:40 | |
*** mrch has quit IRC | 14:41 | |
rloo | dtantsur: our documentation is still using classic drivers. I thought we had switched them to hardware types :( eg: https://docs.openstack.org/ironic/latest/install/enrollment.html | 14:43 |
dtantsur | this is another step we need to take | 14:43 |
rloo | dtantsur: you want me to open a bug against it, or is it on your list of things to do? | 14:44 |
*** alex_xu has quit IRC | 14:44 | |
dtantsur | rloo: just added to the etherpad. I personally don't need a bug, I treat it as part of the deprecation work. | 14:44 |
rloo | dtantsur: ok, as long as you are tracking it/you are doing it :) | 14:44 |
dtantsur | :) | 14:45 |
*** jaypipes has joined #openstack-ironic | 14:49 | |
*** alex_xu has joined #openstack-ironic | 14:49 | |
*** mat128 has joined #openstack-ironic | 14:56 | |
*** baoli has quit IRC | 14:56 | |
*** baoli has joined #openstack-ironic | 14:59 | |
*** mjura has quit IRC | 15:01 | |
*** yangyapeng has joined #openstack-ironic | 15:08 | |
*** makowals has joined #openstack-ironic | 15:10 | |
*** srobert has joined #openstack-ironic | 15:12 | |
*** makowals has quit IRC | 15:13 | |
*** rpioso has joined #openstack-ironic | 15:13 | |
*** chlong has joined #openstack-ironic | 15:17 | |
*** chas has joined #openstack-ironic | 15:25 | |
dtantsur | going a bit earlier today, see you on Monday (hopefully, heh) | 15:29 |
dtantsur | while I'm out, please ping TheJulia about all the things PTL | 15:29 |
dtantsur | if I'm suddenly out next week, please assume I'm doing some paperwork, and ping TheJulia again :) | 15:29 |
*** Sukhdev has joined #openstack-ironic | 15:30 | |
jlvillal | Is the sourceforege issue resolved? I still see it in the whiteboard. | 15:30 |
jlvillal | The one that was affecting IPA and ironic-lib | 15:30 |
TheJulia | jlvillal: I've not looked yet today, I've been heads down on this spec which I might be posting in a few minutes, doing one final read | 15:31 |
jlvillal | TheJulia, No worries and thanks :) | 15:31 |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Update README https://review.openstack.org/507898 | 15:32 |
rloo | bye dtantsur, good luck with the move! | 15:32 |
*** amarao has quit IRC | 15:33 | |
*** dprince has quit IRC | 15:34 | |
jlvillal | I'm marking the gate as good. I see jobs are passing now for ironic-lib | 15:34 |
jlvillal | dtantsur, ditto! Have a safe move :) | 15:34 |
dtantsur | o/ | 15:35 |
*** dtantsur is now known as dtantsur|afk | 15:35 | |
*** dprince has joined #openstack-ironic | 15:35 | |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Update README https://review.openstack.org/507898 | 15:36 |
*** amarao has joined #openstack-ironic | 15:36 | |
*** amarao1 has quit IRC | 15:36 | |
*** rcernin has quit IRC | 15:43 | |
*** crushil_ has joined #openstack-ironic | 15:43 | |
*** amarao has quit IRC | 15:49 | |
*** vsaienk0 has quit IRC | 15:52 | |
*** vsaienk0 has joined #openstack-ironic | 15:53 | |
*** trown is now known as trown|afk | 15:53 | |
*** penick has joined #openstack-ironic | 15:54 | |
*** stendulker has joined #openstack-ironic | 15:54 | |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Update README https://review.openstack.org/507898 | 15:56 |
openstackgerrit | Julia Kreger proposed openstack/ironic-specs master: Add cleanhold specification https://review.openstack.org/507910 | 15:59 |
*** chas has quit IRC | 16:02 | |
*** nicodemos has joined #openstack-ironic | 16:02 | |
TheJulia | apparently sourceforge is having connectivity issues today :( | 16:03 |
rloo | TheJulia: still? ugh. ^^ jlvillal | 16:05 |
*** crushil_ has quit IRC | 16:05 | |
NobodyCam | Good Morning Ironic'ers | 16:05 |
*** amarao has joined #openstack-ironic | 16:06 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic-python-agent master: Rescue extension for CoreOS with DHCP tenant networks https://review.openstack.org/423521 | 16:07 |
*** r-daneel has joined #openstack-ironic | 16:12 | |
*** stendulker has quit IRC | 16:13 | |
*** stendulker has joined #openstack-ironic | 16:14 | |
*** trown|afk has quit IRC | 16:18 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 16:20 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescue interface field to node-related notifications https://review.openstack.org/502889 | 16:21 |
*** crushil has quit IRC | 16:23 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 16:23 |
*** tesseract has quit IRC | 16:24 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Agent driver rescue implementation https://review.openstack.org/400437 | 16:24 |
*** marios has quit IRC | 16:25 | |
*** hjensas has quit IRC | 16:26 | |
jlvillal | rloo: Oh :( And I just updated the whiteboard an hour or so ago | 16:27 |
rloo | jlvillal: undo! :) | 16:27 |
jlvillal | rloo: Okay | 16:27 |
nicodemos | good morning. | 16:32 |
TheJulia | rloo: jlvillal: yup, still https://twitter.com/sfnet_ops | 16:32 |
TheJulia | there was a span where all was good though :) | 16:32 |
*** chas has joined #openstack-ironic | 16:36 | |
*** chas has quit IRC | 16:41 | |
*** crushil has joined #openstack-ironic | 16:44 | |
*** stendulker has quit IRC | 16:44 | |
*** milan has quit IRC | 16:45 | |
*** chas has joined #openstack-ironic | 16:46 | |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Update documentation https://review.openstack.org/507927 | 16:46 |
*** derekh has quit IRC | 16:49 | |
*** vsaienk0 has quit IRC | 16:49 | |
*** d0ugal has quit IRC | 16:50 | |
*** e0ne has quit IRC | 16:52 | |
*** penick has quit IRC | 16:53 | |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Add missing query parameters to 'next' link for GET /nodes?... https://review.openstack.org/506280 | 16:53 |
*** zhangfei has joined #openstack-ironic | 16:56 | |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Remove 'next' for GET /nodes?limit=1&instance_uuid= https://review.openstack.org/506205 | 16:56 |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Add missing query parameters to 'next' link for GET /nodes?... https://review.openstack.org/506280 | 16:57 |
*** penick has joined #openstack-ironic | 16:57 | |
*** penick_ has joined #openstack-ironic | 17:01 | |
*** penick has quit IRC | 17:03 | |
*** rama_y has joined #openstack-ironic | 17:07 | |
*** zhangfei has quit IRC | 17:08 | |
TheJulia | rloo: if you can take a look at https://review.openstack.org/#/c/507910/1/specs/approved/cleanhold-state.rst and let me know if it basically matches what we discussed regarding clean steps at the ptg, I'll go ahead and begin writing the code this afternoon while it is fresh in my head | 17:11 |
patchbot | patch 507910 - ironic-specs - Add cleanhold specification | 17:11 |
rloo | TheJulia: hmm, ok, eating messy lunch. soon... | 17:12 |
TheJulia | well, enjoy the lunch first! | 17:13 |
TheJulia | I'm on pacific time right now | 17:13 |
*** pcaruana has quit IRC | 17:14 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector master: WiP add policy enforcement engine https://review.openstack.org/507826 | 17:18 |
*** trown has joined #openstack-ironic | 17:21 | |
openstackgerrit | Ilya Etingof proposed openstack/ironic-specs master: IPA to report node activity https://review.openstack.org/492702 | 17:22 |
jlvillal | TheJulia: Welcome to our wonderful time zone :D | 17:27 |
openstackgerrit | Ilya Etingof proposed openstack/ironic-python-agent master: Report /dev/disk/by-path on inspection followup https://review.openstack.org/507858 | 17:28 |
*** dpawar has joined #openstack-ironic | 17:33 | |
*** dpawar has quit IRC | 17:35 | |
rloo | TheJulia: i looked at the spec, when you have time, let's discuss. fwiw, i reviewed the ptg notes (https://etherpad.openstack.org/p/ironic-queens-ptg-contentious-topics) and GET API spec (https://review.openstack.org/#/c/501511) | 17:38 |
patchbot | patch 501511 - ironic-specs - spec: Clean Steps that are Available | 17:38 |
rloo | TheJulia: i am unclear as to how your spec & the get-clean-steps spec work together | 17:38 |
rloo | TheJulia: at ptg, did folks mention it might be useful to have a 'hold' state so that the node has the ramdisk loaded but doing nothing? (outside of the need for this API to get the clean steps) | 17:39 |
rloo | TheJulia: was also wondering, what about a 'clean-hold' clean step, and not have a 'clean-hold' state (it'd be 'clean-wait'). or maybe we'd still want a clean-hold state, just thinking maybe for flex, folks might want a clean-hold step to put between/after a particular clean step. | 17:40 |
fellypefca | Devstack is too slow :/ | 17:40 |
*** trown has quit IRC | 17:46 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic master: Reduce complexity of node_power_action() function https://review.openstack.org/507668 | 17:47 |
openstackgerrit | John L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off https://review.openstack.org/507297 | 17:47 |
*** e0ne has joined #openstack-ironic | 17:49 | |
* sambetts --> home | 17:52 | |
sambetts | night all | 17:52 |
*** dpawar has joined #openstack-ironic | 17:52 | |
*** sambetts is now known as sambetts|afk | 17:52 | |
fellypefca | Good night sambetts|afk :) | 17:52 |
*** jtomasek has quit IRC | 17:53 | |
*** rama_y has quit IRC | 17:57 | |
openstackgerrit | Christopher Dearborn proposed openstack/ironic master: python-dracclient driver version update https://review.openstack.org/507942 | 17:57 |
*** rama_y has joined #openstack-ironic | 17:57 | |
openstackgerrit | Christopher Dearborn proposed openstack/ironic master: python-dracclient driver version update https://review.openstack.org/507942 | 17:58 |
openstackgerrit | John L. Villalovos proposed openstack/ironic master: Reduce complexity of node_power_action() function https://review.openstack.org/507668 | 17:58 |
openstackgerrit | John L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off https://review.openstack.org/507297 | 17:58 |
*** penick has joined #openstack-ironic | 17:58 | |
*** penick_ has quit IRC | 18:01 | |
TheJulia | rloo: sorry, I've been on the phone | 18:04 |
rloo | TheJulia: no worries | 18:04 |
openstackgerrit | John L. Villalovos proposed openstack/ironic master: Reduce complexity of node_power_action() function https://review.openstack.org/507668 | 18:05 |
openstackgerrit | John L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off https://review.openstack.org/507297 | 18:05 |
*** ralonsoh has quit IRC | 18:06 | |
rloo | TheJulia: I just added my comments to the spec so I don't forget... | 18:06 |
rloo | TheJulia: so was there a desire from folks to be able to have a 'hold' so they could poke around? | 18:07 |
TheJulia | sorry, phone still | 18:07 |
rloo | TheJulia: oh. heh. let me know when you're avail then :) | 18:07 |
TheJulia | rloo: Okay, finally free | 18:09 |
TheJulia | rloo: so, tl;dr it was the solution to not touching the state machine from the get of the step information | 18:09 |
TheJulia | rloo: it was also reached in more groupmind consensus really quickly during that whole discussion and we quickly moved on | 18:09 |
rloo | TheJulia: but adding a 'hold' *is* touching the state machine? | 18:09 |
TheJulia | absolutely | 18:10 |
*** jkilpatr has quit IRC | 18:10 | |
TheJulia | problem is we shouldn't have multiple ways of triggering state machine changes | 18:10 |
TheJulia | nor can we build something outside that doesn't reflect the current state but that is doing things | 18:10 |
TheJulia | so this kind of decouples the get, makes the get become just what is cached, the user can go to cleanhold, get up to date cleansteps, then trigger manual cleaning with what they want from there | 18:10 |
rloo | TheJulia: first question. is this 'hold' useful outside of the get-cleaning-steps API? i thought people thought it'd be useful | 18:10 |
TheJulia | rloo: hold concept is useful in terms of a long running ramdisk | 18:11 |
rloo | TheJulia: ok, so i think it is worth mentioning that in the spec, because that becomes a usecase/requirement | 18:11 |
TheJulia | I guess in theory, we could have a general hold state, but that could also be onfusing | 18:12 |
TheJulia | confusing | 18:12 |
TheJulia | rloo: I thought I had... hmm | 18:12 |
rloo | TheJulia: and might someone want a 'hold' to happen after a particular clean step was done? | 18:12 |
TheJulia | I wouldn't think a machine would go back to cleanhold, although we could do that with the state machine as long as the step had the ability to go back, but I suspect joining too much on to what is functionally just state machine changes will scope creep and increase complexity | 18:13 |
rloo | TheJulia: i'm not sure what you mean there. I can envision the 'hold' as a separate clean-step. | 18:14 |
TheJulia | So a separate user definable clean-step that can kick the node back to a cleanhold state? | 18:14 |
*** noam_ has joined #openstack-ironic | 18:14 | |
noam_ | Hi | 18:14 |
TheJulia | which then a user could retrigger cleaning or exit? | 18:14 |
rloo | TheJulia: right. what is a cleanhold state anyway. it just means that there is a ramdisk there. | 18:15 |
*** pcaruana has joined #openstack-ironic | 18:15 | |
noam_ | Someone can elborate the difference between netboot and local? | 18:15 |
rloo | TheJulia: which i think would mean, that the user would have to 'continue' | 18:15 |
TheJulia | rloo: basically yeah. Running ramdisk, collected cleansteps. | 18:15 |
rloo | TheJulia: user would have to explicitly say 'continue' | 18:15 |
TheJulia | ugh | 18:16 |
rloo | TheJulia: and then the conductor would go on to the next step if any | 18:16 |
TheJulia | I guess a clean continue verb | 18:16 |
rloo | TheJulia: yes, something like that. i haven't thought out the details. just the concept. | 18:17 |
*** Sukhdev has quit IRC | 18:17 | |
TheJulia | if there is driver internal info stating it can continue, then I guess itcould continue the next cleaning step upon the next heartbeat | 18:17 |
TheJulia | Yeah, it could just be an alias to kick it to CLEANWAIT | 18:17 |
TheJulia | and it could error api side if it is an invalid request | 18:17 |
rloo | TheJulia: heh, i haven't thought about implementation, i'm sure we can implement ;) | 18:18 |
*** ChubYann has joined #openstack-ironic | 18:18 | |
*** hjensas has joined #openstack-ironic | 18:18 | |
rloo | TheJulia: ok, so if you're good with that, it addresses two of my earlier questions. | 18:20 |
rloo | TheJulia: now, how does this fit with the get-clean-steps API. | 18:20 |
TheJulia | rloo: I would prefer that we don't scope creep new mechanics, at least if I'm writing it since I need to get back to some other things to unblock myself | 18:20 |
rloo | TheJulia: I think we still want a GET .../cleaning/steps API, and it returns the cached steps | 18:20 |
TheJulia | rloo: and then nothing else is required... | 18:20 |
rloo | TheJulia: i don't call it 'scope creep new mechanics'. i call it 'support for the feature' | 18:20 |
*** racedo has quit IRC | 18:21 | |
rloo | TheJulia: otherwise, the coding might be done in a way that won't allow it to be flexible later | 18:21 |
TheJulia | ahh, but independent small features can add up and build on each other | 18:21 |
rloo | TheJulia: so the user can do a GET ../cleaning/steps. How does the user do a 'fetch' to get the steps? | 18:21 |
rloo | TheJulia: I had proposed a POST ../cleaning/steps to do that, so that there is a symmetry between the get & fetch. | 18:22 |
rloo | TheJulia: i can't recall if we had decided at the PTG that we didn't or we did like that symmetry | 18:22 |
rloo | TheJulia: wrt 'independent small features can add up...', yes. but hard to do if you don't architect it right. | 18:22 |
rloo | TheJulia: there's a diff conceptually between putting the node in a hold state, vs it being a clean step. | 18:23 |
*** jkilpatr has joined #openstack-ironic | 18:23 | |
rloo | TheJulia: let me give you an example. automated vs manual cleaning. all that would have been easier to code if we had planned for both initially. | 18:23 |
*** rama_y has quit IRC | 18:24 | |
*** rama_y has joined #openstack-ironic | 18:24 | |
TheJulia | phone again | 18:24 |
TheJulia | okay | 18:25 |
openstackgerrit | Fellype Cavalcante proposed openstack/ironic stable/ocata: Fix multitenancy scenario on OneView Driver https://review.openstack.org/507632 | 18:25 |
TheJulia | rloo: the user would fetch steps by requesting the cleanhold state. Having a post means a second method of triggering state machine changes, and there should ideally be only one. | 18:26 |
TheJulia | rloo: agree, hence why I asked you to take a look :) | 18:26 |
rloo | TheJulia: ok, so we decided that it was ok not to have that symmetry. | 18:26 |
rloo | TheJulia: my concern with this hold thing, is that we're expecting the user to issue two commands (and wait in between) | 18:27 |
rloo | TheJulia: in order to get the clean steps | 18:27 |
rloo | TheJulia: 1. put node in hold state | 18:27 |
rloo | 2. wait til node is in hold state | 18:27 |
TheJulia | we did, I remembered the consensus seemed to be that we wanted to maintain existing path for causing things to happen since it would need to be represented back to the user | 18:27 |
rloo | 3. do something to get the node out of hold state | 18:27 |
TheJulia | Well | 18:27 |
*** rama_y has quit IRC | 18:27 | |
TheJulia | Essentially having the post is the same | 18:27 |
TheJulia | because you can't let that hang for five minutes or so for IPA to boot and heartbeat in | 18:28 |
rloo | TheJulia: i *think* the hold was because folks said, if i know i want to do cleaning, but i want to know the clean steps first, i don't want to have to boot the ramdisk twice. | 18:28 |
rloo | TheJulia: does that ring a bell with you? (i wish i had taken more notes at ptg...) | 18:29 |
TheJulia | rloo: That was the side effect, that we functionally begin moving towards long running ramdisks and not having multiple boot cycles for things like, refreshing data in the db. | 18:29 |
rloo | TheJulia: no, the post is different. there is no 'hold' that requires user intervention to get out of | 18:29 |
TheJulia | rloo: so your post would return what to the user? | 18:30 |
rloo | TheJulia: nothing | 18:30 |
rloo | TheJulia: it is async | 18:30 |
rloo | TheJulia: just like most of the requests to put the node in a provision state | 18:30 |
rloo | TheJulia: just like the 'hold' would be. what were you planning on returning for that request? | 18:30 |
TheJulia | so a user would then have to continue to check the state to see if the machine has booted, aside from getting a last updated field | 18:31 |
rloo | TheJulia: yup, if they are waiting to get the available clean steps | 18:31 |
TheJulia | rloo: nothing, but it was going to be via the provision state interface and state machine | 18:31 |
rloo | TheJulia: just like a user has to wait for a node to get deployed to | 18:31 |
TheJulia | exactly, so there is no real difference aside from one follows an existing pattern which was the preference in the room during the discussion | 18:32 |
rloo | TheJulia: I honestly don't recall that this was the preference | 18:32 |
rloo | TheJulia: my concern is that for someone that only wants the clean steps, it looks like your proposal is asking them to do the 3 steps i mentioned above. | 18:33 |
rloo | TheJulia: which may be fine. i have not been given any requirements from any user, as to how they want this to work | 18:33 |
TheJulia | trigger clean hold, wait until it hits the cleanhold state, the steps would b e updated at that point and the user would be able to know that they should have refreshed cleanstep information from the get /cleaning/steps endpoint | 18:34 |
rloo | TheJulia: and then, the user has to issue another request to get it out of hold, right? | 18:34 |
*** fellypefca is now known as fellypefca-afk | 18:35 | |
TheJulia | Yes, that is true, they can issue a manual clean or exit the state. They also may wish to leave the node in that powered on state waiting for days or even potentially (and now i feel bad for the earth) weeks | 18:35 |
rloo | TheJulia: so i think there is value to having the 'hold'. I'm just not sure that it effectively addresses the issue of getting the clean steps (from the user's point of view, although I don't know of any users) | 18:36 |
rloo | s/get/fetch/ | 18:36 |
rloo | caching i guess | 18:36 |
TheJulia | Everyone that I know that has asked for a way to get them, wants to turn around and run manual cleaning | 18:36 |
rloo | i'm not questioning the need for the GET ../clean/steps or the proposed hold thing. | 18:37 |
TheJulia | i know | 18:37 |
rloo | TheJulia: i'm just questioning the fetching/caching of clean steps part | 18:37 |
*** hjensas has quit IRC | 18:37 | |
*** rwsu has quit IRC | 18:38 | |
* TheJulia goes and looks for the instance of irccloud that is using 3.36GB of ram | 18:38 | |
rloo | TheJulia: from the perspective of caching the clean steps, i-the-user might like to issue one command only and come back after coffee to see the clean steps. not have to issue a 'continue' command (or forget to issue that command) | 18:38 |
TheJulia | rloo: but if we also hook in collection along the way with normal processes such as automated cleaning so that we always have cached clean steps.... | 18:39 |
TheJulia | Things won't change that often, if an operator changes out the ramdisk with more hardware managers, they will then need a way to refresh that information | 18:40 |
rloo | TheJulia: huh? we already do this. I mean, we already cache the clean steps, when we start cleaning | 18:40 |
TheJulia | we do, then we nuke it upon completion I thought | 18:41 |
rloo | TheJulia: so i honestly don't recall, i thought someone said that we didn't. but that is ... who cares. we can keep it after that or not, it isn't a big deal. | 18:41 |
TheJulia | I only think we nuke it now because I found and fixed a bug in manual cleaning where we were not cleaning stuff up and getting stuck in a loop | 18:42 |
rloo | TheJulia: ok | 18:42 |
TheJulia | if we timed out, we didn't execute the same cleanup steps to that are normally performed upon exit so re-issuing manual clean wouldn't work | 18:43 |
TheJulia | wow, that was a while ago | 18:43 |
rloo | TheJulia: so I'm a bit lost. I'm not sure what that has to do with what we are discussing wrt the hold & the fetching/caching of clean steps | 18:44 |
TheJulia | I'm just providing background context as to why I think we don't cache them right now unless something has changed, so we would need to always keep them cached. | 18:45 |
TheJulia | err, I think we dont | 18:45 |
TheJulia | if that makes sense | 18:45 |
rloo | TheJulia: yes, for the GET ../cleaning/steps to work, we need to always cache them | 18:45 |
rloo | TheJulia: all i remember, is that when we start cleaning, we get the clean steps from ipa. easy enough to check what happens when cleaning finishes | 18:46 |
rloo | TheJulia: anyway, I think I've given you my feedback about the hold'em state. i can recap in the patch. | 18:48 |
TheJulia | rloo: Agreed, as long as we're careful around logic like on this line https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L313 :) | 18:49 |
TheJulia | That you have, thanks! | 18:49 |
rloo | TheJulia: yup wrt the code! | 18:49 |
*** fellypefca-afk has left #openstack-ironic | 18:52 | |
*** penick has quit IRC | 18:53 | |
*** dprince has quit IRC | 18:56 | |
*** MattMan has quit IRC | 18:59 | |
*** MattMan has joined #openstack-ironic | 19:00 | |
*** pcaruana has quit IRC | 19:00 | |
*** r-daneel has quit IRC | 19:02 | |
*** r-daneel has joined #openstack-ironic | 19:03 | |
*** e0ne has quit IRC | 19:08 | |
*** penick has joined #openstack-ironic | 19:10 | |
*** jcoufal has quit IRC | 19:17 | |
*** rama_y has joined #openstack-ironic | 19:20 | |
*** dprince has joined #openstack-ironic | 19:20 | |
*** livelace2 has quit IRC | 19:25 | |
*** dpawar has quit IRC | 19:26 | |
*** penick_ has joined #openstack-ironic | 19:27 | |
*** penick has quit IRC | 19:30 | |
*** rwsu has joined #openstack-ironic | 19:32 | |
*** ChubYann has quit IRC | 19:58 | |
*** mat128 has quit IRC | 20:04 | |
*** ChubYann has joined #openstack-ironic | 20:12 | |
*** jkilpatr has quit IRC | 20:14 | |
*** r-daneel has quit IRC | 20:14 | |
*** r-daneel has joined #openstack-ironic | 20:15 | |
*** cdearborn has quit IRC | 20:28 | |
*** rwsu has quit IRC | 20:52 | |
*** ChubYann has quit IRC | 20:54 | |
*** noam_ has quit IRC | 20:54 | |
*** e0ne has joined #openstack-ironic | 21:00 | |
*** penick_ has quit IRC | 21:03 | |
*** e0ne has quit IRC | 21:04 | |
*** e0ne has joined #openstack-ironic | 21:05 | |
*** penick has joined #openstack-ironic | 21:07 | |
*** ChubYann has joined #openstack-ironic | 21:07 | |
*** crushil has quit IRC | 21:09 | |
*** penick has quit IRC | 21:11 | |
*** causten has quit IRC | 21:11 | |
*** rwsu has joined #openstack-ironic | 21:12 | |
*** LeoBud has joined #openstack-ironic | 21:13 | |
*** bnemec has quit IRC | 21:14 | |
*** LeoBud has quit IRC | 21:20 | |
*** causten has joined #openstack-ironic | 21:20 | |
*** chas has quit IRC | 21:21 | |
*** chas has joined #openstack-ironic | 21:22 | |
*** e0ne has quit IRC | 21:24 | |
*** chas has quit IRC | 21:27 | |
*** causten has quit IRC | 21:27 | |
*** causten has joined #openstack-ironic | 21:28 | |
*** etingof has quit IRC | 21:31 | |
*** srobert_ has joined #openstack-ironic | 21:31 | |
*** srobert has quit IRC | 21:35 | |
*** srobert_ has quit IRC | 21:36 | |
*** dprince has quit IRC | 21:42 | |
*** charcol has joined #openstack-ironic | 21:43 | |
*** penick has joined #openstack-ironic | 21:46 | |
*** jlabarre has quit IRC | 21:46 | |
*** etingof has joined #openstack-ironic | 21:46 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent master: Build tinyipa with python3 https://review.openstack.org/495385 | 21:51 |
*** baoli has quit IRC | 21:54 | |
*** penick has quit IRC | 21:55 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent master: Build tinyipa with python3 https://review.openstack.org/495385 | 21:55 |
*** jkilpatr has joined #openstack-ironic | 21:56 | |
*** causten has quit IRC | 22:02 | |
*** causten has joined #openstack-ironic | 22:03 | |
*** causten_ has joined #openstack-ironic | 22:05 | |
*** cdearborn has joined #openstack-ironic | 22:06 | |
*** charcol has quit IRC | 22:06 | |
*** rwsu has quit IRC | 22:07 | |
*** causten has quit IRC | 22:07 | |
*** Goneri has quit IRC | 22:08 | |
*** causten has joined #openstack-ironic | 22:13 | |
*** causten_ has quit IRC | 22:16 | |
*** rbudden has quit IRC | 22:24 | |
*** krtaylor has quit IRC | 22:30 | |
*** rbudden has joined #openstack-ironic | 22:38 | |
*** krtaylor has joined #openstack-ironic | 22:43 | |
*** penick has joined #openstack-ironic | 22:45 | |
*** jaypipes has quit IRC | 22:48 | |
*** penick has quit IRC | 22:52 | |
openstackgerrit | Merged openstack/ironic stable/pike: Adds more exception handling for ironic-conductor heartbeat https://review.openstack.org/507373 | 22:56 |
openstackgerrit | Merged openstack/ironic master: Cleanup unit tests for ipmittool https://review.openstack.org/506413 | 22:56 |
*** charcol has joined #openstack-ironic | 23:01 | |
*** baoli has joined #openstack-ironic | 23:18 | |
*** rloo has quit IRC | 23:21 | |
*** baoli_ has joined #openstack-ironic | 23:22 | |
*** baoli has quit IRC | 23:22 | |
*** Sukhdev has joined #openstack-ironic | 23:32 | |
jlvillal | TheJulia: sourceforge says they are now back up as of 5 minutes ago. I could download ipmitool tarball | 23:33 |
openstackgerrit | John L. Villalovos proposed openstack/ironic master: Reduce complexity of node_power_action() function https://review.openstack.org/507668 | 23:34 |
openstackgerrit | John L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off https://review.openstack.org/507297 | 23:34 |
TheJulia | \o/ | 23:34 |
* TheJulia messages friends that were mentioning it | 23:35 | |
*** krtaylor has quit IRC | 23:35 | |
anupn | jlvillal, nice, even i was waiting to see when sourceforge gets up | 23:36 |
*** causten has quit IRC | 23:38 | |
*** causten has joined #openstack-ironic | 23:38 | |
*** srobert has joined #openstack-ironic | 23:39 | |
*** causten has quit IRC | 23:43 | |
*** ijw has joined #openstack-ironic | 23:43 | |
*** ijw has quit IRC | 23:45 | |
*** ijw has joined #openstack-ironic | 23:45 | |
jlvillal | anupn: I noticed some jobs worked on your Python 3 IPA patch. | 23:45 |
jlvillal | anupn: Never mind. Those are the coreos jobs. | 23:47 |
*** rpioso has quit IRC | 23:51 | |
*** bnemec has joined #openstack-ironic | 23:52 | |
*** r-daneel has quit IRC | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!