*** k_mouza has joined #openstack-ironic | 00:05 | |
*** k_mouza has quit IRC | 00:10 | |
janders | good morning Ironic o/ | 00:11 |
---|---|---|
*** k_mouza has joined #openstack-ironic | 00:16 | |
*** k_mouza has quit IRC | 00:20 | |
*** k_mouza has joined #openstack-ironic | 00:48 | |
*** k_mouza has quit IRC | 00:52 | |
stevebaker | janders: hey | 00:58 |
*** tzumainn has quit IRC | 01:01 | |
*** bburns has quit IRC | 01:05 | |
*** bburns has joined #openstack-ironic | 01:15 | |
janders | stevebaker TheJulia I'm (finally!) wrapping up my https/vmedia testing - if you're interested in results, happy to share the report | 01:24 |
janders | spoiler: I got older SuperMicros to work | 01:24 |
janders | (and not with Samba! :) ) | 01:24 |
janders | there was an awful lot of swearing involved | 01:25 |
stevebaker | janders: nice, well done | 01:38 |
janders | thank you stevebaker | 02:41 |
janders | disappearing for half an hour or so, router swap | 02:41 |
janders | the old one got a little unstable after the recent power outage | 02:41 |
*** mkrai has joined #openstack-ironic | 03:01 | |
*** paras333 has joined #openstack-ironic | 03:09 | |
*** derekh has quit IRC | 03:16 | |
*** derekh has joined #openstack-ironic | 03:22 | |
*** rh-jelabarre has quit IRC | 03:26 | |
janders | FAIL! | 03:55 |
janders | rolled back to the old router | 03:55 |
janders | not sure what the new one is doing, but clearly not routing packets | 03:55 |
janders | looks like a support case for later | 03:56 |
stevebaker | heh | 04:01 |
*** k_mouza has joined #openstack-ironic | 04:10 | |
*** k_mouza has quit IRC | 04:11 | |
*** k_mouza_ has joined #openstack-ironic | 04:11 | |
*** k_mouza_ has quit IRC | 04:15 | |
*** ricolin has joined #openstack-ironic | 04:18 | |
*** sdanni has quit IRC | 05:52 | |
*** bfournie has quit IRC | 05:56 | |
*** bfournie has joined #openstack-ironic | 05:59 | |
*** bburns_ has joined #openstack-ironic | 06:01 | |
*** bburns has quit IRC | 06:09 | |
arne_wiebalck | Good morning, ironic! | 06:18 |
*** gyee has quit IRC | 06:29 | |
*** zzzeek has quit IRC | 06:34 | |
*** zzzeek has joined #openstack-ironic | 06:35 | |
*** sshnaidm|afk is now known as sshnaidm | 06:59 | |
iurygregory | good morning janders stevebaker arne_wiebalck and Ironic | 07:00 |
arne_wiebalck | hey iurygregory o/ | 07:00 |
janders | hey iurygregory arne_wiebalck | 07:02 |
janders | going offline for some time - demo recording | 07:02 |
*** mkrai_ has joined #openstack-ironic | 07:04 | |
*** mkrai has quit IRC | 07:07 | |
*** jawad_axd has joined #openstack-ironic | 07:16 | |
*** rpittau|afk is now known as rpittau | 07:18 | |
rpittau | good morning ironic! o/ | 07:19 |
iurygregory | morning rpittau o/ | 07:21 |
rpittau | hey iurygregory :) | 07:22 |
*** tosky has joined #openstack-ironic | 07:22 | |
* rpittau messages buffer overflow | 07:22 | |
iurygregory | hehe | 07:23 |
*** mkrai_ has quit IRC | 07:27 | |
*** Qianbiao has joined #openstack-ironic | 07:37 | |
*** bfournie has quit IRC | 07:40 | |
*** priteau has joined #openstack-ironic | 07:44 | |
*** benj_ has quit IRC | 07:52 | |
*** bfournie has joined #openstack-ironic | 07:54 | |
*** benj_ has joined #openstack-ironic | 07:56 | |
*** jawad_axd has quit IRC | 08:04 | |
*** lucasagomes has joined #openstack-ironic | 08:04 | |
*** jawad_axd has joined #openstack-ironic | 08:05 | |
dtantsur | morning ironic | 08:07 |
*** benj_ has quit IRC | 08:09 | |
rpittau | good morning dtantsur :) | 08:11 |
*** seongsoocho has quit IRC | 08:12 | |
*** coreycb has quit IRC | 08:12 | |
*** mnasiadka has quit IRC | 08:17 | |
*** coreycb has joined #openstack-ironic | 08:17 | |
iurygregory | morning dtantsur o/ | 08:20 |
*** mkrai_ has joined #openstack-ironic | 08:23 | |
dtantsur | cenn: hi! sorry, I told you we would sync on Monday, but I totally forgot I had a public holiday :) | 08:24 |
*** mnasiadka has joined #openstack-ironic | 08:26 | |
*** seongsoocho has joined #openstack-ironic | 08:30 | |
*** dtantsur has quit IRC | 08:34 | |
*** benj_ has joined #openstack-ironic | 08:36 | |
*** dtantsur has joined #openstack-ironic | 08:39 | |
*** ociuhandu has joined #openstack-ironic | 08:41 | |
*** ociuhandu has quit IRC | 08:51 | |
*** benj_ has quit IRC | 08:51 | |
*** ociuhandu has joined #openstack-ironic | 08:52 | |
*** dtantsur has quit IRC | 08:57 | |
*** dtantsur has joined #openstack-ironic | 08:57 | |
*** benj_ has joined #openstack-ironic | 09:06 | |
cenn | morning dtantsur. It's fine. :) | 09:10 |
cenn | morning everyone o/ | 09:10 |
iurygregory | morning cenn | 09:11 |
*** ociuhandu has quit IRC | 09:12 | |
cenn | hey iurygregory :) | 09:12 |
*** ociuhandu has joined #openstack-ironic | 09:28 | |
*** ociuhandu has quit IRC | 09:33 | |
*** uzumaki has joined #openstack-ironic | 09:34 | |
*** ociuhandu has joined #openstack-ironic | 09:35 | |
*** ociuhandu has quit IRC | 09:40 | |
cenn | I'm hitting this red block while installing bifrost. Does anyone know what the `BECOME` password is? | 09:57 |
dtantsur | cenn: sudo password | 09:57 |
openstackgerrit | Verification of a change to openstack/ironic failed: Update min version of tox to use allowlist https://review.opendev.org/c/openstack/ironic/+/791909 | 09:57 |
cenn | oh. okay. so i pass in my sudo password? | 09:58 |
dtantsur | cenn: if it's a testing machine (I hope you're not doing it on your laptop/main computer :), you can set it up for passwordless sudo | 09:58 |
cenn | yes I'm doing it in a VM (which is on my laptop though) | 09:59 |
dtantsur | (I'll get a tea and come back to sync with you in some minutes) | 09:59 |
*** dsneddon has quit IRC | 09:59 | |
cenn | 👍 | 10:00 |
*** k_mouza has joined #openstack-ironic | 10:12 | |
*** pmannidi has quit IRC | 10:12 | |
*** fungi has quit IRC | 10:13 | |
*** arne_wiebalck has quit IRC | 10:13 | |
*** mugsie has quit IRC | 10:13 | |
*** larsks has quit IRC | 10:13 | |
*** fungi has joined #openstack-ironic | 10:14 | |
*** arne_wiebalck has joined #openstack-ironic | 10:14 | |
*** mugsie has joined #openstack-ironic | 10:14 | |
*** larsks has joined #openstack-ironic | 10:14 | |
*** benj_ has quit IRC | 10:14 | |
*** mkrai_ has quit IRC | 10:14 | |
*** Qianbiao has quit IRC | 10:14 | |
*** ricolin has quit IRC | 10:14 | |
*** openstackgerrit has quit IRC | 10:14 | |
*** mkowalski has quit IRC | 10:14 | |
*** dansmith has quit IRC | 10:14 | |
*** akahat has quit IRC | 10:14 | |
*** hjensas has quit IRC | 10:14 | |
*** cenn has quit IRC | 10:14 | |
*** gryf has quit IRC | 10:14 | |
*** ociuhandu has joined #openstack-ironic | 10:16 | |
*** benj_ has joined #openstack-ironic | 10:16 | |
*** mkrai_ has joined #openstack-ironic | 10:16 | |
*** Qianbiao has joined #openstack-ironic | 10:16 | |
*** ricolin has joined #openstack-ironic | 10:16 | |
*** openstackgerrit has joined #openstack-ironic | 10:16 | |
*** mkowalski has joined #openstack-ironic | 10:16 | |
*** dansmith has joined #openstack-ironic | 10:16 | |
*** akahat has joined #openstack-ironic | 10:16 | |
*** hjensas has joined #openstack-ironic | 10:16 | |
*** cenn has joined #openstack-ironic | 10:16 | |
*** gryf has joined #openstack-ironic | 10:16 | |
*** benj_ has quit IRC | 10:16 | |
*** mnaser has quit IRC | 10:16 | |
*** rnoriega_ has quit IRC | 10:16 | |
*** fdegir has quit IRC | 10:16 | |
*** shadower has quit IRC | 10:16 | |
*** strigazi has quit IRC | 10:16 | |
*** bnemec has quit IRC | 10:16 | |
*** mjturek has quit IRC | 10:16 | |
*** k_mouza has quit IRC | 10:17 | |
*** benj_ has joined #openstack-ironic | 10:17 | |
*** pmannidi has joined #openstack-ironic | 10:18 | |
*** rajinir has quit IRC | 10:18 | |
*** parallax has quit IRC | 10:18 | |
*** buhman has quit IRC | 10:18 | |
*** JayF has quit IRC | 10:18 | |
*** sshnaidm has quit IRC | 10:18 | |
*** hoonetorg has quit IRC | 10:18 | |
*** baha has quit IRC | 10:18 | |
*** EmilienM has quit IRC | 10:18 | |
*** lourot has quit IRC | 10:18 | |
*** andrein has quit IRC | 10:18 | |
*** mraineri has quit IRC | 10:18 | |
*** andreaf has quit IRC | 10:18 | |
*** dtantsur has quit IRC | 10:18 | |
*** iurygregory has quit IRC | 10:18 | |
*** mrda has quit IRC | 10:18 | |
*** mushrushu has quit IRC | 10:18 | |
*** lifeless has quit IRC | 10:18 | |
*** Anticimex has quit IRC | 10:18 | |
*** zzzeek has quit IRC | 10:18 | |
*** cp- has quit IRC | 10:18 | |
*** timeu has quit IRC | 10:18 | |
*** eandersson has quit IRC | 10:18 | |
*** SpamapS has quit IRC | 10:18 | |
*** andy_ has quit IRC | 10:18 | |
*** dtantsur has joined #openstack-ironic | 10:20 | |
*** zzzeek has joined #openstack-ironic | 10:20 | |
*** cp- has joined #openstack-ironic | 10:20 | |
*** sshnaidm has joined #openstack-ironic | 10:20 | |
*** hoonetorg has joined #openstack-ironic | 10:20 | |
*** JayF has joined #openstack-ironic | 10:20 | |
*** buhman has joined #openstack-ironic | 10:20 | |
*** parallax has joined #openstack-ironic | 10:20 | |
*** rajinir has joined #openstack-ironic | 10:20 | |
*** EmilienM has joined #openstack-ironic | 10:20 | |
*** iurygregory has joined #openstack-ironic | 10:20 | |
*** mrda has joined #openstack-ironic | 10:20 | |
*** timeu has joined #openstack-ironic | 10:20 | |
*** mushrushu has joined #openstack-ironic | 10:20 | |
*** baha has joined #openstack-ironic | 10:20 | |
*** eandersson has joined #openstack-ironic | 10:20 | |
*** SpamapS has joined #openstack-ironic | 10:20 | |
*** lourot has joined #openstack-ironic | 10:20 | |
*** andreaf has joined #openstack-ironic | 10:20 | |
*** mraineri has joined #openstack-ironic | 10:20 | |
*** andrein has joined #openstack-ironic | 10:20 | |
*** lifeless has joined #openstack-ironic | 10:20 | |
*** Anticimex has joined #openstack-ironic | 10:20 | |
*** andy_ has joined #openstack-ironic | 10:20 | |
*** mnaser has joined #openstack-ironic | 10:21 | |
*** rnoriega_ has joined #openstack-ironic | 10:21 | |
*** fdegir has joined #openstack-ironic | 10:21 | |
*** shadower has joined #openstack-ironic | 10:21 | |
*** strigazi has joined #openstack-ironic | 10:21 | |
*** bnemec has joined #openstack-ironic | 10:21 | |
*** mjturek has joined #openstack-ironic | 10:21 | |
*** mnasiadka has quit IRC | 10:21 | |
*** bfournie has quit IRC | 10:21 | |
*** priteau has quit IRC | 10:21 | |
*** uzumaki has quit IRC | 10:21 | |
*** coreycb has quit IRC | 10:21 | |
*** bburns_ has quit IRC | 10:21 | |
*** jawad_axd has quit IRC | 10:21 | |
arne_wiebalck | dtantsur: I started to look into the cs8 on s/w RAID issue, and found this (unrelated) snippet: https://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/extensions/image.py#L652-L659 ... this umount/mount combination looks like a remnant from some older logic ... or can you make any sense of this? | 10:22 |
dtantsur | arne_wiebalck: Oo this is interesting | 10:23 |
*** uzumaki has joined #openstack-ironic | 10:24 | |
*** mnasiadka has joined #openstack-ironic | 10:24 | |
*** coreycb has joined #openstack-ironic | 10:24 | |
*** jawad_axd has joined #openstack-ironic | 10:24 | |
*** bfournie has joined #openstack-ironic | 10:24 | |
*** priteau has joined #openstack-ironic | 10:24 | |
*** bburns_ has joined #openstack-ironic | 10:24 | |
*** ociuhandu has quit IRC | 10:24 | |
*** trident has quit IRC | 10:27 | |
*** ianw has quit IRC | 10:27 | |
*** trident has joined #openstack-ironic | 10:27 | |
*** ianw has joined #openstack-ironic | 10:27 | |
*** ociuhandu has joined #openstack-ironic | 10:29 | |
*** k_mouza has joined #openstack-ironic | 10:29 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Update min version of tox to use allowlist https://review.opendev.org/c/openstack/ironic/+/791909 | 10:35 |
*** ociuhandu has quit IRC | 10:37 | |
*** ociuhandu has joined #openstack-ironic | 10:37 | |
dtantsur | ajya: hi! is your old outreachy blog still online? cenn could probably use it as an example. | 10:43 |
ajya | it is - https://ajya.github.io/ | 10:44 |
dtantsur | thanks! cenn ^^^ | 10:44 |
cenn | got it. | 10:45 |
arne_wiebalck | mnaser: did you have any additional logs to share on the cs8 on RAID issue? In our deployment it seems the code is trying to mount the already mounted ESP (which breaks the bootloader installation), but this is the same with a c8.2, so these may be different issues ... but I think you or EmilienM mentioned this was only appearing on 8.4, no? | 10:46 |
cenn | nice ajya :) | 10:46 |
cenn | thanks dtantsur, ajya | 10:46 |
ajya | cenn: yw | 10:46 |
arne_wiebalck | mnaser: it seems to work fine on c7 ... not sure I ever tested c8 on RAID, so the issue I see may have been there since ever. | 10:47 |
arne_wiebalck | mnaser: I keep digging :) | 10:47 |
*** mkrai_ has quit IRC | 10:48 | |
dtantsur | cenn: for failing bifrost, could you paste the full error (using e.g. paste.openstack.org) | 10:49 |
cenn | https://paste.centos.org/view/73cc0087 | 10:49 |
cenn | ^^^ | 10:49 |
cenn | fuller log : https://paste.centos.org/view/7f20683d | 10:52 |
dtantsur | cenn: did you run the command from https://docs.openstack.org/bifrost/latest/install/index.html#testing-on-virtual-machines before 'install'? | 10:52 |
dtantsur | the 'testenv' command prepares (nested) virtual machines to be used as fake hardware | 10:52 |
dtantsur | (I assume you don't have actual servers to test on) | 10:52 |
cenn | right. i'll do that again then. sorry, I thought the second one redoes the above setup. | 10:54 |
cenn | based on my previous tries. | 10:54 |
cenn | Thank you! | 10:54 |
dtantsur | generally, you need to run 'testenv' only once in the beginning | 10:55 |
dtantsur | have you done it already? if yes, do you have the virbr0 network device present? | 10:55 |
cenn | I had done it previously, but then reset the VM because of other errors. Then skipped that by mistake. | 10:56 |
cenn | (It didn't complain and said cloud already present so i thought it was fine) | 10:57 |
dtantsur | aah! got it. I think running it and then retrying will solve your problem. | 10:57 |
dtantsur | install works without testenv, but it's created for actual servers, not a virtual setup | 10:57 |
dtantsur | I definitely think that we need a better error message | 10:58 |
*** hoonetorg has quit IRC | 10:58 | |
*** EmilienM has quit IRC | 10:58 | |
*** andrein has quit IRC | 11:00 | |
*** andrein has joined #openstack-ironic | 11:01 | |
*** ociuhandu has quit IRC | 11:05 | |
*** sshnaidm has quit IRC | 11:06 | |
*** sshnaidm has joined #openstack-ironic | 11:07 | |
*** jawad_ax_ has joined #openstack-ironic | 11:12 | |
*** dtantsur is now known as dtantsur|brb | 11:12 | |
*** k_mouza has quit IRC | 11:13 | |
*** k_mouza_ has joined #openstack-ironic | 11:13 | |
openstackgerrit | Merged openstack/ironic-ui master: Drop horizon-nodejs10-jobs template https://review.opendev.org/c/openstack/ironic-ui/+/792206 | 11:15 |
*** jawad_axd has quit IRC | 11:15 | |
*** ociuhandu has joined #openstack-ironic | 11:18 | |
*** ociuhandu has quit IRC | 11:18 | |
openstackgerrit | Merged openstack/ironic master: CI: Collect a snapshot of network connections https://review.opendev.org/c/openstack/ironic/+/770481 | 11:21 |
*** ociuhandu has joined #openstack-ironic | 11:40 | |
openstackgerrit | Merged openstack/ironic-python-agent stable/victoria: Point ipa-builder to stable/wallaby https://review.opendev.org/c/openstack/ironic-python-agent/+/790457 | 11:44 |
cenn | yay! "Ironic is installed and running." | 11:44 |
openstackgerrit | Merged openstack/ironic master: Update min version of tox to use allowlist https://review.opendev.org/c/openstack/ironic/+/791909 | 11:44 |
openstackgerrit | Merged openstack/ironic stable/victoria: Delete unavailable py2 package https://review.opendev.org/c/openstack/ironic/+/791950 | 11:44 |
iurygregory | congrats cenn =D | 11:49 |
cenn | iurygregory: Thank you. On to building docs now. | 11:51 |
*** ociuhandu has quit IRC | 11:58 | |
mnaser | arne_wiebalck: I am able to do a clean centos 8 build and deploy it using dib. Would you like the options used? | 12:02 |
arne_wiebalck | mnaser: thanks, but no: our images are not built with dib :) | 12:03 |
arne_wiebalck | mnaser: so the issue is all solved for you? | 12:03 |
mnaser | arne_wiebalck: nope, I have c8 dib images that work but cs8 do not | 12:04 |
*** ociuhandu has joined #openstack-ironic | 12:04 | |
arne_wiebalck | mnaser: hmm, ok, that is interesting ... none working atm :( will try to catch up ;) | 12:04 |
arne_wiebalck | mnaser: c8 is which version, 8.3 ? | 12:05 |
mnaser | arne_wiebalck: didn’t check, whatever is latest I guess | 12:06 |
arne_wiebalck | arne_wiebalck: should be 8.3 then, I think | 12:06 |
mnaser | Like I have an image that perfectly works with DIB_RELEASE=8 | 12:07 |
arne_wiebalck | mnaser: will keep you posted if I find anything, need to get c8 working first. Once I have that, I will look into cs8. | 12:07 |
mnaser | Swap that out to DIB_RELEASE=8-stream and it doesn’t work | 12:07 |
mnaser | arne_wiebalck: where is it failing exactly ? | 12:07 |
*** tosky has quit IRC | 12:08 | |
*** ociuhandu has quit IRC | 12:09 | |
arne_wiebalck | mnaser: when mounting the ESP, it seems already mounted | 12:12 |
mnaser | During the deploy I assume | 12:12 |
arne_wiebalck | yes | 12:13 |
arne_wiebalck | http://paste.openstack.org/show/805691/ | 12:13 |
mnaser | rootfs_uuid dance is done? | 12:13 |
arne_wiebalck | mnaser: should be, yes | 12:14 |
arne_wiebalck | mnaser: note that this is with the EPS mirror patch | 12:15 |
arne_wiebalck | mnaser: I claimed it is unrelated, maybe I was wrong | 12:15 |
arne_wiebalck | mnaser: I will debug and let you know ... | 12:15 |
mnaser | Ah likely could be an issue, we upgraded to wallaby which I don’t think includes it yet | 12:15 |
arne_wiebalck | correct | 12:19 |
*** ociuhandu has joined #openstack-ironic | 12:22 | |
*** ociuhandu has quit IRC | 12:34 | |
*** rh-jelabarre has joined #openstack-ironic | 12:39 | |
*** ociuhandu has joined #openstack-ironic | 12:42 | |
openstackgerrit | yang yawei proposed openstack/virtualbmc master: Use TOX_CONSTRAINTS_FILE https://review.opendev.org/c/openstack/virtualbmc/+/792964 | 12:45 |
*** ociuhandu has quit IRC | 12:47 | |
*** dtantsur|brb is now known as dtantsur | 12:58 | |
*** ociuhandu has joined #openstack-ironic | 12:58 | |
TheJulia | it is most definitely morning | 13:03 |
TheJulia | Good morning everyone | 13:03 |
openstackgerrit | Merged openstack/ironic-python-agent master: Enable out-of-order writes when writing whole disk images https://review.opendev.org/c/openstack/ironic-python-agent/+/791934 | 13:03 |
rpittau | good morning TheJulia :) | 13:05 |
*** ociuhandu has quit IRC | 13:08 | |
*** rloo has joined #openstack-ironic | 13:10 | |
*** rloo has quit IRC | 13:10 | |
*** rloo has joined #openstack-ironic | 13:11 | |
*** tzumainn has joined #openstack-ironic | 13:13 | |
cenn | Good morning TheJulia! | 13:19 |
cenn | Finished building the docs locally \o/ | 13:25 |
dtantsur | \o/ | 13:25 |
cenn | :) | 13:26 |
dtantsur | you can also run ironic unit tests (it takes a while, especially the 1st time) | 13:26 |
cenn | alright. will it need the bifrost setup? | 13:26 |
dtantsur | cenn: no. unit tests can run locally. | 13:26 |
dtantsur | (same as doc building) | 13:27 |
dtantsur | good morning TheJulia | 13:27 |
cenn | cool. | 13:27 |
*** ricolin has quit IRC | 13:28 | |
*** ociuhandu has joined #openstack-ironic | 13:28 | |
*** ricolin has joined #openstack-ironic | 13:30 | |
*** uzumaki has quit IRC | 13:30 | |
*** ociuhandu has quit IRC | 13:35 | |
*** ociuhandu has joined #openstack-ironic | 13:38 | |
TheJulia | dtantsur: replied on https://review.opendev.org/c/openstack/ironic/+/792274 | 13:40 |
TheJulia | I really wish you would have talked to me first. | 13:40 |
dtantsur | -1 on gerrit is talking, it's not that I left you a negative performance review | 13:41 |
dtantsur | let's actually have zzzeek confirm that what is happening in the patch is the only way | 13:41 |
dtantsur | the examples on https://docs.sqlalchemy.org/en/14/faq/performance.html#result-fetching-slowness-orm have much less code | 13:43 |
*** ociuhandu has quit IRC | 13:43 | |
TheJulia | Okay, our conundrum is the join where multiple rows are returned by the query | 13:44 |
TheJulia | a join to populate chassis_uuid would be *much* simpler. | 13:45 |
dtantsur | the Bundle example seems related | 13:45 |
TheJulia | dtantsur: I guess my frustration, is when you -1 patches like that, getting you to revisit it seems to take far more work. | 13:46 |
TheJulia | dtantsur: my gut feeling is the bundle example is just more object conversion headaches that are not really needed when we have a clear object model | 13:47 |
dtantsur | well, it's not a new problem | 13:47 |
TheJulia | I know | 13:47 |
* dtantsur tries to find what SelectiveListObject is | 13:48 | |
TheJulia | dtantsur: I created it | 13:49 |
dtantsur | ah, so it's Munch pretty much (let's use Munch) | 13:49 |
TheJulia | because it is *FAR* cleaner to not try and hand the upper levels a dictionary result set | 13:49 |
dtantsur | okay, so replacing ORM objects with a Munch. all for it. | 13:49 |
TheJulia | your making a bit of a jump | 13:49 |
dtantsur | (https://pypi.org/project/munch/) | 13:50 |
dtantsur | actually, there is a very short code slice where we use ORM objects | 13:50 |
TheJulia | use another import for magical handling when selective object list is a dict who has a class wrapped around it, and that is literally all it is | 13:50 |
dtantsur | if we just get rid of them, we're going to see performance improvements across the board | 13:50 |
TheJulia | just to speed up the slowest possible query :\ | 13:51 |
dtantsur | TheJulia: we already depend on Munch through openstacksdk | 13:51 |
TheJulia | we literally just can't strip the object model, and I surely can't get time to re-architect the entire data model | 13:51 |
TheJulia | and that becomes exceedingly high risk | 13:51 |
*** ociuhandu has joined #openstack-ironic | 13:51 | |
dtantsur | well, the code you have is already high risk | 13:51 |
TheJulia | where as in this specific case, it is a targetted focused optimization that we can then possibly explore evolving the object use model. | 13:52 |
dtantsur | it is based on your measurement and a lot of practical knowledge of how sqlalchemy works | 13:52 |
TheJulia | not really, we have higher level code in the api which does mass field strips based on the fields argument provided | 13:52 |
TheJulia | That line was in regards to risk | 13:53 |
*** ociuhandu has quit IRC | 13:56 | |
dtantsur | how does it even work with limit? do we have tests to ensure that LIMIT 1 won't result in 1 trait fetched? | 13:57 |
TheJulia | That is a *really* good question | 13:58 |
dtantsur | I'll leave this and a few more minor comments | 13:58 |
TheJulia | k | 13:59 |
TheJulia | fwiw, we have limited tests around sqlalchemy and joins and limit implications, but I'm not sure limit actually gets entirely passed because when you query 1000 nodes on the second marker, it explicitly asks the db to start at that point but I'm not sure about the limit | 14:00 |
*** ociuhandu has joined #openstack-ironic | 14:05 | |
cenn | http://paste.openstack.org/show/fA7jhLG9yUyhcY3SwyGz/ | 14:06 |
cenn | ^^^ is this a pip bug? I don't really see the conflict. | 14:06 |
dtantsur | TheJulia: if we don't pass limit to sqlalchemy, it's a bug | 14:07 |
dtantsur | cenn: I think the message about conflict is a red herring. the actual problem is that you need PostgreSQL headers installed | 14:08 |
dtantsur | cenn: on my Fedora it's in libpq-devel | 14:08 |
TheJulia | dtantsur: I'll need to dig into it later | 14:08 |
dtantsur | probably libpq-dev on debian/ubuntu | 14:08 |
cenn | yes, i fixed it by installing that (searched backwards using whatprovides) | 14:09 |
cenn | the 'conflict' error threw me off for a bit. | 14:09 |
dtantsur | yeah, it's weird | 14:09 |
cenn | my terminal is going crazy haha ^^' ~flickering. (tests running) | 14:11 |
dtantsur | oh yeah, they're super verbose :D | 14:11 |
dtantsur | once you know what you want to test, you can limit the tests with a keyword, like $ tox -epy3 -- something | 14:11 |
dtantsur | there is also a nice flag to only run tests that failed the previous time | 14:12 |
dtantsur | like $ tox -epy3 -- --failing | 14:12 |
cenn | this also prolly needs an trigger warning for newbies :p (for epilepsy etc) | 14:13 |
cenn | dtantsur: oh thanks! i did not know we could do that. | 14:13 |
* cenn taking notes | 14:13 | |
dtantsur | yep. the underlying testing utility, stestr, has some interesting options | 14:13 |
iurygregory | I wasn't aware of "-- --failing" | 14:15 |
iurygregory | adding to my notes also =) | 14:15 |
cenn | that's my favourite too iurygregory. first noted that one down. | 14:16 |
dtantsur | I've only discovered it recently fwiw. maybe we need it in the contributor guide | 14:16 |
iurygregory | ++ | 14:16 |
cenn | was running the whole pep8 tests everytime i made a small change in (openstacksdk) | 14:17 |
cenn | yay ironic Ran 8264 tests in 520.569s | 14:18 |
cenn | OK | 14:18 |
cenn | all green | 14:19 |
*** ociuhandu has quit IRC | 14:19 | |
*** ociuhandu has joined #openstack-ironic | 14:20 | |
*** rloo has quit IRC | 14:20 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Remove inventory time workaround for WS-man BIOS https://review.opendev.org/c/openstack/ironic/+/792142 | 14:24 |
*** paras333_ has joined #openstack-ironic | 14:26 | |
dtantsur | cenn: great! now I suggest you check the ironic's user guide, while simultaneously looking through the source code structure and trying to understand what it is about | 14:27 |
*** k_mouza_ has quit IRC | 14:27 | |
dtantsur | (don't worry if nothing is clear - we can talk about it) | 14:28 |
*** k_mouza has joined #openstack-ironic | 14:28 | |
cenn | alright. i'll do that. | 14:28 |
openstackgerrit | Lin PeiWen proposed openstack/ironic stable/ussuri: Delete unavailable py2 package https://review.opendev.org/c/openstack/ironic/+/792886 | 14:32 |
*** rloo has joined #openstack-ironic | 14:33 | |
*** rloo has quit IRC | 14:34 | |
*** rloo has joined #openstack-ironic | 14:34 | |
TheJulia | wait --failing | 14:39 |
TheJulia | TIL | 14:39 |
*** dsneddon has joined #openstack-ironic | 14:40 | |
*** dsneddon has quit IRC | 14:47 | |
*** rloo has quit IRC | 14:58 | |
*** rloo has joined #openstack-ironic | 14:58 | |
cenn | dtantsur: hmmm. i see the flag under stestr run (`stestr run --help`) but for some reason it's not working for me. http://paste.openstack.org/show/805701/ | 14:59 |
dtantsur | cenn: have any tests failed for you before? if no, it has no tests to (re-)run | 14:59 |
cenn | oh... | 14:59 |
dtantsur | if you try modifying one test so that it fails, you can check it | 14:59 |
cenn | haha okay. that makes sense | 15:00 |
*** ociuhandu has quit IRC | 15:05 | |
*** ociuhandu has joined #openstack-ironic | 15:05 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Remove inventory time workaround for WS-man BIOS https://review.opendev.org/c/openstack/ironic/+/792142 | 15:05 |
*** ociuhandu has quit IRC | 15:09 | |
*** sshnaidm is now known as sshnaidm|afk | 15:11 | |
*** jawad_ax_ has quit IRC | 15:21 | |
*** sshnaidm|afk is now known as sshnaidm | 15:35 | |
*** ociuhandu has joined #openstack-ironic | 15:36 | |
*** ociuhandu has quit IRC | 15:48 | |
*** ociuhandu has joined #openstack-ironic | 15:50 | |
*** lucasagomes has quit IRC | 15:58 | |
*** paras333_ has quit IRC | 16:08 | |
*** Qianbiao has quit IRC | 16:12 | |
*** tosky has joined #openstack-ironic | 16:14 | |
*** ociuhandu has quit IRC | 16:17 | |
*** ociuhandu has joined #openstack-ironic | 16:17 | |
*** ociuhandu has quit IRC | 16:27 | |
*** ociuhandu has joined #openstack-ironic | 16:29 | |
*** ociuhandu has quit IRC | 16:30 | |
*** ociuhandu has joined #openstack-ironic | 16:30 | |
*** ociuhandu has quit IRC | 16:35 | |
*** k_mouza has quit IRC | 16:37 | |
*** gyee has joined #openstack-ironic | 16:39 | |
rpittau | bye! o/ | 16:41 |
*** rpittau is now known as rpittau|afk | 16:41 | |
*** k_mouza has joined #openstack-ironic | 16:43 | |
*** k_mouza has quit IRC | 16:44 | |
*** k_mouza has joined #openstack-ironic | 16:45 | |
*** uzumaki has joined #openstack-ironic | 16:46 | |
*** k_mouza has quit IRC | 16:48 | |
*** k_mouza has joined #openstack-ironic | 16:49 | |
*** ociuhandu has joined #openstack-ironic | 16:53 | |
*** dtantsur is now known as dtantsur|afk | 16:54 | |
dtantsur|afk | o/ | 16:54 |
*** ociuhandu has quit IRC | 16:58 | |
*** uzumaki has quit IRC | 16:59 | |
TheJulia | reviewjam? | 17:00 |
*** derekh has quit IRC | 17:03 | |
TheJulia | looks like all of the patches are pending changes minus the rbac policy change | 17:03 |
*** hoonetorg has joined #openstack-ironic | 17:25 | |
arne_wiebalck | bye everyone o/ | 17:43 |
*** dsneddon has joined #openstack-ironic | 17:43 | |
openstackgerrit | Merged openstack/ironic master: Remove inventory time workaround for WS-man BIOS https://review.opendev.org/c/openstack/ironic/+/792142 | 18:25 |
*** mnaser has quit IRC | 18:30 | |
*** rnoriega_ has quit IRC | 18:30 | |
*** fdegir has quit IRC | 18:30 | |
*** shadower has quit IRC | 18:30 | |
*** strigazi has quit IRC | 18:30 | |
*** bnemec has quit IRC | 18:30 | |
*** mjturek has quit IRC | 18:30 | |
*** strigazi has joined #openstack-ironic | 18:30 | |
*** mnaser has joined #openstack-ironic | 18:30 | |
*** shadower has joined #openstack-ironic | 18:31 | |
*** fdegir has joined #openstack-ironic | 18:31 | |
*** bnemec has joined #openstack-ironic | 18:31 | |
*** rnoriega_ has joined #openstack-ironic | 18:33 | |
*** k_mouza has quit IRC | 19:09 | |
*** k_mouza has joined #openstack-ironic | 19:10 | |
*** k_mouza has quit IRC | 19:15 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/sushy master: EventService support https://review.opendev.org/c/openstack/sushy/+/792053 | 19:25 |
*** early has quit IRC | 19:29 | |
*** early has joined #openstack-ironic | 19:35 | |
*** EmilienM has joined #openstack-ironic | 20:03 | |
*** lourot has quit IRC | 20:03 | |
*** k_mouza has joined #openstack-ironic | 20:04 | |
*** lourot has joined #openstack-ironic | 20:06 | |
*** lourot has quit IRC | 20:07 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-specs master: Event Subscription Spec https://review.opendev.org/c/openstack/ironic-specs/+/785742 | 20:07 |
*** lourot has joined #openstack-ironic | 20:07 | |
*** k_mouza has quit IRC | 20:10 | |
*** lourot has quit IRC | 20:10 | |
*** lourot has joined #openstack-ironic | 20:12 | |
iurygregory | WOOT?! CI finished a job in 42min?O.o | 20:13 |
iurygregory | oh ok normally is 50min so the sushy jobs | 20:13 |
*** lourot has quit IRC | 20:17 | |
*** lourot has joined #openstack-ironic | 20:18 | |
trandles | Is there a SPUC this week (11am MDT)? | 20:25 |
*** Perfidious has joined #openstack-ironic | 20:34 | |
Perfidious | Any here free to answer few question about ironic issues I am having? | 20:35 |
Perfidious | I am having issue with "No valid hosts found". I am following troubleshooting guide and first thing I tried is to validate my ironic node. It said that ```| boot | False | Cannot validate image information for node 2b3df5e1-7c21-48a9-a60d-39e3a35c56c6 because one or more parameters are missing from its instance_info and insufficent | 20:43 |
Perfidious | information is present to boot from a remote volume. Missing are: ['ramdisk', 'kernel', 'image_source'] | | 20:43 |
Perfidious | | console | False | Driver ipmi does not support console (disabled or not implemented). | 20:43 |
Perfidious | | | 20:43 |
Perfidious | | deploy | False | Cannot validate image information for node 2b3df5e1-7c21-48a9-a60d-39e3a35c56c6 because one or more parameters are missing from its instance_info and insufficent information is present to boot from a remote volume. Missing are: ['ramdisk', 'kernel', 'image_source'] | | 20:43 |
Perfidious | ``` | 20:43 |
Perfidious | Also have this error: Failed to provision instance 995a7108-2da3-42a5-bd15-6fe88f26adce: Failed to prepare to deploy: Image 99ff786e-2490-4312-95c1-2642a1e8605c could not be found. That image is located in /tftpboot/master_images but not in openstack image list. How do I add it there? | 20:43 |
TheJulia | Perfidious: so... hmm | 20:55 |
TheJulia | Perfidious: so you need an image_source setting in the node's instance_info to tell it *what* to deploy | 20:55 |
TheJulia | and that image will need to be in glance, or you'll need to provide it a file path url on the conductor (if just a single conductor) via image_source, or as an https url and an image_checksum setting will then be required so we validate the image has not been modified as a bit of a safeguard | 20:56 |
Perfidious | I thought I do that openstack server start command, there I provide an image that is in glance | 20:57 |
*** larsks has quit IRC | 20:57 | |
*** larsks has joined #openstack-ironic | 20:58 | |
Perfidious | Our ironic is part of openstack environment and not stand alone. | 20:58 |
Perfidious | The error about with image 99 is deploy-ramdisk image and not the image that will be loaded on the server. | 20:59 |
NobodyCam | Good Afternoon Ironic Folks | 21:05 |
* NobodyCam shakes fist at our state machine | 21:11 | |
TheJulia | Perfidious: so nova populates that then | 21:19 |
TheJulia | so... your deploy_kernel and deploy_ramdisk in your driver_info, are they a file url, http url, glance image id? | 21:20 |
*** pmannidi has quit IRC | 21:21 | |
*** pmannidi has joined #openstack-ironic | 21:22 | |
*** paras333_ has joined #openstack-ironic | 21:25 | |
*** paras333_ has quit IRC | 21:30 | |
stevebaker | TheJulia: so the vexxhost raid centos8-stream issue is also the grub2-install issue | 21:33 |
*** k_mouza has joined #openstack-ironic | 21:36 | |
*** k_mouza has quit IRC | 21:41 | |
*** ociuhandu has joined #openstack-ironic | 22:07 | |
*** ociuhandu has quit IRC | 22:11 | |
NobodyCam | hummers' getting the a node into error state is more difficult then anticipated | 22:23 |
NobodyCam | s/hummer's/humm/ # darn autocorrect | 22:23 |
Perfidious | They are glance image id I assume, but it's missing from glance. How do I add it back? | 22:43 |
Perfidious | And does ramdisk load which then loads the image I want? | 22:46 |
NobodyCam | Perfidious: Yes the IPA will deploy the image requested | 22:47 |
NobodyCam | TheJulia: Thought question. what would your thoughts be if someone wanted to have the ironic nova driver move a node into error state | 22:53 |
Perfidious | Thank you. I will try to re-add ramdisk image to glance and if it doesn't work, I will try to replace ramdisk glance id with file path in driver_info | 22:57 |
NobodyCam | I don't believe file path will work | 23:00 |
*** tosky has quit IRC | 23:03 | |
Perfidious | hmm. Will have to add it to glance somehow | 23:18 |
TheJulia | stevebaker: lovely :( | 23:25 |
TheJulia | Perfidious: you can do openstack image update I believe | 23:26 |
TheJulia | or upload and replace, there is a slightly funky pattern to replace an existing glance image | 23:26 |
TheJulia | and by funky I mean it was once disjointed steps and now I think it can be one | 23:27 |
TheJulia | or maybe that is vise versa | 23:27 |
TheJulia | NobodyCam: can this question involve food and a beverage, it has been a long day? | 23:27 |
NobodyCam | :) it can and it doesn't have to be right now | 23:28 |
TheJulia | I guess my question is why does nova need to declare error state | 23:28 |
TheJulia | in terms of what does it know that we don't know | 23:28 |
Perfidious | thank you all for your help. I now have few ways forward | 23:29 |
NobodyCam | that's where I 've been hacking placement record removal, but I got a comment that folks would like the node to be placed in maintenance, (which I can do from the nova driver.. But that ends up leaving the node in a clean wait .. eventually clean_fail | 23:30 |
NobodyCam | so I wanted to move the node in to error... (/me really feels it should have been deletefail, but that's just me) | 23:32 |
TheJulia | NobodyCam: because it is in maintenance state, the heartbeats can't actually do anything | 23:34 |
TheJulia | so nova is supposed to be excluding nodes in non-available or active states if memory serves | 23:34 |
* TheJulia is soooo confused and may just need beer | 23:48 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!