| *** lucas-dinner has quit IRC | 00:00 | |
| NobodyCam | jroll: I stole that from http://docs.openstack.org/developer/tripleo-incubator/deploying.html#install-seed | 00:01 |
|---|---|---|
| rfchapman | NobodyCam: is 8775 the port for the metadat service? | 00:01 |
| NobodyCam | yep | 00:01 |
| rfchapman | OK, let me see if that works | 00:01 |
| *** harlowja has quit IRC | 00:02 | |
| jroll | NobodyCam: oh, does trying to talk to 169.254.169.254 just spam the network or something? | 00:02 |
| NobodyCam | its not brodcast | 00:03 |
| *** harlowja has joined #openstack-ironic | 00:03 | |
| NobodyCam | but can be redirected at the gateway | 00:03 |
| NobodyCam | vm host | 00:03 |
| jroll | hm | 00:04 |
| *** yuanying has quit IRC | 00:04 | |
| NobodyCam | nova dose it: https://github.com/openstack/nova/blob/master/etc/nova/rootwrap.d/compute.filters#L61 | 00:04 |
| NobodyCam | lol | 00:05 |
| NobodyCam | well thay actually add the address | 00:05 |
| NobodyCam | but forwarding os the same no? | 00:05 |
| NobodyCam | lol *ducks* | 00:05 |
| jroll | well, they have a hypervisor and stuff | 00:06 |
| *** yuanying has joined #openstack-ironic | 00:06 | |
| jroll | I have no idea how networks work, don't mind me :) | 00:06 |
| NobodyCam | rfchapman: also have you looked at using a configdrive? | 00:06 |
| *** yuanying has quit IRC | 00:07 | |
| rfchapman | NobodyCam: a configdrive? nope haven't tried that. | 00:07 |
| NobodyCam | do we have docs on using configdrives? lol :( | 00:08 |
| jroll | it's a nova thing | 00:09 |
| jroll | use their docs | 00:09 |
| jroll | has nothing to do with us, we just make it work | 00:09 |
| jroll | (other than "install kilo") | 00:09 |
| NobodyCam | rfchapman: are you using packaged ironic or trunk? | 00:10 |
| rfchapman | NobodyCam: it is packaged with "PackStack" | 00:11 |
| NobodyCam | rfchapman: ahh then my of the top of the head response is. I'll bet you don't have configdrive support atm.. | 00:12 |
| NobodyCam | that support landed in this current cycle | 00:13 |
| *** yuanying has joined #openstack-ironic | 00:13 | |
| rfchapman | Ahh, I couln't get devstack to install in my invironment. I'm behind a PROXY and I had errors poping up all over the place. | 00:13 |
| NobodyCam | :( | 00:14 |
| rfchapman | Even thoug I had the proxy configured seven ways to sunday the devstack distro just would not install. | 00:14 |
| NobodyCam | I have had issues behind FW / proxy in the past | 00:15 |
| rfchapman | Neither would packstack until I wen't home and installed a VM with it and it installed without a hitch. I re-tried devstack at home and it still wouldn't install. | 00:15 |
| NobodyCam | rfchapman: :( | 00:15 |
| NobodyCam | fyi: the issues I had were tacked down to a bad cert on the proxy server | 00:16 |
| rfchapman | NobodyCam: Hummm, I wonder... I'm still not that familure with the overall environment, but I'll have to ask around and see if anyone has any ideas. | 00:17 |
| *** yuanying has quit IRC | 00:19 | |
| rfchapman | Nobody: If I add a ssh-key to a qcow2 image, and use that image to build a dib image, shouldn't that key be in the final image? I tried this last friday and it didn't seem to work. | 00:21 |
| rfchapman | I guess it didn't grab the root directory. | 00:21 |
| rfchapman | I must be tired, Im' ytping fnnuy | 00:22 |
| JayF | funny | 00:22 |
| JayF | Hey, me too! /s | 00:22 |
| JayF | :P | 00:22 |
| rfchapman | :-) | 00:22 |
| *** yuanying has joined #openstack-ironic | 00:23 | |
| NobodyCam | lol yep. but then again I always type like that | 00:23 |
| NobodyCam | :-p | 00:23 |
| NobodyCam | or I words out | 00:24 |
| NobodyCam | *leave | 00:24 |
| NobodyCam | lol | 00:24 |
| * BadCub goes away for the night | 00:25 | |
| NobodyCam | ok I need to step away for housey things (digging up the back yard) | 00:25 |
| *** BadCub is now known as BadCub_Hiding | 00:25 | |
| rfchapman | NobodyCam: Thanks for the help.. | 00:26 |
| NobodyCam | rfchapman: sure.. happy to help | 00:28 |
| NobodyCam | lol you always join channel so close to 5 | 00:28 |
| rfchapman | Yea, I got busy tonight | 00:29 |
| NobodyCam | :) | 00:29 |
| NobodyCam | rfchapman: also take a look at the https://github.com/openstack/tripleo-image-elements/tree/master/elements/seed-stack-config element | 00:29 |
| rfchapman | I'll see if I can't try earlier tomorrow. | 00:30 |
| rfchapman | OK thanks | 00:30 |
| NobodyCam | would to be modifyed a little but might do what you need | 00:30 |
| NobodyCam | ok steps away ... night all | 00:31 |
| openstackgerrit | Merged openstack/ironic-python-agent: Add sphinx build + basic documentation https://review.openstack.org/168503 | 00:32 |
| *** Sukhdev has joined #openstack-ironic | 00:32 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Fix Sphinx Autodoc WARNING/ERROR in docs build https://review.openstack.org/169545 | 00:39 |
| *** yuanying has quit IRC | 00:53 | |
| *** yuanying has joined #openstack-ironic | 00:57 | |
| openstackgerrit | Michael Davies proposed openstack/ironic: New field 'name' not supported in port REST API https://review.openstack.org/163730 | 01:00 |
| *** naohirot has joined #openstack-ironic | 01:06 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: pxe deploy fails for whole disk images in UEFI https://review.openstack.org/167882 | 01:09 |
| *** zhenguo has joined #openstack-ironic | 01:12 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: boot_mode is overwritten in node properties https://review.openstack.org/167952 | 01:13 |
| *** ChuckC has quit IRC | 01:16 | |
| *** jerryz_ has joined #openstack-ironic | 01:35 | |
| openstackgerrit | Merged openstack/ironic: Enable pxe_amt driver with localboot https://review.openstack.org/167975 | 01:36 |
| *** oomichi has joined #openstack-ironic | 01:39 | |
| *** chenglch has joined #openstack-ironic | 01:45 | |
| *** ChuckC has joined #openstack-ironic | 01:47 | |
| *** rloo has quit IRC | 01:49 | |
| *** ChuckC has quit IRC | 01:51 | |
| *** ChuckC has joined #openstack-ironic | 01:52 | |
| *** Haomeng has quit IRC | 01:52 | |
| *** Haomeng|2 has joined #openstack-ironic | 01:52 | |
| *** Haomeng has joined #openstack-ironic | 01:53 | |
| *** chenglch|2 has joined #openstack-ironic | 01:54 | |
| *** chenglch|3 has joined #openstack-ironic | 02:08 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: pxe deploy fails for whole disk images in UEFI https://review.openstack.org/167882 | 02:09 |
| *** mtanino has quit IRC | 02:10 | |
| *** ChuckC has quit IRC | 02:10 | |
| *** ChuckC has joined #openstack-ironic | 02:10 | |
| *** Sukhdev has quit IRC | 02:14 | |
| *** chenglch|4 has joined #openstack-ironic | 02:17 | |
| *** chenglc4 has joined #openstack-ironic | 02:18 | |
| *** chenglch|3 has quit IRC | 02:18 | |
| *** chenglch has quit IRC | 02:19 | |
| *** chenglch|2 has quit IRC | 02:19 | |
| *** chenglch|2 has joined #openstack-ironic | 02:19 | |
| *** harlowja is now known as harlowja_away | 02:29 | |
| *** ramineni has joined #openstack-ironic | 02:35 | |
| *** achanda has quit IRC | 02:59 | |
| *** david-lyle has joined #openstack-ironic | 03:13 | |
| *** saripurigopi has joined #openstack-ironic | 03:41 | |
| *** Marga_ has quit IRC | 03:44 | |
| *** Marga_ has joined #openstack-ironic | 03:46 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Improve AMT driver doc https://review.openstack.org/169606 | 03:55 |
| *** Sukhdev has joined #openstack-ironic | 04:00 | |
| openstackgerrit | Merged openstack/ironic: Fix cleaning nits https://review.openstack.org/168197 | 04:02 |
| *** saripurigopi has quit IRC | 04:04 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: iLO driver documentation for UEFI secure boot https://review.openstack.org/168844 | 04:08 |
| *** saripurigopi has joined #openstack-ironic | 04:09 | |
| *** ramineni has quit IRC | 04:11 | |
| *** achanda has joined #openstack-ironic | 04:12 | |
| *** ramineni has joined #openstack-ironic | 04:18 | |
| *** chenglc4 has quit IRC | 04:37 | |
| *** chenglch|2 has quit IRC | 04:37 | |
| *** chenglch|4 has quit IRC | 04:37 | |
| *** chenglch has joined #openstack-ironic | 04:37 | |
| *** Marga_ has quit IRC | 04:42 | |
| *** krtaylor has quit IRC | 04:48 | |
| *** krtaylor has joined #openstack-ironic | 04:50 | |
| *** Marga_ has joined #openstack-ironic | 05:03 | |
| *** Marga_ has quit IRC | 05:03 | |
| *** Marga_ has joined #openstack-ironic | 05:04 | |
| *** Sukhdev has quit IRC | 06:00 | |
| *** pradipta has joined #openstack-ironic | 06:05 | |
| *** kalpase has joined #openstack-ironic | 06:10 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Transifex https://review.openstack.org/169184 | 06:13 |
| *** kalpase has quit IRC | 06:20 | |
| *** dlpartain has joined #openstack-ironic | 06:36 | |
| *** dlpartain has left #openstack-ironic | 06:36 | |
| *** kalpase has joined #openstack-ironic | 06:57 | |
| *** jcoufal has joined #openstack-ironic | 07:01 | |
| *** romcheg has joined #openstack-ironic | 07:04 | |
| *** ukalifon1 has joined #openstack-ironic | 07:05 | |
| openstackgerrit | Michael Davies proposed openstack/ironic: Disable XML now that we have WSME/Pecan support https://review.openstack.org/169643 | 07:12 |
| *** jistr has joined #openstack-ironic | 07:14 | |
| mrda | devananda, BadCub: XML now disabled in our API ^^^ | 07:17 |
| *** jcoufal has quit IRC | 07:32 | |
| *** mrda is now known as mrda-away | 07:33 | |
| *** yog__ has joined #openstack-ironic | 07:36 | |
| *** achanda has quit IRC | 07:36 | |
| *** sambetts has quit IRC | 07:38 | |
| *** sambetts has joined #openstack-ironic | 07:40 | |
| *** achanda has joined #openstack-ironic | 07:42 | |
| *** achanda has quit IRC | 07:43 | |
| *** rameshg87 has joined #openstack-ironic | 07:50 | |
| *** ramineni has quit IRC | 07:50 | |
| rameshg87 | good afternoon ironic | 07:50 |
| *** Marga_ has quit IRC | 07:50 | |
| *** jcoufal has joined #openstack-ironic | 07:56 | |
| *** mgoddard has joined #openstack-ironic | 08:01 | |
| *** mgoddard has quit IRC | 08:04 | |
| *** ndipanov has joined #openstack-ironic | 08:07 | |
| *** romcheg has quit IRC | 08:21 | |
| *** lucasagomes has joined #openstack-ironic | 08:24 | |
| *** jerryz_ has quit IRC | 08:28 | |
| *** romcheg has joined #openstack-ironic | 08:32 | |
| *** ramineni has joined #openstack-ironic | 08:36 | |
| *** athomas has joined #openstack-ironic | 08:38 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add config drive documentation https://review.openstack.org/166201 | 08:39 |
| *** edcragg has joined #openstack-ironic | 08:52 | |
| *** pas-ha has joined #openstack-ironic | 08:57 | |
| *** andreykurilin_ has joined #openstack-ironic | 08:59 | |
| *** dtantsur|afk is now known as dtantsur | 09:00 | |
| dtantsur | Morning Ironic | 09:00 |
| tiagogomes_ | Hi, by configuring compute_driver=nova.virt.ironic.IronicDriver on nova.conf, does this makes the compute nodes to not provision VMs anymore? | 09:11 |
| dtantsur | I think so | 09:12 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Add PXE-AMT driver's support of IPA ramdisk https://review.openstack.org/169664 | 09:13 |
| *** pelix has joined #openstack-ironic | 09:13 | |
| tiagogomes_ | erm, on http://docs.openstack.org/developer/ironic/deploy/install-guide.html, it says that needs to be done for every compute service controller and compute node. So is not possible to provision VMs and baremetal at the same time? | 09:15 |
| yuriyz | morning Ironic | 09:22 |
| yuriyz | tiagogomes you need separate compute nodes for VMs and baremetal (multiple hypervisors configuration), and you can change only scheduler on controller scheduler_host_manager=nova.scheduler.ironic_host_manager.IronicHostManager | 09:25 |
| yuriyz | this scheduler compatible with both VM and baremetal | 09:25 |
| dtantsur | yuriyz, morning | 09:26 |
| yuriyz | and you need filtering for run instances on the proper hypervisor | 09:26 |
| *** takadayuiko has joined #openstack-ironic | 09:34 | |
| takadayuiko | Good evening, Ironic :) | 09:34 |
| dtantsur | takadayuiko, g'evening | 09:36 |
| tiagogomes_ | yuriyz, so in nova.conf on the controller I set scheduler_host_manager=nova.scheduler.ironic_host_manager.IronicHostManager, and in the compute node that I want to use for baremetal I set compute_driver=nova.virt.ironic.IronicDriver? | 09:38 |
| tiagogomes_ | What is the reason that I can't use a compute node both for VM and baremetal provisioning? | 09:39 |
| takadayuiko | dtantsur, Hi! Good morning? :D | 09:39 |
| dtantsur | yeah, it's nearly noon here, but it's usually morning for me personally :) | 09:40 |
| takadayuiko | dtantsur, you're looking forward to lunch, right? BTW, introspection success!! :D | 09:50 |
| yuriyz | tiagogomes AFAIK a compute node can have only one compute_driver | 09:58 |
| *** naohirot has quit IRC | 09:59 | |
| *** chenglch has quit IRC | 09:59 | |
| *** yuanying has quit IRC | 10:00 | |
| *** alexpilotti has joined #openstack-ironic | 10:04 | |
| dtantsur | takadayuiko, awesome! \o/ | 10:07 |
| dtantsur | yeah, I'm going for some food | 10:07 |
| takadayuiko | dtantsur, thank you! o/ | 10:08 |
| takadayuiko | dtantsur, bon appétit! | 10:08 |
| dtantsur | thanks :) | 10:08 |
| *** andreykurilin_ has quit IRC | 10:14 | |
| *** jistr has quit IRC | 10:23 | |
| *** Haomeng|2 has joined #openstack-ironic | 10:27 | |
| *** Haomeng has quit IRC | 10:30 | |
| takadayuiko | dtantsur, And yesterday I said that functest/devstack-test.sh occurd 502 GATE error, right? I found the reason, it was because of using 127.0.0.1 as API's URL. Perhaps my environment is strange. | 10:33 |
| *** jistr has joined #openstack-ironic | 10:36 | |
| *** achanda has joined #openstack-ironic | 10:44 | |
| takadayuiko | dtantsur, openstack command's problem has been resolved :D I set DISCOVERD_URL env val :D | 10:46 |
| *** achanda has quit IRC | 10:48 | |
| *** derekh has joined #openstack-ironic | 10:52 | |
| *** oomichi has quit IRC | 10:58 | |
| *** romcheg has quit IRC | 11:06 | |
| *** ramineni has quit IRC | 11:07 | |
| *** romcheg has joined #openstack-ironic | 11:07 | |
| *** mgoddard has joined #openstack-ironic | 11:07 | |
| *** jistr is now known as jistr|demo | 11:11 | |
| dtantsur | takadayuiko, I see. So we might consider using some other default... mind reporting a bug, so that we don't forget? | 11:13 |
| *** rameshg87 has left #openstack-ironic | 11:18 | |
| *** rameshg87 has joined #openstack-ironic | 11:18 | |
| *** rameshg87 has left #openstack-ironic | 11:18 | |
| *** Haomeng has joined #openstack-ironic | 11:21 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Update doc for test database migrations https://review.openstack.org/169703 | 11:22 |
| *** Haomeng|2 has quit IRC | 11:23 | |
| openstackgerrit | Merged stackforge/ironic-discoverd: First take on integration test for using on devstack https://review.openstack.org/169241 | 11:28 |
| openstackgerrit | Michael Davies proposed openstack/ironic: Disable XML now that we have WSME/Pecan support https://review.openstack.org/169643 | 11:32 |
| *** albertoffb has joined #openstack-ironic | 11:37 | |
| Shrews | LOL. Loving the Clippy on review.o.o | 11:45 |
| *** pradipta has quit IRC | 11:45 | |
| *** romcheg has quit IRC | 11:45 | |
| dtantsur | Shrews ++ | 11:48 |
| dtantsur | and morning :) | 11:48 |
| Shrews | morning | 11:48 |
| TheJulia | good morning | 11:50 |
| TheJulia | oh jeeze, clippy | 11:50 |
| *** dprince has joined #openstack-ironic | 11:51 | |
| Shrews | also, google maps has a pacman version | 11:51 |
| TheJulia | I saw that on ingress last night | 11:52 |
| TheJulia | portals were ghosts, the player was pacman eating all the XM | 11:52 |
| *** romcheg has joined #openstack-ironic | 11:56 | |
| *** trown|outttypeww is now known as trown | 12:01 | |
| *** dencaval has joined #openstack-ironic | 12:06 | |
| *** lucasagomes is now known as lucas-hungry | 12:07 | |
| *** mark-lk has joined #openstack-ironic | 12:14 | |
| *** romcheg has quit IRC | 12:15 | |
| *** athomas has quit IRC | 12:19 | |
| *** romcheg has joined #openstack-ironic | 12:19 | |
| dtantsur | TheJulia, morning | 12:22 |
| *** mgoddard has quit IRC | 12:22 | |
| *** athomas has joined #openstack-ironic | 12:26 | |
| albertoffb | Hello, everyone. I was using devstack with Ironic and, from inside a driver, I wanted to get the nova instance name of an instance I have boot previously. Searching the Ironic code I found a field inside the node, passed through TaskManager, called "name", which I think is the one, but that field is always null. Anyone knows if the "name" field is really the name of the instance? If it is, is the null value a bug? | 12:36 |
| openstackgerrit | Merged stackforge/proliantutils: RIS:Add get_pending_boot_mode https://review.openstack.org/168270 | 12:38 |
| *** saripurigopi has quit IRC | 12:41 | |
| *** mark-lk has quit IRC | 12:45 | |
| Shrews | albertoffb: that should be the name of the ironic node, not the instance it is associated with. you'll have to take node.instance_uuid and map that to an instance name | 12:49 |
| Shrews | but, hrm, probably don't want to do that from inside a driver | 12:50 |
| albertoffb | Shrews, ok I understood Thank you very much :) | 12:56 |
| jroll | \o mornin' everyone | 12:57 |
| albertoffb | good morning | 12:58 |
| *** rloo has joined #openstack-ironic | 13:06 | |
| *** romcheg has quit IRC | 13:07 | |
| *** lucas-hungry is now known as lucasagomes | 13:08 | |
| *** romcheg has joined #openstack-ironic | 13:09 | |
| *** kkoski has joined #openstack-ironic | 13:17 | |
| *** viktors has joined #openstack-ironic | 13:23 | |
| -openstackstatus- NOTICE: gerrit has been restarted to restore event streaming. any change events missed by zuul (between 12:48 and 13:28 utc) will need to be rechecked or have new approval votes set | 13:28 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Clarify message for power action during cleaning https://review.openstack.org/169745 | 13:31 |
| *** alexpilotti has quit IRC | 13:35 | |
| NobodyCam | good morning Ironic | 13:46 |
| *** mjturek1 has joined #openstack-ironic | 13:47 | |
| *** jmanko has quit IRC | 13:48 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-lib: Fix pep8 errors https://review.openstack.org/162162 | 13:48 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-lib: Copy .gitignore from Ironic https://review.openstack.org/169756 | 13:48 |
| TheJulia | ood morning NobodyCam | 13:48 |
| *** jmanko has joined #openstack-ironic | 13:48 | |
| jroll | oh cool ironic-lib is here | 13:48 |
| jroll | morning NobodyCam and TheJulia :) | 13:48 |
| jroll | dtantsur: ^ fixed that up | 13:48 |
| NobodyCam | mornign TheJulia jroll dtantsur | 13:48 |
| dtantsur | jroll, NobodyCam, morning | 13:49 |
| jroll | hey :) | 13:49 |
| NobodyCam | TheJulia: odd in what way. we've had massive winds all night | 13:49 |
| NobodyCam | oh now I see | 13:49 |
| dtantsur | jroll, https://review.openstack.org/#/c/162162/ does not seem fixing pep8 error | 13:49 |
| * NobodyCam goes for first sip of coffee | 13:49 | |
| dtantsur | actually, IIRC pep8 error was already fixed in a parent patch | 13:50 |
| jroll | dtantsur: oh, lol | 13:50 |
| jroll | I didn't actually check the code, I just rebased it | 13:50 |
| TheJulia | NobodyCam: did the wind bury the archaeological dig? | 13:50 |
| NobodyCam | lol; a little :-p | 13:51 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-lib: Copy .gitignore from Ironic https://review.openstack.org/169756 | 13:53 |
| jroll | rebased that to be on master | 13:53 |
| *** jcoufal has quit IRC | 13:55 | |
| *** ChuckC has quit IRC | 13:57 | |
| openstackgerrit | Merged openstack/ironic-lib: Updated from global requirements https://review.openstack.org/169453 | 13:57 |
| *** jmanko has quit IRC | 14:03 | |
| *** jmanko has joined #openstack-ironic | 14:03 | |
| *** jjohnson2 has joined #openstack-ironic | 14:04 | |
| *** zz_jgrimm is now known as jgrimm | 14:05 | |
| *** ijw has joined #openstack-ironic | 14:08 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add display-name option to example apache2 configuration https://review.openstack.org/169765 | 14:10 |
| *** kejiahu has joined #openstack-ironic | 14:12 | |
| *** mtanino has joined #openstack-ironic | 14:18 | |
| lucasagomes | morning all :) | 14:21 |
| lucasagomes | someone knows when the L-1 will open? | 14:22 |
| tiagogomes_ | can I change some kernel args on the pxe config file? I am interested in changing the console | 14:22 |
| *** BadCub_Hiding is now known as BadCub | 14:22 | |
| BadCub | morning folks. | 14:22 |
| NobodyCam | good morning BadCub | 14:23 |
| * BadCub goes to assault coffee pot | 14:23 | |
| lucasagomes | tiagogomes_, yes, you can do it using the pxe_append_params config option | 14:23 |
| lucasagomes | you can add any extra configuration there | 14:23 |
| tiagogomes_ | lucasagomes ta | 14:23 |
| lucasagomes | np | 14:23 |
| NobodyCam | lucasagomes: morning | 14:23 |
| lucasagomes | NobodyCam, BadCub good morning | 14:24 |
| NobodyCam | lucasagomes: like open for specs? | 14:24 |
| lucasagomes | NobodyCam, yeah, and open for code too | 14:24 |
| NobodyCam | I know specs are after rc is cot | 14:24 |
| NobodyCam | cut even | 14:24 |
| *** pradipta has joined #openstack-ironic | 14:24 | |
| lucasagomes | rc1? | 14:24 |
| BadCub | morning NobodyCam and lucasagomes | 14:25 |
| NobodyCam | https://launchpad.net/ironic/+milestone/kilo-rc1 | 14:25 |
| NobodyCam | yep | 14:25 |
| lucasagomes | cool thanks! | 14:26 |
| *** albertoffb has quit IRC | 14:26 | |
| lucasagomes | NobodyCam, the specs that got merged in K | 14:26 |
| lucasagomes | but got bumped, they need to be resubmitted right? | 14:26 |
| lucasagomes | devananda, ^ | 14:27 |
| NobodyCam | i'm not sure I think they are only going to need a minor "yep stilll a thing" check to get bumped in to l | 14:27 |
| NobodyCam | BadCub: ^^^ do you know? | 14:27 |
| *** kalpase has quit IRC | 14:28 | |
| jroll | we'll need to at least move them to the L directory | 14:28 |
| jroll | which requires a review | 14:28 |
| jroll | I think what NobodyCam said is accurate | 14:28 |
| BadCub | NobodyCam: lucasagomes If they have already been approved for K and bumped, I do not see us requiring a resubmission | 14:28 |
| lucasagomes | fair enough | 14:29 |
| lucasagomes | thanks | 14:29 |
| BadCub | anything that was bumped from K should already be on the L Pad too | 14:30 |
| rloo | morning EVERYONE, that includes YOU BadCub, lucasagomes, jroll, NobodyCam, dtantsur, TheJulia, tiagogomes_, Shrews, yuriyz :) | 14:32 |
| dtantsur | rloo, morning :) | 14:32 |
| lucasagomes | rloo, hah morning | 14:32 |
| Shrews | hi rloo | 14:32 |
| yuriyz | morning rloo | 14:32 |
| tiagogomes_ | afternoon rloo :) | 14:32 |
| BadCub | mornin rloo | 14:33 |
| BadCub | :-) | 14:33 |
| NobodyCam | morning rloo Shrews yuriyz and tiagogomes_ | 14:33 |
| NobodyCam | :) | 14:33 |
| jroll | heya rloo :) | 14:33 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Implement basic checks in functest/devstack-test.sh https://review.openstack.org/169776 | 14:34 |
| *** rameshg87 has joined #openstack-ironic | 14:35 | |
| rameshg87 | good morning all | 14:35 |
| jroll | heya rameshg87 :) | 14:36 |
| NobodyCam | :)morning rameshg87 :) | 14:36 |
| rameshg87 | NobodyCam: jroll: o/ | 14:36 |
| rloo | qq folks. Do we want to start adding/checking error strings in our unit tests? I know we've done that for some tests, but is that something we want to do for all tests? Or only tests for patches that fix a string? wrt https://review.openstack.org/#/c/169745/ | 14:36 |
| jroll | rloo: I don't love doing that | 14:37 |
| *** alexpilotti has joined #openstack-ironic | 14:38 | |
| NobodyCam | brb | 14:38 |
| Shrews | rloo: i'm with jroll on this | 14:38 |
| rloo | jroll: so it would be good to have some guidelines. cuz eg if i were to write a test for the string, i'd test for each 'variable' that could be in the string. | 14:38 |
| jroll | rloo: we should either test all log messages or none | 14:38 |
| Shrews | let's not test message content | 14:39 |
| rloo | jroll, Shrews: yes, I agree. that's why I'm asking :-) | 14:39 |
| jroll | :) | 14:39 |
| rloo | on one hand, it is useful, but that seems like an explosion of testing | 14:39 |
| rloo | yuriyz: ^^, what do you think? | 14:39 |
| rloo | and/or are you guys going to make me post to the community? | 14:40 |
| jroll | heh | 14:40 |
| Shrews | i don't think a ML post is necessary | 14:40 |
| jroll | +1 | 14:41 |
| rloo | Shrews, jroll: ok, then we politely say 'no' if someone asks. Cuz I know there have been asks in the past. | 14:41 |
| jroll | I guess so | 14:42 |
| jroll | would be cool to have a source on that, that we could point people to | 14:42 |
| rloo | cuz the problem is that if you see a few tests that check the string, then it seems fair game to ask for other tests to have them. | 14:42 |
| yuriyz | rloo ok agree but in Nova AFAIK patchs not accepted w/o unit test | 14:43 |
| rloo | yuriyz: there is a unit test for that 'no power action in cleaning', there's just no test that 'cleaning' is in the string. | 14:43 |
| rloo | yuriyz: maybe i misunderstood what you wanted | 14:43 |
| yuriyz | rloo no additional test is needed you can add one line to 'no power... | 14:45 |
| jroll | I *really* would be sad if we became like nova | 14:45 |
| jroll | do not want to spend 12 months to land a feature | 14:45 |
| rloo | yuriyz: one line to do what? to check for 'cleaning' in the response? | 14:45 |
| yuriyz | rloo yes | 14:45 |
| *** hj-hp has joined #openstack-ironic | 14:46 | |
| rloo | yuriyz: that's what we (Shrews, jroll) discussed above. | 14:46 |
| rameshg87 | i think that was just a mistake in that code | 14:46 |
| yuriyz | rloo ok agree | 14:46 |
| rloo | yuriyz: my concern is that if we do this for some tests (add a line to check for strings in messages), then people will expect/ask for all tests. | 14:46 |
| rameshg87 | everywhere else we actually give state=node.provision_state | 14:46 |
| rameshg87 | just grepping through ironic right now :) | 14:47 |
| rloo | rameshg87: so it isn't (I don't think) a question of doing it in this particular case. It is a general question of any test where an error is expected. would we want the test to also check the string? | 14:47 |
| rameshg87 | i am +1 without unit tests for that. | 14:48 |
| rloo | rameshg87: a unit test for what? | 14:48 |
| rameshg87 | unit tests don't need to check for error string (unless we have something in very particular to assert) | 14:48 |
| rloo | rameshg87: oh, got it. thx. | 14:49 |
| rloo | rameshg87: then you are in agreement ;) | 14:49 |
| rameshg87 | :) | 14:49 |
| * jroll +A's the patch and moves on with his life :P | 14:50 | |
| rameshg87 | :D | 14:50 |
| *** Marga_ has joined #openstack-ironic | 14:50 | |
| * rameshg87 runs for food | 14:50 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Clarify message for power action during cleaning https://review.openstack.org/169745 | 14:50 |
| *** rameshg87 is now known as rameshg87-away | 14:51 | |
| rloo | jroll: gad, I didn't notice you had approved. I updated the commit for 169745. | 14:51 |
| jroll | thanks for making me re-approve, rloo :) | 14:51 |
| rloo | jroll: anything to get your stats up ;) | 14:52 |
| jroll | no worries :) | 14:52 |
| jroll | lol | 14:52 |
| dtantsur | lol | 14:52 |
| jlvillal | Funny. They were talking about adding 'Clippy' to Gerrit for April Fools at the PDX Hackathon :) | 14:52 |
| yuriyz | rloo finally: look at api test_ports, assertIn with 'error_message' in many places | 14:56 |
| rloo | hey, what happens after RC-1. Is master open for L, and any subsequent changes for kilo have to be done in some non-master branch? | 14:57 |
| jroll | rloo: I think that's correct | 14:57 |
| BadCub | I believe that to be the correct assumption as well | 14:58 |
| jroll | rc-1 == stable/kilo, iirc | 14:58 |
| rloo | yuriyz: yes, that's the problem (or not). Hard to know when/if one should be doing that. We don't do it everywhere. It seems like we should do it everywhere, or nowhere, or only in places where it should be done (but where should it be done)? | 14:58 |
| TheJulia | Clippy seems to make gerrit much slower though.... :( | 14:58 |
| rloo | jroll: ok, so it is important to get things in for rc1, cuz otherwise more hoops to go through. | 14:59 |
| jroll | rloo: thinking about it more, I wonder if api responses should be checked in test. idk. | 14:59 |
| rloo | jroll: could be low-h-f bug. i'm good either way. yuriyz, what do you think? ^^ | 15:01 |
| jroll | yeah | 15:03 |
| tiagogomes_ | Ironic is drunk, it is on 'wait call-back' state but it doesn't power on the baremetal server. If I manually power it up, Ironic powers it down | 15:03 |
| NobodyCam | tiagogomes_: any errors on node-show? | 15:04 |
| NobodyCam | or in the conductor? | 15:04 |
| yuriyz | rloo jroll I think maybe require or do not require unit test should be documented for contributors in dependence of patch type | 15:04 |
| Shrews | rloo: lol on 169101 :) | 15:04 |
| tiagogomes_ | None, and the 'power state' field indicates that it is power on | 15:05 |
| tiagogomes_ | on the conductor: Apr 01 12:04:53 onenode ironic-conductor[12599]: 2015-04-01 12:04:53.575 12599 WARNING ironic.conductor.manager [-] During sync_power_state, node 0037 | 15:05 |
| NobodyCam | tiagogomes_: is there a instance id on the node? | 15:06 |
| *** Marga_ has quit IRC | 15:08 | |
| tiagogomes_ | NobodyCam, no | 15:09 |
| NobodyCam | tiagogomes_: are you "nova boot(ing)" the nnode? | 15:10 |
| tiagogomes_ | NobodyCam, no, running set-provision-state manually | 15:11 |
| NobodyCam | is this in a openstack enviroment? ie is nova setup in you env? | 15:12 |
| rloo | yuriyz: I agree that we should document, but I think we need to agree on what :-) | 15:14 |
| NobodyCam | tiagogomes_: I have seen this type of thing before when others have attempted to manually control nodes that nova is also controling | 15:14 |
| tiagogomes_ | right, deleting the node and creating again solved the problem apparently | 15:15 |
| yuriyz | rloo first version may be based on current practice so no 'in' message test needed | 15:16 |
| tiagogomes_ | Hmm the options that I added to 'pxe_append_params' are not in the pxe config file | 15:17 |
| NobodyCam | tiagogomes_: did you restart ironic after adding them? | 15:18 |
| tiagogomes_ | NobodyCam, no, do I need it? | 15:18 |
| tiagogomes_ | Usually when I change a node configuration I don't restart the service | 15:18 |
| *** mgoddard has joined #openstack-ironic | 15:18 | |
| jroll | any config file changes need a restart | 15:19 |
| NobodyCam | tiagogomes_: thats a conf options and they are only read upon startup | 15:19 |
| *** yog__ has quit IRC | 15:19 | |
| rloo | yuriyz: yeah. | 15:19 |
| tiagogomes_ | hold on, pxe_append_params should be changed on ironic.conf? I was adding them to the node driver_info | 15:19 |
| NobodyCam | ahh that would do it | 15:20 |
| tiagogomes_ | adding them to the driver_info didn't work | 15:20 |
| NobodyCam | tiagogomes_: https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L1392 | 15:20 |
| tiagogomes_ | I'll add them to ironic.conf | 15:21 |
| tiagogomes_ | I think those options should be per node instead of being global no? I am imagining different nodes requiring different consoles | 15:22 |
| NobodyCam | tiagogomes_: sounds like a good spec for the L cycle :-p | 15:23 |
| tiagogomes_ | hehe | 15:23 |
| rameshg87-away | 1 question folks | 15:27 |
| *** jmanko has quit IRC | 15:27 | |
| rameshg87-away | can unit tests be run in different localizations ? | 15:27 |
| rameshg87-away | https://review.openstack.org/#/c/167882/11/ironic/tests/drivers/ilo/test_deploy.py L 1028 | 15:27 |
| rameshg87-away | just asking if it will fail if i (can) run the unit tests in non-english settings | 15:27 |
| rameshg87-away | ?? | 15:27 |
| tiagogomes_ | is it possible somehow to run the tftp server inside the dhcp namespace | 15:28 |
| *** Marga_ has joined #openstack-ironic | 15:28 | |
| NobodyCam | tiagogomes_: ?? yes! not sure I get the question | 15:30 |
| *** Marga_ has quit IRC | 15:31 | |
| openstackgerrit | Merged stackforge/pyghmi: Try setting optional byte in set user acess https://review.openstack.org/169730 | 15:31 |
| jroll | rameshg87-away: I agree, I think it would | 15:31 |
| *** rameshg87-away is now known as rameshg87 | 15:32 | |
| rameshg87 | jroll: okay, just adding a comment | 15:32 |
| jroll | rameshg87: I'm not 100% sure, but I don't see why it wouldn't be translated | 15:32 |
| rameshg87 | jroll: yeah, even i am not sure .. | 15:33 |
| tiagogomes_ | my baremetal machine is in a private network and DHCPs from Neutron, which is running in a namespace. My TFTP server is running on a public network. So to make the baremetal server being able to contact the TFTP server, I need to create a router between the private network and the public network. I was wondering if that could be avoid | 15:33 |
| rameshg87 | jroll: i think we needn't assert the string there (and to be on the safer side) | 15:34 |
| jroll | +1 | 15:34 |
| NobodyCam | tiagogomes_: you can run tftp service on the ironic conductor | 15:36 |
| *** Marga_ has joined #openstack-ironic | 15:40 | |
| *** ChuckC has joined #openstack-ironic | 15:40 | |
| tiagogomes_ | NobodyCam I am already running the tftp on the conductor. Let's say that I create a 10.0.0.1/24 network with neutron for serving an IP to the baremetal. If in the DHCP offer the TFTP server has an IP of e.g. 192.1.1.10, the baremetal will not be able to connect with the tftp server | 15:43 |
| tiagogomes_ | I think firl had the same problem? | 15:44 |
| firl | what’s this? | 15:44 |
| firl | oh yeah | 15:44 |
| firl | I put it on the same flat network that glance / cinder was running on | 15:44 |
| NobodyCam | morning firl: :) | 15:46 |
| firl | I also had to make sure the dhcp options on the port of the qdhcp were proper | 15:46 |
| firl | morning NobodyCam :) | 15:46 |
| tiagogomes_ | mmm, you didn't need to create a router to route traffic between the networks? | 15:46 |
| tiagogomes_ | I needed to that so that `ip netns qdhcp-xxx exec ping tftp-server` worked | 15:47 |
| *** romcheg has quit IRC | 15:48 | |
| openstackgerrit | Merged openstack/ironic: Add PXE-AMT driver's support of IPA ramdisk https://review.openstack.org/169664 | 15:49 |
| openstackgerrit | Merged openstack/ironic: Improve backwards compat on API behaviour https://review.openstack.org/169101 | 15:49 |
| *** mmorais has quit IRC | 15:51 | |
| *** Marga_ has quit IRC | 15:51 | |
| firl | I did an OVS bridge between the flat network and the internal traffic | 15:52 |
| *** Marga_ has joined #openstack-ironic | 15:52 | |
| firl | tiagogomes_ I did external network to a custom bridge, then in my neutron config created the other bridge definition | 15:52 |
| *** mmorais has joined #openstack-ironic | 15:53 | |
| *** hj-hp has quit IRC | 15:55 | |
| *** hj-hp has joined #openstack-ironic | 15:55 | |
| *** Marga_ has quit IRC | 15:55 | |
| *** Marga_ has joined #openstack-ironic | 15:55 | |
| rloo | hey jroll, were you going to review https://review.openstack.org/#/c/169303/ (or should I +A) | 15:56 |
| jroll | rloo: go ahead, I was just making a suggestion since that *really* needs docs | 15:57 |
| rloo | jroll: definitely. we love docs... | 15:57 |
| jroll | :) | 15:57 |
| NobodyCam | :) | 15:58 |
| *** ukalifon1 has quit IRC | 16:01 | |
| rloo | hey, the IronicReviewDay etherpad sez 'bug squash on Thursday April 2'. I'm guessing that isn't an April's fool thing ;) All day or certain time? Anyone know? | 16:01 |
| NobodyCam | rloo: I'll bet BadCub knows | 16:02 |
| BadCub | Yes, bug squash is 02April. | 16:02 |
| rloo | BadCub: what time? | 16:03 |
| BadCub | Was thinking 0800hrs PST as a start time | 16:03 |
| BadCub | I am not planning on adding any more bugs to the RC1 list at this stage. I think our plate is rather full. But that is always open to "we really need this" things | 16:06 |
| rloo | BadCub: ok. wrt the etherpad, I only need to look at patches with 'NEEDS REVIEW' next to them? | 16:12 |
| BadCub | rloo: yeah, if we could get some of those off the plate that would be awesome :-) | 16:12 |
| rloo | BadCub: ok. | 16:13 |
| dtantsur | see you tomorrow | 16:13 |
| *** dtantsur is now known as dtantsur|afk | 16:13 | |
| BadCub | rloo: thank you :-) | 16:14 |
| rloo | BadCub: would you do me a favour and post something to the mail list about the bug squash tomorrow? The more the merrier. | 16:14 |
| BadCub | absolutely! :-) | 16:15 |
| *** Sukhdev has joined #openstack-ironic | 16:16 | |
| rloo | thx BadCub. | 16:16 |
| *** alexpilotti has quit IRC | 16:16 | |
| NobodyCam | where is the devstack tftp dir | 16:17 |
| NobodyCam | nm checking the conf file revels the location | 16:18 |
| BadCub | rloo: email sent :-) | 16:18 |
| rameshg87 | BadCub: bug squash is for triage ? | 16:20 |
| * rloo thinks BadCub isn't so bad after all | 16:20 | |
| * rameshg87 has never participated in one | 16:20 | |
| *** andreykurilin_ has joined #openstack-ironic | 16:21 | |
| rameshg87 | BadCub: and 0800 PST == 1600 GMT, right ? please use GMT :) | 16:21 |
| BadCub | awe.. Thanks rloo :-) | 16:22 |
| BadCub | rameshg87: updated on the pad | 16:22 |
| rameshg87 | BadCub: thanks :) | 16:23 |
| BadCub | rameshg87: yes, we tackle the bug list on the pad and see what we awesomeness we can accomplish :-) | 16:23 |
| * BadCub frowns at his typos | 16:24 | |
| BadCub | I clearly need more coffee | 16:24 |
| *** mgoddard has quit IRC | 16:25 | |
| rameshg87 | BadCub: oaky | 16:25 |
| rameshg87 | *okay | 16:25 |
| * rameshg87 too frowns :) | 16:25 | |
| NobodyCam | f.close() | 16:25 |
| NobodyCam | doh ww | 16:25 |
| BadCub | lol | 16:26 |
| BadCub | typos decrease with increased coffee intake :-) | 16:26 |
| openstackgerrit | Merged openstack/ironic: Clarify message for power action during cleaning https://review.openstack.org/169745 | 16:27 |
| *** absubram has joined #openstack-ironic | 16:28 | |
| JayF | JoshNang: https://review.openstack.org/#/c/165496/ did you notice this passed tempest with the absurdly long timeout? | 16:29 |
| openstackgerrit | Merged openstack/ironic: Remove inspect_ports from ilo inspection https://review.openstack.org/169303 | 16:35 |
| *** mdbooth has quit IRC | 16:35 | |
| *** andreykurilin_ has quit IRC | 16:39 | |
| jroll | JayF: I think we should play with that timeout and then get devstack patch merged | 16:40 |
| JayF | jroll: I somewhat agree | 16:41 |
| *** mdbooth has joined #openstack-ironic | 16:42 | |
| JayF | it took, it seems, about 3 minutes to complete cleaning | 16:45 |
| JayF | http://logs.openstack.org/96/165496/5/check/check-tempest-dsvm-ironic-agent_ssh/06357b3/logs/screen-ir-cond.txt.gz#_2015-03-30_18_16_21_058 18:16:21 - 18:19:56 | 16:45 |
| JayF | and all it did was boot the node and do nothing | 16:45 |
| JayF | man that's insanely slow | 16:45 |
| *** jistr|demo has quit IRC | 16:47 | |
| *** romcheg has joined #openstack-ironic | 16:48 | |
| *** derekh has quit IRC | 16:49 | |
| NobodyCam | why is the M$ paper clip on reviews? | 16:50 |
| JayF | NobodyCam: ask your local calendar | 16:51 |
| NobodyCam | lol | 16:51 |
| * BadCub noticed that too... Happy April Fools Day! | 16:51 | |
| NobodyCam | nice | 16:51 |
| NobodyCam | anyone found the april 1st rfc yet? | 16:52 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Enable cleaning by default https://review.openstack.org/165496 | 16:52 |
| JayF | ^ updated the commit message, it needed a recheck anyway | 16:53 |
| *** jlvillal has quit IRC | 16:53 | |
| *** hj-hp has quit IRC | 16:53 | |
| JayF | I dropped the timeout in https://review.openstack.org/#/c/168175 to 5m from 10m. This is a number that should be reasonable enough for us to land. | 16:54 |
| *** Marga_ has quit IRC | 16:54 | |
| *** Marga_ has joined #openstack-ironic | 16:55 | |
| JayF | I'd love to get core reviews on the first real documentation we've had to IPA; https://review.openstack.org/#/c/168522/ | 16:58 |
| *** jlvillal has joined #openstack-ironic | 17:00 | |
| *** alexpilotti has joined #openstack-ironic | 17:01 | |
| * BadCub goes to nuke food | 17:01 | |
| JoshNang | JayF: thanks, yeah i was thinking 5 should be sufficient | 17:01 |
| JayF | JoshNang: yeah, I just want all this stuff to land! | 17:01 |
| *** harlowja_away is now known as harlowja | 17:02 | |
| JayF | JoshNang: jroll: Straw poll: Are you guys on board with adding a docs job to IPA which would fail if someone added a docstring that added a warning to the docs build | 17:02 |
| JayF | I just squashed all such warnings and so now would *really like* to make sure we don't add more | 17:02 |
| jroll | JayF: idk, is that something people normally do? | 17:02 |
| JayF | I don't know what people normally do. I do know, however, that all the issues I found did cause issues with rendered docs so I'd like to not have code merge that mangles generated docs | 17:03 |
| *** mgoddard has joined #openstack-ironic | 17:03 | |
| JayF | since that's the bulk of docs for IPA today | 17:03 |
| jroll | gopher it | 17:04 |
| JoshNang | i think its a good idea. i'm generally a fan of "if we care about it, there should be an automated test for it" | 17:04 |
| jroll | ++ | 17:04 |
| jroll | otherwise JayF will be scrutinizing every docstring | 17:04 |
| JayF | Good. I'll figure out how to do it and PR it, then I'm going to add a job to run the check on every IPA commit, and shortly thereafter do one that will publish the docs | 17:04 |
| JoshNang | heh | 17:04 |
| *** jlvillal has quit IRC | 17:05 | |
| *** achanda has joined #openstack-ironic | 17:08 | |
| NobodyCam | JayF: reviewed | 17:08 |
| JayF | NobodyCam: cool, gimme 5m | 17:10 |
| JayF | NobodyCam: do you have a link to the DIB element? | 17:10 |
| *** achanda has quit IRC | 17:10 | |
| NobodyCam | one sec | 17:10 |
| JayF | cool. | 17:10 |
| lucasagomes | I will call it a day, have a good night everyone! | 17:10 |
| *** lucasagomes is now known as lucas-dinner | 17:10 | |
| BadCub | g'night lucas-dinner | 17:10 |
| jroll | night lucas :) | 17:10 |
| NobodyCam | night lucas-dinner | 17:11 |
| NobodyCam | JayF: https://github.com/openstack/diskimage-builder/tree/master/elements/ironic-agent | 17:11 |
| *** jlvillal has joined #openstack-ironic | 17:11 | |
| JayF | sweet, I'll fix those things right now | 17:12 |
| NobodyCam | :) | 17:12 |
| NobodyCam | TY | 17:12 |
| *** achanda has joined #openstack-ironic | 17:14 | |
| *** Sukhdev has quit IRC | 17:14 | |
| *** penick has joined #openstack-ironic | 17:14 | |
| *** Marga_ has quit IRC | 17:18 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: pxe deploy fails for whole disk images in UEFI https://review.openstack.org/167882 | 17:22 |
| *** pelix has quit IRC | 17:22 | |
| *** pradipta has quit IRC | 17:24 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Allow IPA to request reboot after clean steps https://review.openstack.org/169504 | 17:25 |
| *** lucas-dinner has quit IRC | 17:28 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: boot_mode is overwritten in node properties https://review.openstack.org/167952 | 17:28 |
| *** davideag_ has joined #openstack-ironic | 17:30 | |
| *** davideagnello has quit IRC | 17:33 | |
| *** trown is now known as trown|lunch | 17:35 | |
| rameshg87 | BadCub: fixing https://launchpad.net/bugs/1422723 would required the dib patch to go in first - https://review.openstack.org/#/c/169354/ | 17:38 |
| openstack | Launchpad bug 1422723 in Ironic "Ironic sets instance as active even if bootloader installation fails when boot_option is 'local'" [Critical,In progress] - Assigned to Ramakrishnan G (rameshg87) | 17:38 |
| rameshg87 | BadCub: any help in getting tripleo reviews might be good. i just pinged in tripleo irc as well. | 17:38 |
| *** tiagogomes_ has quit IRC | 17:38 | |
| NobodyCam | rameshg87: be sure to set that as a dependicy on any patches that would require it | 17:39 |
| rameshg87 | NobodyCam: yeah, i just did it | 17:40 |
| rameshg87 | NobodyCam: but ironic and dib patches are both dependent on each other | 17:40 |
| rameshg87 | NobodyCam: so setting Depends-On was confusing :) | 17:40 |
| NobodyCam | oh :( wounder how jenkins will handle that | 17:41 |
| NobodyCam | ya | 17:41 |
| rameshg87 | NobodyCam: anyway i set ironic patch as "Depends-on" on dib patch | 17:41 |
| rameshg87 | NobodyCam: jenkins won't care for it because there are no ci tests for localboot | 17:41 |
| rameshg87 | :D | 17:41 |
| *** athomas has quit IRC | 17:41 | |
| jroll | :| | 17:43 |
| *** jjohnson2_ has joined #openstack-ironic | 17:46 | |
| *** jjohnson2__ has joined #openstack-ironic | 17:47 | |
| *** spandhe has joined #openstack-ironic | 17:48 | |
| *** ndipanov has quit IRC | 17:49 | |
| *** pas-ha has quit IRC | 17:49 | |
| *** jjohnson2 has quit IRC | 17:49 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Allow IPA to request reboot after clean steps https://review.openstack.org/169504 | 17:50 |
| *** jjohnson2_ has quit IRC | 17:51 | |
| NobodyCam | rameshg87: the DIB failed again jenkins again :( | 17:52 |
| NobodyCam | humm that was all wrong | 17:52 |
| NobodyCam | rameshg87: The dib patch failed Jenkins again :0 | 17:52 |
| rameshg87 | NobodyCam: oh | 17:52 |
| * rameshg87 checks | 17:52 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Docs on HardwareManager, building, and workflow https://review.openstack.org/168522 | 17:53 |
| rameshg87 | NobodyCam: its' still running | 17:53 |
| JayF | NobodyCam: ^ fixed your issues | 17:54 |
| rameshg87 | NobodyCam: i see all jobs status as green in http://status.openstack.org/zuul/ for 169354 | 17:54 |
| * rameshg87 might be missing something | 17:54 | |
| NobodyCam | nope. I just missed it in the queue. I thought it finished the recheck ... sry for the false alarm | 17:55 |
| rameshg87 | NobodyCam: :) | 17:56 |
| *** Marga_ has joined #openstack-ironic | 17:56 | |
| *** mgoddard has quit IRC | 17:56 | |
| * NobodyCam notes he has a apt with the eye doctor already setup for later today :-p | 17:57 | |
| *** ndipanov has joined #openstack-ironic | 17:58 | |
| rameshg87 | NobodyCam: it went through :) | 18:02 |
| *** Nisha has joined #openstack-ironic | 18:03 | |
| NobodyCam | w00t | 18:06 |
| *** athomas has joined #openstack-ironic | 18:06 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Mock time.sleep() for two unittests https://review.openstack.org/169860 | 18:07 |
| jroll | <3 | 18:07 |
| *** romcheg has quit IRC | 18:08 | |
| jlvillal | Anybody bored and want to look at: https://review.openstack.org/169036 :) | 18:08 |
| jlvillal | Not critical though | 18:08 |
| *** romcheg has joined #openstack-ironic | 18:09 | |
| jroll | rameshg87: so it seems like your earlier concern was addressed :) https://review.openstack.org/#/c/164226/ | 18:09 |
| rameshg87 | jroll: yeah, rloo pointed out :) | 18:10 |
| jroll | jlvillal: reviewed, pls fix :) | 18:10 |
| jlvillal | jroll thanks! | 18:11 |
| jlvillal | jroll: Wait. Are we sure we want it set for venv? | 18:12 |
| jlvillal | jroll: Isn't that for people to test stuff? Wouldn't they want it in their language. I'm confused. | 18:12 |
| jlvillal | or unsure | 18:12 |
| jroll | oh hm | 18:12 |
| jroll | I guess I misread | 18:12 |
| jroll | yeah, maybe not | 18:12 |
| jroll | jlvillal: +A | 18:13 |
| jlvillal | jroll: Thanks! :) | 18:13 |
| NobodyCam | morning jlvillal | 18:14 |
| jlvillal | NobodyCam: Morning. I finally got my Znc on Znc working :) Used dhellmann ansible script with some modifications | 18:15 |
| NobodyCam | nice :) | 18:15 |
| rameshg87 | jroll: so afterall my concern is not addressed then | 18:15 |
| rameshg87 | jroll: i could go into venv and then do "testr run" | 18:15 |
| rameshg87 | (which i often do) | 18:15 |
| rameshg87 | in some other language | 18:15 |
| jroll | rameshg87: that's your fault, docs say use tox ;) | 18:16 |
| rameshg87 | heh :) | 18:16 |
| jroll | rameshg87: you could go into the py27 env and do testr run | 18:16 |
| jroll | or just tox -e py27 | 18:16 |
| *** ndipanov has quit IRC | 18:16 | |
| rameshg87 | jroll: how do i go into py27 env ? | 18:16 |
| rameshg87 | tox -epy27env ? | 18:17 |
| jroll | rameshg87: same way as the venv, source .tox/py27/bin/activate | 18:17 |
| * rameshg87 checks if it works | 18:17 | |
| rameshg87 | ah okay | 18:17 |
| jroll | for venv it would be .tox/venv/bin/activate | 18:17 |
| rameshg87 | yeah, got it | 18:17 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Check status of bootloader installation for DIB ramdisk https://review.openstack.org/169419 | 18:20 |
| rameshg87 | jlvillal: ^^^ (with unit tests added and ready for review back again), thanks for the quick review earlier | 18:20 |
| jlvillal | jroll: I think it was rloo's concern and not rameshg87 in https://review.openstack.org/169036 | 18:20 |
| JayF | JoshNang: passed with a 5m timeout; I'm shilling for devstack reviews now | 18:20 |
| jlvillal | rameshg87: Looking now | 18:21 |
| JoshNang | JayF: \o/ | 18:21 |
| rameshg87 | jlvillal: jroll was mentioning another patch | 18:21 |
| jroll | jlvillal: right, I was talking with rameshg87 earlier about if tests can pass in other languages | 18:21 |
| jlvillal | jroll: Oh I missed that part. While trying to get ZNC setup :) | 18:21 |
| jroll | heh | 18:21 |
| *** ndipanov has joined #openstack-ironic | 18:21 | |
| jroll | no worries | 18:21 |
| rameshg87 | jlvillal: https://review.openstack.org/#/c/167882/11/ironic/tests/drivers/ilo/test_deploy.py | 18:21 |
| rameshg87 | jlvillal: discussion started with ^^^^ | 18:22 |
| * jroll is allergic to irc on mobile devices | 18:22 | |
| NobodyCam | jroll: +++++++++++ | 18:22 |
| jlvillal | rameshg87: thanks | 18:23 |
| JayF | BadCub: hey, need some help. Can you find folks to review https://review.openstack.org/#/c/168175 which is required so https://review.openstack.org/#/c/165496/ can land? | 18:23 |
| JayF | BadCub: first patch is devstack, second is Ironic (enable cleaning by default) | 18:23 |
| * BadCub looks | 18:23 | |
| * rameshg87 missed docstrings | 18:23 | |
| JayF | Who is the best person to point PBR questions at? | 18:28 |
| NobodyCam | JayF: mordred: | 18:28 |
| jroll | lifeless knows things | 18:28 |
| jroll | dstufft also knows packaging related things about it | 18:28 |
| JayF | morgabra: lifeless: I'm using pbr's build in support for sphinx builds (python setup.py build_sphinx). Is there any way afayk to force sphinx to exit and error on any WARN or ERROR building docs? | 18:29 |
| jroll | brad won't help you much, unfortunately :( | 18:29 |
| JayF | ha | 18:30 |
| JayF | mordred: ^ | 18:30 |
| JayF | I know sphinx has an option to exit 1 if warnings, but it doesn't appear to be exposed by pbr on a cursory look through the code | 18:30 |
| JayF | i could always be wrong | 18:31 |
| *** ndipanov has quit IRC | 18:31 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Check status of bootloader installation for DIB ramdisk https://review.openstack.org/169419 | 18:34 |
| rameshg87 | jlvillal: sorry if you started looking, but another patch ^^^ (just added the docs) | 18:34 |
| mordred | JayF: yes | 18:35 |
| mordred | there is an option | 18:35 |
| jlvillal | rameshg87: Okay :) | 18:35 |
| NobodyCam | morning mordred :) | 18:35 |
| mordred | [pbr] | 18:35 |
| mordred | warnerrors = True | 18:35 |
| mordred | it may not be the best named option | 18:35 |
| JayF | In what? my tox.ini? | 18:36 |
| JayF | Or the conf.py? | 18:36 |
| rameshg87 | i will call it a day ... | 18:37 |
| rameshg87 | good night folks, see you tomorrow | 18:37 |
| NobodyCam | have a good night rameshg87 :0 | 18:37 |
| rameshg87 | have a great day ... | 18:37 |
| *** rameshg87 has quit IRC | 18:37 | |
| *** trown|lunch is now known as trown | 18:37 | |
| *** Sukhdev has joined #openstack-ironic | 18:38 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Check status of bootloader installation for DIB ramdisk https://review.openstack.org/169419 | 18:39 |
| jroll | TIL http://zuul.openstack.org/ | 18:39 |
| NobodyCam | jroll: I use that every day | 18:39 |
| jroll | NobodyCam: different than http://status.openstack.org/zuul/ | 18:39 |
| * BadCub sent plea for reviews to DevStack folks | 18:40 | |
| NobodyCam | oh so it is. TIL too | 18:40 |
| NobodyCam | :-p | 18:40 |
| jroll | :) | 18:41 |
| NobodyCam | brb | 18:41 |
| *** jmank has joined #openstack-ironic | 18:50 | |
| *** achanda has quit IRC | 18:52 | |
| *** ijw has quit IRC | 19:06 | |
| *** lifeless has quit IRC | 19:11 | |
| *** achanda has joined #openstack-ironic | 19:23 | |
| *** Marga_ has quit IRC | 19:24 | |
| *** hj-hp has joined #openstack-ironic | 19:30 | |
| BadCub | heya folks. Just updated our drivers wiki... Can I put out an ask for folks to go update: http://stackalytics.com/report/driverlog?project_id=openstack%2Fironic to make sure we are aligned? | 19:31 |
| jroll | BadCub: my stuff is there :D | 19:34 |
| BadCub | jroll: awesome! :-) | 19:34 |
| NobodyCam | JayF: 168522 +2'd | 19:34 |
| *** lifeless has joined #openstack-ironic | 19:40 | |
| NobodyCam | JayF: looking at 168175: could we need to bump up other timeout values too: http://logs.openstack.org/75/168175/5/check/check-tempest-dsvm-ironic-pxe_ssh/972d51b/logs/screen-n-cpu.txt.gz#_2015-04-01_17_54_58_876 | 19:40 |
| JayF | NobodyCam: notice that's pxe_ssh | 19:44 |
| JayF | NobodyCam: and that ran without cleaning enabled | 19:44 |
| BadCub | we have three +2s on https://review.openstack.org/#/c/166201/ someone want to throw a +A at it? | 19:45 |
| * NobodyCam looks | 19:45 | |
| rloo | BadCub: that's cuz I'd like lucas to address the nits. | 19:45 |
| *** dencaval has quit IRC | 19:45 | |
| rloo | BadCub: but I can +A and he can fix later | 19:46 |
| NobodyCam | ya was juust looking at the comments | 19:46 |
| rloo | NobodyCam: hmm, just looked them over. I think it is worth another revision, what do you think? | 19:47 |
| BadCub | rloo: read the nits too. Maybe we should have him address first? | 19:47 |
| rloo | BadCub: yeah, there are enough of them. I'll -1. | 19:47 |
| BadCub | rloo: coolness! that works perfect :-) | 19:48 |
| JayF | I mean, lets land something? These nits are minor and someone could push up a new patchset so we can land that | 19:48 |
| JayF | do you all want me to do that? | 19:48 |
| NobodyCam | I'm ok with landing this and doing a follow up | 19:48 |
| BadCub | JayF: I would like to see some of these pending things land too. | 19:49 |
| rloo | it just seems easiest to fix it in this patch, vs creating a new patch, etc. | 19:49 |
| NobodyCam | none of the comments will change the direction or meaning of the patch | 19:49 |
| JayF | rloo: I'm saying one of us should push up a patchset fixing them, then land | 19:49 |
| rloo | oh. could do that too. | 19:49 |
| JayF | rloo: I'm offering to do it so you and NobodyCam could still land the result | 19:49 |
| rloo | JayF: yes please. | 19:49 |
| NobodyCam | JayF: doit | 19:49 |
| NobodyCam | :) | 19:49 |
| BadCub | JayF: I say go for it :-) | 19:49 |
| JayF | I'm "eating lunch" right now | 19:50 |
| JayF | so give me a few to get back combobulated and I'll push it | 19:50 |
| rloo | JayF: ha ha. take your time. | 19:50 |
| NobodyCam | lol mmm food.. | 19:50 |
| BadCub | food first man! | 19:50 |
| * BadCub is thinking food is a good idea too | 19:51 | |
| *** athomas has quit IRC | 19:51 | |
| *** davideag_ has quit IRC | 19:57 | |
| *** Marga_ has joined #openstack-ironic | 19:57 | |
| *** davideagnello has joined #openstack-ironic | 19:58 | |
| *** davideagnello has joined #openstack-ironic | 19:58 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Add config drive documentation https://review.openstack.org/166201 | 20:00 |
| JayF | rloo: NobodyCam ^ | 20:00 |
| *** edcragg has quit IRC | 20:01 | |
| TheJulia | devananda: you mentioned interest in bifrost being able to fire-off tempest, although that seems like it would ultimately require teaching tempest about noauth mode :( | 20:01 |
| jroll | TheJulia: and teaching tempest how to deploy without nova | 20:02 |
| TheJulia | that too :( | 20:02 |
| NobodyCam | JayF: +2'd ... thank you | 20:03 |
| *** kkoski has quit IRC | 20:03 | |
| *** kkoski has joined #openstack-ironic | 20:04 | |
| BadCub | JayF: Ty :-) | 20:07 |
| JayF | rloo: NobodyCam ^ are one of you going to +A that? You both voted +2 on it ... | 20:07 |
| rloo | JayF: done. NobodyCam was faster than me! | 20:08 |
| JayF | woo | 20:08 |
| NobodyCam | lol | 20:08 |
| rloo | JayF: Jenkins has the last word ;) | 20:08 |
| NobodyCam | yep | 20:08 |
| JayF | I just hate when things don't merge for a day because overnight waiting to fix typos | 20:09 |
| JayF | lol | 20:09 |
| BadCub | lol | 20:10 |
| *** hj-hp has quit IRC | 20:13 | |
| jroll | JayF: some comments on https://review.openstack.org/#/c/168522/ | 20:16 |
| jroll | happy to +A if you want to post a follow up | 20:16 |
| JayF | just let me fix this one, I think it'll be easier that way | 20:17 |
| jroll | k | 20:17 |
| JayF | I'll have it done shortly, then I'm probably done with upstream for the day | 20:17 |
| jroll | cool | 20:18 |
| jroll | thanks! | 20:18 |
| lifeless | JayF: I'm not sure, let me have a quick look | 20:22 |
| jroll | lifeless: he got an answer from monty | 20:22 |
| jroll | but thanks :) | 20:22 |
| lifeless | if options.get_boolean_option(option_dict, | 20:23 |
| lifeless | 'warnerrors', 'WARNERRORS'): | 20:23 |
| lifeless | self._sphinx_run() | 20:23 |
| lifeless | else: | 20:23 |
| lifeless | setup_command.BuildDoc.run(self) | 20:23 |
| lifeless | oh, sorry :) | 20:23 |
| jroll | heh, great work :D | 20:23 |
| JayF | that's not the same answer he gave me though | 20:23 |
| JayF | which is interesting | 20:23 |
| JayF | oh yeah, thati s it, nevermind | 20:23 |
| * JayF gets it now | 20:23 | |
| jlvillal | rloo: I can make a patch to undo the LANGUAGE=en_US in the venv if you want. I'm not totally sure which is best. I'm leaning towards not having it in venv so that user can see things in their language. Thoughts? | 20:32 |
| rloo | jlvillal: my thought is that it should be the same in ironic and client. and yeah, some people might want to see in non english. | 20:33 |
| rloo | jlvillal: could ask someone that uses nonEnglish I guess. | 20:33 |
| jroll | jlvillal: I thought we approved it without it in testenv:venv | 20:34 |
| jlvillal | jroll: We did in the client. | 20:34 |
| jroll | oh | 20:34 |
| jlvillal | jroll: Talking about making a change to remove it in ironic | 20:34 |
| rloo | jroll: you know me. consistency :-) | 20:34 |
| jroll | yeah, we should undo that IMO | 20:34 |
| jroll | :) | 20:34 |
| jlvillal | Okay. I will make a patch. | 20:34 |
| jlvillal | rloo: jroll: Thanks | 20:34 |
| jroll | thank YOU | 20:34 |
| rloo | ++ | 20:34 |
| *** romcheg has quit IRC | 20:35 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Remove setting language to en_US for 'venv' https://review.openstack.org/169914 | 20:37 |
| *** Sukhdev has quit IRC | 20:37 | |
| jlvillal | rloo: jroll ^^^^ | 20:38 |
| jroll | bam | 20:38 |
| jroll | thanks | 20:38 |
| *** Nisha_away has joined #openstack-ironic | 20:39 | |
| *** Nisha has quit IRC | 20:39 | |
| jlvillal | :) | 20:41 |
| rloo | NobodyCam: were you going to push up another revision for https://review.openstack.org/#/c/164369/? | 20:44 |
| *** mrda-away is now known as mrda | 20:44 | |
| mrda | Morning Ironic | 20:45 |
| NobodyCam | rloo: I got side tracked and have yet to get to it | 20:45 |
| NobodyCam | mornign mrda | 20:45 |
| rloo | NobodyCam: ok, no worries. | 20:45 |
| mrda | NobodyCam: o/ | 20:46 |
| BadCub | heya mrda | 20:46 |
| mrda | hey BadCub | 20:46 |
| jlvillal | mrda: Morning | 20:47 |
| * TheJulia yawns... and determines its time for more coffee | 20:47 | |
| mrda | hi jlvillal | 20:48 |
| BadCub | TheJulia: it is always_time_for_coffee lol | 20:48 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Docs on HardwareManager, building, and workflow https://review.openstack.org/168522 | 20:48 |
| mrda | oooh, coffee | 20:48 |
| JayF | NobodyCam: jroll ^ fixed jroll's issues, would appreciate your re-vote | 20:49 |
| jroll | JayF: you've added ISO, but it still says coreos/pxe is the only supported method | 20:50 |
| jroll | seems weird | 20:50 |
| JayF | oh. | 20:50 |
| *** Nisha_away has quit IRC | 20:51 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Docs on HardwareManager, building, and workflow https://review.openstack.org/168522 | 20:52 |
| JayF | jroll: let me know what you think about that | 20:52 |
| JayF | That also could use a section about the prebuilt images | 20:53 |
| JayF | man, everytime I look at this I see two dozen things missing | 20:53 |
| jroll | JayF: maybe append "which can be wrapped in a PXE or an ISO image" | 20:53 |
| jroll | or something? | 20:53 |
| JayF | jroll: did you look at my change? | 20:53 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is usable for ipmitool driver https://review.openstack.org/160383 | 20:53 |
| jroll | yes | 20:53 |
| jroll | what I'm looking for is... | 20:54 |
| JayF | jroll: I don't know where you'd want that then | 20:54 |
| JayF | it's also false | 20:54 |
| jroll | maybe a PXE section | 20:54 |
| JayF | the ISO is 100% entirely a pxe image that's encapsulated | 20:54 |
| jroll | right, technically | 20:54 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add check for usable tftp directory to PXE driver https://review.openstack.org/167451 | 20:54 |
| jroll | oh maybe this works | 20:54 |
| jroll | yeah, I think it's fine | 20:54 |
| mrda | git diff | 20:54 |
| mrda | sigh, sorry | 20:54 |
| JayF | jroll: I'm going to add a link to the prebuilt images. | 20:54 |
| jroll | you know there's a 'make iso', right? | 20:54 |
| JayF | jroll: I'm avoiding indicating stuff like that, bceause all of it is changing really soon | 20:55 |
| JayF | jroll: since I'm splitting out hte image builder | 20:55 |
| jlvillal | jroll, NobodyCam and anyone else: Any idea why the gate has "gate-ironic-python26" but the checks do not. | 20:55 |
| jroll | oh, ok | 20:55 |
| jlvillal | This is for 'ironic' not the client. | 20:55 |
| jroll | jlvillal: cause we dun goofed | 20:55 |
| jlvillal | jroll: On which side? The gate or the checks (not sure technical name). | 20:55 |
| jlvillal | :) | 20:55 |
| JayF | py26 is not supported by openstack anymore | 20:56 |
| jroll | jlvillal: we shouldn't gate on py26 IMO | 20:56 |
| mrda | jlvillal: we still want ironic server to run on RHEL7, which has py26 | 20:56 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: WIP: Add a check to pxe driver for TFTP service https://review.openstack.org/169918 | 20:56 |
| jroll | wat | 20:56 |
| jroll | I've never heard this | 20:56 |
| jroll | and kilo is dropping py2.6 globally in openstack | 20:56 |
| mrda | didn't we talk about this like 12 months ago jroll? | 20:56 |
| NobodyCam | I would love to get some feed back on ^^^ | 20:56 |
| jroll | mrda: I barely existed in ironic 12 months ago, no idea | 20:57 |
| JayF | mrda: it's been dropped project wide. Like Openstack doesn't support 2.6 on anything but client libs and shared stuff (like oslo) | 20:57 |
| *** dprince has quit IRC | 20:57 | |
| rloo | jlvillal: where do you see that? | 20:57 |
| mrda | It might be time, but last time this came up we still needed to support py26 because RHEL 7 | 20:57 |
| rloo | jlvillal: I believe we have that for the client still, but not ironic. | 20:57 |
| * mrda hopes this is the case | 20:58 | |
| mrda | i.e. we can drop py26 | 20:58 |
| jlvillal | rloo: http://status.openstack.org/zuul/ search for 'ironic' | 20:58 |
| jlvillal | rloo: expand entry | 20:58 |
| *** hj-hp has joined #openstack-ironic | 20:58 | |
| jlvillal | rloo: Here is a job: https://jenkins05.openstack.org/job/gate-ironic-python26/952/ | 20:58 |
| rloo | jlvillal: that's stable/juno. | 20:58 |
| mrda | it would free up a lot of vms in the gate, that's for sure | 20:59 |
| rloo | jlvillal: still need to support 2.6 in juno | 20:59 |
| jroll | oh, ha | 20:59 |
| mrda | there you go | 20:59 |
| NobodyCam | ok folks I off to the doctors. may or maynot be back ... depending on if I am able to see or not | 20:59 |
| jlvillal | rloo: Ah, thanks! :) | 20:59 |
| rloo | good luck NobodyCam. Just don't rub your eyes :-) | 21:00 |
| jlvillal | Sorry for the confusion everyone :( | 21:00 |
| rloo | jlvillal keeps us on our toes... | 21:00 |
| mrda | jlvillal asks the questions that we need to be asked :) | 21:00 |
| jlvillal | mrda: Well I did learn some things :) | 21:01 |
| *** kkoski has quit IRC | 21:01 | |
| *** kkoski has joined #openstack-ironic | 21:02 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Docs on HardwareManager, building, and workflow https://review.openstack.org/168522 | 21:02 |
| JayF | jroll: NobodyCam: ^ there, done. This time I'm sure :P | 21:02 |
| *** kkoski has quit IRC | 21:02 | |
| NobodyCam | lol | 21:02 |
| *** trown is now known as trown|outttypeww | 21:03 | |
| NobodyCam | oh good addition !!! | 21:03 |
| jroll | JayF: +A | 21:05 |
| *** achanda has quit IRC | 21:06 | |
| *** romcheg has joined #openstack-ironic | 21:11 | |
| *** jjohnson2__ has quit IRC | 21:13 | |
| *** Marga_ has quit IRC | 21:14 | |
| *** Marga_ has joined #openstack-ironic | 21:15 | |
| *** achanda has joined #openstack-ironic | 21:18 | |
| openstackgerrit | Michael Davies proposed openstack/ironic: Disable XML now that we have WSME/Pecan support https://review.openstack.org/169643 | 21:24 |
| mrda | ^^^ if we want xml disabled in K, this is the review you want :) | 21:25 |
| rloo | mrda: what's the diff with https://bugs.launchpad.net/ironic/+bug/1271317 and https://bugs.launchpad.net/ironic/+bug/1251880? | 21:37 |
| openstack | Launchpad bug 1271317 in Ironic "XML support not required for ironic" [Low,Triaged] - Assigned to Michael Davies (mrda) | 21:37 |
| openstack | Launchpad bug 1251880 in Ironic "Public API doesn't work after appending json/xml extension to the URI" [Medium,Fix released] - Assigned to Roman Prykhodchenko (romcheg) | 21:37 |
| rloo | mrda: maybe you referenced the wrong bug? (for the xml patch) | 21:38 |
| * mrda looks, but internets are slow | 21:38 | |
| * mrda made a mistake | 21:39 | |
| openstackgerrit | Michael Davies proposed openstack/ironic: Disable XML now that we have WSME/Pecan support https://review.openstack.org/169643 | 21:39 |
| *** ChuckC has quit IRC | 21:39 | |
| rloo | mrda: i looked at the patch itself. it is mumbo jumbo to me. Will let others review it ;) | 21:39 |
| mrda | rloo: I'll fix the commit | 21:40 |
| rloo | mrda: thx. | 21:40 |
| *** kkoski has joined #openstack-ironic | 21:44 | |
| openstackgerrit | Michael Davies proposed openstack/ironic: Disable XML now that we have WSME/Pecan support https://review.openstack.org/169643 | 21:45 |
| openstackgerrit | Merged openstack/python-ironicclient: Force LANGUAGE=en_US in test runs https://review.openstack.org/169036 | 21:46 |
| rloo | in the last few days I've seen some failures with 'Timed out waiting for Nova hypervisor-stats count >= 1'. Eg https://review.openstack.org/166201 | 21:48 |
| rloo | I think people have been doing 'recheck's. Has anyone looked into them? | 21:48 |
| JayF | I've seen one similar failure | 21:48 |
| JayF | but generally I feel like the tests have been slower | 21:48 |
| JayF | and that's a timeout hitting | 21:49 |
| JayF | so it makes me wonder if we just started hitting the timeout due to other circumstances | 21:49 |
| rloo | right. I'm wondering when that started to happen and if it is something we should worry about | 21:49 |
| rloo | it is waiting 2 minutes for nova resource tracker to pick up the count. don't know if nova is waiting for ironic to do something first though (I'm not familiar with the tests) | 21:52 |
| *** kkoski has quit IRC | 21:53 | |
| *** mjturek1 has quit IRC | 21:54 | |
| *** ekarlso has quit IRC | 22:04 | |
| *** ekarlso has joined #openstack-ironic | 22:04 | |
| *** ChuckC has joined #openstack-ironic | 22:09 | |
| *** Marga_ has quit IRC | 22:21 | |
| *** Marga_ has joined #openstack-ironic | 22:21 | |
| openstackgerrit | Merged openstack/ironic: Mock time.sleep() for two unittests https://review.openstack.org/169860 | 22:30 |
| *** igordcard_ has joined #openstack-ironic | 22:32 | |
| openstackgerrit | Michael Davies proposed openstack/ironic: New field 'name' not supported in port REST API https://review.openstack.org/163730 | 22:37 |
| openstackgerrit | Merged openstack/ironic: Remove setting language to en_US for 'venv' https://review.openstack.org/169914 | 22:40 |
| jroll | adam_g: have you hacked on a nova-less test suite at all? or thought about it? | 22:51 |
| adam_g | jroll, whadya mean? | 22:51 |
| jroll | adam_g: basic functional testing for ironic, without nova | 22:52 |
| adam_g | jroll, the tempest baremetal API tests should run without nova being there, but im not sure how to test actual deployment/provisioning without it | 22:53 |
| jroll | adam_g: yeah, that's my real goal. just wondering if you've thought about it. thanks :) | 22:53 |
| adam_g | jroll, you have any ideas? | 22:54 |
| jroll | adam_g: other than just make the same calls nova does, not really :) | 22:54 |
| *** andreykurilin_ has joined #openstack-ironic | 22:55 | |
| adam_g | rloo, JayF, looking like something bad happening there: http://logs.openstack.org/01/166201/14/check/check-tempest-dsvm-ironic-agent_ssh/ad7c3c5/logs/screen-n-cpu.txt.gz#_2015-04-01_21_26_30_567 | 22:58 |
| JayF | adam_g: interestingly enough | 22:59 |
| JayF | http://logs.openstack.org/01/166201/14/check/check-tempest-dsvm-ironic-agent_ssh/ad7c3c5/logs/screen-ir-api.txt.gz | 22:59 |
| JayF | clearly not 60 attempts there | 22:59 |
| adam_g | JayF, yea | 23:00 |
| adam_g | where are they going | 23:00 |
| jroll | JayF: adam_g: notice that ir-api wasn't up at that point | 23:00 |
| adam_g | uhm | 23:01 |
| *** ijw has joined #openstack-ironic | 23:01 | |
| adam_g | 2015-04-01 21:26:30.567 ERROR nova.virt.ironic.client_wrapper [req-ecd502fe-5f3e-4b74-bd01-52910c116077 None None] Error contacting Ironic server for 'node.list'. Attempt 60 of 60 | 23:02 |
| adam_g | 2015-04-01 21:26:42.355 17092 INFO ironic.cmd.api [-] Serving on http://0.0.0.0:6385 | 23:02 |
| adam_g | hmm | 23:02 |
| JayF | adam_g: given the set of failures I've seen, my hypothesis was basically that it was taking everything longer, more or less, and we were hitting timeouts | 23:02 |
| JayF | this matches that hypothesis fwiw | 23:02 |
| adam_g | JayF, yeah, looks like it | 23:02 |
| jroll | yeah, just longer setup time | 23:03 |
| *** chlong has joined #openstack-ironic | 23:07 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Docs on HardwareManager, building, and workflow https://review.openstack.org/168522 | 23:09 |
| *** penick has quit IRC | 23:11 | |
| *** andreykurilin_ has quit IRC | 23:13 | |
| *** igordcard has quit IRC | 23:14 | |
| *** ijw has quit IRC | 23:23 | |
| *** yuanying has joined #openstack-ironic | 23:25 | |
| *** Marga_ has quit IRC | 23:29 | |
| *** Marga_ has joined #openstack-ironic | 23:30 | |
| *** ijw has joined #openstack-ironic | 23:30 | |
| *** harlowja has quit IRC | 23:31 | |
| *** harlowja has joined #openstack-ironic | 23:32 | |
| *** harlowja has quit IRC | 23:49 | |
| *** harlowja has joined #openstack-ironic | 23:50 | |
| *** romcheg has quit IRC | 23:54 | |
| *** Haomeng|2 has joined #openstack-ironic | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!