| iurygregory | JayF, do you mind if I just +W ? | 00:42 |
|---|---|---|
| iurygregory | seems it seems a very trivial change | 00:43 |
| opendevreview | Merged openstack/virtualpdu master: Fix gitreview; Minor update to readme https://review.opendev.org/c/openstack/virtualpdu/+/880894 | 02:26 |
| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] Handle MissingAttributeError when using OOB inspections to fetch MACs https://review.opendev.org/c/openstack/ironic/+/880575 | 04:27 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Handle MissingAttributeError when using OOB inspections to fetch MACs https://review.opendev.org/c/openstack/ironic/+/880575 | 04:39 |
| rpittau | good morning ironic! o/ | 07:09 |
| opendevreview | Riccardo Pittau proposed openstack/virtualpdu master: Remove ignore F811 flake8 rule https://review.opendev.org/c/openstack/virtualpdu/+/880913 | 07:37 |
| opendevreview | Riccardo Pittau proposed openstack/virtualpdu master: Use hacking 5.x https://review.opendev.org/c/openstack/virtualpdu/+/880914 | 07:38 |
| opendevreview | Riccardo Pittau proposed openstack/virtualpdu master: Remove useless config files https://review.opendev.org/c/openstack/virtualpdu/+/880915 | 07:40 |
| opendevreview | Riccardo Pittau proposed openstack/virtualbmc master: Use generic openstack python3 tests https://review.opendev.org/c/openstack/virtualbmc/+/880916 | 07:43 |
| opendevreview | Riccardo Pittau proposed openstack/virtualpdu master: Add doc8 test to pep8 https://review.opendev.org/c/openstack/virtualpdu/+/880917 | 07:46 |
| Sandzwerg[m] | Morning Ironic | 09:17 |
| Sandzwerg[m] | Has anyone experience with Fujitsu hardware? | 09:18 |
| opendevreview | Mark Goddard proposed openstack/tenks master: Fix CI failures https://review.opendev.org/c/openstack/tenks/+/880866 | 09:33 |
| opendevreview | Mark Goddard proposed openstack/tenks master: Add retries for get_url and package tasks https://review.opendev.org/c/openstack/tenks/+/880759 | 09:33 |
| kaloyank | morning ironic o/ | 09:58 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-specs master: Amend the inspector merger spec with details about PXE filters https://review.opendev.org/c/openstack/ironic-specs/+/880941 | 11:01 |
| dtantsur | hjensas, TheJulia ^^^ | 11:01 |
| dtantsur | Also looking for a 2nd review on https://review.opendev.org/c/openstack/ironic-specs/+/878230 (inspection rules). arne_wiebalck, if you're online, you're probably the best person to check it. | 11:04 |
| hjensas | dtantsur: ++ Thanks, that sounds reasonable to me. | 11:13 |
| iurygregory | morning Ironic | 11:38 |
| opendevreview | Verification of a change to openstack/ironic master failed: Handle MissingAttributeError when using OOB inspections to fetch MACs https://review.opendev.org/c/openstack/ironic/+/880575 | 13:09 |
| iurygregory | dtantsur, tks for the review, I've added my thoughts based on what I understood from your comments o/, if you have more questions or ideas on how I can improve the spec let me know | 13:16 |
| opendevreview | Verification of a change to openstack/ironic master failed: Handle MissingAttributeError when using OOB inspections to fetch MACs https://review.opendev.org/c/openstack/ironic/+/880575 | 13:53 |
| opendevreview | Mark Goddard proposed openstack/tenks master: Fix CI failures https://review.opendev.org/c/openstack/tenks/+/880866 | 15:15 |
| opendevreview | Mark Goddard proposed openstack/tenks master: Add retries for get_url and package tasks https://review.opendev.org/c/openstack/tenks/+/880759 | 15:15 |
| rpittau | good night! o/ | 16:02 |
| NobodyCam | Night rpittau | 16:41 |
| NobodyCam | good morning Ironic Folks | 16:41 |
| JayF | iurygregory: Generally we only need one core review for a docs change proposed by a core. So thanks 👍👍 | 16:42 |
| NobodyCam | anyone have time to criticize https://review.opendev.org/c/openstack/ironic/+/880165/7 | 16:43 |
| NobodyCam | ;) | 16:43 |
| NobodyCam | hey hey JayF wasn't there a tag I should add | 16:43 |
| NobodyCam | and OFC Good Morning | 16:44 |
| JayF | Hashtag ironic-week-prio puts a change in our priority review dashboard. Please only place this on patches that are passing CI and ready to merge. | 16:45 |
| JayF | I am not feeling great today, migraine and nausea. Don't expect me to be around much. | 16:47 |
| dtantsur | get better JayF! | 16:47 |
| dtantsur | I'm migrating my IRC bouncer right now, will be offline for hopefully just a few minutes | 16:48 |
| dtantsur | :qa | 16:48 |
| dtantsur | mmm, no, not this command :D | 16:48 |
| dtantsur | I think I'm back | 16:55 |
| frickler | not directly an ironic issue, but my supermicro boxes seem to permanently change their boot order even if requesting just a one time boot override. has anyone seen that and maybe knows a solution? I verified that this also happens when talking directly to the redfish API | 17:26 |
| dtantsur | Heard that, not aware of any workarounds. | 17:27 |
| dtantsur | In theory, Ironic could enforce a specific boot device, but this feature seems to be highly vendor-specific. | 17:27 |
| TheJulia | frickler: dtantsur: I think we need to start a list, it seems we’re getting observations and questions of most weird super micro behavior. If we can pin down details/patters, there are folks we can reach out to | 21:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!