| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-07-08 | |||
| 14:14:22 | huaqiang | It's better to have them | |
| 14:17:48 | sean-k-mooney | i found this before https://github.com/intel/sst-bf-openstack-setup-automation which tries to automate it but it was not entirely trival to figure out what it was doing with our painstakingly reverse engienriing it | |
| 14:18:44 | sean-k-mooney | huaqiang: its currently the best reference i coudl find however | |
| 14:19:05 | sean-k-mooney | it also only really covers SST-BF | |
| 14:22:19 | alex_xu | gibi: tony su is on the same team with me | |
| 14:22:38 | huaqiang | sean-k-mooney, OVS is a good workload for BF | |
| 14:23:37 | sean-k-mooney | huaqiang: ovs-dpdk even more so because you can trivialy pin the PMDs | |
| 14:24:12 | sean-k-mooney | kernel ovs is tricker since some of the processing is also done in the vhost_threads wo ideally you want toes to also be on the high priority cores | |
| 14:24:48 | sean-k-mooney | thos are kernel thready however that are spawned in responce to qemu so im not sure its easy to do that | |
| 14:25:09 | alex_xu | gibi: oh...just saw huaqiang already help on that | |
| 14:25:12 | sean-k-mooney | for ovs-dpdk however all the packet procesing is done in the pmd so its easy to configure | |
| 14:28:29 | huaqiang | we can use mixed instance and dedicated cores for PMD threads, and shared cores for other threads, right | |
| 14:28:31 | huaqiang | ? | |
| 14:29:19 | gibi | alex_xu: yes, thanks | |
| 14:30:23 | huaqiang | Hie alex_xu, stephenfin, the mixed instance pathces are ready for next review, please review at your convenient time! | |
| 14:30:50 | alex_xu | huaqiang: got it! | |
| 14:31:01 | huaqiang | alex_xu thanks! | |
| 14:31:19 | alex_xu | np | |
| 14:40:14 | sean-k-mooney | huaqiang: partly yes but i dont belive the core affitiy we apply to the vm cpus applyse to the kernel thread spwaned by the vhost kernel module | |
| 14:40:46 | sean-k-mooney | the are not forked or spwaned from the process | |
| 14:41:16 | sean-k-mooney | so i would not assume any cgroup/taskset limits applied to the eumlator thread or to the vcpus would have any affect on where they run | |
| 14:42:25 | sean-k-mooney | you could restict the ovs-vsitchd to a set of thread too but again i thikn that woudl only apply to the user spacee process not the kernel threads. it may be its not clear that it would | |
| 14:42:34 | sean-k-mooney | anyway i think this is out of scope of nova | |
| 14:45:22 | huaqiang | can you limit the kernel threads to some CPUs? | |
| 14:45:37 | huaqiang | I know the interrput thread could be pinned | |
| 14:46:40 | huaqiang | any know nova don't have the capability for kernel pinning | |
| 14:46:57 | huaqiang | s/any know/anyway/ | |
| 14:58:32 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184 | |
| 15:00:05 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: Switch legacy Zuul jobs to native Zuul v3 jobs https://review.opendev.org/725163 | |
| 15:30:21 | openstackgerrit | Merged openstack/nova stable/stein: libvirt: avoid cpu check at s390x arch https://review.opendev.org/701526 | |
| 15:30:31 | openstackgerrit | Merged openstack/nova stable/stein: Reserve DISK_GB resource for the image cache https://review.opendev.org/732529 | |
| 15:30:38 | openstackgerrit | Merged openstack/nova stable/stein: Add admin doc information about image cache resource accounting https://review.opendev.org/732542 | |
| 15:30:46 | openstackgerrit | Merged openstack/nova stable/train: Fix cherry-pick check for merge patch https://review.opendev.org/739177 | |
| 15:42:25 | openstackgerrit | Elod Illes proposed openstack/nova stable/stein: Check cherry-pick hashes in pep8 tox target https://review.opendev.org/737922 | |
| 16:18:01 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Document nova in tree virt drivers https://review.opendev.org/740061 | |
| 16:22:11 | openstackgerrit | sean mooney proposed openstack/nova stable/train: add [libvirt]/max_queues config option https://review.opendev.org/740064 | |
| 16:40:06 | sean-k-mooney | gmann: is there a patch to adress the libffi-dev dep for https://review.opendev.org/#/c/738130/1 | |
| 16:41:20 | sean-k-mooney | i could add it a bindeps file for os-vif adn list it i guess but this is needed for any pyton lib that uses the forign function interface lib | |
| 16:41:30 | sean-k-mooney | we dont use it directly its transitive | |
| 16:44:36 | elod | sean-k-mooney: we've just seen the same issue and I think if you change the cffi in lower-constraints.txt to a newer one then that failure won't come | |
| 16:45:26 | elod | sean-k-mooney: since the problem is that cffi is not there in wheels repo | |
| 16:45:28 | sean-k-mooney | i doutb it this is becasue we dont have the cpackage so when we compile it fails to find the header | |
| 16:45:36 | elod | at least 1.11.5 | |
| 16:45:39 | gmann | sean-k-mooney elod it should be from bindep but i have not tried the cffi new version | |
| 16:45:40 | sean-k-mooney | changing a pip constratint shoudl not alter that | |
| 16:46:07 | elod | sean-k-mooney: that's why only the lower-constraints job fails and any other passes | |
| 16:46:09 | sean-k-mooney | have they started packaging the c header files for cffi? | |
| 16:46:53 | sean-k-mooney | hum ok | |
| 16:47:06 | gmann | cffi==1.11.5 is current now. | |
| 16:47:15 | sean-k-mooney | well we can raise it i guess but really we shoudl just be installing the header file form the distro | |
| 16:47:52 | gmann | ok, let me update the patch and try | |
| 16:48:24 | elod | cffi=1.14.0 is installed without any problem, see https://zuul.opendev.org/t/openstack/build/b8c5b0f682284567949ba9a0b598f61e/log/tox/py38-1.log#63 | |
| 16:48:27 | sean-k-mooney | well we dont have bindeps in os-vif currently are you going to raise the lower-constiat or add a bindep.txt | |
| 16:48:43 | sean-k-mooney | we shoudl add bindep.txt but there are other things we shoudl do in general fo that. | |
| 16:50:33 | elod | of course, adding bindep.txt with all the relevant binary dependency is a good thing :) | |
| 16:51:03 | gmann | yeah bindeps with libffi-dev [platform:dpkg] | |
| 16:51:46 | elod | just wanted to let you know why is the weird behavior (failing with lower-constraints job but not at the other jobs) | |
| 16:51:49 | elod | :) | |
| 16:52:06 | sean-k-mooney | devstafck normally instlls that for us and it used to be installed in the base infra images | |
| 16:52:14 | sean-k-mooney | so we never needed it until now | |
| 16:52:24 | sean-k-mooney | even with older cffi versions | |
| 16:52:38 | sean-k-mooney | that why we are now seeing it | |
| 16:52:57 | sean-k-mooney | gmann: if you start with adding a bindeps file ill propose followups to add our other deps later | |
| 16:53:11 | sean-k-mooney | e.g. openvswith for our functional tests | |
| 17:00:55 | openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: DNM: testing os-vif gate on ubuntu focal https://review.opendev.org/738130 | |
| 17:01:41 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: migrate testing to ubuntu focal https://review.opendev.org/738126 | |
| 17:12:13 | sean-k-mooney | gmann: ya that looks sane. if it passes feel free to drop the DNM the depend on will be enough to hold it untill we can merge it | |
| 17:13:30 | gmann | sean-k-mooney sure | |
| 17:16:43 | sean-k-mooney | i do have a bindeps change here https://review.opendev.org/#/c/728037/ | |
| 17:16:57 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184 | |
| 17:16:58 | sean-k-mooney | but i can rebase that on top of yours | |
| 17:18:39 | gmann | sean-k-mooney either is fine or once it pass then we can add in 728037 also. | |
| 17:18:52 | sean-k-mooney | sure both works | |
| 17:19:07 | sean-k-mooney | do you know what the plannned eta is for the switch to focal | |
| 17:19:09 | gmann | which can merge independent of focaltion work | |
| 17:19:24 | gmann | focal-migration | |
| 17:19:32 | sean-k-mooney | yep | |
| 17:19:42 | gmann | sean-k-mooney July end for now but i think it might delay. | |
| 17:20:01 | sean-k-mooney | as i said the dep used to be in the nodepool images and its install by devstack so that is why we are just seing this now | |
| 17:20:40 | gmann | yeah, there are lot of updates in images, so not sure if that is one of that. infra team can confirm | |
| 17:32:40 | openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: DNM: testing os-vif gate on ubuntu focal https://review.opendev.org/738130 | |
| 17:33:11 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: Switch legacy Zuul jobs to native Zuul v3 jobs https://review.opendev.org/725163 | |
| 22:39:33 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: Testing system scope in tempest https://review.opendev.org/740124 | |
| #openstack-nova - 2020-07-09 | |||
| 02:39:34 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029 | |
| 02:50:37 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | |
| 02:50:59 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | |
| 03:35:19 | openstackgerrit | XiaolinZhao proposed openstack/nova master: Add UEFI support for architecture MIPS https://review.opendev.org/740151 | |
| 04:34:06 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029 | |
| 07:05:05 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | |
| 07:09:43 | gibi | good morning nova | |
| 07:11:09 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | |
| 07:24:31 | tony_su | gibi: sean-k-mooney: I am still taking the provider-config-file job and fully supporting its merging into upstream. | |
| 07:25:58 | gibi | tony_su: cool. So I will move the series to a runway slot and we will try to focus on it in the next two weeks | |
| 07:26:09 | gibi | are you available in the next to week to iterate on the series based on our feedback? | |
| 07:27:39 | openstackgerrit | Elancheran S proposed openstack/nova master: Removes the delta file once image is extracted https://review.opendev.org/734421 | |
| 07:27:50 | tony_su | yes, definitely. | |
| 07:28:31 | tony_su | gibi: yes, definitely. | |
| 07:28:38 | gibi | tony_su: cool | |
| 07:28:39 | gibi | thanks | |
| 07:49:43 | huaqiang | gibi: hi can you review https://review.opendev.org/728480? | |