| *** zhangfei has joined #openstack-ironic | 00:06 | |
| TheJulia | The only other way would be to add fallback/retry logic into nova | 00:09 |
|---|---|---|
| TheJulia | jlvillal: I feel like I should know this, where is our grenade stuff squirreled away? | 00:15 |
| jlvillal | TheJulia: devstack/upgrade/ | 00:15 |
| TheJulia | jlvillal: off the top of your head, do you know where they get invoked at? | 00:16 |
| TheJulia | nvmd | 00:17 |
| jlvillal | TheJulia: Uh, grenade does it. It loads the file as a plugin. | 00:17 |
| jlvillal | TheJulia: Kind of have to walk through it and see what is going on. | 00:17 |
| jlvillal | TheJulia: I'm about to take off :( | 00:17 |
| TheJulia | okay, have a good evening | 00:17 |
| jlvillal | TheJulia: Thanks, you too! | 00:19 |
| *** baoli has joined #openstack-ironic | 00:34 | |
| *** mtanino has quit IRC | 00:35 | |
| *** baoli has quit IRC | 00:38 | |
| *** fragatina has quit IRC | 00:45 | |
| *** fragatina has joined #openstack-ironic | 00:46 | |
| *** syqian_ has quit IRC | 00:48 | |
| *** fragatin_ has joined #openstack-ironic | 00:49 | |
| *** fragatin_ has quit IRC | 00:50 | |
| *** fragatina has quit IRC | 00:50 | |
| *** fragatin_ has joined #openstack-ironic | 00:50 | |
| *** fragatin_ has quit IRC | 00:52 | |
| *** fragatina has joined #openstack-ironic | 00:52 | |
| *** baoli has joined #openstack-ironic | 00:55 | |
| *** fragatina has quit IRC | 00:56 | |
| *** xavierr has quit IRC | 00:58 | |
| *** baoli has quit IRC | 00:59 | |
| *** tuanluong has joined #openstack-ironic | 01:00 | |
| *** phuongnh has joined #openstack-ironic | 01:02 | |
| *** rbartal has joined #openstack-ironic | 01:03 | |
| *** xavierr has joined #openstack-ironic | 01:09 | |
| TheJulia | woot, so... same version all around :\ | 01:09 |
| *** harlowja has quit IRC | 01:13 | |
| *** fragatina has joined #openstack-ironic | 01:25 | |
| *** hshiina has joined #openstack-ironic | 01:25 | |
| *** fragatina has quit IRC | 01:29 | |
| *** rloo has quit IRC | 01:31 | |
| *** yufei has joined #openstack-ironic | 01:32 | |
| *** syqian_ has joined #openstack-ironic | 01:32 | |
| *** xavierrr has joined #openstack-ironic | 01:36 | |
| TheJulia | jlvillal: I think this is the actual issue. http://logs.openstack.org/49/485349/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/2869cfe/logs/grenade.sh.txt.gz#_2017-07-24_23_55_59_040 | 01:36 |
| *** rbartal has quit IRC | 01:39 | |
| *** chlong_ has joined #openstack-ironic | 01:45 | |
| *** chlong__ has quit IRC | 01:47 | |
| *** litao__ has joined #openstack-ironic | 01:48 | |
| *** crushil has joined #openstack-ironic | 01:50 | |
| *** baoli has joined #openstack-ironic | 01:51 | |
| * TheJulia adds another item to the PTG etherpad | 01:52 | |
| *** baoli has quit IRC | 01:56 | |
| *** chlong__ has joined #openstack-ironic | 02:06 | |
| *** chlong_ has quit IRC | 02:07 | |
| *** fragatina has joined #openstack-ironic | 02:16 | |
| *** xavierrr has quit IRC | 02:19 | |
| *** chlong_ has joined #openstack-ironic | 02:25 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Tempest scenario test for boot-from-volume https://review.openstack.org/472740 | 02:26 |
| *** namnh has joined #openstack-ironic | 02:26 | |
| *** chlong__ has quit IRC | 02:27 | |
| TheJulia | jlvillal: I figured it out | 02:31 |
| TheJulia | it is grenade multi-node at it's core | 02:31 |
| TheJulia | the old api that hangs out, and the redirect we put in to it. :\ | 02:31 |
| *** fragatina has quit IRC | 02:34 | |
| *** baoli has joined #openstack-ironic | 02:37 | |
| *** crushil has left #openstack-ironic | 02:38 | |
| *** crushil has joined #openstack-ironic | 02:38 | |
| *** baoli has quit IRC | 02:41 | |
| *** fragatina has joined #openstack-ironic | 02:55 | |
| *** syqian_ has quit IRC | 03:01 | |
| *** syqian_ has joined #openstack-ironic | 03:01 | |
| *** fragatina has quit IRC | 03:12 | |
| *** vishwana_ has joined #openstack-ironic | 03:19 | |
| *** vishwana_ has quit IRC | 03:19 | |
| *** vishwanathj has quit IRC | 03:22 | |
| *** fragatina has joined #openstack-ironic | 03:23 | |
| *** chlong_ has quit IRC | 03:28 | |
| *** chlong__ has joined #openstack-ironic | 03:28 | |
| *** hshiina has quit IRC | 03:41 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: DNM: Unwind Julia's Migraine... maybe. https://review.openstack.org/486852 | 04:01 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Address review feedback for ipxe boot file fix https://review.openstack.org/486853 | 04:01 |
| *** hshiina has joined #openstack-ironic | 04:01 | |
| *** chlong__ has quit IRC | 04:01 | |
| *** chlong_ has joined #openstack-ironic | 04:01 | |
| openstackgerrit | Merged openstack/proliantutils master: Add test cases parity for RIS init https://review.openstack.org/486570 | 04:09 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Tempest scenario test for boot-from-volume https://review.openstack.org/472740 | 04:12 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: DNM: Unwind Julia's Migraine... maybe. https://review.openstack.org/486852 | 04:12 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Address review feedback for ipxe boot file fix https://review.openstack.org/486853 | 04:12 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP/DNM: Try to test boot-from-volume https://review.openstack.org/485349 | 04:12 |
| TheJulia | jlvillal: https://review.openstack.org/#/c/486852/ might explain my headache | 04:12 |
| patchbot | patch 486852 - ironic - DNM: Unwind Julia's Migraine... maybe. | 04:12 |
| *** chlong__ has joined #openstack-ironic | 04:25 | |
| *** chlong_ has quit IRC | 04:27 | |
| *** links has joined #openstack-ironic | 04:30 | |
| *** sasha2 has quit IRC | 04:33 | |
| *** mvpnitesh has joined #openstack-ironic | 04:53 | |
| *** jaypipes has joined #openstack-ironic | 04:56 | |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Redfish: Adds server capabilities https://review.openstack.org/484360 | 05:04 |
| *** marios has joined #openstack-ironic | 05:15 | |
| openstackgerrit | Ankit Kumar proposed openstack/ironic master: Adding grub2 bootloader support to devstack plugin https://review.openstack.org/454026 | 05:16 |
| *** jjung has joined #openstack-ironic | 05:18 | |
| *** vmud213 has joined #openstack-ironic | 05:21 | |
| *** chlong_ has joined #openstack-ironic | 05:32 | |
| *** racedo has joined #openstack-ironic | 05:34 | |
| *** racedo has quit IRC | 05:34 | |
| *** chlong__ has quit IRC | 05:34 | |
| masber | good afternoon all, I have created ironic node but the validation is giving me some errors regarding boot and console http://paste.openstack.org/show/616378/, I was wondering whether I could get any type of help or advise? | 05:36 |
| *** mjura has joined #openstack-ironic | 05:41 | |
| *** syqian__ has joined #openstack-ironic | 05:50 | |
| *** syqian_ has quit IRC | 05:50 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add API for volume resources to api-ref https://review.openstack.org/478798 | 05:51 |
| *** syqian__ has quit IRC | 05:53 | |
| *** syqian__ has joined #openstack-ironic | 05:54 | |
| *** jjung_ has joined #openstack-ironic | 05:58 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add storage interface to api-ref https://review.openstack.org/480975 | 06:00 |
| openstackgerrit | lu.li proposed openstack/virtualbmc master: Update the documentation link for doc migration https://review.openstack.org/486886 | 06:07 |
| *** rcernin has joined #openstack-ironic | 06:10 | |
| *** udesale has joined #openstack-ironic | 06:13 | |
| *** ccamacho has joined #openstack-ironic | 06:35 | |
| *** stendulker has joined #openstack-ironic | 06:39 | |
| *** milan has joined #openstack-ironic | 06:39 | |
| *** rbartal has joined #openstack-ironic | 06:41 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Adds nvdimm_n, logical_nvdimm_n, persistent_memory attributes https://review.openstack.org/478702 | 06:47 |
| *** tiendc has joined #openstack-ironic | 06:50 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: WIP:Redfish: Adds get_essential_properties() https://review.openstack.org/485134 | 06:50 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds macs discovery https://review.openstack.org/483127 | 06:51 |
| *** kaifeng has joined #openstack-ironic | 06:52 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Fix docstring and default value for local_group_info https://review.openstack.org/486911 | 06:52 |
| *** jaganathan has joined #openstack-ironic | 06:52 | |
| kaifeng | masber: you need to create a port for this node | 06:53 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds get_essential_properties() https://review.openstack.org/485134 | 06:53 |
| kaifeng | ramdisk and kernel are also required if you don't want to expect another validation error. | 06:56 |
| *** ChubYann has quit IRC | 06:56 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds get_essential_properties() https://review.openstack.org/485134 | 07:01 |
| *** kaifeng has quit IRC | 07:05 | |
| *** vsaienk0 has joined #openstack-ironic | 07:24 | |
| openstackgerrit | Siyi Luo proposed openstack/sushy master: Enable some off-by-default checks https://review.openstack.org/482380 | 07:31 |
| *** mvpnitesh has quit IRC | 07:33 | |
| *** e0ne has joined #openstack-ironic | 07:38 | |
| *** rbartal has quit IRC | 07:39 | |
| *** ralonsoh has joined #openstack-ironic | 07:54 | |
| *** etingof has quit IRC | 07:57 | |
| *** yibo has left #openstack-ironic | 07:59 | |
| *** ralonsoh has quit IRC | 07:59 | |
| *** rbartal has joined #openstack-ironic | 08:01 | |
| milan | morning ironic! :) | 08:01 |
| *** aarefiev_afk is now known as aarefiev | 08:05 | |
| aarefiev | morning! | 08:05 |
| *** ralonsoh has joined #openstack-ironic | 08:06 | |
| wong | hi, does Ironic have a best practice? | 08:07 |
| *** derekh has joined #openstack-ironic | 08:10 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic master: [doc-migration] Add configuration folder for documentation https://review.openstack.org/486934 | 08:11 |
| *** jhesketh has joined #openstack-ironic | 08:16 | |
| *** etingof has joined #openstack-ironic | 08:16 | |
| *** ralonsoh has quit IRC | 08:16 | |
| *** ralonsoh has joined #openstack-ironic | 08:16 | |
| *** jhesketh_ has joined #openstack-ironic | 08:22 | |
| *** jjung has quit IRC | 08:22 | |
| *** jjung_ has quit IRC | 08:22 | |
| *** lucas-afk is now known as lucasagomes | 08:23 | |
| *** jhesketh has quit IRC | 08:25 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: bifrost-create-vm-nodes: Do not use KVM VMs by default https://review.openstack.org/486645 | 08:28 |
| *** jhesketh_ is now known as jhesketh | 08:30 | |
| etingof | milan, morning ;) | 08:30 |
| milan | morning etingof! :) | 08:30 |
| etingof | milan, can I take a look at the logs you mentioned here -- https://review.openstack.org/#/c/466448/40/ironic_inspector/pxe_filter/dnsmasq.py@40 ? | 08:30 |
| patchbot | patch 466448 - ironic-inspector - Introducing a dnsmasq PXE filter driver | 08:30 |
| etingof | milan, the ones I can see now are probably not failing anymore | 08:31 |
| milan | etingof, http://logs.openstack.org/12/468712/34/check/gate-tempest-dsvm-ironic-inspector-ubuntu-xenial/23ad6fe/logs/screen-ironic-inspector.txt.gz?level=ERROR | 08:31 |
| milan | etingof, it's from the TEST patch that enables the dnsmasq filter | 08:32 |
| * etingof looking | 08:33 | |
| milan | etingof, also the whole design revolves around the --dhcp-hostsdir description from http://www.thekelleys.org.uk/dnsmasq/docs/dnsmasq-man.html | 08:35 |
| * milan emphasising that right now | 08:35 | |
| milan | in the patch | 08:35 |
| *** priteau has joined #openstack-ironic | 08:36 | |
| openstackgerrit | bhavani proposed openstack/bifrost master: Handle log message interpolation by the logger in common/ https://review.openstack.org/485123 | 08:37 |
| *** jjung_ has joined #openstack-ironic | 08:38 | |
| etingof | milan so can you `os.unlink` the existing file and then `os.rename` the temp file into the proper name? | 08:39 |
| *** jjung has joined #openstack-ironic | 08:39 | |
| milan | etingof, I'd like to avoid it; another thread may be running listdir() | 08:39 |
| milan | edge cases | 08:40 |
| milan | etingof, anyways, a MAC address is quite a random beast and the hostsdir isn't public, please see the man page | 08:40 |
| lucasagomes | morning all | 08:41 |
| * milan would like to revert this tempfile thing | 08:41 | |
| milan | it's not tempdir after all ;) | 08:41 |
| *** marios has quit IRC | 08:41 | |
| milan | morning lucasagomes! :) | 08:41 |
| etingof | milan, right. so is there still the same edge case if two threads are touching these files? | 08:41 |
| etingof | morning ;) | 08:41 |
| *** marios has joined #openstack-ironic | 08:42 | |
| etingof | milan, BTW, there is an atomic `os.replace` since Py3.3 | 08:42 |
| milan | yup but the filenames are always mac addresses; the size check is mutually exclusive with a write, correct? | 08:42 |
| milan | the named temp file creates the inconsistency | 08:43 |
| etingof | milan, well, I'm not sure: when you open() for writing file size resets to zero | 08:43 |
| asettle | sambetts|afk: how'd you go with the docs? Lemme know, I have more time today to help out if you need :) | 08:44 |
| milan | etingof, also true | 08:44 |
| * etingof is wondering - do we need concurrent listdir()...? | 08:45 | |
| milan | etingof, actually I'm wrong | 08:46 |
| milan | etingof, I've forgotten it's exclusive (there's the lock decorator) | 08:46 |
| etingof | milan, that's good! so what is the problem we are still confronting here? | 08:47 |
| milan | etingof, basically the file existence as it's cache | 08:48 |
| etingof | milan, can we safely unlink it prior to renaming? | 08:48 |
| milan | etingof, I'm not sure the tempname adds much security to the picture | 08:49 |
| milan | but yeah, we can | 08:49 |
| etingof | milan, well, probably depends on the setup | 08:49 |
| etingof | etingof, it's not critical for sure | 08:49 |
| etingof | milan, anyways, thank you for the explanation! ;-) | 08:50 |
| milan | etingof, thanks for the reviews! :) | 08:50 |
| milan | etingof, one more drawback of the named temp file: the dnsmasq would get posted twice ;) | 08:52 |
| vdrok | morning ironic, milan etingof aarefiev wong lucasagomes ! :) | 08:52 |
| milan | morning vdrok! :) | 08:52 |
| etingof | morning ;) | 08:52 |
| etingof | milan, to nechápu ;) | 08:53 |
| milan | etingof, mind the inotify ;) | 08:53 |
| milan | a new file would pop up with the hardlink, no? | 08:54 |
| etingof | milan, so if we create the tempfile at the hostdir - yes, inotify might send the extra events to dnsmasq | 08:55 |
| milan | etingof, exactly | 08:55 |
| etingof | milan, the solution may be to create tempfiles in a subdir | 08:55 |
| milan | etingof, listdir() would have to adjusted then | 08:55 |
| milan | to filter | 08:55 |
| milan | but again, I'm unsure about the benefits | 08:56 |
| etingof | milan, the hard requirement here is that tempfiles and hostdir/* should be on the same filesystem | 08:56 |
| milan | it's getting more complicated :) | 08:56 |
| etingof | milan, yeah, I clearly see the complications.... | 08:56 |
| etingof | milan, btw, I wonder how many inotify events we produce per a single file update -- one or two? | 08:57 |
| milan | etingof, a single one with the fd.close() | 08:58 |
| milan | at least that's what I've observed | 08:58 |
| etingof | milan, good then | 08:58 |
| milan | etingof, we can examine actually in the logs | 08:59 |
| * milan would amend the patch | 08:59 | |
| milan | then the TEST patch will log dnsmasq events | 08:59 |
| etingof | milan, I'm asking because when you truncate a file (for example), file size is reset to zero and no contents is available anymore, still FD is still open | 08:59 |
| milan | etingof, ack makes sense; that's maybe why inotifyd reacts on the fd.close() | 09:00 |
| wong | morning | 09:00 |
| milan | morning wong! :) | 09:00 |
| etingof | milan, I think it's a single event! | 09:02 |
| etingof | milan, judging from https://github.com/aa65535/dnsmasq/blob/13dee6f49e1d035b8069947be84ee8da2af0c420/src/inotify.c#L121 and https://linux.die.net/man/7/inotify | 09:02 |
| milan | \o/ :D | 09:02 |
| milan | yup | 09:02 |
| milan | so the design of the filter isn't completely hosed ;) | 09:03 |
| *** mgoddard_ has joined #openstack-ironic | 09:03 | |
| etingof | milan, noooo! it paves our way to the inevitable success ;) | 09:04 |
| milan | lol :D | 09:04 |
| milan | etingof, so true about pretty much any piece of code out there! :D | 09:04 |
| openstackgerrit | yuan liang proposed openstack/ironic-inspector master: Add selinux check in Troubleshooting PXE boot document https://review.openstack.org/482848 | 09:05 |
| wong | we are going to provide ironic to our customer, do you have offical best practice? | 09:08 |
| wong | whitepaper of something else | 09:09 |
| wong | sorry , or | 09:09 |
| openstackgerrit | Cédric Ollivier proposed openstack/bifrost master: Fix tools/vagrant_dev_env/vagrant.yml https://review.openstack.org/486954 | 09:10 |
| wong | now I'm create an instance success by using devstack 、pxe_ilo and HP server | 09:14 |
| wong | but I'm afraid if bad things happened in customer environment | 09:15 |
| lucasagomes | vdrok, morning :D | 09:21 |
| vdrok | wong: I'm not sure what you mean, what bad things happened? | 09:23 |
| vdrok | devstack is not the thing generally recommended to deploy ironic in production :) | 09:24 |
| *** tiendc has quit IRC | 09:32 | |
| *** chlong__ has joined #openstack-ironic | 09:33 | |
| wong | so,what should I choose the most suitabale physical machine before I deploy in production | 09:34 |
| *** chlong_ has quit IRC | 09:35 | |
| wong | and the most suitable deployment topology | 09:35 |
| *** amarao1 has joined #openstack-ironic | 09:39 | |
| *** ralonsoh has quit IRC | 09:45 | |
| *** ralonsoh has joined #openstack-ironic | 09:45 | |
| openstackgerrit | Merged openstack/bifrost master: Removes unnecessary utf-8 encoding https://review.openstack.org/486594 | 09:53 |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic master: [doc-migration] Add configuration folder for documentation https://review.openstack.org/486934 | 10:02 |
| sambetts|afk | asettle: basically we've got competeing ideas for the ironic index page now :/ mine: https://review.openstack.org/#/c/486649/ and rloo's https://review.openstack.org/#/c/485820 | 10:06 |
| patchbot | patch 486649 - ironic - Clean up and simplify doc/source/index | 10:06 |
| patchbot | patch 485820 - ironic - [doc] update ironic's landing page | 10:06 |
| *** sambetts|afk is now known as sambetts | 10:06 | |
| asettle | Neat :d | 10:06 |
| asettle | :D * | 10:06 |
| sambetts | yeah great... XD | 10:06 |
| sambetts | literally the first commnet on my patch was "I wish there was guideence from OpenStack land so that we all organisze the info in the same way" | 10:07 |
| *** phuongnh has quit IRC | 10:08 | |
| *** dtantsur|afk is now known as dtantsur | 10:10 | |
| dtantsur | morning ironic | 10:10 |
| dtantsur | sambetts: heh, let the fight begin :) | 10:12 |
| *** syqian has joined #openstack-ironic | 10:16 | |
| mgoddard_ | wong: take a look at http://kayobe.readthedocs.io/en/latest/ for production deployment using ironic. It's based on the kolla-ansible project with a focus on bare metal | 10:16 |
| *** openstackgerrit has quit IRC | 10:17 | |
| *** syqian__ has quit IRC | 10:18 | |
| *** MattMan_1 has quit IRC | 10:19 | |
| *** MattMan_1 has joined #openstack-ironic | 10:20 | |
| mgoddard_ | wong: do you need non-baremetal compute as well? That hasn't been tried with Kayobe yet so it would be treading new ground | 10:22 |
| masber | good afternoon all, node validation is giving some errors http://paste.openstack.org/show/616378/ kaifeng mentioned that I need to create a port, is it a Port groups support? https://docs.openstack.org/ironic/latest/admin/portgroups.html | 10:23 |
| mkrai | Hi Ironic Team | 10:24 |
| mkrai | Can you please help review this https://review.openstack.org/#/c/486934/ ? | 10:24 |
| patchbot | patch 486934 - ironic - [doc-migration] Add configuration folder for docum... | 10:24 |
| dtantsur | masber: hi! I don't think you need port groups, just at least one port will work | 10:24 |
| mkrai | This is necessary for pike docs | 10:24 |
| dtantsur | masber: there is some information about ports on https://docs.openstack.org/ironic/latest/install/enrollment.html#enrollment-process | 10:25 |
| milan | morning asettle, dtantsur, masberg, mgoddard_, mkrai, sambetts! :) | 10:25 |
| dtantsur | hey milan | 10:25 |
| milan | :) | 10:25 |
| asettle | o/ | 10:25 |
| * asettle whispers | 10:25 | |
| asettle | Does this mean I'm a part of ironic now? | 10:25 |
| dtantsur | mkrai: added to my queue, will check once Jenkins passes. thanks! | 10:26 |
| *** syqian has quit IRC | 10:26 | |
| dtantsur | asettle: yep. welcome! :) | 10:26 |
| asettle | Hooray! | 10:26 |
| *** syqian has joined #openstack-ironic | 10:28 | |
| *** jkilpatr has quit IRC | 10:29 | |
| sambetts | asettle: obligatory link to UGT for the new comer http://www.total-knowledge.com/~ilya/mips/ugt.html | 10:30 |
| sambetts | ;) | 10:30 |
| asettle | sambetts: "Your ass will be laminated" well that's nice | 10:30 |
| asettle | Although, I completely adhere to UGT without realising it | 10:31 |
| asettle | Neato | 10:31 |
| sambetts | ;P | 10:31 |
| masber | dtantsur, I added the port to my node-create command but still complains http://paste.openstack.org/raw/616410/ | 10:36 |
| *** jjung_ has quit IRC | 10:37 | |
| *** jjung has quit IRC | 10:37 | |
| sambetts | masber: ironic node-port-list 089d1588-604e-4a1b-9ca1-3a039a935d98 | 10:37 |
| sambetts | masber: is your port listed there | 10:37 |
| mgoddard_ | morning milan & all | 10:37 |
| sambetts | o/ mgoddard_ | 10:38 |
| masber | sambetts, no port listed after running your command | 10:41 |
| sambetts | its sounds like your port create command didn't work then | 10:41 |
| sambetts | you need to do ironic port-create -n <node_uuid> -a <mac_address> | 10:42 |
| *** openstackgerrit has joined #openstack-ironic | 10:46 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Fixes hashing issues for py3.5 https://review.openstack.org/473888 | 10:46 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Fix small issues in the installation documentation https://review.openstack.org/486651 | 10:47 |
| *** jjung has joined #openstack-ironic | 10:51 | |
| *** jjung_ has joined #openstack-ironic | 10:51 | |
| *** mvpnitesh has joined #openstack-ironic | 10:52 | |
| *** chlong_ has joined #openstack-ironic | 10:52 | |
| *** chlong__ has quit IRC | 10:53 | |
| *** stendulker_ has joined #openstack-ironic | 10:54 | |
| *** lucasagomes has quit IRC | 10:55 | |
| *** chlong__ has joined #openstack-ironic | 10:57 | |
| *** stendulker has quit IRC | 10:58 | |
| *** chlong_ has quit IRC | 10:59 | |
| *** jkilpatr has joined #openstack-ironic | 10:59 | |
| *** stendulker_ has quit IRC | 11:00 | |
| *** sai has quit IRC | 11:01 | |
| *** rook has quit IRC | 11:01 | |
| vdrok | good morning asettle sambetts dtantsur masber ! | 11:03 |
| xavierr | good morning asettle sambetts dtantsur masber vdrok :) | 11:05 |
| dtantsur | o/ | 11:05 |
| vdrok | mornin xavierr | 11:06 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Guide on upgrading to hardware types https://review.openstack.org/486696 | 11:07 |
| dtantsur | a bit raw, and I cannot check the syntax, but worth reviewing anyway ^^ | 11:10 |
| asettle | o/ | 11:11 |
| masber | sambetts, so do I still need the option -i ipmi_port=623 when I create a node? | 11:11 |
| sambetts | not unless your using an non-standard ipmi port | 11:19 |
| *** lucasagomes has joined #openstack-ironic | 11:20 | |
| *** lucasagomes is now known as lucas-hungry | 11:20 | |
| *** zhangfei has quit IRC | 11:22 | |
| xavierr | vdrok: do you agree with Shivanand suggestion in https://review.openstack.org/#/c/482209/5/releasenotes/notes/oneview-agent-mixin-removal-b7277e8f20df5ef2.yaml :) | 11:24 |
| patchbot | patch 482209 - ironic - Removes agent mixin from oneview drivers | 11:24 |
| * xavierr asks before send a new patch set | 11:24 | |
| vdrok | xavierr: that wording seems fine to me | 11:25 |
| * xavierr opens vim to change the release notes | 11:26 | |
| masber | sambetts, ok, now I am getting a deploy_randisk and deploy_kernel errors on node-validation http://paste.openstack.org/raw/616419/ documentation says that "When using the Compute Service with the Bare Metal service, it is safe to ignore the deploy interface’s validation error due to lack of image information" I am not clear about that statement as I though everything in Ironic was baremetal service, could you please help me to understand? | 11:27 |
| *** jjung_ has quit IRC | 11:27 | |
| *** jjung has quit IRC | 11:27 | |
| *** syqian has quit IRC | 11:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/487008 | 11:27 |
| *** baoli has joined #openstack-ironic | 11:32 | |
| *** baoli has quit IRC | 11:32 | |
| *** baoli has joined #openstack-ironic | 11:33 | |
| *** ralonsoh has quit IRC | 11:33 | |
| *** sai has joined #openstack-ironic | 11:35 | |
| *** rook has joined #openstack-ironic | 11:35 | |
| *** lucasagomes has joined #openstack-ironic | 11:35 | |
| *** rook is now known as Guest57172 | 11:35 | |
| openstackgerrit | Xavier proposed openstack/ironic master: Removes agent mixin from oneview drivers https://review.openstack.org/482209 | 11:41 |
| sambetts | masber: you need to provide a node with driver_info/deploy_kernal and deploy_ramdisk equal to glance image UUIDs for the deploy ramdisk, e.g. IPA | 11:41 |
| sambetts | masber: the warnings you can ignore are about the image_source and related fields which are populated by nova at deploy time | 11:42 |
| openstackgerrit | Merged openstack/ironic master: Add tempest tests for physical networks https://review.openstack.org/470915 | 11:44 |
| *** chlong__ has quit IRC | 11:45 | |
| *** chlong_ has joined #openstack-ironic | 11:45 | |
| *** chlong_ has quit IRC | 11:58 | |
| masber | sambetts, I think that is what I am doing (assigning the deployment UUIDs to the node) http://paste.openstack.org/show/616426/ but still getting same error, the images have been created using coreOS tools | 11:59 |
| sambetts | masber: you have double labeled the parameters, so you've got driver_info/driver_info/deploy_ramdisk no driver_info/deploy_ramdisk | 12:01 |
| sambetts | not* | 12:01 |
| sambetts | > -i driver_info/deploy_kernel=7c038f5b-1b76-49a9-9add-41ce5e165a35 \ | 12:01 |
| sambetts | > -i driver_info/deploy_ramdisk=b7949118-4296-4050-878c-768532574b93 \ | 12:01 |
| sambetts | should just be deploy_kernel= and deploy_ramdisk= | 12:02 |
| sambetts | the -i == driver_info | 12:02 |
| *** litao__ has quit IRC | 12:05 | |
| *** tuanluong has quit IRC | 12:06 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Remove usage of some of the deprecated methods https://review.openstack.org/487025 | 12:08 |
| vdrok | sambetts: tried to fix some things here https://review.openstack.org/487025, but still the most annoying is the pkg_resources one | 12:09 |
| patchbot | patch 487025 - ironic - Remove usage of some of the deprecated methods | 12:09 |
| *** namnh has quit IRC | 12:09 | |
| sambetts | vdrok: jlvillal has a fix for the pkg_resources one here: https://review.openstack.org/#/c/486796/ | 12:09 |
| patchbot | patch 486796 - oslo.utils - Prevent deprecation error messages from pkg_resources | 12:09 |
| vdrok | sambetts: oh cool! | 12:10 |
| sambetts | vdrok: awesome job! that engine facade one has been annoying the hell out of me for weeks | 12:10 |
| *** lucas-hungry has quit IRC | 12:11 | |
| *** jaganathan has quit IRC | 12:13 | |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Receive and store agent version on heartbeat https://review.openstack.org/364861 | 12:15 |
| *** trown|outtypewww is now known as trown | 12:17 | |
| masber | sambetts, thanks, so does it looks better? http://paste.openstack.org/raw/616430/ | 12:17 |
| sambetts | masber: yup :) ['ramdisk', 'kernel', 'image_source'] these are the ones nova will populate for you | 12:17 |
| *** jjung has joined #openstack-ironic | 12:18 | |
| *** jjung_ has joined #openstack-ironic | 12:18 | |
| *** Guest57172 is now known as rook | 12:21 | |
| *** jcoufal has joined #openstack-ironic | 12:23 | |
| * dtantsur sees his chances to get to the PTG diminishing with every day.. :( | 12:25 | |
| sambetts | :( | 12:26 |
| sambetts | visa issues? | 12:26 |
| dtantsur | well, not related to the US itself. rather issues right here.. | 12:27 |
| *** links has quit IRC | 12:27 | |
| sambetts | :( | 12:28 |
| *** MattMan_1 has quit IRC | 12:28 | |
| *** goldyfruit has joined #openstack-ironic | 12:33 | |
| *** racedo has joined #openstack-ironic | 12:38 | |
| goldyfruit | Morning | 12:42 |
| goldyfruit | Any of you had this issue from a Bifrost (newton branch) fresh install: | 12:42 |
| goldyfruit | Jul 24 23:55:00 bifrost ironic-api[2390]: 2017-07-24 23:55:00.586 2488 DEBUG oslo.messaging._drivers.impl_rabbit [-] Timed out waiting for RPC response: Timeout while waiting on RPC response - topic: "<unknown>", RPC method: "<unknown>" info: "<unknown>" _raise_timeout /usr/local/lib/python2.7/dist-packages/oslo_messaging/_drivers/impl_rabbit.py:1052 | 12:43 |
| goldyfruit | This happens when a node should be validated | 12:43 |
| goldyfruit | My RabbitMQ is running, a curl on 5672 port is responding, I can see Ironic connected to the AMQP bus | 12:44 |
| *** links has joined #openstack-ironic | 12:44 | |
| *** sauloaislan has quit IRC | 12:48 | |
| *** hshiina has quit IRC | 12:50 | |
| TheJulia | Good morning | 12:51 |
| TheJulia | goldyfruit: can you reproduce it with the ironic command line? | 12:51 |
| goldyfruit | TheJulia: Morning | 12:53 |
| goldyfruit | TheJulia: Yes I can | 12:53 |
| *** sauloaislan has joined #openstack-ironic | 12:53 | |
| goldyfruit | TheJulia: https://pastebin.com/BB1xgsvs | 12:53 |
| *** yufei has quit IRC | 12:54 | |
| goldyfruit | TheJulia: And the trace: https://pastebin.com/AnFhy0L0 | 12:54 |
| TheJulia | vsaienk0: regarding https://review.openstack.org/#/c/486852/, basically that means we can never land something in nova the same cycle we land the api in ironic. Kind of insanely stupid in my opinion. :( | 12:54 |
| patchbot | patch 486852 - ironic - DNM: Unwind Julia's Migraine... maybe. | 12:54 |
| TheJulia | goldyfruit: what do you see in your conductor log? | 12:55 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Guide on upgrading to hardware types https://review.openstack.org/486696 | 12:55 |
| dtantsur | morning TheJulia | 12:55 |
| TheJulia | good morning dtantsur | 12:55 |
| TheJulia | dtantsur: wrt, bfv, I think the only way is to begin to craft retry and/or fallback logic into the nova patch. | 12:56 |
| * dtantsur sighs | 12:56 | |
| goldyfruit | TheJulia: same as for the API, timeout: https://pastebin.com/WFFgsfmH | 12:56 |
| dtantsur | is it because of the upgrade? | 12:56 |
| goldyfruit | plop dtantsur | 12:56 |
| TheJulia | dtantsur: yeah, because we explicitly only run an old api endpoint. | 12:57 |
| dtantsur | TheJulia: I think if we document that ironic is always upgraded first, we should also test it in the CI | 12:57 |
| * dtantsur thinks | 12:57 | |
| TheJulia | dtantsur: except, we don't complete the upgrade, we just let the old endpoint run. | 12:57 |
| dtantsur | can we prevent nova from upgrading? can we actually only upgrade one ironic-conductor? | 12:57 |
| sambetts | we should probably complete the upgrade right? | 12:58 |
| TheJulia | no and that is what we presently do | 12:58 |
| *** ralonsoh has joined #openstack-ironic | 12:58 | |
| TheJulia | sambetts: I would think so, but grenade multihost forces a mixed environment. | 12:58 |
| sambetts | having old-api running and new conductors means your 50% through an upgrade | 12:58 |
| TheJulia | goldyfruit: what is the version of oslo messaging that is installed? | 12:59 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver https://review.openstack.org/466448 | 12:59 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack https://review.openstack.org/468712 | 12:59 |
| sambetts | grenade should test going from full old version to full new version, rolling upgrade grenade should test mixed | 12:59 |
| TheJulia | agreed | 12:59 |
| *** sasha2 has joined #openstack-ironic | 13:00 | |
| milan | morning goldyfruit, TheJulia! :) | 13:00 |
| goldyfruit | TheJulia: 5.17.2 | 13:00 |
| goldyfruit | milan: o/ | 13:00 |
| TheJulia | goldyfruit: can you try downgrading to oslo.messaging===5.10.2 ? | 13:01 |
| goldyfruit | TheJulia: I'll | 13:02 |
| TheJulia | that is the upper constraint for newton, sounds like a newer version got on there | 13:02 |
| *** peri_ has joined #openstack-ironic | 13:04 | |
| goldyfruit | TheJulia: That solved the issue \o/ | 13:04 |
| *** vmud213 has quit IRC | 13:04 | |
| TheJulia | dtantsur: any thoughts regarding grenade, since I'm being asked about bfv patches in nova | 13:04 |
| TheJulia | I'm kind of at this "there is no way we can land it" and not break our gate, but our grenade gate does not perform a complete upgrade, so it is already broken anyhow. | 13:05 |
| *** ralonsoh has quit IRC | 13:05 | |
| goldyfruit | TheJulia: Thx | 13:05 |
| *** ralonsoh has joined #openstack-ironic | 13:05 | |
| dtantsur | TheJulia: it's a question of whether we want to 1. change our grenade, 2. or change our upgrade procedure and guarantees around it | 13:05 |
| dtantsur | #1 allows us to merge the nova change and figure out our problems | 13:06 |
| dtantsur | #2 requires us to rework our virt driver to be compatible with future changes (and already merged changes this release) | 13:06 |
| dtantsur | I belive that it's a bit too late to do #2 right now | 13:06 |
| dtantsur | TheJulia: does it make sense& | 13:06 |
| dtantsur | ? | 13:06 |
| TheJulia | dtantsur: I checked history, the past releases are not impacted by #2. This is the first time this cycle we've bumped the microversion | 13:07 |
| sambetts | IMO we should do grenade regardless of #2 because our grenade isn't actually testing ironic post-upgrade correctly | 13:07 |
| TheJulia | the other feature worked without a micro version bump | 13:07 |
| dtantsur | right | 13:07 |
| dtantsur | still, if we say that starting with Pike we allow Nova N-1 to work with Ironic, we have to be quite serious about this claim | 13:08 |
| dtantsur | e.g. test it in the CI | 13:08 |
| sambetts | N-1 should already work with ironic | 13:08 |
| vsaienk0 | folks sorry I'm a bit out of context what is the problem with our upgrades and CI? | 13:09 |
| dtantsur | yeah, sorry, Ironic N-1, Nova N | 13:09 |
| goldyfruit | TheJulia: I have an other error, do you know where can I find the upper constraint for Newton ? | 13:09 |
| TheJulia | So the question becomes, do we let the bfv nova patch land to force us to fix grenade, or do we just punt for now and work on grenade | 13:09 |
| goldyfruit | TheJulia: I think it's the same kind of issue | 13:09 |
| TheJulia | vsaienk0: if we bump the minimum required micro version to a version released _this_ cycle inside of nova, our grenade test will fail because we never start a newer API, we just redirect requests to an older API endpoint on the second node. | 13:10 |
| sambetts | we pin the API version in the ironic-api to last n-1 version | 13:10 |
| TheJulia | goldyfruit: openstack/requirments, look in the stable/newton branch, file upper-constraints.txt | 13:11 |
| dtantsur | I vote "fix the grenade", as the recommended upgrade order is 1. ironic-conductor, 2. ironic-api, 3. nova-compute | 13:11 |
| vsaienk0 | TheJulia: I see, this only affects multinode grenade right? | 13:11 |
| *** mvpnitesh has quit IRC | 13:11 | |
| TheJulia | vsaienk0: yes | 13:11 |
| goldyfruit | TheJulia: +1 Thanks | 13:11 |
| *** Goneri has joined #openstack-ironic | 13:11 | |
| openstackgerrit | Fellype Cavalcante proposed openstack/ironic master: Add a flag to always peform persistent boot on PXE interface https://review.openstack.org/483430 | 13:11 |
| *** jaganathan has joined #openstack-ironic | 13:11 | |
| *** mvpnitesh has joined #openstack-ironic | 13:11 | |
| *** fellypefca has joined #openstack-ironic | 13:12 | |
| vsaienk0 | ok so multinode grenade is like a partial upgrade, an we test there that all works when we have api and conductors with different versions, we may disable nova upgrade there I think | 13:12 |
| dtantsur | I wonder if we should even disable nova there and use one of standalone tests :) | 13:12 |
| dtantsur | but yeah, I agree with vsaienk0 | 13:13 |
| TheJulia | I'm not sure we can realistically, but that is a viable option | 13:13 |
| TheJulia | dtantsur: +++++ | 13:13 |
| fellypefca | Good morning, Ironic :) | 13:13 |
| sambetts | disabling nova upgrade sounds like the right way to go if we want it to represent partial upgrade | 13:13 |
| TheJulia | do we hae a good document written to describe these? | 13:14 |
| TheJulia | describing the test cases we are actually trying to test. | 13:14 |
| sambetts | I think we had a spec or something for grenade testing right? | 13:15 |
| TheJulia | I'm more thinking we need some sort of living document that just details current state as we view them | 13:16 |
| sambetts | I think we wanted 3 different tests, 1. everything shutdown and upgraded and brought back up 2. partial upgrade, ironic only everything else n-1 3. rolling upgrade, conductors -> api -> nova | 13:16 |
| sambetts | TheJulia: +1 | 13:16 |
| dtantsur | ++ to both | 13:16 |
| TheJulia | specs are great, but... not everything lives onward like described in a spec. :) | 13:17 |
| sambetts | there was an etherpad when jlvillal, vsaienk0, and vdrok were working on it right? | 13:17 |
| vdrok | sambetts: I'm not really sure, don't remember :( | 13:18 |
| *** mvpnitesh has quit IRC | 13:18 | |
| vdrok | but what you describe sounds good | 13:18 |
| TheJulia | Anyone up for doing a quick review on https://review.openstack.org/#/c/484032/5 ? There is a follow-up covering jlvillal's review feedback. | 13:18 |
| patchbot | patch 484032 - ironic - Generate iPXE boot script on start up | 13:18 |
| *** mvpnitesh has joined #openstack-ironic | 13:18 | |
| sambetts | I was just going through that | 13:19 |
| sambetts | :) | 13:19 |
| TheJulia | awesome! | 13:19 |
| sambetts | Found this: https://etherpad.openstack.org/p/IronicCI and https://etherpad.openstack.org/p/ironic-newton-summit-grenade-worksession | 13:19 |
| TheJulia | I feel like the consensus is work towards letting the nova bfv patch land, and we independently go and fix our broken grenade job? | 13:19 |
| vsaienk0 | sambetts: I think we cover 1 and 2 already by singlenode grenade, and multinode grenade. I don't think we can cover 3 easily as it will require upgrade scripts to perform upgrades on subnode, which is not allowed by design at the moment | 13:20 |
| sambetts | vsaienk0: right, rolling upgrade is harder | 13:20 |
| dtantsur | TheJulia: I skimmed through it, and just suggested merging it with a follow-up on #openstack-nova :) | 13:21 |
| vsaienk0 | by design I mean that it is not covered by devstack-gate and grenade scripts | 13:21 |
| sambetts | vsaienk0: 2 isn't covered by multinode because multinode upgrades other services, and doesn't fully upgrade ironic | 13:21 |
| vsaienk0 | sambetts: yes but we already have a case when all is upgraded it is single node grenade | 13:22 |
| sambetts | multinode is currently the oposite of partial upgrade, it upgrades everything else, and leaves ironic running in old version mode | 13:22 |
| dtantsur | sambetts: waiting for your review then, I'm +2 on it | 13:22 |
| vsaienk0 | sambetts: multinode grenade should test only stage when we have conductors and api with different versions, we shouldn't touch other services I think | 13:22 |
| sambetts | vsaienk0: the problem we're hitting is the nova is being upgraded in that job so TheJulia's BFV patch breaks because it requests a API version newer than ironic can support | 13:23 |
| sambetts | so its no doing partial upgrade correctly right now, because nova is upgrading | 13:24 |
| vsaienk0 | sambetts: I understand this, I think the propper fix here, is to disable upgrade for all services on multinode that should be upgraded after ironic like nova | 13:24 |
| sambetts | right | 13:25 |
| dtantsur | ++ | 13:25 |
| TheJulia | I think we are in agreement there | 13:25 |
| TheJulia | so the question then becomes, how | 13:25 |
| *** mvpnitesh has quit IRC | 13:25 | |
| sambetts | grenade has a flag for it according to the etherpad I just found | 13:25 |
| *** cdearborn has joined #openstack-ironic | 13:26 | |
| *** rloo has joined #openstack-ironic | 13:26 | |
| vsaienk0 | https://github.com/openstack-dev/grenade/blob/b69f4851950d2e6aa66617cf5b751fb8aa1c1324/projects/60_nova/settings | 13:27 |
| vsaienk0 | so the project for upgrade is registered by nova upgrade plugin | 13:27 |
| sambetts | so we just disable nova upgrade plugin? | 13:28 |
| sambetts | or I guess we can just export UPGRADE_PROJECTS="ironic" | 13:29 |
| sambetts | which is all the register_project_for_upgrade function appends | 13:29 |
| vsaienk0 | sambetts: nova is in grenade core, and nova settings will be always called here https://github.com/openstack-dev/grenade/blob/11dd94308ed5c25a8f28f86b03b20b251f0a05a1/inc/plugin#L35 | 13:31 |
| sambetts | I guess we'll have to over ride that field then ? | 13:31 |
| vsaienk0 | so I think we should modify trigger here https://github.com/openstack-dev/grenade/blob/b69f4851950d2e6aa66617cf5b751fb8aa1c1324/projects/60_nova/settings#L1 to something like if is_service_enabled nova && is_service_enabled ironic && not MULTINODE_JOB ... | 13:32 |
| *** baoli_ has joined #openstack-ironic | 13:33 | |
| sambetts | we shouldn't need to do that, 60_nova shouldn't need to be aware of ironic, otherwise every project's settings file will need a flag for is_service_enabled ironic, because its not just nova we don't want to upgrade, its every other project | 13:34 |
| *** jmorgan1_ has joined #openstack-ironic | 13:34 | |
| *** baoli has quit IRC | 13:35 | |
| *** crushil has quit IRC | 13:35 | |
| *** racedo has quit IRC | 13:35 | |
| *** jmorgan1 has quit IRC | 13:35 | |
| vsaienk0 | sambetts: I think we need to talk with sdague, as it will require a lot of changes | 13:35 |
| *** crushil has joined #openstack-ironic | 13:35 | |
| vsaienk0 | Do we have any other requirements that should be upgraded only after ironic except of Nova? | 13:36 |
| * dtantsur thinks only Nova | 13:36 | |
| *** chlong has joined #openstack-ironic | 13:37 | |
| vsaienk0 | ok so seems having extended if in nova upgrade settings is the only one place | 13:37 |
| masber | hi, I just tried my first deployment, the target server did restart but then it boot the original OS instead of deploying the new user image, this is what I can see from the ironic-conductor logs http://paste.openstack.org/raw/616440/ | 13:37 |
| * sambetts is very -1 to making nova aware of ironic upgrade | 13:37 | |
| masber | openstack server list shows the status of the baremetal server as BUILD | 13:38 |
| vsaienk0 | sambetts we can try to introduce something like DO_NOT_UPRADE_PROJECTS and check that project is not set https://github.com/openstack-dev/grenade/blob/11dd94308ed5c25a8f28f86b03b20b251f0a05a1/inc/plugin#L34 | 13:39 |
| vsaienk0 | the problem with core plugins is that they registered dynamically, can try to switch it to manual fashion, but will require a lot of changes to project-config... | 13:39 |
| sambetts | vsaienk0: like I said the function register_project_for_upgrade just adds the project to the field UPGRADE_PROJECTS so we can just override that field | 13:40 |
| sambetts | like UPGRADE_PROJETS="ironic" | 13:40 |
| vsaienk0 | sambetts: override from where? from ironic grenade plugin? | 13:41 |
| sambetts | yeah, something like that, in upgrade/settings or something | 13:42 |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic master: [doc-migration] Add configuration folder for documentation https://review.openstack.org/486934 | 13:42 |
| TheJulia | I believe upgrade/settings is read after the other projects are read in | 13:43 |
| TheJulia | I kind of like the DO_NOT_UPGRADE_PROJECTS flag | 13:43 |
| TheJulia | since that allows for some order to be imposed for partial cases like ours | 13:43 |
| vsaienk0 | yup it will work, but we can't drop projects that were added after ironic settings are sourced | 13:43 |
| masber | this may be more useful http://paste.openstack.org/show/616445/ | 13:46 |
| TheJulia | masber: do you have cleaning enabled? | 13:47 |
| vsaienk0 | UPGRADE_PROJECTS=$(echo $UPGRADE_PROJECTS | sed -e 's/ nova//g' ) | 13:47 |
| TheJulia | vsaienk0: that is kind of evil, and could just work | 13:47 |
| vsaienk0 | yup and better to add export of that variable | 13:48 |
| dtantsur | +2W on BFV, get your glasses, cool down your wine! | 13:49 |
| *** crushil has quit IRC | 13:49 | |
| *** jaganathan has quit IRC | 13:50 | |
| * jroll sends large bottles of bourbon instead \o/ | 13:50 | |
| *** e0ne has quit IRC | 13:50 | |
| * jroll realizes \o/ also looks like a whiskey glass with large round ice | 13:50 | |
| *** e0ne has joined #openstack-ironic | 13:50 | |
| vdrok | good morning jroll :) | 13:51 |
| jroll | morning! :) | 13:51 |
| *** e0ne has quit IRC | 13:51 | |
| rloo | good morning all, jroll (!), TheJulia, vsaienk0, dtantsur, sambetts masber, vdrok and everyone else! | 13:51 |
| vdrok | mornin rloo :) | 13:51 |
| *** e0ne has joined #openstack-ironic | 13:51 | |
| rloo | wrt order of project upgrades, this is what we documented in the spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/support-rolling-upgrade.html#problem-description | 13:51 |
| dtantsur | morning rloo, jroll | 13:51 |
| *** e0ne has quit IRC | 13:52 | |
| *** e0ne has joined #openstack-ironic | 13:52 | |
| *** e0ne has quit IRC | 13:52 | |
| *** e0ne has joined #openstack-ironic | 13:53 | |
| masber | TheJulia, I did put the cleaning network in the ironic configuration file | 13:53 |
| *** e0ne has quit IRC | 13:53 | |
| rloo | i think we/openstack ought to have a suggested upgrade order so openstack will work... | 13:53 |
| TheJulia | good morning jroll , rloo | 13:53 |
| *** e0ne has joined #openstack-ironic | 13:54 | |
| *** e0ne has quit IRC | 13:54 | |
| TheJulia | rloo: I think that backfired since we are not one of the main projects in grenade. | 13:54 |
| TheJulia | masber: so the question is really, what happened to the node during it's previous deployment, did it make it through cleaning after it was torn down from it's previous deployment or when it was set to manageable | 13:55 |
| rloo | TheJulia: right, so ahem, i think grenade infrastructure needs to be enhanced. I mean, *after* the community agrees with order of upgrades. i believe wrt nova, that they agree that ironic gets upgraded first. | 13:55 |
| *** zhangfei has joined #openstack-ironic | 13:55 | |
| TheJulia | rloo: enhanced is a good way of putting it :) | 13:56 |
| rloo | TheJulia: I suspect it isn't a bug, that grenade was initially written to support the 'core' projects. anyway, wondering if ptg/xproject discussion is in order | 13:57 |
| rloo | TheJulia: in the meantime, i think whatever we can do to hack it to work for us, is ok :) | 13:57 |
| openstackgerrit | Merged openstack/ironic-inspector master: Introducing the contributor content https://review.openstack.org/480222 | 13:57 |
| xavierr | dtantsur: vdrok: may you +2 https://review.openstack.org/#/c/482209/? All reviews done, Jenkins and CI passing :) | 13:58 |
| patchbot | patch 482209 - ironic - Removes agent mixin from oneview drivers | 13:58 |
| masber | TheJulia, automatic_celan is false and I did not manually cleaned the node. I can see the node provisioned state as "wait call-back" and maintenance as false. My guess is that cleaning step did not start because the boot step did not took long | 13:58 |
| TheJulia | rloo: I was kind of thinking the same last night when I got the message from JayF. The nature of the world of openstack has shifted over the past year. | 13:58 |
| *** crushil has joined #openstack-ironic | 13:59 | |
| rloo | TheJulia: ++ | 13:59 |
| *** ricardoas has quit IRC | 13:59 | |
| TheJulia | masber: indeed, you'll need it set to true to really prevent issues like your encountering | 13:59 |
| dtantsur | xavierr: putting on my list, will try to get to it today | 13:59 |
| vdrok | xavierr: will take a look, thx | 13:59 |
| * dtantsur -> short walk | 14:01 | |
| *** dtantsur is now known as dtantsur|brb | 14:01 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Fixes hashing issues for py3.5 https://review.openstack.org/473888 | 14:01 |
| vdrok | (fingers crossed) | 14:01 |
| TheJulia | rloo: I'm going to try and get that discussion started re grenade | 14:02 |
| rloo | TheJulia: thx! | 14:03 |
| TheJulia | at least, so maybe we can influence the order in the future | 14:03 |
| TheJulia | s/we/a plugin can/ | 14:03 |
| TheJulia | vdrok: \o/ | 14:03 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Fixes hashing issues for py3.5 https://review.openstack.org/473888 | 14:03 |
| *** chlong has quit IRC | 14:03 | |
| sambetts | the same issue exists with devstack plugins and order, there is currently no way to determine which order the devstack plugins are run | 14:03 |
| sambetts | so knowing if a service is up, e.g. glance if we need to use it to upload IPA images is hard | 14:04 |
| vdrok | sambetts: iirc there is a variable that defines the order, but yeah changing it does not sound good | 14:04 |
| rloo | wtf.. .i'm still seeing grenade errors, looks like the same neutron "no exchange 'q-agent-notifier-port-delete_fanout' in vhost" | 14:05 |
| rloo | http://logs.openstack.org/32/483432/1/gate/gate-grenade-dsvm-ironic-ubuntu-xenial/b37efc2/logs/new/screen-q-svc.txt.gz?level=ERROR | 14:05 |
| sambetts | rloo: /me smashe | 14:05 |
| * sambetts smashes face into desk | 14:05 | |
| rloo | sambetts: i was hoping you meant you were going to smash that bug :-( | 14:05 |
| sambetts | unless it was on my desk where my face just landed nope :/ | 14:06 |
| * TheJulia wonders if we could have anticipated that and placed a pillow to slightly soften the face smashing into desk | 14:06 | |
| rloo | sambetts: i hope i am wrong, maybe it is due to some other bug that got introduced? we should try to keep an eye on it, see if it fails again. | 14:07 |
| sambetts | +1 might be a different code path not caught by the neutron fix | 14:07 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Rolling upgrades support for create_port RPCAPI https://review.openstack.org/485773 | 14:07 |
| rloo | sambetts: i don't pay much attention to the tests, it is 'normal' (grenade test) for neutron to complain about "Failed to bind port 55c4410a-7547-4ad6-a949-f397b270379c on host 73ae7831-2d15-4b48-8c26-86ea7ef1ca26 for vnic_type baremetal using segments [{'network_id': '66d1b2a9-9d6d-42bb-9407-c4b0fb012e57', 'segmentation_id': 36, 'physical_network': None, 'id': '4c165893-5582-43a6-b937-76b0c5f90c03', 'network_type': u'vxlan'}]" | 14:10 |
| rloo | vsaienk0: ^^ ? | 14:10 |
| vdrok | rloo: that's fine, ml2 plugin in networking-baremetal should solve this | 14:10 |
| sambetts | unfortunatly yes, that is what the ml2 plugin should get rid of | 14:11 |
| rloo | vdrok, sambetts: ok. so we're back to q-agent-notifier-port-delete_fanout error :-( | 14:11 |
| sambetts | :( | 14:12 |
| TheJulia | Can we get a core that is not rloo to look at https://review.openstack.org/#/q/project:openstack/python-ironicclient real quick? easiest review of the week | 14:16 |
| TheJulia | err | 14:16 |
| TheJulia | no | 14:16 |
| TheJulia | wrong link | 14:16 |
| TheJulia | https://review.openstack.org/#/c/486677/ | 14:16 |
| patchbot | patch 486677 - python-ironicclient - Follow up to the API version warning patches | 14:16 |
| rloo | TheJulia: you made a very minor change, i think you can +2/A it | 14:16 |
| *** ricardoas has joined #openstack-ironic | 14:17 | |
| TheJulia | rloo: I touched half of it :\ | 14:17 |
| * TheJulia has a clipboard, and a list of checkboxes, detailing process... because reasons() ;) | 14:18 | |
| rloo | TheJulia: was it that much? :) Anyway, I am fine if you +A. it is a followup after all, and minor changes. We have bigger fish to fry! | 14:18 |
| TheJulia | thank you vdrok | 14:19 |
| vdrok | np :) | 14:19 |
| vsaienk0 | rloo: the error Failed to bind port 55c4410a-7547-4ad6-a949-f397b270379c .... 'network_type': u'vxlan'} is ok, as for unknown reason we still using vxlan networks in neutron for ironic | 14:20 |
| vsaienk0 | for flat case | 14:21 |
| rloo | vsaienk0: thx, that is good to know! | 14:21 |
| sambetts | my devstack plugin patches should change that ^ but I think they need a massive rebase | 14:22 |
| *** mtanino has joined #openstack-ironic | 14:24 | |
| *** mjturek has joined #openstack-ironic | 14:25 | |
| *** links has quit IRC | 14:26 | |
| TheJulia | rloo: It is on the QA ether pad. :) | 14:28 |
| rloo | TheJulia: yay! | 14:28 |
| rloo | TheJulia: so it isn't only ironic that wants that :) | 14:28 |
| TheJulia | rloo: it is likely worth noting other projects that would like or need such functionality. | 14:31 |
| rloo | TheJulia: btw, would be good to update our queens etherpad, to mention other sessions of interest | 14:32 |
| TheJulia | good point | 14:32 |
| *** udesale has quit IRC | 14:33 | |
| TheJulia | rloo: done | 14:34 |
| rloo | TheJulia: thx! | 14:35 |
| TheJulia | vsaienk0: it sounded like we have our plan for now until we can have better things, are you working in that direction or should someone else be doing so? | 14:36 |
| vsaienk0 | TheJulia: you mean try to disable nova upgrade? | 14:36 |
| TheJulia | yes | 14:36 |
| vsaienk0 | TheJulia: I will handle this | 14:36 |
| TheJulia | vsaienk0: Awesome, thanks! | 14:37 |
| *** trown is now known as trown|brb | 14:41 | |
| *** rbartal has quit IRC | 14:42 | |
| rloo | vdrok: wrt https://review.openstack.org/#/c/408556/, did you have any more comments/suggestions before i do another revision? | 14:43 |
| patchbot | patch 408556 - ironic - Add new dbsync command with first online data migr... | 14:43 |
| vdrok | rloo: will take a look in 10 mins | 14:43 |
| rloo | vdrok: thx! | 14:43 |
| *** trown|brb is now known as trown | 14:44 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Follow up to the API version warning patches https://review.openstack.org/486677 | 14:45 |
| sambetts | vsaienk0: ok going through the grenade rc scripts, if we add "export UPGRADE_PROJECTS="ironic"" to a localrc file in the grenade.sh directory that gets sourced after all the grenade plugins are loaded and might allow us to override as late as possible in the process | 14:46 |
| *** jjung has quit IRC | 14:46 | |
| *** jjung_ has quit IRC | 14:46 | |
| goldyfruit | TheJulia: I tried Bifrost's master branch and I have the same issue as before the Messaging Timeout | 14:48 |
| TheJulia | well that is not good | 14:49 |
| TheJulia | goldyfruit: can you send me the steps you used to install so I can stand up something similar | 14:50 |
| goldyfruit | TheJulia: I tried from something cleaned (removed all the python dependencies before), checkout the master branch, ran the bash script, ran the Ansible playbook, but but but but it doesn't like me anymore ! | 14:51 |
| masber | TheJulia, I enabled automatic cleaning and tried to deploy the server again but failed with same error. This is a screenshot of pxe boot process https://imagebin.ca/v/3Ub8OtxbTJKd | 14:52 |
| TheJulia | goldyfruit: can you run test-bifrost.sh? | 14:52 |
| goldyfruit | TheJulia: Sure | 14:52 |
| TheJulia | goldyfruit: save the stdout and stderr | 14:53 |
| TheJulia | it will create a logs folder as well, if you share that with me, I might be able to get to the bottom of your issue quickly | 14:53 |
| goldyfruit | TheJulia: It failed almost at the begin https://paste.debian.net/978085/ | 14:54 |
| goldyfruit | TheJulia: the script needs to be run from /opt/bifrost directory | 14:55 |
| goldyfruit | Running now | 14:55 |
| *** deadnull has joined #openstack-ironic | 14:56 | |
| *** cdearborn_ has joined #openstack-ironic | 14:57 | |
| TheJulia | hmm, okay | 14:57 |
| vdrok | rloo: all makes sense to me, apart from one place, commented | 15:03 |
| rloo | vdrok: thx! looking... | 15:03 |
| rloo | vdrok: good point wrt https://review.openstack.org/#/c/408556/63/ironic/cmd/dbsync.py | 15:07 |
| patchbot | patch 408556 - ironic - Add new dbsync command with first online data migr... | 15:07 |
| rloo | vdrok: i think we need to check before doing the upgrade then | 15:08 |
| rloo | vdrok: also, i forgot to ask you about your comment here: https://review.openstack.org/#/c/408556/63/doc/source/cli/ironic-dbsync.rst | 15:08 |
| patchbot | patch 408556 - ironic - Add new dbsync command with first online data migr... | 15:08 |
| rloo | vdrok: what did you mean? | 15:08 |
| *** baoli_ has quit IRC | 15:09 | |
| *** lifeless has quit IRC | 15:09 | |
| vdrok | rloo: it's just my imagination here :) like, you've upgraded the packages, with all the dependencies etc. and now realize that you need to install old code to run some command. so you'll maybe have to build a virtualenv with old requirements and run the command from inside it. just stating "hey, you'd better not forget to run the online migration" should be fine :) | 15:11 |
| rloo | vdrok: ah, that stuff will be in the documentation about rolling upgrades. I'll see if i can reword it a bit clearer. thx. | 15:12 |
| *** cdearborn has quit IRC | 15:12 | |
| vdrok | thank you! | 15:13 |
| openstackgerrit | Derek Higgins proposed openstack/ironic master: [TEST] [DO NOT MERGE] Nothing to see here https://review.openstack.org/487118 | 15:14 |
| *** MattMan has joined #openstack-ironic | 15:14 | |
| *** chlong has joined #openstack-ironic | 15:16 | |
| *** [1]rpioso has joined #openstack-ironic | 15:16 | |
| *** rcernin has quit IRC | 15:17 | |
| *** mjura has quit IRC | 15:17 | |
| * sambetts instantly wants to click on that ^ | 15:20 | |
| derekh | You'll be very disappointed | 15:21 |
| NobodyCam | Good morning Ironic'ers | 15:21 |
| *** lifeless has joined #openstack-ironic | 15:22 | |
| *** zhangfei has quit IRC | 15:23 | |
| *** vishwanathj has joined #openstack-ironic | 15:24 | |
| *** milan has quit IRC | 15:26 | |
| *** rama_y has joined #openstack-ironic | 15:29 | |
| anupn | Good morning Ironic'ers, NobodyCam :) | 15:29 |
| *** lifeless has quit IRC | 15:29 | |
| NobodyCam | Good Morning anupn :) | 15:29 |
| rama_y | Good morning all, anupn, NobodyCam! | 15:32 |
| NobodyCam | hey hey Good Morning rama_y :) | 15:32 |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/487008 | 15:33 |
| vdrok | morning NobodyCam anupn rama_y and derekh ! | 15:34 |
| NobodyCam | Good Morning vdrok :) | 15:34 |
| anupn | Morning vdrok, rama_y | 15:34 |
| derekh | good morning | 15:35 |
| TheJulia | derekh: I was super disappointed ;) | 15:35 |
| derekh | TheJulia: sorry ;-( | 15:35 |
| *** lifeless has joined #openstack-ironic | 15:36 | |
| *** vishwanathj has quit IRC | 15:36 | |
| *** baoli has joined #openstack-ironic | 15:36 | |
| *** vishwanathj has joined #openstack-ironic | 15:37 | |
| rama_y | Good morning vdrok! | 15:45 |
| *** fellypefca has quit IRC | 15:45 | |
| jlvillal | TheJulia: dtantsur|brb: Can you bring me up to speed on BFV? I saw a comment in the Nova channel about asking if I was okay with something. | 15:47 |
| TheJulia | hmm... where would you like me to start? | 15:47 |
| jlvillal | TheJulia: Well basically am I supposed to say yes or no to something? | 15:48 |
| jlvillal | TheJulia: I'm okay with landing the Nova patch now and finishing up the ironic patches this week. | 15:48 |
| jlvillal | TheJulia: Frankly the sooner we land the Nova patch the better it seems | 15:48 |
| TheJulia | jlvillal: that is what has occurred. :) | 15:48 |
| jlvillal | Okay, great :) | 15:48 |
| jlvillal | TheJulia: I was reading the backscroll where I got pinged :) | 15:49 |
| *** stendulker has joined #openstack-ironic | 15:49 | |
| jlvillal | About two hours ago | 15:49 |
| TheJulia | vsaienk0: is taking on trying to fix the grenade issue sine our test should not be upgrading nova in that specific scenario | 15:49 |
| TheJulia | s/sine/since/ | 15:49 |
| TheJulia | I'm having a latte trying to relax a little, and then some in tree patch revisions | 15:52 |
| *** rama_y has quit IRC | 15:52 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Introducing the installation content https://review.openstack.org/480249 | 15:52 |
| jlvillal | TheJulia: Thanks for the info! | 15:54 |
| jlvillal | TheJulia: And woo hoo! on the Nova patch landing :) | 15:55 |
| mjturek | that's awesome!!!! | 15:56 |
| TheJulia | it is awesome, now just think of all the doc, and next steps in the next cycle. | 15:56 |
| mjturek | congrats TheJulia hshiina | 15:56 |
| mjturek | :) sounds fun | 15:56 |
| *** baoli has quit IRC | 15:57 | |
| rloo | yay!! thanks to you too mjturek :) | 15:58 |
| *** jistr is now known as jistr|mtg | 15:58 | |
| mjturek | :) | 15:58 |
| *** baoli has joined #openstack-ironic | 15:59 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Introducing the user content https://review.openstack.org/480250 | 16:02 |
| openstackgerrit | Merged openstack/ironic-inspector master: Docs migration conclusion https://review.openstack.org/480251 | 16:02 |
| *** dr_gogeta86_ has joined #openstack-ironic | 16:03 | |
| *** mcarden_ has joined #openstack-ironic | 16:03 | |
| *** dr_gogeta86 has quit IRC | 16:06 | |
| *** mcarden has quit IRC | 16:06 | |
| *** kzaitsev_ws has joined #openstack-ironic | 16:08 | |
| kzaitsev_ws | any pointers why a node that's visible under ironic node-list doesn't show up under nova hypervisor-list? | 16:08 |
| kzaitsev_ws | who's logs should I look at? | 16:09 |
| *** ccamacho has quit IRC | 16:11 | |
| *** amarao1 has quit IRC | 16:14 | |
| *** lucasagomes is now known as lucas-hungry | 16:14 | |
| *** lucas-hungry is now known as lucas-afk | 16:14 | |
| *** rcernin has joined #openstack-ironic | 16:14 | |
| *** amarao1 has joined #openstack-ironic | 16:15 | |
| *** jaganathan has joined #openstack-ironic | 16:16 | |
| jlvillal | TheJulia: Are we holding off on merging: https://review.openstack.org/#/c/484032/ ? Has four +2s now. | 16:16 |
| patchbot | patch 484032 - ironic - Generate iPXE boot script on start up | 16:16 |
| *** oomichi has quit IRC | 16:16 | |
| vdrok | kzaitsev_ws: compute logs for the start I suppose, they should be updated periodically | 16:17 |
| vdrok | s/they/cached node list | 16:17 |
| *** oomichi has joined #openstack-ironic | 16:18 | |
| xavierr | mjturek: when performing BFV, should we do cleaning anyways? | 16:18 |
| mjturek | xavierr: cleaning through the ramdisk should work, so if you have other cleaning steps then yes | 16:19 |
| *** jistr|mtg is now known as jistr | 16:19 | |
| *** amarao1 has quit IRC | 16:20 | |
| xavierr | mjturek: the IPA will run on the ram and then verify if there is a disk on that it will perform the operation, if not the operation will be skiped. is that true? :) | 16:21 |
| mjturek | xavierr: I believe so, TheJulia please correct me if I'm wrong ^ | 16:22 |
| xavierr | mjturek: may in the future we could configure each node to tell ironic if perform or not the cleaning operation. does it make sense? | 16:23 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Fixes hashing issues for py3.5 https://review.openstack.org/473888 | 16:23 |
| TheJulia | xavierr: yes, because there could still be local disks | 16:24 |
| xavierr | TheJulia: even if not using the local disk (only the volume) ironic will perform the cleaning? | 16:25 |
| TheJulia | xavierr: i've pondered such a feature actually and wouldn't be too terribly opposed to it, but it would defintiely be an "advanced usage - know what your doing" kind of thing with lots of warnings, and maybe only works once | 16:25 |
| TheJulia | xavierr: it should, because there could be firmware that also needs to be updated | 16:25 |
| TheJulia | that is good question if ipa will error or not. I think we opened a bug on that a long time ago. I think it just tries to wipe any disks that it finds | 16:26 |
| xavierr | TheJulia: is update firmware part of any clean step? | 16:26 |
| TheJulia | xavierr: it is, if so defined by a hardware manager | 16:26 |
| xavierr | TheJulia: oh, ok. I was trying to find 'should_write_image' before perform cleaning but now makes sense... :) | 16:28 |
| xavierr | TheJulia: considering the case we need a noop implementation but returning False on 'should_write_image' instead | 16:28 |
| mjturek | xavierr: I don't think skipping disks in cleaning is supported.. Maybe it is through root device hints? It probably would be a nice feature if you have any disks that you want to retain | 16:28 |
| xavierr | TheJulia: shoud we introduce a new storage interface for that? | 16:29 |
| openstackgerrit | Merged openstack/ironic master: Fix docstring and default value for local_group_info https://review.openstack.org/486911 | 16:29 |
| xavierr | mjturek: ++ | 16:29 |
| *** peri_ has quit IRC | 16:30 | |
| *** derekh has quit IRC | 16:30 | |
| TheJulia | mjturek: yeah, some details there would have to be hashed out. We have preserve_ephemeral for rebuild, but yeah... | 16:31 |
| TheJulia | xavierr: i'm not sure why you would need a storage interface that is noop and returns false thus skipping deploy logic | 16:32 |
| TheJulia | I mean, if you have a magical volume attachment mechanism in the background, then \o/ | 16:32 |
| xavierr | TheJulia: we want just update the server profile instead | 16:33 |
| xavierr | TheJulia: no need for pxe when using such technology | 16:33 |
| JayF | Also re: BfV + Cleaning -- you have to drop the assumption that cleaning is about disk wiping | 16:34 |
| JayF | it's about more than that in some environments - security, validation, etc | 16:34 |
| xavierr | TheJulia: image streamer ;) | 16:34 |
| TheJulia | xavierr: I was suspecting that... | 16:34 |
| xavierr | hahaha | 16:34 |
| mjturek | JayF: very true (and o/!!!) | 16:34 |
| JayF | https://github.com/jayofdoom/ipa-example-hardware-managers has examples of other use cases for cleaning :) | 16:34 |
| JayF | mjturek: o/ | 16:34 |
| JayF | I just read the right few lines of IRC, still not actually here :P | 16:35 |
| mjturek | :) | 16:35 |
| * jroll sees JayF very much here | 16:35 | |
| JayF | With only like, a dozen IRC channels now | 16:35 |
| JayF | it's easy to keep up :P | 16:35 |
| JayF | I was at 90+ when I actually worked on this stuff, hahaha | 16:35 |
| xavierr | hey jroll, JayF o/ | 16:36 |
| TheJulia | xavierr: yeah, I don't remember the mechanics of it anymore, but that could be a viable thing and would be a storage interface if your facilitating that.... although at the same time it is almost like a deploy interface. I guess technically depends on the mechanics and how storage ends up being used in those cases | 16:36 |
| *** fellypefca has joined #openstack-ironic | 16:36 | |
| jroll | \o xavierr | 16:36 |
| TheJulia | o/ JayF and jroll | 16:36 |
| jroll | \o TheJulia | 16:36 |
| goldyfruit | TheJulia: found the issue ! Not related to messing at all, my ramdisk URL had a typo, the validate was never able to validate, so I got an error from the messaging because it reached the timeout | 16:38 |
| goldyfruit | TheJulia: sorry to be... me ! | 16:38 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Changes log level of a message https://review.openstack.org/476847 | 16:38 |
| TheJulia | xavierr: I guess, a good way to scope it would be what are the inputs and desired results, and operational requirements along the way | 16:39 |
| xavierr | TheJulia: we were trying to use oneview of oneview classic drivers w/o boot interface, and updating the profile when allocating the node (applying profile). I got stuck 'waiting for callback' ... | 16:39 |
| xavierr | as axpected... hahaha | 16:39 |
| xavierr | TheJulia: please, clarify :) | 16:40 |
| TheJulia | xavierr: Use of the image streamer tech that you have access to, if it would fall into storage or deploy interfaces | 16:41 |
| *** trown is now known as trown|lunch | 16:41 | |
| *** rama_y has joined #openstack-ironic | 16:41 | |
| TheJulia | xavierr: it would be awesome if you could make it to the PTG, because this is a reason to possibly consider some additional interface changes | 16:41 |
| *** stendulker has quit IRC | 16:42 | |
| xavierr | TheJulia: indeed, we could add moke hooks methods to that :) | 16:42 |
| xavierr | s/moke/more/ | 16:43 |
| xavierr | TheJulia: but, ironic is planning to have agnostic deploy interfaces. isn't that true? | 16:44 |
| *** rama_y is now known as 18WAAOEC6 | 16:44 | |
| *** rama_y has joined #openstack-ironic | 16:44 | |
| *** rama_y has quit IRC | 16:44 | |
| xavierr | maybe it should go to the boot interface (only if storage is not the right place) | 16:45 |
| TheJulia | it has been discussed, part of the conundrum begins what to do about cleaning because deploy and cleaning are mixed together | 16:45 |
| TheJulia | xavierr: the more I think about it, it the more it feels like a variation on the deploy interface. | 16:47 |
| xavierr | that's the case, how to apply a profile w/o change cleaning methods from deploy interface... haha | 16:47 |
| *** mgoddard_ has quit IRC | 16:47 | |
| xavierr | TheJulia: ++ | 16:47 |
| xavierr | maybe in the future we have deploy and cleaning interfaces for nodes :) | 16:50 |
| *** deadnull has quit IRC | 16:50 | |
| * xavierr needs a coffee | 16:50 | |
| *** 18WAAOEC6 has quit IRC | 16:53 | |
| TheJulia | xavierr: Yeah, that is kind of where my brain was going with the discussion, hence why I feel like being at the PTG would be a GoodIdea™ | 16:53 |
| *** links has joined #openstack-ironic | 16:55 | |
| *** racedo has joined #openstack-ironic | 16:56 | |
| *** racedo has quit IRC | 16:56 | |
| *** racedo has joined #openstack-ironic | 16:56 | |
| *** racedo has quit IRC | 16:57 | |
| *** racedo has joined #openstack-ironic | 16:57 | |
| *** racedo has quit IRC | 16:58 | |
| *** ralonsoh has quit IRC | 16:58 | |
| openstackgerrit | Merged openstack/ironic master: Update URL home-page in documents according to document migration https://review.openstack.org/483654 | 17:00 |
| *** jaganathan has quit IRC | 17:03 | |
| *** ricardoas has quit IRC | 17:05 | |
| *** sauloaislan has quit IRC | 17:05 | |
| *** cdearborn has joined #openstack-ironic | 17:05 | |
| xavierr | yeah, TheJulia | 17:05 |
| xavierr | just curious, why GoodIdea™? :D | 17:05 |
| *** ricardoas has joined #openstack-ironic | 17:06 | |
| *** dtantsur|brb is now known as dtantsur | 17:09 | |
| dtantsur | morning NobodyCam, JayF, jlvillal | 17:09 |
| NobodyCam | Good Morning dtantsur :) | 17:09 |
| dtantsur | TheJulia: as a reponsible PTL I should not encourage people to drink, but today feels like opening a bottle :) | 17:09 |
| jlvillal | Hi dtantsur, NobodyCam, JayF :) | 17:10 |
| NobodyCam | hey hey Good morning jlvillal :) | 17:10 |
| * sambetts --> home | 17:10 | |
| sambetts | Night all! | 17:10 |
| NobodyCam | have a good night sambetts | 17:10 |
| *** sambetts is now known as sambetts|afk | 17:10 | |
| TheJulia | dtantsur: oh alright! ;) | 17:10 |
| sambetts|afk | cya NobodyCam | 17:10 |
| NobodyCam | :) | 17:10 |
| TheJulia | xavierr: because it is on the topics list | 17:11 |
| TheJulia | xavierr: and you driver brings a variation of use that forces us to look at things a little differently | 17:11 |
| TheJulia | xavierr: which is a good thing | 17:11 |
| dtantsur | folks, we have 4x +2 on https://review.openstack.org/#/c/484032/. is there any reasons not to approve it? | 17:13 |
| patchbot | patch 484032 - ironic - Generate iPXE boot script on start up | 17:13 |
| *** cdearborn has quit IRC | 17:13 | |
| *** marios has quit IRC | 17:13 | |
| TheJulia | no, I was just about to | 17:13 |
| xavierr | TheJulia: great! justing asking because of the ™ :) | 17:14 |
| * xavierr needs 484032 approved to keep working | 17:15 | |
| *** ChubYann has joined #openstack-ironic | 17:15 | |
| TheJulia | xavierr: it is approved, I fear we might just need to fix our grenade first, we shall see :) | 17:17 |
| *** baoli has quit IRC | 17:17 | |
| xavierr | \o/ | 17:17 |
| TheJulia | xavierr: I have a few minor follow-ups to fix up today, I'll likely just stack them all so you can pull the last one in if you want. | 17:18 |
| *** baoli has joined #openstack-ironic | 17:18 | |
| *** sauloaislan has joined #openstack-ironic | 17:18 | |
| xavierr | TheJulia: send the link | 17:19 |
| TheJulia | xavierr: will do when I get that far | 17:19 |
| xavierr | o/ | 17:19 |
| rloo | dtantsur: wrt https://review.openstack.org/#/c/486651/, let me know if you want to fix the nova link, or shall I +A? | 17:20 |
| patchbot | patch 486651 - ironic - Fix small issues in the installation documentation | 17:20 |
| *** cdearborn_ has quit IRC | 17:20 | |
| dtantsur | rloo: this is a nit-fixing follow-up patch, I guess we don't want a follow-up for a follow-up later? :) | 17:20 |
| dtantsur | will fix | 17:21 |
| rloo | dtantsur: heh. up to you :) | 17:21 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Guide on upgrading to hardware types https://review.openstack.org/486696 | 17:21 |
| dtantsur | rloo: another thing you may be excited to see ^ | 17:21 |
| dtantsur | (or maybe not) | 17:21 |
| rloo | dtantsur: NOOOOOOOO! Am trying to avoid doing docs until close to 'the end' :) | 17:22 |
| dtantsur | ok, it's fine :) it can wait indeed | 17:22 |
| dtantsur | rloo: re s/-/--/ do we actually use -- anywhere? I'm pretty sure I've always used a simple dash.. | 17:22 |
| *** baoli has quit IRC | 17:23 | |
| rloo | dtantsur: grammatically, it is -- (one long dash), but we don't tend to use it cuz I don't think people know about it. | 17:23 |
| rloo | dtantsur: i don't care that much, just thought i'd point it out cuz yeah, i took a real typing course many years ago :) | 17:23 |
| dtantsur | yeah, but it's not a double dash, but a special symbol, right? I'd use it if I know how to input it :D | 17:24 |
| rloo | dtantsur: i suspect there is some way to get the actual symbol. but. low. on. my. list... | 17:24 |
| rloo | dtantsur: it was a double dash with a real typewriter ;) | 17:24 |
| *** etingof has quit IRC | 17:25 | |
| rloo | dtantsur: i suspect that if you put '--', it'll get rendered as long-dash. http://docutils.sourceforge.net/FAQ.html#how-can-i-represent-esoteric-characters-e-g-character-entities-in-a-document | 17:28 |
| *** baoli has joined #openstack-ironic | 17:28 | |
| dtantsur | I can give it a try, but we'll only know it after the Jenkins run :) | 17:28 |
| kzaitsev_ws | vdrok: thanks =) indeed my compute was misconfigured | 17:29 |
| kzaitsev_ws | took me some time to recheck everything %) | 17:29 |
| openstackgerrit | Merged openstack/ironic-lib master: Update and replace http with https for doc links https://review.openstack.org/486047 | 17:29 |
| rloo | dtantsur: you're still having issues generating docs locally? yikes. | 17:29 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Fix small issues in the installation documentation https://review.openstack.org/486651 | 17:30 |
| dtantsur | rloo: anyway, here it is (without the dash change) ^^^ | 17:30 |
| *** links has quit IRC | 17:31 | |
| *** trown|lunch is now known as trown | 17:31 | |
| *** cdearborn__ has joined #openstack-ironic | 17:34 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Changes log level of a message https://review.openstack.org/476847 | 17:43 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: [DNM|WIP] Use mac virtual for OneView drivers https://review.openstack.org/485618 | 17:45 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Address review feedback for ipxe boot file fix https://review.openstack.org/486853 | 17:48 |
| *** oomichi_ has joined #openstack-ironic | 17:51 | |
| *** baoli has quit IRC | 17:52 | |
| *** oomichi has quit IRC | 17:52 | |
| *** oomichi_ is now known as oomichi | 17:52 | |
| *** harlowja has joined #openstack-ironic | 17:55 | |
| *** rama_y has joined #openstack-ironic | 17:57 | |
| *** baoli has joined #openstack-ironic | 17:58 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: BFV Deploy skip minor logging, logic, and test fixes https://review.openstack.org/479326 | 17:59 |
| *** cdearborn_ has joined #openstack-ironic | 17:59 | |
| xavierr | vdrok: pls, remember https://review.openstack.org/#/c/482209/ :) | 18:01 |
| patchbot | patch 482209 - ironic - Removes agent mixin from oneview drivers | 18:01 |
| anupn | TheJulia, rama_y ppiela crushil: Meeting time ;) | 18:01 |
| TheJulia | Thanks for the reminder, my brain was working on some notes | 18:03 |
| jlvillal | TheJulia: The experimental job just got approved :) https://review.openstack.org/#/c/480332/ | 18:04 |
| patchbot | patch 480332 - openstack-infra/project-config - Add an experimental job to test ironic boot from v... | 18:04 |
| jlvillal | TheJulia: So soon we can just do: check experimental | 18:04 |
| jlvillal | To kick off the experimental boot-from-volume test :) | 18:04 |
| *** penick has joined #openstack-ironic | 18:04 | |
| TheJulia | \o/ | 18:06 |
| TheJulia | I'm updating he whiteboard w/r/t the grenade-multinode failures that we discussed earlier. I'll finish typing out the notes in a little bit. | 18:06 |
| *** athomas has quit IRC | 18:09 | |
| *** cdearborn__ has quit IRC | 18:14 | |
| vdrok | Python 3 job passed http://logs.openstack.org/88/473888/11/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/4c2a6d4/ | 18:20 |
| vdrok | dtantsur: ^ just so u know | 18:22 |
| *** baoli has quit IRC | 18:23 | |
| *** penick has quit IRC | 18:27 | |
| rloo | 'Did a compute worker not boot correctly'? That is the question that grenade multinode wants to know :-( http://logs.openstack.org/73/485773/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/7554a75/logs/testr_results.html.gz | 18:28 |
| *** fragatina has quit IRC | 18:28 | |
| rloo | vdrok: nice! | 18:30 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Guide on upgrading to hardware types https://review.openstack.org/486696 | 18:31 |
| TheJulia | rloo: yeah, it didn't, just wrapped the UI meeting and need to finish updating the whiteboard :( | 18:32 |
| *** baoli has joined #openstack-ironic | 18:32 | |
| *** amarao1 has joined #openstack-ironic | 18:33 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Update URL home-page in documents according to document migration https://review.openstack.org/483655 | 18:34 |
| dtantsur | vdrok: awesome! | 18:37 |
| dtantsur | vdrok: I guess both of us should stop working at this hour :) | 18:37 |
| vdrok | :) good night everyone! | 18:42 |
| xavierr | bye vdrok | 18:43 |
| *** crushil has left #openstack-ironic | 18:44 | |
| *** crushil has joined #openstack-ironic | 18:44 | |
| *** priteau_ has joined #openstack-ironic | 18:44 | |
| *** priteau has quit IRC | 18:45 | |
| dtantsur | good night | 18:45 |
| *** dtantsur is now known as dtantsur|afk | 18:45 | |
| vdrok | xavierr: I do remember about that change, will take a look first thing in the morning | 18:45 |
| xavierr | vdrok: awesome, take care! | 18:45 |
| *** baoli has quit IRC | 18:47 | |
| *** baoli has joined #openstack-ironic | 18:47 | |
| TheJulia | Okay, eyes crossed and t's dotted... oh wait, I have that backwards.... | 18:48 |
| TheJulia | vsaienk0: https://bugs.launchpad.net/ironic/+bug/1706418 | 18:48 |
| openstack | Launchpad bug 1706418 in Ironic "Ironic multimode-grenade upgrades nova without upgrading the API" [Critical,Confirmed] - Assigned to Vasyl Saienko (vsaienko) | 18:48 |
| *** priteau_ has quit IRC | 18:48 | |
| TheJulia | vsaienk0: I went ahead and assigned it to you based on the discussion earlier. | 18:49 |
| *** penick has joined #openstack-ironic | 18:51 | |
| *** fragatina has joined #openstack-ironic | 18:51 | |
| *** baoli has quit IRC | 18:52 | |
| openstackgerrit | Rushil Chugh proposed openstack/ironic-specs master: Add spec for XClarity Driver https://review.openstack.org/476922 | 18:54 |
| *** penick has quit IRC | 18:58 | |
| *** baoli has joined #openstack-ironic | 19:02 | |
| *** amarao1 has quit IRC | 19:04 | |
| *** jcoufal has quit IRC | 19:08 | |
| *** penick has joined #openstack-ironic | 19:12 | |
| rloo | TheJulia: sorry, are you saying that the grenade error is due to the bfv stuff? is it random? | 19:15 |
| TheJulia | rloo: it is due to the bfv nova patch merging. Better to merge it and fix our broken grenade job then not get the feature in this cycle. | 19:19 |
| rloo | TheJulia: ahh. So it must be a random failure, otherwise you couldn't have merged the code in the first place? | 19:20 |
| rloo | TheJulia: oh, wait. due to the nova patch merging. It breaks the gate then, recheck won't work? | 19:20 |
| TheJulia | rloo: nova does not gate on it, so they were able to merge it. The next thing we realistically need to merge is the fix to our upgrade scripts | 19:20 |
| TheJulia | nope, it will not work | 19:21 |
| TheJulia | Well, if other tests fail, those might pass :) | 19:21 |
| rloo | TheJulia: ok, so our gate is broken. Would you please update our whiteboard then :) | 19:21 |
| TheJulia | this is purely limited to just that grenade scenario | 19:21 |
| TheJulia | rloo: already did | 19:21 |
| rloo | TheJulia: ok thx! | 19:21 |
| TheJulia | rloo: also while I opened the bug and all | 19:21 |
| TheJulia | err, why | 19:22 |
| rloo | TheJulia: I added a note in whiteboard, not to recheck since it won't help | 19:23 |
| TheJulia | And I just made the text larger :) | 19:24 |
| rloo | TheJulia: :D good idea | 19:24 |
| *** fragatin_ has joined #openstack-ironic | 19:38 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Removes unnecessary utf-8 encoding https://review.openstack.org/480099 | 19:40 |
| *** fragatina has quit IRC | 19:41 | |
| *** racedo has joined #openstack-ironic | 19:46 | |
| openstackgerrit | Jay Bryant proposed openstack/ironic-specs master: Add spec for XClarity Driver https://review.openstack.org/476922 | 19:48 |
| *** mtreinish has quit IRC | 19:55 | |
| openstackgerrit | Rushil Chugh proposed openstack/ironic-specs master: Add spec for XClarity Driver https://review.openstack.org/476922 | 19:59 |
| *** xavierr is now known as xavierr|afk | 20:09 | |
| *** priteau has joined #openstack-ironic | 20:09 | |
| *** fragatin_ has quit IRC | 20:10 | |
| *** mtreinish has joined #openstack-ironic | 20:10 | |
| *** fragatina has joined #openstack-ironic | 20:11 | |
| *** fragatina has quit IRC | 20:11 | |
| *** fragatina has joined #openstack-ironic | 20:12 | |
| rama_y | rloo, TheJulia: ironic node-show <node-id> doesn't show target_raid_config; openstack baremetal node show <node-id> does. | 20:12 |
| rloo | rama_y: oh really? | 20:12 |
| rama_y | However, if api version is specified, it displays. Is that the intended behavior? | 20:12 |
| rloo | rama_y: i would think that they both default to the same api version if api version is not specified. | 20:13 |
| rama_y | Without specifying api-version, I was unable to see target_raid_config | 20:13 |
| rloo | rama_y: if you do something like --debug, you should see the request header. what is the api version there? i think the client sets it but don't recall | 20:14 |
| *** Goneri has quit IRC | 20:14 | |
| *** e0ne has joined #openstack-ironic | 20:14 | |
| rama_y | rloo, it shows X-OpenStack-Ironic-API-Version: 1.9 | 20:18 |
| rloo | rama_y: i just tried it quickly. i think what is happening is that ironic CLI will show the text for 'target-raid-config', but no value. whereas we must have done something with OSC so that it doesn't show the text. the value itself is not shown if the API version is too low. | 20:19 |
| rloo | rama_y: do you actually see a value for the target-raid-config? | 20:20 |
| rama_y | The value is not shown if api version is not explicitly given. | 20:20 |
| rloo | rama_y: right. i think that is a shortcoming with ironic CLI. we never got around to cleaning that up. | 20:20 |
| rama_y | I do; once I give it as --ironic-api-version 1.31 or something like that | 20:20 |
| rama_y | So, should I enter a bug for this? | 20:21 |
| rloo | rama_y: i don't recall if there is a bug against that or not, you can take a look and add one if you want. but i don't think we will fix it; the client release is this week and we're going to deprecate ironic CLI. | 20:21 |
| rama_y | I saw the RFE on deprecating Ironic CLI | 20:21 |
| *** cdearborn__ has joined #openstack-ironic | 20:22 | |
| rloo | rama_y: right. so for 'documentation' purposes, we can mention in the bug that we aren't going to fix it. | 20:22 |
| rama_y | There is no bug against it. Ok, let me enter it then. | 20:22 |
| rama_y | Thanks. | 20:22 |
| rloo | thank you rama_y! | 20:22 |
| *** cdearborn_ has quit IRC | 20:24 | |
| *** jkilpatr has quit IRC | 20:25 | |
| *** fellypefca has quit IRC | 20:28 | |
| *** oomichi has quit IRC | 20:28 | |
| *** oomichi has joined #openstack-ironic | 20:28 | |
| *** crushil has quit IRC | 20:41 | |
| TheJulia | rama_y: That is known, as far as I'm aware, from a standpoint one is api version 1.9, the other is "last known" to receive updates, so it will show more data as I understand it. | 20:41 |
| rama_y | TheJulia: I found the doc on raid config: https://github.com/openstack/ironic/blob/02015a4be6b3333f2d78f852f9a36ec799b93910/doc/source/admin/raid.rst | 20:42 |
| rama_y | It says to use the api-version. | 20:42 |
| TheJulia | rama_y: it is also geared entirely around the CLI | 20:43 |
| rama_y | Yeah | 20:43 |
| * TheJulia sighs that we didn't get the docs all updated by now | 20:45 | |
| jlvillal | TheJulia: experimental job passed :) http://logs.openstack.org/26/479326/9/experimental/gate-tempest-dsvm-ironic-bfv-ubuntu-xenial-nv/0f4b1dc/ | 20:47 |
| TheJulia | rloo: double checking release notes is on my todo list tomorrow. | 20:48 |
| TheJulia | jlvillal: So. Totatally. Awesome! | 20:48 |
| jlvillal | STA! ;) | 20:48 |
| *** priteau has quit IRC | 20:48 | |
| TheJulia | This means I need to actually like... have an agenda for the thursday BFV meeting.. DOH | 20:48 |
| rloo | TheJulia: release notes for client packages? | 20:49 |
| TheJulia | client packages, and ironic-ui. Also need to build/review ironic for the cycle and begin that clean-up effort | 20:49 |
| rloo | TheJulia: ack. and Thanks! :) | 20:50 |
| *** crushil has joined #openstack-ironic | 20:51 | |
| *** trown is now known as trown|outtypewww | 20:53 | |
| *** jkilpatr has joined #openstack-ironic | 20:56 | |
| *** priteau has joined #openstack-ironic | 20:59 | |
| *** Goneri has joined #openstack-ironic | 21:09 | |
| *** priteau has quit IRC | 21:13 | |
| *** e0ne has quit IRC | 21:23 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Deprecate DHCP providers https://review.openstack.org/456127 | 21:31 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: DNM/WIP: Testing https://review.openstack.org/487223 | 21:31 |
| *** baoli has quit IRC | 21:34 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: DNM/WIP: Testing https://review.openstack.org/487224 | 21:34 |
| *** mtreinish has quit IRC | 21:42 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: DNM/WIP: Testing https://review.openstack.org/487224 | 21:44 |
| *** penick has quit IRC | 21:44 | |
| *** mtreinish has joined #openstack-ironic | 21:46 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 21:51 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 21:51 |
| *** penick has joined #openstack-ironic | 21:57 | |
| *** mjturek has quit IRC | 21:59 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: DNM/WIP: Testing https://review.openstack.org/487224 | 22:04 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: [POC|DNM|TEST] Add node.configdrive https://review.openstack.org/455811 | 22:07 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: DNM/WIP: Testing https://review.openstack.org/487224 | 22:21 |
| openstackgerrit | Nate Potter proposed openstack/sushy master: Create mappings for System Indicator LED https://review.openstack.org/485376 | 22:27 |
| *** Goneri has quit IRC | 22:27 | |
| openstackgerrit | Nate Potter proposed openstack/sushy master: Add DELETE method to connector https://review.openstack.org/487231 | 22:29 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: DNM/WIP: Testing https://review.openstack.org/487224 | 22:34 |
| *** sasha2 has quit IRC | 22:38 | |
| openstackgerrit | Nate Potter proposed openstack/sushy master: Add DELETE method to connector https://review.openstack.org/487231 | 22:40 |
| *** mtanino has quit IRC | 22:41 | |
| *** Goneri has joined #openstack-ironic | 22:49 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: DNM/WIP: Testing https://review.openstack.org/487224 | 22:56 |
| *** rcernin has quit IRC | 22:58 | |
| *** [1]rpioso has quit IRC | 23:00 | |
| openstackgerrit | Anup Navare proposed openstack/ironic-ui master: Display enabled interfaces for underlying driver https://review.openstack.org/467804 | 23:13 |
| openstackgerrit | Anup Navare proposed openstack/ironic-ui master: Migrate node-details controller tests to new framework https://review.openstack.org/486711 | 23:13 |
| *** chlong has quit IRC | 23:32 | |
| *** deep-book-gk_ has joined #openstack-ironic | 23:33 | |
| *** deep-book-gk_ has left #openstack-ironic | 23:35 | |
| *** vishwanathj has quit IRC | 23:37 | |
| *** vishwanathj has joined #openstack-ironic | 23:37 | |
| openstackgerrit | Nate Potter proposed openstack/sushy master: Add DELETE method to connector https://review.openstack.org/487231 | 23:38 |
| *** penick has quit IRC | 23:39 | |
| *** penick has joined #openstack-ironic | 23:40 | |
| *** penick has quit IRC | 23:45 | |
| *** jaypipes has quit IRC | 23:54 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!