*** leifmadsen has joined #openstack-ironic | 00:02 | |
*** harlowja has joined #openstack-ironic | 00:06 | |
*** dprince has quit IRC | 00:16 | |
*** bcornec1 has quit IRC | 00:20 | |
*** rbudden has quit IRC | 00:27 | |
*** dsneddon_afk is now known as dsneddon | 00:28 | |
*** nicodemos has quit IRC | 00:29 | |
*** nicodemos has joined #openstack-ironic | 00:29 | |
*** jaosorior has quit IRC | 00:31 | |
*** causten has quit IRC | 00:35 | |
*** bcornec1 has joined #openstack-ironic | 00:41 | |
openstackgerrit | Merged openstack/ironic: Trivial: fix warning message formatting https://review.openstack.org/420789 | 00:46 |
---|---|---|
*** bfournie has quit IRC | 00:48 | |
*** wrouesnel has quit IRC | 00:50 | |
*** wrouesnel has joined #openstack-ironic | 00:50 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI https://review.openstack.org/348191 | 00:51 |
*** hoangcx has joined #openstack-ironic | 00:52 | |
jlvillal | aNuposic, On the CONF options. I think you can use CONF.register_opts() in your unit test for the glance options | 00:54 |
jlvillal | aNuposic, http://docs.openstack.org/developer/oslo.config/cfg.html#registering-options | 00:54 |
aNuposic | jlvillal: checking out register_opts() | 00:58 |
*** bfournie has joined #openstack-ironic | 00:58 | |
*** SerenaFeng has joined #openstack-ironic | 01:00 | |
*** formica has quit IRC | 01:05 | |
*** phuongnh has joined #openstack-ironic | 01:06 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool management driver for Inject NMI https://review.openstack.org/348721 | 01:07 |
aNuposic | jlvillal: Nope failing on same, here is the change and error http://paste.openstack.org/show/595304/ | 01:08 |
jlvillal | aNuposic, I don't see that register opts is setting the group to 'swift' | 01:08 |
*** tuanluong has joined #openstack-ironic | 01:11 | |
aNuposic | jlvillal: should it be register_opts('option_name','group_name')? I followed this http://docs.openstack.org/developer/oslo.config/cfg.html#global-configopts | 01:11 |
*** jerrygb has quit IRC | 01:11 | |
jlvillal | aNuposic, http://docs.openstack.org/developer/oslo.config/cfg.html#option-groups | 01:12 |
*** jerrygb has joined #openstack-ironic | 01:14 | |
*** mtanino has quit IRC | 01:14 | |
*** hw_wutianwei has joined #openstack-ironic | 01:19 | |
aNuposic | jlvillal: http://paste.openstack.org/show/595305/ | 01:20 |
jlvillal | aNuposic, That is not the way register_opts works, from what I understand. | 01:21 |
jlvillal | Look at what the examples show. | 01:21 |
jlvillal | rabbit_host_opt = cfg.StrOpt('host', | 01:22 |
jlvillal | default='localhost', | 01:22 |
jlvillal | help='IP/hostname to listen on.'), | 01:22 |
jlvillal | rabbit_port_opt = cfg.PortOpt('port', | 01:22 |
jlvillal | default=5672, | 01:22 |
jlvillal | help='Port number to listen on.') | 01:22 |
jlvillal | conf.register_opt(rabbit_port_opt, group='rabbit') | 01:22 |
jlvillal | aNuposic, ^^^ | 01:22 |
aNuposic | jlvillal: Yes. We need these group attributes as well like default and help | 01:24 |
*** causten has joined #openstack-ironic | 01:25 | |
aNuposic | jlvillal: I see | 01:25 |
aNuposic | modifying | 01:25 |
jlvillal | aNuposic, It needs to be some type of "Opt" to be registered. | 01:27 |
jlvillal | Not a string like 'username' | 01:27 |
*** athomas has quit IRC | 01:27 | |
*** jralph has quit IRC | 01:27 | |
*** ricardoas has quit IRC | 01:27 | |
*** pmannidi has quit IRC | 01:27 | |
*** stevemar has quit IRC | 01:27 | |
jlvillal | user_opt = cfg.StrOpt('username) | 01:27 |
jlvillal | conf.register_opt(user_opt) | 01:28 |
jlvillal | Or something like that. | 01:28 |
*** athomas has joined #openstack-ironic | 01:29 | |
*** jralph has joined #openstack-ironic | 01:29 | |
*** ricardoas has joined #openstack-ironic | 01:29 | |
*** pmannidi has joined #openstack-ironic | 01:29 | |
*** stevemar has joined #openstack-ironic | 01:29 | |
*** xuao has joined #openstack-ironic | 01:32 | |
aNuposic | http://paste.openstack.org/show/595306/ | 01:34 |
aNuposic | jlvillal: http://paste.openstack.org/show/595306/ | 01:34 |
*** sacharya has quit IRC | 01:35 | |
*** wrouesnel has quit IRC | 01:36 | |
*** jhesketh has quit IRC | 01:36 | |
*** jhesketh has joined #openstack-ironic | 01:36 | |
aNuposic | jlvillal: may be i know what went wrong this time. | 01:37 |
aNuposic | jlvillal: Yes it worked | 01:37 |
aNuposic | jlvillal: cool thanks :) | 01:38 |
* TheJulia whispers "don't reuse the config group...." | 01:41 | |
*** Sukhdev_ has quit IRC | 01:48 | |
*** rbudden has joined #openstack-ironic | 01:52 | |
openstackgerrit | jiang wei proposed openstack/python-ironicclient: Support i18n for baremetal driver,portgroup cmds https://review.openstack.org/421087 | 02:00 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add DB layer support for node tags filter https://review.openstack.org/272624 | 02:00 |
openstackgerrit | jiang wei proposed openstack/python-ironicclient: Support i18n for baremetal driver,portgroup cmds https://review.openstack.org/421087 | 02:03 |
*** wrouesnel has joined #openstack-ironic | 02:07 | |
*** jrist has quit IRC | 02:18 | |
*** aNuposic has quit IRC | 02:29 | |
*** jrist has joined #openstack-ironic | 02:32 | |
*** sacharya has joined #openstack-ironic | 02:36 | |
*** harlowja has quit IRC | 02:36 | |
*** davidlenwell_ has joined #openstack-ironic | 02:38 | |
*** sacharya has quit IRC | 02:40 | |
*** sacharya has joined #openstack-ironic | 02:52 | |
*** gcb has joined #openstack-ironic | 02:54 | |
*** sacharya has quit IRC | 02:56 | |
*** sacharya has joined #openstack-ironic | 02:56 | |
*** sacharya has quit IRC | 03:01 | |
*** jschwarz has quit IRC | 03:01 | |
*** davidlenwell_ has quit IRC | 03:06 | |
*** jschwarz has joined #openstack-ironic | 03:06 | |
*** wrouesnel has quit IRC | 03:07 | |
*** wrouesnel has joined #openstack-ironic | 03:08 | |
*** vsaienko has joined #openstack-ironic | 03:19 | |
*** SerenaFeng has quit IRC | 03:20 | |
*** vsaienko has quit IRC | 03:20 | |
*** vikrant has joined #openstack-ironic | 03:21 | |
*** baoli has joined #openstack-ironic | 03:23 | |
*** wrouesnel has quit IRC | 03:23 | |
*** wrouesnel has joined #openstack-ironic | 03:26 | |
*** amotoki has quit IRC | 03:27 | |
*** Nisha_Agarwal has joined #openstack-ironic | 03:28 | |
*** baoli has quit IRC | 03:29 | |
*** sacharya has joined #openstack-ironic | 03:33 | |
*** Syed__ has quit IRC | 03:35 | |
*** wrouesnel has quit IRC | 03:35 | |
*** wrouesnel has joined #openstack-ironic | 03:37 | |
*** gcb has quit IRC | 03:38 | |
*** gcb has joined #openstack-ironic | 03:39 | |
*** baoli has joined #openstack-ironic | 03:39 | |
*** amotoki has joined #openstack-ironic | 03:42 | |
*** wrouesnel has quit IRC | 03:46 | |
*** wrouesnel has joined #openstack-ironic | 03:49 | |
*** amotoki_ has joined #openstack-ironic | 03:51 | |
*** baoli has quit IRC | 03:52 | |
*** amotoki has quit IRC | 03:54 | |
*** links has joined #openstack-ironic | 04:02 | |
*** amotoki_ has quit IRC | 04:11 | |
*** ijw has quit IRC | 04:19 | |
*** amotoki has joined #openstack-ironic | 04:25 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off https://review.openstack.org/216743 | 04:33 |
*** Sukhdev has joined #openstack-ironic | 04:34 | |
*** udesale has joined #openstack-ironic | 04:37 | |
*** awiddersheim has quit IRC | 04:39 | |
*** sacharya has quit IRC | 04:42 | |
*** sacharya has joined #openstack-ironic | 04:43 | |
*** amotoki has quit IRC | 04:44 | |
*** sacharya has quit IRC | 04:47 | |
*** wrouesnel has quit IRC | 04:53 | |
*** wrouesnel has joined #openstack-ironic | 04:55 | |
*** ijw has joined #openstack-ironic | 04:56 | |
*** nmathew has joined #openstack-ironic | 05:16 | |
*** wrouesnel has quit IRC | 05:21 | |
*** wrouesnel has joined #openstack-ironic | 05:26 | |
openstackgerrit | Naohiro Tamura proposed openstack/python-ironicclient: Add soft reboot/poweroff power states. https://review.openstack.org/247904 | 05:33 |
*** wrouesnel has quit IRC | 05:36 | |
*** causten has quit IRC | 05:36 | |
*** wrouesnel has joined #openstack-ironic | 05:37 | |
*** wrouesnel has quit IRC | 05:37 | |
*** wrouesnel has joined #openstack-ironic | 05:38 | |
*** rbudden has quit IRC | 05:39 | |
*** rama_y_ has quit IRC | 05:40 | |
*** rama_y has quit IRC | 05:40 | |
*** wrouesnel has quit IRC | 05:43 | |
*** jerrygb has quit IRC | 05:45 | |
*** wrouesnel has joined #openstack-ironic | 05:45 | |
*** jerrygb has joined #openstack-ironic | 05:46 | |
*** wrouesnel has quit IRC | 05:48 | |
*** wrouesnel has joined #openstack-ironic | 05:49 | |
*** wrouesnel has quit IRC | 05:49 | |
*** jerrygb has quit IRC | 05:50 | |
*** wrouesnel has joined #openstack-ironic | 05:51 | |
*** wrouesnel has quit IRC | 05:52 | |
openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: snmp inspection for disk size https://review.openstack.org/394858 | 05:54 |
*** wrouesnel has joined #openstack-ironic | 05:54 | |
*** wrouesnel has quit IRC | 05:54 | |
*** wrouesnel has joined #openstack-ironic | 05:57 | |
*** amotoki has joined #openstack-ironic | 06:00 | |
*** wrouesnel has joined #openstack-ironic | 06:01 | |
*** wrouesnel has quit IRC | 06:02 | |
*** wrouesnel has joined #openstack-ironic | 06:04 | |
*** wrouesnel has quit IRC | 06:04 | |
*** wrouesnel has joined #openstack-ironic | 06:06 | |
openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: snmp inspection for disk size https://review.openstack.org/394858 | 06:06 |
*** jrcloud has joined #openstack-ironic | 06:06 | |
*** wrouesnel has quit IRC | 06:06 | |
*** wrouesnel has joined #openstack-ironic | 06:08 | |
*** wrouesnel has joined #openstack-ironic | 06:08 | |
*** zackf has quit IRC | 06:09 | |
*** wrouesnel has quit IRC | 06:11 | |
openstackgerrit | Naohiro Tamura proposed openstack/python-ironicclient: Support soft reboot and soft power off with timeout for OSC https://review.openstack.org/357627 | 06:12 |
*** wrouesnel has joined #openstack-ironic | 06:12 | |
*** wrouesnel has joined #openstack-ironic | 06:12 | |
*** wrouesnel has quit IRC | 06:17 | |
*** wrouesnel has joined #openstack-ironic | 06:19 | |
*** wrouesnel has quit IRC | 06:23 | |
*** wrouesnel has joined #openstack-ironic | 06:25 | |
*** wrouesnel has quit IRC | 06:27 | |
*** wrouesnel has joined #openstack-ironic | 06:29 | |
*** Sukhdev has quit IRC | 06:36 | |
*** mjura has joined #openstack-ironic | 06:40 | |
openstackgerrit | Annie Lezil proposed openstack/ironic-inspector-specs: Collect NIC name given by BIOS https://review.openstack.org/404372 | 06:41 |
*** xuao has quit IRC | 06:44 | |
openstackgerrit | Cao Xuan Hoang proposed openstack/ironic-ui: Add Python 3.5 classifier and venv https://review.openstack.org/421695 | 06:48 |
*** SerenaFeng has joined #openstack-ironic | 06:54 | |
*** phuongnh has quit IRC | 07:03 | |
*** phuongnh has joined #openstack-ironic | 07:04 | |
*** jrist has quit IRC | 07:11 | |
*** jrist has joined #openstack-ironic | 07:12 | |
*** jrist has quit IRC | 07:14 | |
*** bapalm has quit IRC | 07:16 | |
*** jrist has joined #openstack-ironic | 07:16 | |
*** bapalm has joined #openstack-ironic | 07:18 | |
*** ChubYann has quit IRC | 07:22 | |
openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent: Do not review patch. Testing biosdevname package https://review.openstack.org/421706 | 07:24 |
*** ohamada has joined #openstack-ironic | 07:25 | |
*** tesseract has joined #openstack-ironic | 07:25 | |
*** tesseract has quit IRC | 07:25 | |
*** tesseract has joined #openstack-ironic | 07:26 | |
*** rbartal has joined #openstack-ironic | 07:27 | |
openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent: [WIP]Do not review patch. Testing biosdevname package https://review.openstack.org/421706 | 07:28 |
xek | morning ironic! | 07:37 |
*** mgoddard has joined #openstack-ironic | 07:52 | |
*** pmannidi has quit IRC | 07:53 | |
*** phuongnh has quit IRC | 07:54 | |
*** phuongnh has joined #openstack-ironic | 07:55 | |
tuanluong | morning ironic | 08:00 |
openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: snmp inspection for disk size https://review.openstack.org/394858 | 08:04 |
*** e0ne has joined #openstack-ironic | 08:04 | |
Nisha_Agarwal | tuanluong, morning! | 08:05 |
*** ralonsoh has joined #openstack-ironic | 08:18 | |
*** vsaienko has joined #openstack-ironic | 08:21 | |
*** vsaienko has quit IRC | 08:23 | |
tuanluong | morning Nisha_Agarwal | 08:26 |
openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: Add `latest` to baremetal-api-version https://review.openstack.org/420659 | 08:26 |
*** ijw has quit IRC | 08:31 | |
openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: snmp inspection for disk size https://review.openstack.org/394858 | 08:31 |
*** amotoki_ has joined #openstack-ironic | 08:33 | |
*** amoralej|off is now known as amoralej | 08:35 | |
*** amotoki has quit IRC | 08:36 | |
*** amotoki has joined #openstack-ironic | 08:37 | |
*** amotoki_ has quit IRC | 08:41 | |
*** amotoki has quit IRC | 08:44 | |
*** amotoki has joined #openstack-ironic | 08:44 | |
*** mgoddard has quit IRC | 08:48 | |
vdrok | good morning ironic, xek tuanluong and Nisha_Agarwal | 08:51 |
*** priteau has joined #openstack-ironic | 08:52 | |
Nisha_Agarwal | Good morning vdrok | 08:52 |
*** zzzeek has quit IRC | 09:00 | |
*** zzzeek has joined #openstack-ironic | 09:00 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui: Imported Translations from Zanata https://review.openstack.org/418781 | 09:06 |
*** vsaienko has joined #openstack-ironic | 09:14 | |
*** derekh has joined #openstack-ironic | 09:23 | |
*** jhesketh has quit IRC | 09:27 | |
*** amotoki_ has joined #openstack-ironic | 09:29 | |
*** jhesketh has joined #openstack-ironic | 09:30 | |
*** mgoddard has joined #openstack-ironic | 09:30 | |
*** amotoki has quit IRC | 09:31 | |
*** ijw has joined #openstack-ironic | 09:32 | |
*** ijw has quit IRC | 09:37 | |
*** pcaruana has joined #openstack-ironic | 09:41 | |
*** lucas-afk is now known as lucasagomes | 09:45 | |
*** SerenaFeng has quit IRC | 09:46 | |
*** vsaienko has quit IRC | 09:53 | |
*** vsaienko has joined #openstack-ironic | 09:53 | |
*** tuanluong_ has joined #openstack-ironic | 10:06 | |
*** hoangcx has quit IRC | 10:06 | |
*** tuanluong has quit IRC | 10:07 | |
*** vsaienko has quit IRC | 10:12 | |
*** tuanluong_ has quit IRC | 10:14 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack: Configure nodes/environment to boot in UEFI mode https://review.openstack.org/374988 | 10:16 |
lucasagomes | ^ all dependencies are now merged o/ | 10:17 |
*** MattMan has quit IRC | 10:18 | |
*** MattMan has joined #openstack-ironic | 10:18 | |
vdrok | morning lucasagomes :) | 10:25 |
openstackgerrit | Merged openstack/ironic-python-agent: Use sh instead of bash when installing grub https://review.openstack.org/421102 | 10:27 |
*** ijw has joined #openstack-ironic | 10:34 | |
*** milan has joined #openstack-ironic | 10:36 | |
milan | morning Ironic! :) | 10:36 |
phuongnh | lucasagomes, congratulation :) | 10:36 |
*** pcaruana has quit IRC | 10:38 | |
*** ijw has quit IRC | 10:39 | |
milan | aarefiev, morning :) mind a review here: https://review.openstack.org/#/c/421285/ ? | 10:43 |
patchbot | patch 421285 - ironic-inspector - Find a node by multiple attributes | 10:43 |
aarefiev | milan: hey, I'll take a look | 10:44 |
milan | aarefiev, thx! this should fix https://bugs.launchpad.net/tripleo/+bug/1651719 | 10:45 |
openstack | Launchpad bug 1651719 in Ironic Inspector "Introspection fails on nodes with the same IPMI address but different IPMI ports" [High,In progress] - Assigned to milan k (vetrisko) | 10:45 |
aarefiev | nice, I remember we've discussed it :) | 10:46 |
*** sambetts|afk is now known as sambetts | 10:47 | |
sambetts | Morning all | 10:47 |
milan | yup :) | 10:47 |
milan | morning sambetts! :) | 10:47 |
*** csmart has joined #openstack-ironic | 10:53 | |
sambetts | hey milan | 10:53 |
*** phuongnh has quit IRC | 10:53 | |
vsaienk0 | morning Ironic'ers! | 10:53 |
vsaienk0 | sambetts: wondered what do you think about https://review.openstack.org/#/c/419975/ | 10:54 |
patchbot | patch 419975 - nova - Implement interface attach/detach in ironic virt d... | 10:54 |
vdrok | morning milan aarefiev sambetts vsaienk0 | 10:54 |
milan | morning vdrok! :) | 10:54 |
vsaienk0 | hey vrdok :) | 10:55 |
aarefiev | morning sambetts, vdrok, vsaienk0 :) | 10:55 |
* milan just realised a thing, we store the state of introspection in the DB now, so probably dropping the Attributes table as part of the migration might need documenting at least | 10:56 | |
sambetts | vsaienk0: I like the idea of doing that, I have similar concerns to jroll and we'll need code changes on the Ironic side to call the configure_tenant_networks logic for the port if the node is already in active state | 10:59 |
*** gcb has quit IRC | 11:05 | |
vsaienk0 | sambetts: yeah, I've missed Ironic part, definitely it is needed. Regarding CI I saw tempest tests in nova that check attach/detach we can reuse them | 11:05 |
*** udesale has quit IRC | 11:13 | |
*** nmathew has quit IRC | 11:18 | |
*** dtantsur|afk is now known as dtantsur | 11:32 | |
dtantsur | morning Ironic, says the man wondering how to sleep less than 11 hours Oo | 11:32 |
dtantsur | jroll, t-shirt++ | 11:33 |
cdaniel | morning all | 11:34 |
*** ijw has joined #openstack-ironic | 11:35 | |
sambetts | morning dtantsur, cdaniel | 11:36 |
*** ijw has quit IRC | 11:40 | |
jroll | sambetts: ++, nice comments | 11:40 |
sambetts | :) | 11:41 |
jroll | morning y'all | 11:41 |
*** jkilpatr has quit IRC | 11:42 | |
sambetts | o/ | 11:43 |
vdrok | morning dtantsur cdaniel and jroll | 11:44 |
dtantsur | morning jroll, cdaniel, vdrok, sambetts, aarefiev, milan, lucasagomes | 11:44 |
jroll | vdrok: morning, thanks for the driver composition reviews, will update soon :) | 11:44 |
vdrok | yw :) | 11:45 |
lucasagomes | dtantsur, jroll sambetts cdaniel morning | 11:45 |
*** dprince has joined #openstack-ironic | 11:45 | |
lucasagomes | vdrok, too! | 11:45 |
vdrok | \o/ | 11:45 |
jroll | dtantsur: morning, most of the driver composition stuff is close/ready when you have some time ;) | 11:45 |
jroll | s/;)/:)/ | 11:45 |
aarefiev | morning dtantsur, jroll, lucasagomes | 11:45 |
dtantsur | jroll, will definitely find time today, lemme open my eyes first (yes, I wake up the same time as you :D ) | 11:46 |
jroll | dtantsur: lol, thanks | 11:46 |
csmart | \o | 11:48 |
openstackgerrit | Sam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 11:48 |
*** ccamacho is now known as ccamacho|lunc | 11:55 | |
*** ccamacho|lunc is now known as ccamacho|lunch | 11:55 | |
*** pcaruana has joined #openstack-ironic | 11:56 | |
*** SerenaFeng has joined #openstack-ironic | 11:57 | |
dtantsur | jroll, can we really safely add columns with nullable=False? | 12:00 |
jroll | dtantsur: with a default, yes | 12:00 |
dtantsur | I mean, probably this table will be empty on upgrade | 12:00 |
jroll | there shouldn't be any rows anyway | 12:00 |
dtantsur | jroll, do you keep in mind that default!=server_default? I'm not sure how this work on upgrade | 12:01 |
*** amotoki_ has quit IRC | 12:01 | |
dtantsur | ah, we don't register anything yet. then ignore me. | 12:01 |
jroll | dtantsur: I can add some inserts in the migration test to be safe, if you like | 12:01 |
jroll | before the upgrade | 12:01 |
dtantsur | jroll, not really, as I've noticed that we don't populate this table at all yet | 12:02 |
jroll | indeed | 12:02 |
jroll | though ops be crazy | 12:02 |
milan | morning dtantsur, jroll! :) | 12:02 |
dtantsur | jroll, well, if some data is in table already, then blindly adding "false" is not an option too | 12:03 |
dtantsur | so dunno, I'm fine with +2A this now, just lemme know what you think | 12:03 |
milan | oh, and morning lucasagomes! :D | 12:03 |
jroll | dtantsur: yeah, idk I think it's fine | 12:04 |
dtantsur | ok, approved first two, will get to the next when I open my eyes wider (i.e. after more coffee) | 12:10 |
jroll | cool, the third has a -1 anyway | 12:11 |
jroll | will do spec update today as well | 12:12 |
jroll | thanks! | 12:12 |
dtantsur | jroll, oh, we seem to have something to release for ironic-lib (again), see doug's email and http://paste.openstack.org/show/595268/ | 12:13 |
jroll | dtantsur: yep, on my todo list for today | 12:13 |
mrtenio | Good morning y'all, :) | 12:13 |
dtantsur | morning mrtenio | 12:13 |
jroll | morning mrtenio | 12:13 |
*** jkilpatr has joined #openstack-ironic | 12:18 | |
csmart | mrtenio: morning! | 12:19 |
csmart | well, evening here ;-) | 12:19 |
mrtenio | csmart :P | 12:28 |
*** lhinds is now known as lhinds|afk | 12:30 | |
nicodemos | morning, ironicers. :) | 12:33 |
csmart | nicodemos: \o | 12:33 |
nicodemos | csmart: o/ | 12:34 |
*** ijw has joined #openstack-ironic | 12:36 | |
*** vikrant has quit IRC | 12:39 | |
*** vikrant has joined #openstack-ironic | 12:40 | |
*** ijw has quit IRC | 12:41 | |
*** hw_wutianwei has quit IRC | 12:42 | |
*** vikrant has quit IRC | 12:43 | |
jroll | dtantsur: tooz thing, do you want a release note? I tend to think not for just adding a library as ops shouldn't need to worry about it | 12:44 |
dtantsur | jroll, sorry, I don't remember, did I vote -1 based on that? | 12:45 |
dtantsur | I generally thing that adding a short note is useful, but I never insist | 12:45 |
openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: snmp inspection for disk size https://review.openstack.org/394858 | 12:45 |
jroll | dtantsur: dunno, you had other comments too, it was a -1 | 12:45 |
jroll | ok I don't mind adding | 12:45 |
*** rbudden has joined #openstack-ironic | 12:45 | |
*** links has quit IRC | 12:46 | |
*** amotoki has joined #openstack-ironic | 12:46 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Move to tooz hash ring implementation https://review.openstack.org/399028 | 12:47 |
dtantsur | jroll, btw, an RFE to (re)consider: https://bugs.launchpad.net/ironic/+bug/1639688 | 12:47 |
openstack | Launchpad bug 1639688 in Ironic "[RFE] Add "reset bios settings to default" to iRMC drivers as cleaning step" [Wishlist,In progress] - Assigned to Dao Cong Tien (tiendc) | 12:47 |
jroll | hrm | 12:49 |
jroll | dtantsur: I'm good with that | 12:51 |
*** ccamacho|lunch is now known as ccamacho | 12:58 | |
openstackgerrit | Merged openstack/ironic: make sure OVS_PHYSICAL_BRIDGE is up before bring up vlan interface https://review.openstack.org/421279 | 13:00 |
openstackgerrit | Xavier proposed openstack/ironic: Adds validation before delete profile in oneview https://review.openstack.org/386014 | 13:01 |
xavierr | good morning Ironic | 13:05 |
*** udesale has joined #openstack-ironic | 13:06 | |
*** trown|outtypewww is now known as trown | 13:07 | |
*** amotoki has quit IRC | 13:08 | |
*** lhinds|afk has quit IRC | 13:11 | |
openstackgerrit | Merged openstack/ironic: Add default column to ConductorHardwareInterfaces https://review.openstack.org/421464 | 13:11 |
*** rnoriega has quit IRC | 13:12 | |
*** rnoriega has joined #openstack-ironic | 13:13 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Find a node by multiple attributes https://review.openstack.org/421285 | 13:13 |
*** lucasagomes is now known as lucas-hungry | 13:15 | |
openstackgerrit | Merged openstack/ironic: Expose default interface calculation from driver_factory https://review.openstack.org/418573 | 13:15 |
*** lhinds has joined #openstack-ironic | 13:17 | |
*** ccamacho has quit IRC | 13:19 | |
*** vikrant has joined #openstack-ironic | 13:20 | |
*** Nisha_Agarwal has quit IRC | 13:23 | |
dtantsur | sambetts, vsaienk0, do you guys know what switches are currently supported for ironic-neutron integration? | 13:24 |
dtantsur | do they have a list somewhere? | 13:25 |
*** vikrant has quit IRC | 13:25 | |
sambetts | dtantsur: Its down to which ML2 drivers support the baremetal vnic type, and I don't know if there is a list anywhere for that | 13:25 |
vsaienk0 | dtantsur: I doubt that such list exists. It strictly depends on vendros ML2 drivers | 13:26 |
dtantsur | ugh.. I need to figure out somehow which hardware to get.. | 13:26 |
sambetts | dtantsur: I know our Cisco Nexus ml2 driver has supprt, arista has a driver that has support, and ironic has the generic switch driver that supports any switch netmiko supports | 13:26 |
*** Nisha_Agarwal has joined #openstack-ironic | 13:26 | |
vsaienk0 | dtantsur: there list of published ML2 drivers that knows about local_link_info field http://codesearch.openstack.org/?q=local_link_info&i=nope&files=&repos= | 13:27 |
sambetts | OpenStack is quite bad at generally being able to look up what drivers are avaible and what hardware is supported for what feature :( | 13:28 |
*** amotoki has joined #openstack-ironic | 13:28 | |
sambetts | wow! more people support it than I expected! | 13:28 |
dtantsur | hmm, Juniper does not seem to be there | 13:29 |
sambetts | dtantsur: netmiko https://github.com/ktbyers/netmiko has a Juniper switch listed | 13:30 |
*** Nisha_Agarwal has quit IRC | 13:32 | |
* dtantsur tries to wrap his head around the thing | 13:33 | |
*** ashestakov has joined #openstack-ironic | 13:33 | |
sambetts | dtantsur: https://github.com/openstack/networking-generic-switch | 13:33 |
sambetts | dtantsur: thats the ML2 driver vsaienk0 and vdrok built that uses netmiko ^ | 13:33 |
sambetts | dtantsur: its what we use in the gate for testing multi-tenant | 13:34 |
sambetts | 13:34 | |
dtantsur | ok, thanks | 13:35 |
vsaienk0 | dtantsur: seems juniper ML2 is not publicly available, you may check deeper https://www.juniper.net/techpubs/en_US/release-independent/junos/topics/topic-map/openstack-neutron-plugin.html | 13:35 |
dtantsur | I also see that Juniper have their own neutron plugin http://www.juniper.net/documentation/en_US/release-independent/junos/topics/topic-map/openstack-neutron-plugin.html | 13:35 |
dtantsur | right :) | 13:35 |
*** ccamacho has joined #openstack-ironic | 13:37 | |
*** ijw has joined #openstack-ironic | 13:38 | |
dtantsur | jroll, please remind me, have we agreed on not exposing vendor_interface? | 13:43 |
openstackgerrit | Merged openstack/ironic-inspector: Do not use loopback BMC addresses for lookup https://review.openstack.org/419526 | 13:43 |
*** ijw has quit IRC | 13:44 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Do not use loopback BMC addresses for lookup https://review.openstack.org/421909 | 13:46 |
*** dprince has quit IRC | 13:52 | |
*** trown is now known as trown|afk | 13:54 | |
*** vsaienko has joined #openstack-ironic | 13:55 | |
*** jerrygb has joined #openstack-ironic | 13:56 | |
*** amoralej is now known as amoralej|lunch | 13:59 | |
*** jerrygb has quit IRC | 13:59 | |
jroll | dtantsur: you and I have, yes :) | 14:01 |
jroll | I think I haven't removed it from those patches though | 14:01 |
*** sacharya has joined #openstack-ironic | 14:03 | |
*** jerrygb has joined #openstack-ironic | 14:03 | |
*** jerrygb has quit IRC | 14:03 | |
*** jerrygb has joined #openstack-ironic | 14:04 | |
*** amotoki has quit IRC | 14:04 | |
*** athomas has quit IRC | 14:06 | |
TheJulia | Good morning | 14:08 |
jroll | morning TheJulia :) | 14:08 |
dtantsur | morning TheJulia | 14:08 |
dtantsur | jroll, good | 14:08 |
*** sacharya has quit IRC | 14:08 | |
*** jcoufal has joined #openstack-ironic | 14:10 | |
*** tuan_luong has joined #openstack-ironic | 14:11 | |
openstackgerrit | Vadim Hmyrov proposed openstack/ironic: Disable guessing content type from extension in node name. https://review.openstack.org/408597 | 14:12 |
*** tuanla_ has joined #openstack-ironic | 14:12 | |
*** Goneri has joined #openstack-ironic | 14:14 | |
*** udesale has quit IRC | 14:15 | |
*** athomas has joined #openstack-ironic | 14:15 | |
*** srobert has joined #openstack-ironic | 14:15 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Change os_tenant_name to os_project_name in tests configuration script https://review.openstack.org/408235 | 14:15 |
*** udesale has joined #openstack-ironic | 14:17 | |
*** amotoki has joined #openstack-ironic | 14:18 | |
vdrok | morning TheJulia | 14:18 |
*** udesale has quit IRC | 14:18 | |
*** udesale has joined #openstack-ironic | 14:19 | |
TheJulia | jroll: what about CONF.hash_distribution_replicas ? | 14:19 |
jroll | TheJulia: what about it? | 14:19 |
* jroll wonders if he did something wrong | 14:20 | |
TheJulia | It looks like we don't use it with the tooz patch | 14:20 |
*** baoli has joined #openstack-ironic | 14:20 | |
jroll | ooo | 14:20 |
*** baoli has quit IRC | 14:20 | |
jroll | oh, right | 14:21 |
jroll | it's a parameter to get_nodes() now, instead of __init__ | 14:21 |
jroll | https://review.openstack.org/#/c/399028/8/ironic/conductor/base_manager.py | 14:21 |
patchbot | patch 399028 - ironic - Move to tooz hash ring implementation | 14:21 |
*** baoli has joined #openstack-ironic | 14:21 | |
*** udesale has quit IRC | 14:21 | |
TheJulia | ahh, thanks | 14:21 |
jroll | but I did miss it here: https://review.openstack.org/#/c/399028/8/ironic/conductor/rpcapi.py | 14:21 |
patchbot | patch 399028 - ironic - Move to tooz hash ring implementation | 14:21 |
*** udesale has joined #openstack-ironic | 14:21 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Move to tooz hash ring implementation https://review.openstack.org/399028 | 14:22 |
jroll | TheJulia: fixed, thanks for catching that | 14:22 |
*** trown|afk is now known as trown | 14:22 | |
TheJulia | I posted a nit on the prior revision regarding the release note, not really worth doing anything most likely. I just feel it might be useful to understand why the dependency has been added. | 14:24 |
jroll | ah | 14:24 |
*** ccamacho has quit IRC | 14:25 | |
*** lucas-hungry is now known as lucasagomes | 14:25 | |
jroll | TheJulia: "as the consistent hash ring code was moved out of ironic and into tooz" sound good? | 14:25 |
TheJulia | jroll: perfect to me! | 14:25 |
jroll | cool | 14:25 |
jroll | done | 14:25 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Move to tooz hash ring implementation https://review.openstack.org/399028 | 14:25 |
TheJulia | jroll: you rock | 14:25 |
jroll | nou | 14:25 |
jroll | :) | 14:25 |
*** dprince has joined #openstack-ironic | 14:29 | |
* TheJulia awaits rendered doc output :) | 14:29 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Remove unused "service" argument from tempest client manager https://review.openstack.org/421965 | 14:31 |
dtantsur | we may need ^^^ to unblock stable/newton for inspector | 14:31 |
jroll | sigh | 14:32 |
jroll | likely mitaka too | 14:32 |
jroll | dtantsur: should I be waiting for CI or is it fine to just go ahead | 14:32 |
dtantsur | jroll, we don't have non-voting dsvm jobs, so it's fine to try approving | 14:33 |
jroll | ok, thanks | 14:34 |
jroll | +A | 14:34 |
EmilienM | ok I'll switch here, I found a problem with ironic/nova, and I'm sure you guys can help | 14:35 |
EmilienM | https://bugs.launchpad.net/tripleo/+bug/1657475 | 14:35 |
openstack | Launchpad bug 1657475 in tripleo "Impossible to spawn overcloud without Glance v1 API" [High,Triaged] - Assigned to Emilien Macchi (emilienm) | 14:35 |
openstackgerrit | Karthik S proposed openstack/ironic-inspector-specs: Spec for retrieving NUMA node information https://review.openstack.org/396147 | 14:35 |
EmilienM | it sounds like ironic tries to use Glance v1 API by default | 14:36 |
EmilienM | and if you disable v1 API, Ironic fails to spawn a node | 14:36 |
dtantsur | we definitely seem to have v2 support: https://github.com/openstack/ironic/tree/master/ironic/common/glance_service/v2 | 14:36 |
EmilienM | yes but how do you confiugre ironic (or nova?) to use it | 14:38 |
* dtantsur tries to figure out | 14:38 | |
dtantsur | maybe we use v1 because of the default value in https://github.com/openstack/ironic/blob/master/ironic/common/image_service.py#L261 | 14:39 |
*** tuanla_ has quit IRC | 14:39 | |
*** tuan_luong has quit IRC | 14:39 | |
lucasagomes | dtantsur, yeah there's a bunch of places where we are not specifying the version and it's defaulting to v1 | 14:39 |
lucasagomes | image_cache.py as well | 14:40 |
lucasagomes | all calls to get_image_service() should specify a version a think | 14:40 |
lucasagomes | apparently it's not configurable | 14:40 |
*** yolanda has quit IRC | 14:40 | |
* lucasagomes checks | 14:40 | |
dtantsur | lucasagomes, lemme try making it configurable | 14:40 |
EmilienM | do you know v1 is deprecated? | 14:40 |
EmilienM | and v2 is usable since Kilo | 14:40 |
*** yolanda has joined #openstack-ironic | 14:41 | |
dtantsur | yep. probably just fell out of our radar | 14:41 |
lucasagomes | dtantsur, yeah I'm modyfing a running devstack here to test | 14:41 |
*** ijw has joined #openstack-ironic | 14:42 | |
*** tuan_luong has joined #openstack-ironic | 14:42 | |
*** tuan_luong has quit IRC | 14:42 | |
*** tuan_luong has joined #openstack-ironic | 14:42 | |
*** tuan_luong has quit IRC | 14:42 | |
*** ccamacho has joined #openstack-ironic | 14:42 | |
*** rloo has joined #openstack-ironic | 14:42 | |
*** tuan_luong has joined #openstack-ironic | 14:43 | |
*** causten has joined #openstack-ironic | 14:45 | |
lucasagomes | vdrok, dtantsur re:wsme + sphix extension | 14:46 |
lucasagomes | ++ I will release it today | 14:46 |
vdrok | lucasagomes: thanks! | 14:46 |
*** ijw has quit IRC | 14:46 | |
*** gcb has joined #openstack-ironic | 14:47 | |
dtantsur | lucasagomes, I wonder why we hardcode v1 here: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L99 | 14:51 |
*** hw_wutianwei has joined #openstack-ironic | 14:51 | |
EmilienM | dtantsur: can I add ironic project to https://bugs.launchpad.net/tripleo/+bug/1657475 ? | 14:51 |
openstack | Launchpad bug 1657475 in tripleo "Impossible to spawn overcloud without Glance v1 API" [High,Triaged] - Assigned to Emilien Macchi (emilienm) | 14:51 |
lucasagomes | dtantsur, dunno... at ironic/drivers/modules/deploy_utils.py the version is 2 | 14:52 |
dtantsur | EmilienM, already done | 14:52 |
lucasagomes | dtantsur, apparently we mix&match stuff there :-( | 14:52 |
EmilienM | dtantsur++ | 14:52 |
dtantsur | ok, I'll replace all places where we have explicit v1 with an option, then try switching it to 2 by default in another patch | 14:53 |
dtantsur | sounds like a plan? | 14:53 |
*** yufei has joined #openstack-ironic | 14:53 | |
EmilienM | yeah ! | 14:53 |
EmilienM | and ship it | 14:53 |
EmilienM | :P | 14:53 |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Do not merge https://review.openstack.org/421985 | 14:53 |
lucasagomes | dtantsur, /me testing it locally on devstack | 14:54 |
dtantsur | cool, I'll prepare patches in the meantime | 14:54 |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Do not merge https://review.openstack.org/421985 | 14:54 |
EmilienM | lucasagomes, dtantsur : thank you guys | 14:56 |
dtantsur | np | 14:56 |
*** links has joined #openstack-ironic | 14:56 | |
*** links has quit IRC | 14:56 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Allow configuring default Glance version to use https://review.openstack.org/421994 | 14:58 |
dtantsur | here goes the first part ^^^ | 14:58 |
*** jheroux has joined #openstack-ironic | 15:01 | |
*** mtanino has joined #openstack-ironic | 15:01 | |
*** rpioso has joined #openstack-ironic | 15:01 | |
*** nmathew has joined #openstack-ironic | 15:02 | |
*** trown is now known as trown|mtg | 15:03 | |
*** jistr is now known as jistr|mtg | 15:05 | |
EmilienM | dtantsur: I can test this patch in tripleo if you like | 15:06 |
openstackgerrit | Julia Kreger proposed openstack/python-ironicclient: Add soft reboot/poweroff power states. https://review.openstack.org/247904 | 15:06 |
dtantsur | EmilienM, please do | 15:07 |
TheJulia | dtantsur: vdrok ^^^ I edited the type since it was the only thing that seemed to be holding it back | 15:07 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Switch to Glance V2 by default for everything https://review.openstack.org/422002 | 15:09 |
dtantsur | the second part ^^^ | 15:09 |
vdrok | TheJulia: thanks | 15:09 |
openstackgerrit | Aparna proposed openstack/proliantutils: Add HPSUM firmware update support https://review.openstack.org/422004 | 15:10 |
mjturek | JayF: wondering if I could help with https://review.openstack.org/#/c/408341/ You mentioned it needs test cases and rebase. Lemme know! | 15:13 |
patchbot | patch 408341 - python-ironicclient - Add support for RESCUE and UNRESCUE provision states | 15:13 |
*** amoralej|lunch is now known as amoralej | 15:14 | |
*** openstack has joined #openstack-ironic | 15:19 | |
*** JayF has joined #openstack-ironic | 15:19 | |
*** SpamapS has joined #openstack-ironic | 15:19 | |
*** jlvillal_ is now known as jlvillal | 15:20 | |
*** fungi has joined #openstack-ironic | 15:20 | |
*** jlvillal is now known as Guest88977 | 15:20 | |
*** milan has quit IRC | 15:22 | |
*** milan has joined #openstack-ironic | 15:22 | |
*** ionutbalutoiu has joined #openstack-ironic | 15:22 | |
*** mgagne has joined #openstack-ironic | 15:22 | |
*** comstud has joined #openstack-ironic | 15:22 | |
*** Daviey has joined #openstack-ironic | 15:22 | |
*** amotoki has joined #openstack-ironic | 15:22 | |
*** nicodemos has joined #openstack-ironic | 15:22 | |
*** sasha2 has joined #openstack-ironic | 15:22 | |
*** trown|mtg has joined #openstack-ironic | 15:22 | |
*** dsneddon has joined #openstack-ironic | 15:22 | |
*** vsaienk0 has joined #openstack-ironic | 15:22 | |
*** jmccrory has joined #openstack-ironic | 15:22 | |
*** natorious has joined #openstack-ironic | 15:22 | |
*** bl0m1_ has joined #openstack-ironic | 15:22 | |
*** jtomasek has joined #openstack-ironic | 15:22 | |
*** lukl has joined #openstack-ironic | 15:22 | |
*** yhvh has joined #openstack-ironic | 15:22 | |
*** joemonk- has joined #openstack-ironic | 15:22 | |
*** pshige_________ has joined #openstack-ironic | 15:22 | |
*** milan has quit IRC | 15:22 | |
*** milan has joined #openstack-ironic | 15:22 | |
*** [1]cdearborn has joined #openstack-ironic | 15:22 | |
*** mgagne has quit IRC | 15:23 | |
*** mgagne has joined #openstack-ironic | 15:23 | |
*** mgagne is now known as Guest58531 | 15:23 | |
*** joanna has joined #openstack-ironic | 15:24 | |
*** chihhsin has joined #openstack-ironic | 15:24 | |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Do not merge https://review.openstack.org/421985 | 15:25 |
rloo | vsaienk0: let me know if I am wrong, am wondering if I am awake enough: https://review.openstack.org/#/c/364413 | 15:26 |
patchbot | patch 364413 - nova - Support Ironic interface attach/detach in nova virt | 15:26 |
dtantsur | jroll, we may need to backport https://review.openstack.org/#/c/416926/ too, as inspector is not using master-only ironic tempest plugin | 15:26 |
patchbot | patch 416926 - ironic - Remove service argument from tempest plugin client... (MERGED) | 15:26 |
dtantsur | also morning rloo | 15:26 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Remove service argument from tempest plugin client manager https://review.openstack.org/422018 | 15:26 |
rloo | hi dtantsur! | 15:26 |
*** davidlenwell has joined #openstack-ironic | 15:26 | |
rloo | and good morning everyone else, vdrok, sambetts, TheJulia, lucasagomes, jroll | 15:27 |
*** nmathew has quit IRC | 15:27 | |
vdrok | morning rloo ! | 15:28 |
lucasagomes | rloo, morning | 15:29 |
*** fyxim has joined #openstack-ironic | 15:29 | |
*** krtaylor has joined #openstack-ironic | 15:31 | |
jroll | dtantsur: hm, shouldn't we just fix it in project-config instead? | 15:32 |
jroll | morning rloo :) | 15:32 |
dtantsur | jroll, maybe? I dunno how to do it though | 15:32 |
TheJulia | good morning rloo | 15:33 |
jroll | dtantsur: looking at this, it looks like you should be using master always: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L996 and https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L216 | 15:34 |
jroll | er, you already are | 15:34 |
dtantsur | jroll, seems like it does not work for grenade | 15:34 |
*** baoli has quit IRC | 15:35 | |
jroll | dtantsur: hrm, it should be | 15:36 |
* jroll looks | 15:36 | |
jroll | oh, I see why, lame | 15:38 |
JayF | mjturek: I was going to work with mariojv today on ipa rescue extension, if you wanted to rebase + add tests to that client patch, I'm OK with you taking care of it | 15:39 |
mjturek | JayF: cool cool, I'll get something up by EoD and ping you when I submit it | 15:39 |
JayF | mjturek: thanks! | 15:40 |
jroll | hrm, http://logs.openstack.org/65/421965/1/check/gate-grenade-dsvm-ironic-inspector-ubuntu-trusty/769552f/logs/old/devstacklog.txt.gz#_2017-01-18_14_55_55_053 | 15:40 |
*** rbartal has quit IRC | 15:40 | |
jroll | dtantsur: I guess it's fine to backport that thing, and just probably let this be solved by separate repos for tempest plugins | 15:41 |
*** jaosorior has joined #openstack-ironic | 15:41 | |
dtantsur | yep | 15:41 |
*** kencjohnston_ has quit IRC | 15:42 | |
*** ijw has joined #openstack-ironic | 15:42 | |
*** kencjohnston has joined #openstack-ironic | 15:43 | |
vdrok | rloo: notification payload base has schema, it is just {} -- https://github.com/openstack/ironic/blob/13944b589d2911f4d9a5b8da0a58d1b9354a08bc/ironic/objects/notification.py#L132 so hasattr returns true for that. the logic I'm thinking of is -- class is inherited from NotificationPayloadBase AND its schema is different from the SCHEMA of the first object | 15:43 |
vdrok | that has it in mro | 15:43 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add usage documentation for Baremetal OSC Plugin https://review.openstack.org/415261 | 15:43 |
rloo | vdrok: OH. | 15:45 |
rloo | vdrok: honestly, that *could* happen but if a class is inherited, I'd wonder why the SCHEMA reverts back. | 15:45 |
*** jvgrant__ has quit IRC | 15:45 | |
aarefiev | jroll, dtantsur could be the problem https://github.com/openstack/ironic-inspector/blob/master/devstack/upgrade/settings#L2 | 15:46 |
rloo | vdrok: i suppose the unit test would do a bit more testing then, as opposed to missing a class that it should check | 15:46 |
*** Nisha_Agarwal has joined #openstack-ironic | 15:46 | |
rloo | vdrok: i question how much time we want to spend on this one particular test too. i don't particularly want to compare the .SCHEMA with *all* its parents | 15:47 |
*** david-lyle has joined #openstack-ironic | 15:47 | |
vdrok | rloo: we can stop the moment we find the first one | 15:47 |
vdrok | because it's mro | 15:47 |
vdrok | the first one in mro will be used as its SCHEMA | 15:48 |
vdrok | left a comment on the patch | 15:48 |
vdrok | I'm fine with landing as is tho | 15:48 |
rloo | vdrok: do you know how many times i looked at that NotificationPayloadBase code and did NOT see SCHEMA? I think it is because I hate comments that are not separated by a blank space; i just lumped all that to VERSION and didn't see SCHEMA :-( | 15:48 |
*** hw_wutianwei has quit IRC | 15:48 | |
*** ijw has quit IRC | 15:48 | |
vdrok | rloo: hah, yeah, this happens sometimes :) | 15:49 |
rloo | vdrok: i'll 'think' about it. working on priority reviews first :) | 15:49 |
vdrok | sure, np | 15:50 |
amoralej | dtantsur, should https://bugs.launchpad.net/tripleo/+bug/1651719 be fixed now? | 15:50 |
openstack | Launchpad bug 1651719 in Ironic Inspector "Introspection fails on nodes with the same IPMI address but different IPMI ports" [High,In progress] - Assigned to milan k (vetrisko) | 15:50 |
*** Sukhdev has joined #openstack-ironic | 15:51 | |
*** jistr|mtg is now known as jistr | 15:51 | |
lucasagomes | folks if you have a time today mind taking a look at https://review.openstack.org/#/c/374988/ ? Last change for being able to test UEFI images with Ironic (then we just need project-config stuff) | 15:52 |
patchbot | patch 374988 - ironic - DevStack: Configure nodes/environment to boot in U... | 15:52 |
dtantsur | amoralej, not entirely fixed, but should be enough to try re-enabling introspection in RDO CI | 15:52 |
*** udesale has quit IRC | 15:53 | |
amoralej | dtantsur, iiuc https://review.openstack.org/#/c/419526/ just disabled introspection when ipmi address is pointing to 127.0.0.1? | 15:53 |
patchbot | patch 419526 - ironic-inspector - Do not use loopback BMC addresses for lookup (MERGED) | 15:53 |
rloo | jroll: will wait for jenkins before +A'ing tooz patch. is there a nova one too? | 15:53 |
dtantsur | amoralej, no, it just avoids using this 127.0.0.1 for anything. introspection should still work, with lookup based on MAC addresses (like before for pxe_ssh) | 15:54 |
amoralej | ok | 15:54 |
amoralej | i'll try to re-enable it | 15:55 |
dtantsur | amoralej, thanks! feel free to ping me/us in case of any new problems | 15:55 |
amoralej | ok | 15:55 |
jroll | rloo: https://review.openstack.org/#/c/399022/ | 15:57 |
patchbot | patch 399022 - nova - Move to tooz hash ring implementation | 15:57 |
*** trown|mtg is now known as trown | 15:57 | |
*** Sukhdev_ has joined #openstack-ironic | 15:57 | |
rloo | thx jroll (since I'm on a sort of roll...) | 15:57 |
jroll | rloo: no problem, thanks for the reviews :) | 15:58 |
* rloo is really postponing looking at all the driver composition patches... | 15:58 | |
jroll | rloo: the bottom patch has a -1, so wait for an update :) | 16:00 |
rloo | jroll: ok, good to know! | 16:00 |
*** rajinir has joined #openstack-ironic | 16:00 | |
*** yufei has quit IRC | 16:01 | |
*** Syed__ has joined #openstack-ironic | 16:02 | |
*** igordcard has quit IRC | 16:05 | |
openstackgerrit | Charlle Daniel proposed openstack/ironic: Ensures that OneView nodes are free for use by Ironic https://review.openstack.org/422064 | 16:06 |
*** mjura has quit IRC | 16:07 | |
*** igordcard has joined #openstack-ironic | 16:07 | |
*** sacharya has joined #openstack-ironic | 16:09 | |
*** jvgrant has joined #openstack-ironic | 16:10 | |
*** ijw has joined #openstack-ironic | 16:10 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP/DNM Testing tempest "smoke" 2 https://review.openstack.org/407258 | 16:16 |
*** rama_y has joined #openstack-ironic | 16:17 | |
*** rama_y_ has joined #openstack-ironic | 16:17 | |
*** dyasny has quit IRC | 16:20 | |
*** dyasny has joined #openstack-ironic | 16:20 | |
*** dyasny has quit IRC | 16:20 | |
*** dyasny has joined #openstack-ironic | 16:21 | |
openstackgerrit | Merged openstack/python-ironicclient: Add soft reboot/poweroff power states. https://review.openstack.org/247904 | 16:22 |
*** jheroux has quit IRC | 16:23 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient: Support soft reboot and soft power off with timeout for OSC https://review.openstack.org/357627 | 16:24 |
*** Guest88977 is now known as jlvillal | 16:25 | |
jlvillal | vsaienk0, jroll: So why does "TEMPEST_ALLOW_TENANT_ISOLATION" need to be set? | 16:26 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add VIFs attach/detach to api-ref https://review.openstack.org/414590 | 16:26 |
jlvillal | That is the default setting. | 16:26 |
jlvillal | vsaienk0, jroll: So why does "TEMPEST_ALLOW_TENANT_ISOLATION=True" need to be set? | 16:26 |
jlvillal | That is the default setting. | 16:26 |
jroll | jlvillal: oh, is it? idk, I saw the test patch needed it, so I wasn't sure | 16:28 |
jroll | if it's the default it's fine, I can change my vote | 16:28 |
jlvillal | jroll, Let me find it in the code and post it... | 16:28 |
EmilienM | dtantsur: I'm watching zuul now, it's running. We'll know in a few min | 16:30 |
vsaienk0 | jroll, jlvillal yeah it is set by default to true http://logs.openstack.org/60/414160/15/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial-nv/c42e3a6/logs/tempest_conf.txt.gz | 16:30 |
*** aNuposic has joined #openstack-ironic | 16:31 | |
*** SerenaFeng has quit IRC | 16:33 | |
vsaienk0 | jlvillal: https://github.com/openstack-dev/devstack/blob/9a133d1ab07cacbcc01e9568f3a65c5594ef0723/lib/tempest#L568 | 16:33 |
dtantsur | EmilienM, cool | 16:33 |
dtantsur | lucasagomes, EmilienM, ironic CI seems to pass with glance V2 | 16:34 |
jlvillal | vsaienk0, Thanks. That is what I thought. Sorry my browser crashed :( | 16:34 |
*** jheroux has joined #openstack-ironic | 16:34 | |
jlvillal | vsaienk0, So I was late trying to post the link.. | 16:34 |
EmilienM | dtantsur: yeah? wow | 16:34 |
NobodyCam | good morning ironic | 16:34 |
*** rloo has quit IRC | 16:34 | |
dtantsur | EmilienM, apparently: https://review.openstack.org/422002 (I'll double-check in logs later) | 16:35 |
patchbot | patch 422002 - ironic - Switch to Glance V2 by default for everything | 16:35 |
Ng | dtantsur: ping me when you're ready? :) | 16:35 |
jroll | ah ok, thanks jlvillal and vsaienk0, I've +1'd the thing | 16:36 |
*** ashestakov has quit IRC | 16:36 | |
*** rloo has joined #openstack-ironic | 16:36 | |
vsaienk0 | JayF have you dig enough with nested KVM problem? I'm interested if VM always failed to start on 'strange' host with enabled kvm. Might be we may not just check that kvm is enabled while also try to start test VM and see if it succeed? | 16:36 |
NobodyCam | hello there Ng ... long time no see | 16:36 |
Ng | hey NobodyCam :D | 16:36 |
Ng | NobodyCam: I typed that into the wrong channel, so I'll disappear back into the shadows now ;) | 16:37 |
jroll | vsaienk0: it isn't always, just sometimes | 16:37 |
*** causten has quit IRC | 16:37 | |
*** causten has joined #openstack-ironic | 16:37 | |
vsaienk0 | jroll: wondered that why virtualbmc reports that node was started why in fact it wasn't. We may fix vbmc than. | 16:38 |
jroll | vsaienk0: because it is started, and then kvm falls over | 16:39 |
NobodyCam | hehehe | 16:39 |
*** harlowja has joined #openstack-ironic | 16:39 | |
vsaienk0 | jroll: but the domain status is never changed to running it left in shutoff state no? | 16:40 |
*** trown is now known as trown|lunch | 16:40 | |
jroll | vsaienk0: I don't know | 16:40 |
jroll | I don't have time to look into this, rather focus on priorities, disable it and try again in pike imo | 16:40 |
* jroll trusts clarkb on this topic | 16:40 | |
vsaienk0 | jroll: according to log it should stay in shutoff state http://logs.openstack.org/60/414160/15/check/gate-tempest-dsvm-ironic-multitenant-network-ubuntu-xenial/0099b51/logs/libvirt/qemu/node-0.txt.gz#_2017-01-18_12_27_27_216 | 16:41 |
vsaienk0 | but in the next line: 2017-01-18 12:28:02.634+0000: starting up... | 16:42 |
jroll | vsaienk0: I think that's likely a second startup | 16:43 |
jroll | but yes, I see what you mean | 16:43 |
*** mgoddard has quit IRC | 16:43 | |
JayF | vsaienk0: the reasons I proposed that were: 1) If it's a config issue, we don't have access to host configs 2) Everyone in infra I've talked to, at summits, in irc, on ml, etc, have all said nested kvm is trouble | 16:43 |
jroll | still not sure how fixing vbmc would deal with that | 16:43 |
JayF | vsaienk0: I'm not saying it's not fixable, I just am not convinced openstack CI controls enough of the elements to ensure it to work reliably | 16:44 |
*** milan has quit IRC | 16:47 | |
JayF | I differ from jroll in that I don't think weshould try it again in Pike unless we have specific strategies in mind for mitigating the failures | 16:48 |
jroll | well, sure | 16:48 |
jroll | lemme rephrase | 16:48 |
jroll | "I think we should disable it for the rest of ocata and go from there" | 16:49 |
TheJulia | +1 | 16:49 |
*** e0ne has quit IRC | 16:50 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Switch to Glance V2 by default for everything https://review.openstack.org/422002 | 16:51 |
dtantsur | TheJulia, lucasagomes, EmilienM, just updated the release note ^^^ | 16:51 |
EmilienM | dtantsur: tripleo job still running | 16:52 |
*** amarao has quit IRC | 16:52 | |
*** tuanla_ has joined #openstack-ironic | 16:55 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Validate the generated swift temp url https://review.openstack.org/370875 | 16:55 |
*** tuan_luong has quit IRC | 16:56 | |
dtantsur | lucasagomes, please see https://github.com/openstack/ironic/blob/95de92c63798830cc3f387909491eec635d116e0/ironic/common/images.py#L402 | 16:56 |
*** ChubYann has joined #openstack-ironic | 16:57 | |
lucasagomes | dtantsur, yeah it says v1 can't be used... urgh... | 16:58 |
*** ohamada has quit IRC | 17:00 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Validate the generated swift temp url https://review.openstack.org/370875 | 17:02 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 17:07 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 17:08 |
*** causten has quit IRC | 17:08 | |
*** causten has joined #openstack-ironic | 17:09 | |
openstackgerrit | Doug Hellmann proposed openstack/ironic-lib: Update UPPER_CONSTRAINTS_FILE for stable/ocata https://review.openstack.org/422112 | 17:10 |
openstackgerrit | Stenio Araujo proposed openstack/ironic: Adds another validation step when using dynamic allocation https://review.openstack.org/422118 | 17:14 |
*** tesseract has quit IRC | 17:21 | |
*** tuanla_ has quit IRC | 17:27 | |
*** harlowja has quit IRC | 17:33 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 17:33 |
*** baoli has joined #openstack-ironic | 17:33 | |
*** aNuposic has quit IRC | 17:35 | |
*** aNuposic has joined #openstack-ironic | 17:35 | |
*** e0ne has joined #openstack-ironic | 17:36 | |
*** rama_y_ has quit IRC | 17:36 | |
*** rama_y has quit IRC | 17:36 | |
EmilienM | dtantsur: looks like it doesn't work in tripleo | 17:37 |
EmilienM | but I'll wait for logs to see if the reason is the same | 17:37 |
*** david-lyle is now known as bailing-wire | 17:37 | |
*** vsaienko has joined #openstack-ironic | 17:40 | |
*** ccamacho has quit IRC | 17:42 | |
openstackgerrit | Merged openstack/python-ironicclient: Support soft reboot and soft power off with timeout for OSC https://review.openstack.org/357627 | 17:43 |
vdrok | good night everyone! | 17:45 |
TheJulia | goodnight vdrok | 17:45 |
*** bailing-wire has quit IRC | 17:46 | |
NobodyCam | night vdrok | 17:47 |
lucasagomes | vdrok, night | 17:48 |
* lucasagomes is going as well | 17:48 | |
lucasagomes | have a great evening all! | 17:48 |
*** lucasagomes is now known as lucas-afk | 17:48 | |
*** amoralej is now known as amoralej|off | 17:48 | |
rloo | good night lucas-afk, vdrok! | 17:51 |
*** vsaienko has quit IRC | 17:52 | |
*** vsaienko has joined #openstack-ironic | 17:53 | |
*** vsaienko has quit IRC | 17:55 | |
*** trown|lunch is now known as trown | 17:59 | |
*** derekh has quit IRC | 18:01 | |
*** jcoufal has quit IRC | 18:03 | |
openstackgerrit | Xavier proposed openstack/ironic: Adds validation before delete profile in oneview https://review.openstack.org/386014 | 18:04 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Testing multinode job https://review.openstack.org/422149 | 18:04 |
*** ralonsoh has quit IRC | 18:05 | |
*** yolanda has quit IRC | 18:06 | |
*** yolanda has joined #openstack-ironic | 18:06 | |
*** athomas has quit IRC | 18:09 | |
dtantsur | going as well, g'night | 18:10 |
*** Sukhdev has quit IRC | 18:11 | |
*** dtantsur is now known as dtantsur|afk | 18:11 | |
TheJulia | goodnight dtantsur|afk | 18:11 |
*** sambetts is now known as sambetts|afk | 18:11 | |
*** gcb has quit IRC | 18:18 | |
*** e0ne has quit IRC | 18:19 | |
*** jerrygb has quit IRC | 18:23 | |
*** sacharya has quit IRC | 18:27 | |
*** rama_y_ has joined #openstack-ironic | 18:32 | |
*** rama_y has joined #openstack-ironic | 18:32 | |
*** Goneri has quit IRC | 18:33 | |
*** absubram has joined #openstack-ironic | 18:35 | |
*** krtaylor has quit IRC | 18:36 | |
openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: snmp inspection for disk size https://review.openstack.org/394858 | 18:41 |
*** vsaienko has joined #openstack-ironic | 18:45 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 18:46 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add VIFs attach/detach to api-ref https://review.openstack.org/414590 | 18:47 |
*** aNuposic has quit IRC | 18:56 | |
*** aNuposic has joined #openstack-ironic | 18:58 | |
*** Sukhdev_ has quit IRC | 18:59 | |
EmilienM | dtantsur|afk: http://logs.openstack.org/14/417214/10/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/b3baf50/logs/undercloud/var/log/ironic/ironic-conductor.txt.gz#_2017-01-18_17_35_02_383 | 19:02 |
EmilienM | ah he left | 19:02 |
*** krtaylor has joined #openstack-ironic | 19:03 | |
*** jcoufal has joined #openstack-ironic | 19:09 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Tempest tests for portgroups https://review.openstack.org/382476 | 19:11 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: DNM: Test patch to verify bond https://review.openstack.org/388660 | 19:11 |
*** absubram has quit IRC | 19:17 | |
*** aNuposic has quit IRC | 19:19 | |
*** absubram has joined #openstack-ironic | 19:22 | |
*** causten_ has joined #openstack-ironic | 19:23 | |
*** jerrygb has joined #openstack-ironic | 19:24 | |
rloo | jroll: am looking at gate status on whiteboard. are stable/newton & stable/mitaka gates still broken? | 19:25 |
*** vsaienko has quit IRC | 19:26 | |
*** causten has quit IRC | 19:26 | |
*** dyasny has quit IRC | 19:28 | |
xavierr | hey TheJulia, could you look https://review.openstack.org/#/c/386014/ :) | 19:28 |
patchbot | patch 386014 - ironic - Adds validation before delete profile in oneview | 19:28 |
*** jerrygb has quit IRC | 19:29 | |
TheJulia | rloo: they look okay based on the revs that are in review | 19:31 |
rloo | TheJulia: that's what I thought. Also, I meant to ping jlvillal, sorry jroll (hope he's ignoring me) | 19:32 |
*** sacharya has joined #openstack-ironic | 19:33 | |
jlvillal | rloo: Yes stable/newton is working. I have not checked stable/mitaka personally | 19:33 |
rloo | jlvillal: thx -- would you please update the whiteboard with whatever latest status on those? | 19:33 |
jlvillal | rloo: Will do | 19:34 |
rloo | jlvillal: :) | 19:34 |
*** rpioso has quit IRC | 19:35 | |
jlvillal | rloo: Moved data to history page. I checked stable/newton & stable/mitaka are working. Or were as of 2 days ago. | 19:37 |
rloo | jlvillal: sweet. On to the next breakage. or not... | 19:37 |
*** trown is now known as trown|mtg | 19:37 | |
jlvillal | I'm sure it will happen, some time... | 19:37 |
*** Sukhdev has joined #openstack-ironic | 19:37 | |
TheJulia | xavierr: is there a bug associated with that issue? | 19:40 |
*** baoli has quit IRC | 19:42 | |
*** Nisha_Agarwal has quit IRC | 19:43 | |
*** dyasny has joined #openstack-ironic | 19:43 | |
xavierr | TheJulia: reporting the bug now, ty Julia :) | 19:46 |
TheJulia | xavierr: thank you, I posted what were largely nits :) | 19:46 |
xavierr | awesome o/ | 19:46 |
*** mriedem has joined #openstack-ironic | 19:53 | |
*** e0ne has joined #openstack-ironic | 19:58 | |
*** galyna has quit IRC | 19:59 | |
*** vsaienko has joined #openstack-ironic | 19:59 | |
*** bailing-wire has joined #openstack-ironic | 20:00 | |
*** bailing-wire is now known as david-lyle | 20:02 | |
*** dsneddon is now known as dsneddon_afk | 20:04 | |
*** jkilpatr has quit IRC | 20:05 | |
*** jerrygb has joined #openstack-ironic | 20:05 | |
rloo | jroll: sorry to bug you, should i hold off looking at https://review.openstack.org/#/c/412631/ ? | 20:05 |
patchbot | patch 412631 - ironic - Register/unregister hardware interfaces for conduc... | 20:05 |
jroll | rloo: sorry, I haven't got to it today. won't touch it for another hour if you want to leave me things to fix, vdrok's things are small | 20:06 |
*** jkilpatr has joined #openstack-ironic | 20:06 | |
* jroll replies to comments quickly | 20:06 | |
rloo | jroll: ok, if it really close, i'll look at it :) | 20:06 |
*** aNuposic has joined #openstack-ironic | 20:09 | |
*** rpioso has joined #openstack-ironic | 20:10 | |
*** aNuposic has quit IRC | 20:10 | |
*** krtaylor has quit IRC | 20:10 | |
*** vsaienko has quit IRC | 20:11 | |
*** aNuposic has joined #openstack-ironic | 20:11 | |
*** jerrygb has quit IRC | 20:13 | |
*** trown|mtg is now known as trown | 20:15 | |
*** galyna has joined #openstack-ironic | 20:16 | |
*** Goneri has joined #openstack-ironic | 20:18 | |
mriedem | rloo: are you or anyone else interested in touching up https://review.openstack.org/#/c/388756/ for the test thing and maybe leaving some comments about cardinality for ports/portgroups per node etc? | 20:24 |
patchbot | patch 388756 - nova - Generate necessary network metadata for ironic por... | 20:24 |
mriedem | rloo: seems like simple small changes to make and then i'd be +2 | 20:24 |
rloo | mriedem: i could do it, worth it for your +2 :) | 20:24 |
mriedem | strike while the iron is hot | 20:25 |
rloo | mriedem: yup. and anything to avoid reviewing jroll's patches, which I had *just* geared myself to start :) | 20:25 |
*** krtaylor has joined #openstack-ironic | 20:25 | |
mriedem | poor jroll | 20:25 |
jroll | wow | 20:25 |
rloo | jroll: unless you want to update the nova patch? (i'm offering...) | 20:26 |
jroll | rloo: I have a meeting in 3 minutes | 20:26 |
jroll | so not really :/ | 20:26 |
rloo | jroll: more fun than me ;) | 20:26 |
jroll | you don't know what the meeting is! | 20:27 |
rloo | jroll: nor do i want to know! or do I? :) | 20:27 |
jroll | no, it's nothing terribly interesting | 20:27 |
mriedem | alternatively i'll update the nova change | 20:28 |
mriedem | but.. | 20:28 |
jroll | I'll get it after my meeting if rloo does not | 20:28 |
*** jrcloud is now known as zackf | 20:32 | |
*** vsaienko has joined #openstack-ironic | 20:38 | |
*** vsaienko has quit IRC | 20:42 | |
*** absubram has quit IRC | 20:43 | |
*** baoli has joined #openstack-ironic | 20:44 | |
*** ppiela has joined #openstack-ironic | 20:44 | |
*** causten_ has quit IRC | 20:47 | |
*** causten has joined #openstack-ironic | 20:50 | |
*** e0ne has quit IRC | 20:51 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Enable OSProfiler support in Ironic https://review.openstack.org/358142 | 20:53 |
*** jerrygb has joined #openstack-ironic | 20:53 | |
TheJulia | cinerama: you may want to take a look at https://review.openstack.org/#/c/396504/ | 20:57 |
patchbot | patch 396504 - diskimage-builder - Update hpssacli to ssacli in proliant-tools element | 20:57 |
TheJulia | I was building a example slide for a presentation and noticed the element was broken :( | 20:57 |
*** absubram has joined #openstack-ironic | 20:59 | |
*** srobert_ has joined #openstack-ironic | 20:59 | |
*** ijw has quit IRC | 21:01 | |
*** srobert has quit IRC | 21:03 | |
*** srobert_ has quit IRC | 21:04 | |
*** jheroux has quit IRC | 21:05 | |
mriedem | rloo: you *just* beat me to it | 21:06 |
rloo | mriedem: oh, sorry, didn't realize you were updating it. | 21:06 |
mriedem | no it's fine | 21:07 |
mriedem | i wasn't until about 30 seconds ago | 21:07 |
rloo | mriedem: i couldn't answer your cloudinit question. and i didn't change the elif. going to comment on that... | 21:07 |
mriedem | vdrok told me in -nova "cloud init does not validate parameters" | 21:08 |
mriedem | i assume related to the question about those things being documented anywhere | 21:08 |
*** jkilpatr has quit IRC | 21:08 | |
*** jheroux has joined #openstack-ironic | 21:10 | |
rloo | mriedem: yeah. anyway, feel free to update that patch after my updates. i know why i stay away from nova, the local unit tests take too long ;) | 21:10 |
mriedem | that's fine, +2 | 21:11 |
mriedem | thanks | 21:11 |
mriedem | i hope you at least ran pep8 | 21:12 |
rloo | mriedem: thank YOU! :D | 21:12 |
rloo | mriedem: what's pep8? | 21:12 |
rloo | mriedem: j/k | 21:12 |
rloo | mriedem: yes i did | 21:12 |
JayF | it's the newest release of pep, at least 10x peppier than pep7 | 21:13 |
rloo | JayF: I did wonder if it was short for pepto bismol :) | 21:15 |
* mriedem cues the rimshot | 21:15 | |
rloo | :D | 21:16 |
*** ijw has joined #openstack-ironic | 21:16 | |
*** absubram has quit IRC | 21:21 | |
*** jkilpatr has joined #openstack-ironic | 21:22 | |
EmilienM | dtantsur|afk: nevermind, it looks like it worked :D | 21:22 |
*** vsaienko has joined #openstack-ironic | 21:23 | |
*** krtaylor has quit IRC | 21:30 | |
*** jerrygb_ has joined #openstack-ironic | 21:30 | |
*** jcoufal_ has joined #openstack-ironic | 21:33 | |
*** jerrygb has quit IRC | 21:33 | |
*** absubram has joined #openstack-ironic | 21:33 | |
*** jcoufal has quit IRC | 21:35 | |
*** jcoufal__ has joined #openstack-ironic | 21:35 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: WIP: use werkzeug https://review.openstack.org/422255 | 21:35 |
*** jcoufal_ has quit IRC | 21:38 | |
*** jcoufal has joined #openstack-ironic | 21:38 | |
*** jcoufal__ has quit IRC | 21:40 | |
*** jcoufal_ has joined #openstack-ironic | 21:40 | |
openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 21:41 |
*** trown is now known as trown|outtypewww | 21:42 | |
*** ijw has quit IRC | 21:42 | |
*** jcoufal has quit IRC | 21:43 | |
*** priteau has quit IRC | 21:58 | |
*** jcoufal has joined #openstack-ironic | 21:58 | |
*** amotoki has quit IRC | 21:58 | |
*** dprince has quit IRC | 21:59 | |
*** jcoufal_ has quit IRC | 22:00 | |
*** jcoufal_ has joined #openstack-ironic | 22:07 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic: Default ironic to not use nested KVM https://review.openstack.org/421581 | 22:09 |
JayF | vsaienk0: ^ I think I fixed the issue you found, if you have a chance to take another loo | 22:09 |
JayF | *look | 22:09 |
*** jcoufal__ has joined #openstack-ironic | 22:10 | |
*** jcoufal has quit IRC | 22:10 | |
*** jcoufal_ has quit IRC | 22:12 | |
*** ijw has joined #openstack-ironic | 22:13 | |
JayF | mjturek: how'd that patch work go? | 22:15 |
*** jheroux has quit IRC | 22:16 | |
*** Goneri has quit IRC | 22:19 | |
*** jcoufal__ has quit IRC | 22:19 | |
mriedem | jroll: i see that the dependency for this is merged in ironic now, and the ironiclient change itself is merged but unreleased https://review.openstack.org/#/c/247904/ | 22:29 |
patchbot | patch 247904 - python-ironicclient - Add soft reboot/poweroff power states. (MERGED) | 22:29 |
mriedem | got plans to release that thing this week? | 22:29 |
openstackgerrit | Anup Navare proposed openstack/ironic: Config drive support for Ceph https://review.openstack.org/407639 | 22:29 |
jroll | mriedem: yep, I'll do it now | 22:29 |
mriedem | cool | 22:29 |
jroll | haven't had time to check in on that today | 22:29 |
jroll | thanks | 22:29 |
*** vsaienko has quit IRC | 22:31 | |
*** david-lyle has quit IRC | 22:32 | |
jroll | mriedem: https://review.openstack.org/#/q/topic:python-ironicclient-1.10.0 | 22:34 |
*** sacharya has quit IRC | 22:39 | |
*** david-lyle has joined #openstack-ironic | 22:40 | |
*** pmannidi has joined #openstack-ironic | 22:54 | |
*** amotoki has joined #openstack-ironic | 22:59 | |
rloo | JayF: if you want a break and want to review a doc-like patch: https://review.openstack.org/#/c/414590/ | 23:01 |
patchbot | patch 414590 - ironic - Add VIFs attach/detach to api-ref | 23:01 |
*** [1]cdearborn has quit IRC | 23:01 | |
rloo | JayF: how'd you review that so fast? you da man... | 23:02 |
JayF | I was looking right at the channel as you posted | 23:03 |
JayF | read it real quick and landed it | 23:03 |
JayF | those api-ref pieces are autogenerated so I have a lot of faith in the output once the script is updated | 23:03 |
rloo | JayF: takes me that long just to look up and grok what someone is saying on irc ;) | 23:04 |
*** Goneri has joined #openstack-ironic | 23:04 | |
JayF | I may obsessively watch irc :-x | 23:04 |
rloo | JayF: I'm going to take off soon. you could help jroll (if he's ok with it) by fixing this: https://review.openstack.org/#/c/399028/10 | 23:07 |
patchbot | patch 399028 - ironic - Move to tooz hash ring implementation | 23:07 |
rloo | ^^ or anyone else. Easy change :) | 23:08 |
*** absubram has quit IRC | 23:09 | |
openstackgerrit | Merged openstack/ironic: Add VIFs attach/detach to api-ref https://review.openstack.org/414590 | 23:09 |
*** galyna has quit IRC | 23:24 | |
*** fungi has quit IRC | 23:24 | |
*** jralph has quit IRC | 23:24 | |
*** ricardoas has quit IRC | 23:24 | |
*** stevemar has quit IRC | 23:24 | |
*** galyna has joined #openstack-ironic | 23:24 | |
*** fungi has joined #openstack-ironic | 23:24 | |
*** jralph has joined #openstack-ironic | 23:24 | |
*** ricardoas has joined #openstack-ironic | 23:24 | |
*** stevemar has joined #openstack-ironic | 23:24 | |
*** aignatov has quit IRC | 23:34 | |
*** aignatov has joined #openstack-ironic | 23:35 | |
*** jkilpatr has quit IRC | 23:35 | |
*** sacharya has joined #openstack-ironic | 23:41 | |
*** jerrygb_ has quit IRC | 23:42 | |
*** Sukhdev has quit IRC | 23:45 | |
*** jerrygb has joined #openstack-ironic | 23:48 | |
*** Goneri has quit IRC | 23:48 | |
*** jaosorior has quit IRC | 23:49 | |
*** mriedem has left #openstack-ironic | 23:53 | |
*** aignatov has quit IRC | 23:57 | |
*** aignatov has joined #openstack-ironic | 23:57 | |
openstackgerrit | Merged openstack/ironic-python-agent: Skip API related work if no api url configured https://review.openstack.org/408222 | 23:59 |
*** aignatov has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!