| *** pmannidi has joined #openstack-ironic | 00:03 | |
| *** pmannidi has quit IRC | 00:05 | |
| *** pmannidi has joined #openstack-ironic | 00:11 | |
| *** mtanino has joined #openstack-ironic | 00:40 | |
| *** hw_wutianwei has joined #openstack-ironic | 00:41 | |
| *** hoangcx has joined #openstack-ironic | 00:49 | |
| *** phuongnh has joined #openstack-ironic | 00:54 | |
| *** litao has joined #openstack-ironic | 00:55 | |
| *** rbudden has quit IRC | 00:56 | |
| *** SerenaFeng has joined #openstack-ironic | 00:56 | |
| *** jistr has quit IRC | 01:00 | |
| *** jistr has joined #openstack-ironic | 01:00 | |
| *** rob0t__ has joined #openstack-ironic | 01:07 | |
| *** mtanino has quit IRC | 01:08 | |
| *** gcb has joined #openstack-ironic | 01:45 | |
| *** tovin07 has joined #openstack-ironic | 01:49 | |
| *** gcb has quit IRC | 01:49 | |
| *** gcb has joined #openstack-ironic | 01:50 | |
| *** ianychoi has quit IRC | 01:52 | |
| *** zhugaoxiao has joined #openstack-ironic | 01:53 | |
| *** ianychoi has joined #openstack-ironic | 01:54 | |
| *** mtanino has joined #openstack-ironic | 02:02 | |
| csmart | morning mrda | 02:03 |
|---|---|---|
| csmart | (belated) | 02:03 |
| *** hoangcx_ has joined #openstack-ironic | 02:11 | |
| *** tuanluong has joined #openstack-ironic | 02:12 | |
| *** hoangcx has quit IRC | 02:14 | |
| mrda | csmart: o/ | 02:15 |
| *** hoangcx has joined #openstack-ironic | 02:16 | |
| *** hoangcx_ has quit IRC | 02:18 | |
| *** baoli has joined #openstack-ironic | 02:21 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic master: Add supported_power_states field to node APIs https://review.openstack.org/382194 | 02:21 |
| *** baoli has quit IRC | 02:22 | |
| *** jvgrant_ has quit IRC | 02:22 | |
| *** jvgrant has joined #openstack-ironic | 02:22 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic master: Add supported_power_states field to api-ref https://review.openstack.org/427000 | 02:30 |
| *** hoangcx_ has joined #openstack-ironic | 02:43 | |
| *** shu-mutou has joined #openstack-ironic | 02:44 | |
| *** hoangcx has quit IRC | 02:45 | |
| *** yonglihe has joined #openstack-ironic | 03:14 | |
| *** mikal has quit IRC | 03:29 | |
| *** JayF has quit IRC | 03:34 | |
| *** mikal has joined #openstack-ironic | 03:34 | |
| *** ianychoi has quit IRC | 03:37 | |
| *** JayF has joined #openstack-ironic | 03:37 | |
| *** ianychoi has joined #openstack-ironic | 03:39 | |
| *** SerenaFeng has quit IRC | 03:47 | |
| *** amotoki has quit IRC | 03:56 | |
| *** sdake has joined #openstack-ironic | 04:00 | |
| *** nmathew has joined #openstack-ironic | 04:01 | |
| *** links has joined #openstack-ironic | 04:09 | |
| *** amotoki has joined #openstack-ironic | 04:11 | |
| *** ianychoi has quit IRC | 04:12 | |
| *** ianychoi has joined #openstack-ironic | 04:14 | |
| *** vikrant has joined #openstack-ironic | 04:25 | |
| *** wrouesnel has quit IRC | 04:32 | |
| *** wrouesnel has joined #openstack-ironic | 04:33 | |
| *** wrouesnel has quit IRC | 04:35 | |
| *** wrouesnel has joined #openstack-ironic | 04:36 | |
| *** amarao has joined #openstack-ironic | 04:38 | |
| *** amarao has quit IRC | 04:39 | |
| openstackgerrit | Anh Tran proposed openstack/bifrost master: Typo fix: prepatory => preparatory https://review.openstack.org/429500 | 04:40 |
| *** david-lyle has joined #openstack-ironic | 04:46 | |
| *** david-lyle_ has joined #openstack-ironic | 04:50 | |
| *** david-lyle has quit IRC | 04:50 | |
| *** wrouesnel has quit IRC | 04:50 | |
| *** wrouesnel has joined #openstack-ironic | 04:51 | |
| *** david-lyle_ is now known as david-lyle | 04:52 | |
| *** sabeen has joined #openstack-ironic | 04:57 | |
| *** amotoki has quit IRC | 05:04 | |
| *** wrouesnel has quit IRC | 05:06 | |
| *** wrouesnel has joined #openstack-ironic | 05:07 | |
| *** wrouesnel has quit IRC | 05:12 | |
| *** wrouesnel has joined #openstack-ironic | 05:12 | |
| *** wrouesnel has quit IRC | 05:16 | |
| *** wrouesnel has joined #openstack-ironic | 05:17 | |
| *** sdake has quit IRC | 05:18 | |
| *** david-lyle has quit IRC | 05:21 | |
| *** shu-mutou is now known as shu-mutou-AWAY | 05:27 | |
| *** amotoki has joined #openstack-ironic | 05:35 | |
| *** SerenaFeng has joined #openstack-ironic | 05:37 | |
| *** wrouesnel has quit IRC | 05:59 | |
| *** wrouesnel has joined #openstack-ironic | 05:59 | |
| *** aparnav has joined #openstack-ironic | 06:00 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 06:06 | |
| *** MattMan_1 has quit IRC | 06:14 | |
| *** MattMan_1 has joined #openstack-ironic | 06:14 | |
| *** cvstealth has quit IRC | 06:17 | |
| *** cvstealth has joined #openstack-ironic | 06:18 | |
| *** links has quit IRC | 06:19 | |
| *** AlexeyAbashkin has quit IRC | 06:21 | |
| *** links has joined #openstack-ironic | 06:35 | |
| *** yolanda has joined #openstack-ironic | 06:43 | |
| *** mtanino has quit IRC | 06:46 | |
| *** moshele has joined #openstack-ironic | 06:49 | |
| *** vsaienko has joined #openstack-ironic | 06:49 | |
| *** moshele has quit IRC | 06:50 | |
| *** moshele has joined #openstack-ironic | 06:51 | |
| *** g3ek has quit IRC | 06:55 | |
| *** yogi has joined #openstack-ironic | 06:57 | |
| *** moshele has quit IRC | 07:01 | |
| *** g3ek has joined #openstack-ironic | 07:05 | |
| *** links has quit IRC | 07:05 | |
| *** tesseract has joined #openstack-ironic | 07:12 | |
| *** gcb has quit IRC | 07:13 | |
| *** gcb has joined #openstack-ironic | 07:14 | |
| *** links has joined #openstack-ironic | 07:17 | |
| *** vsaienko has quit IRC | 07:18 | |
| *** tovin07 has quit IRC | 07:21 | |
| *** ChubYann has quit IRC | 07:33 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Support `update_firmware` to hpsum based firmware update https://review.openstack.org/422572 | 07:34 |
| *** dsneddon_afk has joined #openstack-ironic | 07:35 | |
| *** chihhsin has quit IRC | 07:42 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:45 | |
| *** ccamacho has joined #openstack-ironic | 07:48 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests https://review.openstack.org/423556 | 07:50 |
| *** gcb has quit IRC | 07:52 | |
| *** gcb has joined #openstack-ironic | 07:54 | |
| *** pcaruana has joined #openstack-ironic | 07:55 | |
| *** moshele has joined #openstack-ironic | 08:00 | |
| *** vsaienko has joined #openstack-ironic | 08:02 | |
| *** pcaruana has quit IRC | 08:07 | |
| *** pcaruana has joined #openstack-ironic | 08:07 | |
| pas-ha | morning Ironic :) | 08:19 |
| *** owalsh-afk has joined #openstack-ironic | 08:22 | |
| vdrok | morning ironic and pas-ha ! | 08:32 |
| *** SerenaFeng has quit IRC | 08:34 | |
| *** stack_ has joined #openstack-ironic | 08:34 | |
| vsaienk0 | morning pas-ha, vdrok and all Ironic'ers! | 08:38 |
| tuanluong | morning ironic team | 08:40 |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Add HPSUM firmware update support https://review.openstack.org/422004 | 08:40 |
| tuanluong | morning pas-ha vdrok vsaienk0 | 08:40 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests https://review.openstack.org/423556 | 08:44 |
| *** links has quit IRC | 08:45 | |
| *** links has joined #openstack-ironic | 08:45 | |
| vdrok | morning vsaienk0 and tuanluong | 08:47 |
| *** fxpester has joined #openstack-ironic | 08:48 | |
| pas-ha | morning tuanluong vsaienk0 vdrok :) | 08:49 |
| *** sabeen has quit IRC | 08:52 | |
| *** ralonsoh has joined #openstack-ironic | 08:54 | |
| *** lifeless has quit IRC | 08:54 | |
| *** lifeless has joined #openstack-ironic | 08:55 | |
| *** mjura has joined #openstack-ironic | 08:59 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:00 | |
| openstackgerrit | Merged openstack/bifrost master: Remove my_ip_address from use https://review.openstack.org/428975 | 09:01 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/429217 | 09:01 |
| *** SerenaFeng has joined #openstack-ironic | 09:02 | |
| *** vsaienko has quit IRC | 09:05 | |
| *** rob0t__ has quit IRC | 09:09 | |
| *** lucasagomes has joined #openstack-ironic | 09:25 | |
| lucasagomes | morning all | 09:26 |
| csmart | lucasagomes: morning \o | 09:27 |
| csmart | pas-ha: vdrok vsaienk0 tuanluong good morning, too! | 09:28 |
| vsaienk0 | dtantsur, jroll: what do you think about https://review.openstack.org/#/c/429573/ ? | 09:28 |
| patchbot | patch 429573 - openstack-infra/project-config - Remove Ironic partition-pxe_snmp job | 09:28 |
| *** tovin07 has joined #openstack-ironic | 09:29 | |
| *** links has quit IRC | 09:35 | |
| *** mgoddard_ has joined #openstack-ironic | 09:39 | |
| *** SerenaFeng has quit IRC | 09:47 | |
| *** derekh has joined #openstack-ironic | 09:50 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-specs master: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 09:53 |
| *** gcb has quit IRC | 09:54 | |
| *** yolanda has quit IRC | 09:56 | |
| vdrok | morning lucasagomes and csmart | 09:56 |
| *** AlexeyAbashkin has quit IRC | 09:57 | |
| *** yolanda has joined #openstack-ironic | 09:57 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient master: Include driver properties to osc `driver show` https://review.openstack.org/381153 | 09:57 |
| *** bcornec has quit IRC | 09:58 | |
| lucasagomes | o/ | 10:01 |
| mkrai | Hi Ironic team | 10:02 |
| mkrai | Can you please help review https://review.openstack.org/#/c/415894/ ? | 10:02 |
| patchbot | patch 415894 - ironic-specs - Replace WSME with jsonschema for api validation | 10:02 |
| mkrai | And this too https://review.openstack.org/#/c/413436/ | 10:03 |
| patchbot | patch 413436 - ironic - Add guru meditation report support | 10:03 |
| openstackgerrit | Andrey Shestakov proposed openstack/bifrost master: Allow deploy-dynamic on remote host https://review.openstack.org/411304 | 10:03 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient master: Include driver properties to osc `driver show` https://review.openstack.org/381153 | 10:06 |
| *** sambetts|afk is now known as sambetts | 10:06 | |
| sambetts | Morning all | 10:07 |
| *** fxpester has quit IRC | 10:12 | |
| *** dtantsur|afk is now known as dtantsur | 10:14 | |
| dtantsur | morning ironic, pas-ha, vdrok, vsaienk0, tuanluong, lucasagomes, csmart and sambetts :) | 10:14 |
| vsaienk0 | o/ | 10:14 |
| lucasagomes | dtantsur, hi there | 10:14 |
| vdrok | morning mkrai sambetts and dtantsur | 10:14 |
| tuanluong | morning lucasagomes mkrai sambetts dtantsur | 10:15 |
| csmart | dtantsur: sambetts morning :-) | 10:15 |
| *** owalsh-afk is now known as owalsh | 10:16 | |
| *** fxpester has joined #openstack-ironic | 10:17 | |
| *** yolanda has quit IRC | 10:26 | |
| *** rob0t__ has joined #openstack-ironic | 10:26 | |
| *** tuanluong has quit IRC | 10:26 | |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Add HPSUM firmware update support https://review.openstack.org/422004 | 10:26 |
| *** phuongnh has quit IRC | 10:27 | |
| *** amarao has joined #openstack-ironic | 10:27 | |
| *** yolanda_ has joined #openstack-ironic | 10:29 | |
| *** hoangcx_ has quit IRC | 10:30 | |
| *** rob0t__ has quit IRC | 10:31 | |
| *** rob0t__ has joined #openstack-ironic | 10:32 | |
| *** rob0t__ has quit IRC | 10:36 | |
| sambetts | o/ dtantsur, vdrok, lucasagomes, vsaienk0, csmart | 10:42 |
| vsaienk0 | morning sambetts | 10:42 |
| lucasagomes | sambetts, o/ | 10:51 |
| *** rob0t__ has joined #openstack-ironic | 10:52 | |
| openstackgerrit | Merged openstack/ironic-inspector-specs master: Endpoint to list introspection statuses https://review.openstack.org/375045 | 10:54 |
| *** amotoki has quit IRC | 10:55 | |
| *** rob0t__ has quit IRC | 10:57 | |
| *** ccamacho is now known as ccamacho|lunch | 10:58 | |
| *** mjura has quit IRC | 11:03 | |
| *** rob0t__ has joined #openstack-ironic | 11:08 | |
| *** rob0t__ has quit IRC | 11:12 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Upgrade phase with multitenancy https://review.openstack.org/428043 | 11:18 |
| *** nmathew has quit IRC | 11:20 | |
| *** links has joined #openstack-ironic | 11:21 | |
| *** mgould|afk is now known as mgould | 11:23 | |
| mgould | morning Ironic dtantsur sambetts mkrai pas-ha vdrok vsaienk0 lucasagomes csmart | 11:23 |
| vsaienk0 | \o | 11:24 |
| lucasagomes | mgould, o/ | 11:24 |
| vdrok | morning mgould :) | 11:24 |
| *** skazi has quit IRC | 11:24 | |
| *** links has quit IRC | 11:25 | |
| csmart | Hehe hey mgould :-) | 11:27 |
| *** rob0t__ has joined #openstack-ironic | 11:27 | |
| *** vsaienko has joined #openstack-ironic | 11:28 | |
| *** rob0t__ has quit IRC | 11:31 | |
| *** rob0t__ has joined #openstack-ironic | 11:32 | |
| *** aarefiev_afk is now known as aarefiev | 11:35 | |
| *** amotoki has joined #openstack-ironic | 11:35 | |
| *** rob0t__ has quit IRC | 11:36 | |
| pas-ha | hi mgould sambetts lucasagomes dtantsur | 11:36 |
| *** skazi has joined #openstack-ironic | 11:36 | |
| *** jkilpatr has quit IRC | 11:40 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Upgrade phase with multitenancy https://review.openstack.org/428043 | 11:40 |
| *** rob0t__ has joined #openstack-ironic | 11:42 | |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Add HPSUM firmware update support https://review.openstack.org/422004 | 11:43 |
| *** vsaienko has quit IRC | 11:46 | |
| *** rob0t__ has quit IRC | 11:47 | |
| *** dtantsur is now known as dtantsur|lunch | 11:48 | |
| openstackgerrit | Merged openstack/ironic master: Fix setting persistent boot device does not work https://review.openstack.org/376322 | 11:48 |
| *** vsaienko has joined #openstack-ironic | 11:50 | |
| *** rob0t__ has joined #openstack-ironic | 11:55 | |
| *** rob0t__ has quit IRC | 11:59 | |
| *** amotoki has quit IRC | 12:03 | |
| jtomasek | dtantsur|lunch: hi, is there a use case for a node capability (properties/capabilities) to have empty value e.g. "cap1:value1,cap2:,cap3:value3"? | 12:05 |
| jtomasek | dtantsur|lunch: should that ever happen? | 12:06 |
| *** dtantsur|lunch is now known as dtantsur | 12:10 | |
| *** amotoki has joined #openstack-ironic | 12:10 | |
| dtantsur | jtomasek, I don't think we will treat such capabilities correctly. when we need a boolean one, we usually do cap2:true | 12:10 |
| *** jkilpatr has joined #openstack-ironic | 12:12 | |
| *** early has quit IRC | 12:12 | |
| *** mjura has joined #openstack-ironic | 12:13 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic master: Add nodes tagging support - RPC objects https://review.openstack.org/233357 | 12:15 |
| *** yolanda_ has quit IRC | 12:18 | |
| *** cdaniel-afk is now known as cdaniel | 12:20 | |
| cdaniel | morning all | 12:20 |
| *** ccamacho|lunch is now known as ccamacho | 12:21 | |
| jroll | morning folks | 12:23 |
| *** yolanda_ has joined #openstack-ironic | 12:23 | |
| lucasagomes | jroll, cdaniel o/ | 12:23 |
| *** vikrant has quit IRC | 12:23 | |
| *** mjura has quit IRC | 12:24 | |
| *** yolanda_ has quit IRC | 12:27 | |
| *** yolanda_ has joined #openstack-ironic | 12:28 | |
| *** early has joined #openstack-ironic | 12:28 | |
| mat128|mtg | good morning everyone | 12:29 |
| dtantsur | morning jroll, cdaniel, mat128|mtg | 12:30 |
| *** mat128|mtg is now known as mat128 | 12:30 | |
| *** amotoki has quit IRC | 12:33 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add testcases for OSC baremetal port group commands https://review.openstack.org/385427 | 12:36 |
| *** owalsh is now known as owalsh-afk | 12:37 | |
| jtomasek | dtantsur: ok, we're trying to resolve the problem whether empty values are valid and whether value should be removed when empty in convertion functions: https://review.openstack.org/#/c/424547/7/src/js/utils/nodes.js | 12:38 |
| patchbot | patch 424547 - tripleo-ui - Nodes Table tagging | 12:38 |
| dtantsur | jtomasek, I think it's safe to use https://github.com/openstack/ironic/blob/77bcccf98c99eecc22b97c73d14ff5d1bdce7e8d/ironic/drivers/utils.py#L149-L156 as a canonical implementation | 12:40 |
| jtomasek | dtantsur: ok, thanks | 12:40 |
| *** nicodemos has joined #openstack-ironic | 12:44 | |
| nicodemos | good morning, Ironic. :) | 12:44 |
| jroll | dtantsur: a couple easy stable patches for you https://review.openstack.org/#/q/topic:test-setup+status:open+project:openstack/ironic | 12:46 |
| * jroll touched them so is not voting | 12:46 | |
| openstackgerrit | Andrey Shestakov proposed openstack/ironic master: Remove trailing slash from base_url in tempest plugin https://review.openstack.org/429679 | 12:47 |
| *** lucasagomes is now known as lucas-hungry | 12:49 | |
| *** _fatalis is now known as aimtsou | 12:50 | |
| dtantsur | jroll, sure | 12:52 |
| jroll | thanks | 12:52 |
| * dtantsur adds a stable dashboard to his gertty.yaml | 13:00 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests https://review.openstack.org/423556 | 13:01 |
| openstackgerrit | Merged openstack/ironic stable/newton: Prepare for using standard python tests https://review.openstack.org/428998 | 13:05 |
| vdrok | morning cdaniel jroll mat128 | 13:08 |
| vdrok | and morning nicodemos | 13:08 |
| nicodemos | hey, vdrok. | 13:09 |
| openstackgerrit | Merged openstack/ironic stable/mitaka: Prepare for using standard python tests https://review.openstack.org/429000 | 13:09 |
| *** mjura has joined #openstack-ironic | 13:09 | |
| *** aparnav has quit IRC | 13:12 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [devstack] Allow using "ipmi" hardware type https://review.openstack.org/429694 | 13:12 |
| *** amotoki has joined #openstack-ironic | 13:12 | |
| dtantsur | jroll, will test ^^^ locally, then we can set up a CI | 13:13 |
| *** dprince has joined #openstack-ironic | 13:15 | |
| *** rbudden has joined #openstack-ironic | 13:19 | |
| *** jheroux has joined #openstack-ironic | 13:22 | |
| *** SerenaFeng has joined #openstack-ironic | 13:25 | |
| *** jcoufal has joined #openstack-ironic | 13:27 | |
| *** cdearborn has joined #openstack-ironic | 13:27 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests https://review.openstack.org/423556 | 13:34 |
| bfournie | morning all | 13:34 |
| jroll | dtantsur: awesome, fwiw, I've been running with ipmi hardware type in devstack, seems to work fine | 13:34 |
| *** mjura has quit IRC | 13:35 | |
| dtantsur | great! | 13:35 |
| dtantsur | morning bfournie | 13:35 |
| bfournie | morning dtantsur | 13:36 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient master: Client support resources filecache https://review.openstack.org/420033 | 13:36 |
| *** sdake has joined #openstack-ironic | 13:37 | |
| *** amotoki has quit IRC | 13:38 | |
| *** sdake_ has joined #openstack-ironic | 13:40 | |
| *** ppiela has joined #openstack-ironic | 13:42 | |
| *** sdake has quit IRC | 13:44 | |
| *** SerenaFeng has quit IRC | 13:46 | |
| *** SerenaFeng has joined #openstack-ironic | 13:46 | |
| *** dims_ has quit IRC | 13:47 | |
| TheJulia | good morning everyone | 13:47 |
| *** SerenaFeng has quit IRC | 13:48 | |
| dtantsur | morning TheJulia | 13:48 |
| *** dims has joined #openstack-ironic | 13:48 | |
| *** rob0t__ has joined #openstack-ironic | 13:49 | |
| ppiela | TheJulia: thanks for the weekend reviews! | 13:54 |
| TheJulia | ppiela: no problem, everything I looked at this weekend seemed fairly strait forward | 13:54 |
| nicodemos | morning TheJulia, bfournie | 13:57 |
| *** rloo has joined #openstack-ironic | 14:00 | |
| *** lucas-hungry is now known as lucasagomes | 14:01 | |
| *** srobert has joined #openstack-ironic | 14:01 | |
| TheJulia | ppiela: You may want to take a look at 429111 again, CI didn't quite like it | 14:02 |
| *** sasha2 has joined #openstack-ironic | 14:02 | |
| ppiela | TheJulia: will do | 14:02 |
| *** srobert has quit IRC | 14:02 | |
| TheJulia | Looks like pep8 and an npm validation error at a quick glance | 14:02 |
| *** awiddersheim has quit IRC | 14:03 | |
| *** srobert has joined #openstack-ironic | 14:03 | |
| *** srobert has quit IRC | 14:03 | |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/429217 | 14:03 |
| TheJulia | looks like an excess ) | 14:03 |
| openstackgerrit | Merged openstack/ironic-ui master: Fix node validation problem https://review.openstack.org/429012 | 14:03 |
| *** srobert has joined #openstack-ironic | 14:04 | |
| *** awiddersheim has joined #openstack-ironic | 14:05 | |
| openstackgerrit | Merged openstack/ironic-ui master: Add support for additional port attributes https://review.openstack.org/429066 | 14:05 |
| *** dtantsur is now known as dtantsur|brb | 14:06 | |
| openstackgerrit | Merged openstack/bifrost master: Typo fix: prepatory => preparatory https://review.openstack.org/429500 | 14:07 |
| *** aparnav has joined #openstack-ironic | 14:10 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient master: Client support resources filecache https://review.openstack.org/420033 | 14:11 |
| TheJulia | ppiela: reviewed the release notes you've added. | 14:12 |
| ppiela | TheJulia: thx, working on the ci failures | 14:13 |
| *** vsaienko has quit IRC | 14:14 | |
| *** owalsh-afk has quit IRC | 14:14 | |
| *** vsaienko has joined #openstack-ironic | 14:15 | |
| TheJulia | Awesome | 14:15 |
| *** owalsh-afk has joined #openstack-ironic | 14:15 | |
| *** owalsh-afk is now known as owalsh | 14:16 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient master: Client support resources filecache https://review.openstack.org/420033 | 14:16 |
| aparnav | Hi JayF, Regarding you comment in https://review.openstack.org/#/c/408441/, Does the doc patch has to be blocked till the tests added in CI? | 14:17 |
| patchbot | patch 408441 - ironic - Doc for sanitize erase support in iLO drivers | 14:17 |
| *** baoli has joined #openstack-ironic | 14:18 | |
| *** chlong has joined #openstack-ironic | 14:19 | |
| *** yolanda_ has quit IRC | 14:21 | |
| *** yolanda_ has joined #openstack-ironic | 14:21 | |
| *** lindycoder has joined #openstack-ironic | 14:22 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Release-notes for recent feature additions https://review.openstack.org/429414 | 14:23 |
| *** lindycoder has quit IRC | 14:25 | |
| *** superdan is now known as dansmith | 14:30 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Add support for editing Ironic network ports https://review.openstack.org/429111 | 14:34 |
| TheJulia | aparnav: I think JayF, and he can correct me if I'm misinterpreting him here, but I think what he is stating is that it is not the best idea to include DIB element documentation in ironic's documentation since it is not explicitly tested as part of CI. My concern is that DIB may change in the next year invalidating that documentation. Realistically, part | 14:38 |
| TheJulia | of it seems like documentation that should be in the dib element and that it should be explicitly referenced. | 14:38 |
| TheJulia | aparnav: Also, thinking about it and putting my operator hat on, my immediate question would be "What generations are supported by this? What model of raid controllers are supported by this?" But that should definitely be in DIB element doc. | 14:39 |
| *** baoli has quit IRC | 14:39 | |
| TheJulia | aparnav: I posted feedback on the doc update to that effect | 14:41 |
| TheJulia | ppiela: Looks like the release note rev now has elements from the prior revision. :) | 14:42 |
| *** ElCoyote_ has joined #openstack-ironic | 14:43 | |
| ppiela | TheJulia: Yep, my mistake, I first checked changes into the wrong branch | 14:45 |
| *** baoli has joined #openstack-ironic | 14:46 | |
| aparnav | TheJulia, Thank you I will update it accordingly | 14:46 |
| mat128 | TheJulia, aparnav: also commented. We should link to the DIB docs. | 14:46 |
| *** sdake_ is now known as sdake | 14:49 | |
| *** chihhsin has joined #openstack-ironic | 14:51 | |
| *** rob0t__ has quit IRC | 14:51 | |
| *** rob0t__ has joined #openstack-ironic | 14:53 | |
| *** aparnav has quit IRC | 14:53 | |
| lucasagomes | jroll, remind me please, is python-redfish part of the ironic umbrella ? | 14:56 |
| jroll | lucasagomes: not yet, no https://governance.openstack.org/tc/reference/projects/ironic.html | 14:56 |
| lucasagomes | jroll, right on :-/ | 14:56 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Find a node by multiple attributes https://review.openstack.org/421285 | 14:57 |
| jroll | lucasagomes: I'd like it to be, but nobody has really talked about it much, if we merge the redfish spec idk why we couldn't start that | 14:57 |
| *** rob0t__ has quit IRC | 14:57 | |
| lucasagomes | jroll, yeah, to be honest I'm trying to package it for RDO and urgh... the setup.py is pretty nasty (it has a execute(["pip", "install" ...] there!!!) | 14:58 |
| jroll | O_o | 14:58 |
| lucasagomes | jroll, so I will fix it and was wondering who is actually reviewing it | 14:58 |
| * TheJulia blinks | 14:58 | |
| lucasagomes | jroll, https://github.com/openstack/python-redfish/blob/master/setup.py#L29-L30 | 14:58 |
| lucasagomes | no joke | 14:58 |
| TheJulia | lucasagomes: bcornec2 most likely | 14:58 |
| lucasagomes | TheJulia, ++ | 14:58 |
| TheJulia | lucasagomes: I have an email someplace from bruno that copies some of the python-redfish contributors, so we could likely get ahold of others if needed | 14:59 |
| lucasagomes | TheJulia, thanks, yeah I will put a patch up first and then ask bruno | 15:00 |
| *** mjura has joined #openstack-ironic | 15:00 | |
| TheJulia | lucasagomes: awesome | 15:00 |
| lucasagomes | another thing is that, the tarball.openstack... for red fish doesn't have anything else than -master | 15:00 |
| lucasagomes | maybe the pypi job is broken ? | 15:00 |
| mgould | TheJulia, lucasagomes: René Ribaud, according to the commit history... | 15:00 |
| TheJulia | lucasagomes: it may not have even been branched yet, and as a library afaik it doesn't have to be | 15:01 |
| TheJulia | lucasagomes: I think it was imported this cycle from github | 15:02 |
| lucasagomes | I see | 15:02 |
| lucasagomes | that might explain it | 15:02 |
| lucasagomes | TheJulia, thanks for all info | 15:02 |
| jroll | well, any releases should still have tarballs | 15:02 |
| lucasagomes | I've meeting now but after that I will try to fix that stuff upstream | 15:02 |
| TheJulia | the last release was from when it was on github | 15:03 |
| TheJulia | 0.3 == 9 months ago | 15:03 |
| jroll | ah | 15:03 |
| *** rob0t__ has joined #openstack-ironic | 15:03 | |
| lucasagomes | :-/ | 15:03 |
| *** dtantsur|brb is now known as dtantsur|mtg | 15:03 | |
| TheJulia | I think it would behove us to be further involved with the redfish stuffs. I just don't have any redfish hardware to play with. :( | 15:03 |
| *** milan has joined #openstack-ironic | 15:03 | |
| milan | morning Ironic! :) | 15:04 |
| * TheJulia wonders if moonshots are cheap on ebay yet | 15:04 | |
| TheJulia | good morning milan | 15:04 |
| milan | morning TheJulia! :) | 15:04 |
| *** srobert_ has joined #openstack-ironic | 15:04 | |
| *** vsaienko has quit IRC | 15:05 | |
| *** baoli has quit IRC | 15:05 | |
| jroll | TheJulia: ++ | 15:05 |
| *** baoli has joined #openstack-ironic | 15:05 | |
| sambetts | jroll: QQ, when you make a request to the Ironic API and you get an error, is there a reason why we return a json in the format {"error_message": "<string encoded json dictionary about the error>"} and not {"error_message": <json dictionary>} ?? | 15:06 |
| jroll | sambetts: that is not a quick question :) | 15:06 |
| jroll | I don't know, off the top of my head | 15:06 |
| jroll | I would wonder if it's always a dict | 15:06 |
| TheJulia | I don't think it always is, at least from historical memory | 15:07 |
| *** rob0t__ has quit IRC | 15:08 | |
| sambetts | my first thought was "something something backward compatiblity", /me goes hunting for FIXME/NOTE comments | 15:08 |
| *** srobert has quit IRC | 15:08 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Release notes for recently added features https://review.openstack.org/429742 | 15:09 |
| TheJulia | $280 for a management module... and no chassis on ebay :( | 15:11 |
| TheJulia | ppiela: ^^^ new change set? | 15:11 |
| robcresswell | Its a bit of a pain having to decode every API return twice :p | 15:11 |
| * TheJulia ducks since she is sure ppiela will soon toss stuff at her :) | 15:11 | |
| *** rob0t__ has joined #openstack-ironic | 15:12 | |
| *** srobert_ has quit IRC | 15:12 | |
| *** srobert has joined #openstack-ironic | 15:12 | |
| *** baoli has quit IRC | 15:16 | |
| *** rob0t__ has quit IRC | 15:16 | |
| *** srobert has quit IRC | 15:17 | |
| *** gcb has joined #openstack-ironic | 15:17 | |
| *** srobert has joined #openstack-ironic | 15:17 | |
| vdrok | morning TheJulia and milan | 15:18 |
| milan | morning vdrok! :) | 15:18 |
| *** baoli has joined #openstack-ironic | 15:19 | |
| *** baoli has quit IRC | 15:19 | |
| *** sabeen has joined #openstack-ironic | 15:20 | |
| TheJulia | ppiela: Ahh, I see you abandoned the prior change. | 15:21 |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic master: Fix directories permission https://review.openstack.org/427077 | 15:21 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Update api-ref for dynamic drivers https://review.openstack.org/428460 | 15:23 |
| *** rob0t__ has joined #openstack-ironic | 15:23 | |
| mariojv | cc jroll ^ thanks for the review, think i fixed everything commented on | 15:24 |
| jroll | mariojv: cool, will look | 15:24 |
| *** fxpester has quit IRC | 15:25 | |
| *** srobert has quit IRC | 15:25 | |
| *** srobert has joined #openstack-ironic | 15:26 | |
| *** rob0t__ has quit IRC | 15:28 | |
| *** rloo has quit IRC | 15:29 | |
| *** rloo has joined #openstack-ironic | 15:29 | |
| vdrok | dtantsur|mtg: I've looked through the last 100 (by report) of open bugs on ironic, can I put the ones that caught my eye as worth fixing to the bugs section of whiteboard? | 15:35 |
| vdrok | * by reporting time | 15:36 |
| dtantsur|mtg | vdrok, please do, much appreciated! | 15:36 |
| * dtantsur|mtg does not have time for that, sorry | 15:36 | |
| vdrok | np :) | 15:36 |
| *** rob0t__ has joined #openstack-ironic | 15:37 | |
| *** jvgrant has quit IRC | 15:39 | |
| *** jvgrant has joined #openstack-ironic | 15:39 | |
| *** beekneemech is now known as bnemec | 15:40 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Retry to connect to RPC if failed https://review.openstack.org/376462 | 15:41 |
| *** rob0t__ has quit IRC | 15:41 | |
| *** milan has quit IRC | 15:42 | |
| *** jaosorior has joined #openstack-ironic | 15:43 | |
| NobodyCam | Good Morning Ironic'ers | 15:44 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Retry to connect to RPC if failed https://review.openstack.org/376462 | 15:45 |
| rloo | hi and good monday morning all, vdrok, dtantsur|mtg, jroll, mariojv, TheJulia, sambetts, lucasagomes, mgould, mat128 | 15:45 |
| lucasagomes | rloo, hi there | 15:45 |
| jroll | morning rloo | 15:45 |
| vdrok | morning NobodyCam and rloo | 15:45 |
| sambetts | jroll: So on quick investigation it turns out that wsme always formats our exceptions for us in to a JSON dict ready for returning to the client, and then we've also added middleware that trys to take the exception wraps it into JSON, which results in a double json encoded response | 15:45 |
| rloo | morning NobodyCam | 15:45 |
| mariojv | morning rloo | 15:46 |
| sambetts | o/ rloo | 15:46 |
| dtantsur|mtg | morning rloo, mariojv | 15:46 |
| rloo | sambetts: isn't double doubly better than single? (ugh) ^^ | 15:46 |
| NobodyCam | Morning dtantsur|mtg rloo vdrok sambetts lucasagomes :) | 15:46 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-ui master: Release notes for recently added features https://review.openstack.org/429742 | 15:47 |
| dtantsur|mtg | morning NobodyCam | 15:47 |
| TheJulia | ppiela: Minor edits ^^^^ | 15:47 |
| TheJulia | rloo: good morning rloo | 15:47 |
| rloo | vdrok: i think it could be possible, L139 https://review.openstack.org/#/c/428475/1/ironic/conductor/manager.py | 15:48 |
| patchbot | patch 428475 - ironic - exception from driver_factory.default_interface() | 15:48 |
| rloo | vdrok: but i think it might not be possible at L368 | 15:48 |
| sambetts | rloo: heh its not a huge issue, you just end up having to decode it twice in the client | 15:49 |
| sambetts | rloo: or in any UI code | 15:50 |
| rloo | vdrok: but i don't want to clean that part up. the only goal of that patch was to replace the check for None with raising an exception sooner. so since the code checked for none, it means the code thinks an exception could be raised. | 15:50 |
| lucasagomes | NobodyCam, o/ | 15:50 |
| rloo | sambetts: but that's icky. i'm guessing our client doesn't decode it twice. if we changed to decode twice, and then ironic changes to only singly encode... | 15:50 |
| rloo | sambetts: (but i haven't been paying attention to the discussion. just doesn't seem 'right' to encode twice...) | 15:51 |
| sambetts | rloo: yeah, we shouldn't be doing it IMO, I'm thinking it might have been left over code for processing exceptions from before we used wsme, which we didn't remove when we switched over | 15:52 |
| rloo | sambetts: that theory sounds good | 15:53 |
| NobodyCam | :) | 15:53 |
| sambetts | rloo: we're double decoding in the client as a work around already ... https://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/http.py#L76 | 15:55 |
| rloo | sambetts: we are?!!# | 15:55 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Adds network check in upgrade phase https://review.openstack.org/428043 | 15:56 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Retry to connect to RPC if failed https://review.openstack.org/376462 | 15:56 |
| vdrok | rloo: on L139 as we might be updating node setting its driver to a hardware type that is not enabled? we'd fail earlier, when loading that hw type. Also, I think that it was checking for None because we were not failing if None was returned by _register_and_validate_hardware_interfaces, now we raise instead, and conductor just stops? | 15:56 |
| robcresswell | The ironic standalone ui had a bunch of code to double decode too; thats why I was asking Sam what was going on :) | 15:56 |
| jroll | EmilienM: reading -release channel, this is the first I've heard of ironic blocking tripleo releases, what's up with that? | 15:56 |
| rloo | vdrok: we could also be updating a node's interface? | 15:57 |
| EmilienM | jroll: I don't think it's on ironic side | 15:57 |
| jroll | EmilienM: oh, perfect, thanks | 15:58 |
| mat128 | hi rloo | 15:58 |
| EmilienM | jroll: we are still investigating the bug | 15:58 |
| EmilienM | jroll: I'll let you know how it goes | 15:58 |
| jroll | EmilienM: cool, happy to help if needed | 15:58 |
| EmilienM | thanks for asking | 15:58 |
| jroll | np | 15:59 |
| vdrok | rloo: yeah, but if it's not set and we're setting it, we'd raise only IncompatibleInterface because interface is not supported if it's not supported, if we're removing that field, we already know that some interface is supported and enabled so can calculate default | 15:59 |
| rloo | vdrok: sigh. I didn't add that exception to L128. | 16:00 |
| rloo | vdrok: i'd rather that stuff get cleaned up in a separate patch, that wasn't the intent of my patch | 16:01 |
| rloo | vdrok: i can remove L139 I added, I don't really care. | 16:02 |
| rloo | vdrok: but i added that to be consistent etc. | 16:02 |
| vdrok | rloo: well, I understand that's not the intent, but with the changes to the _register_smth_interfaces_smth those places are now not able to raise, as conductor stopped already :) jroll dtantsur|mtg if you have time, could you also take a look at https://review.openstack.org/428475? I can be persuaded otherwise if people think it's worth leaving those :) | 16:04 |
| patchbot | patch 428475 - ironic - exception from driver_factory.default_interface() | 16:04 |
| rloo | vdrok: so i think a separate patch should clean that up | 16:04 |
| rybridges2 | Good morning guys! | 16:05 |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:05 | |
| rybridges2 | I was wondering what you guys think about the use of ipmitool vs ipminative? Is one preferred over the other? | 16:05 |
| rloo | jroll: vdrok and i were just discussing 428475. did you look at it? | 16:06 |
| *** dtantsur|mtg is now known as dtantsur | 16:07 | |
| jroll | rloo: am looking now | 16:07 |
| rloo | jroll: thx. i wanted to ask about people's take on i18n (or not) on substrings | 16:07 |
| jroll | rloo: am not following irc conversation well, though | 16:07 |
| rloo | jroll: should be clearer when you see vdrok's comments in that patch | 16:07 |
| jroll | rloo: idk enough about how translation works in openstack to have an opinion on that. however word order is different in many languages and I think that could break things? | 16:08 |
| *** spartacloud has joined #openstack-ironic | 16:08 | |
| jroll | I feel like it's usually assumed that the arguments to string.format whatever for i18n'd strings are not translated themselves | 16:08 |
| rloo | jroll: yup, i think so too. but it occurred to me that we don't have any general rule about that. i think we do i18n on substrings already. | 16:08 |
| jroll | we also probably shouldn't translate driver/hardware type | 16:09 |
| jroll | rloo: I'm not interested in thinking about rules for rare cases that don't affect actual runtime code right now, sorry | 16:09 |
| vdrok | translation thing was mostly a nit :) | 16:09 |
| rloo | jroll: ok | 16:09 |
| rloo | vdrok: right, but good to think about | 16:09 |
| * dtantsur reads the scrollback | 16:09 | |
| *** Syed__ has joined #openstack-ironic | 16:11 | |
| *** sacharya has joined #openstack-ironic | 16:12 | |
| openstackgerrit | Merged openstack/ironic-ui master: Add support for editing Ironic network ports https://review.openstack.org/429111 | 16:16 |
| dtantsur | jroll, "No valid host was found. Reason: No conductor service registered which supports driver ipmi" in my local devstack, huh | 16:18 |
| dtantsur | DriverNotFoundInEntrypoint: Could not find the following items in the 'ironic.drivers' entrypoint: ipmi. | 16:19 |
| dtantsur | I wonder if we did something wrong with enabling it... | 16:19 |
| jroll | dtantsur: did you pip install ironic recently? | 16:19 |
| jroll | it "just worked" for me | 16:19 |
| jroll | but if you do git pull and restart, since we added ipmi to setup.cfg, that isn't enough | 16:19 |
| *** nmathew has joined #openstack-ironic | 16:19 | |
| dtantsur | jroll, I assumed devstack does it... | 16:19 |
| *** sayantani01 has joined #openstack-ironic | 16:20 | |
| dtantsur | the same issue Oo | 16:20 |
| jroll | dtantsur: was it fresh devstack or re-stack? | 16:20 |
| dtantsur | fresh. the same after explicit pip install Oo | 16:20 |
| jroll | what the heck | 16:20 |
| dtantsur | jroll, and he's the answer: | 16:21 |
| dtantsur | it seems like we automatically add IRONIC_DEPLOY_DRIVER to the list of enabled drivers... | 16:21 |
| dtantsur | which is no longer correct :) | 16:21 |
| * dtantsur fixes | 16:21 | |
| jroll | oh, I didn't use IRONIC_DEPLOY_DRIVER, makes sense | 16:21 |
| rybridges2 | dtantsur: check your ironic.conf | 16:22 |
| dtantsur | rybridges2, see above :) | 16:22 |
| rybridges2 | and find enabled_drivers | 16:22 |
| dtantsur | but yeah, that's what I ended up doing | 16:22 |
| dtantsur | thanks | 16:22 |
| JayF | heh | 16:23 |
| JayF | patch to fix a devstack misconfig, of course, causes a misconfig a mere few weeks later | 16:23 |
| JayF | /o\ | 16:23 |
| rybridges2 | lol | 16:23 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [devstack] Allow using "ipmi" hardware type https://review.openstack.org/429694 | 16:24 |
| ricardoas | Hello, ironic'ers! | 16:24 |
| yolanda_ | hi, i've been working on the spec for custom deployment steps: https://review.openstack.org/412523. However, some of the questions are out of my knowledge, can other people provide more feedback, or contribute the spec as well? | 16:24 |
| patchbot | patch 412523 - ironic-specs - Ironic deployment steps | 16:24 |
| *** jcoufal_ has joined #openstack-ironic | 16:25 | |
| JayF | dtantsur: isn't adding a hardware_type to IRONIC_ENABLED_DRIVERS wrong? | 16:25 |
| dtantsur | JayF, it is | 16:25 |
| JayF | dtantsur: won't it still get populated into the wrong config variable? | 16:25 |
| dtantsur | well, we can't easily distinguish hw types and drivers by name only | 16:26 |
| dtantsur | not without inspecting setup.cfg | 16:26 |
| JayF | might be wise just to completely remove that failsafe, then | 16:26 |
| JayF | rather than have devstack automatically misconfigure people in some cases? | 16:26 |
| dtantsur | maybe? maybe we should fail loudly instead | 16:27 |
| *** jcoufal has quit IRC | 16:27 | |
| dtantsur | I'll test this revision locally first, then think about it (or feel free to put a -1 requesting it) | 16:27 |
| * dtantsur -> quick snack while devstack is building | 16:27 | |
| ricardoas | JayF, got a second? I have a question about your comments at https://review.openstack.org/#/c/422118/ and https://review.openstack.org/#/c/429009/ (previously https://review.openstack.org/#/c/422064/ ) | 16:28 |
| patchbot | patch 422118 - ironic (stable/newton) - Adds another validation step when using dynamic al... | 16:28 |
| patchbot | patch 429009 - ironic (stable/newton) - Ensures that OneView nodes are free for use by Ironic | 16:28 |
| patchbot | patch 422064 - ironic (stable/newton) - Ensures that OneView nodes are free for use by Ironic | 16:28 |
| ricardoas | :) | 16:28 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests https://review.openstack.org/423556 | 16:30 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add manual clean step ironic standalone test https://review.openstack.org/429770 | 16:30 |
| *** galyna1 has joined #openstack-ironic | 16:31 | |
| *** owalsh is now known as owalsh-afk | 16:33 | |
| *** moshele has quit IRC | 16:33 | |
| *** gcb has quit IRC | 16:33 | |
| ricardoas | JayF, by that I suppose we're not allowed to change minimal version at stable branches... so, in practice, we can't back port anything that requires a new minimal version of python-oneviewclient, right? | 16:34 |
| JayF | yes, exactly | 16:35 |
| *** baoli has joined #openstack-ironic | 16:35 | |
| *** nmathew has quit IRC | 16:35 | |
| jroll | rloo: reviewed that default_interface change | 16:38 |
| *** pcaruana has quit IRC | 16:38 | |
| rloo | jroll: thx! | 16:38 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Adds network check in upgrade phase https://review.openstack.org/428043 | 16:39 |
| jroll | rloo: np, thanks for writing that. still not convinced it's worth it but hey :) | 16:39 |
| *** crushil has joined #openstack-ironic | 16:39 | |
| rloo | jroll: it's code clean up. if we don't expect it to be None, then we should code for that. | 16:39 |
| jroll | rloo: somehow it's more code :) I think you're right about it being better, though | 16:40 |
| rloo | jroll: heh. it isn't the amount of code, it is the future understandability/maintainability of our code | 16:41 |
| jroll | ya | 16:43 |
| *** galyna_ has joined #openstack-ironic | 16:47 | |
| *** galyna2 has joined #openstack-ironic | 16:49 | |
| *** galyna1 has left #openstack-ironic | 16:50 | |
| *** galyna_ has quit IRC | 16:50 | |
| *** galyna2 has left #openstack-ironic | 16:51 | |
| *** galyna2 has joined #openstack-ironic | 16:51 | |
| *** aNuposic has joined #openstack-ironic | 16:54 | |
| ricardoas | JayF: ok... thanks!! | 16:54 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests https://review.openstack.org/423556 | 16:55 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add manual clean step ironic standalone test https://review.openstack.org/429770 | 16:55 |
| JayF | ricardoas: http://docs.openstack.org/project-team-guide/stable-branches.html appropriate fixes is the best header for stuff like this | 16:55 |
| *** baoli has quit IRC | 16:56 | |
| *** rpioso has joined #openstack-ironic | 16:56 | |
| jroll | who's ready for a meetin | 16:58 |
| jroll | g | 16:58 |
| dtantsur | \o/ | 16:59 |
| *** mjura has quit IRC | 16:59 | |
| *** [1]cdearborn has joined #openstack-ironic | 17:02 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [devstack] Allow using "ipmi" hardware type https://review.openstack.org/429694 | 17:03 |
| *** milan has joined #openstack-ironic | 17:08 | |
| *** [2]cdearborn has joined #openstack-ironic | 17:09 | |
| *** dims has quit IRC | 17:11 | |
| *** mgould is now known as mgould|afk | 17:15 | |
| *** dims has joined #openstack-ironic | 17:18 | |
| *** tesseract has quit IRC | 17:18 | |
| *** cdearborn has quit IRC | 17:18 | |
| *** [2]cdearborn is now known as cdearborn | 17:18 | |
| *** amarao has quit IRC | 17:19 | |
| *** amarao has joined #openstack-ironic | 17:20 | |
| *** harlowja has joined #openstack-ironic | 17:20 | |
| *** [1]cdearborn has quit IRC | 17:21 | |
| *** amarao has quit IRC | 17:27 | |
| *** spartacloud has quit IRC | 17:28 | |
| *** dsneddon_afk is now known as dsneddon | 17:28 | |
| *** mat128_ has joined #openstack-ironic | 17:32 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: devstack: Adding a README for ironic-bm-logs directory https://review.openstack.org/426965 | 17:32 |
| *** crushil has quit IRC | 17:41 | |
| *** crushil has joined #openstack-ironic | 17:42 | |
| *** baoli has joined #openstack-ironic | 17:43 | |
| *** alexpilotti has quit IRC | 17:47 | |
| *** harlowja has quit IRC | 17:49 | |
| *** derekh has quit IRC | 17:49 | |
| *** gmmaha has quit IRC | 17:52 | |
| *** rama_y has joined #openstack-ironic | 17:54 | |
| *** ralonsoh has quit IRC | 17:56 | |
| *** alexpilotti has joined #openstack-ironic | 17:56 | |
| jroll | dtantsur: how many times can someone run node-create with name 'ports' in an installation of ironic? | 17:56 |
| dtantsur | jroll, lemme tell you a sad story: the most popular troubleshooting approach to every bare metal problem I'm seeing is to delete all nodes and start from scratch :( | 17:57 |
| *** mgoddard_ has quit IRC | 17:58 | |
| jroll | ugh, okay, fair enough | 17:58 |
| jroll | I guess a version is the right thing to do | 17:58 |
| * dtantsur was fighting with it for long time - and lost | 17:59 | |
| *** alexpilotti has quit IRC | 18:00 | |
| JayF | lively meeting | 18:00 |
| dtantsur | yep | 18:00 |
| * jroll goes to do errands and things, bbl | 18:00 | |
| vsaienk0 | dtantsur: we need to be sured that ipmi driver works or it is stable enough, related to https://review.openstack.org/#/c/429709/ ? | 18:00 |
| patchbot | patch 429709 - openstack-infra/project-config - Add a non-voting job testing ironic with new "ipmi... | 18:00 |
| lucasagomes | calling it a day! Have a great eveninga ll | 18:01 |
| dtantsur | vsaienk0, just make sure that the job passes. then in e.g. 1 week we switch some of our jobs to it, or something like that | 18:01 |
| lucasagomes | evening all* | 18:01 |
| *** lucasagomes is now known as lucas-afk | 18:01 | |
| dtantsur | I haven't put much thought in it yet | 18:01 |
| dtantsur | see you lucas-afk | 18:01 |
| vdrok | night lucas-afk | 18:01 |
| lucas-afk | o/ | 18:01 |
| rloo | night lucas-afk | 18:01 |
| jlvillal | vdrok: I'd like to get your opinions on this patch: https://review.openstack.org/#/c/429280/ | 18:01 |
| patchbot | patch 429280 - networking-generic-switch - devstack: Use /etc/neutron/neutron.conf file | 18:01 |
| vsaienk0 | dtantsur: we can do temp change to devstack-gate that switch driver, and do some number of rechecks. New job seems not needed | 18:01 |
| jlvillal | vdrok: Since you are one of the experts on networking-generic-switch | 18:02 |
| jlvillal | vsaienk0: I replied to your comment on: https://review.openstack.org/#/c/429280/ | 18:02 |
| patchbot | patch 429280 - networking-generic-switch - devstack: Use /etc/neutron/neutron.conf file | 18:02 |
| rloo | vdrok: wrt those two bugs we just discussed. thx for bringing them up. did you want me to do the first one? (I'm not going to offer the 2nd one, heh) | 18:02 |
| vdrok | jlvillal: hah, the only real expert on that is vsaienk0 :) will do it tomorrow, not ime for it today unfortunately :( | 18:02 |
| dtantsur | vsaienk0, not sure what the problem is, your options require much more code. | 18:02 |
| rloo | vdrok: i'm good if you want to do the first one. | 18:02 |
| jlvillal | vdrok: No worries | 18:02 |
| Nisha_Agarwal | vsaienk0, what is this new ipmi driver? part of driver composition? | 18:03 |
| dtantsur | Nisha_Agarwal, yes | 18:03 |
| vdrok | rloo: I already forgot about the first one :D sure, if you want to grab it :) | 18:03 |
| Nisha_Agarwal | dtantsur, so the driver name need not have any interface? | 18:03 |
| vdrok | otherwise I can do it | 18:03 |
| rloo | vdrok: ha ha. ok :) i'll update the bug with what was discussed at the meeting. | 18:03 |
| vdrok | thanks! | 18:04 |
| dtantsur | Nisha_Agarwal, please check the latest version of the spec :) if you just pass the driver, the default interfaces are used. | 18:04 |
| Nisha_Agarwal | dtantsur, got it :) | 18:06 |
| Nisha_Agarwal | i think i read quite long back :) its much updated than that | 18:07 |
| dtantsur | yep, we did update it a few times | 18:09 |
| dtantsur | ok folks, calling it a day now. see you tomorrow! | 18:09 |
| *** dtantsur is now known as dtantsur|afk | 18:09 | |
| *** gmmaha has joined #openstack-ironic | 18:10 | |
| * sambetts -> home | 18:11 | |
| *** sambetts is now known as sambetts|afk | 18:11 | |
| sambetts|afk | night all | 18:11 |
| galyna2 | good night :) | 18:11 |
| *** galyna2 has left #openstack-ironic | 18:11 | |
| *** galyna2 has joined #openstack-ironic | 18:13 | |
| *** moshele has joined #openstack-ironic | 18:16 | |
| *** Nisha_Agarwal has quit IRC | 18:18 | |
| vdrok | night sambetts|afk dtantsur|afk and galyna | 18:19 |
| *** gmmaha has quit IRC | 18:19 | |
| *** gmmaha has joined #openstack-ironic | 18:20 | |
| *** galyna2 has quit IRC | 18:21 | |
| *** crushil has quit IRC | 18:22 | |
| *** gmmaha has quit IRC | 18:22 | |
| *** gmmaha has joined #openstack-ironic | 18:22 | |
| *** ijw has joined #openstack-ironic | 18:23 | |
| *** moshele has quit IRC | 18:23 | |
| *** aNuposic has quit IRC | 18:27 | |
| *** ijw has quit IRC | 18:32 | |
| *** vsaienko has joined #openstack-ironic | 18:36 | |
| *** pcaruana has joined #openstack-ironic | 18:36 | |
| JayF | A super interesting networking bug, found in newton: https://bugs.launchpad.net/ironic/+bug/1657083 | 18:40 |
| openstack | Launchpad bug 1657083 in Ironic "instance rebuild cause creation of two ports in two networks simultaneously" [Undecided,New] | 18:40 |
| JayF | maybe worth digging into and fixing | 18:40 |
| *** chedda has joined #openstack-ironic | 18:40 | |
| *** moshele has joined #openstack-ironic | 18:42 | |
| mariojv | vsaienk0: thanks for the review on the api ref patch, i didn't know those samples were automatically generaetd | 18:42 |
| mariojv | *generated | 18:42 |
| *** mgoddard_ has joined #openstack-ironic | 18:42 | |
| *** harlowja has joined #openstack-ironic | 18:44 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add port groups configuration documentation https://review.openstack.org/429826 | 18:44 |
| *** crushil has joined #openstack-ironic | 18:45 | |
| mariojv | seems to be broken though, at least on ubuntu 14.04, jq -S doesn't recognize the -S flag | 18:46 |
| mariojv | switched to -s and trying again | 18:48 |
| JayF | sambetts|afk: put a comment on https://bugs.launchpad.net/ironic/+bug/1659836 -- not sure I understand the case you're describing | 18:48 |
| openstack | Launchpad bug 1659836 in Ironic "Ironic Nova Virt driver tries to act on exclusively locked node during tear down" [Undecided,New] | 18:48 |
| *** amarao has joined #openstack-ironic | 18:50 | |
| JayF | jlvillal: https://review.openstack.org/#/c/398992 your comments were addressed here, if you wanna apply your +2A :) | 18:52 |
| patchbot | patch 398992 - ironic-python-agent - Configure and use SSL-related requests options | 18:52 |
| *** chlong has quit IRC | 18:52 | |
| *** ChubYann has joined #openstack-ironic | 18:53 | |
| rloo | hey, someone just asked me how to get started with openstack & ironic. what's the best way to get started? is there openstack docs that someone would recommend? | 18:55 |
| *** aNuposic has joined #openstack-ironic | 18:55 | |
| mat128 | rloo: get started to do what precisely? | 18:55 |
| mat128 | rloo: contributing? using? | 18:55 |
| JayF | started as what, an end user of a cloud? a deployer? a developer? | 18:55 |
| *** sambetts|afk has quit IRC | 18:55 | |
| *** sambetts_ has joined #openstack-ironic | 18:56 | |
| rloo | i think as a developer. but not much programming experience, and they don't know openstack. they have experience racking hw. | 18:56 |
| rloo | seems like a BIG ramp up | 18:56 |
| mat128 | rloo: tell them to join the irc channel | 18:56 |
| JayF | That's sorta a tough spot to start from | 18:56 |
| mat128 | thats the first thing | 18:56 |
| rloo | mat128: i did :) | 18:56 |
| rloo | mat128: some folks are shy | 18:56 |
| mat128 | next up, fix a typo and submit it | 18:56 |
| mat128 | get used to the review process | 18:56 |
| rybridges2 | rloo: do they know python at all? | 18:57 |
| JayF | Shy or not, failing to join the channel isn't conductive to becoming part of the community :) | 18:57 |
| rybridges2 | if they are inexperienced with python still, i would start with just python skills | 18:57 |
| rybridges2 | when i first started openstack, i had never written python before | 18:57 |
| mat128 | rybridges2: to me, there is plenty of stuff non-python in openstack | 18:57 |
| mat128 | docs, reviewing code | 18:57 |
| rybridges2 | i found that reading a textbook was very helpful | 18:57 |
| mat128 | s/code/specs | 18:57 |
| mat128 | Python Koans is a good one to get started on python | 18:58 |
| JayF | When I started on Ironic, I was very inexperienced at python, but knew a lot about the systems side | 18:58 |
| rloo | rybridges2: what textbook? | 18:58 |
| rybridges2 | i would highly recommend https://www.amazon.com/Python-Crash-Course-Hands-Project-Based/dp/1593276036 | 18:58 |
| JayF | for someone without the systems or python chops, it'd be a really hard trek :( | 18:58 |
| * JayF used codecademy to bone up on python basics | 18:58 | |
| rloo | rybridges2: oh, you mean python text book. i thought there might be a good openstack book :) | 18:58 |
| rybridges2 | i used that and ramped up really fast in like a week | 18:58 |
| rybridges2 | oh heh i wish there was an openstack textbook | 18:58 |
| mat128 | rloo: https://github.com/gregmalcolm/python_koans | 18:58 |
| *** dsneddon is now known as dsneddon_afk | 18:59 | |
| mat128 | rloo: is the person an openstack user? | 18:59 |
| rloo | mat128: nope. not yet. | 18:59 |
| rloo | mat128: big learning curve... :) | 18:59 |
| mat128 | rloo: what are their previous experience besides racking hardware? | 18:59 |
| rloo | mat128: dunno, they didn't tell me. | 18:59 |
| *** crushil has quit IRC | 19:00 | |
| mat128 | what is* whatever :P | 19:00 |
| mat128 | oh | 19:00 |
| *** dsneddon has joined #openstack-ironic | 19:00 | |
| rloo | mat128: i'll ask for a cv next time :) | 19:00 |
| mat128 | haha | 19:00 |
| rloo | ok, i'll point them to this. but i think they should probably *use* openstack first... the install/admin guides are probably the best place to start? | 19:00 |
| JayF | ++ great suggestion | 19:00 |
| JayF | try to install openstack; fix bugs in docs | 19:01 |
| *** sayantani01 has quit IRC | 19:01 | |
| JayF | many projects even try to get someone to run through the full install guide each release | 19:01 |
| rloo | JayF: oh, maybe we should too. who do we want to torture? | 19:01 |
| mat128 | rloo: http://tickletux.wordpress.com/2007/01/24/using-fizzbuzz-to-find-developers-who-grok-coding/ comes to mind | 19:01 |
| JayF | rloo: /me touches nose | 19:01 |
| *** vsaienk0 has quit IRC | 19:01 | |
| mat128 | rloo, JayF: ours is huge :( | 19:02 |
| JayF | rloo: I already have a post-release project: research and well-document creating deploy images for ironic | 19:02 |
| mat128 | all OSes and combos | 19:02 |
| JayF | it's specifically a pain point I've heard from most ironic deployers | 19:02 |
| JayF | and I know DIB can do it, we just don't always document it well. | 19:02 |
| mat128 | wget tarballs.o.o/ipa/coreos/{kernel,ramdisk} | 19:02 |
| mat128 | ;) | 19:02 |
| JayF | mat128: I mean /instance/ images | 19:02 |
| mat128 | oh, yeah | 19:02 |
| JayF | I should've said instance images | 19:02 |
| JayF | that's what I meant, not deploy images | 19:03 |
| JayF | images to be imaged onto the disk by the other images | 19:03 |
| JayF | /o\ | 19:03 |
| *** mgoddard_ has quit IRC | 19:03 | |
| mat128 | we use Oz | 19:03 |
| mat128 | yeah | 19:03 |
| rloo | thx for volunteering JayF! | 19:03 |
| rloo | fizzbuzz... | 19:04 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Update api-ref for dynamic drivers https://review.openstack.org/428460 | 19:04 |
| rloo | thx all, you provided some good pointers! | 19:04 |
| *** ijw has joined #openstack-ironic | 19:05 | |
| *** vsaienk0 has joined #openstack-ironic | 19:08 | |
| *** chlong has joined #openstack-ironic | 19:09 | |
| *** pcaruana has quit IRC | 19:11 | |
| *** crushil has joined #openstack-ironic | 19:12 | |
| rybridges2 | Yea one of the very first things that I did with openstack (after learning python a bit) was do a full setup of my own instance with all of the components | 19:12 |
| rybridges2 | and that familiarizes you with the APIs and what each piece does | 19:12 |
| rybridges2 | and you dont have to write any code | 19:13 |
| openstackgerrit | Michael Turek proposed openstack/ironic master: Add OpenBMC class for accessing nodes using OpenBMC https://review.openstack.org/429836 | 19:13 |
| *** ppiela has quit IRC | 19:14 | |
| *** gmmaha has quit IRC | 19:16 | |
| *** vsaienko has quit IRC | 19:19 | |
| openstackgerrit | Xavier proposed openstack/ironic master: Validation before perform node deallocation https://review.openstack.org/386014 | 19:19 |
| *** vsaienko has joined #openstack-ironic | 19:19 | |
| rloo | jroll: i thought soft power off etc was done, but i see two patches still? https://review.openstack.org/#/q/topic:bug/1526226 | 19:20 |
| rloo | dtantsur|afk: ^^ | 19:20 |
| jroll | rloo: that's fun | 19:20 |
| rloo | jroll: yup, we love having fun | 19:21 |
| jroll | rloo: well, from a nova user perspective it's done, at least | 19:21 |
| jroll | one is discoverability api, one is docs | 19:21 |
| rloo | jroll: well, you've got a few hours yet as ptl, so i'll leave for you to decide whether to prioritize, herd cats, etc :) | 19:21 |
| * jroll wonders if the former was in the spec | 19:22 | |
| jroll | rloo: I'm not adding an API version this late in the cycle :P | 19:22 |
| *** vsaienko has quit IRC | 19:22 | |
| rloo | jroll: ok. i just wanted to delete that feature from subteam report, but i'll leave it in with a note about it not being finished. | 19:23 |
| jroll | rloo: ah, I see | 19:23 |
| *** kjw3 has joined #openstack-ironic | 19:23 | |
| jroll | and yeah it's the second bullet here that isn't done http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/enhance-power-interface-for-soft-reboot-and-nmi.html#rest-api-impact | 19:23 |
| rybridges2 | speaking of these power interfaces... | 19:24 |
| rloo | jroll: ok, i'll add that link to our subteam report thing. that reminds me, i still haven't reviewed the driver comp spec to see where we're at. i'm assuming you or dtantsur|afk did. | 19:24 |
| rybridges2 | In my experience, ipmi_native has been a bt unreliable when issuing a reboot or a get_power() command | 19:24 |
| rloo | rybridges2: you're going to say how wonderful they are and how happy you are... | 19:25 |
| rybridges2 | it will just fail the first time with a timeout | 19:25 |
| rybridges2 | hahaha! rloo | 19:25 |
| rybridges2 | well | 19:25 |
| JayF | rybridges2: I'd highly reccomend !native for most production uses today | 19:25 |
| rybridges2 | tbh i am quite happy with them | 19:25 |
| jroll | rloo: I'm fairly confident that I have the remaining work to be done written down | 19:25 |
| rybridges2 | i am just wondering if what i am seeing is normal | 19:25 |
| rloo | rybridges2: that's what i wanted to hear. and now for the 'but ...' :) | 19:25 |
| rybridges2 | JayF: Thanks that is very helpful | 19:25 |
| rloo | jroll: good. | 19:25 |
| *** moshele has quit IRC | 19:26 | |
| rybridges2 | JayF: Just curious, why do you think non-native implementations are the way to go for prod uses right now? | 19:26 |
| rybridges2 | any specific downsides to native? or is it just not as mature yet? | 19:27 |
| JayF | rybridges2: sorta a chicken->egg problem. more people use the ipmitool-based driver so it's more vetted, bugs resolved, etc | 19:27 |
| rybridges2 | ahh that makes a lot of sense | 19:27 |
| JayF | also, ipmi is a really borderline insane protocol | 19:27 |
| rybridges2 | yes i agree | 19:28 |
| JayF | and ipmitool has longer experience of dealing with it than our python ipmi library ;) | 19:28 |
| rybridges2 | the worst part about it is... | 19:28 |
| rybridges2 | half the time the firmware from the vendor fails b/c it is super outdated or buggy or w/e | 19:28 |
| JayF | udp? the security problems? varied implementations? | 19:28 |
| rybridges2 | at that point there is nothing we can do | 19:28 |
| rybridges2 | no matter how good our implementation is on openstack | 19:28 |
| JayF | aha, I'll file that one under 'varied implementations' | 19:28 |
| rybridges2 | if the firmware crashes we are screwed no matter what heh | 19:28 |
| JayF | rybridges2: I had a set of machines once, that the ipmi firmware would freeze up, you'd have to pass "power off" over and over until it finally worked and "unfroze" the bmc | 19:29 |
| rybridges2 | yup | 19:29 |
| JayF | rybridges2: and this is literally the process the hardware vendor folks told us to do | 19:29 |
| rybridges2 | bahaha | 19:29 |
| rybridges2 | yea we have many similar issues | 19:29 |
| rybridges2 | we have thousands of machines running ironic right now | 19:29 |
| *** vsaienko has joined #openstack-ironic | 19:29 | |
| rybridges2 | we get failure rates on provides and boots about 12-15% of the time | 19:30 |
| rybridges2 | almost all of them related to ipmi hardware/firmware failures | 19:30 |
| *** amarao has quit IRC | 19:30 | |
| JayF | rybridges2: on native driver? | 19:30 |
| rybridges2 | no not native | 19:30 |
| rybridges2 | we are using ipmitool right now | 19:30 |
| JayF | that's some pretty incredibly flakey hardware | 19:31 |
| JayF | or massively underscaled conductors | 19:31 |
| *** owalsh-afk has quit IRC | 19:31 | |
| *** spartacloud has joined #openstack-ironic | 19:31 | |
| *** owalsh-afk has joined #openstack-ironic | 19:33 | |
| *** moshele has joined #openstack-ironic | 19:34 | |
| *** dyasny has quit IRC | 19:35 | |
| rybridges2 | JayF: We actually run 20 conductor processes per node | 19:36 |
| rybridges2 | err i should say | 19:36 |
| rybridges2 | per ic | 19:36 |
| rybridges2 | before we did that | 19:36 |
| rybridges2 | our failure rates were like 60% | 19:36 |
| JayF | wait, what? You're running multiple conductor instances on the same node? | 19:37 |
| rybridges2 | we know that it is the hardware in most cases though because when we try to issue raw ipmi commands to them they just dont respond | 19:37 |
| rybridges2 | yes :) | 19:37 |
| JayF | That has to be patched in, right? | 19:37 |
| rybridges2 | we patched it ourselves, yes | 19:37 |
| JayF | either that or you're getting some quantity of failures from conductors stepping on each others' tftp directories | 19:37 |
| JayF | but they have to share a tftp directory, b/c only one service per conductor | 19:38 |
| JayF | yikes, yeah, I don't reccomend that setup at all | 19:38 |
| rybridges2 | yes, we figured you guys would be opposed to it, which is why we did not try to push the changes upstream | 19:38 |
| rybridges2 | but at our scale (50k + nodes) there was no choice | 19:38 |
| rybridges2 | we needed more conductors | 19:38 |
| JayF | Yeah; in our environemnt we virtualized the conductors | 19:39 |
| rybridges2 | Oh that is interesting | 19:39 |
| JayF | ran the entire ironic control plane on vms, for maximum ironic output | 19:39 |
| rybridges2 | cool! | 19:39 |
| JayF | that I think would be the "official" solution to your problem: containers or vms | 19:39 |
| rybridges2 | yes | 19:39 |
| JayF | no rule that your control plan has to be bare nodes :) | 19:39 |
| rybridges2 | that is our goal eventually | 19:39 |
| aNuposic | Hi Folks, do we mention the openstack project names in lower case while referring to them commit messages or release notes? | 19:39 |
| rybridges2 | so, you run all of the conductors on a single hypervisor? | 19:39 |
| JayF | I also think we wouldn't be opposed, in theory, to a "support multiple conductors on a single piece of hardware" spec, but it's hard technically | 19:40 |
| JayF | rybridges2: we have an internal cloud at rackspace, and we built using that | 19:40 |
| JayF | rybridges2: all rackspace cloud control plane, with very little exception, is on vms | 19:40 |
| *** vsaienko has quit IRC | 19:40 | |
| TheJulia | aNuposic: I generally do unless it makes sense to capitalize in particular cases like the first word in a sentence. The doc guidelines are useful in that regard. | 19:40 |
| JayF | aNuposic: http://docs.openstack.org/contributor-guide/writing-style/openstack-components.html | 19:40 |
| *** vsaienko has joined #openstack-ironic | 19:40 | |
| JayF | TheJulia: jinx :P | 19:40 |
| TheJulia | lol | 19:40 |
| *** vsaienko has quit IRC | 19:41 | |
| rybridges2 | JayF: Ah ok that makes sense. | 19:41 |
| aNuposic | TheJulia: JayF: cool | 19:41 |
| aNuposic | yeah i have a suggestion on my patch set to use lower case in commit message so just wanted to confirm :) | 19:42 |
| rybridges2 | JayF: So, the reason that we are trying to switch to ipminative over ipmitool is because we noticed that the performance of ipmitool is pretty slow. The sync_power_state was eating up major resources. We were thinking if we switched to native it might speed things up | 19:42 |
| rybridges2 | JayF: But in my initial testing, native seems pretty finicky | 19:42 |
| mat128 | rybridges2: ipmitool is heavy because of all the process opening, file descriptors, text parsing going on | 19:43 |
| rybridges2 | yes exactly | 19:43 |
| *** dyasny has joined #openstack-ironic | 19:43 | |
| rybridges2 | every time it issues a command it is spawning processes | 19:43 |
| rybridges2 | when you have 15k baremetal nodes on a cluster and you have to spawn a new process for each one to get the power status, it slows things down a lot | 19:43 |
| mat128 | rybridges2: for the record, I'm a core reviewer on pyghmi (our ipmi library) and jjohnson2 is the lead for it | 19:43 |
| mat128 | rybridges2: I feel you :) | 19:44 |
| rybridges2 | mat128: Oh that is awesome! glad to meet you. | 19:44 |
| rybridges2 | i was taking a look at that pyghmi library quite a bit over the weekend actually | 19:44 |
| mat128 | we are running ipmitool too because of the issues with native, but we have a general thread to make native better | 19:44 |
| mat128 | im mostly core because it is used in virtualBMC (as a server rather than a client) and we can make changes if something breaks Ironic | 19:44 |
| mat128 | havent committed much (if at all) | 19:45 |
| mat128 | it's mostly Jarrod's project, but I see no reason valid commits arent considered | 19:45 |
| rybridges2 | I see. very interesting | 19:45 |
| mat128 | fun fact: Jarrod is optimizing for large scale, outside of openstack | 19:46 |
| rybridges2 | when i was doing my initial testing against the pyghmi library... | 19:46 |
| mat128 | we internally had problems with the global variables and state sharing going on with pyghmi being used as a library | 19:46 |
| mat128 | inside of ironic | 19:46 |
| rybridges2 | i noticed that i basically had to wrap my calls to set_power('boot') and get_power() with retries because they timeout on me 100% of the time the first time they run. but then instantly succeed the second time | 19:46 |
| rybridges2 | ah okay | 19:46 |
| mat128 | ugh | 19:46 |
| JayF | rybridges2: I wonder if you're running a too-old ipmitool | 19:47 |
| JayF | rybridges2: newer ipmitool has built-in retry support, and we utilize that | 19:47 |
| mat128 | JayF: I think he was mentuioning pygnmi | 19:47 |
| mat128 | oh god | 19:47 |
| mat128 | mentioning pyghmi* | 19:47 |
| JayF | ooooh | 19:47 |
| rybridges2 | JayF: Yes I was talking about pyghmi with native | 19:47 |
| rybridges2 | ipmitool seems to be working fine, albiet a bit sluggish :P | 19:47 |
| rybridges2 | when that pyghmi library works, it is really fast though which is great | 19:48 |
| mat128 | just looking at ipmitool and noticed it is using libfreeipmi, I wonder if anyone wrote a python wrapper to that lib | 19:49 |
| rybridges2 | hmm | 19:49 |
| rybridges2 | that would be interesting | 19:49 |
| *** moshele has quit IRC | 19:50 | |
| JayF | rybridges2: are you able to say what company you work for? | 19:50 |
| mat128 | cant find any, but that would benefit from the work done in ipmitool without having to open file descriptors and parse text | 19:50 |
| rybridges2 | JayF: I work for Yahoo | 19:50 |
| *** ElCoyote_ has quit IRC | 19:50 | |
| JayF | rybridges2: aha, that makes a lot of sense | 19:50 |
| rybridges2 | mat128: yes that sounds very attractive. ultimately the file descriptors and the processes are the bottleneck that we are trying to eliminate | 19:51 |
| rybridges2 | we actually ended up just completely disabling power sync | 19:51 |
| rybridges2 | because it was taking up so much resources at our scale | 19:51 |
| mat128 | rybridges2: with no side effect? | 19:52 |
| mat128 | for the record, we are using power sync in a "trust hardware rather than database" mode | 19:52 |
| rybridges2 | i mean, obviously the side effect is that nodes that have actual hardware problems do not get put into maintenance mode for us automatically | 19:52 |
| mat128 | oh | 19:52 |
| rybridges2 | but | 19:52 |
| mat128 | yeah.. | 19:52 |
| rybridges2 | other than that | 19:52 |
| rybridges2 | it really helped the performance by an astounding margin | 19:52 |
| rybridges2 | now we want to re-enable that power sync | 19:52 |
| JayF | I'm not surprised. | 19:53 |
| rybridges2 | but do it in such a way where it does not cripple performance | 19:53 |
| rybridges2 | so we are investigating ipminative, but it does not seem as though that is a silver bullet solution yet | 19:53 |
| JayF | I feel like a big part of your answer is | 19:53 |
| JayF | figure out how to scale out conductors more dramatically | 19:53 |
| JayF | using containers, or vms, or whatever | 19:53 |
| JayF | because then it's just a question of spinning up as many conductors as it takes | 19:54 |
| mat128 | rybridges2: you're already scaling out the conductors, no? | 19:54 |
| JayF | then you can use metrics to determine bottlenecks, and optimize from there | 19:54 |
| JayF | rybridges2: you do know about metrics, right? | 19:54 |
| rybridges2 | mat128: Yes we are. we run 20 conductor processes on each of our ic nodes. we have 3 ic nodes per cluster | 19:54 |
| rybridges2 | before we made this change | 19:54 |
| rybridges2 | we had these big beefy ic nodes | 19:54 |
| rybridges2 | with 24 processor cores | 19:54 |
| mat128 | 20 conductors on the same machine? | 19:54 |
| rybridges2 | and only 1 conductor process running on them | 19:55 |
| mat128 | and it was pegging a single cpu, right? | 19:55 |
| mat128 | maybe the answer lies in allowing more power sync "actions" to run at the same time | 19:55 |
| rybridges2 | we have 1 12 core cpu per machine | 19:55 |
| mat128 | rather than having a single one | 19:55 |
| rybridges2 | with hyper threading it is 24 virtual cores | 19:55 |
| rybridges2 | that is a good suggestion mat128! | 19:56 |
| rybridges2 | the only difficulty i see with that is race conditions | 19:56 |
| mat128 | running multiple conductors reaches almost the same effect though | 19:56 |
| rybridges2 | JayF: we do use performance metrics. before we started running 20 conductor processes on each machine, our cpu usage was like 2-4%, but the usage of that 1 core was constantly at 100% | 19:57 |
| mat128 | rybridges2: he meant metrics support built right into ironic | 19:58 |
| rybridges2 | oh. no i was not aware of those | 19:58 |
| *** strigazi is now known as strigazi_AFK | 19:58 | |
| rybridges2 | we have an external monitoring system that takes care of that stuff | 19:58 |
| mat128 | http://docs.openstack.org/developer/ironic/deploy/metrics.html | 19:59 |
| *** ElCoyote_ has joined #openstack-ironic | 19:59 | |
| TheJulia | If anyone wants to do a quick review https://review.openstack.org/#/c/429742/ | 20:01 |
| patchbot | patch 429742 - ironic-ui - Release notes for recently added features | 20:01 |
| *** vsaienko has joined #openstack-ironic | 20:02 | |
| rybridges2 | JayF: Ultimately we want to get to containers. Problem is, with our current setup, that is still months off. Our dev team is pretty small as well. We are looking for something that we can do temporarily to buy us time while we work on containerizing | 20:03 |
| JayF | good luck :) you certainly don't have an easy problem to solve | 20:04 |
| JayF | I have to head out for the day though, have a good one folks | 20:04 |
| mat128 | cya | 20:04 |
| TheJulia | goodnight JayF | 20:04 |
| rybridges2 | take care JayF. Thanks a lot for the chat | 20:05 |
| openstackgerrit | Merged openstack/ironic-python-agent master: [TinyIPA] Avoid running rm with no parameters https://review.openstack.org/428099 | 20:05 |
| mat128 | ^woot | 20:05 |
| JayF | mat128: wanna backport it real quick and I'll vote on it before I walk away? | 20:06 |
| rybridges2 | Oh, mat128: are those metrics only for Newton? | 20:06 |
| rybridges2 | we arent at Newton yet :p | 20:06 |
| *** nicodemos has quit IRC | 20:06 | |
| mat128 | JayF: doing it | 20:06 |
| mat128 | conflicts | 20:06 |
| *** nicodemos has joined #openstack-ironic | 20:07 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent stable/newton: [TinyIPA] Avoid running rm with no parameters https://review.openstack.org/429845 | 20:09 |
| mat128 | did it by hand, without forgetting the cherry-picked-from line this time :) | 20:09 |
| mat128 | JayF: ^ if you're still there | 20:09 |
| JayF | mat128: +2 | 20:10 |
| * JayF actually gone | 20:10 | |
| *** mtanino has joined #openstack-ironic | 20:11 | |
| openstackgerrit | Merged openstack/ironic-ui master: Release notes for recently added features https://review.openstack.org/429742 | 20:12 |
| *** vsaienko has quit IRC | 20:15 | |
| *** milan has quit IRC | 20:15 | |
| *** mat128_ has quit IRC | 20:16 | |
| TheJulia | jroll: I updated https://review.openstack.org/#/c/428814/ with ^^^ merge commit | 20:17 |
| patchbot | patch 428814 - releases - ironic-ui 2.2.0 and stable/ocata | 20:17 |
| openstackgerrit | Hugo Nicodemos proposed openstack/python-oneviewclient master: Add parameter to get Server Hardware MAC address from iLO https://review.openstack.org/429849 | 20:25 |
| *** vsaienko has joined #openstack-ironic | 20:26 | |
| *** vsaienko has quit IRC | 20:27 | |
| *** vsaienko has joined #openstack-ironic | 20:28 | |
| *** chedda has quit IRC | 20:30 | |
| *** chedda has joined #openstack-ironic | 20:30 | |
| *** owalsh-afk is now known as owalsh | 20:31 | |
| *** srart has quit IRC | 20:33 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: exception from driver_factory.default_interface() https://review.openstack.org/428475 | 20:49 |
| *** chlong has quit IRC | 20:49 | |
| *** rajinir has joined #openstack-ironic | 21:00 | |
| jroll | TheJulia: oh cool, so that's good to go any time? | 21:02 |
| TheJulia | I believe so yes, looks like the tox job errored | 21:02 |
| TheJulia | looks like it is already expecting the branch to be there O_o | 21:03 |
| mrda | Morning Ironic | 21:03 |
| mariojv | hi mrda | 21:03 |
| *** ElCoyote_ has quit IRC | 21:03 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Update api-ref for dynamic drivers https://review.openstack.org/428460 | 21:03 |
| mrda | hey mariojv | 21:04 |
| TheJulia | good morning mrda | 21:04 |
| mrda | and TheJulia :) | 21:04 |
| jroll | TheJulia: ooo fun | 21:05 |
| jroll | I'll bug release team | 21:05 |
| TheJulia | k, all I did was update the commit hash.... :( | 21:06 |
| rloo | mariojv: is that ^^ ready to be reviewed? | 21:07 |
| rloo | morning mrda | 21:07 |
| *** rama_y has quit IRC | 21:07 | |
| mariojv | rloo: yes, thx | 21:07 |
| mariojv | the main issue was that i wasn't using the script to auto-generate samples | 21:07 |
| jroll | TheJulia: yeah, it was them, I just rebased on top of the fix | 21:07 |
| TheJulia | ok | 21:07 |
| rloo | mariojv: ok, on my list, today or tomorrow. | 21:08 |
| mariojv | \o/ | 21:09 |
| mrda | rloo: o/ | 21:09 |
| mat128 | one more review for a backport? https://review.openstack.org/#/c/429845/ | 21:12 |
| patchbot | patch 429845 - ironic-python-agent (stable/newton) - [TinyIPA] Avoid running rm with no parameters | 21:12 |
| openstackgerrit | Merged openstack/bifrost master: Remove deprecated nginx_port option https://review.openstack.org/429021 | 21:13 |
| openstackgerrit | Merged openstack/bifrost master: Fixing typo in deprecation note https://review.openstack.org/429022 | 21:16 |
| *** hoangcx_ has joined #openstack-ironic | 21:16 | |
| openstackgerrit | Merged openstack/bifrost master: Remove deprecated test playbooks https://review.openstack.org/429023 | 21:16 |
| *** hoangcx_ is now known as hoangcx | 21:18 | |
| *** srobert has quit IRC | 21:18 | |
| *** ElCoyote_ has joined #openstack-ironic | 21:20 | |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic master: WIP/DNM: Testing grenade multi-tenant experimental job for Tempest failures https://review.openstack.org/403895 | 21:21 |
| mariojv | yuriyz: i reviewed https://review.openstack.org/#/c/427281 - it looks good, just needs to mention that portgroup_uuid is added to port payload in release note | 21:22 |
| patchbot | patch 427281 - ironic - Add ironic portgroup CRUD notifications | 21:22 |
| *** ppiela has joined #openstack-ironic | 21:37 | |
| *** crushil has quit IRC | 21:43 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add should_bind_property to NetworkInterface https://review.openstack.org/429893 | 21:45 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add should_bind_property to NetworkInterface https://review.openstack.org/429893 | 21:48 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add should_bind_port property to NetworkInterface https://review.openstack.org/429893 | 21:50 |
| openstackgerrit | Doug Hellmann proposed openstack/ironic-ui master: standardize release notes filenames and fix contents https://review.openstack.org/429896 | 21:54 |
| *** dprince has quit IRC | 21:55 | |
| *** g3ek has quit IRC | 22:02 | |
| *** hoangcx has quit IRC | 22:03 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-ui master: standardize release notes filenames and fix contents https://review.openstack.org/429896 | 22:06 |
| *** g3ek has joined #openstack-ironic | 22:12 | |
| *** alexpilotti has joined #openstack-ironic | 22:19 | |
| *** alexpilo_ has joined #openstack-ironic | 22:21 | |
| *** alexpilotti has quit IRC | 22:25 | |
| *** ijw has quit IRC | 22:33 | |
| *** mjturek has quit IRC | 22:35 | |
| *** baoli has quit IRC | 22:36 | |
| *** strigazi_AFK has quit IRC | 22:37 | |
| *** strigazi_AFK has joined #openstack-ironic | 22:38 | |
| *** jheroux has quit IRC | 22:38 | |
| *** jrcloud has joined #openstack-ironic | 22:42 | |
| openstackgerrit | Anup Navare proposed openstack/ironic master: Config drive support for Ceph https://review.openstack.org/407639 | 22:45 |
| *** jrcloud has quit IRC | 22:46 | |
| *** sacharya_ has joined #openstack-ironic | 22:51 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Source code documentation updates https://review.openstack.org/429916 | 22:53 |
| *** sacharya_ has quit IRC | 22:54 | |
| *** sacharya has quit IRC | 22:54 | |
| *** sacharya has joined #openstack-ironic | 22:54 | |
| *** sacharya has quit IRC | 23:00 | |
| *** sacharya has joined #openstack-ironic | 23:02 | |
| *** jaosorior has quit IRC | 23:04 | |
| *** sacharya has quit IRC | 23:06 | |
| *** jkilpatr has quit IRC | 23:16 | |
| *** crushil has joined #openstack-ironic | 23:18 | |
| *** awiddersheim has quit IRC | 23:21 | |
| *** awiddersheim has joined #openstack-ironic | 23:21 | |
| *** fyxim_ has joined #openstack-ironic | 23:27 | |
| *** boris-42_ has joined #openstack-ironic | 23:27 | |
| *** sergek_ has joined #openstack-ironic | 23:27 | |
| *** ildikov_ has joined #openstack-ironic | 23:27 | |
| *** rajinir_ has joined #openstack-ironic | 23:27 | |
| *** kozhukalov_ has joined #openstack-ironic | 23:27 | |
| *** mgkwill_ has joined #openstack-ironic | 23:27 | |
| *** pas-ha_ has joined #openstack-ironic | 23:27 | |
| *** evgenyl_ has joined #openstack-ironic | 23:27 | |
| *** Nakato_ has joined #openstack-ironic | 23:28 | |
| *** strigazi1AFK has joined #openstack-ironic | 23:30 | |
| *** tdurakov_ has joined #openstack-ironic | 23:32 | |
| *** mmedvede_ has joined #openstack-ironic | 23:33 | |
| *** rpioso has quit IRC | 23:33 | |
| *** strigazi_AFK has quit IRC | 23:34 | |
| *** rajinir has quit IRC | 23:34 | |
| *** cdearborn has quit IRC | 23:34 | |
| *** sasha2 has quit IRC | 23:34 | |
| *** Nakato has quit IRC | 23:34 | |
| *** alineb has quit IRC | 23:34 | |
| *** clenimar has quit IRC | 23:34 | |
| *** fyxim has quit IRC | 23:34 | |
| *** sergek has quit IRC | 23:34 | |
| *** aweeks has quit IRC | 23:34 | |
| *** ildikov has quit IRC | 23:34 | |
| *** BadCub has quit IRC | 23:34 | |
| *** pas-ha has quit IRC | 23:34 | |
| *** evgenyl has quit IRC | 23:34 | |
| *** mmedvede has quit IRC | 23:34 | |
| *** tdurakov has quit IRC | 23:34 | |
| *** kozhukalov has quit IRC | 23:34 | |
| *** mgkwill has quit IRC | 23:34 | |
| *** boris-42 has quit IRC | 23:34 | |
| *** pas-ha_ is now known as pas-ha | 23:34 | |
| *** sergek_ is now known as sergek | 23:34 | |
| *** rajinir_ is now known as rajinir | 23:34 | |
| *** mmedvede_ is now known as mmedvede | 23:35 | |
| *** jkilpatr has joined #openstack-ironic | 23:35 | |
| *** mgkwill_ is now known as mgkwill | 23:35 | |
| *** evgenyl_ is now known as evgenyl | 23:36 | |
| *** ildikov_ is now known as ildikov | 23:36 | |
| *** fyxim_ is now known as fyxim | 23:36 | |
| *** kozhukalov_ is now known as kozhukalov | 23:36 | |
| *** boris-42_ is now known as boris-42 | 23:37 | |
| *** tdurakov_ is now known as tdurakov | 23:37 | |
| rybridges2 | mat128: Do you know where I can find a reference of what all of the command codes are for pyghmi? For instance I see command=2 being passed to raw_command here -> https://github.com/openstack/pyghmi/blob/master/pyghmi/ipmi/command.py#L275 | 23:38 |
| rybridges2 | Does there exist a legend of some sort for the magic numbers? | 23:39 |
| *** sdake has quit IRC | 23:39 | |
| *** clenimar has joined #openstack-ironic | 23:40 | |
| *** sasha2 has joined #openstack-ironic | 23:43 | |
| bigjools | Just setting up Ironic for the first time and my node fails to deploy because vif_port_id is not set in the Port extra data. What's supposed to set that? | 23:45 |
| joanna | bigjools: did you run ironic port-create? | 23:46 |
| bigjools | joanna: yes | 23:46 |
| joanna | are you following dev-quickstart? | 23:46 |
| bigjools | mostly, yes but I am not using devstack any more since that worked fine for me | 23:47 |
| bigjools | I'm trying to set it up from scratch on my existing deployment | 23:47 |
| bigjools | largely following this at the moment: https://github.com/jriguera/ansible-ironic-standalone/wiki/Using-Ironic-client-with-agent_ipmitool-driver | 23:48 |
| joanna | ok - when you go to db, are there any ports in ports table? | 23:49 |
| *** wrouesnel has quit IRC | 23:49 | |
| bigjools | yes | 23:49 |
| *** wrouesnel has joined #openstack-ironic | 23:49 | |
| joanna | and the command that is failing is node-set-power state <node> on? | 23:49 |
| bigjools | I'm doing ironic node-set-provision-state | 23:50 |
| joanna | and the ports in ports table have nodes properly assigned? | 23:50 |
| bigjools | yep | 23:51 |
| bigjools | I am using the mitaka code, if it makes any difference | 23:51 |
| joanna | your database is upgraded & everything,right? | 23:51 |
| bigjools | yep | 23:51 |
| joanna | do you have {"vif_port_id": <some_uuid>} in extra for the port you have assigned for the node? | 23:52 |
| bigjools | no, that's not there | 23:52 |
| bigjools | AFAICT it's what's causing the deployment to fail | 23:52 |
| joanna | ok - so it's not in the db. Cool | 23:52 |
| joanna | let me think... | 23:52 |
| patchbot | (let <variable> = <value> in <command>) -- Defines <variable> to be equal to <value> in the <command> and runs the <command>. '=' and 'in' can be omitted. | 23:52 |
| bigjools | appreciate your help, thank you | 23:53 |
| bigjools | I grepped the code and the only place it's set is in deploy_utils.py when it does cleaning | 23:53 |
| joanna | I see in one of the release notes that poty cannot have vif_port_id assigned if member of portgroup.standalone_ports_supported=False | 23:54 |
| joanna | portgroup with this setting | 23:54 |
| bigjools | it's not in a portgroup | 23:55 |
| joanna | that's weird - it seems like extra 'vif_port_id' shall be deprecated in mitaka | 23:57 |
| bigjools | it's odd for sure. the code in ironic/common/network.py gets called at deployment time and it looks for the VIFs in extra. | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!