| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-07 | |||
| 12:02:22 | cdent | if what we're really concerned about is performance of the api then we should, _later_, consider a namespaced memcached layer on top of the placement api that allows marking collections as dirty | |
| 12:02:35 | alex_xu | mdbooth: appreciate your review same, it is really helpful :) | |
| 12:02:54 | cdent | alex_xu, mdbooth: I think we should have lots more of this type of conversation, so I think this is good | |
| 12:03:10 | alex_xu | mdbooth: btw, the spec is at https://review.openstack.org/345138, in case you are interesting :) | |
| 12:03:29 | mdbooth | alex_xu: Thanks, I should probably have read that first. | |
| 12:04:12 | cdent | I think jay's concerns are at the database layer, in which case we can address that by simply having all the traits in the same table, and isn't that being done when we sync from os-traits? | |
| 12:04:56 | alex_xu | cdent: yes, all the traits in one table | |
| 12:05:07 | cdent | then why have a cache at all? | |
| 12:05:11 | alex_xu | by 'placement-manage os-traits sync' | |
| 12:06:16 | alex_xu | cdent: I guess jay think the trait/resource class won't update frequently, then to same some db call... | |
| 12:06:37 | cdent | -1 | |
| 12:06:37 | cdent | :) | |
| 12:06:43 | alex_xu | :) | |
| 12:06:58 | cdent | with the resource class stuff the cache is because the classes are _not_ all int he same table | |
| 12:07:10 | cdent | with traits, if they are, then... | |
| 12:07:32 | alex_xu | ah... | |
| 12:07:40 | alex_xu | yea a abstraction layer | |
| 12:09:40 | alex_xu | that is why I put the traits sync in the cache before, now we pull that sync out.. | |
| 12:09:47 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types https://review.openstack.org/430608 | |
| 12:10:44 | cdent | mdbooth: in a round about way you have done a good deed. if your quota is one a day, you're done, good job, time for a beer. | |
| 12:22:30 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237 | |
| 12:24:25 | openstackgerrit | Danil Akhmetov proposed openstack/nova master: Use proper user and tenant in the owner section of libvirt.xml. https://review.openstack.org/399679 | |
| 12:27:28 | gibi | bauzas: Hi! thanks for looking at the scheduler hint API spec. I suggested a way to handle missing hints in the review. | |
| 12:28:50 | rfolco | cdent, review_begging++ https://review.openstack.org/#/c/415885/ | |
| 12:31:12 | cdent | rfolco: yup that's on my list for today | |
| 12:31:23 | rfolco | cdent, thx :) | |
| 12:32:56 | lyarwood | mdbooth: have you played around with lvm2 and sparse files yet? | |
| 12:33:14 | lyarwood | mdbooth: we are already creating thinp volumes AFAICT | |
| 12:34:25 | lyarwood | mdbooth: http://paste.openstack.org/show/601745/ | |
| 12:34:39 | openstackgerrit | Steven Webster proposed openstack/nova master: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/382853 | |
| 13:15:17 | jsheeren | hi, i'm having issues with nova-spiceproxy and spice-html5 using Windows instances. Everything is upside down, which makes it a little difficult to use the horizon console. Has anyone seen this as well? | |
| 13:16:21 | jsheeren | with using spice-html5 from git, i'm seeing the same upside down console | |
| 13:16:37 | jsheeren | with linux instances there is no problem at all, spice console works as expected | |
| 13:22:29 | mnaser | hi nova team, is there plans of cutting a stable/newton release soon? | |
| 13:22:47 | mnaser | we had a lot of problems with upgrades re this: https://review.openstack.org/#/c/408725/ -- we manually worked around it but it would be nice to not have to | |
| 13:28:35 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Use Cinder API v3 as default https://review.openstack.org/420201 | |
| 13:52:15 | Dinesh_Bhor | Vek: Hi, Could you please take a look at it? https://review.openstack.org/#/c/441750/ | |
| 13:55:44 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add test for inventory with invalid ResourceClass https://review.openstack.org/442498 | |
| 13:59:11 | tdurakov | paul-carlton2 eliqiao shaohe_feng pkoniszewski yuntongjin mdbooth kashyap kmartin claudiub tobasco tdurakov abhishekk mrhillsman raj_singh wznoinsk siva_krishnan live migration subteam meeting in 2 minutes on #openstack-meeting-3 | |
| 14:09:07 | cdent | edleafe gets the pedant of the day award | |
| 14:09:38 | edleafe | cdent: After all, I used to head the Department of Redundancy Department | |
| 14:10:31 | cdent | You sure you weren't the head chairperson? | |
| 14:10:49 | sfinucan | sdague: Could I ask you to take a look at this patch and dependencies? There are some merge conflicts in later patches and I don't want to push a 15 patch series rebase again https://review.openstack.org/#/c/431022 | |
| 14:11:16 | sfinucan | it's focused on deprecating remaining nova-net options (of which there are many) | |
| 14:12:14 | sdague | sfinucan: ok, I guess I wonder why it's in 8 patches? | |
| 14:12:38 | sfinucan | sdague: I figured that it would be easier to review in chunks. I can combine if not | |
| 14:14:36 | markus_z | Anyone here has a vagrant multinode setup with Ubuntu1604 and likes to share it? | |
| 14:15:00 | sdague | honestly, I think that all the deprecations in nova/conf/network.py as one patch might make it simpler | |
| 14:15:16 | sfinucan | sdague: Sure thing. Gimme two | |
| 14:15:38 | sdague | I also threw a quick comment in here - https://review.openstack.org/#/c/430914/3 | |
| 14:15:53 | sdague | just took me a second to realize how that could be not used any more | |
| 14:21:17 | sfinucan | sdague: Ah, I remember why I broke it up. The options were used by different components of nova-network and I gave that reason in the commit message (to avoid forcing people to go root that out themselves) | |
| 14:21:37 | sdague | sfinucan: ok, well it could just be a long commit message | |
| 14:21:45 | sdague | I'm fine with long commit messages | |
| 14:21:49 | sfinucan | Aye, that's what I'll do now :) | |
| 14:32:44 | johnthetubaguy | lyarwood mdbooth: thanks for you great comments on this one, would be awesome if you could take another peak at that cinder API spec: https://review.openstack.org/#/c/373203/ | |
| 14:32:58 | mdbooth | johnthetubaguy: Will do | |
| 14:33:20 | lyarwood | johnthetubaguy: ack will try to later today | |
| 14:35:06 | mriedem | edleafe: on https://review.openstack.org/#/c/442254/ i'm not using nova/tests/functional/api/openstack/placement | |
| 14:35:09 | johnthetubaguy | lyarwood: mdbooth: awesome, thank you | |
| 14:35:12 | mriedem | i'm testing the scheduler report client | |
| 14:35:44 | mriedem | well the gabbits anyway | |
| 14:37:21 | mriedem | edleafe: your test is testing the placement API directly, which is good, but the functional test i added is testing the report client, which is needed for the logic in there, as noted here https://review.openstack.org/#/c/441543/2/nova/tests/unit/scheduler/client/test_report.py@1242 | |
| 14:37:35 | cdent | mriedem, edleafe yeah seems we need both | |
| 14:37:44 | cdent | s/need/should have/ | |
| 14:42:16 | openstackgerrit | Rafael Folco proposed openstack/nova-specs master: DELETE all inventories for a resource provider https://review.openstack.org/415885 | |
| 14:44:03 | mriedem | sfinucan: we need to get this backported to ocata - breaks people using libvirt+xen that upgrade to ocata if they have new enough libvirt/qemu for virtlogd https://review.openstack.org/#/c/442209/ | |
| 14:44:14 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Fix indentation in conf/netconf https://review.openstack.org/430717 | |
| 14:44:15 | openstackgerrit | Stephen Finucane proposed openstack/nova master: netutils: Ignore 'use_ipv6' for network templates https://review.openstack.org/430910 | |
| 14:44:15 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate 'use_ipv6' https://review.openstack.org/430911 | |
| 14:44:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Ignore 'use_ipv6' for port filters https://review.openstack.org/430909 | |
| 14:44:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate 'allow_same_net_traffic' https://review.openstack.org/430913 | |
| 14:44:17 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Final cleanups in conf/network https://review.openstack.org/431023 | |
| 14:44:17 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Ignore 'allow_same_net_traffic' for port filters https://review.openstack.org/430912 | |
| 14:44:18 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate most 'network' option https://review.openstack.org/430914 | |
| 14:44:25 | mdbooth | kaboom! | |
| 14:44:27 | mriedem | i'd like to get that backported to ocata and released before a lot of people start upgrading | |
| 14:44:38 | sfinucan | sdague: ^^^ | |
| 14:44:44 | sfinucan | mriedem: Sure. I'll take a look now | |
| 14:46:08 | edleafe | mriedem: not sure I follow. The test you added is in ova/tests/functional/api/openstack/placement/test_report_client.py | |
| 14:46:32 | mriedem | edleafe: yeah which is a functional test on the scheduler report client | |
| 14:46:34 | mriedem | which is what we need | |
| 14:47:06 | mriedem | edleafe: as noted in the review, the functional test is testing the entire report client stack for that negative scenario, rather than just piecemeal unit tests - which are removed in that same change | |
| 14:47:10 | mriedem | because the unit tests can lie | |
| 14:49:04 | mriedem | bauzas: i plan on backport https://review.openstack.org/#/c/442035/ to ocata, do you have any issue with that? | |
| 14:49:09 | mriedem | *backporting | |
| 14:49:46 | ildikov | smcginnis: FYI, I added a few extra characters and whitespaces to that reno and now the test seems happy... :) | |
| 14:50:13 | smcginnis | ildikov: Oh good. I should take a look at how you did that. I wasn't able to get it to work. | |
| 14:50:24 | smcginnis | (and was too lazy to really figure it out) :) | |
| 14:50:59 | mriedem | smcginnis: you need the | for rst formatting | |
| 14:51:00 | sfinucan | mriedem: Yeah, that fix makes sense. Done | |
| 14:51:10 | mriedem | sfinucan: thanks | |
| 14:51:24 | mriedem | smcginnis: https://docs.openstack.org/developer/reno/usage.html#note-file-syntax | |
| 14:51:26 | mdbooth | mriedem: Speaking of backports, you had a W-1 on this Newton backport until the Ocata backport merged, which it now has: https://review.openstack.org/#/c/431540/ | |
| 14:51:31 | ildikov | mriedem: smcginnis: that was my guess too, but wasn't 100% sure | |
| 14:51:36 | smcginnis | ildikov, mriedem: So use a | and guve a little extra indent. | |
| 14:51:37 | sfinucan | mriedem: Someone should probably submit a patch to reno to use '|' in the default templates | |
| 14:51:49 | mriedem | sfinucan: go for it | |
| 14:51:51 | mriedem | i always use | | |
| 14:51:57 | mriedem | regardless of whether i need it or not | |