| *** gkadam has joined #openstack-ironic | 02:45 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/ironic master: DNM: Testing all legacy jobs on bionic https://review.openstack.org/642315 | 02:54 |
|---|---|---|
| openstackgerrit | Ghanshyam Mann proposed openstack/ironic-inspector master: DNM: Testing all legacy jobs on bionic https://review.openstack.org/642316 | 02:56 |
| openstackgerrit | Ghanshyam Mann proposed openstack/ironic-python-agent master: DNM: Testing all legacy jobs on bionic https://review.openstack.org/642317 | 03:01 |
| *** andrein has quit IRC | 03:03 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/python-ironicclient master: DNM: Testing all legacy jobs on bionic https://review.openstack.org/642318 | 03:04 |
| *** whoami-rajat has joined #openstack-ironic | 03:07 | |
| *** MattMan_ has quit IRC | 03:23 | |
| *** MattMan_ has joined #openstack-ironic | 03:24 | |
| openstackgerrit | QianBiao Ng proposed openstack/ironic master: Add Huawei iBMC driver support https://review.openstack.org/639288 | 03:40 |
| *** gkadam has quit IRC | 03:47 | |
| *** andrein has joined #openstack-ironic | 04:08 | |
| *** andrein has quit IRC | 04:10 | |
| openstackgerrit | zhufl proposed openstack/ironic master: Fix missing print format error https://review.openstack.org/642346 | 05:50 |
| *** ianychoi has quit IRC | 06:32 | |
| *** ianychoi has joined #openstack-ironic | 06:32 | |
| *** ianychoi has quit IRC | 06:35 | |
| *** ianychoi has joined #openstack-ironic | 06:36 | |
| *** jtomasek has joined #openstack-ironic | 06:50 | |
| *** e0ne has joined #openstack-ironic | 06:51 | |
| *** Qianbiao has joined #openstack-ironic | 06:57 | |
| Qianbiao | Hello | 06:57 |
| *** pcaruana has joined #openstack-ironic | 07:00 | |
| *** andrein has joined #openstack-ironic | 07:02 | |
| *** e0ne has quit IRC | 07:02 | |
| *** rcernin has quit IRC | 07:03 | |
| Qianbiao | Hello, for patch https://review.openstack.org/#/c/639288/9 | 07:05 |
| patchbot | patch 639288 - ironic - Add Huawei iBMC driver support - 9 patch sets | 07:05 |
| Qianbiao | the py27 unittest is failed. | 07:05 |
| Qianbiao | but it works in my local env. any hints? | 07:05 |
| Qianbiao | Unittest code did not changed, and it pass in preview patch too. Not sure why it failed now. | 07:06 |
| *** ltomasbo has joined #openstack-ironic | 07:07 | |
| *** PabloIranzoGmez[ has left #openstack-ironic | 07:08 | |
| *** andrein has quit IRC | 07:24 | |
| openstackgerrit | Merged openstack/ironic master: Fix typo and docstring in pxe/ipxe https://review.openstack.org/642081 | 07:37 |
| rpittau|afk | good morning ironic! o/ | 08:04 |
| *** tssurya has joined #openstack-ironic | 08:06 | |
| etingof | o/ rpittau|afk and the rest of ironic alive | 08:09 |
| *** rpittau|afk is now known as rpittau | 08:09 | |
| rpittau | now I'm more alive :P | 08:09 |
| arne_wiebalck | good morning, ironic! | 08:14 |
| arne_wiebalck | rpittau: how are the legs? | 08:14 |
| *** andrein has joined #openstack-ironic | 08:15 | |
| rpittau | hi arne_wiebalck , legs are ok, the back not that much :/ | 08:15 |
| arne_wiebalck | :/ | 08:15 |
| rpittau | I had backpain since friday but my stubbornness knows no limits :D | 08:16 |
| *** stevebaker has quit IRC | 08:19 | |
| *** dmellado has quit IRC | 08:20 | |
| arne_wiebalck | I did sth similar ... you'll learn from it ;) | 08:22 |
| *** jgarvita_ has joined #openstack-ironic | 08:23 | |
| *** e0ne has joined #openstack-ironic | 08:27 | |
| *** andrein has quit IRC | 08:34 | |
| *** hwoarang has quit IRC | 08:34 | |
| Qianbiao | Hello, may someone help me check the tox-py27 failed issue of patch https://review.openstack.org/#/c/639288/9 | 08:35 |
| patchbot | patch 639288 - ironic - Add Huawei iBMC driver support - 9 patch sets | 08:35 |
| Qianbiao | Everything is ok in local env. | 08:35 |
| *** hwoarang has joined #openstack-ironic | 08:36 | |
| *** jgarvita_ has quit IRC | 08:37 | |
| *** dtantsur|afk is now known as dtantsur | 08:38 | |
| dtantsur | morning ironic | 08:38 |
| rpittau | hi dtantsur | 08:39 |
| *** preece has quit IRC | 08:39 | |
| rpittau | Qianbiao, I'm checking now, also leaving some comments | 08:39 |
| Qianbiao | ok thanks. | 08:41 |
| Qianbiao | rpittau code did not change, unittest is ok in preview patch too. Not sure why failed this time. | 08:41 |
| *** preece has joined #openstack-ironic | 08:43 | |
| *** anastzhyr has quit IRC | 08:45 | |
| *** andrein has joined #openstack-ironic | 08:46 | |
| *** iurygregory has joined #openstack-ironic | 08:50 | |
| iurygregory | morning o/ | 08:51 |
| *** sburette has joined #openstack-ironic | 08:53 | |
| openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Support firmware upgrade for irmc driver https://review.openstack.org/599992 | 09:00 |
| *** needssleep has quit IRC | 09:01 | |
| *** dougsz has joined #openstack-ironic | 09:08 | |
| arne_wiebalck | Qianbiao: I ran the tests locally, they came back ok. | 09:10 |
| Qianbiao | <arne_wiebalck> yes, me too. | 09:11 |
| *** hwoarang has quit IRC | 09:11 | |
| rpittau | Qianbiao, the issue could be with the imported and unused unittest module in 2 of the test modules, I left some comments, let me know if you have questions | 09:11 |
| arne_wiebalck | Qianbiao: I see they are ok on the review now as well :) | 09:11 |
| Qianbiao | wow... | 09:12 |
| *** hwoarang has joined #openstack-ironic | 09:12 | |
| Qianbiao | rpittau rfc3986>=1.20 is required by oslo_config too | 09:13 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: DNM - Test Functional in a cleaner env https://review.openstack.org/641671 | 09:15 |
| rpittau | Qianbiao,I think that is handled directly by oslo_config, also if you change the url parsing to oslo_utils you won't need rfc3986 | 09:17 |
| Qianbiao | yes, will try. | 09:18 |
| Qianbiao | but rfc3986's version should be upgrade i think | 09:18 |
| Qianbiao | some code are using features of rfc3986 which will be removed soon. | 09:19 |
| rpittau | Qianbiao, the only other place where ironic uses rfc3986 is in redfish and I think it can be changed to use oslo_utils | 09:20 |
| rpittau | so maybe it will be removed instead :) | 09:20 |
| Qianbiao | :) good idea | 09:20 |
| *** e0ne has quit IRC | 09:22 | |
| rpittau | also we don't use validators in redfish so maybe we don't need to bump the version even if we don't convert the code to oslo_utils | 09:22 |
| *** michchap has joined #openstack-ironic | 09:26 | |
| Qianbiao | redfish author has a comment for the code. | 09:28 |
| Qianbiao | # TODO(vdrok): Workaround this check, in py3 we need to use validator class | 09:28 |
| Qianbiao | if not parsed.is_valid(require_scheme=True, require_authority=True): | 09:28 |
| Qianbiao | and is_valid has been deprecated by rfc3986 | 09:29 |
| Qianbiao | rfc3986>=1.2.0 is required by the way, no matter ironic need it or not. oslo.config requires rfc3986>=1.2.0 | 09:30 |
| Qianbiao | By the way, which file in oslo.utils provide urlparse util, i did not find it. | 09:30 |
| iurygregory | rpittau, http://logs.openstack.org/71/641671/6/check/ironicclient-functional/e4a4afc/job-output.txt.gz#_2019-03-11_09_32_24_817957 \o/ | 09:42 |
| *** e0ne has joined #openstack-ironic | 09:42 | |
| iurygregory | Ran: 174 tests in 110.0000 sec. - Passed: 174 | 09:43 |
| iurygregory | \o/ | 09:43 |
| *** derekh has joined #openstack-ironic | 09:43 | |
| dtantsur | w00t! | 09:45 |
| dtantsur | iurygregory: ready for review now? | 09:45 |
| iurygregory | dtantsur, nope going to push in the main patch \o/ | 09:45 |
| dtantsur | k | 09:45 |
| iurygregory | zuulv3 and the black magic XD | 09:46 |
| iurygregory | now is a happy monday \o/ | 09:46 |
| rpittau | iurygregory, awesome! | 09:46 |
| rpittau | Qianbiao, netutils | 09:48 |
| Qianbiao | <rpittau> checking now | 09:49 |
| iurygregory | dtantsur, just to let you know my approach was convert the shell scripts into a playbook and we don't have the generate_testr_results functions anymore (I didn't saw a reason to keep since it only generates testrepository.subunit file that have no information except if you download and convert to html) wdyt? | 09:54 |
| dtantsur | iurygregory: well, it's pretty handy to see the generated HTML in the CI | 09:57 |
| dtantsur | although I'm not sure if the current functional job does it | 09:57 |
| iurygregory | dtantsur, we don't get in the CI o.o at least i didnt found the html only the subunit file | 09:57 |
| dtantsur | ah, ok. then we can look into generation later | 09:59 |
| iurygregory | ack | 09:59 |
| *** priteau has joined #openstack-ironic | 10:03 | |
| rpittau | Qianbiao, I had another look and I think the common approach right now is to use rfc3986 in other projects too, so I'm ok with that, but we need to make a note for the requirements bump and probably change the code in redfish | 10:05 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Fix deploy template tests on <= Stein https://review.openstack.org/642399 | 10:05 |
| dtantsur | mgoddard: ^^^ | 10:05 |
| dtantsur | ugh, this should be < not <= | 10:05 |
| mgoddard | dtantsur: ack | 10:05 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Fix deploy template tests on Rocky and earlier https://review.openstack.org/642399 | 10:05 |
| dtantsur | mgoddard: with corrected summary ^^^ | 10:05 |
| mgoddard | dtantsur: thanks for fixing | 10:06 |
| openstackgerrit | Merged openstack/ironic master: driver-requirements: mark UcsSdk as Python 2 only https://review.openstack.org/641972 | 10:06 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/rocky: devstack: use stable/rocky IPA for stable/rocky ironic https://review.openstack.org/642400 | 10:07 |
| dtantsur | jroll: when you wake up ^^^ | 10:07 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Adding ansible python interpreter as node_info https://review.openstack.org/642402 | 10:08 |
| Qianbiao | rpittau i am working on it now. | 10:09 |
| Qianbiao | will remove rfc3986 directly. | 10:09 |
| Qianbiao | use netutils instead | 10:09 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Move to zuulv3 https://review.openstack.org/633010 | 10:10 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Adding ansible python interpreter as node_info https://review.openstack.org/642402 | 10:10 |
| Qianbiao | Hello, is there any import order config document for pycharm. | 10:12 |
| Qianbiao | it's really not easy to re-order the imports by hand. | 10:12 |
| rpittau | Qianbiao, pep8 test should give some hints | 10:12 |
| Qianbiao | I hope the IDE could auto re-order it. | 10:13 |
| rpittau | Qianbiao, I use pycharm and you can optimize the order | 10:14 |
| Qianbiao | Could u share your settings, pycharm auto-format is quit different with openstack's code style | 10:15 |
| openstackgerrit | Will Szumski proposed openstack/tenks master: Adding python 3.6 unit test https://review.openstack.org/640807 | 10:15 |
| openstackgerrit | Will Szumski proposed openstack/tenks master: Fix ansible lint 301 with ip link set dev <device> up https://review.openstack.org/642406 | 10:15 |
| *** e0ne has quit IRC | 10:43 | |
| *** e0ne has joined #openstack-ironic | 10:47 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/metalsmith master: [WIP] Accept hostname in reserve_node in addition to provision_node https://review.openstack.org/640397 | 10:53 |
| openstackgerrit | QianBiao Ng proposed openstack/ironic master: Add Huawei iBMC driver support https://review.openstack.org/639288 | 10:55 |
| Qianbiao | rpittau pushed | 10:56 |
| *** dmellado_ has joined #openstack-ironic | 11:04 | |
| *** dmellado_ is now known as dmellado | 11:05 | |
| dtantsur | anyone has an idea what happened to the postgres job? | 11:08 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Adding ansible python interpreter as node_info https://review.openstack.org/642402 | 11:12 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Adding ansible python interpreter as driver_info https://review.openstack.org/642402 | 11:15 |
| rpittau | dtantsur, where is the issue with the postgres job ? | 11:16 |
| dtantsur | rpittau: I think it has been RED for long time | 11:16 |
| dtantsur | (it's non-voting, so not blocking the gate) | 11:16 |
| dtantsur | mgoddard: gerrit shows "Cannot merge" on https://review.openstack.org/#/c/641984/ to me | 11:20 |
| patchbot | patch 641984 - ironic - Add python3 unit test with drivers installed - 2 patch sets | 11:20 |
| mgoddard | dtantsur: hmm. I'll check it out | 11:20 |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Marking ironic-standalone job voting again https://review.openstack.org/641967 | 11:21 |
| openstackgerrit | Merged openstack/python-ironicclient stable/rocky: pass endpoint interface to http client https://review.openstack.org/641562 | 11:21 |
| *** stevebaker has joined #openstack-ironic | 11:22 | |
| rpittau | dtantsur, ipxe is disabled for postgres, which is ok, although pxe boot image can't be found | 11:23 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add python3 unit test with drivers installed https://review.openstack.org/641984 | 11:23 |
| rpittau | could be the change to bionic ? | 11:23 |
| openstackgerrit | Will Szumski proposed openstack/tenks master: Adding python 3.6 unit test https://review.openstack.org/640807 | 11:23 |
| openstackgerrit | Will Szumski proposed openstack/tenks master: Add pkg-config as an explicit dependency https://review.openstack.org/642423 | 11:23 |
| rpittau | dtantsur, nvm, the location of the pxelinux file should be the same in bionic | 11:24 |
| *** hwoarang has quit IRC | 11:27 | |
| *** bfournie has quit IRC | 11:32 | |
| openstackgerrit | Merged openstack/metalsmith master: Ensure instance_info is clean before deploy and after failure https://review.openstack.org/642016 | 11:33 |
| *** hwoarang has joined #openstack-ironic | 11:35 | |
| *** e0ne has quit IRC | 11:37 | |
| rpittau | dtantsur, the package pxelinux is not installed :/ | 11:45 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [DNM] testing postgres job https://review.openstack.org/642430 | 11:56 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: RPC changes for Graphical Console https://review.openstack.org/640827 | 11:57 |
| openstackgerrit | Varsha Verma proposed openstack/sushy master: Add mappings for `system_type` https://review.openstack.org/640766 | 11:57 |
| *** rh-jelabarre has joined #openstack-ironic | 12:02 | |
| dtantsur | mgoddard: I think https://review.openstack.org/642399 shows a real problem actually: check on deploy_template_ident correctness happens before microversion check | 12:06 |
| patchbot | patch 642399 - ironic-tempest-plugin - Fix deploy template tests on Rocky and earlier - 2 patch sets | 12:06 |
| michchap | hey guys, I'm trying to run devstack with ironic enabled but nova disabled. I can't get it to complete unless I disable the tempest plugin since it seems to wait for nova services to be enabled. Is there meant to be a coupling between ironic and nova in a devstack+tempest environment? | 12:06 |
| dtantsur | mgoddard: we did not see it before, because 1.54 allows CUSTOM_1 is a valid name, but 1.1 does not | 12:07 |
| mgoddard | dtantsur: :( | 12:07 |
| dtantsur | michchap: check our standalone jobs, they do tempest+ironic without nova | 12:07 |
| dtantsur | mgoddard: I wonder if it's worth fixing though | 12:08 |
| mgoddard | I hope this doesn't mean we can't use .validate() | 12:08 |
| mgoddard | on the wsme type | 12:08 |
| michchap | dtantsur: thanks will do | 12:08 |
| dtantsur | mgoddard: well, at least it happens before the microversion check | 12:09 |
| dtantsur | unless we start messing with routing | 12:10 |
| * dtantsur tries something locally | 12:10 | |
| mgoddard | dtantsur: yeah, just looking at pecan | 12:11 |
| mgoddard | dtantsur: I guess worst case we just check the version at the beginning of .validate() | 12:13 |
| dtantsur | mgoddard: I'm trying to go the route path, but yeah | 12:13 |
| *** andrein has quit IRC | 12:16 | |
| dtantsur | seems to work, will push something soon | 12:18 |
| *** Qianbiao has quit IRC | 12:19 | |
| mgoddard | dtantsur: great, thanks! | 12:21 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Check microversions before validations for allocations and deploy templates https://review.openstack.org/642435 | 12:28 |
| dtantsur | mgoddard: ^^^ | 12:28 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Fix deploy template tests on Rocky and earlier https://review.openstack.org/642399 | 12:29 |
| dtantsur | and will check via ^^^ | 12:29 |
| *** e0ne has joined #openstack-ironic | 12:30 | |
| *** jaypipes has joined #openstack-ironic | 12:31 | |
| mgoddard | dtantsur: looks good, awaiting zuul's response | 12:32 |
| *** bfournie has joined #openstack-ironic | 12:36 | |
| *** andrein has joined #openstack-ironic | 12:37 | |
| *** TheJulia has joined #openstack-ironic | 12:42 | |
| TheJulia | Good morning | 12:42 |
| *** trown|back11mar is now known as trown | 12:46 | |
| iurygregory | morning TheJulia | 12:47 |
| *** edleafe has joined #openstack-ironic | 12:51 | |
| arne_wiebalck | dtantsur: TheJulia : There is no entry yet on the whiteboard for software RAID support, so I guess I should add one ... to which section should that go, Stretch Goals or Stein priorities? ;) | 12:52 |
| TheJulia | Go ahead and add it so we can track it :) | 12:53 |
| TheJulia | dtantsur: any chance you can run the meeting this morning? I have to go see an ear specialist this morning and I have a time conflict :( | 12:54 |
| dtantsur | TheJulia: absolutely | 12:55 |
| dtantsur | and g'morning | 12:55 |
| rpittau | hi TheJulia :) | 12:55 |
| dtantsur | arne_wiebalck: Stein is under soft freeze now, so it's likely to slip into Train. Please do add it somewhere anyway. | 12:56 |
| TheJulia | dtantsur: thanks, they didn't call me to schedule until like 4:30 PM on Friday. :( | 12:57 |
| dtantsur | so nice of them | 12:57 |
| dtantsur | TheJulia: FYI I've put the fast-track RFE for review since it's not approved yet (and the code is quite close) | 12:58 |
| TheJulia | We should have marked it approved when a bunch of people were "ohh that would be nice" | 12:59 |
| TheJulia | But makes sense to revisit | 12:59 |
| dtantsur | TheJulia: yeah, also want to hear if people are okay with FFEing it | 13:00 |
| *** hwoarang has quit IRC | 13:00 | |
| TheJulia | I am all for it because it is not user visible | 13:00 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [DNM] testing postgres job https://review.openstack.org/642430 | 13:01 |
| TheJulia | Anyway... Blah. Doctors. | 13:01 |
| dtantsur | TheJulia: good luck! | 13:01 |
| *** hwoarang has joined #openstack-ironic | 13:04 | |
| *** mjturek has joined #openstack-ironic | 13:07 | |
| *** jaypipes has quit IRC | 13:08 | |
| *** jcoufal has joined #openstack-ironic | 13:11 | |
| *** Qianbiao has joined #openstack-ironic | 13:13 | |
| Qianbiao | Hello | 13:13 |
| Qianbiao | http://logs.openstack.org/88/639288/10/check/openstack-tox-lower-constraints/bee0792/testr_results.html.gz | 13:13 |
| Qianbiao | the unittest failed again. | 13:14 |
| Qianbiao | Sometimes it's ok, sometimes failed | 13:14 |
| *** mbeierl has joined #openstack-ironic | 13:14 | |
| arne_wiebalck | dtantsur: I added 's/w RAID support' to the whiteboard. | 13:14 |
| dtantsur | thnx | 13:14 |
| Qianbiao | But in my local ENV, it never failed. | 13:14 |
| dtantsur | Qianbiao: this suggests an ordering problem in your test (when some tests affect each other) | 13:15 |
| dtantsur | or some other kind of race | 13:15 |
| Qianbiao | Hmmm... will check it | 13:16 |
| Qianbiao | dtantsur @mock.patch.object is isolate in stestr, right? | 13:18 |
| dtantsur | Qianbiao: not sure what you mean, but it's effect is done before each test and undone right after | 13:18 |
| openstackgerrit | Merged openstack/python-ironicclient master: Move to zuulv3 https://review.openstack.org/633010 | 13:18 |
| iurygregory | \o/ its merged \o/ | 13:19 |
| Qianbiao | dtantsur thanks, your suggestion helps, will check code now. | 13:19 |
| dtantsur | congrats iurygregory and great job :) | 13:21 |
| iurygregory | ty sir! \o/ | 13:22 |
| rpittau | iurygregory, that was a pain, thanks! xD | 13:22 |
| iurygregory | rpittau, that was the tip of the iceberg the rest of the iceberg is ironic-tempest-plugin XD | 13:23 |
| * iurygregory need more coffee | 13:23 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Document building configdrive on the server side https://review.openstack.org/642441 | 13:24 |
| *** mjturek has quit IRC | 13:27 | |
| *** ianychoi has quit IRC | 13:28 | |
| *** ianychoi has joined #openstack-ironic | 13:29 | |
| *** w14161_1 has joined #openstack-ironic | 13:41 | |
| *** beekneemech is now known as bnemec | 13:43 | |
| w14161_1 | hi, all, did you know where could find the example of pxelinux.cfg/default for ironic pxe boot(boot_mode:bios)? | 13:49 |
| *** jrist has quit IRC | 13:49 | |
| dtantsur | w14161_1: hi, why do you need it? I think ironic should operate well with just pxelinux.cfg/<MAC> | 13:52 |
| dtantsur | (the latter is generated by ironic) | 13:52 |
| *** sthussey has joined #openstack-ironic | 13:52 | |
| *** jrist has joined #openstack-ironic | 13:53 | |
| w14161_1 | But <MAC> did not work, I changed <MAC> to default, then I could see "boot:" at pxe boot loader. | 13:53 |
| w14161_1 | I saw PXE loader was searching MAC and shorter the name a little one by one util reported a error, at last, it would search default, so put a default file would bypass the error message | 13:55 |
| rpittau | FYI review.o.o seems down | 13:56 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add a non-voting metalsmith job for local boot coverage https://review.openstack.org/642457 | 13:56 |
| *** efried has quit IRC | 13:57 | |
| dtantsur | rpittau: seems to work here.. | 13:57 |
| dtantsur | w14161_1: you have to fix the actual issue, not try to work it around.. | 13:57 |
| dtantsur | the issue may be that you have a wrong MAC on an ironic port | 13:58 |
| rpittau | dtantsur, yep working again | 13:58 |
| w14161_1 | OK, i would try, thx | 13:58 |
| dtantsur | happens | 13:58 |
| dtantsur | hjensas, mgoddard, wdyt about https://storyboard.openstack.org/#!/story/1575520 ? | 14:02 |
| mgoddard | dtantsur: I think keeping the hostnames unique should be enough. As Zachary Ma points out in comments, OVS can legitimately not have IP uniqueness | 14:04 |
| mgoddard | there may be other corner cases | 14:05 |
| dtantsur | mgoddard: could you update the story accordingly? | 14:05 |
| mgoddard | dtantsur: will do | 14:05 |
| michchap | dtantsur: I tried the standalone local.conf, I'm getting a 'FATAL: kernel too old'. I have 3.10.0-693.21.1.el7.x86_64, do I need to swap to ubuntu? | 14:07 |
| dtantsur | michchap: not sure where this comes from. I'm using centos7 for my devstack pretty successfully. | 14:07 |
| michchap | dtantsur: thanks, will dig into it further I guess. It's in rebuild_tinyipa_for_ansible | 14:08 |
| *** baha has joined #openstack-ironic | 14:09 | |
| dtantsur | michchap: do you need ansible deploy btw? if not, just disable it. | 14:09 |
| michchap | dtantsur: I don't, do you know the flag to disable it? | 14:10 |
| dtantsur | michchap: make sure it's not in IRONIC_ENABLED_DEPLOY_INTERFACES | 14:10 |
| michchap | dtantsur: cheers | 14:10 |
| *** e0ne has quit IRC | 14:14 | |
| *** e0ne has joined #openstack-ironic | 14:23 | |
| openstackgerrit | QianBiao Ng proposed openstack/ironic master: Add Huawei iBMC driver support https://review.openstack.org/639288 | 14:26 |
| *** mattmceuen has left #openstack-ironic | 14:27 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [DNM] testing postgres job https://review.openstack.org/642430 | 14:33 |
| *** e0ne has quit IRC | 14:36 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Switch functional tests to python3 https://review.openstack.org/642474 | 14:38 |
| * TheJulia heads out to the doctor :( | 14:38 | |
| *** e0ne has joined #openstack-ironic | 14:38 | |
| *** priteau has quit IRC | 14:44 | |
| dtantsur | iurygregory, rpittau, either of you willing to check the status of https://review.openstack.org/#/c/591828/ and https://review.openstack.org/#/c/591835/ please? | 14:44 |
| patchbot | patch 591828 - sushy - Change sushy devstack job to python3 - 2 patch sets | 14:44 |
| patchbot | patch 591835 - python-ironicclient - Switch ci jobs to Python3 - 2 patch sets | 14:44 |
| iurygregory | dtantsur, ack | 14:45 |
| iurygregory | dtantsur, for python ironicclient i just pushed to switch to python3 https://review.openstack.org/642474 since we had the zuulv3 merged | 14:46 |
| patchbot | patch 642474 - python-ironicclient - Switch functional tests to python3 - 1 patch set | 14:46 |
| iurygregory | or do we want python2 and python3 jobs running? | 14:46 |
| dtantsur | iurygregory: both for now | 14:46 |
| dtantsur | and the same for integration jobs, if it's not done already | 14:46 |
| iurygregory | should i update TheJulia patch or the one i pushed? | 14:46 |
| iurygregory | ack | 14:47 |
| dtantsur | either way | 14:47 |
| *** dnuka has joined #openstack-ironic | 14:47 | |
| *** hafsansari has joined #openstack-ironic | 14:47 | |
| dnuka | good morning ironicers \o/ | 14:47 |
| rpittau | hey dnuka :) | 14:48 |
| iurygregory | morning dnuka =) | 14:48 |
| arne_wiebalck | hey dnuka o/ | 14:48 |
| dtantsur | also anyone wants to look into the UEFI job in https://review.openstack.org/#/c/642015/ ? | 14:48 |
| patchbot | patch 642015 - ironic - Move all testing to Bionic - 2 patch sets | 14:48 |
| dnuka | hey rpittau, iurygregory, arne_wiebalck \o/ | 14:48 |
| dtantsur | hi dnuka | 14:48 |
| dnuka | hi dtantsur :) | 14:48 |
| rpioso | Good morning, ironicers | 14:50 |
| *** FlorianFa has joined #openstack-ironic | 14:50 | |
| dnuka | good morning rpioso :) | 14:50 |
| rpioso | dnuka: o/ | 14:50 |
| *** FlorianFa has quit IRC | 14:52 | |
| *** FlorianFa has joined #openstack-ironic | 14:52 | |
| rpioso | mgoddard: On Friday, I started reviewing https://review.openstack.org/#/c/641731/ I'll provide comments today. | 14:52 |
| patchbot | patch 641731 - ironic - WIP: Add iDRAC RAID deploy steps - 2 patch sets | 14:52 |
| mgoddard | thanks rpioso. Having troubles testing it, I think I need to convert some disks to RAID mode | 14:53 |
| mgoddard | rpioso: I have some changes to it, ended up using a less convoluted approach for the deploy step, as you suggested | 14:53 |
| dtantsur | morning rpioso | 14:53 |
| rpioso | mgoddard: Sounds familiar. The iDRAC clean steps don't presently do that. | 14:54 |
| rpioso | dtantsur: Morning :-) | 14:54 |
| mgoddard | rpioso: might be a nice vendor passthru. There's a dracclient method for it | 14:54 |
| *** kaifeng has joined #openstack-ironic | 14:54 | |
| mgoddard | rpioso: moved onto the API reference for now | 14:54 |
| rpioso | mgoddard: We're looking at adding it to the clean step processing. Seems appropriate there. | 14:55 |
| rpioso | mgoddard: That'll use the python-dracclient support. | 14:55 |
| *** cdearborn has joined #openstack-ironic | 14:55 | |
| mgoddard | rpioso: will it be automatic, or a new clean step? | 14:56 |
| rpioso | mgoddard: Automatic | 14:56 |
| mgoddard | rpioso: cool | 14:56 |
| rpioso | mgoddard: :-) | 14:56 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Run jobs under python2 and python3 https://review.openstack.org/642474 | 14:56 |
| iurygregory | dtantsur, done =) | 14:56 |
| mgoddard | rpioso: would be nice if the deploy step could use it too | 14:56 |
| dtantsur | thanks iurygregory | 14:56 |
| iurygregory | fingers crossed for CI now =) | 14:57 |
| rpioso | mgoddard: I don't see any reason why it won't just pick it up. | 14:57 |
| rpioso | mgoddard: Same method, right? | 14:57 |
| mgoddard | rpioso: should be | 14:57 |
| rpioso | s/method/methods/ | 14:57 |
| dtantsur | iurygregory: immediate failure :-P | 14:57 |
| dnuka | :) | 14:57 |
| rpittau | merciless zuul | 14:58 |
| iurygregory | WTF O.o | 14:59 |
| iurygregory | the job is defined lol | 14:59 |
| iurygregory | meeting | 14:59 |
| rpittau | 20 secs :P | 14:59 |
| *** priteau has joined #openstack-ironic | 14:59 | |
| dtantsur | #startmeeting ironic | 15:00 |
| openstack | Meeting started Mon Mar 11 15:00:02 2019 UTC and is due to finish in 60 minutes. The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot. | 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 |
| bdodd | o/ | 15:00 |
| iurygregory | o/ | 15:00 |
| dtantsur | how's up for some ironic talk? | 15:00 |
| rpioso | o/ | 15:00 |
| hjensas | o/ | 15:00 |
| dnuka | o/ | 15:00 |
| mgoddard | \o | 15:00 |
| arne_wiebalck | o/ | 15:00 |
| rpittau | o/ | 15:00 |
| etingof | o/ | 15:00 |
| kaifeng | o/ | 15:00 |
| *** stendulker has joined #openstack-ironic | 15:00 | |
| dnuka | good morning etingof :) | 15:00 |
| dtantsur | Welcome folks :) Our agenda can be found where always: | 15:00 |
| dtantsur | #link https://wiki.openstack.org/wiki/Meetings/Ironic | 15:01 |
| stendulker | o/ | 15:01 |
| cdearborn | o/ | 15:01 |
| dtantsur | #topic Announcements / Reminder | 15:01 |
| *** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)" | 15:01 | |
| dtantsur | #info Soft feature freeze is upon us | 15:01 |
| dtantsur | We don't plan on completely blocking everything that looks like a feature | 15:02 |
| dtantsur | but please be very considerate when approving things | 15:02 |
| dtantsur | especially if these things add API/RPC versions | 15:02 |
| dtantsur | please concentrate on finishing Stein work, fixing bugs, updating docs | 15:02 |
| dtantsur | #info Jobs are being switched to Bionic, help needed with two jobs on https://review.openstack.org/#/c/642015/ | 15:03 |
| patchbot | patch 642015 - ironic - Move all testing to Bionic - 2 patch sets | 15:03 |
| dtantsur | if I understood jroll correctly, the UEFI and the multinode jobs don't work on Bionic | 15:03 |
| dtantsur | we have (again, IIUC) until Wednesday to fix them. Help very welcome! | 15:03 |
| dtantsur | Now a reminder: | 15:04 |
| dtantsur | #info Train Forum/PTG topic etherpad: https://etherpad.openstack.org/p/DEN-train-ironic-brainstorming | 15:04 |
| *** mjturek has joined #openstack-ironic | 15:04 | |
| dtantsur | The Forum is less than 2 months away. please prepare your approvals, reservations, visas ASAP | 15:04 |
| dtantsur | anything else worth mentioning? | 15:05 |
| iurygregory | "[all][ptl][tc] Missing PTL nominations for 31 more teams" | 15:05 |
| dtantsur | indeed. please nominate yourself :) | 15:05 |
| *** hwoarang has quit IRC | 15:05 | |
| iurygregory | Ironic is on the list =) | 15:05 |
| mgoddard | do we know if TheJulia plans to run again? | 15:06 |
| * dtantsur has no idea | 15:06 | |
| iurygregory | about the problem with uefi and bionic we had a bug with neutron if i do remember | 15:07 |
| iurygregory | not sure if we had a workaround for this | 15:07 |
| dtantsur | I don't know. I would love somebody to follow-up on that. | 15:07 |
| *** hwoarang has joined #openstack-ironic | 15:07 | |
| hjensas | I belive on mailing list the thread: [ironic][neutron] nf_conntrack_helper now disabled by default | 15:07 |
| iurygregory | derekh do you remember anything about the UEFI issue with bionic? | 15:08 |
| dtantsur | let's probably move it to open discussion | 15:08 |
| iurygregory | ++ | 15:08 |
| dtantsur | I wanted to also remind you that the final Stein release is pretty soon. End of this week or next week. | 15:08 |
| dtantsur | please plan accordingly :) | 15:09 |
| dtantsur | anything else? | 15:09 |
| dtantsur | well, "final". since we don't have RC's, our RC is a release, which is a final feature release. | 15:09 |
| iurygregory | the new cores? | 15:09 |
| dtantsur | I'm trying to tame the words, but they're stronger, so much stronger. | 15:09 |
| dtantsur | iurygregory: is it in effect? | 15:09 |
| * iurygregory not sure if need announcement lol | 15:10 | |
| iurygregory | dtantsur, for rpittau it's o.o | 15:10 |
| rpittau | dtantsur, it seems so | 15:10 |
| iurygregory | XD | 15:10 |
| arne_wiebalck | it is :-D | 15:10 |
| dtantsur | #link https://review.openstack.org/#/admin/groups/165,members welcome two new cores: arne_wiebalck and rpittau \o/ | 15:10 |
| rpittau | thank you :) | 15:10 |
| arne_wiebalck | thx! | 15:10 |
| dnuka | \o/ | 15:10 |
| hjensas | thanks arne_wiebalck and rpittau ! Great work! | 15:10 |
| dtantsur | great job! | 15:10 |
| derekh | iurygregory: yup, will talk after meeting | 15:11 |
| iurygregory | \o/ congrats | 15:11 |
| kaifeng | congrats arne_wiebalck and rpitta \o/ | 15:11 |
| dtantsur | on this positive note I hope we're done with the announcements :) | 15:11 |
| dtantsur | no action items last week, so moving right into | 15:11 |
| dtantsur | #topic Review subteam status reports (capped at ten minutes) | 15:11 |
| *** openstack changes topic to "Review subteam status reports (capped at ten minutes) (Meeting topic: ironic)" | 15:11 | |
| dtantsur | #link https://etherpad.openstack.org/p/IronicWhiteBoard around line 252 | 15:11 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Run jobs under python2 and python3 https://review.openstack.org/642474 | 15:15 |
| dtantsur | the list is shrinking, it's good :) but I'm pretty worried about the Bionic thingy | 15:15 |
| rpittau | one note on bionic, if there is a dist filter in the packages list, remember to add bionic or the pkg will not be installed :/ | 15:16 |
| dtantsur | ugh | 15:16 |
| iurygregory | so we need to check the requirements? | 15:17 |
| dtantsur | we should ensure we don't mention unsupported releases there | 15:17 |
| rpittau | no, I just explained it completely wrong, taht is for devstack | 15:17 |
| rpittau | we have a list of pkgs to install | 15:17 |
| rpittau | devstack will filter by dist if mentioned: https://github.com/openstack/ironic/blob/master/devstack/files/debs/ironic | 15:18 |
| iurygregory | gotcha | 15:19 |
| dtantsur | rpittau: yeah, that's what I meant: we should start dropping ancient versions from it | 15:20 |
| rpittau | dtantsur, ack | 15:20 |
| dtantsur | anyway, is everyone done with the statuses? | 15:20 |
| rpittau | yup | 15:21 |
| mgoddard | yeah | 15:21 |
| *** cdearborn has quit IRC | 15:21 | |
| dtantsur | #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 | |
| dtantsur | good progress this time, lemme clean up the list a bit | 15:22 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Run jobs under python2 and python3 https://review.openstack.org/642474 | 15:23 |
| Qianbiao | dtantsur Ironic will release a new version next week? | 15:23 |
| dtantsur | Qianbiao: later this week or next week, yes | 15:23 |
| Qianbiao | wow, may we speed up the merge process of https://review.openstack.org/#/c/639288/10 | 15:24 |
| patchbot | patch 639288 - ironic - Add Huawei iBMC driver support - 11 patch sets | 15:24 |
| dtantsur | Qianbiao: it's on the proposed priority list, if folks are okay with FFE on it | 15:24 |
| Qianbiao | we hope it could be release in next stable version | 15:24 |
| dtantsur | Qianbiao: what's the condition of your CI? | 15:25 |
| Qianbiao | It's almost done. | 15:25 |
| Qianbiao | check Huawei Ironic CI | 15:25 |
| Qianbiao | but the network of china is really make me sad.. | 15:25 |
| dtantsur | Qianbiao: please define "almost". does it validate a deployment with your driver? | 15:25 |
| Qianbiao | it's hard to connect to openstack git | 15:25 |
| dtantsur | I hear you :( | 15:25 |
| Qianbiao | dtantsur yes | 15:25 |
| dtantsur | some instability is fine, as long as it works in most cases and tests what it should | 15:26 |
| dtantsur | great! I'm fine with an FFE for it. Folks, especially cores? | 15:26 |
| stendulker | I'm ok to have it | 15:27 |
| Qianbiao | The ibmc driver has been test again all HUAWEI rack server, we are waiting for review and merge now. | 15:27 |
| arne_wiebalck | Qianbiao: you disabled the failing test now I see? | 15:28 |
| dtantsur | 3200+ LoC, wow | 15:28 |
| Qianbiao | yes. the old test has some problem. | 15:28 |
| TheJulia | Regarding PTL, I do plan on running again | 15:28 |
| rpittau | and also maybe don't remove rfc3986 from reqs as redfish still uses it :P | 15:28 |
| dtantsur | TheJulia: \o/ | 15:28 |
| * TheJulia goes back to waiting for the doctor | 15:28 | |
| dnuka | \o/ | 15:28 |
| dtantsur | TheJulia: and regarding FFE for Huawai? | 15:29 |
| iurygregory | TheJulia ++ | 15:29 |
| rpittau | TheJulia :) | 15:29 |
| mgoddard | I think if we have passing 3rd party CI tests for the driver it's fine | 15:29 |
| dtantsur | sorry, Huawei | 15:29 |
| mgoddard | TheJulia to the rescue! | 15:29 |
| arne_wiebalck | TheJulia: \o/ | 15:29 |
| bfournie | TheJulia ++ | 15:29 |
| TheJulia | dtantsur: if it is passing and there are no disabled unit tests without notes, I'm okay with it in principle. | 15:30 |
| arne_wiebalck | dtantsur: if you remove the image file, the patch becomes digestible in size :) | 15:30 |
| rpioso | TheJulia: :-) | 15:30 |
| hjensas | TheJulia ++ :) | 15:30 |
| rpittau | yeah, it's just 1900 lines :D | 15:30 |
| arne_wiebalck | piece of cake | 15:31 |
| kaifeng | TheJulia :) | 15:31 |
| dtantsur | Qianbiao: okay, let's try to get it in by the end of the week? | 15:31 |
| Qianbiao | <arne_wiebalck> power_state_change_timeout not means timeout times, but timeout seconds. The old assert call count has no meaning. so i just remove the asserts. | 15:31 |
| Qianbiao | dtantsur thanks | 15:31 |
| dtantsur | #agreed The Huawei driver RFE gets an FFE until the end of the week. | 15:32 |
| dtantsur | Now, back to the list of priorities, how does it look? | 15:32 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Deploy Templates: API reference https://review.openstack.org/642498 | 15:33 |
| mgoddard | I'd like to add that ^^ :) | 15:33 |
| dtantsur | mgoddard: I was actually about to ask you ;) | 15:33 |
| mgoddard | and the to-be-written docs for deploy templates | 15:33 |
| mgoddard | (working on it) | 15:33 |
| dtantsur | added | 15:34 |
| dtantsur | okay, let's use your copy :) | 15:34 |
| dtantsur | how's the list looking now? | 15:34 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Run jobs under python2 and python3 https://review.openstack.org/642474 | 15:34 |
| *** cdearborn has joined #openstack-ironic | 15:35 | |
| * dtantsur hears crickets telling him the list is good | 15:35 | |
| bfournie | For the IPv6 fixes https://review.openstack.org/#/c/626552/ and https://review.openstack.org/#/c/642039/ can these be backported to rocky/queens? | 15:35 |
| patchbot | patch 626552 - ironic-inspector - Use getaddrinfo instead of gethostbyname while res... (MERGED) - 10 patch sets | 15:35 |
| patchbot | patch 642039 - ironic-inspector - Fix lookup when ipmi_address is a hostname (MERGED) - 1 patch set | 15:35 |
| dtantsur | bfournie: if it's needed (Rocky goes first though) | 15:35 |
| kaifeng | the deploy template has one patch left https://review.openstack.org/#/c/640446/ | 15:35 |
| patchbot | patch 640446 - ironic - Deploy templates: conductor and API nits - 5 patch sets | 15:35 |
| dtantsur | bfournie: and only together | 15:36 |
| bfournie | dtantsur: k, thanks | 15:36 |
| dtantsur | kaifeng: I think the nits should not be a priority :) | 15:36 |
| kaifeng | dtantsur I guess you just accidentally remove it from the list | 15:36 |
| kaifeng | Oo, works for me :) | 15:36 |
| dtantsur | moving on? | 15:37 |
| dtantsur | #topic RFE review | 15:37 |
| *** openstack changes topic to "RFE review (Meeting topic: ironic)" | 15:37 | |
| dtantsur | I have two for today | 15:37 |
| dtantsur | #link https://storyboard.openstack.org/#!/story/2004965 Fast track deployment (FFE for Stein?) | 15:38 |
| dtantsur | I know some folks already said it's a good idea. But since we're in FF, I'd prefer to confirm it again. | 15:38 |
| dtantsur | I'm +1 on it, and IIRC stendulker put a +2 on the patch | 15:38 |
| arne_wiebalck | It's a good idea, too. | 15:38 |
| arne_wiebalck | I think it's a good idea, too. | 15:39 |
| dtantsur | any objections to granting it an FFE? | 15:39 |
| dtantsur | if nothing, I'll leave the patch open till tomorrow and will approve it if nobody objects to the code as well. | 15:39 |
| rpittau | sounds good | 15:40 |
| arne_wiebalck | dtantsur: ++ | 15:40 |
| dtantsur | #agreed The fast-track RFE is approved and gets FFE till the end of the week | 15:40 |
| kaifeng | works for me as it's off by default :D | 15:40 |
| Qianbiao | rpittau I update the redfish utils too. It now does not depend on rfc3986 too. | 15:41 |
| dtantsur | it's off by default, yeah | 15:41 |
| dtantsur | #link https://storyboard.openstack.org/#!/story/2004969 Make power synchronisation with nova configurable | 15:41 |
| dtantsur | arne_wiebalck: yours one ^^ | 15:41 |
| arne_wiebalck | yes | 15:41 |
| dtantsur | would you like to introduce it? | 15:41 |
| arne_wiebalck | just checked with Surya, she will follow up with nova upstream | 15:41 |
| arne_wiebalck | the spec is up and could in principle get reviews from today onwards | 15:42 |
| rpittau | Qianbiao, please not on the same patch | 15:42 |
| dtantsur | arne_wiebalck: do we depend on Nova before we can land code? | 15:42 |
| arne_wiebalck | this will probably take until after the PTG, though | 15:42 |
| arne_wiebalck | dtantsur: I think so | 15:42 |
| Qianbiao | rpittau ok, will revert code now | 15:42 |
| dtantsur | arne_wiebalck: okay, so I guess I can mark it triaged but not approved yet? | 15:43 |
| arne_wiebalck | we'll need to wait until they decide how we should talk to nova | 15:43 |
| arne_wiebalck | dtantsur: yes | 15:43 |
| dtantsur | ack. any comments on this? | 15:43 |
| arne_wiebalck | one thing | 15:43 |
| arne_wiebalck | one question raised was when ironic should inform nova | 15:43 |
| arne_wiebalck | always vs in unexpected situations | 15:44 |
| arne_wiebalck | jroll raised this I think | 15:44 |
| dtantsur | what do you consider unexpected here? | 15:44 |
| arne_wiebalck | the use case | 15:44 |
| arne_wiebalck | i.e. when a node comes up b/c someone powered it on OOB | 15:44 |
| *** mjturek has quit IRC | 15:45 | |
| arne_wiebalck | initially we thought we should cover this to prevent nova shuts it down again | 15:45 |
| dtantsur | arne_wiebalck: I would probably notify Nova every time the power state is changed.. | 15:45 |
| arne_wiebalck | now there is the suggestion that nova should be informed on every change | 15:45 |
| *** mjturek has joined #openstack-ironic | 15:45 | |
| dtantsur | and note: actually changed, as in node.power_state = <something> | 15:45 |
| arne_wiebalck | dtantsur: what about when you have 10k nodes? | 15:45 |
| dtantsur | and power them on/off at the same time? | 15:46 |
| arne_wiebalck | dtantsur: hopefully not :) | 15:46 |
| rpittau | probably rabbitmq will explode :P | 15:46 |
| arne_wiebalck | but there will be a constant stream of notifications | 15:46 |
| dtantsur | if we only notify Nova when 1. a node is 'active', 2. node's power_state field changes value, it should not be crazy, should it? | 15:47 |
| rpittau | arne_wiebalck, the same notifications happen when you have 10k instances, no ? | 15:47 |
| arne_wiebalck | rpittau: I think so, yes | 15:47 |
| dtantsur | rpittau: it's not rabbit, it's HTTP | 15:47 |
| dtantsur | or is it? | 15:47 |
| * dtantsur just imagined an exploding rabbit | 15:48 | |
| arne_wiebalck | the main question I think is whether we (ironic) have a preference | 15:48 |
| arne_wiebalck | if we do, we should make this clear | 15:48 |
| arne_wiebalck | and try to push for it | 15:48 |
| arne_wiebalck | the idea is on the review | 15:49 |
| arne_wiebalck | maybe we wait for first feedback from the nova folks? | 15:49 |
| arne_wiebalck | and then we see | 15:49 |
| arne_wiebalck | maybe this is out of question for reasons we don't see atm | 15:49 |
| dtantsur | #link https://review.openstack.org/#/c/636132/ Ironic-Nova power state notification spec | 15:49 |
| patchbot | patch 636132 - nova-specs - Support server power state update through external... - 1 patch set | 15:49 |
| dtantsur | yeah, I suggest we all take our time reading that | 15:49 |
| dtantsur | maybe I'll even put it as an optional item to priorities | 15:50 |
| arne_wiebalck | dtantsur: +1 | 15:50 |
| kaifeng | does this has something to do with https://github.com/openstack/ironic/blob/master/ironic/conf/conductor.py#L75 ? | 15:50 |
| arne_wiebalck | sounds good | 15:50 |
| dtantsur | kaifeng: it's a similar situation, but in Nova | 15:51 |
| arne_wiebalck | kaifeng: the idea would be to have sth equivalent in nova | 15:51 |
| arne_wiebalck | :) | 15:51 |
| kaifeng | got it | 15:52 |
| dtantsur | #topic Open discussion | 15:52 |
| *** openstack changes topic to "Open discussion (Meeting topic: ironic)" | 15:52 | |
| dtantsur | the floor is open | 15:52 |
| arne_wiebalck | I have one thing. | 15:52 |
| arne_wiebalck | For the sw RAID support, there are a couple of open questions I'd like to get input for. | 15:52 |
| arne_wiebalck | For instance, how the RAID config should be exposed. | 15:53 |
| arne_wiebalck | It's currently in the generic h/w manager. | 15:53 |
| arne_wiebalck | It was suggested to move that to a dedicated one. | 15:53 |
| arne_wiebalck | Any thoughts on this? | 15:53 |
| dtantsur | well, it's not hardware-specific, is it? | 15:53 |
| * rpioso needs to review the s/w RAID spec | 15:54 | |
| dtantsur | what are we going to win by moving it out? | 15:54 |
| arne_wiebalck | dtantsur: it's not, correct | 15:54 |
| dtantsur | arne_wiebalck: then it will be use by default anyway, right? | 15:54 |
| arne_wiebalck | dtantsur: I'm not clear if we would gain anything. | 15:54 |
| dtantsur | unless we move it out of IPA, which I personally don't want to | 15:54 |
| arne_wiebalck | dtantsur: correct | 15:54 |
| dtantsur | arne_wiebalck: is it possible to use something in target_raid_config to mean "yes, I do mean software RAID" | 15:55 |
| dtantsur | ? | 15:55 |
| arne_wiebalck | dtantsur: the argument was to prevent using it by accident | 15:55 |
| arne_wiebalck | dtantsur: yes | 15:55 |
| dtantsur | yeah, I get this argument. but a hardware manager won't help. | 15:55 |
| arne_wiebalck | I put sth in there to trigger grub on whole disk images anyway. | 15:55 |
| arne_wiebalck | and the target_raid_config needs to comply with a certiny layout | 15:55 |
| dtantsur | I guess I should actually read your patches :) | 15:56 |
| arne_wiebalck | s/certiny/certain/ | 15:56 |
| arne_wiebalck | dtantsur: | 15:56 |
| arne_wiebalck | :) | 15:56 |
| dtantsur | arne_wiebalck: my thought is: if something is only valid for software RAID, we can use it as a sign | 15:56 |
| arne_wiebalck | dtantsur: yes | 15:56 |
| dtantsur | arne_wiebalck: do we have an RFE with the design? | 15:56 |
| arne_wiebalck | I can remove the [WIP], this may make the patches more visible | 15:56 |
| arne_wiebalck | dtantsur: the design of the raid layout? | 15:57 |
| arne_wiebalck | dtantsur: or the s/w RAID as a whole? | 15:57 |
| dtantsur | arne_wiebalck: the design of the whole feature? | 15:57 |
| * arne_wiebalck is looking for the story ... | 15:57 | |
| arne_wiebalck | https://storyboard.openstack.org/#!/story/2004581 | 15:57 |
| arne_wiebalck | spec : https://review.openstack.org/#/c/624413/ | 15:58 |
| patchbot | patch 624413 - ironic-specs - Support for Software RAID (MERGED) - 2 patch sets | 15:58 |
| dtantsur | oh, I should have read this, right? :D | 15:58 |
| arne_wiebalck | The spec does not have that flag I was just mentioning, only realised later we would need sth like that. | 15:58 |
| dtantsur | arne_wiebalck: could you propose a spec update with it? | 15:59 |
| arne_wiebalck | ok, I don't want to occupy the whole meeting | 15:59 |
| arne_wiebalck | dtantsur: sure | 15:59 |
| dtantsur | no problem, thanks for the update! | 15:59 |
| dtantsur | thanks all! | 15:59 |
| rpittau | thanks! | 15:59 |
| arne_wiebalck | I'll remove the WIP, reviews welcome | 15:59 |
| dtantsur | #endmeeting | 16:00 |
| *** 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 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-dev mailing list." | 16:00 | |
| openstack | Meeting ended Mon Mar 11 16:00:00 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:00 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-03-11-15.00.html | 16:00 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-03-11-15.00.txt | 16:00 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-03-11-15.00.log.html | 16:00 |
| dnuka | o/ | 16:01 |
| *** dnuka has quit IRC | 16:01 | |
| openstackgerrit | QianBiao Ng proposed openstack/ironic master: Add Huawei iBMC driver support https://review.openstack.org/639288 | 16:02 |
| * arne_wiebalck needs to run | 16:02 | |
| arne_wiebalck | bye, everyone o/ | 16:02 |
| iurygregory | bye | 16:03 |
| dtantsur | c u arne_wiebalck | 16:03 |
| rpittau | bye arne_wiebalck o/ | 16:03 |
| iurygregory | dtantsur, for sushy i will update TheJulia patches | 16:03 |
| dtantsur | cool | 16:03 |
| iurygregory | hope she dont mind =) | 16:03 |
| dtantsur | I think Julia never minds somebody taking a bit of work off her shoulders :) | 16:04 |
| *** kaifeng has quit IRC | 16:04 | |
| iurygregory | yeah =) | 16:05 |
| *** kaifeng has joined #openstack-ironic | 16:06 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Update dist filter for devstack ubuntu https://review.openstack.org/642507 | 16:12 |
| *** cdearborn has quit IRC | 16:15 | |
| *** Qianbiao has quit IRC | 16:16 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: WIP: Add libvirt-based virtual media emulation https://review.openstack.org/640452 | 16:16 |
| openstackgerrit | Merged openstack/ironic master: Stop requiring root_gb for whole-disk images https://review.openstack.org/641993 | 16:17 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/sushy master: Change sushy devstack job to python3 https://review.openstack.org/591828 | 16:20 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/sushy master: Change sushy devstack job to python3 https://review.openstack.org/591828 | 16:21 |
| * iurygregory forgot to remove the playbooks for python2 XD | 16:21 | |
| iurygregory | done \o/ | 16:22 |
| *** pcaruana has quit IRC | 16:23 | |
| *** pcaruana has joined #openstack-ironic | 16:23 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Clean up release notes and add a prelude https://review.openstack.org/642513 | 16:23 |
| dtantsur | TheJulia: ^^ | 16:24 |
| openstackgerrit | Merged openstack/ironic master: Allocation API: optimize check on candidate nodes https://review.openstack.org/640765 | 16:25 |
| *** stendulker has quit IRC | 16:26 | |
| iurygregory | dtantsur, https://review.openstack.org/642474 python2 and python3 jobs \o/ green yay | 16:27 |
| patchbot | patch 642474 - python-ironicclient - Run jobs under python2 and python3 - 5 patch sets | 16:27 |
| *** hafsansari_ has joined #openstack-ironic | 16:27 | |
| dtantsur | iurygregory: unlikely related to your patch, but something worrying in the logs: http://logs.openstack.org/74/642474/5/check/ironicclient-tempest/a859932/controller/logs/screen-n-cpu.txt.gz#_Mar_11_16_04_40_215579 | 16:29 |
| * dtantsur wonders if jroll has anything to do with it | 16:29 | |
| *** hafsansari has quit IRC | 16:29 | |
| *** hafsansari_ has quit IRC | 16:29 | |
| dtantsur | iurygregory: more related to your patch: why do we see Python 2 deprecation in the Python 3 job? http://logs.openstack.org/74/642474/5/check/ironicclient-functional/b3d9e17/job-output.txt.gz#_2019-03-11_15_58_15_461473 | 16:30 |
| openstackgerrit | Candy Tsai proposed openstack/ironic-python-agent master: Rename build name according to README https://review.openstack.org/642515 | 16:30 |
| *** trown is now known as trown|lunch | 16:30 | |
| iurygregory | dtantsur, maybe the tox env was python2 | 16:30 |
| iurygregory | going to check | 16:31 |
| dtantsur | iurygregory: yeah, let's make sure the tox env matches the python version | 16:32 |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Allocation API: use generate_uuid for random resource classes https://review.openstack.org/641728 | 16:34 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [DNM] testing postgres job https://review.openstack.org/642430 | 16:36 |
| *** e0ne has quit IRC | 16:41 | |
| *** baha has quit IRC | 16:42 | |
| *** mjturek has quit IRC | 16:45 | |
| kandi | dtantsur: when I ran `git review`, is there any command that I should run before `git review` since I got the message "Your change was committed before the commit hook was installed." thanks :) | 16:45 |
| dtantsur | kandi: the first run of git review installs the said hook. you can amend your commit (without making any changes yourself) for it to work. | 16:46 |
| dtantsur | something like $ git commit --amend --no-edit | 16:46 |
| *** gyee has joined #openstack-ironic | 16:46 | |
| kandi | ok i'll try~ | 16:47 |
| openstackgerrit | Candy Tsai proposed openstack/ironic-python-agent master: Rename build name according to README https://review.openstack.org/642515 | 16:47 |
| kandi | the msg didn't show on this run :) | 16:49 |
| dtantsur | kandi: yes, this is what --no-edit does | 16:50 |
| kandi | dtantsur: got it~ | 16:51 |
| *** priteau has quit IRC | 16:54 | |
| *** e0ne has joined #openstack-ironic | 16:57 | |
| *** andrein has quit IRC | 17:00 | |
| *** kaifeng has quit IRC | 17:07 | |
| *** tssurya has quit IRC | 17:07 | |
| *** sburette has quit IRC | 17:08 | |
| rpittau | good night o/ | 17:09 |
| *** rpittau is now known as rpittau|afk | 17:09 | |
| *** v12aml has joined #openstack-ironic | 17:12 | |
| *** priteau has joined #openstack-ironic | 17:12 | |
| *** e0ne has quit IRC | 17:13 | |
| *** patchbot has quit IRC | 17:14 | |
| *** patchbot has joined #openstack-ironic | 17:14 | |
| *** e0ne has joined #openstack-ironic | 17:14 | |
| *** mjturek has joined #openstack-ironic | 17:17 | |
| *** mjturek has quit IRC | 17:18 | |
| *** baha has joined #openstack-ironic | 17:18 | |
| dtantsur | see you | 17:19 |
| *** dtantsur is now known as dtantsur|afk | 17:19 | |
| *** mjturek has joined #openstack-ironic | 17:26 | |
| * TheJulia sighs | 17:28 | |
| TheJulia | iurygregory: yes, I never mind. I greatly appreciate! | 17:29 |
| *** e0ne has quit IRC | 17:31 | |
| *** iurygregory has quit IRC | 17:35 | |
| *** dustinc has joined #openstack-ironic | 17:40 | |
| *** trown|lunch is now known as trown | 17:49 | |
| *** derekh has quit IRC | 18:00 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Deploy Templates: documentation https://review.openstack.org/642531 | 18:03 |
| *** dougsz has quit IRC | 18:06 | |
| openstackgerrit | Lars Kellogg-Stedman proposed openstack/ironic master: honor ipmi_port in serial console drivers https://review.openstack.org/640930 | 18:07 |
| *** dustinc has quit IRC | 18:19 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Fix broken Systems listing in dynamic emulator https://review.openstack.org/642534 | 18:23 |
| *** e0ne has joined #openstack-ironic | 18:23 | |
| *** e0ne has quit IRC | 18:25 | |
| *** jcoufal has quit IRC | 18:25 | |
| *** jcoufal has joined #openstack-ironic | 18:54 | |
| *** priteau has quit IRC | 18:55 | |
| *** e0ne has joined #openstack-ironic | 18:57 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: RPC changes for Graphical Console https://review.openstack.org/640827 | 19:01 |
| larsks | dteselkin: I responded to your comment on https://review.openstack.org/#/c/642247/2. Curious what you think. | 19:04 |
| patchbot | patch 642247 - ironic - monitor socat subprocess - 2 patch sets | 19:05 |
| *** jcoufal has quit IRC | 19:06 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [DNM] testing postgres job https://review.openstack.org/642430 | 19:15 |
| *** dsneddon has quit IRC | 19:29 | |
| *** dsneddon has joined #openstack-ironic | 19:31 | |
| *** e0ne has quit IRC | 19:40 | |
| *** e0ne has joined #openstack-ironic | 19:40 | |
| *** jcoufal has joined #openstack-ironic | 20:03 | |
| larsks | If I see a failure like "Node ... is locked by host ..., please retry after current operation is completed.", is there a way to find out what operation ironic thinks is in progress? | 20:05 |
| arne_wiebalck | larsks: does 'openstack baremetal node show <node>' give a hint? | 20:07 |
| larsks | arne_wiebalck: nothing obvious jumps out. Am I looking for something in particular? | 20:08 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Deploy Templates: documentation https://review.openstack.org/642531 | 20:10 |
| *** e0ne has quit IRC | 20:11 | |
| *** e0ne has joined #openstack-ironic | 20:12 | |
| arne_wiebalck | larsks: what does provision_state report? | 20:13 |
| larsks | arne_wiebalck: "active". But I'm sure this isn't a provisioning issue: the node is mostly up and running, but i'm working on serial console support, and after is 'node console disable', trying to 'node console enable' is hitting that 'node is locked' error. | 20:14 |
| arne_wiebalck | ok | 20:14 |
| larsks | I'm trying to figure out what my diagnostic options are. | 20:14 |
| arne_wiebalck | I think I remember seeing this when trying to set the state to a different provision_state while it was in a transient state | 20:15 |
| *** e0ne has quit IRC | 20:15 | |
| arne_wiebalck | larsks: disabling / enabling the console of an active node works in principle (just tried :)) | 20:19 |
| larsks | arne_wiebalck: sure, and I've done it before, too :) | 20:19 |
| larsks | But that also means that your serial console support is broken, which is what I'm trying to fix. | 20:20 |
| arne_wiebalck | larsks: I don't think I ever tried this on an active node before :) | 20:20 |
| openstackgerrit | Lars Kellogg-Stedman proposed openstack/ironic master: honor ipmi_port in serial console drivers https://review.openstack.org/640930 | 20:28 |
| *** manjeets has joined #openstack-ironic | 20:29 | |
| *** dougsz has joined #openstack-ironic | 20:30 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector stable/rocky: Use getaddrinfo instead of gethostbyname while resolving BMC address https://review.openstack.org/642560 | 20:35 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: RPC changes for Graphical Console https://review.openstack.org/640827 | 20:38 |
| arne_wiebalck | larsks: from what I see, there should be a task manager which holds a lock on the node; seems that the purpose is logged in debug mode | 20:43 |
| openstackgerrit | Nisha Brahmankar proposed openstack/python-ironic-inspector-client master: Deprecates "uuid" parameters to Python calls when node is expected https://review.openstack.org/642564 | 20:45 |
| larsks | arne_wiebalck: Thanks. I'm rethinking my apporach in any case, since having taken a closer look a the code I may be trying to solve the wrong problem :) | 20:47 |
| arne_wiebalck | larsks: :) | 20:48 |
| openstackgerrit | Vlad Sorokin proposed openstack/networking-generic-switch master: Add Mellanox MLNX-OS Switch support https://review.openstack.org/642565 | 20:48 |
| manjeets | TheJulia, qq on enabling another ml2 driver using devstack with ngs ? | 20:49 |
| *** e0ne has joined #openstack-ironic | 20:56 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Adds graphical console implementation for ilo drivers https://review.openstack.org/640973 | 20:58 |
| *** e0ne has quit IRC | 21:04 | |
| *** trown is now known as trown|outtypewww | 21:05 | |
| *** e0ne has joined #openstack-ironic | 21:12 | |
| *** baha has quit IRC | 21:25 | |
| *** whoami-rajat has quit IRC | 21:25 | |
| *** mjturek has quit IRC | 21:25 | |
| *** dougsz has quit IRC | 21:27 | |
| *** jcoufal has quit IRC | 21:36 | |
| *** dsneddon has quit IRC | 21:38 | |
| *** e0ne has quit IRC | 21:38 | |
| *** pcaruana has quit IRC | 21:45 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Create libvirt boot loader element if needed https://review.openstack.org/642581 | 21:47 |
| *** dsneddon has joined #openstack-ironic | 21:50 | |
| *** bfournie has quit IRC | 21:54 | |
| *** dsneddon has quit IRC | 21:55 | |
| *** jtomasek has quit IRC | 21:56 | |
| gyee | has anyone ran into this error with redfish? Message.ExtendedInfo': [{u'MessageArgs': [u'BootSourceOverrideMode'], u'MessageID': u'Base.0.10.PropertyUnknown'}]} | 22:04 |
| *** dsneddon has joined #openstack-ironic | 22:04 | |
| gyee | seem like its choking on 'BootSourceOverrideMode' here https://github.com/openstack/sushy/blob/master/sushy/resources/system/system.py#L242 | 22:04 |
| *** dsneddon has quit IRC | 22:09 | |
| *** dsneddon has joined #openstack-ironic | 22:14 | |
| *** dustinc has joined #openstack-ironic | 22:21 | |
| etingof | gyee, I'd check out conductor log to see the exact redfish exchange | 22:35 |
| etingof | gyee, may be your BMC does not support 'BootSourceOverrideMode' property? | 22:36 |
| *** bfournie has joined #openstack-ironic | 22:40 | |
| *** hwoarang has quit IRC | 22:48 | |
| *** hwoarang has joined #openstack-ironic | 22:49 | |
| *** threestrands has joined #openstack-ironic | 22:50 | |
| gyee | etingof, yeah, it's possible that the ilo does not support that particular version of the spec | 22:52 |
| gyee | it is supported according to https://www.dmtf.org/sites/default/files/standards/documents/DSP2046_2017.0a.pdf | 22:52 |
| gyee | but I don't know which version of redfish supports that | 22:52 |
| TheJulia | manjeets: o/ | 23:02 |
| TheJulia | manjeets: I think you would just have to enable the plugins as you go, but that other ml2 plugin would presumably also need a devstack plugin | 23:03 |
| manjeets | TheJulia, I enabled the plugin as it like it created the ironic-provision network etc | 23:03 |
| manjeets | two things here | 23:03 |
| TheJulia | gyee: fairly early, but given the ilo has been heavily influenced by redfish for a long time, it could also be that the ilo just needs a firmware update | 23:04 |
| manjeets | when I enabled another driver and i tried to create server with --port it sends the port id (some other port) to driver I enabled for bind port request | 23:04 |
| manjeets | which end up in failed to bind port | 23:05 |
| manjeets | second so how would dhcp work with another hardware device, would neutron dhcp interface in ironic handle this anyway ? | 23:05 |
| *** dustinc has quit IRC | 23:05 | |
| gyee | TheJulia, my target server is proliant gen9 and ilo 4. Pretty sure it has the latest firmware. I can double check. | 23:06 |
| *** dustinc has joined #openstack-ironic | 23:06 | |
| gyee | maybe I should be using ilo 5? not sure | 23:06 |
| *** dustinc has quit IRC | 23:07 | |
| *** dustinc has joined #openstack-ironic | 23:07 | |
| gyee | looks like HPE's version is a bit different https://github.com/HewlettPackard/python-redfish-utility/blob/master/src/extensions/BIOS%20COMMANDS/BootOrderCommand.py | 23:11 |
| gyee | makes me nervous when I see "extensions" in the path :-) | 23:12 |
| TheJulia | GAH | 23:16 |
| TheJulia | :( | 23:16 |
| TheJulia | a lot of the python-redfish stuff was also very early on | 23:17 |
| gyee | TheJulia, let me take this up with the HPE folks before going any deeper into this rabbit hole. | 23:17 |
| TheJulia | ++++ | 23:18 |
| gyee | first time using redfish so its entirely possible that I don't know wth I am doing :-) | 23:18 |
| TheJulia | I do know there have been many updates to their ilo4 redfish support, so worth double checking versions as well | 23:19 |
| TheJulia | They may just want people to focus on their ilo specific drivers as well | 23:19 |
| gyee | yep, email sent, lets see | 23:21 |
| TheJulia | For everyone curious if I was going to run again, my candidacy statement has been posted. | 23:38 |
| *** threestrands has quit IRC | 23:45 | |
| *** dsneddon has quit IRC | 23:53 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!