| *** dklyle has quit IRC | 00:03 | |
| *** brinzhang has joined #openstack-nova | 00:17 | |
| *** _ix has joined #openstack-nova | 00:17 | |
| *** zzzeek has quit IRC | 00:31 | |
| *** zzzeek has joined #openstack-nova | 00:31 | |
| *** mriedem has joined #openstack-nova | 00:44 | |
| mriedem | 你好 | 00:44 |
|---|---|---|
| mriedem | smcginnis: if you're around, we need stable cores for https://review.openstack.org/#/c/591872/ RC2 | 00:47 |
| *** alex_xu has joined #openstack-nova | 00:47 | |
| mriedem | anyone know if jichen is going to clean this up? https://review.openstack.org/#/c/533125/ | 00:48 |
| mriedem | efried: do we need to just update it ourselves? | 00:48 |
| openstackgerrit | Merged openstack/nova master: Fix service list for disabled compute using MC driver https://review.openstack.org/583936 | 00:51 |
| openstackgerrit | Merged openstack/nova master: Revisons on notifications doc https://review.openstack.org/591621 | 00:51 |
| openstackgerrit | Merged openstack/nova master: use static pages for mitaka and newton release notes https://review.openstack.org/592150 | 00:51 |
| mriedem | tonyb: i guess you'd be a more appropriately time-zoned stable core for https://review.openstack.org/#/c/591872/ for RC2 | 00:52 |
| mriedem | Kevin_Zheng: want to start backporting https://review.openstack.org/#/c/583936/ ? it needs to go to rocky, queens and pike | 00:54 |
| mriedem | we'll hold it out until we've released rocky though | 00:54 |
| tonyb | mriedem: on it | 00:55 |
| *** dave-mccowan has quit IRC | 01:02 | |
| *** imacdonn has quit IRC | 01:04 | |
| Kevin_Zheng | sure | 01:04 |
| *** dave-mccowan has joined #openstack-nova | 01:04 | |
| tonyb | mriedem: I'll ping lyarwood | 01:05 |
| mriedem | tonyb: thanks but he's probably done for the day | 01:06 |
| tonyb | mriedem: I can single core approve it | 01:07 |
| tonyb | mriedem: given you're a stable core etc | 01:08 |
| dansmith | mriedem: tonyb I got it | 01:09 |
| tonyb | dansmith to the rescue! | 01:10 |
| *** r-daneel has joined #openstack-nova | 01:13 | |
| *** zzzeek has quit IRC | 01:14 | |
| *** zzzeek has joined #openstack-nova | 01:16 | |
| *** imacdonn has joined #openstack-nova | 01:16 | |
| *** mhen has quit IRC | 01:17 | |
| mriedem | cool; i've emailed jichen about the zvm docs since those are the only things remaining on the RC2 todo list | 01:24 |
| *** mhen has joined #openstack-nova | 01:29 | |
| *** r-daneel has quit IRC | 01:29 | |
| *** mrsoul has quit IRC | 01:31 | |
| *** sapd1 has joined #openstack-nova | 01:39 | |
| *** donghm has joined #openstack-nova | 01:39 | |
| *** hongbin_ has joined #openstack-nova | 01:41 | |
| mriedem | efried: edmondsw: frankly i care very little if the powervm out of tree driver supports a feature, i don't plan on adding flags to nova to support out of tree features, re: https://review.openstack.org/#/c/591898/ | 01:41 |
| *** Dinesh_Bhor has joined #openstack-nova | 01:44 | |
| *** donghm has left #openstack-nova | 01:44 | |
| *** _ix has quit IRC | 01:53 | |
| openstackgerrit | fupingxie proposed openstack/nova master: Support list for alias in pci section in nova.conf https://review.openstack.org/592243 | 02:10 |
| *** tinwood has quit IRC | 02:10 | |
| *** tinwood has joined #openstack-nova | 02:11 | |
| *** lei-zh has joined #openstack-nova | 02:39 | |
| *** markmc has quit IRC | 02:42 | |
| *** jpena|off has quit IRC | 02:45 | |
| openstackgerrit | Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252 | 02:48 |
| *** psachin has joined #openstack-nova | 02:48 | |
| *** rosey99_ has quit IRC | 03:06 | |
| *** hongbin_ has quit IRC | 03:29 | |
| pvc | @edmondsw im usign ceph rbd | 03:32 |
| *** rosey99 has joined #openstack-nova | 03:33 | |
| *** Dinesh_Bhor has quit IRC | 03:34 | |
| *** udesale has joined #openstack-nova | 03:36 | |
| *** pvc has quit IRC | 03:39 | |
| *** Bhujay has quit IRC | 03:43 | |
| *** lei-zh has quit IRC | 03:48 | |
| *** pvc has joined #openstack-nova | 03:52 | |
| pvc | hi | 03:52 |
| pvc | anyone using ceph | 03:52 |
| *** gbarros has quit IRC | 03:52 | |
| *** erlon has quit IRC | 03:53 | |
| *** takashin has left #openstack-nova | 04:01 | |
| *** rosey99 has left #openstack-nova | 04:02 | |
| *** dave-mccowan has quit IRC | 04:04 | |
| *** abhishekk has joined #openstack-nova | 04:05 | |
| *** tbachman has quit IRC | 04:10 | |
| *** tbachman has joined #openstack-nova | 04:29 | |
| *** gcb_ has quit IRC | 04:31 | |
| *** janki has joined #openstack-nova | 04:34 | |
| *** Bhujay has joined #openstack-nova | 04:38 | |
| *** tbachman has quit IRC | 04:46 | |
| *** tetsuro has joined #openstack-nova | 04:47 | |
| *** gcb_ has joined #openstack-nova | 04:47 | |
| *** tbachman has joined #openstack-nova | 04:51 | |
| *** Bhujay has quit IRC | 05:07 | |
| *** Bhujay has joined #openstack-nova | 05:08 | |
| openstackgerrit | Merged openstack/nova stable/rocky: placement: ignore policy scope check failures if not enforcing scope https://review.openstack.org/591872 | 05:16 |
| *** Dinesh_Bhor has joined #openstack-nova | 05:16 | |
| *** tbachman has quit IRC | 05:20 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Explicitly fail if trying to attach SR-IOV port https://review.openstack.org/591898 | 05:20 |
| *** pcaruana has quit IRC | 05:26 | |
| *** ratailor has joined #openstack-nova | 05:27 | |
| *** lei-zh has joined #openstack-nova | 05:27 | |
| *** tetsuro has quit IRC | 05:39 | |
| openstackgerrit | Merged openstack/nova master: Imported Translations from Zanata https://review.openstack.org/591129 | 05:39 |
| *** tetsuro has joined #openstack-nova | 05:42 | |
| *** pvc has quit IRC | 05:44 | |
| *** brinzh has joined #openstack-nova | 05:48 | |
| *** brinzhang has quit IRC | 05:52 | |
| *** dpawlik has joined #openstack-nova | 06:01 | |
| *** Bhujay has quit IRC | 06:02 | |
| *** Bhujay has joined #openstack-nova | 06:03 | |
| openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter https://review.openstack.org/591976 | 06:07 |
| *** skatsaounis has joined #openstack-nova | 06:12 | |
| *** Dinesh_Bhor has quit IRC | 06:23 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:24 | |
| openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter https://review.openstack.org/591976 | 06:37 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Force refresh instance info_cache during heal https://review.openstack.org/591607 | 06:40 |
| *** ratailor has quit IRC | 06:43 | |
| *** adrianc has joined #openstack-nova | 06:43 | |
| *** hoonetorg has quit IRC | 06:45 | |
| *** ratailor has joined #openstack-nova | 06:47 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Make monkey patch work in uWSGI mode https://review.openstack.org/592285 | 06:48 |
| *** pcaruana has joined #openstack-nova | 06:48 | |
| openstackgerrit | Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252 | 06:51 |
| *** slaweq_ has joined #openstack-nova | 06:51 | |
| *** hoonetorg has joined #openstack-nova | 06:57 | |
| *** ispp has joined #openstack-nova | 06:59 | |
| *** slaweq_ has quit IRC | 07:00 | |
| openstackgerrit | huanhongda proposed openstack/nova-specs master: Return "deleted" and "estimated_reclaim" time of instance for admin https://review.openstack.org/581638 | 07:01 |
| *** adrianc has quit IRC | 07:02 | |
| openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Support deleting data volume when destroy instance https://review.openstack.org/580336 | 07:03 |
| openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Add support specify volume type when boot instance https://review.openstack.org/579520 | 07:04 |
| *** slaweq_ has joined #openstack-nova | 07:04 | |
| *** rha has joined #openstack-nova | 07:04 | |
| *** tetsuro has quit IRC | 07:14 | |
| *** tetsuro has joined #openstack-nova | 07:16 | |
| *** adrianc has joined #openstack-nova | 07:17 | |
| *** tetsuro has quit IRC | 07:17 | |
| *** tetsuro has joined #openstack-nova | 07:17 | |
| *** adrianc_ has joined #openstack-nova | 07:18 | |
| *** rcernin has quit IRC | 07:20 | |
| *** ispp has quit IRC | 07:22 | |
| *** adrianc has quit IRC | 07:23 | |
| *** lei-zh has quit IRC | 07:24 | |
| *** lei-zh has joined #openstack-nova | 07:24 | |
| *** tikitavi has joined #openstack-nova | 07:26 | |
| *** jpena has joined #openstack-nova | 07:31 | |
| *** tetsuro has quit IRC | 07:31 | |
| *** tetsuro has joined #openstack-nova | 07:33 | |
| *** tetsuro has quit IRC | 07:37 | |
| *** tetsuro has joined #openstack-nova | 07:38 | |
| *** adrianc__ has joined #openstack-nova | 07:40 | |
| *** adrianc_ has quit IRC | 07:43 | |
| *** adrianc__ has quit IRC | 07:43 | |
| *** sahid has joined #openstack-nova | 07:44 | |
| *** adrianc has joined #openstack-nova | 07:44 | |
| *** sayalilunkad has joined #openstack-nova | 07:51 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova stable/rocky: Fix service list for disabled compute using MC driver https://review.openstack.org/592310 | 08:01 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova stable/queens: Fix service list for disabled compute using MC driver https://review.openstack.org/592311 | 08:01 |
| *** tssurya has joined #openstack-nova | 08:04 | |
| mriedem | tssurya: o | 08:05 |
| mriedem | o/ | 08:05 |
| mriedem | i've added a couple of things to the cells meeting agenda if you want to take a peek and start lending brain power | 08:05 |
| tssurya | mriedem: \o/ | 08:05 |
| mriedem | cross cell migration, | 08:05 |
| mriedem | and down cell | 08:05 |
| mriedem | our operators want a config option to not return any VMs from nova list if the tenant has any in a down cell | 08:06 |
| tssurya | mriedem: oh okay | 08:06 |
| mriedem | to avoid confusing users that thought they had like 3 VMs but now only list 2 | 08:06 |
| mriedem | ^ would be pre-microversion | 08:06 |
| mriedem | b/c with the microversion we can list the down cell VMs with the UNKNOWN status | 08:06 |
| *** adrianc has quit IRC | 08:06 | |
| tssurya | right so we need to change this now nad backport also I guess to Queens ? | 08:07 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Make monkey patch work in uWSGI mode https://review.openstack.org/592285 | 08:07 |
| tssurya | before we start merging the UNKNOWN stuff. | 08:07 |
| mriedem | tssurya: not sure about backports, but maybe if it defaults to current behavior | 08:07 |
| tssurya | mriedem: the previous behaviour was 500 for down cells, do you suggest the config option to return an empty list if the user has stuff in the down cell then ? | 08:08 |
| tssurya | instead of skipping | 08:08 |
| tssurya | that cell | 08:09 |
| mriedem | originally the behavior was 500 right? | 08:09 |
| tssurya | yes | 08:09 |
| mriedem | you fixed that to skip down cells | 08:09 |
| tssurya | yes | 08:09 |
| mriedem | so yes, they want an option to say don't skip, but just return an empty list | 08:09 |
| mriedem | well, | 08:09 |
| mriedem | if the project has any instances in a down cell | 08:09 |
| tssurya | okay, | 08:09 |
| mriedem | i think it would only apply to non-admin instance list; an admin doing nova-list --all-tenants wouldn't apply here | 08:10 |
| openstackgerrit | fupingxie proposed openstack/nova master: Support list for alias in pci section in nova.conf https://review.openstack.org/592243 | 08:10 |
| tssurya | ack, | 08:10 |
| *** sean-k-mooney[m] has quit IRC | 08:11 | |
| tssurya | mriedem: in the agenda you say "show is OK", well show wouldn't work at all before the new microversion, would just be a 500 | 08:12 |
| tssurya | that's okay right ? | 08:13 |
| mriedem | oh yeah, right | 08:13 |
| mriedem | yeah | 08:13 |
| mriedem | nova CLI will do the right thing once you're using latest novaclient and server, | 08:13 |
| mriedem | in that it will request the latest microversion which will get users the UNKNOWN status behavior | 08:13 |
| tssurya | yes, speaking of which I am working on that right now: does the new microversion which skips certain keys need the api_samples functional tests ? or is it enough I clearly specify this in the api-ref docs ? | 08:15 |
| mriedem | would you have a sample of what this looks like in the api-ref docs? | 08:17 |
| mriedem | i think we probably should | 08:17 |
| mriedem | and those are generated from the api-samples functional tests | 08:17 |
| tssurya | mriedem: yea, okay it woud surely be nice to have sample jsons of what a response from down cell versus up cell looks like | 08:18 |
| mriedem | yeah | 08:18 |
| tssurya | okay then will add those tests too, but before that will do this pre-microversion thing config | 08:18 |
| tssurya | and ping you bacl | 08:18 |
| tssurya | back* | 08:18 |
| mriedem | ok cool | 08:20 |
| mriedem | not sure if this should be a bug or what | 08:20 |
| mriedem | that's why i added it to the meeting agenda | 08:20 |
| tssurya | right, we kind of skipped yesterday's meeting. so maybe we can discuss it with dansmith when he wkaes up ? | 08:21 |
| *** tikitavi has quit IRC | 08:21 | |
| *** alexchadin has joined #openstack-nova | 08:22 | |
| *** tetsuro has quit IRC | 08:24 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove ChanceScheduler https://review.openstack.org/576384 | 08:25 |
| *** davidsha has joined #openstack-nova | 08:25 | |
| *** Dinesh_Bhor has quit IRC | 08:26 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:28 | |
| sean-k-mooney | mriedem: o/ | 08:28 |
| sean-k-mooney | mriedem: i tested the live migration between ovs with iptables and ovs with connectracker hosts and you multi port binding changes work well for that case. | 08:29 |
| mriedem | ho | 08:30 |
| mriedem | *hot | 08:30 |
| sean-k-mooney | mriedem: i need to still test kernel ovs to ovs-dpdk an ovs to ovn but atleast the usecase called out in the spec is good | 08:30 |
| mriedem | Kevin_Zheng: yikun: your cells perf email mentions needing to monkey_patch eventlet in wsgi mode, but didn't mention you needed to monkey patch the time module, right? | 08:30 |
| *** rpittau has joined #openstack-nova | 08:30 | |
| Kevin_Zheng | Ah, yes, I forget that part | 08:31 |
| mriedem | https://github.com/openstack/oslo.db/blob/9c6695969f6502f52bc86f22af0758715fdc57c0/oslo_db/sqlalchemy/engines.py#L51 | 08:31 |
| mriedem | yeah | 08:31 |
| Kevin_Zheng | yikun could you amend that mail to show why the time module matters? | 08:31 |
| mriedem | i think you also only mention that all 10 cells are using a single DB at the end | 08:32 |
| yikun | yeah, sure | 08:32 |
| mriedem | probably good to mention that early and why | 08:32 |
| *** adrianc has joined #openstack-nova | 08:32 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove ChanceScheduler https://review.openstack.org/576384 | 08:34 |
| Kevin_Zheng | OK, I will be more careful in the future | 08:34 |
| sean-k-mooney | mriedem: what is the importance of time module in this case beyond causeing eventlet to yeild execution of the current green tread? | 08:35 |
| mriedem | sean-k-mooney: we iterate the cells with greenthreads, | 08:36 |
| sapd1 | sean-k-mooney: melwitt hi. I have a solution for octavia to use SR-IOV port. this is a workaround with custom in octavia. | 08:36 |
| mriedem | but without monkey_patching the time module, we aren't actually iterating the cells in parallel | 08:36 |
| mriedem | sapd1: how so? | 08:37 |
| sean-k-mooney | mriedem: ah ok. well even with monkey patching its technicall concurent not parralel but i get the point | 08:37 |
| mriedem | sapd1: see https://review.openstack.org/#/c/591898/ | 08:37 |
| sean-k-mooney | sapd1: do you have a review available or a paste of the patch we can review? | 08:38 |
| sapd1 | mriedem: I launch amphora instance using SR-IOV network instead of plug SR-IOV port to amphora instance. So I can launch amphora sucess. | 08:38 |
| sapd1 | s/sucess/success/ | 08:38 |
| mriedem | i don't know what an amphora instance is | 08:38 |
| sean-k-mooney | mriedem: its a specifc load balancer impmentation | 08:38 |
| sapd1 | mriedem: amphora instance is a instance running haproxy inside | 08:39 |
| sapd1 | s/a/an | 08:39 |
| *** vivsoni has joined #openstack-nova | 08:40 | |
| sean-k-mooney | mriedem: there were a few specs in the past to actully make attach work for sriov. its non trival but doable in most cases | 08:40 |
| sean-k-mooney | mriedem: we are missing the resouce tracker device claim and in future the placemnet allocation update to make it work correctly | 08:41 |
| sean-k-mooney | mriedem: as part of the RT claim we also need to update the neutron port binding with pci adress info but thats a detail. | 08:41 |
| sean-k-mooney | mriedem: sapd1 would like to resubmit the spec for stien so octavia can support loadblancers with sriov interfaces | 08:42 |
| tssurya | Keving_Zheng, yikun: I like this part of the cells perf email "so, now the queries are in parallel, but the whole thing still seems serial.", we have been having the same experience :) and thanks for the efforts | 08:42 |
| *** derekh has joined #openstack-nova | 08:43 | |
| Kevin_Zheng | tssurya, interesting, since you guys got 70 cells, how does it work? | 08:43 |
| tssurya | yea which is why we pushed for the patch of using inst_mappings to narrow it down to 2 or 3 cells per user | 08:44 |
| tssurya | so that way it works fine for us on the user end | 08:44 |
| tssurya | mainly because of the way we have our cells designed | 08:45 |
| *** mdbooth has joined #openstack-nova | 08:45 | |
| Kevin_Zheng | Hmm, that might not work for us, as I mentioned before, we are unable to limit our users to cells | 08:45 |
| tssurya | Kevin_Zheng: yea I remember your comment on that patch, but I guess we still need to find a way to make the data gathering part faster then | 08:46 |
| mriedem | sean-k-mooney: yeah i'm aware of the spec | 08:47 |
| mriedem | sean-k-mooney: i also added you to an sriov-bond spec that our product team is interested in | 08:48 |
| mriedem | tssurya: Kevin_Zheng: that reminds me, | 08:49 |
| mriedem | tssurya: the reason we can't restrict users to cells in our public cloud is because flavors are defined by cell (among other things) | 08:49 |
| mriedem | so if a user wants to use different flavors they can get instances in different cells | 08:49 |
| mriedem | tssurya: how is that handled at CERN? | 08:49 |
| sapd1 | sean-k-mooney: I think nova should care about SR-IOV instead of octavia. | 08:50 |
| tssurya | mriedem: we actually map projects to cells | 08:50 |
| tssurya | the default flavors are available everywhere and then the special flavors are available in certain cells which are then restricted for the tenants as well | 08:51 |
| sean-k-mooney | sapd1: well i think this is a nova issue rather then octavia. i cant unfortunetly make nova care about sriov but enough of our users do that we try to support it :) | 08:51 |
| tssurya | so only certain tenats end up in certain cells | 08:51 |
| sean-k-mooney | mriedem: oh i had not seen it but i will take a look | 08:51 |
| *** rha has quit IRC | 08:52 | |
| sean-k-mooney | sapd1: i started looking into this again last night. i think i know a slightly hacky way to fix this in the libvirt virt diriver. i am look to see if we can adress the missing logic in the compute manager instead as most of the code should not depend on the driver | 08:53 |
| mriedem | tssurya: yeah i know cern mapped projects to cells, | 08:54 |
| mriedem | oh ok so basic flavors are global, | 08:54 |
| mriedem | then per-cell flavors have tenant access for the tenants in those cells | 08:54 |
| mriedem | do you have any idea how many projects exist in CERN? | 08:54 |
| mdbooth | I'm trying to enable debug logs in a functional test. Thought it would be as simple as self.flags(debug=True), but apparently not. | 08:54 |
| sean-k-mooney | tssurya: are you using dansmith's prefilter to do the tenat to cell mapping or something else? | 08:55 |
| *** panda|off is now known as panda | 08:56 | |
| mriedem | mdbooth: OS_DEBUG=True | 08:56 |
| lyarwood | OS_DEBUG=1 tox -e functional [..] | 08:56 |
| lyarwood | yeah sorry | 08:56 |
| mdbooth | mriedem lyarwood: Thanks! You saved me some rtfs :) | 08:56 |
| mriedem | sean-k-mooney: wouldn't most of the work for sriov attach be doing the RT claim for a pci_request and if that fails, we fail the attach request | 08:57 |
| mriedem | and then set the port host binding details as you mentioned | 08:58 |
| mriedem | most of ^ is probably already done in the port binding code | 08:58 |
| openstackgerrit | jichenjc proposed openstack/nova master: Add zvm admin intro and hypervisor information https://review.openstack.org/533125 | 08:59 |
| openstackgerrit | jichenjc proposed openstack/nova master: Add zvm CI information https://review.openstack.org/533512 | 08:59 |
| *** alexchadin has quit IRC | 09:00 | |
| sean-k-mooney | mriedem: ya. which is why i think we can do all the work from the compute manager rather then the virt driver | 09:01 |
| sapd1 | sean-k-mooney: Will you have plan to re-implement this feature? :D | 09:01 |
| *** josecastroleon has quit IRC | 09:01 | |
| *** josecastroleon has joined #openstack-nova | 09:01 | |
| sean-k-mooney | mriedem: sapd1 these are my notes form last night https://etherpad.openstack.org/p/sriov-attach | 09:03 |
| openstackgerrit | Merged openstack/nova master: VMware: add missing os types in vSphere sdk 6.5 https://review.openstack.org/591771 | 09:04 |
| sean-k-mooney | sapd1: https://bugs.launchpad.net/nova/+bug/1708433 was opened by my old team as we had a customer request to enable sriov attach generically | 09:05 |
| openstack | Launchpad bug 1708433 in OpenStack Compute (nova) "Attaching sriov nic VM fail with keyError pci_slot" [Medium,In progress] - Assigned to Matt Riedemann (mriedem) | 09:05 |
| sean-k-mooney | sapd1: that was before my team was pulled off upstream work for internal projects. i have now move to redhat i might be able to work on this but not sure | 09:05 |
| sean-k-mooney | i would have to ask | 09:05 |
| *** cdent has joined #openstack-nova | 09:08 | |
| sean-k-mooney | mriedem: there are a few more edge cases that we would have to handel. e.g. vm runs out of pci ports, updating instace request spec, numa affintiy and updating placement allocation would be the main ones. a spec is the best way to capture all the edgecases | 09:10 |
| *** jaosorior has quit IRC | 09:10 | |
| *** alexchadin has joined #openstack-nova | 09:11 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove the deprecated API extensions policies https://review.openstack.org/586872 | 09:11 |
| sapd1 | sean-k-mooney: good point. Because I am not a developer, So I can't help you. Because nova souce code is too complicated | 09:12 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove the deprecated API extensions policies https://review.openstack.org/586872 | 09:13 |
| mriedem | sean-k-mooney: that's why there was a spec :) | 09:13 |
| mriedem | but it's outdated by now probably | 09:13 |
| mriedem | https://review.openstack.org/#/c/139910/ | 09:13 |
| sean-k-mooney | yes i remember. i think we referenced that spec in the bug or it was pointed out at least | 09:14 |
| sean-k-mooney | sapd1: well if this would have been your first topic in nova, it is proably in one of the most complicated parts. | 09:16 |
| cdent | mriedem: I left a comment on https://review.openstack.org/#/c/592285/ trying to get more info about what is actually being fixed, because it's not clear what the actual problem is. Do you have more info? | 09:16 |
| sean-k-mooney | sapd1: mriedem am i can write a spec for this and submit it if you like. but i will have to check internally if i can do more then that and impelmnte it assumeing it was approved. | 09:17 |
| mriedem | cdent: yeah yikun was going to update the ML thread on cells perf testing, | 09:17 |
| mriedem | but he needed to monkey_patch the time module | 09:17 |
| sean-k-mooney | there are some other topics related to this i was planning on writeing proposals for so its somwhat alinged with what i was going to suggest working on anyway. | 09:18 |
| sapd1 | sean-k-mooney: I will vote for you. :D | 09:18 |
| openstackgerrit | jichenjc proposed openstack/nova master: Add zvm admin intro and hypervisor information https://review.openstack.org/533125 | 09:19 |
| openstackgerrit | jichenjc proposed openstack/nova master: Add zvm CI information https://review.openstack.org/533512 | 09:19 |
| mriedem | sean-k-mooney: i'm indifferent to sriov port attach capabilities in nova | 09:20 |
| cdent | mriedem: would be interesting to see/know what ways nova api (as a wsgi app) needs eventlet at all | 09:20 |
| cdent | or it just happens to be picking it up accidentally and getting into trouble | 09:20 |
| *** Dinesh_Bhor has quit IRC | 09:21 | |
| sean-k-mooney | mriedem: for me originally it was parity thing. e.g. it was yet one more thing you had to be aware of did not work with sriov. | 09:22 |
| sean-k-mooney | mriedem: it became more important with hardware offloaded ovs | 09:22 |
| mriedem | cdent: i left a comment | 09:22 |
| cdent | thanks | 09:22 |
| mriedem | sean-k-mooney: sure i get that | 09:22 |
| mriedem | much like volume-backed instances with 33% of the compute API | 09:22 |
| mriedem | or 80% of the compute API with non-libvirt drivers :) | 09:23 |
| sean-k-mooney | mriedem: :) the main thing i was thinking of working on in stien was a generic device management framework for nova. to unify handeling of all passthough devices via a form of flavor decompostion. that is a lot of work though and im not sure its a high enough proirty to try and do in stien. | 09:27 |
| *** dpawlik has quit IRC | 09:28 | |
| sean-k-mooney | basicaly the openstack server create --device stuff i mentioned in vancouver. | 09:28 |
| *** dpawlik has joined #openstack-nova | 09:28 | |
| mdbooth | Ah, ha! My test is non-deterministic because the fake image service returns images as a list from a dict, hence in non-deterministic order. If the first one returned defined an external kernel then the libvirt driver uses the Flat imagebackend explicitly, which I'm not mocking in the Rbd tests. | 09:30 |
| mdbooth | That was satisfying :) | 09:30 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge extended availability zone response into server controller https://review.openstack.org/502859 | 09:31 |
| sean-k-mooney | mdbooth: if you ran it with python3 the dict has determistic ordering | 09:31 |
| mdbooth | sean-k-mooney: Interestingly I was running py3 | 09:31 |
| mdbooth | Although py35, which doesn't have deterministic dict order | 09:31 |
| mdbooth | iirc that 36 or 37? | 09:31 |
| sean-k-mooney | py36 i think | 09:32 |
| sean-k-mooney | mdbooth: that said the iteration order is fixed not sure about calls to values or keys but would assume they return ordered lists also | 09:33 |
| mdbooth | Yeah, I'd assume so | 09:33 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge config drive extension response into server controller https://review.openstack.org/584223 | 09:33 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge extended server attributes extension response https://review.openstack.org/584590 | 09:33 |
| *** Dinesh_Bhor has joined #openstack-nova | 09:34 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge extended server attributes extension response https://review.openstack.org/584590 | 09:35 |
| mriedem | it's not sorted https://github.com/openstack/nova/blob/master/nova/tests/unit/image/fake.py#L164 | 09:35 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge keypair extension response into server view builder https://review.openstack.org/584748 | 09:35 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server usage extension response into server view builder https://review.openstack.org/585262 | 09:35 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge security groups extension response into server view builder https://review.openstack.org/585475 | 09:36 |
| tssurya | mriedem: correct basic flavors (larger flavors users open special request) are available everywhere, special flavors for special projects in special cells where those tenants have access, by default projects are not mapped (when they are normal tenants/personal projects) and they land in a set of default cells. we have around ~4500 projects roughly which depending on where the resources are allocated, | 09:36 |
| tssurya | get mapped to that cell | 09:36 |
| tssurya | sean-k-mooney: we use the pre-filter from dansmith for scheduling based on the same criteria: projects mapped to cells/aggregates | 09:36 |
| tssurya | so that placement returns only those nodes | 09:37 |
| sean-k-mooney | tssurya: cool. i was wondering if ye were still using the oldway with the tenant affinity/isolation filter but that does not work right post palcement | 09:38 |
| tssurya | right | 09:38 |
| *** josecastroleon has quit IRC | 09:38 | |
| *** adrianc has quit IRC | 09:39 | |
| sean-k-mooney | tssurya: how have ye found the perfromce with the prefilter + placement vs cellsv1 and no placement. is it working well for ye? | 09:39 |
| *** alexchadin has quit IRC | 09:41 | |
| tssurya | sean-k-mooney: not really that good, we use the "limits" thing for limiting placement results for getting a decent performance | 09:41 |
| tssurya | we are trying to improve the scheduling part performance wise, mriedem had some patches | 09:42 |
| tssurya | but we have not seen much improvement in production after applying those | 09:42 |
| sean-k-mooney | thats unfortunet. there is still alot of work going into makeing schdulign faster in general. hopefully the performacne is adaquit for now. | 09:46 |
| mriedem | i imagine the big chunk of the per-request scheduling time is taken in the DB queries which iterate all selected compute nodes and then pull all instance UUIDs on each node | 09:46 |
| mriedem | across all cells | 09:46 |
| mriedem | that could be optimized into a single big join per cell probably | 09:46 |
| mriedem | because CERN isn't using the track_instance_changes config option right? | 09:47 |
| mriedem | where the computes report instance per host changes to the scheduler | 09:47 |
| mriedem | ? | 09:47 |
| mriedem | this method https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L710 | 09:48 |
| mriedem | specifically https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L735 | 09:48 |
| mriedem | that queries the cell db per compute host to find the list of instance uuids on that host | 09:49 |
| sean-k-mooney | mriedem: for the cern case you really only need to fine out what cell the project is mapped too right? so if we had placement aggreates per cell we would just need to add a member of to the placement query for the cells assocated with the tenant and let placement do all the work | 09:50 |
| mriedem | sean-k-mooney: CERN already has that | 09:51 |
| tssurya | mriedem: yea the time consumption is in the pulling of all instances info | 09:51 |
| mriedem | they have tenants mapped to host aggregates which are mapped to resource provider aggregates | 09:51 |
| mriedem | and that does the pre-filtering | 09:51 |
| sean-k-mooney | mriedem: oh ok cool | 09:51 |
| *** josecastroleon has joined #openstack-nova | 09:51 | |
| mriedem | they use this stuff https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html#aggregates-in-placement | 09:52 |
| sean-k-mooney | mriedem: so why do we ever pull the per host info back? | 09:52 |
| openstackgerrit | jichenjc proposed openstack/nova master: Add zvm admin intro and hypervisor information https://review.openstack.org/533125 | 09:52 |
| openstackgerrit | jichenjc proposed openstack/nova master: Add zvm CI information https://review.openstack.org/533512 | 09:52 |
| mriedem | sean-k-mooney: for the affinity filters | 09:52 |
| sean-k-mooney | mriedem: ah but that is after the placmenet query right? | 09:53 |
| mriedem | yes | 09:53 |
| *** sambetts|afk is now known as sambetts | 09:53 | |
| mriedem | pre-filter -> GET /allocation_candidates -> get_host_states -> per-host-instance query -> filters/weighers | 09:53 |
| sean-k-mooney | ok i miss understood. i taught you were implying we were doing the db query in the prefilter | 09:54 |
| mriedem | no the only thing we query there is which host aggregates are restricted for the tenant | 09:54 |
| sean-k-mooney | ya that makes sense. | 09:54 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/scheduler/request_filter.py#L26 | 09:55 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Py3 fix in fake image service https://review.openstack.org/592056 | 09:56 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Add regression test for bug 1550919 https://review.openstack.org/591733 | 09:57 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Don't delete disks on shared storage during evacuate https://review.openstack.org/578846 | 09:57 |
| openstack | bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9) | 09:57 |
| *** ratailor has quit IRC | 09:57 | |
| *** ratailor has joined #openstack-nova | 09:57 | |
| sean-k-mooney | mriedem: right we just get the aggreate uuids and stuff them in the destination object | 09:58 |
| mdbooth | mriedem: ^^^ is my proposed for failed evacuate on shared storage. The functional tests exercise all 4 combinations of shared/nonshared instance directory and share/nonshared block. | 09:59 |
| * mriedem goes to get dumplings | 09:59 | |
| *** mriedem has quit IRC | 10:00 | |
| mdbooth | mriedem: Good call ;) | 10:00 |
| sean-k-mooney | there is no stien spec template ye correct? im going to make one reusing the rocky template. | 10:01 |
| sean-k-mooney | is it stien or stein? | 10:02 |
| mdbooth | sean-k-mooney: The latter | 10:02 |
| sean-k-mooney | damb german words not doing the i before e thing | 10:03 |
| sean-k-mooney | that said more english word dont follow that then do also | 10:03 |
| mdbooth | Our crazy european neighbours | 10:03 |
| mdbooth | Don't understand the weightiness of consistent spelling | 10:04 |
| sean-k-mooney | you have seen my speeling correct ( yes i know i misspelled spelling, however that was kind of a typo). sometimes constency is a good thing but that does not mean i like us spelling more then uk. | 10:07 |
| *** tikitavi has joined #openstack-nova | 10:07 | |
| *** rha has joined #openstack-nova | 10:12 | |
| openstackgerrit | Vlad Gusev proposed openstack/nova stable/pike: Fix service list for disabled compute using MC driver https://review.openstack.org/592337 | 10:14 |
| *** s10 has joined #openstack-nova | 10:15 | |
| *** claudiub has joined #openstack-nova | 10:18 | |
| *** abhishekk has quit IRC | 10:23 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | 10:26 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: autospec classes before they are instantiated https://review.openstack.org/342211 | 10:26 |
| *** jaosorior has joined #openstack-nova | 10:28 | |
| *** lei-zh has quit IRC | 10:30 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: WIP: replace spec with autospec https://review.openstack.org/557299 | 10:30 |
| *** alexchadin has joined #openstack-nova | 10:31 | |
| *** dpawlik has quit IRC | 10:31 | |
| *** Sigyn has quit IRC | 10:31 | |
| *** dpawlik has joined #openstack-nova | 10:32 | |
| *** Sigyn has joined #openstack-nova | 10:32 | |
| *** maciejjozefczyk has joined #openstack-nova | 10:37 | |
| *** vivsoni_ has joined #openstack-nova | 10:37 | |
| openstackgerrit | Chen proposed openstack/nova master: Fix soft deleting vm fails after "nova resize" vm https://review.openstack.org/546920 | 10:38 |
| *** vivsoni has quit IRC | 10:39 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM https://review.openstack.org/478943 | 10:41 |
| *** Bhujay has quit IRC | 10:45 | |
| *** maciejjozefczyk has quit IRC | 10:49 | |
| *** maciejjozefczyk has joined #openstack-nova | 10:51 | |
| *** mvkr has quit IRC | 10:54 | |
| *** panda is now known as panda|lunch | 11:02 | |
| *** adrianc has joined #openstack-nova | 11:05 | |
| *** Dinesh_Bhor has quit IRC | 11:08 | |
| *** jpena is now known as jpena|lunch | 11:08 | |
| *** maciejjozefczyk has quit IRC | 11:20 | |
| *** maciejjozefczyk has joined #openstack-nova | 11:22 | |
| *** maciejjozefczyk has quit IRC | 11:23 | |
| *** mvkr has joined #openstack-nova | 11:25 | |
| *** donghm has joined #openstack-nova | 11:26 | |
| *** gcb_ has quit IRC | 11:27 | |
| *** josecastroleon has quit IRC | 11:28 | |
| *** josecastroleon has joined #openstack-nova | 11:28 | |
| *** udesale has quit IRC | 11:30 | |
| *** josecastroleon has quit IRC | 11:31 | |
| *** josecastroleon has joined #openstack-nova | 11:31 | |
| *** Bhujay has joined #openstack-nova | 11:35 | |
| *** erlon has joined #openstack-nova | 11:38 | |
| *** aloga has joined #openstack-nova | 11:42 | |
| *** Dinesh_Bhor has joined #openstack-nova | 11:45 | |
| *** Dinesh_Bhor has quit IRC | 11:46 | |
| *** adrianc has quit IRC | 11:47 | |
| *** aloga has quit IRC | 11:49 | |
| *** aloga has joined #openstack-nova | 11:50 | |
| *** dpawlik has quit IRC | 11:53 | |
| *** dpawlik has joined #openstack-nova | 11:55 | |
| *** dpawlik has quit IRC | 11:55 | |
| *** dpawlik has joined #openstack-nova | 11:55 | |
| *** maciejjozefczyk has joined #openstack-nova | 11:55 | |
| *** cdent has quit IRC | 11:57 | |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Move static _get_power_off_values to compute_utils https://review.openstack.org/578822 | 11:59 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Move static _get_power_off_values to compute_utils https://review.openstack.org/578822 | 11:59 |
| *** adrianc has joined #openstack-nova | 12:01 | |
| mdbooth | Any py3-loving cores fancy knocking this on the head: https://review.openstack.org/#/c/592056/ Trivial: change to explicit string type required by py3. | 12:07 |
| *** ratailor has quit IRC | 12:07 | |
| *** zul has joined #openstack-nova | 12:08 | |
| *** jpena|lunch is now known as jpena | 12:14 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Make monkey patch work in uWSGI mode https://review.openstack.org/592285 | 12:15 |
| efried | mdbooth: I glanced at that yesterday, but I'm not a py2-to-3 expert and the commit message is... sparse. Can you help me understand what's wrong and why it needs to be fixed? | 12:15 |
| mdbooth | efried: Because the argument has to be binary | 12:16 |
| mdbooth | If it's not, it gives an error that the argument has to be binary | 12:16 |
| efried | according to...? | 12:16 |
| efried | what gives an error? data.write or _imagedata.get? | 12:16 |
| efried | presumably the former | 12:17 |
| mdbooth | Yeah | 12:17 |
| efried | and where does the error show up? It must not be in the py35 tox job...? | 12:17 |
| mdbooth | Yeah, it was happening in the functional job | 12:18 |
| efried | oooohhhh | 12:18 |
| mdbooth | Although it was non-deterministic until I twigged why | 12:18 |
| efried | I was blinded by the fact that the fake driver lives in the /unit/ path. | 12:18 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Make monkey patch work in uWSGI mode https://review.openstack.org/592285 | 12:18 |
| efried | but it's used for the func tests too. | 12:18 |
| mdbooth | They're pretty incestuous :) | 12:18 |
| efried | okay, now that I have a way to try to repro locally I can take a better look at it. | 12:18 |
| efried | If you wouldn't mind enhancing the commit message, future me will be grateful. | 12:19 |
| mdbooth | Maybe if you open the file binary it has to have binary data written to it? | 12:19 |
| efried | Well, it makes sense that you would *want* to write binary if you're opening with 'b'. | 12:19 |
| mdbooth | >>> f = open('/tmp/foo', 'wb+') | 12:20 |
| mdbooth | >>> f.write('foo') | 12:20 |
| mdbooth | Traceback (most recent call last): | 12:20 |
| mdbooth | File "<stdin>", line 1, in <module> | 12:20 |
| mdbooth | TypeError: a bytes-like object is required, not 'str' | 12:20 |
| mdbooth | That's the error you get, anyway | 12:20 |
| mdbooth | >>> f.write(b'foo') | 12:20 |
| mdbooth | 3 | 12:20 |
| efried | yeah, so they must have tightened that down in py3, eh? | 12:21 |
| mdbooth | py2: | 12:21 |
| mdbooth | >>> f = open('/tmp/foo', 'wb+') | 12:21 |
| mdbooth | >>> f.write('foo') | 12:21 |
| mdbooth | >>> f.write(b'foo') | 12:21 |
| mdbooth | Both work | 12:21 |
| efried | which func test would fail? | 12:21 |
| efried | I try not to run full suites locally unless I need to reheat my coffee. | 12:22 |
| mdbooth | So... the following functional test would fail non-deterministically depending on whether or not fake image service gave it an image with a kernel and ramdisk or not | 12:22 |
| *** donghm has left #openstack-nova | 12:23 | |
| efried | We have a test that sometimes uses one kind of image and sometimes another, nondeterministically?? | 12:23 |
| mdbooth | efried: Yep. A few :) | 12:23 |
| efried | that's... I can't... but WHY? | 12:24 |
| mdbooth | I felt really good about myself for find that this morning :) | 12:24 |
| mdbooth | efried: non-deterministic dict ordering | 12:24 |
| *** jaosorior has quit IRC | 12:25 | |
| * efried looks into crystal ball and sees patch earlier in same series splitting such tests to always hit both/all paths... | 12:25 | |
| *** jaosorior has joined #openstack-nova | 12:25 | |
| efried | ...because the reward for good work is... | 12:25 |
| mdbooth | efried: It's not relevant to my bug, or I would have. I was just tripping over it. | 12:26 |
| efried | I was about to say: assume the topic bug isn't really related, so this should be tagged with the py3-first tag (whatever that tag is) | 12:27 |
| efried | btw, I recently discovered with cdent's guidance that you can avoid gerrit re-topic'ing your whole series by using git-review -T (available only in recent versions of git-review) | 12:28 |
| mdbooth | I mean the non-deterministic images, that is. My patch just needs a functioning image service in order to boot. kernel vs non-kernel is irrelevant, except that it caused a bug. | 12:29 |
| mdbooth | I don't really use topics, tbh | 12:29 |
| mdbooth | I've always found it more useful to use the fact that gerrit displays a series in dependency order in 'Related Changes' | 12:30 |
| efried | um. For misc random patch that's fine, but when they're associated with a bug, blueprint, or initiative (like mox removal or py3 first), it's useful. | 12:30 |
| efried | Well, yeah, on that note, in this case the fix really doesn't need to be in that series at all, does it? | 12:31 |
| mdbooth | It does, because the functional test which follows it was tripping up on it | 12:31 |
| mdbooth | non-deterministically | 12:31 |
| efried | but only in py3, which isn't set up in the gate yet? | 12:32 |
| mdbooth | Yeah, we run functional on py3 | 12:32 |
| efried | "we" do? | 12:32 |
| efried | looking again... | 12:32 |
| efried | oic | 12:32 |
| mdbooth | http://logs.openstack.org/33/591733/4/check/nova-tox-functional-py35/d9266d8/ | 12:32 |
| *** eharney has quit IRC | 12:33 | |
| efried | Okay; I was looking at that yesterday (for an unrelated reason) and totally missed it :( | 12:33 |
| efried | hmph | 12:33 |
| mdbooth | I didn't know either until I starting writing this test :) | 12:34 |
| *** josecastroleon has quit IRC | 12:34 | |
| mdbooth | efried: Here's the failure in the gate, btw: | 12:34 |
| efried | anyway, you never got around to telling me which func test fails. I'm getting now that it's one you're introducing in your series... | 12:34 |
| mdbooth | http://logs.openstack.org/33/591733/2/check/nova-tox-functional-py35/c3d298a/job-output.txt.gz | 12:34 |
| efried | ...not an existing one | 12:34 |
| mdbooth | 2018-08-15 14:21:38.077216 | 12:34 |
| kosamara | Hi efried. I'm revising the YAML format to include keys that any driver understands. Some keys like vendor_id make sense to many drivere; however, they might expect different values for them. | 12:35 |
| mdbooth | Anything exercising that in py3 will hit it, because it's basically a syntax error. Seems nothing was previously. | 12:36 |
| efried | Right, my point. | 12:36 |
| kosamara | Eg PowerVM -> \X{4}, libvirt -> \x{4} | 12:36 |
| *** cdent has joined #openstack-nova | 12:36 | |
| *** josecastroleon has joined #openstack-nova | 12:37 | |
| kosamara | I would like to say that the value format is the driver's responsibility as well, but there needs to be a common schema. | 12:37 |
| efried | kosamara: Those values come from the hardware and their form (32-bit integer) is ordained by a standard. | 12:37 |
| mdbooth | Ooh, timestamps are clicky: http://logs.openstack.org/33/591733/2/check/nova-tox-functional-py35/c3d298a/job-output.txt.gz#_2018-08-15_14_21_38_077216 | 12:37 |
| * mdbooth assumed it was just text | 12:38 | |
| kosamara | Let me double check | 12:38 |
| *** tbachman has joined #openstack-nova | 12:38 | |
| efried | kosamara: we could ask for it to be represented as an integer via the schema, so you could do 0xC001, 0xc001, 0140001 (octal), 49153 (decimal), etc. | 12:39 |
| kosamara | efried: libvirt reports \x{4} | 12:40 |
| kosamara | efried: solves the problem, but sounds confusing for the operator | 12:41 |
| efried | kosamara: I agree. | 12:42 |
| *** rha has quit IRC | 12:42 | |
| kosamara | efried: I would prefer to dictate an arbitrary format \X{4} and then it will be the driver's responsibility to normalize it. | 12:42 |
| efried | kosamara: I'm fine with that. Note that yaml tries to outsmart you on this, though. It interprets e.g. C001 (without quotes) as a string, but 4321 (without quotes) as an integer base 10. | 12:43 |
| efried | kosamara: So we should document that these hex values be quoted. | 12:44 |
| efried | mdbooth: Is your test case really valid if it's trying to fetch a nonexistent image? Maybe we're better off fixing the fake by taking off the default to .get() to force the test author to set it up. | 12:46 |
| efried | kosamara: Hm, which makes me wonder whether I did my schema validation correctly in https://review.openstack.org/#/c/579289/ | 12:47 |
| efried | because I remember running across this problem. | 12:47 |
| efried | but now I can't remember if the test failed because 4321 got interpreted as an int or because it failed schema validation because not a string. | 12:47 |
| efried | I would hope the latter. Need to add a test. | 12:48 |
| *** owalsh has quit IRC | 12:53 | |
| *** janki has quit IRC | 12:59 | |
| *** janki has joined #openstack-nova | 13:00 | |
| *** jmlowe has joined #openstack-nova | 13:00 | |
| kosamara | efried: nice, thanks for the tip! | 13:04 |
| *** abhishekk has joined #openstack-nova | 13:05 | |
| *** owalsh has joined #openstack-nova | 13:06 | |
| mdbooth | efried: My test case isn't concerned with images. | 13:07 |
| mdbooth | efried: Except that it needs to boot an instance which has disks. | 13:07 |
| *** jmlowe has quit IRC | 13:12 | |
| *** udesale has joined #openstack-nova | 13:12 | |
| *** jaosorior has quit IRC | 13:14 | |
| *** jaosorior has joined #openstack-nova | 13:15 | |
| *** alex_xu has quit IRC | 13:15 | |
| *** eharney has joined #openstack-nova | 13:17 | |
| efried | mdbooth: I can fix the commit message and +2. | 13:17 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Py3 fix in fake image service https://review.openstack.org/592056 | 13:19 |
| *** jaosorior has quit IRC | 13:20 | |
| efried | mdbooth: Done ^ | 13:20 |
| tobias-urdin | have a weird issue, i have some instances stuck in building (scheduling) state which show up in dashboard and with api, but server show throws NotFound exception so I cant remove them, anyubody? i've restarted nova-api, scheduler and conductors. | 13:21 |
| *** eharney has quit IRC | 13:21 | |
| *** eharney has joined #openstack-nova | 13:22 | |
| mdbooth | efried: Thanks | 13:24 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Add regression test for bug 1550919 https://review.openstack.org/591733 | 13:24 |
| openstack | bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9) | 13:24 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Don't delete disks on shared storage during evacuate https://review.openstack.org/578846 | 13:25 |
| *** imacdonn has quit IRC | 13:29 | |
| *** nicolasbock has joined #openstack-nova | 13:32 | |
| *** alexchadin has quit IRC | 13:33 | |
| *** alexchadin has joined #openstack-nova | 13:34 | |
| *** gbarros has joined #openstack-nova | 13:38 | |
| *** imacdonn has joined #openstack-nova | 13:42 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Making instance listing skipping down cells configurable https://review.openstack.org/592428 | 13:53 |
| tssurya | mriedem: ^ | 13:55 |
| *** awaugama has joined #openstack-nova | 13:58 | |
| kosamara | efried: What does PowerVM want to do, if the YAML it parses includes identification keys it doesn't understand? | 13:59 |
| *** brinzh has quit IRC | 14:00 | |
| efried | kosamara: Not sure. | 14:00 |
| efried | kosamara: Options: ignore, log, error. | 14:00 |
| kosamara | efried: I'm tending towards making it an error in libvirt, with the assumption that the operator has made an oversight and will be baffled to see his devices not recognized if it's just ignored. | 14:00 |
| efried | Right | 14:00 |
| efried | that's a reasonable argument. | 14:01 |
| *** janki has quit IRC | 14:01 | |
| kosamara | Should it be enforced by the spec for all drivers though? | 14:01 |
| efried | Because the conf is going to have to be host-specific, which means driver-specific. | 14:01 |
| efried | kosamara: But my hesitation is this: | 14:01 |
| *** dpawlik has quit IRC | 14:01 | |
| *** mvkr has quit IRC | 14:02 | |
| efried | At some point in the future, we might like to support a global conf on the conductor, whose values percolate down to the computes, which can override (optionally) via local configs of their own. | 14:02 |
| *** josecastroleon has quit IRC | 14:02 | |
| efried | In that scenario, the global conf would presumably want to include the union of all the relevant keys. | 14:03 |
| efried | and the computes would have to ignore the ones it doesn't understand. | 14:03 |
| efried | But maybe we can solve that when we get there. | 14:03 |
| tssurya | dansmith, melwitt: FYI, mriedem has some stuff on the cells meeting agenda | 14:04 |
| dansmith | tssurya: for next week? | 14:04 |
| tssurya | heh not really, just up for open discussion/opinions I guess | 14:05 |
| efried | kosamara: As for attempting to define the same rule for all drivers, I think that's a good idea for consistency. | 14:05 |
| *** josecastroleon has joined #openstack-nova | 14:06 | |
| *** _pewp_ has joined #openstack-nova | 14:08 | |
| *** josecastroleon has quit IRC | 14:10 | |
| *** Bhujay has quit IRC | 14:12 | |
| *** cdent has quit IRC | 14:12 | |
| *** slaweq_ is now known as slaweq | 14:14 | |
| sean-k-mooney | kosamara: i would generally try to minimise divergnce options in dirver | 14:16 |
| sean-k-mooney | efried: host-specfic doen not inherintly mean driver-specific | 14:17 |
| efried | sean-k-mooney: Can we have more than one nova-compute service running on a single host? Or more than one driver loaded in a single nova-compute process? | 14:17 |
| sean-k-mooney | efried: yes we can have more then one nova compute on the same host | 14:20 |
| sean-k-mooney | efried: normally only done with ironic | 14:20 |
| sean-k-mooney | we can only have 1 driver loaded as far as i am aware | 14:21 |
| efried | That sounds really confusing. Are they reporting different host resources? | 14:21 |
| efried | or the compute services each "own" a different subset of the ironic nodes? | 14:22 |
| sean-k-mooney | the compute serveice each own a subset of the ironic nodes | 14:22 |
| efried | okay, so still, is there an example where multiple different drivers could exist on a single host? | 14:22 |
| sean-k-mooney | efried: not really. no. | 14:23 |
| sean-k-mooney | you could have ironic and libvirt on the same node | 14:23 |
| sean-k-mooney | i have done that but only for dev | 14:23 |
| efried | okay. So host <=> driver for all intents and purposes. | 14:23 |
| sean-k-mooney | you not really going to run libvirt and say nova-docker on the same host that said you could | 14:24 |
| efried | sean-k-mooney: If you did have that kind of dual setup, would they be pointing to different configs? | 14:24 |
| sean-k-mooney | efried: yes | 14:24 |
| efried | okay, then the model holds. | 14:24 |
| sean-k-mooney | efried: the point i was more trying to make is that the config them selve are not virt driver specic in the content they contain | 14:25 |
| efried | wtf, of course they are | 14:25 |
| sean-k-mooney | what is different? | 14:25 |
| efried | you try to put a libvirt nova.conf onto a powervm compute node, it won't work at all. | 14:25 |
| sean-k-mooney | efried: only the libvirt section would be different | 14:26 |
| sean-k-mooney | anything outside the virt driver section should be identical | 14:26 |
| efried | That sounds like a theoretical statement, not a practical one. | 14:26 |
| sean-k-mooney | efried: if it not we have serious bugs | 14:26 |
| sean-k-mooney | the only section that should be virt driver sepecific is the virt driver section. | 14:27 |
| efried | Same config *format* I'll grant you, but the idea of being able to compose one config file and then copy it verbatim around a heterogeneous cloud - I have a hard time believing that anybody actually does that. | 14:27 |
| sean-k-mooney | efried: alot of the installers bacsically do that | 14:28 |
| sean-k-mooney | you are right here may be some hardware specific changes such as the vcpu_pin_set | 14:29 |
| sean-k-mooney | but that dose not depend on the virt driver virt driver | 14:29 |
| dansmith | sean-k-mooney: to have more than one compute per host you have to tell them each that they are a different hostname, and thus a different config, which means they might as well be separate hosts | 14:29 |
| efried | kosamara: Based on this discussion, how do you feel about recommending that virt drivers log a warning, but not raise an exception, when they encounter a field they don't recognize/support/use? | 14:31 |
| kosamara | efried: That's what I was converging to, I think that's best | 14:31 |
| efried | this would give the admin the right clue as to why their device doesn't show up, if they were in fact expecting that field to be processed. | 14:32 |
| kosamara | which satisfies my scenario | 14:32 |
| sean-k-mooney | dansmith: that is true. | 14:32 |
| *** jaosorior has joined #openstack-nova | 14:32 | |
| efried | "WARNING: Device identification field XXX in inventory.yaml not recognized by the YYY driver; ignoring" | 14:32 |
| *** mvkr has joined #openstack-nova | 14:33 | |
| sean-k-mooney | efried: do you need to raise an exception? | 14:33 |
| efried | no, that's what kosamara and I are talking about. | 14:34 |
| sean-k-mooney | sorry you said not raise an exception | 14:34 |
| efried | If it's a common thing for configs to be copied around a cloud, then we should log and ignore. | 14:34 |
| sean-k-mooney | efried: ya if we raise an exception from config parsing we normall hard crash the agent. but that seam wrong in this case | 14:35 |
| efried | sean-k-mooney: This processing is happening in the compute service, in the virt driver's update_provider_tree method, on startup and/or SIGHUP, so I don't think it crashes the compute service; but it would make lots of things not work, depending where the exception was raised in update_provider_tree. | 14:37 |
| sean-k-mooney | efried: sorry i was not being clear. it not where the parsing is done. we have two classes of config options. 1 where if its invalid we stop the agent form running and a second where we log a warning and continue on | 14:40 |
| efried | sean-k-mooney: Note that this isn't oslo.config | 14:40 |
| sean-k-mooney | this feels more like a warning to me then stop everything | 14:40 |
| sean-k-mooney | efried: yes. the impentation is not what im concerned about its the semantics im thinking about | 14:41 |
| efried | Right. And semantically we converged on the fact that it's (apparently) fairly common for operators to write a config file once and then copy it around the cloud. | 14:43 |
| efried | For this reason, as well as in anticipation of future support for a global inventory.yaml on the conductor, it makes most sense *not* to treat an unrecognized field as an error. | 14:43 |
| efried | But, to avoid baffling the operator, we should log a warning that she'll see when she goes to investigate why her device isn't showing up. | 14:43 |
| efried | kosamara: Fair summary? ^ | 14:43 |
| *** alexchadin has quit IRC | 14:44 | |
| sean-k-mooney | efried: its not that they write them once it more most deployment tools you a base config template with minimal per host overrides so the confige are largly the same. | 14:44 |
| efried | cool. | 14:45 |
| sean-k-mooney | the downside to that is you typo somthing once and you break everything. | 14:45 |
| sean-k-mooney | i may have hard locked 15 servers due to a config typo in the past.. | 14:46 |
| *** ratailor has joined #openstack-nova | 14:59 | |
| *** cdent has joined #openstack-nova | 15:01 | |
| *** erlon has quit IRC | 15:01 | |
| *** munimeha1_ has joined #openstack-nova | 15:03 | |
| *** abhishekk has quit IRC | 15:05 | |
| *** ratailor has quit IRC | 15:05 | |
| *** ratailor has joined #openstack-nova | 15:06 | |
| *** mvkr has quit IRC | 15:06 | |
| *** r-daneel has joined #openstack-nova | 15:07 | |
| melwitt | . | 15:07 |
| *** erlon has joined #openstack-nova | 15:13 | |
| jaypipes | melwitt: , | 15:21 |
| melwitt | ! | 15:21 |
| jaypipes | ? | 15:21 |
| jaypipes | :P | 15:21 |
| melwitt | we're at an impasse... | 15:22 |
| jaypipes | melwitt: was just a friendly comma saying hi to your full stop. | 15:22 |
| melwitt | heh | 15:22 |
| *** hemna_ has joined #openstack-nova | 15:25 | |
| *** psachin has quit IRC | 15:27 | |
| *** luksky has joined #openstack-nova | 15:32 | |
| kosamara | efried: yes | 15:36 |
| *** efried is now known as efried_afk | 15:37 | |
| *** gbarros has quit IRC | 15:45 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | 15:48 |
| *** dave-mccowan has joined #openstack-nova | 15:49 | |
| *** gyee has joined #openstack-nova | 15:53 | |
| *** gbarros has joined #openstack-nova | 16:02 | |
| sean-k-mooney | melwitt: dansmith. i have found an issue in my livemigration. specifcally there is a case where livemigration fails, the instance is still running on the soruce but the vif_type is none beacuse the host_id is not set. | 16:05 |
| *** udesale has quit IRC | 16:05 | |
| sean-k-mooney | i need to do more digging to localise the actull issue | 16:05 |
| dansmith | because we update the instance info in the db from the failure yeah? | 16:06 |
| dansmith | I think mriedem had a patch for that recently, IIRC | 16:06 |
| dansmith | sean-k-mooney: does the healer periodic fix it after it runs? | 16:07 |
| sean-k-mooney | im not sure. i think we may have activated the port binding for the dest and not reactivated for source. | 16:07 |
| *** davidsha has quit IRC | 16:08 | |
| sean-k-mooney | dansmith: ill get the logs and file a propper bug | 16:08 |
| dansmith | okay there was a situation where we saved the failed binding info cache recently that matt was working on, so might be related | 16:08 |
| dansmith | but yeah | 16:08 |
| *** ratailor has quit IRC | 16:09 | |
| sean-k-mooney | dansmith: i dont think its related as teh vif_type is none in neutron not binding_failed. | 16:09 |
| dansmith | ah okay | 16:09 |
| *** tikitavi has quit IRC | 16:12 | |
| *** jpena is now known as jpena|off | 16:19 | |
| *** luksky has quit IRC | 16:20 | |
| kosamara | efried: On the namespacing of driver-generated traits: if we demand that all driver-generated traits exist within a namespace, we don't allow the drivers to generate standard traits. | 16:23 |
| *** panda|lunch is now known as panda|off | 16:28 | |
| kosamara | I propose to allow the driver to generate either namespaced or standard traits. | 16:28 |
| *** derekh has quit IRC | 16:29 | |
| *** sahid has quit IRC | 16:30 | |
| *** cdent has quit IRC | 16:32 | |
| *** cdent_ has joined #openstack-nova | 16:32 | |
| *** s10 has quit IRC | 16:32 | |
| *** pcaruana has quit IRC | 16:34 | |
| *** dpawlik has joined #openstack-nova | 16:41 | |
| *** dpawlik has quit IRC | 16:46 | |
| *** adrianc has quit IRC | 16:48 | |
| *** openstackgerrit has quit IRC | 16:49 | |
| *** tssurya has quit IRC | 17:28 | |
| *** openstackgerrit has joined #openstack-nova | 17:34 | |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Move rocky implemented specs https://review.openstack.org/592622 | 17:34 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Add explanatory prefix to post_test_perf output https://review.openstack.org/591850 | 17:35 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Add trait query to placement perf check https://review.openstack.org/592624 | 17:35 |
| *** dave-mccowan has quit IRC | 17:39 | |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Dynamically find releases for move-implemented-specs https://review.openstack.org/592628 | 17:42 |
| *** slaweq has quit IRC | 17:44 | |
| *** Sundar has joined #openstack-nova | 17:53 | |
| Sundar | Hi all, could somebody tell me the status of nested resource providers, including virt driver changes (to avoid stomping on custom resources)? If there is a bp to track it, please point me to that. Thanks! | 17:54 |
| melwitt | I think the placement side is considered done, but none of the nova integration side has been started yet. I'm not aware of a bp tracking it yet | 17:55 |
| *** owalsh has quit IRC | 17:58 | |
| *** tbachman has quit IRC | 17:58 | |
| Sundar | melwitt: Thanks. So, in particular, the work to prevent virt drivers from overwriting custom resources has not started yet, right? If so, what is the blocker -- is it still upgrade considerations? | 17:58 |
| dansmith | Sundar: resources or traits? | 17:59 |
| Sundar | dansmith: I think both. When last checked, the virt drivers just overwrite the inventory of the compute node. | 18:00 |
| dansmith | well, nova-compute does that with the help of the virt driver, | 18:00 |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Add a script for counting blueprints https://review.openstack.org/581914 | 18:01 |
| dansmith | but inventory of a provider should only be owned by a single entity, | 18:01 |
| sean-k-mooney | Sundar: in dublin we said external service were not allowed to create inventoei in resouce providers they did not create | 18:01 |
| dansmith | and since there is only one provider for a compute node right now, overwriting inventory entirely is kinda expected | 18:01 |
| dansmith | right, what sean-k-mooney said | 18:01 |
| dansmith | so until the providers start reporting things as nested, | 18:01 |
| *** efried_afk is now known as efried | 18:01 | |
| dansmith | there's no real opportunity for the coexistence of other nested providers with their own inventory they manage | 18:02 |
| efried | kosamara: Yes, tricky question as to who "owns" a standard trait. | 18:02 |
| dansmith | and that is dependent on the upgrade bit getting worked out | 18:02 |
| Sundar | Yes. Even with nested RPs, we had the issue that the libvirt driver would overwrite the nested RP config. This was about a month or so ago. | 18:02 |
| *** tbachman has joined #openstack-nova | 18:03 | |
| dansmith | Sundar: because it's not nested-aware, AFAIK | 18:03 |
| efried | true ^ | 18:03 |
| efried | dansmith: The upgrade (reshape) series is ready for review btw. | 18:04 |
| dansmith | yeah I know | 18:04 |
| efried | :P | 18:04 |
| sean-k-mooney | Sundar: cyborg will have to create a seperate resouce provider tree under the compute node. it should be possible to do that today but im not sure if the virt driver will remove them or not | 18:05 |
| efried | kosamara: Do you understand the concern about trait "ownership" in general? | 18:05 |
| efried | sean-k-mooney: The virt driver won't remove them; but we also won't schedule to them either. | 18:05 |
| sean-k-mooney | efried: can you allocate from them | 18:06 |
| sean-k-mooney | the intent would to still schduler to the compute node but consume the cyborg resouce from the cyborg subtrees | 18:06 |
| efried | sean-k-mooney: basically, that whole code path has some bits that will work, some bits that definitely won't, and some that are still a mystery. This is work that needs to be done in Stein. | 18:06 |
| efried | sean-k-mooney: I think the scheduler *should* consume the cyborg resources. | 18:07 |
| efried | It's just not allowed to muck with the cyborg providers' inventories or traits. | 18:07 |
| Sundar | sean-k-mooney: Yes, cyborg is trying to do nRPs. We hit the virt driver overwrite issue and developers decided to wait for that to be resolved. | 18:07 |
| sean-k-mooney | efried: right. we may also have to revisit the idea of have a seperate root node just for the phyical server and then have per projec subtrees | 18:07 |
| sean-k-mooney | Sundar: again if the overriete issue is you tried to create invtories on the comptue node resouce provider that will never be supported | 18:08 |
| *** gbarros has quit IRC | 18:08 | |
| efried | revisit? I don't remember visiting that in the first place, ever. Per-project subtrees? | 18:08 |
| sean-k-mooney | efried: it came up in dublin on the friday when we were all tired | 18:09 |
| Sundar | sean-k-mooney: no, we created nested RPs within the compute node. | 18:09 |
| efried | I think Sundar is talking about creating child providers of the compute node provider but having them ignored by the scheduler and/or deleted by the resource tracker. The latter may have happened in libvirt when it was still using get_inventory. | 18:09 |
| Sundar | Yes ^ | 18:09 |
| efried | I'm guessing you haven't tried it since we implemented libvirt's update_provider_tree method. | 18:09 |
| Sundar | efried: when was that done? | 18:10 |
| efried | Sundar: So if you have the resources and inclination, you could try that again. I guarantee it will still break, but it may get slightly further than it did before. | 18:10 |
| efried | Sundar: Let me find the patch... | 18:10 |
| sean-k-mooney | efried: the idea being that nova would nolonger own the root node of the tree. the root would have no invetores and just represent teh server then each service on the serve would create there own nested resouce provider beneate that root node | 18:11 |
| Sundar | efried: "I guarantee it will still break" -- that won't help me persuade other Cyborg developers to start moving fwd :) | 18:11 |
| efried | Sundar: https://review.openstack.org/#/c/560444/ merged 5/25 | 18:11 |
| sean-k-mooney | efried: it was a fallback incase we did not figure out how to have multple service own different nodes under the current tree create by nova | 18:11 |
| efried | sean-k-mooney: Nono, nova still owns the root provider. But nova does *not* own the device providers - cyborg owns those. | 18:11 |
| efried | oh, sorry, you're talking about the project-based thing. | 18:12 |
| sean-k-mooney | efried: ya sorry i was | 18:12 |
| *** owalsh has joined #openstack-nova | 18:12 | |
| efried | sean-k-mooney: Without further placement API work, we would probably have to do that by lassoing the project subtree in an aggregate corresponding to the project ID. | 18:12 |
| sean-k-mooney | efried: by project id you mean service id? | 18:13 |
| *** mmedvede has quit IRC | 18:13 | |
| efried | But regardless, that would still rely on scheduler and resource tracker implementation that we have not started yet. Same as the stuff Sundar is blocked behind. | 18:13 |
| sean-k-mooney | when i said per-procject i ment nova, neutron, cinder not tenants | 18:13 |
| efried | um, oh, then forget what I'm saying. | 18:14 |
| efried | But yes, all of that needs to be thought through. | 18:14 |
| efried | sean-k-mooney: I would think that nova would still own the root node. | 18:14 |
| efried | Sundar: Define "moving fwd"? You can't integrate with nova yet, for sure. But I assume there's work you can do while you're waiting for enablement. | 18:15 |
| sean-k-mooney | efried: perhaps, the idea of spliting it out was in the event that nova is not the first service deployed on the node, we won know what uuid it will get so we cant create the provider tree | 18:15 |
| efried | somebody would have to reparent | 18:16 |
| efried | which can be done as long as parent isn't already set. | 18:16 |
| *** mmedvede has joined #openstack-nova | 18:17 | |
| sean-k-mooney | efried: once that reshaper stuff is reviewed :) | 18:17 |
| sean-k-mooney | although i guess you could manually reparent now its just not as simple | 18:18 |
| *** r-daneel has quit IRC | 18:20 | |
| efried | actually I don't think reshaper accounts for reparenting at all. I was talking about: | 18:20 |
| efried | Neutron starts first. Creates the NIC RPs. They have no parent. | 18:20 |
| efried | Cyborg starts next. Creates the dev RPs. They have no parent (and aren't related to the NIC RPs). | 18:20 |
| efried | Now Nova starts. It creates the compute node RP. It also somehow discovers that the NIC and device RPs are out there, and reparents them (PUT /resource_provider/{u} with parent_provider_uuid specified in the payload) to the compute node RP. | 18:20 |
| *** tbachman has quit IRC | 18:22 | |
| Sundar | efried: Why can't Cyborg and Neutron wait for Nova compute to create the top level first? | 18:24 |
| *** awaugama has quit IRC | 18:24 | |
| *** claudiub has quit IRC | 18:26 | |
| Sundar | If that is simpler, that is | 18:26 |
| efried | I don't know. Maybe they can. Ask sean-k-mooney | 18:28 |
| *** awaugama has joined #openstack-nova | 18:28 | |
| *** _pewp_ has quit IRC | 18:37 | |
| *** _pewp_ has joined #openstack-nova | 18:37 | |
| *** hemna_ has quit IRC | 18:38 | |
| openstackgerrit | Merged openstack/nova master: Py3 fix in fake image service https://review.openstack.org/592056 | 18:48 |
| *** hemna_ has joined #openstack-nova | 18:51 | |
| *** harlowja has joined #openstack-nova | 18:51 | |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Repropose reshaper spec for Stein https://review.openstack.org/592650 | 18:53 |
| *** awaugama has quit IRC | 18:55 | |
| *** hemna_ has quit IRC | 18:56 | |
| *** r-daneel has joined #openstack-nova | 18:56 | |
| *** awaugama has joined #openstack-nova | 18:56 | |
| *** slaweq has joined #openstack-nova | 19:03 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: use single-shot INSERT/DELETE agg https://review.openstack.org/592654 | 19:04 |
| *** slaweq has quit IRC | 19:19 | |
| *** slaweq has joined #openstack-nova | 19:22 | |
| *** cdent_ has quit IRC | 19:23 | |
| *** _pewp_ has quit IRC | 19:25 | |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Repropose reshaper spec for Stein https://review.openstack.org/592650 | 19:26 |
| *** _pewp_ has joined #openstack-nova | 19:27 | |
| *** tbachman has joined #openstack-nova | 19:28 | |
| *** tbachman has quit IRC | 19:32 | |
| *** slaweq has quit IRC | 19:38 | |
| *** tbachman has joined #openstack-nova | 19:40 | |
| *** awaugama has quit IRC | 19:50 | |
| *** eharney has quit IRC | 19:58 | |
| *** HW_Peter has joined #openstack-nova | 20:04 | |
| *** HW-Peter has quit IRC | 20:07 | |
| *** eharney has joined #openstack-nova | 20:13 | |
| *** pcaruana has joined #openstack-nova | 20:13 | |
| *** tbachman has quit IRC | 20:15 | |
| *** eharney has quit IRC | 20:24 | |
| *** slaweq has joined #openstack-nova | 20:35 | |
| *** r-daneel has quit IRC | 20:36 | |
| *** takashin has joined #openstack-nova | 20:44 | |
| melwitt | nova meeting in 10 minutes | 20:50 |
| *** r-daneel has joined #openstack-nova | 20:53 | |
| *** pcaruana has quit IRC | 20:53 | |
| *** tbachman has joined #openstack-nova | 20:58 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Batch results per cell when doing cross-cell listing https://review.openstack.org/592698 | 20:59 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: use single-shot INSERT/DELETE agg https://review.openstack.org/592654 | 21:02 |
| *** Sundar has quit IRC | 21:05 | |
| *** mchlumsky has quit IRC | 21:11 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Batch results per cell when doing cross-cell listing https://review.openstack.org/592698 | 21:24 |
| *** erlon has quit IRC | 21:26 | |
| *** mvkr has joined #openstack-nova | 21:29 | |
| *** munimeha1_ has quit IRC | 21:35 | |
| *** rcernin has joined #openstack-nova | 21:49 | |
| *** slaweq has quit IRC | 21:49 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: use single-shot INSERT/DELETE agg https://review.openstack.org/592654 | 22:00 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: use single-shot INSERT/DELETE agg https://review.openstack.org/592654 | 22:04 |
| *** tbachman has quit IRC | 22:13 | |
| *** harlowja has quit IRC | 22:14 | |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Delete approved template in move_implemented_specs https://review.openstack.org/592755 | 22:16 |
| efried | melwitt: In case you were busily working on this enhancement ^ | 22:17 |
| melwitt | hah, thanks | 22:19 |
| *** edmondsw has quit IRC | 22:20 | |
| *** edleafe has quit IRC | 22:22 | |
| efried | melwitt: Mind if I combine your patches for dynamically finding releases and counting blueprints into a series that commonizes the fs walk bit? | 22:24 |
| melwitt | efried: nope, go ahead | 22:25 |
| *** edmondsw has joined #openstack-nova | 22:25 | |
| *** chyka has joined #openstack-nova | 22:32 | |
| *** dklyle has joined #openstack-nova | 22:33 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104 | 22:37 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Making consistent used of GiB and MiB in API ref https://review.openstack.org/559985 | 22:42 |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Move rocky implemented specs https://review.openstack.org/592622 | 22:49 |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Add a script for counting blueprints https://review.openstack.org/581914 | 22:51 |
| efried | melwitt: ^ rebased on top of the dynamic-y one | 22:52 |
| melwitt | cool, thanks | 22:54 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (4) https://review.openstack.org/574106 | 23:03 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (5) https://review.openstack.org/574110 | 23:06 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (6) https://review.openstack.org/574113 | 23:09 |
| *** slaweq has joined #openstack-nova | 23:10 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (7) https://review.openstack.org/574974 | 23:11 |
| *** chyka has quit IRC | 23:13 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8) https://review.openstack.org/575311 | 23:14 |
| *** slaweq has quit IRC | 23:15 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9) https://review.openstack.org/575581 | 23:17 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (10) https://review.openstack.org/576017 | 23:20 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (11) https://review.openstack.org/576018 | 23:22 |
| *** erlon has joined #openstack-nova | 23:23 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (12) https://review.openstack.org/576019 | 23:26 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13) https://review.openstack.org/576020 | 23:29 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14) https://review.openstack.org/576027 | 23:36 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.openstack.org/576031 | 23:41 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.openstack.org/576299 | 23:44 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.openstack.org/576344 | 23:48 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.openstack.org/576673 | 23:51 |
| *** gyee has quit IRC | 23:53 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.openstack.org/576676 | 23:54 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.openstack.org/576689 | 23:57 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.openstack.org/576709 | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!