Earlier  
Posted Nick Remark
#openstack-nova - 2019-07-04
18:04:16 sean-k-mooney for other drivers for other things that expect jsut the os name
18:04:25 sean-k-mooney not the os name + version
18:07:05 openstackgerrit Merged openstack/nova master: Refactor bandwidth related functional tests https://review.opendev.org/641679
#openstack-nova - 2019-07-05
01:01:06 openstackgerrit Merged openstack/nova stable/stein: Ignore hw_vif_type for direct, direct-physical vNIC types https://review.opendev.org/667354
02:18:11 openstackgerrit Boxiang Zhu proposed openstack/nova master: Make evacuation respects anti-affinity rule https://review.opendev.org/649963
03:13:03 lewe Hi, Maybe someone can help me, I am trying to cold migrate an instance but it does not seems to work right. The instance was newly created on the destination hosts rather than copying the original disk from the instance original host. Thank you.
03:36:16 openstackgerrit wangwei1 proposed openstack/nova master: fix spelling error in nova/api/validation/__init__.py https://review.opendev.org/669244
04:26:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix a broken link https://review.opendev.org/669248
04:45:38 openstackgerrit Seyeong Kim proposed openstack/nova stable/queens: Share snapshot image membership with instance owner https://review.opendev.org/661667
06:00:33 openstackgerrit wangwei1 proposed openstack/nova stable/stein: fix spelling error in nova/api/validation/__init__.py https://review.opendev.org/669251
06:32:07 openstackgerrit Ghanshyam Mann proposed openstack/nova-specs master: Fix the cross_az_attach value in AZ-unshelved spec https://review.opendev.org/669261
06:39:10 openstackgerrit Artom Lifshitz proposed openstack/nova master: Deprecate CONF.workarounds.enable_numa_live_migration https://review.opendev.org/640021
07:18:21 openstackgerrit Yongli He proposed openstack/nova master: Clean up orphan instances virt driver https://review.opendev.org/648912
07:18:21 openstackgerrit Yongli He proposed openstack/nova master: clean up orphan instances https://review.opendev.org/627765
07:24:02 openstackgerrit Yongli He proposed openstack/nova master: clean up orphan instances https://review.opendev.org/627765
08:12:55 openstackgerrit Kashyap Chamarthy proposed openstack/nova-specs master: Fix indentation in the "Secure Boot for KVM ... guests" spec https://review.opendev.org/669284
08:13:24 kashyap stephenfin: johnthetubaguy: A trivial cosmetic fix --^ (rST syntax related)
08:25:32 openstackgerrit Lee Yarwood proposed openstack/nova master: Get rid of args to RBDDriver.__init__() https://review.opendev.org/668564
08:35:40 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Remove native LUKS compat code https://review.opendev.org/669121
09:12:29 openstackgerrit Stephen Finucane proposed openstack/nova master: Fix spelling error in 'nova/api/validation/__init__.py' https://review.opendev.org/669244
09:14:08 openstackgerrit Stephen Finucane proposed openstack/nova stable/stein: Fix spelling error in 'nova/api/validation/__init__.py' https://review.opendev.org/669251
09:15:41 kashyap stephenfin: Hope I answered your question there: https://review.opendev.org/#/c/669129/
09:16:02 kashyap (Not an entirely satisfactory one; but it's the truth :-))
09:16:26 kashyap New rev upcoming.
09:17:30 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_PERF_VERSION https://review.opendev.org/669128
09:17:30 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove support for Intel CMT `perf` events https://review.opendev.org/669129
09:17:31 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: vif: Remove MIN_LIBVIRT_MACVTAP_PASSTHROUGH_VLAN https://review.opendev.org/669176
09:25:17 openstackgerrit Merged openstack/nova-specs master: Fix indentation in the "Secure Boot for KVM ... guests" spec https://review.opendev.org/669284
10:38:17 openstackgerrit Boxiang Zhu proposed openstack/nova master: Fix live migration break group policy simultaneously https://review.opendev.org/651969
10:40:40 bauzas gibi: remember me, do we document in the API docs something like https://review.opendev.org/#/c/630723/4/nova/api/openstack/compute/migrate_server.py ?
10:40:48 bauzas not the API guide I mean
10:41:23 bauzas I can read https://docs.openstack.org/neutron/latest/admin/config-qos-min-bw.html tho
10:41:45 bauzas and https://developer.openstack.org/api-guide/compute/port_with_resource_request.html document what's missing
10:42:02 bauzas but I wonder if within the API docs, we said that too
10:42:15 bauzas I mean api-ref
10:42:19 bauzas lemme check actually
10:44:13 bauzas nope, we don't (eg. https://developer.openstack.org/api-ref/compute/?expanded=resize-server-resize-action-detail#resize-server-resize-action )
10:44:31 bauzas anyway, not a big deal, it's just for internal docs
10:44:49 bauzas gibi: FWIW, ping me if you have any changes about ^ btw.
11:04:47 mdbooth >>> x.config_drive = False
11:04:47 mdbooth >>> x.config_drive
11:04:47 mdbooth u'False'
11:04:47 mdbooth >>> x.config_drive = True
11:04:56 mdbooth Don't you just love python?
11:05:06 mdbooth And that field has the wrong type, of course
11:06:06 mdbooth >>> x.config_drive = False
11:06:06 mdbooth >>> bool(x.config_drive)
11:06:06 mdbooth True
11:06:33 openstackgerrit Merged openstack/nova master: api-ref: Fix a broken link https://review.opendev.org/669248
11:47:51 stephenfin kashyap: Replied to https://review.opendev.org/#/c/669129/
11:48:12 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: vif: Remove MIN_LIBVIRT_MACVTAP_PASSTHROUGH_VLAN https://review.opendev.org/669176
11:51:04 gibi bauzas: we have the limitation separately, not in the api-ref. I think it was intentional as it is expected to be a temporary limitation
11:51:34 gibi bauzas: the actual implementation of server move operation still needs my time. it is started up here https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/support-move-ops-with-qos-ports
11:52:00 gibi bauzas: but I was busy with https://review.opendev.org/#/c/637955/
12:29:27 kashyap stephenfin: Will respond (saddled with a couple off meetings); thanks for the thoughtful reply.
12:41:34 openstackgerrit Lee Yarwood proposed openstack/nova master: Get rid of args to RBDDriver.__init__() https://review.opendev.org/668564
12:47:29 bauzas gibi: ack ok
13:05:26 openstackgerrit Balazs Gibizer proposed openstack/nova master: Error out interrupted builds https://review.opendev.org/666857
14:12:42 mriedem lyarwood: have you actually hit a recreate for this? https://review.opendev.org/#/q/I15a7c13edf78884ec223fd531a78a341106b41b8
14:15:04 lyarwood mriedem: did you mean reason?
14:15:32 mriedem no,
14:15:40 mriedem when that was reported there was no recreate in the bug for it,
14:15:45 mriedem and i was unable to recreate it in a functional test
14:15:54 mriedem so i'm hesitant to backport that without knowing how to recreate it
14:15:58 lyarwood mriedem: ah reproducer
14:16:19 mriedem yeah
14:16:21 lyarwood mriedem: no, that's fair it just looked like a trivial thing to change
14:16:36 mriedem ok i'd rather not backport it unless we have someone else reporting the problem
14:16:59 lyarwood mriedem: ack that's fine, I'll take a look next week at reproducing, if I can't I'll close
14:18:56 mriedem btw it occurs to me now we maybe shouldn't have merged this https://review.opendev.org/#/c/663887/
14:19:11 mriedem b/c i can't run fast8/pep8 on my bionic vm (py27 and py36) on rocky changes now
14:19:13 mriedem stephenfin: ^
14:21:40 stephenfin Because bionic doesn't support py3.5?
14:22:02 stephenfin It's a tough one. Without it, I can't run the same on my Fedora 30 machine for the reasons in the commit message
14:23:15 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Add functional recreate test for regression bug 1825537 https://review.opendev.org/669361
14:23:15 openstack bug 1825537 in OpenStack Compute (nova) rocky "finish_resize failures incorrectly revert allocations" [Medium,Confirmed] https://launchpad.net/bugs/1825537
14:23:18 mriedem rocky doesn't support py37 at all though right?
14:23:28 mriedem rocky tox.ini at least has py36 targets
14:23:47 sean-k-mooney stephenfin: it use py36
14:23:51 mriedem iow, if you're doing rocky stuff and only have 3.7, you're going to need another vm
14:24:01 mriedem bionic has 2.7 and 3.6
14:24:12 stephenfin It doesn't, no, but the use of python3 means "give me any version of Python 3.x"
14:24:27 sean-k-mooney stephenfin: we dont support py3.5 anymore
14:24:34 sean-k-mooney its not part of the train PTI
14:24:40 stephenfin We did in rocky though
14:24:48 stephenfin That's a stable-only patch
14:24:48 sean-k-mooney rocky proably yes
14:25:40 stephenfin mriedem: Could we use basepython = python3.6 instead, assuming that's available on both Bionic and Xenial (or whatever 16.04 is?)
14:25:44 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Revert "tox: Use basepython of python3.5" https://review.opendev.org/669362
14:25:58 mriedem stephenfin: in rocky i don't think so,
14:26:12 mriedem because rocky supported (in the gate and the tox default env list) 27 and 35
14:26:17 mriedem but has a 36 target
14:26:39 mriedem there is no official 37 test targeting in rocky though
14:27:04 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Boot a VM with an unaddressed port https://review.opendev.org/641670
14:27:06 mriedem so like i said, i think if you're doing rocky changes and only have 37, you'll just need another dev vm for working on those older branches (which is what i have to do for ocata changes at this point)
14:27:12 mriedem i have a xenial vm just for doing ocata stuff now
14:27:44 mriedem xenial less for the python thing but more b/c of native packages like postgresql
15:04:53 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove nova-consoleauth https://review.opendev.org/652969
15:07:27 stephenfin lyarwood: The stable/stein variant of this has merged now. Could you tackle this one? https://review.opendev.org/#/c/667355/
15:20:58 dansmith sean-k-mooney: mriedem: Artom made several changes to the revert resize patch yesterday but didn't respond to comments and also rebased it so there's a lot of change.. did I miss any discussion about that?

Earlier   Later