Earlier  
Posted Nick Remark
#openstack-nova - 2018-04-12
19:00:18 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
19:00:19 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
19:00:20 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674
19:00:21 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
19:00:22 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
19:00:23 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
19:00:29 mriedem arvindn05_: i'm sure bauzas would love to think about this
19:01:34 arvindn05_ will wait for their comments....will update the spec to make it clear based on your comments for now...thanks mriedem
19:04:31 arvindn05_ mriedem: btw are you convinced with the argument that image properties filter and using traits on images are independent of each other?
19:08:17 mriedem arvindn05_: no
19:08:22 mriedem but,
19:08:27 mriedem i also don't feel strongly about it
19:08:59 arvindn05_ that's half the battle won :)
19:09:14 mriedem ha, now you're getting it
19:09:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Skip placement on rebuild in same host https://review.openstack.org/546357
19:09:54 arvindn05_ but ignoring the rebuild issue, like i mentioned if tommorow someone removed the image properties filter, based on the original BP, there would be no regression of the feature
19:10:27 mriedem hongbin: when you get a minute, can you propose backports for https://review.openstack.org/#/c/546357/ to queens and pike?
19:10:44 hongbin mriedem: sure
19:10:48 mriedem thanks
19:10:59 mriedem you'll have to do it from command line
19:11:38 hongbin i see, sure, no problem
19:12:40 openstackgerrit Merged openstack/nova master: uncap eventlet in nova https://review.openstack.org/560420
19:12:58 openstackgerrit Merged openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451
19:15:18 openstackgerrit Merged openstack/nova master: Remove mox in unit/virt/xenapi/test_vm_utils.py (2) https://review.openstack.org/558993
19:15:30 openstackgerrit Merged openstack/nova master: Remove mox in unit/virt/xenapi/test_vm_utils.py (3) https://review.openstack.org/559258
19:16:42 openstackgerrit Hongbin Lu proposed openstack/nova stable/queens: Skip placement on rebuild in same host https://review.openstack.org/561014
19:22:35 openstackgerrit Hongbin Lu proposed openstack/nova stable/pike: Skip placement on rebuild in same host https://review.openstack.org/561015
19:26:04 openstackgerrit Arvind Nadendla proposed openstack/nova-specs master: Handle rebuild of instance with new image https://review.openstack.org/560718
19:35:20 openstackgerrit Arvind Nadendla proposed openstack/nova-specs master: Handle rebuild of instance with new image https://review.openstack.org/560718
19:43:05 cfriesen melwitt: for the detach_volume() change, are you suggesting a try/except block around the call to encryptor.detach_volume()?
19:43:49 melwitt cfriesen: yes. because if you get there on a second attempt and you already ran it in the past, os-brick will raise "unknown device" because it can't find the attached device
19:44:08 cfriesen makes sense, will do
19:44:25 melwitt we backported the os-brick change back to pike but we can't bump requirements.txt on stable
19:45:30 melwitt so it's not guaranteed that someone running stable/pike will have the os-brick version that will ignore exit code 4 (unknown) for you
19:48:06 melwitt s/you/them/
20:00:55 mriedem i found another cells upcall
20:01:00 mriedem https://github.com/openstack/nova/blob/c531b7905f5a9f8a5bfa355a2047d04032cbf847/nova/virt/xenapi/host.py#L86
20:01:18 mriedem however, it's for that completely broken and deprecated anyway host maintenance / power action stuff
20:08:10 openstackgerrit Matt Riedemann proposed openstack/nova master: Use Queens UCA for nova-multiattach job https://review.openstack.org/554317
20:09:12 cfriesen melwitt: I don't have the actual "target" to report in the error message. do you know how to get it, or can I just use "disk_dev" or something?
20:10:33 melwitt cfriesen: yeah, disk_dev is the equivalent
20:11:11 melwitt this version of the patch was almost right, it just also needed to reraise if not exit code 4 https://review.openstack.org/#/c/515008/3/nova/virt/libvirt/driver.py
20:11:27 cfriesen ah, okay
20:11:55 melwitt because we still want to fail if a legit detach error happened. but we'll want to ignore "unknown" only
20:17:28 openstackgerrit Matt Riedemann proposed openstack/nova master: DNM: test live_migration_wait_for_vif_plug=True https://review.openstack.org/558006
20:17:43 openstackgerrit Matt Riedemann proposed openstack/nova master: DNM: test live_migration_wait_for_vif_plug=True https://review.openstack.org/558006
20:39:31 openstackgerrit Chris Friesen proposed openstack/nova stable/pike: libvirt: disconnect volume from host during detach https://review.openstack.org/560690
20:42:59 mriedem melwitt: i'll be skipping the meeting today for fun times, left some links to gate bugs in the meeting agenda this morning, but the multiattach one is already semi resolved
20:43:42 melwitt mriedem: we shall miss you. okay, I'll take a look at those bugs
20:47:34 openstackgerrit Jay Pipes proposed openstack/nova-specs master: Standardize CPU resource tracking https://review.openstack.org/555081
20:50:12 melwitt nova meeting in 10 minutes
21:02:30 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: Localdisk https://review.openstack.org/549300
21:52:03 cfriesen melwitt: I've got a theory for https://bugs.launchpad.net/nova/+bug/1763181 but I guess mriedem and bauzas are away
21:52:03 openstack Launchpad bug 1763181 in OpenStack Compute (nova) "test_parallel_evacuate_with_server_group intermittently fails" [Medium,Confirmed]
21:59:43 openstackgerrit Michael Still proposed openstack/nova master: Move xenapi disk resizing to privsep. https://review.openstack.org/552242
21:59:44 openstackgerrit Michael Still proposed openstack/nova master: Sync xenapi and libvirt on what flags to pass e2fsck. https://review.openstack.org/554078
21:59:45 openstackgerrit Michael Still proposed openstack/nova master: Move xenapi partition copies to privsep. https://review.openstack.org/553605
21:59:46 openstackgerrit Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437
21:59:47 openstackgerrit Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439
21:59:48 openstackgerrit Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438
22:05:08 openstackgerrit Eric Fried proposed openstack/nova master: add lower-constraints job https://review.openstack.org/555961
22:05:21 openstackgerrit Eric Fried proposed openstack/nova master: add lower-constraints job https://review.openstack.org/555961
22:07:06 melwitt cfriesen: noyce, thanks for the comment on the bug
22:13:53 cfriesen melwitt: there's another variant that can affect migrations, where we'd want to look at the migration records and include any hosts that are the destination or source of a migration for instances in the group.
22:14:34 cfriesen the joys of race conditions
22:15:39 melwitt guh
23:05:39 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_serversV21.py (1) https://review.openstack.org/560821
23:20:01 mriedem_funtime new unit test race fail http://logs.openstack.org/78/554378/3/check/openstack-tox-py27/54b38c0/testr_results.html.gz
23:22:35 mriedem https://bugs.launchpad.net/nova/+bug/1763535
23:22:36 openstack Launchpad bug 1763535 in OpenStack Compute (nova) "nova.tests.unit.virt.xenapi.test_vm_utils.StreamDiskTestCase.test_non_ami intermittently fails" [High,Confirmed]
23:28:01 mriedem takashin: ^ you should probably look at that
23:36:44 mriedem melwitt: https://review.openstack.org/#/q/I78dead482e2ed8262745cb08c9f4ab71035adb33 are what fix the multiattach job
23:36:59 mriedem https://review.openstack.org/#/c/554317/ on top has to depend on the queens backport,
23:37:12 mriedem otherwise zuul pulls the queens job def which has the bad branch specifier in it and borks everything up
23:37:30 mriedem so once those patches to remove the in-tree branch specifier are merged, we can make the multiattach job voting and gating again
23:51:39 openstackgerrit Merged openstack/nova master: Remove :return from update_provider_tree docstring https://review.openstack.org/560442
#openstack-nova - 2018-04-13
00:08:23 melwitt mriedem_afk: alright, thanks
01:05:35 openstackgerrit Merged openstack/nova master: Move get_stashed_volume_connector to compute.utils https://review.openstack.org/557810
01:14:12 openstackgerrit Merged openstack/nova master: ProviderTree.{add|remove}_{traits|aggregates} https://review.openstack.org/553475
02:10:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_serversV21.py (2) https://review.openstack.org/560829
03:01:37 openstackgerrit Merged openstack/nova master: Remove mox in test_neutron_security_groups.py https://review.openstack.org/560267
03:02:55 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
03:13:23 openstackgerrit jichenjc proposed openstack/nova master: Add config drive link to api-guide https://review.openstack.org/561113
03:21:01 openstackgerrit jichenjc proposed openstack/nova master: Update link of metadata https://review.openstack.org/561114
03:33:32 phuongnh hi all, I would like to add one more line in file https://review.openstack.org/#/c/462759/3/os_traits/hw/cpu/x86.py for CPU_FPGA
03:33:40 phuongnh the link to this is: https://en.wikipedia.org/wiki/Field-programmable_gate_array
03:33:44 phuongnh Do I need to propose a blueprint in launchpad?
04:02:49 openstackgerrit Nguyen Hung Phuong proposed openstack/os-traits master: [WIP] add fpga cpu for the future https://review.openstack.org/561117
04:07:57 sapd_ Hi guys, I want to pass volume_type to block_device_mapping_v2, I saw this specs: https://review.openstack.org/#/c/362698/5/specs/ocata/approved/add-volume-type-when-boot-instances.rst , but have it implemented yet?
04:32:04 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: WIP: complex policy https://review.openstack.org/553776
04:47:55 openstackgerrit Takashi NATSUME proposed openstack/nova master: Partially revert "Remove mox in unit/virt/xenapi/test_vm_utils.py (3)" https://review.openstack.org/561121
04:51:06 openstackgerrit jichenjc proposed openstack/nova master: [libvirt] Avoid create swap definition in xml https://review.openstack.org/561122
05:14:36 openstackgerrit Merged openstack/nova master: Move xenapi disk resizing to privsep. https://review.openstack.org/552242
05:53:10 gmann sapd_: this is not implemented and idea is not to implement. you can check the details in that spec
05:57:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
05:58:01 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
06:19:03 openstackgerrit Merged openstack/nova master: Get rid of 406 paths in report client https://review.openstack.org/556633
06:38:33 alex_xu johnthetubaguy_: here is something about policy, want to hear your idea https://review.openstack.org/#/c/554159/
06:52:03 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601
06:52:04 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support relay RP for allocation candidates https://review.openstack.org/533437

Earlier   Later