| *** pmannidi|AFK is now known as pmannidi | 01:29 | |
| opendevreview | Jacob Anders proposed openstack/sushy master: Change defaults - optional insert_media attributes https://review.opendev.org/c/openstack/sushy/+/802452 | 01:40 |
|---|---|---|
| opendevreview | Jacob Anders proposed openstack/ironic master: Make reset_idrac and clear_job_queue verify steps https://review.opendev.org/c/openstack/ironic/+/804032 | 01:52 |
| *** pmannidi is now known as pmannidi|brb | 04:11 | |
| opendevreview | Jacob Anders proposed openstack/ironic master: Make reset_idrac and clear_job_queue verify steps https://review.opendev.org/c/openstack/ironic/+/804032 | 04:26 |
| *** pmannidi|brb is now known as pmannidi | 05:42 | |
| iurygregory | good morning Ironic | 06:10 |
| janders | hey iurygregory | 06:11 |
| iurygregory | hey janders o/ | 06:13 |
| arne_wiebalck | Good iurygregory janders and Ironic! | 06:19 |
| janders | hey arne_wiebalck o/ | 06:19 |
| iurygregory | hey arne_wiebalck p/ | 06:24 |
| *** pmannidi is now known as pmannidi|mtg | 06:30 | |
| *** sshnaidm|afk is now known as sshnaidm | 06:34 | |
| *** pmannidi|mtg is now known as pmannidi|brb | 07:29 | |
| *** rpittau|afk is now known as rpittau | 07:51 | |
| rpittau | good morning ironic! o/ | 07:51 |
| rpittau | Happy Monday! | 07:52 |
| iurygregory | morning rpittau o/ | 07:53 |
| *** pmannidi|brb is now known as pmannidi | 07:54 | |
| rpittau | hey iurygregory :) | 07:58 |
| dtantsur | good morning ironic | 08:43 |
| rpittau | morning dtantsur :) | 08:43 |
| iurygregory | good morning dtantsur =) | 09:15 |
| * iurygregory is wondering if we will switch to DPL for the next cycle | 09:18 | |
| dtantsur | we may if nobody takes the place | 09:21 |
| iurygregory | yeah | 09:22 |
| janders | ajya dtantsur rpittau thank you for your reviews/+1/+2s on https://review.opendev.org/c/openstack/sushy/+/802452 | 09:37 |
| janders | dtantsur w/r/t https://review.opendev.org/c/openstack/sushy/+/802452/15/releasenotes/notes/change-vmedia-insert-default-parameters-48308dbfa4bdb5a8.yaml#12 I think if we had the current Ironic behaviour and this sushy patch in at the same time, this would un-do the SuperMicro fix | 09:38 |
| janders | (Ironic would send True,True which would make sushy put True, True in the payload which would revert things to the state before the stable fix merged) | 09:38 |
| janders | should I look into a follow-up addresing this? | 09:39 |
| janders | (no obvious ideas how to address this off the top of my head, but wanted to get your thoughts before I start going deeper into this) | 09:40 |
| dtantsur | ouch | 09:49 |
| * dtantsur removes W+1 | 09:49 | |
| dtantsur | yeah, I don't think we should do it then. there is a non-zero change that sushy will be upgraded before Ironic | 09:49 |
| dtantsur | then I'd flip write_protected to True and call it a day on the sushy side | 09:50 |
| dtantsur | then in the next release we may consider None/None (after Ironic is updated, not before) | 09:50 |
| dtantsur | janders: ^^ | 09:50 |
| janders | dtantsur: ACK | 09:54 |
| janders | so - do we park this change for the time being and then I create another one flipping write_protected to true and we just do that one? | 09:55 |
| janders | (and then we merge this one some time in the future)? | 09:55 |
| dtantsur | seems so | 09:57 |
| janders | dtantsur: ACK | 10:03 |
| janders | on it | 10:03 |
| janders | dtantsur: good pickup, thanks for that (and sorry about the mess) | 10:05 |
| janders | I did think through the mechanics of this, but did not think enough about the N+1 / N-1 scenario | 10:05 |
| janders | this shows I haven't been working on operations for a while now :) | 10:06 |
| dtantsur | :) | 10:10 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Add better error messages for invalid conf molds https://review.opendev.org/c/openstack/ironic/+/805222 | 10:12 |
| janders | I'm hitting this while trying to run tests on a freshly checked out sushy/master (most likely any repo will do that): https://paste.opendev.org/show/808241/ | 10:22 |
| janders | looks like some version stuff-up.. have you seen this, is there a workaround? | 10:23 |
| rpittau | janders: looking, I just ran tests on different repos and didn't see it | 10:23 |
| janders | rpittau: most likely it's to do with my system | 10:23 |
| rpittau | janders: just ran on sushy and no issues | 10:24 |
| iurygregory | same here .-. | 10:24 |
| iurygregory | no issues | 10:24 |
| janders | hmm I will create a dummy user, git clone and try again | 10:25 |
| janders | wonder if it's some rubbish somewhere in my home directory | 10:25 |
| janders | nah, same, must be system-global | 10:27 |
| rpittau | janders: I use vagrant vms to run tests when in doubt, or containers; helps reproducing clean CI environments | 10:27 |
| janders | I will investigate after dinner | 10:27 |
| janders | maybe operator's blood sugar level is a part of the problem | 10:27 |
| rpittau | :) | 10:27 |
| janders | :) | 10:27 |
| janders | thanks rpittau :) | 10:27 |
| opendevreview | Merged openstack/ironic master: rfc3986: Replace deprecated URIReference.is_valid https://review.opendev.org/c/openstack/ironic/+/805440 | 10:58 |
| opendevreview | Jacob Anders proposed openstack/sushy stable/wallaby: Change defaults: insert_media write_protected attr https://review.opendev.org/c/openstack/sushy/+/805614 | 11:19 |
| opendevreview | Jacob Anders proposed openstack/sushy stable/wallaby: Change default value: insert_media write_protected https://review.opendev.org/c/openstack/sushy/+/805614 | 11:22 |
| janders | ^ this commit message looks less bad | 11:22 |
| janders | :) | 11:22 |
| dtantsur | did you actually intend to update wallaby? | 11:25 |
| janders | what da.. | 11:28 |
| janders | no | 11:29 |
| janders | sorry! | 11:29 |
| janders | not my day today | 11:29 |
| iurygregory | well, it's monday | 11:29 |
| janders | I was poking around trying to get tox to work again in my setup | 11:29 |
| janders | that included trying running tests in different branches | 11:29 |
| janders | so probably that's what happened there | 11:30 |
| janders | switched the wrong copy | 11:30 |
| janders | now... I never needed to fix a stuff-up like this before - can I switch this change to another branch, or do I need to start a new one? | 11:31 |
| iurygregory | I never did this, but I would say to start a new one | 11:33 |
| dtantsur | janders: you can just cherry-pick | 11:33 |
| dtantsur | (without -x) | 11:33 |
| janders | dtantsur: thanks, will try that | 11:34 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy-tools master: [WIP] Basic support for vmedia TLS certificates from version 1.4.0 https://review.opendev.org/c/openstack/sushy-tools/+/805621 | 12:00 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Change default value: insert_media write_protected https://review.opendev.org/c/openstack/sushy/+/805624 | 12:16 |
| janders | ^ hopefully no more stuff-ups | 12:17 |
| janders | see you tomorrow Ironic o/ | 12:20 |
| TheJulia | good morning | 13:04 |
| rpittau | good morning TheJulia :) | 13:04 |
| * TheJulia attempts to wake up | 13:06 | |
| dtantsur | morning TheJulia | 13:07 |
| iurygregory | good morning TheJulia =) | 13:50 |
| cenne | re [setting node info directly](https://review.opendev.org/c/openstack/ironic/+/800084/20/ironic/conductor/utils.py) | 13:55 |
| cenne | So here was my thinking: | 13:55 |
| cenne | We gave the command "driver.set_boot_mode" and it didn't error. That's what we have. | 13:55 |
| cenne | That could mean | 13:55 |
| cenne | a) it (driver) did what was asked and the task is done successfully. | 13:55 |
| cenne | b) it took the command and failed but didn't pass the error up or just ignored it outright. | 13:55 |
| cenne | Since it doesn't explicitly return a `success` back, just a None, I am not sure what happened. | 13:55 |
| cenne | So out of caution I thought it'd be better to just ask the new current state and set node info accordingly. | 13:55 |
| cenne | I think this type of situation may come up regularly, | 13:55 |
| cenne | so I want to ask how to deal with it in general. | 13:55 |
| cenne | How to deal with ~external functions that don't explicitly return info that the requested change was made successfully.? | 13:58 |
| dtantsur | I think (b) assumes a broken driver, which is not our general assumption | 14:07 |
| dtantsur | I mean, it happens.. but a lot of things break this way | 14:07 |
| dtantsur | cenne: my advice is to keep it simple for now, and see if it causes issues | 14:08 |
| dtantsur | I'm more worried about unnecessary calls to the BMC rather than the driver silently swallowing errors | 14:08 |
| cenne | ack | 14:09 |
| opendevreview | Merged openstack/sushy master: Change default value: insert_media write_protected https://review.opendev.org/c/openstack/sushy/+/805624 | 14:12 |
| dtantsur | rebase party begins | 14:14 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy master: Support credentials for VirtualMedia URI https://review.opendev.org/c/openstack/sushy/+/805367 | 14:16 |
| sdanni | Hi folks! Just want to know if you have any feedback on lmcgann's keylime/ironic integration demo video (https://www.youtube.com/watch?v=o7eL95BUGLA). And do you think it's necessary to bring up our work in midcycle meeting (maybe it's too late to ask)? | 14:30 |
| dtantsur | I haven't watched it yet, I must admit | 14:31 |
| dtantsur | re midcycle, wednesday seems pretty empty so far | 14:32 |
| sdanni | dtantsur: nice! what should we prepare for midcycle? | 14:33 |
| TheJulia | Yeah, I was looking at wednesday and thinking "That is either going to be a not fun discussion or going to go very quickly | 14:33 |
| dtantsur | sdanni: if you have an idea which points are the most contentious, bring them | 14:35 |
| dtantsur | a brief (5 mins top) overview of the implementation could be helpful as well | 14:35 |
| * iurygregory hides since he was the one who set up the agenda | 14:35 | |
| TheJulia | iurygregory: no hiding :) | 14:36 |
| TheJulia | Stand tall! | 14:36 |
| iurygregory | ok :D | 14:36 |
| sdanni | Got it! The demo video is around 10 mins long, we'll quickly go over it at the meeting. | 14:36 |
| dtantsur | we can always insert social hours whenever we have time :) | 14:36 |
| iurygregory | well, I asked if people were ok with the agenda last week :D hehe | 14:36 |
| * TheJulia wasn't entirely around last week | 14:37 | |
| * TheJulia may not be entirely around this week at this rate | 14:37 | |
| jssfr | so now that the pipelines have stabilized, my change is green again: https://review.opendev.org/c/openstack/ironic-python-agent/+/803710 | 14:48 |
| jssfr | does someone need to poke it again or will it progress in the workflow automatically? | 14:48 |
| iurygregory | hummm I don't see the gate pipeline running according to zuul | 14:49 |
| iurygregory | jssfr, now you just need to wait | 14:50 |
| jssfr | thanks for poking! | 14:50 |
| iurygregory | np | 14:50 |
| dtantsur | It was now o'clock when I realized that midcycle is this week, not next week :) | 14:52 |
| TheJulia | heh | 14:53 |
| TheJulia | d'oh | 14:53 |
| iurygregory | dtantsur, hehehe | 14:53 |
| opendevreview | Aija Jauntēva proposed x/sushy-oem-idrac master: Fix exposing system constants https://review.opendev.org/c/x/sushy-oem-idrac/+/805655 | 14:55 |
| *** pmannidi is now known as pmannidi|AFK | 14:56 | |
| opendevreview | cenne proposed openstack/ironic master: Add api endpoints for changing boot_mode and secure_boot state https://review.opendev.org/c/openstack/ironic/+/800084 | 14:59 |
| dtantsur | the midcycle is now, right? | 15:00 |
| iurygregory | dtantsur, yes | 15:00 |
| iurygregory | https://meetpad.opendev.org/ironic | 15:00 |
| opendevreview | Merged openstack/ironic master: [doc] Update power sync documentation https://review.opendev.org/c/openstack/ironic/+/805168 | 15:07 |
| * dtantsur is curious if Meetpdad is a dad-joke or a typo :D | 15:15 | |
| iurygregory | Link for the midcycle in case people are not aware https://meetpad.opendev.org/ironic :D | 15:21 |
| rpittau | it was a typo :) | 15:56 |
| rpittau | bye everyone! o/ | 15:58 |
| *** rpittau is now known as rpittau|afk | 15:58 | |
| TheJulia | dtantsur: have we determined dad jokes were appropriate in IRC ?! | 15:59 |
| TheJulia | or, rather, are ? | 15:59 |
| dtantsur | they're never appropriate, that's the point | 15:59 |
| TheJulia | what about required? | 16:00 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Add RAID mode conversion for idrac-redfish https://review.opendev.org/c/openstack/ironic/+/804112 | 16:00 |
| dtantsur | mmm, hard to say :) consensus needed! | 16:00 |
| iurygregory | LOL | 16:00 |
| opendevreview | Merged openstack/ironic-python-agent master: Move loading of IPMI module loading to a single point https://review.opendev.org/c/openstack/ironic-python-agent/+/803710 | 16:14 |
| TheJulia | Less than 24 hours! I'd love to see an election folks! | 17:10 |
| * iurygregory doesn't have enough knowledge about the HW world to be a PTL lol | 17:23 | |
| TheJulia | it is more about driving process/people forward together than obscure hardware knowledge | 17:26 |
| iurygregory | oh really? O.o | 17:26 |
| TheJulia | Project *Team* Leader :) | 17:27 |
| iurygregory | yeah XD | 17:27 |
| iurygregory | well, maybe I will give a try... will do some exercises and think about it :D | 17:28 |
| TheJulia | ok | 17:32 |
| opendevreview | cenne proposed openstack/ironic master: Add api endpoints for changing boot_mode and secure_boot state https://review.opendev.org/c/openstack/ironic/+/800084 | 17:39 |
| arne_wiebalck | bye everyone o/ | 17:43 |
| cenne | bye arne_wiebalck o/ | 17:55 |
| NobodyCam | Good Morning Ironic folks | 20:31 |
| iurygregory | good morning NobodyCam o/ | 20:39 |
| iurygregory | TheJulia, https://review.opendev.org/c/openstack/election/+/805701 =) | 20:39 |
| NobodyCam | howdy iurygregory | 20:39 |
| iurygregory | doing good and you? | 20:39 |
| NobodyCam | so far so good... | 20:40 |
| TheJulia | iurygregory: :) | 20:47 |
| TheJulia | good morning NobodyCam | 20:47 |
| NobodyCam | Morning TheJulia How goes the day! | 20:48 |
| TheJulia | iurygregory: To the point, I like it :) | 20:48 |
| iurygregory | TheJulia, tks! fingers crossed and I hope I can do at least 80% of what you did as a PTL =) | 20:49 |
| TheJulia | iurygregory: You'll kick ass, I'm sure of it | 20:54 |
| TheJulia | NobodyCam: alright, need to go to the bank though | 20:54 |
| iurygregory | \o/ tks \o/ | 20:54 |
| * TheJulia goes to the bank... because actually needs to do things with paper checks | 20:59 | |
| NobodyCam | paper checks | 21:00 |
| NobodyCam | wow | 21:00 |
| TheJulia | yeah, like deposit and get one cut because I don't have a check book for an account.... | 21:00 |
| NobodyCam | Good Luck | 21:06 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org has been restarted for a patch version upgrade, resulting in a brief outage | 21:42 | |
| NobodyCam | iurygregory: congrats! | 22:01 |
| iurygregory | NobodyCam, tks! | 22:02 |
| NobodyCam | :) | 22:02 |
| stevebaker | TheJulia: hey in your opinion should tripleo continue to set [ilo]default_boot_mode=bios? https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/ironic/ironic-conductor-container-puppet.yaml#L440-L441 | 22:10 |
| janders | good morning Ironic o/ | 22:23 |
| iurygregory | morning janders o/ | 22:27 |
| * iurygregory is going to sleep now hehe | 22:27 | |
| TheJulia | stevebaker: unlikely. Any setting should have a why behind it, and that why ideally should ahve a name we can track down | 22:43 |
| TheJulia | stevebaker: that why is semi-weak I guess | 22:44 |
| stevebaker | TheJulia: I think the only 'why' is that bios was the default, so ilo needed to be explicitly set to bios also | 22:45 |
| stevebaker | TheJulia: I fixed it https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805732 | 22:45 |
| TheJulia | stevebaker: yeah, and everything really should be uefi moving forward | 22:45 |
| stevebaker | yup | 22:45 |
| * TheJulia should think about dinner | 22:58 | |
| *** yoctozepto4 is now known as yoctozepto | 23:37 | |
| *** pmannidi|AFK is now known as pmannidi | 23:49 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!