Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-07-05
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?
15:21:28 dansmith looks like he removed the try..except around the signature change which was my deal, but otherwise I either need to eye-diff it or pull it down
15:21:46 sean-k-mooney dansmith: he change it to the db lookup
15:21:53 sean-k-mooney right
15:22:02 sean-k-mooney i havent looked at it this mornig
15:22:09 dansmith alright
15:23:58 mriedem i was going to have to eye-diff anyway

Earlier   Later