| *** mgkwill_ has joined #openstack-ironic | 00:04 | |
| *** zul has quit IRC | 00:04 | |
| *** timirnich_ has joined #openstack-ironic | 00:05 | |
| *** ildikov_ has joined #openstack-ironic | 00:05 | |
| *** sergek__ has joined #openstack-ironic | 00:05 | |
| *** patrickeast_ has joined #openstack-ironic | 00:05 | |
| *** patrickeast has quit IRC | 00:05 | |
| *** patrickeast_ is now known as patrickeast | 00:05 | |
| *** mgkwill has quit IRC | 00:05 | |
| *** mgkwill_ is now known as mgkwill | 00:05 | |
| *** csatari_ has joined #openstack-ironic | 00:05 | |
| *** csatari has quit IRC | 00:05 | |
| *** timirnich has quit IRC | 00:05 | |
| *** ildikov has quit IRC | 00:05 | |
| *** sergek_ has quit IRC | 00:05 | |
| *** csatari_ is now known as csatari | 00:05 | |
| *** timirnich_ is now known as timirnich | 00:05 | |
| *** ildikov_ is now known as ildikov | 00:05 | |
| *** sergek__ is now known as sergek_ | 00:05 | |
| *** mordred has quit IRC | 00:06 | |
| *** zigo has quit IRC | 00:06 | |
| *** afazekas has quit IRC | 00:06 | |
| *** melwitt has quit IRC | 00:06 | |
| *** zigo_ has joined #openstack-ironic | 00:06 | |
| *** r-daneel has quit IRC | 00:07 | |
| *** cz2 has quit IRC | 00:10 | |
| *** melwitt has joined #openstack-ironic | 00:11 | |
| *** afazekas has joined #openstack-ironic | 00:11 | |
| *** cz2 has joined #openstack-ironic | 00:13 | |
| *** zul has joined #openstack-ironic | 00:16 | |
| *** mordred has joined #openstack-ironic | 00:16 | |
| *** yolanda_ has quit IRC | 00:19 | |
| *** yolanda_ has joined #openstack-ironic | 00:20 | |
| *** rwsu has quit IRC | 00:20 | |
| *** vabada has quit IRC | 00:21 | |
| hshiina | TheJulia: i will update the nova rescue patch | 00:24 |
|---|---|---|
| hshiina | this passes a list of api version when creating a client now | 00:24 |
| hshiina | should i change this to pass a version per call with depending on the ongoing client change? https://review.openstack.org/#/c/557850 | 00:25 |
| patchbot | patch 557850 - python-ironicclient - Support per-call version: set_provision_state | 00:25 |
| *** jovial[m] has quit IRC | 00:37 | |
| *** retr0h has quit IRC | 00:37 | |
| *** mkrai has quit IRC | 00:37 | |
| *** csmart has quit IRC | 00:38 | |
| *** melwitt is now known as Guest48153 | 00:39 | |
| *** vabada has joined #openstack-ironic | 00:39 | |
| *** rwsu has joined #openstack-ironic | 00:45 | |
| *** trungnv_ has quit IRC | 00:53 | |
| *** trungnv_ has joined #openstack-ironic | 00:54 | |
| *** harlowja has quit IRC | 01:00 | |
| *** phuongnh has joined #openstack-ironic | 01:06 | |
| *** jovial[m] has joined #openstack-ironic | 01:06 | |
| *** phuongnh has quit IRC | 01:08 | |
| *** phuongnh has joined #openstack-ironic | 01:10 | |
| openstackgerrit | hyunsik Yang proposed openstack/virtualbmc master: vBMC for openstack instance https://review.openstack.org/570737 | 01:11 |
| *** tiendc has joined #openstack-ironic | 01:11 | |
| openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Support raid configuration for BM via irmc driver https://review.openstack.org/512979 | 01:17 |
| *** cjloader has joined #openstack-ironic | 01:21 | |
| *** cjloader has quit IRC | 01:26 | |
| *** gyankum has joined #openstack-ironic | 01:29 | |
| *** mordred has quit IRC | 01:31 | |
| *** gyee has quit IRC | 01:31 | |
| openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Fix authentication issues along with add multi extra volumes https://review.openstack.org/570145 | 01:40 |
| *** mordred has joined #openstack-ironic | 01:45 | |
| openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Make method public to support out-of-band cleaning https://review.openstack.org/570109 | 01:47 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Stop passing IP address to IPA by PXE https://review.openstack.org/570371 | 02:04 |
| *** mkrai has joined #openstack-ironic | 02:07 | |
| *** retr0h has joined #openstack-ironic | 02:07 | |
| *** csmart has joined #openstack-ironic | 02:07 | |
| *** rajinir has joined #openstack-ironic | 02:17 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: DNM: Run a test with PXE and CoreOS with fix https://review.openstack.org/571063 | 02:18 |
| *** zhangfei has joined #openstack-ironic | 02:43 | |
| *** namnh has joined #openstack-ironic | 02:48 | |
| *** harlowja has joined #openstack-ironic | 03:29 | |
| *** links has joined #openstack-ironic | 04:11 | |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic master: [WIP] Add CUSTOM_CPU_FPGA to Traits https://review.openstack.org/569377 | 04:12 |
| *** harlowja has quit IRC | 04:14 | |
| *** cjloader has joined #openstack-ironic | 04:19 | |
| *** cjloader has quit IRC | 04:24 | |
| *** fragatina has quit IRC | 04:31 | |
| *** fragatina has joined #openstack-ironic | 04:33 | |
| *** fragatina has quit IRC | 04:37 | |
| *** marios has joined #openstack-ironic | 04:40 | |
| *** hshiina2 has joined #openstack-ironic | 04:41 | |
| *** hshiina has quit IRC | 04:41 | |
| openstackgerrit | Merged openstack/ironic master: Add Node BIOS support - REST API https://review.openstack.org/512579 | 04:47 |
| *** IgorYozhikov has quit IRC | 04:51 | |
| *** rbartal has joined #openstack-ironic | 04:52 | |
| *** e0ne has joined #openstack-ironic | 05:00 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic-tempest-plugin master: Add manual clean step ironic standalone test https://review.openstack.org/568616 | 05:10 |
| yolanda_ | hi TheJulia , it was a leftover, i modified the commit message | 05:10 |
| *** hshiina2 is now known as hshiina | 05:10 | |
| *** fragatina has joined #openstack-ironic | 05:18 | |
| *** lifeless_ has quit IRC | 05:19 | |
| *** lifeless has joined #openstack-ironic | 05:19 | |
| *** IgorYozhikov has joined #openstack-ironic | 05:20 | |
| openstackgerrit | Merged openstack/ironic master: Reraise exception with converting node ID https://review.openstack.org/569763 | 05:57 |
| *** IgorYozhikov has quit IRC | 05:59 | |
| *** mjura has joined #openstack-ironic | 06:01 | |
| *** e0ne has quit IRC | 06:07 | |
| *** akhilaki has joined #openstack-ironic | 06:09 | |
| *** hjensas has quit IRC | 06:14 | |
| *** akhilaki has quit IRC | 06:17 | |
| *** IgorYozhikov has joined #openstack-ironic | 06:18 | |
| *** cjloader has joined #openstack-ironic | 06:19 | |
| *** e0ne has joined #openstack-ironic | 06:22 | |
| *** lifeless has quit IRC | 06:23 | |
| *** cjloader has quit IRC | 06:24 | |
| *** e0ne has quit IRC | 06:25 | |
| *** lifeless has joined #openstack-ironic | 06:25 | |
| *** jtomasek has joined #openstack-ironic | 06:25 | |
| *** rajinir has quit IRC | 06:27 | |
| *** jtomasek has quit IRC | 06:28 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 06:32 | |
| *** pcaruana has joined #openstack-ironic | 06:33 | |
| *** hjensas has joined #openstack-ironic | 06:56 | |
| *** hjensas has quit IRC | 06:56 | |
| *** hjensas has joined #openstack-ironic | 06:56 | |
| *** milan has joined #openstack-ironic | 06:57 | |
| openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: add admin doc https://review.openstack.org/568504 | 07:02 |
| *** milan has quit IRC | 07:03 | |
| *** ajya has joined #openstack-ironic | 07:07 | |
| *** rbartal has quit IRC | 07:14 | |
| *** tesseract has joined #openstack-ironic | 07:20 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: WIP: Add the ability to setup enabled bios interfaces in devstack https://review.openstack.org/571115 | 07:23 |
| *** racedo has joined #openstack-ironic | 07:25 | |
| *** jtomasek has joined #openstack-ironic | 07:25 | |
| openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Fix authentication issues along with add multi extra volumes https://review.openstack.org/570145 | 07:26 |
| *** rcernin has quit IRC | 07:27 | |
| *** rbartal has joined #openstack-ironic | 07:27 | |
| *** jtomasek has quit IRC | 07:31 | |
| *** dougsz has joined #openstack-ironic | 07:42 | |
| *** zhangfei has quit IRC | 07:53 | |
| *** zhangfei has joined #openstack-ironic | 08:05 | |
| *** mjura has quit IRC | 08:08 | |
| *** mjura has joined #openstack-ironic | 08:08 | |
| *** mgoddard has joined #openstack-ironic | 08:11 | |
| *** cjloader has joined #openstack-ironic | 08:19 | |
| *** cjloader has quit IRC | 08:24 | |
| openstackgerrit | Nguyen Hai proposed openstack/python-ironic-inspector-client master: add lower-constraints job https://review.openstack.org/556139 | 08:29 |
| *** zhangfei has quit IRC | 08:29 | |
| openstackgerrit | Nguyen Hai proposed openstack/python-ironic-inspector-client master: add lower-constraints job https://review.openstack.org/556139 | 08:29 |
| *** skazi has quit IRC | 08:35 | |
| *** rbartal has quit IRC | 08:36 | |
| *** MattMan has quit IRC | 08:40 | |
| *** MattMan has joined #openstack-ironic | 08:40 | |
| *** zhangfei has joined #openstack-ironic | 08:41 | |
| yolanda_ | hi, good morning. I'm writing some cleanup tests, but there is one step, the check_partitions_present, that seems to be very fragile. It times out very often | 08:44 |
| yolanda_ | http://logs.openstack.org/16/568616/26/check/ironic-dsvm-standalone-queens/2aa7b05/job-output.txt.gz | 08:44 |
| yolanda_ | it is executing a check with virt-partitions, and seems to timeout on the subprocess call | 08:44 |
| *** lifeless_ has joined #openstack-ironic | 08:44 | |
| yolanda_ | how could i workaround it, or increase that timeout? maybe we can do another test that is not that risky? | 08:44 |
| *** lifeless has quit IRC | 08:44 | |
| *** rbartal has joined #openstack-ironic | 08:49 | |
| *** Alexey_Abashkin has joined #openstack-ironic | 08:50 | |
| *** Alexey_Abashkin has quit IRC | 08:51 | |
| *** AlexeyAbashkin has quit IRC | 08:51 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 08:52 | |
| *** mrunge has quit IRC | 08:56 | |
| openstackgerrit | ShangXiao proposed openstack/ironic-tempest-plugin master: Update the home-page url in setup.cfg https://review.openstack.org/571142 | 09:03 |
| *** fragatina has quit IRC | 09:05 | |
| *** fragatina has joined #openstack-ironic | 09:06 | |
| *** milan_ has joined #openstack-ironic | 09:16 | |
| *** gcb has joined #openstack-ironic | 09:19 | |
| *** jtomasek has joined #openstack-ironic | 09:24 | |
| *** jtomasek has quit IRC | 09:25 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic-tempest-plugin master: Add manual clean step ironic standalone test https://review.openstack.org/568616 | 09:31 |
| *** dtantsur|afk is now known as dtantsur | 09:38 | |
| dtantsur | morning ironic | 09:38 |
| vdrok | morning ironic, dtantsur | 09:55 |
| *** zigo_ is now known as zigo | 09:56 | |
| vdrok | yolanda_: morning, maybe would be safer to pass timeout to the communicate call and then try a couple more times if that timeout happens | 10:01 |
| yolanda_ | vdrok, why does it take that long, you know? when i test locally, it's quite fast | 10:04 |
| *** namnh has quit IRC | 10:08 | |
| *** cjloader has joined #openstack-ironic | 10:19 | |
| arnewiebalck__ | Good morning Ironic! Which interfaces have to pass the validate check for a node to have a workload scheduled to it? The enrollment guide says “power” and “deploy”, the troubleshooting guide seems to indicate “power” and “management”. | 10:22 |
| *** sambetts|afk is now known as sambetts | 10:23 | |
| sambetts | arnewiebalck__: power, management and deploy | 10:23 |
| *** cjloader has quit IRC | 10:23 | |
| arnewiebalck__ | sambetts: thx, so both docs are not fully correct? ;) | 10:23 |
| dtantsur | arnewiebalck__, sambetts, not exactly, deploy interface is not valid until nova populates some fields | 10:26 |
| dtantsur | if you don't use nova - then you have to populate it yourself | 10:26 |
| dtantsur | so, power and management (and maybe boot?) is the right answer in nova case | 10:27 |
| sambetts | oh yeah forgot boot :/ | 10:27 |
| arnewiebalck__ | dtantsur: thx. So, https://docs.openstack.org/project-install-guide/baremetal/draft/enrollment.html needs an update then? | 10:28 |
| openstackgerrit | zenghui.shi proposed openstack/python-ironicclient master: BIOS Settings support https://review.openstack.org/569318 | 10:28 |
| dtantsur | arnewiebalck__: which part? also this link is outdated | 10:29 |
| arnewiebalck__ | (as it says in the 2nd Note: “Any bare metal node that is visible to the Compute service may have a workload scheduled to it, if both the power and deploy interfaces pass the validate check.”) | 10:29 |
| dtantsur | arnewiebalck__: use https://docs.openstack.org/ironic/latest/install/ instead | 10:29 |
| dtantsur | does it ^^^ have the same issue? | 10:29 |
| sambetts | no its updated: https://docs.openstack.org/ironic/latest/install/enrollment.html#validating-node-information | 10:30 |
| *** AlexeyAbashkin has quit IRC | 10:30 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 10:31 | |
| sambetts | ah the note at the top though still only mentions power and deploy... | 10:31 |
| arnewiebalck__ | I think it has the same issue: https://docs.openstack.org/ironic/latest/install/enrollment.html#enrollment-process | 10:31 |
| * dtantsur asks on infra to take the old version down | 10:31 | |
| dtantsur | yep, good catch. wanna propose a patch? | 10:32 |
| arnewiebalck__ | sure! | 10:32 |
| *** lifeless_ has quit IRC | 10:32 | |
| arnewiebalck__ | dtantsur: sambetts: thx | 10:32 |
| sambetts | no problem :) | 10:32 |
| *** AlexeyAbashkin has quit IRC | 10:36 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: add lower-constraints job https://review.openstack.org/556139 | 10:39 |
| *** gyankum has quit IRC | 10:41 | |
| *** mrunge has joined #openstack-ironic | 10:44 | |
| dtantsur | sambetts: mind reviewing https://review.openstack.org/#/c/555922/ please? | 10:45 |
| patchbot | patch 555922 - networking-baremetal - add lower-constraints job | 10:45 |
| dtantsur | these patches tend to get outdated rather quickly | 10:45 |
| sambetts | dtantsur: I'll take a look now | 10:45 |
| sambetts | dtantsur: looks good | 10:46 |
| dtantsur | ditto https://review.openstack.org/#/c/555572/ | 10:46 |
| patchbot | patch 555572 - ironic-ui - add lower-constraints job | 10:46 |
| *** adreznec has quit IRC | 11:00 | |
| openstackgerrit | Arne Wiebalck proposed openstack/ironic master: [Doc] Scheduling needs validated 'management' interface https://review.openstack.org/571170 | 11:00 |
| *** lifeless has joined #openstack-ironic | 11:01 | |
| *** adreznec has joined #openstack-ironic | 11:01 | |
| *** phuongnh has quit IRC | 11:09 | |
| *** erolg has joined #openstack-ironic | 11:15 | |
| sambetts | dtantsur: done :) | 11:18 |
| *** AlexeyAbashkin has joined #openstack-ironic | 11:21 | |
| *** bfournie has quit IRC | 11:21 | |
| *** bfournie has joined #openstack-ironic | 11:21 | |
| *** liuzz_ has quit IRC | 11:22 | |
| *** liuzz has joined #openstack-ironic | 11:22 | |
| *** zhangfei has quit IRC | 11:23 | |
| openstackgerrit | Merged openstack/ironic-ui master: add lower-constraints job https://review.openstack.org/555572 | 11:25 |
| *** derekh has joined #openstack-ironic | 11:25 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic-tempest-plugin master: Add manual clean step ironic standalone test https://review.openstack.org/568616 | 11:25 |
| *** bfournie has quit IRC | 11:26 | |
| vdrok | yolanda_: not sure, maybe because we don't have nested kvm on the gate? | 11:34 |
| yolanda_ | i tried with the timeout approach, to see if that helps | 11:35 |
| yolanda_ | i'm thinking that the times when it succeeds, it may be because it's running on some specific cloud | 11:36 |
| yolanda_ | and fails for others | 11:36 |
| vdrok | yeah possibly, what's currently in the patch should help I think | 11:36 |
| *** lifeless_ has joined #openstack-ironic | 11:37 | |
| *** lifeless has quit IRC | 11:38 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add the ability to setup enabled bios interfaces in devstack https://review.openstack.org/571115 | 11:40 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-tempest-plugin master: Add manual clean step ironic standalone test https://review.openstack.org/568616 | 11:52 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-tempest-plugin master: Add manual clean step ironic standalone test https://review.openstack.org/568616 | 11:53 |
| *** tiendc has quit IRC | 11:53 | |
| *** gyankum has joined #openstack-ironic | 11:54 | |
| *** dprince has joined #openstack-ironic | 11:55 | |
| *** mikal has joined #openstack-ironic | 11:57 | |
| *** mikal_ has quit IRC | 11:59 | |
| *** bfournie has joined #openstack-ironic | 12:00 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Stop enabling the "fake" classic driver by default in tests https://review.openstack.org/570996 | 12:05 |
| vdrok | dtantsur: you're really having a good time with the unittests :D | 12:13 |
| vdrok | that is huuge | 12:13 |
| dtantsur | vdrok: yep, it's a nightmare :( I've removed W-1 btw, so it's up for review | 12:22 |
| dtantsur | ideally before something conflicts with it | 12:22 |
| vdrok | will take a look by the eod | 12:22 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Remove the fake and fake_soft_power drivers https://review.openstack.org/571191 | 12:28 |
| *** cz2 has quit IRC | 12:29 | |
| TheJulia | Good morning | 12:30 |
| *** cz2 has joined #openstack-ironic | 12:30 | |
| dtantsur | morning TheJulia | 12:31 |
| TheJulia | yolanda_: vdrok: afaik, we do not, but it is provider specific as well | 12:31 |
| * dtantsur -> food (and cry in the corner about our unit tests) | 12:31 | |
| *** trown|outtypewww is now known as trown | 12:31 | |
| *** serlex has joined #openstack-ironic | 12:32 | |
| yolanda_ | that may explain. Then i wonder if that tests is going to be possible, or it's going to fail so often | 12:34 |
| *** e0ne has joined #openstack-ironic | 12:36 | |
| *** lifeless_ has quit IRC | 12:41 | |
| *** rbudden has quit IRC | 12:44 | |
| *** hshiina is now known as hshiina|afk | 12:45 | |
| *** rloo has joined #openstack-ironic | 12:58 | |
| *** ajya has quit IRC | 13:01 | |
| *** ajya has joined #openstack-ironic | 13:02 | |
| *** mjturek has joined #openstack-ironic | 13:02 | |
| TheJulia | \o/ train tickets printed | 13:03 |
| * TheJulia feels ready for her next trip..... minus the packed suitcase | 13:03 | |
| dtantsur | hah, I haven't started packing for Prague yet :) | 13:04 |
| TheJulia | yolanda_: hmmm we could make it an optional test and run it on our gate perhaps? | 13:04 |
| * TheJulia is not sure what would be the best thing to do with a variable long running test like that.. | 13:05 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Implements baremetal inspect abort https://review.openstack.org/565687 | 13:05 |
| TheJulia | yolanda_: have you tried bumping the timeouts temporarily to just see how long it runs? | 13:05 |
| *** jcoufal has joined #openstack-ironic | 13:05 | |
| *** mjturek has quit IRC | 13:06 | |
| yolanda_ | TheJulia, this was running just without timeouts. It was failed at the longest in for process.communicate | 13:06 |
| yolanda_ | not related to our timeouts for nodes | 13:06 |
| TheJulia | but for the actual cleaning action? | 13:06 |
| * TheJulia looks at the messages on the mailing list thread | 13:07 | |
| yolanda_ | TheJulia, but it was not timing out on the clean action, but on the process execution itself | 13:07 |
| TheJulia | and your clean step is just the metadata wipe right? | 13:08 |
| * TheJulia doesn't remember right now | 13:08 | |
| yolanda_ | TheJulia, yes. But what takes time is just the virt-filesystems command | 13:08 |
| yolanda_ | the cleanup step just completes, but the check times out | 13:08 |
| TheJulia | so your trying to check the disk file contents to verify the content is gone? | 13:09 |
| yolanda_ | YES | 13:09 |
| yolanda_ | yes | 13:09 |
| TheJulia | that violates the boundry :( | 13:09 |
| TheJulia | makes total sense in our case | 13:10 |
| TheJulia | but also means someone can't run that test against baremetal | 13:10 |
| sambetts | yeah adding a test like that will not work in third party CI | 13:11 |
| yolanda_ | so can we have some alternative to check that step? | 13:11 |
| TheJulia | we could accept just the action and not the verification for now, and follow-up with something using ramdisk booting on the same node afterwards | 13:11 |
| TheJulia | the cleaning should fail if any part in the underlying process fails, which also helps, the verification is really just going to ensure that our python code down to the c code in the disk/block device utilities are doing the right thing | 13:12 |
| yolanda_ | ok so i can start just removing those validations, and the test should be "cleanup is not failing" | 13:13 |
| sambetts | that seems like something a functional test on IPA should exercise | 13:13 |
| TheJulia | yolanda_: that seems reasonable | 13:14 |
| TheJulia | sambetts: "hi IPA, give me data about the disk contents" | 13:14 |
| TheJulia | can we just create imaging at that point? *ducks* | 13:14 |
| yolanda_ | TheJulia, ok i will amend that patch. It passed very few times | 13:14 |
| TheJulia | yolanda_: thanks! | 13:14 |
| sambetts | TheJulia: I was thinking more a test job that runs IPA locally without devstack, and just exercises the hardware manager actions against local ramfs volume or something | 13:15 |
| sambetts | TheJulia: that validates IPA is working correctly, and then the ironic job can assume IPA is working | 13:16 |
| sambetts | basically taking the same approach as we do with UTs where you break it down, so the more complicated test cases can just assume the internals are doing what they should be | 13:16 |
| dtantsur | TheJulia: when we have boot-into-ramdisk, we can use it to validate that the disk is clean | 13:17 |
| TheJulia | sambetts: people run these tests on long lived systems though, a reconfiguration could go very sideways fast and at that point it would be a pure functional test of IPA outside of tempest which does kind of make sense. | 13:19 |
| * TheJulia thinks were talking about the same thing basically | 13:19 | |
| sambetts | TheJulia: yeah I'm talking about pure tempestless functional testing of IPA | 13:19 |
| *** _milan_ has joined #openstack-ironic | 13:19 | |
| sambetts | where we use unittest, and start IPA as a ficture | 13:20 |
| sambetts | fixture | 13:20 |
| sambetts | and excercise its funtionality against local volumes | 13:20 |
| sambetts | then we don't have to care about its functionality in the tempest tests | 13:21 |
| dtantsur | just make sure it does not need root to run :) | 13:21 |
| openstackgerrit | yolanda.robla proposed openstack/ironic-tempest-plugin master: Add manual clean step ironic standalone test https://review.openstack.org/568616 | 13:21 |
| dtantsur | vdrok: I may still try to squeeze removing the "fake" driver in the same patch. I noticed that I remove a lot of code that I've just changed :) | 13:22 |
| *** milan_ has quit IRC | 13:22 | |
| TheJulia | dtantsur: ++ | 13:23 |
| *** liuzz_ has joined #openstack-ironic | 13:26 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Remove the "fake" and "fake_soft_power" classic drivers https://review.openstack.org/570996 | 13:27 |
| jroll | morning everyone | 13:28 |
| dtantsur | morning jroll | 13:28 |
| *** links has quit IRC | 13:29 | |
| dtantsur | +573, -1231 what an awesome patch \o/ | 13:29 |
| jroll | ++ | 13:29 |
| *** liuzz has quit IRC | 13:30 | |
| *** links has joined #openstack-ironic | 13:30 | |
| TheJulia | good morning jroll | 13:32 |
| jroll | hey TheJulia :) | 13:33 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Remove the "fake" and "fake_soft_power" classic drivers https://review.openstack.org/570996 | 13:34 |
| dtantsur | okay, now this is ready for review, if you're eager to waste a few hours of your life on unit tests | 13:34 |
| * dtantsur wasted several days :( | 13:34 | |
| dtantsur | I do hope that removing other drivers will go easier | 13:36 |
| dtantsur | otherwise I'll rage-quit software industry before I finish | 13:36 |
| *** skazi has joined #openstack-ironic | 13:37 | |
| * jroll passes dtantsur a fishing rod | 13:38 | |
| * TheJulia passes dtantsur something delicious to sip while fishing | 13:39 | |
| dtantsur | heh :) | 13:39 |
| dtantsur | TheJulia: oh, an update: I had to extend my stay in Prague, so I won't be able to run our meeting on Monday | 13:40 |
| * dtantsur has to close CZ bank account finally | 13:40 | |
| TheJulia | Ahh, okay. | 13:41 |
| TheJulia | the fun life things :( | 13:41 |
| dtantsur | yep | 13:42 |
| *** ajya has quit IRC | 13:43 | |
| *** ajya has joined #openstack-ironic | 13:44 | |
| *** links has quit IRC | 13:44 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Remove the "fake" and "fake_soft_power" classic drivers https://review.openstack.org/570996 | 13:45 |
| yolanda_ | dummy question, where are the ironic conductor logs going on devstack? | 13:45 |
| yolanda_ | i can see the bm-deploy-logs, but that's node per node | 13:45 |
| *** hjensas has quit IRC | 13:45 | |
| TheJulia | yolanda_: in journald I guess since they go through standard logging interfaces | 13:46 |
| yolanda_ | ah indeed | 13:48 |
| yolanda_ | thx | 13:48 |
| *** baha has joined #openstack-ironic | 13:48 | |
| *** mjturek has joined #openstack-ironic | 13:49 | |
| dtantsur | yep, something like devstack@ir-cond | 13:50 |
| dtantsur | journalctl -u devstack@ir-cond | 13:50 |
| yolanda_ | thx, found it. The name of the service was confusing me | 13:50 |
| dtantsur | a simple review if anybody has time: https://review.openstack.org/#/c/570373/ | 13:53 |
| patchbot | patch 570373 - python-ironicclient - Do not abort wait_for_provision_state of last_erro... | 13:53 |
| *** AlexeyAbashkin has quit IRC | 14:02 | |
| *** milan_ has joined #openstack-ironic | 14:02 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 14:04 | |
| *** _milan_ has quit IRC | 14:05 | |
| *** skazi has quit IRC | 14:06 | |
| *** EmilienM_PTO is now known as EmilienM | 14:08 | |
| *** cjloader has joined #openstack-ironic | 14:13 | |
| *** marios has quit IRC | 14:16 | |
| *** marios has joined #openstack-ironic | 14:16 | |
| *** r-daneel has joined #openstack-ironic | 14:18 | |
| *** hjensas has joined #openstack-ironic | 14:25 | |
| *** hjensas has quit IRC | 14:25 | |
| *** hjensas has joined #openstack-ironic | 14:25 | |
| *** rajinir has joined #openstack-ironic | 14:25 | |
| dtantsur | TheJulia: forced vacation on a cruise \o/ where to sign up? :D | 14:29 |
| TheJulia | iknowrite! | 14:29 |
| TheJulia | well, work sessions at either end, but vacation! | 14:29 |
| TheJulia | lol | 14:29 |
| TheJulia | Could you imagine a PTG at sea? We would need a rack of gear in terms of mirrors of things, but yeah. | 14:30 |
| sambetts | hehe that would be an interesting edge cloud experiment for OS infra ;) | 14:32 |
| *** rpioso|afk is now known as rpioso | 14:33 | |
| rpioso | Good morning :) | 14:33 |
| TheJulia | indeed. I actually randomly had a discussion about the economics of such while I was in YVR. The person I was talking to had memories of Perl Whirl, and the headaches of corporations approving employees to go to a conference on a cruise ship. | 14:34 |
| TheJulia | Regardless of the economic advantages | 14:34 |
| sambetts | I could imagine the conference flu would be even worse on the cruise ship that it normally is | 14:35 |
| sambetts | s/that/than | 14:36 |
| TheJulia | Yeah... that is another possible issue | 14:36 |
| *** r-daneel_ has joined #openstack-ironic | 14:36 | |
| *** r-daneel has quit IRC | 14:37 | |
| *** r-daneel_ is now known as r-daneel | 14:37 | |
| TheJulia | yolanda_: great reply to the ml :) | 14:41 |
| *** rbartal has quit IRC | 14:44 | |
| jroll | I know a guy that worked on a healthcare records system for cruise ships... interesting problems there :) | 14:45 |
| NobodyCam | Good Morning Ironic'ers | 14:45 |
| *** mjura has quit IRC | 14:48 | |
| yolanda_ | TheJulia, i wanted to add my 2 cents ... | 14:49 |
| rpioso | Hey NobodyCam | 14:50 |
| NobodyCam | hey hey Good morning rpioso :) | 14:50 |
| *** bfournie has quit IRC | 14:59 | |
| TheJulia | jroll: that does seem like an interestingly difficult problem space given various ports of call | 14:59 |
| jroll | TheJulia: indeed! | 14:59 |
| jroll | luckily a human can only be in one place at a time, so no divergent data to sync :P | 15:00 |
| *** gyee has joined #openstack-ironic | 15:05 | |
| *** zhangfei has joined #openstack-ironic | 15:06 | |
| *** serlex has quit IRC | 15:06 | |
| dtantsur | heh | 15:08 |
| *** r-daneel has quit IRC | 15:16 | |
| dtantsur | imagine getting a "nice" weather and getting stuck in some port :D | 15:20 |
| *** r-daneel has joined #openstack-ironic | 15:25 | |
| *** Guest48153 is now known as melwitt | 15:25 | |
| *** zhangfei has quit IRC | 15:25 | |
| bdodd | Good morning ironic | 15:26 |
| *** skazi has joined #openstack-ironic | 15:32 | |
| rpioso | bdodd: \o | 15:32 |
| *** pcaruana has quit IRC | 15:33 | |
| *** bfournie has joined #openstack-ironic | 15:37 | |
| bdodd | morning rpioso | 15:38 |
| * TheJulia feels like a caffeine IV is needed for lunch | 15:38 | |
| *** racedo has quit IRC | 15:43 | |
| *** racedo has joined #openstack-ironic | 15:43 | |
| *** racedo has quit IRC | 15:43 | |
| *** racedo has joined #openstack-ironic | 15:44 | |
| *** racedo has quit IRC | 15:44 | |
| *** akhilaki has joined #openstack-ironic | 15:47 | |
| dtantsur | fantastic, devstack-gate hardcodes the "fake" driver >_< https://github.com/openstack-infra/devstack-gate/blob/b3da8a393c68cc62924ee2f752f442d5c85ea8ed/devstack-vm-gate.sh#L453 | 15:51 |
| sambetts | \o/ burn devstack-gate with fire | 15:52 |
| dtantsur | any practical ideas how to handle it without breaking stable branches? :( | 15:52 |
| dtantsur | actually, maybe we can override it in playbooks.... | 15:53 |
| sambetts | devstack-gate has access to which branch we're on so we could flag it on that, or I'd suggest we just nuke that logic from devstack-gate and just set it all in our local conf | 15:54 |
| sambetts | per job | 15:54 |
| sambetts | this logic should really be in the jobs local config or in our devstack plugin | 15:55 |
| sambetts | in fact I think we already override it in our local conf in almost all jobs | 15:55 |
| sambetts | ah no :( we only do the deploy interface | 15:56 |
| sambetts | and the network interfaces | 15:56 |
| dtantsur | sambetts: I think I'll hack it into our devstack plugin for now. devstack-gate reviews can take months.. | 15:59 |
| *** skazi has quit IRC | 16:00 | |
| *** baha has quit IRC | 16:05 | |
| clarkb | dtantsur: devstack gate exist(s|ed) specifically to run devstack in a gate environment | 16:07 |
| clarkb | with ironic that implied using the fake driver aiui | 16:07 |
| clarkb | dtantsur: the reason for that is the idea is to move away from devstack-gate entirely with the new zuulv3 devstack jobs | 16:08 |
| clarkb | I think we're mostly only reviewing bug fixes affecting existing jobs in devstack-gate at this point | 16:08 |
| dtantsur | clarkb: no, not really.. we support both fake and ipmi in the testing environment | 16:08 |
| dtantsur | damn, is there a way to write a regular expression matching both fake and fake, ? | 16:09 |
| dtantsur | I might be crazy, but nothing I come up with works... | 16:09 |
| clarkb | fake,? | 16:10 |
| dtantsur | clarkb: yep, if you remember that sed defaults to non-extended regexps... which I had to google to realize | 16:10 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Remove the "fake" and "fake_soft_power" classic drivers https://review.openstack.org/570996 | 16:12 |
| dtantsur | okay, this may work for now ^^^ | 16:12 |
| *** ajya has quit IRC | 16:13 | |
| *** ajya has joined #openstack-ironic | 16:14 | |
| dtantsur | vdrok: missed your comments, will update now | 16:14 |
| clarkb | on the pains with d-g front anytime you can have sane defaults in devstack or your plugin(s) it helps quite a bit. I think this particular case is painful because people want to use devstack with real hardware which isn't possible in the gate | 16:14 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Remove the "fake" and "fake_soft_power" classic drivers https://review.openstack.org/570996 | 16:17 |
| dtantsur | clarkb: actually, the defaults in devstack-gate mostly match our defaults. probably nobody cared to clean them up. | 16:17 |
| * dtantsur really likes new-style jobs | 16:18 | |
| openstackgerrit | Merged openstack/networking-baremetal master: add lower-constraints job https://review.openstack.org/555922 | 16:21 |
| *** pcaruana has joined #openstack-ironic | 16:23 | |
| *** trown is now known as trown|lunch | 16:26 | |
| *** marios has quit IRC | 16:27 | |
| *** srart has quit IRC | 16:32 | |
| *** srart has joined #openstack-ironic | 16:34 | |
| *** milan_ has quit IRC | 16:37 | |
| *** milan_ has joined #openstack-ironic | 16:38 | |
| rpioso | dtantsur: I'm adding UEFI boot mode support to the idrac h/w type. What should ManagementInterface.get_boot_device() return as the boot_device when no boot device is present? The comment in the base class states, "Ahe [sic] boot device, one of :mod: `ironic.common.boot_devices` or None if it is unknown." | 16:38 |
| rpioso | dtantsur: None present is different from unknown. | 16:39 |
| *** milan_ has quit IRC | 16:42 | |
| dtantsur | rpioso: get_boot_device or get_boot_mode? or are they somehow related for idrac? | 16:43 |
| rpioso | dtantsur: get_boot_device | 16:43 |
| dtantsur | rpioso: why cannot you detect the boot device? | 16:44 |
| dtantsur | I think you can return None, I'm not sure how ironic will react | 16:44 |
| dtantsur | it's probably fine, we have drivers not supporting boot devices | 16:44 |
| sambetts | its possible to not have a boot device set on some BMCs, I think None is the correct response | 16:44 |
| rpioso | dtantsur: I can. None are configured on the server -- no PXE NIC nor drive with an OS. | 16:44 |
| dtantsur | okay, so I guess it's None indeed | 16:45 |
| mordred | bifrost integration tests are failing on the cmd2 python3 issue - is that a thing people are already working on ? | 16:45 |
| rpioso | dtantsur, sambetts: I can detect the persistent state. | 16:45 |
| *** erolg has quit IRC | 16:45 | |
| dtantsur | mordred: I think it's because of clifff | 16:46 |
| rpioso | dtantsur, sambetts: ty | 16:46 |
| mordred | yah | 16:46 |
| dtantsur | cliff needs a release to strike out the broken version | 16:46 |
| dtantsur | which is 2.12.0 I guess | 16:46 |
| mordred | ah yes, that would make sense | 16:46 |
| mordred | also - do we still need python-ironicclient in bifrost? | 16:46 |
| * dtantsur no idea | 16:47 | |
| *** mgoddard has quit IRC | 16:49 | |
| *** sambetts is now known as sambetts|afk | 16:59 | |
| sambetts|afk | night all o/ | 16:59 |
| rpioso | sambetts|afk: Good nite | 16:59 |
| *** rpioso is now known as rpioso|eat | 16:59 | |
| dtantsur | heading to pycon.cz tomorrow, see you on Tuesday | 17:00 |
| *** dtantsur is now known as dtantsur|afk | 17:00 | |
| *** derekh has quit IRC | 17:01 | |
| *** e0ne has quit IRC | 17:02 | |
| *** e0ne has joined #openstack-ironic | 17:02 | |
| *** tesseract has quit IRC | 17:10 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Power fault recovery: API implementation https://review.openstack.org/556015 | 17:12 |
| *** gyankum has quit IRC | 17:23 | |
| *** baha has joined #openstack-ironic | 17:28 | |
| *** AlexeyAbashkin has quit IRC | 17:29 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 17:31 | |
| *** dprince has quit IRC | 17:33 | |
| *** AlexeyAbashkin has quit IRC | 17:36 | |
| *** trown|lunch is now known as trown | 17:39 | |
| *** cz2 has quit IRC | 17:43 | |
| *** dougsz has quit IRC | 17:56 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 17:57 | |
| *** rpioso|eat is now known as rpioso | 17:58 | |
| *** dprince has joined #openstack-ironic | 17:58 | |
| *** Alexey_Abashkin has joined #openstack-ironic | 18:00 | |
| *** AlexeyAbashkin has quit IRC | 18:02 | |
| *** Alexey_Abashkin is now known as AlexeyAbashkin | 18:02 | |
| *** pcaruana has quit IRC | 18:09 | |
| *** milan has joined #openstack-ironic | 18:12 | |
| *** AlexeyAbashkin has quit IRC | 18:40 | |
| *** jcoufal_ has joined #openstack-ironic | 18:40 | |
| *** jcoufal has quit IRC | 18:44 | |
| *** e0ne has quit IRC | 18:50 | |
| TheJulia | mordred: unlikely, but anyone using cli stuffs would expect it | 18:51 |
| *** rwsu has quit IRC | 18:51 | |
| *** fragatina has quit IRC | 18:56 | |
| *** yolanda has joined #openstack-ironic | 18:57 | |
| *** yolanda_ has quit IRC | 18:59 | |
| *** cjloader has quit IRC | 18:59 | |
| *** cjloader_ has joined #openstack-ironic | 18:59 | |
| *** cjloader_ has quit IRC | 19:01 | |
| *** cjloader has joined #openstack-ironic | 19:01 | |
| mordred | TheJulia: nod | 19:17 |
| *** dougsz has joined #openstack-ironic | 19:28 | |
| *** lifeless has joined #openstack-ironic | 19:31 | |
| *** cjloader has quit IRC | 19:46 | |
| *** cjloader has joined #openstack-ironic | 19:46 | |
| *** cjloader has quit IRC | 19:48 | |
| *** cjloader_ has joined #openstack-ironic | 19:48 | |
| *** cjloader_ has quit IRC | 19:49 | |
| *** cjloader has joined #openstack-ironic | 19:50 | |
| *** dougsz has quit IRC | 19:51 | |
| *** srart has quit IRC | 19:51 | |
| *** ajya has quit IRC | 20:03 | |
| *** srart has joined #openstack-ironic | 20:12 | |
| *** srart has quit IRC | 20:12 | |
| * TheJulia wonders where her brain went | 20:23 | |
| *** zshi has quit IRC | 20:24 | |
| *** zshi has joined #openstack-ironic | 20:24 | |
| *** trown is now known as trown|outtypewww | 20:25 | |
| *** fragatina has joined #openstack-ironic | 20:38 | |
| *** akhilaki has quit IRC | 20:40 | |
| *** akhilaki has joined #openstack-ironic | 20:40 | |
| *** rwsu has joined #openstack-ironic | 20:43 | |
| *** skazi has joined #openstack-ironic | 20:58 | |
| *** sapd_ has joined #openstack-ironic | 21:03 | |
| *** cjloader has quit IRC | 21:05 | |
| *** sapd has quit IRC | 21:07 | |
| *** bfournie has quit IRC | 21:11 | |
| *** milan has quit IRC | 21:13 | |
| *** skazi has quit IRC | 21:16 | |
| *** dprince has quit IRC | 21:24 | |
| *** lifeless has quit IRC | 21:26 | |
| *** lifeless_ has joined #openstack-ironic | 21:26 | |
| *** lifeless has joined #openstack-ironic | 21:38 | |
| *** lifeless_ has quit IRC | 21:38 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Fix task_manager process_event docstring https://review.openstack.org/571316 | 21:39 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Fix task_manager process_event docstring https://review.openstack.org/571316 | 21:40 |
| *** jcoufal_ has quit IRC | 21:41 | |
| *** baha has quit IRC | 21:41 | |
| *** baha has joined #openstack-ironic | 21:42 | |
| *** baha has quit IRC | 21:42 | |
| *** fragatina has quit IRC | 21:46 | |
| *** fragatina has joined #openstack-ironic | 21:46 | |
| *** harlowja has joined #openstack-ironic | 21:52 | |
| *** lifeless has quit IRC | 22:03 | |
| *** lifeless has joined #openstack-ironic | 22:04 | |
| *** rcernin has joined #openstack-ironic | 22:22 | |
| *** lifeless has quit IRC | 22:33 | |
| *** lifeless has joined #openstack-ironic | 22:35 | |
| *** r-daneel has quit IRC | 22:42 | |
| *** bfournie has joined #openstack-ironic | 22:52 | |
| *** lifeless has quit IRC | 23:11 | |
| *** pmannidi has quit IRC | 23:13 | |
| *** lifeless has joined #openstack-ironic | 23:41 | |
| *** cjloader has joined #openstack-ironic | 23:46 | |
| *** akhilaki has quit IRC | 23:54 | |
| *** rpioso is now known as rpioso|afk | 23:55 | |
| *** lifeless has quit IRC | 23:58 | |
| *** lifeless has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!