*** diurnalist has joined #openstack-ironic | 00:01 | |
*** Goneri has quit IRC | 00:22 | |
*** k_mouza has joined #openstack-ironic | 00:30 | |
*** k_mouza has quit IRC | 00:35 | |
*** rloo has quit IRC | 01:10 | |
*** bfournie has joined #openstack-ironic | 01:50 | |
*** markmcclain has quit IRC | 02:01 | |
openstackgerrit | Steve Baker proposed openstack/ironic master: devstack json-rpc, use http basic auth https://review.opendev.org/736398 | 02:02 |
---|---|---|
*** markmcclain has joined #openstack-ironic | 02:02 | |
*** dking has quit IRC | 02:13 | |
*** rcernin has quit IRC | 02:48 | |
*** hoonetorg has quit IRC | 02:51 | |
*** Lucas_Gray has quit IRC | 02:55 | |
*** rcernin has joined #openstack-ironic | 02:59 | |
*** rcernin has quit IRC | 03:05 | |
*** Qianbiao has joined #openstack-ironic | 03:08 | |
*** mkrai has joined #openstack-ironic | 03:17 | |
*** tzumainn has quit IRC | 03:18 | |
*** rcernin has joined #openstack-ironic | 03:20 | |
*** Qianbiao has quit IRC | 03:20 | |
*** Qianbiao has joined #openstack-ironic | 03:21 | |
*** rcernin has quit IRC | 03:22 | |
*** rcernin has joined #openstack-ironic | 03:22 | |
*** ravsingh has joined #openstack-ironic | 03:30 | |
*** hoonetorg has joined #openstack-ironic | 04:35 | |
*** ociuhandu has joined #openstack-ironic | 05:21 | |
*** ociuhandu has quit IRC | 05:26 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Adds boot mode support to iLO management interface https://review.opendev.org/736232 | 05:30 |
arne_wiebalck | Good morning, ironic! | 05:38 |
*** mkrai has quit IRC | 05:59 | |
*** uzumaki has quit IRC | 06:02 | |
*** mkrai has joined #openstack-ironic | 06:04 | |
*** uzumaki has joined #openstack-ironic | 06:07 | |
openstackgerrit | Pratik proposed openstack/bifrost master: optimize size and time using "--no-cache-dir" https://review.opendev.org/736447 | 06:22 |
*** yolanda has quit IRC | 06:31 | |
*** Habib-A has joined #openstack-ironic | 06:31 | |
Habib-A | Good morning #ironic | 06:31 |
*** yolanda has joined #openstack-ironic | 06:36 | |
*** rpittau|afk is now known as rpittau | 06:43 | |
rpittau | good morning ironic! o/ | 06:43 |
Habib-A | Agent returned error for clean step {u'priority': 10, u'interface': u'deploy', u'step': u'erase_devices', u'abortable': True, u'reboot_requested': False} on node 97a50bb3-6b6c-45be-ace1-1f49cbd7d9fc : {u'message': u'Clean step failed: Error performing clean_step erase_devices: No HardwareManager found to handle method: Could not find method: | 06:49 |
Habib-A | erase_block_device', u'code': 500, u'type': u'CleaningError', u'details': u'Error performing clean_step erase_devices: No HardwareManager found to handle method: Could not find method: erase_block_device'}. | 06:49 |
Habib-A | what should i do ?! | 06:49 |
Habib-A | since i switched my 1TB HDDs with 146gb ones and re run the ironic provide command which yielded this | 06:50 |
*** uzumaki has quit IRC | 06:56 | |
rpittau | Habib-A: that looks like the ironic-python-agent can't find a hardware manager to handle the erase, which is kind of weird, do you have the full logs or at least a bigger chunk ? | 07:03 |
Habib-A | they are attached to a raid controller and i disabled ata erase as it caused errors i was hoping shred would do the job instead | 07:03 |
Habib-A | for the conducter yes | 07:04 |
Habib-A | i will send them as soon as i can get them from the server | 07:04 |
gudrutis2 | morning o/ | 07:05 |
*** Qianbiao has quit IRC | 07:16 | |
*** Qianbiao has joined #openstack-ironic | 07:17 | |
iurygregory | good morning ironic | 07:26 |
*** mkrai has quit IRC | 07:32 | |
*** uzumaki has joined #openstack-ironic | 07:33 | |
*** dougsz has joined #openstack-ironic | 07:39 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_base_manager https://review.opendev.org/734811 | 07:45 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_neutron https://review.opendev.org/735915 | 07:45 |
*** Qianbiao has quit IRC | 07:46 | |
*** Qianbiao has joined #openstack-ironic | 07:47 | |
*** Qianbiao_ has joined #openstack-ironic | 07:48 | |
Qianbiao_ | hello ironic. | 07:50 |
Qianbiao_ | good morning folks | 07:50 |
iurygregory | hi Qianbiao_ | 07:51 |
*** Qianbiao has quit IRC | 07:51 | |
*** rcernin_ has joined #openstack-ironic | 07:54 | |
*** rcernin has quit IRC | 07:54 | |
*** Qianbiao_ has quit IRC | 07:55 | |
iurygregory | oh nice driver-libs are failing in stein .-. | 08:00 |
openstackgerrit | Pratik proposed openstack/virtualbmc master: optimize size and time using "--no-cache-dir" https://review.opendev.org/736490 | 08:01 |
*** diurnalist has quit IRC | 08:03 | |
*** lucasagomes has joined #openstack-ironic | 08:08 | |
*** Lucas_Gray has joined #openstack-ironic | 08:12 | |
uzumaki | iurygregory, we might need to open the "stein's gate" to fix those libs xD | 08:17 |
iurygregory | uzumaki, yes | 08:18 |
iurygregory | =d | 08:18 |
iurygregory | =D* | 08:18 |
*** ravsingh has quit IRC | 08:20 | |
*** rcernin_ has quit IRC | 08:20 | |
*** dtantsur|afk is now known as dtantsur | 08:29 | |
*** ravsingh has joined #openstack-ironic | 08:33 | |
mgoddard | Has anyone seen an error like this with virtualbmc? http://paste.openstack.org/show/794922/ | 08:38 |
mgoddard | Fails on ipmitool set bootdev pxe in kayobe CI. Works before upgrade, fails afterwards | 08:39 |
dtantsur | mgoddard: anything in vbmc logs? | 08:40 |
mgoddard | dtantsur: nope | 08:41 |
mgoddard | seems like it might have started after we switched from ussuri to master | 08:41 |
dtantsur | I don't think we've changed anything in vbmc since ussuri.. maybe the pyghmi version? | 08:42 |
dtantsur | or ipmitool itself? | 08:42 |
*** ociuhandu has joined #openstack-ironic | 08:42 | |
mgoddard | Not seeing anything in commit logs | 08:44 |
iurygregory | we forgot to pin ibmcclient in stable/stein and we are ussing 0.2.3 omg | 08:44 |
*** rcernin_ has joined #openstack-ironic | 08:45 | |
dtantsur | sigh | 08:48 |
iurygregory | dtantsur, squash backports would be the approach or I can just change https://github.com/openstack/ironic/blob/stable/stein/driver-requirements.txt#L25 and add to https://review.opendev.org/#/c/736017/ ? | 08:49 |
patchbot | patch 736017 - ironic (stable/stein) - devstack: Use uwsgi binary from path; temporary di... - 1 patch set | 08:49 |
*** rcernin_ has quit IRC | 08:50 | |
dtantsur | iurygregory: just squash them, mentioning it in the commit message | 08:50 |
iurygregory | dtantsur, ack | 08:50 |
dtantsur | can I get some reviews on https://review.opendev.org/#/c/733559/ please? | 09:08 |
patchbot | patch 733559 - bifrost - Quick start Bifrost CLI - 23 patch sets | 09:08 |
dtantsur | it actually works, I've been using it for my development environment | 09:08 |
*** jawad_axd has joined #openstack-ironic | 09:09 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [WIP] Use configure_keystone_authtoken_middleware https://review.opendev.org/736510 | 09:10 |
rpittau | dtantsur: I have that on top of my list, just wanted to do some tests, I'll give it a look today | 09:13 |
Habib-A | rpittau attached the ironic agent log https://pastebin.com/8Eu5h8EH | 09:21 |
Habib-A | @rpittau https://pastebin.com/8Eu5h8EH | 09:21 |
Habib-A | any help is appreciated https://pastebin.com/8Eu5h8EH | 09:21 |
openstackgerrit | Merged openstack/ironic master: Fix mock callable for Python 3.6 and precedent https://review.opendev.org/736131 | 09:22 |
openstackgerrit | Merged openstack/ironic master: Enforce autospec in test_rpc_api and test_steps https://review.opendev.org/734132 | 09:22 |
openstackgerrit | Merged openstack/ironic master: Enforce autospec in test_base_manager https://review.opendev.org/734811 | 09:22 |
*** yaawang_ has quit IRC | 09:24 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic stable/stein: devstack: Use uwsgi binary from path; temporary disable grenade; pin ibmc https://review.opendev.org/736017 | 09:26 |
*** mkrai has joined #openstack-ironic | 09:28 | |
*** yaawang_ has joined #openstack-ironic | 09:41 | |
openstackgerrit | Merged openstack/ironic master: Enforce autospec in test_neutron https://review.opendev.org/735915 | 09:44 |
*** Habib-A has quit IRC | 09:47 | |
*** Qianbiao has joined #openstack-ironic | 09:50 | |
*** ociuhandu has quit IRC | 09:52 | |
dtantsur | would also appreciate some reviews on https://review.opendev.org/#/c/736191/ | 09:54 |
patchbot | patch 736191 - ironic-python-agent - Make get_partition_uuids work with whole disk images - 1 patch set | 09:54 |
*** Qianbiao has quit IRC | 09:54 | |
*** Qianbiao has joined #openstack-ironic | 09:54 | |
*** ociuhandu has joined #openstack-ironic | 09:59 | |
*** diurnalist has joined #openstack-ironic | 10:00 | |
openstackgerrit | Mark Goddard proposed openstack/tenks master: Install python setuptools & virtualenv https://review.opendev.org/736249 | 10:01 |
*** tkajinam has quit IRC | 10:05 | |
*** rpittau is now known as rpittau|bbl | 10:06 | |
*** k_mouza has joined #openstack-ironic | 10:17 | |
openstackgerrit | Merged openstack/tenks stable/1.0: CI: multiple fixes https://review.opendev.org/735917 | 10:21 |
*** ravsingh has quit IRC | 10:32 | |
*** Qianbiao has quit IRC | 10:34 | |
*** rcernin_ has joined #openstack-ironic | 10:47 | |
*** Habib-A has joined #openstack-ironic | 11:14 | |
Habib-A | TheJulia https://pastebin.com/8Eu5h8EH help please | 11:15 |
mgoddard | dtantsur: catching up on PTG discussions. Who was interested in Standalone improvements? | 11:19 |
mgoddard | dtantsur: I saw someone pointed out a-universe-from-nothing. That's probably not a great fit, but you can use kolla-ansible for standalone ironic. I made a PoC here: https://github.com/markgoddard/bikolla | 11:20 |
mgoddard | I expect it needs updating | 11:20 |
mgoddard | wins over bifrost - you don't have to maintain it, it supports multinode & HA, it uses containers | 11:22 |
mgoddard | also you can add as many or few additional openstack services as you like | 11:22 |
Habib-A | 2020-06-18 09:17:38.779 797 ERROR root ironic_python_agent.errors.CleaningError: Clean step failed: Error performing clean_step erase_devices: No HardwareManager found to handle method: Could not find method: erase_block_device | 11:24 |
Habib-A | what should i do | 11:24 |
dtantsur | Habib-A: shred: /dev/sdb: error writing at offset 0: Remote I/O error | 11:27 |
dtantsur | this is the actual failure, it seems a problem with your hardware | 11:27 |
*** derekh has joined #openstack-ironic | 11:27 | |
dtantsur | mgoddard: I was interested :) Honestly, rather than having an alternative to bifrost, I'd prefer converging with bifrost | 11:28 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Stop allocating double space for raw images https://review.opendev.org/736391 | 11:28 |
mgoddard | dtantsur: how do you mean converging? | 11:28 |
dtantsur | mgoddard: making bifrost support this mode of operation maybe? | 11:28 |
Habib-A | dtantsur TheJulia i tried the centos-ipa-train image and worked !! | 11:28 |
mgoddard | dtantsur: bifrost as a python CLI wrapper? | 11:29 |
dtantsur | python and ansible, yeah | 11:29 |
Habib-A | i guess the tinycore ipa image has something missing/different :/ | 11:29 |
dtantsur | Habib-A: note that tinyIPA images are for CI/local development only. They don't necessary support real hardware. | 11:29 |
dtantsur | mgoddard: or rather: two modes of operation in bifrost: containers via kollan-ansible, source code in a venv | 11:30 |
mgoddard | dtantsur: I would like that. There are still useful parts of bifrost, but it could delegate deployment to kolla | 11:30 |
dtantsur | dunno how much of maintenance burden it is though | 11:30 |
mgoddard | I'd probably say just pick a path | 11:31 |
dtantsur | well, I definitely see value in a source-based installation (esp. to replace devstack) | 11:31 |
*** mkrai has quit IRC | 11:33 | |
*** Qianbiao has joined #openstack-ironic | 11:35 | |
*** ociuhandu has quit IRC | 11:50 | |
openstackgerrit | Merged openstack/ironic master: Block port deletions where vif is present https://review.opendev.org/665835 | 11:50 |
openstackgerrit | Merged openstack/ironic stable/train: devstack: Use uwsgi binary from path; temporary disable grenade https://review.opendev.org/736015 | 11:50 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic stable/ussuri: Enable grenade https://review.opendev.org/736664 | 11:54 |
Habib-A | dtantsur thanks for the info i read the first part in the documentation but i didn't realize the second part "They don't necessary support real hardware." thanks :D | 12:00 |
*** k_mouza has quit IRC | 12:10 | |
openstackgerrit | Pratik Raj proposed openstack/ironic master: optimize size and time using --no-cache-dir https://review.opendev.org/736688 | 12:13 |
openstackgerrit | Pratik Raj proposed openstack/ironic-inspector master: optimize size and time using --no-cache-dir https://review.opendev.org/736689 | 12:13 |
*** Qianbiao has quit IRC | 12:14 | |
*** k_mouza has joined #openstack-ironic | 12:14 | |
openstackgerrit | Pratik Raj proposed openstack/ironic-lib master: optimize size and time using --no-cache-dir https://review.opendev.org/736690 | 12:14 |
openstackgerrit | Pratik Raj proposed openstack/ironic-prometheus-exporter master: optimize size and time using --no-cache-dir https://review.opendev.org/736691 | 12:15 |
openstackgerrit | Pratik Raj proposed openstack/ironic-python-agent master: optimize size and time using --no-cache-dir https://review.opendev.org/736692 | 12:15 |
dtantsur | how can we stop this ^^^? | 12:15 |
openstackgerrit | Pratik Raj proposed openstack/ironic-python-agent-builder master: optimize size and time using --no-cache-dir https://review.opendev.org/736693 | 12:16 |
openstackgerrit | Pratik Raj proposed openstack/ironic-tempest-plugin master: optimize size and time using --no-cache-dir https://review.opendev.org/736694 | 12:16 |
* dtantsur -2 everything and considers reaching out to the contributor | 12:17 | |
openstackgerrit | Pratik Raj proposed openstack/ironic-ui master: optimize size and time using --no-cache-dir https://review.opendev.org/736695 | 12:17 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Remove .travis.yml (wow, it is ancient) https://review.opendev.org/736697 | 12:19 |
rpittau|bbl | dtantsur: going to auto -2 them | 12:20 |
dtantsur | yep | 12:20 |
*** rpittau|bbl is now known as rpittau | 12:20 | |
iurygregory | I will help | 12:20 |
iurygregory | =0 | 12:20 |
iurygregory | =) | 12:20 |
iurygregory | -2 or -1 in workflow? | 12:20 |
dtantsur | -2 with an explanation (feel free to copy mine from the patches above) | 12:20 |
iurygregory | http://lists.openstack.org/pipermail/openstack-discuss/2020-June/015515.html | 12:21 |
rpittau | can I just put "me no like it" ? | 12:21 |
iurygregory | we can also use the discussion from the list | 12:21 |
dtantsur | iurygregory: an update: AJaeger asked us to abandon them to free up the CI resources | 12:21 |
dtantsur | can someone please do it (even for patches I already -2ed)? | 12:22 |
rpittau | even better | 12:22 |
iurygregory | yeah | 12:22 |
rpittau | man, I'm too slow :D | 12:25 |
iurygregory | lol | 12:25 |
rpittau | time to refresh gerrit and they were all abandoned already :D | 12:26 |
* iurygregory hides | 12:27 | |
iurygregory | dtantsur, do you have any ideas on https://review.opendev.org/#/c/736170/ or keep trying recheck? .-. | 12:28 |
patchbot | patch 736170 - metalsmith (stable/train) - Use is instead of ambiguous pipe - 1 patch set | 12:28 |
* rpittau goes back to beat libvirt | 12:28 | |
*** rh-jelabarre has joined #openstack-ironic | 12:28 | |
* iurygregory goes test bifrost cli on f30 | 12:30 | |
dtantsur | iurygregory: I haven't checked what the problem is yet. any hints? | 12:33 |
*** derekh has quit IRC | 12:33 | |
iurygregory | dtantsur, it fails during Wait for provisioning of instances | 12:35 |
iurygregory | but it shows some infor about the node that seems correct | 12:36 |
rpittau | iurygregory: have you checked the conductor logs? I see an error that is also in the ipa-ramdisk | 12:37 |
iurygregory | rpittau, haven't looked at conductor yet was looking at the execution logs to get an idea first | 12:38 |
iurygregory | Failed to install a bootloader doesn't look good | 12:38 |
rpittau | iurygregory: UnicodeEncodeError: 'utf-8' codec can't encode characters in position 0-1: surrogates not allowed | 12:38 |
rpittau | in https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_a4f/736170/1/check/metalsmith-integration-glance-localboot-centos7/a4f02f3/controller/ironic-bm-logs/node-0_no_ansi_2020-06-17-21%3A14%3A18.log | 12:38 |
dtantsur | Oo | 12:39 |
iurygregory | wtf | 12:39 |
rpittau | yeah | 12:39 |
rpittau | seems oslo related | 12:39 |
dtantsur | why are we even using python 3 on train? | 12:39 |
rpittau | heh | 12:40 |
iurygregory | because we are a bit crazy? :D | 12:40 |
*** derekh has joined #openstack-ironic | 12:40 | |
TheJulia | dtantsur: uhh.. because we were going to python3 the next cycle and TC was pushing for there to be python3 CI jobs during train | 12:42 |
TheJulia | Good morning everyone | 12:42 |
iurygregory | good morning TheJulia | 12:43 |
rpittau | good morning TheJulia :) | 12:43 |
dtantsur | morning TheJulia | 12:44 |
dtantsur | TheJulia: in a desperate attempt to recover my test environment I've restarted dnsmasq and libvirtd, and now PXE does not work any more :D | 12:45 |
TheJulia | dtantsur: oh... are there multiple dnsmasqs? | 12:45 |
dtantsur | TheJulia: there are ones from libvirt, but tcpdump only shows responses from the correct one | 12:46 |
dtantsur | which seem to have all the necessary information | 12:46 |
TheJulia | did libvirt change the VMs? | 12:47 |
*** jawad_axd has quit IRC | 12:47 | |
dtantsur | that is possible | 12:48 |
dtantsur | meaning, a bifrost test environment won't survive a reboot :( | 12:48 |
dtantsur | TheJulia: speaking of which, do I have any chances of luring you into reviewing https://review.opendev.org/#/c/733559/ today? | 12:49 |
TheJulia | that seems wrong, but I'e also long had a habit of wiping out my test vms | 12:49 |
patchbot | patch 733559 - bifrost - Quick start Bifrost CLI - 23 patch sets | 12:49 |
TheJulia | I can give it a try... my calendar for the nexxt few hours looks like a nightmare | 12:49 |
* TheJulia coffees | 12:50 | |
iurygregory | dtantsur, I'm testing locally on F30 the bifrost cli | 12:51 |
dtantsur | cool, the more eyes the better | 12:52 |
iurygregory | I love when a website only have "payment on delivery" but they don't say if they accept credit/debit cards or only money .-. | 12:55 |
dtantsur | yeah, amazing | 12:55 |
* iurygregory needs a good chair =( | 12:55 | |
*** Qianbiao has joined #openstack-ironic | 12:57 | |
TheJulia | :( | 13:01 |
iurygregory | the best part is: write an email in english for a czech company \o/ | 13:02 |
*** rloo has joined #openstack-ironic | 13:04 | |
openstackgerrit | Mark Goddard proposed openstack/tenks master: Trust receive filters of baremetal nodes https://review.opendev.org/736708 | 13:05 |
dtantsur | iurygregory: been there, done that :) | 13:06 |
iurygregory | dtantsur, hehehe | 13:07 |
rpittau | ok, I'm banging my head since a while, this is an interesting error from a dib job: http://paste.openstack.org/show/794938/ | 13:09 |
rpittau | seems like there's no iscsi connection detected | 13:10 |
rpittau | this is the full ipa-ramdisk log https://e76b50c684498fe349e9-b3e81020c9d0d64d224068ffd33ef104.ssl.cf1.rackcdn.com/716889/40/check/ironic-tempest-ipa-partition-uefi-pxe-grub2/301cdb8/controller/logs/ironic-bm-logs/node-0_console_2020-06-10-08%3A57%3A25_log.txt | 13:11 |
rpittau | any ideas? | 13:12 |
TheJulia | Hmm, I guess I got completely distracted and never managed to send an email regarding the review jam | 13:12 |
iurygregory | TheJulia, you sent | 13:14 |
iurygregory | [ironic] code review jam - Thursday @ 4 PM UTC | 13:14 |
iurygregory | http://lists.openstack.org/pipermail/openstack-discuss/2020-June/015455.html | 13:14 |
dtantsur | yep, I've seen it too | 13:19 |
dtantsur | I'll have to stop by at the release team meeting though | 13:19 |
TheJulia | sweet | 13:21 |
TheJulia | okay | 13:21 |
*** sshnaidm is now known as sshnaidm|mtg | 13:23 | |
*** Habib-A has quit IRC | 13:28 | |
uzumaki | iurygregory, I also want a good chair, I'm stuck in this sofa, which is terrible to sit in. Adds to my already increasing set of excuses to procrastinate .-. | 13:33 |
iurygregory | uzumaki, damm =( I can imagine | 13:33 |
uzumaki | I had a really comfortable chair in the office, working at home just doesn't work for me so well.. | 13:34 |
iurygregory | dtantsur, first try with cli failed because I forgot to start libvirt (facepalm) | 13:34 |
dtantsur | heh | 13:35 |
dtantsur | iurygregory: we should probably fix it in bifrost, but that's not the cli problem | 13:35 |
uzumaki | has anyone seen the new Github interface? It's kinda cool | 13:35 |
iurygregory | dtantsur, yeah | 13:35 |
dtantsur | TheJulia: unfortunately, I don't think https://review.opendev.org/#/c/734709/ will work, see inline | 13:35 |
patchbot | patch 734709 - ironic-python-agent - Add timeout operations to try and prevent hang on ... - 5 patch sets | 13:35 |
dtantsur | and https://github.com/psf/requests/issues/4876#issuecomment-450595079 sounds like they don't care :( | 13:36 |
uzumaki | =O that should be a crime, not caring about your misbehaving code | 13:37 |
dtantsur | I don't entirely disagree with that | 13:37 |
dtantsur | unrelated, I've talked to blivet people re publishing to pypi and they seem open to the idea: https://github.com/storaged-project/blivet/issues/858 | 13:43 |
TheJulia | seriously?!? | 13:43 |
dtantsur | seriously re what: requests or blivet? :) | 13:43 |
TheJulia | dtantsur: requests. fwiw, the field team has tested both patches an even with just the first patch they end up in fail states instead of endless deploy wait, and the second patch with tuned timers the deploy works just fine | 13:44 |
dtantsur | mm, well, I don't know what to say. all I can find explicitly says it shouldn't work :( | 13:44 |
dtantsur | if we're absolutely, totally sure it works, I'm fine with letting it in | 13:45 |
dtantsur | the change is straightforward and shouldn't cause regressions, so whatever | 13:45 |
dtantsur | but I don't understand how it can possibly work, given everything I've found | 13:45 |
TheJulia | dtantsur: that bug is different | 13:46 |
dtantsur | the bug may be different, but I've found two references to "iter_content never yields an empty string" | 13:46 |
dtantsur | (see inline) | 13:46 |
*** Goneri has joined #openstack-ironic | 13:47 | |
TheJulia | I _think_ the only way it works is if it yeilds none | 13:47 |
dtantsur | do you have any references to it ever yielding none? I couldn't find that either.. | 13:48 |
TheJulia | I found a post by lukasa describing how to catch the sort of issues we were hitting and what works and what doesn't work from around 2018 | 13:48 |
* dtantsur may need to dive into urllib3 | 13:48 | |
TheJulia | I'd have to dig through my browser history at this point | 13:48 |
*** tzumainn has joined #openstack-ironic | 13:48 | |
TheJulia | dtantsur: possibly. We're definitely in the area of super not fun socket programming behaviors | 13:48 |
dtantsur | lovely. okay, lemme do my own research before I +2. | 13:50 |
iurygregory | dtantsur, it works \o/ | 13:51 |
dtantsur | yay! | 13:51 |
dtantsur | okay, on one code path it seems impossible: https://github.com/urllib3/urllib3/blob/master/src/urllib3/response.py#L520 | 13:51 |
dtantsur | on the other too: https://github.com/urllib3/urllib3/blob/master/src/urllib3/response.py#L574-L578 | 13:52 |
dtantsur | now the last path | 13:52 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Use configure_keystone_authtoken_middleware https://review.opendev.org/736510 | 13:52 |
TheJulia | hmm, found a requests bug from 2014 where lukasa points at using the time module with iter_content to detect cases | 13:52 |
TheJulia | but not the one I was looking for | 13:53 |
iurygregory | dtantsur, the only thing we are missing is documentation right? | 13:53 |
dtantsur | the third too https://github.com/urllib3/urllib3/blob/master/src/urllib3/response.py#L770-L771 | 13:53 |
dtantsur | iurygregory: yeah, will write it a bit later | 13:53 |
dtantsur | probably more options later on, but not for the MVP | 13:54 |
iurygregory | dtantsur, no worries tks! | 13:54 |
dtantsur | TheJulia: urllib3 seems serious about their promise not to return anything empty.. | 13:54 |
TheJulia | :( | 13:55 |
dtantsur | unrelated, but WTF https://github.com/urllib3/urllib3/pull/1669 | 13:55 |
*** Goneri has quit IRC | 13:55 | |
rpittau | ok..... | 13:56 |
iurygregory | Fix for python4 | 13:57 |
iurygregory | wow | 13:57 |
TheJulia | dtantsur: I guess I feel better that we're not the only folks that have struggled with this exact same thing, so many references to the same basic issue :( | 13:58 |
dtantsur | yeah, it's super annoying | 13:58 |
dtantsur | connection interrupted in the middle is not some exotic case | 13:58 |
dtantsur | these people have clearly forgotten the deal-up times :( | 13:58 |
TheJulia | heh | 14:01 |
TheJulia | there are literally so many different threads to the discussion, I've not yet found the same one | 14:08 |
rpioso | Good morning, ironic | 14:08 |
*** Goneri has joined #openstack-ironic | 14:08 | |
rpioso | rpittau: Thank you for the reviews of https://review.opendev.org/#/c/731644 | 14:09 |
patchbot | patch 731644 - ironic - Fix Redfish handle no continuous override boot src - 5 patch sets | 14:09 |
rpittau | hey rpioso np :) | 14:09 |
rpioso | rpittau: :-) Looks like the Zuul checks liked it, too, or at least it did 10 hours ago. | 14:10 |
rpittau | yep, that looks fine from CI perspective :) | 14:11 |
jamesdenton | QQ with regard to port groups. Is the portgroup mode actually utilized by anything at the moment? The documentation alludes to cloud-init being able to configure the bond automatically based on the mode, but I'm a little confused about the requirements for this behavior to occur (standalone true vs false) | 14:11 |
rpioso | Would appreciate another core review to move it forward. | 14:11 |
*** rcernin_ has quit IRC | 14:14 | |
TheJulia | dtantsur: I really wish I had lukasa on speed dial at the moment | 14:17 |
dtantsur | heh | 14:17 |
openstackgerrit | Merged openstack/bifrost master: Quick start Bifrost CLI https://review.opendev.org/733559 | 14:25 |
dtantsur | wooohoo | 14:27 |
dtantsur | if we can get https://review.opendev.org/#/c/735933/ in too, I'll no longer need to manually patch my bifrost environment :) | 14:28 |
patchbot | patch 735933 - bifrost - Fix fast-track deployment after discovery/inspection - 1 patch set | 14:28 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: CI: clean up non-votings jobs https://review.opendev.org/735581 | 14:29 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Fix uefi jobs with native ubuntu ovmf package https://review.opendev.org/716889 | 14:32 |
*** Goneri has quit IRC | 14:35 | |
rpittau | I just realized I updated this ^ patch and we don't have ipa-ramdisk logs............ :( | 14:35 |
* TheJulia looks at requests | 14:36 | |
TheJulia | hmm | 14:36 |
dtantsur | jamesdenton: IIRC (and my memory on this topic is incomplete) the mode is used by ML2 drivers | 14:39 |
jamesdenton | dtantsur in this case I am using NGS and the portgroup ports are being configured successfully (non-LACP). Docs at https://docs.openstack.org/ironic/ussuri/admin/portgroups.html kinda imply cloud-init will do some on-host bond configuration based on the mode, but i'm not so sure | 14:43 |
TheJulia | eww, entirely different behavior between chunked encoding and non-chunked | 14:46 |
*** ociuhandu has joined #openstack-ironic | 14:50 | |
*** Goneri has joined #openstack-ironic | 14:51 | |
dtantsur | yeah, different code paths | 14:54 |
dtantsur | but both seem to skip empty chunks | 14:54 |
TheJulia | I thought you could yeild without a data, but I'm not seeing anything yet, however I'm just scratching the surface and crying | 14:55 |
*** ociuhandu has quit IRC | 14:55 | |
dtantsur | :( | 14:58 |
*** portdirect has joined #openstack-ironic | 14:59 | |
portdirect | hey! im trying to get a ci env up for metal3 atm, and have a hit an issue with sushi-tools | 15:00 |
portdirect | is it possible to use the Libvirt domain name, rather than the domain uuid here? | 15:00 |
*** dking has joined #openstack-ironic | 15:00 | |
portdirect | the reason for asking is that id like to have a reasonably dynamic environment - ie create x hosts etc for work we are doing in airship | 15:02 |
portdirect | and with uuids this is rather hard to drive, I suppose we could deterministically create uuids - but it seems like a really ugly hack | 15:02 |
Qianbiao | <portdirect> metal3 has a node name | 15:03 |
Qianbiao | * or named bmh node name | 15:03 |
Qianbiao | that is the same as ironic node name | 15:03 |
Qianbiao | you can use that instead of uuid | 15:03 |
portdirect | Qianbiao: what im looking for here comes earlier than that | 15:06 |
portdirect | eg: | 15:06 |
portdirect | https://www.irccloud.com/pastebin/nDHhI6jS/ | 15:06 |
portdirect | ideally id like to be able to have output like: | 15:06 |
portdirect | https://www.irccloud.com/pastebin/0fsvXdcd/ | 15:07 |
*** diurnalist has quit IRC | 15:08 | |
Qianbiao | sorry not familiar with libvirt redfish vbmc. | 15:08 |
Qianbiao | I am testing metal3 with real baremetal | 15:08 |
*** cdearborn has joined #openstack-ironic | 15:11 | |
*** diurnalist has joined #openstack-ironic | 15:16 | |
dtantsur | portdirect: I remember we switched to using UUIDs for some reason at some point, I don't remember why | 15:17 |
iurygregory | name is not unique probably... | 15:25 |
portdirect | dtantsur: I think in many cases that makes sense | 15:26 |
portdirect | but it also makes things rather hard as well :) | 15:26 |
portdirect | I see there is actually a redirect so if you hit the domain name, it ends up in the right place | 15:26 |
portdirect | https://www.irccloud.com/pastebin/3I4eq0Fr/ | 15:26 |
portdirect | I wonder, could we make the default [uuid|name] toggleable to support this? | 15:27 |
dtantsur | I would not mind it, patches are welcome | 15:28 |
*** Lucas_Gray has quit IRC | 15:29 | |
*** diurnalist has quit IRC | 15:34 | |
*** diurnalist has joined #openstack-ironic | 15:40 | |
*** ociuhandu has joined #openstack-ironic | 15:40 | |
*** sshnaidm|mtg is now known as sshnaidm|ruck | 15:41 | |
*** gyee has joined #openstack-ironic | 15:50 | |
*** priteau has joined #openstack-ironic | 15:52 | |
*** Qianbiao has quit IRC | 15:53 | |
*** diurnalist has quit IRC | 15:58 | |
TheJulia | shall we jam through some reviews? | 16:01 |
dtantsur | what's the link? | 16:01 |
TheJulia | meetpad.opendev.org/ironic | 16:01 |
iurygregory | 1min | 16:02 |
iurygregory | going to grab some water | 16:02 |
*** lucasagomes has quit IRC | 16:03 | |
TheJulia | join us! | 16:05 |
openstackgerrit | Merged openstack/ironic master: CI: make grenade voting again https://review.opendev.org/735577 | 16:08 |
iurygregory | \o/ | 16:08 |
openstackgerrit | Merged openstack/bifrost master: CI: clean up non-votings jobs https://review.opendev.org/735581 | 16:28 |
*** dougsz has quit IRC | 16:31 | |
*** diurnalist has joined #openstack-ironic | 16:35 | |
*** sshnaidm|ruck is now known as sshnaidm|off | 16:52 | |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Remove .travis.yml (wow, it is ancient) https://review.opendev.org/736697 | 16:58 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Merge jobs https://review.opendev.org/726121 | 17:00 |
rpittau | bye everyone! o/ | 17:00 |
*** rpittau is now known as rpittau|afk | 17:00 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Merge jobs https://review.opendev.org/726121 | 17:01 |
*** derekh has quit IRC | 17:02 | |
*** dtantsur is now known as dtantsur|afk | 17:10 | |
dtantsur|afk | o/ | 17:10 |
*** dsneddon has quit IRC | 17:19 | |
*** dsneddon has joined #openstack-ironic | 17:21 | |
arne_wiebalck | and the field team's experience grows over time which helps I guess | 17:25 |
arne_wiebalck | (wrong window, sorry) | 17:26 |
arne_wiebalck | bye everyone o/ | 17:27 |
openstackgerrit | Merged openstack/ironic master: Fix Redfish handle no continuous override boot src https://review.opendev.org/731644 | 19:05 |
openstackgerrit | Merged openstack/ironic master: Fix: review from dtantsur of 728123 https://review.opendev.org/735276 | 19:06 |
openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Set tempest_test_timeout for managed job https://review.opendev.org/730515 | 19:06 |
*** gmann is now known as gmann_afk | 19:22 | |
*** k_mouza has quit IRC | 19:24 | |
*** diurnalist has quit IRC | 19:36 | |
*** gmann_afk is now known as gmann | 19:37 | |
*** diurnalist has joined #openstack-ironic | 19:41 | |
*** uzumaki has quit IRC | 19:41 | |
*** gmann is now known as gmann_afk | 20:10 | |
openstackgerrit | Merged openstack/ironic-python-agent master: Remove .travis.yml (wow, it is ancient) https://review.opendev.org/736697 | 20:34 |
*** priteau has quit IRC | 20:34 | |
*** shadower has quit IRC | 20:57 | |
openstackgerrit | Richard G. Pioso proposed openstack/ironic stable/ussuri: Fix Redfish handle no continuous override boot src https://review.opendev.org/736839 | 21:01 |
openstackgerrit | Richard G. Pioso proposed openstack/ironic stable/ussuri: Fix Redfish handle no continuous override boot src https://review.opendev.org/736839 | 21:11 |
*** k_mouza has joined #openstack-ironic | 21:24 | |
*** k_mouza has quit IRC | 21:30 | |
TheJulia | bfournie: looks like we need to get https://review.opendev.org/#/c/731676/ backported to train | 21:41 |
patchbot | patch 731676 - ironic (stable/ussuri) - New configuration parameter to use ipmitool retries (MERGED) - 1 patch set | 21:41 |
*** rcernin_ has joined #openstack-ironic | 21:54 | |
*** k_mouza has joined #openstack-ironic | 21:58 | |
*** k_mouza has quit IRC | 21:59 | |
*** gmann_afk is now known as gmann | 22:04 | |
*** tonyb has joined #openstack-ironic | 22:05 | |
clarkb | TheJulia: did meetpad work a bit better this time around than during ptg? | 22:07 |
clarkb | I think what we've discovered is that there are scaling limits on the server side (which we've alrgely addressed for current use) but also on the client side where the client gets bogged down based on the amount of video input it is receiving | 22:07 |
clarkb | smaller groups means less video and clients are happier but as the conf room grows it gets worse | 22:08 |
TheJulia | clarkb: didn't really notice any issues, although we also encouraged our one firefox user to change browsers | 22:08 |
*** rcernin_ has quit IRC | 22:13 | |
openstackgerrit | Steve Baker proposed openstack/ironic master: ironic-standalone, use http basic auth for json-rpc https://review.opendev.org/736398 | 22:42 |
*** tkajinam has joined #openstack-ironic | 23:00 | |
*** rcernin_ has joined #openstack-ironic | 23:15 | |
*** rcernin_ has quit IRC | 23:15 | |
*** rcernin has joined #openstack-ironic | 23:16 | |
*** diurnalist has quit IRC | 23:55 | |
*** rloo has quit IRC | 23:55 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!