janders | good morning Ironic o/ | 00:02 |
---|---|---|
janders | TheJulia ftarasenko rpioso ajya|afk I spotted the discussion on SSD secure erase in the logs and it's a topic I'm interested in so wanted to share some thoughts/questions. Is there any way in which we can use SSD trim/discard in cleaning? I think this would be the right tool for the job both in terms of speed and maximising SSD lifetime. Do Dell | 00:24 |
janders | controllers support this? If so, can we leverage this in ironic via supported BMCs? | 00:24 |
rpioso | janders: iDRAC supports secure erase of SSDs. Please see the description of the WS-Man DCIM_RAIDService.SecureErase method in https://downloads.dell.com/solutions/general-solution-resources/White%20Papers/Dell_SimpleRAID_Profile.pdf. It is in section 8.30. DCIM_RAIDService.SecureErase (), starting on p. 73. Similar is offered via Redfish, too -- | 00:38 |
rpioso | https://topics-cdn.dell.com/pdf/idrac9-lifecycle-controller-v4x-series_api-guide_en-us.pdf. | 00:38 |
rpioso | ajya|afk, ftarasenko, janders, TheJulia: Are you interested in discussing secure erase during the PTG? | 01:01 |
janders | rpioso thank you for the links, will have a look through | 01:35 |
janders | rpioso regarding PTG - I am! :) | 01:35 |
janders | rpioso reading through the document I see "8.30. DCIM_RAIDService.SecureErase () The Secure Erase operation or full initialization on a PCIe SSD overwrites all blocks in an encrypted way and all | 01:38 |
janders | the data on the PCIe SSD device will be permanently lost." | 01:38 |
janders | is it possible to secure-erase the data by trim/discard as opposed to overwrite ( https://en.wikipedia.org/wiki/Trim_(computing) )? | 01:38 |
janders | idea = maximising SSD lifetime | 01:38 |
rpioso | janders: yw :-) | 01:39 |
rpioso | janders: Dunno. I would have to check. Got it. | 01:39 |
janders | rpioso thanks again, that would be awesome :) | 01:40 |
janders | (and would go very nicely with fastrack deploys, kexec, etc - we could cut deployment/cleaning overheads very very slim) | 01:41 |
janders | if this is useful at some point, maybe it's possible test this out using an SSD running in pass-thru mode and this https://man7.org/linux/man-pages/man8/blkdiscard.8.html | 01:43 |
*** zzzeek has quit IRC | 02:20 | |
*** zzzeek has joined #openstack-ironic | 02:22 | |
*** rh-jelabarre has joined #openstack-ironic | 02:33 | |
openstackgerrit | wu.shiming proposed openstack/ironic master: Remove install unnecessary packages https://review.opendev.org/751636 | 03:10 |
*** Qianbiao has joined #openstack-ironic | 03:17 | |
*** mkrai has joined #openstack-ironic | 03:23 | |
*** rh-jelabarre has quit IRC | 03:25 | |
*** zzzeek has quit IRC | 03:50 | |
*** zzzeek has joined #openstack-ironic | 03:51 | |
*** uzumaki has joined #openstack-ironic | 04:49 | |
*** mkrai has quit IRC | 04:59 | |
*** mkrai has joined #openstack-ironic | 05:00 | |
*** Qianbiao has quit IRC | 05:15 | |
*** abdysn has joined #openstack-ironic | 05:40 | |
*** Qianbiao has joined #openstack-ironic | 05:50 | |
openstackgerrit | wu.shiming proposed openstack/python-ironicclient master: Remove install unnecessary packages https://review.opendev.org/751649 | 06:05 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Add status_code argument to @method.expose https://review.opendev.org/743677 | 06:18 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Add @method.body decorator https://review.opendev.org/743678 | 06:18 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Duplicate trait validation with jsonschema https://review.opendev.org/745778 | 06:18 |
openstackgerrit | Steve Baker proposed openstack/ironic master: New argument validate decorator https://review.opendev.org/742795 | 06:18 |
openstackgerrit | Steve Baker proposed openstack/ironic master: WIP Utility functions for REST API JSON handling https://review.opendev.org/751160 | 06:18 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert allocations endpoint to plain JSON https://review.opendev.org/743679 | 06:18 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert bios endpoint to plain JSON https://review.opendev.org/745475 | 06:18 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert chassis endpoint to plain JSON https://review.opendev.org/745476 | 06:18 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert conductors endpoint to plain JSON https://review.opendev.org/745779 | 06:18 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert deploy_templates endpoint to plain JSON https://review.opendev.org/745780 | 06:18 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert drivers endpoint to plain JSON https://review.opendev.org/741865 | 06:19 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert event endpoint to plain JSON https://review.opendev.org/746623 | 06:19 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert nodes endpoint to plain JSON https://review.opendev.org/747820 | 06:19 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert ports endpoint to plain JSON https://review.opendev.org/750120 | 06:19 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert portgroups endpoint to plain JSON https://review.opendev.org/750864 | 06:19 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert ramdisk endpoint to plain JSON https://review.opendev.org/750857 | 06:19 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume endpoint to plain JSON https://review.opendev.org/750858 | 06:19 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume/connectors endpoint to plain JSON https://review.opendev.org/750859 | 06:19 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume/targets endpoint to plain JSON https://review.opendev.org/750860 | 06:19 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Expunge the internal version of WSME https://review.opendev.org/750861 | 06:19 |
arne_wiebalck | Good morning, ironic! | 06:42 |
janders | good morning arne_wiebalck | 06:43 |
arne_wiebalck | Hey janders o/ | 06:44 |
*** jtomasek has joined #openstack-ironic | 06:58 | |
*** mkrai has quit IRC | 07:16 | |
*** mkrai_ has joined #openstack-ironic | 07:16 | |
iurygregory | good morning arne_wiebalck janders and Ironic! | 07:19 |
arne_wiebalck | hey iurygregory o/ | 07:19 |
ftarasenko | gm, Ironic! janders, rpioso, regarding secure erase via Dell RAID Controller, I think, DCIM_RAIDService.ReKey method can be used. The bad is that security is not supported on 330 series controller, so it's not a solution for all Dell servers. | 07:32 |
ftarasenko | Putting SSD's in pass-through mode seems a solution for my servers, but haven't tested yet. | 07:33 |
*** mkrai_ has quit IRC | 07:35 | |
*** jtomasek has quit IRC | 07:39 | |
*** jtomasek has joined #openstack-ironic | 07:41 | |
uzumaki | good morning ironic! o/ | 07:45 |
iurygregory | morning uzumaki o/ | 07:45 |
uzumaki | how you doing iurygregory ? como estas? | 07:45 |
iurygregory | doing good =) and you? | 07:46 |
uzumaki | estoy bien, gracias =) | 07:46 |
uzumaki | any interesting holidays in CZ? | 07:46 |
iurygregory | I think we have one on 28 Sep | 07:47 |
openstackgerrit | Arne Wiebalck proposed openstack/ironic-inspector master: Add NVIDIA V100 to known accelerator devices https://review.opendev.org/751660 | 07:55 |
*** uzumaki has quit IRC | 07:56 | |
iurygregory | really ironic-inspector was the only one that failed to merge the l-c changes -.-' facepalm | 07:58 |
*** mkrai_ has joined #openstack-ironic | 08:24 | |
openstackgerrit | Merged openstack/ironic master: Follow-up patch for One Button Secure Erase clean step https://review.opendev.org/750624 | 08:27 |
*** Abdallahyas has joined #openstack-ironic | 08:30 | |
*** abdysn has quit IRC | 08:34 | |
*** derekh has joined #openstack-ironic | 08:42 | |
*** lucasagomes has joined #openstack-ironic | 08:55 | |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Use converted ipa jobs https://review.opendev.org/749723 | 08:58 |
Qianbiao | Good morning ironic. | 09:08 |
Qianbiao | Morning arne_wiebalck iurygregory uzumaki | 09:08 |
Qianbiao | Morning everyone | 09:09 |
Qianbiao | Hi, i have an issue while using ironic-inspector to add infiniband ports. | 09:09 |
iurygregory | morning Qianbiao | 09:09 |
Qianbiao | I create the inspect ipa with "disk-image-create ironic-python-agent-ramdisk mellanox centos-minimal devuser -o ipa-centos-8-mlnx" | 09:10 |
Qianbiao | But IB ports is not detected. | 09:10 |
*** sshnaidm|pto is now known as sshnaidm | 09:10 | |
openstackgerrit | Merged openstack/ironic-python-agent master: Fix backup node lookup https://review.opendev.org/750823 | 09:11 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Move ironic-standalone-ipa-src to Ubuntu Focal https://review.opendev.org/748213 | 09:11 |
arne_wiebalck | Hey Qianbiao o/ | 09:14 |
Qianbiao | Hey o/ | 09:14 |
*** tosky has joined #openstack-ironic | 09:31 | |
*** uzumaki has joined #openstack-ironic | 09:50 | |
*** uzumaki has quit IRC | 10:00 | |
*** uzumaki has joined #openstack-ironic | 10:04 | |
*** dtantsur|afk is now known as dtantsur | 10:27 | |
dtantsur | morning ironic | 10:27 |
*** Lucas_Gray has joined #openstack-ironic | 10:30 | |
*** Lucas_Gray has quit IRC | 10:42 | |
Qianbiao | Morning dtantsur | 10:43 |
janders | good morning dtantsur | 10:45 |
*** mkrai_ has quit IRC | 10:47 | |
*** mkrai__ has joined #openstack-ironic | 10:47 | |
*** Lucas_Gray has joined #openstack-ironic | 10:47 | |
*** Lucas_Gray has quit IRC | 10:51 | |
*** Wryhder has joined #openstack-ironic | 10:51 | |
*** Wryhder is now known as Lucas_Gray | 10:51 | |
*** uzumaki has quit IRC | 10:56 | |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Remove unused jobs https://review.opendev.org/749725 | 10:58 |
iurygregory | good morning dtantsur | 11:01 |
*** uzumaki has joined #openstack-ironic | 11:08 | |
*** mkrai__ has quit IRC | 11:37 | |
dtantsur | folks, could I get a 2nd review on https://review.opendev.org/#/c/747136/ please? it's an API/RPC change, we should get it in ASAP. | 11:40 |
patchbot | patch 747136 - ironic - Accept and use a TLS certificate from the agent - 9 patch sets | 11:40 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: tinyipa: use python 3 for building on all systems https://review.opendev.org/751769 | 11:48 |
*** Lucas_Gray has quit IRC | 11:50 | |
*** rh-jelabarre has joined #openstack-ironic | 11:50 | |
*** rh-jelabarre has quit IRC | 11:51 | |
*** rh-jelabarre has joined #openstack-ironic | 11:51 | |
*** mkrai__ has joined #openstack-ironic | 11:52 | |
*** Lucas_Gray has joined #openstack-ironic | 11:53 | |
*** Lucas_Gray has quit IRC | 11:58 | |
*** Abdallahyas has quit IRC | 12:00 | |
*** rh-jelabarre has quit IRC | 12:00 | |
*** rh-jelabarre has joined #openstack-ironic | 12:00 | |
openstackgerrit | Merged openstack/tenks master: Fix ansiblelint 208 https://review.opendev.org/749211 | 12:10 |
*** Goneri has joined #openstack-ironic | 12:11 | |
*** abdysn has joined #openstack-ironic | 12:11 | |
*** priteau has joined #openstack-ironic | 12:15 | |
openstackgerrit | Merged openstack/ironic-inspector master: Fix l-c and mysql GRANT syntex for Ubuntu Focal https://review.opendev.org/751218 | 12:17 |
iurygregory | finally \o/ | 12:30 |
iurygregory | dtantsur, looking now | 12:31 |
janders | see you tomorrow Ironic o/ | 12:34 |
iurygregory | bye janders | 12:39 |
iurygregory | dtantsur, done | 12:41 |
dtantsur | thx | 12:41 |
* iurygregory goes back testing bifrost with ipe | 12:42 | |
*** mkrai__ has quit IRC | 12:50 | |
openstackgerrit | Merged openstack/ironic-python-agent master: Move ironic-standalone-ipa-src to Ubuntu Focal https://review.opendev.org/748213 | 12:52 |
Qianbiao | hi dtantsur does inspect support detect infiniband port? | 12:55 |
dtantsur | Qianbiao: I assume so based on the patches I've seen. I don't have any experience with IB though. | 12:55 |
Qianbiao | mellanox infiniband port. I try to add mellanox element on inspect ipa ramdisk, but not work. | 12:55 |
dtantsur | Qianbiao: you can ask janders tomorrow, he seems to have had some experience | 12:55 |
Qianbiao | ok thanks for the information | 12:56 |
janders | dtantsur Qianbiao it used to work | 12:56 |
janders | (with some catches) | 12:56 |
Qianbiao | janders with solution: "add mellanox element on inspect ipa ramdisk" ? | 12:56 |
janders | yeah that would be one | 12:57 |
Qianbiao | I am try working with mellanox CX5 | 12:57 |
janders | what servers is this on? | 12:57 |
Qianbiao | centos7 | 12:57 |
janders | how about server make/model? | 12:57 |
janders | sometimes there were some hardware-specific catches IIRC | 12:58 |
Qianbiao | i am working with huawei ibmc | 12:58 |
Qianbiao | I am trying make ramdisk by "disk-image-create ironic-python-agent-ramdisk mellanox centos-minimal devuser -o ipa-centos-8-mlnx" | 12:59 |
janders | are you booting off IB as well, or just recording IB details? | 13:00 |
Qianbiao | recording IB guid | 13:01 |
janders | good, this should be simpler | 13:01 |
Qianbiao | I try to get IB guid when inspecting | 13:01 |
janders | do you have another CX5 ethernet port you're booting off? | 13:01 |
Qianbiao | janders we did not dhcp through IB port | 13:02 |
Qianbiao | we have ethernet port for pxe | 13:02 |
Qianbiao | IB port will be used after provision | 13:02 |
janders | on the CX5 or on the motherboard? | 13:02 |
Qianbiao | on CX5 | 13:02 |
Qianbiao | let me take a picture | 13:03 |
janders | are you currently able to PXE-boot over the Ethernet port, just not getting the GUID details? | 13:03 |
Qianbiao | yes | 13:03 |
Qianbiao | http://paste.openstack.org/show/797833/ | 13:03 |
janders | ok! then adding the element should help | 13:04 |
Qianbiao | we are pxe on eno2/eno1. | 13:04 |
Qianbiao | and try to get guid of ibs4f1 | 13:04 |
*** Lucas_Gray has joined #openstack-ironic | 13:04 | |
janders | sometimes getting PXE to work over any port is tricky depending on the hardware, but it sounds like this is not a problem, which is good | 13:04 |
Qianbiao | ok i will try again to make sure nothing wrong operated before. | 13:05 |
janders | is this your internal lab, or is this for a customer of yours? | 13:05 |
Qianbiao | internal lab | 13:05 |
janders | cool! easier to get debug info if need be | 13:05 |
Qianbiao | I am testing on it, and will delivery the solution to our customer | 13:05 |
Qianbiao | This is the start, i need to using the guid to setup partition for IB network in neutron.. | 13:06 |
Qianbiao | need to create a new ml2 driver maybe. | 13:06 |
janders | right! | 13:06 |
janders | let us know how you go with the custom element, hopefully that will work | 13:07 |
janders | I think if it's properly in it should be mentioned in the introspection logs IIRC | 13:07 |
janders | allright! it's getting late here, I will see you later | 13:07 |
janders | good luck Qianbiao | 13:07 |
Qianbiao | Iam using https://github.com/openstack/diskimage-builder/tree/master/diskimage_builder/elements/mellanox | 13:08 |
Qianbiao | thanks, will try. and let you know. | 13:08 |
*** uzumaki has quit IRC | 13:09 | |
*** irclogbot_2 has quit IRC | 13:19 | |
TheJulia | godo morning | 13:24 |
TheJulia | good | 13:24 |
iurygregory | good morning TheJulia =) | 13:24 |
*** irclogbot_0 has joined #openstack-ironic | 13:25 | |
*** rloo has joined #openstack-ironic | 13:26 | |
iurygregory | dtantsur, if I want to add a way to enable the ipe from the bifrost CLI would be something like http://paste.openstack.org/show/797834/ that is needed right? | 13:28 |
*** mnaser has quit IRC | 13:32 | |
*** mnaser has joined #openstack-ironic | 13:32 | |
*** mnaser has quit IRC | 13:32 | |
*** mnaser has joined #openstack-ironic | 13:32 | |
dtantsur | iurygregory: yep, but I'd use --enable-prometheus-exporter | 13:33 |
dtantsur | morning TheJulia, will you believe me if I tell you there is another bug with agent tokens? | 13:34 |
iurygregory | dtantsur, ack =) | 13:34 |
* dtantsur tries to reproduce again | 13:34 | |
dtantsur | yaaaay, power off still does not wipe the token despite me fixing it... | 13:35 |
TheJulia | Sweet! | 13:35 |
TheJulia | So out of band power off while in a workflow? | 13:36 |
TheJulia | Or workflow killing the power? | 13:36 |
dtantsur | power off via CLI | 13:36 |
*** tkajinam has quit IRC | 13:37 | |
*** jtomasek has quit IRC | 13:39 | |
*** tzumainn has joined #openstack-ironic | 13:41 | |
dtantsur | TheJulia: ehmm, we have broken is_agent_token_pregenerated | 13:42 |
dtantsur | that's why it regressed | 13:42 |
dtantsur | no, wait, false alert | 13:42 |
*** cdearborn has joined #openstack-ironic | 13:47 | |
*** ricolin has quit IRC | 13:52 | |
* dtantsur has spotted the issue and is in deep facepalm | 13:57 | |
openstackgerrit | Christopher Dearborn proposed openstack/ironic master: Redfish driver firmware update https://review.opendev.org/749619 | 14:00 |
*** uzumaki has joined #openstack-ironic | 14:01 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Guard conductor from consuming all of the ram https://review.opendev.org/726483 | 14:01 |
openstackgerrit | Merged openstack/tenks master: Use importlib to take place of imp module https://review.opendev.org/748818 | 14:02 |
TheJulia | On the etherpad, https://etherpad.opendev.org/p/IronicWhiteBoard, line 167, is that still relevent? | 14:02 |
bfournie | hi rpioso, re: discussion on this last week, the missing PowerState with FC640 - https://bugzilla.redhat.com/show_bug.cgi?id=1873305, they tried with iDRAC firmware 4.10.10.10 but are seeing the same issue | 14:03 |
openstack | bugzilla.redhat.com bug 1873305 in Bare Metal Hardware Provisioning "Failed to power on /inspect node when using Redfish protocol" [High,New] - Assigned to dtantsur | 14:03 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Also wipe agent token on manual power off or reboot https://review.opendev.org/751813 | 14:03 |
dtantsur | TheJulia: join me in the facepalm ^^^ | 14:03 |
iurygregory | damm if | 14:04 |
TheJulia | sigh | 14:04 |
bfournie | dtantsur: could this be issue with inspection or clean failing and subsequent attempts also fail due to token issues? | 14:05 |
dtantsur | bfournie: not 100% sure | 14:05 |
arne_wiebalck | TheJulia for https://review.opendev.org/726483, I am starting systematic tests with 200 physical nodes (without your patch for now, to get the baseline behaviour, and also to see how scaling out controllers helps) | 14:05 |
patchbot | patch 726483 - ironic - Guard conductor from consuming all of the ram - 15 patch sets | 14:05 |
dtantsur | TheJulia: I also wonder if we should wipe agent_url on power off | 14:06 |
bfournie | dtantsur: k | 14:06 |
dtantsur | since DHCP may deliver us a new IP next time | 14:06 |
rpioso | ftarasenko: There's also DCIM_RAID.RemoveControllerKey(). Its doc states, "CAUTION: All encrypted drives shall be erased." I'm under the impression DCIM_RAID.ReKey() does not erase. At least, that profile doesn't state it does. | 14:06 |
TheJulia | dtantsur: If we have a valid token I think we'll update it now | 14:06 |
rpioso | Good morning, ironicers | 14:06 |
TheJulia | I _think_ | 14:06 |
rpioso | bfournie: Thank you for the update. Drats! | 14:07 |
TheJulia | granted dhcp doing that is a possibility but also means the network config and dhcp is broken | 14:07 |
dtantsur | TheJulia: nope, we outright reject any changes | 14:07 |
TheJulia | hmm | 14:07 |
TheJulia | We likely should then | 14:07 |
TheJulia | because people have broken networks | 14:07 |
dtantsur | okay, updating the patch | 14:07 |
TheJulia | k | 14:07 |
*** sshnaidm is now known as sshnaidm|afk | 14:07 | |
Qianbiao | Hello, is this a ipa compatible issue: Failed looking up node with addresses '00:2e:c7:dc:38:c3,74:5a:aa:63:07:62,74:5a:aa:63:07:60,00:2e:c7:dc:38:c2' at http://192.168.43.144/baremetal, status code: 404 | 14:08 |
Qianbiao | Am i building ramdisk with wrong ipa? | 14:09 |
dtantsur | Qianbiao: ironic-inspector cannot match the MAC addresses with anything it has in its cache | 14:09 |
Qianbiao | Hmm. but 74:5a:aa:63:07:62 this mac can be list in port list command | 14:09 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Also wipe agent token on manual power off or reboot https://review.opendev.org/751813 | 14:10 |
dtantsur | Qianbiao: you need to check ironic-inspector logs for what it's going to use for lookup | 14:10 |
Qianbiao | sure chekcing. | 14:10 |
*** ricolin has joined #openstack-ironic | 14:11 | |
rpioso | arne_wiebalck, rpittau: I'm thinking it's time for a Redfish Interop Profile sync-up. How about late this week or sometime next week? Doodle it? | 14:13 |
Qianbiao | dtantsur It says: Look up error: Could not find a node for attributes {'bmc_address': [], 'mac': []} | 14:14 |
Qianbiao | looks like a compatible issue? | 14:14 |
dtantsur | oh, hmm, interesting, I've confused two lookups | 14:14 |
dtantsur | the first lookup was from ironic, not ironic-inspector and it does use ports | 14:14 |
dtantsur | ironic-inspector fails.. I don't remember if IB ports are used during lookup | 14:14 |
Qianbiao | dtantsur nothing about ib port for now. | 14:15 |
Qianbiao | those mac address are all eth port | 14:15 |
Qianbiao | inspect complians "Failed to load collector {'extra_hardware'}" | 14:16 |
Qianbiao | is this related? | 14:16 |
* rpioso will be right back. Time for a reboot salute :-) | 14:17 | |
TheJulia | rpioso: I think rpittau|afk is on pto. | 14:19 |
*** mkrai has joined #openstack-ironic | 14:19 | |
arne_wiebalck | rpioso: this and next week are both fine with me, should be pretty flexible | 14:20 |
Qianbiao | He said: four weeks IIRC | 14:21 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Minor agent version code cleanup https://review.opendev.org/749552 | 14:22 |
*** kashifkhan has joined #openstack-ironic | 14:22 | |
*** uzumaki has quit IRC | 14:24 | |
Qianbiao | dtantsur I found :: [-] Not updating iptables - no changes in MAC list [u'28:a6:db:a9:92:27', u'84:a9:c4:6f:3f:c5', u'70:c7:f2:69:07:62'] {{(pid=12303) sync /opt/stack/ironic-inspector/ironic_inspector/pxe_filter/iptables.py:120}} | 14:26 |
Qianbiao | the update list does not contains the target mac | 14:26 |
Qianbiao | something wrong? | 14:26 |
*** uzumaki has joined #openstack-ironic | 14:28 | |
Qianbiao | It just success... | 14:30 |
Qianbiao | i changed nothing. | 14:30 |
rpioso | TheJulia: Thank you for informing me rpittau|afk is out. | 14:32 |
rpioso | TheJulia: Do you expect him back next week? | 14:33 |
Qianbiao | rpioso he said 4 weeks IIRC | 14:33 |
bfournie | rpioso: he will be out for 3 more weeks | 14:33 |
TheJulia | dtantsur: I was thinking 2 more weeks, but it could be 3 | 14:34 |
TheJulia | dtantsur: anything from bifrost that you want to add for potential review items? | 14:34 |
rpioso | bfournie, TheJulia, Qianbiao: Thank you! | 14:34 |
rpioso | arne_wiebalck: Looks like rpittau|afk will be out for a while. How about we proceed and sync up with him later? | 14:36 |
rpioso | arne_wiebalck: We could doodle Wednesday-Friday this week and next and see where it falls :-) | 14:37 |
arne_wiebalck | rpioso: sounds good! | 14:37 |
TheJulia | Does anyone remember if last week we reached consensus on the make automated clean work in both directions rfe? | 14:41 |
TheJulia | dtantsur: ^^^ | 14:42 |
dtantsur | TheJulia: I think JayF was "whatever" in the end, so we sort of did? | 14:42 |
dtantsur | I do intend to make this feature opt-in from the operator's perspective after that conversation | 14:42 |
TheJulia | okay, then I think that is fine and I'm totally cool with that. I think that is the right way to head in | 14:43 |
dtantsur | bifrost items.. lemme see what we've already got | 14:43 |
rpioso | arne_wiebalck: I created the doodle: https://doodle.com/poll/s3dttwbbuvc2n6dx | 14:43 |
rpioso | arne_wiebalck: Could we use your video bridge? | 14:44 |
arne_wiebalck | rpioso: sure, once we settle on the slot, I will create a meeting | 14:45 |
rpioso | arne_wiebalck: Thank you! | 14:49 |
Qianbiao | Hello, is there a method to confirm whether a diskimage element is correctly append. | 14:49 |
arne_wiebalck | rpioso: Thanks for driving this! | 14:49 |
rpioso | arne_wiebalck: :-) | 14:50 |
*** abdysn has quit IRC | 14:53 | |
*** stendulker has joined #openstack-ironic | 14:57 | |
JayF | dtantsur: TheJulia: I mean, I'm not happy with the way things are. I won't be happy with the way things will be. We need to take this ask -- for an anti-feature of leaving data preserved on "available" nodes -- as a call to action to handle data-preservation cases better within Ironic, without doing things like that. | 14:59 |
iurygregory | o/ | 15:00 |
TheJulia | #startmeeting ironic | 15:00 |
TheJulia | o/ | 15:00 |
openstack | Meeting started Mon Sep 14 15:00:51 2020 UTC and is due to finish in 60 minutes. The chair is TheJulia. 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 |
JayF | o/ | 15:00 |
iurygregory | o/ | 15:01 |
mgoddard | \o | 15:01 |
rpioso | \o | 15:01 |
bfournie | o/ | 15:01 |
erbarr | o/ | 15:01 |
stendulker | o/ | 15:01 |
TheJulia | Good morning everyone! | 15:01 |
arne_wiebalck | o/ | 15:01 |
TheJulia | Our Ironic meeting agenda can be found on the wiki. | 15:01 |
TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
TheJulia | #topic CFP is open for the Forum | 15:01 |
*** openstack changes topic to "CFP is open for the Forum (Meeting topic: ironic)" | 15:01 | |
*** kashifkhan has quit IRC | 15:01 | |
TheJulia | #undo | 15:01 |
openstack | Removing item from minutes: #topic CFP is open for the Forum | 15:01 |
TheJulia | #topic Announcements / Reminders | 15:02 |
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)" | 15:02 | |
dtantsur | o/ | 15:02 |
rpioso | TheJulia: We have a doodle up for an ironic Redfish Interop Profiles sync-up: https://doodle.com/poll/s3dttwbbuvc2n6dx | 15:02 |
TheJulia | #info CFP for the forum is supposed to be open. Apparently the submission form is closed but that should hopefully get fixed soon. we need to submit any items _today_ | 15:02 |
TheJulia | #info Since we are in the home stretch for the victoria cycle, reviewers, please review changes this week! | 15:03 |
TheJulia | #info A gentle reminder (for even me) to self triage bugs! | 15:03 |
TheJulia | #link https://docs.openstack.org/ironic/latest/contributor/bugs.html#triaging-guide | 15:03 |
TheJulia | #info vPTG time slots have been selected - 2PM-4PM UTC ?Monday through Wednesday? | 15:03 |
TheJulia | #info Redfish Inter-op profiles sync-up call doodle has been proposed | 15:04 |
TheJulia | #link https://doodle.com/poll/s3dttwbbuvc2n6dx | 15:04 |
rloo | o/ | 15:04 |
TheJulia | Does anyone have anything else to announce or remind us of? | 15:04 |
Qianbiao | TheJulia not sure whether is suitable to remind here. the ramdisk deploy interface issue, may someone take a look at. | 15:05 |
Qianbiao | I have add it to the white board in vendor list | 15:05 |
TheJulia | Okay, | 15:06 |
cdearborn | o/ | 15:06 |
TheJulia | It is not really an item for the general community wide reminders, but good to know | 15:07 |
TheJulia | Anyone have anything else? | 15:07 |
TheJulia | I guess not! | 15:08 |
TheJulia | Lets carry on! | 15:08 |
TheJulia | No action items from our previous meeting, so we will carry on directly to reviewing subteam status reports | 15:08 |
TheJulia | #topic Review subteam status reports | 15:09 |
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:09 | |
TheJulia | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:09 |
TheJulia | Starting at line 282. | 15:09 |
TheJulia | dtantsur: do you want me to propose a 8.3.1 release for bifrost? | 15:10 |
openstackgerrit | Merged openstack/bifrost master: Let us finish the rabbit hunt https://review.opendev.org/748265 | 15:10 |
dtantsur | TheJulia: if you don't mind. just remember that it's a manual process for now | 15:11 |
TheJulia | manual as in manual tag or manual as in manual edit the file? | 15:11 |
dtantsur | TheJulia: as in go to #openstack-release and tell them git hash, branch and version | 15:12 |
TheJulia | #sadness | 15:12 |
iurygregory | oh we can't push a patch? O.o | 15:13 |
iurygregory | I can do that since TheJulia seems sad =) | 15:13 |
TheJulia | iurygregory: by all means, and if they object to it point them to the IRC log! | 15:13 |
iurygregory | TheJulia, ack =) | 15:13 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add redfish options to the generated docs https://review.opendev.org/751839 | 15:14 |
dtantsur | iurygregory: anyway ping one of us to double-check | 15:14 |
iurygregory | dtantsur, sure | 15:14 |
TheJulia | dtantsur: looks like the most important ramdisk tls patch is the eventlet monkey patching? is that correct? | 15:14 |
dtantsur | mmm, no | 15:15 |
JayF | that patch was integrated in the static TLS support | 15:15 |
dtantsur | we're no longer blocked by that | 15:15 |
JayF | it's all one patch now | 15:15 |
dtantsur | yes, this | 15:15 |
JayF | I punted on the "we should monkey patch everything" problem, fell back to just monkey patching socket. | 15:15 |
dtantsur | auto-TLS works in the CI once the IPA patch is approved | 15:15 |
TheJulia | okay then, so basically that feature is approved and will hopefully merge today *crosses fingers, toes, etc* | 15:16 |
JayF | One question that I thought of: do we think we should also have *manual* TLS in a job? I'm leaning towards no, but wanted that to be an explicit decision. | 15:16 |
* dtantsur ponders | 15:16 | |
TheJulia | iurygregory: line 408 Make CI manageable: You have two patches up. What is the path forward on those? | 15:16 |
dtantsur | JayF: I'm not sure how much coverage of *our* code it would add | 15:17 |
*** JamesBenson has joined #openstack-ironic | 15:17 | |
* TheJulia is unsure and is leaning towards no on that question because of the amount of resources that would be needed to test it | 15:17 | |
iurygregory | TheJulia, I'm still fixing things in https://review.opendev.org/741886 | 15:17 |
patchbot | patch 741886 - ironic - Add tempest default_boot_option and altflavor - 16 patch sets | 15:17 |
JayF | Really, I think it'd add coverage of the listen_tls agent conf (minimal) and the ipa-tls static config element | 15:17 |
iurygregory | to get all green | 15:17 |
TheJulia | iurygregory: ack, please add that to the notes. | 15:17 |
dtantsur | JayF: we could add such a job to IPA-builder only | 15:18 |
TheJulia | dtantsur: I like that idea | 15:18 |
dtantsur | but it will need some devstacking as well | 15:18 |
JayF | eh, just config for Ironic to ignore validity of IPA certs, then you get the TLS generation and it gets "free"-ish with a properly configured ramdisk | 15:18 |
TheJulia | iurygregory: any estimate on a CI job for dhcp-less? | 15:19 |
dtantsur | we build the ramdisk somewhere inside devstack, and it's not overly customizable. I don't remember if you can add an element. | 15:19 |
dtantsur | not a huge deal, of course | 15:19 |
iurygregory | TheJulia, no estimate atm | 15:19 |
JayF | ah, makes sense, shouldn't be impossible | 15:19 |
TheJulia | Seems like we're good to proceed? | 15:20 |
JayF | +1 | 15:22 |
TheJulia | #topic Priorities for the week | 15:22 |
*** openstack changes topic to "Priorities for the week (Meeting topic: ironic)" | 15:22 | |
TheJulia | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:22 |
TheJulia | Starting at line 141 | 15:22 |
TheJulia | I'll remove the merged items from the list (of which there are a number, one moment. | 15:23 |
TheJulia | I'm going to remove the redfish RAID WIP since it has needed rebasing/updating for a few weeks | 15:24 |
*** ianychoi has joined #openstack-ironic | 15:24 | |
TheJulia | Any objections? | 15:24 |
Qianbiao | <dtantsur> devstack has option to add element | 15:25 |
dtantsur | cool! | 15:25 |
dtantsur | TheJulia: none | 15:25 |
Qianbiao | but devstack ir really not friendly for hardware bm nodes. | 15:25 |
TheJulia | Okay, new items starting at 212. | 15:25 |
TheJulia | The metalsmith changes are around trying to fix a bug/issue in IPA with partition images and UEFI, so I needed a compatible job to test in as well. I think it has identified another issue, but I got derailed on Friday due to the lower-constraints headaches | 15:26 |
TheJulia | Everyone good with those items? | 15:27 |
derekh | Is it ok if I added one to the bug fixes: section ? https://review.opendev.org/#/c/726275/ happy to test etc.. if needed to help it along | 15:28 |
patchbot | patch 726275 - x/sushy-oem-idrac - WIP: Accommodate GET HTTP 202 status without Location - 3 patch sets | 15:28 |
dtantsur | note that most of us don't have core rights there | 15:28 |
dtantsur | so it's more for rpioso, cdearborn and ajya|afk | 15:28 |
rpioso | derekh: It's a priority for us :-) | 15:29 |
derekh | dtantsur: ack | 15:29 |
derekh | rpioso: cool, thanks | 15:29 |
* TheJulia adds the items | 15:30 | |
TheJulia | How does that look for everyone? | 15:32 |
TheJulia | derekh: I guess it is okay to add, but again basically only dell folks have the power to merge it. | 15:32 |
derekh | TheJulia: gotchya, thanks | 15:33 |
TheJulia | ohh, the ipe change has changed | 15:34 |
dtantsur | the list is impressive :) | 15:34 |
TheJulia | Well, it is a week we need reviews performed _AND_ some of those items are already approved | 15:34 |
TheJulia | What is impressive and difficult to put into words is how awesome our leaps and bounds as a community and a project have been | 15:35 |
TheJulia | (I did the cycle highlights on Friday) | 15:35 |
TheJulia | dtantsur: I think I may go add ramdisk tls to that ;) | 15:35 |
JayF | \o/ | 15:35 |
TheJulia | Now we just need to get things out the door and released | 15:35 |
dtantsur | TheJulia: I'd appreciate that ;) | 15:35 |
dtantsur | we may also add our intention to deprecate the iscsi deploy to the highlights | 15:36 |
dtantsur | (I'll try to finish the patch today-tomorrow) | 15:36 |
TheJulia | fyi, the Forum submission tool has been re-opened | 15:36 |
TheJulia | Which means we likely ought to submit topics asap. | 15:36 |
TheJulia | I guess time for discussion | 15:37 |
TheJulia | #topic Discussion | 15:37 |
*** openstack changes topic to "Discussion (Meeting topic: ironic)" | 15:37 | |
TheJulia | I guess this is more informational. | 15:37 |
TheJulia | I held off on getting a kexec meeting scheduled because of the chance of getting some students from Boston University involved. That didn't happen and we got the sad news Friday that they couldn't form up a team to work with us for a few months. But! Ironic is going to be presented to the senior engineering students this week! | 15:38 |
TheJulia | \o/ | 15:38 |
TheJulia | arne_wiebalck: trandles: sorry :( | 15:39 |
TheJulia | Onward to other things at the moment! | 15:39 |
tosky | from the zuulv3 point of view: please remember to merge the remaining zuulv3 job (it's broken, but less broken that the current one) before next week! | 15:39 |
dtantsur | let's put it on the priority list | 15:40 |
dtantsur | maybe we can fix it with a backport later on... | 15:40 |
TheJulia | There is the ongoing topic of the raid_config behavior contract and the desire to change it. We briefly brought this up last week with less than our normal number of attendees last week and the consensus was that we should have it as a Forum discussion topic. Are there any other feelings on this topic? Does everyone that missed last week's meeting concur that the forum is the right place at this point? | 15:40 |
TheJulia | tosky: do you have the link handy? | 15:41 |
tosky | TheJulia: https://review.opendev.org/#/c/705030/ - iurygregory is the expert there, I don't remember if the other failures are unrelate | 15:42 |
patchbot | patch 705030 - ironic - Native zuulv3 grenade multinode multitenant - 33 patch sets | 15:42 |
tosky | unrelated* | 15:42 |
iurygregory | I will update removing the Depends-On so we can merge since it needs to be max till 20 Sep | 15:43 |
* TheJulia takes silence on discussion topics as agreement | 15:44 | |
TheJulia | I guess Baremetal SIG is nexg | 15:44 |
TheJulia | next | 15:44 |
TheJulia | #topic Baremetal SIG | 15:44 |
*** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)" | 15:44 | |
TheJulia | arne_wiebalck: ? | 15:44 |
arne_wiebalck | Two little things. | 15:44 |
arne_wiebalck | The doodle for the regular slot does only have replies from Ironic cores. | 15:45 |
arne_wiebalck | I was hoping for some wider interest, shall we resend the mail, or just start? | 15:45 |
TheJulia | arne_wiebalck: The mailing list is likely a good idea | 15:46 |
iurygregory | maybe resend email and tell it will start on *day time* | 15:46 |
arne_wiebalck | Ok, I will resend. | 15:46 |
arne_wiebalck | 2nd item: Should we ask for a forum slot for the SIG? Or is one at the PTG enough? | 15:46 |
TheJulia | arne_wiebalck: I think that would be a good idea | 15:46 |
TheJulia | Will you make the submission? | 15:46 |
arne_wiebalck | At both, or only one? | 15:47 |
TheJulia | sorry, the forum segment for the SIG | 15:47 |
arne_wiebalck | IMO, the forum would be sufficient. | 15:47 |
arne_wiebalck | right | 15:47 |
TheJulia | I think it is the right venue for the SIG to do requirements gathering to feed into the ptg | 15:47 |
arne_wiebalck | yes, my thinking as well | 15:47 |
arne_wiebalck | ok, I will submit | 15:47 |
arne_wiebalck | That's it from me. | 15:48 |
TheJulia | k, I'll submit the raid_config contract one after the meeting. I don't think anyone has raised any other topics fo rit | 15:48 |
TheJulia | Okay! | 15:48 |
TheJulia | #topic RFE Review | 15:48 |
*** openstack changes topic to "RFE Review (Meeting topic: ironic)" | 15:48 | |
TheJulia | We had one item from last week, which got a little further discussion before the meeting, so I'm not sure we need to rehash it during this meeting and can remove it form the agenda | 15:48 |
TheJulia | Any obejctions? | 15:48 |
dtantsur | not from me | 15:49 |
* dtantsur is still thinking about forum topics | 15:49 | |
TheJulia | #topic Open Discussion | 15:49 |
dtantsur | well, operator feedback can always work | 15:49 |
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:49 | |
arne_wiebalck | dtantsur: ++ | 15:49 |
dtantsur | maybe also solving support matrix around 2 types of images vs 2 boot modes vs software RAID | 15:50 |
TheJulia | It might be a worthwhile forum topic because there is a delineation in between "Ironic as a interface layer for multitenant cloud computing" and "ironic as a general purpose tool in an enterprise" | 15:50 |
TheJulia | dtantsur: sounds like something to submit! | 15:50 |
dtantsur | TheJulia: are you going to bulk-submit everything we come up with? | 15:51 |
TheJulia | I was thinking individually because there is the internal limit | 15:52 |
TheJulia | of 3 submissions per person | 15:52 |
TheJulia | (unless you kindly ask jimmy and make a good case) | 15:52 |
TheJulia | I've already got one for the project update, so I can submit two more | 15:52 |
dtantsur | I can submit something for image types/boot modes, especially if I find a submission link :) | 15:53 |
TheJulia | cfp.openstack.org | 15:53 |
TheJulia | Link is also on the agenda I believe | 15:53 |
dtantsur | okie, will do after the meeting | 15:53 |
*** zzzeek has quit IRC | 15:54 | |
TheJulia | arne_wiebalck: would it make sense to combine operator feedback in with the conundrum I mentioned earlier? | 15:54 |
arne_wiebalck | which conundrum is that? | 15:55 |
arne_wiebalck | ah, yes | 15:55 |
TheJulia | yeah | 15:55 |
*** zzzeek has joined #openstack-ironic | 15:55 | |
rpioso | TheJulia: Regarding the ongoing topic of the raid_config behavior contract and desire to change it ^^^, I and others would like to better understand the basis for the concerns within our project. Could we discuss at both the Forum and PTG? | 15:56 |
dtantsur | I suspect the PTG is a better place even | 15:56 |
rpioso | +1 | 15:56 |
TheJulia | PTG is a far better place for the raid behavior contract change proposed | 15:56 |
dtantsur | not sure how many non-contributors will understand what we'll be talking about | 15:56 |
rpioso | I for one don't :-) | 15:57 |
TheJulia | I think operator feedback on "will this be useful" is vital to come into that discussion with | 15:57 |
rpioso | At least the concerns. | 15:57 |
TheJulia | arne_wiebalck: are you going to submit operator feedback session or do you want me to? | 15:57 |
arne_wiebalck | I can do it. | 15:57 |
TheJulia | k | 15:57 |
rpioso | +1 re: vote for both | 15:57 |
TheJulia | okay then | 15:58 |
TheJulia | Well, Unless there is something else, I think we're done for our meeting this week. | 15:58 |
TheJulia | Thanks everyone! | 15:59 |
arne_wiebalck | Thanks, TheJulia ! | 15:59 |
rpioso | TheJulia: Thank you! | 15:59 |
TheJulia | #endmeeting | 15:59 |
*** 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-discuss mailing list." | 15:59 | |
openstack | Meeting ended Mon Sep 14 15:59:33 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:59 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-09-14-15.00.html | 15:59 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-09-14-15.00.txt | 15:59 |
openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-09-14-15.00.log.html | 15:59 |
Qianbiao | Thanks, TheJulia | 16:00 |
Qianbiao | lol, I have no idea about most of the discussion :) | 16:00 |
JayF | That's true of almost all of us Qianbiao, don't feel alone :) | 16:00 |
Qianbiao | ;) o/ | 16:01 |
JayF | We all have our areas of specialty, except TheJulia and dtantsur who seem to know everything sometimes :D | 16:01 |
*** lucasagomes has quit IRC | 16:01 | |
TheJulia | I know nothing! | 16:01 |
dtantsur | if only :D | 16:01 |
*** zzzeek has quit IRC | 16:02 | |
TheJulia | dtantsur: if only.... we would rule the world! | 16:03 |
dtantsur | \o/ | 16:03 |
Qianbiao | My question is coming. someone familiar with infiniband port? I can get IB IF info from inspector now. | 16:03 |
TheJulia | I guess that means I need to make a roux for lunch.... so I can roux the day! | 16:04 |
Qianbiao | but it is filtered by standard processor. | 16:04 |
Qianbiao | Is there a method that can add IB IF port to ironic port. | 16:04 |
* TheJulia escorts herself out for the bad pun | 16:04 | |
*** zzzeek has joined #openstack-ironic | 16:04 | |
dtantsur | :D | 16:04 |
*** priteau has quit IRC | 16:04 | |
dtantsur | Qianbiao: did you figure out why it's ruled out? | 16:05 |
Qianbiao | yes | 16:05 |
TheJulia | Yeah, that is the biggest question I would have, why is it excluded | 16:05 |
Qianbiao | In "ValidateInterfacesHook" | 16:05 |
iurygregory | dtantsur, for bifrost 8.3.1 on bugfix/8.3 the commit I've found is 681f054d07cd21404525330fe15c86f781094aa9 | 16:05 |
dtantsur | Qianbiao: maybe you need to change [processing]add_ports? | 16:05 |
Qianbiao | I have set it to all for now | 16:05 |
dtantsur | iurygregory: looks about right | 16:06 |
iurygregory | so I only need to ask in the #openstack-release with this information? | 16:06 |
Qianbiao | In "class ValidateInterfacesHook(base.ProcessingHook):", it validate the mac address, with "netutils.is_valid_mac(mac):" | 16:06 |
Qianbiao | all ib mac does not match the rule. | 16:06 |
Qianbiao | inspector data http://paste.openstack.org/show/797846/ | 16:06 |
dtantsur | Qianbiao: the mellanox hardware manager is supposed to provide MACs in the right format.. | 16:07 |
Qianbiao | dtantsur hmmm, there must be something wrong. i found this doc: https://docs.openstack.org/ironic-inspector/latest/user/usage.html#infiniband-support | 16:09 |
Qianbiao | does it means I need to pxe through IB port? | 16:09 |
Qianbiao | currently, we are pxe on eth port. | 16:09 |
dtantsur | Qianbiao: I think this applies to PXE through an IB port | 16:09 |
*** zzzeek has quit IRC | 16:09 | |
dtantsur | if you don't need it, I think it's worth trying to understand why the mlnx hardware manager doesn't work the way you expect | 16:10 |
*** zzzeek has joined #openstack-ironic | 16:10 | |
TheJulia | Did the mac filtering logic get changed anytime recently becasue IB macs are longer if memory serves | 16:10 |
Qianbiao | Yes TheJulia, I am pretty sure with this. IB mac is different with normal mac | 16:11 |
Qianbiao | But whether inspector transform it to normal MAC regex, i am not sure. | 16:11 |
Qianbiao | dtantsur do you mean mlnx code in IPA will transform the IB MAC | 16:12 |
dtantsur | Qianbiao: that's how I read it, I'm completely unfamiliar with it however | 16:12 |
Qianbiao | ok I will check source code. | 16:12 |
dtantsur | TheJulia: Ocata is no longer recently :) but that applies only to PXE via IB, Qianbiao just needs the ports discovered | 16:13 |
dtantsur | the mlnx hardware manager seems to do what is needed. the question is why it does not kick in. ramdisk logs may provide a clue. | 16:13 |
Qianbiao | TheJulia dtantsur yes, we are pxe over eth port. Just want to discovery the IB port, and setup IB partition(vlan) for IB networking. | 16:15 |
*** zzzeek has quit IRC | 16:15 | |
TheJulia | hardware manager logs do seem needed :\ | 16:15 |
Qianbiao | TheJulia dtantsur https://github.com/IamFive/gist/tree/master/ib-inspect-ramdisk-log | 16:17 |
dtantsur | "No Mellanox devices found" ehmmm | 16:17 |
Qianbiao | oh .... | 16:17 |
Qianbiao | I see that too..at the moment. | 16:18 |
dtantsur | I wonder if they have different vendor IDs. or the devices are not detected by the kernel. | 16:18 |
*** zzzeek has joined #openstack-ironic | 16:19 | |
*** stendulker has quit IRC | 16:19 | |
dtantsur | it feels that IPA recognizes only lo and eno2 | 16:20 |
dtantsur | Qianbiao: my wild guess at this point that the operating system does not recognize or does not properly initialize the IB interfaces | 16:20 |
Qianbiao | dtantsur hmmm. it seems all interface is ready | 16:20 |
Qianbiao | I see that in log at somewhere | 16:20 |
dtantsur | do you see any mentions of the IB interfaces? | 16:21 |
dtantsur | not sure if `ip addr` is supposed to show IB interfaces, but they're not there: https://github.com/IamFive/gist/blob/master/ib-inspect-ramdisk-log/ip_addr | 16:21 |
*** derekh has quit IRC | 16:22 | |
Qianbiao | <dtantsur> than why inspect data contains the ib port | 16:22 |
TheJulia | So rebuilds/ephemeral preservation and opt-in/out cleaning. Are there any other sort of identity crisis sort of issues that anyone can think of? | 16:22 |
Qianbiao | http://paste.openstack.org/show/797846/ line 53 | 16:23 |
dtantsur | Qianbiao: mmmmmmmmmmmmm, okay, you got me :) | 16:23 |
*** derekh has joined #openstack-ironic | 16:23 | |
TheJulia | (identify crsis is likely a little strong of a statement, but hopefully conveys the conundrums) | 16:23 |
Qianbiao | dtantsur TheJulia thanks, this issue is pretty closed to fixed. I will check the log again. | 16:24 |
dtantsur | I don't see the devices in lshw either O_o | 16:25 |
*** mkrai has quit IRC | 16:25 | |
dtantsur | I don't know how IPA even learns about them, but at least we know that the mlnx hardware manager does not recognize them for some reason | 16:25 |
Qianbiao | <dtantsur> .... I copied a wrong ramdisk log | 16:25 |
dtantsur | heh, good, because I already started thinking I was insane :D | 16:26 |
dtantsur | (well, less sane than usual) | 16:26 |
TheJulia | could the issue be that the hardware is not enabled because of the boot mode? | 16:26 |
TheJulia | like... maybe it doesn't have uefi firmware or bios firmware? | 16:26 |
TheJulia | we've seen that before with some of the early 10GB nics | 16:26 |
TheJulia | arne_wiebalck: I was thinking the conundrum should really be a separate session. Thoughts? | 16:27 |
Qianbiao | dtantsur log updated | 16:27 |
arne_wiebalck | Ah, I understood you wanted to combine these into one. | 16:27 |
TheJulia | I'm really unsure | 16:27 |
TheJulia | so if anyone has strong feelings either way? | 16:28 |
arne_wiebalck | I think it fits well with the ops session: hopefully there will be people using it either way. | 16:28 |
TheJulia | okay, then mind if I add some words to that submission | 16:29 |
arne_wiebalck | TheJulia: Or did you want to make this a more general overall direction discussion? | 16:29 |
TheJulia | Well, I was thinking general discussion as a larger community would be good | 16:29 |
TheJulia | So maybe we need both? | 16:29 |
arne_wiebalck | Input for this would mostly come from operators, no? | 16:30 |
TheJulia | Yeah | 16:30 |
arne_wiebalck | I mean, everyone will/would have an opinion, but in the end it is what people operating the physical infrastructure would like to see. | 16:31 |
arne_wiebalck | And where the emphasis should go. | 16:31 |
TheJulia | ++ | 16:31 |
arne_wiebalck | Anyway, does not mean I have a strong opinion, happy to take this out again from the ops session. | 16:32 |
openstackgerrit | Verification of a change to openstack/ironic failed: Accept and use a TLS certificate from the agent https://review.opendev.org/747136 | 16:32 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Generate a TLS certificate and send it to ironic https://review.opendev.org/749930 | 16:32 |
dtantsur | Qianbiao: the journal is truncated for some reason | 16:32 |
TheJulia | Lets have it in the ops session then | 16:32 |
Qianbiao | <dtantsur> checking | 16:33 |
arne_wiebalck | TheJulia: Ok, apologies for the misunderstanding! | 16:33 |
TheJulia | arne_wiebalck: made a minor addition | 16:34 |
Qianbiao | dtantsur not sure why, but that is all in ramdisk log tar.gz | 16:34 |
arne_wiebalck | TheJulia: feel free to edit, I was merely the submitter :) | 16:34 |
dtantsur | Qianbiao: bad luck :( | 16:34 |
TheJulia | I do like that I could edit it... thanks jimmy! | 16:35 |
arne_wiebalck | That's indeed handy! | 16:35 |
arne_wiebalck | What are multiple levels of RAID? | 16:35 |
dtantsur | TheJulia: will many people be upset if I cope-paste the same thing into all 3 fields the submission requires? | 16:35 |
* dtantsur hates too many free-form fields | 16:36 | |
TheJulia | dtantsur: not really | 16:36 |
arne_wiebalck | TheJulia: multiple levels of RAID is RAID on top of RAID? | 16:36 |
TheJulia | Yeah, basic input into the raid_config contract change | 16:37 |
arne_wiebalck | Ah, ok. | 16:37 |
arne_wiebalck | Bye everyone o/ | 16:38 |
TheJulia | onboarding session submitted | 16:39 |
Qianbiao | arne_wiebalck raid is pretty complex. I code almost 3000 line to finish ibmc raid config logics. | 16:40 |
trandles | TheJulia: yay! | 16:43 |
TheJulia | raid is one of the most complex and opinionated topics I've ever encountered in this community | 16:43 |
JayF | I suspect if we ever get in the business of doing freeform partitions/LVM it'll get a run for its money in complexity | 16:43 |
TheJulia | Well yes | 16:43 |
JayF | (obviously KS driver will do this, but is doing it by punting all the complexity to external installers) | 16:43 |
* TheJulia remembers the discussion that was had in the very very very cold room in Dublin | 16:44 | |
* dtantsur only remembers the cold room | 16:44 | |
dtantsur | kidding, I remember it as well. it's an awesome feature, but the required heavy lifting is.. enourmous | 16:44 |
JayF | It's nice to have a bridge in the external installer driver zer0c00l is working on, but I don't know if we'll be able to avoid it in Ironic-proper installs forever | 16:45 |
Qianbiao | dtantsur thanks for your help. the mlnx mac issue, i will figure it out tomorrow. (everything wrong must happen in ipa/mlnx.py) | 16:48 |
Qianbiao | need to go, see you all. | 16:49 |
dtantsur | see you Qianbiao | 16:49 |
*** Qianbiao is now known as Qianbiao|afk | 16:49 | |
*** Qianbiao|afk has quit IRC | 16:49 | |
*** derekh has quit IRC | 17:00 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent bugfix/6.3: Fix backup node lookup https://review.opendev.org/751864 | 17:03 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent bugfix/6.3: Update the cache if we don't have a root device hint https://review.opendev.org/751865 | 17:04 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent bugfix/6.2: Update the cache if we don't have a root device hint https://review.opendev.org/751866 | 17:08 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: Update the cache if we don't have a root device hint https://review.opendev.org/751868 | 17:12 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent bugfix/6.3: Fix backup node lookup https://review.opendev.org/751864 | 17:15 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent bugfix/6.3: Fix backup node lookup https://review.opendev.org/751864 | 17:16 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent bugfix/6.2: Fix backup node lookup https://review.opendev.org/751870 | 17:17 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: Fix backup node lookup https://review.opendev.org/751871 | 17:17 |
TheJulia | yay for backports | 17:18 |
* TheJulia takes a short break for a snack | 17:24 | |
*** dtantsur is now known as dtantsur|afk | 17:35 | |
dtantsur|afk | o/ | 17:35 |
*** Lucas_Gray has quit IRC | 17:41 | |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Remove unused jobs https://review.opendev.org/749725 | 18:20 |
*** dsneddon has joined #openstack-ironic | 18:23 | |
*** mgoddard has quit IRC | 18:34 | |
*** mgoddard has joined #openstack-ironic | 18:41 | |
*** jtomasek has joined #openstack-ironic | 18:55 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Clarify HPE Edgeline support https://review.opendev.org/751881 | 19:01 |
*** jtomasek has quit IRC | 19:08 | |
*** zzzeek has quit IRC | 19:14 | |
*** zzzeek has joined #openstack-ironic | 19:16 | |
*** zzzeek has quit IRC | 19:20 | |
*** zzzeek has joined #openstack-ironic | 19:23 | |
*** JamesBenson has quit IRC | 19:24 | |
*** JamesBenson has joined #openstack-ironic | 19:28 | |
*** zzzeek has quit IRC | 19:28 | |
*** zzzeek has joined #openstack-ironic | 19:29 | |
*** zzzeek has quit IRC | 19:33 | |
*** zzzeek has joined #openstack-ironic | 19:35 | |
*** uzumaki has quit IRC | 19:41 | |
*** zzzeek has quit IRC | 19:54 | |
openstackgerrit | Merged openstack/bifrost master: Remove unnecessary overrides of ansible_python_interpreter https://review.opendev.org/749262 | 19:56 |
*** zzzeek has joined #openstack-ironic | 19:57 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Accept and use a TLS certificate from the agent https://review.opendev.org/747136 | 20:16 |
openstackgerrit | Bill Dodd proposed openstack/ironic master: WIP: Add Redfish RAID management to Ironic https://review.opendev.org/730000 | 20:22 |
*** hjensas is now known as hjensas|afk | 20:44 | |
openstackgerrit | Bill Dodd proposed openstack/sushy master: Make Actions field in Volume resource optional https://review.opendev.org/751900 | 21:01 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Minor agent version code cleanup https://review.opendev.org/749552 | 21:10 |
openstackgerrit | Bill Dodd proposed openstack/sushy-tools master: Add Volumes nav property to Storage template https://review.opendev.org/751903 | 21:22 |
TheJulia | erbarr: hey, are you guys carrying a downstream patch on the devstack plugin? I see jobs failed after the ibmc change was proposed and that is the only thing that made sense. | 21:29 |
openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job https://review.opendev.org/748315 | 21:58 |
erbarr | TheJulia, which patch? | 22:02 |
TheJulia | https://review.opendev.org/#/c/748841/ | 22:06 |
patchbot | patch 748841 - ironic - Feat: add ibmc hardware info support for devstack - 3 patch sets | 22:06 |
erbarr | that looks to be a setuptools failure on the one job i checked, I'll do more digging in a little bit, need to walk my cat around the house | 22:23 |
erbarr | yard* | 22:23 |
*** tosky has quit IRC | 22:33 | |
openstackgerrit | Verification of a change to openstack/bifrost failed: Always setup environment in bifrost-cli and make it less verbose https://review.opendev.org/750315 | 22:34 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Add basic retries for inspection https://review.opendev.org/745598 | 22:38 |
TheJulia | erbarr: You mean your cat doesn't walk you? | 22:41 |
TheJulia | :) | 22:41 |
*** tkajinam has joined #openstack-ironic | 22:52 | |
*** dtruong has quit IRC | 22:54 | |
*** dtruong has joined #openstack-ironic | 22:54 | |
erbarr | TheJulia, lol, yea he goes where he pleases, there's no fence and some wildlife around and he's elderly in a new place so, just keeping him from getting in trouble | 22:59 |
*** portdirect has quit IRC | 23:13 | |
*** portdirect has joined #openstack-ironic | 23:13 | |
openstackgerrit | Verification of a change to openstack/bifrost failed: Force updating repos with copy_from_local_path https://review.opendev.org/750617 | 23:17 |
openstackgerrit | Verification of a change to openstack/bifrost failed: Re-download checkums on upgrades and updates https://review.opendev.org/750665 | 23:17 |
janders | good morning Ironic o/ | 23:34 |
*** Goneri has quit IRC | 23:53 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!