| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-08-30 | |||
| 13:34:22 | gibi | I agree to have that coverage in our upstream gate | |
| 13:34:41 | sean-k-mooney | thanks ill let artom know and see if he wants me to wait for the jobs to be split or not first ill start on the WIP patch in anycase | |
| 13:43:21 | artom | Huh, happy coincidence, I was pondering proposing a periodic whitebox job for Nova | |
| 13:43:31 | artom | So, I think it's not yet stable enough for that, actually | |
| 13:43:52 | artom | We think we know the issue, and we're working on it, but until then I'm not sure it's ready yet | |
| 13:45:13 | artom | Every so often, depending on which order tests end up being executed, what we think happens is we attempt to reshape from cpu_dedicated_set to vcpu_pin_set, and that's not allowed, so there's a cascading failure. There are also around how we use admin clients and clean up after ourselves, that can also cause cascading failures | |
| 13:45:17 | gibi | artom: nothing is urgent from upstream perspective. If upstream feedback helps then I'm OK to enable a non voting job | |
| 13:46:01 | artom | gibi, I think even that's premature, as the solution to ^^^ is to change whitebox's own job a bit, so until that's done, let's not add it to nova | |
| 13:46:50 | sean-k-mooney | artom: ok the reason i was bring this up was we did at one point plan to enabel white box for wallaby | |
| 13:47:04 | sean-k-mooney | then we did not have time to actully get it stable in time | |
| 13:47:15 | sean-k-mooney | so i was hoping we could do thatbefore the end of xena | |
| 13:47:24 | artom | Ah, probably not before the end of Xena | |
| 13:47:24 | sean-k-mooney | if you think its not ready however we can hold off | |
| 13:47:31 | artom | ... well, does end == FF? | |
| 13:47:33 | artom | Or release? | |
| 13:48:15 | sean-k-mooney | well i guess i twas thinking before RC1 when stable branch is created | |
| 13:48:34 | sean-k-mooney | although if we were ok with backporting enableing the testing on the stable branch end could be anytime before eol i guess | |
| 13:49:27 | sean-k-mooney | if we dont think it ready however no need to rush | |
| 13:50:00 | sean-k-mooney | i would just like to keep making progress on getting this test coverage enable eithe firstparty or third party | |
| 13:51:11 | artom | Third party I still haven't solved the hardware problem :) | |
| 13:51:13 | artom | Err :( | |
| 13:51:20 | sean-k-mooney | artom: is bauzas back today or is he retruning tomorow | |
| 13:51:31 | sean-k-mooney | artom: yep i know :) | |
| 13:52:19 | artom | Still on PTO today, according to Workday | |
| 13:52:42 | gibi | I personally OK with enabling new jobs on stable but I guess elodilles or lyarwood has more authority about that :) | |
| 13:53:10 | gibi | as per landing it on master, this is not a feature so RC1 is the cut of date due to branching | |
| 13:53:39 | artom | I can try to hurry it up, especially as jparker seems to have more time for this right about now, too | |
| 13:54:58 | sean-k-mooney | gibi: before i recheck are there any gate blockers i should hold off for | |
| 13:55:24 | sean-k-mooney | i was just looking at the failure in bauzas mdev series which dont seam related | |
| 13:56:04 | gibi | sean-k-mooney: the "Add force kwarg to delete_allocation_for_instance" not landed yet that kills at least 1/4 of the tempest jobs all around the gate | |
| 13:56:16 | sean-k-mooney | ah right | |
| 13:56:20 | gibi | I don't know about any full blocker | |
| 13:56:49 | sean-k-mooney | ok i was seeing the nova-ceph-multistore job fail in several patch but have not dug in to see if its the same issue | |
| 13:57:29 | sean-k-mooney | oh "'Failed to delete allocations for consumer 2064788c-9fa0-474e-a66c-72cf97b45922. .." | |
| 13:57:34 | sean-k-mooney | ya so its just that | |
| 13:58:55 | gibi | yes | |
| 13:59:10 | sean-k-mooney | any idea why it would hit the multistore job more often | |
| 13:59:36 | sean-k-mooney | it looks like that is mostly the failure so ill hold off until the force patch lands | |
| 14:00:04 | sean-k-mooney | they dont have +w anyway so they can wait | |
| 14:00:46 | elodilles | well, i don't exactly followed which job you were talking about but I am less concerned enabling new CI jobs on stable than disabling one o:) | |
| 14:00:47 | gibi | Ive no ide about the increased frequency of multistore failure | |
| 14:01:07 | gibi | elodilles: it would be a job running https://opendev.org/openstack/whitebox-tempest-plugin/src/branch/master/whitebox_tempest_plugin/api/compute | |
| 14:01:35 | sean-k-mooney | elodilles: which is defiedn here https://opendev.org/openstack/whitebox-tempest-plugin/src/branch/master/.zuul.yaml#L31-L81 | |
| 14:02:13 | sean-k-mooney | gibi: artom is currntly actully spliting it into two jobs one that use the old cpu pinning config and the main one will only use the new way | |
| 14:02:24 | gibi | ack | |
| 14:02:35 | sean-k-mooney | gibi: right now if the reshap happens at the wong time the job breaks | |
| 14:02:45 | sean-k-mooney | so we are just going to split it to aovid that | |
| 14:02:57 | gibi | sure make sense | |
| 14:03:17 | elodilles | gibi sean-k-mooney : I guess these would land on master and then backported on the most recent stable branch, am I right? | |
| 14:05:06 | elodilles | (hmmm, it looks quite heavy, according to its parent: tempest-multinode-full-py3) | |
| 14:07:10 | gibi | personally I would take it on master first | |
| 14:16:28 | sean-k-mooney | elodilles: well it need 2 nodes but it does not run run all the tempest test becasue we use the regex to limit it | |
| 14:16:59 | sean-k-mooney | tox_envlist: all | |
| 14:17:01 | sean-k-mooney | tempest_concurrency: 1 | |
| 14:17:03 | sean-k-mooney | tempest_test_regex: ^whitebox_tempest_plugin\. | |
| 14:17:30 | sean-k-mooney | so i just ues that job to set up 2 node devstack with tempest then we just run the test from the plugin | |
| 14:20:58 | elodilles | oh, i see, i missed that | |
| 14:21:22 | sean-k-mooney | we proably could inherit form something better to make that more obvious | |
| 14:26:12 | elodilles | well, when someone reviews it thoroughly i think it'll be obvious o:) but that's true that at first glance the tempest-multinode-*full*-py3 suggests some time and resource heavy test job o:) | |
| 14:26:57 | sean-k-mooney | we proably can just use devstack-tempest ill see if there is a better job we can use in the future. that is a simple fix | |
| 15:20:12 | ganso | elodilles, lyarwood: Hi! If you have a spare minute could you please take a quick look at the backport now for victoria? it is clean and same as the one for wallaby from last week. Thanks in advance! https://review.opendev.org/c/openstack/nova/+/806004 | |
| 15:22:44 | opendevreview | Ghanshyam proposed openstack/nova master: Convert features not supported error to HTTPBadRequest https://review.opendev.org/c/openstack/nova/+/806294 | |
| 15:46:03 | elodilles | ganso: +2'd. Thanks for the backport! (fyi, lyarwood is on holiday today) | |
| 15:47:09 | ganso | elodilles: thanks! I will ping him tomorrow =) | |
| 15:50:11 | elodilles | no problem :) | |
| 16:06:20 | opendevreview | Merged openstack/nova master: tests: Validate AZ values https://review.opendev.org/c/openstack/nova/+/801523 | |
| 17:05:36 | opendevreview | Merged openstack/nova master: Add force kwarg to delete_allocation_for_instance https://review.opendev.org/c/openstack/nova/+/688802 | |
| 17:05:45 | sean-k-mooney | :) | |
| 17:15:39 | opendevreview | Merged openstack/nova master: Prevent deletion of a compute node belonging to another host https://review.opendev.org/c/openstack/nova/+/694802 | |
| 17:15:55 | opendevreview | Merged openstack/nova master: Fix inactive session error in compute node creation https://review.opendev.org/c/openstack/nova/+/695189 | |
| 17:16:17 | opendevreview | Merged openstack/nova master: Reduce mocking in test_reject_open_redirect for compat https://review.opendev.org/c/openstack/nova/+/803091 | |
| 17:16:25 | opendevreview | Merged openstack/nova master: extend_volume of libvirt/volume/iscsi should not use device_path https://review.opendev.org/c/openstack/nova/+/801003 | |
| 17:38:03 | opendevreview | sean mooney proposed openstack/nova stable/victoria: address open redirect with 3 forward slashes https://review.opendev.org/c/openstack/nova/+/806626 | |
| 17:39:28 | sean-k-mooney | gibi: elodilles by the way are we goign to backport https://review.opendev.org/c/openstack/nova/+/688802 | |
| 17:56:40 | opendevreview | sean mooney proposed openstack/nova stable/ussuri: address open redirect with 3 forward slashes https://review.opendev.org/c/openstack/nova/+/806628 | |
| 18:03:58 | opendevreview | sean mooney proposed openstack/nova stable/train: address open redirect with 3 forward slashes https://review.opendev.org/c/openstack/nova/+/806629 | |
| 19:51:45 | gibi | sean-k-mooney: I don't konw. The consumer types feature is only on master so we have a smaller issue on stable. And that smaller issue there since stein if I remember correclty. | |
| 21:56:27 | gmann | gibi: added releasenotes in this https://review.opendev.org/c/openstack/nova/+/806294 | |
| #openstack-nova - 2021-08-31 | |||
| 00:52:21 | opendevreview | Merged openstack/nova master: [func test] neutron fixture for extended resource request https://review.opendev.org/c/openstack/nova/+/794306 | |
| 00:52:29 | opendevreview | Merged openstack/nova master: Detect port-resource-request-groups neutron API extension https://review.opendev.org/c/openstack/nova/+/793618 | |
| 00:52:44 | opendevreview | Merged openstack/nova master: Reject server create with extended resource req https://review.opendev.org/c/openstack/nova/+/793619 | |
| 00:53:02 | opendevreview | Merged openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | |
| 01:35:16 | opendevreview | Merged openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | |
| 01:35:25 | opendevreview | Merged openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | |
| 01:35:33 | opendevreview | Merged openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | |
| 01:35:41 | opendevreview | Merged openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | |
| 01:35:49 | opendevreview | Merged openstack/nova master: [func test] refactor asserts in qos tests https://review.opendev.org/c/openstack/nova/+/798930 | |
| 01:35:57 | opendevreview | Merged openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | |
| 05:34:44 | opendevreview | melanie witt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142 | |
| 05:34:44 | opendevreview | melanie witt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/c/openstack/nova/+/712139 | |
| 05:34:45 | opendevreview | melanie witt proposed openstack/nova master: Update limit APIs https://review.opendev.org/c/openstack/nova/+/712707 | |
| 05:34:45 | opendevreview | melanie witt proposed openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/c/openstack/nova/+/712143 | |
| 05:34:46 | opendevreview | melanie witt proposed openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/c/openstack/nova/+/713301 | |
| 05:34:46 | opendevreview | melanie witt proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/c/openstack/nova/+/712749 | |
| 05:34:47 | opendevreview | melanie witt proposed openstack/nova master: Add legacy limits and usage to unified limits https://review.opendev.org/c/openstack/nova/+/713498 | |
| 05:34:47 | opendevreview | melanie witt proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | |
| 05:34:48 | opendevreview | melanie witt proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | |
| 05:34:49 | opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 07:03:02 | bauzas | good morning Nova | |
| 07:04:18 | gibi | bauzas: welcome back | |