| buhman | would bad things happen if I removed all rows from the ironic ports table? I'm not sure why "tenant_vif_port_id" is useful. | 00:34 |
|---|---|---|
| zer0c00l | The ironic port's internal_info is updated with tenant_vif_port_id. Any idea what in ironic actually responsible for updating the port's internal_info? | 00:54 |
| opendevreview | Merged openstack/ironic-inspector master: Handle NodeLocked failures https://review.opendev.org/c/openstack/ironic-inspector/+/803935 | 00:58 |
| *** pmannidi is now known as pmannidi|AFK | 01:24 | |
| *** pmannidi|AFK is now known as pmannidi | 02:41 | |
| opendevreview | Julia Kreger proposed openstack/ironic master: Allow initial versions to not be created yet https://review.opendev.org/c/openstack/ironic/+/804003 | 02:42 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Implements node history: database https://review.opendev.org/c/openstack/ironic/+/768009 | 02:43 |
| buhman | Maybe a better question should have been: what's the correct way to delete a Neutron Port that may or may not be referenced by an Ironic Port? | 03:08 |
| buhman | Maybe a better question should have been: what's the correct way to delete a Neutron Port that may or may not be referenced by an Ironic Port? | 03:10 |
| *** pmannidi is now known as pmannidi|Lunch | 03:25 | |
| *** pmannidi|Lunch is now known as pmannidi|AFK | 04:17 | |
| opendevreview | David Vallee Delisle proposed openstack/ironic-lib master: json_rpc: list_opts() should return tuple https://review.opendev.org/c/openstack/ironic-lib/+/804194 | 04:32 |
| opendevreview | David Vallee Delisle proposed openstack/ironic-lib master: json_rpc: setup.cfg is missing entrypoint for json_rpc's list_opts https://review.opendev.org/c/openstack/ironic-lib/+/804195 | 04:32 |
| opendevreview | Vanou Ishii proposed openstack/ironic master: Make curl in DevStack Bypass Proxy https://review.opendev.org/c/openstack/ironic/+/804201 | 06:16 |
| *** pmannidi|AFK is now known as pmannidi | 06:20 | |
| opendevreview | Vanou Ishii proposed openstack/ironic master: Make curl in DevStack Bypass Proxy https://review.opendev.org/c/openstack/ironic/+/804201 | 06:21 |
| arne_wiebalck | Good morning, Ironic! | 06:49 |
| iurygregory | good morning arne_wiebalck and Ironic o/ | 06:54 |
| arne_wiebalck | hey iurygregory o/ | 06:54 |
| jssfr | Good Morning Ironic! | 06:55 |
| *** pmannidi is now known as pmannidi|brb | 06:56 | |
| iurygregory | morning jssfr | 06:56 |
| iurygregory | arne_wiebalck, by any chance do you have nodes with 4TB disks at CERN? | 06:57 |
| arne_wiebalck | iurygregory: exactly 4TB or >4TB ? | 06:57 |
| iurygregory | can be bigger | 06:57 |
| iurygregory | do you have an idea how long it takes if you do cleaning on it? | 06:58 |
| arne_wiebalck | I think we do, yes. But not for the system. | 06:58 |
| arne_wiebalck | iurygregory: that depends on the type of disk and the type of cleaning | 06:58 |
| arne_wiebalck | iurygregory: if secure erase is available it should be seconds | 06:59 |
| iurygregory | HDDs | 06:59 |
| arne_wiebalck | iurygregory: if it is not, it could be days I guess | 06:59 |
| iurygregory | hehehe | 06:59 |
| arne_wiebalck | iurygregory: if you erase with shred it will take *very* long | 06:59 |
| iurygregory | arne_wiebalck, tks :D | 07:00 |
| arne_wiebalck | iurygregory: this is why janders and I were discussing the "efficient" erase the other week | 07:00 |
| arne_wiebalck | iurygregory: to find a balance between how long it takes and much it cleans | 07:00 |
| iurygregory | yeah | 07:01 |
| arne_wiebalck | iurygregory: days is an issue for public clouds I would think | 07:01 |
| iurygregory | also wondering the effect when we are doing the write_image deploy step | 07:01 |
| arne_wiebalck | what do you mean? | 07:01 |
| iurygregory | we need to write the image to the device | 07:03 |
| iurygregory | so if you have a 4TB disk where you want to write the image it can take very long also | 07:04 |
| arne_wiebalck | that should be independent from the disk size and only depend on the image size, no? | 07:10 |
| *** rpittau|afk is now known as rpittau | 07:23 | |
| rpittau | good morning ironic! o/ | 07:23 |
| iurygregory | yeah that's what I'm thinking but maybe qemu is trying some weird things.. | 07:23 |
| iurygregory | morning rpittau o/ | 07:23 |
| rpittau | hey iurygregory :) | 07:23 |
| rpittau | JayF, cenne, re: openstacksdk, that was probably overlooked when it was committed, feel free to correct it | 07:25 |
| *** pmannidi|brb is now known as pmannidi | 07:47 | |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic bugfix/18.1: Enable priority overrides to enable/disable steps https://review.opendev.org/c/openstack/ironic/+/804156 | 08:15 |
| iurygregory | if any ironic core has some minutes to +A the fix for IPE CI https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/804149 it would be amazing :D | 08:17 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Explicity set IPE job to ML2/OVS https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/804149 | 08:58 |
| cenne | Good morning iurygregory, rpittau, arne_wiebalck, jssfr | 09:10 |
| cenne | Thanks rpittau. I'll make a patch. | 09:11 |
| iurygregory | hey cenne o/ | 09:11 |
| cenne | hey iurygregory o/ | 09:12 |
| arne_wiebalck | hey cenne o/ | 09:14 |
| rpittau | hey cenne :) | 09:14 |
| cenne | hey o/ | 09:14 |
| opendevreview | Merged openstack/ironic-python-agent stable/wallaby: Force immediate NTP time sync with chronyd at IPA startup https://review.opendev.org/c/openstack/ironic-python-agent/+/804096 | 09:56 |
| opendevreview | Merged openstack/ironic-python-agent stable/victoria: Force immediate NTP time sync with chronyd at IPA startup https://review.opendev.org/c/openstack/ironic-python-agent/+/804097 | 09:56 |
| opendevreview | Merged openstack/ironic-python-agent master: Increase version of hacking and pycodestyle https://review.opendev.org/c/openstack/ironic-python-agent/+/802903 | 10:02 |
| iurygregory | merge all the things :D | 10:07 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Make reset_idrac and clear_job_queue verify steps https://review.opendev.org/c/openstack/ironic/+/804032 | 10:45 |
| *** pmannidi is now known as pmannidi|brb | 10:47 | |
| opendevreview | Jacob Anders proposed openstack/ironic master: Make reset_idrac and clear_job_queue verify steps https://review.opendev.org/c/openstack/ironic/+/804032 | 10:51 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Make reset_idrac and clear_job_queue verify steps https://review.opendev.org/c/openstack/ironic/+/804032 | 11:06 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Make reset_idrac and clear_job_queue verify steps https://review.opendev.org/c/openstack/ironic/+/804032 | 11:07 |
| opendevreview | Merged openstack/ironic-python-agent master: Output verbose info from efibootmgr https://review.opendev.org/c/openstack/ironic-python-agent/+/803331 | 11:08 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Make reset_idrac and clear_job_queue verify steps https://review.opendev.org/c/openstack/ironic/+/804032 | 11:17 |
| rpittau | bye everyone, see you tomorrow! o/ | 11:28 |
| *** rpittau is now known as rpittau|afk | 11:28 | |
| janders | see you tomorrow Ironic o/ | 11:48 |
| iurygregory | bye janders o/ | 11:52 |
| *** pmannidi|brb is now known as pmannidi | 11:52 | |
| iurygregory | bye rpittau|afk o/ | 11:52 |
| opendevreview | Merged x/sushy-oem-idrac stable/wallaby: Enable coverage HTML output https://review.opendev.org/c/x/sushy-oem-idrac/+/803447 | 12:22 |
| opendevreview | Merged x/sushy-oem-idrac stable/victoria: Enable coverage HTML output https://review.opendev.org/c/x/sushy-oem-idrac/+/803448 | 12:22 |
| opendevreview | Bob Fournier proposed openstack/ironic master: [WIP] For particular vendors set the boot device prior to installing image https://review.opendev.org/c/openstack/ironic/+/802389 | 12:44 |
| opendevreview | Merged openstack/ironic stable/wallaby: Fix Redfish RAID interface_type physical disk hint https://review.opendev.org/c/openstack/ironic/+/800933 | 13:19 |
| TheJulia | good morning | 13:24 |
| opendevreview | Julia Kreger proposed openstack/ironic-inspector stable/wallaby: Handle NodeLocked failures https://review.opendev.org/c/openstack/ironic-inspector/+/804167 | 13:43 |
| iurygregory | good morning TheJulia | 13:43 |
| iurygregory | TheJulia, by any chance do you remember the irc nick of the person who mentioned about the Lenovo limitation in some slots? I'm trying to look at the irc logs hehe | 13:45 |
| TheJulia | oh jeeze | 13:45 |
| TheJulia | not off hand | 13:46 |
| TheJulia | timeu: o/ | 13:46 |
| TheJulia | iurygregory: it was timeu I believe | 13:47 |
| iurygregory | oh ok :D | 13:49 |
| *** pmannidi is now known as pmannidi|AFK | 13:50 | |
| * TheJulia feels moderately stumped by the database upgrade not working as expected | 13:53 | |
| opendevreview | Julia Kreger proposed openstack/ironic master: Allow initial versions to not be created yet https://review.opendev.org/c/openstack/ironic/+/804003 | 13:59 |
| buhman | What's the correct way to delete a Neutron Port that may or may not be referenced by an Ironic Port? I noticed Ironic throws "Unable to set binding:host_id for neutron port {port_id}. Error: Port {port_id} could not be found" when ironic_port[internal_info][tenant_vif_port_id] exists, but that Neutron Port no longer exists. | 14:21 |
| buhman | I also noticed ironic_port[internal_info] is read-only via the API. | 14:21 |
| iurygregory | if the neutron port no longer exists you probably need to delete the ironic port | 14:23 |
| iurygregory | either via CLI or API | 14:23 |
| buhman | What if a Neutron Port that represents that Ironic Port's mac address still exists, but has a different ID? | 14:25 |
| buhman | I suppose deleting the Ironic Port is still the thing to do, because the outcome would be that tenant_vif_port_id is deleted but the an Ironic Port for that mac address still exists. | 14:26 |
| iurygregory | you can update ussing attach detach vif | 14:27 |
| iurygregory | https://docs.openstack.org/python-ironicclient/latest/cli/osc/v1/index.html#baremetal-node-vif-attach | 14:28 |
| iurygregory | https://docs.openstack.org/python-ironicclient/latest/cli/osc/v1/index.html#baremetal-node-vif-detach | 14:28 |
| iurygregory | I think this would be the way to do | 14:28 |
| opendevreview | Aija Jauntēva proposed x/sushy-oem-idrac master: Fix IncludeInExport allowed value validation https://review.opendev.org/c/x/sushy-oem-idrac/+/800496 | 14:46 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Record node history and manage events in db https://review.opendev.org/c/openstack/ironic/+/803292 | 15:27 |
| TheJulia | buhman: what iurygregory says | 15:28 |
| TheJulia | buhman: it only operates by uuid of the vif record, although if there is another neutron port with the same mac address already, then that too will cause the next deploy to fail | 15:29 |
| TheJulia | since that port is obviously orphaned too | 15:29 |
| arne_wiebalck | bye everyone o/ | 15:31 |
| TheJulia | o/ | 15:34 |
| *** sshnaidm is now known as sshnaidm|afk | 15:35 | |
| buhman | TheJulia: wouldn't it be better to just uh not fail? | 15:37 |
| buhman | afaict, failing to updating binding:host_id on the Neutron Port isn't important enough to be fatal to the deployment. | 15:38 |
| TheJulia | buhman: it should fail on redeploy | 15:44 |
| TheJulia | then that is neutron port binding woudl fail because macs need to be unique in neturon | 15:44 |
| TheJulia | No, that *should* be fatal as far as I remember | 15:45 |
| buhman | right | 15:52 |
| buhman | it is fatal, that's why I'm complaining :) | 15:52 |
| TheJulia | oh | 16:10 |
| TheJulia | so... it is a separate port that is somehow orphaned. We shouldn't delete that port in neutron, its the conflict and the resulting hard fail that prevents dhcp from being able to be setup, at least with neutron | 16:11 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Allow initial versions to not be created yet https://review.opendev.org/c/openstack/ironic/+/804003 | 16:19 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Implements node history: database https://review.opendev.org/c/openstack/ironic/+/768009 | 16:19 |
| TheJulia | NobodyCam: you have an email | 16:22 |
| NobodyCam | Oh ... Checks ... and Good Morning Ironic'ers | 16:23 |
| * TheJulia files paperwork for a non-hurty brain | 16:23 | |
| opendevreview | Verification of a change to openstack/ironic-inspector stable/wallaby failed: Handle NodeLocked failures https://review.opendev.org/c/openstack/ironic-inspector/+/804167 | 16:25 |
| * TheJulia glares at CI | 16:40 | |
| iurygregory | zuul is asking for cookies | 16:53 |
| TheJulia | I don't have the brains to make zuul cookies today | 17:08 |
| * TheJulia thinks they would be cranberry oatmeal though | 17:09 | |
| opendevreview | Bob Fournier proposed openstack/ironic master: [WIP] For particular vendors set the boot device prior to installing image https://review.opendev.org/c/openstack/ironic/+/802389 | 17:12 |
| NobodyCam | mmmm Cranberry oatmeal | 17:23 |
| TheJulia | success, ironic-grenade passed | 17:39 |
| TheJulia | now to figure out explicitly why it is not catching sqlalchemy exceptions | 17:39 |
| TheJulia | stevebaker: looks like your tripleo inspection change passed :) | 17:54 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent-builder master: Allow opt-out of full ramdisk pruning https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/803994 | 18:02 |
| opendevreview | Bob Fournier proposed openstack/ironic master: [WIP] For particular vendors set the boot device prior to installing image https://review.opendev.org/c/openstack/ironic/+/802389 | 18:44 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Allow initial versions to not be created yet https://review.opendev.org/c/openstack/ironic/+/804003 | 19:03 |
| opendevreview | Merged openstack/ironic-inspector stable/wallaby: Handle NodeLocked failures https://review.opendev.org/c/openstack/ironic-inspector/+/804167 | 19:28 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Implements node history: database https://review.opendev.org/c/openstack/ironic/+/768009 | 19:30 |
| TheJulia | buhman: regardless, patches welcome | 19:38 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Allow opt-out of full ramdisk pruning https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/803994 | 22:11 |
| stevebaker | TheJulia: I'm glad my strategy of ignoring it being broken for so long paid off. I won't make a habit of it | 22:36 |
| TheJulia | stevebaker: I'm just glad I dug into the logs and found the actual cause | 22:56 |
| stevebaker | TheJulia: I remember looking at the logs and thinking this locking behaviour must be normal so it must be something else | 22:58 |
| TheJulia | it is... unfortunately | 23:02 |
| janders | good morning Ironic o/ | 23:04 |
| * TheJulia feels completely drained | 23:07 | |
| opendevreview | Julia Kreger proposed openstack/ironic master: Allow initial versions to not be created yet https://review.opendev.org/c/openstack/ironic/+/804003 | 23:09 |
| TheJulia | stevebaker: ^^^ works \o/ | 23:09 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Implements node history: database https://review.opendev.org/c/openstack/ironic/+/768009 | 23:10 |
| TheJulia | ^^^ just a rebase, but now passes | 23:10 |
| TheJulia | which means we don't need to do long series of changes for features anymore \o/ | 23:10 |
| *** pmannidi|AFK is now known as pmannidi | 23:33 | |
| stevebaker | nice | 23:34 |
| stevebaker | I think yesterday's pfizer-1 is making me a little woozy | 23:34 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!