Earlier  
Posted Nick Remark
#openstack-nova - 2020-07-09
12:41:40 sean-k-mooney i almost never write c++ but i watch a tone of talks ans podcasts on c++
12:42:19 sean-k-mooney i hate c but to be honest after 7+ years of python at this point im now coming to like it as much or more then c++
12:42:48 sean-k-mooney type hits certenly remove one of my gripes with python
12:46:35 gibi I wrote c++ last time 7 years ago still I appreciate it as a language. I would go with Rust if I have to choose today
12:48:12 sean-k-mooney i dont like the syntax of rust but i can appreciate some of the design chosies regarding the borrow checker. though the clang based tooling that has been wrrent for c++ is quickly approchign the same level of checking
12:49:08 sean-k-mooney languates that use let to intoduce variables like basic just feel strange to me
12:49:45 sean-k-mooney it took me a very long time to get used to type on the right had side of an identifier too
12:50:47 sean-k-mooney its becomeing the norm now but the reason new lanagues do that is its eaier to write the parser it has nothing to do with human readabilty
12:55:55 gibi stephenfin: you did a nice job splitting up the vtpm series. thank you for that
12:56:36 stephenfin np
12:56:56 stephenfin thanks for the reviews :)
12:57:05 stephenfin be sure to let me know if I can return the favour
12:57:46 gibi sure :)
13:05:37 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Reject boot request for unsupported images https://review.opendev.org/708608
13:05:42 openstack bug 1550919 in OpenStack Compute (nova) rocky "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Lee Yarwood (lyarwood)
13:05:42 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Add functional test for bug 1550919 https://review.opendev.org/732707
13:05:52 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732708
13:07:24 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: compute: Allow snapshots to be created from PAUSED volume backed instances https://review.opendev.org/729177
13:07:39 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: libvirt: check job status for VIR_DOMAIN_EVENT_SUSPENDED_MIGRATED event https://review.opendev.org/711233
13:08:55 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Block rebuild when NUMA topology changed https://review.opendev.org/703116
13:08:55 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Remove 'test_cold_migrate_with_physnet_fails' test https://review.opendev.org/703115
13:08:56 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: FUP for in-place numa rebuild https://review.opendev.org/703118
13:08:56 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Disable NUMATopologyFilter on rebuild https://review.opendev.org/703117
13:09:29 lyarwood sean-k-mooney: ^ rebased for you btw
13:16:46 sean-k-mooney oh cool thanks
13:37:09 openstackgerrit Huaqiang Wang proposed openstack/nova master: libvirt: apply mixed instance CPU policy https://review.opendev.org/728481
13:37:09 openstackgerrit Huaqiang Wang proposed openstack/nova master: objects: Introduce the 'CPUAllocationPolicy.MIXED' enum https://review.opendev.org/716267
13:37:10 openstackgerrit Huaqiang Wang proposed openstack/nova master: Calculate the CPU usage for mixed instance https://review.opendev.org/716269
13:37:10 openstackgerrit Huaqiang Wang proposed openstack/nova master: Validate CPU pinning configuration for mixed instance https://review.opendev.org/716268
13:37:11 openstackgerrit Huaqiang Wang proposed openstack/nova master: flavor: Enable 'hw:cpu_dedicated_mask' for creating a mixed instance https://review.opendev.org/714706
13:37:11 openstackgerrit Huaqiang Wang proposed openstack/nova master: scheduler: mixed instance CPU resource translating https://review.opendev.org/728482
13:37:12 openstackgerrit Huaqiang Wang proposed openstack/nova master: hardware: create 'mixed' instance for realtime and shared CPUs https://review.opendev.org/728485
13:37:12 openstackgerrit Huaqiang Wang proposed openstack/nova master: compute: bump nova-compute service version for mixed instance https://review.opendev.org/716271
13:37:13 openstackgerrit Huaqiang Wang proposed openstack/nova master: metadata: export the vCPU IDs that are pinning on the host CPUs https://review.opendev.org/688936
13:37:13 openstackgerrit Huaqiang Wang proposed openstack/nova master: hardware: refactor numa_get_constraints https://review.opendev.org/718294
13:37:14 openstackgerrit Huaqiang Wang proposed openstack/nova master: release note: add new features introduces by use-pcpu-and-vcpu-in-one-instance https://review.opendev.org/736997
13:38:11 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Simplify '_create_domain' function https://review.opendev.org/739208
13:38:14 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Add vTPM config support https://review.opendev.org/739209
13:38:15 openstackgerrit Stephen Finucane proposed openstack/nova master: crypto: Add support for creating, destroying vTPM secrets https://review.opendev.org/739211
13:38:15 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Request vTPM trait based on flavor or image https://review.opendev.org/739210
13:38:16 openstackgerrit Stephen Finucane proposed openstack/nova master: Add emulated TPM support to Nova https://review.opendev.org/631363
13:38:16 openstackgerrit Stephen Finucane proposed openstack/nova master: manager: Prevent compute startup on invalid vTPM config https://review.opendev.org/739212
13:38:17 openstackgerrit Stephen Finucane proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.opendev.org/639934
13:38:18 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Add docs for vTPM support https://review.opendev.org/739213
13:44:08 openstackgerrit Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184
13:48:29 stephenfin gibi: Have effectively rewritten https://review.opendev.org/#/c/739208/ again from learnings on the previous PS. Hopefully it makes sense
13:57:38 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP/DNM zuul: Attempt to migrate and break up the nova-live-migration job https://review.opendev.org/711604
14:12:32 openstackgerrit Merged openstack/nova master: hardware: Invert order of NUMA topology generation https://review.opendev.org/714701
14:56:24 openstackgerrit Ghanshyam Mann proposed openstack/os-vif master: DNM: testing os-vif gate on ubuntu focal https://review.opendev.org/738130
15:11:22 openstackgerrit Merged openstack/nova stable/queens: Fix os_CODENAME detection and repo refresh during ceph tests https://review.opendev.org/740193
15:22:55 openstackgerrit Ghanshyam Mann proposed openstack/nova master: migrate testing to ubuntu focal https://review.opendev.org/738126
15:42:58 openstackgerrit Merged openstack/python-novaclient master: Switch legacy Zuul jobs to native Zuul v3 jobs https://review.opendev.org/725163
15:44:45 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: Remove stale nested backport from InstancePCIRequests https://review.opendev.org/738199
15:44:48 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: libvirt: Mark e1000e VIF as supported https://review.opendev.org/736962
15:48:24 gibi stephenfin: https://review.opendev.org/#/c/739208/ looks good to me
15:51:30 gibi nova meeting starts in 9 minutes on #openstack-meeting-3
15:56:07 openstackgerrit Artom Lifshitz proposed openstack/nova master: Improve documentation of (unpin|pin)_cpus_with_siblings() https://review.opendev.org/740327
15:56:23 artom sean-k-mooney, stephenfin ^^ quick drive-by comment patch while I work on the downstream bug
16:00:59 sean-k-mooney there are docs for that :P
16:02:42 sean-k-mooney so on master if cpus & sib: should alwasy be empty/false when you have hw:cpu_thread_policy=isolate
16:02:53 sean-k-mooney we change the meaning in train
16:03:18 sean-k-mooney before it ment select host without SMT or claim both siblings if SMT is enabled on the host
16:03:32 sean-k-mooney form train on its just select a host without SMT
16:03:50 artom sean-k-mooney, *shrug* feel free to -1 :)
16:04:17 sean-k-mooney so i dont think there is a valid path to use pin_cpus_with_siblings anymore unless we are allowing the old behavior if you dont report PCPUs
16:04:26 sean-k-mooney stephenfin: are we supporting ^
16:05:04 stephenfin sean-k-mooney: https://review.opendev.org/#/c/705666/
16:05:37 sean-k-mooney hehe i even reviewd it i tought we spoke about this before
16:06:21 sean-k-mooney so we can remove those in victoria right
16:07:08 stephenfin yup
16:07:11 stephenfin just got to find the time
16:07:11 sean-k-mooney should thos be in the runway with the mixed cpus changes or just treated as seperate cleanups
16:07:16 stephenfin separate
16:07:20 stephenfin let's land the feature first
16:07:23 sean-k-mooney ok
16:24:19 openstackgerrit sean mooney proposed openstack/nova master: lookup nic feature by pci address https://review.opendev.org/739131
16:24:52 sean-k-mooney alex_xu: i need to test this on my sriov servers so ill set -w on it for now but ^ should adress your comment
16:25:07 sean-k-mooney it should be much more effiecnt then we had before
16:31:13 gmann sean-k-mooney: there are more deps need to bump lower-constraint, I am getting nova job green first and then same we can apply on os-vif- https://review.opendev.org/#/c/738126/8
16:33:04 gmann i think i need to create Focal env and test in one shot. let's see if there are more failure
16:44:17 sean-k-mooney i have folal devn envs but i have run devstack so they are not clean anymore
16:44:47 sean-k-mooney i can proabnly test the pathcs locally if you need me too i just need to boot a clean vm but not today
16:45:41 sean-k-mooney gmann: i assme infra have no plance to add the deps to the base images so we either need to fix them in bindeps in each project or bump the minor verions
16:46:35 sean-k-mooney gmann: looks like https://review.opendev.org/#/c/738130/ passed for os-vif
16:46:52 gmann sean-k-mooney: cool.
16:47:18 gmann infra side, not at lest min version as that is project side things
16:48:14 sean-k-mooney i was more thinking of the libffi-dev issue
16:48:15 sean-k-mooney but ok
16:48:49 gmann let me update cmt msg and remove depends-on as 738130 has lower-constraint update also
16:49:29 sean-k-mooney sure. the kuryr-kubernetes-tempest job is not an os-vif job by the way so we will need to fix kuryr-kubernetes before that will pass
16:49:58 gmann yeah
16:50:07 sean-k-mooney its uses os-vif form git however so we run it to make sure we dont break them
16:51:46 sean-k-mooney which reminds me with the upcoming change in devstack to use ovn by default - os-vif-ovs will be come an ovn job but we need to make sure that - os-vif-ovs-iptables keeps using ml2/ovs
16:51:58 sean-k-mooney thats unrealted but it just occured to me
16:53:16 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Re-enable live snapshot for paused instances https://review.opendev.org/740334
16:53:17 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: Attempt to restore some sanity to snapshot https://review.opendev.org/740336
16:53:17 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Remove workaround for really old QEMU https://review.opendev.org/740335
16:53:34 gmann yeah, till now we added ovn as n-v job on devstack and check its stability but yes we need to adjust few bits for ovs specific jobs
16:54:43 gmann let me try kuryr-kubernetes migration also and test all things together.
16:55:33 sean-k-mooney given that job install k8s it might be more work then other to get working

Earlier   Later