| *** baoli has quit IRC | 00:00 | |
| *** rpioso has quit IRC | 00:00 | |
| *** jrist has quit IRC | 00:02 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Testing serial console logs https://review.openstack.org/426965 | 00:16 |
|---|---|---|
| *** csmart has quit IRC | 00:20 | |
| *** bfournie has joined #openstack-ironic | 00:20 | |
| *** bfournie has quit IRC | 00:21 | |
| *** bfournie has joined #openstack-ironic | 00:23 | |
| *** rajinir has quit IRC | 00:26 | |
| *** mtanino_ has joined #openstack-ironic | 00:29 | |
| *** mtanino has quit IRC | 00:30 | |
| *** MattMan has quit IRC | 00:31 | |
| *** MattMan has joined #openstack-ironic | 00:32 | |
| *** rloo has quit IRC | 00:45 | |
| *** rama_y has quit IRC | 00:59 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Testing serial console logs https://review.openstack.org/426965 | 01:00 |
| *** rama_y_ has quit IRC | 01:00 | |
| *** baoli has joined #openstack-ironic | 01:02 | |
| *** baoli has quit IRC | 01:02 | |
| *** csmart has joined #openstack-ironic | 01:04 | |
| *** liliars_ has joined #openstack-ironic | 01:15 | |
| *** jaosorior has quit IRC | 01:17 | |
| *** aNuposic has quit IRC | 01:24 | |
| *** Syed__ has quit IRC | 01:25 | |
| *** dims has quit IRC | 01:35 | |
| *** dims_ has joined #openstack-ironic | 01:35 | |
| *** crushil has joined #openstack-ironic | 01:36 | |
| *** baoli has joined #openstack-ironic | 01:38 | |
| *** dims_ has quit IRC | 01:42 | |
| *** baoli has quit IRC | 01:42 | |
| *** dims has joined #openstack-ironic | 01:42 | |
| *** ijw has quit IRC | 02:08 | |
| openstackgerrit | Merged openstack/ironic-inspector: Remove heading "Team and repository tags" https://review.openstack.org/425700 | 02:12 |
| *** wrouesnel has quit IRC | 02:13 | |
| *** wrouesnel has joined #openstack-ironic | 02:13 | |
| *** mtanino has joined #openstack-ironic | 02:17 | |
| *** dyasny has quit IRC | 02:18 | |
| *** mtanino_ has quit IRC | 02:19 | |
| *** baoli has joined #openstack-ironic | 02:20 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add soft reboot, soft power off and power timeout to api-ref https://review.openstack.org/423986 | 02:25 |
| *** dyasny has joined #openstack-ironic | 02:27 | |
| *** Sukhdev has quit IRC | 02:27 | |
| *** absubram has quit IRC | 02:27 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add supported_power_states field to api-ref https://review.openstack.org/427000 | 02:29 |
| *** wrouesnel has quit IRC | 02:31 | |
| *** wrouesnel has joined #openstack-ironic | 02:31 | |
| *** mtanino has quit IRC | 02:34 | |
| *** wrouesnel has quit IRC | 02:37 | |
| *** wrouesnel has joined #openstack-ironic | 02:37 | |
| *** awiddersheim has quit IRC | 02:39 | |
| *** awiddersheim has joined #openstack-ironic | 02:39 | |
| *** rama_y has joined #openstack-ironic | 02:43 | |
| *** liliars_ has quit IRC | 02:43 | |
| *** sasha2 has quit IRC | 02:44 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Testing serial console logs https://review.openstack.org/426965 | 02:47 |
| *** wrouesnel has quit IRC | 02:57 | |
| *** wrouesnel has joined #openstack-ironic | 02:57 | |
| *** ijw has joined #openstack-ironic | 03:00 | |
| *** ijw has quit IRC | 03:04 | |
| *** wrouesnel has quit IRC | 03:08 | |
| *** wrouesnel has joined #openstack-ironic | 03:08 | |
| *** zzzeek has quit IRC | 03:13 | |
| *** rama_y has quit IRC | 03:14 | |
| *** zzzeek has joined #openstack-ironic | 03:15 | |
| *** wrouesnel has quit IRC | 03:19 | |
| *** wrouesnel has joined #openstack-ironic | 03:19 | |
| *** alexpilotti has joined #openstack-ironic | 03:22 | |
| *** wrouesnel has quit IRC | 03:25 | |
| *** wrouesnel has joined #openstack-ironic | 03:25 | |
| *** alexpilotti has quit IRC | 03:26 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Testing serial console logs https://review.openstack.org/426965 | 03:28 |
| *** amotoki has joined #openstack-ironic | 03:30 | |
| *** ijw has joined #openstack-ironic | 03:31 | |
| *** ijw has quit IRC | 03:31 | |
| *** ijw has joined #openstack-ironic | 03:31 | |
| *** ijw has quit IRC | 03:32 | |
| *** mtanino has joined #openstack-ironic | 03:35 | |
| *** wrouesnel has quit IRC | 03:37 | |
| *** wrouesnel has joined #openstack-ironic | 03:37 | |
| *** vikrant has joined #openstack-ironic | 03:37 | |
| *** sdake has quit IRC | 04:00 | |
| *** rbudden has quit IRC | 04:03 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:05 | |
| *** causten has joined #openstack-ironic | 04:13 | |
| *** causten has quit IRC | 04:13 | |
| *** mtanino has quit IRC | 04:19 | |
| *** sdake has joined #openstack-ironic | 04:23 | |
| *** sdake_ has joined #openstack-ironic | 04:26 | |
| *** sdake has quit IRC | 04:28 | |
| *** nmathew has joined #openstack-ironic | 04:34 | |
| *** baoli has quit IRC | 04:36 | |
| *** zackf has quit IRC | 04:39 | |
| *** aparnav has joined #openstack-ironic | 04:46 | |
| *** udesale has joined #openstack-ironic | 04:48 | |
| *** Sukhdev has joined #openstack-ironic | 04:50 | |
| *** rbudden has joined #openstack-ironic | 05:00 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Update the existing APIs due to adding get_supported_power_states https://review.openstack.org/382194 | 05:06 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add supported_power_states field to api-ref https://review.openstack.org/427000 | 05:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Adding a README for ironic-bm-logs directory https://review.openstack.org/426965 | 05:10 |
| *** sdake_ has quit IRC | 05:14 | |
| *** jrist has joined #openstack-ironic | 05:19 | |
| *** jaganathan has joined #openstack-ironic | 05:25 | |
| *** wrouesnel has quit IRC | 05:29 | |
| *** wrouesnel has joined #openstack-ironic | 05:29 | |
| *** wrouesnel has quit IRC | 05:30 | |
| *** wrouesnel has joined #openstack-ironic | 05:30 | |
| *** Nisha_Agarwal has quit IRC | 05:32 | |
| *** rbudden has quit IRC | 05:34 | |
| *** udesale__ has joined #openstack-ironic | 05:51 | |
| openstackgerrit | Aparna proposed openstack/ironic: Support `update_firmware` to hpsum based firmware update https://review.openstack.org/422572 | 05:51 |
| *** udesale has quit IRC | 05:54 | |
| *** zackf has joined #openstack-ironic | 05:59 | |
| *** zackf has quit IRC | 06:04 | |
| *** amotoki has quit IRC | 06:12 | |
| *** amotoki has joined #openstack-ironic | 06:13 | |
| *** amarao has joined #openstack-ironic | 06:17 | |
| *** vsaienko has joined #openstack-ironic | 06:19 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: [Devstack] Add stack user to libvirt group https://review.openstack.org/426664 | 06:21 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Ironic standalone tests https://review.openstack.org/423556 | 06:22 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DNM/DNR: Temp patch to verify ironic standalone tests https://review.openstack.org/426665 | 06:22 |
| *** crushil has quit IRC | 06:28 | |
| *** udesale__ has quit IRC | 06:34 | |
| *** udesale has joined #openstack-ironic | 06:34 | |
| *** vsaienko has quit IRC | 06:39 | |
| *** Sukhdev has quit IRC | 06:49 | |
| *** rcernin has joined #openstack-ironic | 07:12 | |
| *** jtomasek_ has joined #openstack-ironic | 07:26 | |
| *** ChubYann has quit IRC | 07:36 | |
| *** tesseract has joined #openstack-ironic | 07:54 | |
| *** rbartal has joined #openstack-ironic | 08:01 | |
| *** udesale__ has joined #openstack-ironic | 08:06 | |
| *** mjura has joined #openstack-ironic | 08:08 | |
| *** udesale has quit IRC | 08:08 | |
| *** priteau has joined #openstack-ironic | 08:10 | |
| *** rbartal has quit IRC | 08:11 | |
| *** ccamacho has joined #openstack-ironic | 08:12 | |
| *** rbartal has joined #openstack-ironic | 08:13 | |
| openstackgerrit | Hironori Shiina proposed openstack/python-ironicclient: Add volume_connector support https://review.openstack.org/214786 | 08:13 |
| openstackgerrit | Hironori Shiina proposed openstack/python-ironicclient: Add OSC commands for volume connector https://review.openstack.org/427053 | 08:13 |
| *** rbartal has quit IRC | 08:14 | |
| *** rbartal has joined #openstack-ironic | 08:16 | |
| *** mgoddard has joined #openstack-ironic | 08:20 | |
| *** ralonsoh has joined #openstack-ironic | 08:23 | |
| *** vsaienko has joined #openstack-ironic | 08:30 | |
| *** foexle has joined #openstack-ironic | 08:47 | |
| *** nmathew- has joined #openstack-ironic | 08:49 | |
| *** nmathew has quit IRC | 08:50 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: ETAGs maintenance for all api node's requests https://review.openstack.org/392213 | 08:52 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV https://review.openstack.org/397126 | 08:55 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV https://review.openstack.org/397126 | 08:56 |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:01 | |
| *** athomas has joined #openstack-ironic | 09:02 | |
| *** mgoddard has quit IRC | 09:05 | |
| *** vsaienko has quit IRC | 09:09 | |
| *** udesale has joined #openstack-ironic | 09:09 | |
| *** vsaienko has joined #openstack-ironic | 09:09 | |
| *** udesale__ has quit IRC | 09:10 | |
| *** e0ne has joined #openstack-ironic | 09:14 | |
| vsaienk0 | Good morning Ironic'ers! dtantsur, lucasagomes, aarefiev, vdrok, sambetts, milan, mgould! | 09:14 |
| *** rbartal_ has joined #openstack-ironic | 09:15 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Use Ansible to create test VMs https://review.openstack.org/427071 | 09:19 |
| *** udesale__ has joined #openstack-ironic | 09:32 | |
| *** hshiina has joined #openstack-ironic | 09:32 | |
| *** vsaienko has quit IRC | 09:34 | |
| *** udesale has quit IRC | 09:34 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic: Fix directories permission https://review.openstack.org/427077 | 09:41 |
| *** lucas-afk is now known as lucasagomes | 09:44 | |
| lucasagomes | morning all | 09:44 |
| mkrai | Hi lucasagomes | 09:45 |
| mkrai | Morning | 09:45 |
| mkrai | What is the ironic equivalent CLI for 'nova boot' ? | 09:45 |
| *** udesale__ has quit IRC | 09:48 | |
| aarefiev | morning all! | 09:52 |
| *** gcb has joined #openstack-ironic | 09:52 | |
| vdrok | morning vsaienk0 lucasagomes mkrai and aarefiev ! | 09:54 |
| vdrok | mkrai: ironic node-set-provision-state | 09:54 |
| vdrok | though you'll have to do a bunch of stuff manually before doing that | 09:54 |
| lucasagomes | mkrai, vdrok morning | 09:54 |
| lucasagomes | aarefiev, o/ | 09:54 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Make _send_sensors_data concurrent https://review.openstack.org/407429 | 09:54 |
| vdrok | like adding instance_info and creating neutron ports with adding their ids to port extra | 09:54 |
| lucasagomes | mkrai, yeah, are you using ironic stand alone ? | 09:54 |
| aarefiev | morning vsaienk0, vdrok, lucasagomes, mkrai | 09:55 |
| mkrai | lucasagomes: Yes | 09:55 |
| mkrai | vdrok: Is there any doc for it? | 09:56 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Make _send_sensors_data concurrent https://review.openstack.org/407429 | 09:56 |
| vdrok | mkrai: http://docs.openstack.org/project-install-guide/baremetal/draft/standalone.html | 09:58 |
| vdrok | mkrai: that is, if you do ironic completely standalone. do you have glance/neutron? | 09:58 |
| mkrai | vdrok: Yes | 09:59 |
| mkrai | I am trying to use it without Nova | 09:59 |
| mkrai | As far as I know, Nova helps with scheduling only. | 10:00 |
| mkrai | Is there anything more that Nova helps with? | 10:00 |
| vdrok | mkrai: ok, then you don't need to set noauth auth mode and dhcp provider none. but instead you'll have to do neutron port-create and add its id in one of the ports' extra/vif_port_id field before doing set-provision-state | 10:00 |
| vdrok | other stuff in docs should be valid for your case | 10:00 |
| vdrok | maybe we need to have a section on just "no nova" case, not only completely standalone | 10:01 |
| vdrok | mkrai: yeah, nova is for scheduling mostly | 10:01 |
| *** lennyb has quit IRC | 10:02 | |
| *** lennyb has joined #openstack-ironic | 10:03 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-specs: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 10:03 |
| mkrai | Yes I think that will be good | 10:03 |
| mkrai | vdrok: Can I log a bug for it? | 10:04 |
| mkrai | vdrok: Do you get similar questions on "no nova" use case frequently? | 10:05 |
| *** jtomasek_ has quit IRC | 10:06 | |
| *** derekh has joined #openstack-ironic | 10:06 | |
| *** mgoddard has joined #openstack-ironic | 10:09 | |
| vdrok | mkrai: yup, will do a bit later | 10:11 |
| *** zackf has joined #openstack-ironic | 10:12 | |
| *** sambetts|afk is now known as sambetts | 10:12 | |
| sambetts | Morning all | 10:13 |
| mkrai | vdrok: https://bugs.launchpad.net/ironic/+bug/1660577 | 10:14 |
| openstack | Launchpad bug 1660577 in Ironic "Create a "no nova" document for users" [Undecided,New] - Assigned to Madhuri Kumari (madhuri-rai07) | 10:14 |
| mkrai | vdrok: Please have a look whenever you have time | 10:14 |
| mkrai | Thanks! | 10:14 |
| *** zackf has quit IRC | 10:16 | |
| sambetts | vdrok, mkrai: other things that nova does apart from scheduling include generation of the config-drive etc too, the only tools I know of that work for doing that outside of nova is in bifrost | 10:17 |
| vdrok | morning sambetts, oh, right, config drive. good point | 10:18 |
| vdrok | but still, these are like convenience wrappers mostly | 10:19 |
| sambetts | yeah, Ironic can be operated directly, I like to think of Ironic as a hypervisor for baremetal, like libvirt is to VMs. you can use libvirt directly but its a lot more manual than going through nova | 10:20 |
| sambetts | its the same when talking directly to ironic | 10:21 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: ETAGs maintenance for all api node's requests https://review.openstack.org/392213 | 10:21 |
| mat128 | morning | 10:22 |
| mkrai | My question could be very weird. But why aren't Ironic team trying to do that in Ironic itself? | 10:22 |
| mkrai | sambetts: vdrok ^^ | 10:23 |
| vdrok | morning mat128 | 10:26 |
| vdrok | mkrai: what exactly do you mean by "that"? generating configdrive? or creating neutron ports? | 10:26 |
| vdrok | or having a scheduler in ironic? :) | 10:27 |
| vsaienk0 | we have a problem with grenade job, it is unstable now, nova cpu failed to start due to simultaneous restart of apache (keystone) and nova-cpu https://review.openstack.org/#/c/424730/ | 10:27 |
| patchbot | patch 424730 - openstack-dev/grenade - Run the placement service when upgrading to Ocata (MERGED) | 10:27 |
| vsaienk0 | ironic grenade job failed on patch ^^ | 10:28 |
| sambetts | mkrai: IMO (not sure if others agree) as I said before Ironic is like a hypervisor but for baremetal, so that question is like asking why libvirt doesn't have it built in. I think its layering thing, Ironic just provides the resources, what you run inside it is your decision in a layer above that | 10:28 |
| mkrai | vdrok: generating config-drive | 10:30 |
| mkrai | vdrok: and also scheduler which I think is redundant | 10:31 |
| mkrai | So I wanted to get an idea of why Ironic is dependent on Nova | 10:31 |
| mkrai | sambetts: In Ironic case, the virt-driver/technology can be the ipmi/pxe etc tools and above that we provide a way to talk to them and deploy a BM | 10:33 |
| *** alexpilotti has joined #openstack-ironic | 10:42 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: DNM: turn on resource classes https://review.openstack.org/423269 | 10:44 |
| *** alexpilotti has quit IRC | 10:44 | |
| *** alexpilotti has joined #openstack-ironic | 10:45 | |
| *** milan has joined #openstack-ironic | 10:46 | |
| milan | morning Ironic! :) | 10:46 |
| vdrok | morning milan | 10:48 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV https://review.openstack.org/397126 | 10:48 |
| milan | morning vdrok! :) | 10:48 |
| jroll | morning milan, vdrok, everyone else | 10:52 |
| vdrok | morning jroll | 10:52 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Improve conductor driver validation at startup https://review.openstack.org/426213 | 10:53 |
| mat128 | morning jroll | 10:54 |
| jroll | nmi thing merged in nova, woot woot | 10:55 |
| aarefiev | morning milan, jroll, mat128! | 10:58 |
| milan | morning aarefiev, jroll, mat128! :) | 10:58 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Verify grenade fix https://review.openstack.org/427100 | 10:59 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: ETAGs for node in ironicclient https://review.openstack.org/400335 | 10:59 |
| milan | and morning galyna and vsaienk0! :) | 10:59 |
| vsaienk0 | jroll grenade is unstable https://bugs.launchpad.net/devstack/+bug/1660581 should fix https://review.openstack.org/#/c/427098/ | 11:00 |
| openstack | Launchpad bug 1660581 in devstack "nova compute may fail on startup with ironic virt driver" [Undecided,In progress] - Assigned to Vasyl Saienko (vsaienko) | 11:00 |
| patchbot | patch 427098 - openstack-dev/devstack - Ensure that keystone is available when n-cpu starts | 11:00 |
| galyna | morning, milan :) | 11:00 |
| jroll | vsaienk0: thanks | 11:00 |
| *** dmellado has quit IRC | 11:00 | |
| *** nmathew- has quit IRC | 11:01 | |
| *** dmellado has joined #openstack-ironic | 11:03 | |
| *** lhinds|afk is now known as lhinds | 11:10 | |
| *** aparnav has quit IRC | 11:13 | |
| *** ionutbalutoiu has quit IRC | 11:14 | |
| *** ionutbalutoiu has joined #openstack-ironic | 11:16 | |
| *** athomas has quit IRC | 11:16 | |
| *** athomas has joined #openstack-ironic | 11:24 | |
| *** hshiina has quit IRC | 11:25 | |
| *** fellypefca has joined #openstack-ironic | 11:26 | |
| *** dtantsur|afk is now known as dtantsur | 11:29 | |
| sambetts | vsaienk0: I thought we already fixed that once | 11:31 |
| dtantsur | morning ironic, jroll, aarefiev, galyna, milan, sambetts, vsaienk0, vdrok, mat128 and everyone | 11:31 |
| vsaienk0 | sambetts: this is introduced in grenade | 11:31 |
| dtantsur | is it me who's late today or did some of you wake up early? :) | 11:31 |
| vsaienk0 | sambetts: we fixed in in ironic upgrade, but now when placement api enabled it should be fixed in nova | 11:32 |
| sambetts | vsaienk0: ah cool | 11:32 |
| vsaienk0 | sambetts: and then we can remove workarounds from ironic devstack plugin | 11:33 |
| jroll | \o dtantsur, some of both :P | 11:33 |
| aarefiev | morning dtantsur, or both options :) | 11:33 |
| dtantsur | heh | 11:33 |
| aarefiev | heh :) | 11:33 |
| vdrok | morning dtantsur :) | 11:34 |
| jroll | dtantsur: when you get to it, ruby is +2 on the driver comp APIs \o/ | 11:35 |
| sambetts | vsaienk0: oh nice! | 11:35 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: ETAGs maintenance for all api node's requests https://review.openstack.org/392213 | 11:36 |
| dtantsur | jroll, w00t! | 11:36 |
| dtantsur | will check after short snack | 11:36 |
| jroll | cool | 11:36 |
| * jroll needs to go clear snow and such | 11:36 | |
| fellypefca | morning everyone :) | 11:41 |
| sambetts | lucasagomes: I think I have hit a potential issue with your UEFI devstack plugin changes, the edk2-ovmf package doesn't seem to exist in centos7 | 11:41 |
| lucasagomes | sambetts, yeah that's correct | 11:43 |
| lucasagomes | sambetts, but we have a sanity check in the code | 11:43 |
| lucasagomes | we only support ubuntu and fedora for now | 11:43 |
| vsaienk0 | dtantsur do we have a bug to track work related on CI improvements (decrease number of CI jobs) ? | 11:43 |
| sambetts | lucasagomes: where in the code is the sanity check? devstack fails on trying to install the packages before hitting any OS type check | 11:44 |
| sambetts | lucasagomes: trying a completely fresh devstack this morning | 11:44 |
| dtantsur | vsaienk0, I don't remember, likely not | 11:44 |
| lucasagomes | sambetts, https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L425-L434 | 11:45 |
| lucasagomes | L431 | 11:45 |
| vdrok | morning fellypefca | 11:46 |
| sambetts | lucasagomes: ah... problem is I don't have UEFI enabled, but it still trys to install the package and fails because its in here https://github.com/openstack/ironic/blob/master/devstack/files/rpms/ironic#L20 | 11:46 |
| lucasagomes | sambetts, oh, :-( | 11:47 |
| lucasagomes | sambetts, yeah indeed... I will fix it | 11:47 |
| lucasagomes | sorry for that | 11:47 |
| sambetts | lucasagomes: not a problem :) I'm building a devstack to talk to real hardware right now, so I've just removed the line from the rpms file | 11:49 |
| lucasagomes | sambetts, fair enough, I will put up a patch fixing it in a sec... just finshing something first | 11:49 |
| sambetts | lucasagomes: is it possble to put a # dist: fedora or something on that line to make it avoid installing it? not sure how those files work | 11:50 |
| vdrok | sambetts: jroll lucasagomes dtantsur there are some concerns from rloo/JayF on https://review.openstack.org/316149. We did merge one patch already doing that a while ago (commented about that). are we still ok treating it as a bug? | 11:50 |
| patchbot | patch 316149 - ironic - Fix invalid chassis/nodes/chassis_uuid API url | 11:50 |
| lucasagomes | sambetts, exactly | 11:50 |
| lucasagomes | sambetts, like we do at some apt packages | 11:51 |
| sambetts | lucasagomes: Awesome! I guess the gate doesn't have any centos test slaves for us to confirm it works on :/ | 11:51 |
| lucasagomes | sambetts, yeah, nor fedora :-( | 11:51 |
| lucasagomes | vdrok, I iwll look in a sec | 11:55 |
| vdrok | sure, thanks | 11:55 |
| *** ccamacho is now known as ccamacho|lunch | 11:56 | |
| *** jkilpatr has joined #openstack-ironic | 11:56 | |
| *** udesale has joined #openstack-ironic | 12:01 | |
| milan | morning dtantsur fellypefca lucasagomes sambetts and all! :) | 12:02 |
| lucasagomes | milan, morning | 12:02 |
| dtantsur | o/ | 12:03 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Use Ansible to create test VMs https://review.openstack.org/427071 | 12:05 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Remove vendor_passthru mentions from lookup/heartbeat documentation https://review.openstack.org/427126 | 12:06 |
| mat128 | easy one ^ :) | 12:06 |
| *** jtomasek_ has joined #openstack-ironic | 12:08 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 12:12 | |
| *** vsaienko has joined #openstack-ironic | 12:13 | |
| *** sdake has joined #openstack-ironic | 12:16 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Limit memory usage when running qemu-img convert https://review.openstack.org/427134 | 12:19 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Limit memory usage when running qemu-img convert https://review.openstack.org/427135 | 12:19 |
| mat128 | ^ proposing backports to stable branches because it's covered under a CVE https://launchpad.net/bugs/1449062 | 12:20 |
| openstack | Launchpad bug 1449062 in python-oslo.concurrency (Ubuntu Wily) "[OSSA 2016-012] qemu-img calls need to be restricted by ulimit (CVE-2015-5162)" [Medium,Fix committed] | 12:20 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Ironic standalone tests https://review.openstack.org/423556 | 12:20 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DNM/DNR: Temp patch to verify ironic standalone tests https://review.openstack.org/426665 | 12:20 |
| vsaienk0 | dtantsur: opened a bug to track CI refactoring work https://bugs.launchpad.net/ironic/+bug/1660606 | 12:21 |
| openstack | Launchpad bug 1660606 in Ironic "Ironic CI refactoring" [Undecided,New] | 12:21 |
| dtantsur | thanks! | 12:22 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fix invalid chassis/nodes/chassis_uuid API url https://review.openstack.org/316149 | 12:22 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fix all possible node subresources routing collisions https://review.openstack.org/315766 | 12:22 |
| sambetts | vdrok: ^ update to fix your concern with the response codes | 12:22 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Clean up logging related to new state machine https://review.openstack.org/424654 | 12:22 |
| vdrok | sambetts: thanks, looking | 12:22 |
| sambetts | vdrok: I think we might be lacking tests if me changing the response codes like that didn't break anything :/ | 12:23 |
| dtantsur | anyone from iLO team here? I need some help/opinions on https://bugzilla.redhat.com/show_bug.cgi?id=1416622 please | 12:25 |
| openstack | bugzilla.redhat.com bug 1416622 in rhosp-director "Deployment always fails while using shared ILO port." [High,New] - Assigned to athomas | 12:25 |
| * dtantsur thinks of reporting it against proliantutils | 12:27 | |
| *** rbartal__ has joined #openstack-ironic | 12:28 | |
| *** jtomasek_ has quit IRC | 12:28 | |
| dtantsur | https://bugs.launchpad.net/proliantutils/+bug/1660609 | 12:30 |
| openstack | Launchpad bug 1660609 in proliantutils "Ironic loses access to the BMC when iLO shared port is used" [Undecided,New] | 12:30 |
| *** mgould|afk is now known as mgould | 12:30 | |
| vdrok | Nisha_Agarwal: ^^ | 12:31 |
| *** sdake_ has joined #openstack-ironic | 12:31 | |
| *** jtomasek_ has joined #openstack-ironic | 12:32 | |
| *** sdake has quit IRC | 12:34 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack: Only install edk2-ovmf on Fedora https://review.openstack.org/427142 | 12:34 |
| lucasagomes | sambetts, ^ tested locally, works | 12:34 |
| sambetts | lucasagomes: awesome thanks | 12:34 |
| *** vsaienko has quit IRC | 12:36 | |
| *** baoli has joined #openstack-ironic | 12:37 | |
| aarefiev | milan: hey, around? | 12:39 |
| *** Nisha_Agarwal has quit IRC | 12:41 | |
| *** baoli has quit IRC | 12:42 | |
| *** lucasagomes is now known as lucas-hungry | 12:46 | |
| openstackgerrit | Merged openstack/ironic-inspector: [devstack] Stop installing jq, we're not using it in tests any more https://review.openstack.org/425675 | 12:48 |
| openstackgerrit | Merged openstack/ironic-inspector: Update external links in the documentation https://review.openstack.org/425709 | 12:48 |
| *** Nisha_Agarwal has joined #openstack-ironic | 12:54 | |
| Nisha_Agarwal | vdrok, thanks . we will look at it. We never use shared port for deploy. | 12:55 |
| vdrok | thank you! | 12:55 |
| Nisha_Agarwal | the reason is that the ip is assigned to the port used by iLO itself. How wil u assign IP to it when using deploy. for deploy we need to assign IP to the port which already has IP | 12:57 |
| Nisha_Agarwal | I thinki this is the reason for failure. But logs would help if u have any. | 12:57 |
| Nisha_Agarwal | vdrok, ^^^ | 12:57 |
| vdrok | that's more to dtantsur :) | 12:58 |
| Nisha_Agarwal | dtantsur, ^^^ do u have any logs for the failure. My initial thought is that it failed to assign the IP as the shared port must have the IP assigned already. | 12:59 |
| Nisha_Agarwal | dtantsur, and it looks like expected behaviour possibly. | 13:00 |
| *** bfournie has quit IRC | 13:00 | |
| dtantsur | Nisha_Agarwal, thanks! which logs do you have in mind? | 13:01 |
| *** jtomasek_ has quit IRC | 13:01 | |
| *** fellypefca has quit IRC | 13:01 | |
| Nisha_Agarwal | it hangs/timeouts in the end? | 13:01 |
| milan | aarefiev, hi, yup | 13:01 |
| aarefiev | milan: is any reason we don't have transition for timeout in starting state? | 13:02 |
| dtantsur | Nisha_Agarwal, yes, it seems so | 13:02 |
| *** udesale has quit IRC | 13:02 | |
| milan | aarefiev, not sure it's needed, we move to waiting | 13:02 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [devstack] Stop installing jq, we're not using it in tests any more https://review.openstack.org/427192 | 13:02 |
| dtantsur | ditto for newton ^^^ | 13:03 |
| *** udesale has joined #openstack-ironic | 13:03 | |
| milan | aarefiev, we also have unlimited starting -> waiting move | 13:03 |
| milan | to be able to restart | 13:03 |
| milan | aarefiev, is there some issue w/ it? | 13:03 |
| aarefiev | milan: yeah if some error happened before start transition starting -> waiting | 13:03 |
| Nisha_Agarwal | dtantsur, we would check the behavior. But most likely it may be expected when u try to assign IP to shared iLO port as the port is shared by iLO also and it would have its own IP | 13:03 |
| *** trown|outtypewww is now known as trown | 13:04 | |
| milan | aarefiev, that should be starting -> error I guess that's possible, no? | 13:04 |
| aarefiev | milan: it will be always in starting state | 13:04 |
| * milan checks | 13:04 | |
| aarefiev | and I see NodeStateInvalidEvent: Can not transition from state 'starting' on event 'timeout' (no defined transition) | 13:04 |
| *** ccamacho|lunch is now known as ccamacho | 13:05 | |
| dtantsur | Nisha_Agarwal, how does it work btw? is this NIC actually two virtual NICs, one for iLO, one - for the operating system? | 13:05 |
| Nisha_Agarwal | dtantsur, the shared one? | 13:06 |
| milan | aarefiev, https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/introspection_state.py#L127 | 13:06 |
| dtantsur | Nisha_Agarwal, yes | 13:06 |
| milan | error should be possible | 13:06 |
| milan | aarefiev, so the odd thing is why it's still starting and not waiting | 13:06 |
| Nisha_Agarwal | dtantsur, i never tried with shared iLO port...something new for me to try :) | 13:06 |
| Nisha_Agarwal | dtantsur, we always used the free NIC(the embedded one) for deploy | 13:07 |
| dtantsur | oh, I see | 13:07 |
| milan | aarefiev, you've got some logs? | 13:07 |
| aarefiev | milan: yeah it's because another issue | 13:07 |
| aarefiev | heh looks like it should be in error state | 13:07 |
| Nisha_Agarwal | dtantsur, we will get back to u on the bug soon | 13:08 |
| milan | aarefiev, ah I see | 13:08 |
| dtantsur | Nisha_Agarwal, thanks! | 13:08 |
| Nisha_Agarwal | dtantsur, u r welcome :) | 13:08 |
| milan | aarefiev, ok, so some exception wasn't handled the way I imagined it would have been O:-) | 13:08 |
| aarefiev | milan: looks like all exceptions after start_introspection and before _background_introspect_locked will cause this issue | 13:09 |
| * milan checks | 13:10 | |
| milan | aarefiev, I see, you're right | 13:11 |
| milan | aarefiev, there's no top-level transition around the introspect, thus the error transition doesn't happen, and the timeout kicks in eventually | 13:12 |
| aarefiev | milan: I'll file a bug today later | 13:12 |
| milan | aarefiev, please, file a bug I | 13:12 |
| *** vikrant has quit IRC | 13:12 | |
| milan | ack :) | 13:12 |
| milan | aarefiev, thanks for pointing it out! | 13:12 |
| aarefiev | np :) | 13:12 |
| TheJulia | Good morning everyone | 13:13 |
| milan | morning TheJulia! :) | 13:15 |
| Nisha_Agarwal | dtantsur, sambetts could you look at the bug https://bugs.launchpad.net/ironic/+bug/1609622 when u have time , Thank you :) | 13:15 |
| openstack | Launchpad bug 1609622 in Ironic "RFE: Fix iLO drivers to get local_gb for all storages" [Wishlist,In progress] - Assigned to Nisha Agarwal (agarwalnisha1980) | 13:15 |
| dtantsur | Nisha_Agarwal, responded | 13:20 |
| milan | sambetts, https://review.openstack.org/#/c/375045/ O:-) you plan reviewing? | 13:22 |
| patchbot | patch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses | 13:22 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: ETAGs for node in ironicclient https://review.openstack.org/400335 | 13:22 |
| *** awiddersheim has quit IRC | 13:27 | |
| *** jheroux has joined #openstack-ironic | 13:29 | |
| *** bfournie has joined #openstack-ironic | 13:32 | |
| vsaienk0 | dtantsur: once have a minute could you please have look at https://review.openstack.org/#/c/426700 I've resolved your comments | 13:32 |
| patchbot | patch 426700 - openstack-infra/project-config - Update experimental ironic-standalone job | 13:32 |
| dtantsur | k | 13:32 |
| vsaienk0 | dtantsur: thanks! | 13:32 |
| dtantsur | vsaienk0, you have a recursive inclusion of ironic-extra-vars... I believe that you should rollback to using "shell" builder | 13:33 |
| *** rbudden has joined #openstack-ironic | 13:35 | |
| vsaienk0 | dtantsur: and in the shell still left https://review.openstack.org/#/c/426700/2/jenkins/jobs/ironic.yaml@1246 to apply changes from ironic-tempest-plugin and ironic-tempest-regex builders? | 13:35 |
| patchbot | patch 426700 - openstack-infra/project-config - Update experimental ironic-standalone job | 13:35 |
| dtantsur | vsaienk0, yes, I think so. check how devstack-virtual-ironic is implemented | 13:36 |
| TheJulia | NobodyCam: Can you take a look at https://review.openstack.org/#/c/426187/1 I remember you mentioned the same issue to me last week. | 13:37 |
| patchbot | patch 426187 - bifrost - Do not install pxe_amt driver by default | 13:37 |
| vdrok | morning TheJulia | 13:39 |
| vsaienk0 | dtantsur: another question I doubt that set VIRT_DRIVER to '' is right, might be better to fix in ironic devstack https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L649 to something like if is_service_enabled n-cpu && VIRT_DRIVE == 'ironic'; then required_services+='n-cpu' wdyt? | 13:40 |
| dtantsur | hmm.. | 13:40 |
| *** jcoufal has joined #openstack-ironic | 13:41 | |
| dtantsur | I don't have a strong preference here.. | 13:41 |
| *** lucas-hungry is now known as lucasagomes | 13:43 | |
| sambetts | TheJulia: I also hit that issue with the AMT driver the other week, I meant to post a bug whoops | 13:43 |
| TheJulia | doh! | 13:44 |
| *** awiddersheim has joined #openstack-ironic | 13:44 | |
| TheJulia | It happens | 13:44 |
| TheJulia | I barely remember last week with meetings and travel. | 13:44 |
| sambetts | TheJulia: actually that reminds me, I don't know if you've ever experienced this but I've also run into weird issues on centos7 with regards to the ironic root wrap code, I think it was to do with where ironic-rootwrap is installed and the line that is written into the sudoers file | 13:45 |
| TheJulia | I also ended up driving 1800 miles Saturday through Sunday... 8| | 13:45 |
| sambetts | TheJulia: oh wow thats quite a trip! | 13:45 |
| *** jaganathan has quit IRC | 13:46 | |
| TheJulia | sambetts: As in, in sudoers we're hardcoding the path out in sudoers, and the command is elsewhere? | 13:46 |
| *** dprince has joined #openstack-ironic | 13:46 | |
| sambetts | TheJulia: yeah | 13:47 |
| sambetts | TheJulia: bifrost hardcodes /usr/local/bin, and in my install its either in /bin or /usr/bin | 13:47 |
| TheJulia | sambetts: https://review.openstack.org/#/c/422856/ | 13:48 |
| patchbot | patch 422856 - bifrost - Make path to ironic-rootwrap dynamic | 13:48 |
| sambetts | TheJulia: :O! nice! | 13:48 |
| sambetts | TheJulia: shame we can't auto work it out via something like 'which ironic-rootwrap' but then I guess it'll have to be in the path somewhere then | 13:49 |
| *** baoli has joined #openstack-ironic | 13:50 | |
| TheJulia | sambetts: One would hope root a system wide path does not include a user writable folder... but.. yeah.. | 13:51 |
| *** Nisha_Agarwal has quit IRC | 13:53 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: ETAGs for node in ironicclient https://review.openstack.org/400335 | 13:54 |
| dtantsur | huh, inspector grenade apparently went down :( | 13:55 |
| dtantsur | anything similar with ironic itself? | 13:55 |
| jroll | dtantsur: yeah, a bit flaky, not 100% down afaik | 13:56 |
| jroll | 11:00:01 vsaienk0 | jroll grenade is unstable https://bugs.launchpad.net/devstack/+bug/1660581 should fix https://review.openstack.org/#/c/427098/ | 13:56 |
| openstack | Launchpad bug 1660581 in devstack "nova compute may fail on startup with ironic virt driver" [Undecided,In progress] - Assigned to Vasyl Saienko (vsaienko) | 13:56 |
| patchbot | patch 427098 - openstack-dev/devstack - Ensure that keystone is available when n-cpu starts | 13:56 |
| jroll | dtantsur: ^^" | 13:56 |
| dtantsur | ah, ok | 13:57 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Improve conductor driver validation at startup https://review.openstack.org/426213 | 13:57 |
| jroll | vdrok: ^^ forgot to git add :P | 13:57 |
| *** Nisha_Agarwal has joined #openstack-ironic | 13:57 | |
| *** dsneddon_afk is now known as dsneddon | 14:00 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: ETAGs for node in ironicclient https://review.openstack.org/400335 | 14:01 |
| *** sasha2 has joined #openstack-ironic | 14:02 | |
| *** jcoufal has quit IRC | 14:03 | |
| *** sasha2 has quit IRC | 14:03 | |
| *** cdearborn has joined #openstack-ironic | 14:03 | |
| *** sasha2 has joined #openstack-ironic | 14:03 | |
| dtantsur | jroll, +A to both driver comp patches \o/ | 14:04 |
| dtantsur | though inspector grenade seems to fail 100% now | 14:05 |
| jroll | zomgzomgzomgzomgzomgzomg | 14:05 |
| *** jcoufal has joined #openstack-ironic | 14:06 | |
| jroll | dtantsur: the next one in the stack should be easy too | 14:06 |
| jroll | I'm going to hack on the rest of the todos this morning | 14:07 |
| jroll | oh, and the nova patch for resource class stuff is passing, woo | 14:08 |
| dtantsur | also NMI was approved by nova folks (failing CI though) | 14:08 |
| *** rloo has joined #openstack-ironic | 14:08 | |
| jroll | yeah, saw that \o/ | 14:08 |
| jroll | this is probably ready, quick docs patch https://review.openstack.org/#/c/345081/ | 14:09 |
| patchbot | patch 345081 - ironic - Add docs for node.resource_class and flavor creation | 14:09 |
| jroll | dtantsur: I don't think we'll be able to make that mandatory for pike upgrades, btw, so tripleo should be okay to just do that during pike | 14:09 |
| vsaienk0 | jroll, dtantsur: found issue with grenade, we always update nova before ironic which is not right, we are lucky that we finished ironic upgrade before n-cpu max retries to check ironic connectivity reached. I've tried to check that n-cpu started after n-cpu start, but it not happens http://logs.openstack.org/00/427100/1/check/gate-grenade-dsvm-ironic-ubuntu-xenial/a83bb44/logs/new/screen-n-cpu.txt.gz?level=ERROR | 14:09 |
| dtantsur | jroll, ack, good ot know | 14:09 |
| jroll | vsaienk0: ouch | 14:10 |
| dtantsur | meh.. | 14:10 |
| vsaienk0 | jroll seems we should also change upgrade order ironic first than nova | 14:10 |
| jroll | vsaienk0: yep, that's our upgrade process, we should fix it. I wonder if grenade allows for that or plugins always go last... | 14:10 |
| vsaienk0 | jroll I will check | 14:11 |
| *** nmathew has joined #openstack-ironic | 14:14 | |
| amarao | Hello everyone. Is ironic (IPA) supports software raids (linux-raid aka mdadm)? | 14:22 |
| *** nmathew has quit IRC | 14:23 | |
| dtantsur | amarao, hi, no. it's not the first time we hear this request, but no work has been done so far. | 14:23 |
| openstackgerrit | Merged openstack/bifrost: Make path to ironic-rootwrap dynamic https://review.openstack.org/422856 | 14:24 |
| *** gcb has quit IRC | 14:24 | |
| *** srobert has joined #openstack-ironic | 14:24 | |
| amarao | dtantsur Thanks. | 14:25 |
| *** Goneri has joined #openstack-ironic | 14:25 | |
| dtantsur | amarao, a few raw spec: https://review.openstack.org/#/c/327769/ (needs most of work) | 14:26 |
| patchbot | patch 327769 - ironic-specs - LVM/Software RAID support in ironic-python-agent | 14:26 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack: Only install edk2-ovmf on Fedora https://review.openstack.org/427142 | 14:26 |
| jroll | this also is ready and incredibly useful for nova folks, if someone has time: https://review.openstack.org/#/c/407651/ | 14:28 |
| patchbot | patch 407651 - ironic - Allow using resource classes | 14:28 |
| TheJulia | jroll: two comments regarding the commit message, I just marked it -1 as a result. Since the commit message speaks as if the changes are there, but in the code they are just TODOs | 14:30 |
| jroll | TheJulia: oh, oops, thank you | 14:31 |
| *** [1]cdearborn has joined #openstack-ironic | 14:33 | |
| TheJulia | jroll: no problem. I just feel useful with my scrambled zombie brains :) | 14:36 |
| jroll | lol | 14:36 |
| * TheJulia is never driving 1800 miles again in three days. | 14:36 | |
| jroll | yeeeeesh | 14:36 |
| jroll | that'll do it | 14:36 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: ETAGs for node in ironicclient https://review.openstack.org/400335 | 14:38 |
| *** Nisha_Agarwal has quit IRC | 14:41 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Allow using resource classes https://review.openstack.org/407651 | 14:44 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: DNM: turn on resource classes https://review.openstack.org/423269 | 14:44 |
| *** mtanino has joined #openstack-ironic | 14:45 | |
| *** cdearborn has quit IRC | 14:48 | |
| openstackgerrit | Merged openstack/bifrost: roles: bifrost-create-vm-nodes: Allow overriding the disk caching method https://review.openstack.org/426844 | 14:48 |
| openstackgerrit | Merged openstack/ironic-python-agent: Remove vendor_passthru mentions from lookup/heartbeat documentation https://review.openstack.org/427126 | 14:49 |
| *** zackf has joined #openstack-ironic | 14:49 | |
| openstackgerrit | Merged openstack/bifrost: roles: bifrost-ironic-install: SUSE/Debian: Add gcc and python-pip packages https://review.openstack.org/424853 | 14:49 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Use Ansible to create test VMs https://review.openstack.org/427071 | 14:50 |
| dtantsur | so, do we only need that devstack patch to unbreak the CI? | 14:56 |
| *** crushil has joined #openstack-ironic | 14:58 | |
| dtantsur | we've had 5-7 patches failing grenade in a row | 14:59 |
| vdrok | dtantsur: I think vsaienk0 has a plan now, and its going to be in grenade, not devstack | 15:00 |
| dtantsur | ack, thanks | 15:01 |
| *** Nisha_Agarwal has joined #openstack-ironic | 15:02 | |
| *** Nisha_Agarwal has quit IRC | 15:07 | |
| NobodyCam | Good Morning Ironic'ers | 15:11 |
| dtantsur | morning NobodyCam | 15:14 |
| NobodyCam | hey hey Good Morning dtantsur :) | 15:14 |
| *** salmankhan has joined #openstack-ironic | 15:17 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Verify grenade fix https://review.openstack.org/427100 | 15:18 |
| vdrok | NobodyCam: morning! | 15:19 |
| NobodyCam | Good Morning vdrok :) | 15:19 |
| *** rbartal__ has quit IRC | 15:19 | |
| *** rbartal has quit IRC | 15:20 | |
| *** rbartal_ has quit IRC | 15:20 | |
| *** jaosorior has joined #openstack-ironic | 15:20 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Refresh Ansible facts after installing packages https://review.openstack.org/426702 | 15:21 |
| *** vsaienko has joined #openstack-ironic | 15:22 | |
| TheJulia | NobodyCam: ^^^ should address the fact issue we've been seeing with the tests. I just rebased it to head of master and approved it since it should fix that sporadic issue that the larger more recently proposed changes are encountering. | 15:24 |
| NobodyCam | oh nice :) | 15:25 |
| *** zackf has quit IRC | 15:26 | |
| *** salmankhan has quit IRC | 15:27 | |
| *** causten has joined #openstack-ironic | 15:28 | |
| *** zackf has joined #openstack-ironic | 15:34 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: ETAGs for node in ironicclient https://review.openstack.org/400335 | 15:35 |
| *** udesale has quit IRC | 15:35 | |
| *** salmankhan has joined #openstack-ironic | 15:36 | |
| *** vsaienko has quit IRC | 15:36 | |
| *** mtanino_ has joined #openstack-ironic | 15:39 | |
| vdrok | sambetts: how do you think, do we need to add anything apart from api-ref to our docs regarding attach/detach? maybe some network interface developer guide, or something else too? | 15:39 |
| *** mtanino has quit IRC | 15:40 | |
| TheJulia | Three +2's on https://review.openstack.org/#/c/407651/. I'll +A it in 20 minutes if nobody speaks up :) | 15:41 |
| patchbot | patch 407651 - ironic - Allow using resource classes | 15:41 |
| openstackgerrit | Merged openstack/ironic: Add dynamic driver functionality to REST API https://review.openstack.org/417970 | 15:49 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: [WIP] ETag support: caching resources for node https://review.openstack.org/420033 | 15:49 |
| dtantsur | YESSSSSS! | 15:50 |
| jroll | \o/ \o/ \o/ | 15:50 |
| rloo | dtantsur, jroll: congrats (almost) :D | 15:51 |
| rloo | TheJulia: I trust you, +A. (I haven't looked at it.) | 15:51 |
| rloo | TheJulia: or, er, you could wait for jenkins first. | 15:51 |
| openstackgerrit | Merged openstack/ironic: Add soft reboot, soft power off and power timeout to api-ref https://review.openstack.org/423986 | 15:51 |
| TheJulia | Well, it will fail if jenkins does not give it a green light :) | 15:52 |
| TheJulia | err, I guess +2. | 15:52 |
| *** rpioso has joined #openstack-ironic | 15:52 | |
| * TheJulia looks for the "scrambled zombie brains" hat | 15:52 | |
| rloo | TheJulia: true. just depends on your confidence level :) | 15:52 |
| rloo | TheJulia: I mean, I'll +A if I'm fairly sure that jenkins will be happy. otherwise, i'll wait. but you can +A and then see what jenkins thinks too. | 15:53 |
| TheJulia | :) | 15:56 |
| openstackgerrit | Merged openstack/bifrost: Refresh Ansible facts after installing packages https://review.openstack.org/426702 | 15:58 |
| *** foexle has quit IRC | 16:00 | |
| *** absubram has joined #openstack-ironic | 16:02 | |
| *** sayantani01 has joined #openstack-ironic | 16:02 | |
| vdrok | morning rloo ! | 16:02 |
| *** crushil has quit IRC | 16:03 | |
| rloo | hi and good morning vdrok, TheJulia, jroll, dtantsur, early-bird NobodyCam, and everyone else | 16:03 |
| TheJulia | Good morning rloo | 16:04 |
| NobodyCam | Good Morning rloo :) | 16:04 |
| dtantsur | morning rloo | 16:04 |
| NobodyCam | hehehe | 16:04 |
| *** crushil has joined #openstack-ironic | 16:04 | |
| vdrok | rloo: the earliest-bird is jroll though :) | 16:04 |
| *** rcernin has quit IRC | 16:04 | |
| rloo | vdrok: yes, but that is 'normal' for him! | 16:04 |
| *** sasha2 has quit IRC | 16:04 | |
| TheJulia | rloo: so that change, looks like grenade blew up for it :( | 16:05 |
| jroll | heh | 16:05 |
| rloo | TheJulia: :-( | 16:05 |
| jroll | TheJulia: yeah, grenade is shaky right now | 16:05 |
| vdrok | TheJulia: grenade is down, see whiteboard | 16:05 |
| jroll | see the whiteboard, vsaienk0 is working on it | 16:05 |
| *** sasha2 has joined #openstack-ironic | 16:06 | |
| *** absubram_ has joined #openstack-ironic | 16:07 | |
| *** absubram has quit IRC | 16:07 | |
| *** absubram_ is now known as absubram | 16:07 | |
| JayF | FYI, I have to leave today around lunchtime. | 16:10 |
| * JayF doing final walkthrough on a house he's going to have tomorrow \o/ | 16:10 | |
| *** sasha2 has quit IRC | 16:10 | |
| *** sasha2 has joined #openstack-ironic | 16:10 | |
| mariojv | congrats | 16:10 |
| dtantsur | \o/ | 16:10 |
| JayF | BTW; if anyone is interested, I'm giving a tech talk this afternoon for OSIC on gate troubleshooting (in about 3 hours). If anyone !Rackspace/!Intel wants to attend, send me a PM and I'll forward you the info. | 16:11 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui: Remove node-action deleteNodes/deletePorts functions https://review.openstack.org/426334 | 16:12 |
| *** Syed__ has joined #openstack-ironic | 16:15 | |
| *** d0ugal has quit IRC | 16:16 | |
| TheJulia | JayF: congrats, and good luck! :) | 16:16 |
| *** d0ugal has joined #openstack-ironic | 16:17 | |
| *** d0ugal has quit IRC | 16:17 | |
| *** d0ugal has joined #openstack-ironic | 16:17 | |
| *** owalsh has joined #openstack-ironic | 16:23 | |
| rloo | JayF: that reminds me, weren't you going to give that talk upstrem, about 6 months ago? | 16:24 |
| JayF | yep, and it never ended up happening | 16:24 |
| JayF | that's part of why I'm advertising that one here | 16:24 |
| rloo | JayF: what do you think about (re)scheduling it? | 16:24 |
| JayF | rloo: the one I'm about to give will be video'd and posted to youtube. I was going to post it up to the list and ask if folks wanted a live version. Before it fizzled partially because of scheduling | 16:25 |
| rloo | JayF: cuz I don't think that's enough notice for upstream, and only mentioning it via irc :-( | 16:25 |
| *** aNuposic has joined #openstack-ironic | 16:25 | |
| JayF | and youtube videos don't take any requirement for scheduling :) | 16:25 |
| rloo | JayF: ok, i'll wait for the youtube edition then :) | 16:25 |
| owalsh | hi, can anybody tell me when exactly an ironic node is added to the nova resources? managable state or available state? | 16:25 |
| mat128 | JayF: my schedule is (over)booked, but please send the link when it's up :) | 16:26 |
| rloo | owalsh: available | 16:26 |
| *** baoli has quit IRC | 16:26 | |
| JayF | owalsh: managable is for when you (or an operator) is managing a machine. Available means it's available for provisioning. | 16:26 |
| owalsh | thank rloo, JayF | 16:27 |
| rloo | owalsh: this might help (or not): http://docs.openstack.org/developer/ironic/dev/states.html | 16:27 |
| JayF | owalsh: a unsolicited suggestion: use the newest ironic release possible for best results :) | 16:27 |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:30 | |
| owalsh | JayF: CI issue, couldn't be newer | 16:32 |
| JayF | owalsh: awesome | 16:33 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add ironic portgroup CRUD notifications https://review.openstack.org/427281 | 16:35 |
| dtantsur | Nisha_Agarwal, a small question re https://bugs.launchpad.net/ironic/+bug/1609622: what is "pp" in e.g. snmp_auth_prot_pp? | 16:37 |
| openstack | Launchpad bug 1609622 in Ironic "RFE: Fix iLO drivers to get local_gb for all storages" [Wishlist,In progress] - Assigned to Nisha Agarwal (agarwalnisha1980) | 16:37 |
| dtantsur | maybe worth expanding? | 16:37 |
| dtantsur | everything else looks good | 16:37 |
| Nisha_Agarwal | dtantsur, sambetts jroll i updated the bug https://bugs.launchpad.net/ironic/+bug/1609622 as per the given suggestions. | 16:37 |
| Nisha_Agarwal | dtantsur, it is passphrase | 16:38 |
| *** baoli has joined #openstack-ironic | 16:38 | |
| Nisha_Agarwal | the variable becomes too long, hence i truncated it to pp | 16:38 |
| Nisha_Agarwal | it is explained in documentations | 16:38 |
| dtantsur | Nisha_Agarwal, maybe shorten to "pass"? my gut feeling is that it may be easier recognizable | 16:38 |
| dtantsur | input from English-speaking people is welcome ^^^ | 16:38 |
| JayF | pp is completely unrecognizable | 16:39 |
| jroll | since when do we limit these? | 16:39 |
| JayF | pw would make sense, as would pass | 16:39 |
| jroll | I think passphrase is fine | 16:39 |
| JayF | but jroll has a better point | 16:39 |
| Nisha_Agarwal | the complete variables become like "snmp_auth_prot_pass_phrase" | 16:40 |
| jroll | sure | 16:40 |
| jroll | looks good to me | 16:40 |
| Nisha_Agarwal | ok i would expand them. | 16:40 |
| * jroll welcomes other input, but I don't see a problem | 16:40 | |
| *** amarao has quit IRC | 16:41 | |
| * jroll repeats: if you're typing the same node-create or node-update call more than twice, you're doing something wrong | 16:41 | |
| Nisha_Agarwal | jroll, dtantsur done | 16:41 |
| dtantsur | snmp_auth_protocol_password :) | 16:42 |
| jroll | ^^ +1 | 16:42 |
| Nisha_Agarwal | i would modify the patch https://review.openstack.org/372428 as per the bug details now | 16:42 |
| patchbot | patch 372428 - ironic - SNMP agent support for OOB inspection for iLO Drivers | 16:42 |
| Nisha_Agarwal | dtantsur, ok | 16:42 |
| Nisha_Agarwal | dtantsur, jroll done | 16:43 |
| vsaienk0 | jroll, dtantsur: grenade passed with https://review.openstack.org/#/c/427248/ we need to ping folks from infra team to help merge it | 16:44 |
| patchbot | patch 427248 - openstack-dev/grenade - Do not wait for nova-compute when ironic is enabled | 16:44 |
| jroll | vsaienk0: thanks, will do | 16:44 |
| vsaienk0 | jroll thanks! | 16:46 |
| jroll | vsaienk0: it has +A :) | 16:56 |
| *** tesseract has quit IRC | 16:56 | |
| *** absubram has quit IRC | 16:59 | |
| *** aNuposic has quit IRC | 17:00 | |
| JayF | dtantsur: congrats on tripleo core | 17:02 |
| dtantsur | thanks :) | 17:03 |
| jroll | dtantsur: condolences :D | 17:03 |
| dtantsur | haha, I'm not entirely sure either :-P | 17:03 |
| *** zackf has quit IRC | 17:04 | |
| *** spartacloud has joined #openstack-ironic | 17:04 | |
| *** spartacloud is now known as zackf | 17:05 | |
| Nisha_Agarwal | dtantsur, congrats :) | 17:05 |
| sasha2 | trown: https://review.openstack.org/427299 | 17:06 |
| patchbot | patch 427299 - tripleo-quickstart - fix `hostname -f` resolution for undercloud | 17:06 |
| *** dprince has quit IRC | 17:07 | |
| *** e0ne has quit IRC | 17:08 | |
| jroll | dtantsur: I'm +1 on nisha's rfe if you want to approve it or have others look https://bugs.launchpad.net/ironic/+bug/1609622 | 17:08 |
| openstack | Launchpad bug 1609622 in Ironic "RFE: Fix iLO drivers to get local_gb for all storages" [Wishlist,In progress] - Assigned to Nisha Agarwal (agarwalnisha1980) | 17:08 |
| * dtantsur wonders why snmp_priv_protocol instead of snmp_auth_priv_protocol | 17:09 | |
| dtantsur | anyway, jroll, I'm +1 on it as well | 17:10 |
| Nisha_Agarwal | dtantsur, jroll thanks | 17:10 |
| Nisha_Agarwal | dtantsur, no reason as such to choose that variable | 17:10 |
| jroll | vsaienk0: sean had some thoughts on doing that "right" when we have time, I posted them in the bug: https://bugs.launchpad.net/grenade/+bug/1660646 | 17:11 |
| openstack | Launchpad bug 1660646 in grenade "It is not possible to define order when registering project to upgrade" [Undecided,New] | 17:11 |
| jroll | vsaienk0: if you want to give that a try sometime | 17:11 |
| jroll | Nisha_Agarwal: dtantsur: good point, let's make that snmp_auth_priv_protocol and then I'll approve | 17:12 |
| dtantsur | ++ | 17:12 |
| mgould | hi everyone | 17:13 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic: Add hardware type 'irmc' for FUJITSU PRIMERGY servers https://review.openstack.org/425604 | 17:13 |
| Nisha_Agarwal | dtantsur, jroll done | 17:13 |
| mgould | I'm getting "Host is not mapped to any cell" errors in nova boot on a clean devstack on a freshly-installed baremetal machine | 17:13 |
| jroll | Nisha_Agarwal: approved, you should probably link the related patch in there | 17:14 |
| mgould | I assume this means I've done something wrong - anyone know what? | 17:14 |
| jroll | mgould: interesting, I'd ask nova to be honest | 17:14 |
| Nisha_Agarwal | jroll, thanks. will just postthe patch with changes and add the link in bug as well | 17:14 |
| jroll | Nisha_Agarwal: thanks | 17:14 |
| mgould | jroll: will do, thanks | 17:14 |
| jroll | mgould: there's been a lot of devstack changes around cells v2 this cycle, fwiw, that's why I mention that :) | 17:15 |
| mgould | jroll: makes sense, thanks | 17:15 |
| vdrok | mgould: I've seen this in devstack-gate | 17:15 |
| vdrok | https://github.com/openstack-infra/devstack-gate/commit/f1025531af1726e4fff54b31e16d472bae8193de | 17:16 |
| mgould | vdrok: awesome, thanks | 17:17 |
| vdrok | tho that seems to be affecting multinode, if it is a single node, all should be well... anyway, that commit has a depends-on, maybe that would be useful too | 17:17 |
| mgould | vdrok: I ran nova-manage cell_v2 simple_cell_setup, and now it's failing in a different way | 17:19 |
| mgould | so, progress! | 17:19 |
| vdrok | heh | 17:20 |
| *** trown is now known as trown|lunch | 17:22 | |
| *** milan has quit IRC | 17:25 | |
| *** crushil has quit IRC | 17:31 | |
| *** dprince has joined #openstack-ironic | 17:31 | |
| *** athomas has quit IRC | 17:33 | |
| *** kvcobb has joined #openstack-ironic | 17:33 | |
| *** baoli has quit IRC | 17:35 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Improve conductor driver validation at startup https://review.openstack.org/426213 | 17:36 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fail conductor startup if invalid defaults exist https://review.openstack.org/427322 | 17:36 |
| jroll | the first one there should be good now, I believe the second one is too but it hasn't had any reviews yet :P | 17:36 |
| rloo | jroll: will look at them both today | 17:36 |
| jroll | thanks rloo | 17:37 |
| dtantsur | jroll, I think I'm kinda -2 on the 2nd patch | 17:37 |
| jroll | dtantsur: why? building a task will fail later if we don't do this | 17:37 |
| dtantsur | even without a valid default, nodes can be created with an explicit value | 17:37 |
| dtantsur | or can they? | 17:37 |
| jroll | sure, but it would be easy to fail to do this | 17:37 |
| JayF | I am very +1 to anything that fast-fails an invalid configuration | 17:38 |
| jroll | yeah we added it to node-create | 17:38 |
| dtantsur | well, it boils down to whether nodes can still be created | 17:38 |
| dtantsur | if they can, this is not a hard failure | 17:38 |
| *** kvcobb has quit IRC | 17:38 | |
| jroll | dtantsur: well, slow down | 17:38 |
| jroll | the only way to have no valid default, is to have no interfaces enabled that are supported by the hardware type | 17:39 |
| jroll | which makes that hardware type unusable | 17:39 |
| *** kvcobb has joined #openstack-ironic | 17:39 | |
| * rloo is reminded that she wanted to revisit the No*Interface classes and the rationale for all that. | 17:39 | |
| dtantsur | jroll, you're right, objection retracted | 17:39 |
| dtantsur | I probably confused it with some other situation | 17:40 |
| jroll | remember, we fall back to [i for i in supported if i in enabled] | 17:40 |
| jroll | :) | 17:40 |
| jroll | dtantsur: no worries | 17:40 |
| rloo | jroll, dtantsur: so... what was the reason not to allow enabled_*_interface = None again? | 17:40 |
| dtantsur | I just wanted to stop thinking about the driver composition..... | 17:40 |
| jroll | rloo: you mean, the reason to use no-* instead? | 17:41 |
| rloo | jroll, dtantsur: cuz intuitively, i'd think it would make sense to allow it to be None. (from operator point of view if i were to be an operator which i have no idea what it would be like) | 17:41 |
| rloo | jroll: right, why are we using no* instead of allowing it to be None | 17:41 |
| rloo | jroll: i am wondering if we could have handled <whatever reason> in code instead. | 17:41 |
| dtantsur | rloo, what does "None" mean there? | 17:42 |
| jroll | rloo: well, I can search back in history, or you can, I don't have that anywhere near paged into memory | 17:42 |
| jroll | rloo: or we can revert everything and redesign the entire feature | 17:42 |
| * dtantsur ragequits | 17:42 | |
| dtantsur | :) | 17:42 |
| * jroll would prefer to do neither of those, we're close, let's keep on keeping on | 17:42 | |
| rloo | dtantsur: that the hardware type doesn't support that interface. | 17:42 |
| dtantsur | rloo, we don't have a notion of not supporting interface.. | 17:42 |
| * rloo is reminded of other features, microversions, state machines, where we were so tired we pushed things out | 17:43 | |
| dtantsur | btw my initial version used to allow None to mean that | 17:43 |
| dtantsur | but it caused other problems, lemme remember which | 17:43 |
| jroll | I believe we did this for consistency, so we wouldn't have to guess if things are supported | 17:43 |
| dtantsur | yeah | 17:43 |
| jroll | but I don't fully recall | 17:43 |
| dtantsur | so e.g. node.inspect can be None, but node.power cannot | 17:43 |
| dtantsur | also then we would need some other value to express "use the default" | 17:43 |
| rloo | dtantsur, jroll: it is totally possible that i agreed with current behavior. i guess what bothers me is that i don't recall, so does it make sense to our users. | 17:44 |
| dtantsur | so we changes the proposal: we agreed to consider None as "use default" and create no-X things instead of the whole notion of "not supporting an interface" | 17:44 |
| jroll | rloo: I doubt any of this will make sense at first look, either way we go, it'll take good docs and some reading of those docs | 17:44 |
| dtantsur | ++ | 17:45 |
| jroll | anyway, I have an hour until I'm stuck in meetings for the rest of the day, so I need to go rest my brain and eat some food | 17:45 |
| JayF | I really liked the design of explicitly being like "no-raid" instead of None | 17:45 |
| rloo | we allow node.x_interface = None, we allow config: default_x_interface=None, we do not allow enabled_x_interface=None. | 17:46 |
| JayF | as an operator I sorta pet-peeve against config options where "None" has a meaning | 17:46 |
| rloo | ok. i am not the author of the driver comp, just a reviewer. and i personally don't feel like spending hours trying to recall/think whether what we did is the right thing or not. just wanted to bring it up but others feel like it is fine, so i'm going to move along. | 17:47 |
| rloo | much as folks are tired of working on a feature, this reviewer is tired of reviewing so many features | 17:47 |
| dtantsur | so true.. | 17:48 |
| dtantsur | and after we land it, I'll have to slowly switch tripleo to it.. | 17:49 |
| * dtantsur cries in a corner | 17:49 | |
| rloo | dtantsur: be happy. isn't that why they made you core? :) | 17:49 |
| dtantsur | hehe, dunno | 17:50 |
| *** baoli has joined #openstack-ironic | 17:50 | |
| dtantsur | before I go, JayF, do you know a reason why iPXE downloading IPA images via HTTP can be inadequately slow? | 17:50 |
| *** aNuposic has joined #openstack-ironic | 17:50 | |
| dtantsur | maybe you've seen something like that | 17:50 |
| JayF | dtantsur: in tripleo context? | 17:50 |
| dtantsur | JayF, in any that comes to your mind | 17:50 |
| JayF | dtantsur: well, I know that colocating tftp and http has a negative perf impact on both | 17:50 |
| JayF | dtantsur: especially when a lot of nodes are pxeing | 17:51 |
| *** rajinir has joined #openstack-ironic | 17:51 | |
| JayF | dtantsur: if this is more of a "one off" slowness, on real hardware, I'd be suspicious that the UNDI support on the nic is crappy | 17:51 |
| dtantsur | to which degree? they have something like 8 minutes to download an IPA image on a 10G network | 17:51 |
| JayF | file that under crappy UNDI support, I think | 17:51 |
| dtantsur | lucasagomes, this is re today's escalation ^^^ | 17:51 |
| JayF | you can compile better, hardware-specific drivers into ipxe | 17:51 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Treat unknown fields as subcontrollers https://review.openstack.org/426257 | 17:51 |
| JayF | in fact I think some are precompiled in releases | 17:52 |
| JayF | that are potentially better than UNDI | 17:52 |
| dtantsur | hmm, interesting | 17:52 |
| lucasagomes | JayF, dtantsur yeah that was my guess too, the NIC model isn't well supported | 17:52 |
| lucasagomes | I just commented on the BZ about it | 17:52 |
| lucasagomes | (it's private tho) | 17:52 |
| *** absubram has joined #openstack-ironic | 17:52 | |
| dtantsur | thanks JayF | 17:52 |
| JayF | Everytime a private bug is filed against open source software, a kitten crys | 17:53 |
| JayF | lol | 17:53 |
| lucasagomes | JayF, I know right ? >.< | 17:53 |
| dtantsur | <-- this kitten cries a lot about it | 17:53 |
| lucasagomes | apparently ppl do that A LOT | 17:53 |
| *** rama_y has joined #openstack-ironic | 17:53 | |
| dtantsur | yep | 17:53 |
| dtantsur | ok, going now, see you tomorrow | 17:53 |
| JayF | o/ | 17:53 |
| lucasagomes | dtantsur, see ya | 17:53 |
| mgould | BTW, I got an answer from dansmith about my cells problem | 17:53 |
| *** dtantsur is now known as dtantsur|afk | 17:53 | |
| lucasagomes | JayF, the guy in the BZ said it only happens to the compute nodes so I asked if the NIC models are different in the compute and control nodes | 17:54 |
| lucasagomes | waiting for the reply now | 17:54 |
| dansmith | mgould: I just asked this in a roundabout way in -nova, but.. is this breaking in the ironic gate or just in your setup? | 17:54 |
| lucasagomes | but I let you know once he does | 17:54 |
| mgould | dansmith: just my setup, I think | 17:54 |
| dansmith | okay | 17:54 |
| mgould | vdrok: where did you see it happening? | 17:55 |
| dansmith | mgould: is it that you didn't have your ironic nodes up at the time of the devstack run? or do you think you lost a race that the gate wins because it's slower or something? | 17:55 |
| mgould | the former | 17:55 |
| mgould | that would explain it | 17:56 |
| *** mgoddard has quit IRC | 17:56 | |
| mgould | I'm not creating any nodes in the devstack run | 17:56 |
| vdrok | mgould: on multinode multitenant grenade job. which is experimental as of now | 17:56 |
| dansmith | mgould: ah, okay | 17:56 |
| dansmith | mgould: so this is kindof a thing at this point, but there is a kinda ugly workaround I have proposed: | 17:57 |
| dansmith | mgould: https://review.openstack.org/#/c/426826/ | 17:57 |
| patchbot | patch 426826 - nova - Allow scheduler to run cell host discovery periodi... | 17:57 |
| dansmith | mgould: you don't want that running every second or anything, but setting that nonzero will cause the scheduler to eventually call discover_hosts for you | 17:57 |
| vdrok | mgould: dansmith oh, also here, it seems to be voting job on grenade http://logs.openstack.org/79/396679/45/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/11607f2/logs/grenade.sh.txt.gz#_2017-01-30_09_54_13_227 | 17:57 |
| dansmith | vdrok: ah, that's with VMs not ironic right? | 17:59 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Duplicated code in ..api.get_active_driver_dict() https://review.openstack.org/427335 | 17:59 |
| vdrok | yup, that's vms | 17:59 |
| dansmith | vdrok: that's awesome, you know why? | 17:59 |
| mgould | because it's easier to duplicate? | 17:59 |
| dansmith | vdrok: I wrote a patch to make sure we don't race in multinode grenade, and sdague didn't want to merge it, but now I can rub it in his face :P | 17:59 |
| *** crushil has joined #openstack-ironic | 18:00 | |
| vdrok | hah | 18:00 |
| dansmith | please do express your support here: https://review.openstack.org/#/c/426310/ :P | 18:00 |
| patchbot | patch 426310 - openstack-dev/grenade - Make nova verify compute service count after upgrade | 18:00 |
| vdrok | dansmith: will do :) | 18:00 |
| dansmith | \o/ | 18:00 |
| *** Nisha_Agarwal has quit IRC | 18:01 | |
| dansmith | I might need to tweak it just a tad to make sure we do that before we run discover_hosts (or re-run it after) | 18:01 |
| dansmith | but the basic principle is there | 18:01 |
| *** aNuposic has quit IRC | 18:03 | |
| *** aNuposic has joined #openstack-ironic | 18:03 | |
| mgould | dansmith: woo! | 18:06 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Treat unknown fields as subcontrollers https://review.openstack.org/426257 | 18:07 |
| *** eroux has joined #openstack-ironic | 18:11 | |
| *** dprince has quit IRC | 18:12 | |
| owalsh | JayF, rloo... while I have dansmith here... can I assume that nova discover_hosts will run late enought if I do this after the ironic nodes are set to available | 18:13 |
| owalsh | or is there a potential (but short) race until nova know about the nodes | 18:14 |
| *** dprince has joined #openstack-ironic | 18:15 | |
| JayF | basically the resource tracker in nvoa has to run | 18:15 |
| JayF | while the nodes are in available | 18:15 |
| JayF | before nova will pick them up | 18:15 |
| owalsh | JayF: ok, so to be 100% sure I'll need to poll nova until they exist | 18:16 |
| JayF | I am not 100% sure what you're asking about specifically | 18:16 |
| JayF | so don't wanna advise action | 18:17 |
| JayF | but that's how it works: resource tracker periodic task picks up ironic inventor | 18:17 |
| JayF | I think it's configurable how often they run | 18:17 |
| *** ijw has joined #openstack-ironic | 18:18 | |
| *** salmankhan has quit IRC | 18:18 | |
| *** lucasagomes is now known as lucas-afk | 18:20 | |
| *** jcoufal has quit IRC | 18:20 | |
| vdrok | owalsh: I'm no cells expert, but looking here - https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L1424 seems like you just need to have a single ironic node seen by resource tracker per nova compute host for the host mapping to be populated | 18:21 |
| owalsh | vdrok, JayF: context is a race in the tripleo mistral workflows for baremetal | 18:22 |
| owalsh | it appears to work reliably if discover_hosts is run after the node provisioning states have changed to available | 18:23 |
| owalsh | but I'm concerned there is still a short race there. If the provisioning state does not change to available until nova picks it up then I'm good | 18:24 |
| owalsh | if not then I'll need to poll nova until it does | 18:24 |
| vdrok | owalsh: yup, I think race is possible, if you're adding new nodes in "empty" ironic and right after that do discover_hosts it might not help. you might need to poll for hypervisor-stats and then do discover_hosts. I think :) | 18:25 |
| *** jkilpatr_ has joined #openstack-ironic | 18:25 | |
| *** jkilpatr has quit IRC | 18:27 | |
| dansmith | mgould: vdrok: so.. that log you linked is failing on the old side of grenade, not the new one | 18:30 |
| dansmith | mgould: vdrok which shouldn't be hitting the thing it's hitting.. anything weird about that run that I should know about? | 18:30 |
| *** mjura has quit IRC | 18:30 | |
| *** vsaienko has joined #openstack-ironic | 18:31 | |
| *** jcoufal has joined #openstack-ironic | 18:31 | |
| vdrok | dansmith: see the patch it was run on :) I think it does set base to master | 18:31 |
| *** krtaylor has quit IRC | 18:32 | |
| dansmith | vdrok: okay, well, that means that grenade isn't doing the things it needs to before it runs the old side because it thinks it is newton :) | 18:32 |
| *** ralonsoh has quit IRC | 18:33 | |
| *** mgoddard has joined #openstack-ironic | 18:37 | |
| vdrok | ok, time to leave for me, good night everyone@ | 18:41 |
| vdrok | s/@/! | 18:41 |
| *** trown|lunch is now known as trown | 18:47 | |
| *** owalsh is now known as owalsh-brb | 18:49 | |
| *** patriciadomin has quit IRC | 18:51 | |
| *** owalsh-brb has quit IRC | 18:53 | |
| *** baoli has quit IRC | 18:58 | |
| *** owalsh-brb has joined #openstack-ironic | 18:59 | |
| *** sdake_ has quit IRC | 18:59 | |
| *** sdake has joined #openstack-ironic | 19:00 | |
| *** ChubYann has joined #openstack-ironic | 19:00 | |
| *** patriciadomin has joined #openstack-ironic | 19:03 | |
| *** baoli has joined #openstack-ironic | 19:03 | |
| *** mgould is now known as mgould|afk | 19:04 | |
| *** eroux has quit IRC | 19:05 | |
| *** foexle has joined #openstack-ironic | 19:10 | |
| *** vsaienko has quit IRC | 19:14 | |
| *** krtaylor has joined #openstack-ironic | 19:14 | |
| jroll | passing off the ptl baton: https://twitter.com/changelog/status/826496580119363585 | 19:16 |
| *** milan has joined #openstack-ironic | 19:16 | |
| jroll | stevemar: ^ you'll appreciate that | 19:16 |
| jroll | the baby is the new ptl, not the baton, to be clear :P | 19:17 |
| *** jaosorior has quit IRC | 19:18 | |
| stevemar | oh it's totally the baton | 19:18 |
| *** jaosorior has joined #openstack-ironic | 19:18 | |
| *** dsneddon is now known as dsneddon_afk | 19:21 | |
| *** owalsh-brb is now known as owalsh | 19:31 | |
| *** pcaruana has quit IRC | 19:31 | |
| *** crushil has quit IRC | 19:32 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add documentation for using rescue mode https://review.openstack.org/419606 | 19:33 |
| *** dsneddon has joined #openstack-ironic | 19:36 | |
| *** milan has quit IRC | 19:37 | |
| TheJulia | jroll: shouldn't there at least be a bunch of pillows in a raft on that water though? :) | 19:39 |
| *** salmankhan has joined #openstack-ironic | 19:39 | |
| *** SpamapS has quit IRC | 19:39 | |
| jroll | TheJulia: we pretend the pillows are there but they really are just a figment of your imagination :( | 19:39 |
| *** mgoddard has quit IRC | 19:40 | |
| TheJulia | :( | 19:40 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Use virtualbmc for virtual BM nodes https://review.openstack.org/427376 | 19:41 |
| *** SpamapS has joined #openstack-ironic | 19:41 | |
| TheJulia | \o/ | 19:42 |
| *** salmankhan has quit IRC | 19:44 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Do not install pxe_amt driver by default https://review.openstack.org/426187 | 19:46 |
| *** jkilpatr_ has quit IRC | 19:47 | |
| *** vsaienko has joined #openstack-ironic | 19:53 | |
| *** foexle has quit IRC | 19:54 | |
| *** vsaienko has quit IRC | 19:59 | |
| *** vsaienko has joined #openstack-ironic | 20:00 | |
| NobodyCam | jroll: lol | 20:00 |
| *** jkilpatr_ has joined #openstack-ironic | 20:01 | |
| *** Sukhdev has joined #openstack-ironic | 20:03 | |
| *** vsaienko has quit IRC | 20:04 | |
| *** sdake has quit IRC | 20:14 | |
| *** vsaienko has joined #openstack-ironic | 20:15 | |
| *** maratoid_ has joined #openstack-ironic | 20:16 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Log reason for hardware type registration failure https://review.openstack.org/427389 | 20:18 |
| *** aNuposic has quit IRC | 20:19 | |
| openstackgerrit | Merged openstack/bifrost: Do not install pxe_amt driver by default https://review.openstack.org/426187 | 20:22 |
| *** vsaienko has quit IRC | 20:24 | |
| *** mtanino_ has quit IRC | 20:25 | |
| *** vsaienko has joined #openstack-ironic | 20:26 | |
| *** srobert_ has joined #openstack-ironic | 20:27 | |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Use Tempest plugin interface https://review.openstack.org/427395 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Migrate Tempest tests into Ironic tree https://review.openstack.org/427396 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Remove hard-coded DEPLOYWAIT timeout from Baremetal Scenario https://review.openstack.org/427397 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Refresh ssh verification mechanism https://review.openstack.org/427398 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Comment out test options that already exists on tempest's tree https://review.openstack.org/427399 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Replace depricated tempest-lib with tempest.lib https://review.openstack.org/427400 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Tune interval for node provision state check https://review.openstack.org/427401 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Document how to run the tempest tests https://review.openstack.org/427402 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Fix usage of rest_client expected_success() in tests https://review.openstack.org/427403 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Add support for API microversions in Tempest tests https://review.openstack.org/427404 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Introduce provision states: AVAILABLE, ENROLL https://review.openstack.org/427405 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Fix signature for request method https://review.openstack.org/427406 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: tempest: start using get_configured_admin_credentials https://review.openstack.org/427407 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: No need for 'default=None' in config variable https://review.openstack.org/427408 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Add test to verify ironic multitenancy https://review.openstack.org/427409 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Fix incorrect order of params of assertEqual() https://review.openstack.org/427410 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Using assertIsNone() is preferred over assertEqual() https://review.openstack.org/427411 | 20:27 |
| openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: Fix incorrect order of params of assertEqual() https://review.openstack.org/427412 | 20:27 |
| jroll | nice | 20:28 |
| *** aNuposic has joined #openstack-ironic | 20:28 | |
| soliosg | sorry for the spam! | 20:28 |
| jroll | it's the good kind of spam :D | 20:29 |
| jroll | http://s2.quickmeme.com/img/0c/0c567f765d5aaede661760e28aed96199f1a5fc48fbcca36e5d887f73eb8db40.jpg | 20:30 |
| *** srobert has quit IRC | 20:30 | |
| openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 20:32 |
| *** vsaienko has quit IRC | 20:35 | |
| *** vsaienko has joined #openstack-ironic | 20:39 | |
| openstackgerrit | Xavier proposed openstack/ironic: Validation before perform node deallocation https://review.openstack.org/386014 | 20:46 |
| *** dsneddon has quit IRC | 20:52 | |
| *** dsneddon has joined #openstack-ironic | 20:52 | |
| *** e0ne has joined #openstack-ironic | 20:54 | |
| *** rpioso has quit IRC | 20:56 | |
| *** jcoufal_ has joined #openstack-ironic | 21:01 | |
| *** jcoufal has quit IRC | 21:02 | |
| *** sdake has joined #openstack-ironic | 21:08 | |
| *** Marx314 has quit IRC | 21:14 | |
| *** vinsh has quit IRC | 21:17 | |
| *** jkilpatr_ has quit IRC | 21:18 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Agent driver rescue implementation https://review.openstack.org/400437 | 21:19 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 21:19 |
| mariojv | just a rebase again ^ | 21:19 |
| mariojv | '/buffer 26 | 21:19 |
| mariojv | oops | 21:19 |
| *** bfournie1 has joined #openstack-ironic | 21:20 | |
| *** bfournie has quit IRC | 21:21 | |
| *** mtanino has joined #openstack-ironic | 21:22 | |
| *** cdearborn has joined #openstack-ironic | 21:22 | |
| *** jaosorior has quit IRC | 21:23 | |
| *** rpioso has joined #openstack-ironic | 21:23 | |
| *** crushil has joined #openstack-ironic | 21:32 | |
| *** vsaienko has quit IRC | 21:32 | |
| *** [1]cdearborn has quit IRC | 21:35 | |
| *** e0ne has quit IRC | 21:37 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV https://review.openstack.org/397126 | 21:38 |
| *** Goneri has quit IRC | 21:38 | |
| rybridges2 | Hello OpenStackers. | 21:39 |
| rybridges2 | I was having a look at how ipmi tool works in this file https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipmitool.py | 21:39 |
| rybridges2 | I noticed that in the _wait() method (line 507), it looks like there is no retries happening. Let's take an example: | 21:39 |
| rybridges2 | The first time the control flow enters that _wait() method, the 'iter' variable will be set to -1. It will try to call _exec_ipmitool(). Let's say for the sake of argument that the _exec_ipmitool() method throws an exception. Then it gets caught below and a log message is printed. | 21:39 |
| rybridges2 | Then, the finally block is executed which increments the 'iter' variable to 0. Then the sleep time will be calculated and the method will be executed again. On the second iteration in this case, that first if statement on line 510 will fail because 'iter' is now 0, so the else block executes which simply checks the power state. | 21:39 |
| rybridges2 | From this point on, any further iterations of of this method will only check the power state and not actually try to set the power state. Is this intended? Do we really only want to try setting the power state with ipmi tool once? Or do we want to actually retry setting the power state if it fails the first time? | 21:39 |
| *** e0ne has joined #openstack-ironic | 21:40 | |
| *** rushil has joined #openstack-ironic | 21:40 | |
| *** e0ne has quit IRC | 21:42 | |
| *** baoli has quit IRC | 21:43 | |
| *** rushil has quit IRC | 21:46 | |
| *** vsaienko has joined #openstack-ironic | 21:46 | |
| *** vinsh has joined #openstack-ironic | 21:49 | |
| *** jkilpatr_ has joined #openstack-ironic | 21:53 | |
| *** baoli has joined #openstack-ironic | 21:54 | |
| *** jaosorior has joined #openstack-ironic | 21:55 | |
| *** trown is now known as trown|outtypewww | 21:58 | |
| *** baoli has quit IRC | 22:01 | |
| openstackgerrit | Solio Sarabia proposed openstack/ironic: WIP/DNM: Move ironic_tempest_plugin to new repo https://review.openstack.org/424874 | 22:02 |
| *** crushil has quit IRC | 22:04 | |
| *** vinsh has quit IRC | 22:04 | |
| *** rpioso has quit IRC | 22:05 | |
| *** rpioso has joined #openstack-ironic | 22:06 | |
| *** krtaylor has quit IRC | 22:07 | |
| *** dprince has quit IRC | 22:08 | |
| openstackgerrit | Solio Sarabia proposed openstack/ironic: WIP/DNM: Move ironic_tempest_plugin to new repo https://review.openstack.org/424874 | 22:12 |
| *** sayantani01 has quit IRC | 22:13 | |
| _fatalis | Is telemetry service required for ironic? | 22:14 |
| *** sayantani01 has joined #openstack-ironic | 22:17 | |
| jlvillal | _fatalis: Not that I am aware of. | 22:19 |
| rloo | jroll: sigh. are release notes working as expected? http://docs.openstack.org/releasenotes/ironic/. so newton is 6.0.0 - 6.2.x. Now look at newton: http://docs.openstack.org/releasenotes/ironic/newton.html. 6.2.x only. Which means it doesn't include everything that is in the newton release :-( | 22:21 |
| *** jheroux has quit IRC | 22:21 | |
| jroll | rloo: probably a question for #openstack-release | 22:21 |
| jroll | I don't know how reno works well | 22:21 |
| rloo | jroll: my theory is cuz of ironic's release versioning. | 22:22 |
| rloo | jroll: i'll ask tomorrow maybe. | 22:22 |
| jroll | rloo: it's all based on git, it shouldn't matter | 22:22 |
| rloo | jroll: maybe something needs to be configured. on my list for tomorrow unless someone beats me to it. | 22:23 |
| jroll | rloo: thanks | 22:23 |
| jroll | rybridges2: yeah, the goal is to issue the command once, then we poll the status to see if it went through, as some BMCs take some time to do that thing | 22:24 |
| soliosg | rybridges2: well, there's a comment in https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipmitool.py#L509 | 22:24 |
| jroll | rybridges2: though now I see what you're saying, the "# Log failures but keep trying" comment is wrong | 22:24 |
| soliosg | rybridges2: where the author explicitly wants _exec_ipmitoll to run once | 22:25 |
| *** baoli has joined #openstack-ironic | 22:25 | |
| *** vinsh has joined #openstack-ironic | 22:29 | |
| *** ccamacho has quit IRC | 22:29 | |
| *** bfournie1 has quit IRC | 22:29 | |
| soliosg | jroll, rybridges2: probably "… but keep trying" means keep reading/polling the status, not keep issuing power change | 22:30 |
| jroll | https://github.com/openstack/diskimage-builder/ | 22:32 |
| jroll | http://lists.openstack.org/pipermail/openstack-dev/2017-January/109716.html | 22:32 |
| jroll | csmart: ^ relevant | 22:32 |
| *** srobert has joined #openstack-ironic | 22:33 | |
| *** priteau has quit IRC | 22:35 | |
| *** srobert_ has quit IRC | 22:36 | |
| *** srobert has quit IRC | 22:38 | |
| *** wrouesnel has quit IRC | 22:42 | |
| *** wrouesnel has joined #openstack-ironic | 22:42 | |
| jroll | ok I'm done for the day, see y'all tomorrow | 22:42 |
| rloo | have a good evening jroll | 22:43 |
| *** amotoki has quit IRC | 22:44 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Remove support for driver object periodic tasks https://review.openstack.org/427493 | 22:48 |
| *** ElCoyote_ has left #openstack-ironic | 22:49 | |
| csmart | jroll: thanks! | 22:50 |
| *** jcoufal_ has quit IRC | 22:51 | |
| *** krtaylor has joined #openstack-ironic | 22:53 | |
| *** sdake has quit IRC | 22:59 | |
| _fatalis | thank you jlvillal | 23:03 |
| _fatalis | I was confused because it is referred here | 23:03 |
| _fatalis | http://docs.openstack.org/project-install-guide/baremetal/draft/get_started.html | 23:03 |
| *** yuanying has joined #openstack-ironic | 23:04 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Remove deprecated [deploy]erase_devices_iterations https://review.openstack.org/427500 | 23:07 |
| *** sdake has joined #openstack-ironic | 23:08 | |
| soliosg | rloo: I hadn't seen your comment ^^ regarding the release notes, and I posted a question in one of your patches (remove support for periodic task in driver object) :( | 23:14 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Fix a few docstring warnings https://review.openstack.org/427502 | 23:18 |
| *** baoli has quit IRC | 23:21 | |
| *** dsneddon has quit IRC | 23:21 | |
| *** rama_y has quit IRC | 23:26 | |
| *** rama_y has joined #openstack-ironic | 23:26 | |
| *** absubram has quit IRC | 23:27 | |
| *** sdake has quit IRC | 23:32 | |
| rloo | soliosg: no worries | 23:37 |
| *** sayantani01 has quit IRC | 23:42 | |
| *** sayantani01 has joined #openstack-ironic | 23:50 | |
| *** sayantani01 has quit IRC | 23:54 | |
| *** crushil has joined #openstack-ironic | 23:54 | |
| *** bfournie has joined #openstack-ironic | 23:57 | |
| *** sdake has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
