| TheJulia | Oh, I see | 00:00 |
|---|---|---|
| TheJulia | I'm not really sure there is a way around that with the way the ironic driver and nova interact :\ | 00:00 |
| praneshp | ha, ok. thanks for your time! | 00:01 |
| *** davidlenwell has quit IRC | 00:03 | |
| *** praneshp has quit IRC | 00:07 | |
| *** praneshp_ has joined #openstack-ironic | 00:07 | |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack/ironic: Remove websockify from requirements https://review.openstack.org/373559 | 00:08 |
| SpamapS | TheJulia: ^^ A fine opportunity to start Ocata off right ;) | 00:09 |
| TheJulia | :) | 00:12 |
| *** Alexey_Abashkin_ has quit IRC | 00:13 | |
| *** Alexey_Abashkin_ has joined #openstack-ironic | 00:14 | |
| *** davidlenwell has joined #openstack-ironic | 00:16 | |
| *** Alexey_Abashkin_ has quit IRC | 00:18 | |
| *** BadCub has joined #openstack-ironic | 00:19 | |
| *** vishwanathj has quit IRC | 00:22 | |
| *** Goneri has joined #openstack-ironic | 00:30 | |
| *** Alexey_Abashkin_ has joined #openstack-ironic | 00:32 | |
| *** bdeschenes has quit IRC | 00:32 | |
| *** zackf has quit IRC | 00:34 | |
| *** Sukhdev has joined #openstack-ironic | 00:35 | |
| *** bdeschenes has joined #openstack-ironic | 00:36 | |
| *** mtanino has quit IRC | 00:37 | |
| *** bdeschenes has quit IRC | 00:39 | |
| *** Goneri has quit IRC | 00:40 | |
| TheJulia | @cinerama I added you to 363197 if you have any thoughts. | 00:42 |
| *** bdeschenes has joined #openstack-ironic | 00:43 | |
| *** hj-hpe has quit IRC | 00:44 | |
| *** hoangcx has joined #openstack-ironic | 00:48 | |
| *** BadCub has quit IRC | 00:55 | |
| *** Goneri has joined #openstack-ironic | 00:56 | |
| *** rbudden has quit IRC | 00:57 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Change Bifrost to TinyIPA as the default https://review.openstack.org/373574 | 01:01 |
| *** phuongnh has joined #openstack-ironic | 01:02 | |
| *** jabroney has joined #openstack-ironic | 01:12 | |
| *** bdeschenes has quit IRC | 01:13 | |
| *** rbrud has quit IRC | 01:13 | |
| *** manjeets has joined #openstack-ironic | 01:15 | |
| *** Goneri has quit IRC | 01:16 | |
| manjeets | I have a single node devstack with 4 ironic node vms | 01:18 |
| manjeets | when i boot nova instances | 01:18 |
| manjeets | it creates neutron port and delete stuck node status at wait-call and go back to available after some time | 01:18 |
| manjeets | here is trace of error i could find from cond logs | 01:19 |
| manjeets | http://paste.openstack.org/show/582345/ | 01:19 |
| openstackgerrit | zhangyanxian proposed openstack/ironic: Fix a typo in helper.py https://review.openstack.org/372994 | 01:19 |
| *** sdake_ has joined #openstack-ironic | 01:19 | |
| *** mtanino has joined #openstack-ironic | 01:20 | |
| *** sdake has quit IRC | 01:21 | |
| *** amotoki has joined #openstack-ironic | 01:26 | |
| *** rbrud has joined #openstack-ironic | 01:26 | |
| *** rbrud has quit IRC | 01:28 | |
| *** aNuposic has quit IRC | 01:29 | |
| *** zackf has joined #openstack-ironic | 01:34 | |
| *** rloo has quit IRC | 01:35 | |
| TheJulia | @cinerama I left you two questions on 367637 | 01:40 |
| openstackgerrit | Om Kumar proposed openstack/ironic: Add 10s sleep for iPXE boot. https://review.openstack.org/373194 | 01:41 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Make bifrost on stable/mitaka branch deploy stable/mitaka ironic https://review.openstack.org/363197 | 01:43 |
| *** baoli has joined #openstack-ironic | 01:51 | |
| openstackgerrit | Merged openstack/bifrost: bifrost-prepare-for-test-dynamic: Create known_hosts if it's not present https://review.openstack.org/370258 | 01:55 |
| *** tuanla has joined #openstack-ironic | 02:05 | |
| *** bfournie has joined #openstack-ironic | 02:12 | |
| *** amotoki has quit IRC | 02:25 | |
| *** ChubYann has quit IRC | 02:30 | |
| *** marlinc has quit IRC | 02:30 | |
| *** rama_y has quit IRC | 02:31 | |
| *** marlinc has joined #openstack-ironic | 02:32 | |
| *** amotoki has joined #openstack-ironic | 02:41 | |
| *** amotoki has quit IRC | 02:49 | |
| *** cmyster has quit IRC | 02:49 | |
| *** amotoki has joined #openstack-ironic | 02:49 | |
| *** yuanying has quit IRC | 02:52 | |
| *** priteau has joined #openstack-ironic | 02:53 | |
| *** amotoki has quit IRC | 02:54 | |
| *** amotoki has joined #openstack-ironic | 02:58 | |
| *** priteau has quit IRC | 02:58 | |
| *** baoli has quit IRC | 03:01 | |
| *** david-lyle has quit IRC | 03:04 | |
| *** amotoki has quit IRC | 03:04 | |
| *** joprovost has quit IRC | 03:05 | |
| *** vikrant has joined #openstack-ironic | 03:15 | |
| *** appu103_ has joined #openstack-ironic | 03:29 | |
| *** appu103_ is now known as aparnav | 03:36 | |
| *** ijw has quit IRC | 03:40 | |
| *** sdake_ has quit IRC | 03:42 | |
| *** ijw_ has joined #openstack-ironic | 03:45 | |
| *** amotoki has joined #openstack-ironic | 03:46 | |
| *** ijw_ has quit IRC | 03:49 | |
| *** yuanying has joined #openstack-ironic | 03:49 | |
| *** vikrant has quit IRC | 03:51 | |
| *** vikrant has joined #openstack-ironic | 03:53 | |
| *** bdeschenes has joined #openstack-ironic | 04:11 | |
| *** syed_ has quit IRC | 04:13 | |
| *** ijw has joined #openstack-ironic | 04:14 | |
| *** ijw has quit IRC | 04:19 | |
| *** rama_y has joined #openstack-ironic | 04:19 | |
| *** bdeschenes has quit IRC | 04:25 | |
| *** tuanla has quit IRC | 04:35 | |
| *** tuanla has joined #openstack-ironic | 04:36 | |
| *** hemna_ has joined #openstack-ironic | 04:43 | |
| *** hemna_ has left #openstack-ironic | 04:44 | |
| *** hemna_ has quit IRC | 04:44 | |
| *** jaosorior has joined #openstack-ironic | 04:50 | |
| *** vsaienko has joined #openstack-ironic | 05:04 | |
| *** praneshp_ has quit IRC | 05:09 | |
| openstackgerrit | Merged openstack/ironic: Remove websockify from requirements https://review.openstack.org/373559 | 05:09 |
| *** jaosorior has quit IRC | 05:10 | |
| *** jaosorior has joined #openstack-ironic | 05:11 | |
| *** rbartal has joined #openstack-ironic | 05:14 | |
| *** moshele has joined #openstack-ironic | 05:21 | |
| *** wuhg has joined #openstack-ironic | 05:22 | |
| *** rama_y has quit IRC | 05:45 | |
| *** rbartal has quit IRC | 06:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/virtualbmc: Updated from global requirements https://review.openstack.org/373687 | 06:05 |
| *** rcernin has joined #openstack-ironic | 06:07 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/373701 | 06:08 |
| *** vsaienko has quit IRC | 06:11 | |
| *** mjura has joined #openstack-ironic | 06:11 | |
| *** vsaienko has joined #openstack-ironic | 06:14 | |
| *** mjura has quit IRC | 06:17 | |
| *** mjura has joined #openstack-ironic | 06:17 | |
| *** rbartal has joined #openstack-ironic | 06:18 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move root controller tests to functional dir https://review.openstack.org/356188 | 06:18 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move api driver tests to functional dir https://review.openstack.org/350365 | 06:18 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move port api tests to functional dir https://review.openstack.org/356287 | 06:18 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Prepare for functional tests https://review.openstack.org/349421 | 06:18 |
| *** athomas has joined #openstack-ironic | 06:24 | |
| *** strigazi_AFK is now known as strigazi | 06:51 | |
| *** Sukhdev has quit IRC | 07:06 | |
| *** ccamacho has joined #openstack-ironic | 07:08 | |
| *** pcaruana has joined #openstack-ironic | 07:13 | |
| *** fxpester has joined #openstack-ironic | 07:13 | |
| *** vsaienko has quit IRC | 07:14 | |
| *** Alexey_Abashkin_ has quit IRC | 07:20 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:23 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/373701 | 07:27 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/373747 | 07:27 |
| *** mtanino has quit IRC | 07:27 | |
| *** jabroney has quit IRC | 07:29 | |
| *** hshiina has joined #openstack-ironic | 07:33 | |
| *** jpich has joined #openstack-ironic | 07:36 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI https://review.openstack.org/348191 | 07:39 |
| fxpester | hi all, do I understand correctly that currently ironic uses 1 network for everything (provisioning traffic and teanant traffic) ? and VLAN/VXLAN segmentation is not supported ? | 07:41 |
| *** Romanenko_K has joined #openstack-ironic | 07:56 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** e0ne has joined #openstack-ironic | 08:11 | |
| *** jaosorior is now known as jaosorior_brb | 08:28 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool management driver for Inject NMI https://review.openstack.org/348721 | 08:41 |
| *** derekh has joined #openstack-ironic | 08:44 | |
| *** pmannidi has quit IRC | 08:48 | |
| *** hshiina has quit IRC | 08:58 | |
| *** pmannidi has joined #openstack-ironic | 09:09 | |
| *** pmannidi has quit IRC | 09:09 | |
| *** pmannidi has joined #openstack-ironic | 09:10 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: iRMC management driver for Inject NMI https://review.openstack.org/348732 | 09:13 |
| *** bks has joined #openstack-ironic | 09:16 | |
| mgould | morning Ironic! | 09:26 |
| tuanla | morning ironic | 09:45 |
| tuanla | I was have problem about ipmitool: ipmitool -I lanplus -H 10.0.0.10 -U admin -P admin chassis bootdev pxe options=persistent, I don't know how to using bootdev to set pxe but don't delete other boot priority | 09:47 |
| tuanla | I have a bug relate to whole-disk-image and after some research and test i know the causual because when i using bootdev pxe options=persistent every prioritys options will be delete just have only pxe or netboot | 09:49 |
| *** dtantsur|pto is now known as dtantsur | 09:53 | |
| dtantsur | morning Ironic! | 09:53 |
| * dtantsur is back | 09:53 | |
| vdrok | morning mgould and dtantsur ! | 09:59 |
| vdrok | dtantsur: welcome :) | 09:59 |
| mgould | vdrok dtantsur morning! | 10:00 |
| mgould | and welcome back dtantsur :-) | 10:00 |
| dtantsur | o/ | 10:00 |
| fxpester | hi all, do I understand correctly that currently ironic uses 1 network for everything (provisioning traffic and teanant traffic) ? and VLAN/VXLAN segmentation is not supported ? | 10:05 |
| dtantsur | fxpester, hi! this feature just got implemented in Newton, before that we only supported one network indeed. | 10:07 |
| vdrok | fxpester: yeah, it's in 6.1.0 | 10:07 |
| dtantsur | fxpester, http://docs.openstack.org/developer/ironic/deploy/multitenancy.html | 10:08 |
| fxpester | nice, maybe ,multi-rack setup also implemented in newton ? =) | 10:08 |
| fxpester | sorry, not multirack but multi-l3-networks with dhcp relay | 10:09 |
| dtantsur | I wish so :) I don't think it's up to Ironic to implement DHCP relays though | 10:09 |
| dtantsur | Ironic defers DHCP configuration to Neutron, so I guess it's within their domain | 10:10 |
| dtantsur | fxpester, ^^ | 10:10 |
| fxpester | dtantsur: thx a lot! | 10:11 |
| *** bdeschenes has joined #openstack-ironic | 10:27 | |
| *** makowals has quit IRC | 10:29 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch error when wrong temp_url were generated https://review.openstack.org/370875 | 10:35 |
| *** phuongnh has quit IRC | 10:35 | |
| *** hoangcx has quit IRC | 10:36 | |
| *** ohamada has joined #openstack-ironic | 10:36 | |
| *** tuanla has quit IRC | 10:41 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-python-agent: Improve error message while download image https://review.openstack.org/370824 | 10:48 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Switch stable/newton to Newton upper-constraints https://review.openstack.org/374023 | 10:51 |
| *** makowals has joined #openstack-ironic | 10:56 | |
| *** davidlenwell has quit IRC | 10:57 | |
| *** bdeschenes has quit IRC | 10:58 | |
| *** davidlenwell has joined #openstack-ironic | 10:58 | |
| *** ohamada has quit IRC | 10:59 | |
| *** ohamada has joined #openstack-ironic | 11:05 | |
| *** ohamada has quit IRC | 11:05 | |
| *** ijw has joined #openstack-ironic | 11:07 | |
| *** ijw has quit IRC | 11:12 | |
| dtantsur | aarefiev, morning! around? | 11:14 |
| aarefiev | dtantsur:hey | 11:14 |
| dtantsur | will pm you | 11:15 |
| *** baoli has joined #openstack-ironic | 11:15 | |
| *** baoli_ has joined #openstack-ironic | 11:18 | |
| *** lucas-afk is now known as lucasagomes | 11:18 | |
| lucasagomes | morning all | 11:19 |
| *** baoli has quit IRC | 11:21 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Add oslotest to test-requirements https://review.openstack.org/374045 | 11:22 |
| dtantsur | morning lucasagomes | 11:22 |
| *** jaosorior_brb is now known as jaosorior | 11:28 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Change the logic of selecting image for tests https://review.openstack.org/329625 | 11:33 |
| vdrok | morning lucasagomes | 11:34 |
| *** bdeschenes has joined #openstack-ironic | 11:34 | |
| *** mtanino has joined #openstack-ironic | 11:37 | |
| *** daemontool has joined #openstack-ironic | 11:39 | |
| *** bfournie has quit IRC | 11:42 | |
| xavierr | good morning Ironic | 11:42 |
| xavierr | morning dtantsur aarefiev lucasagomes vdrok | 11:43 |
| lucasagomes | xavierr, vdrok hello there :-) | 11:43 |
| vdrok | Morning xavierr :) | 11:47 |
| openstackgerrit | Ankit Kumar proposed openstack/ironic: Documentation change for feature updates in iLO drivers. https://review.openstack.org/374060 | 11:50 |
| *** athomas has quit IRC | 11:52 | |
| TheJulia | o/ | 11:52 |
| mat128 | morning xavierr, lucasagomes, vdrok, TheJulia and everyone else | 11:52 |
| *** mtanino has quit IRC | 11:53 | |
| xavierr | morning mat128 :D | 11:54 |
| * TheJulia wonders how long it takes for the company to update in stacklytics.... | 11:54 | |
| * xavierr asks if TheJulia is back to the channel | 11:55 | |
| xavierr | morning TheJulia \o | 11:55 |
| vdrok | Morning TheJulia and mat128 | 11:56 |
| *** athomas has joined #openstack-ironic | 11:56 | |
| wznoinsk | hi all | 11:56 |
| *** bks has quit IRC | 11:57 | |
| wznoinsk | I'm fixing ironic-inspector deployment via kolla and before I start adding sudoers file for rootwrap I wanted to check whether there's any, and if yes is it advanced, work currently on privsep for it? | 11:57 |
| lucasagomes | TheJulia, mat128 morning! | 11:57 |
| TheJulia | xavierr: yes, I'm back now :) | 11:58 |
| *** amotoki has quit IRC | 11:58 | |
| *** wuhg has quit IRC | 12:01 | |
| dtantsur | hi wznoinsk, we're not working on it right now. | 12:04 |
| dtantsur | morning TheJulia, xavierr, mat128 | 12:04 |
| openstackgerrit | Merged openstack/ironic-inspector: Tempest: add auto-discovery test https://review.openstack.org/369261 | 12:04 |
| dtantsur | aarefiev, mind +2 https://review.openstack.org/#/c/374045/ please? | 12:04 |
| patchbot | patch 374045 - python-ironic-inspector-client (stable/newton) - Add oslotest to test-requirements | 12:04 |
| *** vsaienko has joined #openstack-ironic | 12:05 | |
| lucasagomes | is someone here comfortable enough debugging tftp problems (at least I think it's tftp)... Quite often booting from UEFI (ovmf-edk2) fails with PXE-E18 timeout. Looking at tcpdump apparently DHCP works fine but it fails (apparently) at the tftp part | 12:05 |
| lucasagomes | see: http://paste.openstack.org/show/582391/ | 12:05 |
| dtantsur | "comfortable enough debugging tftp problems" everyone of course :D | 12:06 |
| lucasagomes | that said, the same configuration somtimes works / works every time for non UEFI | 12:06 |
| lucasagomes | dtantsur, hah | 12:06 |
| mat128 | lucasagomes: lovely hostname, "Standard-PC-i440FX-PIIX-1996" | 12:06 |
| lucasagomes | dtantsur, well gotta try, it's giving me some grey hair. Apparently not many people are using VMs with UEFI right now :-P | 12:06 |
| dtantsur | lucasagomes, I see it does request ipxe.efi | 12:07 |
| lucasagomes | mat128, heh default from ubuntu 16.04 :D | 12:07 |
| lucasagomes | dtantsur, yeah exactly I see the read request RRQ | 12:07 |
| lucasagomes | but I don't see any WRQ write request coming | 12:07 |
| lucasagomes | >.< | 12:07 |
| dtantsur | lucasagomes, it uses random ports, you won't see it with this request | 12:07 |
| * dtantsur got hit heavily by it one day | 12:07 | |
| mat128 | lucasagomes: you wont get WRQ unless the machine uploads a file | 12:07 |
| mat128 | what you want to see is the response to the read request | 12:08 |
| dtantsur | "port 67 or port 68 or port 69" won't show any sends or writes IIRC | 12:08 |
| lucasagomes | right hmm | 12:08 |
| lucasagomes | so yeah, apparently everything is good... I'm almost concluding that it may be some problem with the UEFI edk2 firmware itself | 12:08 |
| lucasagomes | because it works, sometimes | 12:08 |
| wznoinsk | dtantsur: ok, thanks | 12:08 |
| mat128 | lucasagomes: dump everything to/from the IP address | 12:09 |
| lucasagomes | ack | 12:09 |
| mat128 | which you wont know in advance, but you can probably guess it | 12:09 |
| mat128 | or change the dhcp range | 12:09 |
| mat128 | lucasagomes: any console log from the machine doing efi boot? anything interesting there? | 12:10 |
| lucasagomes | mat128, just the PXE-E18 then it falls into the UEFI shell | 12:11 |
| mat128 | lucasagomes: from https://github.com/tianocore/edk2/blob/af90df3cb099ed8e009579b7b55e7142dc0fc410/ShellPkg/Library/UefiShellTftpCommandLib/Tftp.c, I reckon it should update the "screen" with progress bars and status | 12:11 |
| mat128 | ah :( | 12:11 |
| lucasagomes | mat128, http://paste.openstack.org/show/582394/ | 12:12 |
| mat128 | :( | 12:12 |
| lucasagomes | mat128, yeah I see the dots ". . ." when it's booting then it times out | 12:12 |
| mat128 | nothing much there | 12:12 |
| lucasagomes | no worries :D I think that's the last bit actually needed | 12:13 |
| lucasagomes | once it works the uefi deployment finishes with no problem | 12:13 |
| *** dprince has joined #openstack-ironic | 12:13 | |
| lucasagomes | there's a trick, if I remove the uefi loader from the libvirt xml and boot in legacy bios the PXE boot works greatly | 12:13 |
| lucasagomes | then I turn off the amchine and add the loader again, then UEFI works as well | 12:14 |
| lucasagomes | it's pretty strange | 12:14 |
| mat128 | eww :S | 12:14 |
| mat128 | but you sometimes got it to boot PXE via UEFI? | 12:14 |
| lucasagomes | maybe I should ask in the IRC channel for tiano core, lemme find one | 12:14 |
| lucasagomes | mat128, yeah that's another option as well | 12:14 |
| lucasagomes | so I don't have to chainload anything | 12:14 |
| lucasagomes | well I still require tftp, more heavily | 12:15 |
| lucasagomes | hmm will see, thanks for the ideas | 12:15 |
| jroll | morning everyone | 12:15 |
| jroll | wb dtantsur \o/ | 12:15 |
| lucasagomes | jroll, hi there | 12:15 |
| dtantsur | hey-hey jroll! release time? | 12:15 |
| jroll | hey lucasagomes :) | 12:15 |
| jroll | dtantsur: hopefully! | 12:16 |
| *** bfournie has joined #openstack-ironic | 12:16 | |
| jroll | I need to look around, it would be nice to land the install guide refactor stuff if we can | 12:16 |
| dtantsur | jroll, I +2ed a few IPA patches as well this morning, might be worth checking | 12:16 |
| jroll | dtantsur: cool, will do | 12:16 |
| * dtantsur looks for the install guide | 12:16 | |
| dtantsur | much English, not wow :( | 12:17 |
| *** wznoinsk has quit IRC | 12:21 | |
| dtantsur | jroll, mat128, do we want to fix mistakes in the guide at the same time? | 12:23 |
| jroll | dtantsur: goal is to move it first, then fix things | 12:23 |
| mat128 | dtantsur: so far we were aiming for simple port, but if find something I can always fix it while fixing other things | 12:23 |
| dtantsur | ok, let's just not forget about the "fix" part | 12:24 |
| mat128 | like typos or smt | 12:24 |
| mat128 | but I found things that werent quite right in there, but wanted to submit as different patches later | 12:24 |
| jroll | dtantsur: the reason to do it for newton is they publish at /draft in master, and /branch for stable/branch. so merge now or deal with backports (because it's partially done) | 12:24 |
| dtantsur | +2 on the first patch with comments | 12:24 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 12:26 |
| *** wznoinsk has joined #openstack-ironic | 12:28 | |
| openstackgerrit | Aparna proposed openstack/proliantutils: Add a clean step 'hardware_disk_erase' https://review.openstack.org/336148 | 12:32 |
| * TheJulia reviews | 12:32 | |
| dtantsur | jroll and others: the proposed requirements updates seem to come from Ocata, don't approve them until we branch | 12:33 |
| jroll | +1 | 12:33 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Add oslotest to test-requirements https://review.openstack.org/374045 | 12:35 |
| *** rbudden has joined #openstack-ironic | 12:36 | |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic-specs: WIP: Add VCA hardware type to Ironic https://review.openstack.org/369309 | 12:39 |
| *** vmud213 has joined #openstack-ironic | 12:39 | |
| *** vikrant has quit IRC | 12:41 | |
| dtantsur | lucasagomes, do you plan to revive https://review.openstack.org/246033 ? I'd love to see it in newton | 12:42 |
| patchbot | patch 246033 - ironic - Conductor fails to do take_over for PXE drivers | 12:42 |
| dtantsur | jroll, ^^^ | 12:42 |
| lucasagomes | dtantsur, I really should | 12:42 |
| dtantsur | cool | 12:43 |
| lucasagomes | dtantsur, lemme do it now | 12:43 |
| jroll | +1 | 12:43 |
| *** Goneri has joined #openstack-ironic | 12:43 | |
| jroll | should be quick | 12:43 |
| TheJulia | dtantsur: jroll: w/r/t the install-guide importing the overview, +2'ed | 12:43 |
| jroll | cool | 12:44 |
| *** aparnav has quit IRC | 12:44 | |
| * jroll tidying up relnotes | 12:44 | |
| *** aparnav has joined #openstack-ironic | 12:44 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Local boot with partition images" https://review.openstack.org/369776 | 12:46 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Flavor creation" https://review.openstack.org/369761 | 12:46 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Enrollment" and "Troubleshooting" sections https://review.openstack.org/369777 | 12:46 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Add link to new guide in old install guide https://review.openstack.org/369802 | 12:46 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import Install and configure sections https://review.openstack.org/367988 | 12:46 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Building or downloading a deploy ramdisk image" https://review.openstack.org/369800 | 12:46 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Trusted boot with partition image" https://review.openstack.org/369801 | 12:46 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Bare Metal service overview" https://review.openstack.org/366329 | 12:46 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import configdrive https://review.openstack.org/369798 | 12:46 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Image requirements" https://review.openstack.org/369754 | 12:46 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Appending kernel parameters to boot instances" https://review.openstack.org/369799 | 12:46 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "integration with other OpenStack components" https://review.openstack.org/369510 | 12:46 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import HTTPS, standalone and root device hints https://review.openstack.org/369797 | 12:46 |
| *** Goneri has quit IRC | 12:46 | |
| *** mjturek has joined #openstack-ironic | 12:46 | |
| * TheJulia re-opens the review :) | 12:47 | |
| mat128 | ouch, race condition here :( | 12:47 |
| mat128 | sorry TheJulia and dtantsur | 12:47 |
| mat128 | want me to address your concerns right now? | 12:47 |
| mat128 | before we merge | 12:47 |
| TheJulia | If your up for it, sure | 12:47 |
| mat128 | ok, let me do this real quick | 12:48 |
| *** Goneri has joined #openstack-ironic | 12:49 | |
| *** mjturek has quit IRC | 12:51 | |
| *** milan has joined #openstack-ironic | 12:52 | |
| milan | morning Ironic! :) | 12:52 |
| TheJulia | good morning milan | 12:53 |
| milan | TheJulia, good morning! :) | 12:53 |
| dtantsur | good (3pm) morning milan :) | 12:53 |
| milan | dtantsur, :P forgotten to load chat | 12:53 |
| milan | guys, how do you feel about null vs infinity in a db column? | 12:54 |
| * milan talks about stuff like node.finished_at (inspector) and how to figure out not-finished things | 12:54 | |
| milan | https://review.openstack.org/#/c/344921/20/ironic_inspector/node_cache.py | 12:55 |
| patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 12:55 |
| * dtantsur tries not to think about null vs infinity in a db column | 12:56 | |
| dtantsur | :D | 12:56 |
| *** david-lyle has joined #openstack-ironic | 12:56 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up release notes for 6.2.0 https://review.openstack.org/373357 | 12:56 |
| jroll | ok, that should be ready to go ^^ | 12:56 |
| *** jcoufal has joined #openstack-ironic | 12:57 | |
| milan | dtantsur, heh :) | 12:57 |
| *** rloo has joined #openstack-ironic | 12:57 | |
| * jroll steps away for a bit and then looks at install guide and sees what else is left | 12:57 | |
| *** krtaylor has joined #openstack-ironic | 12:57 | |
| *** ElCoyote_ has left #openstack-ironic | 12:59 | |
| milan | I'm thinking about an "alias" attribute on the db model that would translate null to infinity on demand: finished_at_inf that one could use directly in pagination and filtering, such as inf >= node.finished_at_inf implying node not finished yet | 13:00 |
| milan | dtantsur, wdyt^ I know you try not to thing about those but still ;) | 13:01 |
| milan | aarefiev, hi, you've got an opinion? ^ | 13:02 |
| TheJulia | We really need to get consistent on tenses, "add" versus "added" :( | 13:02 |
| dtantsur | ++ | 13:02 |
| *** amotoki has joined #openstack-ironic | 13:03 | |
| milan | dtantsur, somehow I guess your ++ went to TheJulia instead of me ;) | 13:03 |
| dtantsur | milan, lol, and this is correct :) | 13:03 |
| TheJulia | dtantsur: we have "Removed" :) | 13:03 |
| dtantsur | milan, I'd drop marker until oslo.db is fixed | 13:04 |
| TheJulia | jroll: dtantsur and I have both posted feedback to the release note clean-up | 13:04 |
| milan | dtantsur, yeah, we had a chat with sambetts|pto and aarefiev on Monday where we agreed we shouldn't use the finished=True/False/None as it would be replaced by state=finished and deprecated | 13:05 |
| rloo | hi and morning milan, jroll. Welcome back TheJulia, dtantsur! | 13:05 |
| dtantsur | milan, correct | 13:05 |
| milan | dtantsur, so I tried to refactor the thing to base the filter on finished_at | 13:05 |
| rloo | TheJulia: ++ for tenses | 13:05 |
| dtantsur | morning rloo :) | 13:05 |
| TheJulia | good morning rloo | 13:05 |
| milan | morning rloo, jroll :) | 13:05 |
| *** vishwanathj has joined #openstack-ironic | 13:06 | |
| milan | dtantsur, but there are the damn nulls :D | 13:06 |
| dtantsur | milan, or primary goal is to have the API itself. limit is only to prevent too many records returned to a user | 13:06 |
| dtantsur | milan, but if you drop marker for now, nulls are no longer the problem, right? | 13:06 |
| milan | dtantsur, I know, but pagination without sort makes no sense as | 13:06 |
| zigo | hi there! | 13:06 |
| *** rbudden_ has joined #openstack-ironic | 13:06 | |
| rloo | jroll, mat128: what happens if we don't merge the install guide patches? will the official install guide be made avail in ../newton/, with links to the existing install guide? | 13:06 |
| zigo | Any idea why building Ironic leads to these 2 unit tests failures? http://logs.openstack.org/63/372763/2/check/gate-deb-ironic-dpkg-debian-jessie/24caf63/console.html | 13:06 |
| dtantsur | milan, we can skip pagination for the first patch.. | 13:07 |
| milan | as you might get the same items couple more than once | 13:07 |
| dtantsur | hey zigo | 13:07 |
| milan | zigo, o/ | 13:07 |
| *** mtanino has joined #openstack-ironic | 13:07 | |
| milan | dtantsur, I'd say we can do better: wait for the state patch | 13:07 |
| *** rbudden has quit IRC | 13:07 | |
| *** rbudden_ is now known as rbudden | 13:07 | |
| *** ElCoyote_ has joined #openstack-ironic | 13:07 | |
| dtantsur | milan, we can, but it's not necessary. the API makes sense even without states | 13:07 |
| milan | dtantsur, the workaround I've got (with the infinity) works | 13:08 |
| milan | but the filtering doesnt | 13:08 |
| milan | * sorting does filtering doesn't | 13:08 |
| dtantsur | did I ask for it? ;) | 13:08 |
| milan | so we could drop finished=True/False/None | 13:08 |
| milan | well, it's in the original bug | 13:08 |
| milan | RFE | 13:08 |
| milan | https://bugs.launchpad.net/ironic-inspector/+bug/1525238 | 13:09 |
| openstack | Launchpad bug 1525238 in Ironic Inspector "[RFE] API for listing introspection statuses" [Wishlist,In progress] - Assigned to milan k (vetrisko) | 13:09 |
| dtantsur | we can fix the RFE to mention states, then make this patch Partial-Bug | 13:09 |
| dtantsur | milan, ^^^ | 13:09 |
| milan | dtantsur, that might be the way out | 13:09 |
| milan | we could even have some filtering on finished_before, _after, started_before, _after queries but without null handling | 13:10 |
| *** jaosorior has quit IRC | 13:10 | |
| milan | aarefiev, wdyt ^ | 13:10 |
| *** jaosorior has joined #openstack-ironic | 13:11 | |
| dtantsur | zigo, the failure is correct, I really wonder why our unit test job does not catch it.. | 13:11 |
| dtantsur | zigo, probably because we mock things, while you're using actual proliantutils in our run | 13:11 |
| zigo | lol ! :) | 13:12 |
| dtantsur | s/our/your/ | 13:12 |
| zigo | For once, I don't get the "it works in devstack!" answer! :) | 13:12 |
| lucasagomes | mat128, dtantsur http://paste.openstack.org/show/582402/ | 13:12 |
| dtantsur | I can propose a patch, but I really wonder if we should actually install driver dependencies while unit testing... | 13:12 |
| *** mandre has joined #openstack-ironic | 13:12 | |
| dtantsur | jroll, ^^^ | 13:12 |
| zigo | dtantsur: FYI, that's the very latest tag of proliantutils that I'm using. | 13:12 |
| dtantsur | lucasagomes, wow, works? | 13:13 |
| milan | dtantsur, aarefiev in total we'd have: pagination&list all but no state filtering (yet), just time-based query/filtering | 13:13 |
| lucasagomes | dtantsur, found a way! Basically I replace the pxe rom with an EFI rom and then I don't need to chainload anymore | 13:13 |
| lucasagomes | dtantsur, sudo ln -s /usr/lib/ipxe/qemu/efi-virtio.rom /usr/share/qemu/pxe-virtio.rom | 13:13 |
| dtantsur | lucasagomes, aha! which is closer to bare metal, right? | 13:14 |
| lucasagomes | (gotta remove the first pxe-virtio.rom first) | 13:14 |
| lucasagomes | dtantsur, yes | 13:14 |
| lucasagomes | dtantsur, so now I think I gotta everything mapped to enable UEFI for devstack and later in gate | 13:14 |
| lucasagomes | the only concern is the cirros EFI which is not released yet | 13:14 |
| dtantsur | zigo, I'll create a patch for your bug (please file it in launchpad), and we'll continue discussing what we're doing in CI.. | 13:14 |
| lucasagomes | so I need to use a daily build one | 13:14 |
| mandre | hello, I'm having an issue with ironic not being able to run the partprobe command, http://paste.openstack.org/show/582401/ | 13:14 |
| mandre | anyone seen that before? | 13:15 |
| zigo | dtantsur: Ok, filling the bug. | 13:15 |
| dtantsur | thanks | 13:15 |
| dtantsur | mandre, ugh, no, lemme check | 13:15 |
| rloo | dtantsur: don't we have 3rd party iLo CI? | 13:15 |
| mandre | dtantsur: I've seen it when I try to deploy using an atomic-host image | 13:16 |
| dtantsur | rloo, it's about unit tests | 13:16 |
| *** hshiina has joined #openstack-ironic | 13:16 | |
| dtantsur | mandre, I wonder if a wrong file gets installed by packaging | 13:17 |
| lucasagomes | mandre, strange, it doesn't print any error in specifc right? Just returns exit 99 | 13:17 |
| rloo | dtantsur: the third party CI's don't run unit tests? | 13:17 |
| jroll | dtantsur: good question, we have talked about this before, until after newton release I have zero opinions | 13:17 |
| lucasagomes | Unauthorized command: partprobe | 13:17 |
| lucasagomes | seems some config problem ? | 13:17 |
| lucasagomes | seems like* | 13:17 |
| dtantsur | lucasagomes, yes, it seems like outdated filters are installed | 13:17 |
| dtantsur | lucasagomes, https://github.com/openstack/ironic/blob/master/etc/ironic/rootwrap.d/ironic-lib.filters instead of https://github.com/openstack/ironic-lib/blob/master/etc/rootwrap.d/ironic-lib.filters | 13:18 |
| jroll | rloo: I believe you're correct about how it will be published - provided we land the index, I guess | 13:18 |
| dtantsur | why do we even have a copy of ironic-lib filters in tree?? | 13:18 |
| lucasagomes | mandre, yeah please take a look at the filters there | 13:18 |
| jroll | because rootwrap is the worst thing ever | 13:18 |
| dtantsur | rloo, I don't think so, I think they run tempest | 13:18 |
| rloo | jroll: can we *not* have anything in the official install guide location? | 13:18 |
| mariojv | morning ironic o/ | 13:18 |
| lucasagomes | dtantsur, I think me and rloo already talked about it... we probably need privesep for ironic-lib | 13:19 |
| jroll | rloo: I think we already landed some things no? | 13:19 |
| dtantsur | jroll, lucasagomes, should we just resync both filters for now? | 13:19 |
| rloo | jroll: i feel like we/i am trying to rush the install guide migration. I'd rather not, and just do the release. | 13:19 |
| dtantsur | to avoid moving to privsep right today | 13:19 |
| * dtantsur puts up a patch | 13:19 | |
| jroll | rloo: I'm rushing it because we already moved some of it :( | 13:19 |
| jroll | dtantsur: yeah, seems fair | 13:19 |
| rloo | jroll: well, we landed stuff into ../draft/... I don't know if that means that when we do stable/newton, it'll get moved to ../newton/ or whatever. | 13:19 |
| zigo | daemontool: Can you confirm that it's only an issue in the tests, and that I can safely ignore them for the moment? | 13:19 |
| jroll | rloo: yes, when we branch, that will all be at newton | 13:20 |
| zigo | dtantsur: https://bugs.launchpad.net/ironic/+bug/1626089 | 13:20 |
| openstack | Launchpad bug 1626089 in Ironic "2 proliantutils related unit test failure when building the Debian package for 6.1.0" [Undecided,New] | 13:20 |
| rloo | jroll: or if we have to do something else to actually get it to appear. | 13:20 |
| jroll | rloo: master == /draft, stable/branch == /branch | 13:20 |
| dtantsur | thanks zigo, I'll fix it soon | 13:20 |
| rloo | jroll: maybe we should just take it out then. | 13:20 |
| jroll | rloo: I'm not opposed to reverting so we can just release | 13:20 |
| jroll | rloo: curious why just moving content is hard :( | 13:21 |
| mandre | dtantsur, lucasagomes: right, I have the old filter file | 13:21 |
| rloo | jroll: I'd prefer to revert. | 13:21 |
| jroll | rloo: but I guess we should try to do it right | 13:21 |
| jroll | instead of rush it | 13:21 |
| lucasagomes | mandre, try to update it and see if works please | 13:21 |
| rloo | jroll: i looked at the first two patches in the install saga. Started on Monday. It is Wed. That's why. | 13:21 |
| jroll | yeah | 13:21 |
| jroll | rloo: if you could propose reverts and -2 the first patch in the open series, I can land them | 13:21 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Sync ironic-lib.filters from ironic-lib https://review.openstack.org/374142 | 13:22 |
| dtantsur | lucasagomes, jroll ^^^ | 13:22 |
| jroll | and then we just need relnotes / rootwrap / proliantutils unit test junk to release IMO | 13:22 |
| jroll | dtantsur: ^ agree? | 13:22 |
| rloo | jroll: ok | 13:22 |
| rloo | mat128, JayF: are you ok with it ^^ reverting install guide stuff? | 13:22 |
| dtantsur | jroll, +1 | 13:22 |
| mat128 | wait, what? | 13:22 |
| jroll | mat128: tl;dr, let's not publish it like that for newton, and then do it right in ocata | 13:23 |
| mat128 | ... seriously? | 13:23 |
| jroll | mat128: instead of rushing it | 13:23 |
| mat128 | I thought we had most of it right | 13:23 |
| mandre | lucasagomes: just kicked a new deployment, give it a few minutes to see if it passes the ironic error this time | 13:23 |
| mat128 | and it's much better than what we had | 13:23 |
| lucasagomes | mandre, ack | 13:24 |
| mat128 | in the end, it's your call | 13:24 |
| jroll | mat128: so, I thought we did too, but apparently it's taking a long time to review and such, and we'd rather get the release out the door | 13:24 |
| jroll | but | 13:24 |
| jroll | let me look at it a bit | 13:24 |
| rloo | mat128: there are how many patches? And I started on Monday. Have only looked at two of them. What if there are other issues in the other patches? | 13:25 |
| dtantsur | mandre, fwiw I have a fix up: https://review.openstack.org/374142 | 13:25 |
| patchbot | patch 374142 - ironic - Sync ironic-lib.filters from ironic-lib | 13:25 |
| jroll | dtantsur: TheJulia: in the past we've always used present tense in release notes... I think of it as e.g. "this release adds support for" | 13:25 |
| mat128 | rloo: others are direct copy/paste with no structure change (they each have their own section) | 13:25 |
| openstackgerrit | Vadim Hmyrov proposed openstack/ironic-python-agent: Add bash scripts style checking for IPA https://review.openstack.org/374147 | 13:25 |
| jroll | dtantsur: TheJulia: 6.1.0 notes were all present tense | 13:26 |
| * TheJulia sighs | 13:26 | |
| mandre | dtantsur: nice, thanks] | 13:26 |
| patchbot | Error: Spurious "]". You may want to quote your arguments with double quotes in order to prevent extra brackets from being evaluated as nested commands. | 13:26 |
| dtantsur | wow | 13:26 |
| mandre | lol | 13:26 |
| lucasagomes | heh | 13:26 |
| dtantsur | is it always like] | 13:26 |
| patchbot | Error: Spurious "]". You may want to quote your arguments with double quotes in order to prevent extra brackets from being evaluated as nested commands. | 13:26 |
| dtantsur | ok, ok, let's not annoy patchbot | 13:27 |
| dtantsur | jroll, I'm fine with anything consistent and wrote down in our FAQ about release notes | 13:27 |
| dtantsur | * written | 13:27 |
| jroll | dtantsur: yeah, that's fair | 13:27 |
| rloo | TheJulia, dtantsur: I think we should come up with some guidelines on tense for the release notes, but maybe leave what we have for newton. we only see the ones that jroll has modified; dunno what the others have. we really need to catch/modify those when they are being merged | 13:27 |
| TheJulia | Agreed | 13:28 |
| dtantsur | even among ones jroll modified, some use "adds", some "add" | 13:28 |
| jroll | yeah, it's hard to get right | 13:28 |
| TheJulia | indeed | 13:28 |
| jroll | in reality it doesn't matter, because both get the point across | 13:28 |
| rloo | mat128: i'm going to put a -2 on the first patch, and review a few for about an hour and see how it goes. | 13:28 |
| lucasagomes | jroll, ++ | 13:28 |
| jroll | I try to keep it consistent because it's prettier, but don't get too bothered if I miss one | 13:28 |
| mat128 | rloo: ok | 13:28 |
| rloo | mat128: i wanted to get this done too, but things didn't move as fast as i wanted the last two days and it is already wed. | 13:29 |
| mandre | seems like I passed the rootwrap issue, but I got a different error this time: http://paste.openstack.org/show/582405/ | 13:30 |
| vdrok | morning jroll milan and rloo | 13:30 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add tense guide to release note FAQ https://review.openstack.org/374149 | 13:30 |
| milan | morning vdrok! :) | 13:30 |
| rloo | hi vdrok | 13:31 |
| openstackgerrit | Merged openstack/python-ironicclient: Hide 'nodes' field from chassis OSC subcommands output https://review.openstack.org/366731 | 13:31 |
| openstackgerrit | Merged openstack/python-ironicclient: Hide 'ports' field from node OSC subcommands output https://review.openstack.org/366703 | 13:31 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Local boot with partition images" https://review.openstack.org/369776 | 13:31 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Flavor creation" https://review.openstack.org/369761 | 13:31 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Enrollment" and "Troubleshooting" sections https://review.openstack.org/369777 | 13:31 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Add link to new guide in old install guide https://review.openstack.org/369802 | 13:31 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import Install and configure sections https://review.openstack.org/367988 | 13:31 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Building or downloading a deploy ramdisk image" https://review.openstack.org/369800 | 13:31 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Trusted boot with partition image" https://review.openstack.org/369801 | 13:31 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Bare Metal service overview" https://review.openstack.org/366329 | 13:31 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import configdrive https://review.openstack.org/369798 | 13:31 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Image requirements" https://review.openstack.org/369754 | 13:31 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Appending kernel parameters to boot instances" https://review.openstack.org/369799 | 13:31 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "integration with other OpenStack components" https://review.openstack.org/369510 | 13:31 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import HTTPS, standalone and root device hints https://review.openstack.org/369797 | 13:31 |
| mat128 | ^ last verison with adjustments in "Install and configure" | 13:32 |
| mandre | dtantsur, lucasagomes: have you seen http://paste.openstack.org/show/582405/ before? | 13:32 |
| * lucasagomes looks | 13:32 | |
| dtantsur | not really | 13:33 |
| lucasagomes | mandre, what driver are you using ? | 13:33 |
| mat128 | mandre: (long shot) maybe the iscsi port is a tuple instead of an int? | 13:34 |
| mat128 | thats the only thing that should be an int that I can guess | 13:34 |
| mandre | lucasagomes: how can I find which driver I'm using? | 13:34 |
| mat128 | mandre: ironic node-show | 13:34 |
| lucasagomes | mandre, ironic node-show <node uuid> | 13:34 |
| mandre | so it's using pxe_ssh | 13:35 |
| mat128 | rloo: thanks for your time, I will understand if we dont have time to review it all and make it for the release. (and because this is text and lacks all the non-verbal part, this is honest) | 13:36 |
| lucasagomes | mandre, can you also post the content of driver_infio ? (Hide the passwords if needed) | 13:36 |
| *** joprovost has joined #openstack-ironic | 13:36 | |
| *** mkam has joined #openstack-ironic | 13:36 | |
| rloo | mat128: :) | 13:36 |
| lucasagomes | mandre, driver_info* | 13:36 |
| *** ijw has joined #openstack-ironic | 13:38 | |
| * dtantsur does not have polite words for ilo unit tests | 13:38 | |
| * jroll tries to think who knows about ilo things | 13:38 | |
| jroll | TheJulia: seems proliantutils is ilo 4 only, is there similar OSS for ilo 3? | 13:38 |
| dtantsur | quite a few tests are broken, daaaamn | 13:39 |
| mandre | lucasagomes: this is 'ironic node-show', http://paste.openstack.org/show/582406/ | 13:39 |
| lucasagomes | jroll, Nisha and stendul<something> | 13:39 |
| jroll | lucasagomes: yeah, looking for people that are online :P | 13:41 |
| jroll | thanks though | 13:41 |
| JayF | rloo: obviously I'm not OK with that, or I wouldn't be pushing for us to get it in before release. | 13:41 |
| TheJulia | jroll: my understanding is ilo3 should work at this point on gen7 hardware which is ilo3 afaik | 13:41 |
| jroll | TheJulia: with proliantutils? | 13:41 |
| TheJulia | yes | 13:42 |
| jroll | awesome, thanks | 13:42 |
| * jroll wonders if friend has gen7 | 13:42 | |
| lucasagomes | mandre, looks alright hmmm... | 13:42 |
| TheJulia | jroll: worth testing, I think it mostly got introduced because a lab was largely gen7 gear | 13:42 |
| *** cdearborn has joined #openstack-ironic | 13:42 | |
| jroll | TheJulia: cool, thank you :) | 13:43 |
| rloo | JayF, mat128: what's your commitment for addressing/pushing up revisions? and what/which cores are willing to spend time reviewing them all in the next day (or less or more)? that's the issue. | 13:43 |
| jroll | rloo: so looking at http://docs.openstack.org/project-install-guide/baremetal/draft/ I don't know that we need to revert - just don't merge a bunch of other things | 13:43 |
| *** ijw has quit IRC | 13:43 | |
| JayF | I mean, I've been reviewing them with pretty good frequency; I don't know anyone else who is core who has committed to that | 13:43 |
| jroll | I'd prefer to release today rather than tomorrow fwiw | 13:43 |
| rloo | jroll: yeah, that's what I think. they point back to our install guide. | 13:43 |
| JayF | but I'm OK if we need to release, we need to release | 13:43 |
| jroll | yeah, so I say get the release out and iterate, maybe don't even bother backporting | 13:44 |
| rloo | jroll: ++ | 13:44 |
| JayF | we should change one thing when stable branch is cut: make those install guide links back to the devdoc-install-guide point to /newton/ instead of the master | 13:44 |
| *** aparnav has quit IRC | 13:45 | |
| rloo | jroll: the *only* thing... after we cut stable/newton, can we change all the links in the new install guide (in stable/newton) to point to ../developer/ironic/newton/..installguide? | 13:45 |
| jroll | rloo: yeah, that's fine | 13:45 |
| rloo | ++ JayF! | 13:45 |
| jroll | there's some other things that are branch-specific, e.g. we'll need to change the IPA download URL in our devstack plugin after we branch | 13:45 |
| rloo | JayF: I'd feel better, that we/folks have time to look at the draft after it is 'all done', before showing it to the world. | 13:46 |
| rloo | jroll: there needs to be a variable that indicates the cycle/master. | 13:46 |
| JayF | I mean, I think we're moving the target: we were going to just do a direct port | 13:46 |
| JayF | but everyone picked a different small thing to get caught up on, so it didn't get merged | 13:46 |
| mandre | lucasagomes: yeah, this odd, I wonder where this incorrect value is coming from | 13:47 |
| rloo | JayF: it wasn't a direct port. The new guide is organized differently. | 13:47 |
| JayF | for Ocata, we're almost certainly going to do more than forklift | 13:47 |
| JayF | which is probably better overall | 13:47 |
| jroll | rloo: when I talked to QA folks, it was kind of "it's complicated, just hardcode it" heh | 13:47 |
| lucasagomes | mandre, yeah I think it might be coming from "get_image_mb" in ironic-lib | 13:47 |
| rloo | jroll: ha ha | 13:47 |
| lucasagomes | by that traceback you sent | 13:47 |
| JayF | just disappointing we couldn't stretch and get the install guide in | 13:47 |
| rloo | JayF: I'm sorry, that's the way reviewing goes. I feel like I've been stretched for most of this cycle. And I was willing to spend time at the end on this, but I've experienced this before. at the end. rush something in. and it isn't quite right. | 13:48 |
| jroll | +1 | 13:48 |
| JayF | It is what it is :) limited resources and all | 13:48 |
| *** baoli_ has quit IRC | 13:48 | |
| rloo | JayF: yes, unfortunately. believe me, there are other features I would have liked to have gotten in this cycle; instead, I worked on stuff that seemed higher priority. | 13:49 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Fix broken unit tests for get_ilo_object https://review.openstack.org/374161 | 13:49 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch error when wrong temp_url were generated https://review.openstack.org/370875 | 13:49 |
| dtantsur | zigo, ^^^ | 13:49 |
| dtantsur | jroll, we need to get https://review.openstack.org/374161 in to fix zigo's problems, I guess | 13:49 |
| patchbot | patch 374161 - ironic - Fix broken unit tests for get_ilo_object | 13:49 |
| zigo | dtantsur: Ok, adding it to my package in debian/patches to see what happens... | 13:50 |
| jroll | dtantsur: yep, was waiting for that, thank you! | 13:50 |
| rloo | dtantsur: why did you change the numbers, eg 60 -> 600? https://review.openstack.org/#/c/374161/1/ironic/tests/unit/drivers/modules/ilo/test_common.py | 13:51 |
| patchbot | patch 374161 - ironic - Fix broken unit tests for get_ilo_object | 13:51 |
| dtantsur | rloo, see the commit message. these are the defaults, so this test did not test overriding the defaults | 13:51 |
| *** jrist has quit IRC | 13:52 | |
| rloo | dtantsur: #$@%. thx for fixing. | 13:52 |
| *** causten has joined #openstack-ironic | 13:52 | |
| fxpester | hi again, what driver I can use for custom play with Ironic in my virtualbox lab ? I want to use 2 VMs Ironic + Ironice-compute ? | 13:53 |
| fxpester | if I create node with Fake driver, attach port to it, will ironice provide PXE Boot for it ? | 13:54 |
| rloo | dtantsur, do you know. how is it that folks that work on ilo and code, don't run unit tests, and/or do they have to do something special to get these tests to run? | 13:54 |
| jroll | rloo: it fails with real proliantutils, but not our mocks, is the difference | 13:54 |
| jroll | so pip install proliantutils | 13:54 |
| rloo | jroll: i would have thought that the ilo folks would be using proliantutils though. | 13:55 |
| lucasagomes | fxpester, there's a vbox driver in tree and also pxe_ssh should support vbox | 13:55 |
| vdrok | fxpester: I think when using fake driver you'll have to do a whole bunch of things manually | 13:55 |
| lucasagomes | fxpester, http://docs.openstack.org/developer/ironic/drivers/vbox.html | 13:55 |
| lucasagomes | check if it fits ur needs | 13:55 |
| rloo | jroll: I mean, before submitting their code, they presumably ran the tests with proliantutils? | 13:55 |
| dtantsur | rloo, I suspect they just run "tox" | 13:55 |
| vdrok | fxpester: also libvirt driver from ironic-staging-drivers supports vbox I think | 13:55 |
| jroll | rloo: you would think, but tox won't do it for someone, so nobody will do it | 13:55 |
| lucasagomes | vdrok, ++ | 13:56 |
| vdrok | fxpester: https://github.com/openstack/ironic-staging-drivers/blob/master/doc/source/drivers/libvirt.rst | 13:56 |
| JayF | I wonder if we should add a tox target that could make it "easier" to run tests with all the various additional libs installed (like proliantutils) | 13:56 |
| rloo | jroll, dtantsur: is that something we need to ask when driver code is submitted? 'did you test...' | 13:56 |
| JayF | so perhaps folks who work on that could would be more apt to actually test it? | 13:56 |
| jroll | https://bugs.launchpad.net/ironic/+bug/1626106 | 13:56 |
| openstack | Launchpad bug 1626106 in Ironic "[RFE] Install optional dependencies when running unit tests" [Wishlist,Confirmed] | 13:57 |
| jroll | let's put ideas there :) | 13:57 |
| dtantsur | rloo, "ask" does not scale ;) I suspect we have to stop mocking and start actually installing stuff | 13:57 |
| *** lifeless has quit IRC | 13:57 | |
| dtantsur | jroll, +1 on this RFE (which I'd call a bug rather than RFE) | 13:57 |
| rloo | dtantsur: that would only work if we can install stuff in gate. | 13:57 |
| jroll | we can! | 13:58 |
| *** r-daneel has joined #openstack-ironic | 13:58 | |
| dtantsur | rloo, just add driver-requirements.txt to tox.ini. but this has very dangerous consequences | 13:58 |
| jroll | pip install -r driver-requirements.txt | 13:58 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch error when wrong temp_url were generated https://review.openstack.org/370875 | 13:58 |
| jroll | right | 13:58 |
| jroll | I think we need a new target, with a separate job | 13:58 |
| lucasagomes | not all dependencies may be there tho | 13:58 |
| JayF | dtantsur: what are the dangerous consequences? | 13:58 |
| dtantsur | not all drivers follow global-requirements and upper-constraints | 13:58 |
| lucasagomes | like iboot is not on pypi | 13:58 |
| jroll | otherwise we can accidentally hard-depend on them | 13:58 |
| jroll | and that | 13:58 |
| lucasagomes | tho it will be removed later... | 13:58 |
| dtantsur | lucasagomes, iboot goes away ;) | 13:58 |
| vdrok | yeah, I'm not sure about the "real" drivers, but iirc one of the binary iboot dependencies stated that it works only on 14.04 ubuntu | 13:58 |
| lucasagomes | dtantsur, right but it might happen in the future for other drivers | 13:59 |
| jroll | but we could land something that hard fails if e.g. proliantutils isn't installed | 13:59 |
| lucasagomes | just an alert | 13:59 |
| *** sdake has joined #openstack-ironic | 13:59 | |
| dtantsur | jroll, ++ for a separate target, maybe even running only tests from drivers/modules | 13:59 |
| *** lifeless has joined #openstack-ironic | 13:59 | |
| *** joprovost1 has joined #openstack-ironic | 13:59 | |
| *** causten_ has joined #openstack-ironic | 13:59 | |
| rloo | unit tests are fast, might as well run them all. | 13:59 |
| jroll | dtantsur: maybe, I'd be fine with all tests, they're fast | 13:59 |
| *** sdake has quit IRC | 13:59 | |
| *** joprovost has quit IRC | 13:59 | |
| *** joprovost1 is now known as joprovost | 13:59 | |
| dtantsur | lucasagomes, fair, I'll start -1ing specs for drivers that are not on PyPI ;) problem solved! | 13:59 |
| *** sdake has joined #openstack-ironic | 14:00 | |
| JayF | I mean, if someone is running 3rd party ci (required to be in tree), any libs they use would have to be in pypi or they're solving weird/hard local dep stuff | 14:00 |
| vdrok | maybe let's instead run the unittests in third party ci then? | 14:01 |
| dtantsur | so, if it's not on PyPI, we can mock their modules *unconditionally* | 14:01 |
| dtantsur | then we won't have such bugs as one we're fixing now | 14:01 |
| vdrok | not having a separate job in gate, but only separate tox target to be used by CIs | 14:02 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Improve error message while download image https://review.openstack.org/370824 | 14:02 |
| lucasagomes | vdrok, ++ that seems like a good approach | 14:02 |
| *** causten has quit IRC | 14:03 | |
| dtantsur | mmm, this is strange http://logs.openstack.org/61/374161/1/check/gate-ironic-python27-db-ubuntu-xenial/cbf3772/console.html#_2016-09-21_14_01_56_477476 | 14:03 |
| vdrok | ooo, dtantsur that was in a recent nova ML thread | 14:04 |
| dtantsur | ah, so it's a known problem, right? | 14:04 |
| jroll | O_o | 14:05 |
| vdrok | dtantsur: title is opportunistic tests breaking randomly | 14:05 |
| vdrok | they had such problems with nova migrations | 14:05 |
| *** ohamada has joined #openstack-ironic | 14:05 | |
| dtantsur | ah, I see | 14:05 |
| vdrok | and I think the solution was to increase some timeout | 14:05 |
| *** jrist has joined #openstack-ironic | 14:07 | |
| rloo | I updated that RFE (1626106) with people's suggestions and a link to this irc conversation. feel free to update/take ownership :) | 14:07 |
| jroll | thanks rloo :) | 14:07 |
| *** pester has joined #openstack-ironic | 14:08 | |
| lucasagomes | vdrok, dtantsur https://review.openstack.org/#/c/291397/ | 14:08 |
| patchbot | patch 291397 - manila - Fix unstable DB migration tests (MERGED) | 14:08 |
| *** vmud213 has quit IRC | 14:08 | |
| lucasagomes | apparently that's how manila worked around that problem | 14:08 |
| *** rbrud has joined #openstack-ironic | 14:08 | |
| rloo | lucasagomes, dtantsur, vdrok: is that something that has been failing randomly for awhile, or just noticed now? (for dtantsur maybe just now). we should probably fix for newton (or backport to newton) | 14:09 |
| lucasagomes | rloo, first time I've seem it | 14:09 |
| vdrok | rloo: first time I see it in our gate | 14:10 |
| rloo | lucasagomes: that manila patch -- commit msg is useful. | 14:10 |
| lucasagomes | rloo, yup, see the bug report as well | 14:10 |
| lucasagomes | rloo, https://bugs.launchpad.net/manila/+bug/1501272 | 14:10 |
| openstack | Launchpad bug 1501272 in Manila "DB walk versions unit test is unstable" [High,Fix released] - Assigned to Valeriy Ponomaryov (vponomaryov) | 14:10 |
| *** rama_y has joined #openstack-ironic | 14:11 | |
| *** rpioso has joined #openstack-ironic | 14:11 | |
| rloo | lucasagomes: interesting. they saw it a year ago! | 14:11 |
| *** fxpester has quit IRC | 14:12 | |
| *** narasimha_SV has joined #openstack-ironic | 14:14 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Add vendor, product to interface information https://review.openstack.org/279940 | 14:14 |
| jroll | dtantsur: those IPA +2s you had are now approved | 14:15 |
| *** hj-hpe has joined #openstack-ironic | 14:15 | |
| jroll | zigo: did that patch fix things for you? | 14:15 |
| zigo | jroll: It's currently building at: telnet 169.45.95.178 19885 | 14:16 |
| zigo | :) | 14:16 |
| jroll | :D | 14:16 |
| jroll | thanks | 14:16 |
| rloo | jroll: besides that patch to fix ilo unit tests, and release notes, what else is there? | 14:17 |
| jroll | rloo: I'm still poking around but that might be it | 14:18 |
| jroll | waiting for this to land https://review.openstack.org/#/c/374142/ | 14:19 |
| patchbot | patch 374142 - ironic - Sync ironic-lib.filters from ironic-lib | 14:19 |
| rloo | jroll: oh, did your security doc get merged? | 14:19 |
| rloo | jroll: and ugh, the resource_class thing. | 14:19 |
| jroll | oh, forgot about my own things :) | 14:19 |
| dtantsur | jroll, rloo, I'd love https://review.openstack.org/246033 but it needs updating.. | 14:20 |
| patchbot | patch 246033 - ironic - Conductor fails to do take_over for PXE drivers | 14:20 |
| jroll | rloo: yeah, you're right, https://review.openstack.org/#/c/345081/ and https://review.openstack.org/#/c/369385/ | 14:20 |
| patchbot | patch 345081 - ironic - Add docs for node.resource_class and flavor creation | 14:20 |
| patchbot | patch 369385 - ironic - Add a note about security groups in install guide | 14:20 |
| jroll | dtantsur: yeah, agree | 14:21 |
| dtantsur | lucasagomes, are you still planning to update https://review.openstack.org/246033 today? | 14:21 |
| patchbot | patch 246033 - ironic - Conductor fails to do take_over for PXE drivers | 14:21 |
| rloo | dtantsur: is that 246033 a candidate to backport? or does someone want to update it? | 14:21 |
| dtantsur | rloo, lucasagomes was working on it | 14:21 |
| lucasagomes | dtantsur, it's on my list for today but I was too deep into the uefi thingy | 14:22 |
| lucasagomes | so I didn't want to stop | 14:22 |
| lucasagomes | lemme take a look at this patch | 14:22 |
| jroll | on the proliantutils patch, 2016-09-21 14:22:26.946970 | [Zuul] Job complete, result: SUCCESS | 14:22 |
| * lucasagomes hopes for an easy rebase | 14:22 | |
| jroll | wooo | 14:22 |
| dtantsur | mgould, mind updating https://review.openstack.org/367451 please? it's probably too late for this release, but still | 14:23 |
| patchbot | patch 367451 - ironic - DRAC: list unfinished jobs | 14:23 |
| lucasagomes | :-/ not-so-easy rebase ahead | 14:23 |
| dtantsur | jroll, unrelated thing I wanted to talk to you about. We probably need to run moar jobs on ironic-lib, now that we've moved most of the image-related logic there.. | 14:24 |
| dtantsur | jroll, like, at least run one partition and one whole disk job | 14:24 |
| rloo | mariojv: if you have a minute or 2, could you check that jroll addressed your concerns? https://review.openstack.org/#/c/369385/ | 14:24 |
| vdrok | ugh, it seems gerrit does not feel good | 14:24 |
| patchbot | patch 369385 - ironic - Add a note about security groups in install guide | 14:25 |
| dtantsur | vdrok, well, why would it feel good on the release day? | 14:25 |
| jroll | dtantsur: +1 | 14:25 |
| vdrok | button pressed :) | 14:25 |
| *** rbudden has quit IRC | 14:29 | |
| mariojv | rloo: sure | 14:30 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add docs for node.resource_class and flavor creation https://review.openstack.org/345081 | 14:30 |
| jroll | rloo: ^ let's see if that is more what you expected | 14:30 |
| jroll | er, that ^ | 14:30 |
| rloo | jroll: i was just reviewing that again. will look at the updated one. | 14:31 |
| mariojv | rloo: jroll: yup, LGTM. thanks! | 14:31 |
| rloo | thx mariojv! | 14:31 |
| *** moshele has quit IRC | 14:33 | |
| openstackgerrit | Merged openstack/ironic: Fix broken unit tests for get_ilo_object https://review.openstack.org/374161 | 14:34 |
| *** jheroux has joined #openstack-ironic | 14:35 | |
| *** rbudden has joined #openstack-ironic | 14:38 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Local boot with partition images" https://review.openstack.org/369776 | 14:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Flavor creation" https://review.openstack.org/369761 | 14:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Enrollment" and "Troubleshooting" sections https://review.openstack.org/369777 | 14:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Add link to new guide in old install guide https://review.openstack.org/369802 | 14:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import Install and configure sections https://review.openstack.org/367988 | 14:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Building or downloading a deploy ramdisk image" https://review.openstack.org/369800 | 14:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Trusted boot with partition image" https://review.openstack.org/369801 | 14:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Bare Metal service overview" https://review.openstack.org/366329 | 14:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import configdrive https://review.openstack.org/369798 | 14:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Image requirements" https://review.openstack.org/369754 | 14:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Appending kernel parameters to boot instances" https://review.openstack.org/369799 | 14:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "integration with other OpenStack components" https://review.openstack.org/369510 | 14:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import HTTPS, standalone and root device hints https://review.openstack.org/369797 | 14:40 |
| *** daemontool_ has joined #openstack-ironic | 14:41 | |
| *** charlledaniel_ has quit IRC | 14:42 | |
| *** cdaniel has quit IRC | 14:42 | |
| rloo | mat128: wrt the install guide patches. I suggest *not* making any changes other than the ones to migrate from old to new. otherwise, you'll be doing revisions forever. | 14:42 |
| mat128 | rloo: not touching it anymore | 14:43 |
| rloo | mat128: :) | 14:43 |
| mat128 | Vlad had a good point as I had fixed some of the things but not the other ones | 14:43 |
| *** syed_ has joined #openstack-ironic | 14:43 | |
| mat128 | leaving the changes as it, unless there is something major, that's how it should be merged | 14:43 |
| rloo | mat128: yeah, I'm not saying he didn't, just got to be careful cuz documentation is one of those things where it is easy to improve on... | 14:44 |
| *** daemontool has quit IRC | 14:44 | |
| JayF | mat128: I think the end result of the previous convo was that this patch chain is getting a -2 from jroll until post-newton (which I think happens today) | 14:44 |
| JayF | mat128: :( but at least we'll have a good install guide for ocata | 14:45 |
| mat128 | JayF: I asked for this decision to be revisited and rloo said she would be reviewing it in a timebox | 14:45 |
| mat128 | I don't want to maintain rebasing this for a week and not having an install guide in newton. If one looks at the rendered result of the last patch, it looks pretty good | 14:45 |
| openstackgerrit | Miles Gould proposed openstack/ironic: DRAC: list unfinished jobs https://review.openstack.org/367451 | 14:47 |
| jroll | rloo: I give up on resource class docs, I'm fairly confident in the plan and the future execution but we can just wait | 14:51 |
| openstackgerrit | Merged openstack/ironic-python-agent: Enable SSH access to tinyipa https://review.openstack.org/371775 | 14:51 |
| rloo | jroll: fine by me. i am fine also +2'ing yours, but i wanted to voice my opinion. | 14:51 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 14:51 |
| lucasagomes | dtantsur, re rebasing #246033, apparently a lot has changed i will need to re-test it after the rebase | 14:52 |
| jroll | rloo: yeah | 14:52 |
| dtantsur | sigh.. | 14:52 |
| dtantsur | lucasagomes, ok. then do you think it's backportable? | 14:52 |
| jroll | heh, was just going to ask that | 14:53 |
| *** baoli has joined #openstack-ironic | 14:54 | |
| lucasagomes | dtantsur, jroll it should be | 14:54 |
| dtantsur | then I'd vote for giving it moar testing (I can revive my tripleo HA environment for that) and then backporting | 14:54 |
| mat128 | pas-ha: https://review.openstack.org/#/c/371775/4/imagebuild/tinyipa/finalise-tinyipa.sh does that mean it's going to use my build machine's public key? | 14:54 |
| patchbot | patch 371775 - ironic-python-agent - Enable SSH access to tinyipa (MERGED) | 14:54 |
| pas-ha | mat128: yes if you do not supply another one | 14:55 |
| mat128 | pas-ha: and SSH_PUBLIC_KEY should be the path to a key, not the key itself, correct? | 14:55 |
| pas-ha | yes, full path | 14:55 |
| mat128 | pas-ha: ok, thanks :) | 14:56 |
| mat128 | pas-ha: ah.. I should have read the README first | 14:56 |
| mat128 | pas-ha: everything was in there :) | 14:56 |
| mgould | dtantsur: done | 14:57 |
| openstackgerrit | Vadim Hmyrov proposed openstack/ironic-python-agent: Add bash scripts style checking for IPA https://review.openstack.org/374147 | 14:57 |
| dtantsur | mgould, cheers! | 14:57 |
| JayF | rloo: how is that grammatically incorrect? | 14:58 |
| jroll | rloo: hm, the final rendering is looking good, I'm almost inclined to just merge it all | 14:58 |
| *** daemontool_ has quit IRC | 14:58 | |
| zigo | jroll: dtantsur: Just making sure you got the message: the patch fixed the issue for me! :) | 14:58 |
| rloo | JayF: am in a meeting. it was something like the Bare Metal service Install Guide. | 14:59 |
| jroll | zigo: yep, I saw the logs, thanks! | 14:59 |
| dtantsur | zigo, thanks for confirming! | 14:59 |
| *** openstack has joined #openstack-ironic | 15:02 | |
| jroll | so I'm +2 on the whole chain | 15:02 |
| jroll | +A on the ones that already have a +2 | 15:02 |
| jroll | and if I can get another +2 on the rest, and rloo's agreement, I'm ready to land the whole thing | 15:03 |
| rloo | jroll: in a meeting for an hour. do you want me to remove my -2? can't look now. | 15:03 |
| jroll | rloo: if you're okay with landing it all. I can wait til after your meeting. | 15:04 |
| JayF | I'll be through my reviews and have +2s on the rest, barring issues, very shortly | 15:04 |
| rloo | ok, later... | 15:04 |
| *** wajdi has joined #openstack-ironic | 15:05 | |
| mandre | what are tgz in /var/log/ironic/deploy/ ? | 15:08 |
| * mandre shows my ironic ignorance | 15:08 | |
| JayF | mandre: http://docs.openstack.org/developer/ironic/deploy/troubleshooting.html#retrieving-logs-from-the-deploy-ramdisk should help | 15:09 |
| mandre | thanks JayF :) | 15:09 |
| zigo | There we go, the first Ironic Debian package built on OpenStack infra is there: http://mirror.dfw.rax.openstack.org/debian-openstack/pool/main/i/ironic/ :) | 15:12 |
| zigo | \o/ | 15:12 |
| dtantsur | \o/ | 15:12 |
| zigo | If you guys see inconsistency in packages, we also do accept patches, FYI ... :P | 15:12 |
| zigo | Anyone up for testing these new debs? | 15:13 |
| jroll | \o/ | 15:14 |
| jroll | maybe, but for now food is calling | 15:14 |
| zigo | Yeah, same over here... | 15:14 |
| *** e0ne has quit IRC | 15:28 | |
| *** wznoinsk has quit IRC | 15:28 | |
| *** rama_y has quit IRC | 15:31 | |
| *** aNuposic has joined #openstack-ironic | 15:32 | |
| *** e0ne has joined #openstack-ironic | 15:33 | |
| *** lucasagomes is now known as lucas-hungry | 15:35 | |
| *** jaosorior has quit IRC | 15:39 | |
| rloo | JayF, jroll, mat128|mtg: so you guys are OK with the old install guide page looking like: http://docs-draft.openstack.org/02/369802/12/check/gate-ironic-docs-ubuntu-xenial/072a288//doc/build/html/deploy/install-guide.html ? | 15:42 |
| rloo | JayF, jroll, mat128|mtg: there is still content at the bottom of that page :-( | 15:42 |
| openstackgerrit | Merged openstack/ironic-python-agent: Add vendor, product to interface information https://review.openstack.org/279940 | 15:43 |
| rloo | JayF, jroll, mat128|mtg: and that content is 'below the fold' of my browser; would someone know to scroll down there...?? | 15:43 |
| dtantsur | it's not even in the bottom, it's in the middle | 15:44 |
| dtantsur | PXE setup, boot mode support and IPMI support | 15:44 |
| dtantsur | is "install-guide" really an "admin-guide"? cause we're also moving troubleshooting there... | 15:45 |
| *** bks has joined #openstack-ironic | 15:45 | |
| rloo | dtantsur: i'm not comfortable with that install guide page. seems like it'll be a worse experience for the user, than having it all there for now. | 15:46 |
| rloo | dtantsur: I didn't even scroll to the bottom of the page. good for you :) | 15:47 |
| *** ccamacho is now known as ccamacho|afk | 15:47 | |
| *** rama_y has joined #openstack-ironic | 15:47 | |
| *** d0ugal has quit IRC | 15:48 | |
| *** amotoki has quit IRC | 15:49 | |
| *** MattMan has quit IRC | 15:49 | |
| *** d0ugal has joined #openstack-ironic | 15:49 | |
| *** MattMan has joined #openstack-ironic | 15:49 | |
| *** kumarb has joined #openstack-ironic | 15:51 | |
| *** rbudden has quit IRC | 15:51 | |
| *** aNuposic has quit IRC | 15:52 | |
| *** bks has quit IRC | 15:52 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 15:54 |
| *** joprovost1 has joined #openstack-ironic | 15:57 | |
| openstackgerrit | Merged openstack/ironic: Sync ironic-lib.filters from ironic-lib https://review.openstack.org/374142 | 15:58 |
| *** rajinir has joined #openstack-ironic | 15:58 | |
| *** joprovost has quit IRC | 15:58 | |
| *** joprovost1 is now known as joprovost | 15:58 | |
| *** anush has joined #openstack-ironic | 15:58 | |
| openstackgerrit | Merged openstack/ironic: Add a note about security groups in install guide https://review.openstack.org/369385 | 16:00 |
| *** moshele has joined #openstack-ironic | 16:03 | |
| *** rcernin has quit IRC | 16:03 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Have bashate run for entire project https://review.openstack.org/374269 | 16:05 |
| *** mat128|mtg is now known as mat128 | 16:05 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/ironic-python-agent: Add bash scripts style checking for IPA https://review.openstack.org/374147 | 16:08 |
| *** baoli has quit IRC | 16:08 | |
| *** BadCub has joined #openstack-ironic | 16:09 | |
| *** baoli has joined #openstack-ironic | 16:09 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Refactor env-setup.sh https://review.openstack.org/371883 | 16:11 |
| *** wznoinsk has joined #openstack-ironic | 16:15 | |
| *** pcaruana has quit IRC | 16:15 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Setup the drivers for the Bare Metal service" https://review.openstack.org/374272 | 16:16 |
| mat128 | rloo, JayF, jroll, dtantsur ^ | 16:17 |
| mat128 | missing section imported as-is | 16:17 |
| *** pcaruana has joined #openstack-ironic | 16:17 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Setup the drivers for the Bare Metal service" https://review.openstack.org/374272 | 16:18 |
| *** aNuposic has joined #openstack-ironic | 16:18 | |
| *** moshele has quit IRC | 16:19 | |
| *** devananda|dinner is now known as devananda | 16:23 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Docs: Add warning and add setting of $LOGFILE https://review.openstack.org/366518 | 16:24 |
| *** BadCub has quit IRC | 16:25 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Docs: Add warning and add setting of $LOGFILE https://review.openstack.org/366518 | 16:26 |
| *** BadCub has joined #openstack-ironic | 16:26 | |
| *** trown is now known as trown|lunch | 16:27 | |
| *** lucas-hungry is now known as lucasagomes | 16:29 | |
| *** rama_y has quit IRC | 16:30 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: [WIP] Test ML2 Fail-Fast feature https://review.openstack.org/370016 | 16:30 |
| NobodyCam | Good Morning Ironic'ers :) | 16:31 |
| *** krtaylor has quit IRC | 16:31 | |
| dtantsur | morning NobodyCam | 16:31 |
| vdrok | good morning NobodyCam | 16:31 |
| NobodyCam | morning vdrok dtantsur :) | 16:31 |
| TheJulia | o/ NobodyCam | 16:33 |
| NobodyCam | o/ .... morning TheJulia :) | 16:33 |
| jlvillal | Good morning NobodyCam TheJulia dtantsur vdrok mat128 rloo jroll JayF pas-ha and everyone else | 16:37 |
| dtantsur | morning jlvillal | 16:37 |
| mat128 | o/ jlvillal, NobodyCam | 16:37 |
| NobodyCam | morning jlvillal & mat128 :) | 16:37 |
| *** hshiina has quit IRC | 16:38 | |
| rloo | morning jlvillal, NobodyCam | 16:39 |
| NobodyCam | morning rloo :) | 16:39 |
| *** BadCub has quit IRC | 16:39 | |
| dtantsur | see you tomorrow, I hope you don't need me for the release | 16:42 |
| * dtantsur has approved one drac patch, would be cool to get it in | 16:42 | |
| NobodyCam | night dtantsur | 16:42 |
| NobodyCam | : | 16:43 |
| NobodyCam | ) | 16:43 |
| lucasagomes | NobodyCam, jlvillal morning | 16:43 |
| *** dtantsur is now known as dtantsur|afk | 16:43 | |
| NobodyCam | morning lucasagomes :) | 16:43 |
| rloo | mat128: thx for that last patch. What I would like to see is that the "Installation Guide" link at http://docs-draft.openstack.org/72/374272/2/check/gate-ironic-docs-ubuntu-xenial/2198541//doc/build/html/, points to the new installation guide. | 16:44 |
| mat128 | rloo: ah good idea, so everyone who has a bookmark ends up on the old page, and if you go through the front door you're directed to the right place | 16:45 |
| rloo | mat128: looking at the old installation guide page, i see one thing that doesn't say 'move to new install': http://docs-draft.openstack.org/72/374272/2/check/gate-ironic-docs-ubuntu-xenial/2198541//doc/build/html/deploy/install-guide.html#configuring-tenant-networks | 16:45 |
| mat128 | because thats a section in the dev docs | 16:45 |
| mat128 | which wasnt part of the install guide | 16:45 |
| *** BadCub has joined #openstack-ironic | 16:45 | |
| mat128 | in the new docs, I put a direct link to it | 16:45 |
| rloo | mat128: right. | 16:45 |
| rloo | mat128: so nothing points to that 'Configuring Tenant Networks' section? | 16:45 |
| jlvillal | Hey lucasagomes, sorry I missed seeing you the first time | 16:46 |
| mat128 | rloo: besides bookmarks, I dont think so | 16:46 |
| rloo | mat128: oh, let me look at the new install, haven't looked at that yet. | 16:46 |
| jlvillal | lucasagomes: Regarding your comment on: https://review.openstack.org/#/c/369617/ | 16:46 |
| patchbot | patch 369617 - ironic - Add config option to specify maximum size for conf... | 16:46 |
| lucasagomes | jlvillal, no worries I was having lunch :-) | 16:46 |
| vdrok | morning jlvillal | 16:46 |
| *** rbudden has joined #openstack-ironic | 16:47 | |
| jlvillal | vdrok: :) | 16:47 |
| mat128 | rloo: rendering docs to check if "Installation guide" link is fine | 16:47 |
| mat128 | then im pushing that update | 16:47 |
| jlvillal | lucasagomes: Any idea how much work it will take to remove storing the configdrive in the database? | 16:47 |
| lucasagomes | jlvillal, so I was waiting for cinerama, if he/she agrees I would glad help with it | 16:47 |
| lucasagomes | jlvillal, we need to change the nova driver, usually nova changes takes some time | 16:48 |
| *** jpich has quit IRC | 16:48 | |
| lucasagomes | the ironic bits shouldn't take much longer afaict | 16:48 |
| cinerama | hi there | 16:48 |
| cinerama | i haven't looked at that bug in a while | 16:48 |
| jlvillal | lucasagomes: How will that impact CLI usage? | 16:48 |
| lucasagomes | cinerama, hello, you are assigned to fix that configdrive change. Are you planning to work on it soon ? | 16:49 |
| jlvillal | lucasagomes: I'm guessing they would need to specify the configdrive when they go to provision. | 16:49 |
| lucasagomes | jlvillal, yeah it already happens for "active" but needs to be extended to "rebuild" as well | 16:49 |
| lucasagomes | lemme take a look at the CLI | 16:49 |
| jlvillal | lucasagomes: Okay. Thanks. | 16:49 |
| jroll | mat128: rloo: thanks for the changes / review / discussion | 16:50 |
| rloo | jroll: yw. | 16:50 |
| lucasagomes | jlvillal, I think the main thing I'm kinda against in your patch is the inclusion of a config option | 16:50 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Setup the drivers for the Bare Metal service" https://review.openstack.org/374272 | 16:50 |
| jlvillal | lucasagomes: That is the whole patch :P | 16:50 |
| jlvillal | heh | 16:50 |
| mat128 | rloo: updated with the direct link^ | 16:50 |
| lucasagomes | jlvillal, heh can't we do it without the config ? Perhaps just checking for 16MB for all config drivers ? | 16:50 |
| mat128 | jroll: thanks to you :) | 16:51 |
| rloo | jroll: i am looking now at the latest version of the new install guide. clearly, there is no time for me to think about a reorg of the content. assuming we are ok with the migration as-is, are we going to be ok later, if we reorg the content? | 16:51 |
| lucasagomes | jlvillal, another thing about ^ is that it's incosistent with nova, which doesn't limit the size | 16:51 |
| jlvillal | lucasagomes: The only reason I did that was because of the patch it is based on. So that operators aren't surprised if their database becomes HUGE | 16:51 |
| jlvillal | lucasagomes: We are currently limited at ~ 64KiB | 16:51 |
| jroll | rloo: anything specifically that would make the re-orging later not okay? | 16:51 |
| lucasagomes | jlvillal, not anymore right? (didn't we update the db to support longer values ? ) | 16:51 |
| jlvillal | lucasagomes: No. That patch has not landed. | 16:52 |
| lucasagomes | right | 16:52 |
| jlvillal | As far as I know | 16:52 |
| rloo | jroll: dunno, depends on the reorg. would/might have pointers/links in existing (so cannot remove existing sections) to say content has move to new location. | 16:52 |
| lucasagomes | yeah would be good to have that | 16:52 |
| *** e0ne has quit IRC | 16:52 | |
| lucasagomes | and apparently only affects mariadb/mysql | 16:52 |
| jroll | rloo: the docs site is able to support redirects, instead of keeping the pages | 16:52 |
| rloo | jroll: ok. let me continue looking then... | 16:53 |
| jlvillal | lucasagomes: Right. I'm fine with not having the config option. I was just thinking it could surprise operators if their DB became large. | 16:53 |
| lucasagomes | jlvillal, right, yeah maybe we should bump the priority of the proper fix then | 16:54 |
| lucasagomes | and get it done | 16:54 |
| jlvillal | lucasagomes: There were some concerns voiced, when the DB field was proposed to be 4GiB, about the database getting very large. Thus my idea for the config option. | 16:54 |
| lucasagomes | jlvillal, the proper fix would fix swift as well | 16:54 |
| lucasagomes | swift + rebuild currently does not work | 16:55 |
| lucasagomes | AFAICT | 16:55 |
| lucasagomes | swfit to store the configdrive I mean | 16:55 |
| *** BadCub has quit IRC | 16:55 | |
| jlvillal | lucasagomes: I remember a patch to fix that. not sure if it landed or not. | 16:55 |
| lucasagomes | jlvillal, 4G ? It's a bit steep heh | 16:55 |
| lucasagomes | from 64kb to 4G, tho, TEXT fields for some dbs are unlimited | 16:56 |
| *** rbartal has quit IRC | 16:56 | |
| jlvillal | lucasagomes: Yeah. I proposed doing the MEDIUMTEXT for 16 MiB. Which seemed more reasonable :) | 16:56 |
| lucasagomes | jlvillal, ++ def | 16:56 |
| jlvillal | lucasagomes: Currently it is proposed for 16MiB. Though PostgreSQL is already unlimited. | 16:56 |
| lucasagomes | 16MiB is reasonable I think | 16:57 |
| jlvillal | lucasagomes: I think so too. But 64KiB has given at least a few customers problems. | 16:57 |
| jlvillal | lucasagomes: FYI: The DB patch: https://review.openstack.org/334967 | 16:57 |
| patchbot | patch 334967 - ironic - Fix non-deployed nodes due to short nodes.instance... | 16:57 |
| jroll | mat128: one thing on that last patch | 16:58 |
| jlvillal | lucasagomes: Not something I would want to land until after Newton though. | 16:58 |
| lucasagomes | jlvillal, indeed 64kb seems way too short | 16:58 |
| mat128 | jroll: sure? | 16:58 |
| lucasagomes | thanks will add to the list to look at | 16:58 |
| jroll | mat128: in the comments :) | 16:58 |
| mat128 | ahh | 16:58 |
| xavierr | do we have QA meeting today? | 16:58 |
| jlvillal | lucasagomes: Or the other plan. To get rid of storing it all together. I don't understand all the ramifications of that. | 16:58 |
| lucasagomes | jlvillal, that's the plan for the right fix | 16:59 |
| lucasagomes | we don't need to store the config drive at all | 16:59 |
| *** BadCub has joined #openstack-ironic | 16:59 | |
| JayF | rloo: jroll mat128: re: "what goes in install guide", it's a hard question. The admin guide is still centralized. I'm going to try to work with docs to get admin-guide setup per-project as well, and when that happens, I think we should migrate more of the "admin" information there. Until then, I think devdocs and install guide are equally "wrong", so I don't care which it goes in :) | 16:59 |
| *** derekh has quit IRC | 16:59 | |
| lucasagomes | we only store so that we can rebuild the image but, the ironic nova driver should be regenerating the configdrive at rebuild time as well | 16:59 |
| rloo | JayF: from user's point of view is all i'm concerned with right now. | 17:00 |
| lucasagomes | so no need to store it at all, and storing may even be considered a potential security flaw. So there's a lot of gains if we fix it properly | 17:00 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Setup the drivers for the Bare Metal service" https://review.openstack.org/374272 | 17:01 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Update developer's guide "Installation Guide" link https://review.openstack.org/374296 | 17:01 |
| mat128 | jroll: fixed | 17:01 |
| lucasagomes | off-topic, cirros images with uefi support: <smoser> lucasagomes, frickler is wanting one soon, and i'm generally on board with that. | 17:01 |
| lucasagomes | o/ | 17:01 |
| mat128 | lucasagomes: nice! | 17:01 |
| JayF | rloo: I think both are equal from that perspective, but getting a real ironic "admin-guide" up should be a docs priority for o/p if we can get the docs team (I will try to get permission to spend some time on it) to do per-project-admin-guide | 17:01 |
| lucasagomes | soon to be released :-) | 17:01 |
| *** ohamada has quit IRC | 17:01 | |
| mat128 | JayF: per-project everything! | 17:02 |
| mat128 | :D | 17:02 |
| JayF | mat128: +++ | 17:02 |
| rloo | mat128: sorry, i commented in patch set 3 of 374272 | 17:02 |
| rloo | mat128: another question. was going to look into it, are there other pages in developer/... that point to the (old) install guide? | 17:02 |
| mat128 | rloo: all the sections were left in place so to not break any of those links | 17:03 |
| mat128 | so the anchors still work | 17:03 |
| mat128 | we will have to discuss how to remove that page | 17:03 |
| mat128 | but I suspect it will have to stay for 1 release, like everything else | 17:03 |
| rloo | mat128: yeah, i realize the anchors will work, but it is ugly. seems like we should update all *our* links to point to the new install guide | 17:03 |
| mat128 | let me grep :) | 17:04 |
| rloo | mat128: but if we do that, it means we'll have to update all those links in stable/newton, to point to the new install guide in ../newton/ instead of ../draft/ | 17:04 |
| *** krtaylor has joined #openstack-ironic | 17:04 | |
| rloo | mat128: i don't know how many links/pages are affected | 17:04 |
| mat128 | I dont think we did that for stable branches before | 17:04 |
| rloo | mat128: before, they all pointed to the 'same' thing. now, we have developer/docs pointing to installguide docs (if we change the links) | 17:05 |
| rloo | mat128: did that make sense? | 17:05 |
| mat128 | rloo: links using URL should have been updated for branches, but I dont think they were | 17:05 |
| *** caiobo has joined #openstack-ironic | 17:05 | |
| rloo | mat128: i'll start again. sec, letm e find an example. | 17:05 |
| mat128 | rloo: https://github.com/openstack/ironic/blame/stable/mitaka/doc/source/drivers/oneview.rst#L230 | 17:06 |
| *** jcoufal has quit IRC | 17:06 | |
| mat128 | source file in mitaka, pointing to "master" docs | 17:06 |
| mat128 | in theory, should have been updated | 17:06 |
| *** rama_y has joined #openstack-ironic | 17:06 | |
| rloo | mat128: ugh. i thought we were using relative links, not absolute | 17:06 |
| mat128 | rloo: the right approach is ":ref:`install-guide`", but that will point to the old page that contains nothing but links | 17:06 |
| mat128 | rloo: yeah, some absolute links slipped in | 17:07 |
| mat128 | rloo: http://paste.openstack.org/show/582432/ | 17:07 |
| *** dprince has quit IRC | 17:07 | |
| mat128 | rloo: I can do a new commit to point them to the new absolute link | 17:08 |
| *** kumarb has quit IRC | 17:09 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 17:09 |
| *** charlledaniel has joined #openstack-ironic | 17:09 | |
| rloo | mat128: so if we change those 4 links to new install/draft/, then in stable/newton, change them to new install/newton/ ? | 17:10 |
| mat128 | rloo: thats correct, we can do a mass sed or find and replace to fix the links | 17:10 |
| mat128 | it will cover links from install-guide.rst and those files too | 17:10 |
| rloo | mat128: i think that would work. | 17:10 |
| mat128 | I'll do it once we branch | 17:10 |
| rloo | mat128: right. | 17:11 |
| rloo | mat128: i will continue 'reviewing'... | 17:11 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Fix absolute links to install-guide.rst in developer docs https://review.openstack.org/374303 | 17:13 |
| mat128 | rloo: ^ done | 17:13 |
| rloo | mat128: thx | 17:13 |
| JayF | If there's a doc for what we do on release, we might wanna add the "fix devdoc links" task to that doc | 17:16 |
| mat128 | JayF: do you mean we should add release notes to my last commit? | 17:17 |
| JayF | mat128: I mean, IDK if there's a "how to release ironic" doc jroll follows or something, but if there is, we should add a "post-release, fix the links in devdocs" | 17:18 |
| JayF | mat128: release notes are user facing; I'm talking about something contributor-facing | 17:18 |
| jroll | there is not because I'm a terrible person >.> | 17:18 |
| *** narasimha_SV has quit IRC | 17:19 | |
| rloo | JayF: i think there is a list of things todo when a branch is cut. and jroll is the keeper of that list. You should drag it out of him :) | 17:19 |
| jroll | rloo: it's in my head and I'm in the same room as him. now I have to run away and/or hide the knives. :| | 17:20 |
| JayF | forks. | 17:20 |
| JayF | I have forks nearby, not knifes. | 17:20 |
| rloo | jroll: I have confidence in JayF; he doesn't need knives! | 17:20 |
| openstackgerrit | Merged openstack/python-ironicclient: Use ConfigParser instead of SafeConfigParser in Python 3 https://review.openstack.org/369838 | 17:21 |
| rloo | mat128: just commenting to you cuz i have no idea now which patch it is in, and maybe it is cuz we have so much stuff: look way down this page: http://docs-draft.openstack.org/72/374272/3/check/gate-ironic-install-guide/162ae00//install-guide/build/html/. | 17:22 |
| rloo | mat128: it is probably ok for now. | 17:22 |
| rloo | mat128: but i dunno who will ever read that text at the bottom :) | 17:22 |
| *** Romanenko_K has quit IRC | 17:25 | |
| *** athomas has quit IRC | 17:27 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Allow user to get iPXE files https://review.openstack.org/367635 | 17:27 |
| *** livelace has joined #openstack-ironic | 17:29 | |
| *** trown|lunch is now known as trown | 17:30 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Refactor env-setup.sh https://review.openstack.org/371883 | 17:32 |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Workaround openSUSE ansible_os_family issue https://review.openstack.org/371082 | 17:33 |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Install process SuSE support https://review.openstack.org/367637 | 17:33 |
| openstackgerrit | Merged openstack/ironic: DRAC: list unfinished jobs https://review.openstack.org/367451 | 17:33 |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: SuSE support for VM node creation https://review.openstack.org/370385 | 17:33 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 17:34 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 17:34 |
| mat128 | rloo: thats the text in index.rst | 17:34 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 17:34 |
| mat128 | below the table of contents | 17:34 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_targets table to save target information https://review.openstack.org/285218 | 17:34 |
| rloo | mat128: does it look odd to you in the rendered page? | 17:35 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce VolumeTarget object https://review.openstack.org/285219 | 17:35 |
| rloo | mat128: maybe it should be on top. dunno. | 17:35 |
| mat128 | our TOC is much longer than the text, which makes it look a bit weird | 17:35 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume target operations https://review.openstack.org/285220 | 17:35 |
| mat128 | rloo: all other pages have TOC at the top, so I'm hesitant to do it differently for this one | 17:35 |
| rloo | mat128: i have another question now. 'Configure the Image service'. I think of 'configure' as in eg modifying configuration options. | 17:35 |
| rloo | mat128: that section describes how to get the desired images into the image service. | 17:36 |
| rloo | mat128: wrt the index page, yeah. sometimes copying isn't a good idea. i don't know what the right answer is, just giving you my impression as a user/reader. | 17:36 |
| mat128 | rloo: "Configure Networking to ..." is the same | 17:36 |
| mat128 | oh wait, there's config too | 17:36 |
| rloo | mat128: i haven't gotten that far yet... | 17:37 |
| mat128 | rloo: "Configure Compute flavors for use with the Bare Metal service" is data-only | 17:37 |
| rloo | mat128: i was comparing the sections in the old install with the new install, which is why i was looking at Configure the Image Service. Was trying to find out where 'IMage requirements' went. | 17:37 |
| rloo | mat128: wrt the compute flavors, I see 'Flavor creation' in the old; is that what got moved to the new 'Configure compute flavors...' ? | 17:38 |
| mat128 | in a previous changeset, JayF commented that it belonged under service integration, I agreed and moved it there | 17:38 |
| mat128 | rloo: correct | 17:38 |
| * JayF <-- all his fault | 17:38 | |
| mat128 | JayF: not blaming | 17:38 |
| JayF | I'm joking :) | 17:38 |
| JayF | I still like it there | 17:38 |
| * mat128 cant tell with this being text only | 17:38 | |
| mat128 | ill get used to it :P | 17:39 |
| JayF | because as an op, if I was setting up Ironic for the first time, I'd think of it being "what do I do to ironic?" "what do I do to other services?" | 17:39 |
| rloo | mat128: it very well might belong there, i haven't thought that far. maybe the section just needs to be renamed, to 'Adding images to the Image Service' or 'Using the Image Service'. dunno. | 17:39 |
| JayF | the difference between changing a config file and adding data via the api is ... insignificant | 17:39 |
| rloo | mat128: but that section also describes using disk-image-builder... | 17:39 |
| mat128 | I have to leave for a downstream meeting (again), bbl | 17:39 |
| mat128 | rloo, JayF ^ | 17:39 |
| *** mat128 is now known as mat128|mtg | 17:39 | |
| rloo | do people REALLY want the install guide migrated today/tomorrow? | 17:40 |
| JayF | I mean, I suspect mat128|mtg wouldn't have been rebasing that patchset constantly if he didn't; I wouldn't be advocating for it and getting it in if I didn't | 17:41 |
| JayF | I know "getting ironic in the official install guide" has been something we've wanted to do a long time | 17:41 |
| rloo | JayF: let me rephrase it. Do people really think it is ready to be migrated. | 17:41 |
| JayF | rloo: I think having an OK install guide in the right place is better than an OK install guide in the wrong place | 17:42 |
| rloo | JayF: I also want it migrated. I just don't want to feel rushed about it and to push something so late in the game. | 17:42 |
| JayF | rloo: and by doing the migration now we make it a hell of a lot easier to improve it next cycle with smaller patches | 17:42 |
| rloo | JayF: IF it was OK as the original is OK. | 17:42 |
| vdrok | Time to leave for me, good night! | 17:42 |
| JayF | rloo: I'm honestly more afraid if we attempt to do more of an ...editorial+migration at the same time, it'll be one of those things that never get done | 17:42 |
| rloo | JayF: we ARE going to do the migration. the question is whether to do it so soon to release. which leaves no time to fix/clean things up, and either backporting fixes or not backporting. | 17:42 |
| JayF | by getting something in at all, that's at least as good as what exists today, we setup the foundation to improve it in bite size pieces | 17:43 |
| rloo | JayF: I am NOT editorial'ing anything. i don't think so. i'm comparing what we had to what is proposed, and i see stuff lacking | 17:43 |
| JayF | rloo: I'm more saying; if it gets pushed to Ocata in order to make it "better", it *will* turn into that kind of review, and I forsee lots of time spent painting bikesheds :( | 17:44 |
| rloo | JayF: so you are saying you are fine with 'Configure the Image Service' not including any config changes, but describing how to generate images and get them into the Image service? | 17:44 |
| JayF | rloo: I suggested the change! I'm more than fine with it, I think it's where it belongs. | 17:44 |
| *** ijw has joined #openstack-ironic | 17:45 | |
| rloo | JayF: I disagree with your comment wrt Ocata and bikeshedding etc. You/we just have to be firm with folks that the initial thing is to migrate, not edit the content. | 17:45 |
| *** anush has quit IRC | 17:46 | |
| JayF | I mean, I obviously disagree :). I just want to know what the step is to break the impasse. If the answer is that it doesn't merge, and we release, then that's going to suck, and I'll be unhappy (mat too, I suspect), but I've been unhappy before. I just think we have the opportunity to release with something better, and it'll be extra-sad since we tried and failed than if we had just pushed it from | 17:48 |
| JayF | the start | 17:48 |
| rloo | JayF: I'm feeling pressured, like I'm the one blocking it from merging. I'm going to remove my -2 from the first patch. You and Jim or other cores are free to +2 +A them. | 17:49 |
| JayF | I mean, I honestly don't know how to resolve stuff like this. | 17:49 |
| JayF | I don't get a supervote, nor do you | 17:49 |
| JayF | Conflict resolution over IRC is a difficult skill I haven't yet acquired :) | 17:49 |
| *** livelace has quit IRC | 17:49 | |
| *** vsaienko has quit IRC | 17:50 | |
| *** vsaienko has joined #openstack-ironic | 17:57 | |
| rloo | JayF: I don't know if it is a conflict. I'm fine if others want to review, +A. I'm just voicing my own opinion. | 17:59 |
| rloo | JayF: what you and mat128|mtg have done is great. but that's what reviewing is about, reviewing... | 17:59 |
| JayF | I mean, honestly, I respect your opinion enough to think if it's not good enough, it's not good enough | 17:59 |
| JayF | just a blow to miss the td by inches, to use a football analogy | 18:00 |
| rloo | JayF: i understand. I have a meeting. will try to look/review more later. | 18:00 |
| *** srobert has joined #openstack-ironic | 18:11 | |
| TheJulia | Truthfully I'd personally rather see something partially functional and iterate upon it in later changes, but that is just me. I guess I look at it from a standpoint that there is no perfection, and that good is a relative thing that is up to the eye of the viewer. We can't necessarily see if something is perceived by others as good without feedback, but | 18:11 |
| TheJulia | everyone is going to have separate criteria by which they evaluate good, so perhaps we should evaluate on functional, does it fill the void, does it (essentially) meet the need so we can start the next step. | 18:11 |
| * TheJulia gets off her soapbox, puts it back in the cardboard box, and puts it in a storage compartment on the RV | 18:12 | |
| *** jpich has joined #openstack-ironic | 18:14 | |
| openstackgerrit | Merged openstack/bifrost: Change Bifrost to TinyIPA as the default https://review.openstack.org/373574 | 18:15 |
| JayF | TheJulia: I suspect the more core piece of the disagreement is whether or not the status quo is ok: today we have an install guide that presumedly works, and has for a while, just in the wrong place. Concerns that just moving it and reorganizing it to "fit" install guide are enough to make it worse are valid. IDK. It's hard. | 18:16 |
| JayF | TheJulia: honestly, if I put away the fact that I want it in because I know mat128|mtg has been working on it, and because I have been working on it, and just look at it objectively, it is something that's a little big to try and slip in under the wire. | 18:17 |
| *** baoli has quit IRC | 18:19 | |
| TheJulia | JayF: indeed, change is hard, but at the same time we won't know until later if it was the right or wrong decision, and all we can do is learn from there. I do agree with you though, it is rather big at the last moment. | 18:19 |
| *** baoli has joined #openstack-ironic | 18:20 | |
| TheJulia | JayF: I kind of see it as a learning opportunity as well, if people are grumpy by the change, we learn. If not, we don't _really_ hear anything that we are not already hearing. | 18:20 |
| JayF | TheJulia: I think I'm going to suggest something more middling: we release w/o merging these, unless all cores reviewing it can feel confident approving it (btw, your reviews would be appreicated :D), then work on it, and if it merges in a week or two, consider (or at least make the case to jroll /stable team) backporting it to newton | 18:20 |
| JayF | TheJulia: one thing I am not on-board with is merging it if other cores are -1 on it, either in vote or because it's something that's going in too late. | 18:21 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 18:23 |
| TheJulia | JayF: More than happy to review btw, just... things have been interesting as of recent and just getting back into the swing of things. I feel the same regarding not merging if other cores are -1 on it, but I also worry about letting things sit in limbo, since we've seen how that goes time and time again. | 18:26 |
| JayF | yeah; I think even if it doesn't hit before release, we should keep the speed up on trying to get the install guide in | 18:26 |
| JayF | and don't change the perspective of a "direct port" first, and "editorial improvements" later | 18:26 |
| *** e0ne has joined #openstack-ironic | 18:29 | |
| *** rcernin has joined #openstack-ironic | 18:30 | |
| TheJulia | JayF: absolutely, speed +++++++ | 18:31 |
| TheJulia | JayF: yeah, changing midstream just becomes scope creep, which just slows us down | 18:32 |
| *** ijw has quit IRC | 18:34 | |
| *** vsaienko has quit IRC | 18:34 | |
| *** jpich has quit IRC | 18:35 | |
| *** ijw has joined #openstack-ironic | 18:36 | |
| *** sdake has quit IRC | 18:39 | |
| *** rbudden has quit IRC | 18:49 | |
| *** ig0r_ has joined #openstack-ironic | 18:51 | |
| *** sdake has joined #openstack-ironic | 18:56 | |
| *** moshele has joined #openstack-ironic | 18:59 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Fix DRAC passthru 'list_unfinished_jobs' desc https://review.openstack.org/374331 | 18:59 |
| *** moshele has quit IRC | 19:03 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/373701 | 19:06 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/373747 | 19:06 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/373749 | 19:06 |
| jroll | rloo: +2 on ^^ did you want that in before the release? | 19:06 |
| * jroll fixing up reno | 19:06 | |
| *** mat128|mtg is now known as mat128 | 19:06 | |
| mat128 | back | 19:06 |
| rloo | jroll: it would be nice but it isn't that important. i don't think. only drac related. | 19:08 |
| jroll | right, ok | 19:10 |
| * mat128 is done reading scrollback | 19:10 | |
| *** catintheroof has joined #openstack-ironic | 19:12 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 19:12 |
| *** BadCub has quit IRC | 19:13 | |
| *** sabeen has joined #openstack-ironic | 19:13 | |
| *** ijw has quit IRC | 19:14 | |
| *** ijw has joined #openstack-ironic | 19:15 | |
| *** BadCub has joined #openstack-ironic | 19:15 | |
| *** BadCub has quit IRC | 19:15 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up release notes for 6.2.0 https://review.openstack.org/373357 | 19:16 |
| jroll | rloo: ^ sorry to bug, but just one more | 19:16 |
| *** BadCub has joined #openstack-ironic | 19:16 | |
| jroll | if other cores could help land that, would be awesome | 19:16 |
| jroll | mat128: so I think I have to agree with ruby and jay, it's just super late for such a big thing. I do kinda want to just merge it but they have valid points. I really don't think we should backport the whole effort, but rather just have an AMAZING install guide in the right place for ocata. | 19:18 |
| jroll | mat128: so, I vote we get newton out and then merge the install guide work tomorrow, how's that sound? :) | 19:18 |
| mat128 | jroll: that's fair. at that point my only concern is fixing merge conflicts. I wish we could have done it but I want everyone to be comfortable with it landing | 19:20 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Enable OSProfiler support in Ironic https://review.openstack.org/358142 | 19:20 |
| jroll | mat128: yeah, that's why I still want to land it asap | 19:21 |
| * jroll steps away for a moment | 19:21 | |
| *** jvgrant__ has quit IRC | 19:24 | |
| *** ijw has quit IRC | 19:25 | |
| rloo | jroll: looking at release notes... | 19:25 |
| JayF | rloo: jroll: release notes are supposed to be present tense, correct? | 19:29 |
| rloo | JayF: yes, err. it isn't consistent so i'm ignoring them. | 19:29 |
| rloo | JayF: see jroll's patch: https://review.openstack.org/#/c/374149/ | 19:30 |
| patchbot | patch 374149 - ironic - Add tense guide to release note FAQ | 19:30 |
| rloo | JayF: we've been sort of using present tense. i'd actually like to know what (if anything) other openstack projects do. | 19:30 |
| rloo | JayF: every time I review these release notes, i think we need to set out some guidelines to make it less painful at the end of a release. I initially thought it was up to the release manager to deal with it, so who cares. ha ha. | 19:31 |
| milan | good night Ironic /me out | 19:31 |
| JayF | +1 writing things down is wonderful | 19:31 |
| rloo | night milan | 19:31 |
| JayF | o/ | 19:31 |
| milan | rloo, JayF o/ | 19:31 |
| rloo | JayF: after we decide what it is! | 19:31 |
| *** milan has quit IRC | 19:31 | |
| *** jvgrant has joined #openstack-ironic | 19:35 | |
| JayF | TheJulia: re: using present tense in https://review.openstack.org/#/c/374149/1/doc/source/dev/faq.rst; I think of it like "6.2.0 adds support for X" vs "6.2.0 added support for X" ... both work, consistency is all that matters | 19:39 |
| patchbot | patch 374149 - ironic - Add tense guide to release note FAQ | 19:39 |
| *** ijw has joined #openstack-ironic | 19:40 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack: Use Jinja2 for templating when creating new VMs https://review.openstack.org/374345 | 19:41 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack: Support for creating UEFI VMs https://review.openstack.org/374346 | 19:41 |
| *** ijw has quit IRC | 19:41 | |
| *** ijw has joined #openstack-ironic | 19:41 | |
| TheJulia | JayF: Well, I kind of view it as 6.2.0 was not what added it, it was a human that added it to be in 6.2.0 in the past, but consistency is indeed the key and I'll agree with anything that drives consistency :) | 19:41 |
| TheJulia | JayF: including if we create new words to denote things, as long as we're consistent :) | 19:43 |
| mat128 | lucasagomes: "The virtio disk controller apparently does not work with UEFI" did you find any official source for this? | 19:44 |
| openstackgerrit | Merged openstack/ironic: Clean up release notes for 6.2.0 https://review.openstack.org/373357 | 19:44 |
| rloo | TheJulia, JayF: fwiw, I just took a quick look at nova's release notes. they aren't consistent either wrt tense. | 19:45 |
| JayF | so we'll be ahead of the curve by standardizing it | 19:45 |
| JayF | (in before the rest of openstack standardizes on $different and people get upset at us) | 19:46 |
| mat128 | JayF: thats why you send it to the ML for prior art | 19:46 |
| devananda | w.r.t. moving the install guide, I haven't looked at the change (I should!). Not having something - anything - in the official place is a serious detriment to th eproject's public perception | 19:46 |
| TheJulia | JayF: you know, that means they will just create some different word, and... ugh | 19:47 |
| devananda | having something subpar but mostly factually correct in the official location is good enough. it doesn't, IMO, need to be perfect. | 19:47 |
| *** e0ne has quit IRC | 19:47 | |
| mat128 | devananda: https://review.openstack.org/#/q/topic:bug/1612278, final version: http://docs-draft.openstack.org/72/374272/4/check/gate-ironic-install-guide/ed5d8b7//install-guide/build/html/ | 19:47 |
| jroll | rloo: JayF: thanks for the help! :) | 19:47 |
| devananda | mat128: thnks! looking now | 19:47 |
| jroll | I think we're ready to release 6.2.0, I'll give people 5 minutes to object :) | 19:48 |
| mat128 | jroll: please give devananda the time to read the install guide :) | 19:48 |
| jroll | and by that, I mean ironic 6.2.0, ironic-inspector, and ipa | 19:48 |
| rloo | jroll: ^^ devananda is looking at the install guide | 19:48 |
| jroll | did we not just decide we should wait? | 19:48 |
| lucasagomes | mat128, not really just testing it locally | 19:49 |
| rloo | i still don't like 'Configure the Image Service' section title. | 19:49 |
| lucasagomes | mat128, found some forums talking about it | 19:49 |
| lucasagomes | mat128, but I can confirm that in ubuntu 16.04 at least it doesn't | 19:49 |
| * jroll -W for now | 19:49 | |
| mat128 | lucasagomes: ok, if there was an official bug I would have suggested linking it | 19:49 |
| lucasagomes | mat128, yeah... lemme google a bit more see if I find something | 19:49 |
| devananda | rloo: that's the title we have in the docs today, isn't it? | 19:49 |
| mat128 | devananda: no, it was "Image requirements" | 19:50 |
| rloo | devananda: no, it was something like 'image requirements' | 19:50 |
| devananda | ah. I see your objection then | 19:50 |
| rloo | devananda: i think it ought to be changed to something like 'creating and adding images to Image service' | 19:50 |
| rloo | jroll: I am looking at the openstack release schedule. RC1 was last week. next week is final rc's. | 19:50 |
| lucasagomes | mat128, http://edk2-devel.narkive.com/I8GEtAXw/virtio-disk-not-bootable-in-ovmf | 19:51 |
| devananda | "Configure Networking to communicate with the bare metal server" is also an incorrect heading | 19:51 |
| jroll | rloo: yes | 19:51 |
| rloo | jroll: can we do a release today. and if the install guide is ready, can we do another release next week? | 19:51 |
| rloo | jroll: i mean, weren't you the one that wanted to release every week? | 19:51 |
| jroll | rloo: technically, we can. I was hoping to branch now so we can start hacking on ocata. | 19:51 |
| mat128 | devananda: "Configure Networking to communicate with the bare metal server" is what it was in the old install guide | 19:51 |
| devananda | jroll: doc-only patch backported to a recent branch ... huh ... | 19:52 |
| rloo | devananda: how important do you think it is to have the new install guide for newton? | 19:52 |
| jroll | rloo: I don't want to leave newton open (blocking big features) just for the install guide | 19:52 |
| jroll | devananda: ? | 19:52 |
| rloo | devananda: i think it is fine if we have the new install guide in master/draft. | 19:52 |
| devananda | rloo: to have it up on the docs website will have a significant benefit to users for the next 12-18 months | 19:52 |
| devananda | I do not actually care if it's in our release -- except that, AIUI, that is how we get it up onto the docs site for THIS release | 19:53 |
| rloo | devananda: the bare bones of it will be on the docs website; all pointing to our dev install guide. | 19:53 |
| jroll | devananda: fwiw, we now have a link to our current guide in the official area http://docs.openstack.org/project-install-guide/baremetal/draft/ | 19:53 |
| jroll | so it's not just missing | 19:53 |
| JayF | and intentions to change that^^ to link to newton dev docs when we branch | 19:53 |
| devananda | jroll: "draft" | 19:53 |
| devananda | jroll: not version specific | 19:53 |
| JayF | devananda: draft == master | 19:53 |
| JayF | devananda: draft becomes newton when we branch | 19:53 |
| jroll | devananda: /draft == master, /branch == stable/branch | 19:53 |
| devananda | ah | 19:53 |
| jroll | so | 19:53 |
| jroll | we've had this same discussion three times this week | 19:53 |
| devananda | that's neat | 19:53 |
| jroll | and walked through the same basic concepts | 19:54 |
| jroll | and as of right now I'm the only core that is pro-merge-now, out of the three that are "in the know" on this effort | 19:54 |
| rloo | jroll: the install guide migration probably requires an rfe/spec | 19:54 |
| devananda | jroll: great. sorry for causing more noise while catching up here ... | 19:54 |
| devananda | barring anything factually incorrect or horrible in it, I am definitely in the pro-merge side | 19:55 |
| jroll | devananda: it's fine, I just don't think 5 minutes before I plan to ship a release is a good time to have a discussion whether we should delay the release to drop in 15 patches to publish this install guide | 19:55 |
| devananda | because it is one of the largest issues operators have raised to me every time I talk with them | 19:55 |
| rloo | devananda: that would have been good to know earlier, so we could have bumped the priority on this sooner. | 19:55 |
| jroll | devananda: specifically, what's the issue they have? hard to find? incorrect? or "isn't the openstack docs theme"? or? | 19:55 |
| devananda | jroll: that's totally fair. I'm out of the loop. I didn't know you were about to shipit until after I began asking questions and scrolled down in my irc buffer | 19:56 |
| jroll | there was no sooner, the work on this started less than 2 weeks ago | 19:56 |
| rloo | regardless. what do we want to do now? | 19:56 |
| rloo | jroll: well, if it was a high priority, someone could have started it sooner. anyway, regardless... | 19:56 |
| devananda | rloo: the tooling to do this was only recently built by the docs team | 19:56 |
| devananda | as in, I became aware of its possibility about 2 weeks ago | 19:56 |
| jroll | ditto | 19:57 |
| jroll | maybe 3 tops | 19:57 |
| devananda | it was actually available only some time in this cycle, I don't know exactly when | 19:57 |
| *** rama_y has quit IRC | 19:57 | |
| JayF | we should celebrate that we got links in in such short order, and that we'll have an awesome install guide come ocata | 19:57 |
| rloo | i opened that bug aug 11 | 19:57 |
| JayF | and buy mat128 beers at the next mid-cycle for championing the work | 19:57 |
| rloo | and i think we all want that install guide migrated. | 19:57 |
| devananda | I think it's absolutely astonishing that it has gotten this far this fast | 19:58 |
| rloo | totally agree on that! | 19:58 |
| devananda | I don't understand the objections to merging it -- aside from jroll's point about this literally stalling the release now | 19:58 |
| devananda | which is completely fair | 19:58 |
| devananda | and not my intent | 19:58 |
| jroll | devananda: it's mostly about, it has insufficient review time | 19:58 |
| devananda | has anyone walked through it and run all the commands to see if they work? << mat128 ? | 19:59 |
| jroll | I'd put money on no | 19:59 |
| mat128 | devananda: my ironic operations background found nothing blatantly wrong, but I confess I did not run all the commands | 20:00 |
| devananda | ok. then I recind my prior statement. I do not think we should merge it | 20:00 |
| mat128 | TBH, that was my first time reading it | 20:00 |
| mat128 | this whole project | 20:00 |
| rloo | the only new commands (I think) are the ones for installing from the diff OS. other than that, what was wrong before, will continue to be wrong which is OK wrt 'only migrating'. | 20:00 |
| jroll | devananda: fwiw, I think it's close enough to merge the moment ocata opens | 20:01 |
| jroll | but that gives us 5 months to fix things | 20:01 |
| JayF | and there are lots of folks interested (I'm one of them) in improving the editorial quality of the install guide over those 5 months once the migration is merged | 20:01 |
| devananda | jroll: yah. it also means that 6 - 9 months from now, when a whole bunch of operators go to upgrade their clouds to Newton, they will not find Newton-install-guide for ironic | 20:01 |
| jroll | devananda: they will, because there will be a link in the official docs to our old thing | 20:02 |
| jroll | insert periodic reminder that dev docs also publish for stable branches | 20:02 |
| devananda | jroll: oh - right. what's currently /draft | 20:02 |
| jroll | yep :) | 20:02 |
| devananda | well then, great | 20:02 |
| jroll | I do understand the concern and I agree | 20:02 |
| jroll | but I think what we have is good enough, and I'd rather not rush things | 20:02 |
| JayF | We still get to have an in-project install guide the first release after it's possilbe, even if it's just links | 20:03 |
| JayF | which is pretty awesome | 20:03 |
| mat128 | ^ this | 20:03 |
| JayF | and I want, this cycle, to try and get admin-guide to be able to be done in-tree too, so maybe in p we can improve that docs | 20:03 |
| devananda | jroll: huh. potential problem, which we can sort out later -- http://docs.openstack.org/project-install-guide/baremetal/draft/install-ubuntu.html#install-and-configure-components links to http://docs.openstack.org/developer/ironic/deploy/install-guide.html#install-the-bare-metal-service which is not versioned, and will be removed once we move the guide over | 20:03 |
| JayF | devananda: it is versioned | 20:04 |
| jroll | devananda: right, we plan to fix it | 20:04 |
| devananda | so we'll need to update the links on the newton stable branch of the docs | 20:04 |
| devananda | cool | 20:04 |
| JayF | devananda: yeah, the generated stuff is versioned and we'll fix those links in stable/newton | 20:04 |
| mat128 | devananda: http://docs.openstack.org/developer/ironic/mitaka/deploy/install-guide.html | 20:04 |
| devananda | that all makes me quite happy then | 20:04 |
| devananda | thanks for catching me up // sorry for repeating the discussion | 20:04 |
| jroll | no worries | 20:04 |
| * rloo glad that devananda is happy. | 20:05 | |
| mat128 | jroll: so you were about to do a release, right? | 20:05 |
| mat128 | :) | 20:05 |
| * rloo hopes that devananda reviews the install guide patches :) | 20:05 | |
| jroll | mat128: indeed :) https://review.openstack.org/374367 | 20:05 |
| patchbot | patch 374367 - releases - Release final newton versions for ironic services | 20:05 |
| * mat128 promises to make the install-guide better once it's in its right place | 20:05 | |
| rloo | jroll must be very happy that we're releasing ... | 20:06 |
| *** bdeschenes has quit IRC | 20:06 | |
| TheJulia | fwiw, I'm slowly going through them now, so far only one thing that makes me kind of raise an eyebrow, but not a big deal | 20:06 |
| rloo | jroll: WAIT. shouldnt we fix that db migration timeout bug? :D | 20:06 |
| jroll | rloo: er, yeah probably, but we can also backport it | 20:07 |
| jroll | rloo: it's only a unit test problem AIUI | 20:07 |
| rloo | jroll: i saw it happen at least once today. it is only a unit test problem, but it means having to recheck. which i don't think we want to do so often, esp not close to release. | 20:07 |
| rloo | jroll: i saw some other gate failures too today. | 20:08 |
| *** catintheroof has quit IRC | 20:08 | |
| *** rama_y has joined #openstack-ironic | 20:08 | |
| jroll | rloo: well, everything we wanted in is done, and the release is proposed, so no worries on that. but I hear ya | 20:08 |
| rloo | jroll: i'll continue on the install guide today and check tomorrow to see if anyone is fixing the timeout issue. | 20:08 |
| lucasagomes | calling it a day | 20:08 |
| rloo | bye lucasagomes | 20:08 |
| lucasagomes | have a great evening all, talk to you tomorrow | 20:08 |
| lucasagomes | rloo, bye bye | 20:08 |
| jroll | rloo: yeah, I need to catch up on that bug/fix | 20:08 |
| *** lucasagomes is now known as lucas-afk | 20:09 | |
| *** BadCub has quit IRC | 20:09 | |
| *** BadCub has joined #openstack-ironic | 20:10 | |
| rloo | mat128: i'm probably going to be reviewing/commenting on the install guide patches in different order, and may go back to them again. so probably best to wait til i have gone through them all before trying to address them. | 20:14 |
| mat128 | rloo: sure, no problem | 20:15 |
| rloo | mat128: i'll let you know :) | 20:15 |
| mat128 | will wait for your ping :) | 20:15 |
| *** jcoufal has joined #openstack-ironic | 20:15 | |
| *** joprovost has quit IRC | 20:17 | |
| *** rbudden has joined #openstack-ironic | 20:17 | |
| *** ijw has quit IRC | 20:18 | |
| *** wajdi_ has joined #openstack-ironic | 20:22 | |
| *** wajdi has quit IRC | 20:25 | |
| *** Goneri has quit IRC | 20:26 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector-specs: It would be useful to display the contents of Link Layer Discovery Protocol (LLDP) TLVs received from network switches that are cached by IPA, in order to help with deployment validation and troubleshooting. The spec presents new 'openstack baremetal' co https://review.openstack.org/374381 | 20:28 |
| *** ijw has joined #openstack-ironic | 20:29 | |
| *** wajdi has joined #openstack-ironic | 20:29 | |
| *** wajdi_ has quit IRC | 20:31 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands https://review.openstack.org/374381 | 20:32 |
| *** baoli has quit IRC | 20:33 | |
| *** bdeschenes has joined #openstack-ironic | 20:33 | |
| *** baoli has joined #openstack-ironic | 20:33 | |
| *** wajdi has quit IRC | 20:33 | |
| *** trown is now known as trown|outtypewww | 20:36 | |
| *** sinval has joined #openstack-ironic | 20:39 | |
| *** bfournie has quit IRC | 20:41 | |
| *** wznoinsk has quit IRC | 20:44 | |
| *** wznoinsk has joined #openstack-ironic | 20:45 | |
| *** ijw has quit IRC | 20:49 | |
| *** ijw has joined #openstack-ironic | 20:50 | |
| *** ijw has quit IRC | 20:53 | |
| *** rbudden has quit IRC | 20:54 | |
| *** krtaylor has quit IRC | 20:56 | |
| *** rama_y has quit IRC | 20:58 | |
| *** sinval has left #openstack-ironic | 21:00 | |
| *** baoli has quit IRC | 21:02 | |
| *** baoli has joined #openstack-ironic | 21:02 | |
| *** rama_y has joined #openstack-ironic | 21:03 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/373701 | 21:03 |
| *** ijw has joined #openstack-ironic | 21:14 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic: Add description for vendor passthru methods https://review.openstack.org/370469 | 21:14 |
| *** baoli_ has joined #openstack-ironic | 21:19 | |
| *** baoli has quit IRC | 21:19 | |
| *** ijw has quit IRC | 21:19 | |
| *** baoli_ has quit IRC | 21:19 | |
| *** baoli has joined #openstack-ironic | 21:20 | |
| *** rcernin has quit IRC | 21:24 | |
| *** xilet has joined #openstack-ironic | 21:27 | |
| *** baoli has quit IRC | 21:28 | |
| *** bfournie has joined #openstack-ironic | 21:29 | |
| *** baoli has joined #openstack-ironic | 21:29 | |
| *** krtaylor has joined #openstack-ironic | 21:40 | |
| *** rbudden has joined #openstack-ironic | 21:42 | |
| mrda | Morning Ironic | 21:44 |
| *** ig0r_ has quit IRC | 21:45 | |
| xavierr | morning mrda | 21:48 |
| mrda | Morning xavierr | 21:48 |
| *** jheroux has quit IRC | 21:48 | |
| *** ccamacho|afk has quit IRC | 21:59 | |
| *** vsaienko has joined #openstack-ironic | 22:01 | |
| *** srobert has quit IRC | 22:01 | |
| *** ijw has joined #openstack-ironic | 22:02 | |
| xavierr | morning devananda. Are these slides available http://devananda.github.io/talks/isnt-it-ironic.html ? | 22:03 |
| xavierr | devananda: they are been referenced on our docs: http://docs.openstack.org/developer/ironic/deploy/user-guide.html#example-1-pxe-boot-and-iscsi-deploy-process | 22:04 |
| xavierr | s/on/in/ | 22:04 |
| devananda | xavierr: interesting - i'm not sure why it's not rendering. source is here | 22:11 |
| devananda | https://github.com/devananda/talks/blob/master/isnt-it-ironic.html | 22:11 |
| *** sabeen has quit IRC | 22:12 | |
| NobodyCam | morning mrda xavierr and devananda :) | 22:13 |
| devananda | xavierr: lemme see if I can fix it ... | 22:14 |
| devananda | NobodyCam: g'morning?afternoon? :) | 22:14 |
| NobodyCam | hehehehe :) | 22:14 |
| mrda | Hey NobodyCam devananda | 22:14 |
| *** rbudden has quit IRC | 22:14 | |
| NobodyCam | :) | 22:15 |
| *** causten_ has quit IRC | 22:17 | |
| *** cdearborn has quit IRC | 22:17 | |
| devananda | xavierr: ok - fixed it! looks like its been broken for a while. sorry about thta | 22:17 |
| *** causten has joined #openstack-ironic | 22:17 | |
| *** joprovost has joined #openstack-ironic | 22:19 | |
| *** jcoufal has quit IRC | 22:20 | |
| *** causten has quit IRC | 22:22 | |
| *** rpioso has quit IRC | 22:23 | |
| rloo | mat128: sorry it took so long. You did a lot :) I think I'm mostly good. I'm not quite sure about the install & configure sections still, i might want to tweak after my suggested changes. thx! will check in with you tomorrow. | 22:29 |
| *** aNuposic has quit IRC | 22:30 | |
| rloo | mat128: 'good' == done looking at the changes (for now) :) | 22:30 |
| xavierr | devananda: thank you so much! :) | 22:30 |
| xavierr | NobodyCam: hello there, good morning! :D | 22:31 |
| TheJulia | sambetts|pto: Question for you at some point when your back: Did we intentionally skip tinyipa in the documentation? I just noticed in mat128's install guide stuff that building dib and coreos IPA are the only ones covered. While... tinycore may not be the solution for everyone, it might be something worth noting. At least, that is what my brain is | 22:31 |
| TheJulia | thinking. | 22:31 |
| NobodyCam | :) | 22:31 |
| devananda | mrda: ohai! :) | 22:31 |
| *** vsaienko has quit IRC | 22:31 | |
| mrda | o/ | 22:32 |
| devananda | TheJulia: I was briefly looking over the install guide when i did the front page restructure - and, off hand, it seemed like it hadn't been significantly updated since about liberty | 22:32 |
| rloo | TheJulia: i'm not positive, but i think i saw some documentation on tinyipa in ipa docs. | 22:32 |
| TheJulia | devananda: I feel like I'm missing something along the way, but I just got off a long call | 22:33 |
| *** ijw has quit IRC | 22:33 | |
| xavierr | TheJulia: OneView CI is back :) | 22:36 |
| NobodyCam | w00t | 22:36 |
| xavierr | o/ | 22:36 |
| devananda | TheJulia: sorry - maybe I'm missing something. I'm curious what sambetts' response will be. | 22:38 |
| TheJulia | devananda: ahh, more context \o/ | 22:38 |
| TheJulia | devananda: Okay, that was my perception as well, but I have noticed some content changes over time :) | 22:40 |
| xavierr | have a good night all | 22:49 |
| *** xavierr is now known as xavierr|afk | 22:49 | |
| NobodyCam | night xavierr|afk | 22:51 |
| TheJulia | xavierr|afk: \o/, and goodnight | 22:53 |
| *** ijw has joined #openstack-ironic | 22:54 | |
| *** ijw has quit IRC | 22:59 | |
| *** baoli has quit IRC | 23:04 | |
| *** hoonetorg has quit IRC | 23:10 | |
| *** BadCub_ has joined #openstack-ironic | 23:20 | |
| *** ptx0 has quit IRC | 23:21 | |
| *** mkam has quit IRC | 23:29 | |
| *** ptx0 has joined #openstack-ironic | 23:29 | |
| *** thrash is now known as thrash|g0ne | 23:31 | |
| *** ijw has joined #openstack-ironic | 23:32 | |
| *** ijw has quit IRC | 23:36 | |
| *** ptx0 has quit IRC | 23:38 | |
| openstackgerrit | Merged openstack/bifrost: Make bifrost on stable/mitaka branch deploy stable/mitaka ironic https://review.openstack.org/363197 | 23:38 |
| *** BadCub has quit IRC | 23:38 | |
| *** BadCub_ is now known as BadCub | 23:38 | |
| *** ijw has joined #openstack-ironic | 23:41 | |
| *** bdeschenes has quit IRC | 23:45 | |
| *** ptx0 has joined #openstack-ironic | 23:51 | |
| *** joprovost has quit IRC | 23:51 | |
| *** mtanino has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!