| opendevreview | melanie witt proposed openstack/nova master: zuul: Add devstack-plugin-nfs-tempest-full to the check queue https://review.opendev.org/c/openstack/nova/+/781139 | 00:38 |
|---|---|---|
| opendevreview | melanie witt proposed openstack/nova master: libvirt: stop using connection_info for NFS file format https://review.opendev.org/c/openstack/nova/+/858836 | 00:38 |
| *** dasm is now known as dasm|off | 04:51 | |
| opendevreview | Eigil Obrestad proposed openstack/nova-specs master: Compute Inventory Customization https://review.opendev.org/c/openstack/nova-specs/+/858858 | 06:51 |
| obre | bauzas: Does the following spec/blueprint make sense for you? ^ | 06:52 |
| obre | Ill tried my best to follow your process, but Ill never done anythng like this before; so I would happily accept any guiding if there are things Im doing wrong. | 06:53 |
| obre | And; the progress are a bit slow as I am doing this work between other duties. | 06:54 |
| gibi | obre: o/ I'm reading the spec now | 07:15 |
| gibi | and dont worry about progress, you proposed the spec quickly so we still remember the context from tuesday, and it is the good time in the cycle to propose specs :) | 07:20 |
| obre | Also; Zuul doesnt like some whitespaces on the end of lines in the spec. When fixing this; do I create a new commit, or ammend to the existing one? So; "git add, git commit, git review" or "git add, git commit --amend, git review" or something else? | 07:22 |
| gibi | obre: amned the exisiting commit. Gerrit uses the Change-Id to identify different versions of the same review | 07:27 |
| opendevreview | Eigil Obrestad proposed openstack/nova-specs master: Compute Inventory Customization https://review.opendev.org/c/openstack/nova-specs/+/858858 | 07:41 |
| obre | gibi: ack | 07:42 |
| gibi | obre: I left feedback in the spec. I think we need a bit more details but agree with what you have written so far | 07:54 |
| gibi | sean-k-mooney[m]: bauzas: ^^ that is an easy read to start the day with :) | 07:55 |
| bauzas | obre: gibi: sorry, I'm on a meeting, but thanks for the spec, I'll look at it | 08:05 |
| obre | bauzas: No need to say sorry. I am not able to be present here at all times, so I do not expect others to be present at all time either :P | 08:07 |
| gibi | nah, everybody needs time to sleep ... at lest 4 hours a day :P | 08:14 |
| sean-k-mooney[m] | left some comments | 08:28 |
| sean-k-mooney[m] | i should proably get up now instead of lying in bed ya. casue its really comfy and soft and warm, but on the other had coffee is down stairs | 08:30 |
| gibi | I think you set up the right motivation structure there :) | 08:31 |
| sean-k-mooney | i made some bread while i was getting coffee too. | 09:09 |
| kashyap | Toasted? | 09:20 |
| kashyap | sean-k-mooney: Wait, you _made_ bread that quick? | 09:21 |
| kashyap | No letting the dough rise? :) | 09:21 |
| kashyap | (Unless it already "rose" overnight) | 09:21 |
| sean-k-mooney | its currently proofing | 09:21 |
| sean-k-mooney | alhtough if i was making irish soda bread it woudl be in the oven already | 09:21 |
| kashyap | I hear first time of this "soda bread". Doesn't sound very appealing by the name of it, TBH :P | 09:28 |
| sean-k-mooney | its what i think of as bread and i think of yeast bread as french bread in my head | 09:28 |
| sean-k-mooney | kashyap: no yeast you use bread soda/baking soda and butter milk instead | 09:29 |
| kashyap | I see | 09:29 |
| kashyap | Oh, like that, I see | 09:29 |
| sean-k-mooney | the butter milk is acidic and the bread soda is basica so it release co2 | 09:29 |
| * kashyap nods; today I learnt :) | 09:29 | |
| sean-k-mooney | so you just mix it up put it in the oven and it rise as it bakes | 09:30 |
| sean-k-mooney | often we put fruit or oats in it and it can be white or brown although tradtionaly its brown just because that was the type of flour that was most avaiable | 09:31 |
| sean-k-mooney | https://www.simplyrecipes.com/recipes/irish_soda_bread/ | 09:32 |
| kashyap | sean-k-mooney: Thank you! (I still don't have an oven, I'm afraid. "Once I move" - I keep telling myself) | 09:37 |
| sean-k-mooney | hehe i have two. when i was buying a micorwave for my house i got one that is also an oven and a grill and has a dough proving setting :) | 09:38 |
| * bauzas missed the whole convo about bread | 09:39 | |
| sean-k-mooney | i dont use the other fucntionst that often but its nice to have the option | 09:39 |
| * bauzas now gets his beret and his baguette and go getting his kid from school | 09:39 | |
| sean-k-mooney | hehe so you missed the part where i creditis all yest breads to france ya :P | 09:40 |
| sean-k-mooney | in ireland i always associated yeast breads with thing you get form a bakery or store and home mande bread is soda bread | 09:41 |
| sean-k-mooney | that not entirly true of course, there are irish yest bread but if your making bread with kids ectra or you by bread mixes in irland they dont normally use yeast | 09:41 |
| sean-k-mooney | so yeast bread were always in my experince the more advanced/difficult breads to make | 09:42 |
| sean-k-mooney | now that i have got used to them its arguable simpler just takes more time | 09:42 |
| sean-k-mooney | i made these for emmas house warming last weekend https://photos.app.goo.gl/HMXtgFFKL6cvKAc46 | 09:45 |
| bauzas | sean-k-mooney: awesome work | 09:59 |
| sean-k-mooney | the last one is still technialy bread and not pastry. its a brioche cinnamon nut sticky bun rather then a danish style pastery | 10:00 |
| sean-k-mooney | with a honney butter glaze | 10:00 |
| sean-k-mooney | https://www.youtube.com/watch?v=JlzmsTiOJnU i adapted that because i was out of maple if you want to try them they were really tasty | 10:02 |
| bauzas | :) | 10:02 |
| bauzas | I don't usually make my own bread but did it a couple of times | 10:03 |
| bauzas | this is just, bread is cheap here compared to the time it takes to make it | 10:03 |
| bauzas | also, the white bread is slightly different from the one I can do | 10:04 |
| bauzas | because of the yeast | 10:04 |
| bauzas | and the flour | 10:04 |
| sean-k-mooney | yep even ehere its not that expensive | 10:04 |
| sean-k-mooney | ya we orderd special bread flour form a mill in bulk | 10:04 |
| bauzas | yeah | 10:05 |
| bauzas | people not having close bakeries do their own breads | 10:05 |
| bauzas | for me, I have 3 different bakeries in a 1km area, and 10 more if I'm driving 5 mins | 10:06 |
| bauzas | so... | 10:06 |
| bauzas | sean-k-mooney: but anyway, your bread looked very nice | 10:06 |
| sean-k-mooney | we got https://kellswholemeal.ie/product/native-purple-wheat-flour-12-5kg/ and https://kellswholemeal.ie/product/marriages-uppermost-flour-16kg/ | 10:06 |
| bauzas | in general, the homemade breads don't look like this | 10:07 |
| bauzas | yours seems better looking | 10:07 |
| bauzas | but I guess your shipping rates are expensive for sending to France :p | 10:07 |
| sean-k-mooney | thanks i have only started making them this year emma has baked bread for years but i was afraid fo messing it up with yeast so never did | 10:08 |
| bauzas | anyway, /me goes cooking for the kid | 10:09 |
| sean-k-mooney | now if only nova had a feature that could make bread for me | 10:09 |
| bauzas | sean-k-mooney: write a spec | 10:09 |
| sean-k-mooney | maybe we could use flavors some how to model the recipe | 10:09 |
| sean-k-mooney | we already know placment is good at trackign whats in your fridge | 10:09 |
| bauzas | and we have placement for the ingredients | 10:09 |
| bauzas | that | 10:09 |
| sean-k-mooney | so we need a bread maker virt dirver to talk to a smart bread maker.... | 10:10 |
| bauzas | but, eventually, 3D-printing the bread doesn't seem a good opportunity | 10:10 |
| bauzas | we need robots | 10:10 |
| sean-k-mooney | yep i think this si clearly the highest priorty for the nova team | 10:10 |
| obre | I believe we have other "computers -> physical world" interfaces than 3d-printers. | 10:10 |
| bauzas | sean-k-mooney: I'll propose this for the sutainability SIG | 10:11 |
| bauzas | sean-k-mooney: I always said Placement can solve many problems, including the world hunger | 10:11 |
| bauzas | read Asimov | 10:12 |
| bauzas | not about the robots but the hydroponic cultures | 10:12 |
| sean-k-mooney | bauzas: can you hit this https://review.opendev.org/c/openstack/placement/+/857900 | 11:38 |
| sean-k-mooney | we need it for https://review.opendev.org/c/openstack/releases/+/858866 | 11:38 |
| *** ozzzo1 is now known as ozzzo | 11:38 | |
| sean-k-mooney | bauzas: we are goning to need an rc2 for placment | 11:47 |
| sean-k-mooney | well maybe not | 11:47 |
| sean-k-mooney | https://github.com/openstack/placement/blob/master/bindep.txt#L31-L36 | 11:47 |
| sean-k-mooney | on master i need to fix the bindeps that is wrong and really out of data | 11:47 |
| sean-k-mooney | *date | 11:47 |
| sean-k-mooney | we should fix it on stable/zed too but it does not nessisarly need to be RC2 | 11:48 |
| sean-k-mooney | we could fix it after the offical release | 11:48 |
| sean-k-mooney | im going to go fix it now | 11:48 |
| sean-k-mooney | and file a bug and we can decided then if we do an RC2 or not | 11:48 |
| sean-k-mooney | ill propose a backport in eitehr case and we can either hold it until after the reelase of merge it depening on how we feel about this | 11:49 |
| sean-k-mooney | this bindep will not install on ubuntu 22.04 | 11:49 |
| sean-k-mooney | which is what we use for the python 3.10 jobs | 11:49 |
| sean-k-mooney | python-dev does not exist in 22.04 | 11:50 |
| bauzas | sean-k-mooney: damn | 12:02 |
| bauzas | sean-k-mooney: you can review this too https://review.opendev.org/c/openstack/nova/+/858087/1 | 12:04 |
| opendevreview | Merged openstack/nova master: Update master for stable/zed https://review.opendev.org/c/openstack/nova/+/858084 | 12:05 |
| bauzas | sean-k-mooney: add your thoughts on the bindep in https://etherpad.opendev.org/p/nova-zed-rc-potential | 12:05 |
| sean-k-mooney | oh yes ill do that now | 12:05 |
| sean-k-mooney | done | 12:05 |
| opendevreview | Merged openstack/placement master: Update master for stable/zed https://review.opendev.org/c/openstack/placement/+/857900 | 12:12 |
| opendevreview | sean mooney proposed openstack/placement master: update bindep for ubuntu 22.04 https://review.opendev.org/c/openstack/placement/+/858927 | 13:19 |
| sean-k-mooney | bauzas: ^ that shoudl be enouch to resolve the issue | 13:20 |
| bauzas | hmm, ok, will look | 13:24 |
| sean-k-mooney | stephenfin: bauzas so there are 3 fix patches https://review.opendev.org/c/openstack/placement/+/829573 for rhel only https://review.opendev.org/c/openstack/placement/+/845074 for debin only that just need 1 +2 by the way | 13:25 |
| sean-k-mooney | and my patch which fixes both | 13:25 |
| sean-k-mooney | we could mege both of the older patches or merge mine | 13:27 |
| bauzas | sean-k-mooney: I have a meeting in 3 mins, so please discuss this in 30 mins | 13:27 |
| sean-k-mooney | i need to go do other stuff so please let me know how what ye want to do | 13:28 |
| sean-k-mooney | sure | 13:28 |
| opendevreview | Merged openstack/nova master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/nova/+/858087 | 13:42 |
| stephenfin | sean-k-mooney: Can you drop the precise identifiers also? https://review.opendev.org/c/openstack/placement/+/858927 | 13:46 |
| sean-k-mooney | oh ya sure ill do that now | 13:54 |
| sean-k-mooney | im going to replace it with test by the way since we currently realy on it being there implictly | 13:55 |
| opendevreview | sean mooney proposed openstack/placement master: update bindep for ubuntu 22.04 https://review.opendev.org/c/openstack/placement/+/858927 | 13:58 |
| sean-k-mooney | stephenfin: ^ happy with that if not ill resping again quickly | 13:58 |
| stephenfin | sean-k-mooney: +2 though I do have more comments that might warrant a respin. Will re +2 if you do | 14:08 |
| sean-k-mooney | im currently doing a review but i am also heere so let me take a look | 14:08 |
| sean-k-mooney | ya | 14:09 |
| sean-k-mooney | so im fine with droping python | 14:09 |
| sean-k-mooney | becuase you knwo bindep needs it | 14:09 |
| sean-k-mooney | so if you dont know why we have it i can remove it but i think its to ensure we have all the python version we need | 14:09 |
| sean-k-mooney | i.e. i think the tox job does not preinstall the python versoin for the current job | 14:10 |
| sean-k-mooney | and bindep is isntalling all of them so when tox runs the required version is there | 14:11 |
| sean-k-mooney | does that make sesne | 14:11 |
| sean-k-mooney | actully no | 14:12 |
| sean-k-mooney | https://packages.ubuntu.com/focal/python3-all | 14:12 |
| sean-k-mooney | so its just pulling the default python looking at the deps | 14:12 |
| sean-k-mooney | althoguh that might be wrong | 14:12 |
| sean-k-mooney | package depending on all supported Python 3 runtime versions | 14:12 |
| sean-k-mooney | so that appears to by why we are using it | 14:13 |
| *** dasm|off is now known as dasm | 14:17 | |
| bauzas | sean-k-mooney: I'm done with my meeting but I need to go taxidading | 14:19 |
| bauzas | I'll look at your patches when I'm back | 14:19 |
| zigo | My 1st Zed VM is up and pings ... :P | 14:40 |
| sean-k-mooney | :) | 14:41 |
| gmann | sean-k-mooney: I thought it was merged. gibi stephenfin please check this https://review.opendev.org/c/openstack/placement/+/845074 | 14:49 |
| gmann | or bauzas ^^ | 14:50 |
| stephenfin | gmann: We were just talking about that. https://review.opendev.org/c/openstack/placement/+/858927 does the same but for RHEL/Fedora also | 14:55 |
| stephenfin | Is this blocking the gate? | 14:55 |
| gmann | i think py310 job is failing whihc is voting now | 14:56 |
| frickler | it is blocking https://review.opendev.org/c/openstack/placement/+/857901 which should be merged before anything else in master | 14:57 |
| stephenfin | gmann: Are you core on placement? Want to look over this? https://review.opendev.org/c/openstack/placement/+/858927 | 14:58 |
| stephenfin | I +W'd it to unblock the gate but it's always good to have another pair of eyes | 14:58 |
| gmann | sure | 14:58 |
| gmann | done | 14:59 |
| opendevreview | Stephen Finucane proposed openstack/placement master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/placement/+/857901 | 14:59 |
| stephenfin | ty | 14:59 |
| stephenfin | rebased that and +W so they should merge together (CC: Frickler) | 15:00 |
| bauzas | sean-k-mooney: gmann: stephenfin: fwiw, +Wd https://review.opendev.org/c/openstack/placement/+/858927 | 15:17 |
| bauzas | tbc, we don't need to backport it to Zed, right? | 15:18 |
| bauzas | this is just for Antelope | 15:18 |
| gmann | bauzas: yes. only for Antelope | 15:18 |
| bauzas | yeah, just verified https://governance.openstack.org/tc/reference/runtimes/zed.html | 15:19 |
| bauzas | 20.04 | 15:19 |
| bauzas | we're all good | 15:19 |
| melwitt | sean-k-mooney: I wanted to get your thoughts on this bug fix, do you think this is an ok thing to do or any problems with it? I was unsure about whether it's proper to use qemu-img info to determine the disk format of a volume https://review.opendev.org/c/openstack/nova/+/858836 | 16:00 |
| sean-k-mooney | is this related ot luks | 16:02 |
| melwitt | sean-k-mooney: no, it's just nfs | 16:02 |
| sean-k-mooney | oh no its differnt | 16:02 |
| sean-k-mooney | ya i tough i swa an nfs related patch for something like this | 16:02 |
| sean-k-mooney | oh i rememebr the bug | 16:02 |
| sean-k-mooney | so snapshots on nfs have to be qcow right | 16:03 |
| melwitt | right | 16:03 |
| sean-k-mooney | like the os-assisted-snapshost api say it must be that | 16:03 |
| sean-k-mooney | ok | 16:03 |
| melwitt | yes it does | 16:03 |
| sean-k-mooney | so when i was looking at the snapshot code i think i say we shoudl usjt update it to be qcow | 16:04 |
| melwitt | but on our side we store the volume attachment connection_info and the format captured there represents the format of the volume (which is raw) not the snapshot (which is qcow2) | 16:04 |
| sean-k-mooney | ya | 16:04 |
| melwitt | yeah, the thing is how to "remember" that when the instance is rebooted for example | 16:05 |
| sean-k-mooney | let me see if i can find the code i was suggesting changing but basicaly i was suggestin updating the format on the image | 16:05 |
| sean-k-mooney | what your dowing sound valid | 16:06 |
| sean-k-mooney | just not sure if its requried | 16:06 |
| melwitt | oh, hm | 16:06 |
| sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L2932-L2958 | 16:07 |
| sean-k-mooney | i tought we could fix that in that code | 16:08 |
| melwitt | ok, I will look at that. thanks for the pointer | 16:09 |
| sean-k-mooney | honestly i think when we create teh snapshot the connection info should get updated to qcow | 16:10 |
| melwitt | sean-k-mooney: I tried that first https://review.opendev.org/c/openstack/cinder/+/857528 and it's run into problems | 16:11 |
| melwitt | if you have any thoughts on that one as well | 16:11 |
| sean-k-mooney | well the backend driver should be reprotign it as qcow no? | 16:12 |
| melwitt | because unless the connection_info gets updated in cinder, we will overwrite it when we do refresh connection_info | 16:12 |
| sean-k-mooney | like its a bug for them to be returnign raw after the snapshot is created | 16:12 |
| melwitt | cinder only keeps the volume format, not the snapshot format | 16:12 |
| melwitt | that's the only thing it knows how to return | 16:12 |
| sean-k-mooney | right so the volume format should be updated once we snapshot right | 16:13 |
| sean-k-mooney | because it has now changed | 16:13 |
| melwitt | yeah, so I tried that and it broke volume extend | 16:13 |
| sean-k-mooney | fun | 16:13 |
| melwitt | because the resize needs to change the size of the raw backing file iiuc | 16:13 |
| sean-k-mooney | so we could work around this in nova but i think we shoudl also bring this up with cinder in the ptg | 16:14 |
| melwitt | so if we stored qcow2, that blows up when it tries to resize raw by specifying qcow2 | 16:14 |
| sean-k-mooney | well extend is not supported with nfs | 16:14 |
| sean-k-mooney | so that shoudl not be a problem | 16:14 |
| melwitt | it is | 16:14 |
| sean-k-mooney | its not | 16:14 |
| sean-k-mooney | we have a spec and open bug for it | 16:14 |
| melwitt | it's in the nfs job, that's how I know it failed :P | 16:14 |
| sean-k-mooney | well its not supported | 16:14 |
| melwitt | ? | 16:15 |
| sean-k-mooney | https://bugs.launchpad.net/cinder/+bug/1870367 | 16:15 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova-specs/+/855490 | 16:15 |
| sean-k-mooney | that is the spec to add extend support in A | 16:16 |
| melwitt | huh. | 16:16 |
| melwitt | I wonder what/how it's running the tests on nfs and passing currently | 16:17 |
| sean-k-mooney | it can work but i think its racy | 16:17 |
| melwitt | ugh, ok. | 16:17 |
| sean-k-mooney | basicaly they tought they fixed it | 16:17 |
| sean-k-mooney | but the external event is async | 16:17 |
| sean-k-mooney | so there is no way for cidner ot know if it works or not | 16:17 |
| sean-k-mooney | there is also https://bugs.launchpad.net/cinder/+bug/1978294 | 16:18 |
| melwitt | yeah.. I have seen that but I didn't remember it when I saw the extend test fail | 16:19 |
| melwitt | I'll add something to the ptg agenda about this | 16:21 |
| melwitt | bc even if we skipped the extend tests for nfs, changing the actual volume format in the metadata to qcow2 afaik isn't correct because it's the snapshot that is qcow2 and the volume itself is still raw. so we're still stuck | 16:22 |
| stephenfin | melwitt: Random question. It appears the '<class>' argument to 'nova quota-class-show <class>' doesn't do anything. Is that expected? | 16:33 |
| sean-k-mooney | melwitt: see the way this works with nfs that is not reallly true | 16:34 |
| melwitt | stephenfin: I don't think that's expected. are you running like 'nova quota-class-show default'? | 16:34 |
| sean-k-mooney | when we create the snapshot the snapshot point to the orginal file and the vm is now runnign form the qcow that is created on top right | 16:34 |
| gibi | gmann, stephenfin: sorry I focused elsewhere. I see stephenfin approved it now | 16:35 |
| melwitt | stephenfin: bc default is the only quota class that automatically exists, any others have to be created by the admin user | 16:35 |
| melwitt | sean-k-mooney: yes right | 16:35 |
| sean-k-mooney | so the snapshot is actully raw and the volume is now qcow? | 16:35 |
| stephenfin | https://paste.opendev.org/show/bUrfKrK6aKCSbqfRJzNZ/ | 16:35 |
| sean-k-mooney | so its the revers of what i sugeste orginally the snapshot remaisn the same but the volume format changes | 16:35 |
| stephenfin | melwitt: ^ | 16:35 |
| sean-k-mooney | stephenfin: why are you using 2.1 | 16:37 |
| stephenfin | to make sure we hadn't broken things in a newer microversion | 16:37 |
| melwitt | stephenfin: ok, I _think_ what that's doing is if you pass a class that doesn't exist, it will show you the default (which is not super helpful, but is accurately showing what would be used if you tried to use a nonexistent quota class) | 16:37 |
| stephenfin | what would a real class be? | 16:37 |
| melwitt | stephenfin: if you create a new class and put different values in it and then show it I think (hopefully) it would show you that new class values | 16:37 |
| melwitt | stephenfin: only default out of the box but you can create quota classes, that's the only way you can get other quota classes | 16:38 |
| stephenfin | Ah, apparently *only* 'default' is supported https://docs.openstack.org/nova/latest/admin/quotas.html | 16:39 |
| stephenfin | Note | 16:39 |
| stephenfin | Only the default class is supported by nova. | 16:39 |
| * stephenfin has literally never played around with quotas before, somehow | 16:40 | |
| melwitt | yeah, at some point in the past we decided that bc quota classes was a rax specific thing they were doing with an external service/system that they had | 16:41 |
| melwitt | and as far as we knew no one else ever used it | 16:41 |
| stephenfin | Nope, tell a lie. Apparently I wrote the quota docs | 16:41 |
| stephenfin | Jaysus | 16:41 |
| melwitt | :) | 16:41 |
| stephenfin | Okay, so that argument means diddly squat in practice | 16:42 |
| melwitt | so, you're right we don't support it but if you were curious how the command could work, that's how I remember it working | 16:42 |
| stephenfin | Right. I must check if any of this is relevant for neutron or cinder. If not, I might get the interns to deprecate all the class-based stuff in OSC. It's just confusing | 16:43 |
| stephenfin | melwitt++ thanks :) | 16:43 |
| melwitt | stephenfin: yeah, I think it's pretty safe to do that. technically someone could change the default quota class values and nova would use that if it's in the right order of precedence ... but I don't think that really helps anyone | 16:44 |
| melwitt | just increases confusion | 16:44 |
| stephenfin | i.e. using 'quota-class-update'? | 16:45 |
| melwitt | yes | 16:45 |
| stephenfin | Okay. I suspect re-implementing that as e.g. 'openstack quota set --default --instances $INSTANCES' or 'openstack default quota set --instances $INSTANCES' would make more sense | 16:46 |
| stephenfin | and deprecate (for removal) all references to quota classes | 16:46 |
| stephenfin | Sound reasonable? | 16:47 |
| melwitt | yeah, I think that makes sense. quota classes is how you can change defaults over the API (as opposed to the config options). so maybe people do do that (?) | 16:47 |
| stephenfin | yeah, I've no idea, but at least this would be a little more discoverable/require less historical knowledge | 16:49 |
| stephenfin | one more thing: you can set your own quota on a per project basis. What do we call those quotas? Custom quotas? Overridden quotas? Project-specific quotas? | 16:49 |
| melwitt | sean-k-mooney: I don't think it's the reverse ... i.e if you qemu-img info <volume path> it returns raw if you qemu-img info <snapshot path> it returns qcow2. unless I'm just totally misunderstanding something | 16:49 |
| melwitt | stephenfin: the last one, project quotas | 16:50 |
| stephenfin | ta | 16:50 |
| melwitt | sean-k-mooney: <source file='/opt/stack/data/nova/mnt/896fb15da6036b68a917322e72ebfe57/volume-89113873-5c74-4980-8396-f876b7b5101c'/> vs <source file='/opt/stack/data/nova/mnt/896fb15da6036b68a917322e72ebfe57/volume-89113873-5c74-4980-8396-f876b7b5101c.484f7406-3169-4ea5-afda-a7b4657c4d4f' index='1'/> | 16:52 |
| melwitt | the latter is what the instance points to after the snapshot and that path/file format is qcow2 | 16:53 |
| sean-k-mooney | melwitt: so when we create a shapshot we are then running form the delta disk | 17:38 |
| melwitt | sean-k-mooney: right | 17:39 |
| sean-k-mooney | yes so the volume is not the new file in qcow format | 17:42 |
| sean-k-mooney | and the snapshot is the old file | 17:42 |
| sean-k-mooney | because if i boot a second vm form the snapshot | 17:42 |
| sean-k-mooney | i should really get the old files content | 17:42 |
| sean-k-mooney | and creatign the new voluem shoudl create a second deleta disk | 17:43 |
| melwitt | oh, ok I think I see what you're saying | 17:43 |
| sean-k-mooney | its kind of the reverse of what you woudl expect | 17:43 |
| melwitt | yeah. I have clearly been confused by all of this 😆 | 17:44 |
| sean-k-mooney | normally we upload a new image to glance with the delta form the base file | 17:44 |
| sean-k-mooney | but the base file does not change with glance | 17:44 |
| sean-k-mooney | but with a voluem it writable | 17:44 |
| sean-k-mooney | so the volume becomes the new file | 17:45 |
| sean-k-mooney | and the snapshot is the old file | 17:45 |
| sean-k-mooney | that i think is how we should look at it but maybe that is not how cinder looks at it | 17:46 |
| sean-k-mooney | to me the voluem is the thing attached to the vm and the snapshot is the backing file | 17:47 |
| melwitt | yeah, it is presented that way as in, the instance remains attached to the same volume uuid, even after snapshots | 17:48 |
| melwitt | (when you look at server show, for example) | 17:49 |
| melwitt | that's part of why it confuses me bc it's attached to the volume but then the xml points at the delta | 17:49 |
| sean-k-mooney | yep so that is why we probly need to change the forma on the volume | 17:54 |
| sean-k-mooney | and keep the format of the snapshot at the current romat of the disk | 17:54 |
| sean-k-mooney | a second snapshot will result in the qcow the vm is now using being the snapshot disk | 17:54 |
| sean-k-mooney | so the second snabp shot format will be qcow | 17:54 |
| sean-k-mooney | but that ok | 17:54 |
| sean-k-mooney | the rule is | 17:55 |
| sean-k-mooney | the format of the snapshot is the current volume format | 17:55 |
| sean-k-mooney | and the volmue format after snapstho is always qcow | 17:55 |
| melwitt | sean-k-mooney: ok. so PS1 of my cinder patch was likely the right approach. and the volume extend failure is expected and should be skipped if nfs is being used, until that spec you linked earlier is implemented (?) | 17:57 |
| sean-k-mooney | i belive so | 18:00 |
| sean-k-mooney | the patch you linked me would also proably work as a workaround | 18:00 |
| sean-k-mooney | but i dont belive it would be the right long term solution | 18:00 |
| melwitt | gotcha | 18:00 |
| sean-k-mooney | o/ | 18:11 |
| opendevreview | Merged openstack/placement master: update bindep for ubuntu 22.04 https://review.opendev.org/c/openstack/placement/+/858927 | 22:23 |
| opendevreview | Merged openstack/placement master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/placement/+/857901 | 22:24 |
| *** dasm is now known as dasm|off | 22:33 | |
| opendevreview | melanie witt proposed openstack/nova stable/yoga: Unify placement client singleton implementations https://review.opendev.org/c/openstack/nova/+/858997 | 23:07 |
| opendevreview | melanie witt proposed openstack/nova stable/yoga: Avoid n-cond startup abort for keystone failures https://review.opendev.org/c/openstack/nova/+/858998 | 23:07 |
| opendevreview | melanie witt proposed openstack/nova stable/xena: Unify placement client singleton implementations https://review.opendev.org/c/openstack/nova/+/858999 | 23:12 |
| opendevreview | melanie witt proposed openstack/nova stable/xena: Avoid n-cond startup abort for keystone failures https://review.opendev.org/c/openstack/nova/+/859000 | 23:12 |
| opendevreview | melanie witt proposed openstack/nova stable/wallaby: Unify placement client singleton implementations https://review.opendev.org/c/openstack/nova/+/859001 | 23:23 |
| opendevreview | melanie witt proposed openstack/nova stable/wallaby: Avoid n-cond startup abort for keystone failures https://review.opendev.org/c/openstack/nova/+/859002 | 23:23 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!