Earlier  
Posted Nick Remark
#openstack-nova - 2018-12-09
17:23:30 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Ignore MoxStubout deprecation warnings https://review.openstack.org/623545
23:41:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992
23:42:21 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993
23:42:40 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix best_match() deprecation warning https://review.openstack.org/611204
23:43:33 openstackgerrit Takashi NATSUME proposed openstack/nova stable/queens: Add description of custom resource classes https://review.openstack.org/619125
23:45:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add descriptions for vol-backed snapshots https://review.openstack.org/615084
23:46:14 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628
23:49:41 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104
23:50:05 openstackgerrit Takashi NATSUME proposed openstack/nova master: Use oslo_db.sqlalchemy.test_fixtures https://review.openstack.org/609352
23:50:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153
#openstack-nova - 2018-12-10
02:20:55 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add descriptions of numbered resource classes and traits https://review.openstack.org/621494
03:05:12 openstackgerrit Zhenyu Zheng proposed openstack/nova master: WIP detach root volume API changes https://review.openstack.org/623981
05:21:21 openstackgerrit Ghanshyam Mann proposed openstack/nova stable/rocky: DNM: For testing multinode and tempest-slow job https://review.openstack.org/623990
05:32:50 openstackgerrit Ghanshyam Mann proposed openstack/nova stable/queens: DNM: For testing multinode and tempest-slow job https://review.openstack.org/623992
06:05:48 openstackgerrit Merged openstack/nova master: Transform scheduler.select_destinations notification https://review.openstack.org/508506
06:10:30 brinzhang #join #oo
06:10:39 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Bump compute service to indicate attach/detach root volume is supported https://review.openstack.org/614750
06:12:28 openstackgerrit Zhenyu Zheng proposed openstack/nova master: WIP detach root volume API changes https://review.openstack.org/623981
06:47:20 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104
07:04:18 belmoreira leakypipes: thanks. I will give you feedback asap
07:23:26 openstackgerrit Rui Zang proposed openstack/nova-specs master: support virtual persistent memory https://review.openstack.org/601596
07:25:54 openstackgerrit Rui Zang proposed openstack/nova-specs master: Virtual persistent memory libvirt driver implementation https://review.openstack.org/622893
08:45:06 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Bump compute service to indicate attach/detach root volume is supported https://review.openstack.org/614750
08:46:43 openstackgerrit Zhenyu Zheng proposed openstack/nova master: WIP detach root volume API changes https://review.openstack.org/623981
08:57:13 kaisers kaisers
09:00:47 openstackgerrit Silvan Kaiser proposed openstack/nova master: Exec systemd-run without --user flag in Quobyte driver https://review.openstack.org/554195
09:14:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: Final release note for versioned notification transformation https://review.openstack.org/624022
09:28:49 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Reject migration requests when source is down https://review.openstack.org/623489
09:44:04 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268
09:44:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: Calculate RequestGroup resource provider mapping https://review.openstack.org/616239
09:44:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: Extend RequestGroup object for mapping https://review.openstack.org/619527
09:44:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Pass resource provider mapping to neutronv2 api https://review.openstack.org/616240
09:44:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fill the RequestGroup mapping during schedule https://review.openstack.org/619528
09:44:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send RP uuid in the port binding https://review.openstack.org/569459
09:44:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: Recalculate request group - RP mapping during re-schedule https://review.openstack.org/619529
09:44:08 openstackgerrit Balazs Gibizer proposed openstack/nova master: Reject interface attach with QoS aware port https://review.openstack.org/570078
09:44:08 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317
09:44:09 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421
09:44:09 openstackgerrit Balazs Gibizer proposed openstack/nova master: Reject networks with QoS policy https://review.openstack.org/570079
09:44:10 openstackgerrit Balazs Gibizer proposed openstack/nova master: Ensure that allocated PF matches the used PF https://review.openstack.org/623543
10:05:09 openstackgerrit Merged openstack/nova master: Add ratio online data migration when load compute node https://review.openstack.org/613499
10:11:34 openstackgerrit Merged openstack/nova master: Add compute_node ratio online data migration script https://review.openstack.org/609995
10:11:40 openstackgerrit Merged openstack/nova master: Note the aggregate allocation ratio restriction in scheduler docs https://review.openstack.org/620713
10:17:52 izza_ hi
10:18:12 izza_ anyone here already deployed volume with attached image (windows) in openstack tripleo
10:18:16 izza_ need help pls
10:30:41 openstackgerrit Merged openstack/nova stable/rocky: Ignore MoxStubout deprecation warnings https://review.openstack.org/623545
11:24:31 openstackgerrit Chris Dent proposed openstack/nova master: Add python 3.7 unit and functional tox jobs https://review.openstack.org/624055
11:29:49 openstackgerrit Chris Dent proposed openstack/nova master: Use external placement in functional tests https://review.openstack.org/617941
11:29:50 openstackgerrit Chris Dent proposed openstack/nova master: Delete the placement code https://review.openstack.org/618215
11:46:55 cdent thanks for continuing to +2 the placement functional stuff gibi, we'll get it merged one of these days
11:48:23 gibi cdent: this morning when I did a git pull I got happy as the pull brought in the deletion of placement related files. But then I had to realize that it was just api sample removal not your patch (yet)
11:48:34 cdent :)
12:23:21 openstackgerrit Chris Dent proposed openstack/nova master: Add python 3.7 unit and functional tox jobs https://review.openstack.org/624055
12:24:10 cdent dansmith looks like the multi_cell query job is not happy with python 3.7 ^
12:24:16 openstackgerrit Yikun Jiang proposed openstack/nova master: Add live migration timeout action https://review.openstack.org/619143
12:24:34 openstackgerrit Yikun Jiang proposed openstack/nova master: Remove live_migration_progress_timeout config https://review.openstack.org/619142
13:20:01 openstackgerrit Balazs Gibizer proposed openstack/nova master: Refactor PortResourceRequestBasedSchedulingTestBase https://review.openstack.org/624080
13:20:01 openstackgerrit Balazs Gibizer proposed openstack/nova master: Ensure that allocated PF matches the used PF https://review.openstack.org/623543
14:02:04 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Add workaround to cleanup instance dir during evac with rbd https://review.openstack.org/618478
14:17:03 lyarwood melwitt: ^ when you're around would you mind taking another look at that? I honestly can't see a way for cleanup to be called erroneously on the source host during a evacuation failure.
14:42:56 sean-k-mooney bauzas: melwitt could one of ye apporve https://review.openstack.org/#/c/618239/ as release liaison/ptl when ye get a chance.
14:43:50 openstack Launchpad bug 1734320 in os-vif "Eavesdropping private traffic" [High,In progress] - Assigned to sean mooney (sean-k-mooney)
14:43:50 sean-k-mooney the main change is fixing the os-vif side of https://bugs.launchpad.net/neutron/+bug/1734320 + https://bugs.launchpad.net/os-vif/+bug/1801072
14:43:51 openstack Launchpad bug 1801072 in os-vif "vif_plug_ovs.linux_net.delete_net_dev is called outside the privsep context" [Critical,Fix released] - Assigned to sean mooney (sean-k-mooney)
14:55:25 KeithMnemonic Is it possible to please get some reviews on this patch? https://review.openstack.org/#/c/573066/
14:58:16 sean-k-mooney KeithMnemonic: you realise that libvirt/qemu has a limit on how many volumes you can attach to an instance too
14:58:33 KeithMnemonic yes there is a pci limit i thought
14:58:40 KeithMnemonic is that 26?
14:58:49 sean-k-mooney i think its 20 not 26
14:59:49 openstack Launchpad bug 1770527 in OpenStack Compute (nova) "openstack server add volume fails over 26vols" [Wishlist,In progress] - Assigned to Tsuyoshi Nagata (yukari-papa)
14:59:49 KeithMnemonic it is more than 20 i am pretty sure, problem is this bug is still open https://bugs.launchpad.net/nova/+bug/1770527 so we have a customer asking for it
14:59:52 sean-k-mooney the 26 limit on device names i sobviousl just because tehre are 26 ascii/english letter
15:00:27 sean-k-mooney KeithMnemonic: it may depend on the qemu/libvirt version
15:00:31 KeithMnemonic https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/5/html/virtualization/sect-virtualization-virtualization_limitations-kvm_limitations
15:01:05 KeithMnemonic Hence, of the 32 available PCI devices for a guest, 4 are not removable. This means there are 28 PCI slots available for additional devices per guest. Every para-virtualized network or block device uses one slot. Each guest can use up to 28 additional devices made up of any combination of para-virtualized network, para-virtualized disk devices, or other PCI devices using VT-d.
15:03:31 sean-k-mooney does the 4 include the vnc/spice resouces
15:04:50 sean-k-mooney you can use virtio-scsi too which may allow mulitple volumes per pci device
15:18:59 mriedem lyarwood: surprise surprise multiattach swap volume across 2 hosts is broken
15:21:28 lyarwood mriedem: wasn't mdbooth looking at blocking all attempts to swap/migrate multiattach volumes?
15:21:49 lyarwood he isn't around today btw
15:22:04 mriedem he was for at least cases of the multiattach volume having >1 read/write attachment
15:22:17 mriedem https://review.openstack.org/#/c/572790/
15:22:23 mriedem but this isn't that issue,
15:22:49 mriedem the swap volume code in compute updates the bdm record for the "old" volume and changes the volume_id to the "new" volume
15:22:52 openstackgerrit Merged openstack/nova master: Add docs for (initial) allocation ratio configuration https://review.openstack.org/622588
15:22:56 mriedem essentially orphaning the old volume, so we don't cleanup properly
15:23:48 mriedem this https://github.com/openstack/nova/blob/ae3064b7a820ea02f7fc8a1aa4a41f35a06534f1/nova/compute/manager.py#L5798-L5806
15:24:10 mriedem the bdm that gets updated is the old bdm (source volume), but save_volume_id is the new volume
15:24:41 mriedem we're also clearly wrongly updating the old bdm.connection_info with the new vol connection_info,
15:24:57 mriedem for new volume attach flows that doesn't matter as we don't use the bdm.connection_info, but it's still wrong
15:25:36 mriedem looks like that code assumes we did a cinder-induced retype/migration, "# correct volume_id returned by Cinder."
15:25:43 mriedem but that's not the case here
15:27:13 mriedem if only we had that volume_id uniqute constraint in the bdms table :)
15:27:20 mriedem the bdm update would blow up hard
15:28:41 lyarwood if only, sooooooooooo Matt also had a plan to block the direct use of this API FWIW at some point recently
15:28:58 lyarwood so only allow it to be used via volume migration API
15:29:20 lyarwood but this still sounds valid, do we have a launchpad bug for this?
15:29:47 mriedem not yet, i'm debugging the failures in the tempest-slow job here https://review.openstack.org/#/c/606981/ - i'll be dumping notes in a launchpad bug
15:30:46 mriedem i'm not sure how the compute api would block swap volume unless we checked the volume status to see if it's either 'retyping' or 'migrating' and fail otherwise

Earlier   Later