| opendevreview | Jacob Anders proposed openstack/sushy master: Re-trying InsertMedia call with TransferProtocolType if required https://review.opendev.org/c/openstack/sushy/+/830322 | 00:28 |
|---|---|---|
| *** pmannidi is now known as pmannidi|Away | 02:10 | |
| *** pmannidi|Away is now known as pmannidi | 04:07 | |
| *** pmannidi is now known as pmannidi|Away | 06:23 | |
| opendevreview | Jacob Anders proposed openstack/sushy master: Retry setting BootSourceOverride with alternate URI if required https://review.opendev.org/c/openstack/sushy/+/830553 | 07:04 |
| arne_wiebalck | Good morning, Ironic! | 07:19 |
| rpittau | good morning ironic! o/ | 07:42 |
| arne_wiebalck | Good morning rpittau o/ | 08:12 |
| rpittau | hey arne_wiebalck :) | 08:13 |
| MahnoorAsghar | o/ | 08:50 |
| arne_wiebalck | TheJulia: I did some tests with the policy generator and left a comment on the change. | 09:09 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Retry setting BootSourceOverride with alternate URI if required https://review.opendev.org/c/openstack/sushy/+/830553 | 09:21 |
| janders | hey rpittau arne_wiebalck MahnoorAsghar and Ironic o/ | 10:00 |
| arne_wiebalck | hey janders o/ | 10:00 |
| janders | arne_wiebalck do I recall correctly I remember you fighting some eTag issues not too long ago? | 10:01 |
| arne_wiebalck | janders: yes | 10:01 |
| arne_wiebalck | janders: more than one even, I think :) | 10:01 |
| janders | does this issue/workaround ring a bell or is this a brand new (and possibly BMC-side) stuffup: https://review.opendev.org/c/openstack/sushy/+/830553/2/sushy/resources/system/system.py#290 | 10:01 |
| arne_wiebalck | janders: what is the incorrect etag which is returned? | 10:03 |
| arne_wiebalck | janders: let me find my patch ... | 10:03 |
| arne_wiebalck | janders: https://review.opendev.org/c/openstack/sushy/+/818110 | 10:04 |
| janders | I briefly looked at this code while trying to figure out what is going wrong | 10:05 |
| janders | in my case, if I use the "mainstream" Ironic code I get a number that looks like an eTag but the BMC does not like it | 10:06 |
| janders | if I replicate what I do with manual curl commands (which is the workaround in my patch) it works fine | 10:07 |
| arne_wiebalck | what about dtantsur 's comment? | 10:07 |
| arne_wiebalck | if the url changes, the etag changes and you have to refetch it, no? | 10:08 |
| janders | yeah that would explain the difference | 10:10 |
| janders | thank you for looking into this arne_wiebalck | 10:10 |
| janders | exotic BMCs.. always keep us entertained | 10:10 |
| arne_wiebalck | for some definition of entertainment at least :-D | 10:11 |
| janders | :D | 10:12 |
| janders | un-tertainment perhaps | 10:13 |
| arne_wiebalck | that is talk title material | 10:15 |
| arne_wiebalck | "Fishy BMC un-tertainment" | 10:15 |
| opendevreview | Merged openstack/ironic bugfix/19.0: CI: force config drive on the multinode job's subnode https://review.opendev.org/c/openstack/ironic/+/830194 | 10:20 |
| janders | arne_wiebalck +1! | 10:22 |
| dtantsur | morning folks | 10:41 |
| opendevreview | Merged openstack/ironic stable/wallaby: CI: force config drive on the multinode job's subnode https://review.opendev.org/c/openstack/ironic/+/830195 | 10:44 |
| opendevreview | Verification of a change to openstack/ironic stable/wallaby failed: Build the new cirros image even when netboot is the default https://review.opendev.org/c/openstack/ironic/+/830353 | 10:50 |
| rpittau | dtantsur, TheJulia, iurygregory: I was looking at the projects mentioned in http://lists.openstack.org/pipermail/openstack-discuss/2022-February/027379.html, I can take care of asking releases for them, some are ready, some are waiting for 1-2 patches to merge | 10:54 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Remove unicode from code https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/825322 | 11:01 |
| dtantsur | rpittau: other than builder, everything else can be safely released | 11:02 |
| dtantsur | the builder can be released as well, but we'll probably have another release | 11:02 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Deprecate instance network boot https://review.opendev.org/c/openstack/ironic/+/828746 | 11:15 |
| janders | see you tomorrow Ironic o/ | 11:35 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy-tools master: Fewer mandatory methods in AbstractSystemsDriver https://review.opendev.org/c/openstack/sushy-tools/+/830157 | 11:48 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy-tools master: [WIP] Add a fake driver https://review.opendev.org/c/openstack/sushy-tools/+/830598 | 11:48 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: dhcp-all-interfaces: let NetworkManager doit. https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/825986 | 12:49 |
| hjensas | review please, on Wallaby backport https://review.opendev.org/c/openstack/ironic-lib/+/830203, thanks. | 12:52 |
| opendevreview | Merged openstack/networking-generic-switch master: Support multiple links in link_local_information dict of portgroups https://review.opendev.org/c/openstack/networking-generic-switch/+/557051 | 12:56 |
| hjensas | thanks rpittau | 13:16 |
| rpittau | hjensas: np :) | 13:17 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy-tools master: [WIP] Add a fake driver https://review.opendev.org/c/openstack/sushy-tools/+/830598 | 13:50 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Use pycdlib to extract deploy iso https://review.opendev.org/c/openstack/ironic/+/819121 | 13:57 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Use pycdlib to extract deploy iso https://review.opendev.org/c/openstack/ironic/+/819121 | 14:41 |
| opendevreview | Verification of a change to openstack/ironic-lib stable/wallaby failed: Add platform:base-py[2|3] to bindep https://review.opendev.org/c/openstack/ironic-lib/+/830203 | 14:44 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy-tools master: Add a fake system driver https://review.opendev.org/c/openstack/sushy-tools/+/830598 | 15:03 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Use pycdlib to extract deploy iso https://review.opendev.org/c/openstack/ironic/+/819121 | 15:05 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Use pycdlib to extract deploy iso https://review.opendev.org/c/openstack/ironic/+/819121 | 15:21 |
| rpittau | ^ this should be ready for reviews now | 15:22 |
| * rpittau needs tea | 15:22 | |
| *** gmann is now known as gmann_afk | 15:59 | |
| dtantsur | see you tomorrow folks o/ | 16:02 |
| arne_wiebalck | bye dtantsur o/ | 16:03 |
| *** gmann_afk is now known as gmann | 16:17 | |
| arne_wiebalck | TheJulia: I finally managed to have a look at the inspector errors: the it was due to TLS required in our haproxy (while the container uses the IP/hostname and http). Once fixed, we could scale up to 400 inspections in parallel. And we only stopped since we ran out of machines to test with ... and were afraid to break other things :) https://usercontent.irccloud-cdn.com/file/3NhOdLML/inspector_scaling.png | 16:55 |
| rpittau | good night! o/ | 17:07 |
| TheJulia | arne_wiebalck: Excellent! | 17:07 |
| opendevreview | Radosław Piliszek proposed openstack/tenks master: [WIP] Add Debian support https://review.opendev.org/c/openstack/tenks/+/830675 | 18:02 |
| arne_wiebalck | bye everyone, see you tomorrow o/ | 18:12 |
| opendevreview | Radosław Piliszek proposed openstack/tenks master: Use ansible_facts to reference facts https://review.opendev.org/c/openstack/tenks/+/830182 | 18:17 |
| opendevreview | Radosław Piliszek proposed openstack/tenks master: CI: switch Ubuntu jobs to Focal 20.04 https://review.opendev.org/c/openstack/tenks/+/828509 | 18:17 |
| opendevreview | Radosław Piliszek proposed openstack/tenks master: docs: linux bridge, supported OS distros https://review.opendev.org/c/openstack/tenks/+/828278 | 18:17 |
| opendevreview | Radosław Piliszek proposed openstack/tenks master: [WIP] Add Debian support https://review.opendev.org/c/openstack/tenks/+/830675 | 18:18 |
| opendevreview | Merged openstack/ironic stable/xena: CI: force config drive on the multinode job's subnode https://review.opendev.org/c/openstack/ironic/+/830193 | 18:52 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/wallaby: Create random character devices so systemd does not crash https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/830274 | 19:25 |
| stevebaker[m] | Good morning | 19:57 |
| stevebaker[m] | TheJulia: Hey I think the rhel-9 image build fixes are ready to go if you have some review time https://review.opendev.org/c/openstack/diskimage-builder/+/829620/3 https://review.opendev.org/c/openstack/diskimage-builder/+/828617/14 | 19:58 |
| TheJulia | stevebaker[m]: good morning | 20:10 |
| TheJulia | stevebaker[m]: I posted a question to the first one | 20:17 |
| stevebaker[m] | TheJulia: Thanks. I replied. Vaguely | 20:25 |
| TheJulia | stevebaker[m]: the second one, that looks like it would no longer handle just a partition image, is that right? | 20:25 |
| stevebaker[m] | TheJulia: it should do, the block device would show up as loop0p1 and ROOT_LOOPDEV would get set | 20:27 |
| TheJulia | uhh, that doesn't sound right | 20:28 |
| TheJulia | because a filesystem on a block device, without a partition table should be loop0 | 20:29 |
| * TheJulia tries | 20:29 | |
| stevebaker[m] | TheJulia: also at this point I think extract-image is only used by rpm based images, and none of them are partition images | 20:31 |
| TheJulia | okay | 20:31 |
| TheJulia | confirmed locally, a filesystem only image does go directlly to loop[x], no aprtition | 20:32 |
| TheJulia | stevebaker[m]: +2'd | 20:33 |
| TheJulia | stevebaker[m]: do you have a couple minutes to talk through a tripleo-ism or two? | 20:33 |
| stevebaker[m] | TheJulia: sure thing | 20:33 |
| TheJulia | https://meet.google.com/bma-tdwu-xyk | 20:34 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/wallaby: Remove centos7 job, python2 support https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/826199 | 22:02 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/wallaby: Move centos9 jobs from experimental to check https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/826200 | 22:06 |
| TheJulia | stevebaker[m]: posted the change to release 2.7.1 for wallaby | 22:16 |
| stevebaker[m] | \o/ | 22:17 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
