| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-12-07 | |||
| 17:35:44 | opendevreview | Merged openstack/nova master: Use SDK for volume_target.* https://review.opendev.org/c/openstack/nova/+/867790 | |
| 17:35:50 | opendevreview | Merged openstack/nova master: Use SDK for node.set_provision_state https://review.opendev.org/c/openstack/nova/+/867791 | |
| 17:35:58 | opendevreview | Merged openstack/nova master: Use SDK for node.set_power_state https://review.opendev.org/c/openstack/nova/+/867792 | |
| 18:02:52 | sean-k-mooney | stephenfin: that is ill docuemtned buyt it depend on when the port is added | |
| 18:49:30 | gmann | melwitt: done. i opened it but somehow i forgot to review it yesterday | |
| 18:51:23 | melwitt | thanks gmann ! | |
| 21:08:31 | opendevreview | Merged openstack/nova master: Remove redundant setting of DEFAULT_IMAGE_NAME(_FILE) https://review.opendev.org/c/openstack/nova/+/902734 | |
| 21:08:38 | opendevreview | Merged openstack/nova master: Set UEC image vars for jobs not defined in Nova https://review.opendev.org/c/openstack/nova/+/902809 | |
| 22:28:46 | opendevreview | Merged openstack/nova master: Use SDK for node.vif* https://review.opendev.org/c/openstack/nova/+/867793 | |
| #openstack-nova - 2023-12-08 | |||
| 07:02:04 | opendevreview | Pranali Deore proposed openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | |
| 07:02:29 | opendevreview | Pranali Deore proposed openstack/nova master: DNM: Test glance new location api https://review.opendev.org/c/openstack/nova/+/891207 | |
| 07:38:27 | opendevreview | Merged openstack/nova master: Use SDK for remaining network operations https://review.opendev.org/c/openstack/nova/+/867794 | |
| 08:23:16 | gibi | hm, nobody can say that our gate is blocking us merging things, we merged 9 patches since I left yesterday | |
| 09:48:18 | opendevreview | Alex Welsh proposed openstack/nova master: Make map_cell0 command update existing mappings https://review.opendev.org/c/openstack/nova/+/903140 | |
| 10:29:55 | opendevreview | Alex Welsh proposed openstack/nova master: Make map_cell0 command update existing mappings https://review.opendev.org/c/openstack/nova/+/903140 | |
| 10:30:38 | opendevreview | Pranali Deore proposed openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | |
| 10:30:49 | opendevreview | Pranali Deore proposed openstack/nova master: DNM: Test glance new location api https://review.opendev.org/c/openstack/nova/+/891207 | |
| 10:32:04 | opendevreview | Alex Welsh proposed openstack/nova master: Make map_cell0 command update existing mappings https://review.opendev.org/c/openstack/nova/+/903140 | |
| 10:44:46 | opendevreview | Amit Uniyal proposed openstack/nova master: enforce remote console shutdown https://review.opendev.org/c/openstack/nova/+/901824 | |
| 11:09:54 | gibi | bauzas: additional power management bugfix is incoming https://bugs.launchpad.net/nova/+bug/2045966 | |
| 11:11:18 | bauzas | ohch | |
| 11:11:30 | bauzas | I thought scaling_governor was supported since 2015 | |
| 11:12:14 | bauzas | https://lwn.net/Articles/682391/ | |
| 11:12:18 | bauzas | my bad, 2016 | |
| 11:12:35 | bauzas | when I meant 'supported', I meant 'enabled by default in the kernel' | |
| 11:12:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: [pwmgmt]ignore missin governor when cpu_state used https://review.opendev.org/c/openstack/nova/+/903169 | |
| 11:13:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: [pwmgmt]ignore missin governor when cpu_state used https://review.opendev.org/c/openstack/nova/+/903169 | |
| 11:13:42 | gibi | bauzas: if you install centos 9 stream, by default it has no governor | |
| 11:14:06 | gibi | bauzas: I can give you access to a beaker node downstream :) | |
| 11:14:15 | gibi | anyhow the fix is fairly simple | |
| 11:14:26 | bauzas | nah nah I trust you | |
| 11:14:28 | gibi | and I will make sure it will be backported to antelope | |
| 11:15:24 | bauzas | fwiw, my fedora (38 IIRC) ships scaling_governor | |
| 11:15:26 | bauzas | [sbauza@sbauza nova-specs]$ ll /sys/devices/system/cpu/cpu1/cpufreq/scaling_governor | |
| 11:15:28 | bauzas | -rw-r--r--. 1 root root 4096 8 déc. 12:14 /sys/devices/system/cpu/cpu1/cpufreq/scaling_governor | |
| 11:15:28 | bauzas | -rw-r--r--. 1 root root 4096 8 déc. 12:14 /sys/devices/system/cpu/cpu1/cpufreq/scaling_governor | |
| 11:15:35 | bauzas | wtf | |
| 11:16:18 | bauzas | humpgf | |
| 11:16:20 | bauzas | [stack@lenovo-sr655-01 nova]$ ll /sys/devices/system/cpu/cpu1/cpufreq/scaling_governor | |
| 11:16:22 | bauzas | -rw-r--r--. 1 root root 4096 Dec 5 13:30 /sys/devices/system/cpu/cpu1/cpufreq/scaling_governor | |
| 11:16:24 | bauzas | [stack@lenovo-sr655-01 nova]$ cat /etc/redhat-release | |
| 11:16:26 | bauzas | Red Hat Enterprise Linux release 9.2 (Plow) | |
| 11:16:28 | bauzas | gibi ^ | |
| 11:17:09 | bauzas | I don't want Stream f... up with cpufreq but they broke ity | |
| 11:17:30 | bauzas | don't know* what* | |
| 11:18:01 | gibi | [root@osp-dev-01 ~]# cat /etc/redhat-release | |
| 11:18:01 | gibi | CentOS Stream release 9 | |
| 11:18:18 | gibi | [root@osp-dev-01 ~]# ll /sys/devices/system/cpu/cpu1/cpufreq/scaling_governor | |
| 11:18:21 | gibi | ls: cannot access '/sys/devices/system/cpu/cpu1/cpufreq/scaling_governor': No such file or directory | |
| 11:18:24 | gibi | [root@osp-dev-01 ~]# ll /sys/devices/system/cpu/cpu1/cpufreq/ | |
| 11:18:26 | gibi | ls: cannot access '/sys/devices/system/cpu/cpu1/cpufreq/': No such file or directory | |
| 11:19:45 | bauzas | that's crazy | |
| 11:20:00 | bauzas | I +2d your change anyway, but I'm very surprised | |
| 11:20:02 | gibi | but I checked onlineing / offlineing works | |
| 11:20:37 | bauzas | cpufreq is one very common kernel module | |
| 11:28:13 | opendevreview | Alex Welsh proposed openstack/nova master: Make map_cell0 command update existing mappings https://review.opendev.org/c/openstack/nova/+/903140 | |
| 11:28:56 | bauzas | gibi: out of curiosity, could you please issue on your c9s box : | |
| 11:29:01 | bauzas | find /lib/modules/$(uname -r)/kernel/ | grep cpufreq | |
| 11:46:56 | gibi | [root@osp-dev-01 ~]# find /lib/modules/$(uname -r)/kernel/ | grep cpufreq | |
| 11:46:59 | gibi | /lib/modules/5.14.0-386.el9.x86_64/kernel/drivers/cpufreq | |
| 11:47:02 | gibi | /lib/modules/5.14.0-386.el9.x86_64/kernel/drivers/cpufreq/acpi-cpufreq.ko.xz | |
| 11:47:05 | gibi | /lib/modules/5.14.0-386.el9.x86_64/kernel/drivers/cpufreq/amd-pstate-ut.ko.xz | |
| 11:47:08 | gibi | /lib/modules/5.14.0-386.el9.x86_64/kernel/drivers/cpufreq/amd_freq_sensitivity.ko.xz | |
| 11:47:11 | gibi | /lib/modules/5.14.0-386.el9.x86_64/kernel/drivers/cpufreq/p4-clockmod.ko.xz | |
| 11:47:14 | gibi | /lib/modules/5.14.0-386.el9.x86_64/kernel/drivers/cpufreq/powernow-k8.ko.xz | |
| 11:47:17 | gibi | /lib/modules/5.14.0-386.el9.x86_64/kernel/drivers/cpufreq/speedstep-lib.ko.xz | |
| 11:52:25 | bauzas | gibi: thanks so cpufreq is shipped | |
| 11:52:38 | bauzas | gibi: lsmod | grep cpufreq ? | |
| 11:52:57 | gibi | that is empty | |
| 11:53:10 | bauzas | I'll ask cfontain if he knows why cpufreq isn't there in c9s | |
| 11:53:17 | gibi | but it is also empty in my desktop where I have cpufreq | |
| 11:53:21 | bauzas | maybe because of tuned | |
| 11:54:47 | admin1 | hi all .. how to get a vm out of " Confirm or Revert Resize/Migrate " stage where both confirm or revert fails to work .. these vms are backed by ceph and came to this state where their hypervisor died and were in shutdown state | |
| 11:55:23 | bauzas | gibi: oh that's a boot kernel config | |
| 11:56:26 | bauzas | gibi: cat /boot/config-$(uname -r) | grep CONFIG_CPU_FREQ | |
| 11:57:28 | gibi | https://paste.opendev.org/show/bknmxAOZRtxwb0pNBaSH/ | |
| 12:01:25 | bauzas | I'm then out of ideas why the sysfs directory isn't popping up | |
| 12:01:47 | bauzas | anyway, the patch is good to me, it's good to be conservative here | |
| 12:02:34 | bauzas | I'll talk to Christophe in case (which is more than possible) he knows the reason :) | |
| 12:03:40 | bauzas | oh damn, he's on PTO :D | |
| 12:03:55 | gibi | bauzas: OK. thanks for looking into it | |
| 12:04:07 | bauzas | np, thanks for testing it :) | |
| 12:04:35 | bauzas | I could have been more cautious | |
| 12:04:56 | bauzas | and not assume every single box (from RPi to large mainframes) includes cpufreq | |
| 12:05:12 | gibi | it was a surprise for me too | |
| 12:05:38 | gibi | first I thought it is due to using a VM as compute. But then I checked centos on a real hw and I see the same | |
| 12:06:02 | bauzas | maybe the journal said the reason | |
| 12:06:12 | bauzas | maybe it was deactivate | |
| 12:06:17 | bauzas | deactivated* | |
| 12:06:19 | bauzas | anyway | |
| 12:06:22 | gibi | nothing in journalctl | grep cpufreq | |
| 12:06:44 | gibi | journalctl | grep cpufreq | |
| 12:06:46 | gibi | sorry | |
| 12:07:19 | gibi | btw, offlining cpus when a compute is a VM works (but only offlines the cpu from the VM perspective obviously) | |
| 12:07:39 | bauzas | makes sense | |
| 12:07:50 | gibi | so at some point we can have some end to end test for this in openstack-k8s-operators CI upstrea | |
| 12:08:15 | gibi | currently I only have https://github.com/openstack-k8s-operators/nova-operator/pull/597#issuecomment-1846870113 as we don't have dedicated CPUs in CI yet | |
| 12:11:10 | bauzas | https://docs.kernel.org/core-api/cpu_hotplug.html | |
| 12:11:20 | bauzas | that's what the offline does | |
| 12:12:01 | bauzas | I meant https://docs.kernel.org/core-api/cpu_hotplug.html#the-offline-case | |