| *** suro-patz has joined #openstack-ironic | 00:00 | |
| *** trown|outtypewww is now known as trown | 00:03 | |
| openstackgerrit | Merged openstack/ironic: Devstack: add check of chassis creating https://review.openstack.org/293781 | 00:03 |
|---|---|---|
| *** dims_ has joined #openstack-ironic | 00:03 | |
| *** suro-patz has quit IRC | 00:04 | |
| *** dims has quit IRC | 00:04 | |
| *** Sukhdev has joined #openstack-ironic | 00:06 | |
| *** phuongnh has quit IRC | 00:09 | |
| *** baoli has quit IRC | 00:09 | |
| *** baoli has joined #openstack-ironic | 00:10 | |
| *** Marga_ has quit IRC | 00:14 | |
| *** Marga_ has joined #openstack-ironic | 00:15 | |
| *** daemontool has quit IRC | 00:15 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: iRMC power driver for soft power off and inject nmi https://review.openstack.org/216743 | 00:17 |
| *** achanda has quit IRC | 00:18 | |
| *** achanda has joined #openstack-ironic | 00:21 | |
| *** garthb has quit IRC | 00:22 | |
| jroll | out for the night, see y'all later. | 00:22 |
| devananda | jroll: ciao! | 00:23 |
| gmmaha | devananda: thanks for the tip.. i have that and still nothing.. what i am seeing is that bifrost master is able to lease an IP and hand the URL to download the boot.ipxe link.. | 00:24 |
| gmmaha | but with some irregularity, the download over htto just halts. | 00:24 |
| gmmaha | once when download ramdisk, once during the kernel image and once even when download the boot.ipxe file | 00:24 |
| gmmaha | i have a feeling i am dealing with a network setting issue thats causing it | 00:25 |
| gmmaha | s/htto/http | 00:25 |
| devananda | gmmaha: that sounds like either network fault or hardware fault | 00:25 |
| gmmaha | devananda: yeah, thats what i am feeling as well.. | 00:25 |
| devananda | gmmaha: once the server has started to net boot and fetch the images, it should complete that | 00:26 |
| devananda | gmmaha: bifrost preferentially uses HTTP instead of TFTP because it is more reliable | 00:26 |
| gmmaha | devananda: i am using http, but thats the one where its halting.. | 00:26 |
| devananda | gmmaha: how large are the image files being transferred? | 00:26 |
| devananda | you can check /httpboot/ dir on the bifrost host | 00:27 |
| gmmaha | devananda: its just cores images | 00:27 |
| devananda | yea. that's odd. sounds like bad network or something wrong with hardware to me | 00:27 |
| TheJulia | gmmaha: check your switchport logs for interface state changes in addition to wire errors | 00:27 |
| TheJulia | on interface counters | 00:27 |
| devananda | yep | 00:27 |
| gmmaha | 22M, 212M and this tme the 4K file | 00:27 |
| devananda | gmmaha: normal sizes | 00:28 |
| *** aginwala has quit IRC | 00:28 | |
| gmmaha | TheJulia: yes, thats hte next step.. | 00:28 |
| gmmaha | devananda: thanks.. let me get into debuggin the network ports nad figure out whats happening at the switch | 00:28 |
| gmmaha | thanks for the help | 00:28 |
| devananda | np | 00:28 |
| TheJulia | gmmaha: let us know what you find, seems really odd, if it was a timer on the switch with state transitions I would think it would typically fail in about the same place every time | 00:30 |
| gmmaha | TheJulia: absolutely will do.. its very odd as well that it doesnt fail at the same point all the time | 00:31 |
| gmmaha | one quickQ.. once bifrost master hands an IP to ipze, should that IP be pingable from the master? I woudl assume yes | 00:31 |
| gmmaha | cause right after the hang, i tried pinging the IP and i got nothing | 00:31 |
| *** baoli has quit IRC | 00:32 | |
| TheJulia | i don't think the network stack in ipxe responds to icmp | 00:32 |
| *** aginwala has joined #openstack-ironic | 00:32 | |
| gmmaha | aah ok | 00:32 |
| gmmaha | thanks for confirming | 00:32 |
| TheJulia | if you can get it to arp, that would be a way to know | 00:32 |
| *** spandhe has quit IRC | 00:32 | |
| NobodyCam | I would also check http server logs | 00:32 |
| * gmmaha goes to see that | 00:33 | |
| TheJulia | you might be able to see some stuff if you do a packet capture, however when the download completes, the node abruptly stops responding and you'll see lots of FIN ACK packets as if it just went away | 00:34 |
| gmmaha | TheJulia: aah.. unfortunately dotn have access to the switch ports.. getting someone else who actually owns it to help debug on their side | 00:39 |
| gmmaha | NobodyCam: thanks.. http logs dont say anythign | 00:40 |
| *** achanda has quit IRC | 00:40 | |
| *** piet has joined #openstack-ironic | 00:41 | |
| *** krotscheck has joined #openstack-ironic | 00:42 | |
| NobodyCam | gmmaha: so you see the request for the file? | 00:49 |
| NobodyCam | have you tried to curl the file? from another system ... just to test. | 00:51 |
| gmmaha | NobodyCam: getting another machine up with an OS to test the curl.. | 00:53 |
| gmmaha | for the logs, i can see the ironic-api logs.. not able to find http logs on the machine.. :| | 00:53 |
| gmmaha | and nothing unusual in the ironic logs | 00:53 |
| gmmaha | i can see that he image started downloading | 00:54 |
| gmmaha | in the ironic conductor logs | 00:54 |
| gmmaha | and thats it | 00:54 |
| gmmaha | nothing beyond that | 00:54 |
| NobodyCam | humm | 00:58 |
| NobodyCam | I have to step away for dinner | 00:59 |
| *** saripurigopi has quit IRC | 01:11 | |
| *** achanda has joined #openstack-ironic | 01:11 | |
| *** Sukhdev has quit IRC | 01:12 | |
| *** phuongnh has joined #openstack-ironic | 01:14 | |
| *** piet has quit IRC | 01:15 | |
| *** aginwala has quit IRC | 01:16 | |
| gmmaha | NobodyCam: no worries.. thanks | 01:17 |
| gmmaha | TheJulia: do the NICs have to be in promiscuous mode to be able to get though the http downloads? | 01:18 |
| TheJulia | gmmaha: no.... | 01:18 |
| *** aginwala has joined #openstack-ironic | 01:19 | |
| gmmaha | TheJulia: the person enabled prmiscuous mode on the NIC (was running tcpdump) and the installation started moving.. | 01:20 |
| * TheJulia tries to think how promisc would impact or change a tcp connection | 01:21 | |
| * gmmaha doesnt know much about tcp and promiscuous mode to comment on that | 01:21 | |
| TheJulia | basically, it shouldn't change anything... confusing. | 01:22 |
| *** alexpilotti has quit IRC | 01:23 | |
| *** mtanino has joined #openstack-ironic | 01:23 | |
| *** baoli has joined #openstack-ironic | 01:24 | |
| *** baoli has quit IRC | 01:30 | |
| *** Marga_ has quit IRC | 01:30 | |
| *** baoli has joined #openstack-ironic | 01:34 | |
| *** dims_ has quit IRC | 01:40 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: [Devstack]Add ability to enable shellinabox SSL certificate https://review.openstack.org/289671 | 01:44 |
| gmmaha | TheJulia: it was apparently that. It continued with install but was slow.. but worked | 01:46 |
| gmmaha | not sure why and what happened | 01:46 |
| *** saripurigopi has joined #openstack-ironic | 01:49 | |
| *** saripurigopi has quit IRC | 01:51 | |
| *** harshs has quit IRC | 01:52 | |
| *** Fdaisuke has joined #openstack-ironic | 01:54 | |
| *** Fdaisuke has quit IRC | 01:54 | |
| *** Fdaisuke has joined #openstack-ironic | 01:55 | |
| TheJulia | gmmaha: still makes no sense :( | 02:05 |
| gmmaha | TheJulia: dont know much about things to comment on those... | 02:06 |
| * gmmaha needs to go and learn about things | 02:06 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Adopt Ironic's own context https://review.openstack.org/295663 | 02:09 |
| lazy_prince | gmmaha: sometimes its due to bad SFP connected to the nic port | 02:09 |
| lazy_prince | I had one experience where image download starts and then it abruptly stops.. on further investigation we found that SFP was faulty.. | 02:09 |
| gmmaha | lazy_prince: aaah... i tried just one other machine and it did the same thing.. | 02:10 |
| gmmaha | maybe i should have tried a few more.. | 02:10 |
| gmmaha | lazy_prince: you might be spot on on this. | 02:10 |
| gmmaha | still curious is that it worked hwen setting promiscuous mode on for the port. thats whats puzzling. | 02:11 |
| lazy_prince | if thats the case, i guess, its something to do either with VLAN or with Routing... | 02:12 |
| gmmaha | lazy_prince: yeah.. trying to figure out where that issue is | 02:13 |
| *** suro-patz has joined #openstack-ironic | 02:19 | |
| *** hoangcx has joined #openstack-ironic | 02:39 | |
| openstackgerrit | Reedip proposed openstack/pyghmi: Remove support for py33/py26 https://review.openstack.org/264570 | 02:45 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: [Devstack]Add ability to enable shellinabox SSL certificate https://review.openstack.org/289671 | 02:47 |
| *** aginwala has quit IRC | 02:51 | |
| *** sivaramakrishna has joined #openstack-ironic | 02:53 | |
| *** suro-patz has quit IRC | 02:54 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Remove unneeded use of task.release_resources() https://review.openstack.org/296149 | 02:54 |
| openstackgerrit | bin proposed openstack/ironic: Fix VirtualBox cannot set boot device when powered on https://review.openstack.org/290951 | 02:55 |
| *** Cameron_C has joined #openstack-ironic | 03:00 | |
| *** spandhe has joined #openstack-ironic | 03:01 | |
| gmmaha | TheJulia: lazy_prince: it seems its the NIC thats causing issues... | 03:04 |
| gmmaha | my colleague tried cobler and it sorta hit the same issue.. download halts after starting | 03:04 |
| *** piet has joined #openstack-ironic | 03:10 | |
| *** Nisha_away has joined #openstack-ironic | 03:15 | |
| *** saripurigopi has joined #openstack-ironic | 03:18 | |
| *** spandhe has quit IRC | 03:18 | |
| *** yuanying has quit IRC | 03:21 | |
| *** yuanying_ has joined #openstack-ironic | 03:21 | |
| *** Cameron_C has quit IRC | 03:21 | |
| *** Cameron_C has joined #openstack-ironic | 03:22 | |
| *** piet has quit IRC | 03:23 | |
| *** yuanying_ has quit IRC | 03:24 | |
| *** yuanying has joined #openstack-ironic | 03:24 | |
| *** Marga_ has joined #openstack-ironic | 03:26 | |
| *** links has joined #openstack-ironic | 03:28 | |
| *** yuanying has quit IRC | 03:29 | |
| *** aginwala has joined #openstack-ironic | 03:29 | |
| *** Marga_ has quit IRC | 03:30 | |
| *** aginwala has quit IRC | 03:33 | |
| *** bharath has joined #openstack-ironic | 03:34 | |
| *** Marga_ has joined #openstack-ironic | 03:39 | |
| *** Marga_ has quit IRC | 03:40 | |
| *** Marga_ has joined #openstack-ironic | 03:40 | |
| *** baoli has quit IRC | 03:46 | |
| *** hoangcx has quit IRC | 03:56 | |
| *** hoangcx has joined #openstack-ironic | 03:57 | |
| *** praneshp_ has joined #openstack-ironic | 03:58 | |
| *** yuanying has joined #openstack-ironic | 04:01 | |
| *** achanda has quit IRC | 04:06 | |
| *** baoli has joined #openstack-ironic | 04:13 | |
| *** baoli has quit IRC | 04:14 | |
| *** baoli has joined #openstack-ironic | 04:15 | |
| *** baoli has quit IRC | 04:24 | |
| openstackgerrit | greghaynes proposed openstack/bifrost: Fix ironic inventory nic creation https://review.openstack.org/296172 | 04:34 |
| *** aginwala has joined #openstack-ironic | 04:41 | |
| *** appu103_ has joined #openstack-ironic | 04:42 | |
| *** appu103_ is now known as aparnav | 04:42 | |
| *** aginwala has quit IRC | 04:43 | |
| *** vinm213 has joined #openstack-ironic | 04:43 | |
| *** ChrisAusten has joined #openstack-ironic | 04:45 | |
| *** aginwala has joined #openstack-ironic | 04:49 | |
| *** linuxgeek has quit IRC | 04:51 | |
| *** achanda has joined #openstack-ironic | 04:53 | |
| *** ChrisAusten has quit IRC | 05:05 | |
| *** ChrisAusten has joined #openstack-ironic | 05:05 | |
| *** saripurigopi has quit IRC | 05:12 | |
| *** ChrisAusten has quit IRC | 05:29 | |
| *** hoangcx has quit IRC | 05:30 | |
| *** hoangcx has joined #openstack-ironic | 05:32 | |
| *** phuongnh has quit IRC | 05:34 | |
| *** suro-patz has joined #openstack-ironic | 05:43 | |
| NobodyCam | gmmaha: is it a Mellanox nic? | 05:49 |
| *** Nisha_away has quit IRC | 05:50 | |
| *** stendulker has joined #openstack-ironic | 05:57 | |
| *** Sukhdev has joined #openstack-ironic | 05:57 | |
| *** jaybeale has quit IRC | 06:00 | |
| *** achanda_ has joined #openstack-ironic | 06:13 | |
| *** achanda has quit IRC | 06:14 | |
| *** suro-patz has quit IRC | 06:32 | |
| *** ChubYann has quit IRC | 06:32 | |
| *** moshele has joined #openstack-ironic | 06:35 | |
| openstackgerrit | Yuiko Takada proposed openstack/ironic: Delete bridge "brbm" in devstack/unstack.sh https://review.openstack.org/293900 | 06:49 |
| lazy_prince | gmmaha: The way you explained made me feel it is either NIC card or SFPs.. Good, you figured it out.. | 06:55 |
| *** e0ne has joined #openstack-ironic | 06:58 | |
| *** moshele has quit IRC | 07:03 | |
| openstackgerrit | Yuiko Takada proposed openstack/ironic: Delete bridge "brbm" in devstack/unstack.sh https://review.openstack.org/293900 | 07:05 |
| *** praneshp_ has quit IRC | 07:08 | |
| *** ndipanov has quit IRC | 07:08 | |
| *** praneshp_ has joined #openstack-ironic | 07:08 | |
| *** vishwanathj is now known as vishwanathj_away | 07:12 | |
| *** lazy_prince has quit IRC | 07:23 | |
| *** Sukhdev has quit IRC | 07:25 | |
| *** davidlenwell has quit IRC | 07:26 | |
| *** lazy_prince has joined #openstack-ironic | 07:26 | |
| *** achanda_ has quit IRC | 07:26 | |
| *** e0ne has quit IRC | 07:28 | |
| *** aginwala has quit IRC | 07:29 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update resources only for specific node during deletion. https://review.openstack.org/287498 | 07:29 |
| *** davidlenwell has joined #openstack-ironic | 07:34 | |
| *** achanda has joined #openstack-ironic | 07:35 | |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic: Use get_admin_context() to create the context object https://review.openstack.org/246036 | 07:35 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Fix NamedTemporaryFile() OSError Exception https://review.openstack.org/293762 | 07:36 |
| *** tesseract has joined #openstack-ironic | 07:41 | |
| *** spandhe has joined #openstack-ironic | 07:41 | |
| *** tesseract is now known as Guest60842 | 07:41 | |
| *** alexpilotti has joined #openstack-ironic | 07:42 | |
| vinm213 | morning ironicers | 07:43 |
| vinm213 | lintan: i have submitted new patch based out of your patch..Would you mind taking a look at it? | 07:44 |
| *** praneshp_ has quit IRC | 07:51 | |
| *** alexpilotti has quit IRC | 07:53 | |
| *** moshele has joined #openstack-ironic | 07:54 | |
| *** alexpilotti has joined #openstack-ironic | 07:54 | |
| *** alexpilotti has quit IRC | 07:58 | |
| *** Nisha_away has joined #openstack-ironic | 08:00 | |
| lintan | sure, vinm213 | 08:00 |
| *** achanda has quit IRC | 08:06 | |
| *** achanda has joined #openstack-ironic | 08:08 | |
| *** achanda has quit IRC | 08:11 | |
| *** achanda has joined #openstack-ironic | 08:12 | |
| *** achanda has quit IRC | 08:12 | |
| *** achanda has joined #openstack-ironic | 08:13 | |
| *** achanda has quit IRC | 08:13 | |
| *** saibarspeis has joined #openstack-ironic | 08:15 | |
| *** spandhe has quit IRC | 08:16 | |
| *** athomas has joined #openstack-ironic | 08:20 | |
| davidlenwell | ml2 | 08:21 |
| *** vinm213 has quit IRC | 08:25 | |
| *** vinm213 has joined #openstack-ironic | 08:27 | |
| vinm213 | lintan: thanks | 08:31 |
| *** xek has quit IRC | 08:32 | |
| *** xek has joined #openstack-ironic | 08:33 | |
| *** mbound has joined #openstack-ironic | 08:34 | |
| *** ohamada has joined #openstack-ironic | 08:41 | |
| *** shu-mutou has joined #openstack-ironic | 08:47 | |
| *** links has quit IRC | 09:01 | |
| *** piet has joined #openstack-ironic | 09:01 | |
| *** e0ne has joined #openstack-ironic | 09:03 | |
| *** openstackgerrit has quit IRC | 09:03 | |
| *** openstackgerrit has joined #openstack-ironic | 09:04 | |
| aarefiev | morning all! | 09:10 |
| *** mkovacik__ has joined #openstack-ironic | 09:10 | |
| *** lucas-dinner is now known as lucasagomes | 09:13 | |
| *** achanda has joined #openstack-ironic | 09:14 | |
| lucasagomes | morning all | 09:14 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Restart consoles on conductor startup https://review.openstack.org/257252 | 09:15 |
| mkovacik__ | morning guys! | 09:16 |
| mkovacik__ | lucasagomes, o/ | 09:16 |
| *** piet has quit IRC | 09:16 | |
| *** ifarkas has joined #openstack-ironic | 09:17 | |
| *** yolanda has quit IRC | 09:17 | |
| *** links has joined #openstack-ironic | 09:18 | |
| *** derekh has joined #openstack-ironic | 09:19 | |
| *** achanda has quit IRC | 09:20 | |
| vdrok | morning ironic, aarefiev, lucasagomes, mkovacik__ | 09:24 |
| *** mgould has joined #openstack-ironic | 09:30 | |
| alineb | morning all! | 09:31 |
| mgould | alineb, morning! | 09:31 |
| mgould | and morning all other Ironicers! | 09:31 |
| alineb | o/ mgould | 09:33 |
| *** yolanda has joined #openstack-ironic | 09:34 | |
| *** daemontool has joined #openstack-ironic | 09:35 | |
| *** daemontool_ has joined #openstack-ironic | 09:36 | |
| *** daemontool_ has quit IRC | 09:36 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Documentation update related to disk label https://review.openstack.org/296341 | 09:40 |
| sergek | o/ | 09:41 |
| *** yolanda has quit IRC | 09:42 | |
| *** Nisha_away has quit IRC | 09:44 | |
| *** shu-mutou is now known as shu-mutou-AFK | 09:44 | |
| lucasagomes | yuriyz, vdrok I just fast approved the intel driver in staging cause the core group is quite small :-( | 09:44 |
| lucasagomes | yuriyz, vdrok if one of you guys want to help as core, lemme know as well | 09:44 |
| openstackgerrit | Merged openstack/ironic-inspector: Better error handling when converting eDeploy data https://review.openstack.org/295327 | 09:45 |
| vdrok | morning alineb and mgould ! | 09:45 |
| yuriyz | morning Ironic | 09:45 |
| vdrok | lucasagomes: thanks! I think I could help with that if there is a need | 09:46 |
| mgould | sergek, lucasagomes vdrok yuriyz morning! | 09:46 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Adopt Ironic's own context https://review.openstack.org/295663 | 09:46 |
| lucasagomes | vdrok, cool I'm going to add you! So far there are not many reviews there, but help is always good | 09:46 |
| vdrok | lucasagomes: sure, will keep an eye on them :) | 09:47 |
| lucasagomes | vdrok, added! You may have to login/logout from gerrit to the change to get effect (happened to me once) | 09:48 |
| yuriyz | lucasagomes thanks, +1 for vdrok in cores, I personally will try to review more if patches will be there, but plan to be mostly like code contributor in staging drivers | 09:48 |
| lucasagomes | yuriyz, awesome! Thank you, vdrok is core now (-: | 09:48 |
| vdrok | lucasagomes: yep, that works! | 09:49 |
| stendulker | vdrok, lucasagomes, yuriyz: Hi | 09:50 |
| lucasagomes | stendulker, morning | 09:50 |
| pas-ha | morning folks | 09:50 |
| stendulker | vdrok, lucasagomes, yuriyz: Please look at this patch https://review.openstack.org/296341 | 09:50 |
| mgould | vdrok, congrats :-) | 09:50 |
| vdrok | morning stendulker | 09:50 |
| vdrok | thanks mgould :) | 09:50 |
| stendulker | vdrok, lucasagomes, yuriyz: If possible better to land it in Mitaka | 09:50 |
| stendulker | vdrok, lucasagomes : Good morning :) | 09:51 |
| vdrok | stendulker: I believe jroll already proposed a release, so idk | 09:51 |
| stendulker | vdrok: oh | 09:51 |
| vdrok | stendulker: https://review.openstack.org/#/c/296049/ | 09:51 |
| lucasagomes | stendulker, will do in a sec | 09:52 |
| stendulker | vdrok: thanks. | 09:52 |
| stendulker | vdrok: will leaving a message in that review will help? | 09:53 |
| stendulker | vdrok: just a tiny thing, but could help users | 09:53 |
| stendulker | lucasagomes: thank you | 09:54 |
| stendulker | vdrok: thank you | 09:54 |
| mkovacik__ | vdrok, o/ | 09:54 |
| mkovacik__ | mgould, o/ | 09:55 |
| lintan | thanks vdrok, lucasagomes and yuriy, I will add a follow-up patch on amt soon. Actually, I have more on hands :) | 09:58 |
| mgould | mkovacik__, stendulker, lintan o/ | 09:58 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 09:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 09:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 09:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 09:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 09:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 09:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 09:59 |
| lintan | thanks jroll for helping update the openstack-request-id patch ! :) | 09:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 09:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 09:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: add network drivers https://review.openstack.org/285852 | 09:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 09:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack https://review.openstack.org/293520 | 09:59 |
| *** dims has joined #openstack-ironic | 10:00 | |
| stendulker | mgould: good Morning :) | 10:00 |
| *** sambetts|afk is now known as sambetts | 10:00 | |
| sambetts | Morning all :) | 10:01 |
| vdrok | lintan: thank you! :) | 10:01 |
| vdrok | morning sambetts | 10:01 |
| mgould | morning sambetts | 10:02 |
| sambetts | o/ vdrok mgould | 10:02 |
| lucasagomes | lintan, thanks a lot! No rush | 10:02 |
| aparnav | hey lucasagomes, vdrok, Can you take a look at this documentation patch https://review.openstack.org/#/c/280533/ ? The code patch for this documentation have already got merged. | 10:05 |
| vdrok | aparnav: looking | 10:05 |
| *** links has quit IRC | 10:05 | |
| aparnav | vdrok, Thank you :) | 10:05 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-specs: Check duplicate nodes enrollment https://review.openstack.org/296357 | 10:06 |
| *** ndipanov has joined #openstack-ironic | 10:09 | |
| mkovacik__ | aarefiev, hi, thanks for the review on https://review.openstack.org/#/c/267637 ! | 10:11 |
| mkovacik__ | aarefiev, wanted to ask one thing, though | 10:11 |
| mkovacik__ | aarefiev, https://review.openstack.org/#/c/267637/15/ironic_inspector/process.py with the exceptions processing on lines 365 and below | 10:12 |
| aarefiev | mkovacik__: hey, whats up | 10:12 |
| mkovacik__ | aarefiev, when handling utils.error the LOG.error() line isn't necessary, correct? | 10:13 |
| mkovacik__ | as when that exception is instantiated and thrown it gets logged implicitly | 10:13 |
| aarefiev | mkovacik__: utils.error has it's own log | 10:13 |
| aarefiev | yep | 10:13 |
| mkovacik__ | OK, so I think I'll kill that line | 10:14 |
| mkovacik__ | aarefiev, but what about a generic exception? | 10:14 |
| aarefiev | mkovacik__: what generic exception? | 10:14 |
| mkovacik__ | line 371 | 10:14 |
| mkovacik__ | you put a comment saying that it needed be logged | 10:15 |
| mkovacik__ | I mean the text/message of the exception itself | 10:15 |
| mkovacik__ | but a generic exception doesn't have logging built-in afaictl | 10:15 |
| mkovacik__ | am I missing something here? | 10:16 |
| aarefiev | mkovacik__ I mean there LOG.exception is used, it will add exception info | 10:16 |
| *** linuxgeek has joined #openstack-ironic | 10:16 | |
| *** moshele has quit IRC | 10:16 | |
| aarefiev | it's not big deal | 10:17 |
| mkovacik__ | aarefiev, so LOG.exception does process last stack trace implicitly> | 10:17 |
| mkovacik__ | ? | 10:17 |
| * mkovacik__ didn't know | 10:17 | |
| aarefiev | mkovacik__: yeah https://docs.python.org/2/library/logging.html#logging.Logger.exception | 10:18 |
| mkovacik__ | aarefiev, thanks! | 10:19 |
| mkovacik__ | I'll fix this nits, better to do it the right way | 10:20 |
| *** links has joined #openstack-ironic | 10:21 | |
| aarefiev | mkovacik__: np, I'm worrying here about existing data(stored before this change ), it will fail if we try to apply, but we can see this data and status of inspection is finished. | 10:22 |
| *** hoangcx has quit IRC | 10:23 | |
| aarefiev | mkovacik__: we can handle this case or make better error to avoid confusing | 10:23 |
| mkovacik__ | aarefiev, there's but a single result kept | 10:24 |
| mkovacik__ | so last introspection run overwrites any previous result | 10:24 |
| mkovacik__ | what do you suggest? maybe I'm missing your point | 10:25 |
| aarefiev | mkovacik__: we have introspection data, then update inspector and try to reapply on existing data | 10:26 |
| aarefiev | it fails because didn't find incoming data, right? | 10:26 |
| * mkovacik__ thinks | 10:27 | |
| mkovacik__ | but that should fail for no data stored, not incompatibility issue | 10:27 |
| aarefiev | mkovacik__: yes, but when I do status or get data, they exists | 10:28 |
| aarefiev | mkovacik__: it's confusing | 10:29 |
| aarefiev | maybe just better error | 10:29 |
| mkovacik__ | aarefiev, right, I think I understand now | 10:29 |
| aarefiev | mkovacik__: sorry, I need to go, back in 30 min | 10:30 |
| mkovacik__ | if you GET on the result data stored in Swift you'll get a valid, passed result, but if you try reapply, it will override that result with a failure | 10:31 |
| mkovacik__ | aarefiev, sure, thx for talking this | 10:31 |
| openstackgerrit | Jane.zhang proposed openstack/ironic-python-agent: Stop tgtd before creating iscsi target via lio. https://review.openstack.org/290335 | 10:31 |
| *** e0ne has quit IRC | 10:35 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-lib: Support configdrive in iscsi deploy for raw images https://review.openstack.org/230924 | 10:36 |
| *** electrofelix has joined #openstack-ironic | 10:41 | |
| *** e0ne has joined #openstack-ironic | 10:45 | |
| *** _degorenko|afk is now known as degorenko | 10:51 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Add disk_label support for partition images https://review.openstack.org/295766 | 10:53 |
| *** alexpilotti has joined #openstack-ironic | 10:55 | |
| *** alexpilotti has quit IRC | 10:59 | |
| jroll | morning y'all | 11:00 |
| jroll | lintan: you're welcome :) | 11:00 |
| sambetts | o/ jroll | 11:00 |
| *** trusted has joined #openstack-ironic | 11:00 | |
| aarefiev | jroll: morning | 11:00 |
| jroll | stendulker: I see vdrok has a -1 on 296341, if you fix that quickly it may still be able to make mitaka :) | 11:00 |
| stendulker | jroll : sure. checking it right away | 11:01 |
| vdrok | morning jroll | 11:02 |
| jroll | stendulker: even if it doesn't make it, we can backport it | 11:02 |
| jroll | morning sambetts, stendulker, aarefiev, vdrok :) | 11:02 |
| *** sivaramakrishna has quit IRC | 11:03 | |
| stendulker | ok | 11:03 |
| aarefiev | mkovacik__: does it override? I thought it fails on getting info from cache | 11:03 |
| *** trusted has quit IRC | 11:04 | |
| *** dims_ has joined #openstack-ironic | 11:05 | |
| *** dims has quit IRC | 11:06 | |
| *** awiddersheim has quit IRC | 11:08 | |
| *** sivaramakrishna has joined #openstack-ironic | 11:13 | |
| *** sivaramakrishna is now known as Guest33334 | 11:14 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 11:14 |
| *** achanda has joined #openstack-ironic | 11:19 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Documentation update for partition image support https://review.openstack.org/296341 | 11:19 |
| stendulker | vdrok, jroll: Have reposted patch for https://review.openstack.org/296341 | 11:20 |
| jroll | thanks | 11:20 |
| lucasagomes | jroll, morning | 11:20 |
| jroll | stendulker: +2 | 11:21 |
| jroll | morning lucasagomes :) | 11:21 |
| stendulker | jroll: thank you :) | 11:21 |
| openstackgerrit | bin proposed openstack/ironic: Fix VirtualBox cannot set boot device when powered on https://review.openstack.org/290951 | 11:21 |
| TheJulia | good morning | 11:22 |
| vdrok | morning TheJulia | 11:23 |
| *** achanda has quit IRC | 11:24 | |
| jroll | \o TheJulia | 11:24 |
| mgould | morning jroll, TheJulia | 11:25 |
| jroll | hey mgould | 11:25 |
| sambetts | lucasagomes: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#pxe-uefi-setup reading through this part of the install guide, it starts by downloading elilo but then only configures grub2 in the following steps, do you think we should drop the elilo part as its been dropped from debian since 2014? Or at least move it into some kind of alternatives section | 11:26 |
| *** Nisha_away has joined #openstack-ironic | 11:27 | |
| vsaienko | morning jroll, lucasagomes, jroll and all ironic community | 11:28 |
| vsaienko | sambetts: could you please have a test cisco CI with https://review.openstack.org/#/c/296237/ | 11:28 |
| jroll | \o vsaienko | 11:28 |
| *** saibarauei has joined #openstack-ironic | 11:29 | |
| vsaienko | that patch changes order, plugins are executed first and extras.d executed after plugins | 11:29 |
| sambetts | vsaienko: I already had a patch out to do this https://review.openstack.org/#/c/293954/ | 11:29 |
| openstackgerrit | Merged openstack/ironic-webclient: Removed phantomjs from test suite https://review.openstack.org/294160 | 11:30 |
| lucasagomes | sambetts, hi there, that wouldn't be a bad thing actually | 11:31 |
| lucasagomes | I think we kept elilo for some reason in the past (backward compat I believe) | 11:32 |
| *** stendulker_ has joined #openstack-ironic | 11:32 | |
| sambetts | jroll: ^ | 11:32 |
| sambetts | jroll: any reason you know of for keeping the elilo part in the UEFI section of the install guide? | 11:33 |
| *** saibarspeis has quit IRC | 11:33 | |
| *** saibarauei is now known as saibarspeis | 11:33 | |
| vsaienko | sambetts: I've abandoned my patch. It looks like we can't fix it this way. Going to update https://review.openstack.org/#/c/293020/ | 11:33 |
| *** stendulker has quit IRC | 11:33 | |
| jroll | sambetts: I don't know much about it, sorry | 11:33 |
| lucasagomes | sambetts, I think the tricky part is, if we drop it, what happens to machines that were dpeloyed with it? | 11:34 |
| openstackgerrit | Merged openstack/ironic: Documentation update for partition image support https://review.openstack.org/296341 | 11:34 |
| lucasagomes | sambetts, e.g if the ring gets rebalanced and another conductor takes it over it needs to regeneate elilo pxe templates right? | 11:34 |
| vsaienko | lucasagomes, jroll, sambetts: what do you think if we completely move ironic tempest configuration https://github.com/openstack-dev/devstack/blob/master/lib/tempest#L508-L523 to ironic devstack plugin? | 11:34 |
| sambetts | lucasagomes: well they weren't following most of the guide anyway, because all the other steps configure the grub2 bootloader | 11:34 |
| lucasagomes | sambetts, we just need to think about it, maybe a way to migrate from it or something | 11:35 |
| lucasagomes | sambetts, right | 11:35 |
| jroll | vsaienko: so, stable branch is the reason we didn't do that, but the recommendation is to do it and always install ironic master in the tempest venv | 11:35 |
| jroll | vsaienko: (which I agree with) | 11:35 |
| jroll | we just haven't done it yet | 11:35 |
| sambetts | jroll, vsaienko: there is a TODO note in the ironic_tempest_plugin/config.py for that | 11:36 |
| jroll | sambetts: vsaienko: yeah, see this also https://review.openstack.org/#/c/283789/ | 11:36 |
| jroll | the comments | 11:36 |
| sambetts | makes sense | 11:37 |
| *** stendulker_ has quit IRC | 11:38 | |
| sambetts | jroll: we're disucssing this IRT https://bugs.launchpad.net/ironic/+bug/1546706 | 11:38 |
| openstack | Launchpad bug 1546706 in Ironic "DEFAULT_INSTANCE_TYPE not honored in the new Ironic's devstack plugin" [Medium,In progress] - Assigned to Mauro S M Rodrigues (maurosr) | 11:38 |
| jroll | IRT? | 11:39 |
| sambetts | in regards to | 11:39 |
| jroll | ah | 11:40 |
| sambetts | basically moving to a plugin changed the order that we're are run relative to the extra.d files in devstack | 11:40 |
| jroll | right right | 11:40 |
| sambetts | so the flavor we create doesn't exist at the time tempest trys to configure its config file and blows up the tests when using a custom DEFAULT_INSTACE_TYPE | 11:41 |
| sambetts | we recently merged a patch changing our docs to make DEFAULT_INSTANCE_TYPE=baremetal | 11:42 |
| *** linuxgeek has quit IRC | 11:43 | |
| sambetts | which from mitaka onwards doesn't get honored | 11:43 |
| jroll | yeah, I see that | 11:44 |
| jroll | what if we just changed devstack/lib/nova to create the baremetal flavor if ironic is enabled | 11:45 |
| sambetts | We've got a couple of different patches out that fix it in different ways, https://review.openstack.org/#/c/293020/4 or https://review.openstack.org/#/c/293954/ | 11:45 |
| sambetts | I personally think fixing the order in devstack makes the most sense | 11:45 |
| jroll | oh, I see someone did that already https://review.openstack.org/#/q/status:open+branch:master+topic:bug/1546706 | 11:46 |
| sambetts | jroll: thats the other option, move the flavor creation back into devstack | 11:47 |
| jroll | sambetts: so what happens when tempest moves to a plugin and happens to run before ironic? :) | 11:47 |
| sambetts | jroll: if tempest became a plugin then we could enforce the order by change the order by changing the order of enable_plugin calls in local.conf | 11:48 |
| sambetts | e.g. like the fact we have to enable_plugin inspector after enable_plugin ironic | 11:49 |
| sambetts | we could make sure that we enable_plugin tempest last | 11:49 |
| jroll | yeah | 11:49 |
| sambetts | but the extra.d thing is just sort of wedged in there and has its own ordering system | 11:50 |
| jroll | so yeah, I think I like the re-order in devstack, if you can talk devstack people into it | 11:50 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added power button to node list https://review.openstack.org/274847 | 11:50 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Created common navigation header https://review.openstack.org/295329 | 11:50 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Introduced btn-gradient styles. https://review.openstack.org/294082 | 11:50 |
| jroll | we should also add a thing in our plugin: if [ $(ini_get tempest default_flavor) != 'baremetal' ]; then die; fi | 11:50 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Updated application dependencies https://review.openstack.org/293963 | 11:51 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Applied table styles and column ordering to node list https://review.openstack.org/295399 | 11:51 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Adjusted flow for header elements https://review.openstack.org/294114 | 11:51 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Updated UX for Default Cloud dropdown https://review.openstack.org/294080 | 11:51 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added node list control buttons https://review.openstack.org/295330 | 11:51 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Removed border from header https://review.openstack.org/294081 | 11:51 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Updated eslint plugins https://review.openstack.org/293953 | 11:51 |
| jroll | :| | 11:51 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Removed account https://review.openstack.org/274848 | 11:51 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Updated ui-bootstrap https://review.openstack.org/293969 | 11:51 |
| jroll | sambetts: we should also add a thing in our plugin: if [ $(ini_get tempest default_flavor) != 'baremetal' ]; then die; fi | 11:51 |
| jroll | (once it's sorted, of course) | 11:51 |
| jroll | and probably backport to mitaka devstack if that's already cut | 11:51 |
| krotscheck | betherly: Rebased and updated with your comments ^^ | 11:51 |
| jroll | which it is | 11:51 |
| betherly | krotscheck: awesome will have a look in a minute | 11:52 |
| jroll | sambetts: this means we're also not running with the recommended filters >.> | 11:52 |
| jroll | like exactram | 11:52 |
| jroll | sigh. | 11:52 |
| krotscheck | Not certain why some of those still have the cannot-merge flag... | 11:53 |
| sambetts | jroll: yeah, thats why I don't like the move it into the ironic plugin option because we'd have to work out all the things that might be affected by our workflow and then duplicate the inisets | 11:54 |
| jroll | sambetts: yeah, I feel like tempest should always be last | 11:54 |
| jroll | if it's going to validate things before running | 11:55 |
| jroll | bbiab | 11:56 |
| *** ohamada has quit IRC | 11:58 | |
| *** david-lyle_ has joined #openstack-ironic | 11:58 | |
| *** david-lyle has quit IRC | 12:00 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector: Set config options for keystoneauth. https://review.openstack.org/294703 | 12:01 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector: Use keystoneauth for Ironic and Swift clients https://review.openstack.org/286510 | 12:01 |
| *** awiddersheim has joined #openstack-ironic | 12:03 | |
| *** kromanenko has joined #openstack-ironic | 12:03 | |
| *** ohamada has joined #openstack-ironic | 12:04 | |
| *** awiddersheim has quit IRC | 12:04 | |
| *** dprince has joined #openstack-ironic | 12:07 | |
| Nisha_away | jroll, lucasagomes is it possible to propose the backport patch for https://review.openstack.org/#/c/295766/ now? or it should be done later? | 12:07 |
| *** awiddersheim has joined #openstack-ironic | 12:07 | |
| jroll | Nisha_away: yes, please propose it now | 12:07 |
| Nisha_away | jroll, ok. :) | 12:07 |
| lucasagomes | Nisha_away, it's aready merged in master so go ahead and propose it | 12:08 |
| lucasagomes | thanks | 12:08 |
| jroll | Nisha_away: I will try to get that released before mitaka final | 12:08 |
| Nisha_away | jroll, ok | 12:08 |
| *** ohamada has quit IRC | 12:09 | |
| *** smoriya_ has quit IRC | 12:09 | |
| *** jcoufal has joined #openstack-ironic | 12:11 | |
| *** m3m0 has joined #openstack-ironic | 12:12 | |
| *** lucasagomes is now known as lucas-hungry | 12:12 | |
| *** vinm213 has quit IRC | 12:13 | |
| *** jaypipes has joined #openstack-ironic | 12:15 | |
| openstackgerrit | Aparna proposed openstack/ironic: Doc update to enable HTTPS in Glance and Ironic comm https://review.openstack.org/280533 | 12:20 |
| *** raildo-afk is now known as raildo | 12:30 | |
| *** krtaylor has quit IRC | 12:32 | |
| *** Goneri has joined #openstack-ironic | 12:35 | |
| *** piet has joined #openstack-ironic | 12:36 | |
| *** cdearborn has joined #openstack-ironic | 12:36 | |
| Nisha_away | jroll, raised https://review.openstack.org/296403 for the same | 12:45 |
| *** krtaylor has joined #openstack-ironic | 12:46 | |
| *** baoli has joined #openstack-ironic | 12:48 | |
| jroll | Nisha_away: thanks | 12:49 |
| *** mbound has quit IRC | 12:49 | |
| *** mtanino has quit IRC | 12:53 | |
| *** alexpilotti has joined #openstack-ironic | 12:56 | |
| mkovacik__ | aarefiev, it fails, you're right (see my reply on the pull request), overrides in case data found but other errors happened | 12:58 |
| mkovacik__ | aarefiev, so would you consider the concern resolved with the comment? | 12:58 |
| *** alexpilotti has quit IRC | 13:00 | |
| *** links has quit IRC | 13:01 | |
| aarefiev | mkovacik__: so it fails with not found in cache? | 13:02 |
| mkovacik__ | aarefiev, it fails yes, logging | 13:06 |
| *** david-lyle_ is now known as david-lyle | 13:06 | |
| mkovacik__ | not storing in the error | 13:06 |
| mkovacik__ | cause there might be other reasons for the failure such as race between saving and reapplying | 13:06 |
| jroll | lucas-hungry: is this done? https://bugs.launchpad.net/ironic/+bug/1544642 | 13:06 |
| openstack | Launchpad bug 1544642 in Ironic "[RFE] Test ipmitool drivers in gate" [Wishlist,In progress] - Assigned to Lucas Alvares Gomes (lucasagomes) | 13:06 |
| *** afaranha has joined #openstack-ironic | 13:07 | |
| jroll | betherly: how's ironic-ui release coming? :) | 13:09 |
| jroll | TheJulia: how's bifrost release coming? :) | 13:09 |
| betherly | jroll: 1 more patch to land then will be there. | 13:09 |
| jroll | betherly: thanks | 13:10 |
| jroll | jlvillal: hey, this was moved to an rfe but it seems like it could either be called done, or never-ending. what should we do with it? https://bugs.launchpad.net/ironic/+bug/1491670 | 13:11 |
| openstack | Launchpad bug 1491670 in Ironic "[RFE] ironic does not have functional testing" [Wishlist,In progress] - Assigned to John L. Villalovos (happycamp) | 13:11 |
| * jroll doing an rfe approval/cleanup spike | 13:11 | |
| lucas-hungry | jroll, yes, it's -nv at the moment but pretty much done | 13:13 |
| *** lucas-hungry is now known as lucasagomes | 13:14 | |
| jroll | lucasagomes: shall I call it done or do you want to leave that rfe until you make it voting? | 13:14 |
| jroll | 13:13:05 openstackgerrit | Merged openstack/releases: ironic 5.1.0 https://review.openstack.org/296049 | 13:14 |
| jroll | \o/ | 13:14 |
| jroll | #success ironic 5.1.0 released as the basis for stable/mitaka | 13:15 |
| openstackstatus | jroll: Added success to Success page | 13:15 |
| lucasagomes | jroll, I marked it as fix committed, because I want to leave the jobs -nv for at least 1 month | 13:15 |
| lucasagomes | see if it's stable and so on | 13:15 |
| lucasagomes | but so far so good | 13:15 |
| jroll | lucasagomes: well to close it out it should be fix released :) | 13:15 |
| jroll | but sounds good | 13:15 |
| * lucasagomes updates it | 13:15 | |
| lucasagomes | jroll, o/ for 5.1.0 | 13:15 |
| jroll | :D | 13:16 |
| *** baoli_ has joined #openstack-ironic | 13:16 | |
| openstackgerrit | Thierry Carrez proposed openstack/ironic: Update reno for stable/mitaka https://review.openstack.org/296428 | 13:17 |
| openstackgerrit | Thierry Carrez proposed openstack/ironic: Update reno for stable/mitaka https://review.openstack.org/296428 | 13:17 |
| *** baoli has quit IRC | 13:19 | |
| *** aparnav has quit IRC | 13:20 | |
| *** athomas has quit IRC | 13:20 | |
| *** mbound has joined #openstack-ironic | 13:20 | |
| aarefiev | mkovacik__: took a look again, I'm wrong it fails with "exception while fetching stored introspection data: " , then looks fine | 13:21 |
| aarefiev | mkovacik__: sorry for misleading | 13:21 |
| *** achanda has joined #openstack-ironic | 13:22 | |
| *** athomas has joined #openstack-ironic | 13:24 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 13:25 |
| * sambetts instant clicks on it to find out what it is | 13:25 | |
| lucasagomes | jroll, btw q related to the ipmitool jobs. Once we made it voting, should we also get rid of the ssh jobs ? | 13:26 |
| lucasagomes | I don't think they will be needed anymore | 13:26 |
| jroll | lucasagomes: yeah, I think so, and then probably figure out how to move that driver out eventually | 13:27 |
| *** achanda has quit IRC | 13:27 | |
| lucasagomes | jroll, right yeah | 13:27 |
| *** Nisha_away has quit IRC | 13:27 | |
| sambetts | lucasagomes, jroll: it would be cool to remove that driver, because I think it only causes confusion :-p | 13:28 |
| *** trown is now known as trown|afk | 13:28 | |
| lucasagomes | sambetts, heh yeah | 13:29 |
| jroll | sambetts: yeah indeed | 13:31 |
| jroll | wow, we're terrible at this rfe thing, e.g. manual cleaning was never approved :P | 13:31 |
| jroll | hopefully we can keep track better once they're cleaned up | 13:32 |
| openstackgerrit | Andrey Shestakov proposed openstack/ironic: Add VNC console support https://review.openstack.org/296437 | 13:32 |
| *** absubram has joined #openstack-ironic | 13:32 | |
| *** daemontool_ has joined #openstack-ironic | 13:33 | |
| sambetts | jroll: oh dear... I really think we need to automate it somehow like how the fix merged comments get applied, if it was an RFE we should be able to make it mark it as approved | 13:33 |
| *** absubram_ has joined #openstack-ironic | 13:33 | |
| *** daemontool has quit IRC | 13:33 | |
| *** rbudden has joined #openstack-ironic | 13:34 | |
| *** absubram has quit IRC | 13:37 | |
| *** absubram_ is now known as absubram | 13:37 | |
| jroll | sambetts: could be neat, but I think if people mark it approved when they +A the spec it could be fine | 13:37 |
| jroll | lucasagomes: I feel like this is done, is there still more to do? https://bugs.launchpad.net/ironic/+bug/1526275 | 13:38 |
| openstack | Launchpad bug 1526275 in Ironic "[RFE] iPXE dynamic configuration" [Wishlist,In progress] - Assigned to Lucas Alvares Gomes (lucasagomes) | 13:38 |
| lucasagomes | jroll, it's actually not :-( I had to stop it to prioritize other stuff | 13:39 |
| jroll | lucasagomes: ah ok, no worries | 13:39 |
| lucasagomes | jroll, all the plumbing work is done but we still have ipxe conf files in the filesystem and all | 13:39 |
| lucasagomes | it's on my list but I didn't have time to finish it up, sorry | 13:40 |
| jroll | lucasagomes: yeah, all good, just curious | 13:41 |
| lucasagomes | jroll, honestly, I sometimes think that we should split PXE and iPXE drivers now that we have a boot interface | 13:42 |
| lucasagomes | it's a bit messy having both things on the same boot interface (they use different technologies) | 13:42 |
| jroll | lucasagomes: maybe, yeah, I wonder how much cleaner the code would be | 13:42 |
| lucasagomes | tho they are similar in some aspects | 13:42 |
| jroll | maybe we should wait until after we to the driver comp? | 13:42 |
| lucasagomes | jroll, yeah I didn't look into it, so I don't have much idea | 13:42 |
| lucasagomes | jroll, yeah that would help also | 13:43 |
| lucasagomes | so we don't have to create more entry points | 13:43 |
| jroll | lucasagomes: yeah, and they can share some of the code | 13:43 |
| lucasagomes | yeah, let's wait for the driver composition and then we can evaluate it | 13:44 |
| lucasagomes | (evaluate if it worth having an separated ipxe boot interface I mean) | 13:44 |
| *** mtanino has joined #openstack-ironic | 13:44 | |
| *** piet has quit IRC | 13:45 | |
| mkovacik__ | aarefiev, no worries, anyways, thanks for the review! I'll post the logging update soon. | 13:46 |
| jlvillal | jroll, on the functional testing RFE. Seems reasonable. I think once there is "some" functional testing it could be called done for the RFE. WDYT? | 13:48 |
| *** trown|afk is now known as trown | 13:51 | |
| jroll | jlvillal: what seems reasonable, closing it? | 13:52 |
| jroll | I'd be good with that | 13:52 |
| jlvillal | jroll, Once we have some functional testing, then closing it would be reasonable. But at the moment we don't have any. | 13:53 |
| jroll | jlvillal: oh, I see what you mean, sounds good | 13:53 |
| jlvillal | Thanks | 13:54 |
| jroll | thank you | 13:54 |
| *** piet has joined #openstack-ironic | 13:57 | |
| *** absubram has quit IRC | 14:02 | |
| *** dims has joined #openstack-ironic | 14:04 | |
| *** dims_ has quit IRC | 14:04 | |
| *** achanda has joined #openstack-ironic | 14:05 | |
| NobodyCam | good morning Ironicers | 14:05 |
| *** achanda has quit IRC | 14:07 | |
| NobodyCam | morning jroll jlvillal devananda mgould lucasagomes sambetts betherly TheJulia krotscheck and all others not directly listed here | 14:07 |
| jroll | morning NobodyCam | 14:07 |
| lucasagomes | NobodyCam, hi there! | 14:07 |
| NobodyCam | :) | 14:07 |
| betherly | Hi NobodyCam :) | 14:07 |
| *** alexpilotti has joined #openstack-ironic | 14:08 | |
| *** openstack has joined #openstack-ironic | 14:23 | |
| *** ametts has joined #openstack-ironic | 14:24 | |
| *** mgoddard has quit IRC | 14:30 | |
| *** dprince has quit IRC | 14:30 | |
| *** piet has quit IRC | 14:31 | |
| *** mgoddard has joined #openstack-ironic | 14:32 | |
| *** piet has joined #openstack-ironic | 14:38 | |
| *** alexpilotti has joined #openstack-ironic | 14:41 | |
| *** david-lyle_ is now known as david-lyle | 14:47 | |
| *** mdorman has joined #openstack-ironic | 14:47 | |
| *** athomas has quit IRC | 14:49 | |
| *** zzzeek has quit IRC | 14:50 | |
| *** athomas has joined #openstack-ironic | 14:53 | |
| sambetts | Morning NobodyCam | 14:57 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add flavor_id baremetal tempest option https://review.openstack.org/293020 | 14:59 |
| *** trown|meeting is now known as trown | 15:00 | |
| betherly | jroll: final patch upstream just waiting for krotscheck to have a chance to review it then will be ready to merge and submit the release | 15:01 |
| jroll | betherly: cool, thanks | 15:02 |
| *** zzzeek has joined #openstack-ironic | 15:02 | |
| *** absubram has joined #openstack-ironic | 15:05 | |
| jroll | lucasagomes: JayF: is this done? https://bugs.launchpad.net/ironic/+bug/1531217 | 15:07 |
| openstack | Launchpad bug 1531217 in Ironic "[RFE] Nodes should not always be powered off on failure" [Wishlist,In progress] - Assigned to Jay Faulkner (jason-oldos) | 15:07 |
| NobodyCam | :) | 15:08 |
| lucasagomes | jroll, I don't think so. I think JayF will update his patch soon https://review.openstack.org/#/c/259119 | 15:08 |
| lucasagomes | or if JayF don't mind, I can take it over | 15:09 |
| lucasagomes | since I'm interested on that feature | 15:09 |
| jlvillal | Hi NobodyCam | 15:09 |
| NobodyCam | hey hi there jlvillal :) | 15:09 |
| *** ddub_ has joined #openstack-ironic | 15:10 | |
| *** openstackstatus has joined #openstack-ironic | 15:12 | |
| *** ChanServ sets mode: +v openstackstatus | 15:12 | |
| *** garthb has joined #openstack-ironic | 15:12 | |
| jroll | lucasagomes: ok cool, thanks | 15:13 |
| *** mgoddard has quit IRC | 15:14 | |
| *** mgoddard_ has joined #openstack-ironic | 15:14 | |
| *** ddub_ has quit IRC | 15:15 | |
| *** saibarspeis has quit IRC | 15:15 | |
| *** mgoddard__ has joined #openstack-ironic | 15:17 | |
| *** gddub has joined #openstack-ironic | 15:19 | |
| *** mgoddard_ has quit IRC | 15:21 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Get root device hints from the node object https://review.openstack.org/296539 | 15:28 |
| jroll | holy cow, I made it through all the rfes :o | 15:30 |
| jroll | still 90+ that need more info or the spec is not merged | 15:30 |
| JayF | I can look at that today, lucasagomes jroll | 15:30 |
| jroll | JayF: no rush, was just curious about status | 15:30 |
| JayF | was going to start on that yesterday but took the time to read some specs and get back up to speed on open reviews | 15:30 |
| lucasagomes | JayF, cool, yeah lemme know otherwise I can just push some patches for that (reuse urs) | 15:30 |
| JayF | yeah I have basically most of this week, and hopefully longer, to work upstream, and the outstanding patches (that one and reboot requested) are at the top of my list | 15:31 |
| *** mgoddard has joined #openstack-ironic | 15:33 | |
| *** mgoddard__ has quit IRC | 15:33 | |
| *** jaybeale has joined #openstack-ironic | 15:37 | |
| betherly | krotscheck: +2'd the updated patches | 15:38 |
| krotscheck | betherly Woot | 15:39 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Allow rerunning introspection on stored data https://review.openstack.org/267637 | 15:40 |
| lucasagomes | JayF, ack thanks | 15:41 |
| *** Sukhdev has joined #openstack-ironic | 15:44 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Code style fix according to flake8 https://review.openstack.org/296552 | 15:45 |
| jroll | stepping away for a while | 15:50 |
| *** moshele has joined #openstack-ironic | 15:54 | |
| *** bharath has quit IRC | 15:56 | |
| *** ohamada has quit IRC | 15:57 | |
| *** mgoddard_ has joined #openstack-ironic | 15:57 | |
| *** praneshp_ has joined #openstack-ironic | 15:59 | |
| *** mgoddard has quit IRC | 16:01 | |
| *** ohamada has joined #openstack-ironic | 16:01 | |
| *** praneshp__ has joined #openstack-ironic | 16:02 | |
| *** mgoddard_ has quit IRC | 16:04 | |
| *** mgoddard has joined #openstack-ironic | 16:04 | |
| *** jaypipes has quit IRC | 16:04 | |
| *** praneshp_ has quit IRC | 16:04 | |
| *** links has joined #openstack-ironic | 16:06 | |
| *** jaypipes has joined #openstack-ironic | 16:07 | |
| *** jaypipes has left #openstack-ironic | 16:07 | |
| *** moshele has quit IRC | 16:15 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Code style fix according to flake8 https://review.openstack.org/296552 | 16:16 |
| *** ohamada has quit IRC | 16:16 | |
| *** Nisha_away has joined #openstack-ironic | 16:17 | |
| *** trown is now known as trown|lunch | 16:19 | |
| *** ohamada has joined #openstack-ironic | 16:19 | |
| *** Guest60842 has quit IRC | 16:21 | |
| jlvillal | JayF does this log look normal? http://paste.openstack.org/show/491617/ | 16:28 |
| *** praneshp__ has quit IRC | 16:28 | |
| *** saripurigopi has quit IRC | 16:28 | |
| JayF | jlvillal: looks sane to me | 16:29 |
| JayF | jlvillal: what kind of problem are you having? | 16:29 |
| jlvillal | JayF: Says the node isn't coming up | 16:29 |
| JayF | hmmm... did ipa not get started? | 16:29 |
| JayF | I think IPA didn't get started. | 16:30 |
| JayF | [ 254.213045] systemd[1]: Starting Untar Provided IPA Container... | 16:30 |
| JayF | [ 257.109100] systemd[1]: Failed to set memory.limit_in_bytes on : Invalid argument | 16:30 |
| JayF | [ 258.660045] systemd[1]: Failed to reset devices.list on /system.slice: Invalid argument | 16:30 |
| JayF | ^ I wonder if it's considering those fatal errors | 16:30 |
| *** harshs has joined #openstack-ironic | 16:30 | |
| jlvillal | I wasn't sure... | 16:30 |
| JayF | those are new things to me as well, haven't seen those before | 16:30 |
| jlvillal | I thought I had seen the memory limit one on working ones. | 16:30 |
| jlvillal | Not sure about the other ones | 16:30 |
| JayF | I mean, idk, but something stopped before it started the ipa service | 16:31 |
| JayF | can you get console on the node/ | 16:31 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Fix NamedTemporaryFile() OSError Exception https://review.openstack.org/293762 | 16:31 |
| *** rajinir has joined #openstack-ironic | 16:32 | |
| jlvillal | JayF: This is during a grenade run. I think it kills the vm. | 16:32 |
| jlvillal | Unsure how to stop that. | 16:32 |
| JayF | ah | 16:32 |
| JayF | wonder why a grenade run would be any different than any other boot | 16:32 |
| JayF | hmm | 16:32 |
| jlvillal | Grenade run is doing a devstack run on stable/liberty | 16:32 |
| jlvillal | Like our gate. | 16:33 |
| jlvillal | It would then go do other stuff | 16:33 |
| Nisha_away | jroll, o/ ironic 5.1.0 released | 16:33 |
| *** david-lyle has quit IRC | 16:33 | |
| JayF | hmm. | 16:33 |
| *** david-lyle has joined #openstack-ironic | 16:33 | |
| JayF | I'd be very interested to see if this is happening in a recent gate | 16:33 |
| betherly | jroll: ready to start the release for ironic-ui 1.1.0 | 16:34 |
| jlvillal | JayF: I think it is local to my environment. Not sure why though. So I was trying to debug it. | 16:35 |
| JayF | Yeah, I'd really try to get one of those booted so you can login | 16:36 |
| jlvillal | I have two different systems. Running the same code. One works and one doesn't. Odd. | 16:36 |
| JayF | if you get to that point I can help debug no problem | 16:36 |
| *** izaakk has joined #openstack-ironic | 16:36 | |
| JayF | are they running exact same ramdisk? like check sha1/md5? | 16:36 |
| *** vishwanathj_away is now known as vishwanathj | 16:39 | |
| jlvillal | JayF: They both dynamically build it up inside a VM. I'll keep looking at it. | 16:40 |
| jlvillal | JayF: Thanks! | 16:40 |
| JayF | np | 16:40 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Joined 'tags' column while getting node https://review.openstack.org/253065 | 16:41 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Config to stop powering off nodes on failure https://review.openstack.org/259119 | 16:43 |
| *** gddub has quit IRC | 16:43 | |
| *** aginwala has joined #openstack-ironic | 16:44 | |
| JayF | lucasagomes: jroll ^^ passing tests and limited to just one of the changes | 16:48 |
| *** aginwala has quit IRC | 16:49 | |
| *** links has quit IRC | 16:49 | |
| *** ifarkas has quit IRC | 16:50 | |
| *** alexpilotti has quit IRC | 16:51 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Uplink Set https://review.openstack.org/295981 | 16:52 |
| *** alexpilotti has joined #openstack-ironic | 16:56 | |
| *** achanda has joined #openstack-ironic | 16:58 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Correct test-bifrost-venv script https://review.openstack.org/296611 | 16:58 |
| *** rpioso has joined #openstack-ironic | 16:59 | |
| *** krtaylor has quit IRC | 17:01 | |
| *** alexpilotti has quit IRC | 17:01 | |
| JayF | jroll: lucasagomes: You all think I need a separate bug for the power_off_after_cleaning half of that patch/ | 17:01 |
| lucasagomes | JayF, reading the bug it talks about deploy and cleaning, so I think we are good | 17:03 |
| JayF | k I'll mark the second as related | 17:03 |
| JayF | it's going up in a sec | 17:03 |
| JayF | btw https://review.openstack.org/259119 is up and ready | 17:03 |
| lucasagomes | JayF, tho you will need a chain of patches first one Partial-Bug and the las Closes-Bug | 17:03 |
| lucasagomes | JayF, will take a look | 17:03 |
| JayF | I mean, I'll make the cleaning patch related-bug | 17:03 |
| *** achanda has quit IRC | 17:03 | |
| JayF | and if it reopens the bug I'll followup and make sure it's closed | 17:03 |
| JayF | rather than forcing order where it doesn't matter | 17:04 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [amt] https://review.openstack.org/295946 | 17:04 |
| sambetts | jlvillal: qa meeting today? | 17:04 |
| lucasagomes | JayF, right, tho in that case I would mark both as Partial and then close it... but I'm not picky :-) | 17:04 |
| jlvillal | sambetts: Sorry! | 17:04 |
| lucasagomes | JayF, just saying because the bug explicitly talks about cleaning, so closing it wihtout the cleaning is less ideal | 17:05 |
| sambetts | jlvillal: its all good :) | 17:05 |
| lucasagomes | (in case the deploy patch merges before the cleaning one) | 17:05 |
| *** dims_ has quit IRC | 17:05 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Make powering off nodes after cleaning optional https://review.openstack.org/296617 | 17:06 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Config to stop powering off nodes on failure https://review.openstack.org/259119 | 17:06 |
| *** dims has joined #openstack-ironic | 17:06 | |
| mkovacik__ | sambetts, hi, thanks a lot for the review on https://review.openstack.org/#/c/253675/14 | 17:06 |
| JayF | lucasagomes: both are now related-bug, and I'll make sure it gets closed once they both merge | 17:07 |
| lucasagomes | JayF, thanks much! I will take a look | 17:07 |
| *** ppiela has quit IRC | 17:07 | |
| JayF | lucasagomes: full disclosure: local tests already passing on deploy one, still running on cleaning (althouhg no reason to expect they won't pass) | 17:07 |
| sambetts | mkovacik__: :) | 17:08 |
| mkovacik__ | sambetts, yeah, too long to read I guess ;) | 17:08 |
| mkovacik__ | sambetts, do you have some design concerns with it? | 17:08 |
| mkovacik__ | I mean stuff like active--active vs active--passive design, splitting different ways, etc.. | 17:09 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [amt] https://review.openstack.org/295946 | 17:10 |
| *** Nisha_away has quit IRC | 17:11 | |
| sambetts | mkovacik__: A lot of my comments are just general language clean up stuff, but I had a couple of questions, the first being the comment around L138 and the other around L1040 | 17:11 |
| * mkovacik__ looks | 17:12 | |
| lucasagomes | JayF, looks good, just question/thought inline. Do you think we should add a similar note to the help text of that configuration option about Nova powering off the node anyway? | 17:12 |
| sambetts | mkovacik__: and I also thought we should define the usecase better as stated in the comment on L32 | 17:12 |
| JayF | lucasagomes: No, because there's no way that gets updated when we patch Nova. | 17:12 |
| JayF | lucasagomes: heh :) | 17:12 |
| NobodyCam | lintan: are you around? | 17:12 |
| JayF | lucasagomes: it's in release notes, which I think is sufficient | 17:12 |
| lucasagomes | JayF, yeah :-/ fair enough | 17:13 |
| *** krtaylor has joined #openstack-ironic | 17:13 | |
| *** achanda has joined #openstack-ironic | 17:14 | |
| lucasagomes | JayF, bumped my vote there | 17:14 |
| mkovacik__ | sambetts, right, with stating the use case, it should sustain couple of hundreds of requests in batches interleaved with silence for couple of minutes; also keeping couple of thousands firewall rules | 17:14 |
| mkovacik__ | but there are no data to base this on, just a size estimate of such a lab | 17:15 |
| mkovacik__ | somewhere 10k nodes | 17:15 |
| JayF | lucasagomes: just in time; tests just passed on my cleaning patch (locally), so you can go vote on that too :P | 17:15 |
| lucasagomes | cool | 17:16 |
| *** mgoddard has quit IRC | 17:16 | |
| mkovacik__ | sambetts, w/r the tooz, true, I have to rephrase maybe. The concern is there's no Queue implementation yet, but that can be addressed | 17:16 |
| *** mgoddard has joined #openstack-ironic | 17:16 | |
| mkovacik__ | sambetts, 138: was the former one; the lock that reduces reboot calls | 17:17 |
| *** sinval has joined #openstack-ironic | 17:17 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Addressing nits on Change-Id: I2984cd9d469622a65201fd9d50f964b144cce625 https://review.openstack.org/296621 | 17:17 |
| *** krtaylor has quit IRC | 17:17 | |
| NobodyCam | doh | 17:18 |
| JayF | easiest patch to review ever | 17:18 |
| *** manjeets has left #openstack-ironic | 17:19 | |
| *** alexpilotti has joined #openstack-ironic | 17:19 | |
| *** MattMan has quit IRC | 17:19 | |
| *** MattMan has joined #openstack-ironic | 17:19 | |
| NobodyCam | jroll: you had a +2 on https://review.openstack.org/#/c/261429 have a minute to take another look and I'll toss up a (real) follow up to address the nits :p | 17:20 |
| mkovacik__ | sambetts, btw the 10k estimate is CERN-sized use case | 17:20 |
| *** ppiela has joined #openstack-ironic | 17:20 | |
| JayF | oh the -v -v -v -v is a good idea | 17:20 |
| *** suro-patz has joined #openstack-ironic | 17:22 | |
| *** alexpilotti has quit IRC | 17:23 | |
| *** krtaylor has joined #openstack-ironic | 17:24 | |
| sambetts | mkovacik__: With the use case I would state that infomation you've just said here in the spec, instead of saying "the baretenant to tenant use case" because that doesn't mean much to me | 17:25 |
| *** derekh has quit IRC | 17:25 | |
| mkovacik__ | sambetts, yeah, I'll do just that | 17:26 |
| openstackgerrit | Lilia Sampaio proposed openstack/ironic: Add Dynamic Allocation feature for the OneView drivers https://review.openstack.org/286192 | 17:26 |
| *** aginwala has joined #openstack-ironic | 17:26 | |
| mkovacik__ | sambetts, thanks again! | 17:27 |
| * mkovacik__ gone | 17:27 | |
| sambetts | mkovacik__: awesome thanks :) | 17:27 |
| krtaylor | jlvillal, looks like you pinged me, my connection bounced except for that one message, missed the meeting :( | 17:27 |
| jlvillal | krtaylor: No worry. Just doing the 3rd Party CI section. | 17:28 |
| krtaylor | I was curious why it was so quiet :) | 17:28 |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Provide attach remote media function https://review.openstack.org/296047 | 17:28 |
| *** mkovacik__ has quit IRC | 17:29 | |
| *** rcernin has quit IRC | 17:29 | |
| *** achanda has quit IRC | 17:29 | |
| *** achanda has joined #openstack-ironic | 17:33 | |
| *** mbound has quit IRC | 17:36 | |
| *** trown|lunch is now known as trown | 17:36 | |
| jroll | NobodyCam: wow, that patch got bigger | 17:38 |
| *** mbound has joined #openstack-ironic | 17:38 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Addressing nits on I2984cd9d469622a65201fd9d50f964b144cce625 https://review.openstack.org/296621 | 17:38 |
| NobodyCam | jroll: ^^^ should address the comments on it | 17:39 |
| jroll | cool, thanks | 17:39 |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Network https://review.openstack.org/294559 | 17:40 |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Uplink Set https://review.openstack.org/295981 | 17:40 |
| *** achanda has quit IRC | 17:41 | |
| *** electrofelix has quit IRC | 17:45 | |
| jlvillal | jroll: Some initial Nova style config centralization work: https://review.openstack.org/#/c/295946/ | 17:46 |
| jlvillal | My co-worker is working on it. | 17:46 |
| *** alexpilotti has joined #openstack-ironic | 17:47 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Network https://review.openstack.org/294559 | 17:47 |
| jroll | jlvillal: yay, I get to use my -2 stick! | 17:48 |
| jlvillal | Heh :) | 17:48 |
| jroll | jlvillal: good stuff though, this is relevant to my interests | 17:48 |
| *** piet has quit IRC | 17:49 | |
| jlvillal | I think it and the oslo-config-generator can mostly be done independently. Some overlap, but not that much. | 17:49 |
| jroll | jlvillal: I'd like the generator thing done first | 17:50 |
| jlvillal | Works for us. | 17:50 |
| jroll | they will collide with each other, and as the generator requires something like https://review.openstack.org/#/c/247331/2/ironic/common/opts.py | 17:50 |
| jroll | it gives us a list to work off of :) | 17:50 |
| jlvillal | lintan: Is working on it, I believe. | 17:50 |
| *** mbound has quit IRC | 17:51 | |
| *** rama_y has joined #openstack-ironic | 17:51 | |
| jroll | yep | 17:51 |
| jroll | so I'm not suggesting they even need to work together (though that would help), just stack the patches accordingly | 17:51 |
| jlvillal | Agreed. They are in contact with each other. | 17:51 |
| *** achanda has joined #openstack-ironic | 17:51 | |
| TheJulia | cinerama: w/r/t 296611, any thoughts of adding a test job, or expanding a base test script? | 17:52 |
| *** piet has joined #openstack-ironic | 17:52 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 17:52 |
| jroll | jlvillal: awesome. thanks :) | 17:53 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Get root device hints from the node object https://review.openstack.org/296539 | 17:54 |
| jlvillal | jroll: I'm telling rama_y to keep working on it in parallel until the config-generator stuff is rebased and working again. She can rebase on top of the config-generator stuff I think without too much effort. | 17:54 |
| *** electrofelix has joined #openstack-ironic | 17:54 | |
| jroll | jlvillal: cool, sounds good | 17:55 |
| lucasagomes | I'm calling it a day, have a great evening all! | 17:56 |
| lucasagomes | see you all tomorrow (-: | 17:56 |
| lucasagomes | JayF, I will take a look at the cleaning patch tomorrow | 17:56 |
| JayF | lucasagomes: Looking at it, I think there are comments from jroll which are unanswered from the old patchset (fro the cleaning one) | 17:57 |
| JayF | lucasagomes: and honestly idk how to resolve them | 17:57 |
| * lucasagomes looks | 17:57 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Network https://review.openstack.org/294559 | 17:57 |
| JayF | lucasagomes: so save your time, I'll W-1 it, and try to nerd-snipe jroll into helping me come up with better ideas | 17:57 |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Uplink Set https://review.openstack.org/295981 | 17:57 |
| jroll | JayF: which one is unresolvable? | 17:57 |
| JayF | lucasagomes: mainly around when/if to cleanup cleaning network ports + pxe boot configs if we don't do it while exiting cleaning | 17:57 |
| JayF | jroll: ^ | 17:57 |
| jroll | mmmmm. | 17:57 |
| jroll | I mean we exit cleaning eventually, right? | 17:58 |
| JayF | jroll: not saying unsolveable, just I'm not clever enough to know how to yet :P | 17:58 |
| *** spandhe has joined #openstack-ironic | 17:58 | |
| jroll | JayF: well, tear_down_cleaning just needs to run eventually, probably when you head to the next state change | 17:58 |
| JayF | Well that's the thing, this doesn't power off after exiting cleaning | 17:58 |
| JayF | so this would be a node, in available, with the ramdisk still booted | 17:58 |
| JayF | jroll: are cleaning network & provisioning network separate settings? if so I think the patch is impossible | 17:59 |
| JayF | jroll: unless I limit it to cleaning failures only | 17:59 |
| *** alexpilotti has quit IRC | 17:59 | |
| lucasagomes | JayF, why available? Can't it be on CLEANFAIL state? | 17:59 |
| jroll | JayF: provisioning network doesn't exist until the neutron work lands | 17:59 |
| * jroll == lucasagomes | 17:59 | |
| JayF | that patch is "don't power off nodes after cleaning" | 18:00 |
| JayF | not "fail all cleaning LOL" | 18:00 |
| lucasagomes | and once you move state, it will take care of tear down the network | 18:00 |
| JayF | Now, if I made the patch "don't power off nodes after cleaning FAILS" then that works | 18:00 |
| JayF | but right now it's never-power-off-after cleaning | 18:00 |
| jroll | ohhhhhh. I see. | 18:00 |
| lucasagomes | JayF, oh right, I thought it was "do not power off after clean failure" | 18:00 |
| lucasagomes | or something like that, just like the deploy | 18:01 |
| JayF | lucasagomes: it not being that is why I had to split the patches | 18:01 |
| *** alexpilotti has joined #openstack-ironic | 18:01 | |
| lucasagomes | yeah if you want to keep it on even on success (is it some optmization !?) the network bits are tricky | 18:01 |
| JayF | lucasagomes: and obviously I think the cleaning one is more involved on further looking :) | 18:01 |
| lucasagomes | you will have to somehow power on the node | 18:01 |
| JayF | lucasagomes: if cleaning network == provisioning network && your ramdisks are the same, it is an optimization | 18:01 |
| lucasagomes | JayF, maybe you should go for "power on nodes on avialable" | 18:01 |
| JayF | lucasagomes: this patch may or may not be lifted directly from our downstream patchset | 18:01 |
| *** alexpilotti has quit IRC | 18:01 | |
| lucasagomes | instead of "after-cleaning" | 18:01 |
| rama_y | jlvillal, jroll: Thanks! | 18:02 |
| JayF | lucasagomes: I'm thinking, yeah, two patches: don't power off machines after cleaning fails && power on machines when going into available | 18:02 |
| JayF | but the second patch might be something we need a spec for | 18:02 |
| *** alexpilotti has joined #openstack-ironic | 18:02 | |
| jroll | JayF: yep, I agree | 18:02 |
| lucasagomes | JayF, it sounds more cleanear yes | 18:02 |
| JayF | at least traditionally the "long running ramdisk" work has been spec'd in the past | 18:02 |
| lucasagomes | fair enough | 18:03 |
| JayF | I'd be OK with something that simply prepares for deployment on transition to available, but I understand if folks want to -2 that in favor of a better long-running-ramdisk design where heartbeat is used and such | 18:03 |
| lucasagomes | JayF, yeah I think the long running ramdisk design would be good in that case, otherwise I don't see the benefit of leaving the nodes powered on for no reasn | 18:04 |
| lucasagomes | reason* | 18:04 |
| JayF | I mean, I see the reason | 18:05 |
| lucasagomes | anyway, maybe we can continue the discussion on the patch? | 18:05 |
| JayF | because today if you just have them on, Ironic will happily deploy to that agent | 18:05 |
| sambetts | Has anyone else ever seen this http://paste.openstack.org/show/491623/ happen in the nova scheduler, directly after a node-update | 18:05 |
| JayF | I know that because that's how it works in my prod environment today :) | 18:05 |
| *** Sukhdev has quit IRC | 18:06 | |
| JayF | but yeah ,I'll make the cleaning patch on cleanfail only, and approach the other problem separately | 18:06 |
| lucasagomes | JayF, oh right, but with the code upstream we are not going to reboot de node prior to start the deployment? | 18:06 |
| JayF | Nope, we wouldn't | 18:06 |
| lucasagomes | if that's the case it's kinda useless to what we have upstream | 18:06 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Addressing nits on I2984cd9d469622a65201fd9d50f964b144cce625 https://review.openstack.org/296621 | 18:06 |
| lucasagomes | JayF, alright, so yeah it sounds good | 18:06 |
| JayF | I'll double check but I don't think we have any patches to change that behavior | 18:06 |
| JayF | and if we do, I'll upstream that too, lol | 18:06 |
| * JayF downstream patch assassin | 18:07 | |
| lucasagomes | JayF, fair enough... yeah I'm not 100% sure either | 18:07 |
| *** yuanying_ has joined #openstack-ironic | 18:07 | |
| lucasagomes | JayF, ++ upstream everything | 18:07 |
| *** alexpilotti has quit IRC | 18:07 | |
| lucasagomes | alright, calling it a day :-) | 18:08 |
| lucasagomes | have a great evening all | 18:08 |
| JayF | nn | 18:08 |
| NobodyCam | have a good night lucasagomes :) | 18:08 |
| *** lucasagomes is now known as lucas-dinner | 18:08 | |
| *** achanda has quit IRC | 18:09 | |
| *** yuanying has quit IRC | 18:09 | |
| *** piet has quit IRC | 18:10 | |
| *** ohamada has quit IRC | 18:10 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [amt] https://review.openstack.org/295946 | 18:11 |
| *** alexpilotti has joined #openstack-ironic | 18:13 | |
| *** alexpilotti has quit IRC | 18:14 | |
| *** alexpilo_ has joined #openstack-ironic | 18:14 | |
| *** krotscheck has quit IRC | 18:14 | |
| *** krotscheck has joined #openstack-ironic | 18:15 | |
| *** gabriel-bezerra has quit IRC | 18:15 | |
| *** sinval has quit IRC | 18:15 | |
| *** afaranha has quit IRC | 18:15 | |
| *** nicodemos has quit IRC | 18:15 | |
| *** clenimar has quit IRC | 18:16 | |
| *** alexpilo_ has quit IRC | 18:17 | |
| *** alexpilotti has joined #openstack-ironic | 18:17 | |
| *** alexpilotti has quit IRC | 18:18 | |
| *** alexpilotti has joined #openstack-ironic | 18:19 | |
| *** alexpilotti has quit IRC | 18:19 | |
| *** nicodemos has joined #openstack-ironic | 18:25 | |
| *** gabriel-bezerra has joined #openstack-ironic | 18:26 | |
| *** krotscheck has quit IRC | 18:26 | |
| *** raildo is now known as raildo-afk | 18:26 | |
| *** raildo-afk is now known as raildo | 18:26 | |
| *** krotscheck has joined #openstack-ironic | 18:26 | |
| *** clenimar has joined #openstack-ironic | 18:28 | |
| *** sinval has joined #openstack-ironic | 18:28 | |
| *** piet has joined #openstack-ironic | 18:28 | |
| *** praneshp_ has joined #openstack-ironic | 18:28 | |
| *** aginwala has quit IRC | 18:30 | |
| * sambetts is heading off | 18:31 | |
| sambetts | see everyone tomorrow | 18:31 |
| JayF | o/ | 18:31 |
| sambetts | cya JayF | 18:31 |
| *** kromanenko has quit IRC | 18:31 | |
| *** sambetts is now known as sambetts|afk | 18:31 | |
| NobodyCam | night sambetts|afk | 18:33 |
| *** electrofelix has quit IRC | 18:34 | |
| *** aginwala has joined #openstack-ironic | 18:37 | |
| *** jaypipes has joined #openstack-ironic | 18:38 | |
| *** krotscheck has quit IRC | 18:38 | |
| *** krotscheck has joined #openstack-ironic | 18:39 | |
| *** shamail_ has joined #openstack-ironic | 18:42 | |
| *** shamail_ has quit IRC | 18:46 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Support reboot_requested bool on agent clean_steps https://review.openstack.org/259135 | 18:46 |
| *** aginwala has quit IRC | 18:47 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Support reboot_requested bool on agent clean_steps https://review.openstack.org/259135 | 18:48 |
| *** mgould has quit IRC | 18:49 | |
| *** sinval has quit IRC | 18:50 | |
| *** gabriel-bezerra has quit IRC | 18:51 | |
| *** clenimar has quit IRC | 18:51 | |
| *** openstack has joined #openstack-ironic | 19:22 | |
| *** openstackstatus has joined #openstack-ironic | 19:23 | |
| *** ChanServ sets mode: +v openstackstatus | 19:23 | |
| *** serverascode_ is now known as serverascode | 19:23 | |
| *** dtantsur has joined #openstack-ironic | 19:24 | |
| *** alexpilotti has joined #openstack-ironic | 19:25 | |
| *** alexpilotti has quit IRC | 19:26 | |
| *** alaski_ is now known as alaski | 19:28 | |
| openstackgerrit | Clenimar Filemon proposed openstack/ironic: Request service token using Identity v3 by default https://review.openstack.org/296714 | 19:29 |
| *** JasonF is now known as JayF | 19:30 | |
| *** [1]cdearborn has joined #openstack-ironic | 19:31 | |
| *** aginwala has joined #openstack-ironic | 19:32 | |
| *** achanda has joined #openstack-ironic | 19:35 | |
| *** penick has joined #openstack-ironic | 19:39 | |
| *** Sukhdev has joined #openstack-ironic | 19:39 | |
| *** nicodemos has quit IRC | 19:39 | |
| *** aginwala has quit IRC | 19:42 | |
| *** [2]cdearborn has joined #openstack-ironic | 19:42 | |
| *** dims_ has joined #openstack-ironic | 19:43 | |
| *** aginwala has joined #openstack-ironic | 19:44 | |
| *** dims has quit IRC | 19:45 | |
| *** ChubYann has joined #openstack-ironic | 19:45 | |
| *** cdearborn has quit IRC | 19:46 | |
| *** [2]cdearborn is now known as cdearborn | 19:46 | |
| *** alexpilotti has joined #openstack-ironic | 19:46 | |
| *** alexpilotti has quit IRC | 19:51 | |
| *** moshele has joined #openstack-ironic | 19:52 | |
| *** dims_ has quit IRC | 19:53 | |
| *** alexpilotti has joined #openstack-ironic | 19:54 | |
| *** alexpilotti has quit IRC | 19:55 | |
| *** alexpilotti has joined #openstack-ironic | 19:55 | |
| *** alexpilotti has quit IRC | 19:56 | |
| *** dims has joined #openstack-ironic | 19:56 | |
| *** alexpilotti has joined #openstack-ironic | 19:56 | |
| *** alexpilotti has quit IRC | 19:56 | |
| *** [1]cdearborn has quit IRC | 19:57 | |
| *** Sukhdev has quit IRC | 20:01 | |
| *** awiddersheim has quit IRC | 20:13 | |
| *** rajinir has quit IRC | 20:15 | |
| *** e0ne has joined #openstack-ironic | 20:17 | |
| jlvillal | Is there a relatively easy way to attach to the console of an ironic bare-metal VM inside devstack? | 20:17 |
| jlvillal | I'm thinking vnc might be the easiest, but thought I would check if other options. | 20:18 |
| *** achanda has quit IRC | 20:18 | |
| jlvillal | I see that the ironic bare-metal VM gets launched with a 'vnc' option. | 20:18 |
| JayF | I think virsh console does it? | 20:18 |
| JayF | you can look at the code that mades the bm logs i nthe gate | 20:19 |
| jlvillal | JayF: I tried that and got on error :( | 20:19 |
| *** openstack has joined #openstack-ironic | 20:33 | |
| *** e0ne has quit IRC | 20:37 | |
| *** Sukhdev has quit IRC | 20:38 | |
| *** Goneri has quit IRC | 20:45 | |
| *** aginwala has quit IRC | 20:56 | |
| *** david-lyle has quit IRC | 20:59 | |
| *** achanda has joined #openstack-ironic | 21:01 | |
| *** alexpilotti has joined #openstack-ironic | 21:01 | |
| *** aginwala has joined #openstack-ironic | 21:01 | |
| *** aginwala has quit IRC | 21:01 | |
| *** aginwala has joined #openstack-ironic | 21:03 | |
| *** david-lyle has joined #openstack-ironic | 21:05 | |
| *** alexpilotti has quit IRC | 21:05 | |
| *** sinval has quit IRC | 21:06 | |
| *** jaypipes has quit IRC | 21:06 | |
| *** awiddersheim has joined #openstack-ironic | 21:06 | |
| *** alexpilotti has joined #openstack-ironic | 21:08 | |
| *** alexpilotti has quit IRC | 21:12 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add flavor_id baremetal tempest option https://review.openstack.org/293020 | 21:16 |
| *** trown is now known as trown|outtypewww | 21:17 | |
| *** mdorman has quit IRC | 21:21 | |
| *** david-lyle has quit IRC | 21:21 | |
| *** alexpilotti has joined #openstack-ironic | 21:23 | |
| *** alexpilotti has quit IRC | 21:23 | |
| *** alexpilo_ has joined #openstack-ironic | 21:23 | |
| *** david-lyle has joined #openstack-ironic | 21:23 | |
| *** Sukhdev has joined #openstack-ironic | 21:26 | |
| *** jaybeale has quit IRC | 21:33 | |
| *** daemontool_ has quit IRC | 21:34 | |
| *** baoli has quit IRC | 21:36 | |
| *** daemontool_ has joined #openstack-ironic | 21:38 | |
| *** saripurigopi has joined #openstack-ironic | 21:39 | |
| *** achanda has quit IRC | 21:46 | |
| *** mannidi_ has joined #openstack-ironic | 22:04 | |
| *** mannidi__ has quit IRC | 22:07 | |
| *** baoli has joined #openstack-ironic | 22:09 | |
| *** greghayn1 is now known as greghaynes | 22:11 | |
| *** mannidi_ has quit IRC | 22:11 | |
| *** ametts has quit IRC | 22:14 | |
| *** achanda has joined #openstack-ironic | 22:16 | |
| *** jcoufal has quit IRC | 22:18 | |
| *** baoli has quit IRC | 22:18 | |
| *** saripurigopi1 has joined #openstack-ironic | 22:27 | |
| *** rbudden has quit IRC | 22:28 | |
| *** moshele has quit IRC | 22:29 | |
| *** saripurigopi has quit IRC | 22:30 | |
| NobodyCam | humm: am I wrong here: https://review.openstack.org/#/c/296428/2/releasenotes/source/index.rst current (line 8) should be 5.1.0 and mitaka should be 4.3.0 - 5.1.0 ?? | 22:31 |
| *** daemontool_ has quit IRC | 22:31 | |
| *** awiddersheim has quit IRC | 22:41 | |
| *** garthb has joined #openstack-ironic | 22:43 | |
| TheJulia | that change kind of seems wrong given its not the stock reno notes | 22:46 |
| NobodyCam | posted a non-voting comment | 22:49 |
| NobodyCam | :p | 22:49 |
| TheJulia | the other part of the change, is correct though | 22:49 |
| NobodyCam | ya | 22:49 |
| TheJulia | mitaka.rst | 22:49 |
| *** aginwala has quit IRC | 22:49 | |
| TheJulia | NobodyCam: I think it might just be best for us to fix that though | 22:50 |
| *** Goneri has joined #openstack-ironic | 22:50 | |
| TheJulia | NobodyCam: given, it is our release notes | 22:50 |
| NobodyCam | I would but was unsure if we needed to wait until the actual release (ie the summit) for the change I menshioned | 22:51 |
| TheJulia | NobodyCam: needs to be changed on master once the branch is cut | 22:51 |
| TheJulia | since the branch reflects the changes, and then the change imho should be cherry picked down to the stable branch | 22:52 |
| * TheJulia might also be crazy | 22:52 | |
| NobodyCam | lol it is late in the day | 22:52 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Support reboot_requested bool on agent clean_steps https://review.openstack.org/259135 | 22:54 |
| *** aginwala has joined #openstack-ironic | 22:56 | |
| *** aginwala has quit IRC | 22:58 | |
| JayF | I have a strange test failure, looks like something I couldn't have caused | 22:59 |
| JayF | but also looks like something that shouldn't be intermittant | 22:59 |
| JayF | http://logs.openstack.org/19/259119/9/check/gate-ironic-python34-db/438bc8e/testr_results.html.gz related to https://review.openstack.org/#/c/259119/ | 23:00 |
| JayF | it's only that new gate-ironic-python34-db job, that I have no idea what it does, lol | 23:00 |
| TheJulia | nice | 23:03 |
| *** aginwala_ has joined #openstack-ironic | 23:03 | |
| TheJulia | oh, they are renaming base tests if it uses a db or not | 23:03 |
| TheJulia | and our unit tests can leverage a db | 23:03 |
| JayF | this is just a weird failure | 23:04 |
| JayF | if you look at the error | 23:04 |
| JayF | doesn't look like something that'd be random | 23:04 |
| JayF | I should find another patch that recently had tests run. | 23:04 |
| TheJulia | yeah, i know, that does look weird :( | 23:04 |
| JayF | hm. it passed here: https://review.openstack.org/#/c/259135/ at like 1:17pm | 23:05 |
| *** alaski has quit IRC | 23:05 | |
| *** greghaynes has quit IRC | 23:05 | |
| JayF | I'm going to recheck but I suspect shenanigans | 23:05 |
| *** aginwala_ has quit IRC | 23:05 | |
| *** aginwala has joined #openstack-ironic | 23:06 | |
| TheJulia | the shenanigan that I can think of is the base image type change.... but yeah :\ | 23:06 |
| JayF | I mean, even that being environmental would be weird, really | 23:07 |
| JayF | I rechecked it for scienec | 23:07 |
| *** rpioso has quit IRC | 23:09 | |
| TheJulia | oh true, espescially with the error :( | 23:10 |
| * TheJulia goes to amazon and begins looking for a lab coat | 23:10 | |
| *** alaski has joined #openstack-ironic | 23:11 | |
| jroll | TheJulia: NobodyCam: I actually meant to fix that today but didn't get to it. I'll do it in the morning | 23:13 |
| JayF | TheJulia: I can totes see you wearing a lab coat and having a great maniacal laugh to go along with it | 23:13 |
| JayF | TheJulia: lol | 23:13 |
| TheJulia | JayF: so I'm hearing, I should have one for summit, now I'm actually really looking for one | 23:13 |
| JayF | haha | 23:14 |
| JayF | wear it to a bbq resturant, and have a lot of fake tools with you | 23:14 |
| JayF | and act like you're measuring like, the smoke in the meat or something | 23:14 |
| JayF | and just go "hmm" and "interesting" a lot | 23:14 |
| TheJulia | lol | 23:15 |
| TheJulia | now... black lab coats or white lab coats... so much to consider | 23:15 |
| * TheJulia wonders if they come in purple | 23:15 | |
| jroll | JayF: why do I feel like that test failure is a really tight race | 23:17 |
| jroll | JayF: https://github.com/openstack/ironic/blob/master/ironic/tests/unit/common/test_hash_ring.py#L262 | 23:17 |
| JayF | jroll: because it is, looking at that code | 23:18 |
| JayF | lol | 23:18 |
| jroll | well, a millisecond should be enough for it not to be | 23:18 |
| jroll | apparently these images run fast :P | 23:18 |
| JayF | I mean, do you know for sure conductor would reset the hash ring immediately? | 23:18 |
| JayF | I'd assume that happens in some kind of loop with a delay | 23:18 |
| jroll | https://github.com/openstack/ironic/blob/master/ironic/common/hash_ring.py#L180 | 23:19 |
| jroll | it happens on access | 23:19 |
| JayF | and the access is in the test, and fails | 23:19 |
| jroll | self.ring hits that code path | 23:19 |
| jroll | right | 23:19 |
| *** greghaynes has joined #openstack-ironic | 23:19 | |
| jroll | actually I wonder if the db stores milliseconds, or rounds | 23:20 |
| *** absubram has quit IRC | 23:21 | |
| jroll | JayF: anyway, this should fix it | 23:21 |
| jroll | or we have a real bug :) | 23:21 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fix race in hash ring refresh unit test https://review.openstack.org/296837 | 23:21 |
| * jroll waits for gerrit | 23:21 | |
| *** openstack has joined #openstack-ironic | 23:24 | |
| * TheJulia ponders ordering a white lab coat and dying it purple | 23:26 | |
| *** Goneri has quit IRC | 23:27 | |
| JayF | TheJulia: I'd prefer light red. Purple is a strong color. <.< >.> | 23:29 |
| TheJulia | <3 | 23:29 |
| JayF | I'll wear it and say "DOOOM" just for you if you get it in light red, lol | 23:29 |
| TheJulia | lol | 23:29 |
| * TheJulia puts item on calendar to remind her to look at lab coats tomorrow :) | 23:30 | |
| TheJulia | in more detail | 23:30 |
| * TheJulia calls it a day | 23:32 | |
| *** harlowja_at_home has quit IRC | 23:45 | |
| *** harlowja_at_home has joined #openstack-ironic | 23:46 | |
| *** aginwala has quit IRC | 23:47 | |
| *** aginwala has joined #openstack-ironic | 23:49 | |
| *** aginwala has quit IRC | 23:50 | |
| *** smoriya_ has joined #openstack-ironic | 23:52 | |
| *** jaybeale has joined #openstack-ironic | 23:56 | |
| *** Sukhdev has quit IRC | 23:57 | |
| JayF | https://review.openstack.org/#/c/261429/ would be an easy one to land for any cores, 4x +1, 1x +1 (nobodycm) | 23:58 |
| *** Sukhdev has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!