Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-08-06
20:28:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Cache host to cell mapping in HostManager https://review.opendev.org/663388
20:28:13 melwitt KeithMnemonic: understood. I was trying to be funny, like what else do you need except cirros :P sounds good, thanks. I'll propose a patch for the console docs
20:28:26 KeithMnemonic ah, sorry was too serious today
20:28:34 KeithMnemonic lol
20:29:37 melwitt nah, I'm sure it's me, I'm not good at jokes
20:38:39 openstackgerrit Matt Riedemann proposed openstack/nova master: Update comments in HostManager._get_instance_info https://review.opendev.org/674929
21:02:37 efried occupational hazard
21:33:27 sean-k-mooney i assume no one here has tried using devstack on centos7 recently?
21:33:52 mriedem you said you were done for the day
21:34:25 sean-k-mooney ya i just checking by devstack install and it failed so im shutting down my sriov hosts
21:34:53 sean-k-mooney but i havent use those in 2 months and i think in that time everything broke
21:35:19 sean-k-mooney so i was hoping it worked so i can do testing tommorw but i think ill be installing ubuntu 18.04 instead tomorrow
21:35:40 sean-k-mooney anyway night o/
21:57:00 abaindur Can a VM be attached to an SR-IOV port as well as an OVS port on different networks?
22:01:20 openstackgerrit Merged openstack/nova master: Pass migration to finish_revert_migration() https://review.opendev.org/668631
22:01:28 openstack bug 1833581 in OpenStack Compute (nova) "instance stuck in BUILD state if nova-compute is restarted" [Low,In progress] https://launchpad.net/bugs/1833581 - Assigned to Balazs Gibizer (balazs-gibizer)
22:01:28 openstackgerrit Merged openstack/nova master: Functional reproduce for bug 1833581 https://review.opendev.org/667913
22:15:27 efried mriedem: Do we have a bug for healing instances with VGPUs?
22:15:36 efried Cause I'm 90% sure that's way busted.
22:15:46 efried healing instance allocations*
22:18:40 mriedem heal_allocations doesn't handle nested allocations
22:19:01 mriedem well, not VGPU anyway, i guess the qos ports stuff would be on nested providers
22:19:05 efried exactly
22:22:51 mriedem efried: why is that coming up?
22:23:11 mriedem you were just like, "i want to work on some more terrible stuff"
22:25:52 efried mriedem: I was reviewing stephenfin's patch to consolidate ResourceRequest building
22:26:11 efried He removed a TODO from a spot where we merge all the allocations into a single dict
22:26:20 efried The only place we use that is in heal allocations
22:26:51 efried and looking at heal allocations it is clear that the TODO still very much applies
22:27:07 efried I was considering opening a bug and/or writing a func test to prove how broken it is.
22:27:19 efried so I wanted to know if there was an existing bug
22:27:22 efried but anyway, not today.
22:37:59 openstackgerrit Matt Riedemann proposed openstack/nova master: docs: update 2.23 REST API version history https://review.opendev.org/674948
22:42:42 openstackgerrit Matt Riedemann proposed openstack/nova master: api-ref: document valid GET /os-migrations?migration_type values https://review.opendev.org/674949
22:45:49 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: docs: clarify nova migration-list --host option https://review.opendev.org/674950
22:46:48 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: docs: clarify nova migration-list --host option https://review.opendev.org/674950
23:51:51 openstackgerrit Huachang Wang proposed openstack/nova-specs master: Use PCPU and VCPU in one instance https://review.opendev.org/668656
#openstack-nova - 2019-08-07
02:09:12 openstackgerrit Li Liu proposed openstack/nova master: WIP: Create and bind Cyborg ARQs. https://review.opendev.org/631244
02:11:37 openstackgerrit Eric Fried proposed openstack/nova master: trivial: Remove unused '_instance_to_allocations_dict' function https://review.opendev.org/674893
02:56:06 fungi i directly reenqueued 674821,1 back into the gate after that last failure, so no need to recheck it
02:56:46 fungi i'd promote it but the only thing ahead is a cinder change which is eta 28 minutes to merge anyway
02:56:58 openstackgerrit ZhengMa proposed openstack/nova master: Check container_format on instance creation https://review.opendev.org/673407
02:59:29 openstackgerrit ZhengMa proposed openstack/nova master: Check container_format on instance creation https://review.opendev.org/673407
03:01:15 fungi the failure was a TimeoutException in test_walk_versions on py36, for the record
03:12:20 openstackgerrit Li Liu proposed openstack/nova master: WIP: Create and bind Cyborg ARQs. https://review.opendev.org/631244
03:36:22 openstackgerrit YAMAMOTO Takashi proposed openstack/nova master: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/675021
03:53:36 openstackgerrit Li Liu proposed openstack/nova master: WIP: Create and bind Cyborg ARQs. https://review.opendev.org/631244
04:11:26 openstackgerrit Li Liu proposed openstack/nova master: WIP: Added Cyborg External Events https://review.opendev.org/675022
04:14:41 openstackgerrit Brin Zhang proposed openstack/python-novaclient master: Microversion 2.76: Add user_id/project_id to migration-list API https://review.opendev.org/675023
04:15:35 openstackgerrit Alex Xu proposed openstack/nova-specs master: Update the virtual persistent memory spec based on discussion https://review.opendev.org/675024
04:17:20 openstackgerrit Jing Zhang proposed openstack/nova master: Remove the current host from cold migration target host list https://review.opendev.org/675025
05:20:53 openstackgerrit Merged openstack/nova master: Replace non-nova server fault message https://review.opendev.org/674821
05:40:27 openstackgerrit ZhengMa proposed openstack/nova master: Check container_format on instance creation https://review.opendev.org/673407
07:01:13 openstackgerrit Merged openstack/nova master: trivial: Remove unused '_instance_to_allocations_dict' function https://review.opendev.org/674893
07:14:33 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix non-existent method of Mock https://review.opendev.org/675041
07:55:50 brinzhang takashi: Why abandon this patch https://review.opendev.org/#/c/450119?
07:57:35 brinzhang takashi: I have a bp, it's codes need to add parameters to support for migration-list command, but there isnot the ``openstack server migration-list`` CLI
08:01:53 brinzhang takashin: :)
08:02:13 takashin brinzhang: Hi
08:02:55 brinzhang https://review.opendev.org/#/c/450119 this patch, looks good, but why are you abandon?
08:03:38 takashin brinzhang: There was a subsequnt BP. It was the function to abort cold migration.
08:04:33 takashin brinzhang: But in community discussin, it is not necessary to abort cold migration. So https://review.opendev.org/#/c/450119 was abondaned.
08:05:39 brinzhang takashin: Now in the openstackclient, there is not has the 'opesntack server migration-list', I was reviewed this patch, found you added it
08:06:19 brinzhang takashin: so I asked for that, if so, I will push a new patch for the migration-list
08:06:41 takashin brinzhang: You can take over it.
08:08:38 brinzhang takashin: cool, I will have a try, and remove the cold migration deal, and add my requirement.
08:23:18 openstackgerrit Lee Yarwood proposed openstack/nova master: Use os-brick locking for volume attach and detach https://review.opendev.org/614190
09:22:18 openstackgerrit Chris Dent proposed openstack/nova master: Add a document that describes profiling eventlet services https://review.opendev.org/674865
09:28:09 bhagyashris stephenfin: Hi , Good morning :)
09:29:19 stephenfin bhagyashris: Evening o/
09:29:49 bhagyashris stephenfin: I saw you have uploaded patches including the upgrade changes, But I found few issues after applying all the patches. #link: http://paste.openstack.org/show/755606/
09:30:24 bhagyashris stephenfin: noted issues in above #link path
09:34:47 bhagyashris stephenfin: As per discussion and mentioned in spec we are not going to support both the PCPU and VCPU in single boot request so IMO we will need to add check some where to reject such request with proper error message to user.
09:34:56 stephenfin bhagyashris: Yeah, I should have a WIP flag on that. There's a few corner cases I still need to cover including that one
09:35:54 stephenfin You're correct about the pinning though. I guess that's because we need to set the 'cpu_policy' field on the 'InstanceNUMATopology' object
09:36:59 bhagyashris stephenfin:yeah
09:37:50 bhagyashris stephenfin: And also I would like to know about are we going tp supporting new syntax of flavor extra specs like resources:PCPU and resources:VCPU in Train?
09:38:05 bhagyashris S/tp/to
09:39:18 stephenfin That would require changes to the InstanceNUMATopology object, but like we discussed previously, they don't need to be done yet
09:40:42 bhagyashris stephenfin: if yes then in that case we will also need to take care the first issue that I have mentioned above ... gust xml will give wrong result if user will create the instance using flavor extra specs resources:PCPU=1
09:41:46 stephenfin Yup. Indeed. That needs to be done regardless.
09:42:09 bhagyashris stephenfin: OK
09:42:14 bhagyashris stephenfin: and one more thing I would like to ask about the translation of flavor extra specs and image meta data .. are you going to use my change https://review.opendev.org/#/c/672223/1 or you are going to continue on your change?
09:44:11 stephenfin I think I'm going to keep going with https://review.opendev.org/#/c/671801/. I don't like the idea of rewriting the flavor behind the users back and this change doesn't require that. I might change my mind again, but I'll go with this for now
09:45:20 bhagyashris stephenfin: OK No problem.
09:45:52 stephenfin bhagyashris: Thanks for testing this stuff, btw. Really good to have as many eyes on it as possible, as we're seeing :)
09:47:14 bhagyashris stephenfin: If you are OK? then I will take up adding the functional test cases and will fix the existing test that you have added for https://review.opendev.org/#/c/674895/2
09:48:07 stephenfin bhagyashris: Yes, please. I kept the Author the same on that. I'm going to be working on the previous two patches and adding support for reporting the 'HW_CPU_HYPERTHREADING' trait today
09:48:58 bhagyashris stephenfin: OK then I will take up this task and will upload the patch soon
10:01:33 bhagyashris stephenfin: Thank you :)
11:40:47 sean-k-mooney stephenfin: gibi alex_xu https://review.opendev.org/#/c/621646/23 has been around for a while and it hasnt gotten any core review. i have review this several times at this point, would ye be able to take a look?
11:48:06 gibi sean-k-mooney: I opened it now on a tab but I cannot promies to reach there today
11:48:42 sean-k-mooney gibi: no worries i was pingged to review it yesterday so i did it this morning
11:49:32 sean-k-mooney i just wanted to flag it to some people since it was opened back in december and they have been rebasing it since
12:03:59 alex_xu sean-k-mooney: I added it to my review list, will reach it when I do review work this week
12:05:04 sean-k-mooney alex_xu: cool thanks
12:07:58 stephenfin sean-k-mooney: sure
12:10:59 sean-k-mooney speaking of reveiw where are we at with https://review.opendev.org/#/c/668656/7/specs/train/approved/use-pcpu-vcpu-in-one-instance.rst
12:11:19 sean-k-mooney oh wrong version
12:11:53 sean-k-mooney all the discusstion seams to be on 7 but the patch is on version 11

Earlier   Later