| opendevreview | Masanori Kuroha proposed openstack/nova master: Retrieve the image metadata from instance.image_meta during resize https://review.opendev.org/c/openstack/nova/+/936910 | 02:14 |
|---|---|---|
| opendevreview | Rajesh Tailor proposed openstack/nova-specs master: Add support for preserve/delete port on vm delete https://review.opendev.org/c/openstack/nova-specs/+/936990 | 07:31 |
| opendevreview | Rajesh Tailor proposed openstack/nova-specs master: Add support for preserve/delete port on vm delete https://review.opendev.org/c/openstack/nova-specs/+/936990 | 10:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: doc: Add missing API samples https://review.opendev.org/c/openstack/nova/+/937012 | 13:13 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use UUIDs in TestOpenStackClient https://review.opendev.org/c/openstack/nova/+/937013 | 13:13 |
| ratailor | sean-k-mooney, could you please review https://review.opendev.org/c/openstack/nova/+/873901 , need one more +2 | 15:19 |
| ratailor | gibi, sean-k-mooney please review this as well https://review.opendev.org/c/openstack/nova/+/911108 | 15:19 |
| sean-k-mooney | the secodn one im not sure if we should restre the orginal AZ | 15:20 |
| sean-k-mooney | i dont nessiarly agree with the epxecation in the bug reprot https://bugs.launchpad.net/nova/+bug/1978573 | 15:21 |
| sean-k-mooney | you can argure that either is correct so i watn input form other on why we shoudl revert vs not | 15:22 |
| sean-k-mooney | the first one im more ok with. ill see if i can review | 15:23 |
| ratailor | sean-k-mooney, ack. Thanks! could you please comment on restore original AZ patch for initiating discussion ? | 15:24 |
| sean-k-mooney | sure | 15:24 |
| ratailor | Thanks! | 15:24 |
| *** tosky_ is now known as tosky | 15:40 | |
| sean-k-mooney | ok i left comments on all 3 patches | 15:51 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hosts APIs https://review.opendev.org/c/openstack/nova/+/937047 | 17:50 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for instance actions https://review.opendev.org/c/openstack/nova/+/937048 | 17:50 |
| opendevreview | Merged openstack/osc-placement master: Update python classifier as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/osc-placement/+/935579 | 18:14 |
| sean-k-mooney | gmann: that test_release job is on focal... | 18:27 |
| sean-k-mooney | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_4cd/935579/2/gate/test-release-openstack/4cd9a9d/job-output.txt | 18:27 |
| sean-k-mooney | what even is that? | 18:28 |
| sean-k-mooney | i assume its building a wheel or something but that not defiend in os-vif is it | 18:28 |
| sean-k-mooney | its presumable part of a shared template | 18:28 |
| gmann | sean-k-mooney: let me check | 18:29 |
| gmann | I am not sure we moved test-release-openstack job to latest distro version | 18:29 |
| sean-k-mooney | so its defiend in project-config | 18:30 |
| gmann | sean-k-mooney: mainly it is running because of change in setup.cfg | 18:30 |
| sean-k-mooney | https://github.com/openstack/project-config/blob/master/zuul.d/jobs.yaml#L50-L63 | 18:30 |
| gmann | but it was running fine in other projects | 18:31 |
| gmann | yeah | 18:31 |
| sean-k-mooney | so the issue is that is pinned to ubuntu-focal | 18:31 |
| gmann | let me move it to Noble and see but even on focal it should pass | 18:31 |
| sean-k-mooney | it passed the second tim | 18:31 |
| gmann | yeah | 18:31 |
| sean-k-mooney | but ya we proablly shoudl update it | 18:31 |
| gmann | yup, doing that | 18:31 |
| sean-k-mooney | looking at the failure https://zuul.opendev.org/t/openstack/build/58b699c223834b00a2755ee35b473a2c/log/job-output.txt#832 | 18:32 |
| sean-k-mooney | im guessing this si related to the versions aviable in the mirros | 18:33 |
| gmann | yeah, seems so | 18:33 |
| sean-k-mooney | it failled in ovh-gra1 and passed in rax-iad | 18:33 |
| opendevreview | Ghanshyam proposed openstack/os-vif master: test test-release-openstack job on noble https://review.opendev.org/c/openstack/os-vif/+/937055 | 18:41 |
| opendevreview | Ghanshyam proposed openstack/os-vif stable/2024.2: test test-release-openstack job on noble https://review.opendev.org/c/openstack/os-vif/+/937056 | 18:41 |
| gmann | sean-k-mooney: https://review.opendev.org/c/openstack/project-config/+/937053 | 18:42 |
| sean-k-mooney | oh you going ot keep focal for the older branches. | 18:44 |
| sean-k-mooney | i woudl proably move for stable and only pin to focal on unmaintained | 18:45 |
| sean-k-mooney | actully | 18:45 |
| sean-k-mooney | unmaintained proably does not need that job? | 18:45 |
| sean-k-mooney | i guess we still push to tarballs.openstack.org for unmaintiend we just dont ot the rest of the release processs | 18:46 |
| sean-k-mooney | ok ya i think i agree with the approch you took in the patch | 18:47 |
| sean-k-mooney | your pinning ot the oldes ubuntu supproted in each set | 18:47 |
| sean-k-mooney | more or less alinging the job to the testing runtime | 18:48 |
| gmann | yeah. we can remove the job definition for branches where it is not running | 18:48 |
| sean-k-mooney | cool well lest see how your test patches go | 18:50 |
| opendevreview | melanie witt proposed openstack/nova master: libvirt: Wrap un-proxied listDevices() and listAllDevices() https://review.opendev.org/c/openstack/nova/+/932669 | 19:24 |
| *** elodilles is now known as elodilles_pto | 19:54 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!