| *** livelace has quit IRC | 00:28 | |
| *** livelace has joined #openstack-ironic | 00:28 | |
| *** MattMan_ has quit IRC | 00:36 | |
| *** zhangfei has joined #openstack-ironic | 00:40 | |
| *** ElCoyote_ has joined #openstack-ironic | 00:40 | |
| *** MattMan_ has joined #openstack-ironic | 00:45 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: DNM/TEST configure transport url https://review.openstack.org/593219 | 01:01 |
|---|---|---|
| *** trungnv has joined #openstack-ironic | 01:09 | |
| *** phuongnh has joined #openstack-ironic | 01:17 | |
| *** bfournie has quit IRC | 01:35 | |
| *** ElCoyote_ has quit IRC | 02:02 | |
| *** zhangfei has quit IRC | 02:14 | |
| *** zhangfei has joined #openstack-ironic | 02:14 | |
| *** early has quit IRC | 03:18 | |
| *** Bhujay has joined #openstack-ironic | 03:18 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Direct deploy serve HTTP images from conductor https://review.openstack.org/565474 | 03:21 |
| *** gcb_ has joined #openstack-ironic | 03:23 | |
| *** Bhujay has quit IRC | 03:26 | |
| *** early has joined #openstack-ironic | 03:28 | |
| *** zhangfei has quit IRC | 03:30 | |
| *** trungnv has quit IRC | 03:55 | |
| *** jaganathan has joined #openstack-ironic | 04:05 | |
| *** Bhujay has joined #openstack-ironic | 04:16 | |
| *** phuongnh has quit IRC | 05:00 | |
| *** phuongnh has joined #openstack-ironic | 05:00 | |
| *** gcb_ has quit IRC | 05:10 | |
| *** hoonetorg has quit IRC | 05:22 | |
| *** e0ne has joined #openstack-ironic | 05:26 | |
| *** hoonetorg has joined #openstack-ironic | 05:39 | |
| *** oanson has joined #openstack-ironic | 05:43 | |
| *** phuongnh has quit IRC | 05:44 | |
| *** trungnv has joined #openstack-ironic | 05:55 | |
| *** e0ne has quit IRC | 06:00 | |
| *** skazi has joined #openstack-ironic | 06:12 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: DNM/TEST configure transport url https://review.openstack.org/593219 | 06:21 |
| *** jtomasek has joined #openstack-ironic | 06:22 | |
| *** ltomasbo has joined #openstack-ironic | 06:29 | |
| *** IamChandra has joined #openstack-ironic | 06:29 | |
| *** IamChandra is now known as sekharvajjula | 06:29 | |
| *** phuongnh has joined #openstack-ironic | 06:38 | |
| *** zhangfei has joined #openstack-ironic | 06:42 | |
| *** e0ne has joined #openstack-ironic | 06:42 | |
| *** e0ne has quit IRC | 06:43 | |
| *** pcaruana has joined #openstack-ironic | 06:46 | |
| *** moshele has joined #openstack-ironic | 06:46 | |
| *** sekharvajjula has quit IRC | 06:49 | |
| *** arne_wiebalck has joined #openstack-ironic | 06:51 | |
| *** rcernin has quit IRC | 07:05 | |
| *** sekharvajjula has joined #openstack-ironic | 07:06 | |
| sekharvajjula | krtaylor: I am Chandra shekar from Nokia. I need some help regarding 3rd party CI setup at Nokia. I am planning to run Ironic Tests specific to Nokia Ironic drivers. Need some guidance on this topic. I got your contact from https://wiki.openstack.org/wiki/Ironic/Testing#Questions. | 07:13 |
| *** yolanda has joined #openstack-ironic | 07:19 | |
| *** rpittau has joined #openstack-ironic | 07:41 | |
| *** S4ren has joined #openstack-ironic | 07:44 | |
| *** w-miller has joined #openstack-ironic | 07:54 | |
| *** moshele has quit IRC | 08:06 | |
| *** moshele has joined #openstack-ironic | 08:06 | |
| *** livelace has quit IRC | 08:12 | |
| *** tssurya has joined #openstack-ironic | 08:16 | |
| *** livelace has joined #openstack-ironic | 08:16 | |
| *** dtantsur|afk is now known as dtantsur | 08:25 | |
| dtantsur | morning ironic | 08:25 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add automated_clean field https://review.openstack.org/585795 | 08:25 |
| dtantsur | sekharvajjula: sambetts|afk may be able to help when he's here | 08:25 |
| etingof | good morning dtantsur \o | 08:26 |
| dtantsur | o/ | 08:26 |
| arne_wiebalck | Good morning, Ironic. | 08:31 |
| sekharvajjula | dtantsur: Thank you. I will await #sambetts|afk arrival and reply. | 08:36 |
| *** mgoddard has joined #openstack-ironic | 08:40 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-specs master: Amendment of the agent http provisioning spec https://review.openstack.org/593545 | 08:47 |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-specs master: Amendment of the agent http provisioning spec https://review.openstack.org/593545 | 08:50 |
| *** e0ne has joined #openstack-ironic | 08:58 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: [WIP] Wrap rpc server into oslo.service https://review.openstack.org/584758 | 09:27 |
| *** sambetts|afk is now known as sambetts | 09:33 | |
| sambetts | dtantsur, sekharvajjula: Hi | 09:34 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/585996 | 09:37 |
| dtantsur | has anyone checked why the inspector CI is failing? | 09:38 |
| dtantsur | vdrok: haven't we fixed http://logs.openstack.org/87/592387/1/check/ironic-inspector-tempest-dsvm-python3/3aba945/logs/screen-n-cpu.txt.gz#_Aug_16_14_31_27_527100 already? | 09:39 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add automated_clean field to the node object and API https://review.openstack.org/585991 | 09:42 |
| openstackgerrit | Merged openstack/sushy stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/584104 | 09:49 |
| *** S4ren has quit IRC | 09:56 | |
| *** S4ren has joined #openstack-ironic | 09:58 | |
| vdrok | dtantsur: morning, we did, scheduler should be using a microversion allowing the value to be equal.. | 09:59 |
| vdrok | scheduler client rather | 09:59 |
| dtantsur | vdrok: well, it does not seem the case in the inspector CI.. | 10:01 |
| vdrok | https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L875-L876 | 10:03 |
| vdrok | hrm, http://logs.openstack.org/87/592387/1/check/ironic-inspector-tempest-dsvm-python3/3aba945/logs/screen-placement-api.txt.gz#_Aug_16_14_31_27_525609 says microversion=1.0 | 10:05 |
| vdrok | maybe this one is calling it https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L1098 | 10:07 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add functionality for individual cleanup on nodes https://review.openstack.org/586277 | 10:07 |
| dtantsur | vdrok: yeah.. I wonder why only the inspector CI is affected | 10:08 |
| dtantsur | vdrok: fun fact: apparently only master is broken, not even rocky | 10:13 |
| vdrok | hrm, maybe there's a race | 10:13 |
| *** w-miller has quit IRC | 10:16 | |
| *** w-miller has joined #openstack-ironic | 10:17 | |
| *** w-miller has quit IRC | 10:21 | |
| *** dtantsur is now known as dtantsur|brb | 10:27 | |
| openstackgerrit | Merged openstack/networking-baremetal stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590316 | 10:30 |
| *** vdrok has quit IRC | 10:39 | |
| *** vdrok has joined #openstack-ironic | 10:40 | |
| openstackgerrit | Merged openstack/bifrost master: Refactor bifrost-integration-tinyipa jobs https://review.openstack.org/592462 | 10:46 |
| openstackgerrit | Merged openstack/bifrost stable/rocky: Refactor bifrost-integration-dhcp jobs https://review.openstack.org/592486 | 10:46 |
| *** e0ne has quit IRC | 10:51 | |
| *** w-miller has joined #openstack-ironic | 10:53 | |
| openstackgerrit | Merged openstack/sushy master: import zuul job settings from project-config https://review.openstack.org/592417 | 10:59 |
| openstackgerrit | Merged openstack/sushy master: switch documentation job to new PTI https://review.openstack.org/592418 | 11:04 |
| openstackgerrit | Merged openstack/bifrost master: Install openstack client, inspector in clouds.yml https://review.openstack.org/550522 | 11:09 |
| openstackgerrit | Merged openstack/bifrost stable/rocky: Refactor bifrost-integration-dibipa-debian jobs https://review.openstack.org/592488 | 11:09 |
| openstackgerrit | Merged openstack/sushy-tools master: import zuul job settings from project-config https://review.openstack.org/592420 | 11:09 |
| openstackgerrit | Merged openstack/sushy-tools master: add python 3.6 unit test job https://review.openstack.org/592421 | 11:09 |
| jroll | morning folks | 11:13 |
| openstackgerrit | Merged openstack/virtualbmc master: import zuul job settings from project-config https://review.openstack.org/592422 | 11:15 |
| openstackgerrit | Merged openstack/virtualbmc master: switch documentation job to new PTI https://review.openstack.org/592423 | 11:15 |
| openstackgerrit | Merged openstack/sushy master: add python 3.6 unit test job https://review.openstack.org/592419 | 11:15 |
| openstackgerrit | Merged openstack/sushy stable/queens: import zuul job settings from project-config https://review.openstack.org/592456 | 11:15 |
| openstackgerrit | Merged openstack/sushy stable/rocky: import zuul job settings from project-config https://review.openstack.org/592470 | 11:15 |
| openstackgerrit | Merged openstack/sushy stable/pike: import zuul job settings from project-config https://review.openstack.org/592445 | 11:15 |
| openstackgerrit | Merged openstack/virtualbmc master: add python 3.6 unit test job https://review.openstack.org/592424 | 11:16 |
| *** Bhujay has quit IRC | 11:17 | |
| *** zhangfei has quit IRC | 11:28 | |
| openstackgerrit | Merged openstack/networking-generic-switch stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590313 | 11:28 |
| *** phuongnh has quit IRC | 11:31 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: DNM/TEST functional https://review.openstack.org/593219 | 11:38 |
| *** Bhujay has joined #openstack-ironic | 11:40 | |
| *** e0ne has joined #openstack-ironic | 11:47 | |
| *** rh-jelabarre has joined #openstack-ironic | 11:49 | |
| *** jcoufal has joined #openstack-ironic | 12:00 | |
| *** jroll has quit IRC | 12:00 | |
| *** jroll has joined #openstack-ironic | 12:01 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: [WIP] Wrap rpc server into oslo.service https://review.openstack.org/584758 | 12:11 |
| *** trown|outtypewww is now known as trown | 12:13 | |
| *** Bhujay has quit IRC | 12:16 | |
| *** Bhujay has joined #openstack-ironic | 12:16 | |
| *** needssleep is now known as TheJulia | 12:23 | |
| TheJulia | Good morning everyone | 12:23 |
| TheJulia | dtantsur|brb: I started to look at inspector on Friday but I don't remember getting very far. | 12:24 |
| TheJulia | dtantsur|brb: it looked like it was something over in nova land with how nova was processing the data in the resource tracker. I feel like I saw cached data not refresh based on log output, but I'm not sure. | 12:27 |
| *** jcoufal has quit IRC | 12:31 | |
| *** jcoufal has joined #openstack-ironic | 12:33 | |
| openstackgerrit | Merged openstack/ironic master: Remove the duplicated word https://review.openstack.org/592887 | 12:39 |
| *** dtantsur|brb is now known as dtantsur | 12:45 | |
| dtantsur | morning TheJulia, jroll | 12:46 |
| * TheJulia thinks casual nick friday should continue | 12:46 | |
| dtantsur | heh | 12:46 |
| TheJulia | I drove 700+ miles this weekend, I feel like a freight train hit me this morning | 12:47 |
| jroll | oof | 12:50 |
| dtantsur | wow | 12:54 |
| *** moshele has quit IRC | 12:57 | |
| TheJulia | 669.5 miles next weekend... not counting stopping for fuel. | 12:59 |
| dtantsur | vdrok, TheJulia, oh, the same nova issue breaks tripleo: https://bugs.launchpad.net/tripleo/+bug/1787910 | 13:00 |
| openstack | Launchpad bug 1787910 in tripleo "OVB overcloud deploy fails on nova placement errors" [Critical,Triaged] - Assigned to Marios Andreou (marios-b) | 13:00 |
| TheJulia | Yay | 13:03 |
| *** rloo has joined #openstack-ironic | 13:07 | |
| vdrok | dtantsur: doing a patch now | 13:15 |
| dtantsur | vdrok: awesome! mind updating the lp bug? maybe add nova to it if they haven't migrated to sb? | 13:19 |
| vdrok | sure, will do | 13:19 |
| dtantsur | thanks! | 13:20 |
| *** zul has joined #openstack-ironic | 13:28 | |
| TheJulia | mgoddard: If you have some free braincells, It would be awesome if you could take a look at | 13:31 |
| TheJulia | wow, helps if I paste the link | 13:33 |
| TheJulia | https://review.openstack.org/#/c/587576/ | 13:33 |
| patchbot | patch 587576 - ironic - A vision - 0h 19m 8s spent in CI | 13:33 |
| TheJulia | sambetts: Also if you have some braincells to spare, if you could take a look at ^^^ patch. Thanks! | 13:34 |
| sambetts | TheJulia: taking a look | 13:34 |
| * TheJulia is hoping to worflow it today and push a follow-up to fix some minor things | 13:34 | |
| TheJulia | sambetts: thanks! :) | 13:38 |
| sambetts | TheJulia: :thumbs up: sounds good | 13:39 |
| vdrok | dtantsur: actually, currently trying to add a test to the patch, and I see that the client uses 1.29 by default, while required version for setting reserved=total is 1.26 | 13:47 |
| vdrok | so this test does not fail even when not specifying 1.26 | 13:47 |
| vdrok | trying to figure out where else the version can be overriden | 13:48 |
| openstackgerrit | Dmitry Tantsur proposed openstack/networking-baremetal master: import zuul job settings from project-config https://review.openstack.org/592406 | 13:48 |
| vdrok | yeah it can be set via [placement]version I think in nova.conf | 13:49 |
| openstackgerrit | Dmitry Tantsur proposed openstack/networking-baremetal master: add python 3.6 unit test job https://review.openstack.org/592408 | 13:51 |
| vdrok | or maybe it's how the PlacementDirect is setup in tests | 13:52 |
| dtantsur | who knows how tox jobs with -neutron are different? | 13:52 |
| sambetts | dtantsur: they add neutron, neutron-lib, python-neutronclient to the required-projects | 13:53 |
| dtantsur | sambetts: then why do net-bm unit tests pass without it? | 13:54 |
| sambetts | dtantsur: its so that the networking-* projects can base their tox jobs off it and not have to specify all the required-projects over and over | 13:54 |
| openstackgerrit | Paul Belanger proposed openstack/bifrost stable/rocky: Refactor bifrost-integration-tinyipa jobs https://review.openstack.org/593626 | 13:54 |
| dtantsur | dhellmann just tried adding a bare py3.6 job and it worked.. | 13:54 |
| sambetts | not sure if networking-bm inherits anything from neutron | 13:54 |
| dhellmann | dtantsur : is the template for the neutron variant still there? | 13:54 |
| dhellmann | sambetts : does it need to install neutron in order to run the tests? | 13:55 |
| dtantsur | dhellmann: it has neutron in requirements | 13:55 |
| dhellmann | ah, ok, that would do it then | 13:55 |
| dtantsur | dhellmann: so, no need for special -neutron jobs for us? | 13:56 |
| dhellmann | the neutron job variant installs neutron from source, iiuc | 13:56 |
| dhellmann | the regular job is only going to install from releases, so if that is sufficient for you then you don't need the variant | 13:56 |
| sambetts | yeah if neutron isn't in the required projects it comes from pypi and you won't be able to do depends-ons to any neutron patch | 13:56 |
| dhellmann | mordred and AJaeger can probably advise more | 13:56 |
| dtantsur | dhellmann: okay, so we only need check-requirements from https://review.openstack.org/#/c/592406/2/zuul.d/project.yaml ? | 13:56 |
| patchbot | patch 592406 - networking-baremetal - import zuul job settings from project-config - 1h 11m 6s spent in CI | 13:56 |
| dhellmann | dtantsur : I would take that patch as it is for now and remove anything you decide you don't need later | 13:57 |
| dhellmann | but you may be right | 13:57 |
| dtantsur | dhellmann: btw, wanted to draw attention to something: last time I checked the requirements CI was expecting to find check-requirements template in project-config for anything that you try to add to projects.txt | 13:57 |
| dtantsur | was it fixed? | 13:57 |
| dhellmann | yeah, we're going to have to change that | 13:57 |
| dhellmann | although we don't really need projects.txt any more since we aren't syncing | 13:58 |
| dhellmann | I'll have to talk to prometheanfire and tonyb about it | 13:58 |
| dtantsur | ok cool | 13:59 |
| openstackgerrit | Dmitry Tantsur proposed openstack/networking-baremetal master: import zuul job settings from project-config https://review.openstack.org/592406 | 13:59 |
| dhellmann | as far as the unit test jobs, you should only need 1 of the templates ultimately, but I'm not sure which you want. As sambetts points out the variant lets you test against master | 13:59 |
| openstackgerrit | Dmitry Tantsur proposed openstack/networking-baremetal master: add python 3.6 unit test job https://review.openstack.org/592408 | 13:59 |
| dtantsur | dhellmann, sambetts, then we need a py36 template for -neutron | 13:59 |
| dtantsur | otherwise we're going to have a mix of different jobs | 13:59 |
| dhellmann | indeed | 13:59 |
| dhellmann | let me drop into -infra and ask about that | 14:00 |
| *** jlv-sick is now known as jlvillal | 14:00 | |
| dtantsur | sambetts, TheJulia, so, in the end, which of the jobs should we run: with -neutron or without? | 14:00 |
| *** baha has joined #openstack-ironic | 14:00 | |
| sambetts | the alternative to using the template is that we include neutron and etc projects to the required-projects for that specific job ourselves | 14:01 |
| dtantsur | sambetts: I'm okay with the template, I just wonder if we should land it as it is or create a py36-neutron template | 14:02 |
| * TheJulia notices chatter and is currently in a meeting | 14:02 | |
| *** Bhujay has quit IRC | 14:02 | |
| dtantsur | sambetts: if we use a neutron job, how does it work with requirements.txt? | 14:02 |
| dtantsur | we seem to just install the version from upper-constraints anyway.. | 14:03 |
| * mordred waves | 14:03 | |
| mgoddard | TheJulia: I'll try to apply some braincells to the vision | 14:03 |
| sambetts | dtantsur: zuul handles the sibling projects magic, so basically requirements.txt gets installed, then zuul evauates the virtual env and checks for projects that have been installed which we have the source for | 14:03 |
| sambetts | dtantsur: then it installs the source version over the package version | 14:04 |
| sambetts | http://logs.openstack.org/08/592408/1/check/openstack-tox-py35/4544c14/job-output.txt.gz#_2018-08-16_14_05_49_962143 | 14:04 |
| dtantsur | FUN | 14:04 |
| sambetts | yup... | 14:04 |
| mordred | it's magic | 14:04 |
| mordred | and by magic - I mean code that you probably shouldn't read | 14:05 |
| dtantsur | lol | 14:05 |
| sambetts | what frustrates me a little about that is that we used to have tox_install.sh so that magic worked locally, but now it only works in the CI | 14:05 |
| mordred | WELL... | 14:05 |
| mordred | http://git.openstack.org/cgit/openstack/pbrx/tree/pbrx/siblings.py | 14:05 |
| sambetts | its better in other ways though ;) | 14:05 |
| mordred | we've been working on extracting that so it can be run locally too | 14:05 |
| sambetts | :O \o/!!! | 14:05 |
| openstackgerrit | Paul Belanger proposed openstack/bifrost stable/queens: Refactor bifrost-integration-dhcp jobs https://review.openstack.org/593631 | 14:06 |
| openstackgerrit | Paul Belanger proposed openstack/bifrost stable/queens: Refactor bifrost-integration-dibipa-debian jobs https://review.openstack.org/593632 | 14:06 |
| openstackgerrit | Paul Belanger proposed openstack/bifrost stable/queens: Refactor bifrost-integration-tinyipa jobs https://review.openstack.org/593633 | 14:06 |
| dtantsur | vdrok: should we create an inspector patch that depends-on your nova fix? | 14:06 |
| *** moshele has joined #openstack-ironic | 14:07 | |
| vdrok | dtantsur: yes we can, tho the functional test now shows the issue clearly | 14:07 |
| dtantsur | mordred: how does it all work with upper-constraints? | 14:07 |
| dtantsur | vdrok: lemme do it | 14:07 |
| mordred | dtantsur: yes | 14:08 |
| mordred | this: https://review.openstack.org/#/c/589465/ is a patch using it in a test job | 14:08 |
| patchbot | patch 589465 - openstackclient - Update openstackclient-check-plugins to honor cons... - 0h 31m 49s spent in CI | 14:08 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: [DNM] Test the proposed gate fix https://review.openstack.org/593635 | 14:08 |
| * dtantsur sees more magic | 14:08 | |
| mordred | (that test job doesn't use tox so the normal tox magic in the gate didn't apply) | 14:08 |
| TheJulia | mgoddard: thanks | 14:09 |
| TheJulia | yay full brainnow | 14:12 |
| TheJulia | vdrok: I don't know if you saw in the #openstack-nova logs from friday, but melwitt was thinking it was always sending the wrong microversion even though the default was et | 14:13 |
| vdrok | TheJulia: well, I'm not sure, the bit of log dtantsur linked shows that reply was with microversion 1.0, so I think it means the request was with 1.0 as well | 14:15 |
| vdrok | now I think it is set explicitly to 1.26 everywhere inventories put happens | 14:15 |
| TheJulia | dtantsur: my opinoin is the tightly coupled jobs need it because we need to make sure we release working code at the end of the cycle, that being said, we could just use pip installed packages now that neutron releases are pushed to pypi | 14:16 |
| TheJulia | vdrok: might be pertinent to take this discussion to #openstack-nova | 14:18 |
| TheJulia | I see a patch :) | 14:21 |
| dtantsur | TheJulia: these are the two options we have, yes | 14:21 |
| dtantsur | if master is preferred (seems the case), we need py36-neutron variant of the job | 14:22 |
| dtantsur | mordred, dhellmann ^^ | 14:22 |
| dhellmann | it sounds like that's right | 14:22 |
| TheJulia | Yeah, that is what will cause things to be detected early on | 14:24 |
| TheJulia | and with the chnage frequency to plugins, earlier is much better | 14:24 |
| dtantsur | dhellmann: do you know where such job should be added? project-config? | 14:25 |
| openstackgerrit | Dmitry Tantsur proposed openstack/networking-baremetal master: import zuul job settings from project-config https://review.openstack.org/592406 | 14:26 |
| *** e0ne has quit IRC | 14:28 | |
| dtantsur | dhellmann: and unrelated question: is it intended for publish-to-pypi to stay in project-config? | 14:30 |
| dhellmann | dtantsur : yes, we want the release jobs to say in project-config for now | 14:30 |
| dtantsur | k | 14:30 |
| dhellmann | adding the job variant will require a change in project-config to define the job and then a change in openstack-zuul-jobs to define the job template | 14:31 |
| dhellmann | if you have time to write those changes, I can help get you started and then review | 14:31 |
| dhellmann | otherwise I should have time later this week | 14:31 |
| * dhellmann looks for an example | 14:32 | |
| dtantsur | dhellmann: I think I figured it out, thanks. see #infra | 14:33 |
| dhellmann | thanks, dtantsur | 14:34 |
| *** e0ne has joined #openstack-ironic | 14:35 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/networking-baremetal master: add python 3.6 unit test job https://review.openstack.org/592408 | 14:36 |
| *** r-daneel has joined #openstack-ironic | 14:37 | |
| *** mjturek has joined #openstack-ironic | 14:44 | |
| *** cdearborn has joined #openstack-ironic | 14:44 | |
| *** jiapei has joined #openstack-ironic | 14:48 | |
| *** stendulker has joined #openstack-ironic | 14:52 | |
| *** kaifeng has joined #openstack-ironic | 14:52 | |
| *** rpioso|afk is now known as rpioso | 14:58 | |
| TheJulia | #startmeeting ironic | 15:00 |
| openstack | Meeting started Mon Aug 20 15:00:09 2018 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| TheJulia | o/ | 15:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| TheJulia | Good morning everyone! | 15:00 |
| mjturek | o/ | 15:00 |
| etingof | o/ | 15:00 |
| rpioso | o/ | 15:00 |
| jroll | \o | 15:00 |
| mgoddard | o/ | 15:00 |
| stendulker | o/ | 15:00 |
| dtantsur | o/ | 15:00 |
| cdearborn | \o | 15:00 |
| bdodd | o/ | 15:00 |
| kaifeng | o/ | 15:00 |
| TheJulia | Our agenda is fairly light today, so hopefully we'll wrap up the meeting quickly, and we can all then go back and make software! | 15:00 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:00 |
| TheJulia | #topic Announcements/Reminders | 15:01 |
| *** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)" | 15:01 | |
| rloo | o/ | 15:01 |
| jiapei | o? | 15:01 |
| jiapei | o/ | 15:01 |
| TheJulia | #info 22 days until the PTG | 15:01 |
| TheJulia | #info Please ensure that you've made your hotel reservation and have registered for the PTG. The hotel room block is expected to sell out. | 15:02 |
| TheJulia | #info The deadline for submitting topics/ideas is the 23rd, which is this week. | 15:03 |
| TheJulia | #link https://etherpad.openstack.org/p/ironic-stein-ptg | 15:03 |
| * etingof discovered that the hotel rooms are already sold out as of last Friday | 15:03 | |
| etingof | (group booking) | 15:03 |
| TheJulia | #info Please +1 items that you feel are important, that will help with trying to determine a schedule and priority for items. | 15:03 |
| TheJulia | #info etingof informs us that the PTG room block has sold out. | 15:04 |
| TheJulia | :( | 15:04 |
| *** moshele has quit IRC | 15:04 | |
| TheJulia | I think that is it for announcements/reminders from my point of view. Does anyone have anything to raise? | 15:04 |
| mjturek | #info Bug day next Thurs (Aug 28th) https://etherpad.openstack.org/p/ironic-bug-day-august-28-2018 | 15:05 |
| mjturek | oh wait, it's a tuesday not a thursday | 15:05 |
| TheJulia | #info Bug day next Tuesday (Aug 28th) | 15:05 |
| TheJulia | #link https://etherpad.openstack.org/p/ironic-bug-day-august-28-2018 | 15:06 |
| TheJulia | Anything else? | 15:06 |
| mjturek | please vote for a time! Seems like it will be the standard 1300 UTC | 15:06 |
| TheJulia | That will be 6 AM for me :\ | 15:07 |
| TheJulia | +1 to >=1400 UTC | 15:07 |
| mjturek | ouuuch, yeah that would work for me but please cast vote here https://doodle.com/poll/ef4m9zmacm2ey7ce | 15:08 |
| TheJulia | #link https://doodle.com/poll/ef4m9zmacm2ey7ce | 15:08 |
| TheJulia | Okay, well I guess we can move on! | 15:08 |
| TheJulia | #topic Review action items from previous meeting | 15:09 |
| *** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)" | 15:09 | |
| TheJulia | We had one action item from last week which was for me to send an email regarding ironic-staging-drivers discussion to get that going on the mailing list. Thank you those that have replied. My apologies for not replying but last week was a bit hectic and was never able to finish writing a reply. | 15:10 |
| rloo | TheJulia: no need to apologize. thx for starting the discussion! | 15:10 |
| *** pcaruana has quit IRC | 15:11 | |
| * TheJulia still feels bad | 15:11 | |
| TheJulia | :) | 15:11 |
| TheJulia | #topic Review subteam status reports | 15:11 |
| *** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:11 | |
| TheJulia | I feel like we don't really have anything actionable to discuss regarding subteam status. We're in that end of cycle quiet time right now, although we now will permit stein patches to merge to master. | 15:12 |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:13 |
| rloo | i just deleted deploy steps, BIOS, and conductor groups | 15:14 |
| rloo | they were all done. | 15:14 |
| rloo | now i don't even remember if it was conductor groups, i've already forgotten what I deleted :) | 15:15 |
| TheJulia | lol | 15:15 |
| rloo | TheJulia: how are we doing wrt project vision? | 15:15 |
| rloo | TheJulia: can we +A it now? | 15:16 |
| TheJulia | One thing, following up on vision, we have +2s on all cores for https://review.openstack.org/#/c/587576/ (minus myself, but I wrote it) | 15:16 |
| patchbot | patch 587576 - ironic - A vision - 0h 19m 8s spent in CI | 15:16 |
| * jroll thinks TheJulia should +A that | 15:16 | |
| dtantsur | ++ | 15:16 |
| TheJulia | So I think we can +A it, I'd prefer to go ahead and get the follow-up patch posted first so I don't forget | 15:16 |
| rloo | line 239 for those that are following :) | 15:16 |
| TheJulia | Works for me | 15:16 |
| rloo | done | 15:17 |
| TheJulia | Awesome | 15:17 |
| rloo | dtantsur: classic drivers work is done?! | 15:17 |
| dtantsur | unbelievable, yeah? | 15:18 |
| TheJulia | rloo: I believe it is | 15:18 |
| jroll | \o/ | 15:18 |
| rloo | whee! | 15:18 |
| * dtantsur can now safely retire | 15:18 | |
| TheJulia | dtantsur: your keeping a tally of how many beers we need to provide to you in berlin right? | 15:18 |
| rloo | dtantsur now moves onto greater glory | 15:18 |
| dtantsur | TheJulia: all of them :) | 15:18 |
| TheJulia | \o/ | 15:18 |
| rloo | All the Steins? :) | 15:19 |
| TheJulia | Okay, lets move on to priorities for this coming week, since it is largely specs. | 15:19 |
| dtantsur | :) | 15:19 |
| * rloo deletes classic driver removal from whiteboard | 15:19 | |
| TheJulia | Are we good to proceed since I think everything else is kind of in spec land? | 15:20 |
| cdearborn | I have one | 15:20 |
| cdearborn | https://review.openstack.org/#/c/591493/ | 15:20 |
| patchbot | patch 591493 - puppet-ironic - Correct default number of threads - 3h 11m 37s spent in CI | 15:20 |
| cdearborn | can we try to get that landed this week? | 15:20 |
| rloo | who owns puppet-ironic? | 15:21 |
| dtantsur | I'm core there, but noone else I think | 15:21 |
| TheJulia | openstack-puppet I believe | 15:21 |
| TheJulia | #topic Deciding on priorities for the coming week | 15:21 |
| *** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:21 | |
| TheJulia | So regarding that, it looks like the linter was just grumpy friday. I didn't see a reason for the failure, I can recheck it and we can go from there | 15:22 |
| cdearborn | Thanks! | 15:22 |
| TheJulia | and I can reach out to the puppet-ironic ptl | 15:22 |
| dtantsur | ++ | 15:22 |
| dtantsur | I'll +2 when it passes | 15:22 |
| TheJulia | k | 15:23 |
| rloo | TheJulia: are there any patches needed for the final Rocky release, that we ought to look at? | 15:23 |
| dtantsur | (though they haven't migrated to storyboard yet) | 15:23 |
| TheJulia | rloo: I think we're in good shape, I know there is an irmc fix that was backported that we should consider releasing | 15:24 |
| dtantsur | TheJulia: it's pending their CI passing | 15:25 |
| TheJulia | Yeah, https://review.openstack.org/#/c/592247/ | 15:25 |
| rloo | ok, so we can take the week off :) | 15:25 |
| patchbot | patch 592247 - ironic (stable/rocky) - Fix not exist deploy image within irmc-virtual-med... - 3h 53m 25s spent in CI | 15:25 |
| TheJulia | rloo: specs :) | 15:25 |
| TheJulia | I'm good with the list as-is, I added the puppet ironic item. I don't think any of the doc changes got traction last week, but I think they may also need revisions | 15:26 |
| openstackgerrit | Merged openstack/ironic master: A vision https://review.openstack.org/587576 | 15:27 |
| TheJulia | Shall we move on? | 15:29 |
| rloo | +1 movin' | 15:30 |
| TheJulia | So I had a discusison item, but I'm going to defer that since we're discussing on the mailing list, so we'll go direct to open discussion. | 15:30 |
| TheJulia | #topic Open Discussion | 15:30 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:30 | |
| *** moshele has joined #openstack-ironic | 15:30 | |
| TheJulia | Anyone have anything? | 15:30 |
| TheJulia | Everyone excited about the PTG? | 15:30 |
| rpioso | I have a thing. | 15:30 |
| cdearborn | Depends on the trains... | 15:30 |
| TheJulia | cdearborn: I swore that was addressed.... | 15:31 |
| TheJulia | rpioso: sure! | 15:31 |
| mjturek | I am! The Renaissance was a nice spot for it last time | 15:31 |
| rpioso | Does defining a new clean/deploy step require an RFE or is it a just do it kind of activity? | 15:31 |
| rloo | TheJulia: if the discussion item is the staging drivers, i think we might as well finish the discussion here. can recap or whatever in mail list. | 15:31 |
| * jroll is totes excited | 15:31 | |
| rloo | rpioso: requires an RFE. | 15:32 |
| dtantsur | rpioso: RFE | 15:32 |
| TheJulia | rpioso: I think it would be a new RFE, we woudl need to better understand exactly what your attempting to achieve and if there is already the same elsewhere. | 15:32 |
| rpioso | Ack | 15:32 |
| rpioso | afaiu, it's entirely new. | 15:33 |
| TheJulia | Okay, then definitely RFE | 15:33 |
| TheJulia | rloo: So it is, I guess my other concern is the lottery factor for pas-ha and the other folks which would leave it entirely pure ironic-core folks if we get ironic-core added | 15:34 |
| rloo | not sure what you mean by lottery factor. | 15:35 |
| TheJulia | rloo: ever heard of the bus factor? | 15:35 |
| rloo | nope | 15:35 |
| TheJulia | oh my | 15:35 |
| rloo | is that an american thing? | 15:35 |
| TheJulia | Think "someone won the lottery, and they are gone forever" | 15:35 |
| rloo | oh, got it | 15:35 |
| rpioso | Has a new version of sushy been released? I'm under the impression it's needed to fix the gate. | 15:36 |
| rloo | so you are concerned if we add ironic-cores as cores to ironic-stging-drivers, that ... ironic cores will end up reviewing only? | 15:36 |
| jroll | bus factor being the more generally understood but more morbid thing, where it's someone hit by a bus instead of winning the lottery | 15:36 |
| rloo | https://en.wikipedia.org/wiki/Bus_factor | 15:36 |
| TheJulia | I can go either way, I didn't see pas-ha reply to the thread, and I feel like he would be the authoritative person at this point. | 15:36 |
| jroll | TheJulia: I don't think where the repo lives in governance affects bus factor, right? | 15:36 |
| rloo | my thoughts exactly. | 15:37 |
| rloo | the bus factor will be an issue regardless | 15:37 |
| TheJulia | I guess takeover/line of succession wise | 15:37 |
| rpioso | TheJulia: ^^^ | 15:37 |
| *** S4ren has quit IRC | 15:37 | |
| TheJulia | but if we're in agreement that we can add ironic-core, then I think that is good | 15:37 |
| rloo | are we concerned with patches not being reviewed in ironic-staging-drivers, or are we concerned with something else? | 15:37 |
| TheJulia | Similarlly we should consideer adding cores to sub-projects | 15:37 |
| rloo | i thought we were already adding cores to sub-projects? | 15:38 |
| TheJulia | rloo: my brain is focused on long term health/survivability | 15:38 |
| TheJulia | rloo: we have on a case by case basis, just we've not done it for a while afaik | 15:38 |
| *** gyee has joined #openstack-ironic | 15:38 | |
| TheJulia | rpioso: I or dtantsur need to put htat request in | 15:38 |
| TheJulia | rpioso: stable/pike correct? | 15:39 |
| rloo | i thought we had decided awhile ago, that ironic-cores were on each subproject and that was done. i don't think we've added any subprojects recently? or maybe we missed/forgot to add cores to them. | 15:39 |
| jroll | yeah, when we re-organized governance around jroll-ptl time we made ironic-core the only cores on all projects in our umbrella | 15:39 |
| TheJulia | we've kind of missed/forgotten to elevate people actively reviewing only on subprojects | 15:39 |
| rpioso | dtantsur, TheJulia: Yes, stable/pike, please. | 15:39 |
| rloo | anyway, i'm already worried about long term health/survivability of existing projects under ironic governance. | 15:39 |
| TheJulia | Anyway, elevating peoples seems like it is something I should spend time slices on soon() | 15:40 |
| rloo | survivability of ironic-staging-drivers is less importnat to me that the other projects | 15:40 |
| jroll | I think sushy is probably the first project with non-cores reviewing more there than in ironic | 15:40 |
| TheJulia | I guess we can alway scome back ot it | 15:40 |
| TheJulia | jroll: I feel like your reading my mind :) | 15:40 |
| TheJulia | s/ot/to/ | 15:40 |
| jroll | :) | 15:41 |
| rloo | oh, is that what we're talking about? yeah, each subproject should be promoting folks to cores as warranted. | 15:41 |
| TheJulia | so re; ironic-staging-drivers, lets get ironic-core added if the other cores are agreeable. That at least maintains access survivability and gives us the ability to sanity check review/approve | 15:41 |
| rloo | (do we need to vote on that? glee) | 15:41 |
| jroll | I don't mind ironic-core having access on that project, but don't take that as a commitment to care about it :) | 15:42 |
| rloo | are there other cores here besides dtantsur, jroll, myself? | 15:42 |
| TheJulia | I'm not really sure, there is nothing preventing someone from adding ironic-core to it today | 15:42 |
| jroll | mgoddard is here | 15:42 |
| rloo | ok, let's just ask the existing cores in ironic-staging-drivers if they want to add ironic-cores to it. | 15:43 |
| TheJulia | rloo: I'll take that as an action item | 15:43 |
| rloo | thx TheJulia! | 15:43 |
| jroll | we don't manage ironic-staging-drivers, they can add whatever cores they want ;D | 15:43 |
| rloo | jroll: true. we can't add us, we have to ask them to add us :) | 15:43 |
| TheJulia | #action TheJulia to follow-up with ironic-staging-drivers cores regarding adding to the group of reviewers | 15:44 |
| rloo | (if they want us, if they don't, i'm fine with that too) | 15:44 |
| TheJulia | Also, I think I need I might as well add an action item to review sushy review traffic and peform some outreach | 15:44 |
| etingof | please ^ | 15:44 |
| TheJulia | :) | 15:44 |
| dtantsur | +++ | 15:44 |
| TheJulia | #action TheJulia to review stats and perform some outreach for subprojects this week. | 15:45 |
| TheJulia | Do we have anything else to discuss? | 15:45 |
| * TheJulia feels like this has been really productive | 15:45 | |
| jroll | so did we just agree not to bring i-s-d under our wing? | 15:45 |
| * TheJulia aslways hopes for like a 20 minute meeting | 15:45 | |
| TheJulia | jroll: I believe we did agree not to, but that we can re-evaluate later. | 15:46 |
| jroll | (or not yet anyway) | 15:46 |
| jroll | yay, decisions :) | 15:46 |
| rloo | yes, we did NOT agree to bring it into govenance :) | 15:46 |
| jroll | who is going to summarize on the list? | 15:46 |
| rloo | I can do that if no one else wants to :) | 15:47 |
| mgoddard | seems like a reasonable decision on i-s-d | 15:47 |
| TheJulia | rloo: please :) | 15:47 |
| rloo | ok. i'll do it :) | 15:47 |
| jroll | thanks rloo! | 15:48 |
| TheJulia | thanks rloo! | 15:49 |
| TheJulia | so, I guess we should now all return to the software mines? | 15:49 |
| cdearborn | Can we discuss 2 RFEs that I just entered? | 15:49 |
| cdearborn | not sure of the process | 15:49 |
| TheJulia | cdearborn: I don't see why not, what do you have? | 15:49 |
| cdearborn | https://storyboard.openstack.org/#!/story/2003495 | 15:49 |
| dtantsur | I think these RFEs get automatic approve as part of consistency between drivers | 15:50 |
| cdearborn | https://storyboard.openstack.org/#!/story/2003494 | 15:50 |
| TheJulia | yeah, it is pretty much an automatic approval from my point of view since it is parity work | 15:50 |
| dtantsur | the second one needs details (in the story or a spec) | 15:50 |
| * TheJulia loads the second one | 15:50 | |
| cdearborn | ok - so no spec necessary for 2003495? | 15:50 |
| TheJulia | cdearborn: no, we have the method calls on management interfaces | 15:51 |
| dtantsur | cdearborn: no. marked it rfe-approved based on the above. | 15:51 |
| cdearborn | ok, great, thx! | 15:51 |
| TheJulia | so unless there is something super special we're unaware of.... | 15:51 |
| openstackgerrit | Paul Belanger proposed openstack/bifrost master: WIP: Remove legacy playbooks https://review.openstack.org/593677 | 15:52 |
| * dtantsur hopes no | 15:52 | |
| cdearborn | no - nothing super special on that one - should be very straight forward | 15:52 |
| TheJulia | The second, the ilo folks have some prior art of sorts in cleaning steps, but I'm leaning towards spec | 15:52 |
| dtantsur | yeah, especially since the story does not talk about iDrac specifically | 15:52 |
| cdearborn | ok, so the next step will be to come up with a spec for firmware upgrade then | 15:52 |
| TheJulia | yeah, need to completely understand the problem and desired outcome too, might be slightly different or have a different take that could really change things up | 15:53 |
| cdearborn | ok. is it ok to add it to the list of possible sessions for the PTG? | 15:53 |
| TheJulia | I think so yeah, it is a common ask so I don't think we can avoid discussing firmware management | 15:53 |
| cdearborn | ok, great! thx very much all | 15:54 |
| rloo | we talked ab it about firmware mgt in last PTG. | 15:54 |
| cdearborn | will review that material | 15:54 |
| rloo | weren't we going to try to get hw vendor agreement on some consistent something? | 15:54 |
| openstackgerrit | Paul Belanger proposed openstack/bifrost master: WIP: Remove legacy playbooks https://review.openstack.org/593677 | 15:55 |
| TheJulia | rloo: yeah, and that didn't go anywhere I think. I did talk to the folks that are doing firmware updates for EFI stuffs and they have gotten some traction, but they've run into similar roadblocks/hurdles with vendors | 15:55 |
| TheJulia | I kind of feel like eventually we could adopt their stuff as an optional cleaning step | 15:56 |
| rloo | TheJulia: ok. it would be good to be focussed on what we are going to talke about wrt firmware upgrades, so we don't say the same thing as in last PTG :) | 15:56 |
| TheJulia | but they were just getting itno server hardware as of spring | 15:56 |
| TheJulia | well, Dell folks being interested might help traction there. | 15:57 |
| TheJulia | rloo: ++++ | 15:57 |
| cdearborn | we're definitely interested! | 15:57 |
| rloo | perhaps Dell could be a leader then; prepare/let us know how you do/propose to do firmware upgrades. | 15:57 |
| cdearborn | ok, i can definitely come up with something for the PTG | 15:58 |
| TheJulia | ++ | 15:58 |
| openstackgerrit | Paul Belanger proposed openstack/bifrost master: WIP: Remove legacy playbooks https://review.openstack.org/593677 | 15:58 |
| TheJulia | Anyway, one minute left. Anyone have anything else? | 15:58 |
| * jroll throws an extra cricket into the channel | 16:00 | |
| NobodyCam | Good morning Ironic'ers | 16:00 |
| NobodyCam | :p | 16:00 |
| TheJulia | Thanks everyone! | 16:00 |
| TheJulia | jroll: we need schematics to be able to print that with a 3d printer.... | 16:00 |
| TheJulia | deploy the crickets! :) | 16:00 |
| TheJulia | Anyway have a wonderful day everyone! | 16:01 |
| jroll | TheJulia: heh, I have enough real ones in the yard for everyone | 16:01 |
| cdearborn | u2 - thx! | 16:01 |
| rpioso | Cheers | 16:01 |
| TheJulia | #endmeeting | 16:01 |
| *** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75" | 16:01 | |
| openstack | Meeting ended Mon Aug 20 16:01:28 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-08-20-15.00.html | 16:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-08-20-15.00.txt | 16:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-08-20-15.00.log.html | 16:01 |
| *** rpioso is now known as rpioso|afk | 16:01 | |
| kaifeng | Hello guys, I need some help for the patch https://review.openstack.org/#/c/584758/ | 16:02 |
| patchbot | patch 584758 - ironic-inspector - [WIP] Wrap rpc server into oslo.service - 16h 29m 50s spent in CI | 16:02 |
| rloo | rpioso|afk: wrt firmware management, here's the etherpad notes from previous PTG: https://etherpad.openstack.org/p/ironic-rocky-ptg-cleaning-evolved | 16:02 |
| rloo | oh sorry, it was cdearborn that mentioned it ^^ | 16:03 |
| kaifeng | I need to setup a rabbitmq server for functional test, but don't know how, tried bindep, but last log seems weird because that oslo.messaging used. | 16:03 |
| rpioso|afk | rloo: Thank you! We're both interested :) | 16:03 |
| * rpioso|afk recalls shivering during that discussion | 16:04 | |
| TheJulia | rpioso|afk: it wsa cold... so very cold | 16:05 |
| *** stendulker has quit IRC | 16:05 | |
| TheJulia | kaifeng: your just using olso.messaging right? | 16:05 |
| kaifeng | TheJulia yes | 16:05 |
| kaifeng | logs said 2018-08-20 11:49:02.059304 | ubuntu-xenial | ometa.runtime.ParseError: | 16:07 |
| kaifeng | 2018-08-20 11:49:02.059390 | ubuntu-xenial | rabbitmq-server [platform:rpm] | 16:07 |
| TheJulia | kaifeng: I kind of feel like contract verification on each side might be what you want to do instead of testing that messgaes pass through rabbitmq... not exactly functional testing though | 16:07 |
| TheJulia | kaifeng: guest/guest might work, but that is going to be packager dependent | 16:07 |
| kaifeng | yes, the it's the default account, but inspector uses openstack-tox-functional, it doesn't install rabbitmq | 16:08 |
| kaifeng | TheJulia: because the patch is splitting api/worker into difference processes, so the rpc is the only way to communicate | 16:09 |
| dtantsur | kaifeng: oslo.msg has some kind of a fake in-memory driver | 16:09 |
| dtantsur | I think the plan was to use it | 16:09 |
| *** moshele has quit IRC | 16:09 | |
| kaifeng | dtantsur: you mean the fake:// one? I don't think so, it's instance level | 16:10 |
| openstackgerrit | Paul Belanger proposed openstack/bifrost master: WIP: Remove legacy playbooks https://review.openstack.org/593677 | 16:10 |
| TheJulia | I think we've reached the "why functional testing for split services is hard" | 16:10 |
| kaifeng | So that why ironic didn't support functional test? ;) | 16:11 |
| dtantsur | kaifeng: I still don't see a problem. you have one process with API and worker co-existing, why not use fake:// for it? | 16:14 |
| kaifeng | dtantsur: I tried, it doesn't work.. | 16:15 |
| TheJulia | dtantsur: because you actually need to fire both up inside the same test runtime aiui | 16:15 |
| kaifeng | actually two processes now | 16:15 |
| dtantsur | kaifeng: should be one process, that's your problem | 16:15 |
| dtantsur | otherwise it's hard to solve. we definitely should not start depending on rabbit for functional tests | 16:15 |
| TheJulia | yeah, the test has to load both parts and pass the info through rpc. I guess you will end up with portions of each side mocked that things crossed back over as expected | 16:16 |
| *** e0ne has quit IRC | 16:16 | |
| *** anupn has joined #openstack-ironic | 16:17 | |
| kaifeng | Oh I thought it was splitted services, shouldn't be that/ | 16:17 |
| kaifeng | ? | 16:17 |
| kaifeng | From the | 16:18 |
| kaifeng | I was thinking two services from the beginning | 16:19 |
| *** strigazi has quit IRC | 16:19 | |
| dtantsur | kaifeng: functional tests are designed to quickly work on a laptop, so it's fine | 16:21 |
| dtantsur | we will exercise the actual split between processes on the integration jobs | 16:22 |
| TheJulia | ++ | 16:22 |
| kaifeng | ok, got it, no rabbitmq then. | 16:24 |
| *** strigazi has joined #openstack-ironic | 16:24 | |
| kaifeng | but I am still not clear on the next move, how would I perform the functional test? | 16:24 |
| kaifeng | should I use thread to start both service instead of launch service from oslo.service? | 16:27 |
| openstackgerrit | Paul Belanger proposed openstack/bifrost master: WIP: Remove legacy playbooks https://review.openstack.org/593677 | 16:27 |
| dtantsur | kaifeng: a service per thread, I think. | 16:29 |
| kaifeng | dtantsur: thanks, I'll have a try | 16:31 |
| kaifeng | TheJulia dtantsur: thanks for the help, good night | 16:34 |
| *** kaifeng has quit IRC | 16:35 | |
| rpioso|afk | dtantsur: In case you hadn't noticed, yet, https://review.openstack.org/#/c/588949/ passed the Dell EMC CI. | 16:35 |
| patchbot | patch 588949 - ironic (stable/ocata) - Fix iDRAC hardware type does not work with UEFI - 0h 52m 36s spent in CI | 16:35 |
| *** trown is now known as trown|lunch | 16:35 | |
| * rpioso|afk is heading to the office | 16:35 | |
| *** mgoddard has quit IRC | 16:37 | |
| openstackgerrit | Paul Belanger proposed openstack/bifrost master: WIP: Remove legacy playbooks https://review.openstack.org/593677 | 16:37 |
| openstackgerrit | Paul Belanger proposed openstack/bifrost master: WIP: Remove legacy playbooks https://review.openstack.org/593677 | 16:53 |
| *** w-miller has quit IRC | 16:57 | |
| *** w-miller has joined #openstack-ironic | 16:58 | |
| *** w-miller has quit IRC | 17:02 | |
| *** skazi has quit IRC | 17:04 | |
| *** e0ne has joined #openstack-ironic | 17:05 | |
| openstackgerrit | Paul Belanger proposed openstack/bifrost master: WIP: Remove legacy playbooks https://review.openstack.org/593677 | 17:10 |
| *** e0ne has quit IRC | 17:15 | |
| *** moshele has joined #openstack-ironic | 17:16 | |
| *** dtantsur is now known as dtantsur|afk | 17:17 | |
| dtantsur|afk | o/ | 17:17 |
| *** sambetts is now known as sambetts|afk | 17:20 | |
| sambetts|afk | night all | 17:20 |
| * TheJulia thinks she is done sending emails (for now...) | 17:26 | |
| *** e0ne has joined #openstack-ironic | 17:42 | |
| openstackgerrit | Paul Belanger proposed openstack/bifrost master: WIP: Remove legacy playbooks https://review.openstack.org/593677 | 17:42 |
| *** tssurya has quit IRC | 17:43 | |
| *** trown|lunch is now known as trown | 17:48 | |
| openstackgerrit | Merged openstack/ironic master: import zuul job settings from project-config https://review.openstack.org/592383 | 17:49 |
| *** e0ne has quit IRC | 17:51 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Clear GPT and MBR headers with dd to avoid sgdisk CRC errors https://review.openstack.org/589831 | 17:58 |
| openstackgerrit | Paul Belanger proposed openstack/bifrost master: WIP: Remove legacy playbooks https://review.openstack.org/593677 | 17:59 |
| *** r-daneel has quit IRC | 18:06 | |
| *** jiapei has quit IRC | 18:08 | |
| *** r-daneel has joined #openstack-ironic | 18:08 | |
| cdearborn | rloo, thanks for the link! | 18:10 |
| rloo | cdearborn: yw :) | 18:10 |
| *** e0ne has joined #openstack-ironic | 18:14 | |
| openstackgerrit | Merged openstack/ironic-python-agent stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590322 | 18:26 |
| openstackgerrit | Paul Belanger proposed openstack/bifrost master: WIP: Remove legacy playbooks https://review.openstack.org/593677 | 18:26 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Minor fixes to contributor vision https://review.openstack.org/593736 | 18:30 |
| *** rpioso|afk is now known as rpioso | 18:30 | |
| rpioso | TheJulia: When you have an opportunity, please review https://review.openstack.org/#/c/588949/ It's a backport of the iDRAC UEFI bug fix to stable/ocata. | 18:34 |
| patchbot | patch 588949 - ironic (stable/ocata) - Fix iDRAC hardware type does not work with UEFI - 0h 52m 36s spent in CI | 18:34 |
| TheJulia | rpioso: okay | 18:34 |
| rpioso | TheJulia: ty | 18:34 |
| TheJulia | btw, putting that sushy release request together now | 18:35 |
| rpioso | You read my mind! | 18:35 |
| rpioso | Thanks much. | 18:35 |
| *** r-daneel has quit IRC | 18:37 | |
| *** r-daneel has joined #openstack-ironic | 18:37 | |
| openstackgerrit | Merged openstack/python-ironicclient stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/585855 | 18:37 |
| TheJulia | pushing up now | 18:39 |
| TheJulia | so... release team will review it, hopefully tomorrow it will be released | 18:39 |
| TheJulia | arne_wiebalck: By chance did you get back to https://review.openstack.org/#/c/592639/ ? | 18:41 |
| patchbot | patch 592639 - ironic-python-agent - Fix multi-device behavior - 2h 13m 29s spent in CI | 18:41 |
| TheJulia | well, not back to as in updating, but figuring out what the issue is | 18:42 |
| *** e0ne has quit IRC | 18:42 | |
| rpioso | TheJulia: Looking forward to it. Thanks, again! | 18:51 |
| openstackgerrit | Julia Kreger proposed openstack/python-ironicclient master: Switch ci jobs to Python3 https://review.openstack.org/591835 | 19:03 |
| *** e0ne has joined #openstack-ironic | 19:06 | |
| *** serlex has joined #openstack-ironic | 19:10 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/585996 | 19:13 |
| openstackgerrit | Julia Kreger proposed openstack/sushy-tools master: Switch sushy-tools devstack job to python3 https://review.openstack.org/591831 | 19:13 |
| *** serlex has quit IRC | 19:14 | |
| openstackgerrit | Paul Belanger proposed openstack/bifrost master: WIP: Remove legacy playbooks https://review.openstack.org/593677 | 19:16 |
| *** moshele has quit IRC | 19:25 | |
| *** e0ne has quit IRC | 19:26 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: import zuul job settings from project-config https://review.openstack.org/592393 | 19:30 |
| *** moshele has joined #openstack-ironic | 19:31 | |
| *** jtomasek has quit IRC | 19:36 | |
| *** r-daneel_ has joined #openstack-ironic | 19:45 | |
| *** mjturek has quit IRC | 19:46 | |
| *** r-daneel has quit IRC | 19:47 | |
| *** r-daneel_ is now known as r-daneel | 19:47 | |
| openstackgerrit | Julia Kreger proposed openstack/networking-generic-switch master: Switch n-g-s devstack job to python3 https://review.openstack.org/591826 | 19:48 |
| *** moshele has quit IRC | 19:53 | |
| openstackgerrit | Julia Kreger proposed openstack/sushy master: Change sushy devstack job to python3 https://review.openstack.org/591828 | 19:54 |
| *** r-daneel_ has joined #openstack-ironic | 20:02 | |
| *** r-daneel has quit IRC | 20:03 | |
| *** r-daneel_ is now known as r-daneel | 20:03 | |
| openstackgerrit | Julia Kreger proposed openstack/virtualbmc master: Change CI job to use python3 https://review.openstack.org/589296 | 20:10 |
| *** jtomasek has joined #openstack-ironic | 20:21 | |
| *** jtomasek has quit IRC | 20:24 | |
| *** jtomasek has joined #openstack-ironic | 20:31 | |
| *** jtomasek has quit IRC | 20:34 | |
| openstackgerrit | Merged openstack/ironic-python-agent stable/rocky: import zuul job settings from project-config https://review.openstack.org/592464 | 20:35 |
| openstackgerrit | Merged openstack/ironic-python-agent stable/queens: import zuul job settings from project-config https://review.openstack.org/592450 | 20:35 |
| openstackgerrit | Merged openstack/ironic-python-agent stable/pike: import zuul job settings from project-config https://review.openstack.org/592439 | 20:35 |
| *** jcoufal has quit IRC | 20:35 | |
| *** r-daneel has quit IRC | 20:44 | |
| *** r-daneel has joined #openstack-ironic | 20:50 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add admin-only provisioner reference arch https://review.openstack.org/586618 | 21:01 |
| *** trown is now known as trown|outtypewww | 21:02 | |
| *** baha has quit IRC | 21:09 | |
| *** cdearborn has quit IRC | 21:45 | |
| *** rcernin has joined #openstack-ironic | 21:50 | |
| rpioso | TheJulia: I'm wondering if an RFE is required, preferred, or not needed for adding a supported interface to a h/w type. The default interface would not be changed. | 22:13 |
| TheJulia | rpioso: in those cases, i think just a patch with a reno | 22:14 |
| TheJulia | It is already supported elsewhere, so im bot aure there is a case to deny addition to a hardware type | 22:15 |
| rpioso | TheJulia: It's for adding the Redfish management and power interfaces to the idrac h/w type. | 22:16 |
| rpioso | So, we're looking at adding support for existing interfaces to an existing h/w type. | 22:17 |
| TheJulia | I think that is fine. The key is going to be delineating things like uefi support | 22:17 |
| rpioso | TheJulia: For idrac, I believe there's no overlap. Just pick one of the supported management interfaces, both of which support UEFI. | 22:22 |
| *** r-daneel has quit IRC | 22:24 | |
| rpioso | TheJulia: There may be a need to enhance tripleo's support of the idrac h/w type, but that's off this channel's topic. The interfaces expect their creds to be on the ironic node object. | 22:26 |
| TheJulia | rpioso: so an rfe worthy item would be credential reuse so one username/password pair | 22:49 |
| rpioso | TheJulia: We don't plan to do that, because the interface implementations would have to change. The idrac WS-Man interfaces look for 'drac_username' and 'drac_password'. The Redfish ones similarly look for 'redfish_username' and 'redfish_password'. | 22:53 |
| rpioso | While our operators should have fun using ironic, that would be too much fun ;-) | 22:54 |
| TheJulia | Yes, but it makes no sense to copy the vars to have two different clients active. | 22:55 |
| *** moshele has joined #openstack-ironic | 22:55 | |
| rpioso | TheJulia: If memory serves, the idrac props formerly had more generic names. Subsequently, they were given the drac_ prologue. | 22:58 |
| TheJulia | Yeah, i kind of remember that, but that was before the days of hardware types | 23:00 |
| TheJulia | Worth revisiting i think | 23:00 |
| rpioso | So, is the answer, "Maybe, maybe not.", and we should discuss it in channel? | 23:01 |
| *** pmannidi has joined #openstack-ironic | 23:29 | |
| *** rbudden has joined #openstack-ironic | 23:57 | |
| *** rbudden has quit IRC | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!