| opendevreview | Merged openstack/metalsmith stable/xena: Write network_data.json metadata to config-dirve https://review.opendev.org/c/openstack/metalsmith/+/810480 | 00:40 |
|---|---|---|
| opendevreview | Merged openstack/metalsmith stable/wallaby: Write network_data.json metadata to config-dirve https://review.opendev.org/c/openstack/metalsmith/+/810481 | 00:58 |
| *** pmannidi is now known as pmannidi|AFK | 01:13 | |
| *** pmannidi|AFK is now known as pmannidi | 03:06 | |
| opendevreview | Jacob Anders proposed openstack/ironic master: Follow up to Add support for verify steps https://review.opendev.org/c/openstack/ironic/+/812792 | 04:38 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Do not append filename parameter to image URL when using local file https://review.opendev.org/c/openstack/ironic/+/812795 | 05:27 |
| arne_wiebalck | Good morning janders and Ironic! | 06:12 |
| iurygregory | good morning arne_wiebalck janders and Ironic o/ | 06:51 |
| arne_wiebalck | he iurygregory o/ | 06:51 |
| rpittau | good morning ironic! o/ | 07:54 |
| iurygregory | good morning rpittau o/ | 07:55 |
| rpittau | hey iurygregory :) | 07:57 |
| rpittau | to whoever is interested in molteniron, I tried to clean the tests a little bit and give it a general better shape https://review.opendev.org/c/openstack/molteniron/+/812520, but it needs a lot more love and hugs | 08:30 |
| iurygregory | rpittau, ack I will add to my list to take a look | 08:37 |
| iurygregory | dtantsur, since in stable/wallaby we have https://opendev.org/openstack/ironic/commit/f43e80411207e9736f88e0deb452aa910015bffe and it has critical in the release notes, do you think we should release as 17.1.0 ? see https://review.opendev.org/c/openstack/releases/+/812599/1/deliverables/wallaby/ironic.yaml | 08:46 |
| dtantsur | morning ironic | 09:01 |
| dtantsur | iurygregory: critical only means that the fix is serious, it does not affect semver | 09:01 |
| iurygregory | dtantsur, ack can you add as a comment in the release patch? | 09:01 |
| dtantsur | link? | 09:02 |
| iurygregory | https://review.opendev.org/c/openstack/releases/+/812599/1/deliverables/wallaby/ironic.yaml | 09:02 |
| dtantsur | done | 09:03 |
| iurygregory | tks! | 09:03 |
| janders | hey arne_wiebalck, iurygregory rpittau dtantsur and Ironic o/ | 09:11 |
| arne_wiebalck | hey janders o/ | 09:12 |
| janders | w/r/t our yesterday discussion about removing "?filename=file.iso" suffix from the vmedia image URL - I've got an early version change up for review: | 09:13 |
| janders | https://review.opendev.org/c/openstack/ironic/+/812795 | 09:13 |
| janders | needs a story and release note, but is this a good direction in general? | 09:13 |
| janders | (local tests passed, I think CI has some problem, I just re-ran it) | 09:14 |
| dtantsur | janders: looks about right | 09:28 |
| janders | thank you dtantsur | 09:28 |
| rpittau | janders: at a glance looks ok | 09:30 |
| janders | thank you rpittau | 09:31 |
| janders | in this case I will create a story reflecting what's been done, add a release note and it should be good to go | 09:31 |
| janders | just getting the FJ PR up first | 09:32 |
| jeeva | i'm getting this error when i do a node validate | 09:34 |
| jeeva | Cannot validate image information for node fc61ba91-24d2-4711-a80e-5631a9d7cc9e because one or more parameters are missing from its instance_info and insufficent information is present to boot from a remote volume. Missing are: ['image_source', 'kernel', 'ramdisk'] | 09:34 |
| jeeva | but | 09:34 |
| jeeva | openstack baremetal node show fc61ba91-24d2-4711-a80e-5631a9d7cc9e -c driver_info -f value | 09:34 |
| jeeva | {'ipmi_port': 623, 'ipmi_username': 'root', 'ipmi_password': '******', 'ipmi_address': '10.102.30.29', 'deploy_kernel': '0d356ee3-e996-4a39-a4ab-f5698f9c4f4a', 'deploy_ramdisk': '13cfdef0-7ed2-4098-a61b-53f5cfdb7214'} | 09:34 |
| jeeva | the images are there | 09:34 |
| dtantsur | jeeva: you need to populate instance_info, you're looking at driver_info though | 09:36 |
| dtantsur | jeeva: https://docs.openstack.org/ironic/latest/user/deploy.html#populating-instance-information | 09:37 |
| opendevreview | Merged openstack/ironic master: Yoga: Change default boot mode to uefi https://review.opendev.org/c/openstack/ironic/+/810249 | 10:04 |
| dtantsur | w00t | 10:07 |
| iurygregory | :O | 10:10 |
| opendevreview | Verification of a change to openstack/ironic master failed: CI: Change CI ipxe file to snponly https://review.opendev.org/c/openstack/ironic/+/812167 | 10:19 |
| opendevreview | Verification of a change to openstack/ironic master failed: Retool devstack plugin to use pxe loaders configuration https://review.opendev.org/c/openstack/ironic/+/812188 | 10:19 |
| iurygregory | default to uefi merged, two changes failed to merge ...coincidence? | 10:20 |
| dtantsur | :D | 10:20 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add and document high-level helpers for async steps https://review.opendev.org/c/openstack/ironic/+/807295 | 10:20 |
| iurygregory | now we need to change the flavor of the cookies we give to zuul... | 10:20 |
| rpittau | well they were passing before and they're on top of the uefi change, so maybe not | 10:21 |
| iurygregory | is just a joke because we are close to friday :D | 10:24 |
| iurygregory | janders, maybe I'm a bit lost, shouldn't https://review.opendev.org/c/openstack/ironic/+/812792 be a follow-up to https://review.opendev.org/c/openstack/ironic/+/800001/ ? | 10:27 |
| janders | iurygregory: well spotted, you are right! | 10:28 |
| janders | must be a copy-paste stuff-up | 10:28 |
| iurygregory | when I clicked in the commit in the commit-message i was directed to https://review.opendev.org/c/openstack/ironic/+/744117 , I was like "what?" | 10:29 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Follow up to Add support for verify steps https://review.opendev.org/c/openstack/ironic/+/812792 | 10:50 |
| janders | iurygregory: should be fixed, sorry about the earlier stuff-up | 10:51 |
| iurygregory | janders, no worries! | 10:51 |
| opendevreview | Verification of a change to openstack/ironic master failed: CI: Change CI ipxe file to snponly https://review.opendev.org/c/openstack/ironic/+/812167 | 11:12 |
| opendevreview | Verification of a change to openstack/ironic master failed: Retool devstack plugin to use pxe loaders configuration https://review.opendev.org/c/openstack/ironic/+/812188 | 11:28 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Update the list of supported database filters https://review.opendev.org/c/openstack/ironic/+/812979 | 11:42 |
| jeeva | [11:37] <dtantsur> jeeva: https://docs.openstack.org/ironic/latest/user/deploy.html#populating-instance-information | 11:45 |
| jeeva | dtantsur, there is no page for ussuri | 11:46 |
| dtantsur | jeeva: it's not much different for ussuri, I think | 11:46 |
| dtantsur | jeeva: I think most of it was in https://docs.openstack.org/ironic/ussuri/install/standalone.html back then | 11:47 |
| jeeva | been following this: https://docs.openstack.org/kolla-ansible/ussuri/reference/bare-metal/ironic-guide.html | 11:53 |
| jeeva | but the only thing i'm able to do is switch the node on & off | 11:53 |
| janders | storyboard is driving me nuts | 12:02 |
| janders | won't let me save the story | 12:02 |
| janders | but won't tell me why | 12:02 |
| janders | do you have any suggestions why this may be happening? | 12:03 |
| janders | oh well, started over, works now | 12:06 |
| janders | go figure | 12:06 |
| dtantsur | a possible reason: you typed the project instead of choosing it from the list :) | 12:13 |
| janders | dtantsur sounds possible :) | 12:13 |
| janders | must have clicked it properly when starting over :) | 12:14 |
| opendevreview | Merged x/sushy-oem-idrac stable/wallaby: Fix IncludeInExport allowed value validation https://review.opendev.org/c/x/sushy-oem-idrac/+/805145 | 12:17 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Do not append filename parameter to image URL when using local file https://review.opendev.org/c/openstack/ironic/+/812795 | 12:24 |
| opendevreview | Merged x/sushy-oem-idrac master: Add Python3 Yoga unit tests https://review.opendev.org/c/x/sushy-oem-idrac/+/811614 | 12:31 |
| janders | see you tomorrow Ironic o/ | 12:50 |
| iurygregory | bye janders o/ | 12:53 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: WIP add a helper for node-based periodics https://review.opendev.org/c/openstack/ironic/+/812495 | 13:14 |
| opendevreview | Verification of a change to openstack/ironic master failed: CI: Change CI ipxe file to snponly https://review.opendev.org/c/openstack/ironic/+/812167 | 14:31 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add a helper for node-based periodics https://review.opendev.org/c/openstack/ironic/+/812495 | 14:45 |
| dtantsur | finally! | 14:45 |
| JayF | dtantsur: that new decorator looks spiffy, nice one! | 14:55 |
| dtantsur | :) | 14:55 |
| *** pmannidi is now known as pmannidi|AFK | 14:56 | |
| iurygregory | the open infra live about Xena was really nice! | 15:02 |
| NobodyCam | Good Morning Ironic folks! | 15:33 |
| rpittau | hey NobodyCam :) | 15:33 |
| NobodyCam | hey hey rpittau top of the morning | 15:34 |
| NobodyCam | :) | 15:34 |
| dtantsur | what's up NobodyCam :) | 15:47 |
| NobodyCam | hey hey dtantsur | 15:47 |
| NobodyCam | having out on a all hands call | 15:48 |
| iurygregory | hey NobodyCam o/ | 15:49 |
| NobodyCam | If anyone has time free for a quick review. https://review.opendev.org/c/openstack/ironic/+/809902 | 15:49 |
| NobodyCam | hey hey iurygregory | 15:49 |
| NobodyCam | Congratulations to all on Xena | 15:49 |
| NobodyCam | awesome work | 15:49 |
| iurygregory | NobodyCam, done =) | 15:52 |
| * NobodyCam looks | 15:52 | |
| dtantsur | fwiw I'm still not convinced by the arbitrary nature of positioning of sleeps.. but I'm not gonna die on that hill | 15:52 |
| NobodyCam | Thank you I have this same patch in our production and it has solved the issues we were encountering | 15:53 |
| iurygregory | dtantsur, don't look at the attestation interface patches =) | 15:54 |
| dtantsur | oh oh | 15:55 |
| dtantsur | NobodyCam: yeah, but if someone invokes the same actions in a different order.. still boom? | 15:55 |
| dtantsur | iurygregory: well, I'm about to enter a long weekend, I'm not willing to spoil the mood :D | 15:55 |
| iurygregory | dtantsur, yup! | 15:56 |
| iurygregory | enjoy your long weekend with your wife ++ | 15:56 |
| NobodyCam | :) | 15:56 |
| rpittau | bye everyone, talk tomorrow o/ | 15:57 |
| iurygregory | bye rpittau o/ | 15:57 |
| NobodyCam | Night rpittau | 15:57 |
| NobodyCam | dtantsur: with this patch we have stopped seeing any errors from the snmp controlled nodes. so from what we have been seeing I would say no to the still boom question.. | 15:58 |
| dtantsur | NobodyCam: yeah, because you rely on the exact order the conductor call driver methods | 15:59 |
| dtantsur | which works, but is honestly a layer violation | 15:59 |
| dtantsur | anyway, I'll leave it here and will gradually move to the weekend :) | 16:03 |
| dtantsur | see you on Monday folks | 16:03 |
| *** hjensas is now known as hjensas|afk | 16:12 | |
| opendevreview | Verification of a change to openstack/ironic master failed: Demote three warning messages https://review.opendev.org/c/openstack/ironic/+/812643 | 16:17 |
| opendevreview | Verification of a change to openstack/ironic master failed: add snmp power action delay https://review.opendev.org/c/openstack/ironic/+/809902 | 17:36 |
| opendevreview | Verification of a change to openstack/ironic master failed: Demote three warning messages https://review.opendev.org/c/openstack/ironic/+/812643 | 18:56 |
| NobodyCam | is it `recheck` for merge gate jobs?? | 20:28 |
| JayF | recheck does it all now :) | 20:52 |
| JayF | no more reverify, not for years | 20:52 |
| janders_ | good morning Ironic o/ | 22:34 |
| *** janders_ is now known as janders | 22:37 | |
| janders | if any of the cores have time: I'm after the second +2 for https://review.opendev.org/c/openstack/ironic/+/812792 | 22:52 |
| janders | ^ is just addressing the comments from https://review.opendev.org/c/openstack/ironic/+/800001/38/ironic/conductor/verify.py#70 and https://review.opendev.org/c/openstack/ironic/+/800001/38/ironic/conf/conductor.py#346 | 22:52 |
| janders | thank you iurygregory | 23:01 |
| iurygregory | janders, np =) | 23:02 |
| *** pmannidi|AFK is now known as pmannidi | 23:14 | |
| NobodyCam | Morning janders | 23:49 |
| janders | hey NobodyCam o/ | 23:49 |
| NobodyCam | Hey hey JayF Thank you ! | 23:50 |
| NobodyCam | o/ janders | 23:51 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!