| *** zackf has quit IRC | 00:19 | |
| *** tuanla_ has joined #openstack-ironic | 00:27 | |
| *** sdake has quit IRC | 00:28 | |
| *** ijw has quit IRC | 00:28 | |
| *** rama_y has quit IRC | 00:38 | |
| *** rama_y has joined #openstack-ironic | 00:40 | |
| *** hoangcx has joined #openstack-ironic | 00:48 | |
| *** baoli has quit IRC | 00:56 | |
| *** baoli has joined #openstack-ironic | 00:59 | |
| *** baoli has quit IRC | 00:59 | |
| *** baoli has joined #openstack-ironic | 00:59 | |
| *** ijw has joined #openstack-ironic | 01:18 | |
| *** zackf has joined #openstack-ironic | 01:23 | |
| *** ijw has quit IRC | 01:25 | |
| *** david-lyle has joined #openstack-ironic | 01:27 | |
| *** david-lyle has quit IRC | 01:31 | |
| *** tangchen has joined #openstack-ironic | 01:36 | |
| *** zackf has quit IRC | 01:36 | |
| *** baoli has quit IRC | 01:39 | |
| *** baoli has joined #openstack-ironic | 01:40 | |
| *** baoli has quit IRC | 01:45 | |
| *** Sukhdev has joined #openstack-ironic | 01:59 | |
| *** hoangcx has quit IRC | 02:10 | |
| *** ijw has joined #openstack-ironic | 02:21 | |
| *** ijw has quit IRC | 02:26 | |
| *** baoli has joined #openstack-ironic | 02:42 | |
| *** baoli has quit IRC | 02:47 | |
| *** baoli has joined #openstack-ironic | 03:07 | |
| *** baoli has quit IRC | 03:32 | |
| *** hoangcx has joined #openstack-ironic | 03:37 | |
| *** vikrant has joined #openstack-ironic | 03:38 | |
| *** michauds has joined #openstack-ironic | 04:04 | |
| tuanla_ | Hello ironic team | 04:09 |
|---|---|---|
| *** ijw has joined #openstack-ironic | 04:24 | |
| *** ijw has quit IRC | 04:28 | |
| *** moshele has joined #openstack-ironic | 04:36 | |
| *** davidlenwell has quit IRC | 04:37 | |
| *** michauds has quit IRC | 04:40 | |
| *** tuanla_ has quit IRC | 04:48 | |
| *** davidlenwell has joined #openstack-ironic | 04:49 | |
| *** vsaienko has joined #openstack-ironic | 05:01 | |
| *** pcaruana has quit IRC | 05:05 | |
| *** jaosorior has joined #openstack-ironic | 05:11 | |
| *** vsaienko has quit IRC | 05:16 | |
| *** vsaienko has joined #openstack-ironic | 05:21 | |
| *** tuanla_ has joined #openstack-ironic | 05:24 | |
| *** ijw has joined #openstack-ironic | 05:25 | |
| *** ijw has quit IRC | 05:30 | |
| *** vsaienko has quit IRC | 05:32 | |
| *** ChubYann has quit IRC | 05:40 | |
| *** monika has joined #openstack-ironic | 05:43 | |
| *** monika is now known as Guest21843 | 05:43 | |
| *** pall has joined #openstack-ironic | 05:44 | |
| pall | hi | 05:44 |
| *** baoli has joined #openstack-ironic | 05:44 | |
| pall | I have installed and configure Ironic service on overcloud controller node. I have done the introspection of baremetal node, however when i try to boot the node it gives me error - - No valid host was found. There are not enough hosts available | 05:46 |
| pall | please help me to fix this | 05:48 |
| *** baoli has quit IRC | 05:49 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Support configdrive in iscsi deploy for whole disk images https://review.openstack.org/225115 | 05:53 |
| *** pall has quit IRC | 05:53 | |
| *** pall_ has joined #openstack-ironic | 05:58 | |
| pall_ | hi | 05:58 |
| *** ccamacho has joined #openstack-ironic | 05:58 | |
| *** rcernin has joined #openstack-ironic | 06:03 | |
| *** mjura has joined #openstack-ironic | 06:04 | |
| *** KrisK19 has joined #openstack-ironic | 06:06 | |
| *** mjura has quit IRC | 06:11 | |
| *** mjura has joined #openstack-ironic | 06:11 | |
| *** vsaienko has joined #openstack-ironic | 06:13 | |
| *** Guest21843 has quit IRC | 06:17 | |
| *** pcaruana has joined #openstack-ironic | 06:17 | |
| *** hoangcx has quit IRC | 06:20 | |
| pas-ha | morning Ironic | 06:24 |
| *** rama_y has quit IRC | 06:25 | |
| *** ijw has joined #openstack-ironic | 06:26 | |
| pas-ha | pall_: this looks more like Nova scheduling issue. verify that a) all Ironic nodes are visible to Nova (nova hypervizor-list) b) ironic nodes have their properties re vcpus, ram and disk set correctly after introspection and c) your flavor matches those node capabilities | 06:28 |
| *** vsaienko has quit IRC | 06:30 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Enable SSH access to tinyipa https://review.openstack.org/371775 | 06:30 |
| *** ijw has quit IRC | 06:32 | |
| *** rbartal has joined #openstack-ironic | 06:33 | |
| *** vsaienko has joined #openstack-ironic | 06:34 | |
| tuanla_ | Hi pas-ha, can you share me n-cpu log | 06:35 |
| tuanla_ | morning ironic team | 06:35 |
| *** vsaienko has quit IRC | 06:37 | |
| *** hoangcx has joined #openstack-ironic | 06:38 | |
| *** vsaienko has joined #openstack-ironic | 06:43 | |
| pas-ha | tuanla_: what's the issue? | 06:44 |
| *** hoangcx has quit IRC | 06:47 | |
| *** hoangcx has joined #openstack-ironic | 06:48 | |
| *** lennyb_ has joined #openstack-ironic | 06:58 | |
| *** Sukhdev has quit IRC | 07:21 | |
| *** rama_y has joined #openstack-ironic | 07:24 | |
| *** vsaienko has quit IRC | 07:26 | |
| *** ohamada has joined #openstack-ironic | 07:27 | |
| *** ijw has joined #openstack-ironic | 07:28 | |
| *** jtomasek has joined #openstack-ironic | 07:28 | |
| *** ccamacho has quit IRC | 07:28 | |
| *** ijw has quit IRC | 07:33 | |
| *** jpich has joined #openstack-ironic | 07:33 | |
| *** ccamacho has joined #openstack-ironic | 07:34 | |
| tuanla_ | pas-ha, I am sorry | 07:41 |
| tuanla_ | pas-ha, I have issue when deploy whole-disk-image | 07:42 |
| tuanla_ | after node was active and reboot show message http://paste.openstack.org/show/580887/ | 07:42 |
| *** amitkqed has quit IRC | 07:53 | |
| *** amitkqed has joined #openstack-ironic | 07:54 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| openstackgerrit | Markos Chandras proposed openstack/bifrost: bifrost-prepare-for-test-dynamic: Create known_hosts if it's not present https://review.openstack.org/370258 | 08:02 |
| *** hoangcx has quit IRC | 08:03 | |
| *** rama_y has quit IRC | 08:08 | |
| vdrok | good morning ironic, happy monday! | 08:09 |
| *** KrisK19 has quit IRC | 08:09 | |
| *** KrisK19 has joined #openstack-ironic | 08:09 | |
| *** mgoddard has joined #openstack-ironic | 08:13 | |
| openstackgerrit | Yibo Cai proposed openstack/ironic: Support multi arch deployment https://review.openstack.org/322632 | 08:13 |
| *** anush__ has quit IRC | 08:13 | |
| *** anish has quit IRC | 08:13 | |
| *** hamzy has quit IRC | 08:14 | |
| *** vgadiraj has quit IRC | 08:14 | |
| *** hamzy has joined #openstack-ironic | 08:14 | |
| *** anush__ has joined #openstack-ironic | 08:14 | |
| *** anish has joined #openstack-ironic | 08:14 | |
| *** sturivnyi_ has quit IRC | 08:15 | |
| *** e0ne has joined #openstack-ironic | 08:15 | |
| *** sturivnyi_ has joined #openstack-ironic | 08:15 | |
| *** vgadiraj has joined #openstack-ironic | 08:17 | |
| *** Romanenko_K has joined #openstack-ironic | 08:22 | |
| *** athomas has joined #openstack-ironic | 08:26 | |
| *** e0ne has quit IRC | 08:27 | |
| pall_ | pas-ha:- i have verified | 08:28 |
| pall_ | [root@overcloud-controller-0 ~]# nova hypervisor-stats +----------------------+-------+ | Property | Value | +----------------------+-------+ | count | 1 | | current_workload | 0 | | disk_available_least | 49 | | free_disk_gb | 49 | | free_ram_mb | 5120 | | local_gb | 49 | | local_gb_used | 0 | | memory_mb | 5120 | | memory_m | 08:28 |
| *** ijw has joined #openstack-ironic | 08:29 | |
| pall_ | [root@overcloud-controller-0 ~]# nova hypervisor-list +----+--------------------------------------+-------+---------+ | ID | Hypervisor hostname | State | Status | +----+--------------------------------------+-------+---------+ | 2 | 835b2d2d-c9d1-4f60-9f63-409ec05a929c | up | enabled | +----+--------------------------------------+-------+---------+ | 08:29 |
| *** e0ne has joined #openstack-ironic | 08:29 | |
| pall_ | [root@overcloud-controller-0 ~]# nova flavor-show bare-01 +----------------------------+-------------------------------------------------------------+ | Property | Value | +----------------------------+-------------------------------------------------------------+ | OS-FLV-DISABLED:disabled | False | 08:30 |
| pall_ | [root@overcloud-controller-0 ~]# ironic node-show bare1 +------------------------+------------------------------------------------------------------------+ | Property | Value | +------------------------+------------------------------------------------------------------------+ | target_power_state | None | 08:30 |
| pall_ | ironic node properties and flavor properties matching | 08:32 |
| pall_ | i see ironic node in hypervisor list | 08:33 |
| pall_ | and in hypervisor-stats i see the baremetal configuration | 08:33 |
| pall_ | i still see error | 08:34 |
| pall_ | "message": "No valid host was found. There are not enough hosts available.", "code": 500, "details": " File \"/usr/lib/python2.7/site-packages/nova/conductor/manager.py\", line 739, in build_instances | | 08:34 |
| patchbot | Error: No closing quotation | 08:34 |
| *** ijw has quit IRC | 08:34 | |
| *** derekh has joined #openstack-ironic | 08:39 | |
| *** pmannidi has quit IRC | 08:47 | |
| *** e0ne has quit IRC | 08:51 | |
| *** e0ne has joined #openstack-ironic | 08:52 | |
| *** hoangcx has joined #openstack-ironic | 08:55 | |
| pas-ha | pall_: please use paste.openstack.org, can't really parse those line with eyes here :( also please include "ironic node-show" output for (some) of your nodes | 08:57 |
| pas-ha | tuanla_: it seems either your TFTP/networking is not configured properly or (if you are using Neutron) Neutron ports have not PXE opts set correctly | 08:59 |
| tuanla_ | pas-ha, Please see more detail in http://paste.openstack.org/show/580941/ | 09:00 |
| pall_ | http://paste.openstack.org/show/Uw8YpPw9R7meI4wnefDh/ | 09:02 |
| pas-ha | tuanla_: now it seems that the boot device was not set correctly - logs show that image was successfully downloaded and written to disk, but node still boots from PXE instead of local disk. AFAIK that might be a problem with firmware on the node, I've met such cases e.g. on some SuperMicro servers - with certain firmware versions when netboot is enabled in | 09:04 |
| pas-ha | BIOS, node is PXE-booted regardless of boot order | 09:04 |
| pall_ | pas-ha: are u able to see the output now ? | 09:05 |
| pas-ha | pall_: yes. | 09:05 |
| pas-ha | pall_: your nodes are configured with 'fake_pxe' driver - is that intentional? | 09:06 |
| pall_ | yes | 09:06 |
| *** hoangcx has quit IRC | 09:06 | |
| pall_ | #pas-ha I am using virtualmachine | 09:06 |
| pas-ha | pall_: so what is your target - verify that request is handled over to Ironic or actually boot the node? because to really boot it you'd still need a proper deploy driver, like agent_ssh or agent_ipmitool | 09:08 |
| pall_ | I need to boot it | 09:08 |
| tuanla_ | Hi pas-ha, I was sucessfull when deploy partition image, | 09:09 |
| pall_ | I am manually start the VM | 09:09 |
| pall_ | i will* | 09:09 |
| *** phuongnh has joined #openstack-ironic | 09:10 | |
| pall_ | i have used fake_pxe driver while deploying overcloud compute and controller node from undercloud | 09:10 |
| pas-ha | pall_: then I'm out of ideas :( let's wait for some triple-o involved folks to show up here | 09:12 |
| *** hoangcx has joined #openstack-ironic | 09:12 | |
| *** stendulker has joined #openstack-ironic | 09:13 | |
| pall_ | ok | 09:13 |
| pas-ha | tuanla_: are you sure your whole disk image contains the installed bootlader and such? OTOH I do not see the node even attempting to boot from disk.. | 09:15 |
| tuanla_ | Hi pas-ha, Can you give some guidle to fix such as you cases. I will try to check my configuration in BIOS and redeploy again | 09:15 |
| pas-ha | tuanla_: for the thing I wrote about some SuperMicros - nothing to fix except update firmware to working version | 09:16 |
| tuanla_ | pas-ha, I was using cirros default image, How do I can create or checking my image to make sure that this image installed bootloader | 09:17 |
| pas-ha | default cirros qcow2 image should be fine | 09:17 |
| patchbot | Error: 'supybot.cirros' is not a valid configuration variable. | 09:17 |
| tuanla_ | pas-ha, I was confused about different between whole-disk-image and partition image because when Deploying partition image when have configuration file in /tftpboot/pxelinux.cfg but when deploy whole-disk-image it was create in deploying and delete after node was active | 09:22 |
| tuanla_ | Hi pas-ha, I was try to using my node boot from local disk and it sucessfull to boot from local disk but when boot with network it have problem. | 09:24 |
| pas-ha | that means you were using partition image with netboot, so that Ironic caches the image kernel and ramdisk and node is still booted from PXE into this kernel and ramdisk served from Ironic's TFTP. whole disk image means that image contains everything to boot by itself from local disk, and that's what Ironic should do - write image to disk, change the node | 09:26 |
| pas-ha | boot sequence to boot from disk and reboot the node | 09:26 |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: Fix display of chassis UUID field if empty https://review.openstack.org/370764 | 09:26 |
| *** hoangcx has quit IRC | 09:26 | |
| *** sambetts|afk is now known as sambetts | 09:27 | |
| sambetts | Morning all | 09:28 |
| *** hoangcx has joined #openstack-ironic | 09:28 | |
| stendulker | morning sambetts | 09:31 |
| *** electrofelix has joined #openstack-ironic | 09:33 | |
| *** mgould|afk is now known as mgould | 09:34 | |
| mgould | morning sambetts stendulker pas-ha tuanla_ and everyone else! | 09:35 |
| tuanla_ | morning mgould | 09:35 |
| *** makowals has joined #openstack-ironic | 09:35 | |
| stendulker | mgould o/ | 09:35 |
| *** makowals_ has quit IRC | 09:36 | |
| tuanla_ | hi pas-ha, what about ERROR ironic_python_agent.ironic_api_client ConnectionError: HTTPConnectionPool | 09:39 |
| *** hoangcx has quit IRC | 09:48 | |
| *** hoonetorg has quit IRC | 09:56 | |
| *** cmyster has quit IRC | 09:57 | |
| *** hoonetorg has joined #openstack-ironic | 09:57 | |
| *** cmyster has joined #openstack-ironic | 09:58 | |
| openstackgerrit | Aparna proposed openstack/ironic: Enhance pxe_ilo driver to do inband-inspection https://review.openstack.org/343577 | 09:59 |
| *** hoangcx has joined #openstack-ironic | 10:10 | |
| *** phuongnh has quit IRC | 10:15 | |
| jroll | morning folks | 10:17 |
| sambetts | hey jroll | 10:17 |
| jroll | just fyi, I'm traveling this morning, should be back online around meeting time | 10:17 |
| jroll | hey sambetts :) | 10:18 |
| *** hoangcx has quit IRC | 10:25 | |
| *** openstackstatus has quit IRC | 10:28 | |
| mgould | morning jroll | 10:29 |
| *** ijw has joined #openstack-ironic | 10:31 | |
| *** narasimha_SV has joined #openstack-ironic | 10:32 | |
| narasimha_SV | if I am using external dhcp server for ironic | 10:32 |
| narasimha_SV | do I need to create port for the node ? | 10:32 |
| *** ijw has quit IRC | 10:36 | |
| *** vsaienko has joined #openstack-ironic | 10:36 | |
| *** dprince has joined #openstack-ironic | 10:39 | |
| *** vsaienko has quit IRC | 10:39 | |
| pas-ha | tuanla_: that ConnecttionError is due to agent first trying the new lookup API, but apparently your Ironic is not new enough (new lookup API is API v1.22). eventually it tries the old one and succeeds - Line 325 of your paste, "Lookup succeeded, node UUID is 6753779f-82a6-461c-bfb7-2e320da749ca" | 10:40 |
| pas-ha | morning mgould jroll sambetts | 10:41 |
| *** thrash|g0ne is now known as thrash | 10:42 | |
| tuanla_ | pas-ha, Thanks you | 10:44 |
| tuanla_ | I have been find out why i have problem | 10:44 |
| tuanla_ | in my BIOS boot setup | 10:44 |
| tuanla_ | pas-ha, recently i can deploy whole-disk-image cirros | 10:44 |
| tuanla_ | hi pas-ha Have you ever try to deploy window image using ironic | 10:45 |
| tuanla_ | hi sambetts.I have been finded out why i have problem in reboot whole-disk-image. In my server i didn't set up priority for local boot but using whole-disk-image will using local boot instead of network boot. If you have free please take time to lookat log bug http://paste.openstack.org/show/580941/ | 10:48 |
| sambetts | tuanla_: what driver are you using ? | 10:50 |
| tuanla_ | i was using pxe_irmc and agent_ipmitool | 10:51 |
| tuanla_ | hi sambetts, the log in agent_ipmitool . good to here you void | 10:51 |
| sambetts | tuanla_: which driver were you using for this particualr node that failed? | 10:52 |
| tuanla_ | sambetts, I was using agent_ipmitool | 10:52 |
| tuanla_ | hi sambetts, as pas-ha said, it just version not fix, I also deploy successfull | 10:53 |
| sambetts | tuanla_: Ironic should set the boot order for you via IPMI | 10:54 |
| sambetts | tuanla_: you shouldn't have to do it manaully | 10:54 |
| tuanla_ | sambetts: How can i make sure that i deploy sucessfull because i just seen it in horizon and ironic node-show | 10:56 |
| sambetts | tuanla_: we need to work out what is causing the machine to not set the correct boot device | 10:59 |
| *** anush has quit IRC | 11:00 | |
| *** yarkot1 has quit IRC | 11:00 | |
| *** anush has joined #openstack-ironic | 11:01 | |
| *** dprince has quit IRC | 11:01 | |
| *** anush has quit IRC | 11:02 | |
| *** anush has joined #openstack-ironic | 11:03 | |
| *** lucas-dinner is now known as lucasagomes | 11:03 | |
| lucasagomes | morning all :-) | 11:04 |
| tuanla_ | sambetts, in my case the horizon show that node is active. and whole disk image boot from local disk | 11:04 |
| *** aslezil has quit IRC | 11:04 | |
| *** mpotdar has quit IRC | 11:04 | |
| *** vgadiraj has quit IRC | 11:04 | |
| *** yarkot has quit IRC | 11:04 | |
| *** anush_ has quit IRC | 11:04 | |
| *** anush_ has joined #openstack-ironic | 11:04 | |
| sambetts | That all sounds correct to me | 11:06 |
| tuanla_ | morning lucasagomes | 11:06 |
| tuanla_ | in vietnam now time to go home. see you guide later. Thanks for time support me sambetts lucasagomes pas-ha | 11:07 |
| *** tuanla_ has quit IRC | 11:07 | |
| lucasagomes | see ya | 11:08 |
| *** makowals has quit IRC | 11:08 | |
| *** makowals has joined #openstack-ironic | 11:09 | |
| *** mpotdar has joined #openstack-ironic | 11:10 | |
| vdrok | morning jroll sambetts and lucasagomes | 11:11 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Fix iLO drivers to not clear local_gb if its not detected https://review.openstack.org/341752 | 11:18 |
| *** yarkot has joined #openstack-ironic | 11:20 | |
| *** yarkot1 has joined #openstack-ironic | 11:20 | |
| *** milan has joined #openstack-ironic | 11:20 | |
| *** pall_ has quit IRC | 11:22 | |
| *** hoonetorg has quit IRC | 11:23 | |
| milan | morning Ironic! :) | 11:23 |
| *** stendulker has quit IRC | 11:25 | |
| milan | sambetts, aarefiev guys, would you mind a review here: https://review.openstack.org/#/c/344921/ not touching any critical paths ;) | 11:25 |
| patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 11:25 |
| sambetts | milan: I'll take a look when I get a minute :)( | 11:25 |
| sambetts | :) | 11:25 |
| milan | sambetts, thx! | 11:26 |
| *** aslezil has joined #openstack-ironic | 11:26 | |
| vdrok | morning milan | 11:28 |
| milan | morning vdrok ! :) | 11:28 |
| *** ccamacho is now known as ccamacho|lunch | 11:31 | |
| *** makowals has quit IRC | 11:32 | |
| *** ijw has joined #openstack-ironic | 11:32 | |
| *** makowals has joined #openstack-ironic | 11:34 | |
| *** hshiina has joined #openstack-ironic | 11:36 | |
| *** makowals_ has joined #openstack-ironic | 11:37 | |
| *** ijw has quit IRC | 11:37 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 11:38 |
| *** makowals has quit IRC | 11:38 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Grub2 by default for PXE + UEFI https://review.openstack.org/367306 | 11:39 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Refactor common checks when instantiating the ipmitool classes https://review.openstack.org/346896 | 11:39 |
| *** sdake has joined #openstack-ironic | 11:49 | |
| *** vgadiraj has joined #openstack-ironic | 11:51 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification https://review.openstack.org/348437 | 11:52 |
| *** hoonetorg has joined #openstack-ironic | 11:56 | |
| *** dprince has joined #openstack-ironic | 11:57 | |
| openstackgerrit | Merged openstack/ironic: Remove NotificationEventTypeError as not needed https://review.openstack.org/371810 | 11:57 |
| *** hoonetorg has quit IRC | 11:57 | |
| *** hoonetorg has joined #openstack-ironic | 11:58 | |
| openstackgerrit | Merged openstack/ironic: Separate WSGIService from RPCService https://review.openstack.org/369724 | 11:59 |
| openstackgerrit | Merged openstack/ironic: [trivial] Fix typo in docstring https://review.openstack.org/371486 | 11:59 |
| *** trown|outtypewww is now known as trown | 12:03 | |
| *** hshiina has quit IRC | 12:07 | |
| aarefiev | milan: hey, done | 12:10 |
| aarefiev | marker doesn't work for me | 12:10 |
| *** sdake has quit IRC | 12:13 | |
| *** sdake has joined #openstack-ironic | 12:14 | |
| *** baoli has joined #openstack-ironic | 12:16 | |
| *** baoli has quit IRC | 12:16 | |
| *** baoli has joined #openstack-ironic | 12:16 | |
| *** baoli_ has joined #openstack-ironic | 12:17 | |
| *** vikrant has quit IRC | 12:19 | |
| *** baoli has quit IRC | 12:21 | |
| *** vsaienko has joined #openstack-ironic | 12:28 | |
| *** KrisK19 has quit IRC | 12:28 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Prepare for functional tests https://review.openstack.org/349421 | 12:29 |
| *** hshiina has joined #openstack-ironic | 12:29 | |
| *** jcoufal has joined #openstack-ironic | 12:32 | |
| milan | aarefiev, thx, me looks | 12:33 |
| sambetts | milan: QQ about that API patch, as far as I can tell we don't reset the finished value when we do reapply, so you could have that API return a node as "finished" that is being worked on by reapply | 12:33 |
| *** ijw has joined #openstack-ironic | 12:33 | |
| milan | sambetts, you mean to get the list of finished nodes, one might abuse the reapply part? | 12:34 |
| milan | :) | 12:34 |
| *** e0ne has quit IRC | 12:35 | |
| *** e0ne has joined #openstack-ironic | 12:35 | |
| sambetts | milan: no, what I mean is that if I inspect a node succesfully, then do reapply on it, it'll still return as finished because we don't clear the finished field | 12:36 |
| milan | sambetts, ah, yeah, we don't reset that flag | 12:36 |
| milan | now I get it | 12:37 |
| *** ccamacho|lunch is now known as ccamacho | 12:38 | |
| milan | sambetts, are you suggesting to fix that behaviour in this patch? | 12:38 |
| *** david-lyle has joined #openstack-ironic | 12:38 | |
| *** ijw has quit IRC | 12:39 | |
| milan | or should it be a follow-up? | 12:39 |
| sambetts | milan: So I think there are 2 things that need changing here, we need to add a patch to set the started_at value when you do reapply, and I think this patch needs to check if started_at is after finished_at | 12:40 |
| milan | sambetts, yeah, that might work indeed | 12:41 |
| *** david-lyle has quit IRC | 12:43 | |
| milan | sambetts, also the states patch might solve this, but would have to filter on the state value instead of the presence/value of the finished_at vs started_at | 12:43 |
| milan | I thought about actually renaming finished query to state query | 12:43 |
| *** leduardo_ has joined #openstack-ironic | 12:44 | |
| sambetts | milan: yup, I would much prefer if we added a state = query | 12:44 |
| *** leduardo has quit IRC | 12:44 | |
| milan | sambetts, aarefiev wdyt ^ less work later on | 12:44 |
| sambetts | milan: and if we add this API we have to support it even if we change to the states one | 12:44 |
| milan | yeah | 12:45 |
| openstackgerrit | Merged openstack/ironic-ui: Improve the display of node information https://review.openstack.org/361226 | 12:45 |
| milan | so let's rename finished=True/False/None to state in {finished, waiting, processing, reapplying, error}? I think we can infer all of those even right now | 12:46 |
| milan | or -2 till we land the states patch? | 12:47 |
| milan | sambetts, aarefiev ^ wdyt? | 12:47 |
| aarefiev | For me it looks like separate patch, it's bug | 12:48 |
| milan | aarefiev, +1 on bug | 12:48 |
| sambetts | the reapply thing totally is a bug | 12:48 |
| aarefiev | as for finished = {finished, waiting, processing, reapplying, error} I'd wait until state patch | 12:48 |
| sambetts | but I'm not super sold on adding this API based on that we are adding the states patch | 12:49 |
| sambetts | I don't want to have to support an extra API if we don't need too | 12:49 |
| milan | I'd prefer API call with a state query, states patch would have to replace the bits with select where state == state query | 12:51 |
| sambetts | ++ | 12:51 |
| milan | matter of the node_cache.get_node_list function | 12:52 |
| sambetts | I think I'd like to have a finished_at/started_at >/</= <a time value> API too | 12:53 |
| sambetts | then you could do thinks like, get node state == REAPPLY and started_at > 1hr ago | 12:53 |
| sambetts | or similar | 12:54 |
| milan | sambetts, that sounds reasonable | 12:54 |
| *** jheroux has joined #openstack-ironic | 12:54 | |
| milan | I thought the same | 12:54 |
| aarefiev | yeah, but it's separate feature | 12:54 |
| milan | just not sure about doing it right now (would grow in number of tests) | 12:54 |
| milan | on the other hand, we could just rely on the finished_at & started_at query (instead of state) and add the state part later? | 12:55 |
| milan | I think it would be less rewriting | 12:55 |
| sambetts | yeah, I wonder if that the right thing to do now, and then add the states when we actually have states :-P | 12:55 |
| milan | sambetts, exactly | 12:55 |
| milan | so to list finished only or unfinished only would be possible, just not sure how to encode the "null/None" thing in the API ;) | 12:56 |
| aarefiev | yeah, we can file a rfe bug now with this | 12:56 |
| milan | but I guess I'd figure that out eventually ;) | 12:56 |
| sambetts | :-P | 12:58 |
| milan | aarefiev, sambetts so lets file: a) bug with the started_at not being updated during the reapply, b) [RFE] to support state query in the list API c) rewrite this patch to filter on started_at finished_at combination rather than finished=[true/false/none] | 12:58 |
| sambetts | ++ makes sense to me | 12:58 |
| aarefiev | sounds good | 12:58 |
| milan | ack, seems we've got a plan :D | 12:59 |
| * milan flies bugs | 12:59 | |
| openstackgerrit | Sofiia Andriichenko proposed openstack/ironic: Add tests for testing Ironic notifications, CRUD operations for node, chassis, port https://review.openstack.org/372466 | 12:59 |
| milan | actually, sambetts you should file the started_at as you've found it and I'd like not to take your credit :P | 12:59 |
| sambetts | haha sure :-P | 13:00 |
| *** rloo has joined #openstack-ironic | 13:01 | |
| *** srobert has joined #openstack-ironic | 13:02 | |
| sambetts | milan, aarefiev: https://bugs.launchpad.net/ironic-inspector/+bug/1625180 | 13:03 |
| openstack | Launchpad bug 1625180 in Ironic Inspector "Reapply doesn't update started_at time" [Undecided,New] | 13:03 |
| milan | sambetts, thx! | 13:03 |
| *** vishwanathj has joined #openstack-ironic | 13:04 | |
| *** cdaniel has joined #openstack-ironic | 13:04 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: [WIP] Test ML2 Fail-Fast feature https://review.openstack.org/370016 | 13:05 |
| openstackgerrit | Sofiia Andriichenko proposed openstack/ironic: Add tests for testing Ironic notifications, CRUD operations for node, chassis, port https://review.openstack.org/372474 | 13:07 |
| *** moshele has quit IRC | 13:08 | |
| *** jaosorior has quit IRC | 13:09 | |
| *** jaosorior has joined #openstack-ironic | 13:10 | |
| milan | sambetts, aarefiev: https://bugs.launchpad.net/ironic-inspector/+bug/1625183 | 13:11 |
| openstack | Launchpad bug 1625183 in Ironic Inspector "[RFE] add a state query to the list introspection statuses API endpoint" [Undecided,New] | 13:11 |
| openstackgerrit | Bharath kumar proposed openstack/ironic-python-agent: Moving Reboot bashscript to python https://review.openstack.org/372478 | 13:11 |
| openstackgerrit | Sofiia Andriichenko proposed openstack/ironic: Add tests for testing Ironic notifications, CRUD operations for node, chassis, port https://review.openstack.org/372479 | 13:11 |
| *** e0ne has quit IRC | 13:12 | |
| *** lennyb has quit IRC | 13:12 | |
| *** lennyb_ has quit IRC | 13:12 | |
| *** lennyb has joined #openstack-ironic | 13:13 | |
| *** ppiela has joined #openstack-ironic | 13:17 | |
| *** bnemec has joined #openstack-ironic | 13:19 | |
| *** cdearborn has joined #openstack-ironic | 13:20 | |
| sambetts | milan: awesome :) | 13:21 |
| milan | :) | 13:21 |
| *** chihhsin has quit IRC | 13:23 | |
| *** moshele has joined #openstack-ironic | 13:23 | |
| *** makowals_ has quit IRC | 13:27 | |
| *** KrisK19 has joined #openstack-ironic | 13:29 | |
| *** makowals has joined #openstack-ironic | 13:30 | |
| *** KrisK19_ has joined #openstack-ironic | 13:31 | |
| *** ijw has joined #openstack-ironic | 13:35 | |
| *** KrisK19 has quit IRC | 13:36 | |
| *** KrisK19_ has quit IRC | 13:36 | |
| *** mkam has joined #openstack-ironic | 13:39 | |
| *** hshiina has quit IRC | 13:39 | |
| *** ijw has quit IRC | 13:41 | |
| *** joprovost has joined #openstack-ironic | 13:45 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Hide 'ports' field from node OSC subcommands output https://review.openstack.org/366703 | 13:46 |
| milan | aarefiev, have you got some details about the marker failing for you in the inspector's list api? | 13:47 |
| aarefiev | milan: sorry, not failing, just returns all list | 13:48 |
| openstackgerrit | Merged openstack/pyghmi: Improve SOL responsiveness https://review.openstack.org/371787 | 13:49 |
| milan | aarefiev, please note the order is descending so last is first | 13:49 |
| *** cdearborn has quit IRC | 13:49 | |
| aarefiev | milan: yeah, even with one element, does it work well for you? | 13:50 |
| milan | hmm | 13:50 |
| milan | I've tried in the func test only | 13:50 |
| milan | but in that case, for me it's 5 statuses | 13:50 |
| milan | and correctly gives 1 | 13:50 |
| *** mkam has quit IRC | 13:51 | |
| milan | I'll try in devstack :) | 13:51 |
| aarefiev | hmm, I'll deploy new env, I'll try on it too | 13:52 |
| *** zackf has joined #openstack-ironic | 13:52 | |
| *** cdearborn has joined #openstack-ironic | 13:52 | |
| milan | could be the sanitisation of the null->+infinity as I'm not messing with anything else | 13:52 |
| * milan hacks func tests | 13:52 | |
| *** causten_ has joined #openstack-ironic | 13:53 | |
| *** Goneri has joined #openstack-ironic | 13:53 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Hide 'nodes' field from chassis OSC subcommands output https://review.openstack.org/366731 | 13:54 |
| *** moshele has quit IRC | 13:54 | |
| *** moshele has joined #openstack-ironic | 13:55 | |
| *** moshele has quit IRC | 13:56 | |
| *** moshele has joined #openstack-ironic | 14:00 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 14:01 |
| *** mkam has joined #openstack-ironic | 14:04 | |
| milan | aarefiev, yeah it's broken somehow (gives me a wrong item (finished: true although it should not be finished at that time)) didn't notice before :-/ | 14:07 |
| aarefiev | :( | 14:08 |
| *** chihhsin has joined #openstack-ironic | 14:08 | |
| milan | one sec, maybe I'm using wrong value | 14:08 |
| milan | aarefiev, sorry I got confused by my own code O:-) so in the test case it seems OK: using self.uuid as a marker and a limit == 1 gives a single-item list with a different uuid | 14:12 |
| milan | so that part seems OK | 14:12 |
| milan | I'll try to sort the list manually and play with the marker a bit | 14:12 |
| aarefiev | milan: ok, thanks | 14:12 |
| *** ElCoyote_ has quit IRC | 14:13 | |
| *** chihhsin_ has joined #openstack-ironic | 14:13 | |
| *** ElCoyote_ has joined #openstack-ironic | 14:13 | |
| *** chihhsin has quit IRC | 14:16 | |
| *** chihhsin_ has quit IRC | 14:19 | |
| *** chihhsin has joined #openstack-ironic | 14:19 | |
| *** wznoinsk has quit IRC | 14:20 | |
| openstackgerrit | Beth Elwell proposed openstack/ironic-ui: Updated to ironic API v1.11 and added manageable state https://review.openstack.org/361098 | 14:25 |
| pas-ha | hi all, WDYT re running bashate on IPA code? do we need its bash scripts to be compliant with bashate, or it is meant just for DevStack? | 14:27 |
| *** wznoinsk has joined #openstack-ironic | 14:27 | |
| pas-ha | there are some bash scripts both in code and lots in imagebuild | 14:27 |
| *** moshele has quit IRC | 14:29 | |
| lucasagomes | pas-ha, I think it's good, but I think it would be even better to convert those scripts to python actually | 14:30 |
| pas-ha | those in IPA code - probably, although I'm not sure that's going to be better, as it most probably will be _less_ readable | 14:31 |
| pas-ha | those in imagebuild - I doubt it makes sense to rewrite them in Py | 14:32 |
| *** strigazi has quit IRC | 14:33 | |
| lucasagomes | pas-ha, true, yeah for those I think bashate is a good thing | 14:33 |
| pas-ha | lucasagomes: btw, there's a pretty cool lib with Py interface for parted, created by RedHat - https://github.com/rhinstaller/pyparted | 14:33 |
| lucasagomes | we can hook the bashate execution in the pep8 job | 14:33 |
| lucasagomes | as we do in ironic | 14:33 |
| pas-ha | would you happen to know why it is not on PyPI | 14:34 |
| pas-ha | yes, filing a wishlist bug for that now :) | 14:34 |
| lucasagomes | pas-ha, nice! I have not, but I can try to find out | 14:34 |
| lucasagomes | pas-ha, one thing about pyparted is that you have to run as root | 14:34 |
| lucasagomes | AFAIR, I used it long long time ago here: https://github.com/umago/carbono/ | 14:34 |
| *** ElCoyote_ has quit IRC | 14:35 | |
| *** strigazi has joined #openstack-ironic | 14:35 | |
| pas-ha | lucasagomes: https://bugs.launchpad.net/ironic-python-agent/+bug/1625215 please put to wishlist | 14:35 |
| openstack | Launchpad bug 1625215 in ironic-python-agent "need to check style for bash scripts too" [Undecided,New] | 14:35 |
| *** syed_ has joined #openstack-ironic | 14:36 | |
| lucasagomes | pas-ha, anaconda also use a pyhton library called blivet to manage the partitioning and things like that https://github.com/rhinstaller/blivet | 14:36 |
| lucasagomes | or https://fedoraproject.org/wiki/Blivet | 14:36 |
| *** rbartal has quit IRC | 14:36 | |
| *** ijw has joined #openstack-ironic | 14:36 | |
| lucasagomes | pas-ha, cool, ty for that bug | 14:36 |
| *** openstackgerrit has quit IRC | 14:37 | |
| *** openstackgerrit has joined #openstack-ironic | 14:37 | |
| mat128 | good morning Ironic | 14:40 |
| *** david-lyle has joined #openstack-ironic | 14:40 | |
| mgould | morning mat128 | 14:41 |
| vdrok | morning mat128 and mgould | 14:42 |
| *** ijw has quit IRC | 14:42 | |
| lucasagomes | mat128, morning | 14:42 |
| mgould | vdrok: morning | 14:44 |
| milan | morning mat128 morning pas-ha! :) | 14:45 |
| *** Goneri has quit IRC | 14:45 | |
| *** david-lyle has quit IRC | 14:45 | |
| pas-ha | morning milan mat128 | 14:49 |
| *** Goneri has joined #openstack-ironic | 14:55 | |
| aarefiev | sambetts, take a look, please, when you have time https://review.openstack.org/#/c/369261/7 | 15:00 |
| patchbot | patch 369261 - ironic-inspector - Tempest: add auto-discovery test | 15:00 |
| *** narasimha_SV has quit IRC | 15:01 | |
| *** bdeschenes has joined #openstack-ironic | 15:01 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 15:04 |
| *** bnemec has quit IRC | 15:04 | |
| *** mjura has quit IRC | 15:05 | |
| *** ijw has joined #openstack-ironic | 15:06 | |
| *** rajinir has joined #openstack-ironic | 15:06 | |
| *** bnemec has joined #openstack-ironic | 15:07 | |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic-specs: WIP: Add VCA hardware type to Ironic https://review.openstack.org/369309 | 15:07 |
| *** narasimha_SV has joined #openstack-ironic | 15:07 | |
| *** pcaruana has quit IRC | 15:07 | |
| *** makowals has quit IRC | 15:09 | |
| jlvillal | Good morning Ironic :) | 15:10 |
| *** mgoddard_ has joined #openstack-ironic | 15:11 | |
| *** mgoddard has quit IRC | 15:11 | |
| *** srobert has quit IRC | 15:11 | |
| *** makowals has joined #openstack-ironic | 15:11 | |
| *** makowals has quit IRC | 15:14 | |
| narasimha_SV | do we need to create port if we use an existing dhcp server rather than ironic ? | 15:14 |
| narasimha_SV | rather than neutron | 15:14 |
| *** makowals has joined #openstack-ironic | 15:15 | |
| *** hshiina has joined #openstack-ironic | 15:18 | |
| *** hpe-hj has joined #openstack-ironic | 15:20 | |
| *** hj-hpe has quit IRC | 15:22 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Serve iPXE configs from Ironic API https://review.openstack.org/369438 | 15:22 |
| pas-ha | lucasagomes: ^ | 15:23 |
| pas-ha | that's dynamic iPXE config | 15:23 |
| pas-ha | for now no unit tests, and most probably will split in several patches. for now would like to have an agreement on implementation path taken | 15:24 |
| pas-ha | especially re manager/drivers part | 15:24 |
| pas-ha | morning jlvillal | 15:24 |
| jlvillal | pas-ha: :) | 15:24 |
| lucasagomes | pas-ha, awesome I will put in my list (is in a meeting | 15:25 |
| pas-ha | lucasagomes: thanks, no problem | 15:25 |
| pas-ha | just wanted to get some of your attention :) | 15:25 |
| pas-ha | and usual disclaimer "works on my local DevStack" :) | 15:26 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Notification event types have status 'error' https://review.openstack.org/372578 | 15:27 |
| * pas-ha for now tries not to think how to rebase that on patch that fetches deploy kernel/ramdisk from Swift, that might be one tough merge conflict | 15:28 | |
| *** moshele has joined #openstack-ironic | 15:28 | |
| *** BadCub has joined #openstack-ironic | 15:28 | |
| *** rcernin has quit IRC | 15:30 | |
| vdrok | morning jlvillal | 15:30 |
| jlvillal | vdrok: :) | 15:31 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: [WIP] Test ML2 Fail-Fast feature https://review.openstack.org/370016 | 15:32 |
| *** david-lyle has joined #openstack-ironic | 15:32 | |
| *** Romanenko_K has quit IRC | 15:42 | |
| *** harlowja_at_home has joined #openstack-ironic | 15:48 | |
| rloo | hi and morning everyone, vdrok, jlvillal, pas-ha | 15:49 |
| *** MattMan has quit IRC | 15:49 | |
| *** MattMan has joined #openstack-ironic | 15:49 | |
| rloo | jlvillal: wrt https://review.openstack.org/#/c/366518/11/doc/source/dev/dev-quickstart.rst, i'm not quite sure what the problem is. | 15:49 |
| patchbot | patch 366518 - ironic - Docs: Add warning and add setting of $LOGFILE | 15:49 |
| rloo | jlvillal: why do we need to put a warning, the doc already had LOGDIR set | 15:49 |
| vdrok | morning rloo | 15:50 |
| rloo | jlvillal: i mean, to some extent, we could put a warning for other lines to say if this isn't set, things won't work. | 15:50 |
| jlvillal | rloo: I don't really know... | 15:51 |
| jlvillal | rloo: I have never really liked the patch. | 15:51 |
| jlvillal | rloo: It seems like the person didn't set LOGDIR, and destroyed their home directory. | 15:52 |
| openstackgerrit | Merged openstack/ironic: Use assertEqual() instead of assertDictEqual() https://review.openstack.org/368700 | 15:52 |
| jlvillal | rloo: So then their solution was to add setting LOGFILE. | 15:52 |
| jlvillal | rloo: But I'm thinking if a person didn't set LOGDIR as the docs say, why are they going to set LOGFILE? | 15:52 |
| rloo | jlvillal: is the problem LOGDIR or LOGFILE? i don't understand, so I -1'd. | 15:53 |
| jlvillal | rloo: if LOGDIR is set, they will be fine. From what I understand. If LOGDIR is not set then it will set LOGDIR based on the value of LOGFILE's directory. | 15:54 |
| *** Sukhdev has joined #openstack-ironic | 15:54 | |
| rloo | jlvillal: well, we already had LOGDIR set in that example thing so ... | 15:54 |
| jlvillal | rloo: Yep... | 15:54 |
| rloo | jlvillal: i'm going on to the next patch. gotta be efficient. ha ha. | 15:54 |
| jlvillal | rloo: Agreed. I don't think that patch is fixing anything. Well they have yet to convince me it is. | 15:55 |
| rloo | jlvillal: really, if they had copy/pasted what was there, they wouldn't have had the problem. and if they don't copy/paste, would they see the WARNING thing. | 15:55 |
| jlvillal | rloo: I know. :( | 15:56 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Enable SSH access to tinyipa https://review.openstack.org/371775 | 15:59 |
| *** athomas has quit IRC | 15:59 | |
| *** vsaienko has quit IRC | 15:59 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Enable SSH access to tinyipa https://review.openstack.org/371775 | 16:03 |
| *** dprince has quit IRC | 16:03 | |
| *** athomas has joined #openstack-ironic | 16:03 | |
| *** vsaienko has joined #openstack-ironic | 16:07 | |
| *** lucasagomes is now known as lucas-hungry | 16:08 | |
| lucas-hungry | rloo, jlvillal morning :-) | 16:09 |
| jlvillal | lucas-hungry: Oi! | 16:09 |
| * jlvillal is not sure if that is how it is spelled | 16:09 | |
| rloo | hi lucas-hungry! | 16:10 |
| rloo | lucas-hungry: is dtantsur|pto away the whole week? | 16:10 |
| rloo | lucas-hungry: found out, he's away today and tomorrow :) | 16:11 |
| *** rpioso has joined #openstack-ironic | 16:11 | |
| *** stendulker has joined #openstack-ironic | 16:12 | |
| *** ohamada has quit IRC | 16:16 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Local boot with partition images" https://review.openstack.org/369776 | 16:17 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Flavor creation" https://review.openstack.org/369761 | 16:17 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Enrollment" and "Troubleshooting" sections https://review.openstack.org/369777 | 16:17 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Add link to new guide in old install guide https://review.openstack.org/369802 | 16:17 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import Install and configure sections https://review.openstack.org/367988 | 16:17 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Building or downloading a deploy ramdisk image" https://review.openstack.org/369800 | 16:17 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Trusted boot with partition image" https://review.openstack.org/369801 | 16:17 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Bare Metal service overview" https://review.openstack.org/366329 | 16:17 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import configdrive https://review.openstack.org/369798 | 16:17 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Image requirements" https://review.openstack.org/369754 | 16:17 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Appending kernel parameters to boot instances" https://review.openstack.org/369799 | 16:17 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "integration with other OpenStack components" https://review.openstack.org/369510 | 16:17 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import HTTPS, standalone and root device hints https://review.openstack.org/369797 | 16:17 |
| *** e0ne has joined #openstack-ironic | 16:17 | |
| mat128 | lmao ^ | 16:17 |
| mat128 | sorry :( | 16:17 |
| mat128 | ready for review :) | 16:17 |
| *** anush__ has quit IRC | 16:22 | |
| *** anush__ has joined #openstack-ironic | 16:22 | |
| sambetts | mat128: nice! | 16:23 |
| openstackgerrit | Sergii Turivnyi proposed openstack/ironic: Add security groups tests https://review.openstack.org/370226 | 16:23 |
| *** Sukhdev has quit IRC | 16:23 | |
| *** rama_y has joined #openstack-ironic | 16:26 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic: Add description for vendor passthru methods https://review.openstack.org/370469 | 16:26 |
| mat128 | sambetts: heh, yeah | 16:27 |
| mat128 | small-ish commits will do that for you | 16:27 |
| mat128 | but it's easier to review | 16:28 |
| *** dprince has joined #openstack-ironic | 16:28 | |
| *** Sukhdev has joined #openstack-ironic | 16:28 | |
| NobodyCam | Good Morning Ironic'ers | 16:31 |
| mat128 | hello NobodyCam | 16:31 |
| NobodyCam | morning mat128 :) | 16:31 |
| *** vsaienko has quit IRC | 16:32 | |
| *** causten_ has quit IRC | 16:33 | |
| *** Sukhdev has quit IRC | 16:33 | |
| *** harlowja_at_home has quit IRC | 16:33 | |
| *** narasimha_SV has quit IRC | 16:33 | |
| openstackgerrit | Alexander Bashmakov proposed openstack/ironic: Rewrite devstack/tools/ironic/scripts/create-node.sh in Python. https://review.openstack.org/346832 | 16:33 |
| *** mkam has quit IRC | 16:34 | |
| *** Sukhdev has joined #openstack-ironic | 16:35 | |
| *** Sukhdev has quit IRC | 16:35 | |
| mgould | NobodyCam: morning! | 16:35 |
| NobodyCam | morning mgould :) | 16:35 |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Refactor env-setup.sh https://review.openstack.org/371883 | 16:37 |
| *** krtaylor has quit IRC | 16:38 | |
| *** Sukhdev has joined #openstack-ironic | 16:38 | |
| stendulker | morning NobodyCam :) | 16:38 |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Workaround openSUSE ansible_os_family issue https://review.openstack.org/371082 | 16:39 |
| NobodyCam | hey hey stendulker Good morning :) | 16:40 |
| *** Sukhdev has quit IRC | 16:43 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Install process SuSE support https://review.openstack.org/367637 | 16:44 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Change 'decom' to clean/cleaning https://review.openstack.org/372625 | 16:44 |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: SuSE support for VM node creation https://review.openstack.org/370385 | 16:44 |
| *** jpich has quit IRC | 16:47 | |
| openstackgerrit | Merged openstack/ironic: Allow using TempURLs for deploy images https://review.openstack.org/344160 | 16:48 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add CRUD notifications for ironic resources https://review.openstack.org/356541 | 16:48 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 16:48 |
| *** ChubYann has joined #openstack-ironic | 16:49 | |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 16:49 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 16:49 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_targets table to save target information https://review.openstack.org/285218 | 16:49 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce VolumeTarget object https://review.openstack.org/285219 | 16:49 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume target operations https://review.openstack.org/285220 | 16:49 |
| hshiina | hi, we're changing rpc signature in https://review.openstack.org/#/c/216730/ | 16:50 |
| patchbot | patch 216730 - ironic - Generic power interface for soft reboot and soft p... | 16:50 |
| jroll | hey y'all | 16:50 |
| jroll | anybody want to run the meeting for me? lucas-hungry devananda | 16:50 |
| hshiina | i'm not sure if we should use can_send_version() mentioned in http://docs.openstack.org/developer/ironic/dev/code-contribution-guide.html#ironic-api-rpc-versions | 16:51 |
| *** rcernin has joined #openstack-ironic | 16:51 | |
| hshiina | it seems that rpc ping has not been introduced yet and most changes don't use can_send_version() | 16:51 |
| hshiina | any advice would be appreciated. | 16:52 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification https://review.openstack.org/348437 | 16:52 |
| *** e0ne has quit IRC | 16:55 | |
| *** anush has quit IRC | 16:57 | |
| *** yarkot1 has quit IRC | 16:57 | |
| jroll | rloo: NobodyCam: how bout y'all, want to run the meeting? | 16:57 |
| *** lucas-hungry is now known as lucasagomes | 16:57 | |
| lucasagomes | rloo, yeah he will be back on wed :-) | 16:58 |
| *** e0ne has joined #openstack-ironic | 16:58 | |
| jroll | oh there's lucas :D | 16:58 |
| NobodyCam | Lol | 16:58 |
| vdrok | morning NobodyCam and stendulker | 16:58 |
| lucasagomes | jroll, hi there, sorry I was having lunch | 16:58 |
| lucasagomes | jroll, I'm on a diff timezone atm (brazil, UTC-3) | 16:58 |
| NobodyCam | Morning jroll vdrok lucasagomes | 16:58 |
| *** anush has joined #openstack-ironic | 16:59 | |
| rloo | hi jroll. I think lucasagomes or NobodyCam will volunteer? :) | 16:59 |
| lucasagomes | rloo, yeah I'll be complete unprepared but I can run | 16:59 |
| lucasagomes | rloo, wanna chair it also ? :D | 16:59 |
| rloo | lucasagomes: i'm sitting on a chair. that's sufficient, isn't it? ;) | 17:00 |
| jroll | ty lucasagomes | 17:00 |
| * jroll brb | 17:00 | |
| *** anush has quit IRC | 17:00 | |
| lucasagomes | jroll, yw | 17:00 |
| rloo | lucasagomes: i've got some ideas that we can push through w/o jroll there. shhh... j/k | 17:00 |
| lucasagomes | rloo, hah fair enough | 17:00 |
| stendulker | morning vdrok | 17:01 |
| *** rama_y has quit IRC | 17:02 | |
| *** derekh has quit IRC | 17:02 | |
| *** rama_y has joined #openstack-ironic | 17:04 | |
| NobodyCam | morning rloo | 17:05 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add CRUD notifications for ironic resources https://review.openstack.org/356541 | 17:05 |
| rloo | morning NobodyCam! | 17:06 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Timing metrics: iRMC drivers https://review.openstack.org/371284 | 17:17 |
| *** trown is now known as trown|lunch | 17:21 | |
| * jlvillal is seeing reviews from dtantsur|pto ... | 17:23 | |
| openstackgerrit | Merged openstack/ironic: Fix docstrings in Tempest Plugin REST client for Ironic API https://review.openstack.org/370090 | 17:25 |
| *** vsaienko has joined #openstack-ironic | 17:26 | |
| *** harlowja has joined #openstack-ironic | 17:30 | |
| openstackgerrit | Merged openstack/ironic: Fix test syntax error in devstack/lib/ironic https://review.openstack.org/370482 | 17:31 |
| *** mgould is now known as mgould|afk | 17:32 | |
| *** vsaienko has quit IRC | 17:34 | |
| *** jaosorior has quit IRC | 17:36 | |
| *** ijw has quit IRC | 17:38 | |
| *** vsaienko has joined #openstack-ironic | 17:41 | |
| *** rbrud has quit IRC | 17:41 | |
| *** moshele has quit IRC | 17:44 | |
| *** rama_y has quit IRC | 17:46 | |
| *** hshiina has quit IRC | 17:46 | |
| lucasagomes | it's a strange feeling to not go for dinner after the upstream meeting on monday | 17:56 |
| jroll | heh | 17:56 |
| lucasagomes | 15pm here still | 17:56 |
| jlvillal | lucasagomes: I'm sure you will adapt ;) | 17:56 |
| jroll | lucasagomes: siesta! | 17:56 |
| lucasagomes | jroll, lol that would be great | 17:57 |
| jlvillal | lucasagomes: Probably fun to have a bunch of people around who speak Portuguese again | 17:57 |
| lucasagomes | jlvillal, yeah :-) I arrived yesterday so I'm still jetlagged and all | 17:57 |
| vdrok | good night everyone! | 17:57 |
| lucasagomes | vdrok, have a good night | 17:57 |
| jlvillal | vdrok: Ciao and welcome back! | 17:57 |
| lucasagomes | jlvillal, yeah, dublin is pretty global nowadays so you hear many languages on the street | 17:57 |
| lucasagomes | jlvillal, here, is very country side so, I only hear portuguese everywhere... | 17:58 |
| jlvillal | lucasagomes: I wonder if they think you have an accent know | 17:59 |
| jlvillal | s/know/now/ | 17:59 |
| jlvillal | lucasagomes: My wife's son thought she had gotten a little bit of an accent after being in the US :) | 17:59 |
| lucasagomes | jlvillal, oh, that's nice actually | 18:00 |
| lucasagomes | I think I'm good, I just don't know all the slangs people are using | 18:01 |
| lucasagomes | or memes they are talking about | 18:01 |
| jlvillal | lucasagomes: I'm sure you'll catch up quick. I hope you have fun there! | 18:01 |
| lucasagomes | cheers | 18:01 |
| rloo | lucasagomes: wrt that bug https://bugs.launchpad.net/ironic/+bug/1427923; i'm not sure we need to take to the ML. i think we agree it should be async. i think talking to dmitry about it would be sufficient. | 18:03 |
| openstack | Launchpad bug 1427923 in Ironic "boot device API blocks while waiting on the BMC" [Medium,Confirmed] - Assigned to bin Yu (froyo-bin) | 18:03 |
| rloo | mat128: is https://review.openstack.org/#/q/topic:bug/1612278 complete? or are there going to be more patches to get it done? (whatever done means) | 18:04 |
| mat128 | we are still missing JayF's patch for driver info, but it is mergeable | 18:04 |
| mat128 | it moves content, section per section | 18:05 |
| mat128 | s/it/the commits | 18:05 |
| rloo | mat128: ok, so less than 20 patches then. :D | 18:05 |
| rloo | s/less/fewer/ | 18:05 |
| mat128 | only 1 left, the drivers | 18:05 |
| mat128 | oh yes | 18:05 |
| rloo | mat128: will try to start later today | 18:05 |
| mat128 | I aimed for "ease of review" | 18:05 |
| mat128 | by cutting it up like that | 18:05 |
| rloo | mat128: i hate it when people say that they have an easy review. seems like 50% of the time, it ends up taking me more than an hour. will see about yours :) | 18:06 |
| mat128 | haha, I said I aim for ease of review, not that they are easy to review | 18:06 |
| mat128 | :) | 18:06 |
| mat128 | s/aim/aimed | 18:07 |
| *** electrofelix has quit IRC | 18:07 | |
| *** stendulker has quit IRC | 18:07 | |
| lucasagomes | rloo, ack, yeah and it's not a big deal either | 18:07 |
| rloo | mat128: touche | 18:07 |
| rloo | lucasagomes: :) | 18:08 |
| lucasagomes | but thanks for bringing those to the meeting | 18:08 |
| *** trown|lunch is now known as trown | 18:11 | |
| lucasagomes | stendulker <gone> | 18:12 |
| *** causten has joined #openstack-ironic | 18:14 | |
| lucasagomes | sambetts, around ? | 18:15 |
| lucasagomes | sambetts, apparently agent_ssh-tinyipa-nv has been broken for a while, do you know if anyone is looking into it ? | 18:16 |
| lucasagomes | (gate-tempest-dsvm-ironic-ipa-partition-agent_ssh-tinyipa-nv) | 18:16 |
| lucasagomes | it's non-voting but would be good to get it green again | 18:16 |
| *** athomas has quit IRC | 18:18 | |
| *** rama_y has joined #openstack-ironic | 18:19 | |
| *** krtaylor has joined #openstack-ironic | 18:21 | |
| *** milan has quit IRC | 18:23 | |
| *** sdake_ has joined #openstack-ironic | 18:23 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-specs: Amend ipxe-swift-tempurls spec https://review.openstack.org/372682 | 18:24 |
| *** amotoki has joined #openstack-ironic | 18:25 | |
| *** sdake has quit IRC | 18:25 | |
| *** e0ne has quit IRC | 18:28 | |
| *** ijw has joined #openstack-ironic | 18:30 | |
| *** moshele has joined #openstack-ironic | 18:33 | |
| *** jaosorior has joined #openstack-ironic | 18:36 | |
| *** moshele has quit IRC | 18:38 | |
| *** tangchen has quit IRC | 18:41 | |
| *** jaosorior has quit IRC | 18:44 | |
| *** tangchen has joined #openstack-ironic | 18:51 | |
| *** aNuposic has joined #openstack-ironic | 18:52 | |
| sambetts | lucasagomes: partition images on the agent driver are experimental still I thought | 19:06 |
| lucasagomes | sambetts, hmm it should work | 19:07 |
| lucasagomes | but yeah I gotta try | 19:07 |
| *** ijw has quit IRC | 19:10 | |
| *** ijw has joined #openstack-ironic | 19:11 | |
| *** ijw has quit IRC | 19:15 | |
| sambetts | lucasagomes: looking at the logs it seems its failing to verify the deployed partitions, but the node is definatly deploying correctly because in the VM logs it is booting into cirros ok | 19:16 |
| sambetts | lucasagomes: http://logs.openstack.org/47/363647/10/check/gate-tempest-dsvm-ironic-ipa-partition-agent_ssh-tinyipa-nv/f019740/console.html.gz#_2016-09-15_12_33_33_137285 | 19:16 |
| lucasagomes | sambetts, oh right, strangely it's checking for the ephemeral partition | 19:16 |
| lucasagomes | I wonder if we actually have ephemeral in the flavor | 19:17 |
| *** davidlenwell has quit IRC | 19:17 | |
| *** rbrud has joined #openstack-ironic | 19:17 | |
| openstackgerrit | Merged openstack/ironic: Support configdrive in iscsi deploy for whole disk images https://review.openstack.org/225115 | 19:17 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-specs: Amend ipxe-swift-tempurls spec https://review.openstack.org/372682 | 19:21 |
| *** tangchen has quit IRC | 19:23 | |
| *** tangchen has joined #openstack-ironic | 19:24 | |
| *** davidlenwell has joined #openstack-ironic | 19:29 | |
| *** ijw has joined #openstack-ironic | 19:31 | |
| rloo | lucasagomes, jlvillal: do you know why line 1190 was added there? https://review.openstack.org/#/c/225115/20/ironic/drivers/modules/deploy_utils.py | 19:34 |
| patchbot | patch 225115 - ironic - Support configdrive in iscsi deploy for whole disk... (MERGED) | 19:34 |
| rloo | lucasagomes, jlvillal: it is there on line 1212; i don't know why it needs to be there twice. | 19:35 |
| *** ijw has quit IRC | 19:36 | |
| lucasagomes | rloo, urgh seems redundant | 19:39 |
| rloo | lucasagomes: yeah. i don't see why that line is needed. | 19:39 |
| lucasagomes | DISK_LAYOUT_PARAMS doesn't check configdrive so it shouldn't be needed before that check | 19:39 |
| rloo | exactly. that's why i noticed it. | 19:40 |
| rloo | lucasagomes: i'll kick it out. just wanted to check in case i was missing something. | 19:40 |
| lucasagomes | rloo, cool, no I think you are correct | 19:40 |
| *** ijw has joined #openstack-ironic | 19:41 | |
| jlvillal | rloo: I think you are correct. It looks like it is unnecessary. | 19:44 |
| rloo | jlvillal: there are several patch sets so it is possible that it was needed at some point, but then the code got changed. dunno and don't want to spend the time trying to figure out. | 19:45 |
| *** ijw has quit IRC | 19:47 | |
| openstackgerrit | Charlle Daniel Da Silva Vieira Dias proposed openstack/ironic: Validate if the node is free or in use by Ironic https://review.openstack.org/372735 | 19:49 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Remove duplicated line wrt configdrive https://review.openstack.org/372736 | 19:49 |
| rloo | jlvillal, lucasagomes: ^^ if interested | 19:49 |
| *** sdake has joined #openstack-ironic | 19:49 | |
| jlvillal | rloo: Thanks | 19:50 |
| rloo | jlvillal: :) | 19:50 |
| *** sdake_ has quit IRC | 19:52 | |
| lucasagomes | rloo, thanks | 19:52 |
| rloo | lucasagomes: thx. I used partial bug cuz that was what the original one did. | 19:52 |
| rloo | lucasagomes: i suppose i could have mentioned 'a follow up...' dunno. | 19:53 |
| lucasagomes | rloo, it's all good :-) | 19:55 |
| lucasagomes | jroll, rloo mgould|afk http://paste.openstack.org/show/581198/ Ubuntu Xenial VM + UEFI + IPXE | 19:55 |
| lucasagomes | rloo, one thing tho, cirros is not UEFI capable :-/ I had to build a ubuntu image to deploy as well | 19:56 |
| lucasagomes | if we have to build such image for our tests in gate that would def increase the time it takes | 19:56 |
| rloo | lucasagomes: nice. isn't the ubuntu image really large? (compared to cirros?) | 19:56 |
| lucasagomes | I wonder tho if it would be possible to cache a UEFI capable image somewhere | 19:56 |
| lucasagomes | rloo, yes way larger than cirros | 19:56 |
| lucasagomes | stack@ubuntu-Standard-PC-i440FX-PIIX-1996:~/devstack$ du -sh my-image.qcow2 | 19:57 |
| lucasagomes | 350M my-image.qcow2 | 19:57 |
| rloo | lucasagomes: i don't recall. was the only issue, the timeout due to size? or were there other issues besides timeout? | 19:57 |
| lucasagomes | cirros might be like 30M heh | 19:57 |
| *** aNuposic has quit IRC | 19:57 | |
| rloo | lucasagomes: whoa. that is funny and sad and ... i wonder if they'll let us do that. | 19:57 |
| lucasagomes | rloo, no because of the size of the image, I mean if we have to build it from scratch than the whole job would need more time to run | 19:58 |
| lucasagomes | but I think we could cache an already generated image somewhere | 19:58 |
| * sambetts -> home | 19:58 | |
| sambetts | night everyone! | 19:58 |
| lucasagomes | I have to also google about cirros + uefi | 19:58 |
| lucasagomes | sambetts, good night | 19:58 |
| rloo | lucasagomes: is fedora smaller? although maybe we can't use that anyway | 19:58 |
| *** sambetts is now known as sambetts|afk | 19:58 | |
| rloo | night sambetts | 19:58 |
| sambetts|afk | cya lucasagomes rloo | 19:58 |
| lucasagomes | rloo, idk, it might be almost the same size | 19:58 |
| lucasagomes | but anyway, it's not a big deal | 19:59 |
| lucasagomes | I hit this too: https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/1483071 | 19:59 |
| openstack | Launchpad bug 1483071 in libvirt (Ubuntu Wily) "Error creating new VM with OVMF" [High,Fix committed] | 19:59 |
| lucasagomes | but I could workaround | 19:59 |
| rloo | lucasagomes: i think it might be a big deal if we want to use ubuntu. hopefully not. maybe someone else will know. | 19:59 |
| rloo | lucasagomes: also wonder how long the test will take. | 20:00 |
| rloo | lucasagomes: but would be nice to have this test. | 20:00 |
| lucasagomes | rloo, so the host is ubuntu, it's just the image we deploy | 20:00 |
| *** BadCub has quit IRC | 20:00 | |
| lucasagomes | if we pre-cache an ubuntu image, the size of it may not be a big deal on the local network | 20:00 |
| lucasagomes | anyway things to discuss with time, I will probably open an RFE for it | 20:01 |
| rloo | lucasagomes: hopefully it won't be a big deal. i haven't paid much attention to the infra/testing side of things. | 20:01 |
| lucasagomes | rloo, yeah | 20:01 |
| lucasagomes | me neither to be honest | 20:01 |
| rloo | lucasagomes: sometimes that is good. sometimes not ;) | 20:01 |
| *** BadCub has joined #openstack-ironic | 20:01 | |
| lucasagomes | rloo, good for your sanity | 20:01 |
| lucasagomes | :D | 20:01 |
| lucasagomes | anyway, I will call it a day as well | 20:02 |
| lucasagomes | brain is pretty slow right now, didn't sleep well this night | 20:02 |
| rloo | have a good evening. hope your jetlag doesn't prevent you from having fun! :) | 20:02 |
| lucasagomes | I talk to you all tomorrow | 20:02 |
| lucasagomes | rloo, just a little and for few days | 20:03 |
| rloo | lucasagomes: :D | 20:03 |
| *** lucasagomes is now known as lucas-afk | 20:03 | |
| *** ijw has joined #openstack-ironic | 20:03 | |
| *** ijw has quit IRC | 20:04 | |
| *** joprovost has quit IRC | 20:04 | |
| mat128 | lucas-afk: can Ubuntu's official image be used? | 20:04 |
| mat128 | https://cloud-images.ubuntu.com/ | 20:05 |
| lucas-afk | mat128, it might, I created this one with diskimage-builder | 20:05 |
| mat128 | I meant in the devstack script | 20:05 |
| lucas-afk | mat128, but I think we could use the official ones yeah | 20:05 |
| mat128 | to save on runtime | 20:05 |
| lucas-afk | mat128, it may not work with local boot tho, since we require grub2 to be installed | 20:05 |
| lucas-afk | + grub-efi-modules for uefi | 20:05 |
| mat128 | ahh | 20:05 |
| lucas-afk | but... something to try | 20:05 |
| mat128 | I think Scott Moser builds those | 20:05 |
| mat128 | we can always ask | 20:05 |
| lucas-afk | o/ that would handier | 20:06 |
| lucas-afk | mat128, we can always deploy w/o local boot | 20:06 |
| mat128 | and, they are qcow2, so probably means more ram for the machines, since streaming mode isnt supported | 20:06 |
| lucas-afk | that should work out of the box | 20:06 |
| lucas-afk | mat128, right yeah | 20:06 |
| lucas-afk | tho we need 1G + for IPA | 20:06 |
| lucas-afk | it should be enough for a cloud image | 20:06 |
| mat128 | I'm really unsure | 20:06 |
| mat128 | I ran the tests to come up with ~13xx mb | 20:07 |
| lucas-afk | mat128, I haven't changed the RAM amount in my tests | 20:07 |
| lucas-afk | and it booted | 20:07 |
| mat128 | and it was tight, not sure 350M will work | 20:07 |
| mat128 | oh | 20:07 |
| mat128 | sweet then :) | 20:07 |
| lucas-afk | mat128, oh right | 20:07 |
| lucas-afk | mat128, I used pxe_ipmitool | 20:07 |
| lucas-afk | that's why | 20:07 |
| mat128 | ahh | 20:07 |
| lucas-afk | it's copied via iscsi so IPA don't have to load it in the memory | 20:08 |
| lucas-afk | because I want to deploy a partition image | 20:08 |
| lucas-afk | well I will take a look at it tomorrow | 20:08 |
| mat128 | sure :) | 20:09 |
| lucas-afk | mat128, I keep you posted | 20:09 |
| lucas-afk | have a good evening | 20:09 |
| mat128 | you too | 20:10 |
| *** e0ne has joined #openstack-ironic | 20:12 | |
| rloo | mat128: about the install guide, https://review.openstack.org/#/c/366329/. the guide isn't being built in the gate, is it? | 20:12 |
| patchbot | patch 366329 - ironic - [install-guide] Import "Bare Metal service overview" | 20:12 |
| *** e0ne has quit IRC | 20:12 | |
| mat128 | it is, see "gate-ironic-install-guide" | 20:12 |
| rloo | mat128: oh yeah, i just saw that. great. was worried i'd have to build them myself. | 20:13 |
| mat128 | back when we introduced the initial commit for the install guide, we didnt have it | 20:13 |
| mat128 | if you wanted to build them, they're just a tox -e install-guide away | 20:13 |
| rloo | mat128: and a few more seconds away :) | 20:13 |
| mat128 | tell me about it :) | 20:13 |
| rloo | mat128: as far as the organization of the new install guide, are there guidelines for that? | 20:13 |
| mat128 | JayF and I were wondering about it, and went with "lets make the structure better without changing the content" | 20:14 |
| rloo | mat128: i mean, do/should i verify that the way you organized the guide, conforms to how openstack install guides are organized? | 20:14 |
| mat128 | rloo: if you find install guide guidelines I'd be happy | 20:14 |
| rloo | mat128, JayF: oh geez. | 20:15 |
| mat128 | so far I looked at the other projects that publish install guides and got inspiration from there | 20:15 |
| mat128 | long story short: we have "installing" under all different OSes, we kept SUSE because there might be autogenerated links pointing there | 20:15 |
| rloo | mat128, JayF: the problem with reorg'ing later, is that the links will be broken. | 20:15 |
| * rloo so sad. | 20:16 | |
| mat128 | yeah, thats why we attacked it with a "best effort reorganization" without changing content | 20:16 |
| mat128 | keeping this in mind, if you believe some section should be under a different section, don't hesitate to comment and I can easily fix it | 20:16 |
| rloo | mat128: it isn't the content of our guide that I'm concerned with, it is the organization of it in the new world. | 20:16 |
| rloo | mat128: i just want it to conform to openstack ways. but you're saying there is no consistent openstack way. | 20:16 |
| mat128 | I wish our docs was flexible without worrying about bookmarks and broken links, maybe sphinx can generate redirects | 20:16 |
| mat128 | rloo: sadly, that's correct | 20:17 |
| mat128 | afaik | 20:17 |
| mat128 | it's already a good start that projects can ship their own install guides | 20:17 |
| rloo | mat128: which install guides did you get inspiration from? | 20:17 |
| rloo | mat128: no, it isn't a good start. | 20:17 |
| rloo | mat128: argh. i'll shut up. | 20:17 |
| mat128 | why not? | 20:17 |
| mat128 | it was previously under the openstack-manuals group, but then we made our own install guide within the developer guide | 20:18 |
| mat128 | and I suspect others did the same | 20:18 |
| rloo | mat128: cuz we aren't tech writers, i'm not an install guide expert. yes, we're going to have more install guides avail in the official url, but they will be org'd differently. | 20:18 |
| mat128 | rloo: thats why I was hoping the docs group would have guidance | 20:19 |
| rloo | mat128: but i guess that's awesome, cuz the projects aren't sheep and can be individualistic and do things their own way. | 20:19 |
| mat128 | :) | 20:19 |
| mat128 | we need oslo.install-guide ;) | 20:19 |
| rloo | mat128: guidance will be nice. it'll be too late, later, when someone does a cross-project 'install-guide' guidelines. | 20:19 |
| rloo | mat128: ok, i'll start reviewing :) | 20:20 |
| mat128 | we can always adapt it | 20:20 |
| * mat128 is forever optimistic | 20:20 | |
| mat128 | so far, projects publishing their own manuals: http://paste.openstack.org/show/581227/ | 20:20 |
| rloo | mat128: optimistic is great. spending time changing things later, isn't, if we can try to 'get it right' now. | 20:21 |
| *** ijw has joined #openstack-ironic | 20:21 | |
| rloo | mat128: did you look at the install guides that are published NOT by the projects? | 20:21 |
| mat128 | Heat: http://docs.openstack.org/project-install-guide/orchestration/draft/ Database: http://docs.openstack.org/project-install-guide/database/draft/ | 20:21 |
| *** aNuposic has joined #openstack-ironic | 20:21 | |
| rloo | mat128: are there install guides that might have been written by someone that has an idea of what/how an install guide should be? | 20:22 |
| mat128 | my biggest gripe so far is that the "project-install-guide" arent linked from docs.openstack.org | 20:22 |
| mat128 | rloo: most of the other projects' published guides are just ports from openstack-manuals | 20:22 |
| mat128 | but we were different | 20:22 |
| mat128 | http://docs.openstack.org/mitaka/install-guide-ubuntu/ | 20:22 |
| rloo | mat128: in what way? | 20:22 |
| mat128 | thats from the global one | 20:22 |
| mat128 | we published ours inside of developer docs | 20:22 |
| mat128 | presumably because we wanted it in our own tree | 20:22 |
| rloo | mat128: oh yeah, i know that. it was cuz the doc folks didn't have anyone to write the install guide and add it to docs.openstack.org. | 20:23 |
| rloo | mat128: it wasn't cuz we wanted it in our tree | 20:23 |
| rloo | mat128: or at least, it wasn't cuz *I* wanted it in our tree | 20:23 |
| mat128 | JayF was saying cinder is/was a good example: http://docs.openstack.org/mitaka/install-guide-ubuntu/cinder.html | 20:24 |
| mat128 | oh really | 20:24 |
| mat128 | not everyone can go and submit docs in openstack/openstack-manuals ? | 20:24 |
| rloo | mat128: dunno. i mean, anyone can submit to any project. | 20:25 |
| rloo | mat128: whether it gets in or not is another question. | 20:25 |
| mat128 | well thats what I meant | 20:25 |
| mat128 | if it was so closed that today it is directly inside projects | 20:25 |
| mat128 | the least docs people should do is provide guidelines | 20:26 |
| mat128 | (and maybe they did, but I couldnt find it) | 20:26 |
| rloo | mat128: so block storage/cinder doesn't seem to have the same outline as ironic's. | 20:26 |
| mat128 | none of the other guides are as through as ours | 20:27 |
| rloo | mat128: ok, let me think/review... | 20:27 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 20:31 |
| rloo | mat128, JayF: sorry, for mitaka, they had separate install guides, for opensuse, RH, and ubuntu. | 20:32 |
| rloo | mat128, JayF: the way the bm guide is organized -- will that be used to generate the three install guides, or is the install guide going a different route in newton? | 20:32 |
| mat128 | rloo: right now the "project-install-guide" is one guide, not 3. Previously, they used to build it three times with sphinx "tags" | 20:33 |
| mat128 | so .rst had :: only: ubuntu | 20:33 |
| mat128 | and then an ubuntu-specific block | 20:33 |
| rloo | mat128: ok, so that's why the new guides have separate sections for the different OS. | 20:33 |
| mat128 | yeah, I wish it was built using tags and I could do ifs | 20:34 |
| mat128 | rather than tracking 3 different (but almost the same) files | 20:34 |
| mat128 | http://docs.openstack.org/project-install-guide/draft/ | 20:34 |
| mat128 | the different "OS" links point to the same address | 20:34 |
| mat128 | instructions: http://docs.openstack.org/contributor-guide/project-install-guide.html | 20:35 |
| mat128 | rloo: reasoning and alternatives listed in the docs-specs: http://specs.openstack.org/openstack/docs-specs/specs/newton/installguide.html and http://specs.openstack.org/openstack/docs-specs/specs/newton/project-specific-installguides.html | 20:37 |
| mat128 | rloo: guidelines are very liberal and already respected in the patch chain: http://specs.openstack.org/openstack/docs-specs/specs/newton/project-specific-installguides.html#content-of-project-specific-install-guides | 20:38 |
| *** Goneri has quit IRC | 20:39 | |
| rloo | mat128: do you know what they meant by "Project specific guides should be versioned, so project teams should publish to the respective subdirectory for their service"? | 20:41 |
| mat128 | versioned and released | 20:41 |
| mat128 | as in | 20:41 |
| mat128 | when we branch for a release, the project guide is also contained in the branch | 20:41 |
| mat128 | thats what I understood | 20:42 |
| rloo | mat128: ok. i was a bit worried they meant we had to explicitly have a newton/ subdir etc | 20:42 |
| mat128 | all other project-install-guide have an install-guide subdir | 20:42 |
| mat128 | what we did is also what was suggested in the main docs: http://docs.openstack.org/contributor-guide/project-install-guide.html | 20:42 |
| rloo | mat128: yeah, that is fine i guess. i'm sorry it isn't under 'doc'. but not a big deal. | 20:43 |
| mat128 | rloo: I have to leave but if you have any concern, don't hesitate to raise them in the reviews | 20:54 |
| mat128 | or directly here by mentioning me | 20:55 |
| rloo | mat128: thx. no worries. am commenting :) | 20:55 |
| openstackgerrit | Charlle Daniel Da Silva Vieira Dias proposed openstack/ironic: Validate if the node is free or in use by Ironic https://review.openstack.org/372735 | 20:57 |
| *** bdeschenes has quit IRC | 20:59 | |
| *** trown is now known as trown|outtypewww | 21:04 | |
| *** ijw has quit IRC | 21:07 | |
| openstackgerrit | Charlle Daniel Da Silva Vieira Dias proposed openstack/ironic: Implement the validation to verify if the node is free or in use by Ironic https://review.openstack.org/372762 | 21:08 |
| *** baoli_ has quit IRC | 21:16 | |
| *** causten has quit IRC | 21:17 | |
| *** causten has joined #openstack-ironic | 21:18 | |
| *** krtaylor has quit IRC | 21:20 | |
| *** rbudden has joined #openstack-ironic | 21:24 | |
| *** rpioso has quit IRC | 21:26 | |
| *** bnemec has quit IRC | 21:33 | |
| *** r-daneel has joined #openstack-ironic | 21:39 | |
| *** vishwanathj has quit IRC | 21:43 | |
| *** jheroux has quit IRC | 21:43 | |
| *** BadCub has quit IRC | 21:51 | |
| *** harlowja has quit IRC | 21:51 | |
| *** wznoinsk has quit IRC | 21:51 | |
| *** jtomasek has quit IRC | 21:51 | |
| *** joemonk has quit IRC | 21:51 | |
| *** TheJulia has quit IRC | 21:51 | |
| *** vsaienk0 has quit IRC | 21:51 | |
| *** ptx0 has quit IRC | 21:51 | |
| *** wznoinsk has joined #openstack-ironic | 21:51 | |
| *** BadCub has joined #openstack-ironic | 21:51 | |
| *** joemonk has joined #openstack-ironic | 21:51 | |
| *** jtomasek has joined #openstack-ironic | 21:51 | |
| *** ptx0 has joined #openstack-ironic | 21:51 | |
| *** ptx0 has joined #openstack-ironic | 21:51 | |
| *** vsaienk0 has joined #openstack-ironic | 21:52 | |
| *** TheJulia has joined #openstack-ironic | 21:53 | |
| *** dprince has quit IRC | 21:54 | |
| *** harlowja has joined #openstack-ironic | 21:55 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Adds a missing space in a help string https://review.openstack.org/372784 | 22:02 |
| jlvillal | Spaces? We ain't got no spaces. We don't need no spaces. I don't have to show you any stinkin' spaces! | 22:04 |
| jlvillal | rloo: :) ^^ | 22:04 |
| rloo | jlvillal: spaces aren't for you, they're for our users :D | 22:04 |
| rloo | jlvillal: what bothers me is that it was a comment in a patch from august. i just got around to looking at that merged patch. so sad that no one followed up on it. | 22:05 |
| jlvillal | rloo: I sort of modified the Treasure of the Sierra Madre quote :) | 22:05 |
| rloo | jlvillal: oh, is that where it came from? it is familiar, just couldn't remember where. | 22:05 |
| jlvillal | rloo: Yeah. s/spaces/badges/ :) | 22:06 |
| rloo | jlvillal: wrt the install guide changes. What do you think about mat & jay's patch just moving the content. you can add patches over those to fix the content. | 22:07 |
| jlvillal | rloo: By 'you' do you mean 'we'? | 22:07 |
| rloo | jlvillal: i had to refrain from commenting on the content :) and want to get these patches done so the install guide is migrated. | 22:07 |
| *** cdearborn has quit IRC | 22:07 | |
| rloo | jlvillal: no, i mean YOU | 22:07 |
| jlvillal | -2!!! | 22:07 |
| rloo | jlvillal: if i have time, i'd like to update the api-ref stuff. time time time... | 22:08 |
| jlvillal | I'm okay with just copy-n-pasting it. | 22:08 |
| jlvillal | We can work on cleaning it up later. | 22:08 |
| jlvillal | Seems reasonable. | 22:08 |
| rloo | jlvillal: ok, i commented about that. hopefully, with mat128 here tomorrow and me 'up to speed' on the guide, we can get them merged quickly. | 22:09 |
| jlvillal | rloo: thanks | 22:10 |
| *** pmannidi has joined #openstack-ironic | 22:10 | |
| mrda | Morning Ironic | 22:11 |
| jlvillal | mrda: No | 22:12 |
| jlvillal | mrda: Good morning!!! :) | 22:12 |
| jlvillal | mrda: It is a supercalifragilisticexpialidocious morning.... | 22:12 |
| mrda | good to hear | 22:13 |
| mrda | I like "spaces aren't for you, they're for our users" above | 22:13 |
| jlvillal | mrda: Don't encourage rloo ;) | 22:14 |
| mrda | :) | 22:15 |
| openstackgerrit | Merged openstack/ironic: Remove duplicated line wrt configdrive https://review.openstack.org/372736 | 22:22 |
| *** vsaienko has quit IRC | 22:25 | |
| *** rcernin has quit IRC | 22:27 | |
| *** amotoki has quit IRC | 22:54 | |
| *** r-daneel has quit IRC | 23:11 | |
| *** aNuposic has quit IRC | 23:17 | |
| *** rbrud has quit IRC | 23:22 | |
| *** rajinir has quit IRC | 23:25 | |
| *** rybridges1 has quit IRC | 23:41 | |
| *** ijw has joined #openstack-ironic | 23:41 | |
| *** rybridges1 has joined #openstack-ironic | 23:41 | |
| *** openstackgerrit has quit IRC | 23:42 | |
| *** rama_y has quit IRC | 23:44 | |
| *** rbudden has quit IRC | 23:47 | |
| *** openstackgerrit has joined #openstack-ironic | 23:47 | |
| *** openstackgerrit has quit IRC | 23:51 | |
| *** hogepodge has quit IRC | 23:53 | |
| *** openstackgerrit has joined #openstack-ironic | 23:55 | |
| *** hogepodge has joined #openstack-ironic | 23:56 | |
| NobodyCam | morning mrda :) | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!