| opendevreview | ZhouHao proposed openstack/ironic master: iRMC: Fix instructions for boot interface https://review.opendev.org/c/openstack/ironic/+/828448 | 03:06 |
|---|---|---|
| arne_wiebalck | Good morning, Ironic! | 07:48 |
| dtantsur | morning folks | 08:14 |
| rpittau | good morning ironic! o/ | 08:15 |
| MahnoorAsghar | o/ | 08:53 |
| *** sshnaidm|afk is now known as sshnaidm | 08:54 | |
| MahnoorAsghar | Good morning Ironic! | 08:59 |
| rpittau | dtantsur: about the ussuri jobs in ipa-builder, we still run them in ipa ussuri branch, doesn't look like we are very committed to that branch since it went EM, but maybe we can leave check-image and build-image for the time being ? | 09:01 |
| rpittau | hey MahnoorAsghar :) | 09:01 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Try to detect the default user based on image name https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/828492 | 09:01 |
| dtantsur | rpittau: I would expect us to use IPA-builder stable/wallaby in ussuri? | 09:01 |
| dtantsur | otherwise you're breaking this job with your tinyipa 13 change | 09:02 |
| rpittau | dtantsur: there's an override-checkout to ipa-builder 2.2.0, is it begin ignored ? | 09:03 |
| rpittau | being* | 09:03 |
| dtantsur | rpittau: should work, but it won't work when running on IPA-builder master | 09:03 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: CI: use a custom centos partition image instead of cirros https://review.opendev.org/c/openstack/ironic/+/827882 | 09:04 |
| MahnoorAsghar | I am getting some CI errors on my patch (https://review.opendev.org/c/openstack/ironic/+/827200) that I dont understand... | 09:04 |
| rpittau | yeah, in the ussuri job itself should work though | 09:04 |
| dtantsur | MahnoorAsghar: you mean https://zuul.opendev.org/t/openstack/build/c8f9a64bde9d43e282e1d461920a23c2/log/job-output.txt#2041 ? it may be parameters vs param | 09:05 |
| MahnoorAsghar | Specifically, there is one Zuul one, one from HPE Proliant and one from Fujitsu iRMC CI | 09:05 |
| dtantsur | ignore the 3rd party CI, only the docs and api-ref ones is relevant for you | 09:06 |
| MahnoorAsghar | dtantsur: No, Ive added a new directive called 'Parameters', thats a reasonable error | 09:06 |
| dtantsur | ah | 09:06 |
| dtantsur | well, the docs job does not know about your extension | 09:06 |
| MahnoorAsghar | okay! What about one Zuul error - https://zuul.opendev.org/t/openstack/build/5eed26ff751f4bc8aa262afa105298ad | 09:06 |
| MahnoorAsghar | Yes..its not possible to tell jobs about my extension via the patch itself, right? | 09:07 |
| dtantsur | this may be problematic, but it's not related to your patch. | 09:07 |
| MahnoorAsghar | ah, okay | 09:07 |
| dtantsur | MahnoorAsghar: you modify api-ref/conf.py but not docs/conf.py | 09:08 |
| dtantsur | there are different, unless you change it somehow | 09:08 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Remove ussuri job https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/828495 | 09:08 |
| MahnoorAsghar | I wasnt looking at docs/conf.py, let me check | 09:08 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix the benchmark job https://review.opendev.org/c/openstack/ironic/+/828498 | 09:12 |
| MahnoorAsghar | I am trying to make an extension for API documentation, being generated from docstrings | 09:13 |
| MahnoorAsghar | The doc/ sphinx setup looks like its generating some other user documentation | 09:14 |
| MahnoorAsghar | oh, youre saying I should try to combine the sources of documentation? | 09:15 |
| dtantsur | MahnoorAsghar: it does, but it also generates Python API documentation, and it sees your directive | 09:15 |
| dtantsur | I'm not sure about combining at this point.. may be too radical | 09:16 |
| MahnoorAsghar | ah | 09:16 |
| MahnoorAsghar | alright | 09:16 |
| MahnoorAsghar | i should modify its conf too then | 09:16 |
| *** mnasiadka_ is now known as mnasiadka | 09:18 | |
| MahnoorAsghar | thank you! | 09:18 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Fix dib ipa jobs https://review.opendev.org/c/openstack/bifrost/+/827293 | 09:23 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Support listening on a Unix socket https://review.opendev.org/c/openstack/ironic/+/828508 | 09:59 |
| opendevreview | Mark Goddard proposed openstack/tenks master: docs: linux bridge, supported OS distros https://review.opendev.org/c/openstack/tenks/+/828278 | 10:01 |
| opendevreview | Mark Goddard proposed openstack/tenks master: CI: switch Ubuntu jobs to Focal 20.04 https://review.opendev.org/c/openstack/tenks/+/828509 | 10:01 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Make ironic-inspector listen on a Unix socket https://review.opendev.org/c/openstack/bifrost/+/827128 | 10:01 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Make ironic and inspector listen on Unix sockets https://review.opendev.org/c/openstack/bifrost/+/827128 | 10:07 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Support listening on a Unix socket https://review.opendev.org/c/openstack/ironic/+/828508 | 10:33 |
| iurygregory | good morning Ironic o/ | 11:27 |
| dtantsur | morning iurygregory! | 11:39 |
| dtantsur | I think we should be more or less ready with a bifrost release | 11:39 |
| iurygregory | nice I'm checking releasenotes today for all projects, since I finally have my gerrit working again :D | 12:11 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: CI: use a custom centos partition image instead of cirros https://review.opendev.org/c/openstack/ironic/+/827882 | 12:17 |
| ajya | Hi dtantsur, when you have time, can you take a look at the comments I left at https://review.opendev.org/c/openstack/ironic/+/825753 ? | 12:47 |
| * dtantsur looking | 12:47 | |
| ajya | thanks | 12:59 |
| TheJulia | good morning | 14:08 |
| *** akahat|rover is now known as akahat|PTO | 14:11 | |
| dtantsur | morning TheJulia | 14:12 |
| dtantsur | "Looking for partition ephemeral0 mounted on /mnt", why do we even assume it's mounted? only because of cirros? | 14:14 |
| TheJulia | where do we do this? | 14:14 |
| dtantsur | in our tempest tests | 14:14 |
| TheJulia | hmm | 14:14 |
| TheJulia | that likely needs to die then | 14:15 |
| dtantsur | I think I'll lose the rest of my sanity on trying to make anything that is not cirros working... | 14:15 |
| TheJulia | or is an artifact of age old partition image deploys | 14:15 |
| dtantsur | https://opendev.org/openstack/ironic-tempest-plugin/src/branch/master/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py#L103-L117 | 14:15 |
| dtantsur | I can only assume cirros auto-mounts it... | 14:16 |
| TheJulia | https://opendev.org/openstack/ironic-tempest-plugin/src/branch/master/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py#L208-L209 | 14:17 |
| TheJulia | yeah, it must | 14:17 |
| TheJulia | so, my thought is just not define a size anymore by default | 14:17 |
| dtantsur | well, I can fix it to add an actual mount | 14:18 |
| TheJulia | you could as well | 14:18 |
| TheJulia | "hey, is this thing here" "oh, mount it" | 14:18 |
| TheJulia | I remember early ec2 instances sometimes mounted ephemeral disks and sometimes didn't. It entirely depended on if it was an official image from AWS or not | 14:19 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Make the basic_ops tests friendlier to non-Cirros images https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/828492 | 14:26 |
| dtantsur | okay, let's try this | 14:26 |
| * arne_wiebalck just handled the first delivery which had no redfish surprises (so far) | 14:35 | |
| opendevreview | Verification of a change to openstack/bifrost master failed: Enable epel repository only when needed https://review.opendev.org/c/openstack/bifrost/+/826442 | 14:39 |
| TheJulia | curious... https://review.opendev.org/c/openstack/ironic/+/828498 didn't run the benchmark job | 15:02 |
| opendevreview | Verification of a change to openstack/bifrost master failed: Remove deprecated os_ironic_facts https://review.opendev.org/c/openstack/bifrost/+/826308 | 15:03 |
| dtantsur | TheJulia: mmm, yeah, ./tools is excluded in bifrost... | 15:06 |
| iurygregory | yeah tools is excluded in bifrost-base | 15:07 |
| iurygregory | https://opendev.org/openstack/bifrost/src/branch/master/zuul.d/bifrost-jobs.yaml#L16 | 15:07 |
| iurygregory | <facepalm> | 15:07 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix the benchmark job https://review.opendev.org/c/openstack/ironic/+/828498 | 15:49 |
| opendevreview | Merged openstack/bifrost master: Add dhcp, vmedia and dibipa CentOS Stream 9 jobs https://review.opendev.org/c/openstack/bifrost/+/827154 | 16:28 |
| opendevreview | Verification of a change to openstack/bifrost master failed: Do not clone repositories that are not used https://review.opendev.org/c/openstack/bifrost/+/826292 | 16:28 |
| opendevreview | Merged openstack/bifrost master: CI: properly publish artifacts for the upgrade job https://review.opendev.org/c/openstack/bifrost/+/827688 | 16:57 |
| rpittau | good night! o/ | 17:19 |
| opendevreview | Arne Wiebalck proposed openstack/ironic master: [trivial] Fix typo in policy error message https://review.opendev.org/c/openstack/ironic/+/828593 | 17:22 |
| arne_wiebalck | TheJulia: it seems that (at least in our current deployment) the default policies reject nova's request every minute to list all nodes ... is that expected? | 17:29 |
| * arne_wiebalck is currently looking into moving to the new default policies as a preparation for Xena | 17:29 | |
| TheJulia | arne_wiebalck: not really expected | 17:38 |
| TheJulia | arne_wiebalck: what role/project does the account nova is using have? If your enforcing the new rbac model, it has to be a system reader to see everything. | 17:39 |
| TheJulia | although, likely system member is what is needed | 17:39 |
| arne_wiebalck | TheJulia: yeah ... I will check | 17:39 |
| arne_wiebalck | TheJulia: yeah, it seems project scoped | 17:43 |
| arne_wiebalck | TheJulia: it is an admin there and I think we have some special rules in place to make it an admin with the old policies | 17:45 |
| arne_wiebalck | TheJulia: thanks, I'll take it from here :) | 17:45 |
| parasiti1 | hi everyone, i've read the doc about portgroups in ironic. It's not very convenient to have to pre create the port group on the switch manually. is there a way to make the information flow up to the generic-agent ml2 plugin and try to realize the portgroup within a port pre commit hook or something ? or is this known to be a deadend ? | 17:53 |
| opendevreview | Merged openstack/bifrost master: Remove deprecated os_ironic_facts https://review.opendev.org/c/openstack/bifrost/+/826308 | 18:16 |
| opendevreview | Verification of a change to openstack/bifrost master failed: Enable epel repository only when needed https://review.opendev.org/c/openstack/bifrost/+/826442 | 19:10 |
| stevebaker[m] | good morning | 19:40 |
| TheJulia | good morning stevebaker[m] | 19:40 |
| opendevreview | Merged openstack/ironic master: [trivial] Fix typo in policy error message https://review.opendev.org/c/openstack/ironic/+/828593 | 22:28 |
| opendevreview | Vanou Ishii proposed openstack/ironic-python-agent master: Rescan device after filesystem creation https://review.opendev.org/c/openstack/ironic-python-agent/+/827936 | 23:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!