| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-12-07 | |||
| 11:51:22 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use clearer syntax for empty generator https://review.opendev.org/c/openstack/nova/+/903076 | |
| 11:51:32 | stephenfin | sean-k-mooney[m]: "remove dark magic" patch | |
| 11:51:37 | stephenfin | ^ | |
| 11:52:42 | sean-k-mooney[m] | thanks | |
| 12:09:52 | opendevreview | Pranali Deore proposed openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | |
| 12:10:02 | opendevreview | Pranali Deore proposed openstack/nova master: DNM: Test glance new location api https://review.opendev.org/c/openstack/nova/+/891207 | |
| 13:36:55 | sean-k-mooney | hi can i get some eyes on https://review.opendev.org/c/openstack/devstack/+/890693 please. | |
| 13:37:06 | sean-k-mooney | oh that was for qa.. | |
| 13:40:48 | sean-k-mooney | tempest-integrated compute often seamm to get pretty close to its 2 hour timeout but there is also a lot of variance | |
| 13:40:54 | sean-k-mooney | https://zuul.opendev.org/t/openstack/builds?job_name=tempest-integrated-compute&project=openstack%2Fnova | |
| 14:10:13 | gibi | stephenfin: nice series, and now it is approved all the way up :) | |
| 14:10:40 | stephenfin | gibi: 👏 thank you! | |
| 14:10:40 | stephenfin | gibi: � thank you! | |
| 15:09:50 | bauzas | if people want to see my face live (after all, this is not like I see most of you everyday on gmeet) : https://www.youtube.com/live/thidlQGX29M?feature=shared | |
| 15:10:14 | bauzas | tl;dr: PTG recap OIL episode | |
| 16:15:07 | melwitt | gmann: if you get a moment https://review.opendev.org/c/openstack/nova/+/902809 | |
| 16:16:11 | melwitt | sean-k-mooney: I have noticed TIMED_OUT too for job running just a little over 2 hours | |
| 16:16:47 | sean-k-mooney | yep | |
| 16:16:51 | sean-k-mooney | i noticed that too | |
| 16:17:11 | sean-k-mooney | thats why i mentioned my zswap changes for devstack | |
| 16:17:15 | melwitt | I said "too" first, I saw your message in the scrollback :) | |
| 16:17:19 | sean-k-mooney | since that was ment to speed up the jobs | |
| 16:17:30 | melwitt | ah, cool, I'll check that out | |
| 16:17:43 | sean-k-mooney | it does not seam to be that common | |
| 16:57:16 | stephenfin | is it expected that adding a port will add the default security group to an instance if not already present? | |
| 16:58:06 | stephenfin | I'm guessing yes assuming a port is using the default security group | |
| 17:04:32 | bauzas | sean-k-mooney: I had zero time today to discuss with you about my mdev live-migration spec and the ping-pong game | |
| 17:09:42 | bauzas | long story short, the ping-pong game is made when the conductor calls check_dest() https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L8501-L8572 | |
| 17:12:26 | bauzas | if we really want to do a lockstep here by once, then we move the type check later in the post claim method, and we just return the current mdev and their types from the source in check_can_live_migrate_source() and we test and claim them in post_claim_migrate_data() | |
| 17:12:39 | bauzas | I'm OK with this change | |
| 17:30:30 | opendevreview | Merged openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/c/openstack/nova/+/659691 | |
| 17:33:32 | opendevreview | Merged openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/c/openstack/nova/+/670213 | |
| 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 | CentOS Stream release 9 | |
| 11:18:01 | gibi | [root@osp-dev-01 ~]# cat /etc/redhat-release | |
| 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 ? | |