| openstackgerrit | Merged openstack/ironic: Adopt config generator https://review.openstack.org/176367 | 00:02 |
|---|---|---|
| *** kan_ has joined #openstack-ironic | 00:05 | |
| *** mtanino has quit IRC | 00:05 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/177231 | 00:06 |
| *** zz_jgrimm has quit IRC | 00:10 | |
| *** zz_jgrimm has joined #openstack-ironic | 00:11 | |
| *** kan_ has quit IRC | 00:12 | |
| *** zz_jgrimm has quit IRC | 00:17 | |
| *** zz_jgrimm has joined #openstack-ironic | 00:18 | |
| *** dprince has joined #openstack-ironic | 00:54 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Include graphviz in install prerequisites https://review.openstack.org/179930 | 00:56 |
| *** zhenguo has joined #openstack-ironic | 00:56 | |
| *** igordcard has quit IRC | 01:22 | |
| *** Marga_ has quit IRC | 01:23 | |
| *** purp_2 is now known as purp | 01:31 | |
| *** purp is now known as stealth_purp | 01:36 | |
| openstackgerrit | Tan Lin proposed openstack/ironic-specs: Bare Metal Trust Using Intel TXT https://review.openstack.org/133902 | 01:38 |
| openstackgerrit | Tan Lin proposed openstack/ironic-specs: Bare Metal Trust Using Intel TXT https://review.openstack.org/133902 | 01:51 |
| *** dprince has quit IRC | 01:56 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-python-agent: Add support for preinstalled efi system partitions https://review.openstack.org/180000 | 02:03 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-python-agent: Use mkdtemp for mounting virtual media device https://review.openstack.org/180001 | 02:03 |
| openstackgerrit | Bruno Cornec proposed openstack/ironic: Specify environment variables needed for a standalone usage https://review.openstack.org/180002 | 02:05 |
| *** rloo has quit IRC | 02:09 | |
| *** pcrews has quit IRC | 02:09 | |
| *** achanda has quit IRC | 02:11 | |
| *** chenglch has joined #openstack-ironic | 02:12 | |
| *** stealth_purp is now known as purp | 02:37 | |
| *** ijw has quit IRC | 02:39 | |
| *** ijw has joined #openstack-ironic | 02:39 | |
| *** ijw has quit IRC | 02:43 | |
| *** early has quit IRC | 02:48 | |
| *** early has joined #openstack-ironic | 03:01 | |
| *** Nisha has joined #openstack-ironic | 03:01 | |
| *** spandhe has quit IRC | 03:10 | |
| *** killer_prince has joined #openstack-ironic | 03:18 | |
| *** pcrews has joined #openstack-ironic | 03:21 | |
| *** coolsvap|afk is now known as coolsvap | 03:23 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add disk layout check on re-provisioning https://review.openstack.org/180014 | 03:23 |
| *** links has joined #openstack-ironic | 03:31 | |
| *** gridinv has joined #openstack-ironic | 03:41 | |
| *** achanda has joined #openstack-ironic | 03:51 | |
| *** saripurigopi has joined #openstack-ironic | 03:59 | |
| openstackgerrit | Merged openstack/ironic: Imported Translations from Transifex https://review.openstack.org/179561 | 04:07 |
| *** achanda has quit IRC | 04:12 | |
| *** gridinv has quit IRC | 04:22 | |
| *** kan_ has joined #openstack-ironic | 04:31 | |
| *** jmanko has joined #openstack-ironic | 04:35 | |
| *** killer_prince has quit IRC | 04:35 | |
| *** jmank has quit IRC | 04:38 | |
| *** kalpase has joined #openstack-ironic | 04:38 | |
| *** kan_ is now known as kan__ | 04:50 | |
| *** gridinv has joined #openstack-ironic | 04:58 | |
| *** killer_prince has joined #openstack-ironic | 04:58 | |
| *** killer_prince has left #openstack-ironic | 04:59 | |
| *** killer_prince has quit IRC | 04:59 | |
| *** killer_prince has joined #openstack-ironic | 05:00 | |
| *** killer_prince has quit IRC | 05:02 | |
| *** killer_prince has joined #openstack-ironic | 05:02 | |
| *** lazy_prince has joined #openstack-ironic | 05:05 | |
| *** killer_prince has quit IRC | 05:05 | |
| openstackgerrit | Kan proposed openstack/python-ironicclient: Disable invalid sort key in list command https://review.openstack.org/176219 | 05:07 |
| *** achanda has joined #openstack-ironic | 05:10 | |
| *** natorious has joined #openstack-ironic | 05:24 | |
| *** gridinv has quit IRC | 05:28 | |
| *** Nisha has quit IRC | 05:34 | |
| *** Nisha has joined #openstack-ironic | 05:34 | |
| *** Nisha_away has joined #openstack-ironic | 05:37 | |
| *** Nisha has quit IRC | 05:37 | |
| *** saripurigopi has quit IRC | 05:40 | |
| *** spandhe has joined #openstack-ironic | 05:54 | |
| *** cui has joined #openstack-ironic | 05:56 | |
| *** spandhe_ has joined #openstack-ironic | 05:57 | |
| *** spandhe has quit IRC | 05:58 | |
| *** spandhe_ is now known as spandhe | 05:58 | |
| *** natorious is now known as zz_natorious | 06:02 | |
| *** saripurigopi has joined #openstack-ironic | 06:12 | |
| *** ijw has joined #openstack-ironic | 06:12 | |
| *** zz_natorious is now known as natorious | 06:14 | |
| *** natorious is now known as zz_natorious | 06:22 | |
| *** cui has quit IRC | 06:23 | |
| *** openstackgerrit has quit IRC | 06:23 | |
| *** cui has joined #openstack-ironic | 06:23 | |
| *** openstackgerrit has joined #openstack-ironic | 06:23 | |
| *** svasheka has quit IRC | 06:25 | |
| *** cui has quit IRC | 06:25 | |
| *** cui has joined #openstack-ironic | 06:26 | |
| *** ijw has quit IRC | 06:27 | |
| *** ijw has joined #openstack-ironic | 06:28 | |
| *** svasheka has joined #openstack-ironic | 06:37 | |
| *** ukalifon has joined #openstack-ironic | 06:38 | |
| *** zhenguo has quit IRC | 06:47 | |
| openstackgerrit | Ghe Rivero proposed openstack/ironic: Use oslo_log lib https://review.openstack.org/157602 | 06:48 |
| *** jcoufal has joined #openstack-ironic | 06:50 | |
| *** e0ne has joined #openstack-ironic | 06:51 | |
| *** zhenguo has joined #openstack-ironic | 06:53 | |
| *** kalpase has left #openstack-ironic | 06:53 | |
| *** ijw has quit IRC | 07:01 | |
| *** ifarkas has quit IRC | 07:03 | |
| *** ifarkas has joined #openstack-ironic | 07:04 | |
| openstackgerrit | Nisha Agarwal proposed stackforge/proliantutils: parsing storage data fails if multiple logical drives or controllers are present https://review.openstack.org/180043 | 07:05 |
| *** e0ne has quit IRC | 07:06 | |
| *** athomas has joined #openstack-ironic | 07:06 | |
| openstackgerrit | Nisha Agarwal proposed stackforge/proliantutils: parsing storage data fails in some scenarios https://review.openstack.org/180043 | 07:09 |
| *** cui has quit IRC | 07:14 | |
| *** achanda has quit IRC | 07:16 | |
| *** ramineni has joined #openstack-ironic | 07:19 | |
| openstackgerrit | Ghe Rivero proposed openstack/ironic: Update config generator to use oslo.i18n https://review.openstack.org/180046 | 07:20 |
| *** achanda has joined #openstack-ironic | 07:22 | |
| *** achanda has quit IRC | 07:24 | |
| openstackgerrit | Ghe Rivero proposed openstack/ironic: Update config generator to use oslo released libs https://review.openstack.org/180046 | 07:27 |
| *** yog__ has joined #openstack-ironic | 07:37 | |
| *** chlong has quit IRC | 07:44 | |
| *** chenglch has quit IRC | 07:47 | |
| *** ndipanov has joined #openstack-ironic | 07:52 | |
| *** lucasagomes has joined #openstack-ironic | 08:02 | |
| *** mdbooth has quit IRC | 08:16 | |
| *** rameshg87 has joined #openstack-ironic | 08:20 | |
| rameshg87 | good afternoon ironic | 08:20 |
| *** mdbooth has joined #openstack-ironic | 08:22 | |
| *** derekh has joined #openstack-ironic | 08:23 | |
| *** e0ne has joined #openstack-ironic | 08:28 | |
| *** e0ne is now known as e0ne_ | 08:28 | |
| *** cui has joined #openstack-ironic | 08:28 | |
| *** cui has quit IRC | 08:30 | |
| *** tiagogomes_ has joined #openstack-ironic | 08:31 | |
| *** cui has joined #openstack-ironic | 08:31 | |
| *** mgoddard has joined #openstack-ironic | 08:32 | |
| *** MattMan has joined #openstack-ironic | 08:33 | |
| lucasagomes | rameshg87, afternoon | 08:44 |
| lucasagomes | rameshg87, btw, I don't know if u saw I added the wol driver | 08:44 |
| *** e0ne_ is now known as e0ne | 08:44 | |
| lucasagomes | I tested it locally it works :-) | 08:45 |
| lucasagomes | rameshg87, https://review.openstack.org/179078 | 08:45 |
| *** e0ne is now known as e0ne_ | 08:45 | |
| rameshg87 | lucasagomes: morning :) | 08:46 |
| *** e0ne_ is now known as e0ne | 08:46 | |
| rameshg87 | lucasagomes: yeah i saw | 08:46 |
| rameshg87 | lucasagomes: i badly wanted to test, but seems like my old bare metal is not powering on. may be some problem with smps. :( | 08:46 |
| rameshg87 | lucasagomes: i need to get that fixed. until then can't test it :( | 08:47 |
| lucasagomes | rameshg87, oh :-( | 08:47 |
| lucasagomes | ok | 08:47 |
| lucasagomes | no worries | 08:47 |
| rameshg87 | lucasagomes: and i have some updates regarding other issue - the exception on uefi localboot | 08:49 |
| rameshg87 | lucasagomes: i tried with dib ramdisk 3 times - it passed all 3 times. i.e, it seems to work when reboot is done from within the bare metal | 08:49 |
| lucasagomes | rameshg87, right hmm | 08:54 |
| lucasagomes | rameshg87, perhaps we should add an option to always to a soft reboot after the deployment is finished? | 08:54 |
| lucasagomes | and add a "reboot" command for IPA ? | 08:55 |
| kejiahu | morning ironic | 08:55 |
| lucasagomes | rameshg87, on Friday I also tried to find you online. I was having some problems with iLO Gen 7. It seems that setting the boot device using ipmitool on those machines doesn't always work | 08:56 |
| rameshg87 | lucasagomes: yeah it seems the good thing to do to me | 08:56 |
| lucasagomes | kejiahu, morning | 08:56 |
| rameshg87 | lucasagomes: oh, friday was a holiday here, so wasn't working | 08:56 |
| lucasagomes | rameshg87, I had to use the pxe_ilo driver, with prolianutils to do the deployment | 08:56 |
| *** jistr has joined #openstack-ironic | 08:56 | |
| lucasagomes | I was doing local boot on the machines, and apparently ipmitool set the next boot device to disk | 08:57 |
| rameshg87 | lucasagomes: and it everything is fine with pxe_ilo driver ? | 08:57 |
| lucasagomes | but it wasn't persistent, and even it was saying DISK for the next boot device it would still pxe boot | 08:57 |
| lucasagomes | rameshg87, yeah wiht pxe_ilo it worked | 08:57 |
| lucasagomes | so I'm just using it now | 08:57 |
| lucasagomes | rameshg87, but I will have to package proliantutils :-) wich is a good thing | 08:58 |
| rameshg87 | lucasagomes: but we had the same problem with some other hardware also ? | 08:58 |
| rameshg87 | :) | 08:58 |
| lucasagomes | rameshg87, I only had access to 2 iLO Gen 7 machines, and both behaved the same | 08:58 |
| lucasagomes | but I haven't tested with any other iLO boxes | 08:58 |
| lucasagomes | I don't even have access to any other so... | 08:58 |
| rameshg87 | lucasagomes: oh okay | 08:58 |
| rameshg87 | lucasagomes: may be i can give a try with the boxes i have - ipmitool driver, right ? | 08:59 |
| rameshg87 | lucasagomes: it has been a while since i use ipmitool driver | 08:59 |
| lucasagomes | rameshg87, yeah the normal pxe_ipmitool driver with local boot | 08:59 |
| lucasagomes | rameshg87, it's ok. As I said, it worked with pxe_ilo | 08:59 |
| *** pelix has joined #openstack-ironic | 08:59 | |
| rameshg87 | lucasagomes: but in juno, we were using ipmitool to get/set boot device | 08:59 |
| lucasagomes | so... unless you have some spare time to give it a go I wouldn't bother | 08:59 |
| rameshg87 | lucasagomes: okay | 08:59 |
| lucasagomes | rameshg87, yeah, probably is something with setting the DISK | 09:00 |
| rameshg87 | okay | 09:00 |
| lucasagomes | I had a paste... lemme see if I can find | 09:00 |
| rameshg87 | lucasagomes: also fedora uefi localboot seems to be broken :( | 09:01 |
| lucasagomes | :-( | 09:01 |
| lucasagomes | any error in specific? | 09:01 |
| rameshg87 | lucasagomes: it seems that fedora follows something different from grub | 09:01 |
| rameshg87 | i mean something different from ubuntu for efi | 09:01 |
| rameshg87 | lucasagomes: i updated https://review.openstack.org/#/c/167929/ to add the required packages for both ubuntu and fedora | 09:02 |
| lucasagomes | rameshg87, ah nice one! | 09:02 |
| rameshg87 | lucasagomes: it's all fine - all the required packages for both ubuntu and fedora are in the image | 09:02 |
| lucasagomes | thanks for that | 09:02 |
| rameshg87 | lucasagomes: ubuntu-bios, ubuntu-uefi, fedora-bios works fine | 09:03 |
| lucasagomes | cool, /me wonders if I can get access to a machine with uefi to give it a go | 09:03 |
| lucasagomes | I will ask for one | 09:03 |
| rameshg87 | lucasagomes: but fedora-uefi seems to follow a different path - when i created the image, it already has a /boot/efi and running grub-install fails | 09:03 |
| lifeless | lucasagomes: https://fedoraproject.org/wiki/Using_UEFI_with_QEMU | 09:04 |
| lifeless | lucasagomes: might help | 09:04 |
| lifeless | night ;) | 09:04 |
| lucasagomes | lifeless, a-ha! or that :-). Thanks | 09:04 |
| lucasagomes | lifeless, g'night | 09:04 |
| rameshg87 | lifeless: thanks. and it worked once for me in virtualbox too - sometimes it crashes though :( | 09:05 |
| *** edcragg has joined #openstack-ironic | 09:10 | |
| *** early has quit IRC | 09:12 | |
| ramineni | lucasagomes: hi | 09:12 |
| ramineni | lucasagomes: I'm trying out ipxe depploy , somehow not able to do a successful one :(, could you help me out | 09:13 |
| lucasagomes | ramineni, hi there, sure | 09:20 |
| lucasagomes | what's up? | 09:20 |
| lucasagomes | ramineni, do you have an HTTP server running where the images will be placed and all? | 09:21 |
| ramineni | lucasagomes: does this configuartion on neutron port seems ok ? http://paste.openstack.org/show/214937/ | 09:21 |
| lucasagomes | is the ipxe tag configured in neutron | 09:21 |
| lucasagomes | ramineni, yes | 09:21 |
| ramineni | lucasagomes: yes, followed installation guide | 09:21 |
| lucasagomes | ramineni, right, and what you see when you try to boot? | 09:22 |
| ramineni | lucasagomes: configured http server on on CC as /opt/stack/data/ironic/httpboot | 09:23 |
| lucasagomes | ramineni, r u trying to do with devstack? | 09:23 |
| ramineni | its not able to get undionly.kpxe i guess , i tried manually also nt wrkng | 09:24 |
| ramineni | lucasagomes: yes | 09:24 |
| ramineni | tftp> get undionly.kpxe | 09:24 |
| ramineni | Error code 1: File not found | 09:24 |
| lucasagomes | rameshg87, is unfionly.kpxe in the tftp folder? | 09:24 |
| lucasagomes | rameshg87, sorry from ping. ramineni | 09:24 |
| ramineni | lucasagomes: yes | 09:24 |
| ramineni | lucasagomes: its there | 09:24 |
| lucasagomes | ramineni, right... hmm is xinted running serving tftp? can you get any other file? it may be permission | 09:24 |
| rameshg87 | :) | 09:25 |
| lucasagomes | ramineni, btw, devstack supports creating a ipxe enviroment | 09:25 |
| ramineni | lucasagomes: ok, let me check | 09:25 |
| *** alexpilotti has joined #openstack-ironic | 09:25 | |
| lucasagomes | ramineni, you just need to add IRONIC_ENABLE_IPXE=True in the local.conf | 09:25 |
| rameshg87 | lucasagomes: this is the problem with fedora on efi | 09:25 |
| rameshg87 | lucasagomes: https://fedoraproject.org/wiki/GRUB_2?rd=Grub2#Updating_GRUB_2_configuration_on_UEFI_systems | 09:25 |
| rameshg87 | lucasagomes: it says "grub2-install shouldn't be used on EFI systems. The grub2-efi package installs a prebaked grubx64.efi on the EFI System partition, which looks for grub.cfg on the ESP in /EFI/fedora/ whereas the grub2-install command creates a custom grubx64.efi, deletes the original installed one, and looks for grub.cfg in /boot/grub2/." | 09:25 |
| rameshg87 | lucasagomes: the problem is fedora has only pre-baked efi system partition contents :( | 09:26 |
| ramineni | lucasagomes: ooh, not aware of that, will create ipxe config automatically using the variable? | 09:26 |
| lucasagomes | ramineni, yes | 09:26 |
| lucasagomes | ramineni, it will do everything actually... configure the HTTP server and all that | 09:27 |
| * lucasagomes haven't tested devstack with it in a while, but should work | 09:27 | |
| lucasagomes | ramineni, IRONIC_IPXE_ENABLED=True | 09:28 |
| lucasagomes | that's the option | 09:28 |
| ramineni | lucasagomes: i'm nt able to get any file :( | 09:28 |
| lucasagomes | ramineni, yeah it seems the TFTP server is not running/working | 09:28 |
| lucasagomes | do a "status" see if it's connected | 09:28 |
| lucasagomes | check permissions, if you're running with selinux enabled check the audit log | 09:28 |
| ramineni | tftp> status | 09:29 |
| ramineni | Connected to 10.10.1.11. | 09:29 |
| ramineni | Mode: netascii Verbose: off Tracing: off | 09:29 |
| ramineni | Rexmt-interval: 5 seconds, Max-timeout: 25 seconds | 09:29 |
| ramineni | it says connected | 09:29 |
| lucasagomes | rameshg87, :-( damn it seems a lot of work to get it working on fedora then | 09:29 |
| rameshg87 | lucasagomes: oh got | 09:29 |
| rameshg87 | lucasagomes: it seems there is another package | 09:29 |
| lucasagomes | ramineni, right, do a get in verbose mode | 09:29 |
| rameshg87 | grub2-efi-modules | 09:29 |
| rameshg87 | lucasagomes: http://www.rpmfind.net//linux/RPM/fedora/updates/21/x86_64/g/grub2-efi-modules-2.02-0.13.fc21.x86_64.html | 09:30 |
| lucasagomes | rameshg87, right | 09:30 |
| rameshg87 | lucasagomes: it seems to just deliver what we want :) | 09:30 |
| lucasagomes | \o/ well would be good to test first | 09:30 |
| lucasagomes | see if that's the case | 09:30 |
| * rameshg87 goes to try that out | 09:30 | |
| lucasagomes | I will try to setup a vm with lifeless guide | 09:30 |
| rameshg87 | okay | 09:31 |
| ramineni | lucasagomes: it says getting from 10.10.1.11:undionly.kpxe to undionly.kpxe [netascii] | 09:32 |
| ramineni | Error code 1: File not found | 09:32 |
| ramineni | lucasagomes: in verbose mode | 09:32 |
| lucasagomes | ramineni, file not found? hmmm | 09:33 |
| lucasagomes | look at the ironic.conf file | 09:33 |
| lucasagomes | see if the tftp_root option is pointing to the same folder you copied the files to | 09:34 |
| *** romcheg has joined #openstack-ironic | 09:34 | |
| *** cui has quit IRC | 09:35 | |
| *** cui has joined #openstack-ironic | 09:36 | |
| ramineni | lucasagomes: its is pointing actually, here are the options i have in conf file http://paste.openstack.org/show/214941/ | 09:36 |
| lucasagomes | ramineni, right odd... I don't know, the tftp server should be configure by devstack with or without ipxe anyway | 09:37 |
| lucasagomes | hmmm | 09:37 |
| ramineni | lucasagomes: ya :( | 09:37 |
| lucasagomes | the xinited tftp configuration looks correct? | 09:37 |
| ramineni | lucasagomes: where would be that /etc/xinetd.d? | 09:39 |
| lucasagomes | hmm I think so | 09:42 |
| lucasagomes | there's a "tftp" file somewhere there | 09:43 |
| lucasagomes | with the tftp configuration | 09:43 |
| * lucasagomes boots a vm to check | 09:43 | |
| lucasagomes | ramineni, but anyway, if you could run devstack with that ipxe option, it would be cool | 09:43 |
| ramineni | lucasagomes: thanks.. checking that :) | 09:44 |
| rameshg87 | lucasagomes: i almost wrote up the code for that without checking if there is another package which delivered the modules - https://review.openstack.org/#/c/180000/ :( | 09:46 |
| lucasagomes | ramineni, no problem | 09:46 |
| lucasagomes | rameshg87, oh | 09:46 |
| lucasagomes | well it cna still be useful no? | 09:46 |
| lucasagomes | for distros that doesn't have any package | 09:46 |
| rameshg87 | lucasagomes: yeah | 09:47 |
| *** early has joined #openstack-ironic | 09:51 | |
| *** romcheg has quit IRC | 10:00 | |
| alexpilotti | rameshg87 lucasagomes: do you think we could have https://review.openstack.org/#/c/176390/ merged by the summit? | 10:00 |
| alexpilotti | we’ll have an OCS chassis there and we’d like to announce the fact that support is available upstream in Ironic :-) | 10:00 |
| *** degorenko has joined #openstack-ironic | 10:00 | |
| rameshg87 | alexpilotti: i think it's in a pretty good shape. i will try to get back to it today itself :) | 10:01 |
| alexpilotti | rameshg87: thanks! | 10:02 |
| lucasagomes | alexpilotti, hi there, I will re-review it. | 10:02 |
| lucasagomes | but yes, I think it's possible | 10:02 |
| alexpilotti | lucasagomes: yeii :-) | 10:03 |
| *** early has quit IRC | 10:03 | |
| rameshg87 | lucasagomes: updated - https://review.openstack.org/#/c/167929/ | 10:08 |
| rameshg87 | lucasagomes: do fedora and redhat cloud images share the same packages ? | 10:08 |
| *** romcheg has joined #openstack-ironic | 10:13 | |
| lucasagomes | rameshg87, thanks! I can check, but I believe yes | 10:14 |
| *** early has joined #openstack-ironic | 10:14 | |
| *** dlpartain has joined #openstack-ironic | 10:16 | |
| *** dlpartain has left #openstack-ironic | 10:17 | |
| *** kbyrne has quit IRC | 10:18 | |
| *** kbyrne has joined #openstack-ironic | 10:20 | |
| *** bigjools has quit IRC | 10:34 | |
| *** bigjools has joined #openstack-ironic | 10:34 | |
| *** bigjools has quit IRC | 10:34 | |
| *** bigjools has joined #openstack-ironic | 10:34 | |
| *** e0ne is now known as e0ne_ | 10:35 | |
| *** e0ne_ is now known as e0ne | 10:36 | |
| *** killer_prince has joined #openstack-ironic | 10:39 | |
| lucasagomes | alexpilotti, I only have 1 comment, should be quick to fix | 10:39 |
| lucasagomes | other than that it looks good to me | 10:39 |
| alexpilotti | lucasagomes: tx! | 10:39 |
| *** lazy_prince has quit IRC | 10:43 | |
| *** early has quit IRC | 10:47 | |
| *** dlpartain has joined #openstack-ironic | 10:49 | |
| alexpilotti | lucasagomes: self.assertIs(..., True) | 10:49 |
| alexpilotti | shouldnt it be: self.assertIs(True, …) | 10:49 |
| alexpilotti | as reference values should appear first? | 10:49 |
| *** early has joined #openstack-ironic | 10:51 | |
| openstackgerrit | Alessandro Pilotti proposed openstack/ironic: Adds OCS Power and Management interfaces https://review.openstack.org/176390 | 10:54 |
| *** e0ne is now known as e0ne_ | 10:56 | |
| *** killer_prince has quit IRC | 11:01 | |
| *** romcheg has quit IRC | 11:01 | |
| *** early has quit IRC | 11:03 | |
| *** lazy_prince has joined #openstack-ironic | 11:08 | |
| *** cui has quit IRC | 11:12 | |
| *** lazy_prince has quit IRC | 11:17 | |
| *** dlpartain has left #openstack-ironic | 11:18 | |
| *** early has joined #openstack-ironic | 11:18 | |
| *** romcheg has joined #openstack-ironic | 11:19 | |
| *** romcheg has quit IRC | 11:20 | |
| lucasagomes | alexpilotti, sure | 11:23 |
| lucasagomes | alexpilotti, sorry I was afk | 11:23 |
| *** romcheg has joined #openstack-ironic | 11:24 | |
| lucasagomes | alexpilotti, +2'd | 11:24 |
| alexpilotti | lucasagomes: tx! :-) | 11:26 |
| *** Nisha_away has quit IRC | 11:26 | |
| bcornec | lucasagomes: thanks for the review. What is the best approach to resubmit now ? Should I try to keep the same ID or should I abandon the current one and make a new one ? | 11:28 |
| lucasagomes | bcornec, 1 sec I'm answering it right now on that review | 11:29 |
| bcornec | Oops great thank you ! | 11:29 |
| lucasagomes | bcornec, https://review.openstack.org/#/c/180002/ | 11:29 |
| lucasagomes | hope that helps | 11:30 |
| bcornec | Yes. May I ask an additional question ? | 11:31 |
| rameshg87 | lucasagomes: it worked \o/ | 11:31 |
| lucasagomes | rameshg87, w00ts! | 11:31 |
| lucasagomes | bcornec, sure of course | 11:31 |
| * rameshg87 waits for the system to boot up | 11:32 | |
| lucasagomes | bcornec, http://docs.openstack.org/infra/manual/developers.html#updating-a-change | 11:32 |
| bcornec | Isn't it better to squash all the modifications in a single commit ? I tried to use that yesterday night to produce the new version of the patch, and that's where a new ID appeared, and I didn't know how to avoid that. As you can guess, I have many problems to solve: learning python, rst and git all at the same time (I'm perl, docbook and svn historically !) | 11:32 |
| lucasagomes | right, by ammending the change you're squashing it into a single commit | 11:34 |
| lucasagomes | so when you do a "git log", you'll only see 1 commit for that change there | 11:34 |
| lucasagomes | but in gerrit it keeps the history of all the changes and comments that patch have been though | 11:35 |
| lucasagomes | through* | 11:35 |
| lucasagomes | which is nice | 11:35 |
| lucasagomes | so new reviewers can look at old patch-sets to see the previous comments and answers from the author | 11:35 |
| *** early has quit IRC | 11:36 | |
| bcornec | Ok so I'm now at the rebase -i phase and it shows to me the 2 commits (with 2 picks by default). I think I need to sqush the first one and pick the second IIIUC right ? | 11:36 |
| bcornec | Humm no I need to squash the second in fact | 11:37 |
| *** early has joined #openstack-ironic | 11:37 | |
| lucasagomes | bcornec, you wouldn't need to rebase that change... Unless you want to rebase it with a new master | 11:37 |
| bcornec | THat's what I did as the master changed in between. Isn't that the correct way ? (the doc you pointed me to advised to do so that's why ;-) | 11:38 |
| openstackgerrit | sonu proposed openstack/python-ironicclient: Added unit tests for command-line shell https://review.openstack.org/172371 | 11:38 |
| lucasagomes | bcornec, right... yeah you can do that if there's a conflict with a new master | 11:38 |
| *** thrash|g0ne is now known as thrash | 11:38 | |
| lucasagomes | bcornec, which doesn't seem to be the case for that change | 11:39 |
| bcornec | Humm no there was no conflict. I just thought it was better to provide a commit on top of the current master id rather than a previsou one, but again I'm still not fully comfortable with how git manages this... :-( | 11:39 |
| lucasagomes | bcornec, right... here what you can do in a nutshell | 11:40 |
| openstackgerrit | Bruno Cornec proposed openstack/ironic: Specify environment variables needed for a standalone usage https://review.openstack.org/180002 | 11:40 |
| lucasagomes | 1) git review -d I7c8864dd08540f3ffa4a1c5489df05077fd51511 | 11:40 |
| lucasagomes | 2) make ur changes | 11:41 |
| lucasagomes | 3) git add | 11:41 |
| lucasagomes | 4) git commit --amend | 11:41 |
| lucasagomes | 5) git review | 11:41 |
| bcornec | Ok so what I did was rather: | 11:41 |
| lucasagomes | Ok yeah you just did it :-) | 11:41 |
| bcornec | 1) git fetch origin | 11:42 |
| bcornec | 2) git checkout master | 11:42 |
| bcornec | 3) git pull origin master | 11:42 |
| bcornec | 4) git checkout stdalone-doc | 11:42 |
| bcornec | 5) git rebase -i master | 11:42 |
| bcornec | 6) git review | 11:42 |
| bcornec | With my modifs between 4 and 5 | 11:42 |
| bcornec | THat's how I understood the doc in fact. | 11:43 |
| lucasagomes | right, yeah it works too | 11:43 |
| lucasagomes | I usually don't use "-i" for rebase with master... It will fail in case of a conflict anyway so you can go add and then issue a "git rebase --continue" | 11:44 |
| bcornec | In your receipe, in fact you work on a dedicated branch mapped to the gerrit ID that correspond to my previous request for review. But in that case, if the master changed on a file I'm also modifying, someone will have to reconcile. If I do it on my side with the rebase it should avoid that potential pb no ? | 11:44 |
| lucasagomes | but anyway... there's few ways of doing this tihngs... The more you use git you will be tunning ur skills with it | 11:44 |
| lucasagomes | bcornec, yeah I use a branch for each commit | 11:44 |
| lucasagomes | bcornec, in case master is updated and you want to get the last one | 11:45 |
| *** saripurigopi has quit IRC | 11:45 | |
| openstackgerrit | Imre Farkas proposed openstack/ironic: [WIP] DRAC: automatic RAID configuration https://review.openstack.org/176319 | 11:45 |
| bcornec | But if I don't solve the conflict on my side before submitting, who will have to do that step ? | 11:45 |
| lucasagomes | you get the new changes on master: 1) "git checkout master" 2) "git pull" | 11:45 |
| *** igordcard_ has quit IRC | 11:45 | |
| *** igordcard_ has joined #openstack-ironic | 11:46 | |
| *** igordcard_ has quit IRC | 11:46 | |
| lucasagomes | then you go back to the branch you want to rebase and rebase it onto the new master: 1) "git checkout <branch name>" 2) "git rebase master" | 11:46 |
| lucasagomes | if it conflicts you fix the conflicts, git add the files you touched and do a "git rebase --continue" | 11:46 |
| lucasagomes | that's it | 11:46 |
| *** igordcard_ has joined #openstack-ironic | 11:46 | |
| bcornec | Ok. I need to look more again at the doc around git rebase to understand really how it works. I hate black magic ;-) | 11:46 |
| lucasagomes | bcornec, heh it's different from svn I get it | 11:47 |
| lucasagomes | but you'll see it makes sense | 11:47 |
| bcornec | Ahhh, keeping the Change-Id makes a patch Set 2. I see : | 11:47 |
| lucasagomes | exactly | 11:47 |
| lucasagomes | bcornec, actually I would suggest to you to read the git man pages | 11:48 |
| lucasagomes | it's pretty well documented and good to understand the way git works | 11:48 |
| lucasagomes | bcornec, https://www.kernel.org/pub/software/scm/git/docs/ each command has it's own page | 11:48 |
| lucasagomes | so... yeah just take a look at the commands you use for every day work | 11:48 |
| bcornec | Very different from SVN, and I'm still unsure I like that. There are good stuff in git like the add -p which you can't do with svn. But the reconciliation is much more tricky for me, even looking at gitk. | 11:48 |
| lucasagomes | bcornec, right... I think that after you understand how git works you will start disliking svn and other centralized version control systems :-) | 11:49 |
| openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC: support for RAID via vendor_passthru https://review.openstack.org/145464 | 11:50 |
| openstackgerrit | Imre Farkas proposed openstack/ironic: [WIP] DRAC: automatic RAID configuration https://review.openstack.org/176319 | 11:50 |
| bcornec | I need to become accustomed to the process *and* the comands to put in front to succeed that understanding. But not sure I'll dislike SVN ;-) Well I use it on small projects with few contributors so I never had the issues to solve that git is trying to resolve. | 11:50 |
| *** e0ne_ is now known as e0ne | 11:51 | |
| lucasagomes | bcornec, yeah, I get that for small projects that's complete fine | 11:53 |
| lucasagomes | bcornec, btw https://try.github.io/levels/1/challenges/1 | 11:54 |
| *** dprince has joined #openstack-ironic | 11:54 | |
| bcornec | Another question: for that patch set 2 I don't see the automated tests that were launched previsouly but I see you made a +2. Is that normal ? Am I not waiting enough for the automated tests to start ? | 11:57 |
| lucasagomes | bcornec, I voted ahead of the jenkins test | 11:57 |
| lucasagomes | bcornec, you can see the tests here http://status.openstack.org/zuul/ | 11:57 |
| lucasagomes | use the filter to find ur change | 11:57 |
| bcornec | I'll continue to ask dumb questions, as I'll relay that workflow internally to other people wanting to contribute, and need to be a bit more comfortable to provide them with appropriate answers | 11:57 |
| lucasagomes | say tested failed for some unknown reason not related to you're change. You can also add a comment "recheck" on the patch set that will re-trigger the tests | 11:58 |
| *** igordcard_ has quit IRC | 11:58 | |
| lucasagomes | bcornec, http://docs.openstack.org/infra/manual/developers.html#automated-testing | 11:58 |
| lucasagomes | read this developers documentation first | 11:58 |
| *** igordcard_ has joined #openstack-ironic | 12:03 | |
| bcornec | Well I've read it, but sometimes there are unexplained aspects that are no pb for people knowing well all the toolset, but more difficult to understand for complete newbye like me here. Also when something is missing and I want to amend it to improve the content, I need to go through the process ;-) And even after reading it questions remains, and only going through the process step by step is really making you getting it (at least I'm th | 12:03 |
| *** saripurigopi has joined #openstack-ironic | 12:05 | |
| bcornec | I see the change 180002 reported as unknown in 2 change queues (all steps reported as queued), but not failed in zuul. I may miss something here. | 12:06 |
| lucasagomes | bcornec, it's queued because the tests are not running yet | 12:07 |
| bcornec | Ok, so I just need to be patient ;-) | 12:07 |
| *** zhenguo has quit IRC | 12:07 | |
| lucasagomes | bcornec, since we have a limited number of resources we queue the changes and test if as the resources are becoming available | 12:07 |
| lucasagomes | yeah | 12:08 |
| bcornec | Many thanks lucasagomes for your help and support. Much appreciated. | 12:08 |
| lucasagomes | bcornec, no problem :-) thanks for contribute with Ironic | 12:08 |
| *** thrash has quit IRC | 12:09 | |
| *** thrash has joined #openstack-ironic | 12:11 | |
| *** lucasagomes is now known as lucas-hungry | 12:15 | |
| *** igordcard_ has quit IRC | 12:17 | |
| *** igordcard_ has joined #openstack-ironic | 12:17 | |
| *** saripurigopi has quit IRC | 12:18 | |
| *** trown|outttypeww is now known as trown | 12:19 | |
| * rameshg87 goes home | 12:24 | |
| *** rameshg87 has quit IRC | 12:24 | |
| *** ukalifon has quit IRC | 12:36 | |
| *** early has quit IRC | 12:40 | |
| *** jjohnson2_ has joined #openstack-ironic | 12:42 | |
| *** early has joined #openstack-ironic | 12:45 | |
| *** links has quit IRC | 12:52 | |
| *** kan__ has quit IRC | 12:52 | |
| *** lucas-hungry is now known as lucasagomes | 13:06 | |
| openstackgerrit | Imre Farkas proposed openstack/ironic: [WIP] DRAC: automatic RAID configuration https://review.openstack.org/176319 | 13:07 |
| jroll | mornin' all | 13:07 |
| jroll | lucasagomes: need an in-band reboot? :) https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/extensions/standby.py#L242 | 13:07 |
| lucasagomes | jroll, oh... morning | 13:08 |
| lucasagomes | jroll, thanks it seems to do what we need https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/shell/reboot.sh | 13:08 |
| jroll | lucasagomes: indeed :) | 13:09 |
| jroll | I don't remember why we didn't just do 'reboot', I think it was sticking sometimes | 13:09 |
| lucasagomes | tho "run_image" :-) | 13:09 |
| jroll | well | 13:09 |
| lucasagomes | naming is hard I know | 13:09 |
| jroll | the goal was to lay down an image and then run it, right? | 13:09 |
| lucasagomes | fair enoff | 13:09 |
| jroll | we hoped to eventually use kexec | 13:10 |
| lucasagomes | hmm -1 | 13:10 |
| jroll | but that's hard(tm) | 13:10 |
| jroll | (optionally) use kexec :P | 13:10 |
| lucasagomes | yeah I had loads and loads of problems with kexec | 13:10 |
| lucasagomes | yeah +1 optionally fine | 13:10 |
| jroll | I mean, if we ever get around to making it work | 13:10 |
| jroll | plus caching images | 13:10 |
| jroll | you'd have like 10 second deploys | 13:10 |
| lucasagomes | it depends a lot on hardware... ACPI, graphical cards many things | 13:10 |
| jroll | right | 13:10 |
| lucasagomes | but yeah, if you know ur hardware and tested to ensure it's working... that's good | 13:11 |
| lucasagomes | skipping having to initialize bios, remap the hardware interruptions and all that is much faster | 13:12 |
| jroll | yep | 13:12 |
| lucasagomes | cool! Man holidays are good... the week should consist of 4 working days and 3 days off | 13:13 |
| lucasagomes | :-) | 13:13 |
| lucasagomes | quality of life! | 13:13 |
| jroll | lol | 13:13 |
| *** yuriyz has quit IRC | 13:14 | |
| NobodyCam | morning Ironic | 13:15 |
| NobodyCam | morning lucasagomes and jroll | 13:16 |
| *** afaranha has quit IRC | 13:16 | |
| lucasagomes | NobodyCam, yo morning | 13:16 |
| NobodyCam | jroll: wow your up earily | 13:16 |
| NobodyCam | hey hey lucasagomes how was your day away | 13:16 |
| lucasagomes | NobodyCam, was awesome! | 13:17 |
| lucasagomes | just knowing that you don't have to wake up early on monday | 13:17 |
| lucasagomes | damn | 13:17 |
| NobodyCam | lol | 13:18 |
| NobodyCam | lucasagomes: ++++ 10000 | 13:18 |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: WIP: Implement event log retrieval from BMCs https://review.openstack.org/179188 | 13:18 |
| *** rloo has joined #openstack-ironic | 13:21 | |
| lucasagomes | jroll, one thing... shouldn't that reboot command be sync? | 13:22 |
| *** yuriyz has joined #openstack-ironic | 13:22 | |
| lucasagomes | so we don't have to pool to check whether it failed or not, it should be quick to just run that echo's | 13:22 |
| *** yog__ has quit IRC | 13:22 | |
| *** athomas has quit IRC | 13:28 | |
| *** afaranha has joined #openstack-ironic | 13:30 | |
| *** athomas has joined #openstack-ironic | 13:35 | |
| *** romcheg has quit IRC | 13:40 | |
| openstackgerrit | Ghe Rivero proposed openstack/ironic: Update config generator to use oslo released libs https://review.openstack.org/180046 | 13:41 |
| *** romcheg has joined #openstack-ironic | 13:44 | |
| rloo | hi and good morning everyone! | 13:49 |
| NobodyCam | good morning rloo :) | 13:49 |
| rloo | hey GheRivero, was https://review.openstack.org/#/c/157602/ all rebasing from revision 17? | 13:49 |
| *** rameshg87 has joined #openstack-ironic | 13:50 | |
| *** lucasagomes_ has joined #openstack-ironic | 13:50 | |
| *** lucasagomes has quit IRC | 13:51 | |
| *** lucasagomes_ is now known as lucasagomes | 13:55 | |
| *** Sukhdev has joined #openstack-ironic | 13:56 | |
| *** e0ne is now known as e0ne_ | 13:56 | |
| *** early has quit IRC | 13:57 | |
| * NobodyCam goes to get more coffee and thinks he really like this answer: http://stackoverflow.com/a/2031195 | 13:58 | |
| *** e0ne_ is now known as e0ne | 13:59 | |
| rameshg87 | alexpilotti: hello | 13:59 |
| *** e0ne is now known as e0ne_ | 13:59 | |
| alexpilotti | rameshg87: hi | 14:00 |
| *** e0ne_ is now known as e0ne | 14:00 | |
| rameshg87 | alexpilotti: just a quick one, i was going through code - https://review.openstack.org/#/c/176390/12/ironic/drivers/modules/msftocs/common.py | 14:00 |
| rameshg87 | alexpilotti: L43-46 | 14:00 |
| *** e0ne is now known as e0ne_ | 14:00 | |
| rameshg87 | alexpilotti: shouldn't that have been msftocs_base_url, msftocs_username, msftocs_password | 14:00 |
| *** e0ne_ is now known as e0ne | 14:00 | |
| rameshg87 | alexpilotti: or am i missing something | 14:01 |
| alexpilotti | rameshg87: looking, tx | 14:01 |
| *** rloo_ has joined #openstack-ironic | 14:03 | |
| *** rloo has quit IRC | 14:03 | |
| Shrews | NobodyCam: LOL, yeah, that's a good one | 14:05 |
| rameshg87 | NobodyCam: :) | 14:05 |
| alexpilotti | rameshg87: well, that’s interesting. It definitely misses the right prefix | 14:05 |
| NobodyCam | hehe | 14:06 |
| *** early has joined #openstack-ironic | 14:06 | |
| *** zz_jgrimm is now known as jgrimm | 14:06 | |
| NobodyCam | morning rameshg87 and alexpilotti | 14:06 |
| rameshg87 | morning NobodyCam | 14:06 |
| alexpilotti | NobodyCam: morning! | 14:07 |
| NobodyCam | :) | 14:07 |
| rameshg87 | alexpilotti: yeah, may be a good candidate for a unit test :) | 14:07 |
| alexpilotti | rameshg87: good catch. Was thinking about putting all those configs in constants | 14:07 |
| rameshg87 | alexpilotti: okay, if you are planning to throw up a new patch, give me some 10-15 mins. i am just going over it. | 14:08 |
| alexpilotti | rameshg87: as I was fearing something like that to happen when refactoring them :-D | 14:08 |
| alexpilotti | rameshg87: ok, adding the unit test in the meantime | 14:08 |
| NobodyCam | morning to Shrews too... /me has only had one cup of coffee | 14:08 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix and enhance "Exercising the Services Locally" docs https://review.openstack.org/180163 | 14:08 |
| Shrews | NobodyCam: morning. i'm used to being ignored ;) | 14:09 |
| NobodyCam | lol ... NOT... | 14:09 |
| NobodyCam | :) | 14:09 |
| lucasagomes | Shrews, morning | 14:09 |
| Shrews | hey lucasagomes | 14:09 |
| lucasagomes | Shrews, quick q, probably dumb. In docstack the config/ironic just configure the Ironic services right? | 14:11 |
| lucasagomes | it doesn't configure other things like glance, keystone etc... | 14:12 |
| * lucasagomes wants to run devstack in a container | 14:12 | |
| Shrews | lucasagomes: that's correct. | 14:12 |
| Shrews | lucasagomes: if you want to just run devstack itself within a single container, dockstack is probably not the right solution | 14:13 |
| lucasagomes | Shrews, right... you know if someone already tried that? | 14:13 |
| *** rloo_ has quit IRC | 14:13 | |
| Shrews | lucasagomes: the idea behind dockstack is to run the individual services in separate containers | 14:13 |
| Shrews | lucasagomes: no, i do not | 14:13 |
| *** rloo has joined #openstack-ironic | 14:14 | |
| lucasagomes | Shrews, fair enuff, yeah I think I can use dockstack for some tests... but sometimes I need the other services as well | 14:14 |
| lucasagomes | such as neutron | 14:14 |
| lucasagomes | Shrews, another q... the conductor service, does it configure tftp and all that? | 14:14 |
| Shrews | lucasagomes: that should be easy enough to add... | 14:14 |
| *** jrist has quit IRC | 14:15 | |
| *** kkoski has joined #openstack-ironic | 14:15 | |
| Shrews | lucasagomes: no. the config/ironic example is really just setup to run a simple fake-driver example | 14:16 |
| Shrews | lucasagomes: i never got around to going further than that, but the tools should be there | 14:16 |
| *** BadCub_Hiding is now known as BadCub | 14:18 | |
| *** kan_ has joined #openstack-ironic | 14:19 | |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: WIP: Implement event log retrieval from BMCs https://review.openstack.org/179188 | 14:19 |
| lucasagomes | Shrews, I see... that's cool I will take a look at it | 14:19 |
| lucasagomes | I mean, not today but soonish I hope | 14:19 |
| *** lifeless has quit IRC | 14:20 | |
| lucasagomes | I want to have a way to build a very quick and light envrioment to test baremetal here | 14:20 |
| Shrews | lucasagomes: awesome | 14:20 |
| lucasagomes | yeah, with that wol driver you also don't need any fancy hardware so | 14:21 |
| lucasagomes | will be cool to have it | 14:21 |
| jroll | lucasagomes: I don't expect run_image to be able to return a response before things start rebooting, that's why it's async | 14:21 |
| lucasagomes | jroll, a-ha... I bet it makes sense haha | 14:22 |
| lucasagomes | +1 | 14:22 |
| lucasagomes | it's not early morning here but I think I didn't thought it through :-) | 14:22 |
| GheRivero | rloo: yeah. The oslo_log change was all rebasing | 14:22 |
| lucasagomes | think it* | 14:22 |
| rloo | GheRivero: good. It had already been approved, so if it was just rebasing, I can just approve it again w/o another +2. | 14:23 |
| lucasagomes | rloo, if you revoke the approval jenkins stop merging it | 14:23 |
| lucasagomes | (as a note) | 14:23 |
| jroll | lucasagomes: heh | 14:24 |
| jroll | no worries | 14:24 |
| rloo | lucasagomes: right. I'm going to add it. You had approved it before, but there was a merge conflict. | 14:24 |
| lucasagomes | ah | 14:24 |
| lucasagomes | rloo, gotcha... anyway morning (GheRivero too) | 14:24 |
| GheRivero | afternoon lucasagomes | 14:25 |
| rloo | lucasagomes: no worries. Hi to you and GheRivero and Shrews and can't remember who else ;) | 14:25 |
| Shrews | morning rloo | 14:25 |
| lucasagomes | rloo, btw I've asked the nova cores about the quota thing... | 14:25 |
| lucasagomes | rloo, it doesn't get committed if the deployment fails | 14:25 |
| rloo | lucasagomes: ok thx. I don't think we want to prevent the user from getting their quota back if the deploy fails. | 14:26 |
| lucasagomes | so it doesn't subtract from the tenant's quota | 14:26 |
| lucasagomes | rloo, yeah | 14:26 |
| NobodyCam | afternoon GheRivero | 14:26 |
| jroll | I believe failed deletes and stuck builds take up quota though | 14:26 |
| jroll | I may be wrong | 14:26 |
| lucasagomes | jroll, right, but that's after deployment succeed | 14:26 |
| lucasagomes | so you delete the instance | 14:27 |
| jroll | right | 14:27 |
| lucasagomes | in this case it's before the deployment succeed | 14:27 |
| lucasagomes | so nova reserve the quota, deploy fails and nova rolls it back | 14:27 |
| *** lifeless has joined #openstack-ironic | 14:27 | |
| rloo | lucasagomes: I need to look at your change to nova. was it to prevent deleting only when deploying? | 14:27 |
| lucasagomes | rloo, it avoid destroy() when the node is in maintenance | 14:28 |
| lucasagomes | it just says "hey you can't destroy because it's in maintenance" | 14:28 |
| *** jrist has joined #openstack-ironic | 14:28 | |
| rloo | lucasagomes: destroy=delete? Am wondering about the quota thing there. | 14:28 |
| lucasagomes | delete calls destroy | 14:28 |
| lucasagomes | but destroy is also triggered when the deployment fails | 14:29 |
| lucasagomes | to clean up the node | 14:29 |
| lucasagomes | and put it back to available etc | 14:29 |
| rloo | lucasagomes: so let's say the instance is active. and at some point, eg power-sync doesn't work and ironic puts the node in maintenance. then user tries to delete instance cuz it doesn't work. the delete will fail? | 14:29 |
| lucasagomes | the delete will alert the user that the instance is in maintenance and can't be destroyed | 14:30 |
| lucasagomes | so he can contact someone to take a look at it | 14:30 |
| lucasagomes | on the other hand if he destroy and the power cycle doesn't work | 14:30 |
| lucasagomes | he gets a stuck instance being destroyed | 14:30 |
| jroll | ehhhhhhhhhhhh | 14:30 |
| jroll | in my environment I'd like to be able to allow deletes while in maintenance | 14:31 |
| lucasagomes | this is only for troubleshoot as well, it's not about running in production | 14:31 |
| lucasagomes | by default we don't keep things running after deployfail etc | 14:31 |
| jroll | we've had a few things where user reports brokenness, we allow them to clean up or ship data off or whatever before deleting, but we put in maintenance before we forget | 14:31 |
| *** mtanino has joined #openstack-ironic | 14:32 | |
| rloo | lucasagomes: how is the change in nova only for troubleshooting? (I know the patch you have in ironic is for troubleshooting) | 14:32 |
| lucasagomes | jroll, oh... so you put in maintenance and then he deletes it? | 14:33 |
| lucasagomes | hmmmm | 14:33 |
| jroll | yeah | 14:33 |
| lucasagomes | ok /me got fix that | 14:33 |
| rloo | i have been wondering what the diff is wrt a node being in some *FAIL state, vs in maintenance mode, and how that affects the user. | 14:33 |
| lucasagomes | rloo, yeah the problem is since nova calls destroy() it doesn't stay in DEPLOYFAIL state | 14:34 |
| lucasagomes | :-/ | 14:34 |
| alexpilotti | rameshg87: unit tests added, please let me know when I can update the new patch | 14:34 |
| rameshg87 | alexpilotti: i just put my comments, others are just nits | 14:37 |
| rameshg87 | lucasagomes: thanks for that .. i just notice that now :) | 14:37 |
| lucasagomes | rameshg87, no worries | 14:37 |
| lucasagomes | I didn't know assertTrue/False did that | 14:37 |
| lucasagomes | I thought it actually asserted that it was a boolean value | 14:37 |
| rameshg87 | lucasagomes: even i thought :) | 14:38 |
| lucasagomes | #butnot | 14:38 |
| lucasagomes | yeah | 14:38 |
| mariojv | good morning all | 14:39 |
| mariojv | rloo: thanks for the comments on https://review.openstack.org/#/c/179910/ | 14:39 |
| NobodyCam | morning mariojv | 14:39 |
| rloo | morning mariojv. yw. let me know if you have questions. | 14:39 |
| jroll | mariojv: rloo is the best reviewer | 14:39 |
| mariojv | yeah, very helpful | 14:39 |
| jroll | 'specially for docs | 14:39 |
| * rloo doesn't really like reviewing or writing docs | 14:40 | |
| jroll | :P | 14:42 |
| *** jrist has quit IRC | 14:42 | |
| lucasagomes | NobodyCam, http://paste.openstack.org/show/214989/ re rebuild provision_state | 14:45 |
| lucasagomes | it puts it back into deploying | 14:45 |
| NobodyCam | lucasagomes: ahh !! | 14:46 |
| openstackgerrit | Alessandro Pilotti proposed openstack/ironic: Adds OCS Power and Management interfaces https://review.openstack.org/176390 | 14:47 |
| alexpilotti | rameshg87: done! | 14:47 |
| rameshg87 | alexpilotti: thanks. will have a look. | 14:48 |
| *** rameshg87 is now known as rameshg87-brb | 14:48 | |
| *** coolsvap is now known as coolsvap|afk | 14:48 | |
| *** zz_natorious is now known as natorious | 14:49 | |
| *** MattMan has quit IRC | 14:53 | |
| *** MattMan has joined #openstack-ironic | 14:55 | |
| *** MattMan has left #openstack-ironic | 14:55 | |
| *** Marga_ has joined #openstack-ironic | 14:57 | |
| *** Marga_ has quit IRC | 14:57 | |
| *** Marga_ has joined #openstack-ironic | 14:58 | |
| *** Sukhdev has quit IRC | 14:58 | |
| *** achanda has joined #openstack-ironic | 15:01 | |
| *** achanda has quit IRC | 15:06 | |
| *** romcheg has quit IRC | 15:13 | |
| tiagogomes_ | hi, do the iscsi_ilo and agent_ilo drivers work with cdrkit? | 15:14 |
| NobodyCam | tiagogomes_: I have not tested that, I'm not sure if others have? | 15:15 |
| lucasagomes | tiagogomes_, IIRC those drivers uses mkisofs to generate the ISO for virtual media | 15:20 |
| lucasagomes | if cdrkit includes mkisofs, I bet it works | 15:21 |
| tiagogomes_ | it includes... something else | 15:24 |
| bcornec | wodim ? | 15:25 |
| tiagogomes_ | genisoimage | 15:25 |
| bcornec | argh yes, wodim is to burn not making the image sorry | 15:26 |
| lucasagomes | tiagogomes_, right, so look at the code apparently it only supports mkisofs | 15:26 |
| *** Marga_ has quit IRC | 15:26 | |
| lucasagomes | looking* | 15:26 |
| NobodyCam | morning bcornec and tiagogomes_ | 15:27 |
| NobodyCam | :-p | 15:27 |
| lucasagomes | tiagogomes_, https://github.com/openstack/ironic/blob/master/ironic/common/images.py#L229 | 15:27 |
| bcornec | morning ! (or afternoon or night depending on TZ) | 15:27 |
| tiagogomes_ | yes it is hardcoded, I wonder if I symlink mkisofs to getnisoimage will work | 15:27 |
| lucasagomes | tiagogomes_, not sure due the parameters passed to each binary | 15:28 |
| bcornec | the commands have the same interface | 15:28 |
| lucasagomes | they probably differ | 15:28 |
| lucasagomes | right if they have the same CLI interface it should work... but I don't know | 15:28 |
| bcornec | I use them without issues as part of MondoRescue | 15:28 |
| tiagogomes_ | they appear to have the same CLI interface | 15:29 |
| lucasagomes | we could abstract that bits and support different binaries if there's a real use case for that | 15:29 |
| lucasagomes | tiagogomes_, right, then what about we try to find if one of the binaries is present in the system | 15:29 |
| lucasagomes | and use it for the commands? we can help a helper function there to do it | 15:29 |
| bcornec | mkisofs is depracated in most recent linux distros. genisoimage is the new "normal" command to use | 15:30 |
| lucasagomes | bcornec, oh nice to know | 15:30 |
| lucasagomes | yeah I think a helper function would be good then | 15:30 |
| bcornec | Example of what we did for MondoRescue: http://trac.mondorescue.org/changeset/2848 (not that this project has the best code base ever written, but gives an idea :-)) | 15:32 |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: WIP: Implement event log retrieval from BMCs https://review.openstack.org/179188 | 15:35 |
| *** jrist has joined #openstack-ironic | 15:35 | |
| *** jrist has quit IRC | 15:35 | |
| *** jrist has joined #openstack-ironic | 15:35 | |
| *** jrist has quit IRC | 15:37 | |
| lucasagomes | if someone has a time mind taking a look at https://review.openstack.org/#/c/178229/ ? | 15:41 |
| *** jrist has joined #openstack-ironic | 15:41 | |
| NobodyCam | lucasagomes: is the spec for that approved.. BP shows needs approval | 15:42 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Add section header to state machines page https://review.openstack.org/180204 | 15:43 |
| lucasagomes | NobodyCam, oh it was approved yes. /me checks | 15:43 |
| *** shakamunyi has joined #openstack-ironic | 15:43 | |
| NobodyCam | lol | 15:43 |
| NobodyCam | yep I approved it | 15:43 |
| NobodyCam | :-p | 15:43 |
| lucasagomes | yeah :D | 15:43 |
| lucasagomes | changed the bp | 15:43 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Adds docstrings to some functions in ironic/conductor/manager.py https://review.openstack.org/179910 | 15:45 |
| alexpilotti | lucasagomes: can I ask you to please +2 / re-review the OCS patch again? https://review.openstack.org/#/c/176390/ | 15:47 |
| lucasagomes | alexpilotti, sure will re-review it | 15:47 |
| lucasagomes | gimme 1 min... just finishing something | 15:47 |
| *** hemnafk is now known as hemna | 15:48 | |
| *** jcoufal has quit IRC | 15:48 | |
| *** jmanko has quit IRC | 15:50 | |
| *** spandhe has quit IRC | 15:51 | |
| *** rameshg87-brb is now known as rameshg87 | 15:52 | |
| * NobodyCam gets ready for a all hands call.. | 15:53 | |
| rameshg87 | lucasagomes: when you get some time, please have a look at rloo and my comments on https://review.openstack.org/#/c/173214/ | 15:54 |
| lucasagomes | ack | 15:54 |
| rloo | rameshg87: didn't I comment on it already? | 15:54 |
| rameshg87 | rloo: yeah, i just came back to it today. | 15:55 |
| rameshg87 | rloo: just wanted lucasagomes opinion on it since he has reviewed it before | 15:55 |
| rameshg87 | rloo: i think i agree to your comments. i am just posting a reply | 15:55 |
| rloo | rameshg87: ah, ok. I feel sorry about that, I hope I'm helping to make the feature better... | 15:55 |
| rameshg87 | rloo: definitely, it's only getting better in my opinion, thanks for the details review :) | 15:56 |
| rameshg87 | *detailed | 15:56 |
| rloo | rameshg87: :) | 15:56 |
| lucasagomes | alexpilotti, did you test that last patch-set and all? | 15:57 |
| lucasagomes | alexpilotti, cause the other was broken no? due the missing "msftocs_" namespace | 15:58 |
| rloo | who's Bob Hansen, and why is he saying that patches have 'Merge Failed'! Is he Jenkins' side-kick? https://review.openstack.org/#/c/179910/ | 15:58 |
| *** kan_ has quit IRC | 15:58 | |
| *** stickpin has joined #openstack-ironic | 15:59 | |
| JoshNang | i had something similar with one of my nova patches, turned out to be misconfigured third party ci (though different name) | 16:00 |
| *** kan__ has joined #openstack-ironic | 16:00 | |
| rameshg87 | lol :D | 16:00 |
| rloo | JoshNang: oops! I'll be on the lookout for Bob Hansen at the summit. ha ha. | 16:00 |
| *** chlong has joined #openstack-ironic | 16:00 | |
| JoshNang | heh | 16:01 |
| *** Marga_ has joined #openstack-ironic | 16:01 | |
| NobodyCam | morning JoshNang | 16:03 |
| alexpilotti | lucasagomes: yes, did it | 16:04 |
| alexpilotti | lucasagomes: added a unit test for that namespace issue as well | 16:04 |
| lucasagomes | alexpilotti, cool! thanks | 16:05 |
| JoshNang | NobodyCam: good morning! | 16:05 |
| rameshg87 | rloo: just replied on https://review.openstack.org/#/c/173214/ | 16:05 |
| alexpilotti | I did quite a few runs with runs REST API class against the actual hardware | 16:05 |
| rameshg87 | rloo: i think we can make the input target_raid_config go to driver_info | 16:05 |
| rloo | rameshg87: great! | 16:06 |
| alexpilotti | but the integration tests I had didnt have the namespace upodated, hence the issue | 16:06 |
| rameshg87 | rloo: and the raid config after the operation goto node.properties.raid_config | 16:06 |
| rameshg87 | rloo: because i think it fits as a property of the node once raid is configured | 16:06 |
| alexpilotti | lucasagomes: ideally, once day we’ll have a full CI for this hardware as well :-) | 16:06 |
| lucasagomes | alexpilotti, awesome! that would be pretty good | 16:07 |
| lucasagomes | alexpilotti, the patch already had 2+2 so I +A it | 16:07 |
| lucasagomes | thanks | 16:07 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Adds docstrings to some functions in ironic/conductor/manager.py https://review.openstack.org/179910 | 16:07 |
| alexpilotti | lucasagomes rameshg87 yuriyz: thaks guys for all the reviews! :-) | 16:08 |
| lucasagomes | alexpilotti, thank you for the code :-) | 16:08 |
| rameshg87 | alexpilotti: welcome and thanks for your changes in ironic too :) | 16:08 |
| alexpilotti | hope to see you in Vancouver and show you thins thing live :-) | 16:08 |
| rloo | rameshg87: I think I'm fine with node.properties.raid_config. it'll be changed if the raid is reconfigured right? | 16:09 |
| lucasagomes | alexpilotti, oh nice! Looking fwd to that | 16:09 |
| rameshg87 | rloo: yes | 16:09 |
| alexpilotti | (and have some beers!) | 16:09 |
| *** minnesotags2 has left #openstack-ironic | 16:09 | |
| rloo | rameshg87: what does ironic node-set-raid-config do? what is it setting? Is it the same as setting driver_info/target_raid_config? | 16:10 |
| rameshg87 | rloo: yeah, but may be a convenient way that ironic node-update <> driver_info/target_raid_config=<> | 16:14 |
| rameshg87 | rloo: yeah it writes to the same place driver_info/target_raid_config | 16:15 |
| rloo | rameshg87: it is convenient. but i would worry that others will want convenient ways to set other driver_info fields. | 16:17 |
| *** chlong has quit IRC | 16:18 | |
| rloo | rameshg87: i have to take off for a bit. i'll try to remember to comment in your patch later today. | 16:18 |
| *** rloo has quit IRC | 16:18 | |
| rameshg87 | rloo: sure. | 16:18 |
| *** rameshg87 is now known as rameshg87-dinner | 16:19 | |
| *** kan__ has quit IRC | 16:22 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Clean up instance specific informantion on tear down https://review.openstack.org/180220 | 16:24 |
| *** jistr has quit IRC | 16:26 | |
| bcornec | It seems that a lot of ironic submissions in Zuul are failing. Not sure why. | 16:36 |
| lucasagomes | bcornec, :-( gate seems very slow today too | 16:43 |
| lucasagomes | we gotta check the logs see if we can spot something about the problem that is causing the tests to fail | 16:44 |
| *** romcheg has joined #openstack-ironic | 16:44 | |
| *** rameshg87-dinner is now known as rameshg87 | 16:54 | |
| * rameshg87 leaves for the day | 16:54 | |
| rameshg87 | good night ironic | 16:54 |
| NobodyCam | night rameshg87 | 16:54 |
| *** rameshg87 has quit IRC | 16:55 | |
| *** saripurigopi has joined #openstack-ironic | 16:55 | |
| *** Marga_ has quit IRC | 16:56 | |
| *** Marga_ has joined #openstack-ironic | 16:57 | |
| *** saripurigopi has quit IRC | 17:04 | |
| -openstackstatus- NOTICE: zuul has been restarted to troubleshoot an issue, gerrit events between 15:00-17:00 utc were lost and changes updated or approved during that time will need to be rechecked or have their approval votes readded to trigger testing | 17:04 | |
| NobodyCam | lucasagomes: ^^^^ | 17:05 |
| lucasagomes | oh | 17:05 |
| *** e0ne has quit IRC | 17:05 | |
| *** harlowja has quit IRC | 17:08 | |
| *** harlowja has joined #openstack-ironic | 17:09 | |
| lucasagomes | folks I will call it a day | 17:13 |
| NobodyCam | night lucasagomes | 17:14 |
| lucasagomes | have a great night everyone | 17:14 |
| lucasagomes | see y'all tomorrow | 17:14 |
| BadCub | night lucasagomes | 17:15 |
| lucasagomes | BadCub, night :D | 17:15 |
| *** lucasagomes is now known as lucas-dinner | 17:15 | |
| *** achanda has joined #openstack-ironic | 17:16 | |
| *** jrist has quit IRC | 17:17 | |
| *** edcragg has quit IRC | 17:19 | |
| *** rloo has joined #openstack-ironic | 17:23 | |
| *** early has quit IRC | 17:26 | |
| *** early has joined #openstack-ironic | 17:26 | |
| NobodyCam | brb | 17:27 |
| *** trown is now known as trown|lunch | 17:29 | |
| *** Sukhdev has joined #openstack-ironic | 17:35 | |
| *** athomas has quit IRC | 17:37 | |
| jlvillal | When doing flavors for Ironic, can we specify the CPU family (SandyBridge, Haswell, etc..)? | 17:47 |
| *** e0ne has joined #openstack-ironic | 17:50 | |
| *** e0ne is now known as e0ne_ | 17:50 | |
| *** e0ne_ is now known as e0ne | 17:51 | |
| jroll | jlvillal: best way to do that would be to use nova flavor extra_specs and ironic node capabilities wherever they are stored | 17:52 |
| jlvillal | jroll: Thanks. So it sounds like it can be done, but not that intuitive. If I'm understanding correctly. | 17:53 |
| jroll | jlvillal: right, it's not a first-class property of a flavor | 17:53 |
| jlvillal | jroll: got it | 17:53 |
| *** raildo has joined #openstack-ironic | 18:12 | |
| *** barra204 has joined #openstack-ironic | 18:12 | |
| *** fungi has joined #openstack-ironic | 18:17 | |
| *** fungi has left #openstack-ironic | 18:18 | |
| *** penick has joined #openstack-ironic | 18:18 | |
| *** ramineni has quit IRC | 18:21 | |
| *** barra204 has quit IRC | 18:27 | |
| *** logan2 has quit IRC | 18:28 | |
| *** natorious is now known as zz_natorious | 18:30 | |
| *** logan2 has joined #openstack-ironic | 18:31 | |
| *** kkoski has quit IRC | 18:36 | |
| *** Marga_ has quit IRC | 18:41 | |
| *** Marga_ has joined #openstack-ironic | 18:42 | |
| *** barra204 has joined #openstack-ironic | 18:42 | |
| *** jrist has joined #openstack-ironic | 18:49 | |
| NobodyCam | wow zuul and mr. jenkins are way backed up today :p | 18:50 |
| openstackgerrit | Nisha Agarwal proposed stackforge/proliantutils: parsing storage data fails in some scenarios https://review.openstack.org/180043 | 18:54 |
| *** penick has quit IRC | 18:55 | |
| *** trown|lunch is now known as trown | 18:57 | |
| *** penick has joined #openstack-ironic | 19:00 | |
| openstackgerrit | Nisha Agarwal proposed stackforge/proliantutils: parsing storage data fails in some scenarios https://review.openstack.org/180043 | 19:02 |
| *** jistr has joined #openstack-ironic | 19:08 | |
| *** spandhe has joined #openstack-ironic | 19:12 | |
| *** ifarkas has quit IRC | 19:12 | |
| *** EmilienM is now known as EmilienM|afk | 19:15 | |
| *** ijw has joined #openstack-ironic | 19:17 | |
| *** kkoski has joined #openstack-ironic | 19:24 | |
| *** subscope_ has joined #openstack-ironic | 19:27 | |
| *** penick has quit IRC | 19:30 | |
| *** ijw has quit IRC | 19:33 | |
| *** mtanino has quit IRC | 19:34 | |
| *** penick has joined #openstack-ironic | 19:34 | |
| *** Sukhdev has quit IRC | 19:35 | |
| *** zz_natorious is now known as natorious | 19:39 | |
| *** jistr has quit IRC | 19:49 | |
| *** kbs1 has quit IRC | 20:10 | |
| *** achanda has quit IRC | 20:11 | |
| NobodyCam | brb | 20:14 |
| *** mtanino has joined #openstack-ironic | 20:20 | |
| *** spandhe has quit IRC | 20:23 | |
| jlvillal | NobodyCam: Are we in Liberty mode yet for Ironic? I notice Stackaltyics doesn't seem to have any Liberty statistics for Ironic. | 20:30 |
| NobodyCam | i think we should be | 20:30 |
| rloo | what does 'liberty mode' mean? | 20:31 |
| jlvillal | NobodyCam: rloo: http://stackalytics.com/?release=liberty&module=ironic-group | 20:31 |
| jlvillal | rloo: Meaning that reviews and commits are being assigned in Stackalytics as for the Liberty cycle. | 20:31 |
| rloo | once kilo-1 or whatever was branched, master was liberty. | 20:31 |
| rloo | jlvillal: not sure how/what stackalytics uses to determine what 'liberty' is. | 20:32 |
| jlvillal | rloo: Stackalytics doesn't even have an "ironic' project when selecting Liberty: http://stackalytics.com/?release=liberty | 20:34 |
| NobodyCam | was in TC meeting | 20:34 |
| *** rsalevsky has quit IRC | 20:35 | |
| *** achanda has joined #openstack-ironic | 20:35 | |
| *** rsalevsky has joined #openstack-ironic | 20:35 | |
| rloo | jlvillal: i'm not versed in stackalytics; I rarely look at it. not sure who maintains it. | 20:35 |
| jlvillal | NobodyCam: Cool :) | 20:35 |
| jlvillal | rloo: I think Mirantis runs it. | 20:35 |
| rloo | jlvillal: might be a matter of 'just' adding Ironic module to it. | 20:36 |
| jlvillal | rloo: I'm taking a look at the code. Thanks! | 20:37 |
| rloo | jlvillal: thank YOU :-) | 20:37 |
| *** barra204 has quit IRC | 20:37 | |
| *** e0ne has quit IRC | 20:38 | |
| *** natorious has quit IRC | 20:39 | |
| *** persia has quit IRC | 20:40 | |
| *** clett has quit IRC | 20:40 | |
| *** russell_h has quit IRC | 20:41 | |
| *** persia has joined #openstack-ironic | 20:41 | |
| *** persia has quit IRC | 20:41 | |
| *** persia has joined #openstack-ironic | 20:41 | |
| *** natorious has joined #openstack-ironic | 20:44 | |
| *** russell_h has joined #openstack-ironic | 20:45 | |
| *** russell_h has quit IRC | 20:45 | |
| *** russell_h has joined #openstack-ironic | 20:45 | |
| *** r-daneel has joined #openstack-ironic | 20:47 | |
| *** raildo is now known as raildo_away | 20:50 | |
| openstackgerrit | Bruno Cornec proposed openstack/ironic: Specify environment variables needed for a standalone usage https://review.openstack.org/180002 | 20:53 |
| *** spandhe has joined #openstack-ironic | 20:59 | |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: WIP: Implement event log retrieval from BMCs https://review.openstack.org/179188 | 21:04 |
| NobodyCam | brb | 21:04 |
| *** kkoski has quit IRC | 21:06 | |
| *** kkoski has joined #openstack-ironic | 21:06 | |
| *** kkoski has quit IRC | 21:06 | |
| *** kkoski has joined #openstack-ironic | 21:10 | |
| *** jjohnson2_ has quit IRC | 21:10 | |
| *** dprince has quit IRC | 21:18 | |
| *** lucas-dinner has quit IRC | 21:22 | |
| krotscheck | devananda: Which was the session where I might give a demo? | 21:27 |
| krotscheck | Is that on thu/fri? | 21:28 |
| rloo | hi krotscheck, here's the etherpad. I don't know what/how much is set in stone yet: https://etherpad.openstack.org/p/liberty-ironic-design-summit-ideas | 21:31 |
| krotscheck | rloo: Thanks! | 21:31 |
| *** dguerri is now known as _dguerri | 21:33 | |
| mrda | Morning Ironic | 21:33 |
| *** _dguerri is now known as dguerri | 21:33 | |
| *** trown is now known as trown|outttypeww | 21:34 | |
| mrda | Haomeng: Could update bug 1446449 with your findings? (after your discussion with deva on the issue) Thanks. | 21:36 |
| openstack | bug 1446449 in OpenStack Compute (nova) "ironic hypervisor resource should be released for booting failed case" [Low,Triaged] https://launchpad.net/bugs/1446449 | 21:36 |
| *** dguerri is now known as _dguerri | 21:38 | |
| *** _dguerri is now known as dguerri | 21:38 | |
| *** kkoski has quit IRC | 21:45 | |
| *** igordcard has joined #openstack-ironic | 21:54 | |
| NobodyCam | morning mrda | 21:55 |
| NobodyCam | mrda: I hear tail devananda: it down on your side of the planet this week | 21:57 |
| jroll | "NobodyCam | mrda: I hear tail devananda..." stop looking at deva's tail, BadCub is gonna get jealous :P | 21:58 |
| NobodyCam | lol | 21:58 |
| BadCub | lol | 21:58 |
| mrda | I'm confused :-S | 21:59 |
| NobodyCam | hey hey mrda | 22:00 |
| * BadCub grumbles about tails and stomps back into his cave | 22:00 | |
| mrda | But good morning :) | 22:00 |
| jroll | mrda: what he meant was deva is in AUS right now :P | 22:00 |
| NobodyCam | lol | 22:00 |
| * mrda hides | 22:00 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Fixes some docstring warnings and upgrades Sphinx https://review.openstack.org/180339 | 22:00 |
| BadCub | lol | 22:01 |
| NobodyCam | yes, I don't know where thou | 22:01 |
| mrda | Do you know where he is in Australia? | 22:01 |
| mrda | What about that GPS tracker implant? | 22:01 |
| BadCub | I think Syndey. But don't quote me on that | 22:01 |
| *** penick has quit IRC | 22:01 | |
| mrda | If he's in Sydney that's good. There's aren't many dropbears in the city. | 22:01 |
| NobodyCam | oh we talked about htat last cycle and I forgot to have it implanted | 22:02 |
| mrda | (that's over 1200mi away from me btw :) | 22:02 |
| NobodyCam | lol I luv the dropbear (koala) pic: http://www.australiangeographic.com.au/news/2013/04/drop-bears-target-tourists,-study-says/ | 22:04 |
| BadCub | wow.. such an evil looking little cuddly thing | 22:04 |
| mrda | They're pretty dangerous unless you know what you're doing | 22:05 |
| mrda | and by doing I mean run | 22:05 |
| *** bnemec has quit IRC | 22:05 | |
| BadCub | shoot them with a tranq dart, cuddle them, then leave | 22:05 |
| mrda | I can send you vegemite if you plan on coming down under | 22:06 |
| *** penick has joined #openstack-ironic | 22:06 | |
| BadCub | no plans in the near future... At least that I am aware of yet. LOL. | 22:07 |
| mrda | I'm hoping for a future non-North American Summit to come here :) | 22:08 |
| jroll | BadCub: "incapacitate them, touch them, and leave." hm, seems.... questionable at best. | 22:08 |
| *** bnemec has joined #openstack-ironic | 22:08 | |
| BadCub | jroll: you question my methods? LOL | 22:08 |
| jroll | hehehe :P | 22:08 |
| BadCub | mrda: that would be cool | 22:08 |
| BadCub | :p | 22:09 |
| NobodyCam | mrda: better then dropgoats : http://viralcircus.com/amazing-moroccan-tree-climbing-goats/ | 22:09 |
| * BadCub wants a tree climbing goat | 22:10 | |
| BadCub | and a cat... and a roomba...... | 22:10 |
| mrda | lol | 22:10 |
| *** thrash is now known as thrash|g0ne | 22:11 | |
| BadCub | Wow.. I think I successfully upgraded HipChat out of existence.. *sighs* | 22:12 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Pluggable locking https://review.openstack.org/180342 | 22:18 |
| jroll | someone approve that pls :D | 22:18 |
| jroll | super easy read | 22:18 |
| NobodyCam | jroll: Data model could be impacted if other then dbapi is selected | 22:28 |
| *** jgrimm_ has joined #openstack-ironic | 22:28 | |
| jroll | NobodyCam: it doesn't change the data model, it just doesn't write to that column anymore | 22:28 |
| NobodyCam | but wow its like reading the code with a narrator | 22:29 |
| NobodyCam | oh very true! | 22:29 |
| jroll | which I guess, if you're looking for that data, changes it | 22:29 |
| *** jgrimm has quit IRC | 22:30 | |
| *** tteggel has quit IRC | 22:30 | |
| *** JayF has quit IRC | 22:30 | |
| *** rwsu_ has quit IRC | 22:30 | |
| *** morgabra has quit IRC | 22:30 | |
| *** ndipanov has quit IRC | 22:30 | |
| *** ndipanov has joined #openstack-ironic | 22:30 | |
| *** dmellado has quit IRC | 22:30 | |
| jroll | but with this spec, it just makes it pluggable, it doesn't add other options | 22:30 |
| *** morgabra has joined #openstack-ironic | 22:30 | |
| *** tteggel has joined #openstack-ironic | 22:30 | |
| *** JayF has joined #openstack-ironic | 22:30 | |
| *** rwsu_ has joined #openstack-ironic | 22:30 | |
| *** dmellado has joined #openstack-ironic | 22:31 | |
| NobodyCam | jroll: I like it! would you kill me if I asked for a Reference to the actual patch? its super nit like. | 22:33 |
| jroll | NobodyCam: what? | 22:33 |
| jroll | spec is supposed to come before patch | 22:33 |
| jroll | patch should ref the spec | 22:33 |
| jroll | right? | 22:33 |
| mrda | ya | 22:34 |
| jlvillal | jroll: +1 | 22:34 |
| *** subscope_ has quit IRC | 22:34 | |
| jroll | jlvillal: gerrit says -1 | 22:34 |
| NobodyCam | yes, but as you already the patch ( https://review.openstack.org/#/c/179962 ) I was thinking add it to hte ref section | 22:34 |
| NobodyCam | doh | 22:34 |
| jroll | are you really going to make me expand the term "DB" and "TTL" | 22:34 |
| jroll | when this is a spec for a distributed system | 22:35 |
| jlvillal | jroll: +1 on your spec before patch :) | 22:35 |
| NobodyCam | no I'm not | 22:35 |
| *** Sukhdev has joined #openstack-ironic | 22:36 | |
| *** romcheg has quit IRC | 22:36 | |
| NobodyCam | oh didn't see those | 22:36 |
| jlvillal | jroll: Up to you. I guess I prefer acronyms defined once in a document. But others may disagree. | 22:36 |
| jroll | :) | 22:36 |
| jroll | jlvillal: if someone reading this document needs those defined, I don't think they have enough context to review something like this | 22:37 |
| jroll | but that's just my opinion | 22:37 |
| jroll | super cynical opinion maybe | 22:37 |
| jlvillal | jroll: True that! I did have two other comments :) | 22:37 |
| jlvillal | Minor nits all of them. | 22:38 |
| jroll | yeah | 22:38 |
| NobodyCam | mrda: you have strange things down there. "KILLER ladybirds(ladybugs)" | 22:42 |
| mrda | NobodyCam: like I said, you should come visit :) | 22:43 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Pluggable locking https://review.openstack.org/180342 | 22:43 |
| NobodyCam | lol.. /me now will have to think twice | 22:43 |
| jroll | jlvillal: ^ | 22:43 |
| jlvillal | jroll: +1 to patch | 22:44 |
| *** penick has quit IRC | 22:45 | |
| NobodyCam | brb | 22:45 |
| *** alex_xu has quit IRC | 22:49 | |
| *** alex_xu has joined #openstack-ironic | 22:51 | |
| *** EmilienM|afk is now known as EmilienM | 22:52 | |
| *** penick has joined #openstack-ironic | 22:57 | |
| *** penick has quit IRC | 23:01 | |
| *** ijw has joined #openstack-ironic | 23:02 | |
| *** alex_xu has quit IRC | 23:10 | |
| *** alex_xu has joined #openstack-ironic | 23:11 | |
| *** penick has joined #openstack-ironic | 23:31 | |
| *** shakamunyi has quit IRC | 23:31 | |
| *** penick_ has joined #openstack-ironic | 23:42 | |
| *** penick has quit IRC | 23:42 | |
| *** penick_ is now known as penick | 23:42 | |
| comstud | jroll: i don't understand.. you don't like stuck locks? | 23:44 |
| * comstud trolls | 23:45 | |
| Madasi | just add a lockpick api command? | 23:47 |
| NobodyCam | comstud: I thought everyone luv'd the mysql cmd line interface, so stuck locks are just a excuse to use it | 23:48 |
| NobodyCam | Madasi: ++ :) | 23:48 |
| comstud | if you can't mysql, you shouldn't be running a service anyway | 23:49 |
| comstud | AM I RIGHT | 23:49 |
| NobodyCam | lol | 23:49 |
| NobodyCam | alright all i think it is about that time..The time when Diet Coke, magically turns in to RUM and Diet Coke... Have a good night All. | 23:56 |
| Madasi | that's a pretty good time of day | 23:56 |
| JayF | NobodyCam: calling that magic is pour form | 23:57 |
| JayF | :P | 23:57 |
| NobodyCam | lol!!!!! | 23:57 |
| BadCub | hmmm.... that sounds like a good plan.... Think I need to turn my Dr. Pepper into Vodka and Dr. Pepper..... | 23:57 |
| * BadCub scampers off to his cave and liqueur cabinet | 23:58 | |
| NobodyCam | lol | 23:58 |
| *** BadCub is now known as BadCub_Hiding | 23:58 | |
| Madasi | I can see I'll be hitting the whiskey tonight so I won't feel left out | 23:58 |
| BadCub_Hiding | Madasi: Cheers!!!!! | 23:58 |
| NobodyCam | Madasi: it's looking like it will be a FUN summit | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!