Thursday, 2020-06-18

*** diurnalist has joined #openstack-ironic00:01
*** Goneri has quit IRC00:22
*** k_mouza has joined #openstack-ironic00:30
*** k_mouza has quit IRC00:35
*** rloo has quit IRC01:10
*** bfournie has joined #openstack-ironic01:50
*** markmcclain has quit IRC02:01
openstackgerritSteve Baker proposed openstack/ironic master: devstack json-rpc, use http basic auth  https://review.opendev.org/73639802:02
*** markmcclain has joined #openstack-ironic02:02
*** dking has quit IRC02:13
*** rcernin has quit IRC02:48
*** hoonetorg has quit IRC02:51
*** Lucas_Gray has quit IRC02:55
*** rcernin has joined #openstack-ironic02:59
*** rcernin has quit IRC03:05
*** Qianbiao has joined #openstack-ironic03:08
*** mkrai has joined #openstack-ironic03:17
*** tzumainn has quit IRC03:18
*** rcernin has joined #openstack-ironic03:20
*** Qianbiao has quit IRC03:20
*** Qianbiao has joined #openstack-ironic03:21
*** rcernin has quit IRC03:22
*** rcernin has joined #openstack-ironic03:22
*** ravsingh has joined #openstack-ironic03:30
*** hoonetorg has joined #openstack-ironic04:35
*** ociuhandu has joined #openstack-ironic05:21
*** ociuhandu has quit IRC05:26
openstackgerritShivanand Tendulker proposed openstack/ironic master: Adds boot mode support to iLO management interface  https://review.opendev.org/73623205:30
arne_wiebalckGood morning, ironic!05:38
*** mkrai has quit IRC05:59
*** uzumaki has quit IRC06:02
*** mkrai has joined #openstack-ironic06:04
*** uzumaki has joined #openstack-ironic06:07
openstackgerritPratik proposed openstack/bifrost master: optimize size and time using "--no-cache-dir"  https://review.opendev.org/73644706:22
*** yolanda has quit IRC06:31
*** Habib-A has joined #openstack-ironic06:31
Habib-AGood morning #ironic06:31
*** yolanda has joined #openstack-ironic06:36
*** rpittau|afk is now known as rpittau06:43
rpittaugood morning ironic! o/06:43
Habib-AAgent 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-Aerase_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-Awhat should i do ?!06:49
Habib-Asince i switched my 1TB HDDs with 146gb ones and re run the ironic provide command which yielded this06:50
*** uzumaki has quit IRC06:56
rpittauHabib-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-Athey are attached to a raid controller and i disabled ata erase as it caused errors i was hoping shred would do the job instead07:03
Habib-Afor the conducter yes07:04
Habib-Ai will send them as soon as i can get them from the server07:04
gudrutis2morning o/07:05
*** Qianbiao has quit IRC07:16
*** Qianbiao has joined #openstack-ironic07:17
iurygregorygood morning ironic07:26
*** mkrai has quit IRC07:32
*** uzumaki has joined #openstack-ironic07:33
*** dougsz has joined #openstack-ironic07:39
openstackgerritRiccardo Pittau proposed openstack/ironic master: Enforce autospec in test_base_manager  https://review.opendev.org/73481107:45
openstackgerritRiccardo Pittau proposed openstack/ironic master: Enforce autospec in test_neutron  https://review.opendev.org/73591507:45
*** Qianbiao has quit IRC07:46
*** Qianbiao has joined #openstack-ironic07:47
*** Qianbiao_ has joined #openstack-ironic07:48
Qianbiao_hello ironic.07:50
Qianbiao_good morning folks07:50
iurygregoryhi Qianbiao_07:51
*** Qianbiao has quit IRC07:51
*** rcernin_ has joined #openstack-ironic07:54
*** rcernin has quit IRC07:54
*** Qianbiao_ has quit IRC07:55
iurygregoryoh nice driver-libs are failing in stein .-.08:00
openstackgerritPratik proposed openstack/virtualbmc master: optimize size and time using "--no-cache-dir"  https://review.opendev.org/73649008:01
*** diurnalist has quit IRC08:03
*** lucasagomes has joined #openstack-ironic08:08
*** Lucas_Gray has joined #openstack-ironic08:12
uzumakiiurygregory, we might need to open the "stein's gate" to fix those libs xD08:17
iurygregoryuzumaki, yes08:18
iurygregory=d08:18
iurygregory=D*08:18
*** ravsingh has quit IRC08:20
*** rcernin_ has quit IRC08:20
*** dtantsur|afk is now known as dtantsur08:29
*** ravsingh has joined #openstack-ironic08:33
mgoddardHas anyone seen an error like this with virtualbmc? http://paste.openstack.org/show/794922/08:38
mgoddardFails on ipmitool set bootdev pxe in kayobe CI. Works before upgrade, fails afterwards08:39
dtantsurmgoddard: anything in vbmc logs?08:40
mgoddarddtantsur: nope08:41
mgoddardseems like it might have started after we switched from ussuri to master08:41
dtantsurI don't think we've changed anything in vbmc since ussuri.. maybe the pyghmi version?08:42
dtantsuror ipmitool itself?08:42
*** ociuhandu has joined #openstack-ironic08:42
mgoddardNot seeing anything in commit logs08:44
iurygregorywe forgot to pin ibmcclient in stable/stein and we are ussing 0.2.3 omg08:44
*** rcernin_ has joined #openstack-ironic08:45
dtantsursigh08:48
iurygregorydtantsur, 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
patchbotpatch 736017 - ironic (stable/stein) - devstack: Use uwsgi binary from path; temporary di... - 1 patch set08:49
*** rcernin_ has quit IRC08:50
dtantsuriurygregory: just squash them, mentioning it in the commit message08:50
iurygregorydtantsur, ack08:50
dtantsurcan I get some reviews on https://review.opendev.org/#/c/733559/ please?09:08
patchbotpatch 733559 - bifrost - Quick start Bifrost CLI - 23 patch sets09:08
dtantsurit actually works, I've been using it for my development environment09:08
*** jawad_axd has joined #openstack-ironic09:09
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Use configure_keystone_authtoken_middleware  https://review.opendev.org/73651009:10
rpittaudtantsur: I have that on top of my list, just wanted to do some tests, I'll give it a look today09:13
Habib-Arpittau attached the ironic agent log https://pastebin.com/8Eu5h8EH09:21
Habib-A@rpittau https://pastebin.com/8Eu5h8EH09:21
Habib-Aany help is appreciated https://pastebin.com/8Eu5h8EH09:21
openstackgerritMerged openstack/ironic master: Fix mock callable for Python 3.6 and precedent  https://review.opendev.org/73613109:22
openstackgerritMerged openstack/ironic master: Enforce autospec in test_rpc_api and test_steps  https://review.opendev.org/73413209:22
openstackgerritMerged openstack/ironic master: Enforce autospec in test_base_manager  https://review.opendev.org/73481109:22
*** yaawang_ has quit IRC09:24
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic stable/stein: devstack: Use uwsgi binary from path; temporary disable grenade; pin ibmc  https://review.opendev.org/73601709:26
*** mkrai has joined #openstack-ironic09:28
*** yaawang_ has joined #openstack-ironic09:41
openstackgerritMerged openstack/ironic master: Enforce autospec in test_neutron  https://review.opendev.org/73591509:44
*** Habib-A has quit IRC09:47
*** Qianbiao has joined #openstack-ironic09:50
*** ociuhandu has quit IRC09:52
dtantsurwould also appreciate some reviews on https://review.opendev.org/#/c/736191/09:54
patchbotpatch 736191 - ironic-python-agent - Make get_partition_uuids work with whole disk images - 1 patch set09:54
*** Qianbiao has quit IRC09:54
*** Qianbiao has joined #openstack-ironic09:54
*** ociuhandu has joined #openstack-ironic09:59
*** diurnalist has joined #openstack-ironic10:00
openstackgerritMark Goddard proposed openstack/tenks master: Install python setuptools & virtualenv  https://review.opendev.org/73624910:01
*** tkajinam has quit IRC10:05
*** rpittau is now known as rpittau|bbl10:06
*** k_mouza has joined #openstack-ironic10:17
openstackgerritMerged openstack/tenks stable/1.0: CI: multiple fixes  https://review.opendev.org/73591710:21
*** ravsingh has quit IRC10:32
*** Qianbiao has quit IRC10:34
*** rcernin_ has joined #openstack-ironic10:47
*** Habib-A has joined #openstack-ironic11:14
Habib-ATheJulia https://pastebin.com/8Eu5h8EH help please11:15
mgoddarddtantsur: catching up on PTG discussions. Who was interested in Standalone improvements?11:19
mgoddarddtantsur: 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/bikolla11:20
mgoddardI expect it needs updating11:20
mgoddardwins over bifrost - you don't have to maintain it, it supports multinode & HA, it uses containers11:22
mgoddardalso you can add as many or few additional openstack services as you like11:22
Habib-A2020-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_device11:24
Habib-Awhat should i do11:24
dtantsurHabib-A: shred: /dev/sdb: error writing at offset 0: Remote I/O error11:27
dtantsurthis is the actual failure, it seems a problem with your hardware11:27
*** derekh has joined #openstack-ironic11:27
dtantsurmgoddard: I was interested :) Honestly, rather than having an alternative to bifrost, I'd prefer converging with bifrost11:28
openstackgerritVladyslav Drok proposed openstack/ironic master: Stop allocating double space for raw images  https://review.opendev.org/73639111:28
mgoddarddtantsur: how do you mean converging?11:28
dtantsurmgoddard: making bifrost support this mode of operation maybe?11:28
Habib-Adtantsur TheJulia  i tried the centos-ipa-train image and worked !!11:28
mgoddarddtantsur: bifrost as a python CLI wrapper?11:29
dtantsurpython and ansible, yeah11:29
Habib-Ai guess the tinycore ipa image has something missing/different :/11:29
dtantsurHabib-A: note that tinyIPA images are for CI/local development only. They don't necessary support real hardware.11:29
dtantsurmgoddard: or rather: two modes of operation in bifrost: containers via kollan-ansible, source code in a venv11:30
mgoddarddtantsur: I would like that. There are still useful parts of bifrost, but it could delegate deployment to kolla11:30
dtantsurdunno how much of maintenance burden it is though11:30
mgoddardI'd probably say just pick a path11:31
dtantsurwell, I definitely see value in a source-based installation (esp. to replace devstack)11:31
*** mkrai has quit IRC11:33
*** Qianbiao has joined #openstack-ironic11:35
*** ociuhandu has quit IRC11:50
openstackgerritMerged openstack/ironic master: Block port deletions where vif is present  https://review.opendev.org/66583511:50
openstackgerritMerged openstack/ironic stable/train: devstack: Use uwsgi binary from path; temporary disable grenade  https://review.opendev.org/73601511:50
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic stable/ussuri: Enable grenade  https://review.opendev.org/73666411:54
Habib-Adtantsur 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 :D12:00
*** k_mouza has quit IRC12:10
openstackgerritPratik Raj proposed openstack/ironic master: optimize size and time using --no-cache-dir  https://review.opendev.org/73668812:13
openstackgerritPratik Raj proposed openstack/ironic-inspector master: optimize size and time using --no-cache-dir  https://review.opendev.org/73668912:13
*** Qianbiao has quit IRC12:14
*** k_mouza has joined #openstack-ironic12:14
openstackgerritPratik Raj proposed openstack/ironic-lib master: optimize size and time using --no-cache-dir  https://review.opendev.org/73669012:14
openstackgerritPratik Raj proposed openstack/ironic-prometheus-exporter master: optimize size and time using --no-cache-dir  https://review.opendev.org/73669112:15
openstackgerritPratik Raj proposed openstack/ironic-python-agent master: optimize size and time using --no-cache-dir  https://review.opendev.org/73669212:15
dtantsurhow can we stop this ^^^?12:15
openstackgerritPratik Raj proposed openstack/ironic-python-agent-builder master: optimize size and time using --no-cache-dir  https://review.opendev.org/73669312:16
openstackgerritPratik Raj proposed openstack/ironic-tempest-plugin master: optimize size and time using --no-cache-dir  https://review.opendev.org/73669412:16
* dtantsur -2 everything and considers reaching out to the contributor12:17
openstackgerritPratik Raj proposed openstack/ironic-ui master: optimize size and time using --no-cache-dir  https://review.opendev.org/73669512:17
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Remove .travis.yml (wow, it is ancient)  https://review.opendev.org/73669712:19
rpittau|bbldtantsur: going to auto -2 them12:20
dtantsuryep12:20
*** rpittau|bbl is now known as rpittau12:20
iurygregoryI will help12:20
iurygregory=012: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
iurygregoryhttp://lists.openstack.org/pipermail/openstack-discuss/2020-June/015515.html12:21
rpittaucan I just put "me no like it" ?12:21
iurygregorywe can also use the discussion from the list12:21
dtantsuriurygregory: an update: AJaeger asked us to abandon them to free up the CI resources12:21
dtantsurcan someone please do it (even for patches I already -2ed)?12:22
rpittaueven better12:22
iurygregoryyeah12:22
rpittauman, I'm too slow :D12:25
iurygregorylol12:25
rpittautime to refresh gerrit and they were all abandoned already :D12:26
* iurygregory hides12:27
iurygregorydtantsur, do you have any ideas on https://review.opendev.org/#/c/736170/ or keep trying recheck? .-.12:28
patchbotpatch 736170 - metalsmith (stable/train) - Use is instead of ambiguous pipe - 1 patch set12:28
* rpittau goes back to beat libvirt12:28
*** rh-jelabarre has joined #openstack-ironic12:28
* iurygregory goes test bifrost cli on f3012:30
dtantsuriurygregory: I haven't checked what the problem is yet. any hints?12:33
*** derekh has quit IRC12:33
iurygregorydtantsur, it fails during Wait for provisioning of instances12:35
iurygregorybut  it shows some infor about the node that seems correct12:36
rpittauiurygregory: have you checked the conductor logs? I see an error that is also in the ipa-ramdisk12:37
iurygregoryrpittau, haven't looked at conductor yet was looking at the execution logs to get an idea first12:38
iurygregoryFailed to install a bootloader doesn't look good12:38
rpittauiurygregory: UnicodeEncodeError: 'utf-8' codec can't encode characters in position 0-1: surrogates not allowed12:38
rpittauin 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.log12:38
dtantsurOo12:39
iurygregorywtf12:39
rpittauyeah12:39
rpittauseems oslo related12:39
dtantsurwhy are we even using python 3 on train?12:39
rpittauheh12:40
iurygregorybecause we are a bit crazy? :D12:40
*** derekh has joined #openstack-ironic12:40
TheJuliadtantsur: uhh.. because we were going to python3 the next cycle and TC was pushing for there to be python3 CI jobs during train12:42
TheJuliaGood morning everyone12:42
iurygregorygood morning TheJulia12:43
rpittaugood morning TheJulia :)12:43
dtantsurmorning TheJulia12:44
dtantsurTheJulia: in a desperate attempt to recover my test environment I've restarted dnsmasq and libvirtd, and now PXE does not work any more :D12:45
TheJuliadtantsur: oh... are there multiple dnsmasqs?12:45
dtantsurTheJulia: there are ones from libvirt, but tcpdump only shows responses from the correct one12:46
dtantsurwhich seem to have all the necessary information12:46
TheJuliadid libvirt change the VMs?12:47
*** jawad_axd has quit IRC12:47
dtantsurthat is possible12:48
dtantsurmeaning, a bifrost test environment won't survive a reboot :(12:48
dtantsurTheJulia: speaking of which, do I have any chances of luring you into reviewing https://review.opendev.org/#/c/733559/ today?12:49
TheJuliathat seems wrong, but I'e also long had a habit of wiping out my test vms12:49
patchbotpatch 733559 - bifrost - Quick start Bifrost CLI - 23 patch sets12:49
TheJuliaI can give it a try... my calendar for the nexxt few hours looks like a nightmare12:49
* TheJulia coffees12:50
iurygregorydtantsur, I'm testing locally on F30 the bifrost cli12:51
dtantsurcool, the more eyes the better12:52
iurygregoryI 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
dtantsuryeah, amazing12:55
* iurygregory needs a good chair =(12:55
*** Qianbiao has joined #openstack-ironic12:57
TheJulia:(13:01
iurygregorythe best part is: write an email in english for a czech company \o/13:02
*** rloo has joined #openstack-ironic13:04
openstackgerritMark Goddard proposed openstack/tenks master: Trust receive filters of baremetal nodes  https://review.opendev.org/73670813:05
dtantsuriurygregory: been there, done that :)13:06
iurygregorydtantsur, hehehe13:07
rpittauok, 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
rpittauseems like there's no iscsi connection detected13:10
rpittauthis 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.txt13:11
rpittauany ideas?13:12
TheJuliaHmm, I guess I got completely distracted and never managed to send an email regarding the review jam13:12
iurygregoryTheJulia, you sent13:14
iurygregory[ironic] code review jam - Thursday @ 4 PM UTC13:14
iurygregoryhttp://lists.openstack.org/pipermail/openstack-discuss/2020-June/015455.html13:14
dtantsuryep, I've seen it too13:19
dtantsurI'll have to stop by at the release team meeting though13:19
TheJuliasweet13:21
TheJuliaokay13:21
*** sshnaidm is now known as sshnaidm|mtg13:23
*** Habib-A has quit IRC13:28
uzumakiiurygregory, 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
iurygregoryuzumaki, damm =( I can imagine13:33
uzumakiI had a really comfortable chair in the office, working at home just doesn't work for me so well..13:34
iurygregorydtantsur, first try with cli failed because I forgot to start libvirt (facepalm)13:34
dtantsurheh13:35
dtantsuriurygregory: we should probably fix it in bifrost, but that's not the cli problem13:35
uzumakihas anyone seen the new Github interface? It's kinda cool13:35
iurygregorydtantsur, yeah13:35
dtantsurTheJulia: unfortunately, I don't think https://review.opendev.org/#/c/734709/ will work, see inline13:35
patchbotpatch 734709 - ironic-python-agent - Add timeout operations to try and prevent hang on ... - 5 patch sets13:35
dtantsurand 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 code13:37
dtantsurI don't entirely disagree with that13:37
dtantsurunrelated, I've talked to blivet people re publishing to pypi and they seem open to the idea: https://github.com/storaged-project/blivet/issues/85813:43
TheJuliaseriously?!?13:43
dtantsurseriously re what: requests or blivet? :)13:43
TheJuliadtantsur: 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 fine13:44
dtantsurmm, well, I don't know what to say. all I can find explicitly says it shouldn't work :(13:44
dtantsurif we're absolutely, totally sure it works, I'm fine with letting it in13:45
dtantsurthe change is straightforward and shouldn't cause regressions, so whatever13:45
dtantsurbut I don't understand how it can possibly work, given everything I've found13:45
TheJuliadtantsur: that bug is different13:46
dtantsurthe 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-ironic13:47
TheJuliaI _think_ the only way it works is if it yeilds none13:47
dtantsurdo you have any references to it ever yielding none? I couldn't find that either..13:48
TheJuliaI 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 201813:48
* dtantsur may need to dive into urllib313:48
TheJuliaI'd have to dig through my browser history at this point13:48
*** tzumainn has joined #openstack-ironic13:48
TheJuliadtantsur: possibly. We're definitely in the area of super not fun socket programming behaviors13:48
dtantsurlovely. okay, lemme do my own research before I +2.13:50
iurygregorydtantsur, it works \o/13:51
dtantsuryay!13:51
dtantsurokay, on one code path it seems impossible: https://github.com/urllib3/urllib3/blob/master/src/urllib3/response.py#L52013:51
dtantsuron the other too: https://github.com/urllib3/urllib3/blob/master/src/urllib3/response.py#L574-L57813:52
dtantsurnow the last path13:52
openstackgerritRiccardo Pittau proposed openstack/ironic master: Use configure_keystone_authtoken_middleware  https://review.opendev.org/73651013:52
TheJuliahmm, found a requests bug from 2014 where lukasa points at using the time module with iter_content to detect cases13:52
TheJuliabut not the one I was looking for13:53
iurygregorydtantsur, the only thing we are missing is documentation right?13:53
dtantsurthe third too https://github.com/urllib3/urllib3/blob/master/src/urllib3/response.py#L770-L77113:53
dtantsuriurygregory: yeah, will write it a bit later13:53
dtantsurprobably more options later on, but not for the MVP13:54
iurygregorydtantsur, no worries tks!13:54
dtantsurTheJulia: urllib3 seems serious about their promise not to return anything empty..13:54
TheJulia:(13:55
dtantsurunrelated, but WTF https://github.com/urllib3/urllib3/pull/166913:55
*** Goneri has quit IRC13:55
rpittauok.....13:56
iurygregoryFix for python413:57
iurygregorywow13:57
TheJuliadtantsur: 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
dtantsuryeah, it's super annoying13:58
dtantsurconnection interrupted in the middle is not some exotic case13:58
dtantsurthese people have clearly forgotten the deal-up times :(13:58
TheJuliaheh14:01
TheJuliathere are literally so many different threads to the discussion, I've not yet found the same one14:08
rpiosoGood morning, ironic14:08
*** Goneri has joined #openstack-ironic14:08
rpiosorpittau: Thank you for the reviews of https://review.opendev.org/#/c/73164414:09
patchbotpatch 731644 - ironic - Fix Redfish handle no continuous override boot src - 5 patch sets14:09
rpittauhey rpioso np :)14:09
rpiosorpittau: :-) Looks like the Zuul checks liked it, too, or at least it did 10 hours ago.14:10
rpittauyep, that looks fine from CI perspective :)14:11
jamesdentonQQ 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
rpiosoWould appreciate another core review to move it forward.14:11
*** rcernin_ has quit IRC14:14
TheJuliadtantsur: I really wish I had lukasa on speed dial at the moment14:17
dtantsurheh14:17
openstackgerritMerged openstack/bifrost master: Quick start Bifrost CLI  https://review.opendev.org/73355914:25
dtantsurwooohoo14:27
dtantsurif we can get https://review.opendev.org/#/c/735933/ in too, I'll no longer need to manually patch my bifrost environment :)14:28
patchbotpatch 735933 - bifrost - Fix fast-track deployment after discovery/inspection - 1 patch set14:28
openstackgerritDmitry Tantsur proposed openstack/bifrost master: CI: clean up non-votings jobs  https://review.opendev.org/73558114:29
openstackgerritRiccardo Pittau proposed openstack/ironic master: Fix uefi jobs with native ubuntu ovmf package  https://review.opendev.org/71688914:32
*** Goneri has quit IRC14:35
rpittauI just realized I updated this ^ patch and we don't have ipa-ramdisk logs............ :(14:35
* TheJulia looks at requests14:36
TheJuliahmm14:36
dtantsurjamesdenton: IIRC (and my memory on this topic is incomplete) the mode is used by ML2 drivers14:39
jamesdentondtantsur 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 sure14:43
TheJuliaeww, entirely different behavior between chunked encoding and non-chunked14:46
*** ociuhandu has joined #openstack-ironic14:50
*** Goneri has joined #openstack-ironic14:51
dtantsuryeah, different code paths14:54
dtantsurbut both seem to skip empty chunks14:54
TheJuliaI thought you could yeild without a data, but I'm not seeing anything yet, however I'm just scratching the surface and crying14:55
*** ociuhandu has quit IRC14:55
dtantsur:(14:58
*** portdirect has joined #openstack-ironic14:59
portdirecthey! im trying to get a ci env up for metal3 atm, and have a hit an issue with sushi-tools15:00
portdirectis it possible to use the Libvirt domain name, rather than the domain uuid here?15:00
*** dking has joined #openstack-ironic15:00
portdirectthe reason for asking is that id like to have a reasonably dynamic environment - ie create x hosts etc for work we are doing in airship15:02
portdirectand with uuids this is rather hard to drive, I suppose we could deterministically create uuids - but it seems like a really ugly hack15:02
Qianbiao<portdirect> metal3 has a node name15:03
Qianbiao* or named bmh node name15:03
Qianbiaothat is the same as ironic node name15:03
Qianbiaoyou can use that instead of uuid15:03
portdirectQianbiao: what im looking for here comes earlier than that15:06
portdirecteg:15:06
portdirecthttps://www.irccloud.com/pastebin/nDHhI6jS/15:06
portdirectideally id like to be able to have output like:15:06
portdirecthttps://www.irccloud.com/pastebin/0fsvXdcd/15:07
*** diurnalist has quit IRC15:08
Qianbiaosorry not familiar with libvirt redfish vbmc.15:08
QianbiaoI am testing metal3 with real baremetal15:08
*** cdearborn has joined #openstack-ironic15:11
*** diurnalist has joined #openstack-ironic15:16
dtantsurportdirect: I remember we switched to using UUIDs for some reason at some point, I don't remember why15:17
iurygregoryname is not unique probably...15:25
portdirectdtantsur: I think in many cases that makes sense15:26
portdirectbut it also makes things rather hard as well :)15:26
portdirectI see there is actually a redirect so if you hit the domain name, it ends up in the right place15:26
portdirecthttps://www.irccloud.com/pastebin/3I4eq0Fr/15:26
portdirectI wonder, could we make the default [uuid|name] toggleable to support this?15:27
dtantsurI would not mind it, patches are welcome15:28
*** Lucas_Gray has quit IRC15:29
*** diurnalist has quit IRC15:34
*** diurnalist has joined #openstack-ironic15:40
*** ociuhandu has joined #openstack-ironic15:40
*** sshnaidm|mtg is now known as sshnaidm|ruck15:41
*** gyee has joined #openstack-ironic15:50
*** priteau has joined #openstack-ironic15:52
*** Qianbiao has quit IRC15:53
*** diurnalist has quit IRC15:58
TheJuliashall we jam through some reviews?16:01
dtantsurwhat's the link?16:01
TheJuliameetpad.opendev.org/ironic16:01
iurygregory1min16:02
iurygregorygoing to grab some water16:02
*** lucasagomes has quit IRC16:03
TheJuliajoin us!16:05
openstackgerritMerged openstack/ironic master: CI: make grenade voting again  https://review.opendev.org/73557716:08
iurygregory\o/16:08
openstackgerritMerged openstack/bifrost master: CI: clean up non-votings jobs  https://review.opendev.org/73558116:28
*** dougsz has quit IRC16:31
*** diurnalist has joined #openstack-ironic16:35
*** sshnaidm|ruck is now known as sshnaidm|off16:52
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Remove .travis.yml (wow, it is ancient)  https://review.opendev.org/73669716:58
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Merge jobs  https://review.opendev.org/72612117:00
rpittaubye everyone! o/17:00
*** rpittau is now known as rpittau|afk17:00
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Merge jobs  https://review.opendev.org/72612117:01
*** derekh has quit IRC17:02
*** dtantsur is now known as dtantsur|afk17:10
dtantsur|afko/17:10
*** dsneddon has quit IRC17:19
*** dsneddon has joined #openstack-ironic17:21
arne_wiebalckand the field team's experience grows over time which helps I guess17:25
arne_wiebalck(wrong window, sorry)17:26
arne_wiebalckbye everyone o/17:27
openstackgerritMerged openstack/ironic master: Fix Redfish handle no continuous override boot src  https://review.opendev.org/73164419:05
openstackgerritMerged openstack/ironic master: Fix: review from dtantsur of 728123  https://review.opendev.org/73527619:06
openstackgerritVerification of a change to openstack/ironic-inspector failed: Set tempest_test_timeout for managed job  https://review.opendev.org/73051519:06
*** gmann is now known as gmann_afk19:22
*** k_mouza has quit IRC19:24
*** diurnalist has quit IRC19:36
*** gmann_afk is now known as gmann19:37
*** diurnalist has joined #openstack-ironic19:41
*** uzumaki has quit IRC19:41
*** gmann is now known as gmann_afk20:10
openstackgerritMerged openstack/ironic-python-agent master: Remove .travis.yml (wow, it is ancient)  https://review.opendev.org/73669720:34
*** priteau has quit IRC20:34
*** shadower has quit IRC20:57
openstackgerritRichard G. Pioso proposed openstack/ironic stable/ussuri: Fix Redfish handle no continuous override boot src  https://review.opendev.org/73683921:01
openstackgerritRichard G. Pioso proposed openstack/ironic stable/ussuri: Fix Redfish handle no continuous override boot src  https://review.opendev.org/73683921:11
*** k_mouza has joined #openstack-ironic21:24
*** k_mouza has quit IRC21:30
TheJuliabfournie: looks like we need to get https://review.opendev.org/#/c/731676/ backported to train21:41
patchbotpatch 731676 - ironic (stable/ussuri) - New configuration parameter to use ipmitool retries (MERGED) - 1 patch set21:41
*** rcernin_ has joined #openstack-ironic21:54
*** k_mouza has joined #openstack-ironic21:58
*** k_mouza has quit IRC21:59
*** gmann_afk is now known as gmann22:04
*** tonyb has joined #openstack-ironic22:05
clarkbTheJulia: did meetpad work a bit better this time around than during ptg?22:07
clarkbI 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 receiving22:07
clarkbsmaller groups means less video and clients are happier but as the conf room grows it gets worse22:08
TheJuliaclarkb: didn't really notice any issues, although we also encouraged our one firefox user to change browsers22:08
*** rcernin_ has quit IRC22:13
openstackgerritSteve Baker proposed openstack/ironic master: ironic-standalone, use http basic auth for json-rpc  https://review.opendev.org/73639822:42
*** tkajinam has joined #openstack-ironic23:00
*** rcernin_ has joined #openstack-ironic23:15
*** rcernin_ has quit IRC23:15
*** rcernin has joined #openstack-ironic23:16
*** diurnalist has quit IRC23:55
*** rloo has quit IRC23:55

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!