Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-07-28
00:33:58 mnaser i should find other places where the db writes had failed and find their stack traces
00:34:03 mnaser ill do that too
00:37:18 melwitt there are some things in here https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/objects/resource_provider.py
00:37:46 melwitt specifically this one https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/objects/resource_provider.py#L4039
00:38:06 melwitt the note there explains it well
00:40:12 melwitt awesome mnaser, thanks
00:47:58 mnaser ok i got a gold mine
00:48:23 mnaser i retrieved a list of all the .save or .create that failed because of db errors
00:48:34 mnaser which should be useful to group up together into a single transaction
00:48:56 mnaser im just deduping the list
00:49:06 mnaser fwiw -- grep _handle_dbapi_exception /var/log/nova/nova-api-wsgi.log -B70 | egrep 'nova.api.openstack.wsgi .*\.(save|create)' -B1
00:50:42 melwitt sweet
00:54:51 melwitt mnaser: do you want to write up a bug for this and include the trace paste and all? else I can do it later, I have to run soon
00:55:23 openstack Launchpad bug 1784074 in OpenStack Compute (nova) "Instances end up with no cell assigned in instance_mappings" [Medium,In progress] - Assigned to Matt Riedemann (mriedem)
00:55:23 mnaser melwitt: https://bugs.launchpad.net/nova/+bug/1784074 i added them all here
00:56:05 melwitt okay
00:57:43 melwitt think I'm going to create a new bug since the db fail thing is a bit different
00:58:17 mnaser melwitt: it's up to you to how you see fit :)
00:58:26 melwitt :)
00:59:36 mnaser https://github.com/openstack/nova/blob/ff5cded9afa65cd4317ba0e3401692ed92c1d150/nova/api/openstack/placement/objects/project.py#L83-L92
00:59:47 mnaser i think this is an easier pattern especially for create this stuff
00:59:53 mnaser rather than a top level function (i think)
01:00:27 melwitt yeah
01:01:13 mnaser https://github.com/openstack/nova/blob/16a2db6f23d150330dc82a56d0cf10366b004210/nova/objects/aggregate.py#L67-L78
01:01:19 mnaser this seems like an interesting pattern too
01:01:26 mnaser im digging around code
01:02:34 mnaser im just curious/worry that wrapping things in the writer twice might break things
01:02:40 mnaser i.e.: https://github.com/openstack/nova/blob/16a2db6f23d150330dc82a56d0cf10366b004210/nova/objects/instance_mapping.py#L75-L86
01:02:58 melwitt it will, don't nest it
01:03:00 melwitt well,
01:03:06 melwitt you can't nest a write under a read
01:03:23 mnaser can you nest two writes? or should i drop it from _create_in_db
01:03:33 melwitt so otherwise nesting is probably ok
01:03:43 mnaser ok cool
01:03:44 melwitt but I don't know if I've tried it, intentionally anyway
01:04:03 mnaser ill make sure i have a failing test first to make sure it actually gets fixed
01:05:58 melwitt awesome, sounds great
01:17:15 openstack Launchpad bug 1784093 in OpenStack Compute (nova) "Build requests can be orphaned without instance mappings" [Medium,Confirmed] - Assigned to Mohammed Naser (mnaser)
01:17:15 melwitt mnaser: https://bugs.launchpad.net/nova/+bug/1784093 fyi, I assigned it to you
01:18:22 mnaser melwitt: cool, i'm building up a test right now, i should probably have a patch later today if this doesnt end up much more complicated :>
01:19:12 melwitt okay, good luck. we'll take a look at it next week
01:19:26 mnaser thanks, have a good weekend!
01:19:37 melwitt thanks and likewise!
02:39:20 openstackgerrit Merged openstack/nova master: Add nova-status upgrade check for request spec migrations https://review.openstack.org/581813
03:13:51 zzzeek does the nova placement database get built as part of nova-manage db sync or nova-manage api_db sync ?
03:14:51 zzzeek api_db....damn
03:54:52 openstackgerrit Merged openstack/nova master: Fix create_all() to replace_all() in comments https://review.openstack.org/586396
04:04:12 openstackgerrit Mohammed Naser proposed openstack/nova master: Create request spec, build request and mappings in one transaction https://review.openstack.org/586742
04:05:03 mnaser ^ tests failing but i dont know if i'm on the right track, i dont wanna dig in too deep and be in the wrong
04:41:44 openstackgerrit Merged openstack/nova master: libvirt: Revert non-reporting DISK_GB if sharing https://review.openstack.org/586614
10:39:38 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Retry allocation writes server side https://review.openstack.org/586048
11:59:43 openstackgerrit Thomas Goirand proposed openstack/nova master: Remove "async" as variable name https://review.openstack.org/586782
12:10:07 openstackgerrit Thomas Goirand proposed openstack/nova master: Remove "async" as variable name https://review.openstack.org/586782
12:50:28 openstackgerrit Merged openstack/nova master: libvirt: Start populating NUMACell.network_metadata field https://review.openstack.org/564441
12:50:34 openstackgerrit Merged openstack/nova master: Add functional tests for numa-aware-vswitches https://review.openstack.org/583288
12:50:43 openstackgerrit Merged openstack/nova master: Consider network NUMA affinity for move operations https://review.openstack.org/585339
14:55:41 openstackgerrit xulei proposed openstack/nova stable/queens: Disable limits if force_hosts or force_nodes is set https://review.openstack.org/584616
15:18:40 openstackgerrit Merged openstack/nova stable/queens: Make host_aggregate_map dictionary case-insensitive https://review.openstack.org/585334
15:54:10 openstackgerrit Mohammed Naser proposed openstack/nova master: Create request spec, build request and mappings in one transaction https://review.openstack.org/586742
15:54:11 openstackgerrit Mohammed Naser proposed openstack/nova master: Check for build request even if instance mapping doesn't exist https://review.openstack.org/586824
16:51:52 openstackgerrit Merged openstack/nova master: Fix accumulated nits from port binding for live migration series https://review.openstack.org/583994
19:46:07 openstackgerrit Vieri proposed openstack/osc-placement master: Add image link in README.rst https://review.openstack.org/586839
#openstack-nova - 2018-07-29
02:11:00 openstackgerrit Thomas Goirand proposed openstack/nova master: Remove "async" as variable name https://review.openstack.org/586782
02:42:32 openstackgerrit Merged openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata https://review.openstack.org/564452
03:41:16 openstackgerrit Merged openstack/nova master: Assorted cleanups from numa-aware-vswitches series https://review.openstack.org/582651
06:36:43 openstackgerrit Merged openstack/nova master: Fix ServerMigrationSampleJsonTests to use sample files from version dir https://review.openstack.org/582702
11:48:11 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Remove the deprecated API extensions policies https://review.openstack.org/586872
11:56:52 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Remove the deprecated API extensions policies https://review.openstack.org/586872
18:12:28 openstackgerrit Merged openstack/python-novaclient master: Update reno for stable/rocky https://review.openstack.org/586366
22:07:30 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104
22:07:52 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (4) https://review.openstack.org/574106
22:08:07 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (5) https://review.openstack.org/574110
22:08:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (6) https://review.openstack.org/574113
23:13:52 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (7) https://review.openstack.org/574974
23:15:40 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8) https://review.openstack.org/575311
23:15:54 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9) https://review.openstack.org/575581
23:39:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (10) https://review.openstack.org/576017
23:40:12 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (11) https://review.openstack.org/576018
23:40:25 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (12) https://review.openstack.org/576019
23:40:36 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13) https://review.openstack.org/576020
23:40:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14) https://review.openstack.org/576027
23:41:08 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.openstack.org/576031
23:43:02 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.openstack.org/576299
23:43:16 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.openstack.org/576344
#openstack-nova - 2018-07-30
00:20:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.openstack.org/576673
01:15:09 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.openstack.org/576676
01:43:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.openstack.org/576689
01:44:27 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.openstack.org/576709
01:44:44 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.openstack.org/576712
04:58:54 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform compute_task notifications https://review.openstack.org/482629
04:59:14 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Create specs directory for Stein https://review.openstack.org/573602
05:31:58 openstackgerrit Thomas Goirand proposed openstack/nova master: Remove "async" as variable name https://review.openstack.org/586782
06:54:03 openstackgerrit Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.openstack.org/556751
06:56:30 openstackgerrit Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.openstack.org/556751
07:21:18 vishakha takashin : Hi, Waiting for your response https://review.openstack.org/#/c/580271/
07:23:38 takashin vishakha: okay. I will review it today.
07:25:20 vishakha takashin, thanks Pl review this too https://review.openstack.org/#/c/586446/
07:48:19 openstackgerrit Merged openstack/nova master: Fix signature of _FakeImageService.download https://review.openstack.org/585747
07:48:26 openstackgerrit Merged openstack/nova master: api-ref: document user_data length restriction https://review.openstack.org/586388
07:57:14 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: libvirt: add method to configure migration speed https://review.openstack.org/586965

Earlier   Later