| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-06 | |||
| 20:13:11 | melwitt | the noVNC people were responsive back when I proposed this nak'd pull request :P https://github.com/novnc/noVNC/pull/1220 | |
| 20:14:25 | melwitt | so there's a chance they'll add some comments on the issue, that would be cool | |
| 20:14:38 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Always set a global_request_id in RequestContext https://review.opendev.org/674138 | |
| 20:14:39 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Correct global_request_id sent to Placement https://review.opendev.org/674129 | |
| 20:14:52 | efried | what a waste of a ci run | |
| 20:23:24 | KeithMnemonic | melwitt: here you go https://github.com/novnc/noVNC/issues/1273 | |
| 20:24:38 | melwitt | KeithMnemonic: woot thanks | |
| 20:24:53 | KeithMnemonic | that describes the issue and fix if needed | |
| 20:25:05 | KeithMnemonic | it was not exahaustively tested ;-) | |
| 20:25:31 | KeithMnemonic | but i could get to cirros and type commands! | |
| 20:25:44 | melwitt | what else do ya need? | |
| 20:26:47 | KeithMnemonic | melwitt: I did not try anything with X for example. I am all set .I will watch the backport progress. If you need anything else from me please let me know. Thanks again | |
| 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 | |