*** sayantani01 has quit IRC | 00:00 | |
*** sdake has quit IRC | 00:01 | |
*** srart has quit IRC | 00:03 | |
*** aNuposic has quit IRC | 00:08 | |
*** baoli has quit IRC | 00:08 | |
*** srart has joined #openstack-ironic | 00:13 | |
*** aNuposic has joined #openstack-ironic | 00:14 | |
*** vinsh has quit IRC | 00:18 | |
*** dsneddon has quit IRC | 00:23 | |
*** Sukhdev has quit IRC | 00:31 | |
*** bfournie has joined #openstack-ironic | 00:31 | |
*** Sukhdev has joined #openstack-ironic | 00:35 | |
*** sayantani01 has joined #openstack-ironic | 00:43 | |
*** rloo has quit IRC | 00:43 | |
*** vsaienko has quit IRC | 00:46 | |
*** trown is now known as trown|outtypewww | 00:47 | |
*** tovin07_ has joined #openstack-ironic | 00:49 | |
*** krtaylor has joined #openstack-ironic | 00:51 | |
openstackgerrit | OpenStack Release Bot proposed openstack/python-ironicclient: Update reno for stable/ocata https://review.openstack.org/425477 | 00:53 |
---|---|---|
*** phuongnh has joined #openstack-ironic | 00:54 | |
openstackgerrit | OpenStack Release Bot proposed openstack/python-ironic-inspector-client: Update reno for stable/ocata https://review.openstack.org/425480 | 00:56 |
*** rama_y has quit IRC | 00:56 | |
*** sacharya has joined #openstack-ironic | 01:08 | |
*** [1]rpioso has quit IRC | 01:20 | |
TheJulia | Aww stable/ocata | 01:23 |
*** baoli has joined #openstack-ironic | 01:28 | |
*** baoli has quit IRC | 01:32 | |
*** sdake has joined #openstack-ironic | 01:33 | |
*** sdake_ has joined #openstack-ironic | 01:35 | |
*** sdake has quit IRC | 01:38 | |
openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 01:39 |
*** sdake_ has quit IRC | 01:43 | |
*** rajinir has quit IRC | 01:46 | |
*** sdake has joined #openstack-ironic | 01:49 | |
*** sdake_ has joined #openstack-ironic | 01:50 | |
*** sdake has quit IRC | 01:54 | |
*** aNuposic has quit IRC | 01:56 | |
*** sacharya_ has joined #openstack-ironic | 02:07 | |
*** sacharya has quit IRC | 02:08 | |
*** sdake_ has quit IRC | 02:09 | |
*** sdake has joined #openstack-ironic | 02:11 | |
*** Sukhdev has quit IRC | 02:14 | |
*** noama has quit IRC | 02:20 | |
*** dansmith has quit IRC | 02:20 | |
*** noama has joined #openstack-ironic | 02:20 | |
*** dansmith has joined #openstack-ironic | 02:21 | |
*** mtanino has quit IRC | 02:23 | |
*** mitchjameson has quit IRC | 02:37 | |
*** Syed__ has quit IRC | 02:45 | |
*** sacharya_ has quit IRC | 02:50 | |
*** sacharya has joined #openstack-ironic | 02:50 | |
*** sayantani01 has quit IRC | 02:50 | |
*** baoli has joined #openstack-ironic | 02:56 | |
*** sdake has quit IRC | 02:58 | |
*** d0ugal has quit IRC | 03:00 | |
*** baoli has quit IRC | 03:10 | |
*** rama_y has joined #openstack-ironic | 03:15 | |
*** d0ugal has joined #openstack-ironic | 03:17 | |
*** newmember has quit IRC | 03:23 | |
*** rushil has quit IRC | 03:23 | |
*** rbudden has quit IRC | 03:27 | |
*** vikrant has joined #openstack-ironic | 03:30 | |
*** ijw has quit IRC | 03:35 | |
*** ijw has joined #openstack-ironic | 03:35 | |
*** ijw has quit IRC | 03:35 | |
*** rama_y has quit IRC | 03:39 | |
*** baoli has joined #openstack-ironic | 03:39 | |
*** rushil has joined #openstack-ironic | 03:41 | |
*** sdake has joined #openstack-ironic | 03:46 | |
*** Sukhdev has joined #openstack-ironic | 03:48 | |
*** gcb has joined #openstack-ironic | 03:50 | |
*** sasha2 has quit IRC | 04:07 | |
*** sdake_ has joined #openstack-ironic | 04:08 | |
*** sdake has quit IRC | 04:09 | |
*** baoli has quit IRC | 04:20 | |
*** sdake has joined #openstack-ironic | 04:31 | |
*** sdake_ has quit IRC | 04:32 | |
*** ianw is now known as ianw_pto | 04:34 | |
*** sdake has quit IRC | 04:49 | |
*** sayantani01 has joined #openstack-ironic | 04:57 | |
*** Sukhdev has quit IRC | 04:58 | |
*** Guest58531 has quit IRC | 05:04 | |
*** comstud has quit IRC | 05:07 | |
*** mgagne has joined #openstack-ironic | 05:07 | |
*** mgagne is now known as Guest33539 | 05:07 | |
*** comstud has joined #openstack-ironic | 05:08 | |
*** Sukhdev has joined #openstack-ironic | 05:09 | |
*** sacharya has quit IRC | 05:14 | |
*** moshele has joined #openstack-ironic | 05:14 | |
*** sacharya has joined #openstack-ironic | 05:14 | |
*** sacharya has quit IRC | 05:15 | |
*** sacharya has joined #openstack-ironic | 05:15 | |
*** jtomasek has quit IRC | 05:18 | |
*** sacharya has quit IRC | 05:20 | |
*** sdake has joined #openstack-ironic | 05:25 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Update iRMC driver doc for soft reboot and soft power off https://review.openstack.org/425541 | 05:41 |
*** sdake has quit IRC | 05:45 | |
*** Sukhdev has quit IRC | 05:49 | |
*** rushil has quit IRC | 06:04 | |
*** sdake has joined #openstack-ironic | 06:06 | |
*** baoli has joined #openstack-ironic | 06:10 | |
*** rbudden has joined #openstack-ironic | 06:10 | |
*** baoli has quit IRC | 06:15 | |
*** sacharya has joined #openstack-ironic | 06:16 | |
*** sdake has quit IRC | 06:16 | |
*** sdake has joined #openstack-ironic | 06:17 | |
*** vsaienko has joined #openstack-ironic | 06:19 | |
*** sdake_ has joined #openstack-ironic | 06:21 | |
*** sacharya has quit IRC | 06:22 | |
*** sdake has quit IRC | 06:23 | |
*** sdake_ has quit IRC | 06:27 | |
openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support https://review.openstack.org/264257 | 06:28 |
*** mjura has joined #openstack-ironic | 06:34 | |
*** e0ne has joined #openstack-ironic | 06:45 | |
openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic-specs: Support additional capabilities discovery for iRMC driver https://review.openstack.org/409044 | 06:46 |
*** e0ne has quit IRC | 06:46 | |
openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic-specs: Support additional capabilities discovery for iRMC driver https://review.openstack.org/409044 | 06:47 |
*** e0ne has joined #openstack-ironic | 06:50 | |
*** e0ne has quit IRC | 06:53 | |
openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient: Add args to CLI 'node-create' for selecting hardware interfaces https://review.openstack.org/418359 | 06:53 |
*** ChubYann has quit IRC | 06:55 | |
*** e0ne has joined #openstack-ironic | 06:57 | |
*** e0ne has quit IRC | 07:00 | |
*** e0ne has joined #openstack-ironic | 07:04 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Update iRMC driver doc for soft reboot and soft power off https://review.openstack.org/425541 | 07:06 |
*** e0ne has quit IRC | 07:09 | |
*** tesseract has joined #openstack-ironic | 07:11 | |
*** e0ne has joined #openstack-ironic | 07:12 | |
*** e0ne has quit IRC | 07:14 | |
*** sayantani01 has quit IRC | 07:15 | |
*** e0ne has joined #openstack-ironic | 07:18 | |
*** AlexeyAbashkin has joined #openstack-ironic | 07:22 | |
*** moshele has quit IRC | 07:23 | |
*** vsaienko has quit IRC | 07:26 | |
*** lennyb has quit IRC | 07:27 | |
*** noama has quit IRC | 07:27 | |
*** noama has joined #openstack-ironic | 07:28 | |
*** lennyb has joined #openstack-ironic | 07:28 | |
*** noama has quit IRC | 07:29 | |
*** vsaienko has joined #openstack-ironic | 07:29 | |
*** vsaienko has quit IRC | 07:38 | |
*** vsaienko has joined #openstack-ironic | 07:46 | |
*** vsaienko has quit IRC | 07:54 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests https://review.openstack.org/423556 | 07:57 |
*** e0ne has quit IRC | 07:59 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: [Devstack] enable only pxe|agent_ipmitool by default https://review.openstack.org/425572 | 08:00 |
*** _fatalis has joined #openstack-ironic | 08:05 | |
*** tovin07_ has quit IRC | 08:05 | |
*** ralonsoh has joined #openstack-ironic | 08:16 | |
openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add support for DBDeadlock handling https://review.openstack.org/394413 | 08:20 |
openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add support for DBDeadlock handling https://review.openstack.org/394413 | 08:21 |
*** rbudden has quit IRC | 08:27 | |
*** phuongnh has quit IRC | 08:27 | |
*** ccamacho has joined #openstack-ironic | 08:27 | |
*** jtomasek has joined #openstack-ironic | 08:50 | |
*** zzzeek has quit IRC | 09:00 | |
*** zzzeek has joined #openstack-ironic | 09:00 | |
*** e0ne has joined #openstack-ironic | 09:01 | |
*** amarao has joined #openstack-ironic | 09:07 | |
openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient: Extends driver-list, driver-show supporting new hardware types https://review.openstack.org/419274 | 09:07 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2 https://review.openstack.org/425068 | 09:08 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Validate port info before assume we may use it https://review.openstack.org/415881 | 09:09 |
*** pester has joined #openstack-ironic | 09:10 | |
*** AlexeyAbashkin has quit IRC | 09:10 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests https://review.openstack.org/423556 | 09:18 |
*** athomas has joined #openstack-ironic | 09:21 | |
*** lucas-afk is now known as lucasagomes | 09:25 | |
lucasagomes | morning all | 09:25 |
sambetts|afk | Morning all | 09:26 |
*** sambetts|afk is now known as sambetts | 09:26 | |
*** amotoki has quit IRC | 09:27 | |
*** e0ne has quit IRC | 09:27 | |
*** mgoddard has joined #openstack-ironic | 09:27 | |
*** amotoki has joined #openstack-ironic | 09:28 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Follow-up iRMC power driver for soft reboot and soft power off https://review.openstack.org/425593 | 09:30 |
*** amotoki has quit IRC | 09:32 | |
lucasagomes | sambetts, o/ | 09:33 |
lucasagomes | the mitaka stable branch tests seems to be completely broken judging by https://review.openstack.org/#/c/422855/ | 09:34 |
patchbot | patch 422855 - ironic-lib (stable/mitaka) - Limit memory usage when running qemu-img | 09:34 |
*** derekh has joined #openstack-ironic | 09:37 | |
*** gcb has quit IRC | 09:37 | |
*** amotoki has joined #openstack-ironic | 09:39 | |
*** e0ne has joined #openstack-ironic | 09:40 | |
openstackgerrit | Sam Betts proposed openstack/ironic-inspector: [WIP] Put logic in place for using new Ironic network simulation https://review.openstack.org/421288 | 09:42 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: [Devstack] Set DEFAULT_IMAGE_NAME variable https://review.openstack.org/425599 | 09:42 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: [Devstack] Set DEFAULT_IMAGE_NAME variable https://review.openstack.org/425599 | 09:43 |
*** gcb has joined #openstack-ironic | 09:50 | |
*** moshele has joined #openstack-ironic | 09:50 | |
*** fxpester has joined #openstack-ironic | 09:54 | |
*** MattMan has quit IRC | 09:55 | |
*** aarefiev_afk is now known as aarefiev | 09:55 | |
aarefiev | morning all! | 09:56 |
*** MattMan has joined #openstack-ironic | 09:56 | |
*** pester has quit IRC | 09:57 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: [Devstack] Set DEFAULT_IMAGE_NAME variable https://review.openstack.org/425599 | 09:58 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: [Devstack] Download both disk and uec images https://review.openstack.org/420692 | 09:58 |
*** noama has joined #openstack-ironic | 09:59 | |
openstackgerrit | Dao Cong Tien proposed openstack/ironic: Add hardware type irmc-s4 for iRMC S4 servers https://review.openstack.org/425604 | 10:04 |
vdrok | good morning ironic, lucasagomes sambetts and aarefiev ! :) | 10:05 |
lucasagomes | sup o/ | 10:05 |
*** hshiina has joined #openstack-ironic | 10:05 | |
vsaienk0 | morning lucasagomes, dtantsur, milan, aarefiev, vdrok, sambetts and all ironicers! | 10:17 |
lucasagomes | o/ | 10:17 |
vdrok | morning vsaienk0 | 10:17 |
aarefiev | morning vdrok, vsaienk0, lucasagomes | 10:18 |
*** amotoki has quit IRC | 10:18 | |
vsaienk0 | sambetts: I found that we can't use nova hotplug NIC tempest tests. Current implementation check for new device after attach, but in ironic case new device will not be added, we just plug that NIC to needed network https://github.com/openstack/tempest/blob/50ca4b2bcd113a838846389f5d66bf7a39babd59/tempest/scenario/test_network_basic_ops.py#L274 | 10:18 |
sambetts | vsaienk0: hmm, I think we should be able to modify that test to work for both cases right? E.g. ensure that after DHCP at least one NIC has the IP address of the new port we attached or something :/ | 10:21 |
sambetts | vsaienk0: otherwise I would suggest that we move the check_new_nic test function into the class, so we only need to override that specific function but the rest of the logic should work right? | 10:22 |
sambetts | vsaienk0: I'n fact looking a couple of lines below we don't even need to use DHCP they assign a static IP for us | 10:23 |
sambetts | vsaienk0: s/I'n/in/ | 10:23 |
vsaienk0 | sambetts: we can't inherit from class that contain tests, it will cause them to run twice | 10:23 |
*** yolanda has quit IRC | 10:23 | |
sambetts | vsaienk0: ah ... of course... | 10:23 |
vsaienk0 | sambetts: and another problem that to run hotplug test we will need to have a custom regex for tests. hotplug is not in the smoke group | 10:24 |
sambetts | vsaienk0: in that case I would suggest that we modify this existing test to work for both use cases | 10:24 |
sambetts | vsaienk0: ultimate goal is that we'll be running the full suite not just smoke | 10:24 |
vsaienk0 | sambetts: I see, but for now we need to run only specific tests, I don't think that we will be able to run full in nearest future | 10:25 |
*** yolanda has joined #openstack-ironic | 10:26 | |
jroll | morning y'all | 10:28 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: [Devstack] Set DEFAULT_IMAGE_NAME variable https://review.openstack.org/425599 | 10:28 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: [Devstack] Download both disk and uec images https://review.openstack.org/420692 | 10:28 |
sambetts | hey jroll | 10:29 |
openstackgerrit | Sam Betts proposed openstack/ironic: [WIP] Don't override device_owner for tenant network ports https://review.openstack.org/424248 | 10:36 |
vdrok | good morning jroll | 10:38 |
lucasagomes | jroll, morning | 10:50 |
*** pester has joined #openstack-ironic | 10:53 | |
openstackgerrit | Oksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV https://review.openstack.org/397126 | 10:56 |
*** dtantsur|afk is now known as dtantsur | 10:57 | |
dtantsur | morning ironic | 10:57 |
lucasagomes | dtantsur, morning | 10:58 |
mat128 | morning everyone | 10:58 |
dtantsur | jroll, mind approving https://review.openstack.org/#/c/425066/ please? | 11:01 |
patchbot | patch 425066 - ironic-inspector (stable/newton) - Make grenade actually upgrade our source code | 11:01 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2 https://review.openstack.org/425068 | 11:03 |
jroll | dtantsur: morning, done | 11:04 |
aarefiev | morning jroll, dtantsur | 11:09 |
aarefiev | dtantsur, sambetts: what features we want to go in O for inspector? | 11:09 |
makowals | Hello everyone | 11:15 |
makowals | Question from beginner about enrolling new node -- I'm constantly getting error "Error: IPMI call failed: power status" for newly added node, even though executing ipmitool manually returns correct result | 11:16 |
makowals | Any tips how to debug the problem? The only idea I have now is there is something wrong with the password which is being provided as "-f" to ipmitool | 11:16 |
*** openstackgerrit has quit IRC | 11:17 | |
vdrok | morning dtantsur and mat128 | 11:18 |
*** vsaienko has joined #openstack-ironic | 11:19 | |
makowals | I'm using agent_ipmitool, but with pxe_ipmitool behaviour is exactly the same | 11:19 |
dtantsur | aarefiev, hey! whatever we can land in a week? :) I'd not start anything new, just maybe finish what is up already, maybe do some small clean ups | 11:20 |
dtantsur | makowals, try looking in ironic-conductor logs. assuming debug logging is enabled, you can find the exact ipmitool command it executes. try it locally, see if there any problems. | 11:20 |
makowals | dtantsur, indeed I did like this and it worked from the console | 11:21 |
makowals | dtantsur, the only difference is conductor provides password as "-f /tmp/tmpnPigQk" | 11:22 |
aarefiev | dtantsur: ok, I thought ff this week, going to review now :) | 11:22 |
dtantsur | there is one thing I'd like to fix due to the move to virtualbmc, stay tuned | 11:23 |
dtantsur | aarefiev, I want to reset https://github.com/openstack/ironic-inspector/blob/master/example.conf#L63 to .* and deprecate it | 11:24 |
dtantsur | with virtualbmc we can't really figure out if we're on virt or not | 11:24 |
dtantsur | and anyway I've heard of jammed DHCP on bare metal as well | 11:24 |
dtantsur | wdyt? also sambetts ^^^ | 11:24 |
dtantsur | jroll, did we release clients yesterday? | 11:25 |
aarefiev | dtantsur: +1 that makes sense, ssh drivers was deprecated while ago | 11:26 |
jroll | dtantsur: we did indeed :) | 11:27 |
dtantsur | w00t | 11:28 |
*** pester has quit IRC | 11:28 | |
aarefiev | dtantsur: does this mean we can't land feature with required client changes | 11:29 |
dtantsur | aarefiev, we can land it, but the client change won't go in Ocata | 11:29 |
dtantsur | e.g. the API will available in Ocata, but the client change only in Pike | 11:30 |
aarefiev | ok, thanks | 11:30 |
*** jkilpatr has quit IRC | 11:40 | |
*** gcb has quit IRC | 11:43 | |
*** openstackgerrit has joined #openstack-ironic | 11:43 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Deprecate introspection_delay_drivers option and make it no-op https://review.openstack.org/425641 | 11:43 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: ETAGs maintenance for all api node's requests https://review.openstack.org/392213 | 11:55 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Deprecated log_bmc_address option https://review.openstack.org/425642 | 11:55 |
*** jkilpatr has joined #openstack-ironic | 11:58 | |
openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support https://review.openstack.org/264257 | 12:00 |
*** pc_m has quit IRC | 12:01 | |
sambetts | dtantsur: +1 from me on deprecating that option | 12:04 |
*** jkilpatr has quit IRC | 12:07 | |
*** pc_m has joined #openstack-ironic | 12:10 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests https://review.openstack.org/423556 | 12:10 |
*** xavierr has quit IRC | 12:13 | |
openstackgerrit | Merged openstack/ironic-inspector: Make grenade actually upgrade our source code https://review.openstack.org/425066 | 12:14 |
dtantsur | w00t, working CI on stable/newton! | 12:15 |
aarefiev | dtantsur: when we are going to switch on tempest plugins from separate repo | 12:18 |
dtantsur | mmm, after Ocata final? dunno, the folks cannot still agree on this as a Pike goal | 12:19 |
aarefiev | I see one issue in grenade with new devstack-tool | 12:19 |
*** priteau has joined #openstack-ironic | 12:19 | |
dtantsur | devstack-tool? what is that? | 12:19 |
aarefiev | dtantsur: https://github.com/openstack/devstack-tools | 12:20 |
aarefiev | it's already used in grenade | 12:20 |
dtantsur | omg | 12:20 |
* dtantsur thinks that if you need a tool for your configuration files, something goes wrong | 12:20 | |
*** jkilpatr has joined #openstack-ironic | 12:21 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add dynamic interfaces fields to nodes API https://review.openstack.org/424720 | 12:21 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add dynamic driver functionality to REST API https://review.openstack.org/417970 | 12:21 |
jroll | whew, that was fun | 12:21 |
* dtantsur thinks of approving that before most of folks wake up :D | 12:22 | |
jroll | lol | 12:22 |
jroll | dtantsur: rloo pointed out we forgot about raid things :P | 12:22 |
dtantsur | oh | 12:22 |
jroll | also found a couple bugs that mocking was hiding | 12:22 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2 https://review.openstack.org/425068 | 12:22 |
dtantsur | fun indeed | 12:22 |
* dtantsur is going to get seriously drunk when we finish this driver composition reform | 12:23 | |
jroll | +1 | 12:23 |
vdrok | yeah, seems like "fast review" is not going well :) | 12:24 |
jroll | dtantsur: here's what I have left: | 12:24 |
jroll | code for "conductor will fail to start if there is a name clash between classic and dynamic" | 12:24 |
jroll | code to allow no classic drivers to be loaded, vice versa | 12:24 |
jroll | CI stuff | 12:24 |
jroll | missing anything obvious? | 12:24 |
dtantsur | " code to allow no classic drivers to be loaded"? | 12:25 |
dtantsur | what's it about? | 12:25 |
*** pester has joined #openstack-ironic | 12:25 | |
*** vsaienko has quit IRC | 12:25 | |
* jroll finds link | 12:25 | |
dtantsur | also please review the IPMI driver: https://review.openstack.org/#/c/422708/ | 12:26 |
patchbot | patch 422708 - ironic - Add hardware type for IPMI using ipmitool | 12:26 |
jroll | dtantsur: https://github.com/openstack/ironic/blob/master/ironic/conductor/base_manager.py#L91 | 12:26 |
dtantsur | oh, good catch | 12:26 |
jroll | get rid of that check and instead do "if not drivers and not hardware_types: fail" later | 12:26 |
jroll | oh, I actually put the todo in code: https://github.com/openstack/ironic/blob/master/ironic/conductor/base_manager.py#L162 | 12:26 |
jroll | will review ipmi driver today | 12:27 |
dtantsur | cool, thnx. yeah, I think the TODO list for Ocata is complete | 12:27 |
jroll | \o/ | 12:27 |
dtantsur | in Pike we should get MOAR HARDWARE TYPES | 12:27 |
jroll | oh I guess we need those config warnings too | 12:27 |
jroll | +1 | 12:27 |
* jroll steps away for a while | 12:30 | |
*** ccamacho is now known as ccamacho|mcdnlds | 12:34 | |
openstackgerrit | Oksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV https://review.openstack.org/397126 | 12:36 |
*** vikrant has quit IRC | 12:37 | |
*** mjura has quit IRC | 12:38 | |
*** mjura has joined #openstack-ironic | 12:40 | |
*** vsaienko has joined #openstack-ironic | 12:45 | |
dtantsur | mmm, moar hardware types already, sweet: https://review.openstack.org/#/c/425604/ | 12:45 |
patchbot | patch 425604 - ironic - Add hardware type irmc-s4 for iRMC S4 servers | 12:45 |
*** vikrant has joined #openstack-ironic | 12:45 | |
*** vsaienko has quit IRC | 12:47 | |
*** vikrant has quit IRC | 12:47 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: ETAGs for subresource requests https://review.openstack.org/425260 | 12:47 |
cdaniel | morning al | 12:48 |
cdaniel | *all | 12:48 |
openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support https://review.openstack.org/264257 | 12:48 |
*** fellypefca has joined #openstack-ironic | 12:48 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/425670 | 12:51 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/425671 | 12:51 |
*** dprince has joined #openstack-ironic | 12:51 | |
*** lucasagomes is now known as lucas-hungry | 12:52 | |
aarefiev | dtantsur: introspection_delay_drivers will match all drivers, so delay is applied for all drivers, right? | 12:53 |
dtantsur | aarefiev, correct, unless introspection_delay is set to 0 | 12:54 |
aarefiev | dtantsur: you just saying that delay is not applied in release notes | 12:55 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [devstack] Stop installing jq, we're not using it in tests any more https://review.openstack.org/425675 | 12:55 |
dtantsur | aarefiev, s/not/now/, ugh :( | 12:56 |
dtantsur | lemme fix | 12:56 |
dtantsur | aarefiev, any more comments to fix? | 12:56 |
*** pester has quit IRC | 12:57 | |
aarefiev | I don't see any | 12:58 |
dtantsur | cool, posting an update | 12:58 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Deprecate introspection_delay_drivers option and make it no-op https://review.openstack.org/425641 | 12:58 |
* dtantsur ALWAYS mistypes "not" vs "now" :( | 12:59 | |
* aarefiev correct dtantsur with s/not/now next time | 13:00 | |
dtantsur | jroll, wdyt about approving https://bugs.launchpad.net/ironic/+bug/1659209 ? | 13:06 |
openstack | Launchpad bug 1659209 in Ironic "[RFE] Add hardware type of iRMC S4 supporting ironic dynamic drivers" [Wishlist,In progress] - Assigned to Dao Cong Tien (tiendc) | 13:06 |
openstackgerrit | Merged openstack/ironic: [Devstack] enable only pxe|agent_ipmitool by default https://review.openstack.org/425572 | 13:08 |
*** bfournie has quit IRC | 13:08 | |
jroll | dtantsur: seems fine to me :) | 13:11 |
jroll | nice small patch too | 13:11 |
* jroll approves it | 13:12 | |
*** vsaienko has joined #openstack-ironic | 13:14 | |
*** galyna has quit IRC | 13:15 | |
*** lucas-hungry is now known as lucasagomes | 13:16 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Restrict access to IPA API https://review.openstack.org/425682 | 13:18 |
*** trown|outtypewww is now known as trown | 13:19 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Add hardware type for IPMI using ipmitool https://review.openstack.org/422708 | 13:23 |
dtantsur | had to rebase :( | 13:23 |
jroll | dtantsur: heh, was just about to ping you for that | 13:23 |
*** amotoki has joined #openstack-ironic | 13:24 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Restrict access to IPA API https://review.openstack.org/425682 | 13:24 |
*** baoli has joined #openstack-ironic | 13:27 | |
*** baoli has quit IRC | 13:27 | |
*** baoli has joined #openstack-ironic | 13:31 | |
*** baoli has quit IRC | 13:31 | |
*** moshele has quit IRC | 13:34 | |
*** baoli has joined #openstack-ironic | 13:38 | |
*** bfournie has joined #openstack-ironic | 13:39 | |
*** sdake has joined #openstack-ironic | 13:39 | |
*** baoli has quit IRC | 13:39 | |
*** baoli has joined #openstack-ironic | 13:39 | |
*** vsaienko has quit IRC | 13:43 | |
*** xavierr has joined #openstack-ironic | 13:49 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Remove heading "Team and repository tags" https://review.openstack.org/425700 | 13:50 |
*** jheroux has joined #openstack-ironic | 13:51 | |
*** ccamacho|mcdnlds is now known as ccamacho | 13:53 | |
*** pester has joined #openstack-ironic | 13:56 | |
*** fellypefca has quit IRC | 13:56 | |
*** rbudden has joined #openstack-ironic | 13:56 | |
*** fxpester has quit IRC | 14:05 | |
openstackgerrit | Merged openstack/ironic-inspector: Do not use loopback BMC addresses for lookup https://review.openstack.org/421909 | 14:05 |
*** Goneri has joined #openstack-ironic | 14:06 | |
*** hw_wutianwei has joined #openstack-ironic | 14:08 | |
*** vsaienko has joined #openstack-ironic | 14:10 | |
*** pester has quit IRC | 14:10 | |
openstackgerrit | Merged openstack/ironic-inspector: [devstack] Remove bridge only if it's not OVB_PHYSICAL_BRIDGE https://review.openstack.org/423758 | 14:11 |
*** rloo has joined #openstack-ironic | 14:15 | |
openstackgerrit | Aparna proposed openstack/proliantutils: Add HPSUM firmware update support https://review.openstack.org/422004 | 14:17 |
vsaienk0 | sambetts: please add to your review queue https://review.openstack.org/#/c/415881/ | 14:17 |
patchbot | patch 415881 - ironic - Validate port info before assume we may use it | 14:17 |
sambetts | vsaienk0: will do :) | 14:18 |
vsaienk0 | thanks! | 14:18 |
*** sasha21 has joined #openstack-ironic | 14:19 | |
*** vsaienko has quit IRC | 14:20 | |
*** absubram has joined #openstack-ironic | 14:22 | |
openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 14:23 |
*** jcoufal has joined #openstack-ironic | 14:24 | |
*** absubram_ has joined #openstack-ironic | 14:26 | |
*** vsaienko has joined #openstack-ironic | 14:26 | |
*** absubram has quit IRC | 14:27 | |
*** absubram_ is now known as absubram | 14:27 | |
*** vsaienko has quit IRC | 14:29 | |
*** srobert has joined #openstack-ironic | 14:30 | |
openstackgerrit | Oksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV https://review.openstack.org/397126 | 14:30 |
*** vsaienko has joined #openstack-ironic | 14:30 | |
*** mrtenio has joined #openstack-ironic | 14:31 | |
xavierr | dtantsur: I have added the traceback to https://bugs.launchpad.net/ironic/+bug/1657892 | 14:33 |
openstack | Launchpad bug 1657892 in Ironic "ValueError when trying to deallocate a non allocated machine in OneView due fail" [Undecided,In progress] - Assigned to Xavier (marcusrafael) | 14:33 |
xavierr | dtantsur: is that ok? :) | 14:33 |
*** hshiina has quit IRC | 14:34 | |
dtantsur | yep, thanks | 14:34 |
*** hw_wutianwei has quit IRC | 14:38 | |
*** ElCoyote_ has joined #openstack-ironic | 14:39 | |
xavierr | sambetts: we were discussing about this patch some time ago. if you have some time to review it :) | 14:43 |
xavierr | sambetts: https://review.openstack.org/#/c/408298/ | 14:43 |
patchbot | patch 408298 - ironic - Shutdown server before change boot order for agent... | 14:43 |
*** rama_y has joined #openstack-ironic | 14:43 | |
rloo | hi and morning all, xavierr, dtantsur, sambetts, vsaienk0 | 14:44 |
*** rama_y has quit IRC | 14:44 | |
dtantsur | morning rloo | 14:44 |
xavierr | hey rloo | 14:44 |
rloo | jroll: client release notes look good :) i probably would have combined the two NMI ones into one but no big deal. http://docs.openstack.org/releasenotes/python-ironicclient/unreleased.html | 14:45 |
*** cdearborn has joined #openstack-ironic | 14:50 | |
*** mtanino has joined #openstack-ironic | 14:51 | |
*** vsaienko has quit IRC | 14:51 | |
*** mtanino has quit IRC | 14:52 | |
jroll | rloo: awesome, thanks :) | 14:53 |
*** rushil has joined #openstack-ironic | 14:54 | |
vdrok | morning xavierr and rloo | 14:55 |
openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 14:55 |
xavierr | hey vdrok o/ | 14:55 |
rloo | jroll: am reviewing 417970 now. was thinking about what you said about the patch being too big. why can't we split these patches up, and merge them in separately but ONLY enable them 'at the end'. | 14:55 |
*** jrist has quit IRC | 14:56 | |
rloo | jroll: not to do for this particular patch. but eg with your two patches, the first one, for utils.allow_dynamic_drivers(), just always return false. then only need to bump microversion once in second patch. | 14:56 |
lucasagomes | rloo, morning | 14:57 |
rloo | hi lucasagomes! | 14:57 |
xavierr | morning lucasagomes | 14:57 |
dtantsur | dunno about this idea, but we can split it into 1. return hw types in drivers API (no microversion), 2. driver filtering (first microversion), 3. node attributes (second) | 14:57 |
dtantsur | but at this stage I suspect it's easier to just merge them finally... | 14:57 |
mgould | hi rloo lucasagomes dtantsur xavierr vdrok jroll sambetts | 14:58 |
rloo | dtantsur: i don't want to split these two patches. just thinking for future features. | 14:58 |
vdrok | morning mgould | 14:58 |
rloo | dtantsur: so that we bump the version only once when the feature is all there. | 14:59 |
rloo | dtantsur: and make the feature avail only when it is all there. | 14:59 |
*** rushil has quit IRC | 15:00 | |
dtantsur | yeah, it makes sense, though it complicates unit testing | 15:01 |
jroll | rloo: yeah, maybe, I'll think about it next time | 15:01 |
* jroll half here half prepping dinner | 15:01 | |
* rloo wonders why jroll is prepping dinner so early in the day | 15:01 | |
rloo | dtantsur: good point. i'll have to think about it more the next time i see a feature with more than one API patch/version bump. | 15:02 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Devstack: Create a "no ansi" logfile for the baremetal console logs https://review.openstack.org/425426 | 15:04 |
*** rushil has joined #openstack-ironic | 15:04 | |
jlvillal | lucasagomes, On the no_ansi patch ^^ I was keeping both versions as the 'no_ansi' version is technically deleting data. So I wasn't sure if someone might need the full version for some debugging. | 15:05 |
jlvillal | lucasagomes, Not sure what you think? | 15:05 |
* dtantsur would probably not need the old version | 15:05 | |
lucasagomes | jlvillal, hmm I thought it was just converting the ansi stuff instead of deleting it | 15:06 |
*** absubram has quit IRC | 15:06 | |
vsaienk0 | morning rloo | 15:06 |
lucasagomes | jlvillal, so idk now, maybe keep both them... but I looked at both logs and the data seems to be there | 15:06 |
jlvillal | lucasagomes, It just strips out any ansi codes it finds. Deleting them. | 15:06 |
lucasagomes | and no ansi is def easier to read so, I thought about just keeping it | 15:06 |
jlvillal | lucasagomes, Yeah the no_ansi is a lot easier to read :) | 15:07 |
jlvillal | This: http://logs.openstack.org/49/422149/7/experimental/gate-grenade-dsvm-ironic-multitenant-ubuntu-xenial-nv/d416cdb/logs/old/ironic-bm-logs/node-4_no_ansi_2017-01-26-00:46:51.txt.gz | 15:09 |
lucasagomes | jlvillal, yeah, I guess we keep both then | 15:09 |
lucasagomes | at least for now idk | 15:09 |
jlvillal | compared to this: http://logs.openstack.org/49/422149/7/experimental/gate-grenade-dsvm-ironic-multitenant-ubuntu-xenial-nv/d416cdb/logs/old/ironic-bm-logs/node-4_console_2017-01-26-00:46:51.txt.gz | 15:09 |
dtantsur | yeah, the latter is completely useless | 15:09 |
*** jrist has joined #openstack-ironic | 15:10 | |
jlvillal | Only makes sense if you cat it into your terminal. At least for me :) | 15:10 |
jlvillal | lucasagomes, Okay. I will leave it for now. If people want to remove the old version it is simple enough. | 15:11 |
*** moshele has joined #openstack-ironic | 15:11 | |
lucasagomes | jlvillal, yeah I will change my vote | 15:11 |
jlvillal | lucasagomes, thanks | 15:13 |
*** mtanino has joined #openstack-ironic | 15:14 | |
*** moshele has quit IRC | 15:15 | |
*** amarao has quit IRC | 15:22 | |
*** Nisha_Agarwal has joined #openstack-ironic | 15:27 | |
*** jaosorior has joined #openstack-ironic | 15:31 | |
*** sacharya has joined #openstack-ironic | 15:31 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests https://review.openstack.org/423556 | 15:33 |
*** stevemar has quit IRC | 15:36 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to session for glance client https://review.openstack.org/410767 | 15:37 |
jroll | rloo: low and slow today :P | 15:37 |
*** stevemar has joined #openstack-ironic | 15:37 | |
rloo | jroll: oooo, will be yummy... | 15:38 |
jroll | indeed! | 15:38 |
jroll | I'm here for real now, what needs some help? | 15:39 |
rloo | jroll: if easy and tasty, please share :) | 15:39 |
*** rushil has quit IRC | 15:39 | |
*** crushil has joined #openstack-ironic | 15:40 | |
jroll | rloo: salt/pepper beef roast. quickly sear to give it a crust. put in crockpot. add one large-ish jar of pepperoncini, including juice. set crockpot to low. wait 8 hours. pull apart and eat. :) | 15:40 |
*** vsaienko has joined #openstack-ironic | 15:41 | |
jroll | one of the easiest meals I do regularly, but so good | 15:41 |
rloo | jroll: easy except for searing (but expected). will have to look into that pepperoncini, not sure what that is. | 15:41 |
soliosg | jfyi, for vim users, I've used AnsiEsc plugin to get a more readable view | 15:42 |
rloo | jroll: oh, spicy peppers... guess that won't work in our household... | 15:42 |
soliosg | here http://www.vim.org/scripts/script.php?script_id=4979, or here https://github.com/vim-scripts/Improved-AnsiEsc | 15:42 |
jroll | rloo: oh :( | 15:42 |
rloo | jroll: well, one web site sez they are 'mild' spicy peppers :) | 15:43 |
soliosg | but without question, jlvillal's script is a great contribution | 15:43 |
*** vsaienko has quit IRC | 15:43 | |
jroll | rloo: they're weaker than jalapeno, stronger than bell pepper, heh. the meat doesn't get spicy, though, so you could just toss the peppers after cooking | 15:44 |
rloo | jroll: i'm going to try it! | 15:46 |
jroll | rloo: let me know how it turns out :) | 15:46 |
rloo | jroll: will do! | 15:47 |
*** rpioso has joined #openstack-ironic | 15:48 | |
*** openstackgerrit has quit IRC | 15:48 | |
dtantsur | $ openstack baremetal port group list | 15:49 |
dtantsur | Resource could not be found. (HTTP 404) | 15:49 |
dtantsur | ehmmmm | 15:49 |
jroll | what api version does osc default to? | 15:49 |
* jroll hopes it's latest | 15:49 | |
dtantsur | good catch. no, it's not latest | 15:49 |
jroll | le sigh | 15:50 |
mariojv | pepperoncini is pretty mild, you can usually get it on a pizza if you want a taste | 15:51 |
dtantsur | this is pretty bad, now instead of a clear message we're getting HTTP 404 | 15:51 |
jroll | dtantsur: we can easily catch that and print a real error | 15:51 |
jroll | though I'd rather default client to latest :| | 15:51 |
dtantsur | well, how do we catch it? | 15:52 |
jroll | except NotFound: ? | 15:52 |
mariojv | https://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/http.py#L48 | 15:52 |
dtantsur | jroll, it's fine for 'portgroup list', but what about 'portgroup show'? | 15:52 |
jroll | dtantsur: yeeeesh | 15:53 |
jroll | do we give a different error in that case? | 15:53 |
mariojv | idk if that's the actually used version though | 15:53 |
jroll | mariojv: it is and I hate it so much | 15:53 |
mariojv | (1.9 seems extremely low) | 15:53 |
dtantsur | jroll, I suspect it's both HTTP 404 | 15:53 |
mariojv | really?? | 15:53 |
*** Syed__ has joined #openstack-ironic | 15:53 | |
mariojv | it should at _least_ go to last named release | 15:53 |
jroll | dtantsur: yeah, but, I believe there's an error code involved | 15:53 |
mariojv | preferably negotiate the highest version it knows about | 15:53 |
jroll | mariojv: people didn't want to ever bump it, because they don't want to break people's bash scripts :( :( :( | 15:54 |
* jroll repeats again, the CLI is not an SDK | 15:54 | |
jroll | s/an SDK/a bash SDK/ | 15:54 |
* dtantsur kind of agrees | 15:54 | |
mariojv | i wonder how hard it would be to just try to roll back to a version where the command is supported if they pass an unrecognized command | 15:55 |
jroll | I feel like we have tons of cleanup work to do | 15:55 |
jroll | 1) get the cli bumped up | 15:55 |
dtantsur | yeah.. | 15:55 |
jroll | 2) actually use new cli stuff in devstack etc | 15:55 |
dtantsur | I wonder if we can fix something right now to avoid people confused as hell | 15:55 |
jroll | 3) so many deprecation warnings everywhere :/ | 15:55 |
mariojv | how would y'all feel about this as a ptg topic, if it isn't one yet? | 15:55 |
dtantsur | I do feel like that | 15:56 |
* dtantsur -> quick snack | 15:56 | |
jroll | dtantsur: I mean, we could hack it, something like: if api version < portgroup version, raise error('bump the version noob') | 15:56 |
jroll | mariojv: +1000 | 15:56 |
dtantsur | jroll, heh, maybe :) | 15:56 |
mariojv | there's already a topic about deprecating ironic CLI | 15:56 |
mariojv | should we mix this in with that, or have a separate topic? | 15:56 |
jroll | let's make it separate | 15:57 |
jroll | it's gonna get heated | 15:57 |
jroll | though | 15:57 |
* jroll points at http://specs.openstack.org/openstack/ironic-specs/specs/kilo-implemented/api-microversions.html#new-client-default-microversion | 15:57 | |
mariojv | i think this is a problem that needs to be talked about as a preqreq to deprecating ironic cli | 15:57 |
mariojv | ooh | 15:57 |
jroll | second paragraph | 15:57 |
jroll | maybe jfdi | 15:57 |
jroll | especially now that we have a good relnotes process | 15:57 |
*** openstackgerrit has joined #openstack-ironic | 15:57 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 15:57 |
mariojv | i'll link to that in whatever i put in https://etherpad.openstack.org/p/ironic-pike-ptg | 15:58 |
jroll | thanks | 15:58 |
jroll | there's probably some prework to do | 15:58 |
mariojv | jroll: dtantsur can i put y'all's name on this topic as well? | 15:58 |
jroll | in devstack and etc | 15:58 |
jroll | mariojv: yes please | 15:58 |
* jroll just plans to yell about every topic anyway | 15:58 | |
mariojv | :) | 15:58 |
*** sayantani01 has joined #openstack-ironic | 15:59 | |
* rloo wonders if jroll can yell once he isn't ptl | 16:01 | |
jroll | rloo: how'd you think I ended up as ptl? :P | 16:01 |
rloo | jroll: being nice? :D | 16:02 |
jroll | heh | 16:02 |
dtantsur | mariojv, ditto as jroll :) | 16:07 |
mariojv | cool; L96, feel free to modify: https://etherpad.openstack.org/p/ironic-pike-ptg | 16:08 |
dtantsur | rloo, if I become a ptl, I'll appoint jroll as "yelling liaison" | 16:08 |
jroll | nice | 16:08 |
JayF | dtantsur: I have a lot of volume to bring to the table | 16:08 |
JayF | dtantsur: don't make any rash decisions | 16:08 |
rloo | dtantsur: Oh, wow, that's a good promise. | 16:08 |
rloo | But JayF has a good point. | 16:08 |
jroll | I tend to think of yelling more about the ranting than the volume :P | 16:08 |
dtantsur | look, we already have a competition :D | 16:08 |
jroll | maybe I can pipe my rants through JayF | 16:09 |
openstackgerrit | Merged openstack/ironic: Add hardware type for IPMI using ipmitool https://review.openstack.org/422708 | 16:09 |
* mariojv would tune into a ptl election debate | 16:09 | |
rloo | too bad we couldn't have a candidate-debate (or whatever they are called) at the ptg :) | 16:09 |
jroll | woooo ipmi | 16:09 |
jroll | heh | 16:09 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Allow diskless nodes introspection https://review.openstack.org/326620 | 16:09 |
dtantsur | rloo, would not work, we like each other too much ^_^ | 16:09 |
*** krtaylor has quit IRC | 16:09 | |
rloo | dtantsur: hmm, maybe you're all not fit to be candidates then ;) | 16:10 |
*** jrist has quit IRC | 16:10 | |
dtantsur | do we have "against everyone" in our voting? | 16:10 |
rloo | dtantsur: we need someone that can buck the establishment. the peons are restless. let them eat cake i say. | 16:10 |
JayF | dtantsur: nope, it's just ranked choice of three, but you can rank people equally | 16:11 |
dtantsur | too bad, rloo will have to pick one of us :D | 16:11 |
rloo | dtantsur: IF I decide to be responsible (or is it a right) in this democratic system, and actually vote :) | 16:12 |
dtantsur | true | 16:12 |
rloo | honestly, i'd prefer if only one candidate ran, would make it easier for me cuz unfortunately i like you all and i think you'd all do a great job. | 16:13 |
JayF | I was commenting to someone the other day, about how good it is for our communtiy that we have 3 folks running, all from different companies, and all very capable of doing the job | 16:13 |
rloo | JayF: why? | 16:13 |
JayF | rloo: just a sign of good and varied participation. As a project, Ironic isn't dominated by a single company or agenda | 16:14 |
JayF | rloo: and having 3 different folks who could step in and be a good PTL is more than I'd suspect for other projects | 16:14 |
rloo | JayF: but that would be true (to me) even if one candidate ran. maybe cuz for me, i know the candidates and what they are capable of. | 16:14 |
JayF | I appreciate when elections aren't unopposed, as a general concept :) | 16:15 |
* dtantsur suggests next time all cores to run. we'll know who's the most popular here :D | 16:15 | |
JayF | If I am elected PTL, I'll be able to draw from the strength of knowing folks had confidence in me to do it | 16:15 |
rloo | what would be good is if the candidate companies could sign (not necessarily in blood) that they fully support the candidates and will let them work 99% upstream. | 16:15 |
dtantsur | JayF++ | 16:15 |
mariojv | rloo: that's a very interesting idea | 16:16 |
jlvillal | I vote for blood! ;) | 16:16 |
rloo | JayF: if there was only one candidate and i opposed them, i probably would mention it... | 16:16 |
rloo | oh oh, didn't realize there were some blood-thirsty folks in the crowd | 16:17 |
jlvillal | o/ | 16:17 |
JayF | I'm usually sweet-tea-thirsty. | 16:17 |
JayF | lol | 16:17 |
rloo | JayF: but do you kiss babies? | 16:17 |
JayF | rloo: I actually kinda hate children | 16:18 |
* dtantsur got distracted for a minute and the discussion is already slightly weird | 16:18 | |
JayF | rloo: like I'm glad they exist as a concept, and that the next generation is happening. I just want them to happen away from me. | 16:18 |
rloo | dtantsur: see, we need 100% attention from ptls. otherwise things degenerate! | 16:18 |
* dtantsur puts his signature under "I'm glad they exist as a concept, and that the next generation is happening. I just want them to happen away from me." | 16:19 | |
dtantsur | rloo, mm, maybe we need all 3 to become PTLs? | 16:19 |
* dtantsur notes that it's not even Friday yet | 16:20 | |
JayF | we can form a mighty triumvirate, Roman style! | 16:20 |
*** aNuposic has joined #openstack-ironic | 16:20 | |
rloo | dtantsur: i thought of that, but i suspect the buck has to stop somewhere and i don't think that is something i want to try to change. there are far more important things i think :) | 16:20 |
*** jcoufal has quit IRC | 16:20 | |
dtantsur | heh | 16:20 |
*** jrist has joined #openstack-ironic | 16:21 | |
*** krtaylor has joined #openstack-ironic | 16:22 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Devstack: Create a "no ansi" logfile for the baremetal console logs https://review.openstack.org/425457 | 16:22 |
* mgould notes that Roman triumvirates tended to collapse into bloody civil war within a few years... | 16:25 | |
dtantsur | well, jlvillal asked for some blood :) | 16:26 |
vdrok | maybe troika then? | 16:26 |
mgould | [ http://thehistoryofrome.typepad.com/ <- strongly recommended podcast, btw] | 16:26 |
dtantsur | oh vdrok +100500 | 16:26 |
jlvillal | :D | 16:26 |
JayF | mgould: I literally just finished that | 16:26 |
JayF | mgould: like listened to the final episode yesterday | 16:27 |
mgould | JayF: awesome! I've got about five episodes to go | 16:27 |
mgould | epically long, but totally worth it | 16:27 |
*** jtomasek has quit IRC | 16:27 | |
JayF | mgould: assuming you've done Hardcore History by Dan Carlin as well? | 16:27 |
jlvillal | vsaienk0: vdrok: Not sure if you have time to look at the failure with the grenade-multitenant job: http://logs.openstack.org/49/422149/7/experimental/gate-grenade-dsvm-ironic-multitenant-ubuntu-xenial-nv/d416cdb/ | 16:28 |
jlvillal | vsaienk0: vdrok: My theory is that it has something to do with grenade changing the networks when it is finished. You discussed it in the summit presentation. | 16:28 |
mgould | JayF: not yet, but I'll need something to replace HoR in a few days :-) | 16:29 |
jlvillal | vsaienk0: vdrok: But I don't know the details on how it was fixed. | 16:29 |
JayF | mgould: it's a huge backlog, and multi-hour long episodes. If you buy the backlog, you'll have podcasts to listen to for a year+ lol | 16:30 |
lucasagomes | sambetts, quick q (possible dumb): For testing the ironic/neutron integration we have a "fake/generic" ml2 driver for neutron, right ? | 16:30 |
lucasagomes | sambetts, if so, what's the code repository for it ? | 16:30 |
lucasagomes | vsaienk0, ^ maybe you know it ? | 16:32 |
vdrok | lucasagomes: github.com/openstack/networking-generic-switch | 16:33 |
lucasagomes | vdrok, thanks much! | 16:33 |
vdrok | jlvillal: yeah, seems like it, we'll take a look, likely tomorrow morning :) | 16:33 |
jlvillal | vdrok: Woot! :) | 16:33 |
jlvillal | vgadiraj: ^^^ | 16:33 |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Update reno for stable/ocata https://review.openstack.org/425480 | 16:36 |
*** baoli has quit IRC | 16:36 | |
*** dprince has quit IRC | 16:37 | |
*** jcoufal has joined #openstack-ironic | 16:37 | |
*** vsaienko has joined #openstack-ironic | 16:38 | |
jroll | TheJulia: dtantsur: would like your eyes on this gerrit acl change: https://review.openstack.org/#/c/425777/ | 16:39 |
patchbot | patch 425777 - openstack-infra/project-config - Clean up ACLs for some ironic projects | 16:39 |
jroll | and krtaylor maybe? ^^ (trying to recall who else is molteniron | 16:39 |
jroll | ) | 16:39 |
dtantsur | jroll, ironic-inspector-core includes ironic-core. do you want this change just for clarity? | 16:40 |
krtaylor | jroll, sorry on a call | 16:41 |
*** e0ne has quit IRC | 16:41 | |
krtaylor | mjturek, hamzy, ^^^^ molten iron ping | 16:41 |
*** sayantani01 has quit IRC | 16:41 | |
jroll | dtantsur: yeah, that was my goal | 16:41 |
*** sayantani01 has joined #openstack-ironic | 16:42 | |
jroll | krtaylor: no rush :) | 16:42 |
dtantsur | jroll, one further suggestion, everything else looks good | 16:42 |
jroll | soliosg: please don't add to the whiteboard's priorities, rather suggest items in the meeting. especially without a link. outside of the meeting please run things by the PTL or some other cores :) | 16:42 |
jroll | dtantsur: thanks | 16:43 |
hamzy | the molteniron bit for that patch seems ok | 16:43 |
*** rama_y has joined #openstack-ironic | 16:43 | |
jroll | hamzy: thanks! | 16:44 |
mjturek | jroll: yep looks good | 16:44 |
hamzy | we need moar reviewers! :) | 16:44 |
*** vsaienko has quit IRC | 16:53 | |
NobodyCam | Good Morning Ironic'ers | 16:53 |
dtantsur | morning NobodyCam | 16:53 |
NobodyCam | :) morning dtantsur :) | 16:53 |
*** e0ne has joined #openstack-ironic | 16:54 | |
*** e0ne has quit IRC | 16:55 | |
*** tesseract has quit IRC | 16:55 | |
*** athomas has quit IRC | 16:56 | |
mgould | morning NobodyCam | 16:59 |
jroll | morning NobodyCam! :) | 17:00 |
*** rajinir has joined #openstack-ironic | 17:00 | |
NobodyCam | morning mogul & roll ... almost Friday :) | 17:01 |
NobodyCam | gah. | 17:01 |
NobodyCam | s/roll/jroll/ | 17:01 |
dtantsur | s/mogul/mgould/ as well I suspect :) | 17:02 |
* dtantsur hands NobodyCam some coffee | 17:02 | |
*** [1]cdearborn has joined #openstack-ironic | 17:02 | |
NobodyCam | lol yep. /me trains autocorrect | 17:02 |
mgould | NobodyCam: rather be skiing? :-) | 17:03 |
jroll | heh | 17:03 |
NobodyCam | heheheh !!! | 17:03 |
soliosg | jroll: thanks, removed 'tempest plugin migration' from that section | 17:03 |
jroll | thanks soliosg | 17:03 |
soliosg | jroll: will bring the topic in the meeting | 17:03 |
soliosg | *bring up | 17:04 |
jroll | thanks | 17:04 |
* dtantsur would prefer to delay discussing it to the PTG | 17:05 | |
jroll | dtantsur: yeah, I asked soliosg to bring it up in the meeting mostly as an FYI, we can punt agreeing to the plan to PTG if we like | 17:07 |
dtantsur | ack | 17:08 |
dtantsur | I know there will be more discussions on the QA track, which I planned to attend | 17:08 |
dtantsur | s/planned/am planning | 17:08 |
joanna | JayF: should I change priority of https://bugs.launchpad.net/ironic/+bug/1513005 to something else than wishlist? (just assuming by 'openstack-wide priority) | 17:09 |
openstack | Launchpad bug 1513005 in Ironic "Auto deploy devstack with ir-api under apache" [Wishlist,In progress] - Assigned to Joanna Taryma (jtaryma) | 17:09 |
JayF | joanna: it's still considered an RFE, in fact, the title should be prefixed with [RFE]] | 17:09 |
patchbot | Error: Spurious "]". You may want to quote your arguments with double quotes in order to prevent extra brackets from being evaluated as nested commands. | 17:09 |
JayF | joanna: I will vote to approve said RFE as long as it's detailed enoughh | 17:10 |
dtantsur | joanna, well, it is still a wish list item, not a bug | 17:10 |
* dtantsur is also ready to approve it | 17:10 | |
jroll | rloo: should I wait for comments on the second API patch or go ahead and address the first now? | 17:10 |
* jroll never wanted to make the [RFE] thing a rule, just a guideline, lol | 17:10 | |
joanna | JayF: do you think it needs additional description now? | 17:11 |
JayF | joanna: I'm OK with it as-is, but I understand the problem well. It never hurts to have a more detailed description :) | 17:11 |
*** crushil has quit IRC | 17:11 | |
joanna | so... assuming I'll add more details to it - is there anything else I should do to push it through the process of being approved and assigned a priority? | 17:12 |
JayF | well not all RFEs have to be a project priority | 17:13 |
JayF | but priorities for pike are determined at the ptg, or at least, drafted there | 17:13 |
mariojv | yes, and usually the community can vote on them afterwards | 17:13 |
JayF | you just need another core to be +2 to the rfe being approved, then it gets its tag changed to 'rfe-approved' | 17:13 |
mariojv | on gerrit | 17:13 |
JayF | as far as you're concerned, I'd just update the bug to be more detailed, keep working, and when folks are less busy sprinting to the end of ocata, it should be easy to get attention and get the rfe approved | 17:14 |
*** sayantani01 has quit IRC | 17:14 | |
dtantsur | JayF, joanna, I'm +2 on it, feel free to add rfe-approved | 17:14 |
sambetts | lucasagomes: sorry I missed your ping, I think vsaienk0 pointed you in the right direction | 17:14 |
JayF | joanna: ^ done | 17:15 |
lucasagomes | sambetts, yeah, no worries! Thanks | 17:15 |
joanna | JayF: are you +2 too? :) | 17:15 |
JayF | I had already put that in the bug, lol | 17:15 |
joanna | dtantsur: thank you :) | 17:15 |
JayF | I am a core, if that's what you're asking? | 17:15 |
*** persia has quit IRC | 17:16 | |
joanna | JayF: amazing thanks! | 17:16 |
dtantsur | :) | 17:16 |
*** mjura has quit IRC | 17:16 | |
joanna | JayF: Nope, I was just making sure you're fine with current description :) | 17:16 |
*** cdearborn has quit IRC | 17:16 | |
JayF | it's pretty straightforward. | 17:16 |
* dtantsur now remembers we have to do the same with inspector, sigh.. | 17:16 | |
jroll | heh | 17:17 |
jroll | yeah that will be a project priority, because it's an openstack priority | 17:18 |
jroll | joanna: major project priorities like this, by the way, are recorded in the specs repo, e.g. http://specs.openstack.org/openstack/ironic-specs/priorities/ocata-priorities.html | 17:18 |
JayF | jroll: we can avoid making it a pike priority by joanna JFDI :P | 17:18 |
jroll | ¯\_(ツ)_/¯ | 17:19 |
lucasagomes | folks, calling it a day | 17:19 |
jroll | we should be focusing on ocata priorities right now | 17:19 |
lucasagomes | have a great evening! | 17:19 |
*** lucasagomes is now known as lucas-afk | 17:19 | |
jroll | later lucas-afk \o | 17:19 |
joanna | JayF: thanks, I know that page - the priority thing was a citation of you, yesterday :D | 17:19 |
lucas-afk | later! | 17:19 |
* lucas-afk runs to the train station | 17:19 | |
mgould | o/ | 17:19 |
JayF | joanna: https://github.com/openstack/governance/blob/master/goals/pike/deploy-api-in-wsgi.rst | 17:20 |
*** persia has joined #openstack-ironic | 17:20 | |
* dtantsur also goes | 17:21 | |
dtantsur | see you tomorrow | 17:21 |
JayF | joanna: properly rendered link: https://governance.openstack.org/tc/goals/pike/deploy-api-in-wsgi.html | 17:21 |
*** dtantsur is now known as dtantsur|afk | 17:21 | |
JayF | o/ have a good day | 17:21 |
mgould | o/ | 17:21 |
joanna | thanks! | 17:21 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/425670 | 17:21 |
jroll | see ya, dtantsur|afk | 17:21 |
*** derekh has quit IRC | 17:24 | |
*** sayantani01 has joined #openstack-ironic | 17:24 | |
rloo | jroll: haven't yet looked at 2nd api patch (lunching) | 17:25 |
openstackgerrit | Merged openstack/ironic: Update iRMC driver doc for soft reboot and soft power off https://review.openstack.org/425541 | 17:26 |
jroll | rloo: ok cool, updating the first then | 17:29 |
jroll | thanks | 17:29 |
openstackgerrit | Sam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 17:31 |
*** aNuposic has quit IRC | 17:34 | |
*** Nisha_Agarwal has quit IRC | 17:35 | |
openstackgerrit | Leif Madsen proposed openstack/bifrost: Make path to ironic-rootwrap dynamic https://review.openstack.org/422856 | 17:38 |
*** mgould is now known as mgould|afk | 17:39 | |
*** baoli has joined #openstack-ironic | 17:41 | |
openstackgerrit | Merged openstack/ironic-inspector: Devstack: don't rely on Ironic local vars https://review.openstack.org/420725 | 17:47 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add dynamic interfaces fields to nodes API https://review.openstack.org/424720 | 17:50 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add dynamic driver functionality to REST API https://review.openstack.org/417970 | 17:50 |
openstackgerrit | Aparna proposed openstack/proliantutils: Add HPSUM firmware update support https://review.openstack.org/422004 | 17:50 |
jroll | rloo: vdrok: I think I got everything & | 17:50 |
jroll | s/&/^/ | 17:50 |
rloo | jroll: will look in a few min | 17:51 |
*** trown is now known as trown|lunch | 17:51 | |
jroll | yeah no rush :) | 17:51 |
* sambetts -> home | 17:51 | |
sambetts | night all | 17:51 |
jroll | \o | 17:51 |
sambetts | cya jroll | 17:51 |
*** sambetts is now known as sambetts|afk | 17:51 | |
jlvillal | Why does ironic/db/sqlalchemy/api.py appear to have no unit tests? I'm just wondering. | 17:56 |
jroll | it does, they're not named the same as that file | 17:57 |
jroll | I did a wtf about that recently too | 17:58 |
jroll | jlvillal: they're here: https://github.com/openstack/ironic/tree/master/ironic/tests/unit/db | 17:58 |
jlvillal | jroll: Ah and :( | 17:58 |
jroll | I do like that they're split but wonder if they should be in that sqlalchemy directory | 17:58 |
*** david-lyle has quit IRC | 17:59 | |
* TheJulia gets a small sign that says "Certified Zombie" and hangs it on herself | 17:59 | |
jlvillal | jroll: Okay. Yeah I was looking at a function like: add_port_filter_by_node() and only found it in api.py. | 17:59 |
jlvillal | jroll: I'm guessing we may not have total coverage. | 18:00 |
jlvillal | Thanks | 18:00 |
jroll | jlvillal: we might not | 18:00 |
jroll | I suspect that gets tested where it's used, given it isn't used outside of api.py | 18:01 |
jroll | ¯\_(ツ)_/¯ | 18:01 |
jlvillal | jroll: Yeah, I would prefer the tests to be in ironic/tests/unit/db/sqlalchemy/api/* for the tests if it has been split into multiple files | 18:02 |
*** aNuposic has joined #openstack-ironic | 18:02 | |
*** mgoddard has quit IRC | 18:02 | |
* jlvillal just though of that idea to create a directory based on the file name. Pure genius | 18:02 | |
* jlvillal strains should trying to pat himself on the back :) | 18:02 | |
jlvillal | s/should/shoulder/ | 18:02 |
jroll | jlvillal: yeah, though I think it's a distraction right now when we should be focusing on priorities | 18:02 |
rloo | jlvillal: ++. historical. not enough people. blah. blah :-( and what jroll sez. | 18:03 |
*** ralonsoh has quit IRC | 18:03 | |
jlvillal | +1 | 18:04 |
* jlvillal got distracted while reviewing a patch set | 18:04 | |
TheJulia | jroll: Reviewed the project config change. Looks like the testing doesn't like it though :\ | 18:05 |
jroll | TheJulia: oh, whomp, I'll take a look. thanks. | 18:05 |
jlvillal | Keep project-config weird, don't normalize it! :) | 18:06 |
TheJulia | lol | 18:06 |
* TheJulia clearly needs less blood in her coffee distribution system this morning. | 18:07 | |
jroll | TheJulia: I can add bifrost-core to bifrost stable maintenance, it was unclear what was going on there to me | 18:07 |
TheJulia | jroll: Up to you. It is really rare for anything to have to be landed to the stable branches, aside from the normal required stuff | 18:09 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/425670 | 18:09 |
jroll | TheJulia: okay, I don't mind adding it back | 18:09 |
TheJulia | Cool | 18:09 |
openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/425671 | 18:10 |
openstackgerrit | Merged openstack/ironic-inspector: Deprecate setting IPMI credentials https://review.openstack.org/417041 | 18:10 |
TheJulia | psutil again... | 18:10 |
JayF | TheJulia: what is psutil doing/ | 18:11 |
JayF | oh | 18:11 |
JayF | oh that's a beaut | 18:11 |
TheJulia | indeed | 18:11 |
JayF | will make my psutil patch a lot easier | 18:11 |
JayF | I'm glad they merged that | 18:11 |
TheJulia | I think housekeeping wants me to checkout. :\ | 18:12 |
jlvillal | rloo: On the python-ironicclient queue. https://review.openstack.org/#/q/status:open+project:openstack/python-ironicclient | 18:13 |
jlvillal | rloo: Any particular ones to focus on? | 18:13 |
jroll | jlvillal: I should cross those off the whiteboard, we released it yesterday | 18:13 |
rloo | jlvillal: nope, not any more. i mean, the client release was cut yesterday | 18:13 |
rloo | jlvillal: so focus on ironic features | 18:14 |
rloo | jlvillal: driver composition | 18:14 |
jlvillal | Okay. | 18:14 |
rloo | jlvillal: welcome to the club :) | 18:14 |
jroll | driver composition and node tags are the big ones right now | 18:14 |
rloo | jroll: node tags didn't make the client cut but i guess it would be good to get the ironic part in. | 18:14 |
rloo | jroll: after we get the driver comp stuff in :) | 18:15 |
jroll | rloo: yep, agree | 18:15 |
*** crushil has joined #openstack-ironic | 18:15 | |
* jroll wonders if I should -2 node tags API until driver comp lands, just in case | 18:15 | |
rloo | jroll: bet you it won't matter! | 18:15 |
jroll | :) | 18:16 |
JayF | does LOG.exception just log the exception, if used in an except block, or does it reraise the exception too? | 18:19 |
* JayF can't find this in any oslo.log docs | 18:19 | |
jroll | the former | 18:19 |
jroll | and it logs the entire traceback | 18:19 |
jroll | at ERROR level | 18:19 |
JayF | perfect, that's exactly what I want in this case, I thikn | 18:19 |
rloo | jroll: geez. since we introduce hw types in 1.30, are we allowing operations on them prior to 1.30, like getting RAID disk properties or vendor-interface-related stuff? | 18:20 |
jroll | LOG.exception() == LOG.error() + traceback.print_exc() or so | 18:20 |
jroll | rloo: yes, since one can set node.driver to a hw type in any API version | 18:20 |
rloo | jroll: which reminds, me i didn't even check the patches against the spec. | 18:20 |
rloo | jroll: Ok. | 18:20 |
jroll | rloo: also why we return hardware types in GET /v1/drivers for all versions | 18:20 |
jroll | rloo: good thing you didn't, the spec forgot raid :) | 18:20 |
*** mrtenio has quit IRC | 18:20 | |
rloo | jroll: hard to remember to include everything in the spec. review, review, review, that's the secret :-( | 18:21 |
jroll | hmm, I should probably lunch before meetings | 18:21 |
jroll | rloo: yep, it's hard | 18:21 |
* jroll bbl | 18:21 | |
*** mrtenio has joined #openstack-ironic | 18:22 | |
*** xek_ has joined #openstack-ironic | 18:22 | |
*** xek has quit IRC | 18:22 | |
*** moshele has joined #openstack-ironic | 18:24 | |
rloo | before i forget, jroll: is there an api-ref patch for driver composition? | 18:26 |
*** jtomasek has joined #openstack-ironic | 18:26 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Remove support for psutil <3 https://review.openstack.org/425284 | 18:27 |
*** dprince has joined #openstack-ironic | 18:28 | |
openstackgerrit | Aparna proposed openstack/proliantutils: Add HPSUM firmware update support https://review.openstack.org/422004 | 18:31 |
*** vinsh has joined #openstack-ironic | 18:32 | |
*** rcernin has joined #openstack-ironic | 18:32 | |
*** amotoki has quit IRC | 18:33 | |
*** dsneddon_afk is now known as dsneddon | 18:35 | |
openstackgerrit | Aparna proposed openstack/proliantutils: Add return code for hpsum firmware update https://review.openstack.org/423856 | 18:37 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/425670 | 18:44 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/425826 | 18:44 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui: Updated from global requirements https://review.openstack.org/423272 | 18:44 |
jroll | rloo: not yet, no | 18:48 |
*** Goneri has quit IRC | 18:48 | |
rloo | jroll: on your list though, right? :) | 18:48 |
openstackgerrit | Merged openstack/bifrost: Typo fix: insufficent = insufficient https://review.openstack.org/425053 | 18:48 |
jroll | rloo: the ever-growing list, yes | 18:48 |
rloo | jroll: what? should be shrinking. | 18:49 |
*** jtomasek has quit IRC | 18:49 | |
*** moshele has quit IRC | 18:49 | |
*** crushil has quit IRC | 18:49 | |
jroll | rloo: always growing! | 18:49 |
jroll | TheJulia: updated the thing https://review.openstack.org/#/c/425777/ | 18:50 |
patchbot | patch 425777 - openstack-infra/project-config - Clean up ACLs for some ironic projects | 18:50 |
TheJulia | jroll: Thank you | 18:50 |
*** ijw has joined #openstack-ironic | 18:51 | |
jroll | np | 18:51 |
* TheJulia goes and finds a taxi | 18:52 | |
*** trown|lunch is now known as trown | 18:54 | |
joanna | <low priority warning> I'm thinking - there are certain values for e.g. boot_interface field od node, right? Why we don't use enums in the db for such cases? | 18:55 |
jroll | joanna: primary reason is, folks can write their own implementations for any interface, and install those into ironic, so we don't know all the values | 18:56 |
joanna | jroll: thanks! :) makes sense | 18:56 |
JayF | joanna: a couple of reasons I can think of, you can have 3rd party implementations, and do we want a DB migration anytime we add a new interface | 18:56 |
jroll | we have a db migration for that anyway :P | 18:57 |
rloo | jroll: you have my +2 on the two driver comp patches; you should be able to get +A by tomorrow at the latest I think. | 18:58 |
joanna | JayF: thanks :) | 18:58 |
jroll | rloo: nice, thanks! | 18:59 |
*** harlowja has quit IRC | 19:02 | |
*** jtomasek has joined #openstack-ironic | 19:02 | |
mariojv | i think JayF might have meant interface implementation | 19:04 |
mariojv | wrt the db migration | 19:04 |
*** derekh has joined #openstack-ironic | 19:05 | |
*** crushil has joined #openstack-ironic | 19:05 | |
*** alexpilotti has joined #openstack-ironic | 19:07 | |
jroll | mmm | 19:08 |
TheJulia | Re: db enum, I seem to remember there is/was an issue with sqlalchemy and using enum when we run db related unit tests without a real database. | 19:08 |
*** alexpilotti has quit IRC | 19:08 | |
*** alexpilotti has joined #openstack-ironic | 19:09 | |
openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 19:12 |
*** alexpilotti has quit IRC | 19:13 | |
*** Sukhdev has joined #openstack-ironic | 19:16 | |
*** yolanda has quit IRC | 19:17 | |
*** absubram has joined #openstack-ironic | 19:18 | |
*** jtomasek has quit IRC | 19:20 | |
openstackgerrit | Merged openstack/ironic: Correctly cache "abortable" flag for manual clean steps https://review.openstack.org/423235 | 19:20 |
*** jtomasek has joined #openstack-ironic | 19:23 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Remove support for psutil <3 https://review.openstack.org/425284 | 19:25 |
*** derekh has quit IRC | 19:27 | |
JayF | TheJulia: ^ pep8 fixes :-C | 19:28 |
openstackgerrit | Merged openstack/python-ironicclient: Update reno for stable/ocata https://review.openstack.org/425477 | 19:30 |
TheJulia | :( | 19:30 |
TheJulia | I feel slightly better that I didn't spot it now that I've looked at it again :) | 19:32 |
*** jjohnson2 has quit IRC | 19:33 | |
*** amotoki has joined #openstack-ironic | 19:34 | |
*** yolanda has joined #openstack-ironic | 19:34 | |
*** jheroux has quit IRC | 19:35 | |
*** jkilpatr has quit IRC | 19:35 | |
*** crushil has quit IRC | 19:39 | |
*** Sukhdev_ has joined #openstack-ironic | 19:44 | |
*** david-lyle has joined #openstack-ironic | 19:45 | |
*** mgoddard has joined #openstack-ironic | 19:47 | |
*** sayantani01 has quit IRC | 19:51 | |
*** jkilpatr has joined #openstack-ironic | 19:51 | |
*** jheroux has joined #openstack-ironic | 19:53 | |
openstackgerrit | Chris Krelle proposed openstack/bifrost: remove pxe_amt driver from pxe enable drivers list https://review.openstack.org/425871 | 19:54 |
*** mtanino has quit IRC | 19:55 | |
jlvillal | jroll: I'll do some more reviewing of the driver composition patch after lunch. Slow going... | 19:55 |
jlvillal | A lot of code! | 19:55 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up nits on driver composition API changes https://review.openstack.org/425873 | 19:56 |
jroll | jlvillal: would prefer a single review, but ^ should fix everything so far | 19:56 |
*** Guest33539 is now known as mgagne | 20:01 | |
*** mgagne has quit IRC | 20:01 | |
*** mgagne has joined #openstack-ironic | 20:01 | |
*** harlowja has joined #openstack-ironic | 20:09 | |
*** Goneri has joined #openstack-ironic | 20:11 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Find a node by multiple attributes https://review.openstack.org/421285 | 20:12 |
*** mgoddard has quit IRC | 20:15 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Find a node by multiple attributes https://review.openstack.org/421285 | 20:15 |
*** jtomasek has quit IRC | 20:16 | |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Use same variable for --[no-]maintenance https://review.openstack.org/425889 | 20:21 |
*** dyasny has quit IRC | 20:22 | |
*** aNuposic has quit IRC | 20:28 | |
rloo | jroll: the resource classes -- do we know for sure that it will be used by nova in pike? | 20:30 |
rloo | jroll: 'it' meaning ironic's resource_class :) | 20:31 |
*** sayantani01 has joined #openstack-ironic | 20:31 | |
jroll | rloo: I am told this is the goal | 20:33 |
rloo | jroll: ok | 20:33 |
jroll | I cannot predict the future | 20:33 |
jroll | keep in mind docs can be edited | 20:34 |
rloo | jroll: yup, i'm keeping that in mind (and glad the doc didn't go into newton) | 20:34 |
jroll | I don't think it would have hurt in newton, but ¯\_(ツ)_/¯ | 20:34 |
rloo | jroll: too far into the future :) | 20:36 |
jroll | rloo: OTOH, deployment projects would have had plenty of time to implement it | 20:37 |
rloo | jroll: perhaps. but what if i implemented it based on the description, and then later decide that the resource classes i created/named didn't make any sense cuz it became (more) clearer later, how they were going to be used. | 20:38 |
rloo | jroll: or what if that feature never happened... | 20:38 |
*** milan has joined #openstack-ironic | 20:38 | |
jroll | rloo: if this stuff doesn't happen I'm quitting | 20:39 |
* rloo hopes no one saw that | 20:39 | |
*** srobert_ has joined #openstack-ironic | 20:39 | |
milan | morning Ironic! :) | 20:40 |
jroll | morning milan | 20:40 |
milan | morning jroll! :) | 20:41 |
*** dyasny has joined #openstack-ironic | 20:41 | |
*** srobert has quit IRC | 20:42 | |
*** aNuposic has joined #openstack-ironic | 20:43 | |
joanna | morning :) | 20:43 |
*** e0ne has joined #openstack-ironic | 20:44 | |
*** srobert_ has quit IRC | 20:48 | |
milan | morning joanna! :) | 20:49 |
*** srobert has joined #openstack-ironic | 20:49 | |
NobodyCam | morning Joanna :) | 20:51 |
joanna | :) | 20:51 |
milan | morning NobodyCam! :) | 20:52 |
NobodyCam | morning milan :) | 20:53 |
*** e0ne has quit IRC | 20:54 | |
*** srart has quit IRC | 20:54 | |
*** e0ne has joined #openstack-ironic | 20:58 | |
*** hecliunyx has quit IRC | 20:59 | |
*** ijw has quit IRC | 20:59 | |
*** ijw has joined #openstack-ironic | 20:59 | |
*** jcoufal has quit IRC | 21:03 | |
*** jcoufal has joined #openstack-ironic | 21:04 | |
*** srart has joined #openstack-ironic | 21:06 | |
*** e0ne has quit IRC | 21:09 | |
* jroll is done for the day, see y'all tomorrow | 21:09 | |
rloo | have a good evening (and dinner) jroll! | 21:10 |
milan | night jroll! :) | 21:12 |
milan | and hi rloo! :) | 21:13 |
*** moshele has joined #openstack-ironic | 21:14 | |
*** crushil has joined #openstack-ironic | 21:14 | |
*** jcoufal has quit IRC | 21:14 | |
*** jcoufal has joined #openstack-ironic | 21:18 | |
*** moshele has quit IRC | 21:19 | |
*** mgoddard has joined #openstack-ironic | 21:30 | |
*** jcoufal has quit IRC | 21:30 | |
*** sacharya has quit IRC | 21:34 | |
*** jkilpatr has quit IRC | 21:35 | |
*** aNuposic has quit IRC | 21:36 | |
*** vsaienko has joined #openstack-ironic | 21:40 | |
*** Sukhdev has quit IRC | 21:41 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests https://review.openstack.org/423556 | 21:42 |
rloo | milan: what are you doing here so late? | 21:43 |
milan | :) | 21:44 |
* milan plans to go to devconf tomorrow O:-) | 21:44 | |
milan | for a bit | 21:44 |
rloo | milan: enjoy! :) | 21:47 |
*** ppiela has quit IRC | 21:48 | |
milan | rloo, thanks! :) | 21:49 |
*** ppiela has joined #openstack-ironic | 21:50 | |
*** milan has quit IRC | 21:51 | |
*** ijw has quit IRC | 21:52 | |
*** moshele has joined #openstack-ironic | 21:52 | |
*** mgoddard has quit IRC | 21:54 | |
openstackgerrit | Joanna Taryma proposed openstack/ironic: Fixed allowing non-UUID value as UUID for port and portgroup patches. https://review.openstack.org/403910 | 21:55 |
*** milan has joined #openstack-ironic | 21:56 | |
*** moshele has quit IRC | 21:57 | |
*** aNuposic has joined #openstack-ironic | 22:00 | |
*** trown is now known as trown|outtypewww | 22:02 | |
*** jheroux has quit IRC | 22:03 | |
openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 22:03 |
mariojv | JayF: does ipa need to update from global requirements before https://review.openstack.org/#/c/425284/ lands ? | 22:04 |
patchbot | patch 425284 - ironic-python-agent - Remove support for psutil <3 | 22:04 |
mariojv | nvm | 22:04 |
mariojv | already there | 22:04 |
JayF | mariojv: https://github.com/openstack/ironic-python-agent/commit/7ec1f254751e2953ffd1cb132a3a661a4440ed8c | 22:04 |
mariojv | https://github.com/openstack/ironic-python-agent/blob/master/requirements.txt#L17 | 22:04 |
mariojv | yup, thanks | 22:04 |
JayF | that was the catalyst to rewrite that patch | 22:04 |
mariojv | i was thinking about the source from yesterday | 22:05 |
mariojv | cool | 22:05 |
JayF | going to try and respond to comments real quick before vanessa calls | 22:05 |
*** milan has quit IRC | 22:06 | |
*** priteau has quit IRC | 22:09 | |
*** mtanino has joined #openstack-ironic | 22:09 | |
*** zackf has quit IRC | 22:09 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests https://review.openstack.org/423556 | 22:11 |
*** ijw has joined #openstack-ironic | 22:12 | |
*** jkilpatr has joined #openstack-ironic | 22:13 | |
csmart | \o morning | 22:14 |
JayF | csmart: you're the smart man mrda has been telling me about | 22:14 |
csmart | JayF: smart by name only... ;-) | 22:15 |
*** dprince has quit IRC | 22:15 | |
*** rama_y has quit IRC | 22:15 | |
csmart | JayF: I think we have a meeting scheduled for next week, but happy to chat here any time | 22:16 |
csmart | I'm in UT+11 | 22:16 |
csmart | *UTC+11 | 22:16 |
* csmart is still getting settled in (first week) | 22:16 | |
*** milan has joined #openstack-ironic | 22:16 | |
*** ijw has quit IRC | 22:17 | |
JayF | csmart: so I heard. Welcome to the group, and we'll get to know each other better next week :) | 22:18 |
*** ijw has joined #openstack-ironic | 22:18 | |
*** ijw has quit IRC | 22:18 | |
*** ijw has joined #openstack-ironic | 22:19 | |
openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 22:19 |
csmart | JayF: rgr, thanks. sounds good | 22:20 |
*** cdearborn has joined #openstack-ironic | 22:20 | |
* JayF introduces csmart to the channel: Chris is a new Racker who is going to be working in one of our cloud product groups. He has some experience with embedded linux so I want to nerd sni^W^W ask him nicely to help with our agent image stuff :) | 22:21 | |
*** sayantani01 has quit IRC | 22:21 | |
*** milan has quit IRC | 22:21 | |
*** sayantani01 has joined #openstack-ironic | 22:22 | |
* csmart waves to everyone, thanks JayF, looking forward to getting my teeth into something | 22:23 | |
*** sacharya has joined #openstack-ironic | 22:23 | |
JayF | csmart: was that intentional? use of the word teeth? | 22:23 |
csmart | JayF: yeah, I assumed that would translate, just means to get deep into something | 22:24 |
csmart | :-S | 22:24 |
JayF | csmart: Ot' | 22:24 |
JayF | csmart: the team that originally launched OnMetal at Rackspace (powered by Ironic) and started our whole involvement in the project was the "Teeth Team" (see: https://journal.paul.querna.org/articles/2014/07/02/putting-teeth-in-our-public-cloud/) | 22:25 |
JayF | really the only remnants of that name is the internal irc channel at rackspace for onmetal and this awesome logo https://journal.paul.querna.org/assets/posts/putting-teeth-in-our-public-cloud/teeth-logo.png lol | 22:26 |
openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 22:27 |
*** baoli has quit IRC | 22:28 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Remove support for older psutil versions https://review.openstack.org/425284 | 22:29 |
JayF | mariojv: jlvillal rloo soliosg ^ your comments have been responded to and/or fixed. TYVM for the reviews. | 22:29 |
csmart | JayF: ahh, gotchat ;-) | 22:29 |
soliosg | csmart: welcome aboard! | 22:30 |
*** aNuposic has quit IRC | 22:31 | |
soliosg | csmart: i'm sure you'll soon find something to work on | 22:32 |
mariojv | thx | 22:32 |
csmart | soliosg: thanks :-) | 22:34 |
*** milan has joined #openstack-ironic | 22:35 | |
*** [1]cdearborn has quit IRC | 22:35 | |
*** vsaienko has quit IRC | 22:37 | |
*** crushil has quit IRC | 22:39 | |
*** hecliunyx has joined #openstack-ironic | 22:44 | |
jlvillal | JayF: +2 | 22:44 |
*** sacharya has quit IRC | 22:46 | |
*** aNuposic has joined #openstack-ironic | 22:51 | |
*** sayantani01 has quit IRC | 22:55 | |
*** sayantani01 has joined #openstack-ironic | 22:56 | |
*** mgoddard has joined #openstack-ironic | 22:58 | |
*** srobert_ has joined #openstack-ironic | 23:03 | |
*** skazi has quit IRC | 23:06 | |
*** srobert has quit IRC | 23:06 | |
*** srobert_ has quit IRC | 23:07 | |
*** absubram has quit IRC | 23:13 | |
*** pmannidi has joined #openstack-ironic | 23:14 | |
*** skazi has joined #openstack-ironic | 23:16 | |
*** mgoddard has quit IRC | 23:17 | |
Sukhdev_ | jlvillal : Hey John - are you there? | 23:24 |
jlvillal | Sukhdev_: I am | 23:24 |
Sukhdev_ | jlvillal : cool - have a QQ - see if you can help | 23:25 |
jlvillal | I'll try :) | 23:25 |
*** skazi has quit IRC | 23:25 | |
* jlvillal hopes it isn't a you work at Intel so you must know about this Intel hardware question :) | 23:25 | |
Sukhdev_ | jlvillal : no nothing of that sort - strictly ironic related | 23:26 |
jlvillal | :) | 23:26 |
Sukhdev_ | jlvillal : I have an Ironic setup (created by devstack), which is all working fine - I have tested using the VM acting a BMs all on a single server | 23:27 |
jlvillal | okay | 23:27 |
Sukhdev_ | jlvillal: now I want to switch to the real HW - I have wired everything up - i.e. a switch, BM machine, etc and connected the controller running Ironic conductor to the switch as well | 23:27 |
jlvillal | okay | 23:28 |
Sukhdev_ | jlvillal: instead of booting the VMs (acting as BMs), want boot the real HW BM server | 23:28 |
Sukhdev_ | jlvillal : so the question is - | 23:28 |
Sukhdev_ | jlvillal : If I update the port created by devstack to change the mac address to the real BM's mac address | 23:29 |
Sukhdev_ | will that work? | 23:29 |
jlvillal | Sukhdev_: Hmm. I'm not sure. | 23:30 |
* jlvillal hopes someone else will chime in too | 23:30 | |
jlvillal | Sukhdev_: What is your deploy driver? | 23:30 |
Sukhdev_ | jlvillal : one second - let me past it | 23:31 |
Sukhdev_ | jlvillal : here is the paste - http://paste.openstack.org/show/596653/ | 23:32 |
jlvillal | Sukhdev_: So your bare-metal node. How is it going to be controlled? IPMI? | 23:33 |
Sukhdev_ | jlvillal : yes | 23:33 |
jlvillal | Sukhdev_: So probably want to change the deploy driver from 'agent_ssh' to 'agent_ipmitool'. I think. | 23:34 |
*** skazi has joined #openstack-ironic | 23:34 | |
jlvillal | Sukhdev_: I am supposed to get my own IPMI box to use for Ironic testing in a few weeks. So I haven't yet set it up in real life. | 23:34 |
Sukhdev_ | jlvillal : :-) | 23:34 |
Sukhdev_ | jlvillal : but, Ironic conductor will need to communicate with IPA over ssh, no? | 23:35 |
Sukhdev_ | jlvillal : so, I thought both will be needed? | 23:35 |
jlvillal | Sukhdev_: Instead of a devstack way. You might consider using Bifrost. I think TheJulia would recommend that. | 23:35 |
Sukhdev_ | jlvillal : do not what is Bifrost way | 23:36 |
jlvillal | Sukhdev_: agent_ssh means use the agent. But control the power via SSH. | 23:37 |
* TheJulia reads | 23:37 | |
aslezil | attaching real hardware to devstack is going to be hard. Devstack has its own limitation | 23:37 |
Sukhdev_ | jlvillal : I thought as long as I have all the nodes created and the ramdisk image created by devstack, I can use all of that as-is and point the mac address and IPMI tool to the correct address, it should do the trick | 23:38 |
jlvillal | https://github.com/openstack/ironic/blob/master/ironic/drivers/agent.py#L64-L70 | 23:38 |
jlvillal | Sukhdev_: The agent_ssh driver doesn't use IPMI though. | 23:39 |
TheJulia | Devstack has a flag for using it with real hardware though | 23:39 |
Sukhdev_ | jlvillal : I should clarify - this is still test setup, just want to replace the VM with real HW :-) | 23:39 |
* jlvillal has never tried that flag | 23:39 | |
TheJulia | About to head into airport security btw | 23:40 |
jlvillal | Sukhdev_: Maybe look for the flag TheJulia mentioned | 23:40 |
Sukhdev_ | TheJulia : what is that flag called? | 23:40 |
jlvillal | Sukhdev_: I have never tried what you are doing. So I'm unsure. | 23:40 |
TheJulia | jlvillal: two or more of the third party CI systems use it, and it was used at HPE for downstream ionic ci | 23:40 |
TheJulia | I don't remember the flag offhand | 23:40 |
TheJulia | I can look once through security | 23:41 |
jlvillal | Sukhdev_: TheJulia: I think it is: IRONIC_IS_HARDWARE | 23:41 |
*** noama has quit IRC | 23:42 | |
*** bfournie has quit IRC | 23:42 | |
jlvillal | Sukhdev_: Look in openstack/ironic/devstack/lib/ironic | 23:43 |
*** noama has joined #openstack-ironic | 23:43 | |
Sukhdev_ | jlvillal : I see it in that file - so, set it to true in local.conf and re-run the devstack, will that do the trick? | 23:44 |
jlvillal | Sukhdev_: I imagine have to set a few other settings | 23:44 |
Sukhdev_ | jlvillal : I was afraid you will say so :-):-) | 23:45 |
jlvillal | Sukhdev_: https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L63-L68 | 23:45 |
TheJulia | That | 23:46 |
jlvillal | Sukhdev_: https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L1371-L1378 | 23:46 |
jlvillal | Sukhdev_: Good luck! | 23:46 |
Sukhdev_ | jlvillal : thanks - looks like I am going to need a lots of luck for being a brave soul | 23:47 |
Sukhdev_ | TheJulia : if you happen to have a moment later, would appreciate if you can point me to anything else that I need to get this going | 23:48 |
*** noama has quit IRC | 23:48 | |
*** noama has joined #openstack-ironic | 23:49 | |
vgadiraj | Sukhdev_: if this doesn't work or you run into a ton of issues, it may be because I think in devstack the baremetal nodes are wired directly into neutron OVS bridge. | 23:51 |
*** david-lyle has quit IRC | 23:51 | |
Sukhdev_ | vgadiraj : hmm. you may be right about it | 23:52 |
*** jaosorior has quit IRC | 23:53 | |
Sukhdev_ | vgadiraj : what information related to ipmi needs to be set in the ironic config file - do you have any idea? | 23:53 |
*** david-lyle has joined #openstack-ironic | 23:54 | |
*** rcernin has quit IRC | 23:56 | |
*** rcernin has joined #openstack-ironic | 23:57 | |
TheJulia | Okay, through security | 23:58 |
TheJulia | Sukhdev_: no ipmi related dates needs to go into ironic.conf, it all should go into the driver_info | 23:59 |
*** rcernin has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!