Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-09-18
22:39:45 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.opendev.org/576031
22:39:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.opendev.org/576031
22:41:13 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.opendev.org/576299
22:41:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.opendev.org/576299
22:42:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.opendev.org/576344
22:42:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.opendev.org/576344
22:43:30 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.opendev.org/576673
22:43:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.opendev.org/576673
22:44:50 openstackgerrit Eric Fried proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640
22:45:23 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.opendev.org/576676
22:45:31 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.opendev.org/576676
22:46:09 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.opendev.org/576689
22:46:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.opendev.org/576689
22:47:39 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.opendev.org/576709
22:47:45 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.opendev.org/576709
22:48:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.opendev.org/576712
22:48:30 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.opendev.org/576712
22:49:16 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add TODO note for mox removal https://review.opendev.org/576758
22:49:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add TODO note for mox removal https://review.opendev.org/576758
23:08:29 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Add a check for --config-drive option on nova boot https://review.opendev.org/653683
23:30:47 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: WIP: Avoid redundant initialize_connection on source post live migration https://review.opendev.org/683008
23:31:24 mriedem KeithMnemonic1: ^ is the pike backport with runtime code conflicts handled, test module conflicts are not handled (yet) but the commit message calls out what caused the conflicts so it's just a matter of resolving those test module conflicts
23:34:29 sean-k-mooney can we actully spligt the cpu resouces series form the vpmem seriese at this point
23:34:56 sean-k-mooney it was intended to prevent merge conflict but i think it has ended up hurting more then it helped
23:35:39 sean-k-mooney although at this point i guess the vpmem serice has merged?
23:36:08 sean-k-mooney looks like we are not on the first useful patch of the pcpu series
23:36:35 alex_xu sean-k-mooney: we already split
23:37:00 sean-k-mooney today?
23:37:38 sean-k-mooney what was the failure out of interest
23:38:14 alex_xu yesterday
23:39:54 alex_xu I only saw two bug, one is test_model_sync, another one is db timeout
23:40:40 sean-k-mooney sorry i though you were refering to https://review.opendev.org/#/c/672693/
23:41:07 sean-k-mooney its going to fail i nthe gate beacue devstack failed on the second node in the live migration job
23:42:38 sean-k-mooney looks like the nova compute service db entry for the subnode never got created properly
23:42:49 alex_xu yea, I thought those two in vpmems, just checking what fail in cpu resource
23:43:13 efried_pto mriedem: I'll try to be around for the meeting tomorrow; but if I'm not, you wanna run it?
23:43:32 mriedem sure
23:43:57 mriedem alex_xu: zuul had 2 restarts today
23:44:00 mriedem so that reset everything
23:44:05 efried_pto alex_xu: totally bug 1823251 again.
23:44:06 openstack bug 1823251 in OpenStack Compute (nova) "Spike in TestNovaMigrationsMySQL.test_walk_versions/test_innodb_tables failures since April 1 2019 on limestone-regionone" [High,Confirmed] https://launchpad.net/bugs/1823251
23:44:21 efried_pto I'm trying hard to nail it down, but so far no luck.
23:44:24 alex_xu same bug after reset?
23:44:42 efried_pto dunno, I just rebased and re+Wed like half an hour ago.
23:45:12 efried_pto correction: 1h ago.
23:46:37 efried_pto (only rebased to requeue since it was failing the constraints job bogusly - on the aforementioned bug)
23:46:55 efried_pto now of course it's going to have to wait twelve freaking hours to get a node again.
23:46:58 mriedem you know, it might be worth just skipping TestNovaMigrationsMySQL until all of these things are merged
23:47:04 efried_pto Agreeeeed
23:47:17 mriedem unskip before rc1
23:47:29 efried_pto that would increase our merge percentage by an order of magnitude.
23:47:41 mriedem do it
23:47:44 efried_pto you wanna propose it quick I'll fast approve it.
23:47:51 mriedem merging all of this stuff in the middle of next week is crazy
23:48:07 efried_pto yeah, I'd really rather merge it *this* week.
23:48:16 efried_pto Hell, I would have rather merged it last Thursday
23:48:23 mriedem was just about to say ^
23:48:38 mriedem i'm in the middle of something so you go ahead
23:50:02 alex_xu i can do it
23:50:06 mriedem well let me see here
23:50:38 sean-k-mooney whne the compute service joing the service group i should see a entry in the cell conductor corresponding to the compute node entry correct
23:50:52 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029
23:50:52 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: YAML file loading and schema validation https://review.opendev.org/673341
23:50:53 openstackgerrit Dustin Cowles proposed openstack/nova master: WIP: Provider Config File: Merge provider configs to provider tree https://review.opendev.org/676522
23:50:59 mriedem alex_xu: i've got it
23:51:39 alex_xu mriedem: ok, so I can help to +2
23:51:50 efried_pto alex_xu: ++
23:52:56 sean-k-mooney hum maybe not
23:55:39 sean-k-mooney that compute node does not have the db config so it woul have to go via the conductor but i guess it may not be logged or rabbit could have dropped the message
23:56:25 sean-k-mooney oh never mined i miss read the devstack message
23:56:51 sean-k-mooney it finishe registering the compute servce and then the neutron openvswitch agent did not start
23:58:36 openstackgerrit Matt Riedemann proposed openstack/nova master: Temporarily skip TestNovaMigrationsMySQL https://review.opendev.org/683009
23:59:21 sean-k-mooney wait is this complianing baour the compute agent or neutron openvswitch agent
23:59:23 sean-k-mooney https://storage.gra1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_5b9/672693/24/gate/nova-live-migration/5b959a8/logs/subnode-2/devstacklog.txt.gz
23:59:42 sean-k-mooney /baour/about/
23:59:51 mriedem alex_xu: efried_pto: there you go https://review.opendev.org/683009
23:59:58 alex_xu mriedem: thanks
#openstack-nova - 2019-09-19
00:02:50 sean-k-mooney ... i think it took ~90 seconds so the 60 second timeout expired but by the time it printed the list the compute service was there so this is just a timeing issue :(
00:06:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Revert "Temporarily skip TestNovaMigrationsMySQL" https://review.opendev.org/683011
00:12:46 openstackgerrit Merged openstack/nova-specs master: Track user_id/project_id for migrations https://review.opendev.org/682511
00:23:42 openstackgerrit Merged openstack/nova master: libvirt: Start reporting PCPU inventory to placement https://review.opendev.org/671793
00:49:43 KeithMnemonic1 mriedem thanks! so how do i proceed, create a qorking branch, apply you the conflicting patches and then try your cherry pick?
00:49:57 KeithMnemonic1 s/qorking/working
00:53:16 melwitt KeithMnemonic1: you can pull down his patch with 'git review -d 683008' and then you just need to resolve the merge conflicts he left in the tests
00:54:06 KeithMnemonic1 and then do a git review once i think they are resolved?
00:54:10 melwitt then --amend the commit and then push a new patch set
00:54:13 melwitt yes
00:54:25 KeithMnemonic1 perfect let me try that, thank you both!
00:55:23 melwitt cool good luck
01:44:07 openstackgerrit Fan Zhang proposed openstack/nova master: Fix exception translation when creating volume https://review.opendev.org/678991
01:51:48 KeithMnemonic1 melwitt quick question, if some of those conflicts are from other patches that cherry pick clean to pike, is the the proper process?
01:52:41 KeithMnemonic1 isnt it better to pull in cleanly any dependencies instead of me trying to shove some stuff in his review?
01:53:41 KeithMnemonic1 and then rebase that patch once they are merged?
01:54:09 melwitt KeithMnemonic1: we don't pull in dependencies that are unrelated to the backport, it can get pretty unwieldy that way. by resolving the conflict you're just making the patch fit within the env of the older release, which shouldn't result in adding a bunch of stuff
01:54:57 melwitt from what I saw when I glanced, the conflicts didn't look too big and mriedem documented all of them in the commit message already. just need to be smoothed out in the .py files
02:29:21 openstackgerrit Takashi NATSUME proposed openstack/nova master: Move Destination object tests to their own test class https://review.opendev.org/683017
03:21:28 openstackgerrit Merged openstack/nova master: Temporarily skip TestNovaMigrationsMySQL https://review.opendev.org/683009
03:21:36 openstackgerrit Merged openstack/nova stable/stein: Fix race in _test_live_migration_force_complete https://review.opendev.org/681744
03:21:45 openstackgerrit Merged openstack/nova stable/stein: Fix a misuse of assertGreaterEqual https://review.opendev.org/682168
05:45:09 openstackgerrit garyk proposed openstack/nova master: Deconstruct the mother of all locks https://review.opendev.org/682242

Earlier   Later