| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-23 | |||
| 17:11:38 | mdbooth | Well, not _find_destination, but some other scheduler call. | |
| 17:11:56 | mriedem | dims: notice what? | |
| 17:12:06 | mriedem | dims: if it's non-voting i'm not noticing it right now | |
| 17:14:42 | bauzas | mriedem: https://github.com/openstack/nova/blob/c9eb9530314d047f5013941ebcfd5ef0192a9dc3/nova/compute/api.py#L3773 | |
| 17:16:08 | dims | mriedem : ack. it's green that's all :) | |
| 17:16:35 | mriedem | dims: oh you were looking for some back patting, i see | |
| 17:16:51 | mriedem | #success dims got gate-tempest-dsvm-py35-ubuntu-xenial-nv green | |
| 17:16:53 | openstackstatus | mriedem: Added success to Success page | |
| 17:17:04 | dims | LOL, just wanted to be sure i won't get yelled at if you notice it later in the week (when it goes RED) :) | |
| 17:18:09 | mriedem | i won't notice | |
| 17:18:35 | mriedem | dims: i'm on vacation so only water temperature and some other key things are high priority for me right now | |
| 17:18:39 | dims | fair enough | |
| 17:19:11 | mriedem | i don't have to officially care about py35 until march | |
| 17:19:12 | mriedem | :) | |
| 17:19:22 | dims | ack mriedem : have fun! | |
| 17:26:34 | openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | |
| 17:27:50 | jaosorior | gibi: you around? | |
| 17:28:07 | openstackgerrit | Stephen Finucane proposed openstack/nova: scheduler: Don't modify RequestSpec.numa_topology https://review.openstack.org/419479 | |
| 17:30:53 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [CLI] Fix token auth type https://review.openstack.org/424256 | |
| 17:31:51 | andreykurilin | mriedem: have a nice vacation! I'll try to ping some other cores to merge fixes when it will be ready and propose a release | |
| 17:33:13 | mriedem | ack | |
| 17:33:21 | mriedem | andreykurilin: thanks for being on top of it | |
| 17:33:41 | sfinucan | dansmith: Added tests to https://review.openstack.org/419479. Looking at the cells limit patch again now | |
| 17:45:07 | openstackgerrit | Stephen Finucane proposed openstack/nova: objects: add new field cpuset_reserved in NUMACell https://review.openstack.org/400874 | |
| 17:45:15 | mdbooth | Can anybody tell me what the 'limits' in the host object returned by scheduler_client.select_destinations() means? | |
| 17:45:19 | mdbooth | bauzas: ^^^ ? | |
| 17:45:52 | bauzas | mdbooth: it's what the claim uses for verifying the limit | |
| 17:46:06 | mdbooth | What's its scope? | |
| 17:46:23 | bauzas | mdbooth: for example, see the CoreFilter, it's telling what the max amount of the resouce is given the allocation ratio | |
| 17:46:31 | mdbooth | i.e. can it race with the clam? | |
| 17:47:05 | mdbooth | Ok, so it's essentially a constant? It doesn't vary as resources are used. | |
| 17:48:28 | mdbooth | I'm looking at a patch where conductor fetches this limits value and passes it to compute, which then uses it to create a claim | |
| 17:49:03 | mdbooth | IIUC, what you describe means there's no race there, as the value can't change in the presence of other placements. | |
| 17:50:09 | mdbooth | I assumed that was the case because in claim: free = limit - used | |
| 17:50:23 | mdbooth | where used is dynamic | |
| 17:50:32 | dansmith | sfinucan: re-reviewed, thanks | |
| 17:52:34 | andreykurilin | dims: I do not what had happened, but rally-nova-py3 jobs is ok now:) | |
| 17:53:02 | mlavalle | mriedem: is it still the plan in Ocata to have the scheduler use the data in resource providers and inventories for instances placement in host? | |
| 17:53:03 | andreykurilin | *I do not know | |
| 17:54:16 | edleafe | mlavalle: yes. The scheduler will get a pared-down list of hosts from placement that match the basic resources in the request | |
| 17:54:33 | edleafe | mlavalle: it will then use its other filters to select, as it currently does | |
| 17:55:47 | mriedem | mlavalle: yes | |
| 17:56:16 | openstackgerrit | John Garbutt proposed openstack/nova: WIP Support cinder v3 API https://review.openstack.org/420201 | |
| 17:58:14 | mlavalle | edleafe, mriedem: so, given that the patches that I wrote for resource providers and routed networks are merging, i can remove the first bullet here, right: http://docs.openstack.org/newton/networking-guide/config-routed-networks.html#limitations? | |
| 17:58:56 | edleafe | mlavalle: no | |
| 17:59:18 | edleafe | mlavalle: I guess I should have specified what I meant by "basic" :) | |
| 17:59:32 | edleafe | mlavalle: just disk, RAM, and VCPU for Ocata | |
| 18:00:19 | mriedem | mlavalle: what ed said. this is also going to be a pre-req for scheduling with routed networks https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/prep-for-network-aware-scheduling-ocata | |
| 18:00:34 | mriedem | mlavalle: and it's looking like at this point that series won't be happening in ocata | |
| 18:01:32 | mlavalle | edleafe, mriedem: so IPV4_ADDRESS inventories are postponed to Pike? | |
| 18:02:23 | mlavalle | as far as being used by the scheduler | |
| 18:02:24 | edleafe | mlavalle: They can be reported and tracked in Ocata. The scheduler won't be able to do anything with them, though | |
| 18:02:52 | mlavalle | edleafe, mriedem: thanks for the update! | |
| 18:03:11 | edleafe | mlavalle: IOW, the placement service will be able to handle it. The scheduler won't be using that information, yet' | |
| 18:03:14 | mriedem | mlavalle: yeah you definitely want them reporting in as soon as possible | |
| 18:03:44 | mriedem | so that when the scheduler can handle it, the data is already there | |
| 18:04:18 | dims | andreykurilin : we must have fixed some stuff in nova while we got the dsvm+tempest working :) | |
| 18:04:29 | dims | andreykurilin : thanks for the good news :) | |
| 18:04:35 | mlavalle | mriedem, edleafe: ok, on our side the functionality is complete. the last patchset remaining has already 2 +2's, so it should merge today. I even update the documentation | |
| 18:05:01 | andreykurilin | dims: there is only one failed scenario. will look at it more close a bit later | |
| 18:05:17 | dims | ack andreykurilin : thanks | |
| 18:05:18 | mlavalle | mriedem, edleafe: I will focus next on creating a tempest test to exercise it in the gate, so we all get feedback | |
| 18:05:35 | kbaegis | Hi all. I'm having some issues with devstack and nova. From horizon I'm getting a ton of 400s, but from nova/openstack cli everything works fine | |
| 18:06:50 | diana_clarke | kbaegis: You might be hitting this. https://bugs.launchpad.net/python-novaclient/+bug/1658148 | |
| 18:06:50 | openstack | Launchpad bug 1654183 in python-novaclient "duplicate for #1658148 Token based authentication in Client class does not work" [High,In progress] - Assigned to Andrey Kurilin (andreykurilin) | |
| 18:10:07 | kbaegis | Thanks guys | |
| 18:16:55 | jbryce | is there a practical upper limit to how many cells a nova deployment might have? this is for a nova cells related question in the user survey | |
| 18:17:26 | openstackgerrit | Dan Smith proposed openstack/nova: Multicell support for instance listing https://review.openstack.org/396775 | |
| 18:17:26 | openstackgerrit | Dan Smith proposed openstack/nova: Make api_samples tests use simple cell environment https://review.openstack.org/410473 | |
| 18:17:27 | openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | |
| 18:17:48 | dansmith | jbryce: assume you're talking about cellsv1? | |
| 18:19:11 | jbryce | dansmith: i’d be curious about both v1 and v2 as this round of updates will probably live into the time when we start getting v2 deployments cataloged | |
| 18:19:34 | dansmith | jbryce: well, as of this moment cellsv2 can't have multiple cells | |
| 18:19:57 | dansmith | jbryce: for cellsv1 you'd probably need to ask someone that actually runs it to see how many they have and/or if they've seen any such practical limits | |
| 18:20:10 | dansmith | jbryce: there is no hard-coded limit or anything like that though | |
| 18:21:16 | jbryce | dansmith: thanks | |
| 18:22:07 | openstackgerrit | Claudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization https://review.openstack.org/398509 | |
| 18:22:08 | openstackgerrit | Claudiu Belu proposed openstack/nova: Adds CPUFeaturesField to ImageMetaProps https://review.openstack.org/420960 | |
| 19:05:09 | amrith | hi andreykurilin ... any luck? | |
| 19:22:30 | oomichi | slaweq_: thanks, now LGTM | |
| 19:27:12 | openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Deny invalid flavor extraspec for cpu_policy https://review.openstack.org/406377 | |
| 19:27:38 | oomichi | gmann: thanks, applied your comments | |
| 19:35:52 | openstackgerrit | Chris Dent proposed openstack/nova: Avoid redundant call to update_resource_stats from RT https://review.openstack.org/424305 | |
| 19:36:28 | cdent | bauzas, edleafe ^ is a potential minor performance fix in the resource tracker | |
| 19:37:11 | cdent | not sure how much it matters but seemed low hanging fruit in the bug list: https://bugs.launchpad.net/nova/+bug/1658629 | |
| 19:37:11 | openstack | Launchpad bug 1658629 in OpenStack Compute (nova) "Resource tracker causes update of compute_nodes table every minute" [Medium,In progress] - Assigned to Chris Dent (cdent) | |
| 19:37:16 | cdent | and with that, good night | |
| 19:42:06 | openstackgerrit | Stephen Gordon proposed openstack/nova: libvirt: avoid generating script with empty path https://review.openstack.org/411936 | |
| 19:46:47 | openstackgerrit | STEW TY proposed openstack/nova: Transform instance.unrescue notifications https://review.openstack.org/388275 | |
| 20:06:24 | slaweq | oomichi: thx for +2 :) | |
| 20:15:56 | openstackgerrit | STEW TY proposed openstack/nova: Transform instance.rescue notifications https://review.openstack.org/385644 | |
| 20:41:10 | openstackgerrit | Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285 | |
| 20:41:11 | openstackgerrit | Ildiko Vancsa proposed openstack/nova: Remove check_attach https://review.openstack.org/335358 | |
| 20:42:30 | openstackgerrit | Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285 | |
| 20:42:31 | openstackgerrit | Ildiko Vancsa proposed openstack/nova: Remove check_attach https://review.openstack.org/335358 | |
| 20:52:51 | amrith | andreykurilin, please let me know if I can help. | |
| 21:17:11 | andreykurilin | amrith: I need just several minutes for latest version of patches. they should help | |
| 21:18:09 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication https://review.openstack.org/419441 | |
| 21:26:38 | moshele | jaypipes: hi, can you review this simple fix https://review.openstack.org/#/c/423608/ | |
| 21:51:10 | openstackgerrit | Ed Leafe proposed openstack/nova: placement: adds REST API for nested providers https://review.openstack.org/384807 | |