Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-01
15:09:19 mriedem they do need it, they are broken
15:09:27 mriedem but that can be fixed in a separate patch
15:09:36 mriedem this is why i asked for that validation yesterday
15:14:28 mgagne oh.... wonderful
15:19:42 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove get_config_vhostuser https://review.openstack.org/565471
15:19:43 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: MIN_LIBVIRT_VHOSTUSER_MQ https://review.openstack.org/565472
15:19:44 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PF_WITH_NO_VFS_CAP_VERSION https://review.openstack.org/565517
15:22:01 openstackgerrit Brianna Poulos proposed openstack/python-novaclient master: Microversion 2.63 - Add trusted_image_certificates https://review.openstack.org/500396
15:22:21 mriedem bpoulos: what's the status on getting a spec amendment for that trusted certs stuff?
15:22:49 bpoulos mriedem: it's written, I'm just waiting for approval to post it.
15:23:05 mriedem i approve
15:23:32 bpoulos mriedem: I wish your approval were enough :)
15:24:10 bpoulos mriedem: unfortunately, it sometimes takes a while, and it can be very frustrating for us when we want to be able to do a quick turnaround
15:24:40 mriedem understandable
15:25:03 mriedem would i get a grant kickback if i just pushed the spec amendment myself?
15:25:18 mriedem you don't have to answer that
15:35:00 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove get_config_vhostuser https://review.openstack.org/565471
15:35:01 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PF_WITH_NO_VFS_CAP_VERSION https://review.openstack.org/565517
15:35:01 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: MIN_LIBVIRT_VHOSTUSER_MQ https://review.openstack.org/565472
15:40:16 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565043
15:40:17 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: Lift the restriction of choices for `cpu_model_extra_flags` https://review.openstack.org/563926
15:47:23 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: Lift the restriction of choices for `cpu_model_extra_flags` https://review.openstack.org/563926
15:48:42 mriedem dansmith: kashyap's https://review.openstack.org/#/q/topic:bug/1750829+status:open changes should be ready, to lift the cpu features flags restrictions so you can specify more than just pcid
15:50:11 dansmith ack
15:58:34 openstackgerrit Surya Seetharaman proposed openstack/nova master: Make association_refresh configurable https://review.openstack.org/565526
15:59:58 tssurya mriedem, efried: ^
16:00:17 efried ack
16:02:24 openstackgerrit Jay Pipes proposed openstack/nova master: add Consumer object https://review.openstack.org/565403
16:02:25 openstackgerrit Jay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer https://review.openstack.org/565404
16:02:26 openstackgerrit Jay Pipes proposed openstack/nova master: move consumer ensure to API layer https://review.openstack.org/565405
16:02:27 openstackgerrit Jay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id https://review.openstack.org/565406
16:02:28 openstackgerrit Jay Pipes proposed openstack/nova master: prep for consumer generation in API layer https://review.openstack.org/565407
16:02:29 openstackgerrit Jay Pipes proposed openstack/nova master: increment consumer generation when allocs change https://review.openstack.org/565513
16:06:04 efried tssurya: Reviewed. Looks good.
16:06:33 tssurya efried: thanks for the quick review,
16:06:42 efried sure thing. easy one :)
16:17:13 melwitt mriedem: thanks
16:18:50 mriedem melwitt: only thing left is cleaning up https://review.openstack.org/#/c/565367/
16:19:03 melwitt k, looking at it now
16:23:08 melwitt phew, lots o docs
16:23:21 melwitt thanks for pointing out all of those
16:28:47 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix shelving a paused instance https://review.openstack.org/541660
16:30:13 mriedem easy fix for a super latent problem ^
16:37:36 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Remove 'tools/releasenotes_tox.sh' https://review.openstack.org/534383
16:37:37 openstackgerrit Stephen Finucane proposed openstack/nova master: tox: Speed things up and document them https://review.openstack.org/534382
16:37:38 openstackgerrit Stephen Finucane proposed openstack/nova master: tox: Switch basepython to 3 https://review.openstack.org/565534
16:39:52 Swami Have a question on PCI-Passthrough. What is the right approach, should we create a neutron port with vnic_type=direct-physical and assign it to the VM or directly create a flavor with Alias and create a VM without specifying the network.
16:40:39 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove stale pip-missing-reqs tox test https://review.openstack.org/565535
16:42:45 openstackgerrit melanie witt proposed openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367
16:52:16 openstackgerrit melanie witt proposed openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367
16:53:13 mriedem melwitt: you were too fast
16:53:30 melwitt oops
17:08:29 openstackgerrit melanie witt proposed openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367
17:14:54 openstackgerrit Surya Seetharaman proposed openstack/nova master: Make association_refresh configurable https://review.openstack.org/565526
17:24:29 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642
17:24:30 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643
17:24:31 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644
17:24:32 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646
17:24:33 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
17:24:34 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
17:24:35 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674
17:24:36 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
17:24:37 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
17:24:38 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
17:25:47 melwitt mriedem: I noticed yesterday that it appears nova-next isn't running console proxy with SSL anymore. says "no cert file" but I see the cert stuff being set up in the devstack log. don't yet know what's wrong http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/devstacklog.txt.gz#_2018-05-01_15_14_45_231
17:27:52 mriedem hmm yeah http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/local.conf.txt.gz has NOVA_CONSOLE_PROXY_COMPUTE_TLS=True
17:28:33 mriedem does the config have to be in the cell?
17:28:39 mriedem if so, it's probably writing to the wrong nova config
17:29:02 mriedem [vnc] is configured properly here http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/etc/nova/nova.conf.txt.gz
17:29:12 mriedem and here http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/etc/nova/nova-cpu.conf.txt.gz
17:29:58 melwitt I would think the config doesn't have the be in the cell until my changes (and the above log I linked is on a change unrelated to my per cell proxy stuff)
17:29:59 mriedem melwitt: i wonder if it's something with using the queens UCA
17:30:01 mriedem libvirt might have changed
17:30:05 mriedem re: https://review.openstack.org/#/c/396186/7/lib/nova_plugins/functions-libvirt
17:30:42 melwitt maybe, guh I hope not. I have no clue how that stuff works
17:30:59 melwitt but this used to work and I can't think of anything that's changed
17:31:00 mriedem this is where that check happens http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/devstacklog.txt.gz#_2018-05-01_15_05_38_543
17:31:48 melwitt yeah, saw that too
17:33:53 melwitt "no cert file" sounds like it can't find the cert file. it deploys them to /etc/pki/libvirt-vnc/
17:34:28 melwitt oh wait, but the config says /etc/pki/nova-novnc/
17:35:11 mriedem looking in the libvirtd logs, i can see qemu being executed with "tls,x509verify=/etc/pki/libvirt-vnc"
17:36:46 melwitt [vnc]
17:36:46 melwitt okay, so I wonder if these are wrong in nova.conf? not sure
17:36:47 melwitt vencrypt_ca_certs = /etc/pki/nova-novnc/ca-cert.pem
17:37:14 mriedem don't know, but that wouldn't really explain how it was working before
17:37:27 mriedem probably need to ask stephenfin or danpb
17:37:38 melwitt yeah, true. I don't even know where teh nova-novnc dir comes from
17:37:58 mriedem it's created by devstack https://review.openstack.org/#/c/396186/7/lib/nova
17:38:57 melwitt fwiw, I was the one who got the devstack patch to actually run with SSL because there were minor things wrong in the devstack changes originally because things had changed over time (the series has been around forever)
17:39:31 melwitt because I didn't want to approve the patches until there was some demonstration of the proxy running with SSL enabled
17:43:40 melwitt wonder if the chown user:group needs to be updated again, that's what I had to change last time https://review.openstack.org/#/c/396186/7/lib/nova_plugins/functions-libvirt@156
17:44:06 mriedem that's what i was thinking,
17:44:11 mriedem because i had to deal with that for the multiattach stuff
17:45:11 mriedem melwitt: this https://github.com/openstack-dev/devstack/blob/master/stackrc#L630
17:45:46 mriedem could add a debug patch to make sure that libvirt-qemu exists
17:46:38 melwitt hm, yeah. this would be the qemu user and group. originally it was just qemu:qemu and I had to update it to libvirt-qemu:libvirt-qemu
17:50:18 mriedem melwitt: you should kick an intern to make sure this article from 2011 is up to date https://wiki.libvirt.org/page/VNCTLSSetup
17:51:19 melwitt heh, that's a good doc
17:54:03 openstackgerrit Aditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371
17:57:20 wolverineav stephenfin: mriedem - updated the review with patch addressing comments ^^ take a look when you get a chance :)

Earlier   Later