Earlier  
Posted Nick Remark
#openstack-nova - 2019-07-12
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: Resolve a TODO and update another https://review.opendev.org/666631
15:40:19 openstackgerrit Stephen Finucane proposed openstack/nova master: vif: Remove dead minimum libvirt checks https://review.opendev.org/666814
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 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:44:01 openstack Launchpad bug 1829896 in OpenStack Compute (nova) "libvirt: "Instance failed to shutdown in 60 seconds." in the gate" [Undecided,New]
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: Add functional test for resize crash compute restart revert https://review.opendev.org/670393
17:42:54 openstackgerrit Artom Lifshitz proposed openstack/nova master: Pass migration to finish_revert_migration() https://review.opendev.org/668631
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 validating instance and node https://review.opendev.org/656028
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: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
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
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: Add functional test for resize crash compute restart revert https://review.opendev.org/670393
14:28:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: Pass migration to finish_revert_migration() https://review.opendev.org/668631
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: Support report multi CPU model traits https://review.opendev.org/670300
01:27:05 openstackgerrit ya.wang proposed openstack/nova master: Add release note https://review.opendev.org/670441
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.
03:19:08 openstack Launchpad bug 1836141 in OpenStack Compute (nova) "vm resize failed due to the remains left by failed actions" [Undecided,New]
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
10:05:54 cdent it should be per test
10:06:05 sean-k-mooney cdent: well yes and no
10:06:12 sean-k-mooney it is in subunit parser
10:06:20 sean-k-mooney but the subunit protocal
10:06:33 cdent so finding and turning off the deprecation warnings (or simply fixing them) is probably the easiest thing
10:06:35 sean-k-mooney allows large packets to be fragmented
10:06:47 sean-k-mooney i think the issue is we are not fragmenting the stream
10:07:04 sean-k-mooney cdent: ya that is how we are fixing it upstream
10:07:18 sean-k-mooney but downstream that is more of a challange
10:07:43 sean-k-mooney it either requires backporting things that may not be backported upstream or downstream only patches
10:13:35 sean-k-mooney cdent: i was wondering if there was a way to redirect all deperecation warnings to a seperate file so that it does not end up in the test output
10:14:05 sean-k-mooney we should be able to install a python logging filter i think but i havent tried
10:15:15 cdent yeah, that does appear to be the case
10:18:43 sean-k-mooney the trick would be using a generic enough regex to match deprecations without redirecting real errors. we would also want the gate jobs to copy the deprecation file so that we can actully track how big it is and squash them
10:22:57 sean-k-mooney it lookse like we already filter a bunch of waring here https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L815-L877
10:24:02 sean-k-mooney cdent: actully https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L820 should be makeing deprecation warning print once
10:24:10 sean-k-mooney but maybe that is per test?

Earlier   Later