| jroll | hm, anyone around from oneview? question on dynamic allocation reno | 00:01 |
|---|---|---|
| JayF | jroll: I reviewed some of it, maybe I can help? | 00:02 |
| jroll | JayF: "pre-allocation" is deprecated, but there's no recommended action for ops to take | 00:03 |
| JayF | jroll: change to dynamic allocation | 00:03 |
| jroll | is that it, just change a boolean? nothing else to do to prepare for it? | 00:04 |
| JayF | jroll: I believe it's laid out how in the documentation, but imbw | 00:04 |
| jroll | it just says set the boolean, but doesn't say if there's any other action to take | 00:04 |
| jroll | in which case why is there a config | 00:04 |
| JayF | dyn allocation is racier if you don't run the ironic-oneviewd | 00:05 |
| JayF | so I presumed it was there to allow operators time to setup that infra | 00:05 |
| jroll | aha | 00:05 |
| JayF | but it sounds like it could use some clarification in the docs, not just in the reno? Maybe file a bug to have the upgrade steps documented in the driver docs? | 00:05 |
| *** PollyZ has quit IRC | 00:06 | |
| jroll | there's a mention that links to https://pypi.python.org/pypi/ironic-oneviewd/ | 00:06 |
| jroll | well, right now I'm focusing on the release, and I hate deprecation warnings with no action | 00:06 |
| * jroll makes a note to ask in the morning | 00:07 | |
| jroll | this reno says nothing about what the change actually is | 00:07 |
| JayF | yeah reading through the docs now | 00:07 |
| JayF | there is no upgrade steps at all | 00:07 |
| JayF | no real call to action | 00:07 |
| jroll | https://github.com/openstack/ironic/blob/master/releasenotes/notes/add-dynamic-allocation-feature-2fd6b4df7943f178.yaml | 00:07 |
| JayF | it does a reasonable job of explaining what's up, but nothing about the transition | 00:07 |
| jroll | as an operator, that means nothing to me | 00:07 |
| JayF | yeah, you're right | 00:08 |
| JayF | I'm going to also file a bug to add upgrade/transition steps to the oneview docs | 00:08 |
| JayF | I should've caught that :( | 00:08 |
| jroll | cool, thank you :) | 00:08 |
| JayF | both cases, I think | 00:08 |
| jroll | it happens | 00:08 |
| jroll | we get anxious to merge long-hanging things | 00:08 |
| JayF | https://bugs.launchpad.net/ironic/+bug/1611155 | 00:10 |
| openstack | Launchpad bug 1611155 in Ironic "No upgrade/transition steps in oneview driver docs" [Undecided,New] - Assigned to Thiago Paiva Brito (thiagop) | 00:10 |
| JayF | then when we get that in, you can just link the docs up in the release notes | 00:10 |
| jroll | cool | 00:11 |
| jroll | thank you | 00:11 |
| JayF | np | 00:11 |
| JayF | Anything else I should poke at before I head out for the day? | 00:12 |
| jroll | idk, I'm just cleaning up renos | 00:13 |
| JayF | Cool; hope your evening is better than your daytime was :) | 00:14 |
| jroll | hah, it's fine, have a good one \o | 00:15 |
| *** adnavare has joined #openstack-ironic | 00:22 | |
| *** aNupoisc has quit IRC | 00:22 | |
| *** PollyZ has joined #openstack-ironic | 00:22 | |
| *** syed_ has quit IRC | 00:32 | |
| *** hoangcx_ has joined #openstack-ironic | 00:44 | |
| *** xuao has joined #openstack-ironic | 00:51 | |
| *** praneshp has quit IRC | 00:51 | |
| *** aNupoisc has joined #openstack-ironic | 00:51 | |
| *** adnavare has quit IRC | 00:53 | |
| *** mtanino has joined #openstack-ironic | 00:54 | |
| *** hoangcx has quit IRC | 00:54 | |
| *** hoangcx has joined #openstack-ironic | 00:55 | |
| *** rloo has quit IRC | 00:56 | |
| *** Sukhdev has quit IRC | 00:57 | |
| *** bvandewa has joined #openstack-ironic | 01:02 | |
| *** zhangjl has joined #openstack-ironic | 01:07 | |
| *** hoangcx has quit IRC | 01:09 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor multitenant networking release notes https://review.openstack.org/352659 | 01:10 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up release notes for 6.1.0 https://review.openstack.org/352660 | 01:10 |
| jroll | ^ could use some reviews on those soon, please :) | 01:10 |
| *** hoangcx has joined #openstack-ironic | 01:10 | |
| *** moshele has joined #openstack-ironic | 01:15 | |
| *** hoangcx has quit IRC | 01:26 | |
| *** sdake has quit IRC | 01:27 | |
| *** chlong has joined #openstack-ironic | 01:27 | |
| *** hoangcx has joined #openstack-ironic | 01:27 | |
| *** sdake has joined #openstack-ironic | 01:29 | |
| *** phuongnh has joined #openstack-ironic | 01:30 | |
| *** jrist has quit IRC | 01:36 | |
| *** bvandewa has quit IRC | 01:36 | |
| *** bvandewa has joined #openstack-ironic | 01:36 | |
| *** bvandewa has quit IRC | 01:36 | |
| *** bvandewa has joined #openstack-ironic | 01:36 | |
| *** PollyZ has quit IRC | 01:41 | |
| *** amitkqed has quit IRC | 01:43 | |
| *** nicodemos has quit IRC | 01:43 | |
| *** amitkqed has joined #openstack-ironic | 01:43 | |
| *** nicodemos has joined #openstack-ironic | 01:44 | |
| *** sabeen has joined #openstack-ironic | 01:44 | |
| *** rbudden has quit IRC | 01:47 | |
| *** jrist has joined #openstack-ironic | 01:50 | |
| *** jrist has quit IRC | 01:50 | |
| *** jrist has joined #openstack-ironic | 01:50 | |
| *** PollyZ has joined #openstack-ironic | 01:53 | |
| *** baoli has joined #openstack-ironic | 01:58 | |
| *** moshele has quit IRC | 01:58 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 02:03 |
| *** hoangcx has quit IRC | 02:12 | |
| *** jrist has quit IRC | 02:14 | |
| *** jrist has joined #openstack-ironic | 02:26 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Use TestCase.config() instead of set_override() https://review.openstack.org/352408 | 02:31 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Improve TestCase.config() to pass enforce_type=True https://review.openstack.org/243430 | 02:31 |
| *** jabroney has quit IRC | 02:31 | |
| *** PollyZ has quit IRC | 02:35 | |
| *** joprovost has joined #openstack-ironic | 02:35 | |
| *** PollyZ has joined #openstack-ironic | 02:36 | |
| *** baoli has quit IRC | 02:43 | |
| *** fragatina has quit IRC | 02:44 | |
| *** hoangcx_ is now known as hoangcx | 02:47 | |
| *** PollyZ has quit IRC | 02:47 | |
| *** PollyZ has joined #openstack-ironic | 02:47 | |
| *** hshiina has joined #openstack-ironic | 02:53 | |
| *** UForgotten has quit IRC | 02:57 | |
| *** UForgotten has joined #openstack-ironic | 02:57 | |
| *** milan has quit IRC | 03:00 | |
| *** hjanssen-hpe has quit IRC | 03:04 | |
| *** hj-hpe has quit IRC | 03:04 | |
| *** tangchen has joined #openstack-ironic | 03:06 | |
| *** milan has joined #openstack-ironic | 03:10 | |
| openstackgerrit | Merged openstack/ironic: Metrics for agent client https://review.openstack.org/352149 | 03:17 |
| *** rama_y has joined #openstack-ironic | 03:18 | |
| *** yibo has quit IRC | 03:20 | |
| *** yibo has joined #openstack-ironic | 03:21 | |
| *** ijw has quit IRC | 03:23 | |
| *** rama_y has quit IRC | 03:25 | |
| *** rama_y has joined #openstack-ironic | 03:25 | |
| *** ijw has joined #openstack-ironic | 03:27 | |
| *** rama_y has quit IRC | 03:31 | |
| *** rama_y has joined #openstack-ironic | 03:31 | |
| *** ijw has quit IRC | 03:32 | |
| *** PollyZ_ has joined #openstack-ironic | 03:36 | |
| *** joprovost has quit IRC | 03:37 | |
| *** PollyZ has quit IRC | 03:38 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-specs: Add synchronize-events-with-neutron spec. https://review.openstack.org/343684 | 03:42 |
| *** ijw has joined #openstack-ironic | 03:43 | |
| *** tangchen has quit IRC | 03:44 | |
| *** ijw has quit IRC | 03:48 | |
| *** tangchen has joined #openstack-ironic | 03:49 | |
| *** aNupoisc has quit IRC | 03:49 | |
| lazy_prince | lucas-dinner: I was trying pxe_ipmitool to deploy uefi image on the target node.. However, I noticed that the PXE config is for node in neutron port is for uefi but the server boots in BIOS mode.. Did I miss anything..? | 03:50 |
| lazy_prince | this was when Ironic is trying to boot node with deploy image.. | 03:51 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/351987 | 03:53 |
| *** sabeen has quit IRC | 03:58 | |
| *** sabeen has joined #openstack-ironic | 04:00 | |
| *** PollyZ_ has quit IRC | 04:07 | |
| *** xuao has quit IRC | 04:10 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 04:10 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 04:11 |
| *** links has joined #openstack-ironic | 04:11 | |
| *** PollyZ has joined #openstack-ironic | 04:28 | |
| *** PollyZ_ has joined #openstack-ironic | 04:34 | |
| *** praneshp has joined #openstack-ironic | 04:35 | |
| *** appu103_ has joined #openstack-ironic | 04:36 | |
| *** PollyZ has quit IRC | 04:36 | |
| *** rama_y has quit IRC | 04:46 | |
| *** xuao has joined #openstack-ironic | 04:47 | |
| yibo | hello dtantsur,lucasagomes, what's your opinion about https://bugs.launchpad.net/ironic/+bug/1582964? it's okay if we're not ready to accept this feature. | 04:50 |
| openstack | Launchpad bug 1582964 in Ironic "[RFE] Support multi arch deployments" [Wishlist,In progress] - Assigned to Yibo Cai (cyb70289) | 04:50 |
| *** bvandewa has quit IRC | 04:54 | |
| *** bvandewa has joined #openstack-ironic | 04:55 | |
| *** bvandewa has quit IRC | 04:55 | |
| *** vmud213 has joined #openstack-ironic | 04:56 | |
| *** bvandewa has joined #openstack-ironic | 04:58 | |
| openstackgerrit | Merged openstack/python-oneviewclient: Raise Not Authorized Exception when connection request is forbidden https://review.openstack.org/343550 | 04:58 |
| *** mtanino has quit IRC | 05:01 | |
| *** moshele has joined #openstack-ironic | 05:03 | |
| *** bvandewa has quit IRC | 05:08 | |
| *** appu103_ has quit IRC | 05:09 | |
| *** aparnav has joined #openstack-ironic | 05:09 | |
| *** david-lyle_ has joined #openstack-ironic | 05:11 | |
| *** tangchen_ has joined #openstack-ironic | 05:13 | |
| *** david-lyle has quit IRC | 05:13 | |
| *** rcernin has joined #openstack-ironic | 05:21 | |
| *** PollyZ_ has quit IRC | 05:25 | |
| *** sdake has quit IRC | 05:26 | |
| *** jtomasek_ has joined #openstack-ironic | 05:26 | |
| *** jtomasek has quit IRC | 05:26 | |
| *** moshele has quit IRC | 05:27 | |
| *** moshele has joined #openstack-ironic | 05:28 | |
| *** bvandewa has joined #openstack-ironic | 05:28 | |
| *** sdake has joined #openstack-ironic | 05:29 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move "server_profile_template_uri" to REQUIRED_ON_PROPERTIES https://review.openstack.org/347127 | 05:32 |
| *** ChubYann has quit IRC | 05:35 | |
| *** bvandewa has quit IRC | 05:36 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Trivial: Remove useless function call in glance service test https://review.openstack.org/352231 | 05:36 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Pass through oslo.context handled params in RequestContext https://review.openstack.org/345758 | 05:37 |
| *** jtomasek_ is now known as jtomasek | 05:37 | |
| *** bvandewa has joined #openstack-ironic | 05:44 | |
| *** xuao has quit IRC | 05:46 | |
| *** bvandewa has quit IRC | 05:47 | |
| *** zhangjl1 has joined #openstack-ironic | 05:47 | |
| *** bvandewa has joined #openstack-ironic | 05:47 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Split API root unit and functional tests https://review.openstack.org/348965 | 05:47 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move api driver tests to functional dir https://review.openstack.org/350365 | 05:47 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Prepare for functional tests https://review.openstack.org/349421 | 05:47 |
| *** bvandewa has quit IRC | 05:48 | |
| *** bvandewa has joined #openstack-ironic | 05:48 | |
| *** zhangjl has quit IRC | 05:49 | |
| *** bvandewa has quit IRC | 05:50 | |
| *** bvandewa has joined #openstack-ironic | 05:50 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 05:51 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update devstack configure_ironic_ssh_keypair https://review.openstack.org/352714 | 05:51 |
| *** bvandewa_ has joined #openstack-ironic | 05:52 | |
| *** bvandewa has quit IRC | 05:53 | |
| *** bvandewa_ has quit IRC | 05:53 | |
| *** vmud213 has quit IRC | 05:53 | |
| *** bvandewa has joined #openstack-ironic | 05:54 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update devstack configure_ironic_ssh_keypair https://review.openstack.org/352714 | 05:56 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 05:56 |
| *** xuao has joined #openstack-ironic | 05:56 | |
| *** bvandewa has quit IRC | 05:59 | |
| *** bvandewa has joined #openstack-ironic | 06:00 | |
| openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Fix i18n problems in shell.py https://review.openstack.org/349343 | 06:04 |
| *** bvandewa has quit IRC | 06:05 | |
| *** vmud213 has joined #openstack-ironic | 06:07 | |
| *** mjura has joined #openstack-ironic | 06:10 | |
| *** bvandewa has joined #openstack-ironic | 06:15 | |
| *** bvandewa has quit IRC | 06:15 | |
| *** appu1034 has joined #openstack-ironic | 06:16 | |
| *** aparnav has quit IRC | 06:16 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic: Remove duplicate parameters from local.conf example https://review.openstack.org/350970 | 06:17 |
| *** bvandewa_ has joined #openstack-ironic | 06:17 | |
| *** bvandewa_ has quit IRC | 06:22 | |
| *** praneshp has quit IRC | 06:25 | |
| *** ljjjustin has quit IRC | 06:26 | |
| *** alineb has quit IRC | 06:27 | |
| *** ljjjustin has joined #openstack-ironic | 06:29 | |
| *** chlong is now known as chlong|mtg | 06:32 | |
| *** sabeen has quit IRC | 06:32 | |
| *** sdake has quit IRC | 06:38 | |
| *** pcaruana has joined #openstack-ironic | 06:39 | |
| *** tesseract- has joined #openstack-ironic | 06:45 | |
| *** bvandewa has joined #openstack-ironic | 06:51 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 06:51 |
| *** bvandewa has quit IRC | 06:52 | |
| *** bvandewa has joined #openstack-ironic | 06:54 | |
| *** rbartal has joined #openstack-ironic | 06:57 | |
| *** bvandewa has quit IRC | 06:57 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 07:06 |
| *** bvandewa has joined #openstack-ironic | 07:08 | |
| *** aparnav has joined #openstack-ironic | 07:17 | |
| *** appu1034 has quit IRC | 07:17 | |
| *** mgoddard has joined #openstack-ironic | 07:18 | |
| *** stendulker has joined #openstack-ironic | 07:19 | |
| *** daemontool has joined #openstack-ironic | 07:21 | |
| *** xuao has quit IRC | 07:24 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Timing metrics: iLO drivers https://review.openstack.org/352745 | 07:25 |
| *** daemontool_ has joined #openstack-ironic | 07:28 | |
| *** daemontool has quit IRC | 07:28 | |
| *** jpich has joined #openstack-ironic | 07:29 | |
| *** hshiina has quit IRC | 07:32 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Allow using introspection data in condition's 'value' field https://review.openstack.org/345535 | 07:32 |
| *** david-lyle has joined #openstack-ironic | 07:43 | |
| *** david-lyle_ has quit IRC | 07:46 | |
| *** bvandewa has quit IRC | 07:48 | |
| *** sdake has joined #openstack-ironic | 07:51 | |
| *** hshiina has joined #openstack-ironic | 07:51 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic: test_console_utils: using mock_open for builtin open() https://review.openstack.org/352756 | 07:53 |
| *** ohamada has joined #openstack-ironic | 07:55 | |
| *** ohamada has quit IRC | 07:55 | |
| *** ohamada has joined #openstack-ironic | 07:56 | |
| *** ifarkas_afk is now known as ifarkas | 07:59 | |
| *** lucas-dinner is now known as lucasagomes | 07:59 | |
| lucasagomes | morning all | 07:59 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| vdrok | morning ironic and lucasagomes :) | 08:01 |
| vsaienk0 | morning lucasagomes, vdrok, sambetts and all Ironic"ers! | 08:02 |
| *** e0ne has joined #openstack-ironic | 08:03 | |
| *** athomas has joined #openstack-ironic | 08:06 | |
| *** sdake has quit IRC | 08:06 | |
| *** tangchen has quit IRC | 08:06 | |
| *** tangchen_ has quit IRC | 08:08 | |
| *** pece has joined #openstack-ironic | 08:08 | |
| *** Romanenko_K has joined #openstack-ironic | 08:08 | |
| stendulker | morning all | 08:09 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:10 |
| *** ljjjustin has quit IRC | 08:11 | |
| *** moshele has quit IRC | 08:11 | |
| *** moshele has joined #openstack-ironic | 08:11 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-python-agent: Parallel erase disk devices https://review.openstack.org/346723 | 08:13 |
| *** vmud213 has quit IRC | 08:13 | |
| *** ljjjustin has joined #openstack-ironic | 08:13 | |
| *** alineb has joined #openstack-ironic | 08:14 | |
| alineb | morning all | 08:14 |
| *** phuongnh has quit IRC | 08:17 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic: test_console_utils: using mock_open for builtin open() https://review.openstack.org/352756 | 08:18 |
| ifarkas | morning all | 08:18 |
| *** Goneri has joined #openstack-ironic | 08:20 | |
| *** vmud213 has joined #openstack-ironic | 08:22 | |
| *** tangchen has joined #openstack-ironic | 08:22 | |
| *** amoralej|off is now known as amoralej | 08:23 | |
| vdrok | morning stendulker alineb and ifarkas | 08:28 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Split API root unit and functional tests https://review.openstack.org/348965 | 08:31 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move api driver tests to functional dir https://review.openstack.org/350365 | 08:31 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Prepare for functional tests https://review.openstack.org/349421 | 08:31 |
| *** milan has quit IRC | 08:37 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Verify JSON response of chassis commands https://review.openstack.org/336596 | 08:42 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 08:48 |
| *** bvandewa has joined #openstack-ironic | 08:48 | |
| *** milan has joined #openstack-ironic | 08:49 | |
| milan | morning Ironic! :) | 08:52 |
| *** bvandewa has quit IRC | 08:54 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add warning message for baremetal node unset command https://review.openstack.org/347856 | 08:55 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Document retrieving logs from the deploy ramdisk https://review.openstack.org/352483 | 08:57 |
| *** xuao has joined #openstack-ironic | 08:59 | |
| *** xuao has quit IRC | 08:59 | |
| *** xuao has joined #openstack-ironic | 09:00 | |
| lucasagomes | if you have a time mind taking a look at this patch https://review.openstack.org/#/c/344164/ ? it's fairly small and already +2'd :-) thanks! | 09:00 |
| openstackgerrit | Gleb Stepanov proposed openstack/ironic: Check keyword arguments https://review.openstack.org/351677 | 09:01 |
| *** moshele has quit IRC | 09:02 | |
| *** moshele has joined #openstack-ironic | 09:03 | |
| moshele | lucasagomes, vdrok: hi, can you please review the infiniband patcheshttps://review.openstack.org/#/c/264263/ https://review.openstack.org/#/c/264257/ | 09:05 |
| lucasagomes | moshele, +1 on my list | 09:06 |
| lucasagomes | lazy_prince, unfortunately pxe_ipmitool can't change the boot mode from BIOS to UEFI on the fly (like iLO does). You will have to change the mode manually :-/ | 09:07 |
| *** milan has quit IRC | 09:07 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 09:08 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Ignore required_services for multinode topology https://review.openstack.org/352793 | 09:08 |
| moshele | lucasagomes: thanks: ) | 09:08 |
| *** electrofelix has joined #openstack-ironic | 09:08 | |
| *** chlong|mtg has quit IRC | 09:10 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 09:16 |
| *** mgould|afk is now known as mgould | 09:19 | |
| mgould | morning Ironic! | 09:19 |
| sambetts | Morning all | 09:20 |
| sambetts | I hope I didn't miss to much yesterday | 09:20 |
| mgould | morning sambetts | 09:20 |
| sambetts | hey mgould | 09:21 |
| lazy_prince | lucasagomes: actually I did set the mode to uefi and then did nova boot.. but ipmi changed it back to bios for some reason.. | 09:21 |
| lucasagomes | ouch, odd | 09:22 |
| * lucasagomes haven't seem that | 09:22 | |
| lazy_prince | I even have flavour and node tagged with uefi but still.. and that the reason I pinged you.. I think I will submit a bug on this.. | 09:23 |
| pas-ha | morning all | 09:24 |
| lazy_prince | ohh.. btw, that dib patch worked for me when using ubuntu uefi image.. | 09:24 |
| openstackgerrit | Merged openstack/ironic-inspector: Provide meaningful error messages in functional tests https://review.openstack.org/349924 | 09:25 |
| lucasagomes | lazy_prince, yeah a bug would be great | 09:26 |
| lucasagomes | lazy_prince, I wonder if the machines goes back to BIOS mode due the boot device that we set | 09:26 |
| sambetts | lucasagomes: I think perhaps if you don't passed the ufiboot option to the bootdev command it might not set it | 09:30 |
| sambetts | efiboot* | 09:30 |
| sambetts | lucasagomes: "In case your server continues to boot with PXE as higher priority, IPMI can set the boot device: | 09:31 |
| sambetts | ipmitool> chassis bootdev disk options=efiboot | 09:31 |
| sambetts | Oh yeah, that efiboot option was so important, because otherwise my server switches down to legacy BIOS mode if you try to override its boot default. " | 09:31 |
| lucasagomes | sambetts, a-ha that's a good call | 09:32 |
| * lucasagomes checks that | 09:32 | |
| sambetts | the only extra option we pass right now is the persistent or not option | 09:32 |
| sambetts | lucasagomes: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipmitool.py#L917 | 09:33 |
| lucasagomes | sambetts, yeah we probably should change that to also pass the efiboot | 09:33 |
| lucasagomes | sambetts, thanks for the info, I will take a look at it | 09:33 |
| lucasagomes | lazy_prince, ^ you may find that interesting | 09:33 |
| mgould | sambetts: that's... really helpful of it. not. | 09:35 |
| sambetts | yeah .. | 09:35 |
| mgould | https://www.happyassassin.net/2013/05/03/a-day-in-the-life-of-a-firmware-engineer/ | 09:35 |
| sambetts | mgould: ahaha | 09:37 |
| lazy_prince | lucasagomes: yeah.. probably thats the cause.. | 09:39 |
| * lazy_prince starts submitting the bug.. | 09:39 | |
| *** hshiina has quit IRC | 09:40 | |
| openstackgerrit | Merged openstack/ironic: Timing metrics: pxe boot and iscsi deploy driver https://review.openstack.org/350317 | 09:43 |
| *** moshele has left #openstack-ironic | 09:44 | |
| *** moshele has joined #openstack-ironic | 09:46 | |
| *** Tuan has joined #openstack-ironic | 09:47 | |
| *** chlong|mtg has joined #openstack-ironic | 09:47 | |
| Tuan | Hello | 09:48 |
| Tuan | Anyone knows can we deploy whole disk image using pxe_* driver? | 09:48 |
| *** Tuan is now known as Tuan_LA | 09:49 | |
| lazy_prince | Well.. I just deployed whole disk image using pxe_ipmitool driver.. | 09:49 |
| lazy_prince | Tuan_LA: ^^ | 09:50 |
| * mgould hopes any firmware engineers in the channel weren't offended :-) | 09:50 | |
| lazy_prince | lucasagomes: sambetts: https://bugs.launchpad.net/ironic/+bug/1611306 | 09:50 |
| openstack | Launchpad bug 1611306 in Ironic "pxe_ipmitool does not boot the server in uefi mode" [Undecided,New] | 09:50 |
| Tuan_LA | lazy_prince: I just got an error | 09:50 |
| lucasagomes | lazy_prince, thanks | 09:50 |
| lazy_prince | Tuan_LA: are you on master branch..? | 09:51 |
| lucasagomes | lazy_prince, 1 sec I will push a patch, can you try it out please? | 09:51 |
| Tuan_LA | "Unable to find requested MBR signature" | 09:51 |
| lazy_prince | lucasagomes: I can try that.. | 09:51 |
| lucasagomes | lazy_prince, cool. thanks! 2 minutes | 09:52 |
| *** xuao has quit IRC | 09:52 | |
| lazy_prince | Tuan_LA: how did you create your image..? | 09:52 |
| Tuan_LA | lazy_prince, lucasagomes: Yes, I got similar situation | 09:53 |
| lazy_prince | Tuan_LA: i hope you used vm element while creating whole disk image using dib.. | 09:53 |
| Tuan_LA | lazy_prince: I follow guide http://docs.openstack.org/image-guide/windows-image.html with some change | 09:54 |
| lazy_prince | ohh.. wow... Windows Image..!! | 09:55 |
| Tuan_LA | But in my case: i was using cirros __disk | 09:55 |
| * lazy_prince checking the usl | 09:55 | |
| lazy_prince | s/usl/url/ | 09:55 |
| lucasagomes | Tuan_LA, a-ha, thanks for the feedback. Yeah it seems it's def something we missed to add for ipmitool | 09:56 |
| Tuan_LA | I was successful deploy cirros- uec firstly, | 09:58 |
| Tuan_LA | after that i change to cirros-disk image (whole disk image) and I meet that problem | 09:59 |
| lazy_prince | you could always inspect the image to see if it has the MBR.. | 09:59 |
| *** dtantsur|sick is now known as dtantsur | 10:00 | |
| dtantsur | Morning/afternoon Ironic | 10:00 |
| mgould | morning dtantsur | 10:00 |
| Tuan_LA | lazy_prince: How can I check that? | 10:01 |
| lazy_prince | Tuan_LA: what the image format..? based on that you could use nbd tools/guestfs tools/fdisk.. so many options.. | 10:02 |
| Tuan_LA | firstly, I used default image in devstack to test. | 10:02 |
| Tuan_LA | lazy_prince: thanks, I will read more and try that | 10:03 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: IPMITool: Check the boot mode when setting the boot device https://review.openstack.org/352820 | 10:03 |
| lucasagomes | lazy_prince, Tuan_LA ^ | 10:03 |
| Tuan_LA | lucasagomes: How can i use new test to test my system | 10:04 |
| lucasagomes | Tuan_LA, to use the patch? You just need to patch the ironic code and restart the services | 10:05 |
| *** Nakato has quit IRC | 10:05 | |
| lazy_prince | lucasagomes: I will wait for the gates first.. :) | 10:05 |
| lucasagomes | fair enough :-) | 10:05 |
| *** early has quit IRC | 10:06 | |
| lucasagomes | lazy_prince, but to be honest, we don't have a gate job for uefi | 10:06 |
| lucasagomes | we def should | 10:06 |
| *** early has joined #openstack-ironic | 10:06 | |
| Tuan_LA | lucasagomes: I was read about case of sambetts at 27-04-2016 | 10:06 |
| *** Nakato has joined #openstack-ironic | 10:06 | |
| *** xuao has joined #openstack-ironic | 10:06 | |
| lazy_prince | KVM VMs can boot into uefi.. just that they cannot maintain their boot order... | 10:07 |
| *** hoangcx has quit IRC | 10:07 | |
| lucasagomes | lazy_prince, +1 it would be great if we could set something like that in the gate | 10:07 |
| Tuan_LA | I just wonder, Does he have solution for that problem | 10:08 |
| Tuan_LA | lazy_prince: It mean I have to using uefi for booting? | 10:08 |
| lucasagomes | Tuan_LA, I need to read the logs from that date. I've no idea which case is that | 10:09 |
| lazy_prince | Tuan_LA: sorry.. I guess in your case, you have image issue.. while in my case, I have some boot issues (as in uefi/bios boot) | 10:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 10:10 |
| Tuan_LA | lucasagomes: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2016-04-06.log.html | 10:10 |
| Tuan_LA | lazy_prince: I used bios boot | 10:11 |
| *** athomas has quit IRC | 10:11 | |
| lazy_prince | CAn you just use your image to boot a VM without ironic.. just to ensure your image is built properly.. | 10:12 |
| Tuan_LA | lazy_prince: I used a default image, but i will try difference image | 10:12 |
| Tuan_LA | lazy_prince: Thanks you so much for you help.. | 10:13 |
| *** jcoufal has joined #openstack-ironic | 10:15 | |
| Tuan_LA | lazy_prince: I will try it, I'm not sure about that but do you have any guide to create right image | 10:15 |
| Tuan_LA | lazy_prince: would you give any suggestion instead of Openstack install guide http://docs.openstack.org/developer/ironic/deploy/install-guide.html | 10:17 |
| sambetts | Tuan_LA: I think the MBR signiture issue I was seeing turned out to be caused by cloud-init expanding the disk on first boot to be bigger than MBR could support | 10:17 |
| sambetts | lucasagomes: ^ do you remember talking about that | 10:18 |
| *** athomas has joined #openstack-ironic | 10:18 | |
| lazy_prince | Tuan_LA: if you want, you could use https://github.com/stackforge/windows-diskimage-builder to generate windows disk images.. Although, it is no longer maintained, I guess, it still works... | 10:18 |
| Tuan_LA | lazy_prince: Thank for you help.I will try it. | 10:19 |
| openstackgerrit | Aline Bousquet proposed openstack/ironic: Add metrics for the ipminative driver https://review.openstack.org/352829 | 10:20 |
| Tuan_LA | sambetts: would you tell me more about the issue. Do you have any suggestion can solve this problem? | 10:22 |
| *** Lord_Phoenix has joined #openstack-ironic | 10:23 | |
| *** xuao has quit IRC | 10:23 | |
| openstackgerrit | Gleb Stepanov proposed openstack/ironic: Check keyword arguments https://review.openstack.org/351677 | 10:23 |
| lucasagomes | sambetts, oh yes! Cause the disk was larger than 2tb | 10:24 |
| *** zhangjl1 has quit IRC | 10:24 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/351987 | 10:25 |
| lucasagomes | Tuan_LA, basically the disk was larger than 2tb (is that ur case ?) so when cloud-init expanded the root partition until the end of the disk, the MBR couldn't represent the size of that partition anymore and the machine failed to boot | 10:25 |
| sambetts | Tuan_LA: you can either ensure that your boot disk is small enough to support MBR when in full use, or force ironic to use GPT, or turn of cloud-init expansion | 10:25 |
| lucasagomes | Tuan_LA, sambetts the solution for that was to introduce a way to force using GPT even for machines in legacy BIOS mode | 10:25 |
| lucasagomes | so you can do that in Ironic now by setting "disk_label:gpt" as a capability to it (make sure the flavor matches) | 10:26 |
| lucasagomes | Tuan_LA, http://docs.openstack.org/developer/ironic/deploy/install-guide.html#choosing-the-disk-label | 10:26 |
| Tuan_LA | lucasagomes: thanks you so much for your help, I will read more about that and try it again base on your suggestion. | 10:28 |
| lucasagomes | Tuan_LA, cool, make sure that's the case tho. Check the size of the disk you are trying to deploy the image onto | 10:28 |
| Tuan_LA | sambetts: Hopefully that we can through this problem. | 10:28 |
| sambetts | :) | 10:29 |
| Tuan_LA | Good bye, | 10:30 |
| *** Tuan_LA has quit IRC | 10:30 | |
| sambetts | thiagop: Good luck | 10:30 |
| sambetts | thiagop: sorry for the noise that was meant for Tuan_LA :-P | 10:30 |
| dtantsur | lucasagomes, do you think you could just approve https://review.openstack.org/#/c/336547/ ? | 10:32 |
| lucasagomes | dtantsur, done | 10:35 |
| dtantsur | \o/ | 10:35 |
| *** vmud213 has quit IRC | 10:37 | |
| *** appu103_ has joined #openstack-ironic | 10:40 | |
| *** aparnav has quit IRC | 10:40 | |
| dtantsur | lucasagomes, could you please check https://review.openstack.org/349510 too? it's small but it's important to do now to do the driver composition cleanly in the next cycle | 10:43 |
| *** jcoufal has quit IRC | 10:44 | |
| lucasagomes | dtantsur, +1 will, just 1 sec finishing something up here | 10:44 |
| dtantsur | sure, sure, I just want it done before Thursday | 10:45 |
| *** lucasagomes has quit IRC | 10:51 | |
| *** sdake has joined #openstack-ironic | 10:53 | |
| *** e0ne has quit IRC | 11:04 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Add erase_devices_metadata cleaning step https://review.openstack.org/344164 | 11:06 |
| *** sdake has quit IRC | 11:08 | |
| *** stendulker has quit IRC | 11:16 | |
| *** sborkows has joined #openstack-ironic | 11:17 | |
| *** lucasagomes has joined #openstack-ironic | 11:22 | |
| *** appu103_ has quit IRC | 11:23 | |
| * lucasagomes got his bouncer working again | 11:26 | |
| jroll | \o/ | 11:30 |
| jroll | morning y'all | 11:30 |
| *** lucasagomes has quit IRC | 11:34 | |
| *** lucasagomes has joined #openstack-ironic | 11:35 | |
| *** lucasagomes has quit IRC | 11:39 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up release notes for 6.1.0 https://review.openstack.org/352660 | 11:40 |
| *** lucasagomes has joined #openstack-ironic | 11:41 | |
| *** vmud213 has joined #openstack-ironic | 11:42 | |
| *** dprince has joined #openstack-ironic | 11:43 | |
| *** lucasagomes is now known as lucas-hungry | 11:44 | |
| TheJulia | jroll: Good morning! | 11:46 |
| dtantsur | morning jroll, TheJulia | 11:47 |
| dtantsur | jroll, something that would be really good to fit in the release: https://review.openstack.org/349510 | 11:47 |
| *** joprovost has joined #openstack-ironic | 11:47 | |
| TheJulia | jroll: would it make sense to to indicate when the keystone_authtoken stuff will be removed? | 11:47 |
| *** milan has joined #openstack-ironic | 11:48 | |
| jroll | TheJulia: probably, but I haven't thought much about when that would be and it was late :) | 11:48 |
| TheJulia | :) | 11:49 |
| *** moshele has quit IRC | 11:50 | |
| *** moshele has joined #openstack-ironic | 11:51 | |
| jroll | dtantsur: reviewed, I agree we don't need a log there | 11:51 |
| dtantsur | okie | 11:51 |
| *** larsks has quit IRC | 11:51 | |
| jroll | dtantsur: any thoughts on when we should remove keystone_authtoken stuff? next cycle or wait for P? | 11:52 |
| jroll | TheJulia: would like your opinions too | 11:52 |
| dtantsur | I'd say the next cycle, why not? | 11:52 |
| dtantsur | this clutters the code and is apparently not compatible with keystone v3 | 11:52 |
| jroll | dunno, it's a relatively big one :) | 11:53 |
| TheJulia | jroll: At this point, wouldn't we technically have to wait for P? | 11:53 |
| jroll | TheJulia: nope, rule is at least one cycle boundary and at least 3 months | 11:53 |
| TheJulia | oh, I thought it was one cycle + 3 months | 11:53 |
| TheJulia | so effectively 9 | 11:53 |
| jroll | ah, no | 11:53 |
| jroll | cycle is for release-followers, 3 months is for CD folks | 11:53 |
| *** dprince has quit IRC | 11:56 | |
| *** larsks has joined #openstack-ironic | 11:56 | |
| *** dprince has joined #openstack-ironic | 11:56 | |
| TheJulia | Well, I stand corrected then. Given how big it is... I think waiting until P would be the best thing to do. | 11:58 |
| ifarkas | dtantsur, could you please review the drac raid patch? https://review.openstack.org/#/c/176319/ | 11:59 |
| TheJulia | But, I don't see much harm nuking it in the next cycle | 11:59 |
| dtantsur | ifarkas, it's on my radar, will come to it asap | 11:59 |
| ifarkas | dtantsur, thanks! | 11:59 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Deprecate putting periodic tasks on a driver object https://review.openstack.org/349510 | 12:00 |
| dtantsur | jroll, TheJulia ^^^ | 12:00 |
| jroll | +2 | 12:01 |
| *** ohamada_ has joined #openstack-ironic | 12:02 | |
| *** ohamada has quit IRC | 12:02 | |
| TheJulia | dtantsur: +2, thank you for the super fast revision :) | 12:03 |
| jroll | TheJulia: please leave a comment there if you want me to add it | 12:03 |
| TheJulia | jroll: was just going back to it :) | 12:03 |
| jroll | I'm going to be afk for the next few hours for an appointment, I'll check up on reno things when I'm back, feel free to ping me with anything else that I can help get in today | 12:03 |
| jroll | and good night to the euro folks, you'll probably be gone when I'm back :) | 12:04 |
| TheJulia | jroll: have a good day! :) | 12:04 |
| *** jcoufal has joined #openstack-ironic | 12:06 | |
| openstackgerrit | Merged openstack/ironic: Officially deprecate agent passthru classes and API https://review.openstack.org/336547 | 12:08 |
| *** nicodemos has left #openstack-ironic | 12:09 | |
| *** amoralej is now known as amoralej|lunch | 12:12 | |
| sambetts | o/ jroll | 12:14 |
| *** mjura has quit IRC | 12:16 | |
| *** mjura has joined #openstack-ironic | 12:17 | |
| *** rbudden has joined #openstack-ironic | 12:19 | |
| *** joprovost1 has joined #openstack-ironic | 12:19 | |
| *** ccamacho is now known as ccamacho|lunch | 12:20 | |
| *** joprovost has quit IRC | 12:20 | |
| *** joprovost1 is now known as joprovost | 12:20 | |
| *** MattMan has quit IRC | 12:21 | |
| *** MattMan has joined #openstack-ironic | 12:21 | |
| *** jcoufal has quit IRC | 12:22 | |
| *** jcoufal has joined #openstack-ironic | 12:22 | |
| *** lucas-hungry is now known as lucasagomes | 12:24 | |
| *** thrash|g0ne is now known as thrash | 12:26 | |
| *** e0ne has joined #openstack-ironic | 12:26 | |
| *** nicodemos has joined #openstack-ironic | 12:27 | |
| *** joprovost has quit IRC | 12:28 | |
| *** ohamada_ has quit IRC | 12:31 | |
| *** ohamada_ has joined #openstack-ironic | 12:31 | |
| *** ayoung has quit IRC | 12:33 | |
| lucasagomes | TheJulia, jroll morning | 12:40 |
| mgould | TheJulia, jroll morning | 12:44 |
| TheJulia | Good morning lucasagomes, mgould | 12:46 |
| *** catintheroof has joined #openstack-ironic | 12:46 | |
| lazy_prince | lucasagomes: your patch did not do its magic.. it did not help.. I am rerunning it just to confirm again.. | 12:49 |
| lucasagomes | lazy_prince, :-/ | 12:49 |
| lucasagomes | lazy_prince, I wonder if that's because we passed both options in the same line | 12:52 |
| lucasagomes | lazy_prince, I've found a blog post saying that it didn't work | 12:52 |
| lucasagomes | lazy_prince, see #7 https://zitseng.com/archives/5966 | 12:52 |
| *** jrist has quit IRC | 12:52 | |
| lazy_prince | hmm.. probably if you can just give me the raw ipmitool cmd, then I can try and let you know in no time. | 12:53 |
| * lazy_prince checking the link.. | 12:54 | |
| lucasagomes | lazy_prince, yeah... try to comment out the bit that add 'persistent' to the options list | 12:54 |
| lucasagomes | line 920 and 921 from ironic/drivers/modules/ipmitool.py | 12:55 |
| lazy_prince | lucasagomes: I have to wait for the current provisioning to complete.. | 12:55 |
| lucasagomes | and see if the machine will keep the uefi mode after changing the boot device | 12:55 |
| lucasagomes | lazy_prince, sure np | 12:55 |
| lucasagomes | lazy_prince, I mean, thanks for trying it out! | 12:55 |
| sborkows | Hi, I have a problem with my patch about pci_devices plugin in inspector: https://review.openstack.org/#/c/352295/ Inspector logs in tempest tests only tell me that the 'pci-devices' collector could not be loaded. I do not know how to check it out. | 12:56 |
| lazy_prince | Well... if I submit bugs, its should be my responsibility to verify the fixes.. | 12:56 |
| *** rook_ is now known as rook | 13:01 | |
| *** vmud213 has quit IRC | 13:03 | |
| *** thiagop has joined #openstack-ironic | 13:04 | |
| thiagop | Good morning, Ironic | 13:04 |
| thiagop | JayF: thanks for reporting the docs bug. Will look into that today | 13:04 |
| *** links has quit IRC | 13:06 | |
| *** rloo has joined #openstack-ironic | 13:06 | |
| *** jrist has joined #openstack-ironic | 13:06 | |
| *** sdake has joined #openstack-ironic | 13:09 | |
| openstackgerrit | Gleb Stepanov proposed openstack/ironic: Check keyword arguments https://review.openstack.org/351677 | 13:10 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 13:11 |
| *** baoli has joined #openstack-ironic | 13:13 | |
| *** bfournie has quit IRC | 13:13 | |
| *** baoli_ has joined #openstack-ironic | 13:14 | |
| *** mgoddard_ has joined #openstack-ironic | 13:16 | |
| *** sdake has quit IRC | 13:17 | |
| *** baoli has quit IRC | 13:18 | |
| *** permalac has joined #openstack-ironic | 13:18 | |
| *** mgoddard has quit IRC | 13:19 | |
| *** sdake has joined #openstack-ironic | 13:19 | |
| *** sdake_ has joined #openstack-ironic | 13:23 | |
| *** sdake has quit IRC | 13:24 | |
| *** amoralej|lunch is now known as amoralej | 13:25 | |
| lazy_prince | lucasagomes: looks like commenting worked.. so how do we retain efiboot and persistent together.. two cmds..? | 13:26 |
| *** ayoung has joined #openstack-ironic | 13:26 | |
| *** Goneri has quit IRC | 13:28 | |
| *** joprovost has joined #openstack-ironic | 13:29 | |
| mgould | lazy_prince: looks like https://zitseng.com/archives/5966 had the same problem | 13:30 |
| mgould | lazy_prince: yeah, a commenter says that two commands should work | 13:30 |
| *** joprovost1 has joined #openstack-ironic | 13:31 | |
| lazy_prince | mgould: lucasagomes: fair enough.. left this as a comment on the patch.. | 13:32 |
| *** afaranha has quit IRC | 13:32 | |
| lucasagomes | lazy_prince, thanks a lot! | 13:32 |
| lucasagomes | lazy_prince, I will make another patch-set for it | 13:32 |
| *** Goneri has joined #openstack-ironic | 13:33 | |
| lazy_prince | lucasagomes: waiting for the patch.. :) | 13:33 |
| *** joprovost has quit IRC | 13:34 | |
| *** joprovost1 is now known as joprovost | 13:34 | |
| lazy_prince | on a side note, do you think, this needs to be back-ported to mitaka..? | 13:34 |
| lucasagomes | lazy_prince, I think it would be good yes | 13:35 |
| lucasagomes | the patch fits the criteria, it doesn't change API or db model | 13:35 |
| lucasagomes | it's a small patch, I will propose it for mitaka | 13:36 |
| lucasagomes | lazy_prince, btw, thanks for giving it a go | 13:36 |
| lazy_prince | lucasagomes: yw.. | 13:36 |
| *** ccamacho|lunch is now known as ccamacho | 13:37 | |
| *** cdearborn has joined #openstack-ironic | 13:39 | |
| *** hoangcx has joined #openstack-ironic | 13:42 | |
| *** bfournie has joined #openstack-ironic | 13:45 | |
| lazy_prince | jroll: so on the compute proxy ha patch. Ihave verified that it works even though capacity is reported by all compute-proxies. Once instance is provisioned on node, the compute-proxies reduces the capacity. | 13:45 |
| lazy_prince | s/Ihave/I have/ | 13:46 |
| *** hoangcx has quit IRC | 13:46 | |
| *** PollyZ has joined #openstack-ironic | 13:50 | |
| openstackgerrit | Gleb Stepanov proposed openstack/ironic: Check keyword arguments https://review.openstack.org/351677 | 13:57 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: [WIP] Change Bifrost to TinyIPA as the default https://review.openstack.org/352919 | 13:58 |
| *** jistr is now known as jistr|debug | 14:00 | |
| *** anush_ has joined #openstack-ironic | 14:00 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Change Vagrant VM to mirror memory/cpu in CI https://review.openstack.org/352921 | 14:01 |
| *** sborkows has quit IRC | 14:02 | |
| *** moshele has quit IRC | 14:05 | |
| *** mtanino has joined #openstack-ironic | 14:09 | |
| *** syed_ has joined #openstack-ironic | 14:09 | |
| *** jcoufal has quit IRC | 14:10 | |
| *** vishwanathj has joined #openstack-ironic | 14:13 | |
| lucasagomes | lazy_prince, I'm wondering if the command actually works with both options but we need to set persistent first and then efiboot | 14:16 |
| lucasagomes | lazy_prince, e.g options=persistent,efiboot (instead of the other way around like we have in the patch) | 14:16 |
| mat128 | good morning ironic! | 14:17 |
| mgould | mat128: morning! | 14:18 |
| mgould | mat128: did you make any progress on your "giant qcow images" problem? | 14:18 |
| lucasagomes | lazy_prince, could you give that I go? | 14:19 |
| * mgould imagines https://www.flickr.com/photos/underbellyltd/2116722894 in mat128's datacenter | 14:19 | |
| *** pece has quit IRC | 14:19 | |
| mat128 | mgould: no, took notes of the great suggestions here in our backlog item. right now our smallest machine is 8G (in production) and biggest image is 5-6G | 14:19 |
| mat128 | but it would greatly help dev machines (where we host multiple machines on the same host, not unlike devstack) to run bigger images | 14:20 |
| mat128 | mgould: haha | 14:20 |
| mgould | mat128: cool | 14:20 |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/351987 | 14:21 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Passing SUPPORTED_ONEVIEW_VERSION as string + transient error fix https://review.openstack.org/352603 | 14:21 |
| mgould | it occurred to me that you can probably get raw.gz "for free" by fiddling with the settings on Glance and/or a web proxy in front of it | 14:21 |
| mgould | at least, in terms of "bytes flying over the network" | 14:21 |
| *** hjanssen-hpe has joined #openstack-ironic | 14:22 | |
| *** hj-hpe has joined #openstack-ironic | 14:22 | |
| mgould | and if the images are huge because they're describing a big but mostly empty disk, they should compress well | 14:22 |
| openstackgerrit | Gleb Stepanov proposed openstack/ironic: Check keyword arguments https://review.openstack.org/351677 | 14:26 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Comply with iLO model of usage for one-time boot https://review.openstack.org/350060 | 14:27 |
| mgould | mat128: ^^^ | 14:27 |
| *** sdake_ is now known as sdake | 14:30 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Comply with iLO model of usage for one-time boot https://review.openstack.org/350060 | 14:32 |
| mat128 | mgould: yup, but the proxy would have to be close to / running on IPA, no? | 14:33 |
| mat128 | that would un-gzip | 14:33 |
| mgould | I don't *think* so | 14:34 |
| mat128 | you think it's standard http stuff | 14:34 |
| mat128 | client uncompresses? | 14:34 |
| mat128 | hmm | 14:34 |
| mgould | yeah | 14:34 |
| mgould | let me check, one minute... | 14:34 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Tests for testing port-create command https://review.openstack.org/291802 | 14:34 |
| mgould | mat128: compression negotiation is built into HTTP | 14:35 |
| mgould | looks like the requests library supports gzip on the client: http://stackoverflow.com/questions/33334606/requests-module-and-compression | 14:36 |
| mgould | are we using requests in IPA? | 14:36 |
| mat128 | lemme check | 14:36 |
| mgould | yes, yes we are | 14:36 |
| mgould | 48 hits in the IPA tree :-) | 14:37 |
| mat128 | mgould: confirmed, _download_file in standby.py | 14:37 |
| *** Goneri has quit IRC | 14:38 | |
| mgould | I just did a `requests.get` in a REPL, and it used gzip without my having to do anything | 14:38 |
| mat128 | so in theory | 14:38 |
| mat128 | I could upload "raw" images to glance, ensure glance is serving compressed content | 14:39 |
| mat128 | and images will be compressed/uncompressed on the fly | 14:39 |
| mgould | yep | 14:39 |
| mat128 | I'll check how big my qcow image will be once raw.gz | 14:39 |
| *** subscope has joined #openstack-ironic | 14:39 | |
| mgould | cool, good luck! | 14:39 |
| mat128 | ty :) | 14:39 |
| *** rbartal has quit IRC | 14:43 | |
| lucasagomes | lazy_prince, got a machine to test here... | 14:44 |
| *** michauds has joined #openstack-ironic | 14:46 | |
| *** catintheroof has quit IRC | 14:47 | |
| *** catintheroof has joined #openstack-ironic | 14:48 | |
| *** rbudden has quit IRC | 14:49 | |
| *** mgoddard_ has quit IRC | 14:50 | |
| *** mgoddard has joined #openstack-ironic | 14:50 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic: Remove duplicate parameters from local.conf example https://review.openstack.org/350970 | 14:50 |
| *** thrash is now known as thrash|biab | 14:51 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix curtosy env-var file for recent auth changes https://review.openstack.org/352949 | 14:53 |
| *** catintheroof has quit IRC | 14:53 | |
| lucasagomes | lazy_prince, http://paste.openstack.org/show/552460/ apparently we can't set efiboot + persistent using the ipmitool command | 14:57 |
| lucasagomes | I will try with raw bytes | 14:57 |
| lucasagomes | >.< | 14:57 |
| *** rpioso has joined #openstack-ironic | 15:00 | |
| mgould | lucasagomes: aaaargh | 15:00 |
| *** praneshp has joined #openstack-ironic | 15:01 | |
| dtantsur | ifarkas, sorry, will have to -1 https://review.openstack.org/#/c/176319 for using deprecated way of creating periodic tasks | 15:01 |
| ifarkas | dtantsur, what's the new way? | 15:01 |
| dtantsur | ifarkas, on interfaces | 15:01 |
| dtantsur | I think I've left the comment previously | 15:02 |
| ifarkas | dtantsur, hmm, yeah, I replied to that comment, I thought it doesn't get registered on the interface | 15:03 |
| ifarkas | I checked it but maybe there was something in my setup | 15:03 |
| ifarkas | will double-check tomorrow | 15:03 |
| dtantsur | ifarkas, both ways are supported, and the interface path is used in inspector interface: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/inspector.py#L116-L118 | 15:03 |
| dtantsur | so I have some degree of confidences that it works :) | 15:03 |
| jlvillal | Woo hoo! 2nd +2 and +A on multiple compute hosts patch :) | 15:04 |
| jlvillal | https://review.openstack.org/#/c/348443/ | 15:04 |
| dtantsur | (unless there is something special about the raid interface, then we have to fix it) | 15:04 |
| dtantsur | jlvillal, w00t!!!!! | 15:04 |
| ifarkas | dtantsur, right :-) | 15:04 |
| thiagop | ifarkas: good example here: https://review.openstack.org/#/c/286192/31/ironic/drivers/modules/oneview/deploy.py@45 | 15:04 |
| ifarkas | dtantsur, will check and update the patch | 15:04 |
| dtantsur | oh, another example, sweet | 15:04 |
| ifarkas | dtantsur, thiagop, then I must have messed up something | 15:04 |
| ifarkas | thanks! | 15:04 |
| lucasagomes | lazy_prince, apparently raw bytes does the job! | 15:05 |
| lucasagomes | http://paste.openstack.org/show/552463/ | 15:05 |
| thiagop | dtantsur: you mean https://review.openstack.org/#/c/176319/29/ironic/drivers/drac.py@88 ? | 15:08 |
| thiagop | it is quite similar to ours | 15:08 |
| thiagop | dtantsur: btw, I hope you got better. Folks said you were sick yesterday... | 15:09 |
| dtantsur | I was, better now, thanks | 15:10 |
| mat128 | mgould: still compressing (20 minutes) and 4.93G so far | 15:11 |
| mat128 | mgould: if that has to be done on the fly I'm unsure of the consequences :S | 15:11 |
| lucasagomes | jlvillal, w0000000t | 15:11 |
| jlvillal | lucasagomes: :) | 15:11 |
| dtantsur | now I need to figure our how to configure this multiple compute feature.. or does it just work (tm)? | 15:11 |
| mgould | mat128: ah well, worth a shot :-) | 15:11 |
| jlvillal | Hopefully jroll saw that the multiple compute host patch got approved :) | 15:12 |
| dtantsur | jlvillal, I think he's out now, will be back much later | 15:12 |
| mgould | jlvillal: woohoo! | 15:12 |
| mat128 | mgould: I'll try it IRL with glance and IPA once I figure how big the file is | 15:12 |
| TheJulia | dtantsur: seems to just work from what I've heard :) | 15:15 |
| *** [1]cdearborn has joined #openstack-ironic | 15:15 | |
| dtantsur | TheJulia, fantastic | 15:16 |
| mgould | mat128: cool, good luck! | 15:16 |
| jlvillal | Is the gate working correctly now? Just wondering if should update the WhiteBoard https://etherpad.openstack.org/p/IronicWhiteBoard | 15:16 |
| jlvillal | I thought I saw some code get merged last night. | 15:17 |
| *** links has joined #openstack-ironic | 15:17 | |
| TheJulia | dtantsur: lazy_prince has been testing it, and he pointed out to me that capacity reporting in nova wouldn't be precisely correct, then again I wonder how many people really manage baremetal via nova capacity data | 15:17 |
| * jlvillal will assume the gate is working as he sees a patch merged about an hour ago. | 15:17 | |
| * dtantsur doesn't even know where this data is shown | 15:17 | |
| dtantsur | jlvillal, we've definitely merged a couple of things today | 15:18 |
| jlvillal | dtantsur: Thanks, I'll update the whiteboard and history page | 15:18 |
| TheJulia | dtantsur: then not stress about that? :) | 15:18 |
| dtantsur | heh | 15:18 |
| *** rbudden has joined #openstack-ironic | 15:20 | |
| *** e0ne has quit IRC | 15:21 | |
| dtantsur | sambetts, around? IIRC the RFE for https://review.openstack.org/279940 was approved, so you can life your -2 | 15:22 |
| *** jistr|debug is now known as jistr | 15:23 | |
| dtantsur | hmm, maybe not really, I'm confused | 15:23 |
| *** e0ne has joined #openstack-ironic | 15:24 | |
| dtantsur | I thought it's a part of the infiniband work | 15:24 |
| * dtantsur leaves a comment | 15:25 | |
| *** catintheroof has joined #openstack-ironic | 15:25 | |
| * lazy_prince leaves for the day.. | 15:30 | |
| *** cdearborn has quit IRC | 15:30 | |
| *** causten has joined #openstack-ironic | 15:30 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add openstack baremetal driver commands https://review.openstack.org/350050 | 15:31 |
| dtantsur | lazy_prince, see you! | 15:33 |
| sambetts | dtantsur: :/ thanks for leaving the coment | 15:34 |
| openstackgerrit | Merged openstack/ironic: Deprecate putting periodic tasks on a driver object https://review.openstack.org/349510 | 15:36 |
| *** ohamada_ has quit IRC | 15:37 | |
| *** PollyZ has quit IRC | 15:38 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 15:39 |
| apughazh | jayf: https://review.openstack.org/#/c/350705/ | 15:41 |
| jroll | jlvillal: I did now :) I've been more focused on getting 6.1.0 out the door | 15:46 |
| devananda | morning, all | 15:46 |
| TheJulia | good morning devananda | 15:46 |
| *** amoralej is now known as amoralej|brb | 15:47 | |
| jroll | heya deva | 15:47 |
| *** pcaruana has quit IRC | 15:48 | |
| mat128 | mgould: really surprised by the following results: http://paste.openstack.org/show/552470/ | 15:50 |
| *** rcernin has quit IRC | 15:50 | |
| mgould | morning devananda | 15:50 |
| mat128 | hi devananda | 15:50 |
| *** tesseract- has quit IRC | 15:50 | |
| mgould | mat128: \o/ | 15:50 |
| mat128 | raw.gz smaller than qcow2! | 15:51 |
| *** fragatina has joined #openstack-ironic | 15:51 | |
| dtantsur | morning devananda | 15:51 |
| dtantsur | jroll, do you think we can call https://trello.com/c/37YuKIB8/28-promote-agent-vendor-passthru-to-core-api done? I can't remember if we need any more patches... | 15:52 |
| dtantsur | hmm, maybe api-ref though... | 15:52 |
| devananda | I can whip up an api-ref change for that real quick, if needed | 15:52 |
| mgould | mat128: apparently qcow2 has "optional zlib based compression" - are you using that? | 15:53 |
| mat128 | yes | 15:53 |
| jroll | dtantsur: api-ref would be nice but I'm fine calling it done as far as trello goes | 15:53 |
| dtantsur | devananda, would be much appreciated! | 15:53 |
| jroll | idk, maybe we should wait to move it | 15:53 |
| mgould | mat128: OK, that's more surprising then | 15:53 |
| * jroll unopinionated | 15:53 | |
| *** PollyZ has joined #openstack-ironic | 15:53 | |
| dtantsur | let's have docs for real, no hurry with moving cards :) | 15:53 |
| devananda | jroll: your multi-compute-host nova patch is +W'd and in the gate!! :-D | 15:54 |
| mgould | duplications between sections of the file that qcow2 can't avoid but a general-purpose compression algorithm can, I guess | 15:54 |
| mat128 | yup, probably | 15:54 |
| jroll | devananda: indeed :) | 15:55 |
| dtantsur | devananda, jroll, does it mean that it's no longer required to use ironic.nova.compute.manager.ClusteredComputeManager? | 15:55 |
| jroll | dtantsur: correct, we can deprecate that now, I think | 15:55 |
| dtantsur | awesome! | 15:56 |
| jroll | s/, I think// | 15:56 |
| devananda | \o/ | 15:56 |
| * jroll does it | 15:56 | |
| mat128 | woohoo! | 15:56 |
| * devananda prepares to +2 that | 15:56 | |
| * dtantsur will remove it from his tripleo patch | 15:56 | |
| mat128 | prepares to push the nova team to upgrade ;) | 15:56 |
| *** sabeen has joined #openstack-ironic | 15:56 | |
| *** Sukhdev has joined #openstack-ironic | 15:57 | |
| sambetts | \o/ | 15:59 |
| *** narasimha_SV has joined #openstack-ironic | 16:02 | |
| jroll | devananda: maybe we should wait to ship this in the final release for this cycle, though? we shouldn't deprecate it in a release until Nova has that in a release | 16:03 |
| narasimha_SV | driver_info/ssh_key_filename what should be the value for this option to the node | 16:03 |
| jroll | devananda: or something, I think you get what I'm saying | 16:03 |
| jroll | devananda: anyway, it's here, since the bot doesn't like me https://review.openstack.org/#/c/352991/ | 16:04 |
| *** dprince has quit IRC | 16:04 | |
| narasimha_SV | if I want to use pxe_ssh what will the options I need to give | 16:04 |
| dtantsur | jroll, fair point.. I think we can defer it to the very final release in Newton | 16:04 |
| jroll | dtantsur: I've -2'd for now, so go ahead and review as needed | 16:05 |
| narasimha_SV | http://paste.openstack.org/show/552473/ | 16:05 |
| *** dprince has joined #openstack-ironic | 16:05 | |
| narasimha_SV | if i provide ssh key file path I am getting above message | 16:05 |
| dtantsur | got a question downstream: do we support changing a port's MAC for an active instance? | 16:06 |
| dtantsur | they've replaced the NIC or something like that | 16:06 |
| jroll | narasimha_SV: probably a permissions problem | 16:06 |
| jroll | dtantsur: hrm, not sure off the top of my head... we might not | 16:07 |
| dtantsur | it seems like a direct update is not possible, so they ended up recreating a port | 16:07 |
| dtantsur | I wonder how many things get broken due to it... | 16:07 |
| jroll | dtantsur: I think maybe you can do a direct update if you put the node in maintenance mode | 16:07 |
| dtantsur | I suspect we don't allow updating port's MAC at all, but I may be wrong | 16:07 |
| *** baoli_ has quit IRC | 16:08 | |
| narasimha_SV | jroll: what permissions should it have | 16:08 |
| jroll | narasimha_SV: whatever permissions allow the user running ironic-conductor to read the key | 16:08 |
| *** e0ne has quit IRC | 16:09 | |
| narasimha_SV | jroll: i gave 644 permissions to ssh_key file | 16:09 |
| JayF | jroll: if we tell people to upgrade ironic, then nova, that means they're going to immediately be using a deprecated thing, right? | 16:10 |
| JayF | jroll: we don't have to wait until ocata to even do the dep *warning*? | 16:10 |
| jroll | JayF: ummmm | 16:11 |
| jroll | I'm okay with people running a deprecated thing for that timeframe | 16:11 |
| jroll | JayF: keep in mind the warning specifies which release of Nova, so it shouldn't be that confusing | 16:11 |
| *** Sukhdev has quit IRC | 16:12 | |
| *** e0ne has joined #openstack-ironic | 16:12 | |
| jroll | oh, I should reno this though | 16:12 |
| dtantsur | it's always the case when we're replacing an old thing with a new thing: for some time you might be running the old thing | 16:12 |
| dtantsur | that's why we don't just remove the old thing right away :) | 16:12 |
| dtantsur | ++ for reno btw | 16:12 |
| milan | guys, I've spent some time looking into this traceback and I cannot locate the issue: http://logs.openstack.org/43/348943/8/check/gate-tempest-dsvm-ironic-inspector-nv/6d5b262/logs/screen-ironic-inspector.txt.gz#_2016-08-09_14_03_21_195 help much appreciated | 16:12 |
| jroll | yeah, the goal isn't to try never to see the warnings, but rather to let people know they have a certain time limit to fix it | 16:13 |
| JayF | I'm just saying it seems a little weird; if a little wierdness is ok then w/e | 16:13 |
| *** PollyZ has quit IRC | 16:13 | |
| dtantsur | milan, wrong formatting in one of log lines. link? | 16:13 |
| dtantsur | (link to the patch) | 16:13 |
| milan | dtantsur, https://review.openstack.org/#/c/344921/ | 16:13 |
| milan | yeah, that's what I cannot locate :-/ | 16:14 |
| milan | should happen somewhere in the utils.py processing_get_logger_prefix, just don't see where | 16:14 |
| *** e0ne has quit IRC | 16:14 | |
| dtantsur | milan, I wonder if it's related to the patch or just something we haven't noticed | 16:15 |
| *** jpich has quit IRC | 16:15 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Add metrics support to IPA https://review.openstack.org/119981 | 16:15 |
| *** PollyZ has joined #openstack-ironic | 16:15 | |
| openstackgerrit | Akilan Pughazhendi proposed openstack/ironic: Updated Dev quickstart for viewing doc changes https://review.openstack.org/350705 | 16:15 |
| milan | dtantsur, I've checked https://review.openstack.org/#/c/344921/ which doesn't have the issue | 16:16 |
| dtantsur | hmm, you gave me the wrong link initially | 16:16 |
| milan | http://logs.openstack.org/21/344921/8/check/gate-ironic-inspector-dsvm-discovery/37c0e38/logs/screen-ironic-inspector.txt.gz#_2016-08-09_10_25_08_293 | 16:16 |
| milan | dtantsur, ah, sry | 16:17 |
| milan | dtantsur, https://review.openstack.org/#/c/348943/ | 16:17 |
| milan | it always happens when there's the MAC address present | 16:17 |
| milan | afaictl | 16:17 |
| *** jcoufal has joined #openstack-ironic | 16:17 | |
| milan | * with my patch | 16:18 |
| dtantsur | milan, https://review.openstack.org/#/c/348943/8/ironic_inspector/node_cache.py@541 | 16:18 |
| dtantsur | incomplete formatting | 16:18 |
| milan | dtantsur, thx | 16:19 |
| *** amoralej|brb is now known as amoralej | 16:20 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Deprecate ClusteredComputeManager https://review.openstack.org/352991 | 16:22 |
| dtantsur | ifarkas, an easy but important review here: https://review.openstack.org/339457 | 16:22 |
| *** Romanenko_K has quit IRC | 16:22 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 16:24 |
| *** aNupoisc has joined #openstack-ironic | 16:25 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: IPMITool: Check the boot mode when setting the boot device https://review.openstack.org/352820 | 16:28 |
| lucasagomes | lazy_prince, it turned out to be more painful than expected to fix this problem :-( | 16:28 |
| lucasagomes | JayF, jroll ^ can I get some eyes on that bug please ? I needed to refactor a big part of the set_boot_device() for ipmitool | 16:28 |
| *** joprovost has quit IRC | 16:28 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 16:29 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 16:31 |
| devananda | jroll: I agree - we can't land a deprecation warning when the thing that supercedes it isn't actually available yet. It's also very odd that the deprecation warning is indicating a release that hasn't been released yet | 16:32 |
| devananda | jroll: imagine someone deploying ironic 6.1.0, before Newton-final is released | 16:32 |
| ifarkas | dtantsur, done ;-) | 16:32 |
| devananda | this message is going to be veeeerrrry odd | 16:32 |
| narasimha_SV | what should be the ip given in ssh_address when use pxe_ssh driver 'ssh_address': '192.168.56.105' | 16:32 |
| jroll | devananda: sure, that's why I don't want to put it in 6.1.0 | 16:32 |
| devananda | :) | 16:32 |
| *** PollyZ has quit IRC | 16:32 | |
| narasimha_SV | i gave the ip of the host in which I deployed | 16:32 |
| jroll | narasimha_SV: it's the host for the VMs you are using to simulate bare metal machines | 16:33 |
| *** PollyZ has joined #openstack-ironic | 16:33 | |
| *** joprovost has joined #openstack-ironic | 16:34 | |
| *** ifarkas is now known as ifarkas_afk | 16:34 | |
| dtantsur | folks, I have a patch that has not had new reviews since June: https://review.openstack.org/#/c/328191/ | 16:35 |
| dtantsur | would be good to land it before the client freeze | 16:35 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: IPMITool: Check the boot mode when setting the boot device https://review.openstack.org/352820 | 16:36 |
| openstackgerrit | Michael Turek proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/332177 | 16:36 |
| *** Lord_Phoenix has quit IRC | 16:37 | |
| NobodyCam | Good Morning Ironic'ers :) | 16:37 |
| *** athomas has quit IRC | 16:37 | |
| *** milan has quit IRC | 16:38 | |
| narasimha_SV | jroll: thanks | 16:38 |
| *** fragatin_ has joined #openstack-ironic | 16:38 | |
| *** joprovost has quit IRC | 16:38 | |
| dtantsur | morning NobodyCam | 16:39 |
| jroll | np narasimha_SV | 16:39 |
| NobodyCam | Morning dtantsur :) | 16:39 |
| dtantsur | also morning rloo | 16:39 |
| *** joprovost has joined #openstack-ironic | 16:40 | |
| rloo | hi dtantsur, morning NobodyCam and everyone else | 16:40 |
| NobodyCam | morning rloo :) | 16:40 |
| *** fragatina has quit IRC | 16:41 | |
| *** jabroney has joined #openstack-ironic | 16:41 | |
| *** dprince has quit IRC | 16:42 | |
| narasimha_SV | if I want to use virsh that means I need to have KVM installed on that ip which I gave in ssh_address right ? | 16:42 |
| jroll | yes | 16:43 |
| narasimha_SV | ok | 16:43 |
| *** dprince has joined #openstack-ironic | 16:43 | |
| dtantsur | jroll, seems like the stevedore problem hits us again with https://review.openstack.org/#/c/314147/ | 16:45 |
| dtantsur | I'm not sure what to do, provided that we can't bump the minimum requirement | 16:46 |
| dtantsur | it probably also affects mitaka | 16:46 |
| jroll | ooooo | 16:46 |
| jroll | probably a question for stable team tbh | 16:46 |
| dtantsur | I see two options: upper-cap stevedore or remove the test and hope for better... | 16:46 |
| jroll | did that conversation happen on the ML before? | 16:46 |
| dtantsur | no, I did talk to folks re master breakage | 16:47 |
| jroll | ok | 16:48 |
| *** Sukhdev has joined #openstack-ironic | 16:50 | |
| *** Sukhdev has quit IRC | 16:50 | |
| *** subscope has quit IRC | 16:51 | |
| *** Sukhdev has joined #openstack-ironic | 16:51 | |
| *** Sukhdev has quit IRC | 16:51 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: IPMITool: Check the boot mode when setting the boot device https://review.openstack.org/352820 | 16:52 |
| NobodyCam | morning jroll :) | 16:52 |
| jroll | hey NobodyCam | 16:52 |
| NobodyCam | :) | 16:53 |
| lucasagomes | NobodyCam, morning | 16:53 |
| NobodyCam | morning lucasagomes :) | 16:53 |
| *** dprince has quit IRC | 16:54 | |
| mgould | morning NobodyCam jroll | 16:54 |
| NobodyCam | morning mgould :) | 16:55 |
| jroll | \o mgould | 16:55 |
| dtantsur | see you tomorrow folks | 16:56 |
| *** dtantsur is now known as dtantsur|afk | 16:56 | |
| NobodyCam | night dtantsur|afk | 16:56 |
| thiagop | night dtantsur|afk | 16:56 |
| NobodyCam | morning thiagop :) | 16:56 |
| thiagop | morning NobodyCam | 16:57 |
| NobodyCam | :) | 16:57 |
| lucasagomes | dtantsur|afk, see ya | 16:57 |
| lucasagomes | I'm calling a day as well | 16:57 |
| lucasagomes | talk to you all tomorrow, have a great evening | 16:57 |
| *** aNupoisc has quit IRC | 16:57 | |
| thiagop | night lucasagomes | 16:58 |
| *** aarefiev_ is now known as aarefiev | 16:58 | |
| *** lucasagomes is now known as lucas-dinner | 16:58 | |
| NobodyCam | night lucas-dinner | 16:59 |
| *** baoli has joined #openstack-ironic | 16:59 | |
| *** anush_ has quit IRC | 17:00 | |
| *** vishwanathj has quit IRC | 17:00 | |
| *** aNupoisc has joined #openstack-ironic | 17:01 | |
| *** e0ne has joined #openstack-ironic | 17:02 | |
| *** jcoufal has quit IRC | 17:10 | |
| *** ijw has joined #openstack-ironic | 17:11 | |
| * mgould -> home; good night! | 17:11 | |
| *** amoralej is now known as amoralej|off | 17:11 | |
| NobodyCam | night mgould | 17:11 |
| mgould | night NobodyCam | 17:11 |
| *** mgould is now known as mgould|afk | 17:11 | |
| *** joprovost has quit IRC | 17:15 | |
| *** thrash|biab is now known as thrash | 17:17 | |
| openstackgerrit | Merged openstack/ironic-inspector: Add callback function to manage missing hooks https://review.openstack.org/339457 | 17:17 |
| *** ijw has quit IRC | 17:18 | |
| *** fragatin_ has quit IRC | 17:19 | |
| *** e0ne has quit IRC | 17:19 | |
| *** joprovost has joined #openstack-ironic | 17:20 | |
| *** vishwanathj has joined #openstack-ironic | 17:21 | |
| *** e0ne has joined #openstack-ironic | 17:23 | |
| *** PollyZ has quit IRC | 17:24 | |
| *** joprovost has quit IRC | 17:25 | |
| *** e0ne has quit IRC | 17:25 | |
| *** ijw has joined #openstack-ironic | 17:27 | |
| openstackgerrit | Merged openstack/ironic: Simplify code by using mask_dict_password (again) https://review.openstack.org/352507 | 17:29 |
| *** joprovost has joined #openstack-ironic | 17:29 | |
| *** dprince has joined #openstack-ironic | 17:32 | |
| *** daemontool_ has quit IRC | 17:36 | |
| *** Sukhdev has joined #openstack-ironic | 17:41 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor multitenant networking release notes https://review.openstack.org/352659 | 17:42 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up release notes for 6.1.0 https://review.openstack.org/352660 | 17:42 |
| jroll | those should be fixed up and passing tests now | 17:42 |
| thiagop | jroll: did you tagged 6.1.0? | 17:42 |
| rloo | jroll: did you need those 2 reviewed soon so you could do a release? | 17:43 |
| jroll | thiagop: no, cleaning up release notes and checking for other things we need in there | 17:43 |
| jroll | rloo: yes | 17:43 |
| openstackgerrit | Matt Mulsow proposed openstack/bifrost: Fixed bifrost to not use a space for token https://review.openstack.org/353041 | 17:43 |
| jroll | thiagop: which reminds me, I had questions for you | 17:43 |
| * jroll finds a link | 17:43 | |
| rloo | jroll: ok. anything else outstanding besides those two? | 17:43 |
| thiagop | jroll: sure, punch it | 17:44 |
| jroll | rloo: if there's any metrics things left, would be nice, otherwise I think that's it. still want to check one more time. | 17:44 |
| jroll | thiagop: https://github.com/openstack/ironic/blob/master/releasenotes/notes/add-dynamic-allocation-feature-2fd6b4df7943f178.yaml | 17:44 |
| rloo | jroll: ok, i'll look at these two. am out of touch with metrics so don't think i can review any quickly if there are any outstanding ones. | 17:44 |
| *** pcaruana has joined #openstack-ironic | 17:44 | |
| jroll | thiagop: this doesn't tell me anything about the feature, and for the deprecation it doesn't tell me what action I need to take | 17:45 |
| *** ChubYann has joined #openstack-ironic | 17:45 | |
| jroll | rloo: okay, metrics is mostly just adding decorators around, pretty easy reviews :) | 17:45 |
| thiagop | jroll: JayF filed a bug for us to make some adds to the docs, but I can improve the release notes on your patch too if you want me to | 17:46 |
| jroll | thiagop: both, please, release notes first so that we can release 6.1.0 :) | 17:46 |
| thiagop | jroll: can I update directly on 352660? | 17:47 |
| jroll | thiagop: separate change, please | 17:47 |
| *** rama_y has joined #openstack-ironic | 17:47 | |
| thiagop | jroll: roger | 17:47 |
| jroll | thanks | 17:48 |
| *** alineb has quit IRC | 17:50 | |
| *** mgoddard has quit IRC | 17:51 | |
| *** mgoddard has joined #openstack-ironic | 17:53 | |
| *** e0ne has joined #openstack-ironic | 17:56 | |
| jroll | api meeting in 2 minutes | 17:58 |
| *** joprovost has quit IRC | 18:00 | |
| thiagop | jroll: wdyt? http://paste.openstack.org/show/552498/ | 18:02 |
| jroll | thiagop: in a meeting, throw it on gerrit and I'll review later | 18:03 |
| thiagop | jroll: ok | 18:03 |
| *** electrofelix has quit IRC | 18:05 | |
| *** joprovost has joined #openstack-ironic | 18:07 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: Refactor OneView dynamic allocation release notes https://review.openstack.org/353049 | 18:10 |
| *** e0ne has quit IRC | 18:11 | |
| *** joprovost has quit IRC | 18:13 | |
| *** rama_y has quit IRC | 18:13 | |
| *** joprovost has joined #openstack-ironic | 18:13 | |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: [WIP] Add storage_interface DB field and object https://review.openstack.org/348005 | 18:13 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: [WIP] Add storage_interface to base driver class https://review.openstack.org/348006 | 18:14 |
| *** fragatina has joined #openstack-ironic | 18:16 | |
| *** PollyZ has joined #openstack-ironic | 18:23 | |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 18:28 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 18:29 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 18:29 |
| *** fragatina has quit IRC | 18:29 | |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_targets table to save target information https://review.openstack.org/285218 | 18:30 |
| *** fragatina has joined #openstack-ironic | 18:30 | |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce VolumeTarget object https://review.openstack.org/285219 | 18:30 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume target operations https://review.openstack.org/285220 | 18:30 |
| *** ppiela has joined #openstack-ironic | 18:33 | |
| *** rama_y has joined #openstack-ironic | 18:39 | |
| *** narasimha_SV has quit IRC | 18:40 | |
| *** links has quit IRC | 18:40 | |
| *** david-lyle has quit IRC | 18:41 | |
| *** david-lyle has joined #openstack-ironic | 18:42 | |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: [WIP] Add storage_interface DB field and object https://review.openstack.org/348005 | 18:43 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: [WIP] Add storage_interface to base driver class https://review.openstack.org/348006 | 18:43 |
| jroll | devananda: we volunteered you for stuff in the ironic api meeting, check out logs or feel free to ping me when you're around :) | 18:44 |
| *** tangchen has quit IRC | 18:46 | |
| *** rcernin has joined #openstack-ironic | 18:51 | |
| * jroll lunches | 18:52 | |
| *** joprovost has quit IRC | 19:06 | |
| openstackgerrit | Merged openstack/python-oneviewclient: Passing SUPPORTED_ONEVIEW_VERSION as string + transient error fix https://review.openstack.org/352603 | 19:06 |
| *** causten_ has joined #openstack-ironic | 19:09 | |
| *** causten has quit IRC | 19:11 | |
| *** catintheroof has quit IRC | 19:12 | |
| *** kragniz has joined #openstack-ironic | 19:14 | |
| devananda | jroll: oh, crap. I've been here. I did not mean to miss the meeting :( | 19:17 |
| devananda | jroll, rloo: I doubt I will have time in the next week to turn that into an actual spec. My goal was after Newton releases / before Barcelona | 19:23 |
| devananda | also, I got sucked into this api-ref update | 19:26 |
| devananda | when I realized that several of the current samples are flat-out wrong | 19:26 |
| devananda | and I wanted a way to regenerate them all | 19:26 |
| devananda | that's about half done now | 19:26 |
| JayF | Those fixes are very appreciated | 19:28 |
| JayF | the api-ref stuff | 19:29 |
| *** sambetts is now known as sambetts|afk | 19:33 | |
| *** PollyZ has quit IRC | 19:37 | |
| jroll | devananda: heh, thank you for that | 19:38 |
| jroll | devananda: not next week is okay. I'd like to do it sooner than later - I may have time to take that on | 19:38 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: DONOTMERGE: Test if stable/liberty is broken in gate https://review.openstack.org/353084 | 19:39 |
| jroll | thiagop: so another question, why is there a config option instead of just making everything pre-allocation? | 19:40 |
| *** david-lyle_ has joined #openstack-ironic | 19:41 | |
| openstackgerrit | Michael Turek proposed openstack/ironic: Add classes for Portgroups API enablement https://review.openstack.org/347549 | 19:43 |
| thiagop | jroll: everything dynamic, you mean? | 19:43 |
| jroll | yes | 19:43 |
| jroll | thiagop: also, what happens if dynamic_allocation=True is not set (or dynamic_allocation=False is set) when the person upgrades to P? | 19:43 |
| *** david-lyle has quit IRC | 19:43 | |
| thiagop | jroll: this is on the spec. Roughly, on P, dynamic_allocation=False is still pre-allocation while lack of dynamic_allocation flag implies dynamic | 19:45 |
| thiagop | on O we'll remove the flag | 19:45 |
| *** aNupoisc has quit IRC | 19:46 | |
| *** penick has joined #openstack-ironic | 19:46 | |
| thiagop | jroll: I think this is the way it is because somebody at some point said to us that this is a breaking change, and needs a transition | 19:46 |
| jroll | thiagop: I guess my question is, is it possible to use dynamic allocation if someone has pre-allocated a node in oneview? | 19:46 |
| jroll | or rather, why is this a breaking change? | 19:46 |
| thiagop | jroll: yup, the operator must just add the flag and remove the server profile (that's what our CLI does) | 19:48 |
| jroll | ahh, the server profile is the issue | 19:48 |
| jroll | ok, thanks | 19:48 |
| jroll | (I wish it could just handle both without a per-node flag) :( | 19:49 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor multitenant networking release notes https://review.openstack.org/352659 | 19:49 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor OneView dynamic allocation release notes https://review.openstack.org/353049 | 19:49 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up release notes for 6.1.0 https://review.openstack.org/352660 | 19:49 |
| jroll | thiagop: I made a couple changes if you want to make sure that looks good to you ^ | 19:49 |
| jroll | rloo: ^ updated those reno patches | 19:49 |
| thiagop | jroll: cli also moves active nodes in pre-allocation too, but then the server profiles will not be removed | 19:49 |
| JayF | jroll: looking at all 3 as well | 19:49 |
| jroll | thiagop: got it | 19:49 |
| jroll | JayF: thanks | 19:49 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: [WIP] Add storage_interface to base driver class https://review.openstack.org/348006 | 19:53 |
| thiagop | jroll: fixing 2 connectors missing ("will +last+ until" and "don't have +the+ dynamic allocation +flag+") | 19:55 |
| * thiagop runs tox | 19:55 | |
| jroll | thiagop: okay, just make sure you don't drop the rebase :) | 19:56 |
| thiagop | I'm editing directly on gerrit to make sure I'll not undo what you did | 19:56 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: Refactor OneView dynamic allocation release notes https://review.openstack.org/353049 | 19:57 |
| openstackgerrit | Michael Turek proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/332177 | 19:57 |
| jroll | thiagop: oh, I meant to rewrite the deprecation part, even left myself a comment :( | 19:57 |
| jroll | mind if I fix that quick? | 19:58 |
| thiagop | jroll: nope | 19:58 |
| jroll | k, one sec | 19:59 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor OneView dynamic allocation release notes https://review.openstack.org/353049 | 20:00 |
| jroll | there you are sir | 20:00 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: [WIP] Add storage_interface to base driver class https://review.openstack.org/348006 | 20:02 |
| *** causten_ has quit IRC | 20:02 | |
| *** causten has joined #openstack-ironic | 20:03 | |
| thiagop | jroll: lgtm, thanks | 20:04 |
| jroll | thiagop: awesome, thank you for helping me understand :) | 20:04 |
| * jroll is super excited for this release | 20:04 | |
| thiagop | I wonder if there is something else we could do to make people understand it just by reading... I'll not be on the channel forever | 20:05 |
| jroll | thiagop: the first part is convincing people to read instead of ask :) | 20:06 |
| * jroll is guilty of that | 20:06 | |
| thiagop | In fact, I'll be out of OS work starting on september until probably January. But our team will be here to help | 20:06 |
| jroll | :( | 20:06 |
| jroll | we'll miss you! | 20:07 |
| * thiagop is going to hike all over Patagonia, but will be sorry for missing Barcelona | 20:07 | |
| jroll | oh, well that's a good reason! | 20:08 |
| *** bvandewa has joined #openstack-ironic | 20:08 | |
| Madasi | take a page from old shareware. Sure, I'll answer that. But first, could you real quick tell me what the third word of the fourth paragraph on the second page is? Thanks! /s | 20:10 |
| *** david-lyle_ is now known as david-lyle | 20:12 | |
| *** causten has quit IRC | 20:12 | |
| *** causten has joined #openstack-ironic | 20:13 | |
| *** baoli has quit IRC | 20:15 | |
| *** baoli has joined #openstack-ironic | 20:15 | |
| jroll | hah | 20:16 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Change Bifrost to TinyIPA as the default https://review.openstack.org/352919 | 20:23 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Comply with iLO model of usage for one-time boot https://review.openstack.org/350060 | 20:24 |
| *** baoli has quit IRC | 20:31 | |
| *** aNupoisc has joined #openstack-ironic | 20:31 | |
| *** jmiu_ has left #openstack-ironic | 20:31 | |
| *** PollyZ has joined #openstack-ironic | 20:32 | |
| *** dprince has quit IRC | 20:37 | |
| *** mgoddard has quit IRC | 20:39 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix DHCP test scenario https://review.openstack.org/353099 | 20:41 |
| openstackgerrit | Kurt Taylor proposed openstack/ironic: Add CI documentation https://review.openstack.org/353102 | 20:47 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Change Bifrost to TinyIPA as the default https://review.openstack.org/352919 | 20:48 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix courtesy env-var file for recent auth changes https://review.openstack.org/352949 | 20:49 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix DHCP test scenario https://review.openstack.org/353099 | 20:50 |
| devananda | jroll: seems we added the node network_interface and resource_class without any api-ref updates | 20:51 |
| devananda | this api-ref update has ended up consuming my whole morning, as there's lots out of date already | 20:51 |
| *** v_davidoff has joined #openstack-ironic | 20:57 | |
| v_davidoff | Hello. I'm following the install guides for ironic and I think I'm done, except that I don't see my baremetal hypervisor in the list of nova hypervisors. What might cause it to not show up there? | 20:57 |
| *** dergrunepunkt has joined #openstack-ironic | 20:58 | |
| dergrunepunkt | hi guys, to deploy with tripleO is it absolutelly neccesary to have IPMI? | 20:59 |
| JayF | dergrunepunkt: If you're asking about tripleo, you shuold probably ask in #tripleo | 21:00 |
| JayF | dergrunepunkt: Ironic supports power management via ipmi/ilo/drac, or various PDUs (via out of tree drivers) | 21:01 |
| dergrunepunkt | Thanks a lot JayF | 21:01 |
| *** Sukhdev has quit IRC | 21:06 | |
| *** v_davidoff has quit IRC | 21:07 | |
| *** Sukhdev has joined #openstack-ironic | 21:09 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 21:10 |
| mrda | Morning Ironic | 21:10 |
| thiagop | morning mrda | 21:13 |
| *** thrash is now known as thrash|g0ne | 21:15 | |
| TheJulia | good morning mrda | 21:15 |
| jlvillal | mrda: Good morning | 21:16 |
| JayF | So I have a question about the flat NetworkInterface | 21:18 |
| JayF | we don't appear to require a separate provisioning network, but we do require a separate cleaning network | 21:18 |
| JayF | is there some kind of logic behind that? | 21:18 |
| *** ayoung has quit IRC | 21:19 | |
| JayF | I'm trying to determine if the flat Network driver needs to implement add/remove rescue network (which is /not/ the network you rescue into, just the network that boots the rescue ramdisk) | 21:19 |
| JayF | jroll: ^ thoughts? | 21:20 |
| TheJulia | JayF: I would think yes if the intent is for the rescue functionality to work on flat networks.... | 21:21 |
| JayF | TheJulia: but we require a cleaning network in that same driver | 21:21 |
| JayF | TheJulia: which is why I'm confused; it doesn' | 21:22 |
| JayF | *it doesn't /actually/ seem flat | 21:22 |
| JayF | https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/network/flat.py#L82 | 21:22 |
| mrda | o/ | 21:22 |
| *** dergrunepunkt has quit IRC | 21:23 | |
| TheJulia | JayF: as I interpret it, it just ends up updating the dhcp info that neutron hands out | 21:25 |
| TheJulia | JayF: but that interpretation that I have uses context in to how the dhcp driver worked | 21:25 |
| jroll | devananda: yeah, I have plans to update the docs for resource_class, it's on my todo list for tomorrow in fact, shame we missed it on network_interface | 21:25 |
| JayF | TheJulia: that does make sense | 21:26 |
| jroll | JayF: we don't require a separate cleaning network for the flat case, but ironic needs to know which network it is, and we use the config for that | 21:26 |
| JayF | TheJulia: so we don't play vlan-swap, but we do play network-swap | 21:26 |
| jroll | for provisioning we use what's passed in by nova | 21:26 |
| jroll | not even network swap, we need to know where to create the port | 21:26 |
| TheJulia | JayF: basically yes | 21:26 |
| JayF | okay that makes a lot of sense' | 21:26 |
| jroll | so yeah, likely the same deal for rescue | 21:26 |
| JayF | No, I get it now | 21:27 |
| JayF | just didn't grok the difference between changing dhcp configs and changing network ports actually around | 21:27 |
| *** michauds has quit IRC | 21:31 | |
| *** zigo has quit IRC | 21:32 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor OneView dynamic allocation release notes https://review.openstack.org/353049 | 21:34 |
| jroll | rloo: updated once more ^ | 21:34 |
| jroll | let's get those in the gate tonight so they're merged in the morning | 21:34 |
| *** zigo has joined #openstack-ironic | 21:35 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Update api-ref for v1.22 https://review.openstack.org/353117 | 21:39 |
| devananda | regenerating all the API samples is now taking me < 30 seconds | 21:40 |
| devananda | well, almost all | 21:40 |
| * devananda afk's to look at houses | 21:41 | |
| openstackgerrit | Merged openstack/ironic: Update devstack configure_ironic_ssh_keypair https://review.openstack.org/352714 | 21:41 |
| rloo | jroll: reviewed. any idea why release notes take so long with jenkins? should be like docs, quick! | 21:53 |
| JayF | rloo: I suspect we haven't done the bits we need | 21:53 |
| JayF | rloo: to make those only run tests on releasenotes | 21:53 |
| rloo | JayF: hopefully that's the only reason... | 21:54 |
| JayF | rloo: it looks like it's configured to skip dsvm... | 21:56 |
| *** Goneri has joined #openstack-ironic | 21:56 | |
| JayF | https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L1172 | 21:57 |
| JayF | I need to finish up these rescue patches and can't dig further now though | 21:57 |
| *** sdake has quit IRC | 21:57 | |
| rloo | JayF: no worries. this isn't urgent. that link seems to indicate it'll run the same tests for docs as for release notes. i'll try to keep an eye on that. | 21:59 |
| *** ppiela has quit IRC | 22:00 | |
| *** hj-hpe has quit IRC | 22:01 | |
| *** hjanssen-hpe has quit IRC | 22:01 | |
| *** rcernin has quit IRC | 22:02 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Use constraints for all the things https://review.openstack.org/353124 | 22:06 |
| jlvillal | JayF: jroll ^^^ To fix broken stable/liberty gate for IPA | 22:07 |
| JayF | jlvillal: so question, was only unit tests fixed, or the code too? | 22:07 |
| jlvillal | JayF: Neither | 22:07 |
| JayF | jlvillal: you misunderstand the question | 22:08 |
| jlvillal | JayF: Have it use the upper-constraints.txt file is the fix. | 22:08 |
| JayF | sure | 22:08 |
| JayF | but without that fix in the ramdisk | 22:08 |
| JayF | is it just going to enable the CI to package broken software | 22:08 |
| jlvillal | JayF: I don't understand. | 22:08 |
| JayF | This also doesn't enable constraints for our image builds, as it isn't | 22:09 |
| JayF | possible to pass an environment variable into the Dockerfile to allow | 22:09 |
| JayF | folks to choose their own constraints file (and we don't want to force | 22:09 |
| JayF | people to rely on git.o.o for building images). | 22:09 |
| JayF | from the commit message | 22:09 |
| JayF | so if this "fixes" tests | 22:09 |
| JayF | and it doesn't do anything inside the ramdisk | 22:09 |
| JayF | are we just leaving the ramdisk broken unawares | 22:09 |
| jlvillal | That makes sense what you are saying. But I don't know how to fix that issue :( | 22:10 |
| jlvillal | JayF: As far as I can tell there is no usage of constraints in the Docker image. | 22:11 |
| jlvillal | That is a TBD item | 22:11 |
| jroll | rloo: the gate has been super backed up recently, due to under-resourcing, over-working, and some misconfiguration | 22:17 |
| jroll | s/gate/CI pipeline/ | 22:17 |
| jroll | JayF: yeesh, we should use constraints in that build | 22:17 |
| JayF | it's a sneakily hard problem | 22:18 |
| rloo | jroll: oh. ugh. | 22:18 |
| JayF | we should at least document the need in a bug though | 22:18 |
| jroll | JayF: jlvillal: we should fix it in a separate patch (that we put on master and backport down), it's just an argument to pip (-c) | 22:18 |
| jroll | yeah | 22:19 |
| JayF | jroll: the problem was that we don't know how to get that file into docker easily | 22:19 |
| JayF | jroll: iirc | 22:19 |
| jroll | JayF: I think it accepts a url? | 22:19 |
| JayF | jroll: you should look at the original patch, I remember bikeshedding a bit about it | 22:19 |
| jroll | yes, it does take a URL | 22:19 |
| jroll | install_command = pip install -U -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages} | 22:19 |
| jlvillal | jroll: Separate patch for the Dockerfile issue? | 22:19 |
| jroll | we don't know how to override it, I think was the problem | 22:19 |
| jroll | jlvillal: yes | 22:19 |
| jlvillal | Okay. | 22:20 |
| jroll | JayF: I'm thinking, have equivalent to what I pasted > have upstream upper-constraints in use always > no constraints at all | 22:20 |
| jroll | rloo: too bad you don't work for someone with a public cloud so I could nudge you :P | 22:21 |
| JayF | jroll: sounds good to me? | 22:21 |
| jroll | cool | 22:22 |
| rloo | jroll: where would you like me to apply to? :) | 22:22 |
| jroll | rloo: :D | 22:22 |
| rloo | jroll: or are you suggesting that Intel should get into the public cloud business :D | 22:23 |
| jroll | rloo: I just want more CI resources, I'm unopinionated as to how that happens :) | 22:23 |
| rloo | jroll: yeah, i want more too. | 22:23 |
| rloo | jroll: i guess it isn't in OSIC's mandate? | 22:24 |
| jroll | rloo: OSIC provides some | 22:24 |
| jroll | they have more, but those are for other testing like kolla is doing right now | 22:24 |
| *** rbudden has quit IRC | 22:24 | |
| rloo | jroll: did internap offer any? | 22:24 |
| jlvillal | JayF: When you said, "you should look at the original patch, I remember bikeshedding a bit about it" Which patch was that? | 22:24 |
| *** bfournie has quit IRC | 22:25 | |
| jroll | rloo: I *think* internap is there, though I don't remember if they have VMs. they have offered to help get baremetal machines in the CI pool. | 22:25 |
| jlvillal | JayF: I looked at: https://review.openstack.org/#/c/318669/ but only saw your vote. | 22:25 |
| *** davidlenwell has quit IRC | 22:25 | |
| rloo | jroll: oh. maybe that was it. we/openstack should periodically ask for more resources... | 22:26 |
| jroll | jlvillal: we probably discussed it in IRC, or all knew it would be hell to pass that env variable in and just sighed and voted | 22:26 |
| mgagne | jroll: we have VMs http://grafana.openstack.org/dashboard/db/nodepool-internap | 22:26 |
| jroll | rloo: I believe they do | 22:26 |
| jlvillal | jroll: Ah okay :) | 22:26 |
| jroll | mgagne: and in the CI pool, yay | 22:26 |
| JayF | jlvillal: i was wrong | 22:26 |
| JayF | jlvillal: or more like == jroll | 22:26 |
| mgagne | not as much as we would like though =) | 22:27 |
| rloo | jroll: alternatively, openstack has a lot of $$, maybe they can use some of it to get more resources somehow... | 22:27 |
| jlvillal | heh | 22:27 |
| rloo | jroll: at our end, we could try to do better with our patches so there are fewer revisions etc... | 22:27 |
| jroll | rloo: infra is working on setting up a pool that I think is partially funded by the foundation, actually :) | 22:27 |
| rloo | jroll: ah, great idea! | 22:28 |
| jroll | rloo: I trust smarter people than me have it covered, but more always helps, hence the 'too bad you work for intel' shot :) | 22:28 |
| rloo | jroll: ha ha. i think our contribution as developers is to try to spend a few extra minutes with our patches before we push them. | 22:28 |
| jroll | JayF: you +A'd https://review.openstack.org/#/c/352660/ , but part of that patch was to fix formatting, so I think we wanted to wait to see what it looked like :) | 22:29 |
| jroll | rloo: +1 | 22:29 |
| JayF | jroll: what approval <.< >.> I don't see no approval | 22:29 |
| * jroll has been trying to make sure he runs unit tests at least | 22:29 | |
| jroll | JayF: thanks :) | 22:29 |
| * jroll goes back to making dinner | 22:30 | |
| *** sdake has joined #openstack-ironic | 22:30 | |
| JayF | you know you can run that locally too, right? | 22:30 |
| *** sdake has quit IRC | 22:30 | |
| JayF | the docs build and then view how it rendesr | 22:30 |
| JayF | *renders | 22:30 |
| JayF | I always do that with docs, or else it'd take me days to get it to look like I want :P | 22:30 |
| *** sdake has joined #openstack-ironic | 22:30 | |
| jroll | JayF: I do, but I want the reviewers to say "yes this looks good" | 22:30 |
| jroll | I guess what I'm saying is, I want whoever +A's that to be happy with the rendering | 22:31 |
| jroll | anyway, see y'all tomorrow | 22:31 |
| JayF | no, you're right | 22:31 |
| JayF | I should've waited | 22:31 |
| thiagop | Folks, I'm calling it a day (finally) | 22:33 |
| thiagop | see you tomorrow | 22:33 |
| JayF | o/ | 22:33 |
| *** fragatin_ has joined #openstack-ironic | 22:33 | |
| *** v_davidoff has joined #openstack-ironic | 22:33 | |
| *** sdake_ has joined #openstack-ironic | 22:33 | |
| thiagop | JayF: I was unable to get to that docs thing today. Tomorrow is a sprint review /retrospective around here, but I'll try to get to it between meetings | 22:34 |
| thiagop | Good evening all | 22:34 |
| JayF | thiagop: I think we'd like to get it in before the release, so please make it happen if you can | 22:34 |
| JayF | although I'm out the rest of the week so others will haev to review | 22:34 |
| thiagop | JayF: sure, but now I need to get something (microweaveable) to dinner, I'm starving | 22:34 |
| *** sdake has quit IRC | 22:34 | |
| thiagop | see ya | 22:35 |
| JayF | I'm more suggesting to work and ignore your meetings, not to work and ignore your dinner :P | 22:35 |
| JayF | #priorities | 22:35 |
| thiagop | JayF: I'm the scrum master so... ¯\_(ツ)_/¯ | 22:35 |
| thiagop | But I'll really try | 22:35 |
| *** thiagop has quit IRC | 22:36 | |
| *** fragatina has quit IRC | 22:36 | |
| *** davidlenwell has joined #openstack-ironic | 22:38 | |
| *** sdake_ is now known as sdake | 22:38 | |
| *** rbudden has joined #openstack-ironic | 22:51 | |
| JayF | I'm going to close the overall metrics RFE bug as released | 22:52 |
| jlvillal | JayF: Woo hoo! :) | 22:52 |
| JayF | and file bugs for 1) vendor drivers that need metrics 2) remaining IPA metrics and 3) Metrics dev docs in Ironic | 22:52 |
| *** Goneri has quit IRC | 22:53 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: WIP: Agent rescue implementation https://review.openstack.org/353147 | 22:54 |
| *** sabeen has quit IRC | 22:56 | |
| *** PollyZ has quit IRC | 23:10 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: WIP: Add rescuewait timeout perodic task https://review.openstack.org/353156 | 23:11 |
| *** Sukhdev has quit IRC | 23:15 | |
| *** Sukhdev has joined #openstack-ironic | 23:16 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: [POC] Enable support for OSprofiler in Ironic. https://review.openstack.org/337987 | 23:18 |
| *** david-lyle has quit IRC | 23:20 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Enable support for OSprofiler in Ironic. https://review.openstack.org/337987 | 23:24 |
| *** rpioso has quit IRC | 23:29 | |
| *** aNupoisc has quit IRC | 23:34 | |
| JayF | If anyone feels up to some simple bug triage + RFE-approvals; as I said earlier; I marked the original Metrics support RFE as released, given we're about to release a large quantity of metrics in 6.1 \o/, but I made sure to file bugs for the gaps, one per driver, one for IPA, and one for devdocs https://bugs.launchpad.net/ironic/+bug/1611553 https://bugs.launchpad.net/ironic/+bug/1611554 | 23:34 |
| openstack | Launchpad bug 1611553 in Ironic "[RFE] Metrics for IPA extensions and GenericHardwareManager" [Undecided,New] | 23:34 |
| openstack | Launchpad bug 1611554 in Ironic "[RFE] Metrics for iLo drivers" [Undecided,New] | 23:34 |
| JayF | https://bugs.launchpad.net/ironic/+bug/1611555 https://bugs.launchpad.net/ironic/+bug/1611556 https://bugs.launchpad.net/ironic/+bug/1611557 https://bugs.launchpad.net/ironic/+bug/1611559 | 23:34 |
| openstack | Launchpad bug 1611555 in Ironic "[RFE] Metrics for DRAC driver" [Undecided,New] | 23:34 |
| openstack | Launchpad bug 1611556 in Ironic "[RFE] Metrics for iRMC driver" [Undecided,New] | 23:34 |
| openstack | Launchpad bug 1611557 in Ironic "[RFE] Metrics support for OneView drivers" [Undecided,New] | 23:34 |
| openstack | Launchpad bug 1611559 in Ironic "Developer docs for Metrics" [Undecided,New] | 23:34 |
| JayF | When I'm back on Monday, I'll work on the metrics devdocs bug and the IPA metrics bugs if nobody else has picked them up yet | 23:35 |
| JayF | thanks! | 23:35 |
| *** ayoung has joined #openstack-ironic | 23:35 | |
| openstackgerrit | Merged openstack/ironic: Remove duplicate parameters from local.conf example https://review.openstack.org/350970 | 23:36 |
| JayF | lol I didn't expect it to lookup all of them :) | 23:36 |
| *** david-lyle has joined #openstack-ironic | 23:38 | |
| *** ppiela has joined #openstack-ironic | 23:43 | |
| *** v_davidoff has quit IRC | 23:43 | |
| *** sdake has quit IRC | 23:45 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Update api-ref for v1.22 https://review.openstack.org/353117 | 23:45 |
| devananda | take two. now with corrected /v1/driver/ samples! | 23:46 |
| * JayF adds himself as a reviewer to take a look when gate finishes | 23:50 | |
| devananda | :) | 23:51 |
| devananda | JayF: if you're interested now, the https://review.openstack.org/#/c/353117/2/api-ref/regenerate-samples.sh could use eyes and won't be in the rendered output | 23:51 |
| devananda | by eyes I mean, tell me if ya'll object to shoving a shell script in there like this | 23:52 |
| devananda | it makes my life maintaining these much easier, but I can just as well put it on my github | 23:52 |
| JayF | I mean, I have no objection at all to having that in there | 23:54 |
| JayF | I wonder if that's the best dir for it | 23:54 |
| devananda | JayF: I like my bikesheds green ;) | 23:54 |
| *** Fdaisuke has quit IRC | 23:56 | |
| JayF | lol | 23:56 |
| JayF | I just didn't know if we have like a tools/ directory | 23:57 |
| JayF | yeah, that under tools/ would be perfect imo | 23:57 |
| JayF | I won't -1 for it but you asked ;) | 23:57 |
| *** rbudden has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!