Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-07-12
13:16:38 mriedem and we don't test restarting crashed computes in tempest
13:16:49 artom Oh right, it's in _init_host, and we never have instances stuck in migrating in the gate
13:16:51 mriedem that's kind of hard to do in a single node parallel test worker environment
13:17:05 mriedem this is why i wanted to write that functional test patch
13:17:14 mriedem b/c unit tests aren't sufficient for that complicated _init_instance mess
13:17:59 artom Yep, I think to be safe I'll rebase on top of your patch, once the patch on the bottom merges
13:19:14 mriedem artom: note that you do'nt need my functional test patch for backports, just fyi for when the time comes
13:19:25 mriedem although - you're not backporting the interface change anyway, that's why we did it
13:19:26 mriedem so nvm
13:19:34 openstack Launchpad bug 1836105 in OpenStack Compute (nova) "Instance does not start - Error during following call to agent: ovs-vsctl" [Undecided,New]
13:19:34 mriedem sean-k-mooney: if anyone can debug this it's probably you https://bugs.launchpad.net/nova/+bug/1836105
13:19:39 mriedem *debug/triage
13:20:09 sean-k-mooney ill take a look.
13:20:21 sean-k-mooney i have been putting off fixing some dpdk specific bugs
13:20:28 sean-k-mooney so i can proably add it to my list
13:20:57 sean-k-mooney i have 2 small ones to fix already so if i start them soon i can try and repoduce that too
13:21:04 sean-k-mooney for now ill triage it
13:23:59 sean-k-mooney the first thing that jumps out at me is there vhost user sockets are in a weired place under /run/libvirt-vhost-user/
13:26:17 sean-k-mooney my guess is that ovs is not currently listening on the unix soecket since the neutorn l2 agent log keeps saying ovs is dead that is proably the issue
13:47:07 mriedem efried: fyi https://review.opendev.org/670345 novaclient 14.2.0 release request
13:51:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional test for resize crash compute restart revert https://review.opendev.org/670393
13:51:13 mriedem artom: updated ^
13:51:20 mriedem bauzas: ^ is also another scenario where we orphan allocations
13:52:18 sean-k-mooney mriedem: im pretty sure the dpdk thing is ovs was not running, i have marked it as incomplete for nova and added the openstack charms compnent as it think its likely a deployment tool issue
13:52:38 sean-k-mooney if they come back with more logs ill take a look again
14:04:40 efried stephenfin: yes sir, I'm on it.
14:13:24 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Convert nova-next to a zuul v3 job https://review.opendev.org/670196
15:17:12 efried stephenfin: I think there's an error in https://review.opendev.org/#/c/666630/
15:18:37 stephenfin efried: There sure is. Thanks for the in-depth review. I'll respin shortly
15:22:41 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Deprecate 'devname' field of '[pci] passthrough_whitelist' https://review.opendev.org/670585
15:22:50 stephenfin sean-k-mooney: ^ you're probably interested in that
15:23:54 stephenfin I'm thinking we might want to backport it too, for the same reasons we backported the '[workarounds] enable_numa_live_migration' config option (we don't want people opting into broken behavior)
15:40:18 openstackgerrit Stephen Finucane proposed openstack/nova master: vif: Stop using getattr for VIF lookups https://review.opendev.org/666630
15:40:19 openstackgerrit Stephen Finucane proposed openstack/nova master: vif: Remove dead minimum libvirt checks https://review.opendev.org/666814
15:40:19 openstackgerrit Stephen Finucane proposed openstack/nova master: vif: Resolve a TODO and update another https://review.opendev.org/666631
15:40:20 openstackgerrit Stephen Finucane proposed openstack/nova master: vif: Remove 'plug_vhostuser', 'unplug_vhostuser' https://review.opendev.org/670588
15:42:37 openstackgerrit Stephen Finucane proposed openstack/nova master: Integrate 'pre-commit' https://review.opendev.org/665518
15:44:01 openstack Launchpad bug 1829896 in OpenStack Compute (nova) "libvirt: "Instance failed to shutdown in 60 seconds." in the gate" [Undecided,New]
15:44:01 mriedem if someone is looking for something to investigate/debug next week which should help speed up tempest run times for anything that requires stopping a server https://bugs.launchpad.net/nova/+bug/1829896 (like shelve, stop/start, etc)
15:46:26 mriedem "While performing stop, rescue and shelve, rebuild operations"
15:46:36 mriedem yikes at 60 seconds a pop that's a lot of wasted time in the gate
15:53:20 mriedem here is a devstack change to work around that https://review.opendev.org/670591
15:53:33 mriedem which is probably something we should have done in the first place
15:55:59 mriedem gmann: ^ probably of interest to you in speeding up gate times
16:05:42 openstackgerrit Artom Lifshitz proposed openstack/nova master: Device tags: don't pass pf_interface=True to get_mac_by_pci_address https://review.opendev.org/670593
16:23:19 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Convert nova-next to a zuul v3 job https://review.opendev.org/670196
16:24:01 mriedem efried: just fyi you can probably remove https://review.opendev.org/#/q/topic:bp/add-host-and-hypervisor-hostname-flag-to-create-server+(status:open+OR+status:merged) from the runway slot since the only thing not yet merged is the docs patch (but it's approved) and the osc change which is dependent on the novaclient release
16:24:23 efried mriedem: ack, thanks
16:25:12 mriedem with that i think i'm out, have fun
17:25:52 openstackgerrit Merged openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/667177
17:27:57 artom \o/
17:42:54 openstackgerrit Artom Lifshitz proposed openstack/nova master: Pass migration to finish_revert_migration() https://review.opendev.org/668631
17:42:54 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add functional test for resize crash compute restart revert https://review.opendev.org/670393
17:42:55 openstackgerrit Artom Lifshitz proposed openstack/nova master: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/664442
19:38:48 spatel sean-k-mooney: yt?
19:50:09 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Multiple API cleanup changes https://review.opendev.org/666889
20:01:11 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Multiple API cleanup changes https://review.opendev.org/666889
20:03:44 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Multiple API cleanup changes https://review.opendev.org/666889
20:20:07 openstackgerrit Artom Lifshitz proposed openstack/nova stable/stein: Revert resize: wait for events according to hybrid plug https://review.opendev.org/670645
20:22:09 openstackgerrit Artom Lifshitz proposed openstack/nova stable/rocky: Revert resize: wait for events according to hybrid plug https://review.opendev.org/670648
20:23:10 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Multiple API cleanup changes https://review.opendev.org/666889
20:47:33 openstackgerrit François Palin proposed openstack/nova master: Call volume detach even if terminate_connection fails https://review.opendev.org/669674
23:16:41 openstackgerrit Artom Lifshitz proposed openstack/nova stable/stein: Revert resize: wait for events according to hybrid plug https://review.opendev.org/670645
23:24:01 openstackgerrit Artom Lifshitz proposed openstack/nova stable/rocky: Revert resize: wait for events according to hybrid plug https://review.opendev.org/670648
#openstack-nova - 2019-07-13
00:34:11 openstackgerrit Merged openstack/nova master: Remove needs:* todo from deprecated APIs api-ref https://review.opendev.org/668234
00:44:24 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for node.list https://review.opendev.org/656027
00:44:25 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for setting instance id https://review.opendev.org/659690
00:44:25 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for validating instance and node https://review.opendev.org/656028
00:44:26 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for getting network metadata from node https://review.opendev.org/670213
00:44:26 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for add/remove instance info from node https://review.opendev.org/659691
04:43:57 openstackgerrit Merged openstack/nova master: Fix no propagation of nova context request_id https://review.opendev.org/662715
04:49:48 openstackgerrit Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851
11:41:31 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Multiple API cleanup changes https://review.opendev.org/666889
12:15:47 openstackgerrit Takashi NATSUME proposed openstack/nova stable/stein: Fix no propagation of nova context request_id https://review.opendev.org/670694
#openstack-nova - 2019-07-14
02:59:27 openstackgerrit Takashi NATSUME proposed openstack/nova stable/stein: Fix no propagation of nova context request_id https://review.opendev.org/670694
07:37:52 openstackgerrit Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851
14:28:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: Pass migration to finish_revert_migration() https://review.opendev.org/668631
14:28:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add functional test for resize crash compute restart revert https://review.opendev.org/670393
14:28:35 openstackgerrit Artom Lifshitz proposed openstack/nova master: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/664442
#openstack-nova - 2019-07-15
01:26:17 openstackgerrit Arthur Dayne proposed openstack/python-novaclient stable/rocky: Add Python 3 Train unit tests https://review.opendev.org/670749
01:27:04 openstackgerrit ya.wang proposed openstack/nova master: Add compatibility checks for CPU mode and CPU models and extra flags https://review.opendev.org/670299
01:27:05 openstackgerrit ya.wang proposed openstack/nova master: Add release note https://review.opendev.org/670441
01:27:05 openstackgerrit ya.wang proposed openstack/nova master: Support report multi CPU model traits https://review.opendev.org/670300
03:19:08 openstack Launchpad bug 1836141 in OpenStack Compute (nova) "vm resize failed due to the remains left by failed actions" [Undecided,New]
03:19:08 Spencer_Yu https://bugs.launchpad.net/nova/+bug/1836141 Some failed actions would leave the remains in resize instance, which will lead next resize to fail.
04:37:29 openstackgerrit Ghanshyam Mann proposed openstack/nova master: DRY get_flavor in flavor manage tests https://review.opendev.org/668281
06:54:30 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Proposal for a safer noVNC console with password authentication https://review.opendev.org/623120
08:23:56 alex_xu efried: bauzas need your help on the direction https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:claim_for_instance
09:38:16 sean-k-mooney stephenfin: do you know the history behind why we went form testr to ostestr to stestr? im wondering if we should consider regerting to testr or look at finding a non subunit based alternitive
09:40:55 sean-k-mooney https://blog.kortar.org/?p=370
09:41:54 sean-k-mooney found the history ^
09:42:36 stephenfin sean-k-mooney: I was pretty sure testr was subunit-based, tbh
09:42:54 sean-k-mooney it is
09:43:02 sean-k-mooney that does not mean it has the same bug
09:57:10 cdent sean-k-mooney: are you talking about the overflow bug with too much logging? or something else?
10:04:23 sean-k-mooney cdent: ya we are now hitting it downstream
10:04:51 sean-k-mooney RHEL8 ship newer versions of some libs then upper constratis uses for stien
10:05:11 sean-k-mooney cdent: so we get more deprecation warnings then upstream
10:05:19 cdent the same bug is present in testr and ostestr (which is a testr wrapper). My understanding is the issue is in testtools and/or subunit not testr or stestr
10:05:43 cdent and it really comes down to a binary data structure being too small

Earlier   Later