| opendevreview | Jacob Anders proposed openstack/sushy master: [WIP] Improve resiliency of eTag handling https://review.opendev.org/c/openstack/sushy/+/856123 | 04:02 |
|---|---|---|
| kamlesh6808c | Good Morning Ironic ! | 04:44 |
| kamlesh6808c | Team , can someone help to second review this patch https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/853621 ? | 04:45 |
| opendevreview | Jacob Anders proposed openstack/sushy master: [WIP] Retry BootSourceOverride request when SettingsURI is read-only https://review.opendev.org/c/openstack/sushy/+/856597 | 04:50 |
| opendevreview | Nisha Agarwal proposed openstack/ironic master: Adds create_csr and add_https_certificate clean step https://review.opendev.org/c/openstack/ironic/+/804486 | 05:11 |
| dtantsur | JayF: wait until you learn metal3 uses sqlite in prod :D | 08:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/ironic-ui/+/856697 | 10:27 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/ironic-ui/+/856698 | 10:27 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui master: Update master for stable/zed https://review.opendev.org/c/openstack/ironic-ui/+/856699 | 10:27 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui master: Add Python3 antelope unit tests https://review.opendev.org/c/openstack/ironic-ui/+/856700 | 10:27 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/856705 | 10:27 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/856706 | 10:27 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder master: Update master for stable/zed https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/856707 | 10:27 |
| iurygregory | good morning Ironic | 10:57 |
| dtantsur | morning iurygregory | 11:02 |
| opendevreview | Merged openstack/ironic-ui master: Update master for stable/zed https://review.opendev.org/c/openstack/ironic-ui/+/856699 | 11:12 |
| opendevreview | Merged openstack/ironic-ui stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/ironic-ui/+/856697 | 11:15 |
| opendevreview | Merged openstack/ironic-ui stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/ironic-ui/+/856698 | 11:15 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/856705 | 11:22 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/856706 | 11:22 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Update master for stable/zed https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/856707 | 11:22 |
| opendevreview | Merged openstack/ironic-ui master: Add Python3 antelope unit tests https://review.opendev.org/c/openstack/ironic-ui/+/856700 | 11:25 |
| Nisha_Agarwal | iurygregory, hi | 11:35 |
| Nisha_Agarwal | iurygregory, we fixed broken proliantutils test cases and released it this week(7th Sept). The patch https://review.opendev.org/c/openstack/ironic/+/804486 contains the updated proliantutils version also . | 11:36 |
| iurygregory | Nisha_Agarwal, ack I will take a look =) | 11:40 |
| Nisha_Agarwal | iurygregory, Thanks | 11:40 |
| opendevreview | Jacob Anders proposed openstack/sushy master: [WIP] Retry BootSourceOverride request when SettingsURI is read-only https://review.opendev.org/c/openstack/sushy/+/856597 | 11:46 |
| kubajj | dtantsur: I am writing a short description of the project and Arne is on vacation now. He mentioned that inspector was previously already part of ironic. Why was it taken out? (I would like to explain it in the description.) | 12:49 |
| dtantsur | kubajj: it has never been. It was proposed by me (back in 2014 heh) and rejected because the scope of Ironic was perceived to be much smaller back then. | 12:50 |
| dtantsur | Some team members were also not sure if the idea gains popularity at all | 12:50 |
| kubajj | dtantsur: do you reckon what I could set as a minimal objective? One which I could start with and if things go well build on top of with the other steps. | 13:11 |
| kubajj | Would the addition of inventory API to Ironic be reasonable to begin with? | 13:12 |
| iurygregory | inspector-client why you do this .-. | 13:17 |
| dtantsur | kubajj: yes, this is a self-contained goal that would already be quite helpful (e.g. metal3 will no longer need to access inspector directly) | 13:18 |
| kubajj | dtantsur: if I understand it correctly, it is supposed to list the hardware on the node from IPA to conductor instead of passing it to inspector? | 13:24 |
| kubajj | Would this solve the concern you had about the volume names for software RAID? | 13:24 |
| dtantsur | kubajj: it's not a functional change for now | 14:07 |
| dtantsur | essentially, Ironic will serve as a proxy for Inspector until other items are done | 14:07 |
| JayF | dtantsur: sqlite is a *great* DB as long as you don't need any form of HA whatsoever :D | 14:08 |
| JayF | dtantsur: I was just shocked we were using it in unit tests for apps that run on mysql | 14:08 |
| kubajj | dtantsur: so would Ironic get the information and just pass it to Inspector? | 14:09 |
| dtantsur | JayF: yeah, metal3 is a bit special (the DB is basically ephemeral, rebuilt from kubernetes on the pod start) | 14:09 |
| dtantsur | kubajj: the other way around: inspector gets it now (and will get it until we switch over) | 14:10 |
| JayF | dtantsur: That makes sense. At my older job that used sqlite heavily; it was basically just a normalized cache of things that were already on the system so it was similarly possible to rebuild it | 14:10 |
| kubajj | dtantsur: so this step will provide access for applications like metal3 to read the inventory from Ironic instead of Inspector | 14:11 |
| dtantsur | kubajj: more importantly, it's one step towards inspector no longer being a separate thing | 14:11 |
| JayF | dtantsur: btw; I went to metal3's page looking for it in slack. It linked to the k8s slack, but I couldn't find the channel there | 14:11 |
| dtantsur | kubajj: you see, merging two projects in a lockstep is basically impossible | 14:11 |
| JayF | dtantsur: you have a pointer? | 14:11 |
| dtantsur | JayF: #cluster-api-baremetal | 14:12 |
| dtantsur | slack is terrible, its links may or may not work.. | 14:12 |
| kubajj | dtantsur: I understand. I am just trying to explain what I aim to do to my supervisor. Need to define some success criteria. So I am trying to figure out what should it do when this step is done. | 14:12 |
| JayF | dtantsur: heh, yeah, the link was only to the kubernetes slack when I looked at it, and there's no channel name. You might wanna point some website folks at it :P | 14:13 |
| dtantsur | kubajj: two goals for the 1st step: 1) access to introspection data via the ironic API, 2) possibility to provide introspection data for inspect implementations other than ironic-inspector (redfish, idrac, etc) | 14:13 |
| dtantsur | JayF: I suspect the link does contain it, but it gets lost during auth... or some non-sense like that | 14:14 |
| JayF | ack; well either way I found it thanks :D | 14:14 |
| kubajj | dtantsur: the 2) means that ironic will pass the introspection data to redfish, idrac, etc or do I understand it wrong? | 14:19 |
| dtantsur | kubajj: the other way around again :) ironic is the frontend in all cases | 14:30 |
| kubajj | dtantsur: so ironic handles the baremetal inspect command and then either inspector, redfish or idrac provide the data? | 14:31 |
| dtantsur | kubajj: exactly | 15:03 |
| opendevreview | Radosław Piliszek proposed openstack/tenks master: Use venv instead of virtualenv https://review.opendev.org/c/openstack/tenks/+/856296 | 15:56 |
| dtantsur | have a great weekend folks o/ | 16:08 |
| opendevreview | Merged openstack/networking-generic-switch master: Implements basic functionality for use with Pluribus switches. https://review.opendev.org/c/openstack/networking-generic-switch/+/853427 | 19:04 |
| opendevreview | Merged openstack/tenks master: Use venv instead of virtualenv https://review.opendev.org/c/openstack/tenks/+/856296 | 20:30 |
| opendevreview | Radosław Piliszek proposed openstack/tenks master: [CI] Gate on CentOS Stream 9 and Rocky Linux 9 https://review.opendev.org/c/openstack/tenks/+/856859 | 20:43 |
| opendevreview | Radosław Piliszek proposed openstack/tenks master: Remove CentOS 7 support leftovers https://review.opendev.org/c/openstack/tenks/+/856860 | 20:52 |
| opendevreview | Radosław Piliszek proposed openstack/tenks master: [CI] Remove CentOS 7 support leftovers https://review.opendev.org/c/openstack/tenks/+/856860 | 20:56 |
| opendevreview | Radosław Piliszek proposed openstack/tenks master: [CI] Optimise ovs setup https://review.opendev.org/c/openstack/tenks/+/856861 | 21:14 |
| NobodyCam | Really: | 21:23 |
| NobodyCam | remote: commit 180c0db: warning: subject >50 characters; use shorter first paragraph | 21:23 |
| NobodyCam | remote: commit 0d011ae: warning: subject >50 characters; use shorter first paragraph``` | 21:23 |
| JayF | https://wiki.openstack.org/wiki/GitCommitMessages it's been enforced in CI for a long time iirc | 21:35 |
| JayF | I think my first foray into openstack had me wrestling CI over a git commit message error | 21:35 |
| TheJulia | It is just a warning…. And I think it helps prevent the one line books | 21:37 |
| * TheJulia guesses she is finally awake | 21:37 | |
| opendevreview | Chris Krelle proposed openstack/ironic master: Cleanup submitted SNMP driver code for additional PDUs https://review.opendev.org/c/openstack/ironic/+/840986 | 21:45 |
| opendevreview | Chris Krelle proposed openstack/ironic master: Cleanup submitted SNMP driver code for additional PDUs https://review.opendev.org/c/openstack/ironic/+/840986 | 23:47 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!