| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-03-02 | |||
| 15:37:43 | bauzas | fucking fingers | |
| 15:40:17 | dansmith | wow, no microversions in yoga? | |
| 15:40:24 | dansmith | that's both maybe a little scary and kinda amazing | |
| 15:40:50 | Uggla | bauzas, cool so maybe I will not have to change it in my patch. \o/ | |
| 15:41:15 | bauzas | dansmith: we only had two open changes for it | |
| 15:41:30 | bauzas | dansmith: one for tenant > project usage | |
| 15:42:30 | bauzas | the other one for a /servers new UUID parameter | |
| 15:43:12 | bauzas | (for booting to a specific hypervisor instead of using a very bad AZ param) | |
| 15:43:28 | dansmith | bauzas: and the volume rebuild | |
| 15:43:33 | bauzas | ah right | |
| 16:12:09 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova/+/831507 | |
| 16:22:59 | melwitt | gmann: I understand there are no additional tempest test running on skip level job but nova-grenade-multinode will only be testing that policy change doesn't break N -> N+1 but only the skip level job will test N -> N+2. maybe I am missing something? | |
| 16:25:47 | melwitt | nvm just saw your later comment on the review | |
| 16:26:00 | dansmith | oh sorry, | |
| 16:26:07 | dansmith | I just realized my git-review failed to merge with master | |
| 16:27:50 | opendevreview | Dan Smith proposed openstack/nova master: Add grenade-skip-level irrelevant-files config https://review.opendev.org/c/openstack/nova/+/831229 | |
| 16:27:54 | dansmith | thar we go ^ | |
| 16:28:59 | gmann | melwitt: yeah | |
| 16:29:02 | gmann | dansmith: thanks | |
| 17:22:54 | opendevreview | Balazs Gibizer proposed openstack/nova master: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/829248 | |
| 17:42:57 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova/+/831507 | |
| 19:44:33 | ade_lee_ | sean-k-mooney, dansmith - hey - could you guys take a look at some jobs -- https://review.opendev.org/c/openstack/nova/+/827895 | |
| 19:45:26 | ade_lee_ | sean-k-mooney, dansmith trying to figure out why the centos/fips jobs are failing -- this seems to be the same problem in other jobs too. | |
| 19:46:33 | ade_lee_ | sean-k-mooney, dansmith eharney looked and in tempest.api.compute.servers.test_server_rescue.ServerStableDeviceRescueTest.test_stable_device_rescue_disk_virtio_with_volume_attached claims that the volume doesn't detach but if you look at the cinder and nova logs, it seems to be doing all the right things.. | |
| 19:46:42 | ade_lee_ | (for instance) | |
| 19:54:43 | sean-k-mooney | that could be due to libvirt | |
| 19:54:49 | sean-k-mooney | is this happening on centos 9 stream | |
| 19:55:05 | sean-k-mooney | if its only centos 8 stream we shoudl proably ignore it | |
| 19:55:28 | sean-k-mooney | ya the fips job is still centos 8 | |
| 19:55:46 | sean-k-mooney | so there are issues with q35 and device detach | |
| 19:55:57 | sean-k-mooney | does the fips job use the q35 manchien type? | |
| 19:56:28 | ade_lee_ | sean-k-mooney, q35? | |
| 19:57:09 | ade_lee_ | sean-k-mooney, let me retry it with centos-9 stream -- we want to move in that direction in any case | |
| 19:58:49 | clarkb | sean-k-mooney: ade_lee_ I think the default was updated to q35 beacuse the previous default in devstack wouldn't boot centos-9 | |
| 19:59:02 | opendevreview | Merged openstack/nova stable/victoria: [rt] Apply migration context for incoming migrations https://review.opendev.org/c/openstack/nova/+/820559 | |
| 20:03:12 | sean-k-mooney | clarkb: that should not be done globally | |
| 20:03:24 | sean-k-mooney | that will break our testing expectations | |
| 20:03:37 | sean-k-mooney | we expect q35 to only be enabled in nova next currently | |
| 20:04:29 | clarkb | ah maybe we didn't jump that far ahead. We definitely had to move the machine type ahead to boot centos 9 | |
| 20:04:47 | sean-k-mooney | clarkb: you should not have too | |
| 20:04:54 | sean-k-mooney | centos 9 shoudl boot fine with pc | |
| 20:05:40 | sean-k-mooney | clarkb: do you have any link to that change | |
| 20:05:42 | clarkb | ah we bumped the cpu model not the chipset | |
| 20:05:47 | sean-k-mooney | ah right | |
| 20:05:51 | clarkb | e06d954229fc4fca827105f5bb0809a19075d590 | |
| 20:05:53 | sean-k-mooney | to nehelm | |
| 20:05:55 | clarkb | yes | |
| 20:06:01 | sean-k-mooney | ya that is differnt | |
| 20:06:03 | sean-k-mooney | that is fien | |
| 20:06:15 | clarkb | Q35 is older than nehalem though | |
| 20:06:34 | clarkb | But I guess those things don't move in lockstep when it comes to virt | |
| 20:06:52 | sean-k-mooney | q35 is basicaly the motherboard | |
| 20:07:01 | sean-k-mooney | rhather then cpu | |
| 20:07:07 | sean-k-mooney | but ya they are independent | |
| 20:07:10 | clarkb | right but the era it comes from is pre nehalem | |
| 20:07:17 | sean-k-mooney | ya | |
| 20:07:21 | clarkb | nehalem was ~2021 and q35 was ~2007 | |
| 20:07:24 | clarkb | er 2012 | |
| 20:07:29 | clarkb | too much typing 2021 last year | |
| 20:07:32 | sean-k-mooney | :) | |
| 21:33:20 | ade_lee_ | sean-k-mooney, trying centos-9 -- lets see how that goes .. https://review.opendev.org/c/openstack/tempest/+/831607/ | |
| 22:37:38 | melwitt | gmann: just a heads up that the following devstack and tempest changes are dependencies for enabling unified limits test coverage in nova-next https://review.opendev.org/q/topic:bp/unified-limits-nova+AND+is:open+AND+(project:openstack/devstack+OR+project:openstack/tempest) | |
| 23:50:08 | gmann | melwitt: ack, I will review tomorrow | |
| 23:53:38 | melwitt | thanks gmann | |
| #openstack-nova - 2022-03-03 | |||
| 07:30:40 | gibi | dmitriis, sean-k-mooney: I made a fix for the func test pci device issue in https://review.opendev.org/c/openstack/nova/+/829248/4/nova/compute/resource_tracker.py let me know what you think. I'm happy to split that patch up if needed | |
| 10:25:51 | opendevreview | ribaudr proposed openstack/python-novaclient master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/python-novaclient/+/831651 | |
| 10:50:59 | opendevreview | OpenStack Release Bot proposed openstack/os-vif stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/os-vif/+/831664 | |
| 10:51:03 | opendevreview | OpenStack Release Bot proposed openstack/os-vif stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/os-vif/+/831670 | |
| 10:51:11 | opendevreview | OpenStack Release Bot proposed openstack/os-vif master: Update master for stable/yoga https://review.opendev.org/c/openstack/os-vif/+/831678 | |
| 10:51:29 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/osc-placement/+/831695 | |
| 10:51:31 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/osc-placement/+/831698 | |
| 10:51:35 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement master: Update master for stable/yoga https://review.opendev.org/c/openstack/osc-placement/+/831701 | |
| 10:51:37 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/python-novaclient/+/831704 | |
| 10:51:39 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/python-novaclient/+/831705 | |
| 10:51:41 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient master: Update master for stable/yoga https://review.opendev.org/c/openstack/python-novaclient/+/831706 | |
| 12:55:02 | opendevreview | Merged openstack/os-vif stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/os-vif/+/831664 | |
| 12:57:52 | opendevreview | Merged openstack/os-vif stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/os-vif/+/831670 | |
| 13:09:15 | opendevreview | ribaudr proposed openstack/python-novaclient master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/python-novaclient/+/831651 | |
| 13:22:03 | sean-k-mooney | by the way am i the only one who sees this when they run commands | |
| 13:22:06 | sean-k-mooney | /usr/lib/python3/dist-packages/secretstorage/dhcrypto.py:15: CryptographyDeprecationWarning: int_from_bytes is deprecated, use int.from_bytes instead | |
| 13:22:08 | sean-k-mooney | from cryptography.utils import int_from_bytes | |
| 13:22:10 | sean-k-mooney | /usr/lib/python3/dist-packages/secretstorage/util.py:19: CryptographyDeprecationWarning: int_from_bytes is deprecated, use int.from_bytes instead | |
| 13:22:12 | sean-k-mooney | from cryptography.utils import int_from_bytes | |
| 13:22:26 | sean-k-mooney | im seing it for nova manage and also osc | |
| 13:25:59 | sean-k-mooney | i assume secretstorage is managed by uc | |
| 13:26:12 | sean-k-mooney | i wonder if that has beeen adressed there yet | |
| 13:54:05 | gibi | sean-k-mooney: I see that in a fairly recent devstack from master too so I think this is still happening | |
| 14:12:19 | sean-k-mooney | gibi: ack i think this need to be updated in secrtstorage but i dont even know what we use that for | |
| 14:13:08 | sean-k-mooney | like sure it provides a way for securly storing password | |
| 14:13:10 | sean-k-mooney | https://pypi.org/project/SecretStorage/ | |
| 14:13:27 | sean-k-mooney | but why woudl we need dbus integratoin for nova-manage | |
| 14:13:43 | sean-k-mooney | liek we are not going to be pulling things form the gnome keyring | |
| 14:14:17 | sean-k-mooney | im guessing this is really from oslo | |
| 14:15:37 | sean-k-mooney | actully i dont see it as a direct dep in any openstack porject that woudl make sense | |
| 14:15:39 | sean-k-mooney | https://codesearch.opendev.org/?q=secretstorage&i=nope&literal=nope&files=&excludeFiles=&repos= | |
| 14:19:50 | sean-k-mooney | based on strace it looks like its coming form keyring | |
| 14:21:44 | sean-k-mooney | it might be a sideffect fo stevador and the entrypoint scan it does | |
| 14:26:45 | sean-k-mooney | gibi: ok so this is from python-keystoneclient | |
| 14:26:48 | sean-k-mooney | https://opendev.org/openstack/python-keystoneclient/commit/5939541bc771e1205394b05e757d7b23b3aca862 | |
| 14:32:47 | opendevreview | Merged openstack/python-novaclient master: Update master for stable/yoga https://review.opendev.org/c/openstack/python-novaclient/+/831706 | |