| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-22 | |||
| 13:29:48 | dr_gogeta86 | or some other service ? | |
| 13:44:15 | mriedem | dr_gogeta86: nova.conf | |
| 13:44:27 | dr_gogeta86 | doesn't fit | |
| 13:44:29 | mriedem | sean-k-mooney: is there anything bad that can happen if we are plugging an inactive port? | |
| 13:44:42 | dr_gogeta86 | i've replaced a strange value as address | |
| 13:44:50 | dr_gogeta86 | doesn't change in what i see with strace | |
| 13:45:18 | sean-k-mooney | mriedem: i am not sure. it would depend on the backend i think. | |
| 13:45:38 | mriedem | ovs | |
| 13:45:49 | mriedem | Successfully plugged vif VIFOpenVSwitch(active=False,address=fa:16:3e:2b:a9:39,bridge_name='br-int',has_traffic_filtering=True,id=b835ff51-0262-4237-ab52-7b08206bd099,network=Network(6c9a79d8-b508-4a97-a15d-bf4acee8b4a8),plugin='ovs',port_profile=VIFPortProfileOpenVSwitch,preserve_on_delete=False,vif_name='tapb835ff51-02') | |
| 13:45:52 | sean-k-mooney | mriedem: for linux bridge we are just using brctl to say when a netdev with this name appears attach it to this bridge | |
| 13:46:35 | sean-k-mooney | for ovs its similar but depends on if its hybridg plug true/false or vhost-user | |
| 13:46:40 | sean-k-mooney | vhost user is safe | |
| 13:47:15 | mriedem | this is the gate so not vhostuser | |
| 13:47:28 | mriedem | "ovs_hybrid_plug": false | |
| 13:47:29 | sean-k-mooney | hybridge-plug=false is the same as linux bridge. we are just adding a port to ovs for a netdev that currently does not exists. ovs will wire it up when it gets created | |
| 13:49:25 | sean-k-mooney | hybrid-plug=true is less safe. we are adding a veth pair to ovs and adding the other end to a linux bridge. traffic can flow form ovs to the linux bridge and back but it cant leave the linux bridge and enter the host. | |
| 13:49:27 | mriedem | ok was just trying to debug a gate failure with resize revert https://bugs.launchpad.net/nova/+bug/1788403 | |
| 13:49:27 | openstack | Launchpad bug 1788403 in OpenStack Compute (nova) "test_server_connectivity_cold_migration_revert randomly fails ssh check" [Medium,Confirmed] | |
| 13:49:41 | mriedem | this test does a cold migratoin and then reverts, | |
| 13:49:50 | mriedem | then tries to ssh into the guest which intermittently fails | |
| 13:50:07 | mriedem | but tempest asserts the floating IP is active before trying to ssh | |
| 13:51:16 | sean-k-mooney | mriedem: my gess would be mac learning gets messed up because of RARP packets emited by qemu as part of the migrate. i would have to check however. | |
| 13:51:37 | sean-k-mooney | pluging the deactivated port should not cause issues however | |
| 13:53:05 | mriedem | ok | |
| 13:53:25 | mriedem | we also aren't waiting for any vif unplug/plug events before saying the server is active, | |
| 13:53:30 | mriedem | but that seems to be on purpose | |
| 13:54:11 | mriedem | https://review.openstack.org/#/c/179228/ | |
| 13:55:16 | sean-k-mooney | did you repoduce this locally by the way. it would be interesting to see if the vm actully had an ip when it failed. i can try to repoduce after i test your live migration lifcycled event fixes if that would help | |
| 13:55:44 | mriedem | no this is something i've seen failing in the gate | |
| 13:56:58 | sean-k-mooney | the gate job was just default ovs live migration env | |
| 13:57:10 | mriedem | this isn't live migration | |
| 13:57:11 | mriedem | cold migrate | |
| 13:57:18 | mriedem | it's also totally intermittent | |
| 13:57:22 | sean-k-mooney | sorry yes that is what i ment | |
| 13:57:43 | sean-k-mooney | its just a default ovs multnode devstack install. | |
| 13:59:22 | dr_gogeta86 | mriedem, CONF = nova.conf.CONF what is this ? | |
| 13:59:33 | mriedem | it's a global | |
| 13:59:50 | mriedem | variable that contains the conf information | |
| 14:01:01 | sean-k-mooney | dr_gogeta86: its just creating an alias to the global config object that was populated when the config was read on start up as mriedem said | |
| 14:01:26 | dr_gogeta86 | i'm wandering why doesn't read the correct one | |
| 14:01:33 | dr_gogeta86 | i dunno which one at this point | |
| 14:03:19 | sean-k-mooney | dr_gogeta86: you can spcify the config to read with --config-file or --config-dir e.g. nova-manage --config-dir /etc/nova or nova-manage --config-file /etc/nova/nova.conf | |
| 14:03:31 | sean-k-mooney | dr_gogeta86: that should be the default behavior however | |
| 14:03:36 | sean-k-mooney | so you should not need too | |
| 14:05:20 | dr_gogeta86 | i need to | |
| 14:05:32 | dr_gogeta86 | cause if put --config-file /etc/nova/nova.conf | |
| 14:05:36 | dr_gogeta86 | works like a charm | |
| 14:06:03 | sean-k-mooney | let me check the code to see what the default path actully is | |
| 14:06:10 | dr_gogeta86 | tnx you | |
| 14:08:39 | s10 | Is there any way to specify custom logging config for nova-api running under uwsgi without using separate nova.conf for nova-api/nova-scheduler/nova-conductor? For simple nova-api we can pass --log-config-append option, but for uwsgi it seems not possible. | |
| 14:11:38 | openstackgerrit | Dan Smith proposed openstack/nova master: Make RecordWrapper record RequestContext and expose cell_uuid https://review.openstack.org/594572 | |
| 14:12:05 | dansmith | yikun_: Kevin_Zheng tssurya: fixed that one thing in the tests ^ | |
| 14:12:42 | dansmith | I have suddenly collected a "cells performance mafia" following | |
| 14:12:55 | tssurya | dansmith: :P (ack) | |
| 14:13:31 | dansmith | tssurya: I certainly don't want to insert myself in front of any of your patches, but if you are waiting on another set anyway, it might make sense to go at the end of this series | |
| 14:13:40 | dansmith | as you said | |
| 14:14:48 | tssurya | yea, looks like the last patches needed from gmann are not there yet and existing last one needs update, so I can go after you.. | |
| 14:15:12 | dansmith | okay | |
| 14:15:34 | tssurya | however I will wait for your initial patches so start geting merged before rebasing, since some are WIPs | |
| 14:15:34 | dr_gogeta86 | sean-k-mooney, no news | |
| 14:15:34 | dr_gogeta86 | ? | |
| 14:16:43 | tssurya | dr_gogeta86: the default path is /etc/nova/nova.conf, you need not have to put it explicitly | |
| 14:16:49 | dansmith | tssurya: ack, let's plan to have the cells meeting today so we can get melwitt and mriedem on board with getting all these merged quickly | |
| 14:17:00 | tssurya | dansmith: cool :) | |
| 14:17:05 | dr_gogeta86 | but doens't | |
| 14:18:03 | holser_ | thanks mriedem | |
| 14:18:56 | sean-k-mooney | tssurya: actully looking at the code i can find where we set the defult. for the wsgi scipt for the api and placement i found it but i dont see it anywayre for the rest of the agents or command files | |
| 14:23:55 | sean-k-mooney | dr_gogeta86: i assume this is a production deployment where you need a password to connect to the db and not devstack correct | |
| 14:25:41 | sean-k-mooney | dr_gogeta86: tssurya i think the reason nova-manage works for devstack deployments without --config-file is that devstack install allows access to the db from localhost without any password | |
| 14:33:04 | mriedem | but i plan on talking about cross cell migration in the cells meeting... | |
| 14:33:10 | mriedem | an infinitely more fun topic | |
| 14:41:48 | openstackgerrit | Dan Smith proposed openstack/nova master: Batch results per cell when doing cross-cell listing https://review.openstack.org/592698 | |
| 14:41:49 | openstackgerrit | Dan Smith proposed openstack/nova master: List instances from all cells explicitly https://review.openstack.org/593717 | |
| 14:41:50 | openstackgerrit | Dan Smith proposed openstack/nova master: Make instance_list perform per-cell batching https://review.openstack.org/593131 | |
| 14:41:51 | openstackgerrit | Dan Smith proposed openstack/nova master: Record cell success/failure/timeout in CrossCellLister https://review.openstack.org/594265 | |
| 14:41:52 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Optimize global marker re-lookup in multi_cell_list https://review.openstack.org/594577 | |
| 14:48:12 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Wait for network-vif-plugged on resize revert https://review.openstack.org/595069 | |
| 14:51:12 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Wait for network-vif-plugged on resize revert https://review.openstack.org/595069 | |
| 15:07:33 | dansmith | mriedem: I think we should be good on this now: https://review.openstack.org/#/c/591735/5 | |
| 15:10:45 | melwitt | . | |
| 15:19:20 | mriedem | ying wang doesn't think so | |
| 15:38:21 | openstackgerrit | sahid proposed openstack/nova master: compute: reset instance events https://review.openstack.org/420026 | |
| 15:55:23 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Add subtree filter for GET /resource_providers https://review.openstack.org/595236 | |
| 16:03:27 | openstackgerrit | Eric Fried proposed openstack/nova master: Compute: Handle reshaped provider trees https://review.openstack.org/576236 | |
| 16:03:34 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Placement: support mixing required traits with any traits https://review.openstack.org/565741 | |
| 16:09:21 | openstackgerrit | Eric Fried proposed openstack/nova master: [placement] Regex consts for placement schema https://review.openstack.org/591863 | |
| 16:09:22 | openstackgerrit | Eric Fried proposed openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927 | |
| 16:09:23 | openstackgerrit | Eric Fried proposed openstack/nova master: reshaper: Look up provider if not in inventories https://review.openstack.org/585033 | |
| 16:09:24 | openstackgerrit | Eric Fried proposed openstack/nova master: Make get_allocations_for_resource_provider raise https://review.openstack.org/584598 | |
| 16:09:25 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: Real get_allocs_for_consumer https://review.openstack.org/584599 | |
| 16:09:26 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: get_allocations_for_provider_tree https://review.openstack.org/584648 | |
| 16:09:27 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: _reshape helper, placement min bump https://review.openstack.org/585034 | |
| 16:09:28 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: update_from_provider_tree w/reshape https://review.openstack.org/585049 | |
| 16:09:29 | openstackgerrit | Eric Fried proposed openstack/nova master: Compute: Handle reshaped provider trees https://review.openstack.org/576236 | |
| 16:10:31 | efried | Does anyone have the power to remove a bogus review? viz https://review.openstack.org/#/c/584598/ | |
| 16:10:48 | efried | Not that we would necessarily let that stop us from merging the thing, but it's potentially distracting. | |
| 16:11:55 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/595243 | |
| 16:14:28 | melwitt | efried: not that I know of. I think if anyone could, it might be infra people | |
| 16:15:08 | efried_rollin | ight | |
| 16:17:33 | dansmith | trivial change'll fix it :) | |