| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-01-11 | |||
| 22:25:38 | openstackgerrit | Eric Fried proposed openstack/nova master: Rip the report client out of SchedulerClient https://review.openstack.org/617042 | |
| 22:25:38 | openstackgerrit | Eric Fried proposed openstack/nova master: Commonize _update code path https://review.openstack.org/615705 | |
| 22:25:39 | openstackgerrit | Eric Fried proposed openstack/nova master: Use a static resource tracker in compute manager https://review.openstack.org/620711 | |
| 22:25:39 | openstackgerrit | Eric Fried proposed openstack/nova master: Rip out the SchedulerClient https://review.openstack.org/617049 | |
| 23:13:35 | openstackgerrit | Eric Fried proposed openstack/nova master: Reduce calls to placement from _ensure https://review.openstack.org/615677 | |
| 23:13:36 | openstackgerrit | Eric Fried proposed openstack/nova master: Commonize _update code path https://review.openstack.org/615705 | |
| 23:13:36 | openstackgerrit | Eric Fried proposed openstack/nova master: Consolidate inventory refresh https://review.openstack.org/615695 | |
| 23:13:37 | openstackgerrit | Eric Fried proposed openstack/nova master: Rip out the SchedulerClient https://review.openstack.org/617049 | |
| 23:13:37 | openstackgerrit | Eric Fried proposed openstack/nova master: Rip the report client out of SchedulerClient https://review.openstack.org/617042 | |
| 23:13:38 | openstackgerrit | Eric Fried proposed openstack/nova master: Use a static resource tracker in compute manager https://review.openstack.org/620711 | |
| 23:15:00 | fried_rice | jaypipes: This series ought to be ready again now ^ | |
| 23:16:12 | fried_rice | Caveat: I haven't gotten to a point where I can tox locally on my shiny new (Windows) system, and don't yet have other hardware on which to install Linux, so I didn't get to do any local testing. | |
| #openstack-nova - 2019-01-12 | |||
| 19:28:53 | openstackgerrit | Eric Fried proposed openstack/nova master: Consolidate inventory refresh https://review.openstack.org/615695 | |
| 19:28:53 | openstackgerrit | Eric Fried proposed openstack/nova master: Reduce calls to placement from _ensure https://review.openstack.org/615677 | |
| 19:28:54 | openstackgerrit | Eric Fried proposed openstack/nova master: Rip the report client out of SchedulerClient https://review.openstack.org/617042 | |
| 19:28:54 | openstackgerrit | Eric Fried proposed openstack/nova master: Commonize _update code path https://review.openstack.org/615705 | |
| 19:28:55 | openstackgerrit | Eric Fried proposed openstack/nova master: Use a static resource tracker in compute manager https://review.openstack.org/620711 | |
| 19:28:55 | openstackgerrit | Eric Fried proposed openstack/nova master: Rip out the SchedulerClient https://review.openstack.org/617049 | |
| 19:29:23 | efried1 | that oughtta fix the unit tests, silly mistake. | |
| 21:48:19 | openstackgerrit | Eric Fried proposed openstack/nova master: Turn off rp association refresh in nova-next https://review.openstack.org/616033 | |
| 21:59:51 | openstackgerrit | Eric Fried proposed openstack/nova master: Use a static resource tracker in compute manager https://review.openstack.org/620711 | |
| #openstack-nova - 2019-01-13 | |||
| 03:37:48 | openstackgerrit | Eric Fried proposed openstack/nova master: Use a static resource tracker in compute manager https://review.openstack.org/620711 | |
| 09:22:14 | openstackgerrit | garyk proposed openstack/nova master: Enhance exception raiased when invalid power state https://review.openstack.org/630496 | |
| 14:53:59 | openstackgerrit | garyk proposed openstack/nova master: Enhance exception raised when invalid power state https://review.openstack.org/630496 | |
| #openstack-nova - 2019-01-14 | |||
| 00:19:30 | eandersson | What kind of impact does getting additional filtering / weighting data have on scheduler performance? | |
| 00:19:54 | eandersson | e.g. flavor information | |
| 00:20:59 | eandersson | It's easy enough to test with a few computes and VMs, but not sure how to test it on more realistic PROD scales. | |
| 07:44:22 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Allow run metadata api per cell https://review.openstack.org/624612 | |
| 08:05:56 | maciejjozefczyk | jaypipes: hey. I did a little update on counters here: https://review.openstack.org/#/c/614167/ Please check that ;) Sorry for spamming a lot about this | |
| 11:48:59 | finucannot | bauzas: If you're about today, fancy pushing https://review.openstack.org/630154 and https://review.openstack.org/630138 ? | |
| 12:35:36 | jaypipes | maciejjozefczyk: no worries! will look at it shortly. | |
| 13:28:22 | sean-k-mooney | o/ | |
| 13:51:24 | gibi | mriedem: hi! welcome back! I'm trying to find a slot for a hangouts call about the bandwidth patches and need your input. Possible slots this week: Wednesday 18:00 UTC (after cdent's placement hangouts), Thursday 17:00 UTC, Friday 17:00 UTC | |
| 13:51:26 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix configure() called after DatabaseAtVersion fixture https://review.openstack.org/619723 | |
| 13:53:40 | gibi | mriedem: and one more not so good spot on Tuesday 17:00 UTC | |
| 13:55:07 | mriedem | any of those should work for me | |
| 13:56:47 | efried | n-sch meeting in 10 minutes in #openstack-meeting-alt | |
| 14:00:07 | gibi | mriedem: thanks | |
| 14:00:14 | gibi | mriedem: will post the final time on the ML | |
| 14:05:14 | openstackgerrit | caoyuan proposed openstack/nova stable/rocky: Update the description to make it more accuracy https://review.openstack.org/630681 | |
| 14:33:54 | efried | belmoreira: Howdy. Are you around today? | |
| 14:39:29 | asmita | gibi: Hi | |
| 14:41:17 | gibi | asmita: hu | |
| 14:41:20 | gibi | asmita: hi | |
| 14:42:25 | sean-k-mooney | adrianc: do you mind if i rebase the sriov patch series and start addressing some of stephenfin's nits | |
| 14:43:13 | asmita | gibi: How do we use this mehod?[1]:https://github.com/openstack/python-novaclient/blob/master/novaclient/tests/unit/utils.py#L30 | |
| 14:43:27 | asmita | method* | |
| 14:44:22 | sean-k-mooney | mriedem: are you ok with removing your procedual -2 in https://review.openstack.org/#/c/616120/ since the spec has been merged and blueprint approved? | |
| 14:44:24 | mriedem | you don't | |
| 14:44:26 | mriedem | asmita: ^ | |
| 14:44:42 | mriedem | it's patching the mock library at import time to avoid adding tests which call invalid assert methods | |
| 14:44:54 | mriedem | which used to be a problem, but is fixed in latest mock i believe | |
| 14:45:20 | gibi | asmita: mriedem was faster. But I think the mock library already fixed to reject those wrong asserts | |
| 14:45:36 | gibi | asmita: so you don't even need that patch any more | |
| 14:45:49 | mriedem | sean-k-mooney: done | |
| 14:46:49 | sean-k-mooney | mriedem: thanks it need to be rebased and some nits addressed but i think the feature is effectivly complete. we just need to test it properly and clean it up at this point. | |
| 14:48:16 | asmita | Okay.There is a test scenario in test_hypervisors.py file of novaclient which I need to implement.I need to raise exception in that test case..But because of the fixtures,I get a error as "No mock address".So how do I overcome this situation? | |
| 14:49:26 | gibi | asmita: do you have a patch up on gerrit I can look at? | |
| 14:51:12 | asmita | gibi: Not yet. | |
| 14:51:27 | gibi | asmita: just push the not working code up so I can look at it | |
| 14:56:57 | asmita | gibi: Please have a look.https://github.com/openstack/python-novaclient/blob/master/novaclient/tests/unit/utils.py#L30 | |
| 14:57:06 | asmita | gibi: https://github.com/openstack/python-novaclient/blob/master/novaclient/tests/unit/utils.py#L30 | |
| 14:57:29 | asmita | gibi : Sorry wron link | |
| 14:57:34 | asmita | wrong link | |
| 14:58:35 | asmita | gibi: paste.openstack.org/show/742306..Thanks. | |
| 14:58:45 | gibi | asmita: looking... | |
| 14:59:49 | tobias-urdin | any good way to actually health check novncproxy? we've had some fallouts where it stops working with "handler exception: [Errno 32] Broken pipe" on console requests | |
| 15:00:17 | tobias-urdin | i guess i'll have to look into some doing some vnc client that does a handshake and tries to find some response that could be checked | |
| 15:04:32 | gibi | asmita: this patch you pasted works for me, both py27 and py35 test passes for me on it | |
| 15:05:21 | gibi | asmita: http://paste.openstack.org/show/742307/ | |
| 15:07:06 | asmita | gibi :Thanks . | |
| 15:07:24 | gibi | asmita: so I don't know why it doesn't work for you | |
| 15:09:10 | asmita | gibi : My concern here is whether it is okay to mock.patch and raise exception,the way I have done? | |
| 15:13:38 | asmita | gibi : Thanks.I will look into it. | |
| 15:17:35 | gibi | amotoki: is the self.cs.hypervisors.search() you call and the novaclient.v2.hypervisors.HypervisorManager.search that you mocked the same thing? if they are the same then you are not testing anything (except the mock library) | |
| 15:17:40 | gibi | amotoki: sorry | |
| 15:17:45 | gibi | asmita: ^^ | |
| 15:21:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Allow run metadata api per cell https://review.openstack.org/624612 | |
| 15:22:26 | mriedem | dansmith: the channel topic is pretty out of date, maybe we should just link to https://etherpad.openstack.org/p/nova-runways-stein ? | |
| 15:22:36 | mriedem | "Current review runways: https://etherpad.openstack.org/p/nova-runways-stein" or something | |
| 15:22:40 | dansmith | aye | |
| 15:22:49 | dansmith | will update in a bit | |
| 15:22:59 | mriedem | thanks | |
| 15:27:04 | mriedem | easy runway bp change with a +2 https://review.openstack.org/#/c/624612/ - run meta-api per cell | |
| 15:27:10 | openstackgerrit | Jan Gutter proposed openstack/os-vif master: Extend port profiles with datapath offload type https://review.openstack.org/572081 | |
| 15:27:48 | belmoreira | efried: hi | |
| 15:28:25 | efried | belmoreira: Greetings. Today in the nova-scheduler meeting we brought up a couple of patches that CERN had showed interest in. | |
| 15:28:53 | efried | We were looking to find out if y'all could (re)deploy them to make sure they do what's intended and don't break the world. | |
| 15:29:00 | efried | (jaypipes heads up) | |
| 15:29:48 | efried | The first one was the placement traffic improvement series starting at https://review.openstack.org/#/c/615677/ | |
| 15:30:11 | efried | but | |
| 15:30:11 | efried | IIRC, you had deployed it and demonstrated the expected improvements, which is great. | |
| 15:30:22 | efried | there was some concern about how it would behave in large ironic deployments. | |
| 15:30:36 | efried | Late last week I made some updates to hopefully take care of that | |
| 15:31:25 | efried | Last I recall, you were about to start deploying to your ironic nodes - not sure how far you got on that or whether you stalled it because of the perceived problems with the patches. | |
| 15:31:43 | belmoreira | efried: that are great news | |
| 15:31:54 | efried | Anyway, we were hoping you could redeploy the series in its current form to see if it's copacetic | |
| 15:34:03 | belmoreira | before the end of the year I was trying it with the ironic compute nodes. It was in our small test cell, and it was OK. Scalability issues will only be noticed in our prod instance. Was planning to do it next week. | |
| 15:34:37 | belmoreira | will check the new code, thanks. | |
| 15:36:16 | efried | belmoreira: Great. | |