| rpittau | good morning ironic! o/ | 08:20 |
|---|---|---|
| rpittau | Guest978: if I remember correctly the software RAID config (as well as hardware) can't be remove with regular cleaning, you can probably find more info and details here https://docs.openstack.org/ironic/latest/admin/raid.html | 08:31 |
| kubajj | Good morning Ironic! | 08:32 |
| rpittau | hey kubajj :) | 08:33 |
| janders | dtantsur ajya thank you for reviews on https://review.opendev.org/c/openstack/sushy/+/856597. 100% agreed on JSON parsing improvements, it kind of grew that way while making small changes iterating through reviews. I tend to miss things that happen this way :) Will upload a new version later today | 08:42 |
| ajya | janders: yw, that's normal, part of the process | 08:43 |
| opendevreview | Riccardo Pittau proposed openstack/sushy master: Make server connection retries configurable https://review.opendev.org/c/openstack/sushy/+/864102 | 09:10 |
| opendevreview | Riccardo Pittau proposed openstack/sushy master: Make server connection retries configurable https://review.opendev.org/c/openstack/sushy/+/864102 | 09:11 |
| *** bbezak_ is now known as bbezak | 09:51 | |
| *** bbezak is now known as Guest1060 | 09:52 | |
| *** Guest1060 is now known as bbezak | 09:56 | |
| janders | dtantsur I'm not sure if I understand this comment: https://review.opendev.org/c/openstack/sushy/+/856597/comments/c8e57493_ae728f9b are you thinking of a slicker way of handling the different eTags, or are you concerned that running two GETs in sequence can mix up etags? | 10:19 |
| janders | dtantsur I can do something like this: https://paste.openstack.org/show/b9iYcfhlJs5GUBD1zDLU/ (jump to L84) | 10:30 |
| janders | this way vars don't sit around if they never get used I suppose | 10:30 |
| janders | let me know if you like this style more and I will push an updated revision (I unified var names too so there's no settings_etag and etag in different conditionals) | 10:32 |
| janders | actually I will just push it and we talk in the review, for async that's prob better anyway | 10:34 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Fix setting boot related attributes https://review.opendev.org/c/openstack/sushy/+/856597 | 10:36 |
| arozman | Hi Ironic! | 10:53 |
| * TheJulia waves from a time zone nine hours off from her normal time zone | 10:59 | |
| dtantsur | janders: I'd probably not act upon it for now.. but *in theory*, when you issue a patch to a subresource, the etag of the main resource may change | 11:07 |
| janders | dtantsur thank you for the explanation. With the latest revision, if such eTag change were to happen, would it still work? That was my goal tweaking it this way | 11:17 |
| janders | (TLDR I moved fetching eTags to right prior to respective PATCH) | 11:17 |
| janders | hey TheJulia o/ | 11:43 |
| arozman | hi I have a question, it might be very naive but I am trying to ask a Ironic node to transition from enroll state to manageable via idrac but I am hitting this issue during verify step https://pastebin.com/MB3dutMk I see this error both in the response and in the Ironic logs, as I understand drac module implements these priority decorators for certain verification actions and then conductor's task manager executes them but in | 11:48 |
| arozman | the code all decorator instances have the priority set to 0 and I see there is something called verify_step_priority_override that is part of the conductor but how do I interact with it form outside (e.g. config file) or is it just for internal use for Ironic | 11:48 |
| arozman | Ohh yeah Ironic is installed from source from this commit 872ede25a072850873e20477bb8bc75fdce1ed37 | 11:52 |
| arozman | Maybe it is also important info that it is real physical dell server with BMC firmware with iDrac firmware 5.00.00.00 | 12:02 |
| dtantsur | arozman: this has been fixed in the upstream code, could you update your ironic? | 12:17 |
| TheJulia | \o/ | 12:17 |
| dtantsur | good afternoon TheJulia :) | 12:17 |
| dtantsur | how's Brno? | 12:17 |
| arozman | @dtantsur I think I can I will ask around it is downstream from me, but thanks for the info !!! | 12:19 |
| iurygregory | good morning Ironic | 12:21 |
| TheJulia | dtantsur: exhausting | 12:46 |
| TheJulia | Like a PTG of olde | 12:46 |
| TheJulia | actually, Dev summit | 12:46 |
| dtantsur | wow! but at least productive? | 12:50 |
| stevebaker[m] | yes, lots of good discussions | 12:59 |
| kubajj | dtantsur: I've implemented the changes from yesterday, so if you have a moment, could I ask for a re-review? https://review.opendev.org/c/openstack/ironic/+/862569 | 13:04 |
| dtantsur | will do eventually\ | 13:12 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 13:41 |
| dtantsur | does anyone by chance have an ubuntu VM around? what does $ sudo dmidecode -s 'system-product-name' output? | 14:22 |
| ashinclouds[m] | Wouldn’t that get decoded out of smbios data presented by libvirt… and the data version supported by the vim architecture? | 14:24 |
| dtantsur | yeah, I guess my question is more about a VM on an ubuntu host | 14:24 |
| * dtantsur is trying to detect KVM | 14:25 | |
| dtantsur | maybe I should just go with systemd-detect-virt | 14:26 |
| ashinclouds[m] | I think that is data from qemu, and indication of native virtualization might not be great… although I guess in some cases that is a feature | 14:26 |
| * ashinclouds[m] gives into sleeeep | 14:29 | |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 14:37 |
| frickler | dtantsur: mine says "OpenStack Nova" | 14:57 |
| dtantsur | hmm, fair enough. thanks! | 14:58 |
| rpittau | bye everyone, see you on monday! o/ | 15:33 |
| *** knikolla[m] is now known as knikolla | 16:06 | |
| JayF | Can folks please take a look at https://review.opendev.org/c/openstack/ironic-specs/+/862538 | 16:09 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 16:29 |
| *** iurygregory__ is now known as iurygregory | 16:44 | |
| Guest978 | rpittau: Thank you very much. I suppose that's as much of a confirmation as I need. I can probably implement it in our custom hardware manager. | 17:05 |
| *** mnaser__ is now known as mnaser | 17:05 | |
| JayF | There's a trick for creating new DB migrations, right? An alembic command? | 19:15 |
| * JayF trying to get the boilerplate done for https://review.opendev.org/c/openstack/ironic-specs/+/861803/10/specs/approved/shard-key.rst#277 | 19:16 | |
| TheJulia | JayF: umm something like alembic new something something | 22:39 |
| JayF | yeah, I figured it out | 22:56 |
| JayF | about to go live at youtube.com/jayofdoom and keep working on that, actually ... office hours from 2300-2400 utc | 22:56 |
| TheJulia | Cool! I’m going to crash shortly | 22:56 |
| TheJulia | Have fun! | 22:56 |
| JayF | Shouldn't you already be crashed? lol | 22:57 |
| TheJulia | Only about midnight | 22:57 |
| TheJulia | Alarm set for 6 am | 22:57 |
| TheJulia | Train to start the trip back is at 8:45 | 22:58 |
| JayF | have a good one | 23:02 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: WIP: DB implementation of shards / shard_key https://review.opendev.org/c/openstack/ironic/+/864236 | 23:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!