| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Change config option from 'ironic-inspector' to 'ironic_inspector' https://review.openstack.org/527815 | 00:10 |
|---|---|---|
| openstackgerrit | Ilya Etingof proposed openstack/ironic-python-agent master: Report /dev/disk/by-path on inspection followup https://review.openstack.org/507858 | 00:12 |
| *** rama_y has quit IRC | 00:15 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Change config option to 'ironic_inspector' and False by default https://review.openstack.org/527815 | 00:16 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/527730 | 00:19 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Remove ironic_tempest_plugin/ directory https://review.openstack.org/527733 | 00:19 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: DNM/WIP: Testing tempest issues https://review.openstack.org/527757 | 00:19 |
| *** hjensas has quit IRC | 00:19 | |
| *** yuanying_ has joined #openstack-ironic | 00:25 | |
| *** yuanying has quit IRC | 00:27 | |
| *** fragatina has quit IRC | 00:28 | |
| *** fragatina has joined #openstack-ironic | 00:28 | |
| *** fragatin_ has joined #openstack-ironic | 00:33 | |
| *** fragatina has quit IRC | 00:33 | |
| *** fragatin_ has quit IRC | 00:34 | |
| *** harlowja has quit IRC | 00:45 | |
| *** fragatina has joined #openstack-ironic | 00:49 | |
| *** rama_y has joined #openstack-ironic | 00:52 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 00:53 | |
| *** fragatina has quit IRC | 00:54 | |
| *** AlexeyAbashkin has quit IRC | 00:57 | |
| *** yuanying_ has quit IRC | 01:00 | |
| *** phuongnh has joined #openstack-ironic | 01:09 | |
| *** yuanying has joined #openstack-ironic | 01:09 | |
| *** yangyapeng has joined #openstack-ironic | 01:12 | |
| *** Goneri has quit IRC | 01:13 | |
| *** zhangfei has joined #openstack-ironic | 01:14 | |
| *** rloo has quit IRC | 01:16 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-inspector master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/527742 | 01:16 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-inspector master: Remove ironic_inspector/test/inspector_tempest_plugin/ directory https://review.openstack.org/527743 | 01:16 |
| *** rloo has joined #openstack-ironic | 01:17 | |
| *** hw_wutianwei has joined #openstack-ironic | 01:17 | |
| *** Sukhdev has quit IRC | 01:19 | |
| *** rloo has quit IRC | 01:29 | |
| *** namnh has joined #openstack-ironic | 01:37 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Remove deprecated ironic.common.policy.enforce() https://review.openstack.org/527831 | 01:39 |
| *** snasir has quit IRC | 01:44 | |
| *** snasir has joined #openstack-ironic | 01:48 | |
| *** Goneri has joined #openstack-ironic | 01:55 | |
| *** jkilpatr has quit IRC | 02:00 | |
| *** litao__ has joined #openstack-ironic | 02:01 | |
| *** jlabarre has joined #openstack-ironic | 02:05 | |
| *** absubram has quit IRC | 02:20 | |
| *** threestrands_ has joined #openstack-ironic | 02:22 | |
| *** d0ugal has quit IRC | 02:22 | |
| *** threestrands_ has quit IRC | 02:22 | |
| *** threestrands_ has joined #openstack-ironic | 02:22 | |
| *** threestrands has quit IRC | 02:22 | |
| *** threestrands_ has quit IRC | 02:22 | |
| *** threestrands_ has joined #openstack-ironic | 02:23 | |
| *** threestrands_ has quit IRC | 02:23 | |
| *** threestrands_ has joined #openstack-ironic | 02:23 | |
| *** d0ugal has joined #openstack-ironic | 02:24 | |
| *** absubram has joined #openstack-ironic | 02:26 | |
| *** Goneri has quit IRC | 02:27 | |
| *** trungnv has quit IRC | 02:29 | |
| *** trungnv has joined #openstack-ironic | 02:29 | |
| *** absubram has quit IRC | 02:30 | |
| *** threestrands_ has quit IRC | 02:32 | |
| *** yuanying has quit IRC | 02:32 | |
| *** kzaitsev_pi has quit IRC | 02:32 | |
| *** ijw has quit IRC | 02:32 | |
| *** amotoki has quit IRC | 02:32 | |
| *** openstackstatus has quit IRC | 02:32 | |
| *** etingof has quit IRC | 02:32 | |
| *** dmellado has quit IRC | 02:32 | |
| *** Grenth has quit IRC | 02:36 | |
| *** threestrands_ has joined #openstack-ironic | 02:36 | |
| *** yuanying has joined #openstack-ironic | 02:36 | |
| *** kzaitsev_pi has joined #openstack-ironic | 02:36 | |
| *** ijw has joined #openstack-ironic | 02:36 | |
| *** amotoki has joined #openstack-ironic | 02:36 | |
| *** openstackstatus has joined #openstack-ironic | 02:36 | |
| *** etingof has joined #openstack-ironic | 02:36 | |
| *** dmellado has joined #openstack-ironic | 02:36 | |
| *** barjavel.freenode.net sets mode: +v openstackstatus | 02:36 | |
| *** Grenth has joined #openstack-ironic | 02:37 | |
| *** rbudden has quit IRC | 02:38 | |
| *** rbudden has joined #openstack-ironic | 02:43 | |
| openstackgerrit | zenghui.shi proposed openstack/ironic-specs master: Hardware interface for BIOS configuration https://review.openstack.org/496481 | 02:46 |
| *** TuanLA has joined #openstack-ironic | 02:55 | |
| *** Gr3nth has joined #openstack-ironic | 03:02 | |
| *** Grenth has quit IRC | 03:02 | |
| *** rbudden has quit IRC | 03:11 | |
| *** penick has quit IRC | 03:25 | |
| *** udesale has joined #openstack-ironic | 04:03 | |
| *** rama_y has quit IRC | 04:15 | |
| *** karthiks has quit IRC | 04:17 | |
| *** karthiks has joined #openstack-ironic | 04:17 | |
| *** MattMan has quit IRC | 04:27 | |
| *** MattMan has joined #openstack-ironic | 04:27 | |
| *** TuanLA has quit IRC | 04:36 | |
| *** TuanLA has joined #openstack-ironic | 04:36 | |
| *** namnh has quit IRC | 04:40 | |
| *** phuongnh has quit IRC | 04:40 | |
| *** trungnv has quit IRC | 04:40 | |
| *** phuongnh has joined #openstack-ironic | 04:40 | |
| *** trungnv has joined #openstack-ironic | 04:40 | |
| *** namnh has joined #openstack-ironic | 04:40 | |
| *** rama_y has joined #openstack-ironic | 05:06 | |
| *** adisky__ has joined #openstack-ironic | 05:18 | |
| *** tzumainn has quit IRC | 05:29 | |
| *** links has joined #openstack-ironic | 05:35 | |
| *** penick has joined #openstack-ironic | 05:48 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Adds rescue_interface to base driver class https://review.openstack.org/509335 | 05:53 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Adds RPC calls for rescue interface https://review.openstack.org/509336 | 05:53 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescue related methods to network interface https://review.openstack.org/509342 | 05:53 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 05:53 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Agent rescue implementation https://review.openstack.org/400437 | 05:53 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add API methods for [un]rescue https://review.openstack.org/350831 | 05:53 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescue interface field to node-related notifications https://review.openstack.org/502889 | 05:53 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Update description for config params of 'rescue' interface https://review.openstack.org/517919 | 05:53 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add documentation for rescue interface https://review.openstack.org/419606 | 05:53 |
| *** penick_ has joined #openstack-ironic | 05:53 | |
| *** penick has quit IRC | 05:54 | |
| *** udesale__ has joined #openstack-ironic | 05:54 | |
| *** sasha2 has quit IRC | 05:55 | |
| *** udesale has quit IRC | 05:56 | |
| *** udesale has joined #openstack-ironic | 05:57 | |
| *** udesale__ has quit IRC | 05:59 | |
| *** links has quit IRC | 06:13 | |
| *** links has joined #openstack-ironic | 06:21 | |
| *** links has quit IRC | 06:21 | |
| *** marios has joined #openstack-ironic | 06:24 | |
| *** etingof has quit IRC | 06:30 | |
| *** karthiks has quit IRC | 06:33 | |
| *** namnh has quit IRC | 06:41 | |
| *** trungnv has quit IRC | 06:41 | |
| *** phuongnh has quit IRC | 06:41 | |
| *** namnh has joined #openstack-ironic | 06:41 | |
| *** trungnv has joined #openstack-ironic | 06:41 | |
| *** phuongnh has joined #openstack-ironic | 06:42 | |
| *** SerenaFeng has joined #openstack-ironic | 06:42 | |
| *** SerenaFeng has quit IRC | 06:43 | |
| *** karthiks has joined #openstack-ironic | 06:46 | |
| *** openstackgerrit has quit IRC | 06:47 | |
| *** TuanLA_ has joined #openstack-ironic | 06:48 | |
| *** jaganathan has joined #openstack-ironic | 06:49 | |
| *** TuanLA has quit IRC | 06:50 | |
| *** mjura has joined #openstack-ironic | 06:52 | |
| *** threestrands_ has quit IRC | 06:53 | |
| *** karthiks has quit IRC | 06:55 | |
| *** stendulker has joined #openstack-ironic | 07:07 | |
| *** karthiks has joined #openstack-ironic | 07:09 | |
| *** e0ne has joined #openstack-ironic | 07:09 | |
| *** vsaienk0 has joined #openstack-ironic | 07:12 | |
| *** namnh has quit IRC | 07:14 | |
| *** phuongnh has quit IRC | 07:14 | |
| *** trungnv has quit IRC | 07:14 | |
| *** trungnv has joined #openstack-ironic | 07:14 | |
| *** namnh has joined #openstack-ironic | 07:14 | |
| *** phuongnh has joined #openstack-ironic | 07:14 | |
| *** rbartal has joined #openstack-ironic | 07:18 | |
| *** rama_y has quit IRC | 07:26 | |
| *** openstackgerrit has joined #openstack-ironic | 07:28 | |
| openstackgerrit | Vu Cong Tuan proposed openstack/ironic-python-agent master: Remove /ironic-python-agent/api/app.wsgi https://review.openstack.org/523341 | 07:28 |
| *** karthiks has quit IRC | 07:31 | |
| *** makowals has joined #openstack-ironic | 07:37 | |
| *** karthiks has joined #openstack-ironic | 07:49 | |
| *** ddurst- has quit IRC | 07:54 | |
| *** ddurst has joined #openstack-ironic | 07:59 | |
| *** marios has quit IRC | 08:03 | |
| *** marios has joined #openstack-ironic | 08:06 | |
| *** dpawar has joined #openstack-ironic | 08:07 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 08:09 | |
| *** hjensas has joined #openstack-ironic | 08:12 | |
| *** rcernin has quit IRC | 08:15 | |
| *** penick_ has quit IRC | 08:18 | |
| *** absubram has joined #openstack-ironic | 08:21 | |
| *** absubram_ has joined #openstack-ironic | 08:25 | |
| *** absubram has quit IRC | 08:25 | |
| *** absubram_ is now known as absubram | 08:25 | |
| *** TuanLA__ has joined #openstack-ironic | 08:26 | |
| *** pmannidi has quit IRC | 08:26 | |
| *** tesseract has joined #openstack-ironic | 08:28 | |
| *** TuanLA_ has quit IRC | 08:28 | |
| *** dsneddon has quit IRC | 08:34 | |
| *** hoonetorg has quit IRC | 08:37 | |
| *** dsneddon has joined #openstack-ironic | 08:39 | |
| *** absubram has quit IRC | 08:42 | |
| *** zzzeek has quit IRC | 08:43 | |
| *** zzzeek has joined #openstack-ironic | 08:45 | |
| openstackgerrit | Nikolay Fedotov proposed openstack/ironic-python-agent master: Try to find partition 3 times https://review.openstack.org/525577 | 08:45 |
| *** links has joined #openstack-ironic | 08:52 | |
| *** e0ne has quit IRC | 08:53 | |
| *** ccamacho has joined #openstack-ironic | 08:53 | |
| *** patchbot has quit IRC | 08:53 | |
| *** patchbot has joined #openstack-ironic | 08:54 | |
| *** hoonetorg has joined #openstack-ironic | 08:55 | |
| *** huats has quit IRC | 08:55 | |
| *** bkero has quit IRC | 08:55 | |
| *** huats has joined #openstack-ironic | 08:55 | |
| *** bkero has joined #openstack-ironic | 08:57 | |
| *** links has quit IRC | 08:57 | |
| *** ijw has quit IRC | 09:03 | |
| *** ijw has joined #openstack-ironic | 09:05 | |
| *** ijw has quit IRC | 09:07 | |
| *** ijw has joined #openstack-ironic | 09:08 | |
| *** vsaienk0 has quit IRC | 09:08 | |
| *** ijw has quit IRC | 09:09 | |
| *** dsneddon_ has joined #openstack-ironic | 09:09 | |
| *** links has joined #openstack-ironic | 09:10 | |
| *** vsaienk0 has joined #openstack-ironic | 09:10 | |
| *** ijw has joined #openstack-ironic | 09:10 | |
| *** ijw has quit IRC | 09:11 | |
| *** dsneddon has quit IRC | 09:13 | |
| *** ijw has joined #openstack-ironic | 09:13 | |
| *** ijw has quit IRC | 09:14 | |
| *** mgoddard has joined #openstack-ironic | 09:15 | |
| *** charcol has joined #openstack-ironic | 09:16 | |
| *** ijw has joined #openstack-ironic | 09:16 | |
| openstackgerrit | Mark Goddard proposed openstack/networking-generic-switch master: Support configuration of trunk ports https://review.openstack.org/524135 | 09:17 |
| openstackgerrit | Mark Goddard proposed openstack/networking-generic-switch master: Add support for trunk ports on Dell switches https://review.openstack.org/524138 | 09:17 |
| *** etingof has joined #openstack-ironic | 09:20 | |
| *** ijw has quit IRC | 09:20 | |
| *** rcernin has joined #openstack-ironic | 09:23 | |
| *** ijw has joined #openstack-ironic | 09:25 | |
| *** ijw_ has joined #openstack-ironic | 09:25 | |
| *** ijw has quit IRC | 09:26 | |
| *** yangyapeng has quit IRC | 09:29 | |
| *** yangyapeng has joined #openstack-ironic | 09:30 | |
| *** lucas-afk is now known as lucasagomes | 09:33 | |
| *** derekh has joined #openstack-ironic | 09:42 | |
| *** zhangfei has quit IRC | 09:48 | |
| *** yangyapeng has quit IRC | 09:49 | |
| *** yangyapeng has joined #openstack-ironic | 09:49 | |
| *** milan_ has joined #openstack-ironic | 09:59 | |
| *** yangyapeng has quit IRC | 10:00 | |
| milan_ | morning Ironic! :) | 10:02 |
| *** charcol has quit IRC | 10:04 | |
| *** TuanLA__ has quit IRC | 10:05 | |
| *** e0ne has joined #openstack-ironic | 10:07 | |
| etingof | milan_, o/ | 10:09 |
| milan_ | morning etingof! :) | 10:09 |
| *** namnh has quit IRC | 10:09 | |
| *** charcol has joined #openstack-ironic | 10:20 | |
| pas-ha | morning ironic etingof milan_ :) | 10:20 |
| *** sambetts|afk is now known as sambetts | 10:21 | |
| *** amarao has joined #openstack-ironic | 10:24 | |
| *** dtantsur|afk is now known as dtantsur | 10:25 | |
| dtantsur | morning ironic | 10:25 |
| *** yangyapeng has joined #openstack-ironic | 10:28 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: devstack: set [service_available]ironic-inspector = True https://review.openstack.org/527923 | 10:29 |
| *** vsaienk0 has quit IRC | 10:34 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Fix pep8 issue https://review.openstack.org/527925 | 10:35 |
| milan_ | morning pas-ha ! :) | 10:36 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Change config option to 'ironic_inspector' and False by default https://review.openstack.org/527815 | 10:36 |
| *** vsaienk0 has joined #openstack-ironic | 10:36 | |
| *** yangyapeng has quit IRC | 10:37 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: devstack: set [service_available]ironic-inspector = True https://review.openstack.org/527923 | 10:37 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/527730 | 10:38 |
| *** yangyapeng has joined #openstack-ironic | 10:40 | |
| *** udesale has quit IRC | 10:41 | |
| *** udesale has joined #openstack-ironic | 10:42 | |
| *** karthiks has quit IRC | 10:42 | |
| *** links has quit IRC | 10:43 | |
| *** jkilpatr has joined #openstack-ironic | 10:46 | |
| *** jtomasek has joined #openstack-ironic | 10:47 | |
| sambetts | milan_: just glancing through your inspector patches regarding the sync method, which is the slow part calling ironic to get the list or calling iptables to update the rules? | 10:47 |
| milan_ | sambetts, the iptables | 10:48 |
| milan_ | mgoddard has 210 ports, each update means 210 calls to iptables; for each node if I'm not mistaken | 10:48 |
| milan_ | so 6300 shell commands executed :D | 10:49 |
| * milan_ hopes he's wrong about the count though | 10:49 | |
| milan_ | sambetts, moreover, the lock is held during the sync and even worse, the API is single-threaded | 10:50 |
| dtantsur | sambetts: morning, could you please land https://review.openstack.org/#/c/527925/ and https://review.openstack.org/#/c/527815/ to unblock the tempest plugin work? | 10:50 |
| patchbot | patch 527925 - ironic-tempest-plugin - Fix pep8 issue | 10:50 |
| patchbot | patch 527815 - ironic-tempest-plugin - Change config option to 'ironic_inspector' and Fal... | 10:50 |
| sambetts | milan_: I wonder if there is a way we can use the iptables save and load feature to make it a single iptables request instead of many requests | 10:52 |
| sambetts | or at least batch them together somehow | 10:52 |
| milan_ | sambetts, that was my idea, the batching, with the patch I just abandoned (relying on the eventual consistency by periodic sync) | 10:53 |
| sambetts | milan_: basically right all the rules we want to apply to a file, and then do "iptables-apply <our rules file>" | 10:54 |
| milan_ | sambetts, it's the same semantically as to relax the lock | 10:54 |
| milan_ | or use a queue | 10:54 |
| milan_ | there's no guarantee that having called sync() | 10:54 |
| milan_ | a node is able to boot | 10:55 |
| milan_ | or wont' boot after a finish | 10:55 |
| milan_ | so dunno | 10:55 |
| mgoddard | sambetts: that could work. Do you know if iptables-restore is able to batch up calls to iptables? | 10:55 |
| milan_ | mgoddard, how would you be able to tell that a node got in the batch? | 10:56 |
| sambetts | mgoddard: iptables-apply is what you want I think, it'll apply new rules to the existing instead of replacing the whole set | 10:56 |
| sambetts | milan_: basically instead of "iptables ...; iptables ...; iptables ...;" we're now doing "iptables-apply ..." | 10:56 |
| sambetts | milan_: the locks and everyting remains as they are today | 10:57 |
| *** links has joined #openstack-ironic | 10:57 | |
| mgoddard | sambetts: are you sure? Looks like it just adds rollback support http://man7.org/linux/man-pages/man8/iptables-apply.8.html | 10:57 |
| milan_ | I see but the issue is the same: you can't guarantee that a node got in a batch, or am missing something? | 10:57 |
| milan_ | ofc, performance would be better | 10:58 |
| *** jtomasek has quit IRC | 10:58 | |
| sambetts | mgoddard: ah :/ I must have understood the man page wrong | 10:58 |
| * milan_ wonders about the situation of multiple syncs running in parallel; same issue we have but with fewer comands | 10:58 | |
| mgoddard | milan_: with multithreading, nodes would be batched if they block on the sync lock | 10:58 |
| mgoddard | milan_: as long as their node_info is correct | 10:59 |
| milan_ | mgoddard, each node would try to dump the batch file and upload it to iptables, correct? | 10:59 |
| milan_ | 30 x 210 ports in a batch this time | 11:00 |
| milan_ | less calls I agree | 11:00 |
| milan_ | so better performance | 11:00 |
| milan_ | but then one can just use the dnsmasq filter | 11:00 |
| milan_ | and call it a day | 11:00 |
| mgoddard | milan_: yes, but if the first sync blocks for 10s, then other nodes queue up behind it, the next sync will work for all of them. After that, calls to sync can be a noop due to the blacklist comparison | 11:01 |
| milan_ | mgoddard, it's exactly what the dnsmasq driver does | 11:01 |
| sambetts | mgoddard: ah looks like if you do "iptables-restore --noflush" it looks like it won't flush the table before applying the changes so then it'll apply new rules ontop of the old ones (I guess) /me experiments locally | 11:03 |
| milan_ | https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/pxe_filter/dnsmasq.py#L74,#L76 | 11:03 |
| mgoddard | sambetts, milan_: Is there a reason not to just use the dnsmasq filter? | 11:04 |
| sambetts | I thought you'd have to cherry-pick it | 11:04 |
| *** dyasny_ has joined #openstack-ironic | 11:04 | |
| milan_ | mgoddard, I'd strongly recommend using dnsmasq filter | 11:04 |
| milan_ | yeah, that | 11:04 |
| milan_ | cherry-pick | 11:04 |
| mgoddard | milan_: it's also what the iptables filter does: https://github.com/openstack/ironic-inspector/blob/stable/pike/ironic_inspector/firewall.py#L192 | 11:05 |
| *** zzzeek has quit IRC | 11:05 | |
| mgoddard | I guess it's cherry pick vs. develop new code for iptables | 11:05 |
| mgoddard | cherry pick seems the better option | 11:06 |
| milan_ | hmmm | 11:06 |
| sambetts | iptables is deprecated right? | 11:06 |
| * sambetts is out of context so just checking | 11:06 | |
| *** etingof has quit IRC | 11:07 | |
| milan_ | mgoddard, that checks if blacklist cache didn't change, which is going to change every time the nodes chime in thru the post@/v1/continue that I think why you experience the issue | 11:07 |
| milan_ | sambetts, IIRC we didn't deprecate it yet | 11:07 |
| *** jkilpatr has quit IRC | 11:08 | |
| *** dyasny has quit IRC | 11:08 | |
| sambetts | so if we know there is a perforcmance bug in iptables and its not deprecated yet, then I wonder if we should try to fix it or not | 11:08 |
| milan_ | but as dtantsur said the other day, let's just kill iptables with fire :D | 11:09 |
| dtantsur | ++ | 11:09 |
| *** ricardoas has quit IRC | 11:09 | |
| *** AlexeyAbashkin has quit IRC | 11:10 | |
| milan_ | ...strict consistency vs eventual consistency... | 11:10 |
| milan_ | cache anyone? | 11:10 |
| milan_ | #pixiesay -msad no, thank you, milan_ better slow than broken! | 11:10 |
| PixieBoots | ʕ•͡ᴥ•ʔ: no, thank you, milan_ better slow than broken! | 11:10 |
| *** sai has quit IRC | 11:11 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 11:11 | |
| *** sai has joined #openstack-ironic | 11:11 | |
| mgoddard | milan_: the cache won't necessarily have changed if multithreading is enabled - all nodes could update their node_info at the same time, then sync() only sees a change once | 11:12 |
| *** zzzeek has joined #openstack-ironic | 11:12 | |
| mgoddard | milan_: in that case, a long-ish sync time actually helps with batching | 11:12 |
| mgoddard | milan_: although a minute is pushing it a bit :) | 11:12 |
| milan_ | mgoddard, probably hopefully you're right! but same effect should appear with the dnsmasq driver, no? | 11:13 |
| *** ricardoas has joined #openstack-ironic | 11:14 | |
| milan_ | if Flask(__name__).run(threaded=True) was used | 11:14 |
| milan_ | then again, why not relax the locks? ;) | 11:14 |
| * milan_ still considers inspector DB, ironic and the filter state a cache of 3 elements (no matter dnsmasq or iptables); the sync() is a flush call or a dirty cache flag, depends on how we'd like to look at it; someone might be cool with an eventual consistency here | 11:17 | |
| milan_ | ie relaxing the locks (together with a config option to allow it) | 11:18 |
| *** ricardoas has quit IRC | 11:18 | |
| mgoddard | milan_: eventual consistency -> eventual fail? | 11:19 |
| milan_ | lol :D | 11:20 |
| milan_ | mgoddard, fail & retry | 11:20 |
| milan_ | manually introspect :D | 11:20 |
| milan_ | hence the config option to allow for it :D | 11:20 |
| milan_ | dtantsur loves the options that allow the operator to shoot their foot, right? :P | 11:21 |
| dtantsur | omg, more option | 11:21 |
| * dtantsur -2 | 11:21 | |
| milan_ | mgoddard, I agree with your concern, hence the patch was abandoned | 11:21 |
| milan_ | sambetts, speaking about inspector: https://review.openstack.org/#/c/523196/7 https://review.openstack.org/#/c/525685/8 ;) | 11:23 |
| patchbot | patch 523196 - ironic-inspector - fix dvsm config deprecations | 11:23 |
| patchbot | patch 525685 - ironic-inspector - Make discovery use dnsmasq dhcp filter | 11:23 |
| *** ricardoas has joined #openstack-ironic | 11:23 | |
| * milan_ -> lunch :P | 11:24 | |
| *** stendulker_ has joined #openstack-ironic | 11:24 | |
| *** stendulker has quit IRC | 11:27 | |
| *** stendulker_ has quit IRC | 11:28 | |
| *** ijw_ has quit IRC | 11:32 | |
| *** linkmark has joined #openstack-ironic | 11:40 | |
| *** lucasagomes is now known as lucas-hungry | 11:42 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for neutronclient https://review.openstack.org/476170 | 11:43 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Finalize migration to keystoneauth adapters https://review.openstack.org/478825 | 11:43 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: [WIP][DNM] Experiment with batch through iptables-restore https://review.openstack.org/527943 | 11:48 |
| sambetts | mgoddard: because I was curious ^ | 11:48 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: [WIP][DNM] Experiment with batch through iptables-restore https://review.openstack.org/527943 | 11:50 |
| mgoddard | sambetts: nice! | 11:52 |
| *** jtomasek has joined #openstack-ironic | 11:54 | |
| dtantsur | vsaienk0, sambetts: sorry, may I please re-ping you about https://review.openstack.org/#/c/527925/ and https://review.openstack.org/#/c/527815/ to unblock the tempest plugin work? we're a bit past schedule with it, and you seem to be the only cores online. | 11:58 |
| patchbot | patch 527925 - ironic-tempest-plugin - Fix pep8 issue | 11:58 |
| patchbot | patch 527815 - ironic-tempest-plugin - Change config option to 'ironic_inspector' and Fal... | 11:58 |
| sambetts | dtantsur: on it now! | 11:59 |
| dtantsur | ah, thanks :) | 11:59 |
| vsaienk0 | dtantsur: +A on both patches | 12:00 |
| sambetts | :) | 12:01 |
| dtantsur | \o/ | 12:04 |
| dtantsur | the next one is https://review.openstack.org/527923 but it hasn't passed the CI yet | 12:05 |
| patchbot | patch 527923 - ironic-inspector - devstack: set [service_available]ironic-inspector ... | 12:05 |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Fix pep8 issue https://review.openstack.org/527925 | 12:08 |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Change config option to 'ironic_inspector' and False by default https://review.openstack.org/527815 | 12:08 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Remove ironic_tempest_plugin/ directory https://review.openstack.org/527733 | 12:09 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/527742 | 12:10 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Remove ironic_inspector/test/inspector_tempest_plugin/ directory https://review.openstack.org/527743 | 12:11 |
| *** dpawar has quit IRC | 12:14 | |
| *** dpawar has joined #openstack-ironic | 12:20 | |
| *** udesale has quit IRC | 12:23 | |
| *** etingof has joined #openstack-ironic | 12:23 | |
| *** ricardoas has quit IRC | 12:28 | |
| *** bfournie has quit IRC | 12:29 | |
| *** bfournie has joined #openstack-ironic | 12:29 | |
| *** bfournie has quit IRC | 12:34 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO https://review.openstack.org/468288 | 12:38 |
| *** ijw has joined #openstack-ironic | 12:38 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Correct introspection rules path https://review.openstack.org/527949 | 12:39 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/527730 | 12:40 |
| *** hw_wutianwei has quit IRC | 12:42 | |
| dtantsur | sambetts, vsaienk0, please check https://review.openstack.org/#/c/527923/ now, it seems good | 12:44 |
| patchbot | patch 527923 - ironic-inspector - devstack: set [service_available]ironic-inspector ... | 12:45 |
| *** catintheroof has joined #openstack-ironic | 12:49 | |
| *** lucas-hungry is now known as lucasagomes | 12:50 | |
| *** fellypefca has joined #openstack-ironic | 12:58 | |
| dtantsur | sambetts, vsaienk0, quick question guys: do we support VXLAN networks with ironic "neutron" interface? do ML2 drivers (like Cisco) support it? | 12:59 |
| sambetts | dtantsur: our's does not right now, but it is thoerically possible to do if the hardware supports terminating VXLANs onto the ports on the switch | 13:00 |
| sambetts | I think our hardware can do it | 13:00 |
| sambetts | but our ml2 driver doesn't have the code to support it | 13:00 |
| dtantsur | I see. do you know some ML2 driver that does? or if n-g-s does? | 13:00 |
| fellypefca | Hey Ironic, good morning. :) | 13:02 |
| sambetts | I don't know of any, I remember talking to Sukdev at one of the summits about it, so I don't know if arista added support for it when he worked on that stuff | 13:02 |
| dtantsur | ok thanks | 13:02 |
| vsaienk0 | dtantsur: n-g-s doesn't support vxlan, it will require vxlan to vlan mapping supported on hardware, and usually it will be only on expensive switches. vendor plugin should support this, I'm sure arista does | 13:02 |
| sambetts | n-g-s definatly doesn't have it | 13:02 |
| dtantsur | thanks guys, yeah. this matches my expectations | 13:03 |
| *** kzaitsev_pi has quit IRC | 13:03 | |
| *** phuongnh has quit IRC | 13:03 | |
| *** kzaitsev_pi has joined #openstack-ironic | 13:04 | |
| openstackgerrit | Michael Turek proposed openstack/ironic-python-agent master: Use lshw in place of dmidecode for the default hardware manager https://review.openstack.org/501799 | 13:07 |
| *** vsaienk0 has quit IRC | 13:13 | |
| *** dpawar has quit IRC | 13:14 | |
| *** vsaienk0 has joined #openstack-ironic | 13:16 | |
| *** bfournie has joined #openstack-ironic | 13:24 | |
| *** tzumainn has joined #openstack-ironic | 13:27 | |
| *** vsaienk0 has quit IRC | 13:28 | |
| *** jcoufal has joined #openstack-ironic | 13:31 | |
| openstackgerrit | Fellype Cavalcante proposed openstack/ironic master: Migrate oneview hardware type to use python-hpOneView https://review.openstack.org/524310 | 13:34 |
| *** jcoufal_ has joined #openstack-ironic | 13:36 | |
| *** dprince has joined #openstack-ironic | 13:39 | |
| *** jcoufal has quit IRC | 13:39 | |
| *** ricardoas has joined #openstack-ironic | 13:41 | |
| TheJulia | Good morning everyone | 13:41 |
| *** vsaienk0 has joined #openstack-ironic | 13:41 | |
| fellypefca | Morning TheJulia | 13:42 |
| dtantsur | anyone with core rights on inspector, could you please merge https://review.openstack.org/#/c/527923/ ? | 13:43 |
| patchbot | patch 527923 - ironic-inspector - devstack: set [service_available]ironic-inspector ... | 13:43 |
| *** dpawar has joined #openstack-ironic | 13:54 | |
| milan_ | dtantsur, done | 13:55 |
| dtantsur | thnx! | 13:55 |
| milan_ | morning fellypefca, TheJulia! :) | 13:55 |
| fellypefca | hey hey milan_ | 13:55 |
| *** dpawar has quit IRC | 13:56 | |
| *** rcernin has quit IRC | 13:59 | |
| *** amarao1 has joined #openstack-ironic | 14:04 | |
| *** zhangfei has joined #openstack-ironic | 14:06 | |
| *** jtomasek has quit IRC | 14:06 | |
| *** sasha2 has joined #openstack-ironic | 14:09 | |
| *** rloo has joined #openstack-ironic | 14:09 | |
| *** sasha2 has quit IRC | 14:10 | |
| *** sasha2 has joined #openstack-ironic | 14:11 | |
| *** Goneri has joined #openstack-ironic | 14:27 | |
| *** jaganathan has quit IRC | 14:28 | |
| *** rbudden has joined #openstack-ironic | 14:31 | |
| *** ccamacho has quit IRC | 14:34 | |
| *** baoli has joined #openstack-ironic | 14:35 | |
| *** yangyapeng has quit IRC | 14:38 | |
| *** ccamacho has joined #openstack-ironic | 14:40 | |
| * TheJulia needs much more coffeeeeee | 14:47 | |
| rloo | good morning ironic'ers | 14:48 |
| * rloo passes carafe of coffee to TheJulia | 14:48 | |
| hwoarang | TheJulia: yolanda hello. Could you have a look at https://review.openstack.org/#/c/526324/ ? TIA :) this is mostly needed by the OPNFV 3rd party ci since we are using completely clean os images | 14:48 |
| patchbot | patch 526324 - bifrost - scripts: install-deps.sh: Ensure distros have pack... | 14:48 |
| hwoarang | thank you | 14:48 |
| TheJulia | hwoarang: approved | 14:49 |
| hwoarang | awesome thank you | 14:49 |
| dtantsur | morning rloo | 14:49 |
| rloo | hi dtantsur | 14:50 |
| *** exodusftw has quit IRC | 14:51 | |
| *** Gr3nth is now known as Grenth | 14:54 | |
| dtantsur | folks, the next tempest-related review please: https://review.openstack.org/#/c/527949/ | 14:54 |
| patchbot | patch 527949 - ironic-tempest-plugin - Correct introspection rules path | 14:54 |
| dtantsur | rloo, TheJulia, sambetts, whoever has time ^^^ | 14:56 |
| *** exodusftw has joined #openstack-ironic | 14:56 | |
| *** yangyapeng has joined #openstack-ironic | 14:56 | |
| TheJulia | reviewed | 14:57 |
| dtantsur | thnx | 14:57 |
| *** amarao1 has quit IRC | 14:58 | |
| *** rbartal has quit IRC | 14:58 | |
| dtantsur | this makes the inspector job green, now only multitenant job has to pass | 14:58 |
| rloo | dtantsur: you need two +2? | 14:58 |
| dtantsur | rloo: ideally :) and +A won't hurt as well | 14:58 |
| rloo | dtantsur: ok, give me a few sec... | 14:58 |
| rloo | dtantsur: done. profit! | 14:58 |
| TheJulia | heh | 14:58 |
| dtantsur | \o/ | 14:58 |
| dtantsur | if the last job is ok too, we can pull the trigger on ironic as soon as jlvillal wakes up | 14:59 |
| dtantsur | looks at this, it's all green: https://review.openstack.org/#/c/527730/ !! | 15:00 |
| patchbot | patch 527730 - ironic - Use the tempest plugin from openstack/ironic-tempe... | 15:00 |
| rloo | dtantsur: sweet. calling up the drummers... | 15:00 |
| rloo | green is such a nice colour... | 15:01 |
| *** yangyapeng has quit IRC | 15:02 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/527742 | 15:05 |
| openstackgerrit | Merged openstack/bifrost master: scripts: install-deps.sh: Ensure distros have package metadata available https://review.openstack.org/526324 | 15:06 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Remove ironic_tempest_plugin/ directory https://review.openstack.org/527733 | 15:06 |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Correct introspection rules path https://review.openstack.org/527949 | 15:06 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Remove ironic_inspector/test/inspector_tempest_plugin/ directory https://review.openstack.org/527743 | 15:08 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Remove ironic_inspector/test/inspector_tempest_plugin/ directory https://review.openstack.org/527743 | 15:08 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: Test OPNFV CI verify job. Please ignore. https://review.openstack.org/367608 | 15:10 |
| *** litao__ has quit IRC | 15:10 | |
| *** yangyapeng has joined #openstack-ironic | 15:14 | |
| *** r-daneel has joined #openstack-ironic | 15:16 | |
| *** mjturek has joined #openstack-ironic | 15:17 | |
| jlvillal | Good morning Ironic | 15:18 |
| jlvillal | dtantsur, Thanks for keeping up the work on the tempest stuff! And fixing issues :) | 15:18 |
| *** yangyapeng has quit IRC | 15:18 | |
| jlvillal | And rloo / TheJulia / sambetts / vsaienk0 for helping out too :) | 15:19 |
| dtantsur | morning jlvillal. np, we're very close :) actually, we can move forward with the ironic part iMO | 15:20 |
| rloo | morning jlvillal, a big day for you and tempest :) | 15:20 |
| jlvillal | dtantsur, I hope I wasn't too mean/rude in https://review.openstack.org/#/c/523341/ Maybe I shouldn't look at patches right before I go to sleep... | 15:20 |
| patchbot | patch 523341 - ironic-python-agent - Remove /ironic-python-agent/api/app.wsgi | 15:20 |
| *** charcol has quit IRC | 15:20 | |
| dtantsur | jlvillal: I'm really close to start doing the same all the time | 15:20 |
| jlvillal | dtantsur, I had forgotten but we do need to add some DSVM type jobs to the plugin repo | 15:21 |
| dtantsur | jlvillal: oh yeah, wanted to discuss with you. | 15:21 |
| dtantsur | jlvillal: we should probably think carefully which jobs we need there | 15:21 |
| jlvillal | I think we can copy a job or two over from ironic and a job over from inspecotr. | 15:21 |
| jlvillal | I'm not sure which ones though... | 15:21 |
| dtantsur | jlvillal: we need 1. one full job from ironic (multinode?), 2. one standalone job from ironic, 3. the default inspector job (the same that runs on ironic) | 15:22 |
| dtantsur | this is for the start, then we can think about adding other jobs | 15:22 |
| dtantsur | like, we'll probably need jobs from stable branches as well | 15:22 |
| jlvillal | dtantsur, That sounds like a good starting point | 15:22 |
| *** amotoki has quit IRC | 15:22 | |
| jlvillal | dtantsur, Yeah. I wonder if there is a way to have a stable branch job run??? | 15:23 |
| jlvillal | Maybe zuul v3 allows that. | 15:23 |
| dtantsur | jlvillal: there is a way, I'm pretty sure. e.g. tripleo-ci repo does it. | 15:23 |
| dtantsur | but we need to ask infra | 15:23 |
| jlvillal | Or we can manually do it by setting some devstack-gate variables | 15:23 |
| jlvillal | Cool | 15:23 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Switch emphasis to hardware types in the installation guide https://review.openstack.org/517290 | 15:24 |
| dtantsur | rloo: the next iteration ^^^ :) | 15:24 |
| *** rama_y has joined #openstack-ironic | 15:24 | |
| dtantsur | jlvillal: wanna unblock https://review.openstack.org/#/c/527730/ in the meantime? | 15:24 |
| patchbot | patch 527730 - ironic - Use the tempest plugin from openstack/ironic-tempe... | 15:24 |
| rloo | dtantsur: on my todo list, will wait til it renders | 15:24 |
| dtantsur | sure | 15:25 |
| jlvillal | dtantsur, Sure | 15:25 |
| *** zhangfei has quit IRC | 15:25 | |
| jlvillal | dtantsur, unblocked | 15:25 |
| dtantsur | rloo: wanna pull the trigger on ^^^? :) | 15:25 |
| rloo | dtantsur: looking... | 15:26 |
| rloo | sooooo many changes... :) | 15:26 |
| *** links has quit IRC | 15:26 | |
| dtantsur | yep :) | 15:27 |
| jlvillal | rloo, Yeah. We need to re-organize our zuul jobs so that we have more modularity... | 15:27 |
| rloo | jlvillal: that's the clean up that no one has offered to do yet :) | 15:27 |
| jlvillal | rloo, Yeah. Hopefully I will have some time to peek at it. But no promises :( | 15:28 |
| rloo | jlvillal: so that they aren't 'legacy' but beautiful things that represent the future! | 15:28 |
| dtantsur | rloo: oh, don't +A that change for now, one of dependencies is still in-flight | 15:29 |
| dtantsur | IIRC zuul won't like it :) | 15:29 |
| jlvillal | rloo, So basically the same basic change was made to each job (run.yaml) And then the one other change. | 15:29 |
| dtantsur | a +2 (if applicable) is enough :) | 15:29 |
| rloo | dtantsur: ok, i'm still reviewing anyway. i'll just +2 (if I'm happy with it) | 15:29 |
| rloo | jlvillal: at least it is easy to review after the first file :) | 15:30 |
| jlvillal | :) | 15:30 |
| jlvillal | dtantsur, about 19 minutes or so away from the in-flight change finishing the gate job... | 15:30 |
| dtantsur | neat | 15:30 |
| jlvillal | Okay time to get out of bed and make some breakfast :) | 15:31 |
| dtantsur | heh | 15:32 |
| jlvillal | dtantsur, The 'ironic-inspector' vs 'ironic_inspector' was strange. I looked at the oslo.config code and it didn't seem like it should work with that. | 15:32 |
| dtantsur | yeah, it's weird indeed | 15:32 |
| jlvillal | dtantsur, But when I changed it to 'ironic_inspector' in the INI file it magically worked. very weird. | 15:32 |
| dtantsur | previously we did not use this option, even though it was defined | 15:32 |
| dtantsur | this is a problem btw: http://logs.openstack.org/42/527742/4/check/ironic-inspector-grenade-dsvm/6ec0371/job-output.txt.gz#_2017-12-14_13_48_44_854590 | 15:34 |
| *** hw_wutianwei has joined #openstack-ironic | 15:34 | |
| dtantsur | I remember we had a similar issue before | 15:34 |
| *** yangyapeng has joined #openstack-ironic | 15:36 | |
| jlvillal | dtantsur, Failure http://logs.openstack.org/30/527730/6/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/e271c93/job-output.txt.gz#_2017-12-13_22_51_58_064326 | 15:36 |
| jlvillal | dtantsur, ini file with "ironic-inspector = False" http://logs.openstack.org/30/527730/6/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/e271c93/logs/tempest_conf.txt.gz | 15:36 |
| jlvillal | Strange... | 15:36 |
| jlvillal | Now I'm really going to go get breakfast | 15:37 |
| openstackgerrit | Merged openstack/ironic-inspector master: devstack: set [service_available]ironic-inspector = True https://review.openstack.org/527923 | 15:37 |
| jlvillal | dtantsur, rloo ^^ landed | 15:38 |
| jlvillal | dtantsur, Yeah, we have to backport stuff to pike | 15:38 |
| rloo | dtantsur: you want to do the honours? | 15:38 |
| dtantsur | rloo: feel free to go ahead | 15:38 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/pike: devstack: set [service_available]ironic-inspector = True https://review.openstack.org/527993 | 15:38 |
| rpioso | Good morning, ironicers. | 15:38 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ocata: devstack: set [service_available]ironic-inspector = True https://review.openstack.org/527994 | 15:39 |
| dtantsur | jlvillal: backports ^^^ | 15:39 |
| dtantsur | morning rpioso | 15:39 |
| rpioso | dtantsur: :) | 15:39 |
| dtantsur | ugh, I should not have done it.. | 15:39 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/527730 | 15:40 |
| *** yangyapeng has quit IRC | 15:40 | |
| dtantsur | jlvillal, rloo, had to remove dependencies and re-approve. the second I created backports, it started depending on them as well.. | 15:40 |
| rloo | dtantsur: oops | 15:41 |
| dtantsur | I always forget about this thing | 15:41 |
| rloo | dtantsur: no worries. if all our troubles were so easily fixed... :) | 15:42 |
| dtantsur | true | 15:42 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Remove ironic_tempest_plugin/ directory https://review.openstack.org/527733 | 15:44 |
| *** yangyapeng has joined #openstack-ironic | 15:56 | |
| openstackgerrit | Merged openstack/networking-generic-switch master: Support configuration of trunk ports https://review.openstack.org/524135 | 15:57 |
| *** catintheroof has quit IRC | 16:00 | |
| *** mjura has quit IRC | 16:00 | |
| *** catintheroof has joined #openstack-ironic | 16:00 | |
| *** yangyapeng has quit IRC | 16:01 | |
| *** hw_wutianwei has quit IRC | 16:04 | |
| *** catintheroof has quit IRC | 16:05 | |
| *** sasha2 has quit IRC | 16:14 | |
| *** yangyapeng has joined #openstack-ironic | 16:17 | |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add baremetal neutron agent https://review.openstack.org/456235 | 16:17 |
| fellypefca | TheJulia After the discussions with ricardoas, I thinks that it's everything ok with the first migration patch. Also, it's all green on CI :) https://review.openstack.org/#/c/523943/ | 16:17 |
| patchbot | patch 523943 - ironic - Introduce hpOneView and ilorest to OneView | 16:17 |
| *** amarao has quit IRC | 16:17 | |
| *** amarao has joined #openstack-ironic | 16:18 | |
| fellypefca | ^ And dtantsur, with you have some time to take a look :) | 16:18 |
| *** ccamacho has quit IRC | 16:19 | |
| *** amarao has quit IRC | 16:19 | |
| dtantsur | I hope I will.. unlikely today though | 16:19 |
| fellypefca | dtantsur np | 16:19 |
| *** amarao has joined #openstack-ironic | 16:20 | |
| *** yangyapeng has quit IRC | 16:21 | |
| *** rama_y has quit IRC | 16:22 | |
| *** bfournie has left #openstack-ironic | 16:23 | |
| *** amarao has quit IRC | 16:23 | |
| *** amarao has joined #openstack-ironic | 16:24 | |
| *** bfournie has joined #openstack-ironic | 16:28 | |
| *** sasha2 has joined #openstack-ironic | 16:33 | |
| *** hoonetorg has quit IRC | 16:35 | |
| rloo | hey, there are three +2 on this spec, it'll be +A'd sometime today unless someone has concerns: https://review.openstack.org/#/c/525870/ | 16:36 |
| patchbot | patch 525870 - ironic-specs - Expose supported power states | 16:36 |
| *** yangyapeng has joined #openstack-ironic | 16:37 | |
| * milan_ is away: I'm busy | 16:37 | |
| *** absubram has joined #openstack-ironic | 16:38 | |
| *** e0ne has quit IRC | 16:38 | |
| *** yangyapeng has quit IRC | 16:41 | |
| *** absubram has quit IRC | 16:43 | |
| *** milan_ has quit IRC | 16:43 | |
| *** amarao has quit IRC | 16:44 | |
| TheJulia | rloo: thanks for the note | 16:46 |
| TheJulia | fellypefca: sure, give me a little bit and I'll take a look | 16:46 |
| rloo | TheJulia: :) | 16:46 |
| *** hoonetorg has joined #openstack-ironic | 16:48 | |
| *** absubram has joined #openstack-ironic | 16:50 | |
| *** jkilpatr has joined #openstack-ironic | 16:52 | |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Add detail=[True, False] query string to API list endpoints https://review.openstack.org/431497 | 16:56 |
| jlvillal | hjensas, I just wanted to thank you for taking over the routed network patch :) Even if I am giving it -1 reviews :) | 16:56 |
| hjensas | jlvillal: The reviews are very welcome! :) | 16:57 |
| fellypefca | TheJulia ok :) :) | 16:58 |
| *** yangyapeng has joined #openstack-ironic | 16:58 | |
| jlvillal | sambetts, Regarding patch ^^^ Still need to update doc/source/contributor/webapi-version-history.rst | 16:58 |
| *** dougsz has joined #openstack-ironic | 16:58 | |
| *** derekh has quit IRC | 16:58 | |
| jlvillal | hjensas, :) | 16:58 |
| sambetts | jlvillal: +1 thanks for reminding me | 16:59 |
| jlvillal | sambetts, Not sure if the release mapping was updated either or if it is necessary. I'm not sure. rloo is the expert :) | 16:59 |
| dougsz | Has anyone seen IPA get stuck in the clean-wait state if cleaning is attempted whilst the node is in maintenance mode? | 16:59 |
| sambetts | dougsz: that its the intended behaviour | 16:59 |
| rloo | jlvillal: huh? which patch? | 16:59 |
| jlvillal | rloo, https://review.openstack.org/431497 | 17:00 |
| TheJulia | jlvillal: only when we cut a tag/stable release | 17:00 |
| patchbot | patch 431497 - ironic - Add detail=[True, False] query string to API list ... | 17:00 |
| dougsz | sambetts: ok, thanks | 17:00 |
| jlvillal | TheJulia, rloo: Hironori mentioned that it should be updated. I wasn't sure as I don't think we are updating the objects. But I'm no expert... | 17:00 |
| rloo | jlvillal: only if there is an API bump, RPC bump, object version bump | 17:01 |
| jlvillal | rloo, There is an API bump | 17:01 |
| rloo | jlvillal: then yes | 17:01 |
| jlvillal | rloo, Good to know! :) We probably should update the API update text notes | 17:01 |
| dtantsur | folks, https://review.openstack.org/#/c/527742/ is green. you know what to do, right? ;) | 17:01 |
| patchbot | patch 527742 - ironic-inspector - Use the tempest plugin from openstack/ironic-tempe... | 17:01 |
| rloo | jlvillal: at the time this patch was started, it didn't. but the API thing landed recently | 17:01 |
| TheJulia | hmm, yeah, the version mapping, not release mapping | 17:02 |
| sambetts | dougsz: when a node is in maintenance=True that tells ironic to be hands off the node so it won't do anything automated to the node | 17:02 |
| TheJulia | dtantsur: push the button right? | 17:02 |
| rloo | jlvillal: i thought the docs/stuff was updated. let me know if you see someplace that was missed | 17:02 |
| dtantsur | \o/ | 17:02 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Add detail=[True, False] query string to API list endpoints https://review.openstack.org/431497 | 17:02 |
| jlvillal | rloo, Ah I see it does have that text there already :) https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/versions.py#L115-L119 | 17:02 |
| jlvillal | sambetts, FYI: https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/versions.py#L115-L119 | 17:02 |
| *** yangyapeng has quit IRC | 17:03 | |
| rloo | sambetts should know, i think he updated the mapping for his heartbeat change :) | 17:03 |
| jlvillal | sambetts, I think need to update the common/.../release_mappings.py | 17:03 |
| * rloo has managed to avoid reviewing that patch, phew :) | 17:03 | |
| jlvillal | heh | 17:03 |
| sambetts | rloo: yeah, its just because this patch is a rebase from before those changes where written up so I didn't have a change in that file ] | 17:03 |
| sambetts | jlvillal: ^ | 17:04 |
| *** penick has joined #openstack-ironic | 17:04 | |
| dougsz | sambetts: Make sense - although it does turn the node on. For example, if the node is off, and in maintenance mode, and I trigger cleaning by moving it from 'manage' to 'provide', it boots up IPA and starts cleaning. | 17:04 |
| rloo | sambetts: that's what i figured. we have to get better at getting patches done sooner rather than later | 17:04 |
| * jlvillal grumbles that PC memory so expensive now. Memory he bought in 2016 is now more than twice the price today :( | 17:05 | |
| rloo | jlvillal: huh, isn't memory cheaper now than in 2016 then? | 17:05 |
| sambetts | dougsz: yeah its a little odd :/ what would you expect the behviour to be? block the move from manage to provide? | 17:06 |
| jlvillal | rloo, Nope! I bought 64GB of memory in 2016 for $300. Now the same exact memory model I bought is $800 :( | 17:06 |
| rloo | jlvillal: wow! (I must have misinterpreted your comment above). I'm surprised. | 17:07 |
| jroll | dougsz: sambetts: I feel like the idea there is to give one a way to boot up ipa to troubleshoot a machine, but not totally sure | 17:07 |
| rloo | jlvillal: i just stick with the 1MB of memory from 1970 | 17:07 |
| rloo | j/k | 17:07 |
| jlvillal | rloo, Me too and slightly sad as I was thinking about building a new system. | 17:07 |
| dougsz | sambetts: naively, that would seem more consistent to me. | 17:08 |
| sambetts | jroll,dougsz: you could do that by allowing to move to manual cleaning in maintenance, but not provide to avaiable in maintnance | 17:08 |
| jroll | sure, that's fair | 17:08 |
| rloo | jlvillal: :-( | 17:08 |
| jroll | sambetts: just pointing out what I thought I remember about it :P | 17:09 |
| sambetts | jroll: yeah, I think thats what I rememeber too | 17:09 |
| *** fragatin_ has joined #openstack-ironic | 17:10 | |
| *** ijw has quit IRC | 17:13 | |
| *** rama_y has joined #openstack-ironic | 17:14 | |
| dougsz | jroll, sambetts: Thanks for your thoughts. I guess the question is: bug or feature? | 17:15 |
| *** vsaienk0 has quit IRC | 17:16 | |
| *** alex_xu has quit IRC | 17:17 | |
| jroll | dougsz: I'm honestly not sure. in any case, both are filed as bugs on launchpad :) | 17:18 |
| *** alex_xu has joined #openstack-ironic | 17:18 | |
| *** yangyapeng has joined #openstack-ironic | 17:18 | |
| dougsz | jroll: alright, thanks! | 17:19 |
| *** AlexeyAbashkin has quit IRC | 17:23 | |
| *** yangyapeng has quit IRC | 17:24 | |
| *** baoli has quit IRC | 17:27 | |
| rloo | wow, the more i look at the BIOS config spec, the more questions I have. seems like the devil is in the details. Would be good for others to look cuz it'd be nice to get that moving faster: https://review.openstack.org/#/c/496481/ | 17:28 |
| patchbot | patch 496481 - ironic-specs - Hardware interface for BIOS configuration | 17:28 |
| *** ijw has joined #openstack-ironic | 17:29 | |
| *** marios has quit IRC | 17:30 | |
| *** sasha2 has quit IRC | 17:30 | |
| *** ijw has quit IRC | 17:31 | |
| *** fragatin_ has quit IRC | 17:31 | |
| *** ijw has joined #openstack-ironic | 17:31 | |
| *** sasha2 has joined #openstack-ironic | 17:32 | |
| *** ijw has quit IRC | 17:32 | |
| *** ijw has joined #openstack-ironic | 17:32 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: [WIP][DNM] Experiment with batch through iptables-restore https://review.openstack.org/527943 | 17:33 |
| *** ijw has quit IRC | 17:33 | |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Add detail=[True, False] query string to API list endpoints https://review.openstack.org/431497 | 17:33 |
| *** ijw has joined #openstack-ironic | 17:33 | |
| *** lucasagomes is now known as lucas-afk | 17:35 | |
| *** ijw has quit IRC | 17:38 | |
| *** baoli has joined #openstack-ironic | 17:38 | |
| jlvillal | dtantsur, Any idea why this failed? https://review.openstack.org/527743 | 17:40 |
| patchbot | patch 527743 - ironic-inspector - Remove ironic_inspector/test/inspector_tempest_plu... | 17:40 |
| * dtantsur looks | 17:41 | |
| * jlvillal looks too | 17:41 | |
| dtantsur | Error when trying to get requirement for VCS system Command "git config --get-regexp remote\..*\.url" failed with error code 1 in /opt/stack/new/tempest, falling back to uneditable format,Could not determine repository location of /opt/stack/new/tempest | 17:41 |
| jlvillal | I'm not quite sure what that means: http://logs.openstack.org/43/527743/7/check/ironic-tempest-dsvm-ironic-inspector/50ad42b/job-output.txt.gz#_2017-12-14_15_44_30_494912 | 17:42 |
| jlvillal | Yeah, that | 17:42 |
| dtantsur | and python 3 http://logs.openstack.org/43/527743/7/check/ironic-inspector-tempest-dsvm-python3/811107d/logs/devstacklog.txt.gz#_2017-12-14_15_55_45_934 | 17:43 |
| dtantsur | I think we should recheck | 17:43 |
| *** ijw has joined #openstack-ironic | 17:43 | |
| TheJulia | rloo: I was pondering digging back into that spec, and then saw the comment count yesterday :( | 17:44 |
| *** logan- has quit IRC | 17:44 | |
| jlvillal | dtantsur, Okay. Can try that. | 17:44 |
| rloo | TheJulia: well, it was updated. i am wondering if we should just approve and then deal with the rest at the code level. dunno. | 17:44 |
| *** crushil has joined #openstack-ironic | 17:44 | |
| dtantsur | some things are hard to reason about by looking at text only.. | 17:45 |
| *** logan- has joined #openstack-ironic | 17:45 | |
| jlvillal | dtantsur, +1 | 17:45 |
| rloo | TheJulia: it isn't even main stuff 'apply the bios config changes', it is keeping things in sync when setting/changing the bios interface of a node, which isn't clear to me | 17:45 |
| dtantsur | jlvillal: I think you should -2 https://review.openstack.org/#/c/527733/ until we switch stable branches, wdyt? | 17:47 |
| patchbot | patch 527733 - ironic - Remove ironic_tempest_plugin/ directory | 17:47 |
| dtantsur | or maybe just fix stable branches.. | 17:47 |
| *** ijw has quit IRC | 17:47 | |
| jlvillal | Sure | 17:47 |
| jlvillal | dtantsur, Both | 17:48 |
| jlvillal | dtantsur, -2 applied | 17:48 |
| dtantsur | jlvillal: do you know if IPA requires any updates? | 17:52 |
| jlvillal | dtantsur, I do not know | 17:52 |
| dtantsur | and other projects that reuse jobs from ironic and inspector? | 17:52 |
| jlvillal | dtantsur, Yes IPA needs changes | 17:53 |
| jlvillal | And probably ironic-lib | 17:53 |
| jlvillal | I'll start working on it. | 17:53 |
| *** tesseract has quit IRC | 17:53 | |
| jlvillal | dtantsur, Basically just updates to the playbooks | 17:54 |
| dtantsur | and ironicclient.. probably all projects in the end | 17:54 |
| *** rmcall has joined #openstack-ironic | 17:54 | |
| TheJulia | dtantsur: jlvillal: is there an etherpad someplace where this is being tracked? I've not seen a common topic | 17:55 |
| jlvillal | TheJulia, There is | 17:55 |
| dtantsur | TheJulia: https://etherpad.openstack.org/p/ironic-tempest-plugin-migration | 17:55 |
| TheJulia | awesome, thanks! | 17:55 |
| *** rmcall has quit IRC | 17:56 | |
| *** catintheroof has joined #openstack-ironic | 17:57 | |
| *** dougsz has quit IRC | 18:00 | |
| dtantsur | calling it a day, and leaving it up to you folks! please do keep the etherpad updated for me to take over in the morning | 18:01 |
| jlvillal | dtantsur, Thanks! | 18:01 |
| *** dtantsur is now known as dtantsur|afk | 18:01 | |
| NobodyCam | Good Morning Ironic'ers | 18:03 |
| TheJulia | good morning NobodyCam | 18:03 |
| jlvillal | Hi NobodyCam :) | 18:04 |
| NobodyCam | :) heheheh :) | 18:04 |
| NobodyCam | morning TheJulia and jlvillal :) | 18:04 |
| *** mgoddard has quit IRC | 18:05 | |
| *** penick has quit IRC | 18:06 | |
| fellypefca | hey NobodyCam :) | 18:07 |
| NobodyCam | hey hey Good Morning fellypefca :) | 18:07 |
| *** penick has joined #openstack-ironic | 18:08 | |
| *** absubram has quit IRC | 18:08 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528024 | 18:10 |
| jlvillal | TheJulia, Just for you I did: git push origin HEAD:refs/publish/master/tempest-plugin :) | 18:11 |
| jlvillal | TheJulia, So it has a topic | 18:11 |
| * jlvillal foolishly doesn't use git-review... | 18:11 | |
| jlvillal | And I decided to change it to "ironic-tempest-plugin" | 18:12 |
| *** trown|outtypewww is now known as trown | 18:13 | |
| *** baoli has quit IRC | 18:14 | |
| *** rmcall has joined #openstack-ironic | 18:15 | |
| *** baoli has joined #openstack-ironic | 18:16 | |
| TheJulia | jlvillal: but git-review makes it so easy :) | 18:17 |
| jlvillal | TheJulia, You don't think: git push origin HEAD:refs/publish/master/ironic-tempest-plugin is easy?? | 18:17 |
| * jlvillal is incredulous! | 18:18 | |
| jlvillal | ;) | 18:18 |
| TheJulia | eh, I was more speaking as a whole it makes it easy | 18:18 |
| *** ijw has joined #openstack-ironic | 18:18 | |
| openstackgerrit | Merged openstack/ironic-lib master: Fix issue with double mocking of utils.execute functions https://review.openstack.org/524329 | 18:21 |
| *** etingof has quit IRC | 18:22 | |
| * sambetts --> home | 18:23 | |
| sambetts | night all | 18:23 |
| TheJulia | goodnight | 18:23 |
| *** sambetts is now known as sambetts|afk | 18:23 | |
| jlvillal | sambetts, Good night | 18:23 |
| sambetts|afk | o/ | 18:23 |
| *** ijw has quit IRC | 18:24 | |
| *** trown is now known as trown|ruck | 18:24 | |
| *** harlowja has joined #openstack-ironic | 18:29 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528033 | 18:30 |
| openstackgerrit | Merged openstack/bifrost stable/ocata: Clean up zuul files https://review.openstack.org/525300 | 18:31 |
| *** ijw has joined #openstack-ironic | 18:31 | |
| *** baha has joined #openstack-ironic | 18:31 | |
| openstackgerrit | John L. Villalovos proposed openstack/python-ironicclient master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528034 | 18:33 |
| *** rama_y has quit IRC | 18:34 | |
| *** ijw has quit IRC | 18:35 | |
| *** etingof has joined #openstack-ironic | 18:36 | |
| *** ijw has joined #openstack-ironic | 18:36 | |
| *** crushil has quit IRC | 18:36 | |
| *** ijw has quit IRC | 18:36 | |
| *** crushil has joined #openstack-ironic | 18:37 | |
| *** ijw has joined #openstack-ironic | 18:37 | |
| *** ijw has quit IRC | 18:37 | |
| *** ijw has joined #openstack-ironic | 18:38 | |
| *** ijw has quit IRC | 18:41 | |
| *** ijw has joined #openstack-ironic | 18:44 | |
| *** penick has quit IRC | 18:44 | |
| *** mgoddard has joined #openstack-ironic | 18:45 | |
| *** penick has joined #openstack-ironic | 18:45 | |
| jlvillal | TheJulia, rloo: Stable backport patch for your review: https://review.openstack.org/#/c/527993/ For ironic-tempest-plugin work. Thanks | 18:45 |
| patchbot | patch 527993 - ironic-inspector (stable/pike) - devstack: set [service_available]ironic-inspector ... | 18:45 |
| *** ijw has quit IRC | 18:46 | |
| *** ijw has joined #openstack-ironic | 18:46 | |
| TheJulia | +2'ed | 18:46 |
| jlvillal | Thanks | 18:46 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: boot mode support at ManagementInterface https://review.openstack.org/526773 | 18:47 |
| *** ijw has quit IRC | 18:47 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:47 | |
| *** ijw has joined #openstack-ironic | 18:48 | |
| *** AlexeyAbashkin has quit IRC | 18:52 | |
| *** ijw has quit IRC | 18:53 | |
| TheJulia | rloo: wrt that stable backport, dmitry did backport it, but he also created it to begin with, so I think we nee 2x+2 | 18:56 |
| TheJulia | last chance to review https://review.openstack.org/#/c/525870/ :) 3x+2 | 19:00 |
| patchbot | patch 525870 - ironic-specs - Expose supported power states | 19:00 |
| * jlvillal trusts those three +2s | 19:00 | |
| fellypefca | Anyone with this issue running devstack? "No matching distribution found for grpcio===1.8.0" | 19:02 |
| *** catintheroof has quit IRC | 19:06 | |
| TheJulia | looks like 1.8.0 has been cut but is not on pypi | 19:06 |
| TheJulia | 2 days ago | 19:07 |
| *** ijw has joined #openstack-ironic | 19:07 | |
| *** ijw has quit IRC | 19:08 | |
| fellypefca | yeah :/ already talk to people from openstack-requirements | 19:08 |
| rpioso | Would adding UEFI support to the iDRAC driver require a Launchpad bug, spec, and/or RFE? | 19:08 |
| *** ijw has joined #openstack-ironic | 19:08 | |
| *** fellypefca is now known as fellypefca-afk | 19:09 | |
| TheJulia | rpioso: just an rfe if it conforms with what has been previously done | 19:09 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Report /dev/disk/by-path on inspection followup https://review.openstack.org/507858 | 19:09 |
| openstackgerrit | Merged openstack/ironic-python-agent stable/ocata: Clean-up zuul.d/ files https://review.openstack.org/525311 | 19:09 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: boot mode support at ManagementInterface https://review.openstack.org/526773 | 19:09 |
| rpioso | TheJulia: Cool :) | 19:10 |
| TheJulia | rpioso: also, basing upon etingof's patch above might be a good idea ;) | 19:11 |
| *** adisky__ has quit IRC | 19:11 | |
| *** ijw has quit IRC | 19:12 | |
| *** ijw has joined #openstack-ironic | 19:12 | |
| rpioso | TheJulia: This one -- https://review.openstack.org/#/c/526773/ ? | 19:14 |
| patchbot | patch 526773 - ironic - boot mode support at ManagementInterface | 19:14 |
| TheJulia | yeah, in an effort to move the calls out to the management interface | 19:15 |
| TheJulia | ugh, I completely forgot about bfv meeting today | 19:15 |
| *** openstack has joined #openstack-ironic | 20:32 | |
| *** ChanServ sets mode: +o openstack | 20:32 | |
| *** ijw has joined #openstack-ironic | 20:32 | |
| rloo | jlvillal: ok. i guess we normally have it? | 20:32 |
| jlvillal | rloo, Yes. Almost all of our text files have an end of line character | 20:33 |
| rloo | jlvillal: odd that most run.yaml have it but this project doesn't. | 20:33 |
| rloo | jlvillal: oh well, now it does :) | 20:33 |
| jlvillal | rloo, Yeah I'm not sure why these don't | 20:33 |
| rloo | jlvillal: a mystery that will go unsolved... | 20:34 |
| jlvillal | rloo, Probably vdrok's fault. He must use a strange editor :) | 20:34 |
| *** yangyapeng has quit IRC | 20:34 | |
| jlvillal | git blame says he is the culprit ;) | 20:34 |
| rloo | jlvillal: heh | 20:34 |
| *** ijw has quit IRC | 20:36 | |
| jlvillal | I think vdrok must be missing the weather at home. 7C and rain there versus 21C and sunny where he is now. I feel sorry for him ;) | 20:38 |
| *** ijw has joined #openstack-ironic | 20:38 | |
| rloo | yeah, poor vdrok. But he'll make the best out of it, I'm sure :D | 20:38 |
| jlvillal | heh | 20:38 |
| rloo | jlvillal: are we backporting this tempest stuff? | 20:38 |
| jlvillal | rloo, Yes. I am going to work on that next | 20:39 |
| jlvillal | rloo, We need to backport it before we can remove it | 20:39 |
| rloo | jlvillal: the list of projects here might help: https://etherpad.openstack.org/p/ironic-zuulv3-intree-tracking | 20:39 |
| rloo | jlvillal: look at the bottom, the non-ironic ones | 20:39 |
| jlvillal | rloo, Thanks | 20:40 |
| rloo | jlvillal: i'm assuming they use our tempest stuff too | 20:40 |
| jlvillal | rloo, Probably. I'm working my way through that list. I added what I have done so far to it. | 20:41 |
| jlvillal | rloo, Sorry I was thinking of: https://etherpad.openstack.org/p/ironic-tempest-plugin-migration | 20:41 |
| rloo | jlvillal: it is like deja vu. third time i am staring at a list of ironic projects etc | 20:41 |
| jlvillal | You have to laugh or you'd cry :) | 20:42 |
| *** ijw has quit IRC | 20:42 | |
| * rloo cries | 20:42 | |
| *** e0ne has quit IRC | 20:43 | |
| *** e0ne has joined #openstack-ironic | 20:44 | |
| *** e0ne has quit IRC | 20:45 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 20:46 | |
| *** yangyapeng has joined #openstack-ironic | 20:50 | |
| *** AlexeyAbashkin has quit IRC | 20:51 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528080 | 20:52 |
| *** rmcall has quit IRC | 20:54 | |
| *** yangyapeng has quit IRC | 20:55 | |
| *** dprince has quit IRC | 21:02 | |
| *** crushil has quit IRC | 21:05 | |
| vdrok | jlvillal: hah, indeed iirc it was vim | 21:09 |
| vdrok | good morning rloo jlvillal and all! | 21:09 |
| jlvillal | vdrok, My vim automatically adds an EOL character. Might be something in my vimconfig | 21:09 |
| rloo | morning vdrok! | 21:09 |
| *** openstack has joined #openstack-ironic | 21:12 | |
| *** ChanServ sets mode: +o openstack | 21:12 | |
| jlvillal | vdrok, vim-fugitive, YouCompleteMe, vim-python-pep8-indent, fzf, vim-gitgutter, and powerline | 21:12 |
| jlvillal | A couple more too. But nothing that interesting | 21:12 |
| jlvillal | I use Vundle to manage my plugins | 21:13 |
| vdrok | aha, was just going to ask this :) | 21:13 |
| *** rpioso|brb is now known as rpioso | 21:13 | |
| *** athomas has quit IRC | 21:14 | |
| TheJulia | jlvillal: it is easy to not have an \n :) | 21:14 |
| *** yangyapeng has quit IRC | 21:15 | |
| jlvillal | TheJulia, :) | 21:16 |
| rpioso | TheJulia: I caught up on the back scroll. Thank you. It seems reasonable to consolidate it into the ManagementInterface. If not during the UEFI effort, then later :) | 21:18 |
| TheJulia | rpioso: awesome! :) | 21:19 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-inspector stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528088 | 21:19 |
| TheJulia | rpioso: implementers choice :) | 21:23 |
| openstackgerrit | Merged openstack/python-ironicclient master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528034 | 21:25 |
| TheJulia | hmmm.... http://logs.ironic-oneview-ci.lsd.ufcg.edu.br/80/528080/1/check/dsvm-tempest-ironic-agent_pxe_oneview/e04e2f9/logs/devstacklog.txt.gz#_2017-12-14_21_14_08_582 | 21:25 |
| openstackgerrit | John L. Villalovos proposed openstack/sushy master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528090 | 21:27 |
| TheJulia | mjturek: krtaylor: eek https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/ironic/80/528080/1/check-ironic/tempest-dsvm-ironic-agent_ipmitool/c9d7426/devstacklog.txt.gz :( | 21:28 |
| mjturek | uh oooooh | 21:28 |
| * TheJulia will remember this moment forever | 21:29 | |
| TheJulia | :) | 21:29 |
| *** yangyapeng has joined #openstack-ironic | 21:30 | |
| openstackgerrit | John L. Villalovos proposed openstack/python-ironic-inspector-client master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528091 | 21:30 |
| mjturek | heheh, thanks for the heads up TheJulia, I'll see what we can do | 21:31 |
| *** threestrands has joined #openstack-ironic | 21:31 | |
| TheJulia | looks like grpcio is impacting dell's ci :\ | 21:32 |
| *** yangyapeng has quit IRC | 21:35 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Add DSVM to the gate for ironic-tempest-plugin https://review.openstack.org/528070 | 21:35 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Add DSVM jobs to the gate for ironic-tempest-plugin https://review.openstack.org/528070 | 21:35 |
| jlvillal | TheJulia, vdrok rloo: ^^^ if you want to add that to your review queue | 21:37 |
| TheJulia | and fujitsu's ci also doesn't seem happy either :( | 21:37 |
| *** chihhsin_ has quit IRC | 21:39 | |
| rajinir | TheJulia: Just heard Dell Ci mentioned. | 21:39 |
| *** Goneri has quit IRC | 21:39 | |
| mjturek | we were having problems with grpcio before too. For now we have a workaround in our pipeline to add the package to switch to 1.8.0rc2 | 21:40 |
| rloo | jlvillal: which ones have passed CI? | 21:40 |
| jlvillal | rloo, Let me investigate... | 21:41 |
| TheJulia | rajinir: https://stash.dellemc-community.org/logs/33/527733/11/check/dell-hw13G-tempest-dsvm-ironic-idrac/a6ecea2/console.html looks like a $PATH issue there | 21:41 |
| jlvillal | rloo, At the moment nothing. I will update you when we have some through the CI and passing. | 21:42 |
| rloo | jlvillal: ok, thx | 21:42 |
| TheJulia | I would expect third party ci to break potentially once the in-tree tempest plugin is removed | 21:43 |
| rloo | jlvillal: i'm around for another hour or so, which is why i don't want to add my name to those patches. hopefully they will be merged by next Monday :) | 21:43 |
| jlvillal | rloo, No worries. Enjoy your weekend :) | 21:43 |
| rloo | TheJulia: oh yeah, good point... | 21:43 |
| jlvillal | rloo, I can try to force vdrok to do it ;) | 21:43 |
| rloo | jlvillal: yup, vdrok is a nice guy :) | 21:43 |
| vdrok | jlvillal: do we want grenade on plugin? | 21:43 |
| jlvillal | vdrok, I'm not sure. When discussing it with dtantsur|afk today he said. /me scrolls back to find it | 21:44 |
| jlvillal | jlvillal: we need 1. one full job from ironic (multinode?), 2. one standalone job from ironic, 3. the default inspector job (the same that runs on ironic) | 21:45 |
| jlvillal | <dtantsur> this is for the start, then we can think about adding other jobs | 21:45 |
| rajinir | TheJulia>: looks like the Ci's shoud fix the PATH | 21:45 |
| jlvillal | vdrok, ^^ | 21:45 |
| vdrok | maybe instead of just multinode, we could do multinode grenade | 21:45 |
| *** chihhsin has joined #openstack-ironic | 21:45 | |
| * jlvillal shrugs and is willing to do either | 21:46 | |
| jlvillal | vdrok, I can change it :) | 21:46 |
| vdrok | because we need to ensure that changes to plugin are passing for all releases? this way, we will at least know previous one is not broken | 21:46 |
| jlvillal | vdrok, Okay. I will use: ironic-grenade-dsvm-multinode-multitenant | 21:47 |
| vdrok | thank you! | 21:47 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Add DSVM jobs to the gate for ironic-tempest-plugin https://review.openstack.org/528070 | 21:47 |
| jlvillal | vdrok, ^^^ :) Thanks for the help | 21:48 |
| *** penick has quit IRC | 21:48 | |
| *** mgoddard has quit IRC | 21:48 | |
| *** yangyapeng has joined #openstack-ironic | 21:52 | |
| openstackgerrit | Merged openstack/ironic-inspector stable/pike: devstack: set [service_available]ironic-inspector = True https://review.openstack.org/527993 | 21:57 |
| *** yangyapeng has quit IRC | 21:57 | |
| *** rcernin has joined #openstack-ironic | 21:58 | |
| *** crushil has joined #openstack-ironic | 21:59 | |
| *** trown|ruck is now known as trown|outtypewww | 22:00 | |
| *** crushil_ has joined #openstack-ironic | 22:00 | |
| openstackgerrit | Merged openstack/ironic-specs master: Expose supported power states https://review.openstack.org/525870 | 22:03 |
| rloo | jlvillal: by the way, i believe that inspector stable/ocata CI is still failing | 22:03 |
| rloo | jlvillal: guess i'll update the whiteboard | 22:03 |
| jlvillal | rloo, yeah :( And thanks. | 22:03 |
| *** penick has joined #openstack-ironic | 22:04 | |
| *** crushil has quit IRC | 22:04 | |
| rloo | jlvillal: i was going to say that if the tempest migration causes the same errors, the change is ok, but you can't merge it if CI fails. unless you change them to non-voting for a patch :) | 22:07 |
| *** bfournie has quit IRC | 22:09 | |
| jlvillal | rloo, stable/ocata is broken unrelated to the tempest code. | 22:09 |
| jlvillal | rloo, dtantsur|afk I think is looking into it. | 22:09 |
| rloo | jlvillal: exactly | 22:09 |
| jlvillal | rloo, dtantsur|afk pointed out in https://etherpad.openstack.org/p/ironic-tempest-plugin-migration on line 40 that it was broken. | 22:10 |
| jlvillal | rloo, So he is aware of it. I think most of the inspector people are. I myself have not investigated the issue. | 22:10 |
| *** absubram has quit IRC | 22:10 | |
| jlvillal | Basically I'm waiting for them to fix it before we can merge the patch | 22:11 |
| *** baha has quit IRC | 22:11 | |
| rloo | jlvillal: ok, i didn't look at that etherpad, well i did, but i skimmed it :) wanted to make sure you were aware | 22:11 |
| jlvillal | rloo, Thanks, I appreciate it | 22:11 |
| *** ijw has joined #openstack-ironic | 22:18 | |
| *** yangyapeng has joined #openstack-ironic | 22:19 | |
| *** jcoufal_ has quit IRC | 22:20 | |
| jlvillal | rloo, This has passed: https://review.openstack.org/528090 | 22:21 |
| patchbot | patch 528090 - sushy - Use the tempest plugin from openstack/ironic-tempe... | 22:21 |
| rloo | jlvillal: looking... | 22:21 |
| jlvillal | Thanks | 22:22 |
| rloo | vdrok, TheJulia ^^ ready for quick review | 22:22 |
| *** ijw has quit IRC | 22:23 | |
| *** yangyapeng has quit IRC | 22:24 | |
| vdrok | that was too quick :) | 22:24 |
| TheJulia | heh | 22:24 |
| jlvillal | vdrok, Well they've seen that change diff pattern before... :) | 22:24 |
| TheJulia | exactly | 22:25 |
| *** rwsu has quit IRC | 22:26 | |
| rloo | vdrok: you must be out of practise; you need to be reviewing more :D | 22:27 |
| vdrok | yeah, shame on me | 22:27 |
| rloo | :) | 22:27 |
| *** ijw has joined #openstack-ironic | 22:28 | |
| *** ijw has quit IRC | 22:30 | |
| *** ijw has joined #openstack-ironic | 22:30 | |
| *** e0ne has joined #openstack-ironic | 22:33 | |
| *** ijw has quit IRC | 22:34 | |
| rpioso | TheJulia: Thank you! | 22:34 |
| * rpioso is pro developer choice :) | 22:34 | |
| TheJulia | :) | 22:34 |
| *** yangyapeng has joined #openstack-ironic | 22:37 | |
| jlvillal | rloo, TheJulia vdrok: Another one just went Zuul +1 https://review.openstack.org/528091 | 22:38 |
| patchbot | patch 528091 - python-ironic-inspector-client - Use the tempest plugin from openstack/ironic-tempe... | 22:38 |
| jlvillal | And the stable branch: https://review.openstack.org/528080 | 22:39 |
| patchbot | patch 528080 - ironic (stable/pike) - Use the tempest plugin from openstack/ironic-tempe... | 22:39 |
| *** rwsu has joined #openstack-ironic | 22:39 | |
| rloo | vdrok is faster, practise helps :) | 22:40 |
| openstackgerrit | Merged openstack/ironic-lib master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528033 | 22:41 |
| jlvillal | heh | 22:41 |
| vdrok | \o/ | 22:41 |
| *** ijw has joined #openstack-ironic | 22:41 | |
| jlvillal | Oooh. I can do a stable backport now... | 22:41 |
| *** yangyapeng has quit IRC | 22:42 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Do not try unmounting the EFI partition if it was not mounted https://review.openstack.org/521114 | 22:43 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528109 | 22:44 |
| *** ijw_ has joined #openstack-ironic | 22:44 | |
| *** rloo has quit IRC | 22:45 | |
| *** ijw has quit IRC | 22:45 | |
| *** ijw_ has quit IRC | 22:46 | |
| *** ijw has joined #openstack-ironic | 22:46 | |
| *** ijw has quit IRC | 22:47 | |
| *** ijw has joined #openstack-ironic | 22:47 | |
| *** ijw has quit IRC | 22:48 | |
| *** bfournie has joined #openstack-ironic | 22:49 | |
| *** bfournie has quit IRC | 22:50 | |
| *** ijw has joined #openstack-ironic | 22:50 | |
| *** rloo has joined #openstack-ironic | 22:52 | |
| *** rloo has quit IRC | 22:54 | |
| *** ijw has quit IRC | 22:55 | |
| *** absubram has joined #openstack-ironic | 22:57 | |
| *** baoli has quit IRC | 22:57 | |
| openstackgerrit | John L. Villalovos proposed openstack/python-ironicclient stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528116 | 22:58 |
| *** yangyapeng has joined #openstack-ironic | 22:59 | |
| *** e0ne has quit IRC | 22:59 | |
| *** ijw has joined #openstack-ironic | 22:59 | |
| *** ijw has quit IRC | 23:00 | |
| *** ijw has joined #openstack-ironic | 23:00 | |
| *** ijw has quit IRC | 23:01 | |
| *** ijw_ has joined #openstack-ironic | 23:01 | |
| *** crushil_ has quit IRC | 23:01 | |
| *** ijw_ has quit IRC | 23:02 | |
| *** ijw has joined #openstack-ironic | 23:02 | |
| openstackgerrit | Merged openstack/sushy master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528090 | 23:03 |
| *** ijw has quit IRC | 23:03 | |
| *** ijw has joined #openstack-ironic | 23:03 | |
| *** yangyapeng has quit IRC | 23:04 | |
| openstackgerrit | John L. Villalovos proposed openstack/sushy stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528121 | 23:05 |
| *** ijw has quit IRC | 23:08 | |
| *** ijw has joined #openstack-ironic | 23:08 | |
| *** threestrands_ has joined #openstack-ironic | 23:10 | |
| *** ijw has quit IRC | 23:13 | |
| *** threestrands has quit IRC | 23:13 | |
| *** masuberu has quit IRC | 23:15 | |
| *** Sukhdev has joined #openstack-ironic | 23:16 | |
| *** weshay is now known as weshay_pto | 23:17 | |
| *** ijw has joined #openstack-ironic | 23:17 | |
| *** ijw has quit IRC | 23:18 | |
| *** ijw has joined #openstack-ironic | 23:18 | |
| *** ijw has quit IRC | 23:19 | |
| *** yangyapeng has joined #openstack-ironic | 23:19 | |
| *** ijw has joined #openstack-ironic | 23:19 | |
| *** ijw has quit IRC | 23:20 | |
| *** ijw has joined #openstack-ironic | 23:20 | |
| *** ijw has quit IRC | 23:21 | |
| *** ijw_ has joined #openstack-ironic | 23:21 | |
| *** ijw has joined #openstack-ironic | 23:22 | |
| *** ijw has quit IRC | 23:23 | |
| *** ijw_ has quit IRC | 23:23 | |
| *** baoli has joined #openstack-ironic | 23:23 | |
| *** yangyapeng has quit IRC | 23:24 | |
| *** ijw has joined #openstack-ironic | 23:25 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528091 | 23:26 |
| *** ijw has quit IRC | 23:26 | |
| openstackgerrit | John L. Villalovos proposed openstack/python-ironic-inspector-client stable/pike: Use the tempest plugin from openstack/ironic-tempest-plugin https://review.openstack.org/528126 | 23:26 |
| *** ijw has joined #openstack-ironic | 23:27 | |
| *** baoli has quit IRC | 23:28 | |
| *** ijw has quit IRC | 23:28 | |
| *** ijw_ has joined #openstack-ironic | 23:28 | |
| *** ijw_ has quit IRC | 23:29 | |
| *** ijw_ has joined #openstack-ironic | 23:31 | |
| *** harlowja has quit IRC | 23:32 | |
| *** ijw_ has quit IRC | 23:32 | |
| *** ijw has joined #openstack-ironic | 23:32 | |
| *** ijw has quit IRC | 23:33 | |
| *** ijw has joined #openstack-ironic | 23:34 | |
| *** ijw_ has joined #openstack-ironic | 23:35 | |
| *** ijw has quit IRC | 23:35 | |
| *** ijw_ has quit IRC | 23:36 | |
| *** ijw has joined #openstack-ironic | 23:36 | |
| *** ijw has quit IRC | 23:37 | |
| *** harlowja has joined #openstack-ironic | 23:39 | |
| *** yangyapeng has joined #openstack-ironic | 23:40 | |
| *** ijw has joined #openstack-ironic | 23:40 | |
| *** harlowja has quit IRC | 23:40 | |
| *** jkilpatr has quit IRC | 23:40 | |
| *** ijw has quit IRC | 23:41 | |
| *** ijw has joined #openstack-ironic | 23:41 | |
| *** harlowja has joined #openstack-ironic | 23:42 | |
| *** ijw has quit IRC | 23:42 | |
| *** ijw has joined #openstack-ironic | 23:42 | |
| *** ijw has quit IRC | 23:44 | |
| *** ijw_ has joined #openstack-ironic | 23:44 | |
| *** yangyapeng has quit IRC | 23:44 | |
| *** ijw_ has quit IRC | 23:45 | |
| *** ijw has joined #openstack-ironic | 23:45 | |
| *** ijw_ has joined #openstack-ironic | 23:48 | |
| *** ijw has quit IRC | 23:48 | |
| *** mjturek has quit IRC | 23:48 | |
| *** ijw_ has quit IRC | 23:48 | |
| *** ijw has joined #openstack-ironic | 23:49 | |
| *** ijw has quit IRC | 23:50 | |
| *** ijw has joined #openstack-ironic | 23:50 | |
| *** harlowja has quit IRC | 23:50 | |
| *** lennyb has quit IRC | 23:51 | |
| *** ijw_ has joined #openstack-ironic | 23:51 | |
| *** ijw_ has quit IRC | 23:53 | |
| *** ijw has quit IRC | 23:53 | |
| *** ijw_ has joined #openstack-ironic | 23:53 | |
| *** ijw_ has quit IRC | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!