| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-03-11 | |||
| 15:10:33 | sean-k-mooney | dansmith: yep basically | |
| 15:10:36 | sean-k-mooney | https://bugzilla.redhat.com/show_bug.cgi?id=2007129 | |
| 15:10:38 | kashyap | That's the comment from a PCI(e) dev from QEMU (from a RHT bug) | |
| 15:10:44 | sean-k-mooney | ^ that is the main bug right | |
| 15:10:46 | kashyap | (Where "latest machine type" == 8.6 / 9) | |
| 15:12:08 | dansmith | ack | |
| 15:12:10 | sean-k-mooney | from that bug the say "This bug is related to some change in qemu-6.2 so it should not be there in RHEL 8.4/8.2," | |
| 15:12:32 | sean-k-mooney | however we have had detach issue on 8.4 | |
| 15:12:48 | sean-k-mooney | anyway it might just be down to the use of 6.2+ on c9s | |
| 15:50:21 | opendevreview | Andre Aranha proposed openstack/nova master: Move FIPS jobs to experimental and periodic queue https://review.opendev.org/c/openstack/nova/+/833431 | |
| 15:55:14 | opendevreview | sean mooney proposed openstack/nova master: [WIP] enable block VDPA operations https://review.opendev.org/c/openstack/nova/+/832330 | |
| 15:55:45 | opendevreview | sean mooney proposed openstack/nova master: [WIP] enable blocked VDPA operations https://review.opendev.org/c/openstack/nova/+/832330 | |
| 15:56:31 | opendevreview | Andre Aranha proposed openstack/nova master: Test setting the nova job to centos-9-stream https://review.opendev.org/c/openstack/nova/+/831844 | |
| 15:56:34 | sean-k-mooney | i need to add a few unit tests and a release note but i think ^ that is basically done | |
| 15:57:03 | sean-k-mooney | i still want to test this with real hardware however before i do and i proably need to update the docs too | |
| 15:58:30 | bauzas | \o | |
| 16:01:24 | sean-k-mooney | o/ | |
| 16:14:50 | opendevreview | sean mooney proposed openstack/nova stable/xena: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833411 | |
| 16:21:28 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833435 | |
| 16:22:33 | opendevreview | sean mooney proposed openstack/nova stable/victoria: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833436 | |
| 16:23:28 | opendevreview | sean mooney proposed openstack/nova stable/ussuri: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833437 | |
| 16:23:48 | opendevreview | sean mooney proposed openstack/nova stable/train: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833438 | |
| 16:49:35 | opendevreview | Takashi Natsume proposed openstack/nova master: Update min supported service version for Zed https://review.opendev.org/c/openstack/nova/+/833440 | |
| 17:05:35 | zigo | I'm packaging nova RC1. I've seen that os-win is removed. Is that library useless now? | |
| 17:06:18 | zigo | Oh... setup.cfg ... :P | |
| 17:06:51 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for Zed https://review.opendev.org/c/openstack/nova/+/833441 | |
| 17:21:40 | sean-k-mooney | zigo: its an optional dep (and always was) so its no in extras | |
| 17:22:46 | sean-k-mooney | zigo: https://github.com/openstack/nova/commit/86d87be8db588cc3125d53cd92e271fb45b1a3aa for context | |
| 17:23:27 | sean-k-mooney | zigo: this was partly propeted by unmaintained packages that were breakign the gate | |
| 17:24:17 | zigo | sean-k-mooney: Is zVMCloudConnector completely gone? | |
| 17:24:28 | zigo | Or will it stay ... | |
| 17:25:03 | zigo | In other words: should I ask for its removal from Debian and erase all traces of it? | |
| 17:25:10 | gmann | dansmith: kashyap sean-k-mooney for detach failure/SSHable things, this last one needs to be merged, rescue negative test which this patch making SSHable was failing in reported bug. https://review.opendev.org/c/openstack/tempest/+/831608 | |
| 17:25:26 | gmann | it is not ready, need to debug on change failure though | |
| 17:29:34 | dansmith | gmann: ah are you saying that the sshable patches that already merged are working and that there are just a few remaining ones needing to be converted (in that patch)? | |
| 17:30:05 | sean-k-mooney | zigo: we still have it in tree https://github.com/openstack/nova/tree/master/nova/virt/zvm im not sure what its state is | |
| 17:30:48 | gmann | dansmith: patches merged are few volume detach are made SSH-able but the failing test in centos9-stream was rescue negative which is in-progress in 831608 | |
| 17:31:46 | gmann | dansmith: those merged were not failing, may be due to the wait between server create and detach operation call. in rescue negative timing were playing key role | |
| 17:32:38 | dansmith | gmann: okay the latest failure I'm looking at includes test_rescued_vm_detach_volume but there's another in there, which may or may not be related | |
| 17:32:46 | dansmith | but yeah good to know | |
| 17:34:16 | sean-k-mooney | dansmith: how oftten are you seeing the failure by the way | |
| 17:34:24 | sean-k-mooney | is it blockign the gate consitently | |
| 17:34:25 | gmann | dansmith: yeah that test and in negative test just try the detach and assert expected failure as detach cannot be done on rescue server but later this test does un-rescue server and detach in cleanup there it stuck | |
| 17:34:40 | dansmith | sean-k-mooney: on centos, this was one very common one we were suffering in the glance job | |
| 17:34:42 | gmann | * in that negative test | |
| 17:35:04 | dansmith | gmann: do those tests actually ssh for some reason, or are we just using ssh to determine readiness? | |
| 17:35:04 | sean-k-mooney | does glance need to test this? | |
| 17:35:34 | gmann | dansmith: just to check readiness | |
| 17:35:36 | dansmith | sean-k-mooney: yeah, this was a glance-cinder-multistore job which needs to run volume-related tests of course | |
| 17:35:46 | sean-k-mooney | some volume operation are certenly glance related but detach proably isnt | |
| 17:36:01 | dansmith | gmann: we could use the login prompt via console instead to reduce the need for secgroups, if that's hard for some reason | |
| 17:36:25 | dansmith | sean-k-mooney: it's a job that tests cinder-glance multistore arrangements | |
| 17:36:28 | gmann | dansmith: that is one try if that fix it. but there might be some other issue. we will see if that patch (once pass gate) can pass cento9 job too | |
| 17:36:46 | sean-k-mooney | dansmith: we removed usign the console in an eairler patch | |
| 17:36:49 | dansmith | sean-k-mooney: but the only reason we were running on centos was because we were trying to get a fips job and used one of our existing wide-coverage jobs to do that | |
| 17:37:01 | dansmith | gmann: ack | |
| 17:37:11 | gmann | yeah SSH-able was preferred than console check | |
| 17:37:25 | dansmith | it's definitely good, it's just more than required for this but fair enough | |
| 17:37:58 | gmann | let me debug sec group thing today or monday. I compared and it was same as other test doing but i might have missing something | |
| 17:38:17 | sean-k-mooney | i think the console check failed in a specific edgecase but i dont recall what it was exactly. | |
| 17:38:32 | dansmith | it's just out-of-band, so a little less fragile, | |
| 17:38:46 | dansmith | but it's cool if we're trying to stick to sshable as the indicator | |
| 18:10:03 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add whitebox-devstack-multinode job to periodic https://review.opendev.org/c/openstack/nova/+/833453 | |
| 21:46:48 | opendevreview | Dan Smith proposed openstack/nova master: Attempt to thin out nova-ceph-multistore https://review.opendev.org/c/openstack/nova/+/833470 | |
| #openstack-nova - 2022-03-12 | |||
| 00:20:53 | opendevreview | Merged openstack/nova master: Changes Emulation CI to weekly-periodic https://review.opendev.org/c/openstack/nova/+/833163 | |
| #openstack-nova - 2022-03-14 | |||
| 09:57:07 | bauzas | happy Hugarian Holidays elodilles and gibi | |
| 11:00:13 | opendevreview | Andre Aranha proposed openstack/nova master: Move FIPS jobs to experimental and periodic queue https://review.opendev.org/c/openstack/nova/+/833431 | |
| 11:23:09 | elodilles | bauzas: thx :D | |
| 11:48:29 | opendevreview | Andre Aranha proposed openstack/nova master: Move FIPS jobs to experimental and periodic queue https://review.opendev.org/c/openstack/nova/+/833431 | |
| 11:48:30 | opendevreview | Andre Aranha proposed openstack/nova master: Test setting the nova job to centos-9-stream https://review.opendev.org/c/openstack/nova/+/831844 | |
| 11:54:19 | opendevreview | Andre Aranha proposed openstack/nova master: Move FIPS jobs to experimental and periodic queue https://review.opendev.org/c/openstack/nova/+/833431 | |
| 11:56:17 | opendevreview | Andre Aranha proposed openstack/nova master: Test setting the nova job to centos-9-stream https://review.opendev.org/c/openstack/nova/+/831844 | |
| 13:33:20 | Uggla | lyarwood, I have a question regarding the manila, virtiofs specs. (https://specs.openstack.org/openstack/nova-specs/specs/yoga/approved/libvirt-virtiofs-attach-manila-shares.html) Any reason not to store the share_proto (nfs, cephfs...) on the object ? | |
| 13:43:09 | tobias-urdin | elodilles: please see stable backport request https://review.opendev.org/c/openstack/nova/+/828407 thanks! | |
| 13:45:53 | lyarwood | Uggla: is that not encoded in the export_locaiton we get from Manila? If it's not then I don't think caching it in Nova would be an issue tbh. | |
| 13:50:43 | Uggla | lyarwood, no this is not in the export location. Only the ip + path are available. So I think I will add the proto. | |
| 13:55:11 | lyarwood | https://docs.openstack.org/api-ref/shared-file-system/?expanded=show-single-export-location-detail,show-all-share-metadata-detail,show-share-details-detail#id27 - Yeah odd I thought it was in the export_location for some reason, it's there in the share details at least so it's not that hard to grab a copy | |
| 14:05:10 | dansmith | bauzas: are you okay with this? https://review.opendev.org/c/openstack/nova/+/833470 | |
| 14:05:31 | dansmith | that job has been OOMing a lot lately, taking out mysqld when it does | |
| 14:05:49 | dansmith | reducing the scope to exclude swift should lower the pressure a bit I think, without compromising what the job is intending to test | |
| 14:06:40 | dansmith | gmann: ^ | |
| 15:15:05 | bauzas | dansmith: sorry, was with Uggla and also had to help my kid | |
| 15:15:36 | bauzas | dansmith: oh, I see | |
| 15:16:03 | bauzas | dansmith: oh yes, not sure why we should use Swift | |
| 15:17:23 | dansmith | thanks | |
| 17:06:53 | artom | bauzas, as requested, the backport link: https://review.opendev.org/c/openstack/nova/+/828414 | |
| 17:07:07 | opendevreview | Elod Illes proposed openstack/nova stable/pike: [stable-only] Fix sphinx doc job https://review.opendev.org/c/openstack/nova/+/833666 | |
| 17:12:42 | opendevreview | ribaudr proposed openstack/nova-specs master: libvirt: Allow Manila shares to be directly attached to instances https://review.opendev.org/c/openstack/nova-specs/+/833669 | |
| 17:39:11 | gmann | dansmith: lgtm but can you add note there so that nobody will accidently enable them it https://review.opendev.org/c/openstack/nova/+/833470/1/.zuul.yaml#588 | |
| 17:39:28 | dansmith | gmann: sure | |
| 17:39:44 | gmann | thanks | |
| 17:40:44 | opendevreview | Dan Smith proposed openstack/nova master: Attempt to thin out nova-ceph-multistore https://review.opendev.org/c/openstack/nova/+/833470 | |
| 17:40:48 | dansmith | gmann: ^ | |
| 17:41:40 | gmann | dansmith: +2. bauzas ^^ | |
| 17:47:26 | bauzas | gmann: dansmith: sent to the gate | |
| 17:48:59 | dansmith | bauzas: everyone thanks you, including me :P | |
| 17:49:06 | bauzas | :) | |
| 20:06:56 | admin1 | i have cinder and glance in ceph, but nova is local storage .. when i take snapshot .. it says saving .. and then quickly disappears .. is snapshot a glance or a nova thing and where would the logs for those be ? | |
| 20:09:20 | dansmith | admin1: nova does the snapshot, and uploads it to glance when it's done | |
| 20:09:31 | dansmith | nova-compute, specifically | |