| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-07-15 | |||
| 21:42:31 | JayF | I guess targetting a bunch of different qemu versions might have a similar problems to what we see in Ironic with ... unfortunately designed hardware BMCs :P | |
| 21:43:39 | sean-k-mooney | ill ask gibi tomorrow if we should consider making the job non voteing. it generally has not been wedgeing the gate but it has been failing frequetly enough that we might want to consider skiping the test or just manually reviewing the job for now | |
| 21:44:47 | JayF | thanks! I'll keep idling in here, feel free to ping me if there's anything I can do to help :) | |
| 21:45:07 | JayF | [or if you ever need someone with the big review hammer in Ironic :D] | |
| 21:45:48 | sean-k-mooney | i wonder if we could make the live migation jobs skip on change that only affect the ironic driver | |
| 21:46:15 | sean-k-mooney | you know since i dont precive that mattering to your use case unless the laws of physic cahnge | |
| 21:46:51 | JayF | I think that's a potentially wise change to make, but don't underestimate the ability of python to violate the laws of physics :P | |
| 21:47:43 | sean-k-mooney | im pretty sure we could add nova/test/unit/virt/ironic and nova/virt/ironic to the irrelevent files list | |
| 21:48:23 | JayF | I'd be happy to +1 such a change | |
| 21:49:03 | sean-k-mooney | so we would jsut need to add those here https://github.com/openstack/nova/blob/master/.zuul.yaml#L93-L107 | |
| 21:49:44 | JayF | and, for purposes of my patch, it'd also need to be backported | |
| 21:50:20 | sean-k-mooney | well that used selse where so we would have to refactor slightly it should not be terrible. yep we would have to backport it first | |
| 21:50:52 | JayF | hmm where else would it need to go :) | |
| 21:51:03 | JayF | I was already on the way to making the change against main branch | |
| 21:52:20 | sean-k-mooney | currently the deffinition of *nova-base-irrelevant-files is in the live migraiton job we would need to move that to say nova-next which also uses it | |
| 21:52:44 | sean-k-mooney | then defein a new nova-live-migration-irrelavent-files that the ceph job would also use | |
| 21:53:01 | sean-k-mooney | although that may or may not be enough to get your patch to pass | |
| 21:53:19 | sean-k-mooney | e.g. the grenade job might fail in a similar way | |
| 21:53:45 | JayF | Yeah, we've seen it fail on several different jobs for sure. Reducing the number of times we have to roll the dice will only raise the likelihood of that patch getting landed, though | |
| 21:54:25 | sean-k-mooney | am its late for me corrently so im going to call it a night | |
| 21:54:45 | JayF | o/ have a good one | |
| 21:54:52 | sean-k-mooney | for now i would just recheck it again o/ | |
| 21:55:07 | JayF | I'm getting really good at it by now :D Thanks again, have a nice night | |
| 23:07:20 | opendevreview | MarkMielke proposed openstack/nova master: extend_volume of libvirt/volume/iscsi should not use device_path https://review.opendev.org/c/openstack/nova/+/801003 | |
| #openstack-nova - 2021-07-16 | |||
| 07:32:54 | opendevreview | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | |
| 07:32:56 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | |
| 07:32:59 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - create arqs https://review.opendev.org/c/openstack/nova/+/758944 | |
| 07:33:01 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - build instance with smartnic arqs https://review.opendev.org/c/openstack/nova/+/798249 | |
| 07:33:02 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - cleanup arqs https://review.opendev.org/c/openstack/nova/+/798054 | |
| 07:33:12 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | |
| 07:33:28 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | |
| 08:32:30 | opendevreview | Merged openstack/nova stable/wallaby: [ironic] Minimize window for a resource provider to be lost https://review.opendev.org/c/openstack/nova/+/799772 | |
| 10:58:41 | opendevreview | Balazs Gibizer proposed openstack/placement master: Bump os-resource-classes requirements https://review.opendev.org/c/openstack/placement/+/796595 | |
| 11:22:52 | gibi | kashyap: I'm just loking at https://blueprints.launchpad.net/nova/+spec/remove-support-for-floppy-disks which seem to be in a limbo state as it is targeted to xena but it was never set to approved. Do you happen to remember if there was agreemet about that in Xena timeframe? | |
| 11:29:00 | sean-k-mooney | at a minium we proably want to deprecate in xena | |
| 11:29:34 | sean-k-mooney | i dont recalls if we said we would remvoe or not in xena | |
| 12:27:06 | gibi | sean-k-mooney: my problem is that bp state is in limbo so I'm not sure we ever approve this bp | |
| 12:27:16 | gibi | I hope kashyap remembers :D | |
| 13:04:27 | kashyap | gibi: Hi, was away for lunch | |
| 13:06:19 | kashyap | gibi: I only recall a "general agreement" (no active objections on the list - from more than a year ago) | |
| 13:06:40 | kashyap | gibi: Yes, deprecate for sure in Xena | |
| 13:07:10 | gibi | kashyap: do you have intention to working on that bp before Milestone 3 (Sept 3)? | |
| 13:07:16 | kashyap | gibi: Yes | |
| 13:07:47 | kashyap | gibi: Can we remove it in Xena, even? Who uses floppy drives anyway | |
| 13:07:57 | kashyap | I'm expecting a "well, actually" here :D | |
| 13:12:28 | gibi | so there was generic agreement around it so I guees I made a mistake not recording that properly in the bp early in xena | |
| 13:12:43 | gibi | so I will move it to approved now | |
| 13:15:50 | gibi | kashyap: as of the implementation I think we need deprecation first | |
| 13:16:36 | kashyap | gibi: Yeah fair enough | |
| 13:27:18 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova-specs master: Integration With Off-path Network Backends https://review.opendev.org/c/openstack/nova-specs/+/787458 | |
| 14:08:11 | kashyap | gibi: Thanks; I saw the changes | |
| 14:42:07 | dmitriis | gibi, sean-k-mooney: re-proposed https://review.opendev.org/c/openstack/nova-specs/+/787458 for Yoga. There's some progress with Neutron and OVN. | |
| 14:42:07 | dmitriis | Neutron: the RFE https://bugs.launchpad.net/neutron/+bug/1932154 got rfe-approved after the discussion in the drivers meeting https://meetings.opendev.org/irclogs/%23openstack-neutron/%23openstack-neutron.2021-07-09.log.html. We also had a +1 from Brian Haley on the spec https://review.opendev.org/c/openstack/neutron-specs/+/788821 (auto-cleared | |
| 14:42:07 | dmitriis | after a re-upload with nit fixes). | |
| 14:42:07 | dmitriis | OVN: Things moved forward in this thread https://patchwork.ozlabs.org/project/ovn/patch/20210509140305.1910796-1-frode.nordahl@canonical.com/#2699171 and some OVN-side implementation details have changed but nothing Nova needs to care about. We are currently waiting for feedback from Ilya Maximets (hopefully next week) and had some feedback from | |
| 14:42:08 | dmitriis | others already. | |
| 14:44:24 | gibi | dmitriis: thanks for the update. I hope we can make this happen in the next cycle | |
| 16:50:59 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Replace implicit conversion of SELECT into FROM https://review.opendev.org/c/openstack/placement/+/800910 | |
| 16:50:59 | opendevreview | Stephen Finucane proposed openstack/placement master: tox: Remove psycopg2 warning filter https://review.opendev.org/c/openstack/placement/+/800911 | |
| 16:51:00 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Replace 'as_scalar()' with 'scalar_subquery()' https://review.opendev.org/c/openstack/placement/+/801100 | |
| 16:51:00 | opendevreview | Stephen Finucane proposed openstack/placement master: tests: Silence noisy tests https://review.opendev.org/c/openstack/placement/+/801101 | |
| 16:51:01 | opendevreview | Stephen Finucane proposed openstack/placement master: setup: Replace dashes with underscores https://review.opendev.org/c/openstack/placement/+/801102 | |
| 16:51:02 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Update 'select()' calls https://review.opendev.org/c/openstack/placement/+/801103 | |
| 16:51:03 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Remove use of non-integer/slice indices https://review.opendev.org/c/openstack/placement/+/801104 | |
| 16:51:04 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Replace deprecated 'FromClause.select().whereclause' parameter https://review.opendev.org/c/openstack/placement/+/801105 | |
| 16:51:05 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Use explicit transactions https://review.opendev.org/c/openstack/placement/+/801106 | |
| 16:51:05 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Remove unnecessary use of '_mapping' https://review.opendev.org/c/openstack/placement/+/801107 | |
| 16:51:07 | opendevreview | Stephen Finucane proposed openstack/placement master: tox: Enable SQLAlchemy 2.0 warnings https://review.opendev.org/c/openstack/placement/+/801108 | |
| 17:00:17 | gibi | stephenfin: I know it is Friday afternoon, so it is just for the future... I think the first 15 patches of the pps series is ready for review. This includes all the prerequisit work up until the first successful boot. The patch that actually enables the boot still needs work | |
| 17:00:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: Bump os-resource-classes to 1.1.0 https://review.opendev.org/c/openstack/nova/+/800976 | |
| 17:00:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] create pps resource on OVS agent RP https://review.opendev.org/c/openstack/nova/+/787205 | |
| 17:00:40 | stephenfin | excellent | |
| 17:00:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move port creation to the NeutronFixture https://review.opendev.org/c/openstack/nova/+/787206 | |
| 17:00:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] neutron fixture for extended resource request https://review.opendev.org/c/openstack/nova/+/794306 | |
| 17:00:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Detect extended_resource_request neutron API extension https://review.opendev.org/c/openstack/nova/+/793618 | |
| 17:00:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server create with extended resource req https://review.opendev.org/c/openstack/nova/+/793619 | |
| 17:00:45 | stephenfin | I can start at that Monday | |
| 17:00:50 | gibi | stephenfin: thank you | |
| 17:04:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | |
| 17:04:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | |
| 17:07:54 | opendevreview | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | |
| 17:07:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | |
| 17:10:25 | opendevreview | Jay Faulkner proposed openstack/nova stable/victoria: [ironic] Minimize window for a resource provider to be lost https://review.opendev.org/c/openstack/nova/+/800873 | |
| 17:11:14 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | |
| 17:11:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor asserts in qos tests https://review.opendev.org/c/openstack/nova/+/798930 | |
| 17:11:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | |
| 17:16:10 | opendevreview | Balazs Gibizer proposed openstack/nova master: Parse extended resource request from the port data https://review.opendev.org/c/openstack/nova/+/800085 | |
| 17:18:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 17:18:22 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support boot with extended resource request https://review.opendev.org/c/openstack/nova/+/800086 | |
| 17:26:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support move ops with extended resource request https://review.opendev.org/c/openstack/nova/+/800087 | |
| 17:26:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test]Refactor interface attach with qos https://review.opendev.org/c/openstack/nova/+/800088 | |
| 17:29:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support interaface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | |
| 17:31:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 21:58:06 | opendevreview | melanie witt proposed openstack/nova master: Correct reno for archiving task_log database records https://review.opendev.org/c/openstack/nova/+/801144 | |
| #openstack-nova - 2021-07-17 | |||
| 06:58:50 | opendevreview | Merged openstack/nova stable/stein: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/791402 | |
| #openstack-nova - 2021-07-19 | |||
| 02:22:15 | opendevreview | chengsheng proposed openstack/nova master: libvirt: Use improved guest CPU config APIs https://review.opendev.org/c/openstack/nova/+/762330 | |
| 04:38:56 | opendevreview | chengsheng proposed openstack/nova master: libvirt: Use improved guest CPU config APIs https://review.opendev.org/c/openstack/nova/+/762330 | |
| 08:46:21 | lyarwood | elodilles: gibi: \o morning, btw I'm not away for 2 weeks now. Surgery was unfortunately cancelled at the last minute and hopefully rescheduled for sometime in the next 4 weeks, until then I'm back as normal. | |
| 08:49:11 | gibi | lyarwood: morning. thanks of the information. | |
| 08:55:38 | bauzas | lyarwood: hope you'll eventually get all sorted as quickly as possible | |