*** sdake_ has quit IRC | 00:13 | |
*** pmannidi is now known as pmannidi|brb | 00:33 | |
*** mtanino has quit IRC | 00:34 | |
*** hoangcx has joined #openstack-ironic | 00:39 | |
*** tuanluong has joined #openstack-ironic | 00:45 | |
*** arahal_ has joined #openstack-ironic | 00:46 | |
*** portdirect has left #openstack-ironic | 00:46 | |
*** sdake has joined #openstack-ironic | 00:47 | |
*** arahal_ has quit IRC | 00:50 | |
*** awidders_ has joined #openstack-ironic | 00:53 | |
*** awiddersheim has quit IRC | 00:53 | |
*** vsaienko has joined #openstack-ironic | 01:01 | |
*** vishwana_ has joined #openstack-ironic | 01:04 | |
*** alex_xu has quit IRC | 01:05 | |
*** vishwanathj has quit IRC | 01:07 | |
*** alex_xu has joined #openstack-ironic | 01:08 | |
openstackgerrit | gecong proposed openstack/virtualbmc: Replace 'MagicMock' with 'Mock' https://review.openstack.org/384755 | 01:13 |
---|---|---|
*** vsaienko has quit IRC | 01:20 | |
*** mgoddard has joined #openstack-ironic | 01:20 | |
*** sdake has quit IRC | 01:31 | |
*** mgoddard has quit IRC | 01:38 | |
*** yarkot has joined #openstack-ironic | 02:03 | |
*** sdake has joined #openstack-ironic | 02:07 | |
*** sdake has quit IRC | 02:07 | |
*** sdake has joined #openstack-ironic | 02:07 | |
*** jcook has quit IRC | 02:08 | |
*** sdake has quit IRC | 02:09 | |
*** sdake has joined #openstack-ironic | 02:12 | |
*** caiobo has quit IRC | 02:12 | |
*** jcook has joined #openstack-ironic | 02:13 | |
*** hoangcx_ has joined #openstack-ironic | 02:18 | |
*** hoangcx_ has quit IRC | 02:18 | |
-openstackstatus- NOTICE: Jobs running on osic nodes are failing due to network issues with the mirror. We are temporarily disabling the cloud. | 02:48 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Update the existing APIs due to adding get_supported_power_states https://review.openstack.org/382194 | 02:48 |
*** MattMan has quit IRC | 02:52 | |
*** MattMan has joined #openstack-ironic | 02:53 | |
*** xuao has joined #openstack-ironic | 02:53 | |
*** sdake_ has joined #openstack-ironic | 02:56 | |
*** sdake has quit IRC | 02:59 | |
*** amotoki has quit IRC | 03:19 | |
*** amotoki has joined #openstack-ironic | 03:30 | |
*** davidlenwell has quit IRC | 03:34 | |
*** vikrant has joined #openstack-ironic | 03:35 | |
*** amotoki has quit IRC | 03:35 | |
*** amotoki has joined #openstack-ironic | 03:40 | |
*** davidlenwell has joined #openstack-ironic | 03:43 | |
*** huiweics has joined #openstack-ironic | 03:45 | |
openstackgerrit | haris proposed openstack/python-ironicclient: Enables ironic list commands to use labels https://review.openstack.org/380671 | 03:48 |
*** yuanying_ has joined #openstack-ironic | 03:57 | |
*** yuanying has quit IRC | 04:01 | |
*** links has joined #openstack-ironic | 04:01 | |
*** amotoki has quit IRC | 04:01 | |
*** yuanying_ has quit IRC | 04:02 | |
*** hshiina has joined #openstack-ironic | 04:04 | |
*** amotoki has joined #openstack-ironic | 04:07 | |
*** bfournie has quit IRC | 04:15 | |
openstackgerrit | Xu Ao proposed openstack/ironic: Make ironic console supporting the nokeepalive mode https://review.openstack.org/372830 | 04:15 |
*** robcresswell has quit IRC | 04:23 | |
*** robcresswell has joined #openstack-ironic | 04:25 | |
*** davidlenwell has quit IRC | 04:35 | |
*** amotoki has quit IRC | 04:51 | |
*** amotoki has joined #openstack-ironic | 04:57 | |
*** e0ne has joined #openstack-ironic | 05:00 | |
*** bfournie has joined #openstack-ironic | 05:02 | |
*** e0ne has quit IRC | 05:04 | |
*** praneshp has joined #openstack-ironic | 05:12 | |
*** rama_y has joined #openstack-ironic | 05:27 | |
*** vishwana_ has quit IRC | 05:32 | |
*** vishwanathj has joined #openstack-ironic | 05:33 | |
*** adam_g` has quit IRC | 05:38 | |
*** d0ugal has quit IRC | 05:40 | |
*** d0ugal has joined #openstack-ironic | 05:41 | |
*** e0ne has joined #openstack-ironic | 05:47 | |
*** yuanying has joined #openstack-ironic | 05:50 | |
*** e0ne has quit IRC | 05:56 | |
*** rama_y has quit IRC | 05:59 | |
*** rcernin has joined #openstack-ironic | 06:07 | |
*** mjura has joined #openstack-ironic | 06:13 | |
*** adam_g has joined #openstack-ironic | 06:18 | |
*** adam_g has quit IRC | 06:18 | |
*** adam_g has joined #openstack-ironic | 06:18 | |
pas-ha | morning Ironic | 06:19 |
tuanluong | mroning ironic | 06:20 |
*** mjura has quit IRC | 06:20 | |
*** vsaienko has joined #openstack-ironic | 06:20 | |
*** bjolo_ has joined #openstack-ironic | 06:21 | |
openstackgerrit | Xu Ao proposed openstack/ironic: Make ironic console supporting the nokeepalive mode https://review.openstack.org/372830 | 06:28 |
*** mjura has joined #openstack-ironic | 06:33 | |
*** pcaruana has joined #openstack-ironic | 06:36 | |
xuao | morning ironic ~~~ | 06:36 |
*** praneshp has quit IRC | 06:37 | |
*** yuanying has quit IRC | 06:41 | |
*** davidlenwell has joined #openstack-ironic | 06:51 | |
*** davidlenwell has quit IRC | 06:56 | |
*** watanabe_isao has joined #openstack-ironic | 07:01 | |
*** Vinsh has quit IRC | 07:02 | |
*** jaosorior has joined #openstack-ironic | 07:03 | |
*** Vinsh has joined #openstack-ironic | 07:03 | |
*** sabeen has quit IRC | 07:04 | |
*** ccamacho has joined #openstack-ironic | 07:10 | |
*** sabeen has joined #openstack-ironic | 07:10 | |
*** AlexeyAbashkin has joined #openstack-ironic | 07:14 | |
*** davidlenwell has joined #openstack-ironic | 07:16 | |
*** Alexey_Abashkin_ has quit IRC | 07:17 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Help a user to enable console redirection https://review.openstack.org/384578 | 07:24 |
*** amotoki has quit IRC | 07:26 | |
*** vsaienko has quit IRC | 07:27 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 07:33 |
*** vsaienko has joined #openstack-ironic | 07:33 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work https://review.openstack.org/376322 | 07:37 |
*** davidlenwell has quit IRC | 07:44 | |
*** davidlenwell has joined #openstack-ironic | 07:46 | |
*** amotoki has joined #openstack-ironic | 07:51 | |
*** amotoki has quit IRC | 07:51 | |
*** hogepodge has quit IRC | 07:51 | |
*** amotoki has joined #openstack-ironic | 07:51 | |
*** Vinsh has quit IRC | 07:52 | |
*** aarefiev has quit IRC | 07:55 | |
*** aarefiev has joined #openstack-ironic | 07:56 | |
*** hogepodge has joined #openstack-ironic | 07:58 | |
vsaienk0 | morning xuao, dtantsur, pas-ha, vdrok, aarefiev, lucasagomes, sambetts and all Ironic'ers! | 07:58 |
xuao | morning ~ vsalenk0 : ) | 07:59 |
xuao | Oh sorry ~ vsaienk0! | 07:59 |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:00 | |
*** e0ne has joined #openstack-ironic | 08:00 | |
*** ohamada has joined #openstack-ironic | 08:09 | |
*** hoangcx has quit IRC | 08:12 | |
*** davidlenwell has quit IRC | 08:14 | |
*** athomas has joined #openstack-ironic | 08:15 | |
*** vsaienko has quit IRC | 08:17 | |
*** tuanluong_ has joined #openstack-ironic | 08:17 | |
*** hoangcx has joined #openstack-ironic | 08:18 | |
*** davidlenwell has joined #openstack-ironic | 08:18 | |
*** tuanluong has quit IRC | 08:21 | |
*** sdake_ has quit IRC | 08:21 | |
*** watanabe_isao has quit IRC | 08:22 | |
*** Romanenko_K has joined #openstack-ironic | 08:23 | |
*** sabeen has quit IRC | 08:24 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Fix broken xenial job https://review.openstack.org/384652 | 08:25 |
*** yuanying has joined #openstack-ironic | 08:27 | |
*** tuanluong__ has joined #openstack-ironic | 08:28 | |
*** tuanluong_ has quit IRC | 08:32 | |
*** pester has joined #openstack-ironic | 08:32 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off https://review.openstack.org/216730 | 08:33 |
*** derekh has joined #openstack-ironic | 08:34 | |
*** hoangcx has quit IRC | 08:34 | |
*** hoangcx has joined #openstack-ironic | 08:35 | |
*** hshiina2 has joined #openstack-ironic | 08:36 | |
*** yibo has joined #openstack-ironic | 08:36 | |
*** tuanluong has joined #openstack-ironic | 08:38 | |
*** early has joined #openstack-ironic | 08:38 | |
*** sambetts has joined #openstack-ironic | 08:39 | |
*** another_larsks has joined #openstack-ironic | 08:39 | |
sambetts | Morning all, my ZNC died last night because of a power outage, did I miss anything important | 08:40 |
sambetts | ? | 08:40 |
*** vicky has joined #openstack-ironic | 08:40 | |
*** jraju has joined #openstack-ironic | 08:41 | |
*** davidlenwell has quit IRC | 08:41 | |
*** mjura has quit IRC | 08:41 | |
*** adam_g has quit IRC | 08:41 | |
*** hshiina has quit IRC | 08:41 | |
*** links has quit IRC | 08:41 | |
*** vikrant has quit IRC | 08:41 | |
*** yarkot has quit IRC | 08:41 | |
*** hoonetorg has quit IRC | 08:41 | |
*** yibo_ has quit IRC | 08:41 | |
*** early` has quit IRC | 08:41 | |
*** makowals has quit IRC | 08:41 | |
*** charlledaniel has quit IRC | 08:41 | |
*** mrtenio has quit IRC | 08:41 | |
*** fxpester has quit IRC | 08:41 | |
*** amit213 has quit IRC | 08:41 | |
*** evgenyl has quit IRC | 08:41 | |
*** tdurakov has quit IRC | 08:41 | |
*** fungi has quit IRC | 08:41 | |
*** rm_work has quit IRC | 08:41 | |
*** mgagne has quit IRC | 08:41 | |
*** JayF has quit IRC | 08:41 | |
*** odyssey4me has quit IRC | 08:41 | |
*** natorious has quit IRC | 08:41 | |
*** larsks has quit IRC | 08:41 | |
*** tuanluong__ has quit IRC | 08:41 | |
*** yarkot has joined #openstack-ironic | 08:42 | |
*** hoangcx has quit IRC | 08:44 | |
*** mrtenio has joined #openstack-ironic | 08:44 | |
*** charlledaniel has joined #openstack-ironic | 08:45 | |
*** hoonetorg has joined #openstack-ironic | 08:46 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 08:46 |
*** hoangcx has joined #openstack-ironic | 08:48 | |
*** davidlenwell has joined #openstack-ironic | 08:49 | |
*** mjura has joined #openstack-ironic | 08:49 | |
*** adam_g has joined #openstack-ironic | 08:49 | |
*** makowals has joined #openstack-ironic | 08:49 | |
*** amit213 has joined #openstack-ironic | 08:49 | |
*** evgenyl has joined #openstack-ironic | 08:49 | |
*** tdurakov has joined #openstack-ironic | 08:49 | |
*** fungi has joined #openstack-ironic | 08:49 | |
*** rm_work has joined #openstack-ironic | 08:49 | |
*** mgagne has joined #openstack-ironic | 08:49 | |
*** JayF has joined #openstack-ironic | 08:49 | |
*** odyssey4me has joined #openstack-ironic | 08:49 | |
*** natorious has joined #openstack-ironic | 08:49 | |
*** vgadiraj has joined #openstack-ironic | 08:50 | |
*** aslezil has joined #openstack-ironic | 08:51 | |
*** anush has joined #openstack-ironic | 08:52 | |
*** strigazi_AFK is now known as strigazi | 08:52 | |
*** evgenyl has quit IRC | 08:53 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 08:56 |
*** evgenyl has joined #openstack-ironic | 08:57 | |
*** syed_ has quit IRC | 09:04 | |
*** adam_g has quit IRC | 09:06 | |
*** hshiina2 has quit IRC | 09:06 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Help a user to enable console redirection https://review.openstack.org/384578 | 09:06 |
*** adam_g has joined #openstack-ironic | 09:07 | |
*** adam_g has quit IRC | 09:08 | |
*** adam_g has joined #openstack-ironic | 09:08 | |
*** vsaienko has joined #openstack-ironic | 09:14 | |
*** huiweics has quit IRC | 09:15 | |
vdrok | good morning pas-ha tuanluong xuao and sambetts | 09:19 |
tuanluong | morning vdrok | 09:19 |
xuao | morning~~~~vdrok | 09:19 |
xuao | Thanks vdrok~ I think I just got a lot new skill from your reviewing~~ | 09:21 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Remove wrong check from conductor periodic task https://review.openstack.org/384899 | 09:21 |
vdrok | xuao: you're welcome, glad to hear that :) | 09:21 |
xuao | Ahahahaah~ | 09:21 |
*** davidlenwell has quit IRC | 09:24 | |
*** openstack has joined #openstack-ironic | 09:31 | |
openstackgerrit | haris proposed openstack/python-ironicclient: Enables ironic list commands to use labels https://review.openstack.org/380671 | 09:31 |
*** jaosorior has quit IRC | 09:39 | |
*** jaosorior has joined #openstack-ironic | 09:39 | |
*** jpich has joined #openstack-ironic | 09:41 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update configure_tenant_networks https://review.openstack.org/356460 | 09:44 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Update stable/mitaka driver requirement for iRMC https://review.openstack.org/365392 | 09:47 |
*** tuanluong has quit IRC | 09:57 | |
*** mgould|afk is now known as mgould | 09:59 | |
mgould | morning vdrok pas-ha sambetts xuao vsaienk0 | 10:00 |
vdrok | morning mgould | 10:00 |
xuao | morning ~ mgould~ | 10:00 |
*** hoangcx has quit IRC | 10:02 | |
*** Vinsh has joined #openstack-ironic | 10:04 | |
*** milan has joined #openstack-ironic | 10:06 | |
milan | morning Ironic! :) | 10:06 |
mgould | morning milan | 10:07 |
milan | hi mgould! :) | 10:07 |
*** _vsaienko has joined #openstack-ironic | 10:09 | |
*** vsaienko has quit IRC | 10:09 | |
vdrok | good morning milan | 10:09 |
milan | morning vdrok! :) | 10:09 |
vsaienk0 | morning mgould | 10:10 |
milan | morning vsaienk0! :) | 10:10 |
vsaienk0 | morning milan :) | 10:11 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update configure_tenant_networks https://review.openstack.org/356460 | 10:11 |
*** jraju has quit IRC | 10:15 | |
tangchen | morning mgould, milan, vdrok, vsaienk0, xuao, and Ironic | 10:15 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 10:16 |
vsaienk0 | morning tangchen | 10:16 |
milan | morning tangchen! :) | 10:16 |
mgould | morning tangchen | 10:17 |
*** bjolo_ has quit IRC | 10:18 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic-lib: DO NOT MERGE: testing new jobs https://review.openstack.org/383687 | 10:19 |
*** dtantsur|afk is now known as dtantsur | 10:20 | |
dtantsur | Morning Ironic | 10:20 |
tangchen | morning dtantsur | 10:20 |
vsaienk0 | morning dtantsur | 10:20 |
sambetts | o/ dtantsur | 10:20 |
openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Support i18n for baremetal chassis cmds https://review.openstack.org/375972 | 10:21 |
openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Support i18n for baremetal port cmds https://review.openstack.org/375971 | 10:21 |
openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Support i18n for baremetal node cmds https://review.openstack.org/357639 | 10:21 |
vdrok | morning tangchen and dtantsur | 10:21 |
vsaienk0 | dtantsur: there is the fix for xenial job: https://review.openstack.org/#/c/384652/ | 10:21 |
patchbot | patch 384652 - ironic - Fix broken xenial job | 10:21 |
openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Add missing unit tests for provision state commands https://review.openstack.org/377607 | 10:22 |
openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Trivial: Rename and reorder deploy baremetal test https://review.openstack.org/378140 | 10:22 |
dtantsur | vsaienk0, wow, srsly? you rock! | 10:22 |
*** sdake has joined #openstack-ironic | 10:23 | |
dtantsur | vsaienk0, one suggestions inline | 10:23 |
mgould | morning dtantsur | 10:24 |
openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Make console commands support multiple node operation https://review.openstack.org/378477 | 10:25 |
vsaienk0 | dtantsur: we can't because default for PUBLIC_BRIDGE_MTU is 1500 | 10:25 |
sambetts | vsaienk0: but you override it to the new number right? | 10:26 |
vsaienk0 | sambetts: right I set it to 1450 | 10:27 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Fix broken xenial job https://review.openstack.org/384652 | 10:27 |
vsaienk0 | sambetts, dtantsur: I need to drink more coffee :) | 10:28 |
dtantsur | vsaienk0, go ahead :) | 10:28 |
sambetts | ;) | 10:28 |
jroll | morning everyone :) | 10:32 |
mgould | morning jroll | 10:32 |
vsaienk0 | morning jroll | 10:33 |
sambetts | hey jroll | 10:33 |
mgould | vsaienk0: LGTM, one probably-stupid query inline :-) | 10:36 |
*** alex_xu has quit IRC | 10:36 | |
*** alex_xu has joined #openstack-ironic | 10:39 | |
*** xuao has quit IRC | 10:42 | |
vdrok | morning jroll | 10:43 |
dtantsur | morning jroll! In case you haven't seen: stable/liberty enters security-only phase, we should land whatever can be landed and release it by Thursday | 10:44 |
dtantsur | at least IPA has non-requirements changes, so it should be released | 10:44 |
jroll | dtantsur: morning :) indeed, we should release mitaka too | 10:44 |
milan | morning dtantsur, jroll, sambetts! :) | 10:44 |
* jroll will review things shortly | 10:45 | |
dtantsur | yeah, but mitaka is not pressing | 10:45 |
jroll | well, it's moving to phase 2, so anything that isn't critical / security needs to get done | 10:46 |
jroll | and as someone else mentioned in the thread, would be nice to release for the phase change anyway | 10:46 |
dtantsur | hmm, yeah, fair | 10:47 |
*** hoangcx has joined #openstack-ironic | 10:47 | |
dtantsur | I don't see anything outstanding for stable/liberty, checking mitaka now | 10:47 |
jroll | https://review.openstack.org/#/q/(project:openstack/ironic+OR+project:openstack/ironic-python-agent+OR+project:openstack/ironic-inspector+)+AND+NOT+branch:master+AND+status:open | 10:47 |
jroll | not much to do I guess :) | 10:47 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for node list and show with specific fields https://review.openstack.org/384923 | 10:48 |
dtantsur | huh, does mitaka use upper-constraints? | 10:49 |
jroll | hm, mitaka gate down? https://review.openstack.org/#/c/365392/ | 10:49 |
patchbot | patch 365392 - ironic (stable/mitaka) - Update stable/mitaka driver requirement for iRMC | 10:49 |
jroll | it does | 10:49 |
dtantsur | this looks like it's pulling the latest ironic-lib: http://logs.openstack.org/92/365392/3/check/gate-ironic-python34-db/8f158bc/console.html#_2016-10-11_09_54_19_087037 | 10:49 |
jroll | (I think?) | 10:49 |
TheJulia | Good morning | 10:49 |
dtantsur | morning TheJulia | 10:49 |
jroll | https://github.com/openstack/ironic/blob/stable/mitaka/tox.ini#L9 | 10:49 |
jroll | 1.2.0 https://github.com/openstack/requirements/blob/stable/mitaka/upper-constraints.txt#L152 | 10:50 |
jroll | what the heck? | 10:50 |
dtantsur | jroll, oh https://github.com/openstack/ironic/blob/stable/mitaka/tox.ini#L9 is totally wrong | 10:50 |
dtantsur | iirc we've switched away from a separate "constraints" thing there | 10:51 |
jroll | oh, fun | 10:51 |
jroll | you got that or want me to? | 10:51 |
jroll | same thing for liberty | 10:51 |
jroll | this means this one will likely also fail if I approve it :( https://review.openstack.org/#/c/354735/ | 10:52 |
patchbot | patch 354735 - ironic (stable/mitaka) - Fix updating port MAC address for active nodes | 10:52 |
*** mmedvede has joined #openstack-ironic | 10:52 | |
dtantsur | jroll, we probably need https://review.openstack.org/#/c/300146/ backported | 10:53 |
patchbot | patch 300146 - ironic - Make tox respect upper-constraints.txt (MERGED) | 10:53 |
dtantsur | I can do it, yeah | 10:53 |
jroll | ah yes | 10:53 |
jroll | dtantsur: probably want this one too https://github.com/openstack/ironic/commit/6e775b82f4a9b82fdf72f82239d0e8d53add6ba2 | 10:54 |
jroll | https://review.openstack.org/#/c/355488/ | 10:54 |
patchbot | patch 355488 - ironic - Use upper-constraints for all tox targets (MERGED) | 10:54 |
*** hoangcx2 has joined #openstack-ironic | 10:54 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Make tox respect upper-constraints.txt https://review.openstack.org/384927 | 10:55 |
dtantsur | jroll, this is the second, right. | 10:55 |
dtantsur | jroll, here we go ^^^ | 10:55 |
*** hoangcx has quit IRC | 10:55 | |
jroll | +2A | 10:56 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Use upper-constraints for all tox targets https://review.openstack.org/384929 | 10:56 |
dtantsur | and the second ^^^ | 10:57 |
jroll | whee, thanks | 10:57 |
jroll | +2A | 10:57 |
dtantsur | now the same for liberty, stay tuned | 10:57 |
vsaienk0 | jroll, sambetts: I've created a nova blueprint https://blueprints.launchpad.net/nova/+spec/ironic-portgroups-support and wondered if we need to propose a spec as well for this | 10:57 |
* jroll rebases the rest on that | 10:57 | |
jroll | vsaienk0: a nova spec? I don't think so, I'll bug matt today | 10:58 |
jroll | vsaienk0: wait, this sounds like dynamic portgroups... | 10:58 |
*** vsaienko has joined #openstack-ironic | 10:59 | |
jroll | vsaienk0: let's have one for using portgroups like we planned, and then later another for dynamic port groups (which will need a spec because API changes) | 10:59 |
*** _vsaienko has quit IRC | 10:59 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Make tox respect upper-constraints.txt https://review.openstack.org/384933 | 11:00 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Update stable/mitaka driver requirement for iRMC https://review.openstack.org/365392 | 11:00 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fix updating port MAC address for active nodes https://review.openstack.org/354735 | 11:01 |
*** thrash|g0ne is now known as thrash | 11:01 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Use upper-constraints for all tox targets https://review.openstack.org/384934 | 11:01 |
dtantsur | jroll, I should have squashed them in one probably, but still ^^^ | 11:01 |
jroll | dtantsur: is this not a backport? https://review.openstack.org/#/c/365392/ | 11:01 |
patchbot | patch 365392 - ironic (stable/mitaka) - Update stable/mitaka driver requirement for iRMC | 11:01 |
jroll | or is it just missing cherry-pick line | 11:02 |
dtantsur | jroll, it's not; they've bumped master to a feature version (0.4.0) | 11:02 |
jroll | okay, thanks | 11:02 |
dtantsur | jroll, ugh, it looks like we'll have to squash the patches, otherwise the coverage one fails.... | 11:04 |
vsaienk0 | jroll: just update description to highlight only static portgroup case | 11:04 |
jroll | dtantsur: oops :/ | 11:05 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Make tox respect upper-constraints.txt https://review.openstack.org/384927 | 11:05 |
jroll | vsaienk0: thanks, I'll run it by matt/john today | 11:05 |
dtantsur | jroll, check this ^^^ | 11:05 |
*** _vsaienko has joined #openstack-ironic | 11:06 | |
jroll | dtantsur: yeah, seems fine | 11:06 |
*** vsaienko has quit IRC | 11:06 | |
dtantsur | if it passes, I'll do the same for liberty | 11:07 |
* dtantsur sigh... | 11:07 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Update stable/mitaka driver requirement for iRMC https://review.openstack.org/365392 | 11:07 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fix updating port MAC address for active nodes https://review.openstack.org/354735 | 11:07 |
jroll | rebased on it | 11:07 |
*** links has joined #openstack-ironic | 11:09 | |
dtantsur | jroll, hah, liberty has passed, so no need for squash there, I guess :) | 11:10 |
jroll | dtantsur: O_o | 11:10 |
sambetts | SCIENCE ;) | 11:10 |
dtantsur | it does not use ironic-lib, so maybe everything else is fine | 11:11 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Fix broken xenial job https://review.openstack.org/384652 | 11:11 |
jroll | dtantsur: ah, true | 11:11 |
dtantsur | so please approve https://review.openstack.org/#/c/384933/ and https://review.openstack.org/#/c/384934/1 | 11:11 |
patchbot | patch 384933 - ironic (stable/liberty) - Make tox respect upper-constraints.txt | 11:11 |
patchbot | patch 384934 - ironic (stable/liberty) - Use upper-constraints for all tox targets | 11:11 |
jroll | yep, please hold | 11:12 |
dtantsur | mitaka has passed too, good | 11:13 |
jroll | done | 11:13 |
dtantsur | thanks! hopefully that's all we need to finish the last patches | 11:14 |
*** sdake_ has joined #openstack-ironic | 11:14 | |
jroll | indeed | 11:14 |
jroll | I've also +2'd the newton backports we have | 11:14 |
*** _vsaienko has quit IRC | 11:15 | |
dtantsur | in the meanwhile, I'm proposing liberty release for IPA, as it has the install_grub fix | 11:15 |
jroll | ++ | 11:15 |
*** vsaienko has joined #openstack-ironic | 11:15 | |
*** sdake has quit IRC | 11:15 | |
mgould | w00t | 11:17 |
openstackgerrit | Merged openstack/ironic: Make tox respect upper-constraints.txt https://review.openstack.org/384927 | 11:20 |
openstackgerrit | Merged openstack/ironic: Make tox respect upper-constraints.txt https://review.openstack.org/384933 | 11:22 |
openstackgerrit | Merged openstack/ironic: Use upper-constraints for all tox targets https://review.openstack.org/384934 | 11:22 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Fix broken xenial job https://review.openstack.org/384652 | 11:24 |
*** yolanda has quit IRC | 11:25 | |
jroll | I'm +2 on ^^ if anyone else wants to have a look | 11:31 |
jroll | would be nice to check experimental before landing | 11:31 |
* jroll does it | 11:32 | |
dtantsur | jroll, please wait for the postgres job to pass | 11:37 |
dtantsur | (this is the only job using xenial right now) | 11:37 |
jroll | dtantsur: ah, ok | 11:37 |
bfournie | good morning ironic | 11:37 |
dtantsur | morning bfournie | 11:38 |
jroll | dtantsur: oh, the experimental jobs are on ironic-lib, right? shall we kick those off? | 11:38 |
jroll | hey bfournie :) | 11:38 |
dtantsur | jroll, I think vsaienk0 already did | 11:38 |
jroll | cool | 11:38 |
* jroll missed that | 11:38 | |
*** adam_g has quit IRC | 11:43 | |
*** ohamada_ has joined #openstack-ironic | 11:45 | |
*** ohamada has quit IRC | 11:45 | |
*** dprince has joined #openstack-ironic | 11:47 | |
*** lucas-afk is now known as lucasagomes | 11:48 | |
lucasagomes | morning all | 11:48 |
dtantsur | morning lucasagomes | 11:48 |
*** openstackgerrit has quit IRC | 11:48 | |
*** openstackgerrit has joined #openstack-ironic | 11:49 | |
jroll | morning lucasagomes | 11:52 |
jroll | hm, did we not have an api meeting last week? | 11:53 |
bfournie | morning lucasagomes | 11:54 |
jroll | guess not :/ | 11:54 |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Add instructions to deploy bifrost on virsh https://review.openstack.org/383792 | 11:59 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Fix broken xenial job https://review.openstack.org/384652 | 12:05 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Fix setting custom IRONIC_VM_NETWORK_BRIDGE https://review.openstack.org/365116 | 12:05 |
openstackgerrit | Merged openstack/bifrost: Enable release notes translation https://review.openstack.org/383202 | 12:07 |
openstackgerrit | Merged openstack/bifrost: Allow management of vlans when specifying mac https://review.openstack.org/380183 | 12:08 |
*** trown|outtypewww is now known as trown | 12:15 | |
openstackgerrit | Bob Fournier proposed openstack/ironic-inspector-specs: Add LLDP processing hook and new CLI commands https://review.openstack.org/374381 | 12:17 |
openstackgerrit | Fabio Dassan proposed openstack/pyghmi: Implement M4 and Kent support https://review.openstack.org/361457 | 12:17 |
*** openstackgerrit has quit IRC | 12:18 | |
*** ccamacho is now known as ccamacho|lunch | 12:18 | |
*** openstackgerrit has joined #openstack-ironic | 12:18 | |
jroll | dtantsur: mat128: want to start drafting a proposal for raid/partitioning? https://etherpad.openstack.org/p/ironic-ocata-summit-deploy-time-raid | 12:24 |
dtantsur | jroll, sure, not right now though | 12:24 |
mat128 | jroll: I still need to address comments, but the first draft was pushed last week: https://review.openstack.org/#/c/382091/ | 12:24 |
patchbot | patch 382091 - ironic-specs - RAID configuration during deployment | 12:24 |
dtantsur | this one is mostly about Nova side, right? | 12:24 |
jroll | dtantsur: yeah, just giving you the etherpad link :) | 12:25 |
jroll | indeed | 12:25 |
dtantsur | mat128, your spec does not touch Nova, right? | 12:25 |
jroll | https://www.openstack.org/summit/barcelona-2016/summit-schedule/events/16947 | 12:25 |
mat128 | dtantsur: as decided in the midcycle, the nova side shoudl be a different discussion/spec | 12:25 |
mat128 | we need to have that capability in Ironic first | 12:25 |
dtantsur | mat128, well, the summit discussion IS the Nova discussion | 12:25 |
jroll | mat128: yeah, we need to get ^^^ | 12:25 |
jroll | the nova part is the hard part | 12:25 |
mat128 | dtantsur, jroll: I agree | 12:25 |
dtantsur | jroll, cool timing, we'll be able to continue with beers right afterwards :) | 12:25 |
jroll | dtantsur: indeed :) | 12:26 |
jroll | that timing worked best for nova folks | 12:26 |
jroll | I may have to pay them with beer to come, since it's so late | 12:26 |
*** bjolo_ has joined #openstack-ironic | 12:27 | |
dtantsur | heh | 12:27 |
mat128 | I wont be at the summit, but I am convinced that the group will take the right decision | 12:27 |
mat128 | brb | 12:27 |
jroll | mat128: you're quite optimistic :) | 12:27 |
jroll | mat128: but yeah, if you could help propose a good option for nova (or maybe one API option, one flavor-based option?) | 12:27 |
*** jpich has quit IRC | 12:28 | |
mat128 | jroll: ok sure | 12:29 |
jroll | thanks :) | 12:29 |
mat128 | brb for real now :) | 12:29 |
*** mat128 is now known as mat128|afk | 12:29 | |
dtantsur | do we have schedule posted for any official parties at the summit? | 12:35 |
*** mgoddard has joined #openstack-ironic | 12:43 | |
jroll | I haven't seen any this time :/ | 12:44 |
*** jcoufal has joined #openstack-ironic | 12:44 | |
dtantsur | strange.. do we plan on a team gathering somewhere? | 12:44 |
*** baoli has joined #openstack-ironic | 12:45 | |
jroll | I would like to | 12:45 |
jroll | I haven't even began to think about planning one, though :( | 12:45 |
jroll | and I don't believe I could pay for one, either | 12:45 |
* jroll did not plan well | 12:45 | |
*** amotoki has quit IRC | 12:45 | |
dtantsur | heh, I see. well, I can easily pay for myself, unless we go to a really fancy place | 12:46 |
*** baoli_ has joined #openstack-ironic | 12:46 | |
jroll | yeah | 12:47 |
*** vicky has quit IRC | 12:47 | |
jroll | lucasagomes: how's your spanish? think you could find us somewhere to eat? :P | 12:47 |
vsaienk0 | jroll, dtantsur, sambetts: to fix xenial https://review.openstack.org/#/c/384652/ and https://review.openstack.org/#/c/365116/ xenial passed on ironic-lib https://review.openstack.org/#/c/383687/ | 12:47 |
patchbot | patch 384652 - ironic - Fix broken xenial job | 12:47 |
patchbot | patch 365116 - ironic - Fix setting custom IRONIC_VM_NETWORK_BRIDGE | 12:47 |
patchbot | patch 383687 - ironic-lib - DO NOT MERGE: testing new jobs | 12:47 |
lucasagomes | jroll, hi there haha it's dodgy but I can manage to speak some | 12:48 |
vsaienk0 | lucasagomes please review as well ^^ xenial unblock patches | 12:48 |
jroll | vsaienk0: thanks | 12:48 |
lucasagomes | jroll, sure, do we have a date already ? | 12:48 |
jroll | lucasagomes: heh, no, we haven't planned for it at all | 12:48 |
* lucasagomes should send something to the ML if not | 12:48 | |
dtantsur | vsaienk0, let's see the CI pass on them (especially the postgres job) | 12:48 |
jroll | lucasagomes: I'm half joking, but if you were able to get a reservation for 20-30 somewhere it would be amazing | 12:48 |
jroll | not sure we can be picky about the date at this point | 12:49 |
lucasagomes | jroll, I can try... Will look at something | 12:49 |
* lucasagomes knows some folks in barcelona, will ask some opinions | 12:49 | |
*** baoli has quit IRC | 12:50 | |
jroll | lucasagomes: you rock! don't feel bad if you can't find something easily :) | 12:50 |
jroll | we can always just randomly invade some tapas place | 12:50 |
jroll | woo, summit is scheduled https://www.openstack.org/summit/barcelona-2016/summit-schedule/global-search?t=Ironic%3A | 12:50 |
* jroll will email the list | 12:50 | |
lucasagomes | true, let's try! | 12:50 |
mat128|afk | jroll: the tags search is a little bit flaky | 12:53 |
*** mat128|afk is now known as mat128 | 12:54 | |
mat128 | https://www.openstack.org/summit/barcelona-2016/summit-schedule/global-search?t=Ironic gives better results | 12:54 |
jroll | mat128: I added the colon to try and restrict it to only design summit sessions, I don't want all the talks in there | 12:55 |
mat128 | jroll: oic, thought it was a typo | 12:56 |
jroll | :) | 12:56 |
jroll | it worked well except that mrda named his talk with a colon >:) | 12:56 |
*** jpich has joined #openstack-ironic | 12:58 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Introduce default_boot_option configuration option https://review.openstack.org/375458 | 12:59 |
*** rloo has joined #openstack-ironic | 12:59 | |
*** skazi has quit IRC | 12:59 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 12:59 |
*** skazi has joined #openstack-ironic | 13:00 | |
dtantsur | jroll, lucasagomes, we can still have a doodle, but they fall back to available options | 13:00 |
dtantsur | e.g. RH folks are not quite available on Monday (some of us also prefer to avoid Tue) | 13:01 |
jroll | dtantsur: yeah | 13:01 |
jroll | surprisingly I have no commitments yet O_o | 13:01 |
*** jheroux has joined #openstack-ironic | 13:02 | |
mat128 | dtantsur: which approaches did you have in mind for nova<>ironic raid? | 13:03 |
lucasagomes | dtantsur, jroll true | 13:04 |
dtantsur | still dumping my thoughts | 13:04 |
lucasagomes | that said I'll arrive on monday afternoon/evening we could at least go to a pub | 13:04 |
lucasagomes | if you guys are there too | 13:04 |
jroll | lucasagomes: yeah, I'll be around, I arrive the wednesday before | 13:05 |
*** amotoki has joined #openstack-ironic | 13:06 | |
lucasagomes | jroll, a-ha, nice! Totally worth it, barcelona is pretty amazing | 13:06 |
mgould | lucasagomes: I land at 0920 on Monday morning, but will need some time to travel into town, check in etc | 13:06 |
*** mgoddard has quit IRC | 13:06 | |
* sambetts isn't a fan of the market place stall crawl being on the Monday this time... | 13:06 | |
*** Goneri has joined #openstack-ironic | 13:07 | |
* jroll s/being on the Monday this time...// | 13:07 | |
jroll | :D | 13:07 |
sambetts | hahaha | 13:07 |
sambetts | I agree sooooo much | 13:07 |
*** hoangcx2 has quit IRC | 13:07 | |
dtantsur | lucasagomes, mgould, don't forget about RH private party on Monday | 13:08 |
dtantsur | jroll++ | 13:08 |
* sambetts lands on Sunday midday | 13:08 | |
* mgould had not forgotten :-) | 13:08 | |
lucasagomes | dtantsur, yup | 13:08 |
* jroll wishes someone would throw him a private party | 13:08 | |
* jroll will have to make his own | 13:08 | |
*** hoangcx has joined #openstack-ironic | 13:09 | |
mgould | jroll: you're all welcome to drop by my AirBnB and drink duty-free Scotch one night :-) | 13:10 |
* lucasagomes handles jroll a pint | 13:10 | |
jroll | mgould: ++ | 13:10 |
*** hamzy has quit IRC | 13:10 | |
dtantsur | oh, seems like we have a venue! | 13:11 |
*** cdearborn has joined #openstack-ironic | 13:11 | |
openstackgerrit | boden proposed openstack/ironic: Replace retrying with tenacity https://review.openstack.org/376574 | 13:13 |
*** ppiela has joined #openstack-ironic | 13:15 | |
*** hshiina has joined #openstack-ironic | 13:15 | |
* mgould wonders what he's let himself in for | 13:16 | |
sambetts | I think the owner of the AirBnB doesn't know what they've let themselves into | 13:17 |
sambetts | ;) | 13:17 |
mgould | heh | 13:17 |
openstackgerrit | Merged openstack/ironic-python-agent: Use CoreOS 1068.10.0 https://review.openstack.org/366142 | 13:21 |
mat128 | ^oh nice | 13:21 |
dtantsur | yep | 13:21 |
mat128 | thanks dtantsur for rechecking :) | 13:21 |
dtantsur | I can't believe, our CI is working again? :) | 13:21 |
dtantsur | np | 13:21 |
mat128 | yup :) | 13:21 |
sambetts | \o/ | 13:21 |
*** ralonsoh has joined #openstack-ironic | 13:21 | |
jroll | woot | 13:21 |
*** adam_g has joined #openstack-ironic | 13:22 | |
*** adam_g has quit IRC | 13:22 | |
*** adam_g has joined #openstack-ironic | 13:22 | |
openstackgerrit | Merged openstack/ironic: Update stable/mitaka driver requirement for iRMC https://review.openstack.org/365392 | 13:23 |
openstackgerrit | Merged openstack/ironic: Fix updating port MAC address for active nodes https://review.openstack.org/354735 | 13:23 |
jroll | \o/ | 13:23 |
jroll | time for mitaka release eh | 13:23 |
*** ccamacho|lunch is now known as ccamacho | 13:26 | |
lucasagomes | dtantsur, vsaienk0 fix for xenial works too (/me tested locally) | 13:27 |
dtantsur | lucasagomes, \o/ | 13:28 |
openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: Install grub in the EFI "Fallback path" https://review.openstack.org/384996 | 13:28 |
derekh | lucasagomes: so ^ is what I've been using to use UEFI with OVB, as far as I can see any time the virvirt domain is restarted the NVRAM gets reset | 13:29 |
mariojv | good morning ironic | 13:29 |
lucasagomes | derekh, hi there, will take a look | 13:30 |
lucasagomes | derekh, does nova sets the <nvram> in the libvirt domain xml with the absolute path to the nvram file ? | 13:31 |
derekh | lucasagomes: installing grub at the fallback path means the boot will succeed even if there are no image specific entries in the nvram | 13:31 |
derekh | lucasagomes: <nvram>/var/lib/libvirt/qemu/nvram/instance-0000001e_VARS.fd</nvram> | 13:32 |
*** amotoki has quit IRC | 13:32 | |
lucasagomes | derekh, right, yeah that seems to be the one libvirt created because none was specified at the beggining | 13:33 |
derekh | lucasagomes: but AIUI nova doesn't put it there, libvirt does as it wasn't specifiedf | 13:33 |
*** sabeen has joined #openstack-ironic | 13:33 | |
lucasagomes | derekh, yeah. Some firmwares (including edk2) ships their own nvram. Maybe nova should allow passing those too | 13:33 |
lucasagomes | to avoid creating an empty one all the time | 13:34 |
lucasagomes | derekh, anyway, if that code in IPA works for you. I think it's fine | 13:34 |
lucasagomes | derekh, needs unittest tho | 13:34 |
*** mgoddard has joined #openstack-ironic | 13:34 | |
derekh | lucasagomes: but even if it was specified I think it would get reset anyways, the master "/usr/share/OVMF/OVMF_VARS.fd" is copied on domain start | 13:34 |
dtantsur | yeah, left comments about unit tests, release notes and a bug report for that | 13:35 |
derekh | lucasagomes: yup, its working for me, with a centos IPA image booting the tripleo overcloud image on OVB | 13:35 |
lucasagomes | derekh, cool. Yeah you can check the vars at /sys/firmware/efi (or efivars command) | 13:36 |
lucasagomes | derekh, good stuff btw! Thanks for looking into it | 13:36 |
dtantsur | derekh, btw, how does OVB work? I guess it uses regular openstack credentials for power management, but when does it spawn the fake bare metal? | 13:37 |
dtantsur | and how does it detect flavor, etc? | 13:37 |
derekh | lucasagomes: dtantsur will sort out the unit test etc... now | 13:38 |
derekh | dtantsur: on the hosting cloud you precreate baremetal nodes and a fake bmc , you then register these with ironic | 13:39 |
* jroll steps away for a while | 13:39 | |
derekh | dtantsur: ironic runs ipmitool as usual and in the fake bmc these are translated into nova commands | 13:39 |
dtantsur | derekh, a fake bmc -> is it set up per VM? | 13:39 |
dtantsur | I wonder if creating a fake bm requires admin rights on the cloud | 13:40 |
derekh | dtantsur: no, 1 fake bmc for the users test env, the bmc has 1 IP per baremetal node | 13:40 |
sambetts | I'm confused why you need to fake baremetal for tripleO on VMs, I thought that the point was that TripleO just used Heat Templates and Puppet, so it went through nova and didn't matter what hypervisor was under nova?? | 13:40 |
dtantsur | sambetts, this is for testing on vms | 13:40 |
dtantsur | to avoid nested virt | 13:40 |
*** amotoki has joined #openstack-ironic | 13:41 | |
dtantsur | sambetts, OVB implies that you use openstack vms as fake BM (similar to devstack, but without nested virt) | 13:41 |
dtantsur | derekh, so, in case of tripleo, do you run fake bmc on undercloud or virt host? | 13:41 |
derekh | dtantsur: on the underlying nova cloud | 13:42 |
derekh | dtantsur: e.g. this is my test env at the moment http://paste.openstack.org/show/585323/ | 13:43 |
sambetts | I don't get why you need to fake baremetal though, can't you test the heat templates and puppet part just straight onto the public cloud? using the public cloud as your undercloud? | 13:43 |
derekh | dtantsur: and undercloud VM, this talks to "bmc" to control the power state of the baremetal_* instances | 13:43 |
dtantsur | sambetts, these parts - yes, but not ironic | 13:43 |
dtantsur | derekh, ah, so "bmc" is a separate vm? I wonder if this can be avoided, as it will consume more quota.. | 13:44 |
derekh | sambetts: ya, on a public cloud you can't pxe boot from ironic | 13:44 |
derekh | dtantsur: ya, its a seperate VM but on 512MB of RAM and it can control a lot more the 5 baremetal instances | 13:45 |
sambetts | derekh: you don't have to, the public cloud is your undercloud | 13:45 |
sambetts | derekh: you just throw your heat template at it | 13:45 |
dtantsur | sambetts, iirc, there is a tripleo job doing that. but this is not an end-to-end testing, so sometimes we need OVB | 13:46 |
*** dprince has quit IRC | 13:47 | |
*** dprince has joined #openstack-ironic | 13:47 | |
* mgould thought the point was to test the bits of tripleo that call Ironic, eg `openstack baremetal introspection bulk start` | 13:48 | |
mgould | for which you need something approximating baremetal | 13:49 |
mgould | also the bits of tripleo installing ironic | 13:49 |
mgould | what have I misunderstood? | 13:49 |
*** amotoki has quit IRC | 13:50 | |
dtantsur | this is correct | 13:50 |
mgould | oh good :-) | 13:50 |
derekh | without OVB we can't test Ironic, the images, neutron on the undercloud, node registration, introspection, the deploy command and probably a couple of other things I can't think of | 13:52 |
* mgould nods - thanks! | 13:54 | |
*** adam_g has quit IRC | 13:54 | |
*** jcoufal_ has joined #openstack-ironic | 13:54 | |
sambetts | derekh: so this is actually OpenStack on OpenStack on OpenStack, I read TripleO on OpenStack and assumed that this was for running OpenStack on Openstack public cloud | 13:54 |
sambetts | TripleO on public cloud* | 13:55 |
mgould | sambetts: it used to be called QuintupleO | 13:55 |
mat128 | just use bare metal cloud instances and remove one virtualization layer :) | 13:55 |
derekh | sambetts: yes, its for testing tripleo end to end on a cloud | 13:56 |
derekh | mat128: if only we had enough HW ;-( | 13:56 |
sambetts | yeah, I understand now, "TripleO on Public OpenStack" just doesn't sould to me like you would be running a second undercloud | 13:57 |
mat128 | derekh: shameless plug: http://www.internap.com/bare-metal/ | 13:57 |
mat128 | haha | 13:57 |
mat128 | but I totally understand | 13:57 |
sambetts | s/sould/sound | 13:57 |
*** jcoufal has quit IRC | 13:57 | |
sambetts | mgould: haha nice | 13:57 |
mgould | mat128: I think you still have the under^2 cloud in that situation, it's just providing BM rather than VM | 13:58 |
mat128 | mgould: yup | 13:58 |
mgould | under^2 cloud provides BM nodes to the undercloud, which installs OpenStack services on them to create the overcloud | 13:58 |
mgould | turtles all the way down, etc | 13:59 |
sambetts | *mind splodes* | 13:59 |
mat128 | there must be a way to directly provide IPMI :) | 13:59 |
mat128 | OH | 13:59 |
mat128 | change VirtualBMC so that it doesnt talk to libvirt but to nova | 14:00 |
mat128 | bam, you can instantly use any cloud :) | 14:00 |
mgould | cunning | 14:00 |
dtantsur | mat128, https://github.com/cybertron/openstack-virtual-baremetal | 14:00 |
*** anush_ has joined #openstack-ironic | 14:00 | |
sambetts | mat128: as I understand it thats what ^ does | 14:01 |
*** absubram has joined #openstack-ironic | 14:01 | |
derekh | mat128 yup thats what we do | 14:01 |
dtantsur | mat128, we were talking about using a bare metal cloud several times.. but seems that there is no agreement on that | 14:01 |
dtantsur | e.g. infra wants at least 2 providers before they enable something like that | 14:01 |
*** amotoki has joined #openstack-ironic | 14:02 | |
derekh | e.g. I've used ironic too deploy a 81 node overcloud hosted on a nova cloud | 14:02 |
vsaienk0 | sambetts: regarding your comments here: https://review.openstack.org/#/c/384652/10/devstack/settings we can move PUBLIC_BRIDGE_MTU to project config in future | 14:03 |
patchbot | patch 384652 - ironic - Fix broken xenial job | 14:03 |
*** e0ne has quit IRC | 14:03 | |
* mgould notes that there's still a "deploy-quintupleo" script in OVB | 14:03 | |
*** absubram_ has joined #openstack-ironic | 14:03 | |
mat128 | dtantsur, sambetts, derekh thats interesting! | 14:04 |
mgould | ah, and here's the IPMI-to-nova translator: https://github.com/cybertron/openstack-virtual-baremetal/blob/master/openstack_virtual_baremetal/openstackbmc.py | 14:04 |
*** absubram has quit IRC | 14:05 | |
*** absubram_ is now known as absubram | 14:05 | |
*** e0ne has joined #openstack-ironic | 14:06 | |
sambetts | vsaienk0: can we not just put the logic into setup_network.sh script? | 14:06 |
sambetts | vsaienk0: and not override the neutron one | 14:07 |
*** e0ne has quit IRC | 14:07 | |
*** rcernin has quit IRC | 14:07 | |
vsaienk0 | sambetts: but overriding mtu in setup_network.sh will override the value that user set in local.conf | 14:08 |
sambetts | vsaienk0: I'm talking about using a local variable only for that script | 14:09 |
sambetts | vsaienk0: so that it doesn't affect the rest of the devstack process | 14:09 |
*** anush_ has quit IRC | 14:10 | |
vsaienk0 | sambetts: so if we update mtu on external bridge in setup_network, and user set PUBLIC_BRIDGE_MTU to something else our 'ironic' settings will be still applied what is the benefit setting PUBLIC_BRIDGE_MTU in local.conf that doesn't affect? | 14:11 |
vsaienk0 | that is overridden by value set in setup_network? | 14:12 |
vsaienk0 | all that is PUBLIC_BRIDGE_MTU variable doing is set mtu on external bridge https://github.com/openstack-dev/devstack/search?utf8=%E2%9C%93&q=PUBLIC_BRIDGE_MTU | 14:13 |
vsaienk0 | sambetts: it seems there is only one way to allow user customize MTU is to set it in project config and update all documentation | 14:15 |
openstackgerrit | Merged openstack/ironic: Remove reservation from sync power states db filter https://review.openstack.org/384576 | 14:16 |
mariojv | yuriyz: i agree that the except clause in https://review.openstack.org/#/c/321865/39/ironic/conductor/notification_utils.py is mainly just for information. there's actually a comment about it on the line below too which will help indicate this | 14:17 |
patchbot | patch 321865 - ironic - Add power state change notifications | 14:17 |
mariojv | i'm about to push a new patch set which cleans up the if statement xu noticed in conductor/manager.py, too | 14:18 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 14:19 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 14:20 |
*** Guest96386 has quit IRC | 14:22 | |
*** dnikishov has joined #openstack-ironic | 14:22 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Help a user to enable console redirection https://review.openstack.org/384578 | 14:22 |
sambetts | vsaienk0: how about adding some logic so that ironic will only override that variable if that variable is set to the default from the neutron | 14:25 |
sambetts | side | 14:25 |
sambetts | e.g. if $PUBLIC_BRIDGE_MTU == 1500: $PUBLIC_BRIDGE_MTU=<new logic> | 14:26 |
mariojv | vdrok: what do you think the imports in https://review.openstack.org/#/c/321865/39/ironic/conductor/notification_utils.py should be named? i was thinking oslo_msg_exc for oslo_messaging.exceptions | 14:27 |
patchbot | patch 321865 - ironic - Add power state change notifications | 14:27 |
vsaienk0 | sambetts: yes it is possible but it will not work when user set mtu to 1500 explicitly | 14:27 |
mariojv | vdrok: and ovo_exc for oslo_versionedobjects.exceptions | 14:27 |
mariojv | will that work? | 14:27 |
vdrok | mariojv: whatever you like, it just should be something :) | 14:27 |
mariojv | cool | 14:27 |
mariojv | maybe oslo_vo_exc to be consistent | 14:27 |
vdrok | works for me, thanks | 14:27 |
*** bjolo_ has quit IRC | 14:28 | |
mariojv | rloo: vdrok: yuriyz: it's 2 votes vs. 2 votes for leaving the specific exceptions in here: https://review.openstack.org/#/c/321865/39/ironic/conductor/notification_utils.py | 14:30 |
patchbot | patch 321865 - ironic - Add power state change notifications | 14:30 |
*** hamzy has joined #openstack-ironic | 14:30 | |
mariojv | openstack guidelines generally dislike catching a generic exception | 14:30 |
*** anush_ has joined #openstack-ironic | 14:31 | |
mariojv | i'm flexible for changing it though if we have consensus on something | 14:31 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification https://review.openstack.org/348437 | 14:32 |
rloo | mariojv: I like having it there because yes, 1. we do not like capturing 'any' Exceptions; 2. but we do that in certain cases where not capturing it could cause the system to go down or other bad things. | 14:32 |
mariojv | i tend to agree. yuriyz vdrok what do you find amiss about what rloo said above? ^ | 14:32 |
rloo | mariojv: in general, i don't like seeing just 'Exception'. but maybe it is just a coding thing. | 14:32 |
vdrok | mariojv: rloo if we say in the docs that custom notifications may only use base notification classes defined in your patch, I think we can only state tehm | 14:33 |
vdrok | and morning mariojv and rloo ! :) | 14:33 |
mariojv | morning | 14:33 |
mariojv | it took a non-trivial effort (~5-10 minutes) to find all of those notification exceptions, fwiw | 14:33 |
mariojv | and i wrote the code :) | 14:34 |
vdrok | mariojv: well, you've documented them now right? :) | 14:34 |
vdrok | so they are more visible | 14:34 |
mariojv | they were already documented; i just meant for tracking down what could occur in this particular place | 14:34 |
rloo | hi and morning everyone :) mariojv, vdrok, yuriyz, vsaienk0, sambetts, mgould, mat128, dtantsur, lucasagomes, jroll, and EVERYONE ELSE! :D | 14:34 |
lucasagomes | rloo, hi there | 14:34 |
vsaienk0 | morning rloo | 14:35 |
vdrok | and morning lucasagomes and mat128 :) | 14:35 |
mariojv | rloo: do you know if there are other places in the ironic codebase that have this style of exception handling? maybe that would help | 14:35 |
mariojv | i can poke around, too | 14:35 |
rloo | vdrok: what do you mean by 'documented'? I guess we're trying to figure out what to do about https://review.openstack.org/#/c/321865/39/ironic/conductor/notification_utils.py, line 54 ish | 14:35 |
patchbot | patch 321865 - ironic - Add power state change notifications | 14:35 |
rloo | mariojv: no, which i think is a shame. | 14:35 |
rloo | mariojv: not exactly like that, we have try except <expected exceptions>:... except Exception:... | 14:36 |
mariojv | ok - the only difference here is that we want to have similar messages for both cases, so we merge them | 14:36 |
mariojv | to decrease number of lines | 14:36 |
mariojv | so i think this is consistent with those other places in ironic | 14:36 |
rloo | mariojv: right. so could go back to what you had before. with 'Unknown' or 'Unexpected exception' for the general Exception case. | 14:37 |
mariojv | if vdrok and yuriyz are more comfortable with that, i'm willing to do it | 14:37 |
mariojv | i think the argument was more about that generic Exception will catch everything, but imbw | 14:37 |
vdrok | mariojv: so, again, is this something we should consider? that people will overload thing like emit and populate schema? | 14:38 |
jlvillal | Good morning Ironic | 14:38 |
vdrok | morning jlvillal | 14:38 |
jlvillal | vdrok: :) | 14:38 |
jlvillal | vdrok: How goes the presentation? Is your section ready? I have to do some work on it this morning. | 14:39 |
mariojv | vdrok: i don't think people will want to override emit, maybe populate_schema. i think this is more for when you're writing a new notification and it's throwing errors, it points you to whether that error can be expected during the dev process | 14:39 |
jroll | morning rloo :) | 14:39 |
vdrok | rloo: by documented I mean the possible exceptions are stated in the object.notifications classes | 14:39 |
vsaienk0 | jroll, dtantsur, sambetts, lucasagomes: do you think we should update PUBLIC_BRIDGE_MTU only when it is set to default neutron value and do not touch for other cases? | 14:39 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for node list and show with specific fields https://review.openstack.org/384923 | 14:39 |
jroll | vsaienk0: I don't have a reason to say yes or no | 14:40 |
lucasagomes | vsaienk0, I'm in a meeting atm. I think I need to read some to understand the implications | 14:41 |
rloo | morning jlvillal | 14:41 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 14:41 |
vsaienk0 | morning jlvillal | 14:41 |
jlvillal | rloo: vsaienk0 :) | 14:41 |
sambetts | it worries me when we take the control away from the local.conf, local.conf should be all power IMO | 14:41 |
vdrok | jlvillal: I was too confident about that last week, not yet :( I'm planning to finish it this evening | 14:41 |
jlvillal | vsaienk0: If you get some free time. https://review.openstack.org/370492 Your input would be appreciated. | 14:42 |
patchbot | patch 370492 - ironic - Increase verbosity of devstack/lib/ironic | 14:42 |
jlvillal | vdrok: Totally understand! | 14:42 |
mariojv | rloo: yuriyz: vdrok: i updated with interleaving the comment for now, hopefully that is a good compromise | 14:42 |
*** sdake_ is now known as sdake | 14:42 | |
mariojv | i view putting the expected exceptions there as extra documentation | 14:42 |
vdrok | mariojv: I really don't know what's the best way, I just usually rely on docstrings rather than on code | 14:42 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone capability to base install https://review.openstack.org/385040 | 14:43 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add initial keystone use documentation https://review.openstack.org/385041 | 14:43 |
mariojv | i think it just takes extra time to go through all the possible call trees and figure out which docstrings apply to that case | 14:43 |
mariojv | i usually rely on those, too | 14:44 |
mariojv | maybe someone else will review and have a deciding vote :) | 14:44 |
*** mkam has joined #openstack-ironic | 14:45 | |
*** mjura has quit IRC | 14:45 | |
*** rajinir has joined #openstack-ironic | 14:46 | |
rloo | jroll: was there an ironic meeting yesterday? I don't see any meeting logs | 14:46 |
vdrok | mariojv: ok, after all that was said, I think it's not worth blocking because of that :) | 14:46 |
mariojv | rloo: meeting bot was down | 14:46 |
rloo | mariojv: :-( | 14:46 |
mariojv | thanks vdrok! i appreciate the discussion | 14:46 |
jroll | rloo: I could paste logs for you | 14:47 |
*** rbrud has joined #openstack-ironic | 14:47 | |
* jroll will send to the list | 14:47 | |
rloo | jroll: that would be good, thx. | 14:47 |
sambetts | vsaienk0: IMO as brbm is a special bridge, we should be safe to set the MTU based on the local_mtu logic in the setup-network.sh, because local_mtu should always end up smaller or equal to PUBLIC_BRIDGE_MTU | 14:48 |
jroll | oh my, weechat's log format is terrible, oh well | 14:48 |
vsaienk0 | sambetts: so you propose to set mtu in setup_network without overriding PUBLIC_BRIDGE_MTU variable? | 14:50 |
sambetts | vsaienk0: yes | 14:50 |
jroll | rloo: sent | 14:50 |
rloo | thx jroll! | 14:51 |
jroll | np :) | 14:51 |
vsaienk0 | sambetts: what is the benefit to set it directly without overriding? | 14:52 |
sambetts | vsaienk0: the user maintains control over real OpenStack stuff via local.conf, and magic ironic stuff doesn't get in the way of that | 14:52 |
*** mkam has quit IRC | 14:53 | |
*** caiobo has joined #openstack-ironic | 14:54 | |
vsaienk0 | sambetts: but if we set mtu directly, it doesn't matter if user set PUBLIC_BRIDGE_MTU in local.conf or not. Ironic's value will take affect anyway. I thought that settings file was created specially for this case. | 14:54 |
*** mkam has joined #openstack-ironic | 14:54 | |
sambetts | vsaienk0: that depends on if any other devstack scripts want to use PUBLIC_BRIDGE_MTU after the ironic devstack script runs for what ever reason | 14:55 |
*** hoangcx has quit IRC | 14:56 | |
*** hoangcx has joined #openstack-ironic | 14:56 | |
vsaienk0 | sambetts: but if we change actual mtu value other scripts will be deceived, because in local.conf PUBLIC_RDIGE_MTU is not the same as on physycal interface | 14:57 |
sambetts | vsaienk0: brbm isn't used by anyone else is it, its a magic ironic thing that shouldn't really exist | 14:58 |
vsaienk0 | sambetts: we should override mtu on br-ex also not only on brbm | 14:59 |
vsaienk0 | sambetts: setting mtu on brbm only will not work | 14:59 |
rloo | jroll: i feel like I should object to the selection of design summits, just-to-do-it. Except that I am fine with it. Thx all for the decisions there :) | 15:01 |
jroll | rloo: :) | 15:01 |
sambetts | vsaienk0: can we not do something like ip link show $PUBLIC_BRIDGE | sed mtu ... and get the real public bridge MTU as set by neutron ? | 15:04 |
*** caiobo is now known as caiobo-lunch | 15:05 | |
*** jtomasek is now known as jtomasek|bbl | 15:05 | |
vsaienk0 | sambetts: you mean this line? https://review.openstack.org/#/c/384652/10/devstack/settings@10 | 15:06 |
patchbot | patch 384652 - ironic - Fix broken xenial job | 15:06 |
sambetts | vsaienk0: that same logic but specifical get it for the $PUBLIC_BRDIGE | 15:07 |
sambetts | specifically* | 15:07 |
vsaienk0 | sambetts: but the idea is to set mtu on br-ex to the smalest mtu among all interfaces | 15:07 |
*** makowals has quit IRC | 15:08 | |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Add script to help build rpms https://review.openstack.org/382044 | 15:08 |
jcook | JayF reading up on the rescue spec | 15:08 |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Check for buffer and use memoryview if missing https://review.openstack.org/385051 | 15:09 |
vsaienk0 | sambetts: this logic is used by devstack-gate: https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L57 why we need other logic? | 15:09 |
*** syed_ has joined #openstack-ironic | 15:11 | |
sambetts | vsaienk0: devstack-gate creates local.conf so simulates user input via local.conf, doing this sort of thing in the Ironic devstack plugin is quick to merge but undermines the local.conf | 15:12 |
vsaienk0 | sambetts: so the proper fix is to set it in local.conf via project-config and update documentation | 15:15 |
*** [1]cdearborn has joined #openstack-ironic | 15:16 | |
jcook | JayF why do we inject a password for rescue mode instead of a pub key into authorized keys? | 15:17 |
sambetts | so we need br-ex and brbm to match MTUs right? | 15:20 |
sambetts | vsaienk0: ^ | 15:20 |
JayF | jcook: because passing a password back from the nova rescue call is the nova api | 15:21 |
*** rpioso has joined #openstack-ironic | 15:22 | |
jcook | JayF gotcha, so using ssh keys would be a new nova feature? | 15:22 |
JayF | jcook: also there's no way after an instance has booted to change the pubkey, so for example, my public key is compromised. Later, I need to rescue a long-running-cloud-instance that was originally booted with my pubkey. Clearly I don't want that compromised key tossed into the instance. | 15:22 |
JayF | jcook: yeah; and honestly, in this case, we're just a virt driver implementing their API. I'd strongly suggest sticking to their API and if we wanna change it later, to do that separately | 15:22 |
JayF | jcook: otherwise no chance whatsoever of landing it in ocata :D | 15:22 |
JayF | jcook: I'd be happy to spend some time going over what exists today with you, if you want? | 15:23 |
jcook | JayF Makes sense. I was thinking that you'd specify a key similar to a boot | 15:24 |
jcook | JayF but that's probably not supported in nova at all | 15:24 |
jcook | never tried tbh | 15:24 |
JayF | the api doesn't have any mechanism for sending in a pub... yep | 15:24 |
jcook | gotcha | 15:24 |
jcook | yeah, totes on going over it | 15:24 |
JayF | cool; after rackerpulse sync? | 15:24 |
*** jistr is now known as jistr|biab | 15:25 | |
JayF | well, that's lunchtime for you. I'm free anytime after that, except 2-230 pst | 15:25 |
jcook | JayF I'm blocked for rest of week lol, my Fridays are work days so if not ad hoc sooner, maybe then | 15:28 |
JayF | I doubt it'd take long. Just ping me if you have 15m | 15:29 |
jcook | JayF sure | 15:29 |
*** wajdi has joined #openstack-ironic | 15:29 | |
jroll | JayF: we should get the nova bp filed for this and make sure we get the ironic bits done very soon | 15:30 |
jroll | if we want to land nova this cycle | 15:30 |
JayF | jroll: nova bp already exists | 15:30 |
jroll | is approved? | 15:30 |
JayF | jroll: jcook volunteered to pick up ironic side bits | 15:30 |
JayF | jroll: yesa | 15:31 |
jroll | \o/ | 15:31 |
JayF | jroll: unless I have to redo that for O? Last time I asked I was more or less told JFDI | 15:31 |
jroll | approved for ocata even? | 15:31 |
JayF | It might have even been said I don't need a bp? I think that's right | 15:31 |
jroll | I think you may need a quick re-approval | 15:31 |
JayF | virt driver implementing existing api = no bp | 15:31 |
*** cdearborn has quit IRC | 15:31 | |
jroll | no, every feature needs a BP | 15:31 |
*** pcaruana has quit IRC | 15:31 | |
jroll | specless though, for this case | 15:31 |
dtantsur | morning rloo, JayF | 15:32 |
jroll | idk, I guess you could have been told differently | 15:32 |
jcook | yeah, I'm gonna hack on it. My next two weeks are lousy with meetings though. Was going to spend my Fridays on it most likely + mornings when I can | 15:32 |
*** lucasagomes is now known as lucas-hungry | 15:32 | |
rloo | hi dtantsur :) | 15:32 |
jroll | sambetts: do you use a different MTU today? | 15:34 |
rloo | dtantsur: you want me to +A this? https://review.openstack.org/#/c/382794/ | 15:35 |
patchbot | patch 382794 - ironic - Log node state transitions at INFO level | 15:35 |
dtantsur | rloo, I definitely won't be against it :) | 15:36 |
rloo | dtantsur: heh. done! | 15:36 |
sambetts | jroll: no, but I already know the pain of having local.conf control taken away by the SUBNETPOOL fix we put into the settings file | 15:36 |
jroll | sambetts: would you be fine with a todo then? | 15:37 |
sambetts | jroll: If there are already plans to fix it elsewhere then probably, I just wonder if it would be solved by getting the MTU directly from br-ex via ip link show as they need to matc, e.g. brbm mtu = `ip link show br-ex <sed the mtu out>` because the neutron plugin will set the MTU on br-ex, and then we can just steal it off it | 15:40 |
jroll | sambetts: yeah idk | 15:40 |
jroll | whatever you and vsaienk0 think is best, I just want to unblock this | 15:40 |
JayF | sambetts: that's exactly what I was thinking, just now looking at the patch | 15:40 |
*** vsaienko has quit IRC | 15:41 | |
*** _vsaienko has joined #openstack-ironic | 15:41 | |
JayF | sambetts: we should have all the info we need in lib/ironic to enforce a maximum mtu regardless of settings | 15:41 |
*** mkam has quit IRC | 15:43 | |
sambetts | vsaienk0, JayF: something like: pub_br_mtu=$(ip link show $PUBLIC_BRIDGE | sed -ne 's/.*mtu \([0-9]\+\).*/\1/p') | 15:45 |
sambetts | then use sudo ip link set dev ${BRIDGE_NAME} mtu $pub_br_mtt | 15:46 |
sambetts | mtu* | 15:46 |
*** e0ne has joined #openstack-ironic | 15:51 | |
*** wajdi has quit IRC | 15:53 | |
*** links has quit IRC | 15:54 | |
*** caiobo-lunch has quit IRC | 15:54 | |
*** mkam has joined #openstack-ironic | 15:55 | |
rajinir | sambetts: Patch soon to be merged to fix the subet creation issue in the CI's https://review.openstack.org/#/c/378063/ | 16:01 |
patchbot | patch 378063 - openstack-dev/devstack - Neutron L3 subnetpool creation should be optional | 16:01 |
sambetts | rajinir: Yeah I saw :D thanks for pushing on that | 16:01 |
*** joprovost has joined #openstack-ironic | 16:04 | |
*** milan is now known as milan|afk | 16:07 | |
*** milan|afk has quit IRC | 16:07 | |
*** Romanenko_K has quit IRC | 16:07 | |
*** caiobo-lunch has joined #openstack-ironic | 16:07 | |
*** jistr|biab is now known as jistr | 16:07 | |
*** rpioso has quit IRC | 16:09 | |
jlvillal | vsaienk0: Are you still around? Curious if you had the script you use to run the multi-node job locally? | 16:11 |
jlvillal | vsaienk0: We are trying to do it here :) | 16:11 |
rloo | jroll: just found out who lost and are now on the TC. congrats on staying focussed on ironic! :D | 16:12 |
jroll | rloo: heh. thanks. | 16:12 |
*** hoangcx has quit IRC | 16:15 | |
*** yuriyz|2 has joined #openstack-ironic | 16:16 | |
mgould | sambetts: your 1640BST post is the kind of thing I meant by "query neutron for the MTU size" :-) | 16:21 |
sambetts | mgould: awesome :) | 16:21 |
yuriyz|2 | hi ironicers. rloo mariojv some time ago I create small notifications-related rfe https://bugs.launchpad.net/ironic/+bug/163044 what you think about? | 16:23 |
openstack | Launchpad bug 163044 in xserver-xorg-video-ati (Ubuntu) "[M6] terminal display error when reading PDF files" [Low,Fix released] - Assigned to Rolf Leggewie (r0lf) | 16:23 |
yuriyz|2 | sorry https://bugs.launchpad.net/ironic/+bug/1630442 | 16:24 |
openstack | Launchpad bug 1630442 in Ironic "[RFE] FSM event for skipping automatic cleaning" [Undecided,New] - Assigned to Varun Gadiraju (varun-gadiraju) | 16:24 |
mgould | sambetts: I was just missing the crucial *how* bit :-) | 16:25 |
NobodyCam | Good Morning Ironic'ers :) | 16:25 |
mgould | morning NobodyCam! | 16:25 |
NobodyCam | morning mgould :) | 16:25 |
yuriyz|2 | morning NobodyCam | 16:25 |
*** athomas has quit IRC | 16:25 | |
rloo | morning NobodyCam! | 16:25 |
sambetts | mgould: yeah, I only thought about it when I saw the local_mtu logic in the patch | 16:25 |
NobodyCam | morning yuriyz|2, rloo and sambetts | 16:25 |
sambetts | Hey NobodyCam | 16:25 |
NobodyCam | :) | 16:26 |
rloo | yuriyz|2: I'll take a look today | 16:26 |
yuriyz|2 | thanks rloo it's very simple | 16:26 |
mariojv | looking today also | 16:27 |
rloo | yuriyz|2: i am always afraid when someone pings me and says something is 'simple' :-) | 16:27 |
yuriyz|2 | thanks mariojv | 16:27 |
*** jpich has quit IRC | 16:27 | |
yuriyz|2 | rloo it's not about me | 16:27 |
yuriyz|2 | :) | 16:28 |
rloo | yuriyz|2: there was something else of yours that mariojv wanted me to look at too... so much to do, so little time... | 16:28 |
mgould | rloo: "simple" does not necessarily mean "small" - see https://en.wikipedia.org/wiki/Monster_group :-) | 16:29 |
rloo | mgould: OH, that explains it! ;) | 16:29 |
yuriyz|2 | rloo this point is not important now | 16:29 |
rloo | yuriyz|2: which point? No need to look at that rfe then? | 16:29 |
dtantsur | g'night | 16:31 |
*** dtantsur is now known as dtantsur|afk | 16:31 | |
mgould | night dtantsur|afk | 16:31 |
yuriyz|2 | rloo it can be done later provision notifications should works w/o implementing of this rfe | 16:31 |
rloo | yuriyz|2: ah, got it. thx. | 16:31 |
*** jaosorior has quit IRC | 16:33 | |
*** lucas-hungry is now known as lucasagomes | 16:34 | |
*** hoonetorg has quit IRC | 16:35 | |
*** baoli_ has quit IRC | 16:38 | |
*** amotoki has quit IRC | 16:42 | |
_vsaienko | jroll: tests passed please approve https://review.openstack.org/#/c/384652 and https://review.openstack.org/#/c/365116 | 16:43 |
patchbot | patch 384652 - ironic - Fix broken xenial job | 16:43 |
patchbot | patch 365116 - ironic - Fix setting custom IRONIC_VM_NETWORK_BRIDGE | 16:43 |
jroll | sambetts: you good with me approving that and we can follow up with something better later? | 16:46 |
*** derekh has quit IRC | 16:46 | |
sambetts | jroll: go for it, and if anyone blows up we'll soon know about it | 16:47 |
sambetts | ;) | 16:47 |
jroll | :D | 16:47 |
openstackgerrit | Julia Kreger proposed openstack/ironic-specs: Boot from Volume - Reference Drivers https://review.openstack.org/294995 | 16:47 |
*** raghusab has quit IRC | 16:47 | |
*** hoonetorg has joined #openstack-ironic | 16:48 | |
*** tomstatt has joined #openstack-ironic | 16:56 | |
*** yuriyz|2 has quit IRC | 16:57 | |
* sambetts -> home | 17:10 | |
sambetts | cya'll later | 17:10 |
*** sambetts is now known as sambetts|afk | 17:10 | |
tomstatt | Hi all, I followed guide here: http://docs.openstack.org/developer/ironic/drivers/vbox.html to deploy ironic for vbox driver but got some error while running stack.sh | 17:11 |
tomstatt | the log is here: http://paste.openstack.org/show/585359/ | 17:12 |
tomstatt | I have changed cpu of compute node from 2 cores to 4 cores and run again but still got this error. | 17:12 |
openstackgerrit | Merged openstack/ironic: Log node state transitions at INFO level https://review.openstack.org/382794 | 17:13 |
tomstatt | please give me some suggestion | 17:14 |
*** xavierr has quit IRC | 17:15 | |
*** ricardoas has quit IRC | 17:15 | |
*** clenimar has quit IRC | 17:16 | |
*** gabriel-bezerra has quit IRC | 17:16 | |
*** charlledaniel has quit IRC | 17:16 | |
*** mtanino has joined #openstack-ironic | 17:16 | |
*** mrtenio has quit IRC | 17:16 | |
mgould | tomstatt: I've never used those instructions, sorry | 17:19 |
mgould | but it looks like there aren't *any* vcpus available | 17:19 |
mgould | so increasing the size of your controller won't help | 17:20 |
* mgould could be wrong, of course | 17:20 | |
lucasagomes | tomstatt, before you run ./unstack.sh can you run "ironic node-list/show" ? | 17:20 |
tomstatt | I have run lscpu command and I got 4 cores in CPU | 17:20 |
mgould | yeah, that suggests to me that those cores aren't being made available to nova | 17:21 |
mgould | for whatever reason | 17:21 |
mgould | but yes, please post the output of `ironic node-list` | 17:21 |
*** hshiina has quit IRC | 17:22 | |
*** rpioso has joined #openstack-ironic | 17:23 | |
tomstatt | lucasagomes, mgould : here is the result for "ironic node-list" | 17:24 |
tomstatt | http://paste.openstack.org/show/585366/ | 17:24 |
*** xavierr has joined #openstack-ironic | 17:25 | |
lucasagomes | tomstatt, please do a "ironic node-show <uuid/name>", see the "properties" field | 17:25 |
*** clenimar has joined #openstack-ironic | 17:25 | |
lucasagomes | it should have a "cpus" key there | 17:25 |
lucasagomes | that should be propagated to nova and will appear at "nova hypervisor-stats" | 17:25 |
tomstatt | here is result for "ironic node-show <uuid/name>" | 17:27 |
tomstatt | http://paste.openstack.org/show/585367/ | 17:27 |
*** gabriel-bezerra has joined #openstack-ironic | 17:27 | |
*** charlledaniel has joined #openstack-ironic | 17:27 | |
*** mrtenio has joined #openstack-ironic | 17:27 | |
mgould | interesting | 17:28 |
*** ohamada_ has quit IRC | 17:28 | |
mgould | and `nova hypervisor-stats`? | 17:28 |
tomstatt | result for 'nova hypervisor-stats': http://paste.openstack.org/show/585368/ | 17:28 |
*** ralonsoh has quit IRC | 17:29 | |
*** ricardoas has joined #openstack-ironic | 17:29 | |
mgould | huh | 17:30 |
mgould | lucasagomes: any ideas? | 17:30 |
* mgould has to go home now, but wishes you luck | 17:30 | |
*** mgould is now known as mgould|afk | 17:30 | |
tomstatt | thank you mgould | 17:30 |
lucasagomes | tomstatt, I can see driver_info is using ssh_* options | 17:32 |
lucasagomes | I bet your validation (ironic node-validate <node uuid>) is failing | 17:32 |
lucasagomes | and that's the cause | 17:32 |
*** baoli has joined #openstack-ironic | 17:33 | |
lucasagomes | tomstatt, can you check ironic node-validate command please ? | 17:33 |
tomstatt | lucasagomes: in my local.conf file I have search and see no ssh driver | 17:33 |
*** trown is now known as trown|lunch | 17:34 | |
lucasagomes | tomstatt, right, I mean in the node-show output (http://paste.openstack.org/show/585367/) | 17:35 |
*** portdirect has joined #openstack-ironic | 17:35 | |
JayF | rloo: you literally just asked on-list exactly what I was going to present to everyone during next upstream meeting | 17:35 |
lucasagomes | see the "driver_info" field, it looks like it's using the driver's options for the *_ssh driver ? | 17:35 |
JayF | rloo: I'm on the side of "put our admin-guide in tree in dev docs, point back to it until we can have it in-tree" | 17:35 |
rloo | JayF: oh, sorry :) | 17:35 |
pas-ha | does our devstack plugin can even configure the vbox driver correctly? this is not the ssh+vbox, but pure vbox driver | 17:35 |
JayF | rloo: don't apologize, it's a good thing when we agree! I feel like it's a sign we're seeing the world correctly | 17:36 |
rloo | JayF: yeah, I think that'd be the easiest/fastest thing to do | 17:36 |
lucasagomes | tomstatt, you must be stuck here https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L635-L639 | 17:36 |
openstackgerrit | Fabio Dassan proposed openstack/pyghmi: Implement M4 and Kent support https://review.openstack.org/361457 | 17:36 |
rloo | JayF: i replied to that cuz i don't really think that some folks are aware of how much work we have to do if they don't provide support... | 17:36 |
* lucasagomes thinks | 17:36 | |
JayF | I mean, I think we have a bit of a ... project-wide feeling that we don't like doing things outside of tree | 17:37 |
rloo | JayF: the fact that someone added that info to the guide, but didn't coordinate with us, even though they clearly copied from our dev docs... :-( | 17:37 |
JayF | for various reasons, including our explicit exclusion from being in the out-of-tree install guide historically | 17:37 |
pas-ha | lucasagomes: there's not a single mention of "vbox" in our devstack plugin | 17:38 |
rloo | JayF: well, ironic was excluded from the official docs cuz (I think) 'they' didn't have the people-resources to handle that along with everything else they wanted to do. then the tent got bigger... | 17:38 |
*** sabeen has quit IRC | 17:38 | |
pas-ha | so I presume it does not support configuring it appropriately | 17:38 |
rloo | JayF: anyway, moving forward into the future... | 17:39 |
JayF | I'll make sure it gets on the agenda for the next meeting to make sure there's a consensus | 17:39 |
*** ccamacho has quit IRC | 17:39 | |
JayF | and I'll reply to your ML thread, agreeing :) | 17:39 |
lucasagomes | pas-ha, right | 17:39 |
rloo | thx JayF | 17:39 |
tomstatt | lucasagomes: yes, I keep the original local.conf file for ironic devstack, just change driver to pxe_vbox | 17:39 |
tomstatt | lucasagomes: maybe some setting is not correct for pxe_vbox | 17:40 |
lucasagomes | tomstatt, right. Yeah so, like pas-ha said devstack may not have the code to configure that driver correctly | 17:40 |
lucasagomes | and it wouldn't be a surprise | 17:40 |
pas-ha | I think it is slated for removal from in-tree to e.g. ironic-staging-drivers | 17:41 |
tomstatt | here is the result for 'ironic node-validate' command: http://paste.openstack.org/show/585370/ | 17:41 |
lucasagomes | tomstatt, you can figure out what are the required properties for that vbox driver by issuing "ironic driver-properties pxe_vbox" | 17:41 |
*** amotoki has joined #openstack-ironic | 17:42 | |
*** sabeen has joined #openstack-ironic | 17:42 | |
lucasagomes | tomstatt, right, you can see there the power/management interface failing because it's missing some options | 17:42 |
lucasagomes | tomstatt, see "ironic driver-properties pxe_vbox" | 17:42 |
tomstatt | yes, I will try it | 17:43 |
tomstatt | thanks | 17:43 |
lucasagomes | tomstatt, you will need to configure the nodes manually (via ironic node-update) | 17:43 |
tomstatt | lucasagomes: yes, I got it, I am reading property description | 17:44 |
tomstatt | lucasagomes: what will I do next, after I have updated the node? | 17:45 |
lucasagomes | tomstatt, check if they are not in maintenance mode (via ironic node-list) | 17:45 |
lucasagomes | because since the power interface is failing validation it may be put in that mode | 17:45 |
lucasagomes | and then the hypervisor-stats command should now be propagating correctly (it may take few minutes) | 17:46 |
lucasagomes | if you want to speed-up the process, just manually restart the n-cpu service | 17:46 |
*** rama_y has joined #openstack-ironic | 17:46 | |
*** amotoki has quit IRC | 17:47 | |
tomstatt | lucasagomes: I see, thank for your support | 17:49 |
lucasagomes | tomstatt, no worries, I might be missing something too | 17:51 |
lucasagomes | but lets see :-) | 17:51 |
tomstatt | lucasagomes: I see, your debugging helped me a lot, I will follow that | 17:52 |
jroll | devananda: rloo: wanna api meeting? | 17:59 |
rloo | jroll: oh, thought it was every second week. i can if you want. | 18:00 |
pas-ha | soo, deployed myself the devstack-master on Xenial (apt-upgraded to the latest) and using pxe_ipmitool driver. one in two deploy attempts succeded o_O, but now I have two nodes sitting in clean_wait, maintenace off, recving heartbeats from them - but they do nothing | 18:00 |
jroll | rloo: I wasn't here last week but couldn't find evidence of one | 18:00 |
rloo | jroll: we cancelled last week's. i think it gave devananda time to write his specs :) | 18:00 |
jroll | rloo: let's wait for devananda I guess :) | 18:01 |
rloo | jroll: ok, i'm here. just ping me if it is a go. | 18:01 |
pas-ha | and the clean can not be aborted, looks like smth has constant exclusive lock on those nodes | 18:02 |
lucasagomes | pas-ha, is it locked on a "wait" state!? | 18:05 |
pas-ha | yes | 18:05 |
lucasagomes | that's odd, sounds like a bug to me | 18:05 |
JayF | I bet there's something else at play | 18:05 |
JayF | like a slow BMC + power status loop leaving the node locked for an extra long time | 18:05 |
JayF | I've seen that happen in OnMetal before | 18:06 |
pas-ha | JayF: that's devstack with VMs | 18:06 |
pas-ha | and the power status sync keeps trying to get a shared lock for those two nodes | 18:07 |
pas-ha | anyway, will continue tomorrow, have a nice evening all :) | 18:07 |
rloo | pas-ha: is that using iscsii? | 18:08 |
pas-ha | yes | 18:08 |
pas-ha | pxe_ipmitool | 18:08 |
rloo | pas-ha: thought i saw a bug about iscsii being REALLY SLOW | 18:08 |
rloo | was in yesterday's meeting i think... am thinking... | 18:08 |
JayF | on xenial? | 18:08 |
pas-ha | yes | 18:08 |
rloo | yeah. | 18:08 |
JayF | if it's on xenial we know what's up, and have a patch to fix it | 18:08 |
JayF | yeah let me find the link | 18:08 |
rloo | is that the xenial blocker | 18:08 |
JayF | https://review.openstack.org/#/c/384652/ | 18:09 |
patchbot | patch 384652 - ironic - Fix broken xenial job | 18:09 |
JayF | It's all about MTU | 18:09 |
pas-ha | I know, that's why I decided to test it | 18:09 |
*** caiobo-lunch has quit IRC | 18:09 | |
JayF | https://review.openstack.org/#/c/365116/ | 18:09 |
patchbot | patch 365116 - ironic - Fix setting custom IRONIC_VM_NETWORK_BRIDGE | 18:09 |
JayF | are the two patches we merged to fix it, I believe | 18:09 |
pas-ha | wonderfully second deploy attempt succeded | 18:09 |
lucasagomes | jroll, latest version of ironic-lib is 2.2.0 right? (not 2.1.1) | 18:09 |
* lucasagomes is confused by pypi | 18:10 | |
lucasagomes | https://pypi.python.org/pypi/ironic-lib/2.2.0 (is saying Latest Version: 2.1.1 ) | 18:10 |
pas-ha | the "fixing" patch is still in merge queue, so I definitely do not have it | 18:10 |
*** bjolo_ has joined #openstack-ironic | 18:11 | |
rloo | lucasagomes: that doesn't look good :-( | 18:11 |
lucasagomes | rloo, yeah, looks a bit odd | 18:11 |
lucasagomes | rloo, anyway, looking at the changelog 2.2.0 is the version which includes the root device hint stuff and fix the whole disk image + config driver | 18:15 |
lucasagomes | drive* | 18:15 |
lucasagomes | so: | 18:15 |
lucasagomes | https://review.openstack.org/#/c/385115/ | 18:15 |
patchbot | patch 385115 - requirements - Bump ironic-lib version to 2.2.0 | 18:15 |
rloo | lucasagomes: i would think 2.2.0 is the first ocata version; 2.1.1. is from stable/newton. | 18:15 |
openstackgerrit | Merged openstack/ironic: Fix setting custom IRONIC_VM_NETWORK_BRIDGE https://review.openstack.org/365116 | 18:15 |
lucasagomes | rloo, yeah it seems to be it | 18:15 |
rloo | lucasagomes: maybe doug hellman knows? | 18:16 |
rloo | lucasagomes: or someone else in stable infra. me, i dunno. | 18:16 |
* lucasagomes can ask | 18:16 | |
openstackgerrit | Merged openstack/ironic: Fix broken xenial job https://review.openstack.org/384652 | 18:29 |
lucasagomes | o/ | 18:31 |
jroll | lucasagomes: hm, I guess probably that means "most recently uploaded version" | 18:31 |
lucasagomes | jroll, right | 18:31 |
mariojv | yuriyz: took a look at the RFE https://bugs.launchpad.net/ironic/+bug/1630442, it makes sense to me | 18:32 |
openstack | Launchpad bug 1630442 in Ironic "[RFE] FSM event for skipping automatic cleaning" [Undecided,New] - Assigned to Varun Gadiraju (varun-gadiraju) | 18:32 |
lucasagomes | I wonder which version it will take if you do a "pip install" | 18:32 |
lucasagomes | without passing the version | 18:32 |
* lucasagomes tries | 18:32 | |
lucasagomes | "Successfully installed ironic-lib-2.2.0" | 18:33 |
lucasagomes | ok seems good | 18:33 |
*** hoonetorg has quit IRC | 18:35 | |
*** mkam has quit IRC | 18:37 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 18:42 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 18:42 |
*** amotoki has joined #openstack-ironic | 18:43 | |
*** trown|lunch is now known as trown | 18:46 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add support for root device hints with operators https://review.openstack.org/366742 | 18:47 |
*** amotoki has quit IRC | 18:47 | |
*** hoonetorg has joined #openstack-ironic | 18:51 | |
*** mkam has joined #openstack-ironic | 18:51 | |
*** mgoddard has quit IRC | 18:52 | |
openstackgerrit | Fabio Dassan proposed openstack/pyghmi: Implement M4 and Kent support https://review.openstack.org/361457 | 18:53 |
*** mgoddard has joined #openstack-ironic | 19:01 | |
rloo | hey lucasagomes, just wondering if you wanted to change your vote here; there are a few -1's: https://review.openstack.org/#/c/302143/ | 19:07 |
patchbot | patch 302143 - ironic-python-agent - Fix local boot issue with fedora in uefi mode | 19:07 |
lucasagomes | rloo, looking | 19:08 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/377446 | 19:09 |
*** [1]rpioso has joined #openstack-ironic | 19:16 | |
mariojv | JayF: added a comment on this old patch set: https://review.openstack.org/#/c/263842 | 19:17 |
patchbot | patch 263842 - ironic - Make agent driver always PXE before cleaning | 19:17 |
mariojv | i think the only change really needed based on comments is moving the deprecation message to service startup if possible | 19:18 |
mariojv | and maybe a little doc change | 19:18 |
mariojv | but i'm just going to get it rebased for now | 19:18 |
*** rpioso has quit IRC | 19:18 | |
*** bjolo_ has quit IRC | 19:22 | |
*** Sukhdev has joined #openstack-ironic | 19:25 | |
JayF | mariojv: oh man, you're taking it back over! <3 | 19:26 |
JayF | mariojv: anything I can do to help lmk | 19:26 |
*** _vsaienko has quit IRC | 19:32 | |
*** [1]rpioso has quit IRC | 19:34 | |
*** _vsaienko has joined #openstack-ironic | 19:34 | |
*** e0ne has quit IRC | 19:37 | |
*** tomstatt has quit IRC | 19:38 | |
devananda | jroll: bah - sorry about missing the meeting. I don't have anything to discuss, just waiting for reviews right now. | 19:39 |
*** e0ne has joined #openstack-ironic | 19:41 | |
*** Sukhdev has quit IRC | 19:41 | |
*** ElCoyote_ has left #openstack-ironic | 19:42 | |
*** dprince has quit IRC | 19:48 | |
*** ElCoyote_ has joined #openstack-ironic | 19:49 | |
*** e0ne has quit IRC | 19:50 | |
*** joprovost has quit IRC | 19:54 | |
*** e0ne has joined #openstack-ironic | 19:55 | |
*** [1]rpioso has joined #openstack-ironic | 19:56 | |
*** e0ne has quit IRC | 19:58 | |
*** Goneri has quit IRC | 19:58 | |
*** e0ne has joined #openstack-ironic | 20:02 | |
lucasagomes | folks I'm calling it a day | 20:04 |
lucasagomes | have a great evening all! | 20:04 |
jlvillal | lucasagomes: Good night! | 20:05 |
*** another_larsks is now known as larsks | 20:05 | |
lucasagomes | jlvillal, g'night! | 20:06 |
*** _vsaienko has quit IRC | 20:06 | |
*** mkam has left #openstack-ironic | 20:06 | |
*** lucasagomes is now known as lucas-afk | 20:21 | |
openstackgerrit | Merged openstack/python-ironicclient: Update .gitignore to ignore .idea of PyCharm https://review.openstack.org/384293 | 20:24 |
*** hamzy has quit IRC | 20:28 | |
openstackgerrit | Merged openstack/ironic-inspector: TrivialFix: Remove default=None when set value in Config https://review.openstack.org/377201 | 20:34 |
JayF | Does anyone know the correct configs/etc for getting a devstack working w/neutron network interface? | 20:39 |
JayF | We don't have it documented in dev docs, but I presume I can get some kind of success looking at the localrc of the multitenant networking job? | 20:39 |
*** sdake_ has joined #openstack-ironic | 20:39 | |
JayF | jcook: ^ | 20:40 |
*** sdake has quit IRC | 20:41 | |
*** amotoki has joined #openstack-ironic | 20:45 | |
rloo | JayF: yeah, I think it is in the mulltitenancy docs | 20:47 |
JayF | holy crap we documented a thing \o/ http://docs.openstack.org/developer/ironic/dev/ironic-multitenant-networking.html | 20:48 |
JayF | I'm going to push a patch to link that in from the normal devstack setup | 20:48 |
JayF | or go back and double check and find out (most likely) that I just didn't see it because it was in the most obvious place possible | 20:48 |
rloo | JayF: i think it is linked from the install guide but i could very well be wrong | 20:49 |
jcook | \o/ | 20:49 |
JayF | sure, but the doc I just linked it about how to setup multitenant in devstack | 20:49 |
JayF | which is more a dev thing than a install thing | 20:49 |
JayF | and in this case I cared more about getting jcook a testing env w/multitenant networking since he's going to be implementing rescue mode | 20:50 |
jcook | managers can code too \o/ (if they let us) | 20:50 |
* jcook gets back to his TPS reports. | 20:50 | |
*** amotoki has quit IRC | 20:50 | |
*** trown is now known as trown|outtypewww | 20:51 | |
*** sdake has joined #openstack-ironic | 20:51 | |
rloo | JayF: oh. devstack. i don't think we have any doc there; it is in the devstack config files | 20:51 |
mrda | jroll: sorry about the talk title :) | 20:51 |
JayF | That doc /is/ for devstack | 20:51 |
*** jheroux has quit IRC | 20:51 | |
JayF | not for actual installs | 20:51 |
mrda | And good morning :) | 20:52 |
JayF | http://docs.openstack.org/developer/ironic/dev/ironic-multitenant-networking.html is 100% devstack 0% real life | 20:52 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Make agent driver always PXE before cleaning https://review.openstack.org/263842 | 20:52 |
JayF | to be explicit | 20:52 |
rloo | JayF: it is? oh right, it is :) | 20:52 |
rloo | morning mrda | 20:52 |
mrda | hey rloo | 20:52 |
JayF | o/ | 20:52 |
mrda | o/ | 20:52 |
*** sdake_ has quit IRC | 20:53 | |
rloo | jcook: there are some managers in openstack that also code, welcome to that club :) | 20:55 |
*** jcoufal_ has quit IRC | 20:56 | |
jcook | rloo \o/ | 20:57 |
rloo | jcook: like michael stills. you can aspire to be like him :D | 21:00 |
*** _vsaienko has joined #openstack-ironic | 21:01 | |
JayF | mikal is nowhere near as awesome as jcook :P | 21:03 |
* JayF says, while waiting for mikal in a meeting room | 21:03 | |
* rloo stays away from commenting on that | 21:04 | |
JayF | I'm joking :) we're all one big happy family | 21:05 |
rloo | JayF: I was wondering if some sort of contest would be appropriate. ha ha. | 21:06 |
*** rama_y has quit IRC | 21:07 | |
*** rama_y has joined #openstack-ironic | 21:08 | |
mikal | Heh | 21:08 |
*** anush_ has quit IRC | 21:08 | |
*** Sukhdev has joined #openstack-ironic | 21:15 | |
openstackgerrit | Marc Aubry proposed openstack/python-ironicclient: Fix python3 compatibility when HTTP Error are returned https://review.openstack.org/378050 | 21:16 |
*** portdirect has quit IRC | 21:18 | |
*** Sukhdev has quit IRC | 21:28 | |
*** Sukhdev has joined #openstack-ironic | 21:31 | |
*** vishwanathj has quit IRC | 21:33 | |
*** vishwanathj has joined #openstack-ironic | 21:34 | |
*** Sukhdev has quit IRC | 21:37 | |
*** aNuposic has joined #openstack-ironic | 21:39 | |
*** aNuposic has quit IRC | 21:42 | |
*** aNuposic has joined #openstack-ironic | 21:42 | |
jcook | I'm not nearly as sarcastic (rtt of 44 minutes, not bad) | 21:48 |
*** e0ne has quit IRC | 21:48 | |
*** e0ne has joined #openstack-ironic | 21:49 | |
*** e0ne has quit IRC | 21:50 | |
*** e0ne has joined #openstack-ironic | 21:50 | |
*** e0ne has quit IRC | 21:50 | |
*** e0ne has joined #openstack-ironic | 21:51 | |
*** e0ne has quit IRC | 21:51 | |
*** e0ne has joined #openstack-ironic | 21:52 | |
*** e0ne has quit IRC | 21:52 | |
*** e0ne has joined #openstack-ironic | 21:52 | |
*** e0ne has quit IRC | 21:53 | |
*** e0ne has joined #openstack-ironic | 21:53 | |
*** e0ne has quit IRC | 21:54 | |
*** e0ne has joined #openstack-ironic | 21:54 | |
*** e0ne has quit IRC | 21:54 | |
*** zigo has quit IRC | 22:02 | |
*** zigo has joined #openstack-ironic | 22:05 | |
*** zigo is now known as Guest50630 | 22:05 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/385180 | 22:18 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/377446 | 22:18 |
*** baoli has quit IRC | 22:21 | |
*** mgoddard has quit IRC | 22:32 | |
*** _vsaienko has quit IRC | 22:33 | |
*** ppiela has quit IRC | 22:33 | |
*** aNuposic has quit IRC | 22:33 | |
*** rama_y has quit IRC | 22:35 | |
*** rama_y has joined #openstack-ironic | 22:46 | |
*** amotoki has joined #openstack-ironic | 22:47 | |
*** [1]rpioso has quit IRC | 22:48 | |
*** baoli has joined #openstack-ironic | 22:51 | |
*** amotoki has quit IRC | 22:52 | |
*** sdake has quit IRC | 23:16 | |
*** rajinir has quit IRC | 23:26 | |
JayF | tonyb: if you have a moment, a couple of minor ironic docs fixes to stable need approval: https://review.openstack.org/#/c/382694/ and https://review.openstack.org/#/c/382478/ | 23:35 |
patchbot | patch 382694 - ironic (stable/newton) - Keep numbering of list in Install Guide | 23:35 |
patchbot | patch 382478 - ironic (stable/newton) - [install-guide] describe pxe.ipxe_swift_tempurl | 23:35 |
*** pmannidi|brb is now known as pmannidi | 23:35 | |
tonyb | JayF: np | 23:44 |
JayF | thanks | 23:44 |
openstackgerrit | Merged openstack/ironic: [install-guide] describe pxe.ipxe_swift_tempurl https://review.openstack.org/382478 | 23:47 |
openstackgerrit | Merged openstack/ironic: Keep numbering of list in Install Guide https://review.openstack.org/382694 | 23:47 |
*** amotoki has joined #openstack-ironic | 23:48 | |
*** sdake has joined #openstack-ironic | 23:50 | |
*** sdake_ has joined #openstack-ironic | 23:51 | |
*** amotoki has quit IRC | 23:53 | |
openstackgerrit | Merged openstack/python-ironicclient: Fix import of ironicclient and reformat docstring https://review.openstack.org/378526 | 23:53 |
*** sdake has quit IRC | 23:55 | |
*** natorious has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!