| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-08 | |||
| 19:49:05 | kevinbenton | it's on the network node that runs the proxy | |
| 19:49:05 | mriedem | well i can't answer that | |
| 19:49:14 | kevinbenton | so not compute node | |
| 19:49:20 | dansmith | kevinbenton: so is it a single config for the whole deployment? | |
| 19:49:30 | kevinbenton | dansmith: essentially yes | |
| 19:49:37 | kevinbenton | dansmith: you could have different configs per network node | |
| 19:49:40 | dansmith | okay, not sure that's a good thing, although it does make this easier | |
| 19:49:44 | kevinbenton | but i don't think that would do anything useful | |
| 19:49:52 | mriedem | kevinbenton: where is that in http://logs.openstack.org/94/436094/33/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bed0e6a/logs/etc/neutron/neutron.conf.txt.gz ? | |
| 19:49:53 | kevinbenton | since networks nodes aren't correlated with cells | |
| 19:49:57 | mriedem | oh nvm http://logs.openstack.org/94/436094/33/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bed0e6a/logs/etc/neutron/metadata_agent.ini.txt.gz | |
| 19:50:01 | mriedem | nova_metadata_ip = 10.40.184.140 | |
| 19:50:05 | kevinbenton | mriedem: +1 | |
| 19:50:14 | dansmith | from a scale point of view I mean, but since we already effectively have it global I guess that's a totally different conversation | |
| 19:50:43 | mriedem | don't people already disable metadata service b/c of perf issues, and security | |
| 19:51:00 | mriedem | i know that's irrelevant | |
| 19:51:01 | dansmith | depends on their goals, but some do yeah | |
| 19:51:34 | dansmith | alright anyway, I'll fix that up | |
| 19:51:55 | openstackgerrit | Dan Smith proposed openstack/nova master: Remove old service version check for mitaka https://review.openstack.org/472395 | |
| 19:52:28 | mriedem | i suppose this meta bug has to be backported | |
| 19:52:58 | mriedem | although i'm only seeing it in the multi-cell devstack change | |
| 19:56:51 | kevinbenton | dansmith, mriedem: is there a way we can know which cell to send something to? | |
| 19:57:09 | kevinbenton | we could alter the proxy logic to route in some manner because it does do a port lookup to inject headers | |
| 19:57:18 | dansmith | mriedem: I don't think we need to backport it | |
| 19:57:25 | dansmith | kevinbenton: you don't and shouldn't know any of that | |
| 19:58:33 | kevinbenton | dansmith: ok, just checking if this is an issue we need to fix on our metadata proxy logic | |
| 19:59:16 | mriedem | i wonder how cells v1 people are doing the metadata service | |
| 19:59:17 | dansmith | kevinbenton: no it sounds like you're effectively forcing a global metadata server now, which makes a fix for this easy on our end | |
| 19:59:30 | dansmith | mriedem: not sure it works in cellsv1, or they just run it in a cell I'd imagine | |
| 19:59:44 | mriedem | i'll ask in ops | |
| 19:59:51 | kevinbenton | just put a load balancer in front of it that fans every request out to a cell until it doesn't get a 404 :) | |
| 20:02:28 | mriedem | you don't get a 404 here, | |
| 20:02:30 | mriedem | you get a 500 :( | |
| 20:02:41 | mriedem | we probably also need to fix that | |
| 20:02:56 | mriedem | http://logs.openstack.org/94/436094/33/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bed0e6a/logs/screen-n-api.txt.gz#_Jun_08_18_36_06_549748 | |
| 20:03:12 | mriedem | 10.1.0.11,10.40.184.140 "POST /openstack/2013-10-17/password HTTP/1.1" status: 500 len: 139 time: 0.0445061: InstanceNotFound: Instance 6cd26f78-9c8a-4b37-8c62-38ad003a095f could not be found. | |
| 20:03:53 | mriedem | i'll open a separate bug for that | |
| 20:04:03 | mriedem | which is backportable | |
| 20:05:58 | openstack | Launchpad bug 1696848 in OpenStack Compute (nova) "POST /openstack/2013-10-17/password returns 500 if instance is not found" [Medium,Triaged] | |
| 20:05:58 | mriedem | https://bugs.launchpad.net/nova/+bug/1696848 | |
| 20:08:11 | mriedem | claudiub: are you doing anything with this? https://blueprints.launchpad.net/nova/+spec/hyperv-sriov | |
| 20:08:16 | mriedem | else i'm going to defer it to queens | |
| 20:13:12 | mriedem | gdi, you know what's awesome, | |
| 20:13:31 | mriedem | overriding a method and tests start failing, you spend too much time trying to figure out why, | |
| 20:13:43 | mriedem | and realize you're not returning the results of the method you're now delegating too | |
| 20:13:45 | mriedem | *to | |
| 20:18:10 | openstackgerrit | Maciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_spice_channel https://review.openstack.org/470787 | |
| 20:18:11 | openstackgerrit | Maciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_memory_balloon https://review.openstack.org/470786 | |
| 20:18:11 | openstackgerrit | Maciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_video_device https://review.openstack.org/470783 | |
| 20:18:12 | openstackgerrit | Maciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_pci_devices https://review.openstack.org/470784 | |
| 20:18:12 | openstackgerrit | Maciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_watchdog_action https://review.openstack.org/470785 | |
| 20:18:56 | melwitt | does anyone know if it matter whether you give a sqlalchemy in_() call a list or a set? is there any benefit in a set vs a list? | |
| 20:19:01 | melwitt | *matters | |
| 20:25:39 | bauzas | melwitt: AFAIK, it just needs a container | |
| 20:26:33 | bauzas | ie. anything having an __in__ | |
| 20:26:45 | melwitt | bauzas: yeah, I know it accepts list, tuple, set. I was just curious if there was a benefit of any over others. I didn't see anything when I googled so far | |
| 20:27:15 | dansmith | isn't it generating a text query? | |
| 20:27:22 | dansmith | such that you need iter() not __in__ ? | |
| 20:27:37 | dansmith | in that case, I can't imagine it would matter, other than that a set avoids potential dupes | |
| 20:28:01 | bauzas | mmmm, http://docs.sqlalchemy.org/en/latest/core/sqlelement.html#sqlalchemy.sql.operators.ColumnOperators.in_ | |
| 20:28:09 | melwitt | yeah, that would make sense why there no mention about it mattering | |
| 20:28:14 | bauzas | it just says lists and tuples | |
| 20:28:18 | melwitt | *there's. guh, I can't type today | |
| 20:28:39 | bauzas | honestly, I think it's not a big deal | |
| 20:28:58 | melwitt | I assumed it's not a big deal, I was just curious about it | |
| 20:30:05 | bauzas | melwitt: if it accepts sets, then cool, as dansmith said | |
| 20:30:30 | melwitt | I know. I know what it accepts. I was curious how it worked and decided to ask if anyone happened to know | |
| 20:30:32 | bauzas | for dupes I mean | |
| 20:31:50 | bauzas | unrelated, could someone tell me the cases where when we have a live-migration, the instance can be shutoff ? | |
| 20:32:29 | bauzas | AFAIK, when it's timing out, it's not shut the instance off, right? | |
| 20:35:37 | dansmith | bauzas: afaik live migration only works on a running instance, and if you want to live migrate it you need to start it paused to do so | |
| 20:36:41 | openstackgerrit | Dan Smith proposed openstack/nova master: Fix lookup of instance mapping in metadata set-password https://review.openstack.org/472405 | |
| 20:37:04 | bauzas | dansmith: sorry, the question is about the output of a live-migration | |
| 20:37:29 | openstack | Launchpad bug 1696025 in OpenStack Compute (nova) "live-migration sometimes failed" [Undecided,New] | |
| 20:37:29 | bauzas | dansmith: I'm trying to understand https://bugs.launchpad.net/nova/+bug/1696025 | |
| 20:37:35 | bauzas | but I'll ask for more details | |
| 20:44:18 | openstackgerrit | Merged openstack/nova master: Add PCIWeigher https://review.openstack.org/379524 | |
| 20:46:51 | openstackgerrit | Steve Noyes proposed openstack/nova master: WIP: update live migration to use v3 cinder api https://review.openstack.org/463987 | |
| 20:56:43 | melwitt | this is weird, api sample test fail because of the v2.1 in the url? http://logs.openstack.org/59/457859/17/check/gate-nova-tox-functional-py35-ubuntu-xenial/6e6bd37/testr_results.html.gz | |
| 20:57:09 | mriedem | dansmith: you can live migrate a paused instance | |
| 20:57:15 | mriedem | we have a tempest test for it | |
| 20:57:37 | dansmith | mriedem: right | |
| 20:58:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Handle uuids in os-hypervisors API https://review.openstack.org/471899 | |
| 20:58:55 | sdague | melwitt: that actually looks like the template didn't fill for some reason | |
| 20:59:42 | melwitt | sdague: oh, I see. so it shouldn't be showing the placeholders | |
| 20:59:57 | sdague | no, or they should match | |
| 21:00:05 | openstackgerrit | Corey Bryant proposed openstack/nova master: Pass CLI options to nova-placement-api https://review.openstack.org/472378 | |
| 21:00:10 | mriedem | meeting time | |
| 21:00:44 | melwitt | yeah. everything looks to match, except possibly the "v2.1" part. but that parameter is called "versioned_compute_endpoint" so it seems like it would encompass the v2.1 | |
| 21:02:31 | openstackgerrit | melanie witt proposed openstack/nova master: Add FloatingIPList.get_count_by_project() https://review.openstack.org/446247 | |
| 21:02:31 | openstackgerrit | melanie witt proposed openstack/nova master: Add FixedIPList.get_count_by_project() https://review.openstack.org/446246 | |
| 21:02:32 | openstackgerrit | melanie witt proposed openstack/nova master: Add SecurityGroupList.get_counts() https://review.openstack.org/446245 | |
| 21:02:32 | openstackgerrit | melanie witt proposed openstack/nova master: Add get_count_by_vm_state() to InstanceList object https://review.openstack.org/446244 | |
| 21:02:33 | openstackgerrit | melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242 | |
| 21:02:35 | openstackgerrit | melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | |
| 21:02:37 | openstackgerrit | melanie witt proposed openstack/nova master: Count networks to check quota https://review.openstack.org/446241 | |
| 21:02:39 | openstackgerrit | melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521 | |
| 21:02:41 | openstackgerrit | melanie witt proposed openstack/nova master: Add InstanceGroup._remove_members_in_db https://review.openstack.org/457858 | |
| 21:02:44 | openstackgerrit | melanie witt proposed openstack/nova master: Count server group members to check quota https://review.openstack.org/457859 | |
| 21:02:49 | openstackgerrit | melanie witt proposed openstack/nova master: Count security groups to check quota https://review.openstack.org/457860 | |