| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-08-23 | |||
| 08:41:40 | gibi | yeah, that is a valid fix for the test case | |
| 08:44:12 | Uggla | Hi gibi , do we have the nova meeting this evening ? | |
| 08:44:38 | gibi | Uggla: it is tuesday so yes | |
| 08:44:42 | gibi | I will chair it | |
| 08:44:46 | gibi | as the last two | |
| 08:44:56 | Uggla | ok cool | |
| 08:47:23 | sean-k-mooney[m] | still embing caffine but assuming im done with vdpa for today. what seriese should i review first | |
| 09:06:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Basics for PCI Placement reporting https://review.opendev.org/c/openstack/nova/+/846187 | |
| 09:06:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Extend device_spec with resource_class and traits https://review.opendev.org/c/openstack/nova/+/846218 | |
| 09:06:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject PCI dependent device config https://review.opendev.org/c/openstack/nova/+/846435 | |
| 09:06:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject mixed VF rc and trait config https://review.opendev.org/c/openstack/nova/+/846436 | |
| 09:06:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore PCI devs with physical_network tag https://review.opendev.org/c/openstack/nova/+/846219 | |
| 09:06:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject devname based device_spec config https://review.opendev.org/c/openstack/nova/+/846466 | |
| 09:06:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support [pci]device_spec reconfiguration https://review.opendev.org/c/openstack/nova/+/846470 | |
| 09:06:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: Stop if tracking is disable after it was enabled before https://review.opendev.org/c/openstack/nova/+/847009 | |
| 09:06:48 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move provider_tree RP creation to PciResourceProvider https://review.opendev.org/c/openstack/nova/+/850546 | |
| 09:06:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: Retry /reshape at provider generation conflict https://review.opendev.org/c/openstack/nova/+/851358 | |
| 09:06:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: Heal missing simple PCI allocation in the resource tracker https://review.opendev.org/c/openstack/nova/+/851359 | |
| 09:06:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Heal PCI allocation during resize https://review.opendev.org/c/openstack/nova/+/852396 | |
| 09:06:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Handle PCI dev reconf with allocations https://review.opendev.org/c/openstack/nova/+/852397 | |
| 09:06:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow enabling PCI tracking in Placement https://review.opendev.org/c/openstack/nova/+/850468 | |
| 09:06:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: Generate request_id for Flavor based InstancePCIRequest https://review.opendev.org/c/openstack/nova/+/853835 | |
| 09:06:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: Request filter for PCI in placement https://review.opendev.org/c/openstack/nova/+/852771 | |
| 09:06:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support resource_class and traits in PCI alias https://review.opendev.org/c/openstack/nova/+/853316 | |
| 09:06:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Map PCI pools to RP UUIDs https://review.opendev.org/c/openstack/nova/+/854118 | |
| 09:06:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make allocation candidates available for scheduler filters https://review.opendev.org/c/openstack/nova/+/854119 | |
| 09:06:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Filter PCI pools based on Placement allocation https://review.opendev.org/c/openstack/nova/+/854120 | |
| 09:06:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Store allocated RP in InstancePCIRequest https://review.opendev.org/c/openstack/nova/+/854121 | |
| 09:07:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Func test for PCI in placement scheduling https://review.opendev.org/c/openstack/nova/+/854122 | |
| 09:09:07 | gibi | sean-k-mooney[m]: the pci stuff can be reviewed up until https://review.opendev.org/c/openstack/nova/+/851358/ and even further if you have the willpower | |
| 09:09:25 | gibi | the top of the series missing unit test coverage and has spotty functional coverage though | |
| 09:09:33 | sean-k-mooney | ack | |
| 09:10:32 | sean-k-mooney | ok so that is everything up to just before the allocation healling | |
| 09:14:29 | opendevreview | Jan Hartkopf proposed openstack/python-novaclient master: add support for microversion 2.93 https://review.opendev.org/c/openstack/python-novaclient/+/816158 | |
| 09:25:23 | Uggla | gibi, sean-k-mooney may I request you to check https://review.opendev.org/c/openstack/nova/+/853986 and merge it if ok. | |
| 10:18:45 | gibi | Uggla: done, and thanks | |
| 10:20:09 | sean-k-mooney | gibi: by the way i have started to notice tempest.api.volume.test_volumes_extend.VolumesExtendAttachedTest.test_extend_attached_volume failing semi often in nova-next | |
| 10:20:58 | Uggla | gibi, thank you ! | |
| 10:21:03 | gibi | sean-k-mooney: do you have an example failure? | |
| 10:21:11 | sean-k-mooney | https://zuul.opendev.org/t/openstack/build/5e64ad2283a24653a424ad91d99a3004 | |
| 10:21:16 | gibi | ack, I will quickly check | |
| 10:21:22 | sean-k-mooney | it failed on teh vdpa patch | |
| 10:21:30 | sean-k-mooney | but i have seen it on several others recnetly | |
| 10:21:53 | sean-k-mooney | do you know where the new elastic search deployment is hosted | |
| 10:22:08 | sean-k-mooney | im kind of wonderign how often this happens | |
| 10:23:24 | gibi | interesting | |
| 10:23:29 | gibi | https://zuul.opendev.org/t/openstack/build/5e64ad2283a24653a424ad91d99a3004/log/compute1/logs/screen-n-cpu.txt#52040 | |
| 10:23:38 | gibi | libvirt.libvirtError: invalid argument: disk '/dev/sda' was not found in the domain config | |
| 10:23:44 | sean-k-mooney | ya | |
| 10:23:48 | sean-k-mooney | thats the same failure | |
| 10:24:03 | gibi | sean-k-mooney: https://opensearch.logs.openstack.org/_dashboards/app/login?nextUrl=%2F_dashboards%2Fapp%2Fdiscover%2F%3Fsecurity_tenant%3Dglobal this is the logstash | |
| 10:24:04 | sean-k-mooney | we dont guarentee the adress of th disk | |
| 10:24:26 | sean-k-mooney | ack i was trying http://logstash.openstack.org/ | |
| 10:24:40 | sean-k-mooney | which is obviously dead but maybe infra coudl add a redirect to the new location | |
| 10:25:52 | gibi | yah a redirect would be nice | |
| 10:27:23 | gibi | hm from the last 10 novan-next failure 9 was due to the above error | |
| 10:27:29 | sean-k-mooney | do you know the login info? | |
| 10:27:50 | sean-k-mooney | i think they sent one ot the mail list but i cant find it in my email | |
| 10:28:39 | sean-k-mooney | gibi: ya so we have a log message that says "the libvirt driver cannot honner the request path" or somethign like that when we attach so im supresed we care when detaching | |
| 10:28:40 | gibi | openstack openstack | |
| 10:29:13 | sean-k-mooney | thanks | |
| 10:30:13 | gibi | it is almost like it started failing on 08.19. | |
| 10:30:51 | gibi | and we merged https://review.opendev.org/c/openstack/nova/+/843700/ in 19th | |
| 10:31:44 | sean-k-mooney | yep | |
| 10:31:44 | gibi | so I think we enabled an unstable test | |
| 10:31:58 | gibi | I'm proposing a revert | |
| 10:32:02 | sean-k-mooney | i can revert that i guess | |
| 10:32:04 | sean-k-mooney | ah go for it | |
| 10:33:24 | sean-k-mooney | so its calling | |
| 10:33:25 | sean-k-mooney | def attach_detach_volume(self, server, volume): | |
| 10:33:27 | sean-k-mooney | attached_volume = self.nova_volume_attach(server, volume) | |
| 10:33:29 | sean-k-mooney | self.nova_volume_detach(server, attached_volume) | |
| 10:33:43 | sean-k-mooney | if that is not waiting for sshable then that is l ikely the issue | |
| 10:34:03 | sean-k-mooney | https://review.opendev.org/c/openstack/tempest/+/836065/4/tempest/scenario/test_encrypted_cinder_volumes.py | |
| 10:34:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: Revert "Test attached volume extend actions in the nova-next job" https://review.opendev.org/c/openstack/nova/+/854132 | |
| 10:34:57 | sean-k-mooney | i wonder is this another case of that type of error | |
| 10:35:01 | gibi | yeah that can be one issue yes | |
| 10:35:13 | sean-k-mooney | we might have removed it form teh persitent tomain but not the active domain | |
| 10:35:21 | sean-k-mooney | and fail with not found on a retry or something | |
| 10:35:37 | sean-k-mooney | i wonder if the logs shows anything like that | |
| 10:36:20 | sean-k-mooney | hum no its the resize that is failing | |
| 10:36:33 | sean-k-mooney | but perhaps the attach has not completed | |
| 10:38:09 | gibi | I'm table this for now. we can merge the revert or somebody can propose a fix to the tempest test but I let that be done by others :) | |
| 10:38:31 | sean-k-mooney | sure :) | |
| 10:41:16 | gibi | (we actually did a naked recheck that showed this error to merge the original patche enabling this test case) | |
| 10:42:33 | sean-k-mooney | thats unfortunet | |
| 10:43:43 | opendevreview | Merged openstack/nova master: Alphabetizes objects https://review.opendev.org/c/openstack/nova/+/853986 | |
| 10:44:48 | gibi | that shows why we are against naked rechecks :D | |
| 10:46:21 | sean-k-mooney | i left a comment on the tempest chagne https://review.opendev.org/c/openstack/tempest/+/836065/4#message-a8c9c9794c75555728517230fcab3b7beb8427de | |
| 10:46:26 | gibi | thanks | |
| 10:48:37 | sean-k-mooney | is logsearch a cli for logstash/opensearch ? | |
| 10:49:10 | sean-k-mooney | that looks like its quite nice to use vs the ui | |
| 10:49:26 | sean-k-mooney | the ui is not bad bu the cli output is very short and too the point | |
| 10:50:04 | gibi | nope it is my own tool grepping logs locally | |
| 10:50:22 | gibi | https://github.com/gibizer/zuul-log-search | |
| 10:50:37 | sean-k-mooney | oh ok | |
| 10:50:51 | sean-k-mooney | so its pulling the logs locally | |
| 10:50:55 | gibi | that is one of my hack and hustle target | |
| 10:51:25 | gibi | yes it pulls the logs locally based on the zuul query you make | |
| 10:51:30 | gibi | caches it locally | |