| *** naohirot has joined #openstack-ironic | 00:12 | |
| *** lazy_prince has quit IRC | 00:13 | |
| *** killer_prince has joined #openstack-ironic | 00:14 | |
| *** killer_prince is now known as lazy_prince | 00:14 | |
| *** lazy_prince has quit IRC | 00:21 | |
| *** killer_prince has joined #openstack-ironic | 00:22 | |
| *** killer_prince is now known as lazy_prince | 00:22 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add dhcp-all-interfaces to get IP to NIC other than eth0 https://review.openstack.org/158156 | 01:05 |
|---|---|---|
| *** kbs1 has quit IRC | 01:13 | |
| *** achanda has quit IRC | 01:21 | |
| *** kbyrne has quit IRC | 01:23 | |
| *** kbyrne has joined #openstack-ironic | 01:25 | |
| *** lazy_prince has quit IRC | 01:33 | |
| *** killer_prince has joined #openstack-ironic | 01:50 | |
| *** killer_prince is now known as lazy_prince | 01:50 | |
| *** lazy_prince has quit IRC | 02:05 | |
| *** killer_prince has joined #openstack-ironic | 02:07 | |
| *** killer_prince is now known as lazy_prince | 02:07 | |
| *** lazy_prince has quit IRC | 02:20 | |
| *** achanda has joined #openstack-ironic | 02:22 | |
| *** achanda has quit IRC | 02:27 | |
| *** achanda has joined #openstack-ironic | 02:27 | |
| *** killer_prince has joined #openstack-ironic | 02:33 | |
| *** killer_prince is now known as lazy_prince | 02:33 | |
| *** lazy_prince has quit IRC | 02:38 | |
| *** achanda has quit IRC | 02:38 | |
| *** killer_prince has joined #openstack-ironic | 02:40 | |
| *** killer_prince is now known as lazy_prince | 02:40 | |
| *** yuanying has quit IRC | 02:47 | |
| *** lazy_prince has quit IRC | 02:48 | |
| *** ramineni has joined #openstack-ironic | 02:49 | |
| *** killer_prince has joined #openstack-ironic | 02:53 | |
| *** killer_prince is now known as lazy_prince | 02:53 | |
| *** lazy_prince has quit IRC | 02:57 | |
| *** killer_prince has joined #openstack-ironic | 02:58 | |
| *** killer_prince is now known as lazy_prince | 02:58 | |
| *** lazy_prince has quit IRC | 03:05 | |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: Add Cleaning Operations for iLO drivers https://review.openstack.org/157715 | 03:06 |
| *** killer_prince has joined #openstack-ironic | 03:07 | |
| *** killer_prince is now known as lazy_prince | 03:07 | |
| *** oomichi_ has joined #openstack-ironic | 03:30 | |
| *** Marga_ has joined #openstack-ironic | 03:37 | |
| *** yuanying has joined #openstack-ironic | 03:38 | |
| *** saripurigopi has joined #openstack-ironic | 03:51 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: iLO implementation for hardware inspection https://review.openstack.org/151596 | 03:59 |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: Update Ilo drivers to use REST API interface to iLO https://review.openstack.org/154403 | 04:12 |
| *** rwsu-afk has quit IRC | 04:19 | |
| *** Marga_ has quit IRC | 04:25 | |
| *** Marga_ has joined #openstack-ironic | 04:25 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node https://review.openstack.org/155731 | 04:30 |
| *** rameshg87 has joined #openstack-ironic | 04:30 | |
| rameshg87 | good morning ironic | 04:32 |
| *** penick has quit IRC | 04:32 | |
| *** spandhe has quit IRC | 04:35 | |
| mrda | hey rameshg87 | 04:36 |
| rameshg87 | hello mrda | 04:40 |
| rameshg87 | mrda, what's the time over there ? lunch-time ? :) | 04:40 |
| *** saripurigopi has quit IRC | 04:42 | |
| *** saripurigopi has joined #openstack-ironic | 04:42 | |
| mrda | 3:15pm in the afternoon | 04:46 |
| mrda | and you? | 04:46 |
| *** spandhe has joined #openstack-ironic | 04:52 | |
| *** rameshg87_ has joined #openstack-ironic | 04:52 | |
| *** rameshg87 has quit IRC | 04:55 | |
| *** rameshg87_ has quit IRC | 04:57 | |
| *** pensu has joined #openstack-ironic | 04:59 | |
| *** pradipta has joined #openstack-ironic | 05:12 | |
| *** lazy_prince has quit IRC | 05:14 | |
| *** Marga_ has quit IRC | 05:16 | |
| *** lazy_prince has joined #openstack-ironic | 05:16 | |
| *** lazy_prince is now known as killer_prince | 05:19 | |
| *** killer_prince is now known as lazy_prince | 05:20 | |
| *** rameshg87 has joined #openstack-ironic | 05:23 | |
| *** rameshg87 has quit IRC | 05:24 | |
| *** rameshg87_ has joined #openstack-ironic | 05:24 | |
| *** rameshg87_ has quit IRC | 05:30 | |
| *** lazy_prince has quit IRC | 05:34 | |
| *** rameshg87 has joined #openstack-ironic | 05:34 | |
| *** achanda has joined #openstack-ironic | 05:35 | |
| *** rameshg87_ has joined #openstack-ironic | 05:36 | |
| rameshg87_ | mrda, it's 11am now | 05:38 |
| *** lazy_prince has joined #openstack-ironic | 05:39 | |
| *** rameshg87 has quit IRC | 05:40 | |
| *** lazy_prince is now known as killer_prince | 05:40 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Common changes for secure boot support https://review.openstack.org/153974 | 05:41 |
| *** killer_prince is now known as lazy_prince | 05:42 | |
| *** pcaruana has quit IRC | 05:45 | |
| *** lazy_prince has quit IRC | 05:45 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for pxe_ilo driver https://review.openstack.org/154808 | 05:47 |
| *** killer_prince has joined #openstack-ironic | 05:51 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for iscsi_ilo driver https://review.openstack.org/154814 | 05:52 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for agent_ilo driver https://review.openstack.org/154816 | 05:55 |
| *** killer_prince has quit IRC | 05:55 | |
| *** killer_prince has joined #openstack-ironic | 06:15 | |
| *** killer_prince is now known as lazy_prince | 06:15 | |
| *** lazy_prince has quit IRC | 06:19 | |
| *** lazy_prince has joined #openstack-ironic | 06:22 | |
| *** lazy_prince is now known as killer_prince | 06:24 | |
| *** achanda has quit IRC | 06:25 | |
| *** killer_prince is now known as lazy_prince | 06:27 | |
| *** coolsvap_ is now known as coolsvap | 06:27 | |
| *** achanda has joined #openstack-ironic | 06:56 | |
| *** jcoufal has joined #openstack-ironic | 06:57 | |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Update get_persistent_boot for UEFI https://review.openstack.org/158205 | 07:06 |
| rameshg87_ | naohirot, i just posted comment on your bug | 07:15 |
| *** rameshg87_ is now known as rameshg87 | 07:15 | |
| rameshg87 | naohirot, https://bugs.launchpad.net/ironic/+bug/1423809 | 07:16 |
| openstack | Launchpad bug 1423809 in Ironic "Error returned from deploy ramdisk: Could+not+find+disk+to+use" [Undecided,New] | 07:16 |
| rameshg87 | naohirot, i think it should be something like disk_device='cciss/c[0-9]d[0-9],sd[a-z],hd[a-z],vd[a-z]' | 07:16 |
| naohirot | rameshg87: Hi good morning | 07:16 |
| naohirot | rameshg87: I read that. which section should I add disk_device='cciss/c[0-9]d[0-9],sd[a-z],hd[a-z],vd[a-z]' in the config file? | 07:18 |
| ramineni | BadCub: hi | 07:20 |
| ramineni | BadCub: Could you please add "https://blueprints.launchpad.net/ironic/+spec/ilo-cleaning-support" , also to kilo3 | 07:22 |
| naohirot | rameshg87: I got it, [pxe] section. | 07:23 |
| rameshg87 | naohirot, yeah | 07:23 |
| rameshg87 | naohirot, does it make sense ? | 07:23 |
| rameshg87 | naohirot, this will require changes in dib first - to make the glob work | 07:24 |
| rameshg87 | naohirot, then change ironic to give the globs | 07:24 |
| naohirot | rameshg87: Yes, thanks. Okay, but disk_device='cciss/c0d0,sda,sdb,sdc,hda,vda' is still valuable :) | 07:25 |
| naohirot | rameshg87: Regarding CloudInit NoCloud you told me last week worked, thanks too. | 07:27 |
| rameshg87 | naohirot, yeah it should remain compatible with previous things | 07:27 |
| naohirot | rameshg87: And regarding virtual floppy, I have a few question. | 07:28 |
| rameshg87 | naohirot, good thing is glob syntax is compatible with this | 07:28 |
| rameshg87 | naohirot, great :) nocloud is so easy when testing only from Ironic | 07:28 |
| rameshg87 | naohirot, yeah please go ahead | 07:28 |
| naohirot | rameshg87: After getting the idea what is cloudinit, why didn't iLo virtual floppy follow the user_data format? | 07:30 |
| rameshg87 | naohirot, virtual floppy is used only for deploy | 07:30 |
| *** dlpartain has joined #openstack-ironic | 07:30 | |
| rameshg87 | naohirot, not while booting user images after the deploy | 07:31 |
| rameshg87 | naohirot, which booting user images after the deploy, ironic anyway supports configdrives which are written to disk | 07:31 |
| naohirot | rameshg87: I see, the fist question has done. | 07:31 |
| naohirot | rameshg87: second question is here https://github.com/openstack/diskimage-builder/blob/master/elements/ramdisk-base/extra-data.d/scripts/d/init-func#L22 | 07:32 |
| rameshg87 | naohirot, yeah | 07:32 |
| rameshg87 | naohirot, that is only for hp hardware right now | 07:33 |
| rameshg87 | naohirot, for irmc hardware, the model might be something else | 07:33 |
| naohirot | rameshg87: the contents of SYSFS_VMEDIA_DEVICE_MODEL is "Virtual Floppy0" in case of irmc. | 07:33 |
| rameshg87 | naohirot, since it is used in grep just below | 07:33 |
| rameshg87 | naohirot, you can add it as a grep pattern | 07:33 |
| naohirot | rameshg87: So I have to change the code little bit. | 07:33 |
| rameshg87 | naohirot, something like SYSFS_VMEDIA_DEVICE_MODEL="(virtual media|Virtual Floppy0)" | 07:34 |
| rameshg87 | naohirot, even better if you can pass this from Ironic | 07:34 |
| rameshg87 | naohirot, iLO can pass this as "virtual media" | 07:34 |
| rameshg87 | naohirot, irmc can pass this parameter as "Virtual Floppy0" | 07:34 |
| naohirot | rameshg87: Yeah, Is the grep actually egrep? | 07:34 |
| *** dlpartain has left #openstack-ironic | 07:34 | |
| rameshg87 | naohirot, ah yes, might change it to egrep as well | 07:36 |
| rameshg87 | :) | 07:36 |
| naohirot | rameshg87: I'm not sure the initrd environment, I have to try, but I asked you :) | 07:36 |
| *** pas-ha has joined #openstack-ironic | 07:36 | |
| naohirot | rameshg87: the third question is test environment | 07:37 |
| rameshg87 | naohirot, i think even basic grep has that or facility in regular expression | 07:37 |
| rameshg87 | naohirot, should work | 07:37 |
| rameshg87 | naohirot, but i prefer making it as a parameter which can be passed to the kernel command-line | 07:37 |
| rameshg87 | naohirot, may be keep "virtual media" as default if nothing is passed | 07:37 |
| rameshg87 | naohirot, yeah | 07:37 |
| rameshg87 | naohirot, and third question ? :) | 07:38 |
| naohirot | rameshg87: the second question has done thanks! | 07:38 |
| naohirot | rameshg87: I created a provider network using devstack, and run nova, neutron, keystone, etc. | 07:39 |
| naohirot | rameshg87: You said last week, you only run ironic and test | 07:39 |
| naohirot | rameshg87: How do you do that? | 07:39 |
| *** ukalifon1 has joined #openstack-ironic | 07:40 | |
| *** athomas has joined #openstack-ironic | 07:40 | |
| naohirot | rameshg87: through ironic python client? | 07:42 |
| *** achanda has quit IRC | 07:42 | |
| rameshg87 | naohirot, i have something like this in my cheetsheat :) | 07:43 |
| rameshg87 | naohirot, http://paste.openstack.org/show/180351/ | 07:43 |
| rameshg87 | naohirot, i use ironic python client to trigger the deploy | 07:44 |
| rameshg87 | naohirot, it seemed much easy for me and fast | 07:44 |
| *** dlpartain1 has joined #openstack-ironic | 07:46 | |
| naohirot | rameshg87: When I tried nocloud image, I passed user_data and meta_data through virtual cdrom by creating iso. | 07:46 |
| naohirot | rameshg87: what is the contents of ~/configdrive? | 07:47 |
| *** yog_ has joined #openstack-ironic | 07:47 | |
| *** achanda has joined #openstack-ironic | 07:48 | |
| rameshg87 | naohirot, cloud-localds has an option --filesystem | 07:48 |
| rameshg87 | naohirot, you can give it as vfat for it to work with ironic | 07:48 |
| naohirot | rameshg87: I noticed that instance_info/configdrive=$CONFIG_DRIVE | 07:49 |
| rameshg87 | naohirot, ~/configdrive has output of cloud-localds gzipped and base-64 encoded | 07:49 |
| naohirot | rameshg87: is $CONFIG_DRIVE vfat image of gizipped and base64? yes, I got. | 07:50 |
| *** dlpartain1 has left #openstack-ironic | 07:50 | |
| naohirot | rameshg87: but why do you need source like "source ~/configdrive | 07:51 |
| naohirot | "? | 07:51 |
| rameshg87 | naohirot, basically cloud-localds --filesystem vfat my-seed.img my-user-data | 07:51 |
| rameshg87 | naohirot, gzip my-seed.img | 07:51 |
| rameshg87 | naohirot, base64 --wrap 0 my-seed.img.gz | 07:52 |
| rameshg87 | naohirot, this will give the contents | 07:52 |
| naohirot | rameshg87: I still didn't get it. source needs text file right? | 07:52 |
| rameshg87 | naohirot, yeah it's something like this http://paste.openstack.org/show/180354/ | 07:52 |
| rameshg87 | naohirot, so you get it in a shell variable :) | 07:52 |
| naohirot | rameshg87: Aha, I got it :) | 07:53 |
| naohirot | rameshg87: Is this command "ironic node-set-provision-state $NODE active | 07:55 |
| naohirot | " for boot deploy iso? | 07:55 |
| naohirot | rameshg87: and does this command ironic node-set-provision-state $NODE deleted trigger to boot user image? | 07:56 |
| rameshg87 | naohirot, yeah | 07:57 |
| openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: ILO: Add support for GET_EMBEDDED_HEALTH https://review.openstack.org/158041 | 07:57 |
| rameshg87 | naohirot, exactly | 07:57 |
| naohirot | rameshg87: I have to check state diagram, but let me ask. | 07:57 |
| naohirot | rameshg87: Okay, thanks, I asked all questions :) | 07:58 |
| rameshg87 | naohirot, great :) | 07:58 |
| rameshg87 | naohirot, brb | 07:58 |
| *** rameshg87 is now known as rameshg87-lunch | 07:59 | |
| *** rameshg87-lunch has quit IRC | 08:10 | |
| *** rameshg87_ has joined #openstack-ironic | 08:10 | |
| *** saripurigopi_ has joined #openstack-ironic | 08:11 | |
| *** saripurigopi has quit IRC | 08:12 | |
| *** saripurigopi_ is now known as saripurigopi | 08:12 | |
| *** ifarkas has joined #openstack-ironic | 08:18 | |
| *** rameshg87_ is now known as rameshg87 | 08:21 | |
| *** achanda has quit IRC | 08:21 | |
| *** achanda has joined #openstack-ironic | 08:22 | |
| *** jcoufal has quit IRC | 08:22 | |
| *** jcoufal has joined #openstack-ironic | 08:23 | |
| rameshg87 | vdrok, hey | 08:23 |
| *** ndipanov has joined #openstack-ironic | 08:24 | |
| *** mgoddard has joined #openstack-ironic | 08:29 | |
| *** achanda has quit IRC | 08:32 | |
| *** dtantsur|afk is now known as dtantsur | 08:35 | |
| dtantsur | Morning! | 08:35 |
| *** erwan_taf has joined #openstack-ironic | 08:41 | |
| rameshg87 | dtantsur, o/ | 08:42 |
| dtantsur | o/ | 08:42 |
| rameshg87 | dtantsur, i remember you had said before someone wanted to use inband discovery with ilo drivers | 08:43 |
| rameshg87 | dtantsur, is there some provision for that ? i mean since we connect ilo discovery <-> ilo driver in code | 08:44 |
| rameshg87 | dtantsur, there wouldn't be provision, right ? | 08:44 |
| dtantsur | rameshg87, yep, with all the drivers. We have quite a few plugins to discoverd that we need. | 08:44 |
| dtantsur | rameshg87, I guess we'll end up with downstream ILO drivers replacing 'inspect = ' string, but I didn't research it yet | 08:45 |
| rameshg87 | dtantsur, okay .. | 08:45 |
| *** achanda has joined #openstack-ironic | 08:45 | |
| rameshg87 | dtantsur, and anyone working on virtual media stuff for booting discoverd ramdisk ? | 08:45 |
| dtantsur | rameshg87, nobody for now | 08:46 |
| rameshg87 | dtantsur, okay. i was just thinking we have so much of code in ironic for that | 08:46 |
| dtantsur | rameshg87, 2 important things you can't (to my best knowledge) discovery OOB: 1. which NIC system PXE boots from; 2. which NIC's are connected to which switch ports | 08:46 |
| dtantsur | the former being very important actually | 08:46 |
| rameshg87 | dtantsur, yeah | 08:47 |
| rameshg87 | dtantsur, why do we need to know which NIC system pxe boots | 08:47 |
| *** spandhe has quit IRC | 08:47 | |
| dtantsur | rameshg87, using virtual media is an interesting to research for discoverd but it does need support code in Ironic; will think about it in L cycle, I guess | 08:47 |
| dtantsur | rameshg87, because if you add ports for all NIC's, chances are high your deployment will fail | 08:47 |
| rameshg87 | dtantsur, any reason for that ? | 08:48 |
| dtantsur | because Nova chooses port randomly, and it may be not the port 1. you can PXE boot from; 2. that is on provision network | 08:48 |
| dtantsur | so until we have proper hints for Nova about networks and PXE booting ports discoverd creates only one port | 08:48 |
| rameshg87 | dtantsur, nova creates neutron port for all ironic ports, right ? | 08:49 |
| dtantsur | rameshg87, no, only for one random :) | 08:49 |
| dtantsur | that's the problem | 08:49 |
| dtantsur | but it's not so unreasonable - you have only one IP for a node, not N | 08:49 |
| dtantsur | (N being number of ports) | 08:49 |
| rameshg87 | dtantsur, but https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L949-L955 | 08:51 |
| dtantsur | that's not neutron ports | 08:51 |
| dtantsur | I can try to find a link for you, but you can well believe me, we had enough failed bare metal (not on vm) deployments due to this :) | 08:52 |
| rameshg87 | dtantsur, but it plugs vifs for all the ports in ironic, right ? | 08:52 |
| rameshg87 | dtantsur, okay :) | 08:52 |
| rameshg87 | anyway we started topic randomly | 08:52 |
| rameshg87 | dtantsur, and regarding virtual media | 08:53 |
| rameshg87 | dtantsur, if ironic can provide some apis (vendor passthru?) for attaching/detaching virtual media | 08:53 |
| rameshg87 | dtantsur, discoverd can use it, right ? | 08:53 |
| dtantsur | definitely | 08:53 |
| rameshg87 | we have most of the code for that in ironic, just need to expose it in an api | 08:54 |
| *** jistr has joined #openstack-ironic | 08:55 | |
| dtantsur | yeah | 08:55 |
| dtantsur | rameshg87, IIRC problem with ports is due to this [0]: https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L420-L421 | 08:55 |
| dtantsur | but I may be confusing the exact place, Nova code is not the easiest :) | 08:56 |
| * rameshg87 wonders how this code is connected with the virt driver | 08:56 | |
| rameshg87 | dtantsur, yeah me too..let me first figure out how it is connected to our virt driver | 08:57 |
| dtantsur | rameshg87, this should be called for any neutron-based deployment | 08:57 |
| rameshg87 | dtantsur, okay so if it is a problem | 08:58 |
| *** viktors|afk is now known as viktors | 08:58 | |
| rameshg87 | dtantsur, if we discover 8 nics using oob ilo discovery | 08:58 |
| rameshg87 | dtantsur, and create ironic ports for all of them | 08:59 |
| rameshg87 | dtantsur, and if nova boot has just 2 networks in it's command line | 08:59 |
| rameshg87 | dtantsur, we might pick any 2 ports in random | 09:00 |
| rameshg87 | dtantsur, in nova virt driver | 09:00 |
| dtantsur | yep | 09:00 |
| rameshg87 | dtantsur, oh and if we don't put the proper dhcp entries | 09:00 |
| dtantsur | (that's to my best understanding, obviously) | 09:00 |
| *** Nisha has joined #openstack-ironic | 09:00 | |
| rameshg87 | dtantsur, deploy ramdisk will fail getting an ip address | 09:00 |
| dtantsur | there will be no deploy ramdisk :) | 09:00 |
| Nisha | rameshg87, hi | 09:00 |
| rameshg87 | Nisha, ^^ | 09:00 |
| Nisha | dtantsur, rameshg87 can i gxet the | 09:01 |
| dtantsur | at least in our case the node was not able to PXE boot at all | 09:01 |
| Nisha | dtantsur, rameshg87 can i get the context? | 09:01 |
| rameshg87 | Nisha, dtantsur and myself we were just discussing on registering ports in discovery | 09:01 |
| * rameshg87 discusses with Nisha | 09:02 | |
| *** romcheg has joined #openstack-ironic | 09:10 | |
| rameshg87 | dtantsur, Nisha will just check what happens when you enroll all the ports and do an ilo driver's deploy | 09:12 |
| dtantsur | yep | 09:12 |
| dtantsur | that's of course depends on the lab setup | 09:12 |
| Nisha | :) dtantsur i will check and update | 09:12 |
| Nisha | dtantsur, meanwhile please review inspection patches | 09:13 |
| dtantsur | yeah | 09:14 |
| *** lucasagomes has joined #openstack-ironic | 09:14 | |
| *** foexle has joined #openstack-ironic | 09:14 | |
| *** subscope has joined #openstack-ironic | 09:15 | |
| oomichi_ | dtantsur: hi, | 09:15 |
| *** bauwser is now known as bauzas | 09:15 | |
| dtantsur | o/ | 09:15 |
| oomichi_ | dtantsur: I am interested in ironic-discoverd, and I have a question about it. | 09:15 |
| oomichi_ | dtantsur: why the implementation is different from ironic repo? | 09:15 |
| *** achanda has quit IRC | 09:16 | |
| *** pas-ha has quit IRC | 09:16 | |
| dtantsur | it's not different, it's separate. that's per Ironic team decision back in Juno, we were not sure that this idea will work at all, and we did know that the implementation is tricky | 09:16 |
| dtantsur | so I started it as a separate project | 09:16 |
| dtantsur | oomichi_, now we have plans to use it from within Ironic in K cycle: http://specs.openstack.org/openstack/ironic-specs/specs/kilo/inband-properties-discovery.html | 09:17 |
| oomichi_ | dtantsur: I got it, thanks a lot :) I will read the spec carefully. | 09:18 |
| dtantsur | np :) feel free to ping me if you have questions | 09:18 |
| oomichi_ | dtantsur: thanks again ;) | 09:18 |
| *** mgoddard has quit IRC | 09:26 | |
| *** MattMan has joined #openstack-ironic | 09:33 | |
| *** andreykurilin_ has joined #openstack-ironic | 09:34 | |
| *** oomichi_ has quit IRC | 09:34 | |
| *** pas-ha has joined #openstack-ironic | 09:41 | |
| *** manab has joined #openstack-ironic | 09:53 | |
| *** manab has quit IRC | 09:56 | |
| *** saripurigopi has quit IRC | 09:57 | |
| *** pas-ha has quit IRC | 10:00 | |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: ILO: Update get_persistent_boot for UEFI https://review.openstack.org/158205 | 10:01 |
| vdrok | morning ironic | 10:01 |
| vdrok | morning rameshg87 | 10:02 |
| rameshg87 | vdrok, hi | 10:02 |
| *** ParsectiX has joined #openstack-ironic | 10:02 | |
| rameshg87 | vdrok, i just wanted to say i am posting some comments on non-glance patch | 10:02 |
| rameshg87 | vdrok, i just thought you were about to raise a new patchset :) | 10:02 |
| vdrok | yeah, saw them ^) | 10:02 |
| rameshg87 | okay | 10:03 |
| vdrok | rameshg87, i will today :) | 10:03 |
| *** Nisha has quit IRC | 10:06 | |
| *** mgoddard has joined #openstack-ironic | 10:13 | |
| *** andreykurilin_ has quit IRC | 10:23 | |
| *** pas-ha has joined #openstack-ironic | 10:26 | |
| *** pas-ha has quit IRC | 10:30 | |
| vdrok | rameshg87, regarding your comment here - https://review.openstack.org/#/c/136741/23/ironic/drivers/modules/image_cache.py | 10:41 |
| vdrok | rameshg87, I thought about that | 10:41 |
| rameshg87 | vdrok, yeah | 10:41 |
| vdrok | rameshg87, but I don't know a good way to do it | 10:41 |
| vdrok | rameshg87, we should then somehow store filename - href mapping | 10:42 |
| vdrok | rameshg87, how do you think, should it be in db? or maybe additional file in cache with this mapping? | 10:42 |
| *** yog_ has quit IRC | 10:43 | |
| rameshg87 | vdrok, i thought the only problem was with the name of master_file_name | 10:43 |
| rameshg87 | vdrok, both the files http://abc.com/ubuntu.qcow2 and file:///ubuntu.qcow2 would be mapped to the same file in master_path, right ? | 10:43 |
| rameshg87 | vdrok, isn't that the problem ? | 10:44 |
| vdrok | rameshg87, yes | 10:44 |
| vdrok | rameshg87, but the file name is just the last part of it | 10:44 |
| rameshg87 | vdrok, if we just use hash(service_utils.parse_image_ref(href)) | 10:45 |
| rameshg87 | vdrok, where hash is some hashing function | 10:45 |
| rameshg87 | vdrok, i thought that could solve the problem | 10:45 |
| vdrok | rameshg87, use it instead of file name you mean? | 10:45 |
| rameshg87 | vdrok, yes | 10:45 |
| rameshg87 | vdrok, we would always do the same hash and then check everytime later before downloading the same file again | 10:45 |
| vdrok | rameshg87, yeah got it, thanks :) | 10:46 |
| vdrok | rameshg87, will change | 10:46 |
| rameshg87 | vdrok, yeah i was thinking some sha hashing functions should take care | 10:47 |
| rameshg87 | vdrok, so that we always get a unique name for a unique href | 10:47 |
| rameshg87 | vdrok, even glance uuid's could be hashed again using sha | 10:48 |
| vdrok | rameshg87, I think even plain hash() function will do the thing as strings are immutable | 10:48 |
| rameshg87 | vdrok, yeah anything would do | 10:48 |
| rameshg87 | :) | 10:48 |
| *** kbyrne has quit IRC | 10:49 | |
| *** kbyrne has joined #openstack-ironic | 10:49 | |
| *** Marga_ has joined #openstack-ironic | 10:56 | |
| *** ramineni has quit IRC | 11:00 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Allow capabilities for scheduling from Nova https://review.openstack.org/158243 | 11:01 |
| dtantsur | dear god, one more spec... | 11:02 |
| dtantsur | rameshg87, there was some spec touching capabilities by one of J* and I guess we postponed everything to L | 11:02 |
| rameshg87 | dtantsur, ah yes .. | 11:03 |
| rameshg87 | dtantsur, but this is not requesting anything new | 11:03 |
| rameshg87 | dtantsur, just kind of getting into an agreement | 11:03 |
| rameshg87 | dtantsur, for Kilo | 11:03 |
| dtantsur | ah ok | 11:03 |
| dtantsur | please make it clear in the title, otherwise people are gonna -2 it even before reading :D | 11:03 |
| rameshg87 | dtantsur, "This patch is more of a proposal of how to make use of the | 11:03 |
| rameshg87 | capabilities for node scheduling for Kilo." | 11:03 |
| rameshg87 | dtantsur, commit msg ? | 11:04 |
| dtantsur | rameshg87, yeah, commit msg ~= title btw | 11:04 |
| rameshg87 | :) | 11:05 |
| *** Marga_ has quit IRC | 11:06 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Allow capabilities for scheduling from Nova https://review.openstack.org/158243 | 11:07 |
| rameshg87 | dtantsur, i am really wondering if i need even a shared-lock here: https://review.openstack.org/#/c/155230/12/ironic/conductor/manager.py | 11:20 |
| rameshg87 | dtantsur, comment on L1379 | 11:20 |
| dtantsur | I don't know. I remember on the summit we decided not to overdo it with exclusive locks, and this one sounds like a valid use case | 11:21 |
| dtantsur | ifarkas, hey, could you review 3 small discoverd patches? https://review.openstack.org/#/projects/stackforge/ironic-discoverd,dashboards/important-changes:review-inbox-dashboard | 11:21 |
| rameshg87 | dtantsur, yeah so then i do a direct update here: https://review.openstack.org/#/c/155230/12/ironic/common/raid.py L169 | 11:22 |
| rameshg87 | dtantsur, both are basically very-similar things | 11:22 |
| ifarkas | dtantsur, sure, will do that! | 11:23 |
| rameshg87 | dtantsur, unless the caller of update_raid_info has already acquired the lock | 11:23 |
| rameshg87 | dtantsur, without a sort of lock | 11:24 |
| dtantsur | and what's the problem? | 11:24 |
| rameshg87 | dtantsur, driver_internal_info = node.driver_internal_info; <edit driver_internal_info>; node.driver_internal_info = driver_internal_info | 11:25 |
| rameshg87 | dtantsur, isn't this a basic problem in concurrency ? | 11:25 |
| dtantsur | well yeah, it's not very atomic... | 11:25 |
| dtantsur | rameshg87, feel free to leave lock for now, I'm not really sure how we should solve all this | 11:26 |
| rameshg87 | dtantsur, what if thread switches in <edit driver_internal_info> and then some other thread modifies it | 11:26 |
| rameshg87 | dtantsur, yeah | 11:26 |
| rameshg87 | dtantsur, may be we need a way for atomically updating node properties in database without locks | 11:26 |
| dtantsur | it means rewriting the whole objects module, I guess | 11:27 |
| rameshg87 | dtantsur, may be yes | 11:28 |
| openstackgerrit | Victor Sergeyev proposed openstack/ironic: Use functions from oslo.utils https://review.openstack.org/158254 | 11:31 |
| *** faizan has joined #openstack-ironic | 11:36 | |
| *** pensu has left #openstack-ironic | 11:40 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add driver interface for RAID configuration https://review.openstack.org/155230 | 11:42 |
| openstackgerrit | Victor Sergeyev proposed openstack/ironic: Remove unused modules from ironic/openstack/common https://review.openstack.org/158264 | 11:49 |
| *** pradipta has quit IRC | 11:54 | |
| thelexx | dtantsur: rameshg87: PaulMurray has discussed this a little with dansmith, adding a way for objects to ask their children if they've changed | 11:55 |
| *** thelexx is now known as lxsli | 11:56 | |
| dtantsur | oh nice | 11:56 |
| lxsli | Still needs work... PaulMurray is gassing in the hallway atm or I'd chase it | 11:56 |
| lxsli | I think dansmith is open to the idea but the implications need to be chased through | 11:56 |
| openstackgerrit | Alexis Lee proposed openstack/ironic: Don't write PXE config during ACTIVE takeover https://review.openstack.org/156250 | 11:58 |
| *** dlpartain has joined #openstack-ironic | 11:58 | |
| *** dlpartain has left #openstack-ironic | 12:04 | |
| rameshg87 | lxsli, any links or references to discussions ? | 12:05 |
| *** jcoufal has quit IRC | 12:10 | |
| *** jcoufal_ has joined #openstack-ironic | 12:10 | |
| *** yog_ has joined #openstack-ironic | 12:14 | |
| *** lucasagomes is now known as lucas-hungry | 12:24 | |
| *** rameshg87 has quit IRC | 12:27 | |
| *** erwan_taf has quit IRC | 12:44 | |
| *** derekh has joined #openstack-ironic | 12:45 | |
| *** faizan has quit IRC | 12:48 | |
| *** zigo has quit IRC | 12:52 | |
| openstackgerrit | Merged stackforge/ironic-discoverd: Update functest to new ramdisk code https://review.openstack.org/156179 | 12:56 |
| *** zigo has joined #openstack-ironic | 12:56 | |
| *** pelix1 has joined #openstack-ironic | 12:59 | |
| *** dprince has joined #openstack-ironic | 13:00 | |
| *** yarker has quit IRC | 13:06 | |
| *** yarker has joined #openstack-ironic | 13:09 | |
| *** ParsectiX has quit IRC | 13:10 | |
| *** ParsectiX has joined #openstack-ironic | 13:16 | |
| *** lsmola has quit IRC | 13:23 | |
| *** lucas-hungry is now known as lucasagomes | 13:23 | |
| *** yog_ has quit IRC | 13:23 | |
| *** lsmola has joined #openstack-ironic | 13:27 | |
| *** Nisha has joined #openstack-ironic | 13:32 | |
| lxsli | dtantsur: rameshg87: turns out this has merged! https://review.openstack.org/#/c/66627 | 13:32 |
| dtantsur | good news! we need to try it | 13:33 |
| dtantsur | (after backporting to our objects) | 13:33 |
| lxsli | There is a gotcha, if you modify a ListOfObjects (EG add an object) then you need to assign it back still | 13:33 |
| *** lazy_prince is now known as killer_prince | 13:39 | |
| *** yog_ has joined #openstack-ironic | 13:42 | |
| *** jjohnson2 has joined #openstack-ironic | 13:42 | |
| *** Guest30952 has joined #openstack-ironic | 13:48 | |
| *** lucasagomes has quit IRC | 13:50 | |
| dtantsur | Nisha, how came you had vif assigned after inspection? Is it an effect of inspection? | 13:53 |
| Nisha | dtantsur, no | 13:53 |
| Nisha | dtantsur, vif is not assigned after inspection | 13:53 |
| *** lucasagomes has joined #openstack-ironic | 13:53 | |
| Nisha | but to contact BM itself u need to have vif on the ironic port correct? | 13:54 |
| dtantsur | Nisha, no, only Ironic port itself | 13:54 |
| dtantsur | Nisha, usually VIF is assigned by Nova during deploy | 13:54 |
| Nisha | so its before inspection i have to assign one ironic port with its corresponding vif port id | 13:54 |
| Nisha | but when we do it from ironic we do it , correct? | 13:55 |
| Nisha | As i understand nova will not be able to honor inspection | 13:55 |
| dtantsur | Nisha, it's usually done automatically | 13:55 |
| *** rloo has joined #openstack-ironic | 13:55 | |
| dtantsur | sorry, I don't understand. Nova has no idea about inspection, neither does Ironic deploy driver | 13:55 |
| Nisha | dtantsur, yes. so to start inspection, the node has to be moved to manage state | 13:56 |
| Nisha | As i understand nova cannot schedule node from manageable state for any action | 13:56 |
| Nisha | may be i am wrong | 13:57 |
| Nisha | Ironic deploy driver needs the node to be in available state same as nova | 13:57 |
| lucasagomes | Nisha, right, nova will only schedule nodes if they are in the AVAILABLE state | 13:57 |
| *** yog_ has quit IRC | 13:58 | |
| *** pas-ha has joined #openstack-ironic | 13:58 | |
| Nisha | so all other actions need to be done at ironic shell level | 13:58 |
| Nisha | lucasagomes, am i correct? | 13:58 |
| dtantsur | Nisha, so you have to advance node MANAGEABLE -> AVAILABLE and it's going to be available to Nova | 13:58 |
| dtantsur | not sure how VIFs are related here... | 13:59 |
| Nisha | Yes dtantsur | 13:59 |
| lucasagomes | Nisha, yea mostly (IIRC rescue can also be invoked from nova) | 13:59 |
| Nisha | for OOB deploy/inspect thru ironic shell needs ironic port to have neutron port id as vif port id for conductor to contact BM | 14:00 |
| Nisha | when the same is done through nova, we dont need to do so | 14:00 |
| dtantsur | oh, so it's specific for your drivers? | 14:00 |
| dtantsur | or your want to deploy without Nova? | 14:00 |
| Nisha | dtantsur, i dont think it is specific to ilo drivers. | 14:01 |
| Nisha | dtantsur, no i dont want to deploy without nova | 14:01 |
| dtantsur | Nisha, then you don't have to bother about VIFs | 14:01 |
| lucasagomes | Nisha, why you need the neutron port and/or vif to talk to ur BMC? | 14:01 |
| lucasagomes | OOB you're going to interrogate the BMC to get information about the physical characteristics of a node, I don't see the neutron port being used for it | 14:02 |
| lucasagomes | Or you mean in-band? Where a ramdisk will be loaded on the node? | 14:03 |
| lucasagomes | ironic-discoverd run it's on DHCP server, so it doesn't depend on neutron to PXE boot a machine (if that's what you meant) | 14:03 |
| *** dlpartain has joined #openstack-ironic | 14:05 | |
| *** dtantsur is now known as dtantsur|brb | 14:06 | |
| Nisha | lucasagomes, yes even ilo drivers depend on dhcp server | 14:08 |
| lucasagomes | sure, for deployment the pxe drivers will depend on that | 14:08 |
| lucasagomes | Nisha, I just don't get the OOB inspect depending on neutron here | 14:09 |
| Nisha | https://github.com/openstack/ironic/blob/2c0a090bd5cc30c094a3ea3e679ff7bf24495a05/ironic/drivers/modules/ilo/deploy.py#L270 | 14:10 |
| lucasagomes | Nisha, ok, that's deploy | 14:11 |
| lucasagomes | deploy we are going to boot a ramdisk on the machine | 14:11 |
| Nisha | yes, | 14:11 |
| lucasagomes | but OOB inspection we won't | 14:11 |
| *** mjturek1 has joined #openstack-ironic | 14:12 | |
| Nisha | oh i kept the same configuration. let me try if that works without vif port id in ironic port... | 14:12 |
| lucasagomes | right | 14:12 |
| * lucasagomes is a bit confused | 14:12 | |
| Nisha | lucasagomes, thanks i will try and see if inspection works without this configurations | 14:13 |
| *** Guest28285 is now known as mgagne | 14:13 | |
| *** mgagne has joined #openstack-ironic | 14:13 | |
| lucasagomes | Nisha, ok | 14:13 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Return required properties for agent deploy driver https://review.openstack.org/158313 | 14:15 |
| *** yog_ has joined #openstack-ironic | 14:18 | |
| *** pas-ha has quit IRC | 14:27 | |
| *** moorryan has quit IRC | 14:29 | |
| *** Nisha has quit IRC | 14:32 | |
| *** dlpartain has left #openstack-ironic | 14:36 | |
| *** jcoufal_ has quit IRC | 14:38 | |
| *** jcoufal has joined #openstack-ironic | 14:38 | |
| *** Guest30952 is now known as annegentle | 14:42 | |
| *** beekneemech is now known as bnemec | 14:48 | |
| *** dtantsur|brb is now known as dtantsur | 14:57 | |
| *** PaulCzar has joined #openstack-ironic | 15:03 | |
| jroll | morning ironic :) | 15:08 |
| NobodyCam | Mroning Ironicers, says the man make-n coffee | 15:08 |
| jroll | hiya NobodyCam | 15:09 |
| NobodyCam | morning jroll :) | 15:09 |
| TheJulia | good morning | 15:09 |
| NobodyCam | morning TheJulia :) nice to home again? | 15:09 |
| jroll | morning TheJulia :) | 15:10 |
| dtantsur | morning folks :) | 15:10 |
| NobodyCam | morning dtantsur :) | 15:12 |
| rloo | morning jroll, NobodyCam, TheJulia, dtantsur, lucasagomes and everyone else :) | 15:14 |
| lucasagomes | rloo, NobodyCam jroll TheJulia good morning | 15:15 |
| jroll | morning rloo and lucasagomes :) | 15:15 |
| rloo | dtantsur: qq. Why is this bug high, if changing the config value can fix it? https://bugs.launchpad.net/ironic/+bug/1423809 | 15:16 |
| openstack | Launchpad bug 1423809 in Ironic "Error returned from deploy ramdisk: Could+not+find+disk+to+use" [High,Confirmed] | 15:16 |
| dtantsur | rloo, because it breaks deployments on the whole group of hardware. the config option is reason why it's not critical :) | 15:17 |
| dtantsur | it's not so easy to figure out that you need to modify some config option, so I would not call it an easy workaround | 15:17 |
| rloo | dtantsur: oh, I thought high was if there was no workaround | 15:17 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Disable n-novnc, heat, cinder and horizon on devstack https://review.openstack.org/158333 | 15:17 |
| jroll | presumably if you're deploying these machines you know that the disk is on /dev/sdc | 15:18 |
| dtantsur | well... it's still subjective IMO :) | 15:18 |
| jroll | so we need more discoverable docs on this stuff | 15:18 |
| jroll | sure | 15:18 |
| rloo | dtantsur: if something doesn't work, it isn't always obvious how to fix it regardless. | 15:18 |
| rloo | dtantsur: yeah, what jroll said. | 15:18 |
| jroll | rloo: openstack. | 15:18 |
| jroll | or "the story of openstack", or something | 15:18 |
| rloo | jroll: it isn't just openstack, is it? | 15:19 |
| rloo | jroll: to be fair. and it is monday and i'm starting with an optimistic view of things. wait til the end of the week ;) | 15:19 |
| jroll | heh | 15:21 |
| rloo | dtantsur: also, the reporter of that bug knew what a temporary fix was, so it wasn't that hard for them to figure it out. | 15:22 |
| dtantsur | rloo, for naohirot - yes, for mere use maybe not | 15:22 |
| dtantsur | I don't insist though, feel free to change to Medium :) | 15:23 |
| NobodyCam | morning rloo :) | 15:23 |
| *** BadCub_ has joined #openstack-ironic | 15:27 | |
| BadCub_ | Morning Ironic | 15:27 |
| jroll | morning BadCub_ :) | 15:27 |
| BadCub_ | morning jroll | 15:32 |
| jlvillal|afk | Good morning: NobodyCam BadCub jroll dtantsur rloo and anyone else :) | 15:32 |
| jroll | heya jlvillal|afk :) | 15:32 |
| *** jlvillal|afk is now known as jlvillal | 15:32 | |
| NobodyCam | morning jlvillal|afk :) | 15:33 |
| jlvillal | So who's OpenStack presentations should I be voting for? :) | 15:33 |
| rloo | morning jlvillal | 15:33 |
| jroll | \o | 15:33 |
| jroll | I need to do that too | 15:33 |
| jroll | dang | 15:33 |
| NobodyCam | jlvillal: the ones you'd like to see | 15:34 |
| *** achanda has joined #openstack-ironic | 15:34 | |
| jlvillal | NobodyCam: Darn, I have to make a decision ;) I did vote for devananda's | 15:34 |
| jroll | wow, there's *so many* deployment presentations | 15:34 |
| jroll | jlvillal: shameless plug https://www.openstack.org/vote-vancouver/presentation/deploying-openstack-services-without-worrying | 15:35 |
| jlvillal | I'm interested in using Ironic/OpenStack to deploy Docker containers. That idea sounds interesting to me. | 15:36 |
| *** zigo_ has joined #openstack-ironic | 15:36 | |
| jroll | there's a big thing to make nova deploy containers, I tend to think it's kind of silly | 15:36 |
| jroll | I'd rather just spin up a bunch of coreos instances and run fleet on top | 15:37 |
| *** alexpilotti has joined #openstack-ironic | 15:37 | |
| devananda | g'morning, all | 15:40 |
| NobodyCam | morning devananda :) | 15:40 |
| jlvillal | jroll: True. I can see the benefit of having one system to do both VMs and containers. Not sure I agree with running the containers in VMs though. Well not always. We were doing Android builds here and the load average would go above 300. Not sure we would want that on a VM. | 15:40 |
| jroll | heya devananda :) | 15:40 |
| jroll | jlvillal: ya, well, those coreos instances could be bare metal, too :) | 15:41 |
| jlvillal | jroll: Yep. But why I was thinking Ironic and then containers. Ironic deploys CoreOS? | 15:41 |
| jroll | ironic deploys whatever you want! | 15:42 |
| jroll | we have coreos images in onmetal | 15:42 |
| NobodyCam | on this one might be fun: https://www.openstack.org/vote-vancouver/presentation/installers-are-dead-deploying-our-bits-is-a-continuous-process | 15:42 |
| lucasagomes | devananda, morning | 15:43 |
| NobodyCam | moorning lucasagomes :) | 15:44 |
| lucasagomes | NobodyCam, hi there, good morning | 15:44 |
| NobodyCam | (think I missed ya ealier) | 15:44 |
| NobodyCam | :-p | 15:44 |
| lucasagomes | oh what's up? | 15:44 |
| NobodyCam | nothing | 15:44 |
| *** achanda has quit IRC | 15:44 | |
| NobodyCam | :-p just missed saying G'morning | 15:44 |
| lucasagomes | ahh gotcha :D | 15:45 |
| jroll | NobodyCam: whoa, red hat saying installers are dead? | 15:45 |
| NobodyCam | ??? guess we got vote and attemd to find out | 15:45 |
| NobodyCam | attend even | 15:45 |
| devananda | jroll: interesting times, eh? | 15:45 |
| jroll | devananda: I'm just happy more people are seeing the light | 15:46 |
| lucasagomes | :D | 15:46 |
| *** ChuckC has joined #openstack-ironic | 15:49 | |
| *** ChuckC_ has quit IRC | 15:52 | |
| devananda | apparently, ubuntu has issues with >200 DPI resolution displays | 15:53 |
| devananda | and by issues i mean both chrome and firefox do not render pages in a readable way | 15:53 |
| lucasagomes | ew | 15:54 |
| dtantsur | devananda, hi! Is it expected that ironic node-create now creates node is 'available' state, not 'manageable'? | 16:00 |
| devananda | dtantsur: changing that will affect users who expect the current behavior. iow, it will break any automation scripts | 16:01 |
| lucasagomes | oh | 16:02 |
| devananda | dtantsur: once we have the whole enroll->(validate)->manageable->(inspect/zap) chain done, I'd like to make the default initial state be enrolled | 16:02 |
| dtantsur | devananda, yeah... but are we going to default to MANAGEABLE if appropriate micro version is set? otherwise MANAGEABLE makes no sense | 16:02 |
| dtantsur | ah, I see | 16:02 |
| dtantsur | for now to test inspection we need to go back to MANAGEABLE, inspect, then go forward to AVAILABLE :) sounds weird.. | 16:03 |
| *** ndipanov has quit IRC | 16:04 | |
| devananda | dtantsur: yea, that is odd. in either case, I think the client should be able to specify the state when creating the node | 16:04 |
| devananda | dtantsur: that should solve your current issue | 16:05 |
| dtantsur | makes sense | 16:05 |
| devananda | without breaking backwards compat | 16:05 |
| dtantsur | there's a patch for microversions in client: https://review.openstack.org/155624 | 16:05 |
| dtantsur | devananda, would be nice if you have a look ^^^ (if you didn't already) | 16:06 |
| dtantsur | The requested action \\\"manageable\\\" can not be performed on node \\\"59fd3b32-8fb0-4a28-a8e8-fb35dfa89704\\\" while it is in state \\\"available\\\ | 16:10 |
| dtantsur | hmmmmmmm | 16:10 |
| jroll | dtantsur: "manage" | 16:11 |
| dtantsur | thanks, yeah | 16:11 |
| *** anderbubble has joined #openstack-ironic | 16:14 | |
| *** jcoufal has quit IRC | 16:15 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: iLO implementation for hardware inspection https://review.openstack.org/151596 | 16:16 |
| openstackgerrit | Merged stackforge/ironic-discoverd: Use less verbose imports for i18n https://review.openstack.org/157419 | 16:19 |
| *** devlaps has joined #openstack-ironic | 16:21 | |
| openstackgerrit | Merged stackforge/ironic-discoverd: Always use non-localized strings in unit tests https://review.openstack.org/157422 | 16:21 |
| openstackgerrit | Merged stackforge/ironic-discoverd: Change i18n domain to match what is generated in oslo-incubator sync https://review.openstack.org/157451 | 16:23 |
| *** andreykurilin_ has joined #openstack-ironic | 16:25 | |
| *** ukalifon1 has quit IRC | 16:27 | |
| *** stendulker has joined #openstack-ironic | 16:30 | |
| *** Nisha has joined #openstack-ironic | 16:34 | |
| NobodyCam | gah! some how I thought the meeting was tonight and not this morning. I was wrong.. Just checking to see if the status reports are up to date... and requestion any last minute issues for the Agenda? | 16:34 |
| jroll | NobodyCam: we should go over k3 priorities | 16:36 |
| NobodyCam | ++ | 16:36 |
| rloo | NobodyCam: feature proposal freeze is March 5. Do we want to prioritize any specs, to get them approved? | 16:39 |
| *** killer_prince is now known as lazy_prince | 16:40 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: INSPECTFAIL value is more readable https://review.openstack.org/158351 | 16:42 |
| dtantsur | rloo, I would even say: let's see if we approve more specs | 16:42 |
| *** devlaps has quit IRC | 16:42 | |
| NobodyCam | there are a couple on the agenda by others :) | 16:42 |
| rloo | dtantsur: that too but realistically, how many more specs can/should we approve, wrt time it takes to get them approved and then code reviews following that. | 16:42 |
| * dtantsur updates bug stats | 16:43 | |
| rloo | NobodyCam: so the authors of the specs would presumably want their specs approved. If there are any specs that "we" think we need (eg the states one was needed) | 16:44 |
| NobodyCam | rloo: jroll: updated agenda | 16:44 |
| viktors | devananda: around? | 16:45 |
| jroll | NobodyCam: thanks :) | 16:46 |
| rloo | thx NobodyCam | 16:46 |
| viktors | devananda: I've answered to your notes in patch https://review.openstack.org/#/c/156192/ (Run tests in py34 environment). Please look there one more time. Thanks! | 16:46 |
| *** devlaps has joined #openstack-ironic | 16:48 | |
| NobodyCam | ten minutes til meeting.. last chance for coffee and what nots :-p | 16:49 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Generic changes for Node Inspection https://review.openstack.org/149823 | 16:51 |
| dtantsur | coffee yeah | 16:51 |
| devananda | viktors: hi! it fails for me locally with this error | 16:54 |
| devananda | urgh ... tox decided to rebuild ... gimme a few to recreate the error | 16:55 |
| dtantsur | tox is nice, it likes rebuilding :D | 16:55 |
| viktors | devananda: sure | 16:55 |
| devananda | rloo: like non-glance image refs | 16:56 |
| devananda | oh. you're talking about specs, not code. /me drinks more coffee | 16:57 |
| jlvillal | Nisha: Hi there :) | 16:57 |
| jlvillal | Not sure if you saw my upload. | 16:57 |
| Nisha | jlvillal, yes | 16:57 |
| *** ChuckC_ has joined #openstack-ironic | 16:57 | |
| Nisha | jlvillal, link | 16:57 |
| rloo | devananda: well, jroll was talking about code ;) | 16:57 |
| jlvillal | Nisha: https://review.openstack.org/151596 | 16:57 |
| Nisha | jlvillal, today i wasnt so active | 16:57 |
| NobodyCam | two minutes | 16:58 |
| jlvillal | One thing my pylint check noticed (happens when I save a file in vim). Was the issue with the error message that I did a comment on. | 16:58 |
| jlvillal | flake8 doesn't catch it :( | 16:58 |
| jroll | what was I talking about now | 16:58 |
| *** ChuckC_ has quit IRC | 16:58 | |
| rloo | jroll: k3 priorities | 16:59 |
| dtantsur | jlvillal, hacking (not flake8) is sometimes more opinionated | 16:59 |
| Nisha | jlvillal, i found the issue with that and was to upload the new patch. Anyway its fine , what all have you addressed in the new patch? | 17:00 |
| jroll | rloo: I'm talking about specs and code, really. more generally, what are our priorities for reviews for the next few weeks? | 17:00 |
| * devananda heads to the meeting room | 17:00 | |
| jlvillal | dtantsur: It was a thing where it doesn't tell you that a statement in the form of string does nothing. | 17:00 |
| *** ChuckC has quit IRC | 17:00 | |
| rloo | jroll: gotcha | 17:00 |
| *** ChuckC_ has joined #openstack-ironic | 17:00 | |
| *** faizan has joined #openstack-ironic | 17:00 | |
| *** foexle has quit IRC | 17:01 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Support for non-Glance image references https://review.openstack.org/136741 | 17:01 |
| dank_ | Devananda: don't hurry they are running late | 17:01 |
| *** rameshg87 has joined #openstack-ironic | 17:01 | |
| *** viktors is now known as viktors|afk | 17:02 | |
| *** vdrok is now known as vdrok_afk | 17:05 | |
| *** rwsu has joined #openstack-ironic | 17:05 | |
| jlvillal | Nisha: I had fixed the message text issue and then used sets. I think if you do a diff you can see the changes. | 17:07 |
| Nisha | jlvillal, i will need to test how the changes work | 17:07 |
| Nisha | yes i saw | 17:07 |
| jlvillal | Nisha: Okay. 'tox' passed for me here | 17:08 |
| Nisha | yes, but i should get the desired output also | 17:08 |
| *** eghobo has joined #openstack-ironic | 17:08 | |
| jlvillal | Nisha: But it is your patch. So I won't be offended if you ignore them :) | 17:08 |
| Nisha | so i will just test and update you | 17:08 |
| Nisha | jlvillal, it is all our united work | 17:08 |
| Nisha | and i should thank you for your comments. am not so expert in python | 17:08 |
| jlvillal | Nisha: One thought on the unittest. Maybe check the message text that is expected. | 17:09 |
| NobodyCam | lucasagomes: are you joining the meeting?? | 17:09 |
| lucasagomes | NobodyCam, oh sh*t forgot | 17:09 |
| lucasagomes | yes | 17:09 |
| NobodyCam | :-p | 17:09 |
| lucasagomes | NobodyCam, thanks :) | 17:09 |
| NobodyCam | :) | 17:09 |
| *** eghobo has quit IRC | 17:16 | |
| *** kbs has joined #openstack-ironic | 17:20 | |
| jroll | 17:21:28 lucasagomes | jroll, but also we are making glance optional by only requiring a http url | 17:21 |
| jroll | this is also trivial in the agent :P | 17:21 |
| lucasagomes | +1 | 17:22 |
| lucasagomes | yeah | 17:22 |
| lucasagomes | the v3 trust could be used to fetch the image from glance too AFAIUI | 17:22 |
| lucasagomes | but will depend on the keystone v3, so I think iscsi will continue for a good while | 17:23 |
| jroll | yeah, idk much about trusts at this point | 17:23 |
| jroll | also seems deva has another reason to keep iscsi deploys | 17:23 |
| lucasagomes | right, memory related I bet | 17:23 |
| lucasagomes | cause IPA you gotta convert the image in the ramdisk right? | 17:23 |
| lucasagomes | in gate it makes it a bit worse :/ | 17:23 |
| JoshNang | relevant (very large) patch to drop swfit dependency in IPA: https://review.openstack.org/#/c/136741/ | 17:24 |
| lucasagomes | but that said, I think that fetching in the ramdisk is way more scalable | 17:24 |
| openstackgerrit | Joshua Harlow proposed openstack/ironic: Update the states generator and regenerate the image https://review.openstack.org/158102 | 17:24 |
| jroll | lucasagomes: well, we want to fix that | 17:24 |
| jroll | to just stream it | 17:24 |
| lucasagomes | JoshNang, nice will look at it | 17:24 |
| lucasagomes | jroll, oh nice one | 17:24 |
| jroll | it's just not high priority | 17:24 |
| *** stendulker_ has joined #openstack-ironic | 17:26 | |
| lucasagomes | I see, but sounds like a pretty good idea :) | 17:26 |
| jroll | indeed | 17:26 |
| jroll | comstud started on it, nobody picked it up | 17:26 |
| jroll | or more started on making image writing pluggable https://review.openstack.org/#/c/128701/ | 17:27 |
| jroll | and vhd support | 17:27 |
| *** jistr has quit IRC | 17:29 | |
| *** stendulker has quit IRC | 17:29 | |
| *** Marga_ has joined #openstack-ironic | 17:34 | |
| *** Marga_ has quit IRC | 17:34 | |
| *** Marga_ has joined #openstack-ironic | 17:35 | |
| *** derekh has quit IRC | 17:36 | |
| *** ChuckC_ has quit IRC | 17:36 | |
| *** athomas has quit IRC | 17:37 | |
| *** spandhe has joined #openstack-ironic | 17:44 | |
| *** ChuckC has joined #openstack-ironic | 17:45 | |
| dtantsur | lazy_prince, driver vendor passthru is just an API endpoint, it allows you everything | 17:47 |
| dtantsur | so I was really surprised you didn't use it for your tool | 17:47 |
| dtantsur | it would save us this discussion | 17:47 |
| lazy_prince | aha.. k.. actually, we started with vendor passthrought but then we were advised by the community on writing an independent tool.. and hence we tok this approach.. | 17:49 |
| lazy_prince | s/tok/took/ | 17:49 |
| lucasagomes | lazy_prince, sigh :( | 17:49 |
| dtantsur | we probably changes some policies recently | 17:49 |
| lucasagomes | probably people haven't understand the tool yet | 17:50 |
| dtantsur | last time adding vendor passthru didn't require a spec even | 17:50 |
| dtantsur | lazy_prince, ^^^ | 17:50 |
| jroll | ^ | 17:50 |
| lazy_prince | thats okay.. we will come back with ven_pass through and see if it helps.. | 17:50 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add driver interface for RAID configuration https://review.openstack.org/155230 | 17:51 |
| dtantsur | lazy_prince, and that means you can just jump into proposing code into ilo passthru | 17:51 |
| dtantsur | (unless people change their mind again) | 17:51 |
| lazy_prince | Well iLO pass_pass through may not work for us as we are targetting micro_servers from HP for now.. | 17:52 |
| *** coolsvap is now known as coolsvap_ | 17:52 | |
| dtantsur | hmm... | 17:52 |
| dtantsur | then don't know, I was assuming it's something close | 17:52 |
| lazy_prince | but we wanted to provide options for other microservers too.. so we decided to shape it as framework.. | 17:53 |
| *** Marga_ has quit IRC | 17:53 | |
| *** Marga_ has joined #openstack-ironic | 17:54 | |
| dtantsur | lazy_prince, it sounds like a separate project within Ironic umbrella then | 17:54 |
| lazy_prince | yup.. thats true.. | 17:55 |
| rloo | jlvillal: I only convinced you that if we had one *ED, we have them all. dtantsur has a usecase for *ED, so I think we should have them. I don't know who suggested getting rid of them though. | 18:00 |
| NobodyCam | good meeting everyone thank you | 18:00 |
| jlvillal | rloo: :) | 18:01 |
| dtantsur | rloo, if it's about inspection, we're already living without it | 18:01 |
| * BadCub_ grabs a breath of air | 18:01 | |
| dtantsur | devananda, re moving ironic-discoverd under Ironic umbrella: if you want to do it asap, what prevents us from slowly starting? | 18:02 |
| devananda | viktors|afk: db type could not be determined | 18:02 |
| *** wanyen has joined #openstack-ironic | 18:02 | |
| devananda | dtantsur: does it have functional and integration testing? | 18:03 |
| dtantsur | devananda, functional - quite primitive, not integration testing. I have to find time to implement devstack plugin for it... | 18:03 |
| devananda | dtantsur: since it has an operator-facing API service, it will need tempest coverage. since it integrates with Ironic, it needs to have support in devstack to start and configure it, and then in tempest to test it | 18:03 |
| devananda | dtantsur: right. so that's all that blocks it from my POV | 18:04 |
| dtantsur | devananda, not sure how tempest is operated, but I don't think these folks will be happy to accept tests for some random stackforge thingy... | 18:04 |
| *** jjohnson2 has quit IRC | 18:05 | |
| devananda | dtantsur: tempest supports plugins for things on stackforge already | 18:05 |
| dtantsur | ah cool | 18:05 |
| * dtantsur needs to research | 18:06 | |
| *** faizan has quit IRC | 18:06 | |
| devananda | dtantsur: it would be really good to see other companies contributing. not a requirement, but it helps | 18:06 |
| jlvillal | yuriyz: https://review.openstack.org/#/c/158351/1/ironic/common/states.py You mention that 15 is the max size. Where would that be? I'm wondering if we should add some documentation to the file? | 18:06 |
| dtantsur | devananda, oh I'd really love it to happen too... but it's a chicken-egg problem here | 18:06 |
| jlvillal | Where would that be, as in where is it defined as 15 maximum size. | 18:06 |
| jroll | jlvillal: models.py | 18:06 |
| jlvillal | jroll: Thanks! | 18:07 |
| jroll | :) | 18:07 |
| jroll | same for driver names :( | 18:07 |
| devananda | dtantsur: yah. I get that :( | 18:07 |
| *** BadCub has quit IRC | 18:07 | |
| *** BadCub_ has quit IRC | 18:08 | |
| devananda | dtantsur: I will continue encouraging other teams in HP to use/contribute. I know there are folks who want discovery | 18:08 |
| devananda | dtantsur: but most of that interest is coming from the direction of our hardware integration (eg, chassis discovery) | 18:08 |
| dtantsur | nice, thanks | 18:08 |
| devananda | dtantsur: so I can't promise anything on that front. there are also other companies interested in the generic approach | 18:08 |
| *** BadCub_ has joined #openstack-ironic | 18:08 | |
| *** BadCub_ is now known as BadCub | 18:08 | |
| devananda | dtantsur: like yahoo -- ohhai rloo :) | 18:08 |
| dtantsur | yeah, I know some people are interested. Yuiko from NEC has contributed basic i18n support | 18:09 |
| devananda | dtantsur: or cray -- stelfer isn't around but you might poke him | 18:09 |
| *** david-lyle_afk is now known as david-lyle | 18:09 | |
| *** lazy_prince is now known as killer_prince | 18:09 | |
| dtantsur | still, it's mostly our effort for now :( | 18:09 |
| devananda | dtantsur: *nod* like I said, that's not a requirement. and I understand how folks view stackforge as unofficial (even if I find that view to be very damaging to the community) | 18:10 |
| devananda | dtantsur: integration testing is really the blocker | 18:10 |
| devananda | dtantsur: if you can show that it's ready -- ie, have an etherpad that shows anyone how to set it up in devstack and run tempest against it | 18:10 |
| devananda | even if those projects haven't merged the patches yet | 18:10 |
| *** ChuckC has quit IRC | 18:11 | |
| dtantsur | I see. I'll try to fit at least devstack work into my schedule :) | 18:11 |
| devananda | dtantsur: that would probably be enough. because then we can land it / move it to ironic, and then iterate to get those changes into devstack & tempest & infra | 18:11 |
| devananda | dtantsur: fwiw, this is what we did with IPA | 18:11 |
| *** BadCub has quit IRC | 18:11 | |
| devananda | it took a whole cycle after they felt IPA was good enough | 18:11 |
| dtantsur | yeah | 18:12 |
| *** BadCub has joined #openstack-ironic | 18:12 | |
| jroll | s/good enough/in production/ | 18:12 |
| jroll | :P | 18:12 |
| dtantsur | well, integration tests are nice to have even now | 18:12 |
| dtantsur | (though it already has one feature that is impossible to test on vms...) | 18:12 |
| lucasagomes | alright folks, I will call it a day | 18:13 |
| lucasagomes | have a good night everyone! | 18:13 |
| jroll | night lucasagomes :) | 18:13 |
| NobodyCam | have a good night lucasagomes :) | 18:13 |
| *** lucasagomes is now known as lucas-dinner | 18:14 | |
| lucas-dinner | g'nigh! | 18:14 |
| *** stendulker_ has quit IRC | 18:15 | |
| dtantsur | calling it a day too, g'night | 18:16 |
| *** dtantsur is now known as dtantsur|afk | 18:16 | |
| jlvillal | lucas-dinner: dtantsur|afk Good night/bye | 18:16 |
| *** achanda has joined #openstack-ironic | 18:16 | |
| naohirot | good night all | 18:16 |
| *** naohirot has quit IRC | 18:17 | |
| devananda | jroll: so the thing I mentioned hasn't been approved by the TC yet. here was the original proposal, now abandoned - https://review.openstack.org/#/c/131422/1/reference/project-release-schedules.rst | 18:17 |
| jroll | devananda: oh, it's abandoned? are they picking it back up? | 18:17 |
| devananda | jroll: I'm trying to figure out why I dont see anything like that in the current proposals | 18:18 |
| *** achanda has quit IRC | 18:18 | |
| devananda | jroll: oh! found it | 18:18 |
| devananda | jroll: https://review.openstack.org/#/c/157322/ | 18:18 |
| *** achanda has joined #openstack-ironic | 18:18 | |
| jroll | awesome, thanks | 18:18 |
| * jroll reads | 18:18 | |
| NobodyCam | oh devananda do happen to have a talk up for the summit? and its url for voting? | 18:19 |
| devananda | NobodyCam: I do. no clue where the link is | 18:19 |
| NobodyCam | lol | 18:20 |
| * BadCub needs to gobble down food real fast | 18:20 | |
| NobodyCam | lol : https://www.openstack.org/vote-vancouver//Presentation/fifty-shades-of-openstack-how-to-quickly-launch-different-openstack-configurations | 18:21 |
| jroll | NobodyCam: there's a search box in the top left | 18:21 |
| jroll | can search by presenter too | 18:21 |
| NobodyCam | ahh yes you can: https://www.openstack.org/vote-vancouver/presentation/isnt-it-ironic-the-bare-metal-cloud | 18:22 |
| devananda | jroll: icehouse & juno - we would have been classified as release:common. in kilo, we'll be classified as release:coordinated | 18:22 |
| jroll | devananda: right | 18:23 |
| devananda | jroll: swift would be classified as release:compatible. I think you'd like it if we were, too | 18:23 |
| jroll | maybe | 18:23 |
| NobodyCam | https://www.openstack.org/vote-vancouver/presentation/what-is-in-your-toolbox-for-deploying-an-openstack-cloud | 18:23 |
| jroll | Ijust hate that I can't land code for 2-3 months out of the year | 18:23 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add a note about maximum size for a state value https://review.openstack.org/158393 | 18:27 |
| *** eghobo has joined #openstack-ironic | 18:33 | |
| *** harlowja has joined #openstack-ironic | 18:33 | |
| *** ifarkas has quit IRC | 18:37 | |
| *** jjohnson2 has joined #openstack-ironic | 18:42 | |
| *** Marga_ has quit IRC | 18:44 | |
| jlvillal | Anyone else running devstack in a VM? It keeps failing for me when trying to launch an instance. | 18:49 |
| jlvillal | I'm following along with: http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html | 18:49 |
| jroll | what's the error etc | 18:49 |
| jroll | it does tend to work in a VM | 18:49 |
| jlvillal | jroll: No valid host was found. Exceeded max scheduling attempts 3 for instance be8f62e8-fad7-4083-98e9-a6ca73916d7f. Last exception: [u'Traceback (most recent call last):\n', u' File "/opt/stack/nova/nova/compute/manager.py", line 2087, in _do_build_and_run_in | 18:49 |
| jlvillal | Not sure if that is a VM issue or something else :( | 18:50 |
| jroll | jlvillal: anything bad in ir-cond logs? | 18:50 |
| jlvillal | Brand new devstack instance running in KVM | 18:50 |
| jroll | also, "ironic node-list" or "nova hypervisor-stats" may be helpful | 18:50 |
| jroll | [/b 163 | 18:50 |
| jroll | oops | 18:50 |
| jlvillal | jroll: Nothing I notice in ir-cond. I did a: http://paste.openstack.org/show/180721/ | 18:51 |
| jlvillal | jroll: ironic node-list http://paste.openstack.org/show/180722/ | 18:52 |
| *** viktors1 has joined #openstack-ironic | 18:52 | |
| jroll | huh | 18:52 |
| viktors1 | devananda: ping | 18:52 |
| *** priteau has joined #openstack-ironic | 18:53 | |
| jlvillal | jroll: nova hypervisor-stats http://paste.openstack.org/show/180723/ | 18:53 |
| jlvillal | jroll: When it was trying to start I did see ironic node-list change state. | 18:53 |
| jroll | jlvillal: yeah, that all looks fine assuming you're using the baremetal flavor | 18:53 |
| jlvillal | But then went back to all off. | 18:53 |
| jlvillal | I think I am. I followed the http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack | 18:54 |
| jlvillal | So setup for baremetal, but running in a KVM instance. Which sounds like an oxymoron :) | 18:55 |
| jlvillal | I'm going to go to lunch now. Play with it some more when I get back... | 18:55 |
| jlvillal | jroll: Thanks! | 18:55 |
| jroll | what's your nova boot line? | 18:55 |
| jroll | heh, ok, np :) | 18:55 |
| jlvillal | jroll: Uh, where do I look. Sorry still new | 18:56 |
| jlvillal | I just did ./stack.sh | 18:56 |
| jroll | jlvillal: like, how are you booting the instance | 18:56 |
| jroll | "nova boot ..." | 18:56 |
| jlvillal | jroll: nova boot --flavor baremetal --image $image --key-name default testin | 18:56 |
| jlvillal | jroll: nova boot --flavor baremetal --image $image --key-name default testing | 18:56 |
| jroll | hrm. | 18:56 |
| jroll | no clue | 18:56 |
| jlvillal | jroll: Me either! :( | 18:57 |
| jlvillal | Okay, time for food :) | 18:57 |
| *** zer0c00l has joined #openstack-ironic | 18:59 | |
| *** faizan has joined #openstack-ironic | 19:00 | |
| NobodyCam | :) | 19:01 |
| *** kbs has quit IRC | 19:01 | |
| priteau | Hello! I have a quick question: is it possible to configure Ironic / Neutron so that each bare-metal node gets its own static IP? | 19:04 |
| jroll | priteau: yes, with cloud-init and configdrive, though neutron will still be the service that decides the IP | 19:06 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Generic changes for Node Inspection https://review.openstack.org/149823 | 19:07 |
| *** viktors1 has quit IRC | 19:07 | |
| *** Marga_ has joined #openstack-ironic | 19:08 | |
| priteau | jroll: but can I configure Neutron to decide to always give the same unique IP to a specific node? | 19:10 |
| jroll | priteau: I have no idea, maybe | 19:10 |
| jroll | though I kind of doubt it | 19:10 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: iLO implementation for hardware inspection https://review.openstack.org/151596 | 19:10 |
| *** viktors1 has joined #openstack-ironic | 19:12 | |
| *** viktors1 has quit IRC | 19:15 | |
| *** viktors1 has joined #openstack-ironic | 19:16 | |
| *** viktors1 has quit IRC | 19:19 | |
| *** pelix1 has quit IRC | 19:23 | |
| *** alexpilotti has quit IRC | 19:24 | |
| *** penick has joined #openstack-ironic | 19:27 | |
| *** rameshg87 has quit IRC | 19:30 | |
| *** lucas-dinner has quit IRC | 19:35 | |
| priteau | jroll: I am afraid you're right, I can't find any way to configuring it this way | 19:37 |
| *** andreykurilin_ has quit IRC | 19:38 | |
| *** wanyen has quit IRC | 19:39 | |
| *** bauzas has quit IRC | 19:43 | |
| *** Nisha has quit IRC | 19:46 | |
| *** kbs has joined #openstack-ironic | 19:46 | |
| *** belle_ has joined #openstack-ironic | 19:47 | |
| *** belle_ has quit IRC | 19:48 | |
| priteau | jroll: It looks like the solution might arrive in Kilo: http://specs.openstack.org/openstack/neutron-specs/specs/kilo/neutron-ipam.html | 19:49 |
| *** openstackgerrit has quit IRC | 19:51 | |
| *** openstackgerrit has joined #openstack-ironic | 19:52 | |
| *** viktors1 has joined #openstack-ironic | 19:54 | |
| *** viktors1 has quit IRC | 19:55 | |
| *** anderbubble has quit IRC | 19:55 | |
| *** achanda has quit IRC | 19:57 | |
| *** penick has quit IRC | 19:58 | |
| *** achanda has joined #openstack-ironic | 19:59 | |
| openstackgerrit | Merged openstack/ironic: Disable n-novnc, heat, cinder and horizon on devstack https://review.openstack.org/158333 | 19:59 |
| *** kbs has quit IRC | 19:59 | |
| openstackgerrit | Merged openstack/ironic: Use functions from oslo.utils https://review.openstack.org/158254 | 19:59 |
| openstackgerrit | Merged openstack/ironic: Update Ilo drivers to use REST API interface to iLO https://review.openstack.org/154403 | 20:00 |
| devananda | jroll: did yall ever review the RAID interface spec thoroughly before it got approved? https://review.openstack.org/#/c/135899/ | 20:00 |
| *** gabriel-bezerra has joined #openstack-ironic | 20:11 | |
| *** Marga__ has joined #openstack-ironic | 20:14 | |
| *** Marga_ has quit IRC | 20:14 | |
| jroll | devananda: not sure, why? | 20:16 |
| *** jjohnson2 has quit IRC | 20:17 | |
| *** andreykurilin_ has joined #openstack-ironic | 20:19 | |
| *** eghobo has quit IRC | 20:23 | |
| devananda | jroll: doesn't look like it had eyes from your team, and seems like something ya'll have a reason to care about | 20:24 |
| devananda | jroll: considering the spec is now approved and thus i presume folks are going ahead with the code | 20:25 |
| devananda | also, this looks like an easy review, if any cores want to +2/+A it - https://review.openstack.org/152859 | 20:25 |
| *** faizan has quit IRC | 20:26 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Minor changes to InspectInterface https://review.openstack.org/158436 | 20:29 |
| NobodyCam | where there any client patches for logical name support? | 20:32 |
| rloo | devananda: wrt 52859, you ok with grammatical errors? ('AMT driver use WS-MAN protocol to interactive with...') | 20:32 |
| devananda | rloo: in published docs, not really. | 20:33 |
| devananda | rloo: that's what I get for speed-reading | 20:33 |
| *** Marga__ has quit IRC | 20:34 | |
| rloo | devananda: i can see a bunch of grammatical stuff. | 20:34 |
| *** harlowja has quit IRC | 20:34 | |
| devananda | rloo: (just fwiw, I'm fine with that in inline comments) | 20:34 |
| *** harlowja_ has joined #openstack-ironic | 20:34 | |
| rloo | devananda: we could/i could approve it and just fix it in a separate patch. or do i comment with all my nits? | 20:34 |
| devananda | darn. well, I was hoping that'd be an easy one to land | 20:34 |
| devananda | rloo: up to you. I'd be fine with a follow on patch too | 20:34 |
| *** Marga_ has joined #openstack-ironic | 20:34 | |
| *** Marga_ has quit IRC | 20:35 | |
| rloo | devananda: ok, i think it is faster for me to do a follow on patch, than to comment on everything. | 20:35 |
| *** Marga_ has joined #openstack-ironic | 20:35 | |
| *** Marga_ has quit IRC | 20:35 | |
| *** Marga_ has joined #openstack-ironic | 20:36 | |
| *** Marga_ has quit IRC | 20:36 | |
| *** Marga_ has joined #openstack-ironic | 20:37 | |
| *** Marga_ has quit IRC | 20:38 | |
| *** Marga_ has joined #openstack-ironic | 20:38 | |
| *** penick has joined #openstack-ironic | 20:42 | |
| *** wanyen has joined #openstack-ironic | 20:43 | |
| *** priteau has quit IRC | 20:45 | |
| *** achanda has quit IRC | 20:45 | |
| *** Marga_ has quit IRC | 20:46 | |
| wanyen | Hi BadCub | 20:46 |
| harlowja_ | rloo what colors u want for https://review.openstack.org/#/c/158102/ :-P | 20:47 |
| harlowja_ | no color? | 20:47 |
| harlowja_ | *for start ones | 20:47 |
| rloo | harlowja_: what is a 'start' state? | 20:47 |
| harlowja_ | ones that the machine may start off in i guess, ha | 20:48 |
| harlowja_ | AVAILABLE, NOSTATE, MANAGEABLE? | 20:48 |
| rloo | harlowja_: they're all supposed to start in 'enroll' state which doesn't exist yet. | 20:48 |
| harlowja_ | oh | 20:48 |
| harlowja_ | kk | 20:48 |
| rloo | harlowja_: I'd say no colours until we get things sorted out | 20:48 |
| harlowja_ | k | 20:48 |
| rloo | harlowja_: stop being so fancy and adding more code ;) | 20:48 |
| harlowja_ | :-P | 20:49 |
| harlowja_ | kk | 20:49 |
| harlowja_ | yes ma'm | 20:49 |
| devananda | rloo: on your "inspect failed" change, that's going to break anyone who's using trunk and has a node in that state currently | 20:49 |
| devananda | rloo: I think that probably affects exactly zero users -- but it might be worth calling out in the commit message | 20:49 |
| rloo | devananda: did code get in that actually uses inspect failed? | 20:49 |
| rloo | devananda: so does it need a DocImpact? Seems like you'd need to microversion to even use that and there's no code for it | 20:50 |
| devananda | rloo: hm, probably true | 20:50 |
| jroll | devananda: it's probably fine, I did skim and didn't see anything outrageous. thanks for the ping | 20:50 |
| devananda | jroll: great, thanks | 20:50 |
| mrda | Morning Ironic | 20:51 |
| NobodyCam | rloo: doc and possiably database | 20:51 |
| rloo | NobodyCam: huh? | 20:51 |
| devananda | rloo: oh, I dont see the API code for inspection yet :) | 20:51 |
| NobodyCam | oh no api code | 20:51 |
| NobodyCam | nm | 20:51 |
| NobodyCam | :-p | 20:51 |
| rloo | devananda: yeah, that's what I mean. Unless you manually update the db or have your own out of tree code | 20:51 |
| devananda | right .. so this is fine | 20:52 |
| rloo | phew :-) | 20:52 |
| rloo | can't believe I missed that when reviewing; harlowja_'s picture was worth it ;) | 20:52 |
| harlowja_ | :) | 20:53 |
| rloo | harlowja_: go do some other stuff for awhile and wait for this to land first, before updating 158102: https://review.openstack.org/#/c/158351/ | 20:54 |
| harlowja_ | k | 20:55 |
| harlowja_ | sounds good to me :-P | 20:55 |
| rloo | thx harlowja_ | 20:56 |
| harlowja_ | np boss | 20:56 |
| rloo | for everyone that doesn't know me, I am definitely NOT harlowja_'s boss | 20:57 |
| jroll | lol, I was starting to wonder | 20:57 |
| harlowja_ | np boss | 20:58 |
| harlowja_ | whatever u say boss | 20:58 |
| harlowja_ | sorry boss | 20:58 |
| rloo | ha ha. harlowja_ is also at Yahoo ;) | 20:58 |
| mrda | rloo: you have direct report? huh | 20:58 |
| harlowja_ | lol | 20:58 |
| jroll | I knew josh was a yahooligan so it made sense :D | 20:58 |
| * harlowja_ runs away (causing to much trouble) | 20:58 | |
| jroll | (is that what y'all call yourselves?) | 20:58 |
| * harlowja_ i just call myself 'trouble' | 20:58 | |
| harlowja_ | lol | 20:58 |
| jroll | heh | 20:58 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: iLO implementation for hardware inspection https://review.openstack.org/151596 | 20:59 |
| *** penick has quit IRC | 21:08 | |
| *** hj-hp has joined #openstack-ironic | 21:08 | |
| dank_ | Welcome hj-hp | 21:11 |
| *** penick has joined #openstack-ironic | 21:11 | |
| NobodyCam | :-p | 21:11 |
| *** jjohnson2 has joined #openstack-ironic | 21:12 | |
| *** achanda has joined #openstack-ironic | 21:13 | |
| *** Marga_ has joined #openstack-ironic | 21:17 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: iLO implementation for hardware inspection https://review.openstack.org/151596 | 21:18 |
| NobodyCam | brb | 21:24 |
| *** ChuckC has joined #openstack-ironic | 21:25 | |
| *** ijw has joined #openstack-ironic | 21:26 | |
| *** Marga_ has quit IRC | 21:32 | |
| *** Marga_ has joined #openstack-ironic | 21:32 | |
| * BadCub takes a break for food and things | 21:32 | |
| *** ijw_ has joined #openstack-ironic | 21:34 | |
| *** ijw has quit IRC | 21:36 | |
| *** ijw has joined #openstack-ironic | 21:41 | |
| *** ijw_ has quit IRC | 21:44 | |
| *** chuckc__ has joined #openstack-ironic | 21:44 | |
| *** chuckc__ has left #openstack-ironic | 21:44 | |
| *** achanda has quit IRC | 21:46 | |
| rfchapman | NobodyCam: How you doing, hope you had a nice weekend. | 21:47 |
| *** achanda has joined #openstack-ironic | 21:47 | |
| rfchapman | Could anyone help with deploy issues? The depoy is failing at the dd portion of the install. It looks like the first dd of 8k is successfull, but the seccond dd is not. Anyone have any ideas? | 21:50 |
| rfchapman | This is on a HP dl360 G2 box | 21:50 |
| *** gabriel-bezerra has quit IRC | 21:58 | |
| NobodyCam | hey rfchapman :) yep I'z had a very nice weekend, Hope yours was the same | 22:00 |
| NobodyCam | can you paste the conductor log ? | 22:01 |
| NobodyCam | if DD starts I am thinking its maybe disk releated. could be network? | 22:01 |
| rfchapman | Yes not bad. Got caught up a little bit. slept quite a bit. | 22:02 |
| rfchapman | Sure I'll pastbin it. | 22:02 |
| NobodyCam | :) | 22:02 |
| rfchapman | what was the address for pastebin? | 22:03 |
| rfchapman | ah shucks, I need to repete the test. Give me a few mins | 22:06 |
| jlvillal | rfchapman: paste.openstack.org | 22:11 |
| *** romcheg has quit IRC | 22:12 | |
| openstackgerrit | Merged openstack/ironic: Address final comments of a4cf7149fb https://review.openstack.org/156671 | 22:12 |
| openstackgerrit | Merged openstack/ironic: Add dhcp-all-interfaces to get IP to NIC other than eth0 https://review.openstack.org/158156 | 22:13 |
| openstackgerrit | Merged openstack/ironic: INSPECTFAIL value is more readable https://review.openstack.org/158351 | 22:13 |
| * jlvillal sometimes really hates mock... | 22:15 | |
| jlvillal | rloo: I saw your comment on my note about maximum size | 22:17 |
| *** romcheg has joined #openstack-ironic | 22:17 | |
| rloo | jlvillal: that was fast ;) | 22:17 |
| jlvillal | I did make a patch to check for things greater than 15 characters. But it was a little hacky. | 22:18 |
| rloo | jlvillal: you feel like updating the comments in that file? | 22:18 |
| jlvillal | for key, value in globals.iteritems(): | 22:18 |
| jlvillal | if key.upper() == key and isinstance(key, str): | 22:18 |
| rloo | jlvillal: that assumes that they're all states | 22:18 |
| jlvillal | assert len(value) < 15, "blah blah blah" | 22:19 |
| jlvillal | I mean, isinstance(value, str) | 22:19 |
| jlvillal | But I think you get the idea. I wasn't sure if it would be accepted so I didn't submit it. | 22:19 |
| jroll | I think that should be a unit test or something | 22:19 |
| *** anderbubble has joined #openstack-ironic | 22:19 | |
| jroll | maybe export all_states = (AVAILABLE, MANAGED, ...) | 22:20 |
| jroll | and then a unit test, for state in states.all_states: check_length() | 22:20 |
| *** jjohnson2 has quit IRC | 22:20 | |
| jlvillal | jroll: How do we enforce states get put into all_states? | 22:20 |
| rloo | jlvillal, jroll: +1 for unit test. I think it is ok to assume that all keys like jlvillal suggested are states. cuz they are for now. | 22:20 |
| jroll | jlvillal: idk, code review? | 22:21 |
| rfchapman | nobodyCam: Here is the log http://paste.openstack.org/show/180826/ | 22:21 |
| jroll | I'd just rather the assertion be in tests and not the actual code | 22:21 |
| rloo | I think for now, they are all states. if we need to export them, we can do that later. hopefully we won't need to. and yeah, i'm worried about having to make sure that all the states get added to all_states. | 22:21 |
| jlvillal | jroll: That shouldn't be hard. As we there is a unit test for that file. | 22:21 |
| jlvillal | jroll: I can do my idea but in the unittest file. | 22:22 |
| jroll | jlvillal: yep, cool :) | 22:22 |
| jlvillal | I'll work on that where I look at the globals. | 22:22 |
| jlvillal | As I'm getting frustrated with mock on a different task | 22:22 |
| rloo | thx jlvillal | 22:22 |
| jroll | I'm just not sure how to reference it other than having a variable that is a list of them all | 22:22 |
| jroll | oh, dir(states) | 22:22 |
| jroll | right. | 22:22 |
| NobodyCam | TheJulia: have you seen: https://www.openstack.org/vote-vancouver/presentation/deploying-openstack-with-ansible | 22:22 |
| jlvillal | jroll: Yep! | 22:23 |
| jlvillal | If it is ALL UPPERCASE and a string value, assume it is a state name. | 22:23 |
| jroll | TheJulia: are you working on ironic/ansible things? | 22:23 |
| jroll | jlvillal: ++ | 22:23 |
| NobodyCam | rfchapman: line 66-69 seem to be in play here | 22:26 |
| jroll | NobodyCam: "unit MiB" may be related | 22:26 |
| *** hj-hp has quit IRC | 22:27 | |
| rfchapman | NobodyCam: yes, as well as 47-50 | 22:29 |
| *** harlowja_ has quit IRC | 22:29 | |
| NobodyCam | rfchapman: yep ... looks like same command | 22:29 |
| *** harlowja has joined #openstack-ironic | 22:30 | |
| *** gabriel-bezerra has joined #openstack-ironic | 22:30 | |
| rfchapman | NobodyCam: I'm not sure what is happening on line 31-50... It looks like the first dd of 8k works with out a hitch, but the second one with a seek of 143305884.. What is it doing | 22:34 |
| NobodyCam | oh this looks good: https://www.openstack.org/vote-vancouver/presentation/bare-metal-network-isolation-between-tenants-with-vxlan | 22:36 |
| jroll | huh | 22:38 |
| jroll | I wonder if they've actually done it yet | 22:38 |
| NobodyCam | rfchapman: line 31 is wipping out any mbr on the disk | 22:39 |
| NobodyCam | so its just writting zeros to the first 36 blocks | 22:39 |
| NobodyCam | rfchapman: how did you install this. I am not seeing logs I would expect to see in the latest trunk versions | 22:40 |
| rfchapman | NobodyCam, Yes, but when blockdev is called right after it's reporting that lun-1 to be the entire size of the disk, does that mean that the boot block was not successfully written? | 22:41 |
| rloo | harlowja: https://review.openstack.org/158351 merged. so whenever you want to update the state diagram... | 22:41 |
| rfchapman | blockdev should be reporting 18432 | 22:42 |
| harlowja | kk | 22:42 |
| harlowja | thx boss | 22:42 |
| jroll | :D | 22:42 |
| rloo | anytime underling harlowja ;) | 22:42 |
| NobodyCam | lol | 22:42 |
| harlowja | :) | 22:42 |
| rfchapman | I used RDO packstack | 22:42 |
| *** achanda has quit IRC | 22:43 | |
| NobodyCam | ahh ok packaged version. | 22:43 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Ensure state values are 15 characters or less https://review.openstack.org/158393 | 22:43 |
| TheJulia | NobodyCam: Saw it, thank you for the reminder :) | 22:44 |
| harlowja | rloo will get that fixed soon boss, having password issues @ work | 22:44 |
| jlvillal | rloo: jroll : ^^^ is patch with check | 22:44 |
| NobodyCam | any chance you could update? we've added a fair amount of logging to help with some of the deploy issues | 22:44 |
| *** hj-hp has joined #openstack-ironic | 22:44 | |
| TheJulia | jroll: I started about two weeks ago and had to context switch back to another project for a little bit, which just so happened to wrap up today, so I'm heading back in that direction. | 22:44 |
| rfchapman | I updated saturday, but nothing new for ironic. | 22:45 |
| rloo | harlowja: no worries | 22:45 |
| rloo | jlvillal: looking... | 22:45 |
| *** achanda has joined #openstack-ironic | 22:45 | |
| NobodyCam | rfchapman: you did a rpm update? | 22:45 |
| rfchapman | NobodyCam: yes. | 22:45 |
| NobodyCam | TheJulia: very kewl :) | 22:46 |
| rloo | jlvillal: did you copy that test file from somewhere else? | 22:46 |
| NobodyCam | rfchapman: I'm sure RH hasn't packaged a new version yet | 22:46 |
| jlvillal | rloo: Kind of. fsm.py | 22:46 |
| jlvillal | I deleted almost all of it | 22:46 |
| rloo | jlvillal: that explains it. | 22:47 |
| jroll | rloo: he copied it from yahoo's proprietary code somewhere | 22:47 |
| rloo | jroll: ha ha. just that he's giving credit to yahoo for his work, that's all ;) | 22:48 |
| jroll | TheJulia: gotcha, we also deploy ironic with ansible if you ever want to bounce ideas off of me | 22:48 |
| jlvillal | rloo: Okay, I'll change it | 22:48 |
| jlvillal | Since in reality I had only have import line | 22:48 |
| NobodyCam | rfchapman: how large is sda on the node your testing with? | 22:48 |
| rloo | jlvillal: you don't have to change it, but I thought I should mention it. Am trying to be impartial here :-) | 22:49 |
| jlvillal | rloo: I'll change and update with jroll's comment | 22:49 |
| NobodyCam | jroll: does deva know that? | 22:50 |
| TheJulia | jroll: cool, right now the initial focus is more on being able to add/update/remove nodes in ironic and not the actual deployment of an ironic node | 22:50 |
| *** Marga_ has quit IRC | 22:50 | |
| rloo | jlvillal: jroll is a faster reviewer than me. | 22:52 |
| jlvillal | rloo: On your None question. it is because there is a state: NOSTATE = None | 22:52 |
| jlvillal | I'm just saying we don't care about it. | 22:52 |
| jlvillal | So we don't check it, because it doesn't have a length | 22:53 |
| jroll | TheJulia: I see, we don't use ansible for that, just control plane deployment :) | 22:54 |
| jroll | NobodyCam: I believe so, yes | 22:54 |
| rloo | jlvillal: oh, i think it is the isinstance() that excludes it. ok. | 22:54 |
| NobodyCam | :) | 22:54 |
| jlvillal | rloo: Yep! | 22:54 |
| jlvillal | rloo: It also excludes LOG (which is the logging class) | 22:54 |
| rloo | jlvillal: and VERBS | 22:55 |
| jlvillal | That too :) | 22:55 |
| *** Marga_ has joined #openstack-ironic | 22:55 | |
| *** Marga_ has quit IRC | 22:56 | |
| *** Marga_ has joined #openstack-ironic | 22:56 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Ensure state values are 15 characters or less https://review.openstack.org/158393 | 22:57 |
| *** kbs has joined #openstack-ironic | 22:58 | |
| NobodyCam | oh the priorities google doc was updated this morning :) https://docs.google.com/spreadsheets/d/1Hxyfy60hN_Fit0b-plsPzK6yW3ePQC5IfwuzJwltlbo | 22:59 |
| *** wanyen has quit IRC | 23:01 | |
| *** harlowja has quit IRC | 23:04 | |
| *** rwsu is now known as rwsu-afk | 23:04 | |
| *** rloo is now known as rloo_afk | 23:04 | |
| *** romcheg has quit IRC | 23:08 | |
| *** romcheg has joined #openstack-ironic | 23:08 | |
| *** romcheg has quit IRC | 23:09 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement Cleaning States https://review.openstack.org/153444 | 23:10 |
| *** achanda has quit IRC | 23:12 | |
| *** devlaps has quit IRC | 23:14 | |
| *** devlaps has joined #openstack-ironic | 23:14 | |
| *** kbs has quit IRC | 23:15 | |
| *** harlowja has joined #openstack-ironic | 23:16 | |
| rfchapman | NobodyCam: 80Gig is the size of the disk. | 23:20 |
| rfchapman | NobodyCam: sorry for the delay, I got pulled away. | 23:21 |
| NobodyCam | rfchapman: line 38 shows it as a 73Gb disk I think | 23:21 |
| rfchapman | NobodyCam: I would expect that for a formatted size.. | 23:22 |
| jroll | GiB vs GB coming in... 1... 2... | 23:23 |
| NobodyCam | jroll: yep thats my guess | 23:23 |
| NobodyCam | rfchapman: try setting it up as a 70 gb disk | 23:23 |
| NobodyCam | :-p | 23:23 |
| rfchapman | Yep, My guess as well. | 23:23 |
| rfchapman | Actually It my be 79Gig. | 23:24 |
| rfchapman | I'm not really sure, I'll run down and check. brb | 23:24 |
| *** achanda has joined #openstack-ironic | 23:25 | |
| *** oomichi has joined #openstack-ironic | 23:25 | |
| NobodyCam | rfchapman: just make the make the size on the flavor and node smaller | 23:26 |
| NobodyCam | does not need to exactlly match | 23:26 |
| * NobodyCam assumes your in POC or testing phase | 23:27 | |
| *** anderbubble has quit IRC | 23:27 | |
| *** dprince has quit IRC | 23:29 | |
| * NobodyCam corrects himself.. does not need to match the actual size of the disk ... small is ok to.. (flavor and node do still need to match) | 23:32 | |
| *** ijw has quit IRC | 23:33 | |
| *** krtaylor has quit IRC | 23:35 | |
| rfchapman | Yes, I just came to that conclusion that I have it as 80 gig and the disk is smaller than that. | 23:36 |
| *** alexpilotti has joined #openstack-ironic | 23:36 | |
| NobodyCam | :) if this is just testing then set to some thing you know will pass | 23:37 |
| NobodyCam | :-p | 23:37 |
| devananda | NobodyCam: I dont think that we're done updating that doc ... | 23:41 |
| * NobodyCam never said it was finished being updated :-p just that it had been updated | 23:42 | |
| * NobodyCam ducks | 23:42 | |
| NobodyCam | hehehe | 23:42 |
| *** andreykurilin_ has quit IRC | 23:43 | |
| *** krtaylor has joined #openstack-ironic | 23:47 | |
| NobodyCam | any reason we haven't landed : https://review.openstack.org/#/c/154730 | 23:49 |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement Cleaning in DriverInterfaces https://review.openstack.org/157208 | 23:55 |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement Cleaning States https://review.openstack.org/153444 | 23:55 |
| *** penick has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!