| rloo | hi gcb, here for a few secs, what's up? | 00:00 |
|---|---|---|
| *** amotoki has quit IRC | 00:00 | |
| *** amotoki has joined #openstack-ironic | 00:01 | |
| *** cdearborn has quit IRC | 00:04 | |
| *** rloo has quit IRC | 00:05 | |
| *** amotoki has quit IRC | 00:05 | |
| *** ijw has joined #openstack-ironic | 00:08 | |
| *** amotoki has joined #openstack-ironic | 00:13 | |
| *** ijw has quit IRC | 00:13 | |
| *** [1]cdearborn has joined #openstack-ironic | 00:19 | |
| *** ijw has joined #openstack-ironic | 00:26 | |
| *** ijw has quit IRC | 00:27 | |
| *** ijw_ has joined #openstack-ironic | 00:28 | |
| *** ijw_ has quit IRC | 00:29 | |
| *** aNuposic has quit IRC | 00:37 | |
| *** [1]cdearborn has quit IRC | 00:39 | |
| *** mtanino has joined #openstack-ironic | 00:40 | |
| *** adu has quit IRC | 00:41 | |
| *** baoli has joined #openstack-ironic | 00:47 | |
| *** baoli_ has joined #openstack-ironic | 00:49 | |
| *** adu has joined #openstack-ironic | 00:49 | |
| *** ijw has joined #openstack-ironic | 00:50 | |
| *** hoangcx has joined #openstack-ironic | 00:50 | |
| *** thrash is now known as thrash|g0ne | 00:50 | |
| *** baoli has quit IRC | 00:52 | |
| *** ijw has quit IRC | 00:54 | |
| *** phuongnh has joined #openstack-ironic | 00:59 | |
| *** amotoki has quit IRC | 00:59 | |
| *** amotoki has joined #openstack-ironic | 01:00 | |
| *** purp has quit IRC | 01:00 | |
| *** awidders_ has quit IRC | 01:03 | |
| *** awiddersheim has joined #openstack-ironic | 01:03 | |
| *** amotoki has quit IRC | 01:04 | |
| *** amotoki has joined #openstack-ironic | 01:08 | |
| *** kwilke has joined #openstack-ironic | 01:09 | |
| *** baoli_ has quit IRC | 01:09 | |
| *** baoli has joined #openstack-ironic | 01:09 | |
| *** tuanluong has joined #openstack-ironic | 01:13 | |
| *** amotoki has quit IRC | 01:24 | |
| *** amotoki has joined #openstack-ironic | 01:24 | |
| *** amotoki has quit IRC | 01:29 | |
| *** kwilke has quit IRC | 01:29 | |
| *** adu has quit IRC | 01:33 | |
| *** adu has joined #openstack-ironic | 01:37 | |
| *** rama_y has quit IRC | 01:47 | |
| *** wajdi has joined #openstack-ironic | 01:48 | |
| *** amotoki has joined #openstack-ironic | 01:59 | |
| *** bdeschenes has quit IRC | 02:11 | |
| *** syed_ has quit IRC | 02:44 | |
| *** adu has quit IRC | 02:45 | |
| *** amotoki_ has joined #openstack-ironic | 02:55 | |
| *** amotoki has quit IRC | 02:58 | |
| *** ijw has joined #openstack-ironic | 03:10 | |
| *** links has joined #openstack-ironic | 03:10 | |
| *** ijw_ has joined #openstack-ironic | 03:10 | |
| *** ijw has quit IRC | 03:14 | |
| *** Romanenko_K has joined #openstack-ironic | 03:16 | |
| *** baoli has quit IRC | 03:20 | |
| *** bathris has joined #openstack-ironic | 03:22 | |
| *** bathri-sa has joined #openstack-ironic | 03:26 | |
| *** bathris has quit IRC | 03:27 | |
| *** hpe-hj has quit IRC | 03:38 | |
| gcb | rloo, just want you to check https://review.openstack.org/#/c/379415 again, I removed the hacking check :-) | 03:43 |
| patchbot | patch 379415 - ironic - Do not use mutable object as func default param | 03:43 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Do not use mutable object as func default param https://review.openstack.org/379415 | 03:43 |
| *** syed_ has joined #openstack-ironic | 03:44 | |
| openstackgerrit | Alexander Bashmakov proposed openstack/ironic: Rewrite create-node.sh in Python. https://review.openstack.org/346832 | 03:46 |
| *** rama_y has joined #openstack-ironic | 03:46 | |
| openstackgerrit | Alexander Bashmakov proposed openstack/ironic: Rewrite create-node.sh in Python. https://review.openstack.org/346832 | 03:47 |
| *** bathri-sa has quit IRC | 03:50 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/bifrost: Fix a typo in documentation https://review.openstack.org/381383 | 03:50 |
| *** vikrant has joined #openstack-ironic | 03:54 | |
| *** rama_y has quit IRC | 03:55 | |
| *** zackf has quit IRC | 03:56 | |
| *** yuanying has quit IRC | 04:01 | |
| *** yuanying_ has joined #openstack-ironic | 04:01 | |
| *** mtanino has quit IRC | 04:03 | |
| *** vishwanathj_ has quit IRC | 04:14 | |
| *** absubram has joined #openstack-ironic | 04:17 | |
| *** absubram_ has joined #openstack-ironic | 04:18 | |
| *** amotoki has joined #openstack-ironic | 04:18 | |
| *** amotoki_ has quit IRC | 04:21 | |
| *** absubram has quit IRC | 04:21 | |
| *** absubram_ is now known as absubram | 04:21 | |
| *** pmannidi has quit IRC | 04:39 | |
| *** Sukhdev has joined #openstack-ironic | 04:41 | |
| *** zackf has joined #openstack-ironic | 04:49 | |
| *** zackf has quit IRC | 04:54 | |
| *** pmannidi has joined #openstack-ironic | 04:56 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI https://review.openstack.org/348191 | 04:58 |
| *** Sukhdev has quit IRC | 05:35 | |
| *** ccamacho has quit IRC | 05:43 | |
| *** jaosorior has joined #openstack-ironic | 05:46 | |
| *** e0ne has joined #openstack-ironic | 05:52 | |
| openstackgerrit | Merged openstack/ironic: Use method delete_if_exists from oslo.utils https://review.openstack.org/379159 | 06:10 |
| *** mjura has joined #openstack-ironic | 06:12 | |
| *** mjura has quit IRC | 06:18 | |
| *** mjura has joined #openstack-ironic | 06:19 | |
| *** gcb has quit IRC | 06:30 | |
| *** gcb has joined #openstack-ironic | 06:40 | |
| *** e0ne has quit IRC | 06:43 | |
| *** amotoki has quit IRC | 06:43 | |
| *** rcernin has joined #openstack-ironic | 06:43 | |
| *** e0ne has joined #openstack-ironic | 06:44 | |
| *** ccamacho has joined #openstack-ironic | 06:47 | |
| *** amotoki has joined #openstack-ironic | 06:49 | |
| *** tesseract- has joined #openstack-ironic | 06:52 | |
| *** e0ne has quit IRC | 06:55 | |
| *** hoangcx has quit IRC | 07:00 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Retry to connect to RPC if failed https://review.openstack.org/376462 | 07:18 |
| *** amotoki_ has joined #openstack-ironic | 07:24 | |
| *** dtantsur|afk is now known as dtantsur | 07:26 | |
| dtantsur | Morning Ironic | 07:26 |
| *** amotoki has quit IRC | 07:27 | |
| gcb | morning dtantsur ! | 07:34 |
| dtantsur | o/ | 07:34 |
| gcb | dtantsur, what's your timezone? | 07:35 |
| *** vmud213 has joined #openstack-ironic | 07:35 | |
| gcb | I'm in UTC+8 | 07:35 |
| dtantsur | UTC+1 (+2 in summer) - central Europe | 07:36 |
| gcb | cool, we can work together in some time :-) | 07:38 |
| dtantsur | yeah :) | 07:44 |
| *** snecklifter has joined #openstack-ironic | 07:44 | |
| *** jpich has joined #openstack-ironic | 07:44 | |
| *** jtomasek|afk is now known as jtomasek | 07:51 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Remove unused methods in common/utils.py https://review.openstack.org/379364 | 07:53 |
| *** syed_ has quit IRC | 07:54 | |
| *** jistr has joined #openstack-ironic | 07:57 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:01 | |
| *** zackf has joined #openstack-ironic | 08:03 | |
| *** zackf has quit IRC | 08:08 | |
| *** ohamada has joined #openstack-ironic | 08:08 | |
| *** e0ne has joined #openstack-ironic | 08:10 | |
| *** stendulker has joined #openstack-ironic | 08:11 | |
| *** athomas has joined #openstack-ironic | 08:15 | |
| *** amotoki_ has quit IRC | 08:18 | |
| pas-ha | morning Ironic | 08:18 |
| *** milan has joined #openstack-ironic | 08:20 | |
| *** amotoki has joined #openstack-ironic | 08:21 | |
| milan | morning Ironic! :) | 08:23 |
| stendulker | morning milan pas-ha :) | 08:24 |
| milan | morning pas-ha stendulker! :) | 08:24 |
| pas-ha | morning milan stendulker | 08:25 |
| *** bks has joined #openstack-ironic | 08:26 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Move build_instance_info_for_deploy to deploy_utils https://review.openstack.org/377608 | 08:26 |
| dtantsur | morning pas-ha, milan; afternoon (?) stendulker | 08:27 |
| milan | morning dtantsur! :) | 08:27 |
| milan | spec review time? :P | 08:27 |
| dtantsur | well, today, but not right now, I guess | 08:27 |
| stendulker | morning dtantsur :) Yes, its afternoon here. | 08:28 |
| dtantsur | milan, in return, please review the remaining LLC patch from sambetts|afk (you've approved the second one in a chain) | 08:28 |
| milan | dtantsur, ack | 08:28 |
| milan | link handy? | 08:28 |
| dtantsur | milan, https://review.openstack.org/380394 | 08:30 |
| patchbot | patch 380394 - ironic-inspector - LLC Hook: Fix patching Ironic ports | 08:30 |
| milan | thanks! | 08:30 |
| milan | w/r the reviews I think we should decide about the raw data in the LLDP spec besides other stuff later | 08:31 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add entry_point for oslo policy scripts https://review.openstack.org/380962 | 08:32 |
| dtantsur | what do you mean by the raw data there? | 08:32 |
| milan | the data returned by ramdisk | 08:33 |
| milan | atm, the suggestion is the "processed" data to have one more level of nesting with keys: 'processed', 'unprocessed' | 08:34 |
| milan | I think it's not optimal | 08:34 |
| * milan considers -1 it | 08:34 | |
| vsaienk0 | good morning dtantsur, milan, sambetts, lucasagomes, pas-ha, vdrok, aarefiev and all Ironic'ers who are around! | 08:34 |
| milan | morning vsaienk0! :) | 08:35 |
| dtantsur | milan, I don't see how this is related to LLDP at all.... | 08:35 |
| milan | dtantsur, https://review.openstack.org/#/c/374381/11/specs/lldp-reporting.rst@160 | 08:36 |
| patchbot | patch 374381 - ironic-inspector-specs - Display LLDP information via new commands | 08:36 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Remove unneeded exception handling from conductor https://review.openstack.org/381530 | 08:36 |
| dtantsur | hmm, yeah, I see now. I'm -1 to changing inventory at all. | 08:37 |
| milan | right | 08:39 |
| gcb | dtantsur, please help review https://review.openstack.org/379581 and https://review.openstack.org/#/c/379081/ | 08:41 |
| patchbot | patch 379581 - ironic - Enable DeprecationWarning in test environments | 08:41 |
| patchbot | patch 379081 - ironic - Trivial: Fix some typos in comments and docstring | 08:41 |
| gcb | two simple patches | 08:41 |
| dtantsur | I'll get to them eventually, but sorry I don't quite understand why I should give them priority.. | 08:42 |
| *** wajdi_ has joined #openstack-ironic | 08:43 | |
| gcb | dtantsur, ironic review depends priority, right ? As oslo , at the beginning of the new release , we would like to merge simple patches. and review with priority at the end of release. | 08:45 |
| openstackgerrit | Markos Chandras proposed openstack/bifrost: Serialize localhost actions during final host deployment https://review.openstack.org/381541 | 08:45 |
| gcb | It seems a little difference between oslo and ironic . I just started with ironic, can contribute simple patch now, will deep dive later :-) | 08:46 |
| *** wajdi has quit IRC | 08:47 | |
| *** ijw_ has quit IRC | 08:49 | |
| *** wajdi_ has quit IRC | 08:50 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: [install-guide] describe pxe.ipxe_swift_tempurl https://review.openstack.org/379358 | 08:50 |
| dtantsur | gcb, sure, I don't say I won't review them, just wonder if they're really worth out-of-order processing implied by pinging a core reviewer personally :) | 08:51 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Clean exceptions handling in conductor manager https://review.openstack.org/381554 | 08:52 |
| gcb | dtantsur, will follow ironic way, thanks :-) | 08:52 |
| dtantsur | :) | 08:52 |
| *** dtantsur is now known as dtantsur|bbl | 08:52 | |
| *** sturivnyi_ has joined #openstack-ironic | 08:58 | |
| *** electrofelix has joined #openstack-ironic | 09:00 | |
| vdrok | morning ironic, dtantsur|bbl gcb pas-ha milan stendulker and vsaienk0 | 09:03 |
| sturivnyi_ | dtantsur|bbl, devananda, sambetts|afk Hello Would be grateful if you could review patches https://review.openstack.org/#/c/301687/ https://review.openstack.org/#/c/293634/ | 09:03 |
| patchbot | patch 301687 - python-ironicclient - Add basic tests for OSC plugin baremetal node comm... | 09:03 |
| patchbot | patch 293634 - python-ironicclient - Tests for testing chassis-create command | 09:03 |
| *** derekh has joined #openstack-ironic | 09:04 | |
| *** strigazi_AFK is now known as strigazi | 09:04 | |
| gcb | morning vdrok :） | 09:11 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add warning message for baremetal node set/unset commands https://review.openstack.org/347856 | 09:12 |
| tuanluong | morning ironic | 09:14 |
| *** sambetts|afk is now known as sambetts | 09:21 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: [tinyipa] allow skipping python compileall https://review.openstack.org/381260 | 09:24 |
| *** ijw has joined #openstack-ironic | 09:35 | |
| milan | morning vdrok! :) | 09:40 |
| milan | morning tuanluong! :) | 09:40 |
| *** ijw has quit IRC | 09:40 | |
| *** dtantsur|bbl is now known as dtantsur | 09:50 | |
| *** amotoki has quit IRC | 09:51 | |
| *** wajdi has joined #openstack-ironic | 09:53 | |
| vsaienk0 | dtantsur: please review https://review.openstack.org/#/c/381570/ one line patch that disables ramdisk building for ironic-multitest experimental job | 09:53 |
| patchbot | patch 381570 - openstack-infra/project-config - Disable building ramdisk for ironic-multitest job. | 09:53 |
| dtantsur | vsaienk0, is it for mitaka? I thought in newton we disabled it by default.. | 09:54 |
| vsaienk0 | dtantsur: it is an experimental job for Ocata | 09:54 |
| dtantsur | vsaienk0, I suspect we should fix https://github.com/openstack-infra/devstack-gate/blob/140642ec6055818b1b00341de7dfa67def26b223/devstack-vm-gate-wrap.sh#L204 instead, but the fix is fine for now | 09:55 |
| dtantsur | jroll, ^^^ | 09:55 |
| vsaienk0 | dtantsur: not really devstack-gate enables it by default if it is not explicitly disabled: https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh | 09:55 |
| vsaienk0 | and also it is enabled by default https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L137 | 09:56 |
| dtantsur | vsaienk0, this is something to fix IMO. the majority of our jobs do not build the ramdisk nowadays. | 09:56 |
| *** bdeschenes has joined #openstack-ironic | 09:56 | |
| dtantsur | wut? I was pretty sure we disabled it.... ok then | 09:56 |
| *** jistr is now known as jistr|mtg | 09:59 | |
| *** zigo has quit IRC | 10:01 | |
| *** zigo has joined #openstack-ironic | 10:04 | |
| *** zigo is now known as Guest84780 | 10:05 | |
| *** amotoki has joined #openstack-ironic | 10:07 | |
| *** Guest84780 has quit IRC | 10:14 | |
| *** zigo_ has joined #openstack-ironic | 10:16 | |
| openstackgerrit | Merged openstack/ironic-inspector: LLC Hook: Fix patching Ironic ports https://review.openstack.org/380394 | 10:19 |
| stendulker | morning vdrok :) | 10:20 |
| jroll | morning y'all | 10:23 |
| dtantsur | morning jroll | 10:25 |
| jroll | dtantsur: did you need something up there or just awareness? | 10:25 |
| *** mgould|afk is now known as mgould | 10:25 | |
| mgould | morning Ironic | 10:26 |
| vdrok | morning jroll and mgould | 10:26 |
| * mgould ponders one of those silicone wristbands for Ramdisk Awareness | 10:26 | |
| dtantsur | jroll, looking if you have any objections | 10:26 |
| jroll | dtantsur: I'd rather just kill that variable, have the default in our devstack plugin, and override in project-config as needed | 10:28 |
| openstackgerrit | Merged openstack/ironic: Fix typos in docstring/comments https://review.openstack.org/377261 | 10:29 |
| dtantsur | ++ | 10:29 |
| jroll | I don't find d-s-g variables very useful | 10:29 |
| jroll | and when project-config can be in our tree d-s-g will slow us down | 10:29 |
| dtantsur | jroll, FYI I'm working on a new set of ironic-lib jobs covering the same cases as ironic's jobs | 10:29 |
| jroll | dtantsur: cool! | 10:29 |
| openstackgerrit | Merged openstack/ironic-inspector: LLC Hook: Do not assume interfaces are added to Ironic https://review.openstack.org/379614 | 10:33 |
| *** phuongnh has quit IRC | 10:35 | |
| *** ijw has joined #openstack-ironic | 10:36 | |
| *** thrash|g0ne is now known as thrash | 10:37 | |
| *** amotoki has quit IRC | 10:39 | |
| *** amotoki has joined #openstack-ironic | 10:40 | |
| *** ijw has quit IRC | 10:42 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: Fix display of chassis UUID field if empty https://review.openstack.org/370764 | 10:47 |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: Fix display of chassis UUID field if empty https://review.openstack.org/370764 | 10:51 |
| *** ralonsoh has joined #openstack-ironic | 10:53 | |
| *** rcernin has quit IRC | 10:57 | |
| *** rcernin has joined #openstack-ironic | 10:58 | |
| *** jistr|mtg is now known as jistr | 11:00 | |
| *** dprince has joined #openstack-ironic | 11:01 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for 'baremetal port create' command https://review.openstack.org/332880 | 11:09 |
| *** baoli has joined #openstack-ironic | 11:14 | |
| *** ccamacho is now known as ccamacho|lunch | 11:14 | |
| *** baoli has quit IRC | 11:15 | |
| jroll | dtantsur: your ironic-lib project-config job lgtm | 11:20 |
| dtantsur | cool | 11:20 |
| jroll | I wonder if we want a job on ironic from master ironic-lib | 11:21 |
| jroll | probably too complex, idk | 11:21 |
| dtantsur | jroll, I don't think it's complex, but we probably should shrink our jobs count first.. | 11:21 |
| jroll | yeah :) | 11:22 |
| *** bks has quit IRC | 11:22 | |
| *** zigo_ is now known as zigo | 11:23 | |
| dtantsur | jroll, do you know the reason of https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L572 ? is it only because of coreos? | 11:32 |
| *** karthiks has quit IRC | 11:33 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Add CRUD and node states notifications to ironic https://review.openstack.org/347242 | 11:34 |
| *** hshiina has joined #openstack-ironic | 11:34 | |
| *** ijw has joined #openstack-ironic | 11:38 | |
| TheJulia | Good morning | 11:39 |
| mat128 | good morning TheJulia | 11:40 |
| sambetts | dtantsur: thats the number of VMs to create during the devstack run, we only need 1 for our basic ops tempest run, and we configure multiple (7) VMs for our tempest smoke run # | 11:40 |
| *** stendulker has quit IRC | 11:40 | |
| dtantsur | sambetts, yeah, I know, I wonder if there are reasons not to bump it for basic ops (I want to merge several jobs to one) | 11:40 |
| sambetts | dtantsur: only that it'll never use them | 11:40 |
| dtantsur | ok, that's what I wanted to know :) | 11:41 |
| dtantsur | morning mat128, TheJulia | 11:41 |
| milan | morning jroll mat128 sambetts TheJulia and every one else! :) | 11:42 |
| *** Goneri has quit IRC | 11:43 | |
| *** ijw has quit IRC | 11:44 | |
| *** bdeschenes has quit IRC | 11:45 | |
| *** bdeschenes has joined #openstack-ironic | 11:46 | |
| *** bdeschenes has quit IRC | 11:46 | |
| openstackgerrit | Merged openstack/bifrost: Update flake8 ignore list https://review.openstack.org/378257 | 11:49 |
| *** karthiks has joined #openstack-ironic | 11:49 | |
| *** baoli has joined #openstack-ironic | 11:55 | |
| milan | sambetts, please, check when you've got the time: https://review.openstack.org/#/c/375045/5/specs/list-introspection-statuses.rst | 11:57 |
| patchbot | patch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses | 11:57 |
| *** amotoki has quit IRC | 11:57 | |
| sambetts | milan: will do :) | 11:58 |
| milan | thanks! :) | 11:58 |
| *** lucas-afk is now known as lucasagomes | 11:59 | |
| mgould | morning mat128 TheJulia lucasagomes | 12:00 |
| *** krtaylor has quit IRC | 12:01 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: [POC] db: use eventlet's thread pooling https://review.openstack.org/381718 | 12:03 |
| *** tuanluong has quit IRC | 12:05 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Introduce default_boot_option configuration option https://review.openstack.org/375458 | 12:05 |
| dtantsur | vdrok, should be fine now ^^^ | 12:05 |
| vdrok | morning mat128 and TheJulia | 12:05 |
| vdrok | dtantsur: thanks, will take a look | 12:05 |
| dtantsur | essentially, I switched to warning on start up, and dropped the global variable | 12:05 |
| *** karthiks has quit IRC | 12:06 | |
| dtantsur | milan, do you think you could upgrade your vote on https://review.openstack.org/#/c/371470/ ? | 12:07 |
| patchbot | patch 371470 - python-ironic-inspector-client - Move documentation from README to Sphinx | 12:07 |
| dtantsur | I can fix the nits in a follow up | 12:07 |
| milan | dtantsur, sure! | 12:08 |
| * milan checks | 12:08 | |
| milan | done | 12:09 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for maintenance mode commands https://review.openstack.org/337259 | 12:11 |
| *** amotoki has joined #openstack-ironic | 12:12 | |
| jroll | dtantsur: what sambetts said :D | 12:14 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Rely on portgroup standalone_ports_supported https://review.openstack.org/360647 | 12:17 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add PortGroups API https://review.openstack.org/347549 | 12:17 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Integrate portgroups with ports to support LAG https://review.openstack.org/332177 | 12:17 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: devstack portgroup support https://review.openstack.org/381743 | 12:17 |
| lucasagomes | mgould, morning | 12:19 |
| *** karthiks has joined #openstack-ironic | 12:19 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Skip tempest tests if ironic isn't available https://review.openstack.org/379256 | 12:22 |
| *** trown|outtypewww is now known as trown | 12:22 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add user and project domains to ironic context https://review.openstack.org/295371 | 12:25 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add user and project domains to ironic context https://review.openstack.org/295371 | 12:28 |
| *** bdeschenes has joined #openstack-ironic | 12:28 | |
| *** ralonsoh_ has joined #openstack-ironic | 12:30 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Move documentation from README to Sphinx https://review.openstack.org/371470 | 12:33 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: Fix docstrings in tempest plugin baremetal json client https://review.openstack.org/377774 | 12:33 |
| *** ralonsoh has quit IRC | 12:34 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add support to update keystone for bifrost https://review.openstack.org/380226 | 12:35 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml https://review.openstack.org/380227 | 12:36 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml https://review.openstack.org/380227 | 12:36 |
| *** bdeschenes has quit IRC | 12:36 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone install and clouds.yaml to testing https://review.openstack.org/380228 | 12:37 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Convert ironic.conf to template generated https://review.openstack.org/305634 | 12:37 |
| *** hoangcx has joined #openstack-ironic | 12:37 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Convert ironic.conf to template generated https://review.openstack.org/305634 | 12:39 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Rename classes in osc plugin unit tests https://review.openstack.org/342913 | 12:40 |
| *** ijw has joined #openstack-ironic | 12:40 | |
| *** ccamacho|lunch is now known as ccamacho | 12:42 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: db: use eventlet's thread pooling https://review.openstack.org/381718 | 12:42 |
| *** baoli_ has joined #openstack-ironic | 12:44 | |
| *** ijw has quit IRC | 12:45 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Retry to connect to RPC if failed https://review.openstack.org/376462 | 12:46 |
| *** baoli has quit IRC | 12:47 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add support to update keystone for bifrost https://review.openstack.org/380226 | 12:48 |
| *** amotoki has quit IRC | 12:49 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml https://review.openstack.org/380227 | 12:49 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone install and clouds.yaml to testing https://review.openstack.org/380228 | 12:49 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Convert ironic.conf to template generated https://review.openstack.org/305634 | 12:49 |
| *** jcoufal has joined #openstack-ironic | 12:50 | |
| *** amotoki has joined #openstack-ironic | 12:54 | |
| *** ralonsoh_ is now known as ralonsoh | 12:54 | |
| *** vmud213 has quit IRC | 13:00 | |
| milan | guys, unittest.TestCase.assertRaises (as a ctx manager) isn't working anymore? (in inspector at least) | 13:00 |
| mgould | milan: IIRC we use some weird subclass/reimplementation of unittest instead? | 13:01 |
| * milan doesn't like regexp when there's an exact match | 13:02 | |
| *** Goneri has joined #openstack-ironic | 13:02 | |
| *** rloo has joined #openstack-ironic | 13:02 | |
| mgould | milan: pastebin? | 13:02 |
| milan | mgould, http://paste.openstack.org/show/584195/ | 13:04 |
| mgould | milan: how odd | 13:07 |
| *** [1]cdearborn has joined #openstack-ironic | 13:07 | |
| milan | mgould, http://paste.openstack.org/show/584198/ yeah, the unittest.TestCase.assertRaises is overridden | 13:07 |
| milan | :-/ | 13:07 |
| milan | not sure that's a good idea | 13:07 |
| * milan would like to have the ctx manager back, what to do? | 13:08 | |
| mgould | dunno | 13:09 |
| mgould | path of least resistance is "wrap your block in a lambda, and be sad about it" | 13:09 |
| * milan sad lol | 13:10 | |
| mgould | path of greatest resistance is "refactor all ironic-inspector tests to remove all reliance on the weird test library, while cackling evilly" | 13:10 |
| milan | dtantsur, ^ :D | 13:10 |
| mgould | release of flying monkeys is optional, but encouraged | 13:11 |
| dtantsur | some people want weird libraries, some don't ¯\_(ツ)_/¯ | 13:11 |
| *** vikrant has quit IRC | 13:11 | |
| dtantsur | I'm a bit afraid of it moving back and forth.. can we just do whatever ironic is doing? | 13:11 |
| * milan checks | 13:12 | |
| milan | dtantsur, ironic uses https://github.com/openstack/ironic/blob/master/ironic/tests/base.py#L71 which is the same as oslotest.base.BaseTestCase | 13:21 |
| dtantsur | ack, let's use it too (I suspect we do already) | 13:22 |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands https://review.openstack.org/374381 | 13:23 |
| milan | dtantsur, oslotest just inherits all from it | 13:24 |
| bfournie | morning ironic | 13:24 |
| milan | so it's basically the same modulo couple of setup fixtures | 13:24 |
| milan | morning bfournie! :) | 13:24 |
| bfournie | dtantsur, milan: moved processed lldp struct out of inventory ^^ | 13:25 |
| milan | bfournie, thx! will look | 13:25 |
| dtantsur | hey bfournie, cool | 13:25 |
| bfournie | thx | 13:25 |
| *** ralonsoh has quit IRC | 13:30 | |
| *** ralonsoh_ has joined #openstack-ironic | 13:30 | |
| milan | dtantsur, just a quick question: would you -2 right away a patch that did this: http://paste.openstack.org/show/584204/ ;) | 13:31 |
| *** daemontool has joined #openstack-ironic | 13:33 | |
| *** jaosorior has quit IRC | 13:34 | |
| *** jaosorior has joined #openstack-ironic | 13:34 | |
| mgould | milan: heh | 13:35 |
| jroll | why not just self.assertRaises(FooExc, func_call, args) ? | 13:35 |
| dtantsur | milan, likely yes, to be honest... | 13:35 |
| milan | jroll, honestly I'm lazy rewriting a patch as the behaviour changed a while ago | 13:36 |
| milan | jroll, I used ctx manager version of that call | 13:36 |
| milan | that got overridden by testutils | 13:36 |
| jroll | heh | 13:36 |
| milan | I guess I didn't pay attention to some deprecation log ;) | 13:37 |
| *** amotoki has quit IRC | 13:38 | |
| dtantsur | I guess it's a problem in testtools.. I also used it as a context manager a couple of times | 13:40 |
| *** ijw has joined #openstack-ironic | 13:41 | |
| *** ijw has quit IRC | 13:46 | |
| *** joprovost has joined #openstack-ironic | 13:48 | |
| *** links has quit IRC | 13:49 | |
| milan | jroll, dtantsur, mgould http://paste.openstack.org/show/584210/ <- this sort of thing is now going to require a wrapper function every time now | 13:50 |
| dtantsur | milan, this is a case for assertRaisesRegex | 13:51 |
| jroll | milan: seems like another good reason to raise our own exceptions, no? | 13:51 |
| mgould | jroll: how would that help? | 13:51 |
| milan | dtantsur, it's rather about changing the state | 13:51 |
| milan | jroll, not really, it's from the sqlite, you'd have to monkey patch | 13:52 |
| jroll | mgould: if the method checked the exception and raised some CheckConstraintFailed exception instead, the tests don't need to | 13:52 |
| milan | s/sqlite/sqlalchemy | 13:52 |
| mgould | oh, right, scoping variables | 13:52 |
| jroll | milan: _set_state() would catch DBError and raise something else | 13:52 |
| dtantsur | self.assertRaisesRegex(oslo_db.exception.DBError, "CHECK constraint failed", self.node_info._set_state, 'foo') | 13:52 |
| dtantsur | or what am I missing here? | 13:52 |
| jroll | dtantsur: ++ that also works | 13:53 |
| milan | ah, sry | 13:53 |
| * milan missed it's a call | 13:53 | |
| milan | imagine it was an assignment that changed the state | 13:54 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/378819 | 13:54 |
| *** hamzy has quit IRC | 13:54 | |
| milan | jroll, OK, that way would work, custom exception indeed | 13:54 |
| jroll | I am surprised that context manager was removed though :/ | 13:54 |
| milan | I'd like to say silently | 13:55 |
| milan | removed | 13:55 |
| milan | but I'm not sure, maybe I ignored a deprecation log | 13:55 |
| * mgould remembers running into this problem a while ago | 13:56 | |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being restarted to address performance degradation and should return momentarily | 14:00 | |
| *** daemontool_ has joined #openstack-ironic | 14:06 | |
| *** wajdi has quit IRC | 14:07 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add negative tests for baremetal node commands https://review.openstack.org/337719 | 14:08 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone installation playbook https://review.openstack.org/380225 | 14:10 |
| *** daemontool has quit IRC | 14:10 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add support to update keystone for bifrost https://review.openstack.org/380226 | 14:10 |
| *** jheroux has joined #openstack-ironic | 14:11 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add support to update keystone for bifrost https://review.openstack.org/380226 | 14:11 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml https://review.openstack.org/380227 | 14:11 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone install and clouds.yaml to testing https://review.openstack.org/380228 | 14:11 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Convert ironic.conf to template generated https://review.openstack.org/305634 | 14:12 |
| xavierr | good morning Ironic | 14:14 |
| mgould | morning xavierr | 14:15 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add user and project domains to ironic context https://review.openstack.org/295371 | 14:16 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for 'baremetal port create' command https://review.openstack.org/332880 | 14:16 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for 'baremetal port create' command https://review.openstack.org/332880 | 14:17 |
| xavierr | dtantsur: I saw you asked about our CI in the inspection interface, we are rebuilding our CI. the vm's that runs our CI and the logs server are broken :( | 14:17 |
| *** amotoki has joined #openstack-ironic | 14:17 | |
| xavierr | dtantsur: sorry about that | 14:18 |
| dtantsur | I see | 14:19 |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Extend FPC status https://review.openstack.org/381878 | 14:19 |
| dtantsur | also morning xavierr | 14:19 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: [install-guide] describe pxe.ipxe_swift_tempurl https://review.openstack.org/379358 | 14:20 |
| *** zackf has joined #openstack-ironic | 14:24 | |
| *** rpioso has joined #openstack-ironic | 14:26 | |
| *** hamzy has joined #openstack-ironic | 14:27 | |
| *** wajdi has joined #openstack-ironic | 14:28 | |
| *** mkam has joined #openstack-ironic | 14:29 | |
| *** mkam has quit IRC | 14:30 | |
| *** mjura has quit IRC | 14:31 | |
| *** mkam has joined #openstack-ironic | 14:32 | |
| milan | morning xavierr! :) | 14:33 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Correctly count the number of primary partitions https://review.openstack.org/381893 | 14:34 |
| *** jjohnson2 has joined #openstack-ironic | 14:37 | |
| *** rook_ is now known as rook | 14:39 | |
| *** mkam has quit IRC | 14:41 | |
| *** mkam has joined #openstack-ironic | 14:41 | |
| *** gcb has quit IRC | 14:42 | |
| *** ijw has joined #openstack-ironic | 14:42 | |
| *** syed_ has joined #openstack-ironic | 14:44 | |
| *** ijw has quit IRC | 14:47 | |
| *** mtanino has joined #openstack-ironic | 14:48 | |
| *** hj-hpe has joined #openstack-ironic | 14:51 | |
| *** vishwanathj has joined #openstack-ironic | 14:53 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: devstack portgroup support https://review.openstack.org/381743 | 14:53 |
| *** gcb has joined #openstack-ironic | 14:54 | |
| lucasagomes | is someone working on add more tests to the ironic-lib gate ? It's a bit too "light" at the moment | 14:56 |
| lucasagomes | e.g https://review.openstack.org/381893 wasn't actually tested (only unittested) with the current tests | 14:57 |
| patchbot | patch 381893 - ironic-lib - Correctly count the number of primary partitions | 14:57 |
| xavierr | morning milan :) | 15:00 |
| xavierr | morning mgould :) | 15:01 |
| *** rajinir has joined #openstack-ironic | 15:01 | |
| dtantsur | lucasagomes, I am | 15:03 |
| dtantsur | lucasagomes, https://review.openstack.org/381652 | 15:03 |
| patchbot | patch 381652 - openstack-infra/project-config - Rework ironic-lib tempest jobs | 15:03 |
| lucasagomes | dtantsur, cool. Yeah that's important | 15:03 |
| jlvillal | Good morning Ironic :) | 15:03 |
| milan | morning jlvillal! :) | 15:04 |
| jlvillal | Hi milan :) | 15:04 |
| dtantsur | morning jlvillal | 15:04 |
| jlvillal | dtantsur: :) | 15:05 |
| lucasagomes | jlvillal, morning | 15:06 |
| *** electrofelix has quit IRC | 15:07 | |
| *** electrofelix has joined #openstack-ironic | 15:07 | |
| jlvillal | lucasagomes: :) | 15:07 |
| *** rcernin has quit IRC | 15:10 | |
| *** rbrud has joined #openstack-ironic | 15:12 | |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Add portgroup support to osc plugin https://review.openstack.org/362130 | 15:13 |
| *** lucasagomes is now known as lucas-hungry | 15:16 | |
| JayF | just marked https://bugs.launchpad.net/ironic/+bug/1629304 invalid as it was filed against unmerged patchsets | 15:17 |
| openstack | Launchpad bug 1629304 in Ironic "Portgroup with empty node_uuid using ironic api version 1.23" [Undecided,Invalid] | 15:18 |
| JayF | found by aslezil :) so thanks for that annie | 15:18 |
| openstackgerrit | Merged openstack/ironic: Remove unused methods in common/utils.py https://review.openstack.org/379364 | 15:19 |
| *** xavierr is now known as xavierr|afk | 15:19 | |
| openstackgerrit | Merged openstack/ironic: Trivial: Fix some typos in comments and docstring https://review.openstack.org/379081 | 15:19 |
| *** daemontool_ has quit IRC | 15:19 | |
| openstackgerrit | Merged openstack/ironic-lib: Changed the home-page of ironic-lib https://review.openstack.org/381085 | 15:20 |
| openstackgerrit | Merged openstack/ironic: Do not use mutable object as func default param https://review.openstack.org/379415 | 15:20 |
| *** gcb has quit IRC | 15:21 | |
| *** daemontool has joined #openstack-ironic | 15:27 | |
| *** rama_y has joined #openstack-ironic | 15:28 | |
| *** aNuposic has joined #openstack-ironic | 15:28 | |
| jlvillal | vdrok: What is the difference between "dynamic portgroups" and "portgroups"? | 15:31 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 15:31 |
| jlvillal | Or if anyone else knows :) | 15:32 |
| vdrok | morning jlvillal | 15:32 |
| jlvillal | vdrok: hi :) | 15:32 |
| jlvillal | I'm assuming "portgroups" is what we have patches up for right now. And close to being done. | 15:32 |
| vdrok | jlvillal: dynamic portgroups are created automatically, based on the requested parameters provided to nova boot | 15:32 |
| vdrok | just porgroups are created manually by an operator | 15:33 |
| jlvillal | vdrok: Okay. I think I get it. Thanks. | 15:33 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: DNM: testing FIXED_RANGE outside of 10/8 https://review.openstack.org/381934 | 15:40 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: DNM: testing 380023 https://review.openstack.org/380351 | 15:40 |
| mgould | morning jlvillal | 15:40 |
| jlvillal | mgould: Hi | 15:40 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: DNM: testing SUBNETPOOL_PREFIX_V4 outside of 10/8 https://review.openstack.org/381934 | 15:42 |
| *** ijw has joined #openstack-ironic | 15:43 | |
| *** ijw has quit IRC | 15:48 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for 'baremetal port create' command https://review.openstack.org/332880 | 15:58 |
| *** karthiks has quit IRC | 15:59 | |
| *** rama_y has quit IRC | 16:00 | |
| *** jaosorior has quit IRC | 16:02 | |
| *** gcb has joined #openstack-ironic | 16:02 | |
| *** amotoki has quit IRC | 16:03 | |
| JayF | has anyone seen a failure like this? http://logs.openstack.org/57/377957/2/check/gate-grenade-dsvm-ironic/b595944/logs/grenade.sh.txt.gz#_2016-10-03_20_27_13_542 | 16:04 |
| JayF | looks like SSH into the instance is timing out in the "old" side of grenade | 16:05 |
| *** Romanenko_K has quit IRC | 16:05 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui: Fixed processing of node properties with null value https://review.openstack.org/381958 | 16:10 |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: If no resource, don't call Resource.to_dict() https://review.openstack.org/379567 | 16:10 |
| *** jcoufal_ has joined #openstack-ironic | 16:11 | |
| rloo | jroll, devananda: i've lost track of whether we're having an API meeting today or not. Are there any specs we should be reviewing? | 16:12 |
| *** jcoufal has quit IRC | 16:12 | |
| *** karthiks has joined #openstack-ironic | 16:15 | |
| *** gcb has quit IRC | 16:16 | |
| *** links has joined #openstack-ironic | 16:17 | |
| *** links has quit IRC | 16:18 | |
| *** lucas-hungry is now known as lucasagomes | 16:21 | |
| *** openstackgerrit has quit IRC | 16:26 | |
| *** openstackgerrit has joined #openstack-ironic | 16:27 | |
| *** rpioso has quit IRC | 16:27 | |
| *** baoli_ has quit IRC | 16:27 | |
| *** openstackgerrit has quit IRC | 16:28 | |
| *** tesseract- has quit IRC | 16:29 | |
| NobodyCam | Good Morning Ironic'ers :) | 16:29 |
| *** openstackgerrit has joined #openstack-ironic | 16:29 | |
| dtantsur | morning NobodyCam | 16:29 |
| NobodyCam | :) | 16:29 |
| jroll | rloo: we didn't meet last week so I believe we are meeting this week | 16:29 |
| *** openstackgerrit has quit IRC | 16:30 | |
| *** openstackgerrit has joined #openstack-ironic | 16:30 | |
| *** e0ne has quit IRC | 16:32 | |
| lucasagomes | NobodyCam, rloo morning | 16:34 |
| NobodyCam | morning lucasagomes, rloo, jroll JayF :) | 16:34 |
| JayF | o/ | 16:35 |
| jroll | \o NobodyCam | 16:35 |
| NobodyCam | :) o/ | 16:35 |
| *** daemontool has quit IRC | 16:35 | |
| *** ohamada has quit IRC | 16:35 | |
| *** karthiks has quit IRC | 16:35 | |
| milan | morning JayF NobodyCam ! :) | 16:35 |
| rloo | jroll: ok, i'll be there! | 16:35 |
| rloo | morning NobodyCam, lucasagomes | 16:35 |
| milan | and morning rloo! :) | 16:35 |
| NobodyCam | :) | 16:35 |
| rloo | oh, morning everyone! :) | 16:36 |
| milan | ...and good night! /me gone :D | 16:36 |
| jroll | heh, good morning :) | 16:36 |
| NobodyCam | might milan | 16:36 |
| milan | o/ | 16:37 |
| *** milan has quit IRC | 16:37 | |
| jroll | rloo: I've been feeling a bit off today, I'll let you know after lunch if I won't be there | 16:38 |
| *** hshiina has quit IRC | 16:38 | |
| rloo | jroll: ok, take it easy. | 16:39 |
| *** trown is now known as trown|lunch | 16:40 | |
| mgould | morning NobodyCam | 16:43 |
| NobodyCam | morning mgould :) | 16:43 |
| *** nicodemos has quit IRC | 16:43 | |
| *** ijw has joined #openstack-ironic | 16:44 | |
| *** mgould is now known as mgould|afk | 16:49 | |
| *** ijw has quit IRC | 16:50 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Update playbooks to support os_client_config https://review.openstack.org/380224 | 16:50 |
| devananda | rloo, jroll: I have a new spec to add to the discussion today. | 16:52 |
| devananda | but that's about it | 16:52 |
| *** karthiks has joined #openstack-ironic | 16:52 | |
| jroll | rloo: devananda: lunch did not help, I think I'm out for the day :/ | 16:52 |
| rloo | devananda: can we take the discussion to the spec itself? | 16:52 |
| rloo | jroll: hope you feel better soon! | 16:53 |
| jroll | rloo: thanks :) | 16:53 |
| * rloo catches up with ML, maybe should have run for TC. NOT! :D | 16:54 | |
| devananda | rloo: thatnks fine with me | 16:54 |
| devananda | jroll: ugh. no worries - hope you feel better! | 16:54 |
| NobodyCam | morning devananda : | 16:54 |
| devananda | NobodyCam: g'morning! | 16:54 |
| NobodyCam | :) | 16:54 |
| NobodyCam | feel better jroll :) | 16:54 |
| openstackgerrit | Michael Turek proposed openstack/ironic: Enable PXE for systems using petitboot https://review.openstack.org/185987 | 16:58 |
| rloo | devananda: is it this WIP spec: https://review.openstack.org/#/c/367583/? | 16:58 |
| patchbot | patch 367583 - ironic-specs - [WIP] API Evolution | 16:58 |
| *** vsaienko has joined #openstack-ironic | 16:59 | |
| *** ccamacho has quit IRC | 17:00 | |
| *** strigazi is now known as strigazi_AFK | 17:00 | |
| *** ralonsoh_ has quit IRC | 17:00 | |
| *** derekh has quit IRC | 17:00 | |
| *** rama_y has joined #openstack-ironic | 17:04 | |
| *** rbrud has quit IRC | 17:04 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: API Evolution - Error Responses https://review.openstack.org/381989 | 17:05 |
| *** penick has joined #openstack-ironic | 17:05 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 17:07 |
| *** krtaylor has joined #openstack-ironic | 17:08 | |
| *** vsaienko has quit IRC | 17:08 | |
| *** jpich has quit IRC | 17:09 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands https://review.openstack.org/374381 | 17:10 |
| *** strigazi_AFK is now known as strigazi | 17:11 | |
| *** vsaienko has joined #openstack-ironic | 17:11 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic: Adding Timing metrics for DRAC drivers. https://review.openstack.org/377957 | 17:12 |
| *** dtantsur is now known as dtantsur|afk | 17:18 | |
| *** strigazi is now known as strigazi_AFK | 17:22 | |
| *** MattMan has quit IRC | 17:22 | |
| *** MattMan has joined #openstack-ironic | 17:23 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic: Adding Timing metrics for DRAC drivers. https://review.openstack.org/377957 | 17:26 |
| *** vsaienko has quit IRC | 17:26 | |
| *** vsaienko has joined #openstack-ironic | 17:27 | |
| *** baoli has joined #openstack-ironic | 17:29 | |
| *** vsaienko has quit IRC | 17:30 | |
| * sambetts -> home | 17:30 | |
| sambetts | night all | 17:30 |
| *** sambetts is now known as sambetts|afk | 17:30 | |
| lucasagomes | sambetts|afk, see ya | 17:32 |
| *** vsaienko has joined #openstack-ironic | 17:32 | |
| *** rbrud has joined #openstack-ironic | 17:33 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Add support to update keystone for bifrost https://review.openstack.org/380226 | 17:35 |
| *** vsaienko has quit IRC | 17:36 | |
| *** baoli has quit IRC | 17:37 | |
| *** baoli has joined #openstack-ironic | 17:38 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: API Evolution - Discoverable Resource Actions https://review.openstack.org/382012 | 17:39 |
| *** hamzy has quit IRC | 17:41 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: API Evolution - Driver Info https://review.openstack.org/382015 | 17:42 |
| *** praneshp has joined #openstack-ironic | 17:43 | |
| *** praneshp has quit IRC | 17:44 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: API Evolution - Implement Tasks Resource https://review.openstack.org/382017 | 17:45 |
| devananda | jroll, rloo: as requested, a spec for adding tasks ^ | 17:46 |
| *** Goneri has quit IRC | 17:46 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Add ability to write clouds.yaml https://review.openstack.org/380227 | 17:46 |
| rloo | thx devananda! | 17:48 |
| openstackgerrit | Annie Lezil proposed openstack/ironic: Adding Timing metrics for DRAC drivers. https://review.openstack.org/377957 | 17:49 |
| openstackgerrit | Annie Lezil proposed openstack/ironic: Add description for vendor passthru methods https://review.openstack.org/370469 | 17:49 |
| *** bjolo_ has joined #openstack-ironic | 17:53 | |
| *** hamzy has joined #openstack-ironic | 17:55 | |
| *** Sukhdev has joined #openstack-ironic | 17:58 | |
| *** sdake has quit IRC | 17:58 | |
| *** rcernin has joined #openstack-ironic | 17:58 | |
| *** ccamacho has joined #openstack-ironic | 18:00 | |
| *** rpioso has joined #openstack-ironic | 18:02 | |
| *** harlowja has quit IRC | 18:03 | |
| *** electrofelix has quit IRC | 18:06 | |
| *** jjohnson2 has quit IRC | 18:09 | |
| *** Goneri has joined #openstack-ironic | 18:13 | |
| lucasagomes | this is quite sad: https://twitter.com/hintjens/status/783254242052206592 (he was the founder of te zeromq project) | 18:19 |
| *** e0ne has joined #openstack-ironic | 18:20 | |
| openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 18:20 |
| Marx314 | bump ^^ https://review.openstack.org/#/c/378050/ | 18:21 |
| patchbot | patch 378050 - python-ironicclient - Fix python3 compatibility when HTTP Error are retu... | 18:21 |
| *** trown|lunch is now known as trown | 18:21 | |
| *** bdeschenes has joined #openstack-ironic | 18:25 | |
| *** athomas has quit IRC | 18:25 | |
| *** jjohnson2 has joined #openstack-ironic | 18:34 | |
| *** bnemec has quit IRC | 18:34 | |
| TheJulia | lucasagomes: :( Indeed | 18:39 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add support to update keystone for bifrost https://review.openstack.org/380226 | 18:42 |
| *** ijw has joined #openstack-ironic | 18:47 | |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Add script to help build rpms https://review.openstack.org/382044 | 18:49 |
| *** ijw has quit IRC | 18:52 | |
| *** hoangcx has quit IRC | 19:05 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml https://review.openstack.org/380227 | 19:06 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml https://review.openstack.org/380227 | 19:07 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone install and clouds.yaml to testing https://review.openstack.org/380228 | 19:09 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Convert ironic.conf to template generated https://review.openstack.org/305634 | 19:10 |
| JayF | https://review.openstack.org/#/c/377957/ pretty easy core review if someone has a moment, I just tossed my +2 on it | 19:15 |
| patchbot | patch 377957 - ironic - Adding Timing metrics for DRAC drivers. | 19:15 |
| xavierr|afk | lucasagomes: omg :( | 19:15 |
| *** xavierr|afk is now known as xavierr | 19:15 | |
| rloo | JayF: I just looked; not so easy to +2 | 19:26 |
| JayF | rloo: uh-oh, did I miss something? | 19:27 |
| rloo | JayF: well, you know the metrics stuff isn't quite clear in my mind. i pointed out what seemed like inconsistencies to me, like why metric x but not metric y. also wonder if a drac person should review | 19:27 |
| JayF | fwiw; When you initialize the metrics logger with __name__ above, you're adding the module name/path to the metric value, so it'll include drac/bios | 19:28 |
| JayF | so the module name is included | 19:28 |
| rloo | JayF: also, there were some metrics for 'simple' stuff like changing node.driver_internal_info or something and doing a .save. | 19:28 |
| JayF | aslezil: ^ you may want to take a look at ruby's comments on your patch and respond | 19:29 |
| JayF | rloo: okay, thanks for having a look, I'll work with Annie to help resolve the issues | 19:29 |
| rloo | JayF: ok, i thought it did but i wasn't sure. (the module name) | 19:29 |
| rloo | JayF: does our metric documentation give hints as to what might be useful to gather metrics for? | 19:29 |
| JayF | http://docs.openstack.org/developer/ironic/deploy/metrics.html#types-of-metrics-emitted ... not really | 19:30 |
| rloo | JayF: if you have in your head, any useful guidelines, might be good to add there. Eg, passthrus, anything that 'talks' to the HW, ?? | 19:31 |
| JayF | I don't have too many things in my head | 19:31 |
| JayF | probably more along the lines of a negative list if anything | 19:31 |
| JayF | stuff you mentioned, like if it's just simply updating a node and doing a db call | 19:31 |
| rloo | JayF: negative is good too | 19:31 |
| JayF | we should be timing that db save at the db layer, not at every method that touches it | 19:32 |
| rloo | JayF: if we want to time db stuff? | 19:32 |
| JayF | yeah | 19:33 |
| JayF | like def update_node(thing): node.driver_info = {'foo':'bar'} node.save() shouldn't be metric'd, because we should have a timing metrics in the node.save() hook | 19:33 |
| JayF | tbh I don't know if we do today or not | 19:34 |
| JayF | but that's probably a better way to go about it | 19:34 |
| rloo | i'd be concerned about timings for db stuff, cuz we update a node fairly frequently. but i have no idea whether someone out there might want that metric | 19:34 |
| JayF | that's a metric I'd care the most about | 19:35 |
| JayF | because it's a choke point for a large part of the code | 19:35 |
| *** ijw has joined #openstack-ironic | 19:35 | |
| JayF | most people aren't going to be concerned, talking about metrics infra, about sending lots of data points for a given metric name | 19:35 |
| rloo | JayF: oh, interesting. Just the thing.save()? | 19:35 |
| JayF | generally the harder piece to scale is individual metrics (i.e. unique names) | 19:35 |
| JayF | statsd actually supports automatically doing math so your storage backend wouldn't see the 1000 times node.save() was called in a minute, it'd roll them up | 19:36 |
| rloo | JayF: well, if you're interested in those numbers, go for it! | 19:37 |
| *** ijw has quit IRC | 19:40 | |
| *** tomstatt has joined #openstack-ironic | 19:47 | |
| *** bjolo_ has quit IRC | 19:49 | |
| *** bjolo_ has joined #openstack-ironic | 19:49 | |
| tomstatt | hi all, I'd like to deploy ironic with Virtualbox instead of a baremetal for training purpose, where can I get the instruction? | 19:55 |
| *** aNuposic has quit IRC | 20:00 | |
| *** dprince has quit IRC | 20:01 | |
| *** bjolo_ has quit IRC | 20:01 | |
| lucasagomes | tomstatt, I think you have few options #1 use the pxe_ssh driver, #2 the vbox driver, #3 the libvirt driver from ironic-staging-drivers project | 20:01 |
| *** aNuposic has joined #openstack-ironic | 20:01 | |
| openstackgerrit | Ricardo Araújo Santos proposed openstack/ironic: Fixes parameters validation in SSH power manager. https://review.openstack.org/366150 | 20:02 |
| lucasagomes | tomstatt, http://docs.openstack.org/developer/ironic/drivers/vbox.html (this is for #2) | 20:02 |
| *** aNuposic has quit IRC | 20:06 | |
| *** ccamacho has quit IRC | 20:08 | |
| *** Goneri has quit IRC | 20:09 | |
| *** e0ne has quit IRC | 20:09 | |
| *** harlowja has joined #openstack-ironic | 20:15 | |
| tomstatt | lucasagomes: Thanks for your guide, I read in the link and as my understand, I use a Windows computer so the correct choice is #2, is that correct? | 20:17 |
| lucasagomes | tomstatt, yes, that's correct | 20:18 |
| * lucasagomes never used it, but knows that the author of that driver used windows | 20:18 | |
| NobodyCam | :p | 20:19 |
| tomstatt | lucasagomes: I see, and as my understand, the section http://docs.openstack.org/developer/ironic/drivers/vbox.html#setting-up-development-environment is for #2 | 20:22 |
| *** phschwartz has quit IRC | 20:22 | |
| lucasagomes | tomstatt, yeah this whole document is for #2 | 20:23 |
| tomstatt | lucasagomes: Do you think step "Set up devstack." means "git clone <devstack>" only or include running "./stack.sh" as well? | 20:25 |
| *** joprovost has quit IRC | 20:29 | |
| mat128 | tomstatt: I would only do the initial devstack part, because you are asked to change configurations right after | 20:32 |
| mat128 | tomstatt: to leverage virtualbox as an ironic driver | 20:32 |
| tomstatt | mat128: as my understand, I will follow this instruction: http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack to deploy ironic with devstack first | 20:34 |
| vdrok | folks, I'd appreciate your opinions about https://review.openstack.org/295371 when you have a bit of time, especially regarding some comments in PS#13. I think it would be great to make it work as it is supposed to | 20:35 |
| patchbot | patch 295371 - ironic - Add user and project domains to ironic context | 20:35 |
| mat128 | tomstatt: if you follow the section entirely, it's going to try and spawn libvirt VMs inside of your initial virtualbox VM | 20:35 |
| tomstatt | mat128: after that /etc/ironic/ironic.conf will be created and modified follow instruction in http://docs.openstack.org/developer/ironic/drivers/vbox.html | 20:39 |
| *** ElCoyote_1 has quit IRC | 20:39 | |
| tomstatt | mat128: in http://docs.openstack.org/developer/ironic/drivers/vbox.html, the driver is one of pxe_vbox, agent_vbox, or fake_vbox, is that correct? I don't see instruction how to use libvirt driver. | 20:41 |
| *** ElCoyote_ has joined #openstack-ironic | 20:42 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 20:42 |
| mat128 | tomstatt: that's correct, you don't want to use libvirt | 20:42 |
| mat128 | tomstatt: you want to use virtualbox | 20:42 |
| mat128 | tomstatt: thats why I suggested stopping after creating local.conf, to apply the virtualbox-specific edits | 20:43 |
| mat128 | tomstatt: "Install pyremotevbox" and on | 20:43 |
| *** aNuposic has joined #openstack-ironic | 20:44 | |
| *** ElCoyote_ has left #openstack-ironic | 20:46 | |
| openstackgerrit | Alexander Bashmakov proposed openstack/ironic: Rewrite create-node.sh in Python. https://review.openstack.org/346832 | 20:46 |
| tomstatt | mat128: "apply the virtualbox-specific edits" mean update pxe_vbox, agent_vbox, or fake_vbox in IRONIC_ENABLED_DRIVERS and IRONIC_DEPLOY_DRIVER, is that correct? | 20:46 |
| mariojv | hey rloo - yuriyz has a really small notification patch set here, if you're not too sick of reviewing patch sets related to this :) https://review.openstack.org/#/c/379469 | 20:47 |
| patchbot | patch 379469 - ironic - Add a possibility early to determine if notificati... | 20:47 |
| mariojv | basically lets us be lazy in sending notifications if we want too (for example, if creating the payload is an expensive operation, just skip it if notifications aren't enabled) | 20:47 |
| mariojv | *to | 20:47 |
| rloo | mariojv: I'm SICK! :) will try to look later today or this week anyway | 20:47 |
| *** ElCoyote_ has joined #openstack-ironic | 20:48 | |
| mat128 | tomstatt: yes | 20:48 |
| * mariojv notifies a doctor | 20:48 | |
| mat128 | mariojv: I'm sure she can find a doctor on her own, unless the situation is similar to Montreal ;) | 20:48 |
| tomstatt | mat128: thank you and lucasagomes a lot. | 20:48 |
| mariojv | mat128: i was making a really bad pun :P | 20:49 |
| *** phschwartz has joined #openstack-ironic | 20:49 | |
| lucasagomes | yvw! | 20:49 |
| *** bdeschenes has quit IRC | 20:50 | |
| lucasagomes | I'm going to call it a day! Have a great evening all | 20:52 |
| *** rbudden has joined #openstack-ironic | 20:53 | |
| xavierr | evening, see you tomorrow lucasagomes :) | 20:56 |
| *** krtaylor has quit IRC | 21:02 | |
| mariojv | where would people like reviewer attention focused before the summit? i'm guessing the "Bump to Ocata" list on trello would be a good place to look? | 21:06 |
| * mariojv won't be at the summit unfortunately, just curious | 21:06 | |
| jlvillal | mariojv: I have a good feeling you can go to the Boston one! | 21:09 |
| jlvillal | mariojv: Very sorry you won't be in Barcelona though :( | 21:10 |
| mariojv | indeed jlvillal. i know it's SO far from somerville (/s), but i'll try to make the 10 min subway ride | 21:10 |
| jlvillal | heh! The benefits of moving. | 21:10 |
| jlvillal | mariojv: You will have to figure out where the next North American summit will be after that and you can move near there too. | 21:11 |
| ElCoyote_ | mat128: is the Montreal situation really so bad that it's now common news? | 21:12 |
| *** trown is now known as trown|outtypewww | 21:12 | |
| rybridges2 | Hey OpenStackers! When I do 'nova quota-show', I see 3 fields. 'Quota', 'Limit', and 'In Use'. I am writing an app that essentially needs to get these 3 fields from some cluster. So in the python code in the app, I am currently using the 'quotas.get' api in the nova client documented here -> http://docs.openstack.org/developer/python-novaclient/ref/v2/quotas.html however that api is only returning | 21:13 |
| rybridges2 | 'Quota' and 'Limit', but it is NOT returning 'In Use.' Is there some API that I can call to get that 'In Use' part of 'nova quota-show'? | 21:13 |
| mat128 | ElCoyote_: Unless you have a family doctor, seeing one for a semi-urgent matter is very difficult | 21:13 |
| ElCoyote_ | Oh, that.. | 21:14 |
| mat128 | ElCoyote_: yes, what did you think? | 21:14 |
| ElCoyote_ | the scarcity of doctors and the lack of preventive measures, that's what I thought. | 21:14 |
| mat128 | well thats it :) | 21:15 |
| mat128 | ElCoyote_: preventive measures are probably a good mid/long-term investment | 21:15 |
| ElCoyote_ | funny you should mention that.. it's part of the reason wifey wants to go back to Europe (we've been in YUL for 3 years now.._ | 21:16 |
| *** bdeschenes has joined #openstack-ironic | 21:19 | |
| *** r-daneel has joined #openstack-ironic | 21:20 | |
| *** adam_g` is now known as adam_g | 21:25 | |
| *** jheroux has quit IRC | 21:30 | |
| mariojv | heading out for the evening o/ | 21:32 |
| *** mkam has quit IRC | 21:32 | |
| mat128 | ElCoyote_: there's always https://www.bonjour-sante.ca to get an appointment in an walk-in clinic | 21:34 |
| mat128 | ElCoyote_: it saved our ass a few times | 21:35 |
| mat128 | you have to pay though, so much a for a universal healthcare system | 21:35 |
| *** jcoufal_ has quit IRC | 21:45 | |
| *** krtaylor has joined #openstack-ironic | 21:47 | |
| *** sabeen has joined #openstack-ironic | 21:59 | |
| *** baoli has quit IRC | 22:03 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic: Adding Timing metrics for DRAC drivers. https://review.openstack.org/377957 | 22:05 |
| *** tomstatt has quit IRC | 22:07 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-specs: RAID configuration during deployment https://review.openstack.org/382091 | 22:16 |
| mat128 | dtantsur|afk: here's the first patchset ^ enjoy | 22:16 |
| vgadiraj | Could not reproduce this bug https://bugs.launchpad.net/ironic/+bug/1626977 on latest master branch. An error is in fact, being raised. I think this bug may have been fixed and can be closed. | 22:16 |
| openstack | Launchpad bug 1626977 in Ironic "Ironic should raise an error when updating unsupported field" [High,Triaged] - Assigned to Vadim Hmyrov (vhmyrov) | 22:16 |
| JayF | two CNR reports in that bug | 22:18 |
| JayF | unless there's objection, I'm going to mark it as invalid | 22:19 |
| JayF | well, I'll go incomplete and ask for more information wrt reproducing it | 22:19 |
| JayF | thanks vgadiraj, I went ahead and added a comment and marked it incomplete | 22:21 |
| *** wajdi_ has joined #openstack-ironic | 22:21 | |
| *** bdeschenes has quit IRC | 22:23 | |
| jlvillal | rybridges2: I don't know the answer to your question. I wonder if they would know in #openstack-nova | 22:24 |
| jlvillal | rybridges2: I don't think Ironic has much to do with the quotas. I could be wrong though. | 22:25 |
| *** wajdi has quit IRC | 22:25 | |
| *** wajdi_ has quit IRC | 22:26 | |
| rybridges2 | thanks jlvillal | 22:31 |
| openstackgerrit | Annie Lezil proposed openstack/ironic: Add description for vendor passthru methods https://review.openstack.org/370469 | 22:32 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui: Fix Jasmine unit tests https://review.openstack.org/382095 | 22:34 |
| *** sabeen has quit IRC | 22:37 | |
| *** baoli has joined #openstack-ironic | 22:39 | |
| *** rbudden has quit IRC | 22:47 | |
| *** ijw has joined #openstack-ironic | 22:48 | |
| *** rcernin has quit IRC | 22:50 | |
| *** rpioso has quit IRC | 22:51 | |
| *** r-daneel has quit IRC | 22:52 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: API Evolution - Error Responses https://review.openstack.org/381989 | 22:52 |
| *** wajdi has joined #openstack-ironic | 22:53 | |
| *** ijw has quit IRC | 22:53 | |
| *** sdake has joined #openstack-ironic | 22:58 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: API Evolution - Implement Tasks Resource https://review.openstack.org/382017 | 22:59 |
| *** rbrud has quit IRC | 22:59 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: DNM: Testing IPA usability with no valid cert https://review.openstack.org/382105 | 23:00 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: DNM: Testing IPA usability with no valid cert https://review.openstack.org/382106 | 23:00 |
| *** wajdi has quit IRC | 23:03 | |
| *** rajinir has quit IRC | 23:05 | |
| *** sdake_ has joined #openstack-ironic | 23:05 | |
| *** sdake has quit IRC | 23:09 | |
| openstackgerrit | Michael Davies proposed openstack/ironic: Update ironic config docs for keystone v3 https://review.openstack.org/382108 | 23:10 |
| *** wajdi has joined #openstack-ironic | 23:16 | |
| *** wajdi has quit IRC | 23:22 | |
| *** wajdi_ has joined #openstack-ironic | 23:24 | |
| *** penick has quit IRC | 23:26 | |
| *** wajdi_ has quit IRC | 23:29 | |
| *** wajdi has joined #openstack-ironic | 23:29 | |
| openstackgerrit | Merged openstack/ironic: [install-guide] describe pxe.ipxe_swift_tempurl https://review.openstack.org/379358 | 23:29 |
| *** penick has joined #openstack-ironic | 23:30 | |
| *** bdeschenes has joined #openstack-ironic | 23:36 | |
| *** awiddersheim has quit IRC | 23:47 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Invalid URL and typo in .rst files https://review.openstack.org/380942 | 23:48 |
| *** awiddersheim has joined #openstack-ironic | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!