| *** jistr has quit IRC | 00:00 | |
| *** jistr has joined #openstack-ironic | 00:01 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Enable support for OSprofiler in Ironic. https://review.openstack.org/337987 | 00:05 |
|---|---|---|
| openstackgerrit | Merged openstack/ironic: Documentation fixes for iLO SSL Certificate feature https://review.openstack.org/349800 | 00:16 |
| *** rbudden has joined #openstack-ironic | 00:16 | |
| *** mgoddard has joined #openstack-ironic | 00:18 | |
| *** rama_y has quit IRC | 00:18 | |
| *** rbudden has quit IRC | 00:21 | |
| *** mtanino has quit IRC | 00:23 | |
| penick | @devananda I prefer to not discuss the color for the bike shed until we discuss the font on the signage | 00:26 |
| penick | I prefer catweasle | 00:26 |
| jroll | penick: we need to decide on the copy before we choose a font | 00:29 |
| jroll | the copy imo should have the color of the bikeshed | 00:29 |
| jroll | so colors first! | 00:29 |
| *** syed_ has quit IRC | 00:32 | |
| penick | Well, if the background is a nice nature background i’m not sure I want it to stand out. So for that we should go with a camo job. | 00:36 |
| penick | In other news.. Launching Ironic at Yahoo today | 00:36 |
| jroll | zomg | 00:37 |
| jroll | \o/ | 00:37 |
| * jroll passes a case of beer knowing you need it by now | 00:37 | |
| penick | right? \o/! | 00:37 |
| openstackgerrit | Merged openstack/python-oneviewclient: Workaround to OneView pagination https://review.openstack.org/348532 | 00:37 |
| zer0c00l_ | yeah yeah! | 00:37 |
| zer0c00l_ | :P | 00:37 |
| jroll | congrats dude :D | 00:37 |
| * jroll wants a copy of the postmortem | 00:37 | |
| zer0c00l_ | hahahaha | 00:38 |
| penick | hahahaha | 00:38 |
| penick | haha | 00:38 |
| * penick weeps | 00:38 | |
| *** hoangcx has joined #openstack-ironic | 00:39 | |
| *** ijw has quit IRC | 00:52 | |
| *** ijw has joined #openstack-ironic | 00:52 | |
| *** sdake has joined #openstack-ironic | 00:59 | |
| * TheJulia wonders if whisky is now indicated to remedy weeping | 01:00 | |
| penick | Probably. So. Much. Whiskey. | 01:04 |
| *** Sukhdev has quit IRC | 01:06 | |
| *** xuao has joined #openstack-ironic | 01:07 | |
| *** xuao has quit IRC | 01:09 | |
| *** xuao has joined #openstack-ironic | 01:11 | |
| *** fragatin_ has quit IRC | 01:12 | |
| *** fragatina has joined #openstack-ironic | 01:12 | |
| openstackgerrit | Merged openstack/python-oneviewclient: Comply with iLO model of usage for one-time boot https://review.openstack.org/350060 | 01:12 |
| *** xuao has quit IRC | 01:16 | |
| *** ijw has quit IRC | 01:16 | |
| *** xuao has joined #openstack-ironic | 01:16 | |
| *** bfournie has joined #openstack-ironic | 01:22 | |
| *** zhangjl has joined #openstack-ironic | 01:25 | |
| *** ijw has joined #openstack-ironic | 01:30 | |
| *** amotoki has joined #openstack-ironic | 01:40 | |
| *** aNupoisc has joined #openstack-ironic | 01:43 | |
| *** zhangjl has quit IRC | 01:44 | |
| *** amotoki has quit IRC | 01:48 | |
| *** jabroney has quit IRC | 01:49 | |
| *** mbound has joined #openstack-ironic | 01:51 | |
| *** mbound has quit IRC | 01:52 | |
| *** amotoki has joined #openstack-ironic | 01:58 | |
| *** amotoki has quit IRC | 01:59 | |
| *** amotoki has joined #openstack-ironic | 02:00 | |
| *** aNupoisc has quit IRC | 02:05 | |
| *** [1]cdearborn has quit IRC | 02:07 | |
| *** ayoung has quit IRC | 02:11 | |
| *** bvandewa has quit IRC | 02:12 | |
| *** ijw has quit IRC | 02:16 | |
| *** zhangjl has joined #openstack-ironic | 02:26 | |
| *** PollyZ has joined #openstack-ironic | 02:35 | |
| openstackgerrit | Xu Ao proposed openstack/ironic: Fix the mistakes in Installation Guide https://review.openstack.org/353224 | 02:40 |
| rloo | congrats penick & zer0c00l_ !!! I wondered... :) | 02:45 |
| zer0c00l_ | :) | 02:46 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 02:57 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: 'As of' in documentation is incorrect https://review.openstack.org/353227 | 03:00 |
| *** xuao has quit IRC | 03:03 | |
| *** xuao has joined #openstack-ironic | 03:08 | |
| *** xuao has quit IRC | 03:10 | |
| *** xuao has joined #openstack-ironic | 03:12 | |
| *** sabeen has joined #openstack-ironic | 03:15 | |
| *** sdake has quit IRC | 03:17 | |
| *** appu103_ has joined #openstack-ironic | 03:27 | |
| *** appu103_ is now known as aparnav | 03:27 | |
| *** TuanLA has joined #openstack-ironic | 03:29 | |
| *** TuanLA_ has joined #openstack-ironic | 03:37 | |
| *** TuanLA__ has joined #openstack-ironic | 03:38 | |
| *** tiendc has joined #openstack-ironic | 03:39 | |
| *** phuongnh has joined #openstack-ironic | 03:39 | |
| *** TuanLA__ has quit IRC | 03:40 | |
| *** tuanla__ has joined #openstack-ironic | 03:41 | |
| *** TuanLA_ has quit IRC | 03:42 | |
| tuanla__ | Good morning, | 03:42 |
| *** TuanLA has left #openstack-ironic | 03:42 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-python-agent: Parallel erase disk devices https://review.openstack.org/346723 | 03:49 |
| *** penick has quit IRC | 03:53 | |
| *** rloo has quit IRC | 04:00 | |
| *** links has joined #openstack-ironic | 04:01 | |
| *** tangchen has joined #openstack-ironic | 04:03 | |
| *** xuao has quit IRC | 04:17 | |
| *** permalac has quit IRC | 04:25 | |
| *** permalac has joined #openstack-ironic | 04:26 | |
| *** ppiela has quit IRC | 04:31 | |
| *** vmud213 has joined #openstack-ironic | 04:33 | |
| *** fragatin_ has joined #openstack-ironic | 04:36 | |
| *** rama_y has joined #openstack-ironic | 04:37 | |
| *** fragatina has quit IRC | 04:39 | |
| *** fragatin_ has quit IRC | 04:40 | |
| *** zhangjl has quit IRC | 04:40 | |
| *** tiendc_ has joined #openstack-ironic | 04:50 | |
| *** PollyZ has quit IRC | 04:50 | |
| *** tiendc has quit IRC | 04:51 | |
| *** ChubYann has quit IRC | 04:51 | |
| *** zhangjl has joined #openstack-ironic | 04:51 | |
| *** xuao has joined #openstack-ironic | 04:52 | |
| *** zhangjl has left #openstack-ironic | 04:55 | |
| *** xuao has quit IRC | 04:55 | |
| *** pcaruana has quit IRC | 05:01 | |
| *** phuongnh has quit IRC | 05:09 | |
| *** rama_y has quit IRC | 05:10 | |
| *** tiendc_ has quit IRC | 05:18 | |
| *** xuao has joined #openstack-ironic | 05:24 | |
| -openstackstatus- NOTICE: zuul is being restarted to reload configuration. Jobs should be re-enqueued but if you're missing anything (and it's not on http://status.openstack.org/zuul/) please issue a recheck in 30min. | 05:24 | |
| *** phuongnh has joined #openstack-ironic | 05:25 | |
| *** xuao has quit IRC | 05:26 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 05:26 |
| *** xuao has joined #openstack-ironic | 05:31 | |
| *** tiendc_ has joined #openstack-ironic | 05:31 | |
| *** Sukhdev has joined #openstack-ironic | 05:31 | |
| *** tiendc_ is now known as tiendc | 05:37 | |
| *** ijw has joined #openstack-ironic | 05:38 | |
| *** ijw has quit IRC | 05:43 | |
| *** vmud213 has quit IRC | 05:43 | |
| *** penick has joined #openstack-ironic | 05:51 | |
| *** vishwanathj has quit IRC | 05:56 | |
| *** zhangjl has joined #openstack-ironic | 05:57 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Fix iLO drivers inconsistent boot mode default value https://review.openstack.org/345231 | 05:57 |
| *** rcernin has joined #openstack-ironic | 06:04 | |
| *** ijw has joined #openstack-ironic | 06:14 | |
| *** ijw has quit IRC | 06:19 | |
| *** vishwanathj has joined #openstack-ironic | 06:28 | |
| *** vishwanathj has quit IRC | 06:29 | |
| *** ijw has joined #openstack-ironic | 06:31 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Timing metrics: iLO drivers https://review.openstack.org/352745 | 06:31 |
| *** Tuan has joined #openstack-ironic | 06:32 | |
| *** ijw has quit IRC | 06:35 | |
| openstackgerrit | Aparna proposed openstack/proliantutils: Add a clean step 'hardware_disk_erase' https://review.openstack.org/336148 | 06:36 |
| *** fragatina has joined #openstack-ironic | 06:37 | |
| *** fragatina has quit IRC | 06:42 | |
| *** tesseract- has joined #openstack-ironic | 06:44 | |
| *** penick has quit IRC | 06:45 | |
| *** Lord_Phoenix has joined #openstack-ironic | 06:47 | |
| *** e0ne has joined #openstack-ironic | 06:47 | |
| *** moshele has joined #openstack-ironic | 06:49 | |
| *** praneshp has quit IRC | 06:51 | |
| *** Sukhdev has quit IRC | 06:52 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 06:52 |
| *** e0ne has quit IRC | 06:54 | |
| *** sabeen has quit IRC | 06:55 | |
| *** e0ne has joined #openstack-ironic | 06:58 | |
| *** milan has joined #openstack-ironic | 06:59 | |
| *** e0ne has quit IRC | 07:00 | |
| *** ccamacho has quit IRC | 07:01 | |
| pas-ha | morning all | 07:02 |
| *** ccamacho has joined #openstack-ironic | 07:04 | |
| *** e0ne has joined #openstack-ironic | 07:05 | |
| *** e0ne has quit IRC | 07:07 | |
| *** cmyster_ is now known as cmyster | 07:09 | |
| *** cmyster has quit IRC | 07:09 | |
| *** cmyster has joined #openstack-ironic | 07:09 | |
| *** e0ne has joined #openstack-ironic | 07:11 | |
| *** stendulker has joined #openstack-ironic | 07:15 | |
| *** jistr has quit IRC | 07:18 | |
| *** crinkle has quit IRC | 07:19 | |
| *** greghaynes has quit IRC | 07:20 | |
| *** bvandewa has joined #openstack-ironic | 07:20 | |
| openstackgerrit | Aparna proposed openstack/proliantutils: Add a clean step 'hardware_disk_erase' https://review.openstack.org/336148 | 07:20 |
| *** greghaynes has joined #openstack-ironic | 07:20 | |
| *** athomas has joined #openstack-ironic | 07:21 | |
| *** crinkle has joined #openstack-ironic | 07:21 | |
| *** pcaruana has joined #openstack-ironic | 07:24 | |
| *** ifarkas_afk is now known as ifarkas | 07:25 | |
| *** jistr has joined #openstack-ironic | 07:25 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 07:29 |
| milan | morning Ironic! :) | 07:29 |
| *** e0ne has quit IRC | 07:31 | |
| stendulker | morning milan :) | 07:32 |
| milan | morning stendulker :) | 07:33 |
| *** ccamacho is now known as ccamacho|afk | 07:33 | |
| *** rbartal has joined #openstack-ironic | 07:35 | |
| *** jpich has joined #openstack-ironic | 07:43 | |
| *** dtantsur|afk is now known as dtantsur | 07:45 | |
| dtantsur | Morning Ironic | 07:45 |
| milan | morning dtantsur :) | 07:45 |
| xuao | Morning~~~ | 07:49 |
| tangchen | Morning | 07:54 |
| tangchen | Hi guys, please take a look at this trivial fix. https://review.openstack.org/#/c/352231/ Thanks a lot. :) | 07:54 |
| *** ccamacho|afk is now known as ccamacho | 07:56 | |
| milan | morning xuao tangchen :) | 07:56 |
| tangchen | :) | 07:57 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** xuao has quit IRC | 08:02 | |
| *** lucas-dinner is now known as lucasagomes | 08:06 | |
| lucasagomes | morning all | 08:06 |
| *** vmud213 has joined #openstack-ironic | 08:07 | |
| zhangjl | hey all | 08:08 |
| dtantsur | morning milan, zhangjl, lucasagomes, tangchen, pas-ha | 08:09 |
| milan | morning lucasagomes zhangjl :) | 08:10 |
| *** xuao has joined #openstack-ironic | 08:10 | |
| openstackgerrit | Aparna proposed openstack/ironic: Update proliantutils version required for Newton release https://review.openstack.org/353301 | 08:13 |
| xuao | I want to ask that, how can I `nova boot` using the `pxe` driver rather than the `ipxe`? 😂 | 08:17 |
| *** openstackgerrit has quit IRC | 08:18 | |
| zhangjl | lucasagomes: i use pxe to deploy my physical node, but failed with the error "FATAL: No or empty root= argument" | 08:18 |
| zhangjl | could you give me some advices? | 08:18 |
| *** openstackgerrit has joined #openstack-ironic | 08:18 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 08:19 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Ignore required_services for multinode topology https://review.openstack.org/352793 | 08:19 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Source openrc on subnode in multinide topology https://review.openstack.org/353302 | 08:19 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Skip db configuration on subnodes https://review.openstack.org/353303 | 08:19 |
| *** alineb has joined #openstack-ironic | 08:19 | |
| alineb | morning all | 08:20 |
| lucasagomes | xuao, you can disable ipxe in the ironic.conf, [pxe]ipxe_enabled=False (you also need to change the pxe_bootfile_name and pxe_config_template) | 08:20 |
| lucasagomes | zhangjl, looks like your pxe configuration does have any value for the root parameter | 08:20 |
| xuao | Thank you so much~ I'll get a try!💪 | 08:21 |
| lucasagomes | zhangjl, was Ironic that generated that configuration ? If so, it may be becuase you are trying to deploy a whole disk image as a partition image | 08:21 |
| lucasagomes | so Ironic can't get the UUID from the filesystem | 08:21 |
| milan | morning alineb | 08:21 |
| milan | :) | 08:21 |
| lucasagomes | (it should fail in that case tho) | 08:21 |
| lucasagomes | alineb, morning | 08:22 |
| zhangjl | lucasagomes: my ironic.conf was generated by ironic. | 08:24 |
| zhangjl | and , did you mean i need to deploy node only with kernel and ramdisk file ? | 08:25 |
| *** amoralej|off is now known as amoralej | 08:26 | |
| zhangjl | i created my images with disk-image-create , like follows: | 08:27 |
| zhangjl | disk-image-create fedora baremetal dhcp-all-interfaces grub2 -o fedora24 | 08:27 |
| zhangjl | my steps was here: http://paste.openstack.org/show/553496/ | 08:28 |
| *** sborkows has joined #openstack-ironic | 08:35 | |
| lucasagomes | zhangjl, no I mean the PXE configuration | 08:35 |
| lucasagomes | zhangjl, not the ironic configuration | 08:36 |
| lucasagomes | the one in ur tftp or http directory (under pxelinux.cfg) | 08:36 |
| *** electrofelix has joined #openstack-ironic | 08:36 | |
| lucasagomes | zhangjl, pxelinux.cfg/01-84-2b-2b-5e-62-39 this one | 08:37 |
| zhangjl | yes, the pxe config file generated by ironic, with the template file ironic/drivers/modules/pxe_config.template | 08:39 |
| zhangjl | yes | 08:39 |
| *** permalac has quit IRC | 08:40 | |
| zhangjl | is there any problem? | 08:40 |
| lucasagomes | zhangjl, check the content, see if root= is empty | 08:45 |
| lucasagomes | if so, it may be a problem in Ironic getting the UUID of the image's filesystem | 08:45 |
| * lucasagomes finds the code | 08:45 | |
| *** ohamada has joined #openstack-ironic | 08:46 | |
| lucasagomes | zhangjl, https://github.com/openstack/ironic-lib/blob/7e926fd3fbb15b2e247e5bbc8c490d4c85857fbc/ironic_lib/disk_utils.py#L295-L300 | 08:47 |
| *** Romanenko_K has joined #openstack-ironic | 08:53 | |
| zhangjl | yes, in my pxe configuration, root= is empty. | 08:55 |
| zhangjl | my pxe configuration file is here: http://paste.openstack.org/show/553503/ | 08:56 |
| zhangjl | is it correct? | 08:58 |
| lucasagomes | zhangjl, it's not, it whould be pointing to the UUID of the root filesystem of your image | 08:59 |
| lucasagomes | zhangjl, can you see in the logs if Ironic failed to get it ? (Maybe grep for "blkid" | 08:59 |
| vdrok | good morning ironic and all! | 09:01 |
| zhangjl | grep -i blkid /var/log/ironic/* | 09:01 |
| zhangjl | i cannot find any | 09:01 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: IPMITool: Check the boot mode when setting the boot device https://review.openstack.org/352820 | 09:12 |
| lazy_prince | lucasagomes: looks like ^^ patch will take some while to settle down.. | 09:14 |
| lucasagomes | lazy_prince, yeah, many comments | 09:15 |
| lucasagomes | lazy_prince, the last patch-set should work for ya setting efiboot+persistent | 09:15 |
| *** bks has joined #openstack-ironic | 09:16 | |
| sborkows | Hi I have a problem with my patch about new pci_devices plugin in Inspector: https://review.openstack.org/#/c/352295/ Tempest logs only say that the collector 'pci-devices' cannot be loaded. Is this IPA issue? I also cannot reproduce it on my local setup. | 09:17 |
| *** vicky has joined #openstack-ironic | 09:17 | |
| *** vicky is now known as Guest42782 | 09:17 | |
| lazy_prince | lucasagomes: Yeah.. I tested it. but so many comments, I will wait for patch to settle down before I retest again.. | 09:17 |
| lucasagomes | lazy_prince, ++ totally | 09:17 |
| openstackgerrit | Merged openstack/ironic: Trivial: Remove useless function call in glance service test https://review.openstack.org/352231 | 09:17 |
| lucasagomes | lazy_prince, once it's merged I will propose a backport to mitaka as well | 09:18 |
| lucasagomes | lazy_prince, and again, thanks a lot for testing it | 09:18 |
| lazy_prince | lucasagomes: thanks for the patch.. | 09:18 |
| tuanla__ | Konichiwa, | 09:19 |
| tuanla__ | Hello, i have been tried to boot instance from whole disk image | 09:20 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Split API root unit and functional tests https://review.openstack.org/348965 | 09:21 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move api driver tests to functional dir https://review.openstack.org/350365 | 09:21 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Prepare for functional tests https://review.openstack.org/349421 | 09:21 |
| zhangjl | lucasagomes: my ironic conductor log is here: http://paste.openstack.org/show/553506/ | 09:21 |
| tuanla__ | I also got a same prolem "Unable to find requested MBR signature" even the Horizon show that the node is active and running | 09:22 |
| lazy_prince | tuanla__: can you share your flavor and the node properties thats being used.. | 09:23 |
| zhangjl | i found ironic build pxe configuration file through the method create_pxe_config https://github.com/openstack/ironic/blob/master/ironic/common/pxe_utils.py#L232 | 09:26 |
| lucasagomes | zhangjl, wait a second, can you paste the output of ironic node-show (make sure you hide ur credentials) ? | 09:26 |
| tuanla__ | lazy_prince, lucasagomes : wait me a second | 09:26 |
| zhangjl | lucasagomes: my node info is here:http://paste.openstack.org/show/553507/ | 09:27 |
| zhangjl | it`s a dell PowerEdge R510 Physical node | 09:28 |
| tuanla__ | lazy_prince: http://paste.openstack.org/show/553509/ | 09:28 |
| lucasagomes | zhangjl, a-ha found the problem... You basically are setting the driver_info/deploy_ramdisk and driver_info/deploy_kernel to the ramdisk and kernel of your image in glance | 09:29 |
| lucasagomes | zhangjl, you need to use a deployment ramdisk for it | 09:29 |
| *** e0ne has joined #openstack-ironic | 09:29 | |
| lucasagomes | zhangjl, like https://github.com/openstack/ironic-python-agent | 09:29 |
| tuanla__ | lazy_prince : and about flavor: http://paste.openstack.org/show/553510/ | 09:30 |
| lucasagomes | zhangjl, here http://docs.openstack.org/developer/ironic/deploy/install-guide.html#building-or-downloading-a-deploy-ramdisk-image | 09:31 |
| xuao | lucasagomes, If you mean we should have a `deploy_image` ? | 09:32 |
| xuao | we are doing the same thing, with zhangjl | 09:33 |
| lucasagomes | xuao, I mean, you need a deploy ramdisk | 09:33 |
| tuanla__ | lazy_prince: and here is final status i get from hardware machine: http://paste.openstack.org/show/553511/ | 09:33 |
| lucasagomes | xuao, zhangjl check the link I've just sent to you | 09:34 |
| zhangjl | thanks | 09:34 |
| zhangjl | i would do it follow your advice | 09:35 |
| xuao | get√~~I'll have try. Thank you so much for your help ~~~~ | 09:35 |
| lucasagomes | zhangjl, xuao once you build the deploy ramdisk (or download it) you need to: 1. upload it to glance | 09:36 |
| lucasagomes | 2. update the ironic node driver_info/deploy_ramdisk and driver_info/deploy_kernel to point to the deploy ramdisk and deploy kernel UUID in glance | 09:36 |
| xuao | Oh! we don't do the step 2 ~ | 09:37 |
| zhangjl | well, i did like aboved | 09:37 |
| lucasagomes | xuao, you haven't done 1. either | 09:37 |
| zhangjl | http://paste.openstack.org/show/553496/ | 09:38 |
| xuao | Oh no~ we've done it maybe | 09:38 |
| lucasagomes | zhangjl, xuao you guys are confusing tenant image (the fedora iage you built) with deploy image | 09:38 |
| lucasagomes | zhangjl, xuao the image you build is the final image, the image that will be running on the node after you deploy it | 09:38 |
| lucasagomes | where the deploy image is a special ramdisk, that Ironic uses to deploy the final image onto the node | 09:38 |
| *** mgould|afk is now known as mgould | 09:39 | |
| tuanla__ | lazy_prince, lucasagomes: In my opinion, I think I trap in Reboot step to reboots into user instance. Do you have any ways to check status or test imformation about the issue i met | 09:39 |
| mgould | morning Ironic! | 09:39 |
| lucasagomes | zhangjl, xuao here: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#image-requirements | 09:39 |
| xuao | Oh !!!! Really got it !!!!!!! | 09:39 |
| zhangjl | thanks !! | 09:39 |
| xuao | I think I've understood what you mean!!! | 09:40 |
| lucasagomes | yeah see the last bullet point of the step 1. in that guide | 09:40 |
| lucasagomes | about building ur deploy image | 09:40 |
| milan | morning mgould ! :) | 09:40 |
| lucasagomes | zhangjl, xuao once you have done that, you should be sorted :-) | 09:40 |
| xuao | If we success, must share with you at the first time!!! hah, thank you so much ! | 09:42 |
| *** mbound has joined #openstack-ironic | 09:44 | |
| *** sdake has joined #openstack-ironic | 09:47 | |
| *** Guest42782 is now known as vikrant | 09:49 | |
| *** tangchen has quit IRC | 09:50 | |
| sambetts|afk | Morning all | 09:59 |
| *** sambetts|afk is now known as sambetts | 09:59 | |
| lucasagomes | lazy_prince, newer versions of ipmitool works setting persistent+efiboot at the same time | 09:59 |
| *** psachin has joined #openstack-ironic | 10:00 | |
| lucasagomes | lazy_prince, we still need the fix tho, because the packaged versions are buggy (at least for the distros I tried) | 10:00 |
| lucasagomes | I've added another comment to that bug | 10:01 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 10:05 |
| vikrant | ping openstack-ironic all - i am trying to change the provisioning MAC address of already deployed ironic node. I have followed procedure mentioned in https://paste.fedoraproject.org/405287/ is any caveats of following this procedure ? | 10:09 |
| *** xuao has quit IRC | 10:10 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 10:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 10:10 |
| *** xuao has joined #openstack-ironic | 10:12 | |
| *** xuao has quit IRC | 10:14 | |
| *** xuao has joined #openstack-ironic | 10:15 | |
| *** xuao has quit IRC | 10:17 | |
| *** tangchen has joined #openstack-ironic | 10:19 | |
| milan | morning sambetts | 10:22 |
| *** xuao has joined #openstack-ironic | 10:22 | |
| mgould | morning sambetts milan xuao zhangjl lucasagomes | 10:22 |
| zhangjl | lucasagomes: Thanks for your help!!!! | 10:22 |
| lucasagomes | zhangjl, did it work? | 10:23 |
| zhangjl | yes!!! | 10:23 |
| xuao | lucasagomes, Wahahahahaahah!!! | 10:23 |
| xuao | We have succeed!!!!! | 10:23 |
| lucasagomes | xuao, zhangjl great :-) | 10:23 |
| mgould | \o. | 10:23 |
| mgould | \o/, rather | 10:23 |
| xuao | morning ~ mgould | 10:23 |
| xuao | Thank you so much!!! mua~~~~lucasagomes | 10:24 |
| lucasagomes | you welcome! Glad it worked for you | 10:25 |
| *** tiendc has quit IRC | 10:27 | |
| sambetts | hey mgould milan lucasagomes o/ | 10:27 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: IPMITool: Check the boot mode when setting the boot device https://review.openstack.org/352820 | 10:27 |
| lucasagomes | sambetts, hi there | 10:27 |
| *** xdreamer has joined #openstack-ironic | 10:28 | |
| *** xdreamer has left #openstack-ironic | 10:31 | |
| *** xdreamer has joined #openstack-ironic | 10:32 | |
| *** xdreamer has left #openstack-ironic | 10:32 | |
| *** vmud213 has quit IRC | 10:32 | |
| lazy_prince | tuanla__: what image is this..? Windows..? MBR based or GPT based..? | 10:32 |
| *** mbound has quit IRC | 10:33 | |
| tuanla__ | lazy_prince, : I am trying into cirros first | 10:33 |
| openstackgerrit | Yuiko Takada Mori proposed openstack/ironic: Add node serial console documentation https://review.openstack.org/293872 | 10:33 |
| tuanla__ | but i also try in ubuntu but it have same problem | 10:33 |
| tuanla__ | lazy_prince: please see http://paste.openstack.org/show/553522/ | 10:34 |
| tuanla__ | . | 10:34 |
| tuanla__ | lazy_prince: It the last message I get before messaage "Unable...." | 10:35 |
| lazy_prince | tuanla__: can you share the glance image-show for the image..? I still suspect image as issue.. | 10:36 |
| *** zhangjl has left #openstack-ironic | 10:37 | |
| *** vmud213 has joined #openstack-ironic | 10:39 | |
| tuanla__ | lazy_prince: | 10:39 |
| tuanla__ | http://paste.openstack.org/show/553542/ | 10:39 |
| *** xuao has quit IRC | 10:40 | |
| *** mkoderer_ has joined #openstack-ironic | 10:42 | |
| *** mkoderer_ has quit IRC | 10:42 | |
| *** hoangcx has quit IRC | 10:44 | |
| *** rajinir has quit IRC | 10:45 | |
| *** tuanla__ has quit IRC | 10:46 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs: Add support for hardware composition https://review.openstack.org/253452 | 10:52 |
| *** permalac has joined #openstack-ironic | 10:53 | |
| *** xavierr has quit IRC | 10:56 | |
| *** xavierr has joined #openstack-ironic | 10:56 | |
| *** dprince has joined #openstack-ironic | 10:59 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 11:04 |
| *** vikrant is now known as vikrant|T | 11:11 | |
| *** stendulker has quit IRC | 11:12 | |
| sborkows | Hi I have a problem with my patch about new pci_devices plugin in Inspector: https://review.openstack.org/#/c/352295/ Tempest logs only say that the collector 'pci-devices' cannot be loaded. Is this IPA issue? I also cannot reproduce it on my local setup. | 11:18 |
| dtantsur | sborkows, I think the prebuild ramdisk is outdated and does not get build, hence your problem. IIRC sambetts is working on fixing it. | 11:19 |
| sborkows | dtantsur, ok thanks for info, so it seems I need to wait to check it out | 11:20 |
| *** e0ne has quit IRC | 11:20 | |
| *** cmyster has quit IRC | 11:21 | |
| *** lucasagomes is now known as lucas-hungry | 11:22 | |
| dtantsur | sambetts, any news re ^^^? | 11:24 |
| sambetts | dtantsur: just looking into it now, the "fix" has merged, but I'm seeing other failures now :'( | 11:25 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor multitenant networking release notes https://review.openstack.org/352659 | 11:28 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor OneView dynamic allocation release notes https://review.openstack.org/353049 | 11:28 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up release notes for 6.1.0 https://review.openstack.org/352660 | 11:28 |
| jroll | whee | 11:28 |
| jroll | morning y'all | 11:28 |
| sambetts | hey jroll | 11:28 |
| sambetts | dtantsur: https://review.openstack.org/#/c/353393/ :'( | 11:30 |
| dtantsur | sborkows, also left some review comments | 11:30 |
| dtantsur | morning jroll | 11:30 |
| *** vmud213 has quit IRC | 11:31 | |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Deprecate -l option for port creation in OSC plugin https://review.openstack.org/351836 | 11:32 |
| vdrok | morning jroll | 11:33 |
| mat128 | morning jroll, sambetts, dtantsur and vdrok | 11:33 |
| sambetts | hey mat128 | 11:33 |
| dtantsur | morning mat128 | 11:34 |
| vdrok | morning mat128 | 11:34 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Deprecate ClusteredComputeManager https://review.openstack.org/352991 | 11:34 |
| dtantsur | hmm, it seems like ironic gives HTTP 500 when updating a port bound to an instance... | 11:35 |
| sborkows | dtantsur, thanks for review. | 11:35 |
| *** fragatina has joined #openstack-ironic | 11:36 | |
| *** aparnav has quit IRC | 11:38 | |
| *** fragatina has quit IRC | 11:41 | |
| *** vikrant|T is now known as vikrant | 11:41 | |
| *** ccamacho is now known as ccamacho|lunch | 11:42 | |
| *** bfournie has quit IRC | 11:42 | |
| mat128 | sambetts: have you need the new POST job output now? http://logs.openstack.org/df/dfaf92b611d7c4abb98345fbcf21ef3ed4bb9d3b/post/ironic-python-agent-buildimage-tinyipa/be57153/console.html#_2016-08-09_23_24_31_001521 | 11:42 |
| mat128 | s/need/seen | 11:42 |
| jroll | mat128: he has :P https://review.openstack.org/#/c/353393/1/jenkins/jobs/ironic-python-agent-jobs.yaml | 11:43 |
| mat128 | ah :) | 11:43 |
| mat128 | thanks | 11:44 |
| sambetts | mat128: yup :) so frustrating, I must have zoned out when I wrote the original patch | 11:48 |
| *** ohamada has quit IRC | 11:49 | |
| *** ohamada has joined #openstack-ironic | 11:49 | |
| *** amotoki has quit IRC | 11:51 | |
| *** sdake has quit IRC | 11:51 | |
| *** sdake has joined #openstack-ironic | 11:51 | |
| *** athomas has quit IRC | 11:52 | |
| *** jcoufal has joined #openstack-ironic | 11:54 | |
| *** amotoki has joined #openstack-ironic | 11:56 | |
| *** sdake_ has joined #openstack-ironic | 11:56 | |
| *** sborkows has quit IRC | 11:57 | |
| *** sborkows has joined #openstack-ironic | 11:57 | |
| *** athomas has joined #openstack-ironic | 11:58 | |
| *** sdake has quit IRC | 11:59 | |
| *** amotoki has quit IRC | 12:00 | |
| jroll | could folks give these a look today, please? https://review.openstack.org/#/q/topic:6.1-reno | 12:01 |
| jroll | hoping to land those so I can release 6.1.0 | 12:01 |
| *** links has quit IRC | 12:01 | |
| vsaienk0 | jroll: http://logs.openstack.org/32/296432/89/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa-multinode-nv/1f75f2e/console.html | 12:04 |
| *** bks has quit IRC | 12:04 | |
| sambetts | vsaienk0: multinode job passed? | 12:05 |
| jroll | vsaienk0: O_O \o/ | 12:05 |
| sambetts | vsaienk0: is it actually running across both nodes? | 12:05 |
| vsaienk0 | we still have a lot of work to be done on multinode stuff, but setup with 2 nodes, on subnode n-cpu, ir-api and conductor passed | 12:05 |
| sambetts | \o/ | 12:05 |
| *** rbudden has joined #openstack-ironic | 12:05 | |
| jroll | looks like it http://logs.openstack.org/32/296432/89/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa-multinode-nv/1f75f2e/logs/subnode-2/ | 12:06 |
| jroll | that's so cool | 12:06 |
| sambetts | thats awesome | 12:06 |
| jroll | vsaienk0: what's that running, --regex ironic? | 12:06 |
| dtantsur | so, re my question about live-updating ports: https://bugzilla.redhat.com/show_bug.cgi?id=1365881 | 12:07 |
| openstack | bugzilla.redhat.com bug 1365881 in openstack-ironic "Not able to update/replace the ironic node provisoning MAC address" [Medium,New] - Assigned to lmartins | 12:07 |
| aarefiev | vsaienk0: awesome | 12:07 |
| aarefiev | morning all! | 12:07 |
| dtantsur | it seems like we try to call neutron which asks us to go to hell | 12:07 |
| dtantsur | morning aarefiev | 12:07 |
| sambetts | dtantsur: yup, its because you have to unbind the port first before updated the mac, which I don't think we do | 12:08 |
| dtantsur | hmm, I wonder if it can be fixed at all | 12:08 |
| dtantsur | replacing a NIC does not sound unrealistic | 12:08 |
| * dtantsur files an lp bug now | 12:08 | |
| jroll | hm, no ilo CI maintainers in channel | 12:09 |
| jroll | :( | 12:09 |
| jroll | bbiab | 12:09 |
| *** dprince has quit IRC | 12:10 | |
| *** dprince has joined #openstack-ironic | 12:11 | |
| dtantsur | jroll, sambetts, filed https://bugs.launchpad.net/ironic/+bug/1611744 wdyt? | 12:11 |
| openstack | Launchpad bug 1611744 in Ironic "HTTP 500 when trying to update port MAC for an active node" [Undecided,New] | 12:11 |
| sambetts | dtantsur: have you seen https://paste.fedoraproject.org/405287/ | 12:12 |
| dtantsur | I'd at least fix HTTP 500, but I'd prefer also to make it possible to actually update | 12:12 |
| sambetts | dtantsur: it kind of discribes what you have to do on the neutron side, and we fail to do | 12:12 |
| dtantsur | sambetts, yeah, that's what I saw today | 12:12 |
| dtantsur | so I wonder if it counts as a valid RFE | 12:13 |
| sambetts | dtantsur: https://github.com/openstack/ironic/blob/master/ironic/dhcp/neutron.py#L73 | 12:13 |
| sambetts | dtantsur: we should do the unbind in there ^ | 12:13 |
| sambetts | I guess its a bug atm because we do actually support that update but it just broken | 12:13 |
| dtantsur | sambetts, so it's only matter of doing step 4 and 6 from https://paste.fedoraproject.org/405287/ ? | 12:14 |
| dtantsur | cool | 12:14 |
| sambetts | dtantsur: yeah | 12:14 |
| *** amotoki has joined #openstack-ironic | 12:15 | |
| * dtantsur will try fixing it | 12:15 | |
| *** bvandewa has quit IRC | 12:15 | |
| sambetts | I hope all this sort of stuff moves into the network providers soon, because we manipulate the host:id and bindings in there, and I would like it to be consistent | 12:16 |
| *** thiagop has joined #openstack-ironic | 12:16 | |
| sambetts | dtantsur: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/network/neutron.py#L180 | 12:16 |
| *** e0ne has joined #openstack-ironic | 12:17 | |
| openstackgerrit | Teng Fei proposed openstack/ironic-inspector: Modify the SQL's string from double quote to single quote. https://review.openstack.org/347795 | 12:18 |
| *** jcoufal has quit IRC | 12:18 | |
| thiagop | Good morning, Ironic | 12:19 |
| sambetts | thiagop o/ | 12:19 |
| *** baoli has joined #openstack-ironic | 12:19 | |
| *** bfournie has joined #openstack-ironic | 12:19 | |
| *** baoli_ has joined #openstack-ironic | 12:20 | |
| *** phuongnh has quit IRC | 12:21 | |
| *** MattMan has quit IRC | 12:21 | |
| *** MattMan_ has joined #openstack-ironic | 12:22 | |
| *** baoli has quit IRC | 12:24 | |
| openstackgerrit | Teng Fei proposed openstack/ironic-inspector: Modify the SQL's string from double quote to single quote. https://review.openstack.org/347795 | 12:26 |
| *** amotoki has quit IRC | 12:28 | |
| jroll | dtantsur: sounds good to me | 12:28 |
| *** lucas-hungry is now known as lucasagomes | 12:29 | |
| *** mtanino has joined #openstack-ironic | 12:30 | |
| lucasagomes | hi all, can I get some eyes at https://review.openstack.org/#/c/344179/ ? This is the last bit of code to get the cleaning step to erase the disks metadata. The IPA part is all merged already | 12:31 |
| lucasagomes | thanks :-) | 12:32 |
| *** ccamacho|lunch is now known as ccamacho | 12:33 | |
| *** jcoufal has joined #openstack-ironic | 12:35 | |
| *** e0ne has quit IRC | 12:43 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add openstack baremetal driver commands https://review.openstack.org/350050 | 12:43 |
| *** vikrant has quit IRC | 12:43 | |
| *** e0ne has joined #openstack-ironic | 12:43 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Fix updating port MAC address for active nodes https://review.openstack.org/353450 | 12:47 |
| dtantsur | sambetts, does it make any sense ^^^? | 12:47 |
| *** baoli_ has quit IRC | 12:48 | |
| *** thrash|g0ne is now known as thrash | 12:48 | |
| *** rloo has joined #openstack-ironic | 12:50 | |
| sambetts | dtantsur: looks pretty good to me, although it needs some UTs to prevent regressing back to a broken state, and I'd love to see us cover an operation like this in the tempest tests but I don't know how easy that would be | 12:50 |
| dtantsur | sambetts, yeah, I wanted to make sure it's not completely dumb before diving into unit tests and local testing.. | 12:51 |
| *** amoralej is now known as amoralej|lunch | 12:51 | |
| dtantsur | sambetts, as to tempest... we need to somehow simulate replacing the NIC.. not impossible but requires messing with virsh, I guess | 12:51 |
| sambetts | yeah true, I was thinking more of just an API level test, e.g. making sure all the right calls were made to neutron and that it didn't blow up, rather than testing a full scenario | 12:52 |
| sambetts | logic looks sound based on the paste from earlier though :) | 12:53 |
| *** mtanino has quit IRC | 12:56 | |
| dtantsur | cool, I'll try it locally anyway before landing | 12:57 |
| sambetts | awesome :D | 12:57 |
| * dtantsur has to update his devstack | 12:58 | |
| TheJulia | Wouldn't somehow touching virsh locally break the ability for tempest to be executed remotely? | 13:01 |
| *** priteau has joined #openstack-ironic | 13:02 | |
| sambetts | I not sure what would break if we added a call in our tests to update the virsh domain to have a different mac address :/ | 13:02 |
| sambetts | hopefully if there is an instance running on that domain, and then we update it, and update all of Ironic and neutron, then the instance should be OK, but :/ | 13:04 |
| TheJulia | sambetts: I know we run the tests remotely here... so that definitely wouldn't work here :( | 13:05 |
| sambetts | I'm not sure what you mean we run them remotely, we don't run the tests inside the fake BM VM | 13:05 |
| sambetts | do we? | 13:05 |
| TheJulia | we have teams that run tempest jobs against full clouds including real baremetal | 13:06 |
| dtantsur | sambetts, do you think we should prevent updating MACs for nodes with instance_uuid and not in maintenance mode? | 13:06 |
| sambetts | dtantsur: we already do :) | 13:07 |
| dtantsur | I don't think we have special code for it, except for the error I'm fixing | 13:08 |
| sambetts | dtantsur: we prevent any update to the port unless in enroll, inspecting, management or maintenance | 13:08 |
| dtantsur | sambetts, it's only to local_link_connection | 13:08 |
| dtantsur | not any updates | 13:08 |
| dtantsur | sambetts, https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1660-L1668 | 13:09 |
| sambetts | oh :/ I misread the code, I thought it was all updates :/ yeah current code allows it, so I'm not sure | 13:10 |
| dtantsur | yeah, but it does not work currently, soo... we have some freedom :) | 13:10 |
| vsaienk0 | team, please help to review: https://review.openstack.org/#/c/340695/ switching to flat network by default. It is required for multinode job testing | 13:11 |
| sambetts | TheJulia: oh so you mean if we do anything involving virsh in the tests it'll not work in test environments against real hardware | 13:11 |
| TheJulia | sambetts: exactly | 13:11 |
| dtantsur | vsaienk0, the dependency https://review.openstack.org/#/c/338116/ is in merge conflict | 13:11 |
| openstackgerrit | Michael Turek proposed openstack/ironic: Add classes for Portgroups API enablement https://review.openstack.org/347549 | 13:12 |
| dtantsur | and it in turn depends on https://review.openstack.org/#/c/338117/ | 13:12 |
| dtantsur | and it depends on the patch you mention Oo | 13:12 |
| dtantsur | vsaienk0, looks like you have a dependency loop ^^^ | 13:12 |
| sambetts | TheJulia: yeah, thats why I think an API only test with the fake driver would be a good idea for ensuring that the calls to neutron happen, but its hard to test the real scenario | 13:13 |
| vsaienk0 | dtantsur: there is no loop, inspector patch depends on 2nd patch in ironic chain, 3rd patch in ironic chain depends on ironic-inspector | 13:14 |
| *** amotoki has joined #openstack-ironic | 13:15 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-inspector: Use IRONIC_TEST_NETWORK_NAME to boot instance. https://review.openstack.org/338116 | 13:15 |
| *** PollyZ has joined #openstack-ironic | 13:17 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/python-oneviewclient: Add auditing instrumentation for python-oneviewclient https://review.openstack.org/337281 | 13:17 |
| *** sdake_ is now known as sdake | 13:18 | |
| *** appu103_ has joined #openstack-ironic | 13:22 | |
| *** harlowja has quit IRC | 13:23 | |
| sambetts | dtantsur: tbh I'm interested if things like the local_link_connection could be updated in the same way the mac_address here, e.g. if we unbind and then update everything we know about the port and then rebind it | 13:23 |
| sambetts | because right now if you put an active node in maintenace then update the local_link_connection it won't actually update neutron atm | 13:24 |
| dtantsur | yeah, interesting | 13:25 |
| *** vishwanathj has joined #openstack-ironic | 13:30 | |
| sambetts | dtantsur: I'm thinking about an update_connectivity or sync_connectivty function in the network providers for superceeding the dhcp provider update_port_address function | 13:31 |
| jroll | sambetts: well, llc would be updated when you move a cable to a different switchport, right? | 13:31 |
| jroll | and in that case, the unbind wouldn't actually re-configure the other switchport | 13:32 |
| jroll | which is the frightening part of doing that | 13:32 |
| sambetts | the unbind would unconfigure the old switch, and then the rebind with the new llc would bind the new switch right/ | 13:32 |
| sambetts | ? | 13:32 |
| sambetts | (states expected behaviour, not tested behaviour) | 13:33 |
| * sambetts states ^ | 13:33 | |
| jroll | sambetts: oh wow, I was thinking about that wrong | 13:34 |
| sambetts | oh. | 13:34 |
| sambetts | ? | 13:34 |
| jroll | for some reason was thinking the cable needed to be connected to deconfigure the switch | 13:34 |
| sambetts | ooooh XD | 13:34 |
| jroll | that'd be interesting, hopefully the ML2 plugins do the right thing :) | 13:35 |
| jroll | I know port-create with binding:host_id: NULL won't even get to the ML2 plugins | 13:35 |
| jroll | I wonder what port-update does in that case, call unbind() ? | 13:35 |
| sambetts | yeah as I understand it | 13:35 |
| *** ametts has joined #openstack-ironic | 13:36 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack guide: Bump IRONIC_VM_SPECS_RAM to 1280 https://review.openstack.org/353474 | 13:37 |
| sambetts | so it should unconfigure the switches, and then we can update the binding profile and rebind with new llc pointing at the new switch | 13:37 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Fix updating port MAC address for active nodes https://review.openstack.org/353450 | 13:37 |
| dtantsur | sambetts, code finished ^^ haven't tested it yet | 13:37 |
| sambetts | and if it all works as expected we should be able to move active nodes between switches | 13:38 |
| jroll | with a blip in service, but that's expected :) | 13:38 |
| *** [1]cdearborn has joined #openstack-ironic | 13:39 | |
| sambetts | yeah | 13:39 |
| *** joprovost has joined #openstack-ironic | 13:39 | |
| sambetts | dtantsur: in the conductor manager change, couldn't we just add address to the connectivity_attr list? | 13:40 |
| dtantsur | sambetts, it will ban updating "available" nodes, which works now | 13:41 |
| sambetts | hmm... why do we ban that for the other things I wondered | 13:42 |
| sambetts | wonder* | 13:42 |
| dtantsur | dunno... | 13:42 |
| sambetts | jroll: any idea ^ | 13:43 |
| sambetts | only reason I could think is if we're scheduling on those fields and we want to prevent a race, but we don't as far as I know | 13:43 |
| sambetts | unless something something long running ramdisks comes into this somewhere | 13:44 |
| *** amoralej|lunch is now known as amoralej | 13:44 | |
| jroll | kinda feel like we shouldn't update mac address for available, but if we do it today... sigh | 13:44 |
| *** rbartal has quit IRC | 13:45 | |
| dtantsur | yeah, when we first allowed that we didn't have manageable state :) | 13:45 |
| jroll | indeed | 13:46 |
| sambetts | rm -rf APIv1 | 13:46 |
| jroll | this isn't in the API, funny enough :D | 13:46 |
| sambetts | huh? updating the ports address and what state the nodes in isn't part of the API? | 13:48 |
| sambetts | i guess its a behaviour thing rather than an actual API thing | 13:50 |
| *** hjanssen-hpe has joined #openstack-ironic | 13:51 | |
| *** hj-hpe has joined #openstack-ironic | 13:51 | |
| dtantsur | oh, when we implement APIv2, we'll have to move a lot of checks to the API level... or have two sets of conductor RPC APIs | 13:51 |
| sambetts | ooooh you mean its only checked once its passed to the conductor, not in the API itself | 13:52 |
| jroll | sambetts: it's in conductor manager | 13:52 |
| jroll | yeah | 13:52 |
| sambetts | I guess this is a case of conductor API affecting Ironic REST API oooo *mind blown* | 13:52 |
| jroll | lol | 13:53 |
| *** wajdi has joined #openstack-ironic | 13:53 | |
| jroll | I mean, really the check should be in both places | 13:53 |
| jroll | the conductor side prevents races | 13:53 |
| sambetts | so IMO the exsting check we added for the llc and other connectivty options shouldn't be in the conductor manager, it should be in the API | 13:55 |
| sambetts | REST API that is | 13:55 |
| sambetts | then the conductor side only does things like the if address change then do these extra things | 13:55 |
| sambetts | but :/ | 13:55 |
| sambetts | there is probably some problem to do with avaible information or something | 13:56 |
| jroll | well, the API doesn't have the lock, is the thing | 13:56 |
| jroll | so state could change | 13:56 |
| *** PollyZ has quit IRC | 13:57 | |
| *** rama_y has joined #openstack-ironic | 13:58 | |
| sambetts | hmm, interesting, so we can't even read the state in the API, unless we have a contract between the conductor and API that we pass the state we read in the API without the lock and then the conductor checks that it hasn't changed since we read it | 13:59 |
| *** anush_ has joined #openstack-ironic | 14:00 | |
| jroll | mhm | 14:00 |
| sambetts | e.g. in the conductor have if state_passed_from_api is not node.state: raise StateHasChanged('Can not proceed with transaction because node has changed state since API call was made') | 14:01 |
| *** rama_y has quit IRC | 14:01 | |
| *** rama_y has joined #openstack-ironic | 14:01 | |
| *** PollyZ has joined #openstack-ironic | 14:03 | |
| *** joprovost1 has joined #openstack-ironic | 14:04 | |
| *** joprovost has quit IRC | 14:06 | |
| *** joprovost1 is now known as joprovost | 14:06 | |
| *** rama_y has quit IRC | 14:07 | |
| *** catintheroof has joined #openstack-ironic | 14:09 | |
| *** Lord_Phoenix has quit IRC | 14:10 | |
| *** marulkan1 has quit IRC | 14:23 | |
| *** syed_ has joined #openstack-ironic | 14:23 | |
| *** mtanino has joined #openstack-ironic | 14:30 | |
| *** athomas has quit IRC | 14:32 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor multitenant networking release notes https://review.openstack.org/352659 | 14:32 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor OneView dynamic allocation release notes https://review.openstack.org/353049 | 14:32 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up release notes for 6.1.0 https://review.openstack.org/352660 | 14:32 |
| jroll | dtantsur: fixed your things :) | 14:32 |
| dtantsur | cool | 14:32 |
| dtantsur | I'll wait for the release note job to publish results before voting | 14:33 |
| openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC RAID configuration https://review.openstack.org/176319 | 14:33 |
| ifarkas | dtantsur, you were right about the periodic task^ | 14:34 |
| openstackgerrit | Merged openstack/ironic: Use devstack test-config phase https://review.openstack.org/351650 | 14:34 |
| ifarkas | dtantsur, the mistake I made previously was that I defined it on the module, not on the class | 14:34 |
| dtantsur | aha, I see. good, I hope we can fit it in Newton still | 14:35 |
| ifarkas | yeah | 14:35 |
| ifarkas | jroll, hi, any chance we can add drac raid patch to the newton schedule? i think it's in a pretty good shape, it already had a +2 from lucasagomes | 14:36 |
| openstackgerrit | Merged openstack/ironic: Check keyword arguments https://review.openstack.org/351677 | 14:36 |
| openstackgerrit | Merged openstack/ironic: Updated Dev quickstart for viewing doc changes https://review.openstack.org/350705 | 14:36 |
| dtantsur | +2, we need some non-Red Hat review on it | 14:37 |
| lucasagomes | ++ to include it, since iLO already has the code in place | 14:37 |
| lucasagomes | it would be good to have RAID for DRAc too | 14:37 |
| jroll | I'm okay with that, but one question | 14:41 |
| *** rbrud has joined #openstack-ironic | 14:42 | |
| jroll | I haven't seen dell CI running anytime recently | 14:42 |
| jroll | will it be running in the next few weeks? | 14:42 |
| jroll | ifarkas: ^ | 14:42 |
| jroll | also, wow that's a large patch | 14:42 |
| ifarkas | yeah, sorry | 14:43 |
| ifarkas | jroll, as for the CI, I am not sure about the status but will check with cdearborn | 14:43 |
| *** athomas has joined #openstack-ironic | 14:43 | |
| *** appu103_ has quit IRC | 14:44 | |
| *** appu103_ has joined #openstack-ironic | 14:44 | |
| lucasagomes | dtantsur, mind taking a quick look at https://review.openstack.org/#/c/344179/ when you have time ? | 14:44 |
| lucasagomes | fairly small patch, would be good to have it included in the next realease if possible | 14:45 |
| *** appu103_ is now known as aparna | 14:45 | |
| jroll | ifarkas: ok, thanks, no sense in merging that if we're going to drop the driver anyway | 14:45 |
| dtantsur | sure, but now I'm trying to figure out why nova-compute can't access ironic | 14:45 |
| dtantsur | jroll, maybe worth emailing official contacts for this CI? | 14:46 |
| openstackgerrit | Sam Betts proposed openstack/ironic-specs: Add spec for interface attach and detach API https://review.openstack.org/317636 | 14:46 |
| ifarkas | jroll, afaik the goal is to have a fully functional ci | 14:46 |
| * ifarkas writes an email now | 14:46 | |
| * lucasagomes is expecting drac to have a CI indeed | 14:46 | |
| jroll | dtantsur: yeah, I'm in the middle of that, emailed some other CIs this morning | 14:47 |
| jroll | ifarkas: alright, thanks | 14:47 |
| jroll | lucasagomes: some things on your reno in case that makes it, trying to ship the release today | 14:47 |
| lucasagomes | jroll, thanks a lot! Will fix right away | 14:48 |
| jroll | cool, ty | 14:48 |
| *** rbrud has quit IRC | 14:49 | |
| *** rbrud has joined #openstack-ironic | 14:49 | |
| * dtantsur wishes keystone had meaningful logs in case of authentication problems... | 14:50 | |
| * krtaylor ears perks up on CI driver drops... | 14:50 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add "erase_devices_metadata_priority" config option https://review.openstack.org/344179 | 14:50 |
| lucasagomes | should be good now ^ (/me rebased the patch as well) | 14:50 |
| *** rbrud has left #openstack-ironic | 14:51 | |
| *** joprovost1 has joined #openstack-ironic | 14:51 | |
| jroll | krtaylor: there's an openstack-dev email and a couple emails in your gmail inbox about this too :) | 14:51 |
| *** rbrud has joined #openstack-ironic | 14:51 | |
| krtaylor | jroll, sorry, still catching up, thats what I get for taking a silly vacation | 14:51 |
| jroll | krtaylor: no worries, they're all from this morning | 14:51 |
| krtaylor | jroll, I'll read up and then ask my questions :) | 14:52 |
| jroll | cool, shuoldn't be any surprises :) | 14:52 |
| * jroll steps away for a bit | 14:52 | |
| *** joprovost has quit IRC | 14:53 | |
| *** joprovost1 is now known as joprovost | 14:53 | |
| *** rbrud has quit IRC | 14:53 | |
| *** rbrud has joined #openstack-ironic | 14:53 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Tempest: increase ironic sync timeout https://review.openstack.org/350499 | 14:53 |
| aparna | Hi dtantsur, What do you think about this (https://bugs.launchpad.net/ironic/+bug/1593194/comments/9) suggestion for the generic way to configure clean steps? | 14:56 |
| openstack | Launchpad bug 1593194 in Ironic "[RFE]Add hardware disk erase as cleaning step" [Undecided,Incomplete] - Assigned to Aparna (aparnavtce) | 14:56 |
| dtantsur | aparna, I'm even fine with configuration options, it just should not be specific to one clean step | 14:57 |
| dtantsur | aparna, e.g. oslo.config supports dict types, you can define a dict with all overrides | 14:57 |
| *** causten has quit IRC | 14:58 | |
| *** causten has joined #openstack-ironic | 14:59 | |
| rloo | hi dtantsur, TheJulia, sambetts: that driver composition defaults meeting, is it to discuss sambetts ' email? | 15:00 |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Fail with more meaningful error while creating client https://review.openstack.org/344791 | 15:00 |
| *** d0ugal has quit IRC | 15:01 | |
| dtantsur | rloo, I don't remember whose email is was, but it's about having or not having defaults for all drivers, and if yes, how to treat them | 15:01 |
| *** sborkows has quit IRC | 15:01 | |
| rloo | dtantsur: ok, going to go through the emails etc. I'm behind... | 15:01 |
| sambetts | rloo, dtantsur: I think my email is the only one that has a full alternative to the current suggested method of handling defaults | 15:03 |
| rloo | sambetts: isn't it because of you (or method) that we're having this meeting? And perhaps to deal with how we're handling node.network_interface... | 15:04 |
| aparna | When you say dict with all the overrides, do you mean adding a generic clean step priority from which other third party clean steps overrides? | 15:04 |
| *** sdake_ has joined #openstack-ironic | 15:04 | |
| aparna | dtantsur, When you say dict with all the overrides, do you mean adding a generic clean step priority from which other third party clean steps overrides? | 15:04 |
| sambetts | rloo: probably, I wanted to have system that works for all drivers, and all interfaces, so we don't have to think about the special cases like network_interface etc | 15:05 |
| dtantsur | aparna, something like CONF.deploy.clean_step_priority_overrides which is a dict {'step name': new priority} | 15:05 |
| rloo | sambetts: ++ | 15:06 |
| *** sdake has quit IRC | 15:07 | |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Explicitly declare strings as binary https://review.openstack.org/353556 | 15:07 |
| sambetts | rloo, TheJulia, dtantsur: I'm not going to make this meeting today, I have an appointment I have to go too, I didn't even see the doodle poll | 15:07 |
| sambetts | :'( | 15:07 |
| rloo | sambetts: oh oh. how effective is this meeting going to be, w/o you? | 15:08 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Remove unused variables assignments in OSC plugin unit tests https://review.openstack.org/353558 | 15:08 |
| jlvillal | Good morning Ironic | 15:10 |
| TheJulia | rloo: sambetts: I think it can be effective, and I like sam's idea tbh, but I worry about ability to leverage third party drivers | 15:10 |
| jlvillal | vsaienk0: Would you have time to do a chat about the multi-node work? | 15:10 |
| *** david-lyle has quit IRC | 15:12 | |
| sambetts | rloo, TheJulia: I hope my email explains things well enough that you'll be able to disucss it without me, I've talked to dtantsur and jroll about it on here before so they should be able to clarify some points with regards to implictness etc | 15:12 |
| *** david-lyle has joined #openstack-ironic | 15:12 | |
| *** aparna has quit IRC | 15:12 | |
| rloo | sambetts: I just read your email. I like the idea. How does that work with 1. node.FOO_interface (and in light of how we are handling network_interface; 2. config option default_FOO_interface? | 15:13 |
| sambetts | rloo: so the goal is to not have a default_FOO_interface separate from the enabled_FOO_interface config option | 15:15 |
| rloo | sambetts: a hardware_type.default_FOO_interface. Don't we have configuration options default_FOO_interface? | 15:15 |
| *** fragatina has joined #openstack-ironic | 15:16 | |
| *** fragatina has quit IRC | 15:16 | |
| sambetts | there would be no hardware_type.default_FOO_interface only hardware_type.FOO_interface = [a list of interface in order of preference by the vendor] | 15:17 |
| openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC RAID configuration https://review.openstack.org/176319 | 15:17 |
| *** fragatina has joined #openstack-ironic | 15:17 | |
| rloo | sambetts: the change that was made to node.network_interface means that node.network_interface always reflects the actual interface. Would it mean that after your algorithm, node.FOO_interface would be set to the determined value? | 15:17 |
| ifarkas | dtantsur, sorry, forgot to fix on of you comments previously ^ | 15:18 |
| rloo | sambetts: right, i understand that there won't be a hardware_type.default_FOO_interface. But we still have configuration option default_FOO_interface. How would that config option be used? | 15:18 |
| sambetts | rloo: no, no config option default_FOO_inteface either | 15:18 |
| rloo | sambetts: oh, i missed that part. and why did folks want to add those configuration options? | 15:18 |
| *** sabeen has joined #openstack-ironic | 15:19 | |
| sambetts | because currently network_interface is a special case | 15:19 |
| *** vmud213 has joined #openstack-ironic | 15:19 | |
| jroll | rloo: people wanted to add the default configs for ops convenience | 15:19 |
| rloo | sambetts: so with your solution, network_interface would work properly (ie, as folks want?) and the volume stuff too? | 15:19 |
| sambetts | rloo: right, it would behave the same as all the other interfacs | 15:20 |
| rloo | jroll: the default configs were meant to override the hardware_type choices, right? | 15:20 |
| jroll | correct | 15:20 |
| rloo | sambetts: ^^ so the operator would specify their desired interface via the enabled_FOO_interface config? | 15:20 |
| sambetts | rloo: and it would use the enabled_network_interfaces field + the hardware_types.network_interfaces list to determine the network_interface to use | 15:20 |
| *** appu103_ has joined #openstack-ironic | 15:20 | |
| *** appu103_ is now known as aparna | 15:20 | |
| aparna | dtantsur, Got it, Thank you :) | 15:21 |
| sambetts | rloo: right, so if the operator only want to use neutron they just set enabled_network_interfaces = neutron | 15:21 |
| *** sabeen1 has joined #openstack-ironic | 15:21 | |
| rloo | jroll: so would sambetts 's solution address the operator's needs? | 15:21 |
| *** aparna has quit IRC | 15:21 | |
| sambetts | rloo: if they want neutron and flat then they set enabled_network_interfaces = neutron,flat and then it uses the hardware_tpyes preference to pick which to use | 15:22 |
| *** jrist has quit IRC | 15:22 | |
| * rloo thinks so | 15:22 | |
| jroll | rloo: I need to re-read, honestly | 15:22 |
| rloo | jroll: no worries. we can discuss at meeting :) | 15:22 |
| sambetts | super super bummed im missing this meeting :'( | 15:22 |
| rloo | i'm trying to pick sambetts brains while he is here... | 15:22 |
| jroll | rloo: yeah, I had planned to do some listening, sad that sam is missing | 15:22 |
| sambetts | I got to head out now | 15:23 |
| rloo | oh, ciao sambetts. | 15:23 |
| jlvillal | sambetts: Good night | 15:23 |
| TheJulia | goodnight sambetts | 15:23 |
| *** jrist has joined #openstack-ironic | 15:23 | |
| sambetts | thanks guys, if there is any issues, drop me and email and I'll try to clarify on the ML if not | 15:23 |
| *** sabeen has quit IRC | 15:23 | |
| sambetts | s/and/an | 15:24 |
| sambetts | night all | 15:24 |
| *** sambetts is now known as sambetts|afk | 15:24 | |
| mat128 | good night sambetts | 15:24 |
| rloo | TheJulia: were were going to use the same etherpad or a new one: https://etherpad.openstack.org/p/ironic_network_interface_discussion | 15:24 |
| devananda | morning, all | 15:26 |
| jlvillal | morning devananda | 15:27 |
| TheJulia | rloo: I honestly had not thought about it. Truly, I just want us all to get on the same page and have consensus on a path forward. | 15:27 |
| rloo | morning devananda | 15:27 |
| rloo | TheJulia: I was going to jot down some notes, like link to sam's email ... | 15:28 |
| TheJulia | rloo: ++ | 15:28 |
| rloo | TheJulia: https://etherpad.openstack.org/p/ironic_driver_composition_interface_defaults | 15:31 |
| BadCub | howdy strangers | 15:34 |
| devananda | well hellooo there, BadCub ! :) | 15:36 |
| BadCub | helooo devananda | 15:36 |
| BadCub | :) | 15:36 |
| jroll | a wild BadCub appears \o/ | 15:36 |
| *** ifarkas is now known as ifarkas_afk | 15:36 | |
| BadCub | lol jroll :p | 15:36 |
| BadCub | been hibernating. figured I would pop my head out of the cave and sniff about | 15:37 |
| TheJulia | Good morning BadCub | 15:37 |
| BadCub | Morning TheJulia | 15:37 |
| thiagop | Hiya BadCub, long time no see | 15:38 |
| BadCub | heya thiagop yes it has been | 15:38 |
| thiagop | BadCub: how are you doing? | 15:39 |
| rloo | howdy stranger BadCub :) | 15:39 |
| *** rpioso has joined #openstack-ironic | 15:39 | |
| BadCub | thiagop: I am well. You? | 15:39 |
| jlvillal | Hey BadCub! :) | 15:39 |
| jroll | TheJulia: just to be sure, the call is at 1600 UTC, yes? | 15:39 |
| BadCub | howdy rloo :) | 15:39 |
| thiagop | BadCub: doing good o/ | 15:39 |
| TheJulia | jroll: yes | 15:40 |
| krtaylor | lucasagomes, is pxe_wakeonlan CI tested, if so, under what CI system? (trying to fill in all the ? in the stackalytics patch) | 15:41 |
| jroll | krtaylor: it isn't | 15:42 |
| lucasagomes | krtaylor, hi there, no it's not | 15:42 |
| BadCub | heya jlvillal :) | 15:42 |
| jlvillal | :) | 15:42 |
| lucasagomes | krtaylor, I've already moved the code to the ironic-staging-drivers project, and I'm expecting it to be removed from ironic soon | 15:42 |
| krtaylor | lucasagomes, ah, ok, I'll move it out then, thanks! | 15:42 |
| jroll | krtaylor: this lists all the drivers (as far as I know) that don't currently have a CI system: http://lists.openstack.org/pipermail/openstack-dev/2016-August/101331.html | 15:43 |
| jroll | might be worth double-checking drac | 15:43 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Trivial: remove redundant parentheses https://review.openstack.org/353592 | 15:43 |
| krtaylor | jroll, btw, I just pushed a patch to document the third-party CI, so #1 on that list will be covered in this patch | 15:44 |
| *** moshele has quit IRC | 15:44 | |
| jroll | krtaylor: oh, excellent | 15:44 |
| * jroll replies to himself | 15:44 | |
| *** sdake has joined #openstack-ironic | 15:45 | |
| krtaylor | jroll, I am :) unless you just want to | 15:45 |
| jroll | I just did :P | 15:45 |
| *** praneshp has joined #openstack-ironic | 15:47 | |
| *** mjura has quit IRC | 15:47 | |
| vsaienk0 | morning jlvillal | 15:48 |
| *** sdake_ has quit IRC | 15:48 | |
| jlvillal | vsaienk0: Hey :) | 15:49 |
| jlvillal | vsaienk0: I was hoping to chat with you, if you had time, about the multi-node stuff | 15:49 |
| jlvillal | vsaienk0: But I have to head home in a few minutes to meet the plumber | 15:50 |
| jlvillal | vsaienk0: Or maybe you can give us an update in the meeting today. | 15:50 |
| *** sabeen1 has quit IRC | 15:52 | |
| vsaienk0 | jlvillal: ok, np: Today I got first green grenade job. on primary node installed and configured all services ironic VMs were added on it. on secondary node ir-api,ir-cond,n-cpu and some ceilomenter services are installed. I skipped adding VMs on subnode, because it will work only with flat network or multitenant case, create_ovs_taps() is not working on subnode. | 15:52 |
| jroll | vsaienk0: maybe just worth it to make multinode grenade do multitenant only? | 15:53 |
| jroll | I think we'll want to move to that as default soon | 15:54 |
| jroll | and maybe even drop 'flat', because I think we can do flat the same way we do multitenant | 15:54 |
| jroll | flat network interface, that is | 15:54 |
| vsaienk0 | jroll: we need to update project config than | 15:54 |
| vsaienk0 | using flat network interface not guarantee that VMs are configred in flat fashion | 15:55 |
| jroll | vsaienk0: yeah, we'll figure out the details of network interfaces things later | 15:55 |
| krtaylor | jroll, so I feel like I should also remove all those non-ci systems from the stackalytics driver patch (maybe wait on drac), comments? | 15:55 |
| jroll | vsaienk0: but I think for now it's okay if multinode is multitenant only | 15:55 |
| jroll | krtaylor: I'm okay with that | 15:56 |
| vsaienk0 | jroll: we need to switch to real flat https://review.openstack.org/#/c/340695/ to support it on multinode | 15:56 |
| *** Sukhdev has joined #openstack-ironic | 15:56 | |
| krtaylor | jroll, will do, thanks | 15:56 |
| jroll | vsaienk0: why not just do multitenant on multinode? | 15:56 |
| vsaienk0 | jroll, I will switch multinode job to use multitenancy | 15:57 |
| jroll | vsaienk0: cool | 15:57 |
| vsaienk0 | or might be jlvillal can assist here | 15:57 |
| jroll | vsaienk0: when multinode is working, we can keep adding nodes with VMs until we have enough VMs to do tempest full :D | 15:57 |
| vsaienk0 | :) | 15:57 |
| vsaienk0 | btw, at the moment multiple n-cpu seems to be not working with Ironic | 15:58 |
| dtantsur | folks, does somebody remember the SIP link to use? | 15:58 |
| jroll | vsaienk0: what do you mean by not working? | 15:59 |
| jroll | dtantsur: https://wiki.openstack.org/wiki/Infrastructure/Conferencing | 15:59 |
| vsaienk0 | jroll: I remember las time when I enabled 2 n-cpu with ironic virt driver, all enrolled VMs were doubled | 16:01 |
| *** rcernin has quit IRC | 16:01 | |
| *** rbudden has quit IRC | 16:01 | |
| vdrok | dtantsur: I think it's pbx.openstack.org | 16:02 |
| jroll | vsaienk0: it should settle after RT runs | 16:02 |
| jroll | vsaienk0: well, with latest nova, that patch landed this week | 16:02 |
| TheJulia | The etherpad that rloo started: https://etherpad.openstack.org/p/ironic_driver_composition_interface_defaults | 16:02 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Deprecate -l option for port creation in OSC plugin https://review.openstack.org/351836 | 16:03 |
| rloo | hi meeting starting to discuss driver composition reform, how to handle default interfaces. | 16:04 |
| *** aNupoisc has joined #openstack-ironic | 16:04 | |
| jlvillal | vsaienk0, Thanks for the update :) | 16:05 |
| mat128 | shit just remembered the meeting | 16:06 |
| mat128 | jroll: memory pressure does that ;) | 16:08 |
| *** Sukhdev has quit IRC | 16:15 | |
| *** penick has joined #openstack-ironic | 16:17 | |
| jroll | just realized I've been staring at the wrong etherpad until now, which is why I've been confused | 16:22 |
| jroll | oops | 16:22 |
| mat128 | I'm on https://etherpad.openstack.org/p/ironic_driver_composition_interface_defaults | 16:22 |
| mat128 | not talking much | 16:22 |
| *** praneshp has quit IRC | 16:29 | |
| *** pcaruana has quit IRC | 16:29 | |
| *** milan has quit IRC | 16:31 | |
| *** Romanenko_K has quit IRC | 16:32 | |
| *** mgould is now known as mgould|afk | 16:32 | |
| devananda | in my haste to update api-ref yesterday, I left out /v1/heartbeat | 16:33 |
| devananda | and now as I test it, I am wondering if it works at all | 16:34 |
| *** milan has joined #openstack-ironic | 16:34 | |
| dtantsur | devananda, wanna join the driver composition defaults call? | 16:34 |
| dtantsur | devananda, it seemed to work the last time I tried it :/ | 16:34 |
| *** amoralej is now known as amoralej|off | 16:35 | |
| *** psachin has quit IRC | 16:37 | |
| *** ametts has quit IRC | 16:38 | |
| *** moshele has joined #openstack-ironic | 16:38 | |
| devananda | dtantsur: yah | 16:39 |
| * devananda has joined | 16:41 | |
| jroll | heartbeat seems to work: http://logs.openstack.org/81/119981/33/check/gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa-src/67ab629/logs/screen-ir-api.txt.gz#_2016-08-09_01_22_37_626 | 16:43 |
| *** vmud213 has quit IRC | 16:43 | |
| penick | rloo: thanks! | 16:46 |
| *** vmud213 has joined #openstack-ironic | 16:47 | |
| *** PollyZ has quit IRC | 16:48 | |
| lucasagomes | I'm calling it a day | 16:48 |
| lucasagomes | have a great evening all! Talk to you soon | 16:48 |
| jlvillal | good night lucasagomes | 16:49 |
| lucasagomes | jroll, https://review.openstack.org/#/c/344179/ release note was updated, if we can fit it in the release today it would be good. If not, the next one :D | 16:49 |
| jroll | lucasagomes: I'll try to take a look. have a good night! | 16:52 |
| *** vmud213 has quit IRC | 16:52 | |
| lucasagomes | cool, thanks! | 16:52 |
| *** lucasagomes is now known as lucas-dinner | 16:52 | |
| *** vmud213 has joined #openstack-ironic | 16:52 | |
| *** jpich has quit IRC | 16:53 | |
| *** vmud213 has quit IRC | 16:53 | |
| *** vmud213 has joined #openstack-ironic | 16:55 | |
| *** aNupoisc has quit IRC | 16:58 | |
| jlvillal | krtaylor, Any chance you can chair the QA meeting for a bit? | 16:59 |
| jlvillal | The plumber just showed up... | 16:59 |
| jlvillal | krtaylor, And could you link to: https://etherpad.openstack.org/p/ironic-newton-grenade-whiteboard ?? | 16:59 |
| *** anush_ has quit IRC | 17:00 | |
| krtaylor | jlvillal, sure | 17:01 |
| *** vmud213 has quit IRC | 17:01 | |
| *** ohamada has quit IRC | 17:02 | |
| *** vmud213 has joined #openstack-ironic | 17:02 | |
| *** vmud213 has quit IRC | 17:03 | |
| *** vmud213 has joined #openstack-ironic | 17:03 | |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Add vendor id, vendor name and product_id to interface information https://review.openstack.org/279940 | 17:05 |
| *** ChubYann has joined #openstack-ironic | 17:07 | |
| *** vmud213 has quit IRC | 17:13 | |
| *** vmud213 has joined #openstack-ironic | 17:13 | |
| *** milan has quit IRC | 17:15 | |
| TheJulia | Thank you everyone who attended the call | 17:17 |
| mat128 | sorry for being late, calendar decided it wouldnt remind me :( | 17:17 |
| mat128 | alert: None | 17:17 |
| mat128 | there you gou :( | 17:17 |
| TheJulia | I'll send out an email tomorrow morning, which will give me a chance to check in with sambetts first thing | 17:18 |
| rloo | thx TheJulia | 17:19 |
| dtantsur | calling it a day, see you tomorrow | 17:19 |
| *** aNupoisc has joined #openstack-ironic | 17:19 | |
| *** dtantsur is now known as dtantsur|afk | 17:19 | |
| rloo | bye dtantsur|afk | 17:19 |
| *** vmud213 has quit IRC | 17:20 | |
| *** praneshp has joined #openstack-ironic | 17:21 | |
| *** e0ne has quit IRC | 17:21 | |
| TheJulia | goodnight dtantsur|afk | 17:21 |
| *** moshele has quit IRC | 17:22 | |
| jroll | cores, can you help get release notes done so we can release 6.1? :) https://review.openstack.org/#/q/topic:6.1-reno | 17:22 |
| *** penick has quit IRC | 17:24 | |
| *** harlowja has joined #openstack-ironic | 17:27 | |
| * jroll goes off to find tacos | 17:27 | |
| *** ametts has joined #openstack-ironic | 17:28 | |
| * TheJulia wishes there was a build_the_docs_now button | 17:29 | |
| *** alineb has quit IRC | 17:30 | |
| jlvillal | krtaylor, thanks for covering for me! | 17:32 |
| jlvillal | Sorry about the last minute notice :( | 17:32 |
| krtaylor | jlvillal, no problem at all | 17:32 |
| krtaylor | glad I was here :) | 17:32 |
| jlvillal | Me too :) | 17:32 |
| *** aNupoisc has quit IRC | 17:33 | |
| *** penick has joined #openstack-ironic | 17:36 | |
| *** priteau has quit IRC | 17:41 | |
| vdrok | good night everyone! | 17:42 |
| *** praneshp has quit IRC | 17:42 | |
| *** praneshp has joined #openstack-ironic | 17:43 | |
| *** priteau has joined #openstack-ironic | 17:45 | |
| rloo | night vdrok! | 17:46 |
| *** electrofelix has quit IRC | 17:49 | |
| jlvillal | vdrok, Good night | 17:49 |
| *** aNupoisc has joined #openstack-ironic | 17:52 | |
| NobodyCam | Good morning Ironic'ers | 17:58 |
| TheJulia | Regarding https://review.openstack.org/#/c/353049/7, the extra space between the link and the period is rendered, as opposed to when it is absent there is no excess space. I can push a fix right now if there are no objections | 17:58 |
| TheJulia | ^^^ jroll | 17:58 |
| NobodyCam | jroll: great tweet last night !!!!! | 17:58 |
| jroll | TheJulia: will it be rendered as part of the URL, though? | 17:59 |
| jroll | NobodyCam: :) | 17:59 |
| TheJulia | jroll: the period, no | 17:59 |
| TheJulia | it renders as </a>. | 17:59 |
| jroll | oh, fantastic | 17:59 |
| TheJulia | at least when I remove the space | 17:59 |
| jroll | yeah, I can fix that, there's a couple of them iirc | 18:00 |
| thiagop | morning NobodyCam | 18:00 |
| xavierr | morning Ironic | 18:00 |
| TheJulia | jroll: you sure? | 18:00 |
| xavierr | morning NobodyCam | 18:00 |
| *** abashmak has left #openstack-ironic | 18:01 | |
| jroll | TheJulia: already on it :) | 18:01 |
| TheJulia | ok | 18:01 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor OneView dynamic allocation release notes https://review.openstack.org/353049 | 18:02 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up release notes for 6.1.0 https://review.openstack.org/352660 | 18:03 |
| jroll | TheJulia: fixed that inline comment too | 18:03 |
| TheJulia | jroll: you rock | 18:03 |
| jroll | nou! | 18:03 |
| *** fragatina has quit IRC | 18:07 | |
| TheJulia | jroll: did you mean to only rebase the OneView release notes rev? | 18:08 |
| NobodyCam | morning xavierr :) | 18:09 |
| jroll | TheJulia: oh, same problem there eh | 18:09 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor OneView dynamic allocation release notes https://review.openstack.org/353049 | 18:10 |
| jroll | there we go | 18:10 |
| TheJulia | jroll: yeah, that was the one I noticed it in :) | 18:10 |
| jroll | sorry | 18:10 |
| jroll | heh | 18:10 |
| *** PollyZ has joined #openstack-ironic | 18:10 | |
| TheJulia | jroll: Thank you! | 18:10 |
| jroll | no problem | 18:10 |
| * jroll needs a bigger break than 20m lunch, bbl | 18:11 | |
| devananda | 2016-08-10 16:33:01.098 7772 INFO keystonemiddleware.auth_token [-] Rejecting request | 18:11 |
| devananda | 2016-08-10 16:33:01.099 7772 INFO eventlet.wsgi.server [-] 172.17.0.1 "POST /v1/heartbeat?node_ident=test_node&callback_url=bar HTTP/1.1" status: 401 len: 293 time: 0.0008261 | 18:11 |
| devananda | I am quite confused as to why I'm getting this error ^ | 18:12 |
| *** e0ne has joined #openstack-ironic | 18:12 | |
| *** tesseract- has quit IRC | 18:13 | |
| jroll | devananda: what state is the node in? | 18:13 |
| devananda | jroll: this error is coming from keystone - it's not even getting into our API method, afaict | 18:13 |
| jroll | oh | 18:14 |
| jroll | /v1/heartbeat/uuid? | 18:14 |
| jroll | the node ident is part of the path | 18:14 |
| jroll | /v1/heartbeat/uuid?callback_url=foo | 18:14 |
| jroll | is what I meant | 18:14 |
| jroll | devananda: and while pecan/wsme magic junk will usually allow ?node_ident with code like that, we only whitelist '/v1/heartbeat/[a-z0-9\-]+' | 18:15 |
| jroll | whitelist meaning without auth | 18:15 |
| * jroll really steps away now | 18:16 | |
| devananda | oh | 18:17 |
| openstackgerrit | Michael Turek proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/332177 | 18:20 |
| devananda | jroll: for when you're back, it works with the node uuid, but not necessarily node name, because the name allows other characters | 18:21 |
| devananda | is there a reason that the exposing of '/v1/lookup' and '/v1/heartbeat/[a-z0-9\-]+' are different? Both accept a node_uuid... but in different ways. | 18:22 |
| *** catintheroof has quit IRC | 18:34 | |
| *** Sukhdev has joined #openstack-ironic | 18:41 | |
| *** ametts has quit IRC | 18:42 | |
| *** e0ne has quit IRC | 18:43 | |
| *** ametts has joined #openstack-ironic | 18:43 | |
| *** fragatina has joined #openstack-ironic | 18:44 | |
| *** e0ne has joined #openstack-ironic | 18:44 | |
| mat128 | devananda: the last part is the driver name | 18:45 |
| mat128 | [a-z0-9\-]+ | 18:45 |
| mat128 | devananda: https://github.com/openstack/ironic/commit/45adeedd437cb6f40be751fd0f1fcf2d1aa5d79a | 18:46 |
| mat128 | hmm, never mind I'm not up on the new contract :) heartbeating doesnt require driver name since it's not a driver passthrough anymore | 18:48 |
| *** sabeen has joined #openstack-ironic | 18:48 | |
| *** e0ne_ has joined #openstack-ironic | 18:49 | |
| *** e0ne has quit IRC | 18:49 | |
| devananda | mat128: right - that's the node name | 18:52 |
| devananda | except it breaks if I name my node "test_node" | 18:52 |
| devananda | because of the underscore | 18:52 |
| *** cryptarium has joined #openstack-ironic | 18:56 | |
| *** cryptarium has quit IRC | 18:56 | |
| *** cryptarium has joined #openstack-ironic | 18:57 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Update api-ref for v1.22 https://review.openstack.org/353117 | 18:58 |
| devananda | JayF: do you recall why /heartbeat is allowed for nodes in _any_ state, rather than restricted to nodes only in specific states? | 18:59 |
| *** e0ne_ has quit IRC | 19:00 | |
| *** cryptarium has quit IRC | 19:01 | |
| *** ijw has joined #openstack-ironic | 19:08 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Restrict /heartbeat to only nodes in LOOKUP_ALLOWED states https://review.openstack.org/353696 | 19:12 |
| * devananda proposes that fix anyway | 19:12 | |
| *** syed_ has quit IRC | 19:12 | |
| *** baoli_ has joined #openstack-ironic | 19:14 | |
| mat128 | devananda: reviewed :) | 19:14 |
| mat128 | but I agree with the fact that /heartbeat should follow the same restriction | 19:14 |
| *** jaypipes has joined #openstack-ironic | 19:15 | |
| jaypipes | NobodyCam, jroll, JayF: hey can any of you describe what exactly boot Ironic node from volume actually is? would this be an Ironic node netbooting from an image stored on a Cinder volume and then laying that image down onto the node's local storage? Or is this something entirely different? I'm having a tough time wrapping my brain around it. | 19:17 |
| mat128 | jaypipes: I know TheJulia is part of this work and IIRC from the spec, we had different modes, one of which is similar to what you are describing | 19:18 |
| mat128 | another was "consider the volume already contains an image" | 19:18 |
| jaypipes | mat128: right, but in that second case, what happens to local storage on the Ironic node? | 19:19 |
| mat128 | jaypipes: https://review.openstack.org/#/c/294995/ - a total of 5 scenarios | 19:19 |
| mat128 | jaypipes: good question | 19:19 |
| * jaypipes goes reading :) | 19:19 | |
| mat128 | not sure your question is answered in there, but it wouldnt hurt to ask | 19:20 |
| mat128 | :) | 19:20 |
| mat128 | brb | 19:20 |
| *** mtanino has quit IRC | 19:34 | |
| mat128 | back | 19:35 |
| *** rbudden has joined #openstack-ironic | 19:53 | |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: [WIP] Add storage_interface DB field and object https://review.openstack.org/348005 | 19:55 |
| TheJulia | jaypipes: local storage would just be presented as storage to whatever boots as a result of the volume attachment or boot config... and afterwards would go through cleaning | 19:56 |
| *** d0ugal has joined #openstack-ironic | 20:01 | |
| *** joprovost has quit IRC | 20:10 | |
| *** causten has quit IRC | 20:13 | |
| *** causten has joined #openstack-ironic | 20:15 | |
| jaypipes | TheJulia: thanks! things were quite a bit clearer after reading the spec :) | 20:15 |
| TheJulia | :) | 20:15 |
| *** dprince has quit IRC | 20:16 | |
| *** bvandewa has joined #openstack-ironic | 20:16 | |
| *** aNupoisc has quit IRC | 20:19 | |
| *** ijw has quit IRC | 20:23 | |
| *** ijw has joined #openstack-ironic | 20:30 | |
| *** ametts has quit IRC | 20:34 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic: Add support for the OneView audit API https://review.openstack.org/337751 | 20:36 |
| *** michauds has joined #openstack-ironic | 20:40 | |
| jroll | devananda: likely because heartbeat doesn't leak any info | 20:45 |
| jroll | devananda: also, node name likely was overlooked because ramdisks use uuids, there's really no need to accept a name | 20:46 |
| *** rpioso has quit IRC | 20:50 | |
| *** PollyZ has quit IRC | 20:55 | |
| *** baoli_ has quit IRC | 20:56 | |
| *** PollyZ has joined #openstack-ironic | 21:04 | |
| *** aNupoisc has joined #openstack-ironic | 21:07 | |
| *** fragatin_ has joined #openstack-ironic | 21:08 | |
| *** causten has quit IRC | 21:11 | |
| *** wajdi_ has joined #openstack-ironic | 21:12 | |
| *** fragatina has quit IRC | 21:12 | |
| jroll | rloo: do you mind slamming these home before you go? https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:master+topic:6.1-reno | 21:13 |
| rloo | jroll: looking now... | 21:13 |
| rloo | jroll: where's jenkins? ##$@%@$ | 21:14 |
| jroll | I know :( | 21:14 |
| rloo | jroll: you want me to +2 and +A if I am ok, or just +2 then? | 21:14 |
| *** wajdi has quit IRC | 21:14 | |
| jroll | rloo: sorry, I didn't realize jenkins hadn't come back yet, it's up to you | 21:15 |
| rloo | jroll: i'll wait another 30 min or so for jenkins. or look later tonight. | 21:16 |
| jroll | rloo: thank you :) | 21:16 |
| rloo | jroll: cuz i should check things in the rendered page. although they are 'just' release notes... heh... | 21:16 |
| jroll | lol | 21:16 |
| *** causten has joined #openstack-ironic | 21:16 | |
| *** wajdi_ has quit IRC | 21:17 | |
| jroll | rloo: if you checked the rendered page before, maybe diff against that and it won't be large changes? | 21:17 |
| jroll | or just wait | 21:17 |
| rloo | jroll: you just want them done by end of day, right? i'll make sure... | 21:18 |
| jroll | rloo: morning is still okay but pushing the limits | 21:18 |
| *** ijw has quit IRC | 21:19 | |
| rloo | jroll: ok. from what i remember, the content of the release notes is good enough. so best to do it by end of day. | 21:19 |
| jroll | ok. thanks. | 21:20 |
| *** ijw has joined #openstack-ironic | 21:23 | |
| *** ijw has quit IRC | 21:24 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add openstack baremetal driver commands https://review.openstack.org/350050 | 21:37 |
| *** bvandewa has quit IRC | 21:38 | |
| *** jcoufal has quit IRC | 21:49 | |
| mrda | Morning Ironic | 21:50 |
| *** priteau has quit IRC | 21:54 | |
| *** [1]cdearborn has quit IRC | 22:06 | |
| *** michauds has quit IRC | 22:13 | |
| *** ijw has joined #openstack-ironic | 22:22 | |
| jroll | heya mrda :) | 22:22 |
| *** ijw has quit IRC | 22:24 | |
| *** fragatin_ has quit IRC | 22:26 | |
| *** fragatina has joined #openstack-ironic | 22:27 | |
| *** causten has quit IRC | 22:33 | |
| *** causten has joined #openstack-ironic | 22:34 | |
| *** ijw has joined #openstack-ironic | 22:35 | |
| openstackgerrit | Merged openstack/ironic: Refactor multitenant networking release notes https://review.openstack.org/352659 | 22:38 |
| *** sdake has quit IRC | 22:38 | |
| *** causten has quit IRC | 22:38 | |
| *** thiagop has quit IRC | 22:41 | |
| *** ijw has quit IRC | 22:45 | |
| mrda | hey jroll | 22:47 |
| *** praneshp has quit IRC | 22:55 | |
| *** PollyZ has quit IRC | 22:56 | |
| *** Sukhdev has quit IRC | 23:01 | |
| devananda | mrda: o/ | 23:11 |
| *** sabeen has quit IRC | 23:15 | |
| *** bfournie has quit IRC | 23:15 | |
| *** sdake has joined #openstack-ironic | 23:26 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Mask configdrive contents in API responses https://review.openstack.org/326768 | 23:30 |
| *** chlong|mtg has quit IRC | 23:31 | |
| *** Sukhdev has joined #openstack-ironic | 23:32 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Update documentation for keystone policy support https://review.openstack.org/345556 | 23:36 |
| mrda | hey devananda :) | 23:38 |
| *** ksumit has joined #openstack-ironic | 23:38 | |
| *** ijw has joined #openstack-ironic | 23:48 | |
| *** ijw has quit IRC | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!