Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-07-28
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
07:58:02 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: libvirt: add method to configure migration speed https://review.openstack.org/586965
08:11:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove unused stubbing function from test https://review.openstack.org/586968
09:05:48 kashyap sean-k-mooney: Thanks for the review here: https://review.openstack.org/#/c/567258/
09:06:27 sean-k-mooney did you get a chance to look into why the live migration was failing friday?
09:06:28 kashyap sean-k-mooney: Darn, I haven't yet dug into the Tempest failures. And I'll be off from Wednesday for 2 1/2 weeks
09:06:31 kashyap :-(
09:06:53 kashyap sean-k-mooney: I looked at it a couple of times, there were Cloud Provider issues
09:07:04 kashyap Let me look at it now
09:07:52 kashyap sean-k-mooney: I wonder if we can get this merged by tomm...
09:17:23 openstackgerrit huanhongda proposed openstack/nova master: Allow unicode keys for metadata https://review.openstack.org/536236
09:21:15 sean-k-mooney kashyap: well if you think its a cloud provider issue you chould just recheck. but this looked like it just did not migrate to the correct host so i would guesss there is a traceback in the n-cpu log
09:21:48 kashyap sean-k-mooney: Right, I did 'recheck' 2 times, and got over the Cloud Provider issues. But just looking through https://orangebox.com/products/Do
09:21:52 kashyap Err, http://logs.openstack.org/58/567258/10/check/nova-live-migration/17bbee9/job-output.txt.gz
09:21:58 bauzas kashyap: yup, looks like infra has a cloud provider issue
09:21:58 kashyap To find the right failure
09:28:43 kashyap bauzas: Hmm
09:28:45 kashyap Still?
09:30:01 bauzas no fresh news I know of, I'm afraid
09:30:30 sean-k-mooney kashyap: looks to me like this was a failure in the code you changed http://logs.openstack.org/58/567258/10/check/nova-live-migration/17bbee9/logs/screen-n-cpu.txt.gz?level=DEBUG#_Jul_27_10_28_57_610656
09:31:15 kashyap sean-k-mooney: Ah, let me look
09:32:02 kashyap sean-k-mooney: Which test failure is it?
09:32:11 kashyap sean-k-mooney: Is there a page with a precise list of test failures?
09:32:26 sean-k-mooney test_live_block_migration[id-1dce86b8-eb04-4c03-a9d8-9c1dc3ee0c7b]
09:32:32 sean-k-mooney http://logs.openstack.org/58/567258/10/check/nova-live-migration/17bbee9/logs/testr_results.html.gz
09:32:36 kashyap I see: http://logs.openstack.org/58/567258/10/check/nova-live-migration/17bbee9/job-output.txt.gz#_2018-07-27_10_29_43_862692

Earlier   Later