Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-07-08
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?
07:55:48 openstackgerrit Jie Li proposed openstack/nova-specs master: add volume backed server rebuild spec https://review.opendev.org/739349
07:55:55 openstackgerrit Huaqiang Wang proposed openstack/nova master: doc: correct the link to user/flavor.rst https://review.opendev.org/740176
07:57:08 gibi huaqiang: is this in the middle of the series?
07:57:55 huaqiang gibi: do you mean https://review.opendev.org/739349
07:58:23 huaqiang for https://review.opendev.org/739349, yes it is, it belongs to bp/mixed-instance
07:58:25 gibi I mean https://review.opendev.org/#/c/728480/
07:58:58 gibi I can review https://review.opendev.org/#/c/728480/ but I reviewd the ancestors of that patch in the series already
07:59:04 huaqiang my mistake
07:59:24 huaqiang https://review.opendev.org/#/c/728480/ belongs to bp/mixed-instances
07:59:43 huaqiang Stephen suggests to have your review,
08:00:22 huaqiang let me check
08:07:36 gibi huaqiang: sure I will try to look deeper into the series. I have feedback in https://review.opendev.org/#/c/714701
08:07:47 gibi and in https://review.opendev.org/#/c/714703
08:08:08 gibi and also alex_xu has feedback in that series
08:10:38 brinzhang0 gibi: do you have time to review the nova-cyborg patch? https://review.opendev.org/#/c/716186/13
08:11:02 brinzhang0 gibi: and the cyborg evacuate support patch https://review.opendev.org/#/c/715326/15
08:11:59 brinzhang0 the cyborg shelve/unshelve support depends on these patch, and these pataches are already to review, thanks

Earlier   Later