*** harlowja has joined #openstack-ironic | 00:06 | |
*** bdeschenes has joined #openstack-ironic | 00:22 | |
*** hoangcx_ has quit IRC | 00:29 | |
*** xek_ has quit IRC | 00:35 | |
*** robcresswell has quit IRC | 00:35 | |
*** xek_ has joined #openstack-ironic | 00:35 | |
*** robcresswell has joined #openstack-ironic | 00:37 | |
*** david-lyle_ has joined #openstack-ironic | 00:42 | |
*** krtaylor has joined #openstack-ironic | 00:44 | |
*** david-lyle has quit IRC | 00:45 | |
*** mtanino has quit IRC | 00:47 | |
*** hoangcx has joined #openstack-ironic | 00:53 | |
*** rloo has quit IRC | 00:56 | |
*** phuongnh has joined #openstack-ironic | 00:59 | |
*** xuao has joined #openstack-ironic | 01:03 | |
*** yuanying has quit IRC | 01:03 | |
*** [1]cdearborn has quit IRC | 01:04 | |
*** yonglihe has joined #openstack-ironic | 01:06 | |
*** yuanying has joined #openstack-ironic | 01:07 | |
*** baoli has joined #openstack-ironic | 01:14 | |
*** zhugaoxiao has joined #openstack-ironic | 01:16 | |
*** tuanluong has joined #openstack-ironic | 01:18 | |
openstackgerrit | Merged openstack/ironic: No need to clear "target_provision_state" again from conductor https://review.openstack.org/380951 | 01:18 |
---|---|---|
*** yuanying_ has joined #openstack-ironic | 01:23 | |
*** bdeschenes has quit IRC | 01:25 | |
*** yuanying has quit IRC | 01:26 | |
openstackgerrit | Xavier proposed openstack/ironic: Reusing oneview_client when possible https://review.openstack.org/358041 | 01:28 |
*** jaosorior has quit IRC | 01:40 | |
*** jaosorior has joined #openstack-ironic | 01:40 | |
*** dprince has joined #openstack-ironic | 01:47 | |
*** zackf has joined #openstack-ironic | 01:51 | |
*** zackf has quit IRC | 01:52 | |
*** sdake_ is now known as sdake | 02:01 | |
*** zackf has joined #openstack-ironic | 02:01 | |
*** dprince has quit IRC | 02:04 | |
*** syed_ has quit IRC | 02:04 | |
*** dprince has joined #openstack-ironic | 02:05 | |
*** zackf has quit IRC | 02:13 | |
*** thrash is now known as thrash|g0ne | 02:24 | |
*** dprince has quit IRC | 02:25 | |
*** zackf has joined #openstack-ironic | 02:25 | |
*** zackf has quit IRC | 02:30 | |
*** mtanino has joined #openstack-ironic | 02:35 | |
*** rama_y has quit IRC | 02:39 | |
*** rbudden has joined #openstack-ironic | 02:48 | |
*** yuanying_ has quit IRC | 02:51 | |
*** yuanying has joined #openstack-ironic | 02:52 | |
*** hamzy has joined #openstack-ironic | 02:53 | |
*** MattMan has quit IRC | 02:53 | |
*** MattMan has joined #openstack-ironic | 02:53 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Update guide for PXE multi-architecture setup https://review.openstack.org/386313 | 02:59 |
*** zhangyufei_ has joined #openstack-ironic | 03:05 | |
*** baoli has quit IRC | 03:12 | |
*** yuanying has quit IRC | 03:15 | |
*** yuanying has joined #openstack-ironic | 03:15 | |
*** yuanying_ has joined #openstack-ironic | 03:17 | |
*** yuanying has quit IRC | 03:18 | |
*** yuanying_ has quit IRC | 03:22 | |
*** mtanino has quit IRC | 03:34 | |
*** hj-hpe has quit IRC | 03:36 | |
*** zackf has joined #openstack-ironic | 03:41 | |
*** zackf has quit IRC | 03:47 | |
*** mtanino has joined #openstack-ironic | 03:52 | |
*** yuanying has joined #openstack-ironic | 03:52 | |
*** vmud213 has joined #openstack-ironic | 03:53 | |
*** mtanino has quit IRC | 03:57 | |
*** links has joined #openstack-ironic | 03:59 | |
*** zackf has joined #openstack-ironic | 04:05 | |
*** zackf has quit IRC | 04:09 | |
*** gzholtkevych has quit IRC | 04:12 | |
*** hj-hpe has joined #openstack-ironic | 04:12 | |
*** hj-hpe has quit IRC | 04:12 | |
*** absubram has joined #openstack-ironic | 04:14 | |
*** david-lyle_ has quit IRC | 04:39 | |
*** david-lyle has joined #openstack-ironic | 04:39 | |
openstackgerrit | KATO Tomoyuki proposed openstack/python-ironicclient: Add plug-in summary for osc doc https://review.openstack.org/386333 | 04:46 |
*** zhangyufei_ has quit IRC | 04:48 | |
*** e0ne has joined #openstack-ironic | 04:48 | |
*** jaosorior has quit IRC | 04:49 | |
*** e0ne has quit IRC | 04:51 | |
openstackgerrit | Nam Nguyen Hoai proposed openstack/ironic-lib: Fix typo in error message https://review.openstack.org/378470 | 04:51 |
*** e0ne has joined #openstack-ironic | 04:55 | |
*** e0ne has quit IRC | 05:02 | |
*** e0ne has joined #openstack-ironic | 05:14 | |
*** ChubYann has quit IRC | 05:17 | |
*** e0ne has quit IRC | 05:17 | |
*** e0ne has joined #openstack-ironic | 05:21 | |
*** sdake has quit IRC | 05:23 | |
*** e0ne has quit IRC | 05:27 | |
*** e0ne has joined #openstack-ironic | 05:28 | |
*** rbrud has joined #openstack-ironic | 05:30 | |
*** e0ne has quit IRC | 05:31 | |
*** e0ne has joined #openstack-ironic | 05:36 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/386361 | 05:36 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/386362 | 05:36 |
pmannidi | lazy_prince, ping | 05:37 |
*** e0ne has quit IRC | 05:39 | |
*** mmedvede has quit IRC | 05:42 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/385746 | 05:43 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/386376 | 05:43 |
*** amotoki has joined #openstack-ironic | 05:51 | |
*** bks has joined #openstack-ironic | 05:52 | |
*** mjura has joined #openstack-ironic | 06:00 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: fix a typo in document https://review.openstack.org/386387 | 06:03 |
*** mjura has quit IRC | 06:09 | |
*** mjura has joined #openstack-ironic | 06:09 | |
*** tuanluong has quit IRC | 06:10 | |
openstackgerrit | Andy Yan proposed openstack/ironic-specs: Power RSD servers using Valence https://review.openstack.org/384032 | 06:11 |
*** yonglihe has quit IRC | 06:16 | |
openstackgerrit | Nam Nguyen Hoai proposed openstack/ironic-python-agent: TrivialFix: Fix typo https://review.openstack.org/379112 | 06:22 |
*** pcaruana has joined #openstack-ironic | 06:23 | |
*** xuao has quit IRC | 06:25 | |
*** vmud213 has quit IRC | 06:27 | |
*** vmud213 has joined #openstack-ironic | 06:27 | |
*** phuongnh has quit IRC | 06:29 | |
*** phuongnh has joined #openstack-ironic | 06:30 | |
*** makowals has quit IRC | 06:33 | |
*** makowals has joined #openstack-ironic | 06:40 | |
*** zackf has joined #openstack-ironic | 06:42 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Zanata https://review.openstack.org/386402 | 06:44 |
*** zackf has quit IRC | 06:46 | |
*** vsaienko has joined #openstack-ironic | 06:55 | |
*** zackf has joined #openstack-ironic | 06:58 | |
*** phuongnh has quit IRC | 07:00 | |
*** zackf has quit IRC | 07:03 | |
*** ralonsoh has joined #openstack-ironic | 07:03 | |
*** makowals has quit IRC | 07:04 | |
*** tesseract- has joined #openstack-ironic | 07:05 | |
*** zackf has joined #openstack-ironic | 07:10 | |
*** zackf has quit IRC | 07:16 | |
*** vsaienko has quit IRC | 07:17 | |
*** rcernin has joined #openstack-ironic | 07:20 | |
*** makowals has joined #openstack-ironic | 07:20 | |
openstackgerrit | haris proposed openstack/python-ironicclient: Enables ironic list commands to use labels https://review.openstack.org/380671 | 07:32 |
bks | hi, what is the use case of deb-python-ironic-lib project in ironic? | 07:37 |
*** hshiina has joined #openstack-ironic | 07:41 | |
*** makowals has quit IRC | 07:42 | |
*** ccamacho has joined #openstack-ironic | 07:47 | |
*** vsaienko has joined #openstack-ironic | 07:52 | |
*** makowals has joined #openstack-ironic | 07:54 | |
openstackgerrit | haris proposed openstack/python-ironicclient: Enables ironic list commands to use labels https://review.openstack.org/380671 | 07:56 |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:03 | |
*** makowals has quit IRC | 08:05 | |
*** jtomasek_ has joined #openstack-ironic | 08:12 | |
*** athomas has joined #openstack-ironic | 08:14 | |
*** zhugaoxiao has quit IRC | 08:16 | |
*** zhugaoxiao has joined #openstack-ironic | 08:16 | |
*** jaosorior has joined #openstack-ironic | 08:19 | |
*** tangchen has joined #openstack-ironic | 08:24 | |
milan | morning Ironic! :) | 08:27 |
*** jtomasek_ has quit IRC | 08:27 | |
*** makowals has joined #openstack-ironic | 08:29 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui: Imported Translations from Zanata https://review.openstack.org/385832 | 08:34 |
tangchen | morning milan, and ironic | 08:35 |
milan | morning tangchen! :) | 08:35 |
*** zackf has joined #openstack-ironic | 08:41 | |
*** dnikishov has joined #openstack-ironic | 08:43 | |
*** zackf has quit IRC | 08:45 | |
*** phuongnh has joined #openstack-ironic | 08:49 | |
*** dtantsur|afk is now known as dtantsur | 08:54 | |
dtantsur | Morning Ironic | 08:54 |
tangchen | morning dtantsur | 08:55 |
*** tuanluong has joined #openstack-ironic | 09:03 | |
*** mgould|afk is now known as mgould | 09:04 | |
mgould | morning Ironic! | 09:04 |
*** milan has quit IRC | 09:06 | |
tangchen | hi mgould, morning | 09:06 |
*** milan has joined #openstack-ironic | 09:06 | |
milan | morning dtantsur! :) | 09:13 |
milan | and morning all! My connection is flaky today. | 09:14 |
dtantsur | hey mgould, milan | 09:15 |
milan | morning mgould! :) | 09:15 |
milan | aarefiev, morning! Could you please check https://review.openstack.org/#/c/344921/25/ironic_inspector/conf.py@28 I'd like to avoid a third rewrite here ;) | 09:18 |
patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 09:18 |
dtantsur | huh, why is API version history empty? http://docs.openstack.org/developer/ironic/dev/webapi.html#rest-api-versions-history | 09:20 |
*** zackf has joined #openstack-ironic | 09:30 | |
*** e0ne has joined #openstack-ironic | 09:31 | |
*** vsaienko has quit IRC | 09:31 | |
*** vsaienko has joined #openstack-ironic | 09:32 | |
*** e0ne has quit IRC | 09:33 | |
*** zackf has quit IRC | 09:34 | |
*** e0ne has joined #openstack-ironic | 09:38 | |
*** hshiina has quit IRC | 09:39 | |
milan | sambetts_ morning! :) could you please check: https://review.openstack.org/#/c/344921/25/doc/source/http-api.rst@85 I'd like to get your feedback before I proceed with the patch | 09:40 |
patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 09:40 |
*** jaosorior has quit IRC | 09:40 | |
*** jaosorior has joined #openstack-ironic | 09:40 | |
*** amotoki has quit IRC | 09:41 | |
openstackgerrit | pawnesh kumar proposed openstack/ironic: doc:Log warning when can't get information from git https://review.openstack.org/382713 | 09:42 |
*** thrash|g0ne is now known as thrash | 09:42 | |
*** openstackstatus has quit IRC | 09:44 | |
*** openstack has joined #openstack-ironic | 09:46 | |
*** bks has quit IRC | 09:49 | |
*** amotoki has joined #openstack-ironic | 09:55 | |
openstackgerrit | Merged openstack/python-ironicclient: OSC plugin support microversions 1.21 & 1.22 https://review.openstack.org/366275 | 10:00 |
*** tuanluong has quit IRC | 10:03 | |
*** amotoki has quit IRC | 10:04 | |
*** zackf has joined #openstack-ironic | 10:06 | |
*** milan has quit IRC | 10:06 | |
*** milan has joined #openstack-ironic | 10:07 | |
*** zackf has quit IRC | 10:11 | |
*** openstackstatus has quit IRC | 10:13 | |
*** openstack has joined #openstack-ironic | 10:15 | |
*** hoangcx has quit IRC | 10:15 | |
*** dtantsur is now known as dtantsur|bbl | 10:20 | |
*** bks has joined #openstack-ironic | 10:21 | |
*** phuongnh has quit IRC | 10:27 | |
*** hshiina has joined #openstack-ironic | 10:28 | |
*** hoangcx has joined #openstack-ironic | 10:28 | |
*** derekh has joined #openstack-ironic | 10:32 | |
*** ohamada has joined #openstack-ironic | 10:34 | |
*** zackf has joined #openstack-ironic | 10:37 | |
*** [1]cdearborn has joined #openstack-ironic | 10:41 | |
*** zackf has quit IRC | 10:42 | |
*** fxpester has joined #openstack-ironic | 10:43 | |
*** pester has quit IRC | 10:45 | |
*** amotoki has joined #openstack-ironic | 10:49 | |
*** ralonsoh_ has quit IRC | 10:52 | |
*** dprince has joined #openstack-ironic | 11:04 | |
*** [1]cdearborn has quit IRC | 11:04 | |
*** hoangcx_ has joined #openstack-ironic | 11:05 | |
*** hoangcx has quit IRC | 11:06 | |
jroll | morning y'all | 11:10 |
jroll | bks: deb-python-ironic-lib is part of the debian packaging project, and makes debian packages for the ironic-lib library, which is some code shared between some ironic projects | 11:11 |
jroll | dtantsur|bbl: O_o | 11:11 |
*** mmedvede has joined #openstack-ironic | 11:13 | |
*** bks has quit IRC | 11:13 | |
*** zackf has joined #openstack-ironic | 11:18 | |
mgould | morning jroll milan tangchen dtantsur|bbl sambetts_ | 11:19 |
milan | morning jroll, mgould! :) | 11:19 |
mgould | dtantsur|bbl: http://docs.openstack.org/developer/ironic/dev/webapi-version-history.html still has the info, at least | 11:22 |
*** zackf has quit IRC | 11:22 | |
*** zackf has joined #openstack-ironic | 11:25 | |
* mgould can't see any difference between the code in webapi.rst and the code for other toctrees that work, other than one extra space of indentation | 11:26 | |
*** e0ne has quit IRC | 11:28 | |
*** zackf has quit IRC | 11:29 | |
*** e0ne has joined #openstack-ironic | 11:29 | |
*** ohamada has quit IRC | 11:33 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_targets table to save target information https://review.openstack.org/285218 | 11:34 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce VolumeTarget object https://review.openstack.org/285219 | 11:34 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce new RPCs to support volume target operations https://review.openstack.org/285220 | 11:34 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connectors and volume_targets to task https://review.openstack.org/366196 | 11:35 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fix API docs to include API version history https://review.openstack.org/386548 | 11:38 |
jroll | dtantsur|bbl: mgould: I think that's the issue but haven't rendered locally | 11:39 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Convert ironic.conf to template generated https://review.openstack.org/305634 | 11:39 |
*** joprovost has joined #openstack-ironic | 11:42 | |
*** strigazi is now known as strigazi_AFK | 11:48 | |
jroll | yep, that did it | 11:51 |
*** zackf has joined #openstack-ironic | 11:56 | |
mgould | jroll: removing the space? | 11:56 |
jroll | mgould: removing dev/ | 11:58 |
jroll | it's a relative path, they're in the same directory | 11:58 |
*** joprovost1 has joined #openstack-ironic | 11:58 | |
*** joprovost has quit IRC | 11:59 | |
*** joprovost1 is now known as joprovost | 11:59 | |
*** baoli has joined #openstack-ironic | 11:59 | |
*** baoli_ has joined #openstack-ironic | 12:00 | |
*** zackf has quit IRC | 12:01 | |
*** lucas-afk is now known as lucasagomes | 12:01 | |
lucasagomes | morning all | 12:02 |
*** links has quit IRC | 12:02 | |
jroll | morning lucasagomes | 12:03 |
*** baoli has quit IRC | 12:03 | |
mgould | morning lucasagomes | 12:06 |
mgould | jroll: ah, ofc | 12:06 |
*** awiddersheim has quit IRC | 12:13 | |
*** awiddersheim has joined #openstack-ironic | 12:13 | |
*** jcoufal has joined #openstack-ironic | 12:14 | |
*** links has joined #openstack-ironic | 12:14 | |
*** amotoki has quit IRC | 12:15 | |
*** trown|outtypewww is now known as trown | 12:19 | |
*** krtaylor has quit IRC | 12:29 | |
*** zackf has joined #openstack-ironic | 12:31 | |
*** joprovost has quit IRC | 12:31 | |
*** jcoufal_ has joined #openstack-ironic | 12:32 | |
openstackgerrit | Ricardo Araújo Santos proposed openstack/ironic: Fixes parameters validation in SSH power manager. https://review.openstack.org/366150 | 12:35 |
*** jcoufal has quit IRC | 12:35 | |
jroll | dtantsur|bbl: mat128: I added a thing at line 55 here, surprised it didn't come up already: https://etherpad.openstack.org/p/ironic-ocata-summit-deploy-time-raid | 12:35 |
*** vmud213 has quit IRC | 12:35 | |
*** zackf has quit IRC | 12:35 | |
*** jcoufal has joined #openstack-ironic | 12:36 | |
jroll | vsaienk0: vdrok: can you all drop some context in here? https://etherpad.openstack.org/p/ironic-ocata-summit-neutron-events | 12:37 |
jroll | (when you have time, maybe monday) | 12:37 |
*** jcoufal_ has quit IRC | 12:38 | |
jroll | mat128: ditto here? https://etherpad.openstack.org/p/ironic-ocata-summit-vnc-console | 12:38 |
vdrok | jroll: we will, a holiday in Ukraine today :) | 12:40 |
*** openstackstatus has quit IRC | 12:43 | |
*** openstack has joined #openstack-ironic | 12:45 | |
mat128 | good morning jroll | 12:49 |
mat128 | was on pto yesterday, going through everything i missed this morning | 12:49 |
*** mjura has quit IRC | 12:49 | |
mat128 | jroll: line 55 on raid, lmao, most obvious solution isnt always the first thought of | 12:50 |
mat128 | jroll: re: vnc console, we are (downstream) not changing anything in nova, besides implementing get_vnc_console in the ironic driver | 12:52 |
mat128 | ah | 12:52 |
mat128 | will do today | 12:52 |
*** skazi has quit IRC | 12:54 | |
jlvillal | dtantsur|bbl, I added 'type hints' to https://etherpad.openstack.org/p/ironic-ocata-summit-qa :) Might be cool/interesting if we were the first OpenStack project that added type hints to their code. | 12:55 |
*** gcb has quit IRC | 12:57 | |
*** sdake_ has joined #openstack-ironic | 13:03 | |
jroll | mat128: :D thanks | 13:06 |
jroll | vdrok: awesome, thanks, enjoy the holiday :) | 13:06 |
jroll | jlvillal: heh, from another channel: 11:31:36 jroll | begin influx of discussion on making everything in openstack typed | 13:07 |
jroll | only took 1.5 hours | 13:07 |
*** zackf has joined #openstack-ironic | 13:07 | |
*** zackf has quit IRC | 13:07 | |
*** Goneri has joined #openstack-ironic | 13:07 | |
*** zackf has joined #openstack-ironic | 13:07 | |
*** bdeschenes has joined #openstack-ironic | 13:10 | |
*** jheroux has joined #openstack-ironic | 13:11 | |
*** links has quit IRC | 13:16 | |
*** daemontool has joined #openstack-ironic | 13:22 | |
mgould | morning mat128 | 13:22 |
mgould | jroll: what was the conclusion? | 13:22 |
jroll | mgould: of? the docs thing? | 13:23 |
mgould | or am I being overly optimistic in expecting there was one? :-) | 13:23 |
mgould | of the "type all the things" discussion | 13:23 |
jroll | oh | 13:23 |
jroll | jlvillal just started it ;) | 13:23 |
mgould | heh | 13:23 |
* mgould runs and hides | 13:23 | |
jroll | my comment was in reply to http://blog.zulip.org/2016/10/13/static-types-in-python-oh-mypy/ | 13:23 |
*** portdirect has quit IRC | 13:24 | |
*** portdirect has joined #openstack-ironic | 13:24 | |
mgould | jroll: looks interesting | 13:25 |
*** [1]cdearborn has joined #openstack-ironic | 13:28 | |
*** jroll is now known as jrollinhatin | 13:29 | |
*** joprovost has joined #openstack-ironic | 13:30 | |
*** makowals_ has joined #openstack-ironic | 13:31 | |
*** vsaienko has joined #openstack-ironic | 13:31 | |
*** vsaienko has quit IRC | 13:31 | |
*** makowals has quit IRC | 13:33 | |
*** jcoufal has quit IRC | 13:33 | |
*** jcoufal has joined #openstack-ironic | 13:35 | |
*** dtantsur|bbl is now known as dtantsur | 13:39 | |
dtantsur | morning jroll, mat128, jlvillal and everyone else | 13:40 |
jrollinhatin | morning dtantsur :) | 13:40 |
dtantsur | jrollinhatin, extending block_device_mapping_v2, hmm... interesting, I wonder what nova folks think about it | 13:45 |
mariojv | morning ironic o/ | 13:45 |
jrollinhatin | dtantsur: right, that's the hard part :) | 13:45 |
jrollinhatin | dtantsur: that's probably the best user-facing option, though, right? | 13:50 |
jrollinhatin | or the most sensible, at least | 13:50 |
*** pester has joined #openstack-ironic | 13:51 | |
*** jcoufal has quit IRC | 13:52 | |
*** mgoddard has joined #openstack-ironic | 13:52 | |
*** dstepanenko has quit IRC | 13:53 | |
*** ccamacho is now known as ccamacho|afk | 13:53 | |
*** dansmith is now known as superdan | 13:53 | |
*** jcoufal has joined #openstack-ironic | 13:53 | |
*** dstepanenko has joined #openstack-ironic | 13:53 | |
*** fxpester has quit IRC | 13:53 | |
*** mgoddard has quit IRC | 13:59 | |
*** mgoddard_ has joined #openstack-ironic | 13:59 | |
*** makowals_ has quit IRC | 14:00 | |
*** sdake has joined #openstack-ironic | 14:00 | |
*** anush has joined #openstack-ironic | 14:01 | |
jlvillal | jrollinhatin: Ah, you read the same Hacker News link I did. Though I did have a summit proposal about "our experiences adding type hints to an OpenStack project" | 14:02 |
*** rbrud has joined #openstack-ironic | 14:02 | |
jrollinhatin | jlvillal: neat :) | 14:02 |
jrollinhatin | I didn't see it on HN, but yeah | 14:03 |
jlvillal | Luckily we didn't get chosen, because then we would have had to add type hints to an OpenStack projects. We were thinking ironic-lib :) | 14:03 |
jrollinhatin | heh | 14:03 |
*** makowals has joined #openstack-ironic | 14:04 | |
mgould | jrollinhatin: can I add something to https://etherpad.openstack.org/p/ironic-ocata-summit-qa ? | 14:04 |
jrollinhatin | mgould: of course | 14:04 |
*** joprovost has quit IRC | 14:04 | |
*** sdake_ has quit IRC | 14:04 | |
mgould | excellent | 14:04 |
jrollinhatin | these are for the community :) | 14:04 |
* jlvillal wonders if there is a "Hatin, Michigan"? | 14:04 | |
*** mgoddard_ has quit IRC | 14:04 | |
mgould | done | 14:05 |
jrollinhatin | jlvillal: nah, just a Hell, MI | 14:05 |
lucasagomes | jlvillal, jrollinhatin what was that on hacker news? | 14:06 |
jlvillal | heh. I was trying to decipher your nick | 14:06 |
mgould | jlvillal: there's a Hatín, Czech Republic... | 14:06 |
jrollinhatin | lucasagomes: http://blog.zulip.org/2016/10/13/static-types-in-python-oh-mypy/ | 14:06 |
* lucasagomes reads, ouch | 14:06 | |
jlvillal | lucasagomes: https://news.ycombinator.com/ | 14:06 |
mgould | jlvillal: https://www.youtube.com/watch?v=CtwJvgPJ9xw | 14:06 |
jrollinhatin | https://www.youtube.com/watch?v=CtwJvgPJ9xw | 14:06 |
jlvillal | lucasagomes: Is Hacker News | 14:06 |
jrollinhatin | heh | 14:06 |
*** mgoddard has joined #openstack-ironic | 14:06 | |
lucasagomes | jlvillal, heh I know hn, just wondering which news was it | 14:06 |
jrollinhatin | jlvillal: and all sorts of associated memes http://weknowmemes.com/wp-content/uploads/2012/06/hamster-wagon-they-see-me-rollin.jpg | 14:07 |
jlvillal | lucasagomes: And the one jrollinhatin posted. About using Python type hints | 14:07 |
jrollinhatin | jlvillal: and then... you know my last name :) | 14:07 |
* jlvillal has so many links to open | 14:07 | |
jrollinhatin | heh | 14:07 |
jrollinhatin | mgould: I haven't actually listened to this in forever | 14:07 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 14:08 |
mgould | jrollinhatin: me neither | 14:08 |
* jlvillal remembers that video | 14:08 | |
mgould | I've listened to Weird Al's version more recently, though :-) | 14:08 |
jrollinhatin | jlvillal: fyi, that youtube video almost certainly has nsfw language, I haven't actually seen the video so can't comment on the visuals | 14:08 |
jlvillal | I have no good idea on how to make a fun Friday nick with 'jlvillal' | 14:08 |
* mgould wonders if Hell, MI is named after https://en.wikipedia.org/wiki/Hell,_Norway, but apparently not | 14:08 | |
*** joprovost has joined #openstack-ironic | 14:08 | |
jlvillal | jrollinhatin: I turned my laptop speakers down from 100%. Luckily it is 7am here. | 14:09 |
mgould | the perfect time for hip-hop :-) | 14:09 |
jrollinhatin | heh | 14:09 |
mgould | "Temperatures in Hell can reach −25 °C (−13 °F)[4] during winter." | 14:10 |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Add instructions to deploy bifrost on virsh https://review.openstack.org/383792 | 14:10 |
jlvillal | mgould: Hell does freeze over ;) | 14:10 |
mgould | apparently so :-) | 14:10 |
jlvillal | TheJulia: hexchat + ZNC instead of irccloud? | 14:13 |
* jlvillal reads some infra backlog | 14:13 | |
TheJulia | jlvillal: heh, I might take a look at that... or just killing all autocorrect | 14:14 |
jlvillal | TheJulia: ZNC is pretty cool. I setup a $5 DigitalOcean VM and run it there. | 14:15 |
jlvillal | TheJulia: I did it like dhellmann suggested here: https://doughellmann.com/blog/2015/03/12/deploying-nested-znc-services-with-ansible/ | 14:15 |
jlvillal | TheJulia: Do you know how to use Ansible? :P | 14:16 |
* jrollinhatin <3 tmux+weechat | 14:16 | |
* TheJulia tosses a wetcat at jlvillal | 14:16 | |
* jlvillal knows TheJulia knows much much much more than he does about Ansible. | 14:16 | |
TheJulia | :) | 14:16 |
jlvillal | jrollinhatin: I see weechat and I think of that program that is huge in China. | 14:17 |
*** mgoddard has quit IRC | 14:17 | |
jrollinhatin | ? | 14:17 |
jlvillal | jrollinhatin: Instead of WhatsApp or Viber they use WeChat. http://www.businessinsider.com/wechat-why-it-dominates-china-2015-8 | 14:18 |
jrollinhatin | ah :) | 14:18 |
*** anush has quit IRC | 14:20 | |
jrollinhatin | people might find this useful, get a python shell with Node/Port/etc and an admin context imported: https://gist.github.com/jimrollenhagen/da36a646af9555a36060fbac94658f5f | 14:22 |
* lucasagomes has an internal vm running ZNC too | 14:22 | |
jrollinhatin | worth tidying up and dropping in ironic/tools/ ? or just throw it in a random repo somewhere | 14:23 |
* mgould votes for ironic/tools | 14:23 | |
*** krtaylor has joined #openstack-ironic | 14:25 | |
* sambetts_ uses a rasppi as a ZNC server locally | 14:28 | |
*** sambetts_ is now known as sambetts | 14:28 | |
*** mtanino has joined #openstack-ironic | 14:28 | |
jlvillal | sambetts: co-worker of mine does that too. Maybe I should since I have three unused Raspberry Pi systems (of different versions) laying around my house :( | 14:31 |
lucasagomes | folks two small patches that will complete the root device hints work: https://review.openstack.org/386204 (Ironic) and https://review.openstack.org/366742 (IPA) | 14:31 |
patchbot | patch 386204 - ironic - Replace parse_root_device_hints with the ironic-li... | 14:31 |
patchbot | patch 366742 - ironic-python-agent - Add support for root device hints with operators | 14:31 |
lucasagomes | when you get some time, thanks! | 14:32 |
* lucasagomes needs to work on docs still | 14:32 | |
*** fxpester has joined #openstack-ironic | 14:33 | |
*** pas-ha has quit IRC | 14:33 | |
*** vdrok has quit IRC | 14:34 | |
*** vdrok has joined #openstack-ironic | 14:34 | |
*** pas-ha has joined #openstack-ironic | 14:34 | |
*** pester has quit IRC | 14:35 | |
*** mgoddard has joined #openstack-ironic | 14:36 | |
*** rama_y has joined #openstack-ironic | 14:36 | |
jrollinhatin | lucasagomes: +2 with nits on the ironic patch, thanks! | 14:36 |
jrollinhatin | easy reviews indeed | 14:37 |
lucasagomes | jrollinhatin, cool! I will take a look | 14:37 |
lucasagomes | yeah, it's just reusing things from ironic-lib | 14:37 |
jrollinhatin | yep | 14:37 |
*** sdake_ has joined #openstack-ironic | 14:38 | |
*** sdake_ has quit IRC | 14:40 | |
*** vishwanathj has joined #openstack-ironic | 14:41 | |
*** sdake has quit IRC | 14:41 | |
*** amotoki has joined #openstack-ironic | 14:41 | |
*** pester has joined #openstack-ironic | 14:44 | |
*** michauds has joined #openstack-ironic | 14:44 | |
*** fxpester has quit IRC | 14:44 | |
*** pester has quit IRC | 14:45 | |
*** bdeschenes has quit IRC | 14:48 | |
*** joprovost1 has joined #openstack-ironic | 14:49 | |
*** amotoki has quit IRC | 14:50 | |
*** joprovost has quit IRC | 14:51 | |
*** joprovost1 is now known as joprovost | 14:51 | |
*** hshiina has quit IRC | 14:52 | |
*** sdake has joined #openstack-ironic | 14:53 | |
*** portdirect has quit IRC | 14:57 | |
*** portdirect_ has joined #openstack-ironic | 14:59 | |
*** portdirect_ is now known as portdirect | 14:59 | |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Mock ironic-lib properly in test_deploy_utils https://review.openstack.org/386255 | 15:00 |
*** mgoddard has quit IRC | 15:01 | |
*** rcernin has quit IRC | 15:02 | |
*** e0ne has quit IRC | 15:02 | |
*** mgoddard has joined #openstack-ironic | 15:03 | |
*** sdake_ has joined #openstack-ironic | 15:03 | |
*** strigazi_AFK is now known as strigazi | 15:04 | |
*** daemontool_ has joined #openstack-ironic | 15:05 | |
*** sdake has quit IRC | 15:05 | |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Add instructions to deploy bifrost on virsh https://review.openstack.org/383792 | 15:07 |
*** daemontool_ has quit IRC | 15:08 | |
*** daemontool has quit IRC | 15:08 | |
*** ccamacho|afk has quit IRC | 15:10 | |
*** e0ne has joined #openstack-ironic | 15:12 | |
*** lucasagomes is now known as lucas-hungry | 15:15 | |
*** pcaruana has quit IRC | 15:15 | |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 15:15 |
*** syed_ has joined #openstack-ironic | 15:19 | |
*** e0ne_ has joined #openstack-ironic | 15:20 | |
*** e0ne has quit IRC | 15:21 | |
*** bnemec is now known as beekneemech | 15:29 | |
*** narasimha_SV has joined #openstack-ironic | 15:40 | |
narasimha_SV | http://paste.openstack.org/show/585723/ | 15:40 |
narasimha_SV | according to the logs after deploying the image to the disk | 15:40 |
narasimha_SV | while rebooting the system it is getting failed | 15:41 |
*** mgoddard has quit IRC | 15:49 | |
*** sdake has joined #openstack-ironic | 15:54 | |
*** tesseract- has quit IRC | 15:54 | |
JayF | vsaienk0: around/ | 15:55 |
*** rbrud has quit IRC | 15:55 | |
JayF | vsaienk0: I approved https://review.openstack.org/#/c/337223/ last night, and this mornign when to mark the bug rfe-approved... only to see it was associated with the same RFE bug as lucas's work. Can you file a RFE bug specific to shipping syslog logs, and change the pointer in the spec in your followup? | 15:56 |
patchbot | patch 337223 - ironic-specs - Collect Ironic agent ramdisk logs (MERGED) | 15:56 |
*** sdake_ has quit IRC | 15:56 | |
*** penick has joined #openstack-ironic | 16:00 | |
*** narasimha_SV has quit IRC | 16:02 | |
*** derekh has quit IRC | 16:16 | |
*** lucas-hungry is now known as lucasagomes | 16:16 | |
NobodyCam | Good Morning Ironic'ers ... and .... (ofc) ..... | 16:17 |
NobodyCam | TGIF! | 16:17 |
lucasagomes | NobodyCam, morning! | 16:23 |
NobodyCam | Morning lucasagomes :) | 16:23 |
*** rbrud has joined #openstack-ironic | 16:25 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Testing multi-node https://review.openstack.org/383241 | 16:25 |
*** xek_ has quit IRC | 16:25 | |
*** xek_ has joined #openstack-ironic | 16:26 | |
openstackgerrit | abdul nizamuddin proposed openstack/ironic: Constraints are ready to be used for tox.ini https://review.openstack.org/386684 | 16:27 |
mgould | morning NobodyCam! | 16:35 |
* mgould -> weekend; have a great weekend, everyone! | 16:36 | |
NobodyCam | Happy Friday mgould | 16:36 |
NobodyCam | have a great weekend | 16:36 |
NobodyCam | I'll see you Tuesday | 16:36 |
*** sdake_ has joined #openstack-ironic | 16:38 | |
*** sdake has quit IRC | 16:40 | |
lucasagomes | mgould, enjoy it! | 16:41 |
jlvillal | mgould: That Hypothesis looks interesting | 16:42 |
dtantsur | morning NobodyCam, jlvillal, JayF; and now g'night everyone and have a nice weekend | 16:46 |
JayF | o/ | 16:47 |
NobodyCam | Good morning and happy Friday dtantsur | 16:47 |
*** dtantsur is now known as dtantsur|afk | 16:47 | |
NobodyCam | morning JayF jlvillal :) | 16:47 |
jlvillal | Hi dtantsur|afk :) And good bye I guess. | 16:47 |
*** aNuposic has joined #openstack-ironic | 16:50 | |
*** xek_ has quit IRC | 16:55 | |
*** mgoddard has joined #openstack-ironic | 16:56 | |
*** xek_ has joined #openstack-ironic | 16:56 | |
jlvillal | mariojv: So on the unit test patch. I think it should be a two-parter. | 17:00 |
*** phschwartz has quit IRC | 17:00 | |
*** mgoddard has quit IRC | 17:00 | |
jlvillal | mariojv: Ping me when you get a moment please. | 17:00 |
*** athomas has quit IRC | 17:00 | |
*** jcoufal has quit IRC | 17:05 | |
*** jcoufal has joined #openstack-ironic | 17:07 | |
*** michauds has quit IRC | 17:12 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Enable inspector discovery by default https://review.openstack.org/386701 | 17:14 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Enhance introspection status https://review.openstack.org/386703 | 17:15 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/386704 | 17:15 |
TheJulia | NobodyCam: https://review.openstack.org/386701 may interest you | 17:17 |
patchbot | patch 386701 - bifrost - Enable inspector discovery by default | 17:17 |
*** links has joined #openstack-ironic | 17:18 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 17:18 |
*** trown is now known as trown|lunch | 17:18 | |
*** e0ne_ has quit IRC | 17:20 | |
*** athomas has joined #openstack-ironic | 17:21 | |
*** jcoufal has quit IRC | 17:22 | |
*** jcoufal has joined #openstack-ironic | 17:22 | |
JayF | So question for folks | 17:29 |
JayF | do you think devstack should validate IRONIC_DEPLOY_DRIVER? | 17:30 |
JayF | I just had an issue where someone failed to stack properly because the driver name was typo'd, and it just seems like something we should check for people | 17:30 |
JayF | but wanted a 2nd opinion before filing a bug about it | 17:30 |
lucasagomes | JayF, it failed cause the driver didn't exist right (and ironic failed to create the node with it) ? | 17:31 |
lucasagomes | cause recently someone added a valid driver (pxe_vbox) but devstack didn't configured it correctly | 17:31 |
JayF | it was 100% a typo'd driver (agent_ipmi vs agent_ipmitool) | 17:32 |
lucasagomes | it finished with success but the node was misconfigured (I think devstack assumed the driver was pxe_ssh) | 17:32 |
JayF | and it failed silently | 17:32 |
milan | JayF, I think a patch/bug that reduces folk's frustration is a good idea | 17:32 |
lucasagomes | JayF, oh that's bad then, so we need to check for valid drivers yes | 17:32 |
lucasagomes | IMHO | 17:32 |
JayF | Cool, we're all on board then. I'll file the bug and mark it low-hanging-fruit | 17:32 |
milan | JayF, ++ | 17:32 |
jrollinhatin | grep -q $IRONIC_DEPLOY_DRIVER /opt/stack/ironic/setup.cfg; if [[ $? != 0 ]] boom | 17:34 |
JayF | https://bugs.launchpad.net/ironic/+bug/1633567 | 17:35 |
openstack | Launchpad bug 1633567 in Ironic "Devstack fails silently when provided an invalid driver" [Undecided,New] | 17:35 |
JayF | jrollinhatin: also we should check to make sure it's in IROINC_ENABLED_DRIVERS | 17:36 |
jrollinhatin | JayF: ya | 17:36 |
*** alex_xu_ has quit IRC | 17:39 | |
*** heyongli has quit IRC | 17:40 | |
*** jaosorior has quit IRC | 17:40 | |
xavierr | when using iPXE the TFTP server is used to do anything? Because the images transference occurs only by HTTP, so no TFTP needed in this case, right? | 17:40 |
*** phschwartz has joined #openstack-ironic | 17:41 | |
*** jaosorior has joined #openstack-ironic | 17:41 | |
jrollinhatin | xavierr: it's used to transfer the ipxe image for NICs that don't have ipxe flashed | 17:41 |
*** joprovost has quit IRC | 17:42 | |
*** alex_xu has joined #openstack-ironic | 17:42 | |
*** absubram has quit IRC | 17:43 | |
*** heyongli has joined #openstack-ironic | 17:43 | |
xavierr | hey jrollinhatin, what you mean by flashed? that doesn't has ipxe capabilities? | 17:43 |
jrollinhatin | xavierr: yes, not all nics have ipxe built in | 17:44 |
JayF | xavierr: so most NICs boot "regular" pxe, they would use "regular" pxe to tftp over ipxe, then ipxe doesn't need tftp anymore | 17:44 |
*** narasimha_SV has joined #openstack-ironic | 17:44 | |
narasimha_SV | : http://paste.openstack.org/show/585723/ | 17:45 |
xavierr | jrollinhatin: u said 'ipxe image' anything special configuration in the images when using ipxe? | 17:45 |
jrollinhatin | xavierr: no, it's just a small binary that's installed by whatever OS package installs ipxe | 17:45 |
* jrollinhatin finds config option | 17:45 | |
xavierr | JayF: great! make a lot of sence :) | 17:46 |
jrollinhatin | oh, I guess it isn't configurable, huh, unless it's pxe_bootfile_name and is shared | 17:46 |
*** athomas has quit IRC | 17:47 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Replace parse_root_device_hints with the ironic-lib version one https://review.openstack.org/386204 | 17:48 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Docs: Document using operators with root device hints https://review.openstack.org/386714 | 17:48 |
*** pcaruana has joined #openstack-ironic | 17:48 | |
xavierr | thanks :) | 17:48 |
*** krtaylor has quit IRC | 17:48 | |
openstackgerrit | abdul nizamuddin proposed openstack/ironic: Removing -U flag in install_command https://review.openstack.org/386684 | 17:52 |
narasimha_SV | after writing the image to disk reboot of the server is failed | 17:55 |
narasimha_SV | as per the above logs | 17:55 |
sai | ping ironic | 17:56 |
sai | seeing this in the api after firing the deploy command | 17:56 |
sai | Client-side error: Invalid input for field/attribute addresses. Value: '80:00:02:48:fe:80:00:00:00:00:00:00:f4:52:14:03:00:54:06:c2,f4:52:14:54:06:c1,a0:d3:c1:04:44:17,a0:d3:c1:04:44:14,a0:d3:c1:04:44:16,a0:d3:c1:04:44:15'. unable to convert to list format_exception /usr/lib/python2.7/site-packages/wsme/ | 17:56 |
sai | nodes remain in wait-call-back | 17:56 |
sai | introspection finishes successfully though | 17:57 |
sai | any pointers? | 17:57 |
*** portdirect has quit IRC | 18:02 | |
*** portdirect has joined #openstack-ironic | 18:04 | |
*** jcoufal_ has joined #openstack-ironic | 18:08 | |
*** jcoufal has quit IRC | 18:10 | |
*** nizam037 has joined #openstack-ironic | 18:13 | |
*** Sukhdev has joined #openstack-ironic | 18:17 | |
Sukhdev | sambetts : Ping | 18:17 |
lucasagomes | sai, looks like the node is calling lookup() (can you confirm?) | 18:24 |
lucasagomes | sai, are you using the same version (e.g newton) for the ironic services and deploy ramdisk ? | 18:24 |
*** krtaylor has joined #openstack-ironic | 18:24 | |
*** Sukhdev has quit IRC | 18:25 | |
*** michauds has joined #openstack-ironic | 18:25 | |
*** Sukhdev has joined #openstack-ironic | 18:26 | |
*** Sukhdev has quit IRC | 18:28 | |
*** hoonetorg has quit IRC | 18:30 | |
openstackgerrit | Merged openstack/ironic-python-agent: Add support for root device hints with operators https://review.openstack.org/366742 | 18:31 |
lucasagomes | o/ | 18:31 |
JayF | \o | 18:33 |
narasimha_SV | http://paste.openstack.org/show/585735/ | 18:33 |
*** jcoufal has joined #openstack-ironic | 18:40 | |
*** links has quit IRC | 18:41 | |
*** jcoufal_ has quit IRC | 18:41 | |
*** hoonetorg has joined #openstack-ironic | 18:41 | |
*** penick_ has joined #openstack-ironic | 18:42 | |
*** cppforlife__ has joined #openstack-ironic | 18:43 | |
*** sergek_ has joined #openstack-ironic | 18:43 | |
*** tdurakov_ has joined #openstack-ironic | 18:44 | |
*** vdrok_ has joined #openstack-ironic | 18:44 | |
*** romcheg_ has joined #openstack-ironic | 18:45 | |
*** rama_y has quit IRC | 18:46 | |
*** rama_y has joined #openstack-ironic | 18:47 | |
*** jcook- has joined #openstack-ironic | 18:49 | |
*** odyssey4me_ has joined #openstack-ironic | 18:49 | |
*** evgenyl has quit IRC | 18:50 | |
*** sukhdev has joined #openstack-ironic | 18:51 | |
*** natorious_ has joined #openstack-ironic | 18:51 | |
*** hoangcx has joined #openstack-ironic | 18:51 | |
*** penick has quit IRC | 18:51 | |
*** vdrok has quit IRC | 18:51 | |
*** mmedvede has quit IRC | 18:51 | |
*** hoangcx_ has quit IRC | 18:51 | |
*** francoblanco has quit IRC | 18:51 | |
*** ElCoyote_ has quit IRC | 18:51 | |
*** Marx314 has quit IRC | 18:51 | |
*** tdurakov has quit IRC | 18:51 | |
*** NobodyCam has quit IRC | 18:51 | |
*** cppforlife_ has quit IRC | 18:51 | |
*** bcornec1 has quit IRC | 18:51 | |
*** odyssey4me has quit IRC | 18:51 | |
*** natorious has quit IRC | 18:51 | |
*** jistr has quit IRC | 18:51 | |
*** fungi has quit IRC | 18:51 | |
*** mrtenio has quit IRC | 18:51 | |
*** d0ugal has quit IRC | 18:51 | |
*** jcook has quit IRC | 18:51 | |
*** sergek has quit IRC | 18:51 | |
*** romcheg has quit IRC | 18:51 | |
*** _fortis has quit IRC | 18:51 | |
*** jcook- is now known as jcook | 18:51 | |
*** penick_ is now known as penick | 18:51 | |
*** natorious_ is now known as natorious | 18:51 | |
*** odyssey4me_ is now known as odyssey4me | 18:51 | |
*** vdrok_ is now known as vdrok | 18:52 | |
*** evgenyl has joined #openstack-ironic | 18:52 | |
*** romcheg_ is now known as romcheg | 18:52 | |
*** sergek_ is now known as sergek | 18:52 | |
*** tdurakov_ is now known as tdurakov | 18:52 | |
*** rama_y has quit IRC | 18:53 | |
narasimha_SV | ironic ipmitool drivers both are failing to power on after writing image to disk | 18:53 |
*** cppforlife__ is now known as cppforlife_ | 18:54 | |
*** jistr has joined #openstack-ironic | 18:54 | |
openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/386361 | 18:54 |
sukhdev | sambetts: Will you be able to chair ironic-neutron meeting on Monday? I have a Dr. appointment | 18:54 |
*** jcoufal has quit IRC | 18:54 | |
openstackgerrit | Merged openstack/ironic: fix a typo in document https://review.openstack.org/386387 | 18:56 |
*** mrtenio has joined #openstack-ironic | 18:56 | |
openstackgerrit | Merged openstack/ironic: Remove "agent_last_heartbeat" internal field from agent drivers https://review.openstack.org/376978 | 18:57 |
milan | good night Ironic! :) | 18:58 |
*** mmedvede has joined #openstack-ironic | 18:58 | |
openstackgerrit | Merged openstack/ironic: Update configure_tenant_networks https://review.openstack.org/356460 | 18:58 |
*** trown|lunch is now known as trown | 18:58 | |
*** bcornec1 has joined #openstack-ironic | 18:59 | |
*** ElCoyote_ has joined #openstack-ironic | 18:59 | |
*** milan has quit IRC | 18:59 | |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Add instructions to deploy bifrost on virsh https://review.openstack.org/383792 | 19:00 |
*** francoblanco has joined #openstack-ironic | 19:00 | |
*** NobodyCam has joined #openstack-ironic | 19:00 | |
*** d0ugal has joined #openstack-ironic | 19:01 | |
*** fungi has joined #openstack-ironic | 19:01 | |
*** _fortis_ has joined #openstack-ironic | 19:01 | |
xavierr | NobodyCam: morning :) | 19:07 |
*** manjeets has quit IRC | 19:08 | |
*** rajinir has joined #openstack-ironic | 19:09 | |
*** _fortis_ is now known as _fortis | 19:09 | |
*** aNuposic has quit IRC | 19:12 | |
*** aNuposic has joined #openstack-ironic | 19:12 | |
NobodyCam | morning xavierr :) | 19:14 |
*** jcoufal has joined #openstack-ironic | 19:17 | |
*** BadCub has quit IRC | 19:20 | |
clif_h | is there a channel dedicated to devstack discussion? | 19:20 |
*** BadCub has joined #openstack-ironic | 19:20 | |
JayF | #openstack-qa | 19:20 |
*** manjeets has joined #openstack-ironic | 19:22 | |
clif_h | ty JayF | 19:22 |
JayF | np | 19:22 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Testing multi-node https://review.openstack.org/383241 | 19:24 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Test timeout extension for inspection https://review.openstack.org/375050 | 19:27 |
*** joprovost has joined #openstack-ironic | 19:47 | |
*** jcoufal has quit IRC | 19:51 | |
lucasagomes | folks, I'm calling it a day! | 19:53 |
lucasagomes | have a great evening all, enjoy the weekend | 19:53 |
*** lucasagomes is now known as lucas-afk | 19:54 | |
*** joprovost1 has joined #openstack-ironic | 19:54 | |
*** joprovost has quit IRC | 19:55 | |
*** joprovost1 is now known as joprovost | 19:55 | |
*** clenimar has quit IRC | 19:58 | |
*** jaosorior has quit IRC | 19:59 | |
*** aNuposic has quit IRC | 20:07 | |
mariojv | jlvillal: i think that patch is mocking out work_on_disk correctly | 20:09 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone installation playbook https://review.openstack.org/380225 | 20:09 |
mariojv | jlvillal: all the ironic_lib calls in the patch are done externally by ironic itself | 20:09 |
mariojv | jlvillal: agree that work_on_disk needs testing in ironic_lib though, if it's not tested already | 20:10 |
jlvillal | mariojv: Really? That function that is being called looks like five lines. | 20:10 |
jlvillal | mariojv: But I haven't looked closely though. | 20:10 |
jlvillal | mariojv: So more than likely you are correct :) | 20:10 |
*** rama_y has joined #openstack-ironic | 20:10 | |
mariojv | jlvillal: so what i did (i may have messed up one of them?) is look at everything in ironic_lib that test_deploy_utils calls, mock that | 20:11 |
mariojv | and mock out methods within test_deploy_utils that call out to external programs (iscsiadm) | 20:11 |
mariojv | i don't think anything internal to ironic-lib is mocked there anymore | 20:12 |
mariojv | but before merging, i agree it'd be good to make sure those individual methods are united tested in ironic-lib itself | 20:12 |
jlvillal | mariojv: I was looking at: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/deploy_utils.py#L358-L373 | 20:13 |
jlvillal | mariojv: And my thought is that the code should be mocking out disk_utils.work_on_disk() | 20:13 |
jlvillal | mariojv: Which maybe it is???? | 20:14 |
mariojv | jlvillal: L424: https://review.openstack.org/#/c/386255/2/ironic/tests/unit/drivers/modules/test_deploy_utils.py | 20:14 |
patchbot | patch 386255 - ironic - Mock ironic-lib properly in test_deploy_utils | 20:14 |
* jlvillal thinks he gets in trouble for doing a drive-by review | 20:14 | |
mariojv | i think all the other mocks are called from _iscsi_setup_and_handle_errors | 20:14 |
jlvillal | mariojv: Ah. Just surprising that 14 lines of code has so many mocks | 20:15 |
mariojv | yeah :| maybe mocking out the _iscsi_setup... and unit testing it separately is better | 20:16 |
mariojv | if that's not already done, didn't look | 20:16 |
jlvillal | mariojv: I'm not sure. | 20:16 |
jlvillal | mariojv: I removed my -1 as I haven't reviewed it enough at this point. | 20:17 |
jlvillal | I'm sort of stuck in multinode testing land right now. So I don't have time to look more. | 20:17 |
mariojv | no worries, thanks for looking :) | 20:17 |
jlvillal | mariojv: Thank you. | 20:17 |
* jlvillal is learning how to use 'jq' | 20:17 | |
*** pcaruana has quit IRC | 20:18 | |
* jlvillal is also thinking these hacks he is doing are getting ugly... | 20:18 | |
*** strigazi is now known as strigazi_AFK | 20:22 | |
*** joprovost has quit IRC | 20:26 | |
*** jcoufal has joined #openstack-ironic | 20:30 | |
*** mgoddard has joined #openstack-ironic | 20:32 | |
mat128 | jrollinhatin: running late on schedule for VNC console document, lindycoder (coworker of mine) will take care of it since he will be present at the summit | 20:35 |
*** lindycoder has joined #openstack-ironic | 20:38 | |
*** aNuposic- has quit IRC | 20:39 | |
*** mgoddard has quit IRC | 20:40 | |
*** aNuposic has joined #openstack-ironic | 20:40 | |
openstackgerrit | Hugo Nicodemos proposed openstack/python-oneviewclient: Refactor the validation to Server Profile Template https://review.openstack.org/377680 | 20:47 |
*** narasimha_SV has quit IRC | 20:50 | |
*** e0ne has joined #openstack-ironic | 20:50 | |
*** Goneri has quit IRC | 20:54 | |
*** e0ne has quit IRC | 20:55 | |
*** e0ne has joined #openstack-ironic | 20:57 | |
*** Sukhdev_ has joined #openstack-ironic | 20:57 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Cleanup: adding linebreaks for readability https://review.openstack.org/386775 | 21:01 |
*** jcoufal_ has joined #openstack-ironic | 21:04 | |
*** jcoufal has quit IRC | 21:05 | |
*** michauds has quit IRC | 21:06 | |
*** jcoufal has joined #openstack-ironic | 21:07 | |
*** e0ne has quit IRC | 21:08 | |
*** jcoufal_ has quit IRC | 21:09 | |
*** krtaylor has quit IRC | 21:12 | |
*** mgoddard has joined #openstack-ironic | 21:13 | |
jlvillal | jrollinhatin: Did you ever send an email to openstack-dev saying that Ironic will be attending the PTG? I looked but could not find anything. | 21:17 |
jlvillal | jrollinhatin: Just asking since I see a lot of projects sending out that type of email. | 21:17 |
*** sukhdev has quit IRC | 21:21 | |
*** jheroux has quit IRC | 21:23 | |
*** jcoufal_ has joined #openstack-ironic | 21:27 | |
*** trown is now known as trown|outtypewww | 21:29 | |
*** jcoufal has quit IRC | 21:30 | |
*** portdirect has quit IRC | 21:31 | |
*** portdirect_ has joined #openstack-ironic | 21:31 | |
*** portdirect_ is now known as portdirect | 21:31 | |
*** ppiela has quit IRC | 21:32 | |
*** ppiela has joined #openstack-ironic | 21:32 | |
*** jcoufal_ has quit IRC | 21:37 | |
*** dprince has quit IRC | 21:38 | |
*** e0ne has joined #openstack-ironic | 21:40 | |
*** jcoufal has joined #openstack-ironic | 21:42 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for node list and show with specific fields https://review.openstack.org/384923 | 21:45 |
*** Goneri has joined #openstack-ironic | 21:46 | |
*** e0ne has quit IRC | 21:52 | |
*** ppiela has quit IRC | 22:08 | |
*** sdake_ has quit IRC | 22:14 | |
jlvillal | On the devstack-gate.qcow2 image. Does 7-8GB sound about right for the size? | 22:17 |
jlvillal | I just built it. | 22:17 |
jlvillal | whoops, wrong channel | 22:17 |
*** rama_y has quit IRC | 22:20 | |
*** rama_y has joined #openstack-ironic | 22:24 | |
openstackgerrit | Merged openstack/python-oneviewclient: Adds attribute mac_type to a SPT and a validation https://review.openstack.org/377054 | 22:37 |
*** sdake has joined #openstack-ironic | 22:44 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: [WIP] Add storage_interface DB field and object https://review.openstack.org/348005 | 22:47 |
*** rbrud has quit IRC | 22:47 | |
*** jcoufal_ has joined #openstack-ironic | 22:49 | |
*** jcoufal has quit IRC | 22:51 | |
*** jcoufal has joined #openstack-ironic | 22:54 | |
*** jcoufal_ has quit IRC | 22:55 | |
*** sdake has quit IRC | 22:57 | |
*** sdake has joined #openstack-ironic | 22:58 | |
*** sdake has quit IRC | 22:58 | |
*** mgoddard has quit IRC | 23:00 | |
*** jcoufal_ has joined #openstack-ironic | 23:01 | |
*** jcoufal has quit IRC | 23:02 | |
*** penick has quit IRC | 23:03 | |
*** sdake has joined #openstack-ironic | 23:06 | |
*** superdan is now known as dansmith | 23:11 | |
*** mgagne_ is now known as mgagne | 23:12 | |
*** sdake has quit IRC | 23:21 | |
*** Goneri has quit IRC | 23:22 | |
*** rajinir has quit IRC | 23:26 | |
*** jcoufal has joined #openstack-ironic | 23:31 | |
*** rama_y has quit IRC | 23:32 | |
*** lindycoder has quit IRC | 23:32 | |
*** jcoufal_ has quit IRC | 23:33 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!