| vanou | Regarding unittest (ironic-tox-unit-with-driver-libs) failure on https://review.opendev.org/c/openstack/ironic/+/858896 , I dealt with it when I revert wrong modification for that backport patch. The cause of unit test failure is commit (https://review.opendev.org/c/openstack/ironic/+/839122) is only backported to stable/yoga but not to bugfix/19.0. I incorporate that into | 00:04 |
|---|---|---|
| vanou | https://review.opendev.org/c/openstack/ironic/+/858896. | 00:04 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/xena: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859502 | 01:15 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/wallaby: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859503 | 01:17 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/victoria: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859504 | 01:18 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/wallaby: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859503 | 01:22 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Retry BootSourceOverride request when SettingsURI is read-only https://review.opendev.org/c/openstack/sushy/+/856597 | 01:31 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/wallaby: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859503 | 01:32 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/victoria: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859504 | 01:34 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/ussuri: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859505 | 02:08 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/train: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859506 | 02:18 |
| opendevreview | Vanou Ishii proposed openstack/ironic bugfix/19.0: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859508 | 02:22 |
| opendevreview | Vanou Ishii proposed openstack/ironic bugfix/19.0: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859508 | 02:25 |
| opendevreview | Vanou Ishii proposed openstack/ironic bugfix/18.1: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859510 | 02:30 |
| opendevreview | Vanou Ishii proposed openstack/ironic bugfix/18.1: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859510 | 02:37 |
| vanou | I make series of backport of https://review.opendev.org/c/openstack/ironic/+/839122 . When I made backport patch of https://review.opendev.org/c/openstack/ironic/+/851802, I incorporated change in 839122. However it should have been backported as independent patch. So I made series of backport. | 02:57 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/wallaby: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859503 | 04:20 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/victoria: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859504 | 04:22 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/ussuri: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859505 | 04:24 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/train: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859506 | 04:26 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Retry BootSourceOverride request when SettingsURI is read-only https://review.opendev.org/c/openstack/sushy/+/856597 | 04:57 |
| janders | ^ hopefully the last revision for now... for some reason local tests don't pick up that syntax problem for me :/ | 04:57 |
| *** mat_fechner is now known as matfechner | 05:53 | |
| janders | yeah that worked \o/ | 06:06 |
| jm1 | dtantsur: hello :) i am still fighting with baremetal modules in aoc or rather ironic. requesting a baremetal node from ironic api via /nodes/detail?instance_uuid=XYZ does not return any result although this instance_uuid definitely exists and is returned by ironic. wth? | 06:26 |
| jm1 | dtantsur: example at "2022-09-27 18:19:21.280081" in https://84440920cff20500b44c-3aa94c2a6e1611b5dab8f8c2e7977d1f.ssl.cf2.rackcdn.com/854113/18/check/bifrost-collections-src/04fdeb3/job-output.txt | 06:27 |
| jm1 | dtantsur: first call from openstacksdk to ironic api returns a result/node. But when openstacksdk uses its instance_uuid on /nodes/detail?instance_uuid=XYZ, then ironic does not return any result | 06:28 |
| jm1 | dtantsur: reading ironic's api maybe instance_uuid is not what i want.. | 06:55 |
| kubajj | Good morning Ironic | 07:21 |
| rpittau | good morning ironic! o/ | 07:39 |
| opendevreview | Pierre Riteau proposed openstack/ironic master: Add missing space to log message https://review.opendev.org/c/openstack/ironic/+/859533 | 07:59 |
| opendevreview | Pierre Riteau proposed openstack/ironic master: Add missing space to log message https://review.opendev.org/c/openstack/ironic/+/859533 | 08:00 |
| janders | hey kubajj rpittau and Ironic o/ | 08:34 |
| rpittau | hey janders :) | 08:35 |
| opendevreview | Jakob Meng proposed openstack/bifrost master: [DNM] Debugging calls to openstack.cloud modules https://review.opendev.org/c/openstack/bifrost/+/859430 | 08:46 |
| opendevreview | Merged openstack/ironic bugfix/19.0: CI: disable bfv testing on bugfix/19.0 https://review.opendev.org/c/openstack/ironic/+/859475 | 09:16 |
| opendevreview | Merged openstack/ironic stable/victoria: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/852804 | 09:24 |
| vanou | hi tpittau o/ Thanks for review | 09:25 |
| rpittau | hey vanou :) | 09:26 |
| opendevreview | Merged openstack/ironic stable/train: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/852806 | 09:40 |
| opendevreview | Merged openstack/ironic stable/ussuri: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/852805 | 09:43 |
| dtantsur | jm1: yeah, I guess the first question is: what are you trying to do? | 09:49 |
| jm1 | dtantsur: trying to list all nodes with a specific uuid/id | 09:50 |
| dtantsur | jm1: instance ID or node ID? | 09:51 |
| jm1 | dtantsur: yeah that is the issue. i thought both are the same but after reading the api it looks like one is ironic the other is nova? | 09:51 |
| dtantsur | jm1: yeah, instance ID was created to track the Nova's server ID (can also be used for other purposes) | 09:51 |
| dtantsur | jm1: "listing nodes by node ID" makes little sense: it's unique | 09:52 |
| dtantsur | so just /v1/nodes/<ID>/detail | 09:52 |
| dtantsur | (instance ID is also unique, by you need to go through the listing API indeed) | 09:52 |
| jm1 | jm1: yes but it would have resulted in less code. anyway, i am using exactly what you suggested now :) | 09:53 |
| jm1 | dtantsur: ^ | 09:53 |
| jm1 | dtantsur: and it works :D | 09:53 |
| dtantsur | :) | 09:53 |
| jm1 | dtantsur: ok not really but at least it fails at a different step :D | 09:54 |
| dtantsur | progress \o/ | 09:54 |
| jm1 | dtantsur: looks like your aoc modules require least porting effort because you were already using the resource proxies before | 09:55 |
| dtantsur | nice! | 09:55 |
| jm1 | dtantsur: *porting efforts for users :) | 09:55 |
| jm1 | dtantsur: aka bifrost | 09:55 |
| opendevreview | Merged openstack/ironic stable/wallaby: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/852803 | 09:55 |
| opendevreview | Jakob Meng proposed openstack/bifrost master: [DNM] Debugging calls to openstack.cloud modules https://review.opendev.org/c/openstack/bifrost/+/859430 | 10:25 |
| opendevreview | Merged openstack/ironic master: Add missing space to log message https://review.opendev.org/c/openstack/ironic/+/859533 | 11:19 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/ussuri: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859505 | 11:28 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/train: [iRMC] Change the way to get irmc-info in raid https://review.opendev.org/c/openstack/ironic/+/859506 | 11:29 |
| janders | TheJulia I heard back from Moshe regarding https://etherpad.opendev.org/p/ironic-antelope-ptg#L85 - 7am UTC works for him. Etherpad says "early in the week" so I asked about Monday/Tuesday specifically. | 12:08 |
| iurygregory | good morning Ironic | 12:10 |
| rpittau | hey iurygregory :) | 12:16 |
| janders | hey iurygregory | 12:19 |
| Nisha_Agarwal | hey iurygregory | 12:20 |
| Nisha_Agarwal | TheJulia, Is there any prerequiste for the mirror image to work for anaconda deploy? | 12:21 |
| iurygregory | hey o/ | 12:30 |
| iurygregory | Ironic folks, we have another round for Outreachy , I'm thinking about submitting a project (no idea yet), but I'm wondering if someone would like to be a mentor so we can work together | 12:47 |
| dtantsur | I don't think it's appropriate for me since I'm a coordinator :) derekh__, janders ^^ | 12:55 |
| iurygregory | dtantsur, yeah ofc =) | 12:56 |
| dtantsur | non-RH people are obviously welcome | 12:56 |
| iurygregory | ++ | 12:56 |
| * dtantsur is curious if JayF wants to give it a try | 12:56 | |
| JayF | Honestly, I appreciate the thought but I am mildly welmed at the moment trying to figure out how to TC and PTL | 13:08 |
| dtantsur | fair enough | 13:11 |
| TheJulia | Nisha_Agarwal: I don't know, you'll need to check the anaconda/kickstart documentation. i know initial amdisk/stage2 ramdisk need to be gzip compressed cpio ramdisks. The image for liveimg, I don't remember | 13:41 |
| TheJulia | janders: okay | 13:44 |
| * TheJulia feels super sleepy | 13:44 | |
| TheJulia | iurygregory: I guess I might be up for doing outreachy again | 13:44 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/858896 can and should be landed, especially since it's been backported further than this now | 13:57 |
| JayF | heads up: I'm here today but have been up since the wee-hours of the morning sick with a cold+sore throat. So expect me to be extra-async today. | 13:58 |
| rpittau | as t his one I guess https://review.opendev.org/c/openstack/ironic/+/859482 | 13:59 |
| JayF | rpittau: you land mine and I'll land yours lol | 14:02 |
| rpittau | deal! | 14:02 |
| JayF | In the future it'd be really nice if we landed these in order | 14:03 |
| rpittau | oh I can't land yours, I propsed it :P | 14:03 |
| JayF | Eh, vanou has done most of the editing | 14:03 |
| JayF | I am OK with you putting a +2 on it | 14:03 |
| JayF | then I can workflow it | 14:03 |
| JayF | we have never been super strict about that kind of things for backports | 14:03 |
| rpittau | done | 14:03 |
| iurygregory | TheJulia, ok =) | 14:08 |
| opendevreview | Merged openstack/ironic bugfix/18.1: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/859482 | 15:13 |
| opendevreview | Merged openstack/ironic bugfix/19.0: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/858896 | 15:22 |
| rpittau | bye everyone, have a great evening! o/ | 15:58 |
| opendevreview | Nisha Agarwal proposed openstack/ironic master: Fix the anaconda deploy for the ISO mounted https://review.opendev.org/c/openstack/ironic/+/859397 | 19:01 |
| janders | good morning Ironic o/ | 21:32 |
| opendevreview | Merged openstack/sushy master: Update release versions for yoga and zed https://review.opendev.org/c/openstack/sushy/+/859018 | 22:37 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!