Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-07-08
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
08:15:32 huaqiang gibi: sorry about that, I missed your comments even I go through some of my abandoned changes.
08:15:49 huaqiang gibi: can you have a look again :)
08:16:09 huaqiang gibi: get your message, thanks
08:20:56 openstackgerrit Wenping Song proposed openstack/nova-specs master: Add no user token when get Cyborg client https://review.opendev.org/740184
08:22:42 openstackgerrit Wenping Song proposed openstack/nova-specs master: Add no user token when get Cyborg client https://review.opendev.org/740184
08:29:32 gibi brinzhang0: if the cyborg patches are ready then please add them to the runway queue. At the momement I will focus on the 3 series that is currently in the 3 slots.
08:29:55 gibi huaqiang: ack I will look
08:30:10 brinzhang0 gibi: ok, I will add these to the runway queue
08:31:25 gibi brinzhang0: thanks
08:31:37 brinzhang0 gibi: np
08:32:01 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Fix os_CODENAME detection and repo refresh during ceph tests https://review.opendev.org/739608
08:32:02 gibi brinzhang0: also summer vacation time will soon arrives. I will take 3 weeks of during july - august
08:32:13 gibi s/of/off/
08:32:34 gibi I guess others from EU will do similar things

Earlier   Later