Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-08-06
19:45:19 KeithMnemonic it was pretty simple to fix once i got looking in the right place
19:45:31 melwitt oh, I see
19:48:24 melwitt well, I think I'm in support of the nova rocky backport given there seems to be no other way to have esx hypervisors consoles work out-of-the-box. though that also comes with caveat of the fact that I'm guessing for rocky noVNC v1.1.0 isn't what's installed out-of-the-box
19:48:50 efried mriedem: you tell me, do we usually require a blueprint for novaclient parity?
19:48:53 melwitt so people would have to know that they need v1.1.0 and install it
19:51:01 mriedem efried: most blueprints in novaclient these days are tied to nova blueprints for microversion parity,
19:51:17 mriedem i think a standalone feature in novaclient should probably use a blueprint for tracking yeah,
19:51:33 mriedem but don't necessarily think it should fall under spec freeze or anything since it's not on the same release schedule
19:51:51 efried mriedem: specless?
19:52:13 efried Seems like you could fit the necessary into a bp description.
19:52:13 mriedem yes, we don't need specs for novaclient blueprints :)
19:52:14 mriedem https://blueprints.launchpad.net/python-novaclient/+spec/more-migration-list-filters
19:52:17 mriedem i did
19:52:42 mriedem if people want i'll just throw it under open discussion for thursday's meeting
19:52:48 efried wfm
19:52:59 efried should at least do that as a formality.
19:53:54 mriedem ack
19:54:02 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for node.list https://review.opendev.org/656027
19:54:02 openstackgerrit Dustin Cowles proposed openstack/nova master: Introduces SDK to IronicDriver and uses for node.get https://review.opendev.org/642899
19:54:03 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for setting instance id https://review.opendev.org/659690
19:54:03 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for validating instance and node https://review.opendev.org/656028
19:54:04 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/670213
19:54:04 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/659691
19:55:03 KeithMnemonic melwitt: that would be great to have it ported, if that does not fly, i can share the patch to noVNC 1.0 if we need to document or something
19:58:23 melwitt KeithMnemonic: ack. feels like either way, we need to add a note on our consoles docs https://docs.openstack.org/nova/latest/admin/remote-console-access.html that calls out the issue for esx hypervisors and instructions on what to do. I would need your help to know what to write there as far as explaining what/how it is broken for esx
20:03:23 KeithMnemonic ok so lets try for the backport and then the note just mentions needing 1.1? and if that does not work we add the steps to fix with 1.0? does that seem reasonable
20:05:42 melwitt KeithMnemonic: right. my thought: I'd like to have the doc have a note about esx hypervisors and explain what is broken in < v1.1.0 and then instruction will be "install v1.1.0" if the backport succeeds, else "apply this patch" and maybe we could host it either our repo or one of the ops-oriented repos and link to it from the doc
20:06:38 KeithMnemonic ok so here was the issue, https://github.com/novnc/websockify/issues/402 i originally thought it was the websockify piece since it was just disconnecing
20:06:45 KeithMnemonic i can update the issue or try and move it
20:07:20 KeithMnemonic melwitt: this is the issue here "rfb.js:685 Failed while connected: Unsupported encoding (encoding: -260)"
20:07:34 KeithMnemonic so the older 1.0 does not handle encoding "-260" correctly
20:07:36 KeithMnemonic 1.1 does
20:07:38 melwitt ah, perfect, yeah we could just link to that issue for explaining the problem
20:08:05 KeithMnemonic should i move it under novnc? or close and reopen a new one ?
20:08:55 KeithMnemonic i dont think i can move it
20:09:06 melwitt oh, it's under the wrong component (I didn't understand that :P). yeah, I dunno, if it's possible to move it then that makes sense to me
20:09:07 KeithMnemonic so i could close and reopen under novnc/novnc
20:09:19 KeithMnemonic simple enough to close and reopen
20:10:07 melwitt yeah, might be worth doing for the sake of avoiding confusion
20:10:46 openstackgerrit Merged openstack/nova master: Add functional test for resize crash compute restart revert https://review.opendev.org/670393
20:12:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Cache host to cell mapping in HostManager https://review.opendev.org/663388
20:12:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Convert HostMapping.cells to a dict https://review.opendev.org/663387
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

Earlier   Later