| *** rama_y has quit IRC | 00:00 | |
| *** baoli_ has quit IRC | 00:05 | |
| *** srobert has quit IRC | 00:11 | |
| *** yangyapeng has quit IRC | 00:13 | |
| *** yangyapeng has joined #openstack-ironic | 00:14 | |
| *** CharlesXavier has quit IRC | 00:17 | |
| *** CharlesXavier has joined #openstack-ironic | 00:18 | |
| *** yangyapeng has quit IRC | 00:18 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Reduce complexity of node_power_action() function https://review.openstack.org/507668 | 00:19 |
|---|---|---|
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off https://review.openstack.org/507297 | 00:19 |
| *** baoli has joined #openstack-ironic | 00:20 | |
| *** srobert has joined #openstack-ironic | 00:22 | |
| *** chas has joined #openstack-ironic | 00:29 | |
| *** CharlesXavier has quit IRC | 00:29 | |
| *** causten has joined #openstack-ironic | 00:30 | |
| *** chas has quit IRC | 00:34 | |
| *** yee37927 has quit IRC | 00:39 | |
| *** yee379 has joined #openstack-ironic | 00:39 | |
| *** litao__ has joined #openstack-ironic | 00:49 | |
| *** kzaitsev1pi has quit IRC | 00:52 | |
| *** kzaitsev_pi has joined #openstack-ironic | 00:54 | |
| *** CharlesXavier has joined #openstack-ironic | 00:58 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent master: Build tinyipa with python3 https://review.openstack.org/495385 | 00:58 |
| *** rbudden has quit IRC | 01:00 | |
| *** phuongnh has joined #openstack-ironic | 01:02 | |
| *** lennyb has quit IRC | 01:11 | |
| *** lennyb has joined #openstack-ironic | 01:11 | |
| *** causten has quit IRC | 01:12 | |
| *** causten has joined #openstack-ironic | 01:15 | |
| *** yangyapeng has joined #openstack-ironic | 01:20 | |
| *** mikal_ has joined #openstack-ironic | 01:22 | |
| *** yangyape_ has joined #openstack-ironic | 01:25 | |
| *** yangyapeng has quit IRC | 01:25 | |
| *** mikal has quit IRC | 01:25 | |
| *** mikal_ has quit IRC | 01:28 | |
| *** hw_wutianwei1 has joined #openstack-ironic | 01:29 | |
| *** gcb has joined #openstack-ironic | 01:31 | |
| *** bnemec has quit IRC | 01:34 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent master: Build tinyipa with python3 https://review.openstack.org/495385 | 01:38 |
| *** bnemec has joined #openstack-ironic | 01:39 | |
| *** mikal has joined #openstack-ironic | 01:56 | |
| *** cdearborn has quit IRC | 02:11 | |
| *** rbudden has joined #openstack-ironic | 02:13 | |
| *** rbudden has quit IRC | 02:17 | |
| *** krtaylor has joined #openstack-ironic | 02:20 | |
| *** baoli has quit IRC | 02:24 | |
| *** chas has joined #openstack-ironic | 02:30 | |
| *** namnh has joined #openstack-ironic | 02:31 | |
| *** baoli has joined #openstack-ironic | 02:34 | |
| *** chas has quit IRC | 02:34 | |
| *** baoli has quit IRC | 02:42 | |
| *** baoli has joined #openstack-ironic | 02:44 | |
| *** IgorYozhikov has joined #openstack-ironic | 02:54 | |
| *** srobert has quit IRC | 02:55 | |
| *** Sukhdev has quit IRC | 03:15 | |
| *** baoli has quit IRC | 03:21 | |
| *** baoli has joined #openstack-ironic | 03:22 | |
| *** Sukhdev has joined #openstack-ironic | 03:24 | |
| *** bnemec has quit IRC | 03:26 | |
| *** crushil has joined #openstack-ironic | 03:26 | |
| *** causten has quit IRC | 03:30 | |
| *** causten has joined #openstack-ironic | 03:31 | |
| *** Tengu has quit IRC | 03:33 | |
| *** baoli has quit IRC | 03:34 | |
| *** Tengu has joined #openstack-ironic | 03:39 | |
| *** udesale has joined #openstack-ironic | 03:42 | |
| *** links has joined #openstack-ironic | 03:44 | |
| *** vmud213 has joined #openstack-ironic | 03:47 | |
| *** dpawar has joined #openstack-ironic | 04:00 | |
| *** crushil_ has joined #openstack-ironic | 04:10 | |
| *** aparnav has joined #openstack-ironic | 04:21 | |
| *** yangyape_ has quit IRC | 04:22 | |
| *** yangyapeng has joined #openstack-ironic | 04:23 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent master: Build tinyipa with python3 https://review.openstack.org/495385 | 04:47 |
| *** exodusftw has quit IRC | 05:06 | |
| *** exodusftw has joined #openstack-ironic | 05:16 | |
| *** jaianshu has joined #openstack-ironic | 05:23 | |
| *** Sukhdev has quit IRC | 05:29 | |
| *** yangyape_ has joined #openstack-ironic | 05:29 | |
| *** yangyapeng has quit IRC | 05:29 | |
| *** yangyape_ has quit IRC | 05:31 | |
| *** yangyapeng has joined #openstack-ironic | 05:32 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent master: Build tinyipa with python3 https://review.openstack.org/495385 | 05:35 |
| *** marios has joined #openstack-ironic | 05:42 | |
| *** yangyapeng has quit IRC | 05:50 | |
| *** yangyape_ has joined #openstack-ironic | 05:50 | |
| *** ChubYann has quit IRC | 05:51 | |
| *** vsaienk0 has joined #openstack-ironic | 05:55 | |
| *** noam_ has joined #openstack-ironic | 06:00 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-python-agent master: Rescue extension for CoreOS with DHCP tenant networks https://review.openstack.org/423521 | 06:00 |
| *** jaganathan has joined #openstack-ironic | 06:02 | |
| *** jaganathan has quit IRC | 06:02 | |
| *** crushil_ has quit IRC | 06:10 | |
| *** crushil has quit IRC | 06:11 | |
| *** mjura has joined #openstack-ironic | 06:15 | |
| *** yangyape_ has quit IRC | 06:18 | |
| *** yangyapeng has joined #openstack-ironic | 06:19 | |
| *** rbartal has joined #openstack-ironic | 06:19 | |
| *** pcaruana has joined #openstack-ironic | 06:19 | |
| *** yangyapeng has quit IRC | 06:23 | |
| *** yangyapeng has joined #openstack-ironic | 06:29 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic master: Update after recent removal of cred manager aliases https://review.openstack.org/506983 | 06:32 |
| *** yangyapeng has quit IRC | 06:34 | |
| *** dpawar has quit IRC | 06:35 | |
| *** makowals has joined #openstack-ironic | 06:53 | |
| *** rcernin has joined #openstack-ironic | 07:00 | |
| *** yangyapeng has joined #openstack-ironic | 07:03 | |
| *** andymccr has quit IRC | 07:07 | |
| *** yangyapeng has quit IRC | 07:07 | |
| *** asettle has quit IRC | 07:08 | |
| *** amarao has quit IRC | 07:09 | |
| *** amarao1 has joined #openstack-ironic | 07:09 | |
| *** andymccr has joined #openstack-ironic | 07:09 | |
| *** asettle has joined #openstack-ironic | 07:10 | |
| *** hjensas has joined #openstack-ironic | 07:10 | |
| *** hjensas has quit IRC | 07:10 | |
| *** hjensas has joined #openstack-ironic | 07:10 | |
| *** asettle is now known as Guest3828 | 07:10 | |
| *** yangyapeng has joined #openstack-ironic | 07:17 | |
| *** amarao1 has quit IRC | 07:17 | |
| *** yangyapeng has quit IRC | 07:18 | |
| *** yangyapeng has joined #openstack-ironic | 07:18 | |
| *** ralonsoh has joined #openstack-ironic | 07:19 | |
| vdrok | morning ironic! | 07:20 |
| noam_ | Hi, can someone elaborate on the difference between local boot and netboot? | 07:24 |
| *** yangyapeng has quit IRC | 07:29 | |
| *** yangyape_ has joined #openstack-ironic | 07:29 | |
| *** yangyape_ has quit IRC | 07:30 | |
| *** yangyapeng has joined #openstack-ironic | 07:30 | |
| *** dpawar has joined #openstack-ironic | 07:36 | |
| pas-ha | morning ironic :) | 07:40 |
| pas-ha | noam_: netboot means that the kernel and ramdisk of the user image will be booted over (i)PXE (network boot) | 07:41 |
| pas-ha | while for localboot ironic will edit the image written to disk a bit to init the bootloader so the node is booted from local storage (disk) | 07:41 |
| pas-ha | also, re multi-tenancy support for instances running on ironic nodes, netboot is not supported currently | 07:43 |
| *** ccamacho has quit IRC | 07:45 | |
| *** ccamacho has joined #openstack-ironic | 07:45 | |
| noam_ | So in case of netboot changes not saved after reboot? | 07:49 |
| *** amarao has joined #openstack-ironic | 07:54 | |
| pas-ha | noam_: depends on what changes - the root partition is still on the disk, so files etc will stay there after reboot, but if you for example update the kernel - it won't have any effect, the node will boot over network from the same kernel as before | 07:57 |
| pas-ha | netboot != ramdisk | 07:58 |
| pas-ha | only kernel and initramfs are supplied over network, the root file system is on local disk still | 07:58 |
| noam_ | ok i understand better now | 08:02 |
| noam_ | thanks | 08:02 |
| *** hjensas has quit IRC | 08:06 | |
| *** hjensas has joined #openstack-ironic | 08:08 | |
| *** amarao1 has joined #openstack-ironic | 08:17 | |
| *** derekh has joined #openstack-ironic | 08:26 | |
| *** chas has joined #openstack-ironic | 08:35 | |
| *** vsaienk0 has quit IRC | 08:36 | |
| IgorYozhikov | pas-ha | 08:37 |
| *** Guest3828 is now known as asettle | 08:39 | |
| *** e0ne has joined #openstack-ironic | 08:42 | |
| *** vsaienk0 has joined #openstack-ironic | 08:48 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Rescue implementation for iLO Virtual Media drivers https://review.openstack.org/453012 | 08:54 |
| *** ralonsoh_ has joined #openstack-ironic | 09:00 | |
| *** ralonsoh has quit IRC | 09:03 | |
| *** yangyapeng has quit IRC | 09:04 | |
| *** yangyapeng has joined #openstack-ironic | 09:08 | |
| pas-ha | IgorYozhikov: hi! :) | 09:08 |
| *** mrch has joined #openstack-ironic | 09:10 | |
| *** stendulker has joined #openstack-ironic | 09:11 | |
| *** marios has quit IRC | 09:21 | |
| *** sambetts|afk is now known as sambetts | 09:24 | |
| *** amarao has quit IRC | 09:24 | |
| *** amarao has joined #openstack-ironic | 09:24 | |
| sambetts | Morning all | 09:25 |
| stendulker | Good morning sambetts :) | 09:26 |
| *** alex__ has joined #openstack-ironic | 09:30 | |
| alex__ | pas-ha: hello | 09:31 |
| IgorYozhikov | pas-ha, colleagues. alex__ and I have question about scaling of ironic services to handle big deployments | 09:33 |
| alex__ | pas-ha: pa-ha, we have a project which need us to use ironic to deploy baremetal server. The scale is about 1000/week. | 09:36 |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Update proliantutils version for master to "2.4.1" https://review.openstack.org/505506 | 09:37 |
| IgorYozhikov | alex__, as far as I know it is also important to know version of OpenStack | 09:37 |
| alex__ | Fuel 9.2 openstack | 09:37 |
| IgorYozhikov | So, it is Mitaka | 09:38 |
| alex__ | but we use the ironic version with ocata. | 09:38 |
| alex__ | because customer wants to use ironic to support tenants network. | 09:39 |
| pas-ha | alex__: do you use nova to drive ironic? | 09:39 |
| pas-ha | and also, is it going to be a mixed baremetal+vm instances in nova, or baremetal only? | 09:42 |
| alex__ | pas-ha: yeah, this is a mixed in nova | 09:44 |
| *** etingof has quit IRC | 09:44 | |
| alex__ | [ironic] api_endpoint = http://10.131.0.98:6385 admin_username = ironic admin_password = ironic admin_url = http://10.131.0.137:5000/v2.0 admin_tenant_name = services | 09:44 |
| alex__ | pas-ha: these are configuration in nova.conf. | 09:45 |
| *** hw_wutianwei1 has quit IRC | 09:46 | |
| vsaienk0 | IgorYozhikov: in Mitaka if I remember correctly we were able to deploy nova-compute in active/backup mode only | 09:46 |
| *** ijw has quit IRC | 09:47 | |
| vsaienk0 | nova compute will not be scalable horizontally | 09:47 |
| pas-ha | yep, in mitaka nova + ironic has lots of issues, especially for mixed hypervisor case | 09:47 |
| alex__ | pas-ha: Do u have the suggestion about how to deploy baremetal servers via multiple conductors? | 09:48 |
| pas-ha | ironic-conductors are perfectly scalable horizontally, it is nova-compute with ironic virt-driver that can be only a single instance in Mitaka | 09:48 |
| pas-ha | on Fuel 8 we had a test lab where 3 (beefy) physical nodes with ironic-conductors were managing 4k nodes, but we were able to provision them via nova only in batches of like 100 | 09:50 |
| *** aparnav has quit IRC | 09:51 | |
| pas-ha | alex__: so again, the main 'scaling' question is - do you need that many nodes simply managed or they have to be frequently deployed/redeployed via nova? | 09:51 |
| pas-ha | this 1k nodes/week - is it enroll and deploy once? they will be redeployed frequently enough? | 09:52 |
| alex__ | pas-ha: No, they don't need to redeploy frequently. | 09:52 |
| alex__ | But we want to have the ability to redeploy as required. | 09:54 |
| alex__ | but not frequently | 09:54 |
| pas-ha | with ironic-conductors on of the biggest limiting factors of average churn is power state sync. If you have it enabled, I'd suggest to test the HW/network first, and then decide on how many nodes your single ironic-conductor can handle | 09:54 |
| *** jaianshu has quit IRC | 09:55 | |
| pas-ha | (as each power sync task consumes/blocks one thread from ironic-conductor pool of threads, though pool size is configurable) | 09:55 |
| alex__ | ok, | 09:56 |
| pas-ha | with redeploy, your main limiting factor would be concurrent deploys via nova, as there most probably will be scheduling collisions | 09:56 |
| pas-ha | for ironic nodes | 09:56 |
| alex__ | pas-ha: so I can improve the pool size, can i improve the performance? | 09:57 |
| pas-ha | but what we have seen, as long as your "concurrency" (how many boot requests for ironic nodes are processed by nova in parallel) is substantially smaller than the number of currently 'available' ironic nodes, scheduling succeeds more often | 09:58 |
| pas-ha | performance in the sense of how many nodes to manage by single ironic-conductor service - yes, but you might need a beefier host for that as well.. | 09:59 |
| *** marios has joined #openstack-ironic | 09:59 | |
| alex__ | pas-ha: yeah, I have a physical server as conductor with 4 10G network Nics and ssd. | 10:00 |
| pas-ha | also note that all these scheduling problems should be almost gone in Pike release of nova and ironic due to a completely new scheduling mechanism in nova (placement API and resource classes) | 10:01 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Implements `rescue` interface for 'ilo' hardware type https://review.openstack.org/453012 | 10:01 |
| *** etingof has joined #openstack-ironic | 10:01 | |
| *** chas has quit IRC | 10:02 | |
| *** chas has joined #openstack-ironic | 10:02 | |
| *** yangyapeng has quit IRC | 10:02 | |
| *** ralonsoh__ has joined #openstack-ironic | 10:03 | |
| *** yangyapeng has joined #openstack-ironic | 10:03 | |
| alex__ | pas-ha: we are not able to update the ironic from ocata to pike,since we have deployed about 300 nodes. | 10:03 |
| *** yangyapeng has quit IRC | 10:04 | |
| pas-ha | to have full potential you'd have to deploy Pike nova as well.. | 10:05 |
| *** ralonsoh_ has quit IRC | 10:06 | |
| *** masber has quit IRC | 10:06 | |
| *** masber has joined #openstack-ironic | 10:07 | |
| *** chas has quit IRC | 10:07 | |
| IgorYozhikov | nova placement was introduced in ocata or newton, no? | 10:07 |
| alex__ | pas-ha: And customer wants us to provide a design about how to improve the performance, since hardwares are not the question. | 10:08 |
| sambetts | Ironic virt driver wasn't updated to use it until the last release | 10:08 |
| *** etingof has quit IRC | 10:10 | |
| *** marios has quit IRC | 10:12 | |
| alex__ | pas-ha: what is simply managed by ironic? | 10:16 |
| pas-ha | by this I mean just a background churn ironic performs on 'available' or 'active' nodes, w/o many nodes being deployed/cleaned, with occasional API commands to execute against the nodes (like power on/off etc) | 10:18 |
| pas-ha | when many nodes are being provisioned/cleaned at the same time, requirements might be different | 10:19 |
| pas-ha | for example many nodes trying to access TFTP on ironic-conductor produces quite noticable CPU spikes, so your conductor host has to be able to deal with it | 10:20 |
| pas-ha | for some example figures see http://pshchelo.github.io/ansible-deploy-perf.html (the purpose of this post was a bit different, but it has some CPU graphs for ironic-conductor provisioning 100 nodes at once) | 10:22 |
| *** ralonsoh__ is now known as ralonsoh | 10:24 | |
| pas-ha | bwt it ^ uses ironic as of Ocata | 10:24 |
| *** etingof has joined #openstack-ironic | 10:25 | |
| *** etingof has quit IRC | 10:27 | |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Switch to none auth for standalone mode https://review.openstack.org/359061 | 10:29 |
| *** jkilpatr has quit IRC | 10:35 | |
| *** udesale has quit IRC | 10:39 | |
| *** phuongnh has quit IRC | 10:40 | |
| *** chas has joined #openstack-ironic | 10:47 | |
| *** ijw has joined #openstack-ironic | 10:47 | |
| *** stendulker_ has joined #openstack-ironic | 10:47 | |
| *** stendulker has quit IRC | 10:50 | |
| *** gcb has quit IRC | 10:50 | |
| *** jkilpatr has joined #openstack-ironic | 10:52 | |
| *** stendulker_ has quit IRC | 10:52 | |
| *** ijw has quit IRC | 10:52 | |
| *** alex__ has quit IRC | 10:53 | |
| *** yolanda has quit IRC | 10:54 | |
| *** vmud213 has quit IRC | 11:03 | |
| *** marios has joined #openstack-ironic | 11:05 | |
| *** gcb has joined #openstack-ironic | 11:07 | |
| *** yolanda has joined #openstack-ironic | 11:08 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost master: Fix bug in setting dns servers https://review.openstack.org/508113 | 11:14 |
| *** ricardoas has quit IRC | 11:19 | |
| *** mrtenio has quit IRC | 11:19 | |
| *** CharlesXavier has quit IRC | 11:19 | |
| *** mrtenio has joined #openstack-ironic | 11:21 | |
| *** ricardoas has joined #openstack-ironic | 11:21 | |
| *** CharlesXavier has joined #openstack-ironic | 11:22 | |
| robcresswell | o/ Hey all; what does "the example template" in https://docs.openstack.org/bifrost/latest/user/virsh.html refer too? | 11:25 |
| robcresswell | Ah wait, maybe this isn't what I want. | 11:26 |
| robcresswell | Basically, trying to figure out how I can chuck bifrost on a NUC and manage a couple of "virtual" baremetals for testing Ironic APIs. | 11:26 |
| robcresswell | Currently trying to adapt the gate test script, but that has both more than I need and also makes assumptions about the env I think, as the upstream stuff likely has extra packages built into the image. | 11:27 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector master: Use ostestr unit test runner https://review.openstack.org/508122 | 11:33 |
| *** jlabarre has joined #openstack-ironic | 11:41 | |
| *** jaganathan has joined #openstack-ironic | 11:41 | |
| yolanda | hi dtantsur|afk or other cores, can i get reviews on https://review.openstack.org/505146 and https://review.openstack.org/505315 ? just missing the final +2 | 11:43 |
| patchbot | patch 505146 - bifrost - Install the latest version of setuptools | 11:43 |
| patchbot | patch 505315 - bifrost - Different pip install with/without venv | 11:43 |
| *** bfournie has quit IRC | 11:45 | |
| *** bfournie has joined #openstack-ironic | 11:45 | |
| *** bfournie has quit IRC | 11:49 | |
| *** dpawar has quit IRC | 11:54 | |
| *** trown has joined #openstack-ironic | 11:55 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector master: Use ostestr unit test runner https://review.openstack.org/508122 | 11:57 |
| *** chas has quit IRC | 12:02 | |
| *** fellypefca has joined #openstack-ironic | 12:04 | |
| fellypefca | Hi and good morning, Ironic :) | 12:05 |
| *** francoblanco has quit IRC | 12:05 | |
| *** dpawar has joined #openstack-ironic | 12:07 | |
| *** cdearborn has joined #openstack-ironic | 12:11 | |
| *** gcb has quit IRC | 12:12 | |
| *** gcb has joined #openstack-ironic | 12:12 | |
| *** rbudden has joined #openstack-ironic | 12:14 | |
| *** srobert has joined #openstack-ironic | 12:17 | |
| *** bfournie has joined #openstack-ironic | 12:19 | |
| *** dsneddon has quit IRC | 12:19 | |
| *** bfournie has quit IRC | 12:19 | |
| *** trown has quit IRC | 12:21 | |
| *** dpawar has quit IRC | 12:23 | |
| *** litao__ has quit IRC | 12:26 | |
| *** dprince has joined #openstack-ironic | 12:29 | |
| *** srobert has quit IRC | 12:30 | |
| *** srobert has joined #openstack-ironic | 12:30 | |
| *** pcaruana has quit IRC | 12:31 | |
| *** cdearborn has quit IRC | 12:31 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost master: Fix bug in setting dns servers https://review.openstack.org/508113 | 12:44 |
| *** jaypipes has joined #openstack-ironic | 12:45 | |
| *** trown has joined #openstack-ironic | 12:45 | |
| *** rloo has joined #openstack-ironic | 12:47 | |
| *** chas has joined #openstack-ironic | 12:53 | |
| *** chas has quit IRC | 12:54 | |
| *** chas has joined #openstack-ironic | 12:54 | |
| *** Goneri has joined #openstack-ironic | 12:55 | |
| *** gcb has quit IRC | 12:58 | |
| *** pcaruana has joined #openstack-ironic | 13:00 | |
| *** dpawar has joined #openstack-ironic | 13:01 | |
| *** mat128 has joined #openstack-ironic | 13:02 | |
| *** gcb has joined #openstack-ironic | 13:02 | |
| *** bfournie has joined #openstack-ironic | 13:03 | |
| *** cdearborn has joined #openstack-ironic | 13:04 | |
| *** ianychoi_ is now known as ianychoi | 13:06 | |
| *** jcoufal has joined #openstack-ironic | 13:07 | |
| *** ijw has joined #openstack-ironic | 13:12 | |
| *** trown has quit IRC | 13:13 | |
| *** chlong has quit IRC | 13:14 | |
| *** trown has joined #openstack-ironic | 13:14 | |
| *** yangyapeng has joined #openstack-ironic | 13:15 | |
| *** catintheroof has joined #openstack-ironic | 13:15 | |
| *** baoli has joined #openstack-ironic | 13:16 | |
| *** ijw has quit IRC | 13:18 | |
| *** parlos has joined #openstack-ironic | 13:19 | |
| TheJulia | good morning | 13:26 |
| TheJulia | robcresswell: the test playbook might serve as an example, it spawns a configurable number of virtual baremetal machines, at the end of testing, it removes them... | 13:28 |
| TheJulia | NobodyCam: When your up, please take a look at https://review.openstack.org/505315 | 13:30 |
| patchbot | patch 505315 - bifrost - Different pip install with/without venv | 13:30 |
| *** bfournie1 has joined #openstack-ironic | 13:31 | |
| *** jrist has quit IRC | 13:31 | |
| *** bfournie has quit IRC | 13:32 | |
| *** jrist has joined #openstack-ironic | 13:35 | |
| *** trown|brb has joined #openstack-ironic | 13:37 | |
| *** trown has quit IRC | 13:37 | |
| *** trown|brb is now known as trown | 13:38 | |
| *** livelace2 has joined #openstack-ironic | 13:42 | |
| robcresswell | TheJulia: Yeah, thats what I had been looking at :) | 13:43 |
| openstackgerrit | Merged openstack/bifrost master: Install the latest version of setuptools https://review.openstack.org/505146 | 13:45 |
| TheJulia | robcresswell: feel free to post a playbook ;) | 13:47 |
| robcresswell | TheJulia: Will do when I figure out how to Ansible | 13:47 |
| robcresswell | TheJulia: Or manage to bully sambetts into Ansible-ing for me. Whichever happens first. | 13:47 |
| TheJulia | heh | 13:47 |
| * TheJulia pull sopen the file | 13:48 | |
| TheJulia | robcresswell: delete https://github.com/openstack/bifrost/blob/master/playbooks/test-bifrost.yaml#L140 | 13:48 |
| robcresswell | Yup, was gonna cut it down so I get rid of the Zuul specific bits too. Need to go and nuke my h/w first though cause its got some old deployment packages that seems to be confusing things. | 13:50 |
| TheJulia | ahh, yeah | 13:50 |
| TheJulia | make sure virsh list is clean, and if you have pre-existing databases locally, you will likely want to drop them | 13:51 |
| TheJulia | or, rename them | 13:51 |
| robcresswell | Yeah | 13:52 |
| *** bnemec has joined #openstack-ironic | 13:53 | |
| *** alex_xu has quit IRC | 13:54 | |
| *** gcb has quit IRC | 13:54 | |
| *** alex_xu has joined #openstack-ironic | 13:54 | |
| *** alex_xu has quit IRC | 14:02 | |
| *** links has quit IRC | 14:02 | |
| *** gcb has joined #openstack-ironic | 14:02 | |
| *** ijw has joined #openstack-ironic | 14:03 | |
| *** yolanda has quit IRC | 14:08 | |
| *** ijw has quit IRC | 14:08 | |
| *** yolanda has joined #openstack-ironic | 14:08 | |
| *** crushil has joined #openstack-ironic | 14:09 | |
| jlvillal | Good morning Ironic | 14:11 |
| jlvillal | Does anyone have ideas on the random unit test failures? I updated the whiteboard about them. | 14:11 |
| sambetts | jlvillal: just looking now | 14:15 |
| jlvillal | sambetts, Thanks | 14:16 |
| *** jrist has quit IRC | 14:17 | |
| *** dpawar has quit IRC | 14:24 | |
| *** rbartal has quit IRC | 14:25 | |
| *** amarao has quit IRC | 14:25 | |
| *** amarao has joined #openstack-ironic | 14:26 | |
| *** srobert has quit IRC | 14:32 | |
| *** mrch has quit IRC | 14:34 | |
| *** ijw has joined #openstack-ironic | 14:36 | |
| *** chlong has joined #openstack-ironic | 14:39 | |
| *** ijw has quit IRC | 14:42 | |
| -openstackstatus- NOTICE: The infra team is now taking Zuul v2 offline and bringing Zuul v3 online. Please see https://docs.openstack.org/infra/manual/zuulv3.html for more information, and ask us in #openstack-infra if you have any questions. | 14:42 | |
| *** SpamapS is now known as Zuulv2 | 14:43 | |
| *** Zuulv2 is now known as SpamapS | 14:43 | |
| *** makowals has quit IRC | 14:49 | |
| rloo | hi and good morning all, jlvillal, sambetts, TheJulia, robcresswell | 14:54 |
| sambetts | o/ rloo | 14:54 |
| rloo | cores -- are you good with approving https://bugs.launchpad.net/ironic-inspector/+bug/1666232? | 14:54 |
| openstack | Launchpad bug 1666232 in Ironic Inspector "[RFE] replace testrepository with stestr" [Wishlist,In progress] - Assigned to Pavlo Shchelokovskyy (pshchelo) | 14:54 |
| sambetts | rloo: LGTM | 14:56 |
| rloo | thx sambetts, I've approved it :) | 14:57 |
| *** srobert has joined #openstack-ironic | 14:58 | |
| *** jrist has joined #openstack-ironic | 14:59 | |
| TheJulia | good morning rloo | 15:01 |
| rloo | TheJulia: :) | 15:01 |
| NobodyCam | Good Morning Ironic'ers :) | 15:04 |
| NobodyCam | TheJulia: looking | 15:04 |
| *** vsaienk0 has quit IRC | 15:04 | |
| *** jaganathan has quit IRC | 15:05 | |
| *** rpioso has joined #openstack-ironic | 15:07 | |
| TheJulia | NobodyCam: thanks! | 15:10 |
| NobodyCam | so omit is setting var = None va var = <undefined> | 15:11 |
| NobodyCam | va = vs | 15:11 |
| TheJulia | basically what it is doing now :\ | 15:12 |
| TheJulia | NobodyCam: also, https://review.openstack.org/#/c/508113/ | 15:12 |
| patchbot | patch 508113 - bifrost - Fix bug in setting dns servers | 15:12 |
| *** parlos has quit IRC | 15:13 | |
| NobodyCam | oh wow. That's def a bug. Awesome catch yolanda | 15:14 |
| NobodyCam | +a | 15:15 |
| yolanda | NobodyCam, i guess i wrote that bits, so my fault to have that bug... | 15:15 |
| NobodyCam | :) | 15:16 |
| NobodyCam | no fault. bugs happen | 15:17 |
| *** jaganathan has joined #openstack-ironic | 15:18 | |
| TheJulia | NobodyCam: So true :) | 15:19 |
| *** penick has joined #openstack-ironic | 15:25 | |
| *** penick has quit IRC | 15:29 | |
| *** penick has joined #openstack-ironic | 15:29 | |
| *** jaganathan has quit IRC | 15:31 | |
| *** jkilpatr has quit IRC | 15:34 | |
| *** crushil has quit IRC | 15:34 | |
| *** marios has quit IRC | 15:35 | |
| *** crushil has joined #openstack-ironic | 15:35 | |
| *** e0ne has quit IRC | 15:36 | |
| *** jrist has quit IRC | 15:36 | |
| *** parlos has joined #openstack-ironic | 15:45 | |
| *** rcernin has quit IRC | 15:46 | |
| *** zhangfei has joined #openstack-ironic | 15:46 | |
| *** parlos_ has joined #openstack-ironic | 15:47 | |
| *** crushil has quit IRC | 15:47 | |
| *** parlos has quit IRC | 15:49 | |
| *** parlos has joined #openstack-ironic | 15:50 | |
| *** parlos_ has quit IRC | 15:51 | |
| *** chas has quit IRC | 15:57 | |
| *** chas has joined #openstack-ironic | 15:58 | |
| *** cdearborn_ has joined #openstack-ironic | 16:00 | |
| *** cdearborn__ has joined #openstack-ironic | 16:01 | |
| *** chas has quit IRC | 16:02 | |
| *** zhangfei has quit IRC | 16:02 | |
| *** cdearborn has quit IRC | 16:04 | |
| *** cdearborn_ is now known as cdearborn | 16:04 | |
| sambetts | jlvillal: I've worked out whats causing the extra os.stat call... and I'm not sure how to avoid it... | 16:09 |
| TheJulia | sambetts: where? | 16:10 |
| *** jrist has joined #openstack-ironic | 16:10 | |
| sambetts | basically there is a deprecation warning being thrown by oslo_serialiszation /home/sam/Work/ironic/.tox/py27/local/lib/python2.7/site-packages/oslo_serialization/jsonutils.py:176: UserWarning: Cannot convert OSError() to primitive, will raise ValueError instead of warning in version 3.0 | 16:10 |
| sambetts | "instead of warning in version 3.0" % (value,)) | 16:10 |
| sambetts | and in order to get the file path part for the deprecation warning it uses os.stat | 16:11 |
| sambetts | tbh I feel like that UT code is extremely delicate mocking os.stat because os.stat is used in a lot of places in the standard.lib too | 16:12 |
| *** mjura has quit IRC | 16:12 | |
| sambetts | added by this commit: https://github.com/openstack/oslo.serialization/commit/fecad3c31fff10c8c88be64c3c845b840270d00d | 16:14 |
| *** noam_ has quit IRC | 16:17 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Deprecate the ironic CLI https://review.openstack.org/508218 | 16:17 |
| jlvillal | sambetts: Awesome! | 16:18 |
| jlvillal | sambetts: That you figured it out | 16:19 |
| sambetts | yeah :/ I think I have an idea for a solution, not sure if its the right one but :/ | 16:25 |
| *** causten has quit IRC | 16:27 | |
| *** jaganathan has joined #openstack-ironic | 16:27 | |
| jlvillal | sambetts: I'm poking away in parallel | 16:31 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Stop passing raw Exceptions as the reasons for ironic Image exceptions https://review.openstack.org/508225 | 16:33 |
| *** ralonsoh has quit IRC | 16:33 | |
| sambetts | jlvillal: ^^ | 16:33 |
| sambetts | thats my initial solution | 16:34 |
| jlvillal | sambetts: Thanks | 16:34 |
| sambetts | the only alternative I could think of was to edit the test so that it used mocks has_calls() function instead of checking the call count, but then I thought we'd have to modify every test instead of solving it in one place | 16:35 |
| jlvillal | sambetts: Where does it use os.stat() for the third time? Is that in oslo.serialization? | 16:36 |
| jlvillal | sambetts: Ah, I see in ironic/commons/exceptions.py it serializes stuff | 16:36 |
| sambetts | jlvillal: https://github.com/openstack/oslo.serialization/blob/fecad3c31fff10c8c88be64c3c845b840270d00d/oslo_serialization/jsonutils.py#L172 this line | 16:37 |
| sambetts | jlvillal: uses the warnings standard lib | 16:37 |
| sambetts | jlvillal: which under the covers uses os.stat | 16:37 |
| jlvillal | sambetts: Oh wonky :) | 16:37 |
| *** zhangfei has joined #openstack-ironic | 16:38 | |
| *** rpioso__ has joined #openstack-ironic | 16:38 | |
| sambetts | its why I'm kind of concerned with us touching/mocking something like os.stat because it such a low level python library function its used by literally everything | 16:38 |
| sambetts | it feels very fragile | 16:39 |
| *** rpioso has quit IRC | 16:39 | |
| jlvillal | sambetts: s/literally/virtually/ ;) | 16:39 |
| sambetts | looking through the python source code it feels like literally ;) | 16:40 |
| jlvillal | heh | 16:40 |
| *** penick has quit IRC | 16:44 | |
| *** karts has joined #openstack-ironic | 16:44 | |
| *** rama_y has joined #openstack-ironic | 16:45 | |
| *** trown is now known as trown|lunch | 16:46 | |
| *** karthiks has quit IRC | 16:47 | |
| *** rwsu has joined #openstack-ironic | 16:47 | |
| *** penick has joined #openstack-ironic | 16:48 | |
| *** jaganathan has quit IRC | 16:55 | |
| *** derekh has quit IRC | 16:56 | |
| jlvillal | sambetts: Tested locally. Fixes it for me. Thanks! Not sure why I don't see it in the zuul status queue though. | 16:56 |
| *** crushil has joined #openstack-ironic | 16:56 | |
| jlvillal | Maybe the Zuul v3 change-over has something to do with it | 16:56 |
| NobodyCam | rloo: is it wrong that your patch makes me :( | 16:56 |
| sambetts | jlvillal: that would be my guess | 16:57 |
| rloo | NobodyCam: the ironic cli deprecation one? No, it isn't wrong. | 16:57 |
| NobodyCam | :) yea :( | 16:57 |
| rloo | NobodyCam: come sit down and tell me why you feel :-( | 16:57 |
| *** rpioso has joined #openstack-ironic | 16:58 | |
| NobodyCam | :) | 16:58 |
| rloo | NobodyCam: is it for nostalgic reasons or ?? | 16:58 |
| NobodyCam | little of both. I am sad to see it go.. I'm also kinda sad that I will have to install os_client for my standalone ironic | 16:59 |
| *** baoli has quit IRC | 17:00 | |
| rloo | NobodyCam: wrt os_client. The plan is for a lightweight osc command that doesn't pull in all the other project commands | 17:00 |
| rloo | NobodyCam: but that osc command won't be avail in the near future. Not sure when. | 17:01 |
| *** baoli has joined #openstack-ironic | 17:01 | |
| sambetts | +1000 to wanting this ^ it'll make using osc standalone a much easier pill to swallow | 17:01 |
| NobodyCam | :) yea | 17:01 |
| sambetts | I wonder if we can get involved in any way to help push it through | 17:01 |
| * NobodyCam ready's alias ironic=`OSC baremetal $1 $2` | 17:03 | |
| rloo | sambetts: are you volunteering? I can connect you to the osc folks. I mentioned to them that I'd like it in Rocky. | 17:04 |
| rloo | NobodyCam: heh | 17:04 |
| NobodyCam | :) | 17:04 |
| sambetts | I'd be interested to see how far they've got with it, (I would have thought it would be a straight forklift of the plugin loading code but you never know) | 17:05 |
| NobodyCam | just asking (because I haven't looked) is there a matrix of old ironic cmd to new osc cmd? | 17:05 |
| rloo | sambetts: i think it involves a bit more than that, and OSC 4.0 (in the works) will make it easier, so I was fine waiting | 17:05 |
| rloo | NobodyCam: nope. Would it be useful? | 17:06 |
| NobodyCam | yes! | 17:06 |
| rloo | NobodyCam: i guess it would or you wouldn't ask... | 17:06 |
| NobodyCam | :) | 17:06 |
| rloo | NobodyCam: would it be useful to see the equivalent when issueing an ironic command, or are you good with documentation only? | 17:06 |
| rloo | NobodyCam: and thanks for making me do more work :) | 17:07 |
| NobodyCam | :( | 17:07 |
| rloo | NobodyCam: anything to help folks migrate... | 17:07 |
| NobodyCam | I was thinking of a simple pivot table type thing | 17:07 |
| rloo | NobodyCam: what is a 'pivot table'? | 17:08 |
| jlvillal | sambetts: +2 and thanks again :) | 17:08 |
| *** jaganathan has joined #openstack-ironic | 17:08 | |
| sambetts | np | 17:08 |
| sambetts | :) | 17:08 |
| jlvillal | http://zuulv3.openstack.org/ is the new status page for Zuul v3 | 17:08 |
| jlvillal | sambetts: Strange that when I only test the one unit test, it always fails. | 17:08 |
| jlvillal | sambetts: But when I do all unit tests it doesn't fail. Weird. | 17:09 |
| jlvillal | So this would always fails: tox -vvv -e py27 -- -r ironic.tests.unit.common.test_image_service.FileImageServiceTestCase.test_download_hard_link_fail | 17:09 |
| jlvillal | But just: tox | 17:09 |
| jlvillal | Would always work locally | 17:09 |
| sambetts | jlvillal: it because the warnings library internally handles the only display the deprecation warning once thing, so another test that doesn't care as probably already triggered the warning | 17:10 |
| *** dpawar has joined #openstack-ironic | 17:10 | |
| sambetts | jlvillal: which is why its racey in the CI, because it depends on the random order the tests are run in | 17:11 |
| jlvillal | sambetts: Ah! | 17:11 |
| sambetts | NobodyCam: if you hadn't arlready found it: https://docs.openstack.org/python-ironicclient/latest/cli/osc_plugin_cli.html#command-reference | 17:12 |
| sambetts | NobodyCam: the OSC commands are more self explanitary than the old ironic cli ones | 17:12 |
| rloo | sambetts, NobodyCam: do you think it might be useful to print out the equiv osc command, when a user issues an ironic command? right not, it just sez use osc: https://review.openstack.org/#/c/508218/ | 17:14 |
| patchbot | patch 508218 - python-ironicclient - Deprecate the ironic CLI | 17:14 |
| rloo | sambetts, NobodyCam: I could change it, so for each ironic command, it prints out the equiv. more work but if it would be useful..? | 17:14 |
| rloo | more to review too :) | 17:15 |
| NobodyCam | I think command-reference is good. I had not seen it. | 17:15 |
| *** baoli has quit IRC | 17:16 | |
| * sambetts ---> calling it a day and getting some dinner | 17:16 | |
| sambetts | cya'll tomorrow | 17:16 |
| NobodyCam | have a good night sambetts .. and TY | 17:16 |
| sambetts | no problem :D | 17:17 |
| NobodyCam | rloo: I think that would be too much work | 17:17 |
| *** sambetts is now known as sambetts|afk | 17:17 | |
| rloo | good night sambetts|afk | 17:17 |
| rloo | NobodyCam: ok. So you're good with the OSC docs? | 17:17 |
| NobodyCam | yea, :) | 17:17 |
| rloo | NobodyCam: great! | 17:18 |
| *** dsneddon has joined #openstack-ironic | 17:18 | |
| *** dsneddon has quit IRC | 17:18 | |
| *** dsneddon has joined #openstack-ironic | 17:18 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrades related dev documentation https://review.openstack.org/419439 | 17:21 |
| *** baoli has joined #openstack-ironic | 17:24 | |
| *** karts has quit IRC | 17:25 | |
| *** zhangfei has quit IRC | 17:27 | |
| *** retr0h has quit IRC | 17:28 | |
| *** retr0h has joined #openstack-ironic | 17:28 | |
| *** baoli has quit IRC | 17:32 | |
| *** baoli has joined #openstack-ironic | 17:32 | |
| *** karts has joined #openstack-ironic | 17:38 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Update documentation https://review.openstack.org/507927 | 17:55 |
| *** livelace2 has quit IRC | 17:55 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Deprecate the ironic CLI https://review.openstack.org/508218 | 17:56 |
| *** baoli has quit IRC | 17:56 | |
| *** baoli has joined #openstack-ironic | 17:57 | |
| *** amarao2 has joined #openstack-ironic | 17:59 | |
| *** amarao has quit IRC | 17:59 | |
| *** amarao1 has quit IRC | 18:01 | |
| *** amarao2 has quit IRC | 18:04 | |
| *** parlos has quit IRC | 18:04 | |
| *** dprince has quit IRC | 18:05 | |
| jlvillal | Just an FYI. The Zuul v3 changeover is going on now. So don't expect any of your patches to build for awhile. | 18:07 |
| *** baoli has quit IRC | 18:07 | |
| *** dprince has joined #openstack-ironic | 18:07 | |
| jlvillal | And any jobs that were building in Zuul v2, are no longer building, even if they are showing up on the old zuul status page. | 18:08 |
| *** cdearborn has quit IRC | 18:10 | |
| *** baoli has joined #openstack-ironic | 18:10 | |
| rloo | jlvillal: thx. wish they had waited to friday but anyway... | 18:10 |
| jlvillal | rloo: I wish they had waited till Saturday ;) | 18:11 |
| rloo | jlvillal: heh, yeah, but i don't expect them to work weekends ;) | 18:11 |
| *** rpioso has quit IRC | 18:11 | |
| *** dprince has quit IRC | 18:13 | |
| *** trown|lunch is now known as trown | 18:13 | |
| *** dprince has joined #openstack-ironic | 18:14 | |
| *** jkilpatr has joined #openstack-ironic | 18:14 | |
| jlvillal | rloo: TheJulia: FYI: Fix for the random unit test failures we have been seeing in the gate: https://review.openstack.org/508225 | 18:16 |
| patchbot | patch 508225 - ironic - Stop passing raw Exceptions as the reasons for iro... | 18:16 |
| rloo | jlvillal: thx for heads up, will look in a few... | 18:17 |
| *** amarao has joined #openstack-ironic | 18:20 | |
| *** rpioso has joined #openstack-ironic | 18:25 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Deprecate the ironic CLI https://review.openstack.org/508218 | 18:28 |
| *** baoli has quit IRC | 18:30 | |
| *** dprince has quit IRC | 18:31 | |
| *** ChubYann has joined #openstack-ironic | 18:34 | |
| *** baoli has joined #openstack-ironic | 18:34 | |
| *** srobert_ has joined #openstack-ironic | 18:35 | |
| *** dpawar has quit IRC | 18:35 | |
| *** srobert has quit IRC | 18:38 | |
| *** crushil_ has joined #openstack-ironic | 18:42 | |
| *** MattMan has quit IRC | 19:00 | |
| *** MattMan has joined #openstack-ironic | 19:00 | |
| *** dprince has joined #openstack-ironic | 19:01 | |
| *** bnemec has quit IRC | 19:07 | |
| *** causten has joined #openstack-ironic | 19:11 | |
| *** bnemec has joined #openstack-ironic | 19:13 | |
| *** bnemec has quit IRC | 19:16 | |
| *** rpioso__ has quit IRC | 19:17 | |
| *** e0ne has joined #openstack-ironic | 19:17 | |
| *** Sukhdev has joined #openstack-ironic | 19:21 | |
| *** crushil_ has quit IRC | 19:23 | |
| *** jaganathan has quit IRC | 19:28 | |
| *** penick_ has joined #openstack-ironic | 19:31 | |
| *** pcaruana has quit IRC | 19:31 | |
| *** srobert_ has quit IRC | 19:32 | |
| *** penick has quit IRC | 19:33 | |
| *** srobert has joined #openstack-ironic | 19:33 | |
| *** bnemec has joined #openstack-ironic | 19:35 | |
| *** etingof has joined #openstack-ironic | 19:39 | |
| *** e0ne has quit IRC | 19:40 | |
| *** e0ne has joined #openstack-ironic | 19:44 | |
| *** causten has quit IRC | 19:48 | |
| *** srobert has quit IRC | 19:53 | |
| *** srobert has joined #openstack-ironic | 19:54 | |
| *** inc0 has joined #openstack-ironic | 19:58 | |
| *** ccamacho has quit IRC | 19:59 | |
| inc0 | hey:) I need an opinion regarding dev based on Kolla-Ansible (devstack alternative) | 19:59 |
| inc0 | (we had session in PTG about that) | 19:59 |
| inc0 | so currently I'm workflow I'm experimenting with looks like that: | 20:00 |
| inc0 | 1. deploy kolla-ansible with ironic devmode | 20:00 |
| inc0 | that will create /opt/stack/src/ironic, checkout code from git there | 20:00 |
| inc0 | also create virtual env in /opt/stack/venvs/ironic | 20:01 |
| *** ijw has joined #openstack-ironic | 20:01 | |
| inc0 | install code in venv | 20:01 |
| inc0 | so when kolla starts container, it will mount this venv and do it's magic to make it work inside container | 20:01 |
| *** fellypefca has quit IRC | 20:02 | |
| inc0 | this scenario comes with one cost - it will require some binary stuff installed on host, like libcrypto for cryptography | 20:02 |
| inc0 | that means cleanup from kolla will leave some leftovers (these few binary libs) | 20:02 |
| inc0 | but all IDE's that supports venv will work fine with autocompletions and stuff | 20:03 |
| *** chlong has quit IRC | 20:03 | |
| *** e0ne has quit IRC | 20:03 | |
| inc0 | alternative is to have code on host | 20:03 |
| inc0 | but not installed | 20:03 |
| inc0 | and run pip install whenever you restart container | 20:03 |
| inc0 | then cleanups will be full, but IDEs might not work well and you won't have option to easily run your service on host (regular terminal) instead of container | 20:04 |
| inc0 | which can be useful for debugging and such | 20:04 |
| inc0 | sooo...opinions? | 20:04 |
| rloo | inc0: ironic isn't the only project that requires binary stuff installed on host, right? | 20:07 |
| inc0 | rloo: no, also you won't need to worry about stuff like tgtd or whatever | 20:07 |
| rloo | inc0: is it 'just' when using devmode, that you are trying to figure out where/how to handle these binaries? | 20:07 |
| inc0 | only host-installed things we need are things that will be required to run pip install ironic | 20:08 |
| inc0 | rloo: yeah, it's just about what lands inside container and what lands on host | 20:08 |
| inc0 | any non-devmode services (keystone and such) will have all it's deps in container | 20:08 |
| inc0 | binary or otherwise | 20:08 |
| rloo | inc0: is it only possible to have one service be in dev mode? | 20:09 |
| inc0 | yes | 20:09 |
| rloo | inc0: at most one service in devmode | 20:09 |
| *** catintheroof has quit IRC | 20:09 | |
| rloo | inc0: ah, ok | 20:09 |
| inc0 | well, you can have as much as you want | 20:09 |
| rloo | inc0: well, if you have as many as you want, and they need similar stuff on host, but diff versions? | 20:09 |
| inc0 | but I assume most of people will only care about their active project in devmode | 20:09 |
| inc0 | that's unlikely because openstack wouldn't install;) | 20:10 |
| inc0 | this dep separation thing is feature of containers, devstack share everything | 20:10 |
| rloo | inc0: oh, i thought maybe your stuff solved everything ;) | 20:10 |
| inc0 | devstack, fuel, tripleo-pre-container... | 20:10 |
| *** weshay has quit IRC | 20:10 | |
| inc0 | well in regular deployments (non-dev) everything is separated | 20:11 |
| *** weshay has joined #openstack-ironic | 20:11 | |
| inc0 | so technically this requirement isn't needed | 20:11 |
| rloo | inc0: so, i don't know the answer to your question. there are pros/cons to each | 20:11 |
| inc0 | right, that's why I'm asking for opinions;) | 20:11 |
| rloo | inc0: i'm hesitant to ask for a flag/switch. | 20:11 |
| rloo | inc0: personally, if i am debuggin, i just want to debug. so i'd want the binaries on the host | 20:12 |
| inc0 | bad idea, it would be different logic each | 20:12 |
| inc0 | well, again, by "binaries" I mean things like python-devc | 20:12 |
| inc0 | or libcrypto, libxml | 20:12 |
| inc0 | so not running things | 20:12 |
| inc0 | other than that you just have your trusty venv mechanics | 20:13 |
| rloo | inc0: what's wrong with having those on the host? can anyone assume that after testing etc, their host is 'clean'? | 20:13 |
| inc0 | source /opt/stack/venvs/ironic/bin/activate | 20:13 |
| inc0 | well, with kolla it will be pretty damn clean after testing | 20:13 |
| rloo | inc0: sambetts|afk is not around now, he is always opinionated, would be good to ask him | 20:13 |
| inc0 | that's one of values vs devstack | 20:14 |
| inc0 | you remove containers, your host is pristine | 20:14 |
| inc0 | minus these few libs | 20:14 |
| rloo | inc0: I'd go with ^^. but I don't know if I'll use your stuff, so I don't know if I count :) | 20:15 |
| inc0 | well I don't expect everyone to jump on bandwagon, but I'd love to know why not use our stuff so we can improve (later) | 20:15 |
| inc0 | right now I'm jsut gathering ideas for initial thing | 20:16 |
| rloo | inc0: ok. there are usually more folks around earlier in the day, if you want to try to gather more opinions | 20:17 |
| *** jcoufal has quit IRC | 20:24 | |
| *** rcernin has joined #openstack-ironic | 20:42 | |
| rloo | hey jlvillal, how many tests do you want for https://review.openstack.org/#/c/505837/4/ironic/tests/unit/api/controllers/v1/test_types.py? | 20:54 |
| patchbot | patch 505837 - ironic - ListType preserves the order of the input | 20:54 |
| rloo | jlvillal: want to make sure i understand so i don't have to revise more than once | 20:54 |
| jlvillal | rloo: Well the two I mentioned. I just wanted to be clear that the order changes. | 20:54 |
| rloo | jlvillal: do you mean to just add those two lines? | 20:54 |
| jlvillal | rloo: Maybe I'm being foolish | 20:54 |
| rloo | jlvillal: no comment | 20:55 |
| rloo | jlvillal: really, only the first one you suggested is needed. not sure why the 2nd one is needed | 20:55 |
| jlvillal | rloo: Okay, I'm good with the one then. | 20:56 |
| rloo | jlvillal: ok, thx | 20:56 |
| jlvillal | rloo: Might be able to be talked into zero. | 20:56 |
| rloo | jlvillal: after spending this much time on it, might as well go all the way :) | 20:56 |
| jlvillal | I just thought since showing ordering is preserved. So why I suggested it | 20:56 |
| rloo | jlvillal: it's really an indication that the original tests were incomplete | 20:57 |
| jlvillal | Heresy! ;) | 20:57 |
| rloo | jlvillal: always blame the long departed ;) | 20:59 |
| jlvillal | heh | 20:59 |
| rloo | jlvillal: although who knows, it could have been me! | 20:59 |
| * rloo wonders if there is something wrong when running unit tests seems faster than generating the docs | 21:00 | |
| *** trown is now known as trown|outtypewww | 21:01 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: ListType preserves the order of the input https://review.openstack.org/505837 | 21:01 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Remove 'next' for GET /nodes?limit=1&instance_uuid= https://review.openstack.org/506205 | 21:02 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add missing query parameters to 'next' link for GET /nodes?... https://review.openstack.org/506280 | 21:02 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add missing query parameters to 'next' link for GET /nodes?... https://review.openstack.org/506280 | 21:17 |
| *** Goneri has quit IRC | 21:29 | |
| *** bfournie1 has quit IRC | 21:31 | |
| rloo | jlvillal: btw, please backport https://review.openstack.org/#/c/507297/ | 21:36 |
| patchbot | patch 507297 - ironic - ipmitool: reboot: Don't power off node if already off | 21:36 |
| jlvillal | rloo: That's the plan. I just want to wait until the master branch patches merge. | 21:37 |
| rloo | jlvillal: they are approved. | 21:37 |
| *** causten has joined #openstack-ironic | 21:37 | |
| jlvillal | rloo: Once that happens I will backport and propose the two patches to stable/pike | 21:37 |
| rloo | jlvillal: if you wait, it'll take that much longer. on the other hand, it is thursday. so it won't get approved before monday anyway. | 21:38 |
| jlvillal | rloo: Okay. Good point. I'll work on back-porting them now | 21:38 |
| rloo | cuz only julia will be around tomorrow | 21:38 |
| *** rama_y has quit IRC | 21:40 | |
| *** chlong has joined #openstack-ironic | 21:41 | |
| *** rama_y has joined #openstack-ironic | 21:42 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic stable/pike: Reduce complexity of node_power_action() function https://review.openstack.org/508321 | 21:48 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic stable/pike: ipmitool: reboot: Don't power off node if already off https://review.openstack.org/508322 | 21:48 |
| *** causten has quit IRC | 21:49 | |
| jlvillal | rloo: ^^^^ | 21:49 |
| rloo | thx jlvillal | 21:49 |
| *** causten has joined #openstack-ironic | 21:51 | |
| *** cdearborn__ has quit IRC | 21:57 | |
| *** bnemec has quit IRC | 22:05 | |
| *** baoli has quit IRC | 22:06 | |
| *** causten has quit IRC | 22:07 | |
| *** srobert_ has joined #openstack-ironic | 22:08 | |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: WIP allow to use none auth in functional tests https://review.openstack.org/508330 | 22:09 |
| * jlvillal checks what the time is in Kharkiv | 22:11 | |
| * jlvillal discovers it is 1:11AM | 22:11 | |
| * jlvillal realizes vdrok is working way too late... | 22:11 | |
| *** srobert has quit IRC | 22:12 | |
| *** srobert_ has quit IRC | 22:13 | |
| JayF | WIP = Working Inebriated, please ignore? | 22:13 |
| JayF | code commits at 1am only mean one thing: chasing a ballmer peak :P | 22:13 |
| JayF | (citation: http://www.xkcd.com/323/) | 22:14 |
| *** r-daneel has joined #openstack-ironic | 22:18 | |
| *** bfournie has joined #openstack-ironic | 22:21 | |
| *** catintheroof has joined #openstack-ironic | 22:24 | |
| *** r-daneel has quit IRC | 22:29 | |
| vdrok | jlvillal: JayF :D what can be better than writing some test patches before going to bed | 22:35 |
| jlvillal | JayF: heh, just now read it :) | 22:35 |
| jlvillal | vdrok: Go to sleep! ;) | 22:35 |
| *** penick_ has quit IRC | 22:36 | |
| * vdrok is vacationing in Croatia since today | 22:37 | |
| vdrok | So it's not that late yet :) | 22:37 |
| *** penick has joined #openstack-ironic | 22:38 | |
| JayF | maybe not late, but doesn't sound like much of a vacation | 22:39 |
| JayF | if you're pushing up code | 22:39 |
| JayF | haha | 22:39 |
| vdrok | :) | 22:40 |
| *** bfournie has quit IRC | 22:48 | |
| *** bfournie has joined #openstack-ironic | 22:49 | |
| *** mat128 has quit IRC | 22:58 | |
| *** masber has quit IRC | 23:02 | |
| *** Goneri has joined #openstack-ironic | 23:04 | |
| *** causten has joined #openstack-ironic | 23:10 | |
| *** crushil has quit IRC | 23:13 | |
| *** causten_ has joined #openstack-ironic | 23:14 | |
| *** MattMan has quit IRC | 23:17 | |
| *** causten has quit IRC | 23:18 | |
| *** rloo has quit IRC | 23:20 | |
| *** rpioso has quit IRC | 23:23 | |
| *** Goneri has quit IRC | 23:31 | |
| *** jaypipes has quit IRC | 23:42 | |
| *** Goneri has joined #openstack-ironic | 23:52 | |
| *** causten_ has quit IRC | 23:53 | |
| *** rpioso has joined #openstack-ironic | 23:58 | |
| *** baoli has joined #openstack-ironic | 23:59 | |
| *** Goneri has quit IRC | 23:59 | |
| *** rpioso has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!