| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-10-05 | |||
| 17:36:05 | sean-k-mooney | so instead of inheriting form NovaPersistentObject the manila share ones could inherit form ovoo_base.TimestampedObject which is also called NovaTimestampObject | |
| 17:36:23 | sean-k-mooney | we dont need to add the mixin for the orgianl usecse | |
| 17:36:49 | stephenfin | Same thing, different approach. If I was Uggla, I'd probably do that to avoid this blocking things | |
| 17:36:58 | sean-k-mooney | gibi: Uggla did i miss why we are not just doing that | |
| 17:37:35 | sean-k-mooney | stephenfin: thats basically what i suggested a month ago https://review.opendev.org/c/openstack/nova/+/854355/5#message-839b7637eabf1d6dcf4e95050e9244c954dc1056 | |
| 17:37:55 | sean-k-mooney | at that time i did not see that we had NovaTimestampObject | |
| 17:38:03 | sean-k-mooney | and did not need ot intoduce a new class at all | |
| 17:38:43 | stephenfin | That change does still make sense though. As unlikely as it is that anyone will bump those major versions, it is confusing and the TODOs are helpful to highlight that | |
| 17:39:15 | sean-k-mooney | i dont dissagree that we might also want to do this | |
| 17:39:24 | sean-k-mooney | but i dont think we want to do this in the manilla share seriese | |
| 17:39:30 | stephenfin | agreed | |
| 17:39:39 | sean-k-mooney | which si why i ask Uggla to not do this when i first reviewed | |
| 17:40:34 | gibi | I cannot recall the reason we went that way | |
| 17:40:47 | gibi | probably to fix the other ovos where we can fix | |
| 17:41:01 | sean-k-mooney | gibi: Uggla orgingial patch alredy had updated all the other objects | |
| 17:41:02 | Uggla | sean-k-mooney, stephenfin I tried to make objects without soft delete the default behavior to avoid future misused. I also started with a less intrusive patch, but gibi wanted to extend it. | |
| 17:41:39 | sean-k-mooney | Uggla: ya but if we did that it shoudl be a sperate blueprint/spec not mixed into your share work | |
| 17:43:37 | sean-k-mooney | bauzas: melwitt when i mentioned stpehens change in the bug call by the way i ment https://review.opendev.org/c/openstack/nova/+/860401 | |
| 17:43:52 | sean-k-mooney | not https://review.opendev.org/c/openstack/nova/+/854355 | |
| 17:44:07 | sean-k-mooney | i just saw that stephen updated that also | |
| 17:44:45 | melwitt | sean-k-mooney: thanks | |
| 17:51:10 | Uggla | sean-k-mooney, ok if it is needed. | |
| 22:17:19 | atmark | I messed up nova cells and now computes states are down. I corrected the cells but they still show as down. How can I reregister the computes? | |
| 22:17:46 | atmark | This is a new deployment | |
| 22:18:44 | atmark | `openstack compute service delete id` won't let me delete | |
| #openstack-nova - 2022-10-06 | |||
| 05:41:25 | opendevreview | Amit Uniyal proposed openstack/nova master: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/791135 | |
| 05:41:25 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/854499 | |
| 07:09:38 | gokhanisi | Hello folks, after deleting vm glance image, we can not reboot vm, because it can't find _base image and after that nova destroyed my vm automatically. this the expected state or is there any mechanism to prevent automatic deletion? | |
| 09:46:56 | sean-k-mooney[m] | gokhanisi: there is no code in nova to do that automatic deletion | |
| 09:47:17 | sean-k-mooney[m] | gokhanisi: so it sounds like you have somethign else in your cloud that is doing that | |
| 09:47:51 | sean-k-mooney[m] | deleting the glance image will have no effect on running vms | |
| 09:49:00 | sean-k-mooney[m] | evacuate may fail if the image is deleted and the vm is not on shared sotrage as it wont be able to download the image but reboots and normal move operations like cold/live migrate will work | |
| 10:56:55 | auniyal_ | Hello #openstack-nova | |
| 10:57:00 | auniyal_ | please review these: | |
| 10:57:02 | auniyal_ | https://review.opendev.org/c/openstack/nova/+/854979 | |
| 10:57:07 | auniyal_ | https://review.opendev.org/c/openstack/nova/+/854980 | |
| 10:57:34 | auniyal_ | https://review.opendev.org/c/openstack/nova/+/858673 | |
| 11:43:26 | sean-k-mooney | bauzas: care to be the second reviewer on https://review.opendev.org/c/openstack/nova/+/827467 | |
| 11:51:06 | opendevreview | sean mooney proposed openstack/nova master: update default overcommit https://review.opendev.org/c/openstack/nova/+/830829 | |
| 11:52:20 | sean-k-mooney | gibi: stephenfin johnthetubaguy ^ care to take a look at that. im hoping we can merge that sooner rather then later | |
| 11:55:11 | opendevreview | Kirill proposed openstack/nova stable/wallaby: GCLOUD2-6989:vnc workable varsion https://review.opendev.org/c/openstack/nova/+/860555 | |
| 11:55:53 | opendevreview | Stephen Finucane proposed openstack/nova master: objects: Validate object-DB model field parity https://review.opendev.org/c/openstack/nova/+/860556 | |
| 11:55:53 | opendevreview | Stephen Finucane proposed openstack/nova master: objects: Add NovaSoftDeleteObject mixin https://review.opendev.org/c/openstack/nova/+/854355 | |
| 12:00:17 | opendevreview | Kirill proposed openstack/nova stable/wallaby: add support of vnc console for ironic https://review.opendev.org/c/openstack/nova/+/860557 | |
| 12:09:53 | opendevreview | Merged openstack/nova master: Fix typos in nova docs https://review.opendev.org/c/openstack/nova/+/858673 | |
| 12:14:37 | gibi | sean-k-mooney: I'm +2 | |
| 12:19:56 | stephenfin | Me too. Low risk since as you note, these are the _initial_ ratios and therefore only affect new resources | |
| 12:21:39 | bauzas | sean-k-mooney: I can look at https://review.opendev.org/c/openstack/nova/+/827467 | |
| 12:22:19 | bauzas | stephenfin: sean-k-mooney: I don't really want to discuss at https://review.opendev.org/c/openstack/nova/+/860401 for the moment, as I have the OpenInfra Live session in 2 hours | |
| 12:22:36 | bauzas | stephenfin: sean-k-mooney: but if you want, we can discuss this tomorro< | |
| 13:03:18 | sean-k-mooney | bauzas: there is no really rush. so focus on the talk | |
| 13:04:16 | sean-k-mooney | anyone else can also reveiw https://review.opendev.org/c/openstack/nova/+/827467 by the way i just pinnged you since i tought you might be around | |
| 13:04:44 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/860401 also can wait till the ptg | |
| 13:04:56 | sean-k-mooney | or we can chat about it tomrrow or next week | |
| 13:43:39 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Remove deleted projects from flavor access list https://review.opendev.org/c/openstack/nova/+/849131 | |
| 14:41:27 | frickler | fyi neutron broke integrated CI, they are working on a fix | |
| 14:52:28 | sean-k-mooney | fun | |
| 14:54:39 | sean-k-mooney | frickler: by the way were you asking about zuul and devstack recently | |
| 14:54:54 | sean-k-mooney | i think we have a bug in our new default policy | |
| 14:55:12 | sean-k-mooney | its blocking people with member form callign the server detail api | |
| 14:56:49 | frickler | sean-k-mooney: that doesn't ring a bell for me | |
| 14:57:36 | sean-k-mooney | it was proably corvus or clarkb | |
| 14:57:52 | sean-k-mooney | basically i repodcued it when i redeploy my home cloud and zuul | |
| 14:58:10 | sean-k-mooney | im not sure if its a nova issue or zuul one but it breaks | |
| 14:58:19 | sean-k-mooney | ill hop on the zuul matrix and follow up later | |
| 15:06:05 | lajoskatona | frickler: Hi, could you point to the failure, perhaps also on neutron channel? | |
| 15:07:28 | frickler | lajoskatona: ralonsoh is on it, https://review.opendev.org/c/openstack/devstack/+/860577 should be the fix | |
| 15:08:25 | lajoskatona | frickler: thanks | |
| 15:08:41 | ralonsoh | I've sent an email right now | |
| 15:24:19 | opendevreview | Vladislav Belogrudov proposed openstack/nova master: Nova instance snapshot should wait for volumes https://review.opendev.org/c/openstack/nova/+/848638 | |
| 15:54:59 | dansmith | bauzas: did you see that we're in conflict with the cinder ops hour? | |
| 15:55:22 | bauzas | dansmith: if so, they added it after ours | |
| 15:55:30 | dansmith | yeah I know | |
| 15:55:33 | bauzas | I just took a placeholder | |
| 15:55:52 | bauzas | and thanks for the notice, haven't seen it yet | |
| 15:56:55 | bauzas | dansmith: at least we'll have one hour for us | |
| 15:57:14 | dansmith | bauzas: could we move ours easily? | |
| 15:57:35 | bauzas | dansmith: problem is, I already published it in the openinfra live episode today | |
| 15:57:48 | dansmith | cinder moving at this point might be difficult, so even though we were first, if we're flexible we probably should be | |
| 15:57:58 | dansmith | yeah, that's unfortunate.. did cinder do the same? | |
| 15:58:01 | bauzas | no | |
| 15:58:06 | dansmith | heh | |
| 15:58:16 | bauzas | hence me thinking cinder would be easier to move | |
| 15:58:24 | dansmith | whoami-rajat: ^ thoughts? | |
| 15:58:40 | bauzas | and that's a 2-hour slot for ops | |
| 15:59:22 | bauzas | so, moving the operator slot is actually moving *two* hours for nova | |
| 15:59:42 | bauzas | again, I'd prefer cinder to move his slot by one hour | |
| 16:00:15 | bauzas | and if they can't, well, as we have 2 hours on that day, that'd be unfortunate but operators wanting to attend both could still be able to discuss with nova for one hour | |
| 16:00:50 | dansmith | bauzas: okay so the placeholders were just one hour I think, | |
| 16:01:02 | dansmith | so you must have chosen the placeholder and placeholder+1 right? | |
| 16:01:11 | bauzas | can't remember sorry how I booked | |
| 16:01:25 | bauzas | the eavesdrop logs could tell me tho :) | |
| 16:01:33 | bauzas | but yeah, probablyu | |
| 16:01:45 | dansmith | well, cinder chose one of the placeholders, so they feel like they did the right thing | |
| 16:01:52 | bauzas | so, nova has two slots back-to-back on tues 1-3 | |
| 16:02:01 | dansmith | but maybe it's less of a problem because we only have an hour overlap for the second hour | |
| 16:02:02 | bauzas | cinder has one slot 2-3 | |
| 16:02:07 | bauzas | zactly | |
| 16:02:17 | dansmith | so it was *nova* that caused the conflict :) | |
| 16:02:41 | bauzas | I pledge non-guilty | |