| *** betherly has quit IRC | 00:04 | |
| *** slaweq has quit IRC | 00:12 | |
| *** slaweq has joined #openstack-nova | 00:13 | |
| *** k_mouza has quit IRC | 00:16 | |
| *** mlavalle has quit IRC | 00:26 | |
| prometheanfire | well, devspace works | 00:28 |
|---|---|---|
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Spec for cross-cell resize https://review.openstack.org/616037 | 00:29 |
| mriedem | yay for upper-constraints and integration testing | 00:30 |
| prometheanfire | https://dev.gentoo.org/~prometheanfire/images/nova-reqs.svg | 00:30 |
| prometheanfire | :D | 00:30 |
| mriedem | a lot of that stuff is probably not really required | 00:30 |
| mriedem | depending on what you use | 00:30 |
| mriedem | we just don't use 'extras' | 00:30 |
| prometheanfire | ya | 00:30 |
| prometheanfire | dep tree, so deps of deps mostly | 00:31 |
| mriedem | but yeah if you pip install you're in for some frun | 00:31 |
| mriedem | *fun | 00:31 |
| mriedem | dansmith: i have birthed it https://review.openstack.org/#/c/616037/ | 00:31 |
| mriedem | i shall call it, damien | 00:31 |
| prometheanfire | dep of testtools/subunit it looks like | 00:31 |
| prometheanfire | lol | 00:31 |
| dansmith | mriedem: remind me tomorrow.. if there is a tomorrow. | 00:32 |
| mriedem | i don't think i'm going to do anything with it until summit | 00:32 |
| mriedem | i just wanted to get it out there | 00:32 |
| dansmith | ack | 00:32 |
| *** mriedem has quit IRC | 00:36 | |
| *** irclogbot_2 has quit IRC | 00:39 | |
| *** tetsuro has joined #openstack-nova | 00:42 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Use SleepingFixture instead of mocking _ThreadingEvent.wait https://review.openstack.org/615724 | 00:47 |
| *** erlon has quit IRC | 00:51 | |
| *** Swami has quit IRC | 01:01 | |
| *** TuanDA has joined #openstack-nova | 01:16 | |
| *** bhagyashris has joined #openstack-nova | 01:17 | |
| *** wangy_ has joined #openstack-nova | 01:18 | |
| *** takashin has joined #openstack-nova | 01:22 | |
| *** tetsuro has quit IRC | 01:33 | |
| *** Dinesh_Bhor has joined #openstack-nova | 01:36 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Improve formats of the Compute API guide https://review.openstack.org/615809 | 01:39 |
| *** brinzhang has joined #openstack-nova | 01:40 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add method to allow fetch root_volume BDM by instance_uuid https://review.openstack.org/614672 | 01:45 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Bump compute service to indicate attach/detach root volume is supported https://review.openstack.org/614750 | 01:45 |
| *** markvoelker has quit IRC | 02:00 | |
| *** markvoelker has joined #openstack-nova | 02:00 | |
| *** markvoelker has quit IRC | 02:02 | |
| *** cfriesen has quit IRC | 02:06 | |
| *** mhen has quit IRC | 02:07 | |
| *** dave-mccowan has joined #openstack-nova | 02:16 | |
| *** dave-mccowan has quit IRC | 02:27 | |
| *** threestrands has joined #openstack-nova | 02:30 | |
| *** zhanglong has joined #openstack-nova | 02:31 | |
| *** jiaopengju has quit IRC | 02:40 | |
| *** jiaopengju has joined #openstack-nova | 02:43 | |
| *** zhanglong has quit IRC | 02:44 | |
| *** mrsoul has quit IRC | 02:48 | |
| *** zhanglong has joined #openstack-nova | 02:59 | |
| *** wangy_ has quit IRC | 03:01 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628 | 03:11 |
| *** prometheanfire has left #openstack-nova | 03:12 | |
| *** hongbin has joined #openstack-nova | 03:12 | |
| *** dave-mccowan has joined #openstack-nova | 03:24 | |
| *** dave-mccowan has quit IRC | 03:36 | |
| *** tbachman has quit IRC | 03:43 | |
| *** brinzhang has quit IRC | 03:47 | |
| *** owalsh_ has joined #openstack-nova | 03:47 | |
| *** brinzhang has joined #openstack-nova | 03:48 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628 | 03:49 |
| *** brinzh has joined #openstack-nova | 03:51 | |
| *** owalsh has quit IRC | 03:51 | |
| *** brinzhang has quit IRC | 03:53 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Improve formats of the Compute API guide https://review.openstack.org/615809 | 04:01 |
| *** Dinesh_Bhor has quit IRC | 04:15 | |
| *** bhagyashris has quit IRC | 04:20 | |
| *** zhanglong has quit IRC | 04:21 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add a link to the doc contrib guide https://review.openstack.org/616063 | 04:21 |
| *** jmlowe has quit IRC | 04:21 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:34 | |
| *** sapd1_ has quit IRC | 04:35 | |
| *** sapd1 has joined #openstack-nova | 04:36 | |
| *** bhagyashris has joined #openstack-nova | 04:40 | |
| *** hongbin has quit IRC | 04:47 | |
| *** TuanDA has quit IRC | 05:00 | |
| *** hamzy has joined #openstack-nova | 05:18 | |
| *** rnoriega- has quit IRC | 05:28 | |
| *** med_ has quit IRC | 05:29 | |
| *** rnoriega has joined #openstack-nova | 05:29 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289 | 05:40 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | 05:41 |
| *** wznoinsk has quit IRC | 05:46 | |
| gmann | alex_xu: you around for API office hour today ? | 05:57 |
| alex_xu | gmann: yea | 05:57 |
| gmann | alex_xu: anything specific you have to discuss otherwise i would like to skip, little busy for summit things | 05:58 |
| *** zhanglong has joined #openstack-nova | 06:03 | |
| alex_xu | gmann: really nothing specific | 06:03 |
| gmann | alex_xu: ok, i will skip then. btw you will come to summit right ? | 06:06 |
| openstackgerrit | Merged openstack/nova master: [Trivial Fix] Correct spelling error of "should" and "resource" https://review.openstack.org/614899 | 06:09 |
| *** tetsuro has joined #openstack-nova | 06:14 | |
| *** TuanDA has joined #openstack-nova | 06:17 | |
| *** threestrands has quit IRC | 06:23 | |
| alex_xu | gmann: no, I won't, I didn't get budget this time | 06:42 |
| gmann | alex_xu: oh | 06:43 |
| *** Luzi has joined #openstack-nova | 06:51 | |
| *** lyarwood has quit IRC | 06:53 | |
| *** andreaf has quit IRC | 06:53 | |
| *** andreaf has joined #openstack-nova | 06:55 | |
| *** tbachman has joined #openstack-nova | 07:03 | |
| *** tbachman has quit IRC | 07:07 | |
| *** tbachman has joined #openstack-nova | 07:09 | |
| *** tbachman has quit IRC | 07:11 | |
| *** dpawlik has joined #openstack-nova | 07:11 | |
| *** alexchadin has joined #openstack-nova | 07:15 | |
| *** dpawlik has quit IRC | 07:17 | |
| *** maciejjozefczyk has joined #openstack-nova | 07:17 | |
| *** dpawlik has joined #openstack-nova | 07:17 | |
| *** ccamacho has joined #openstack-nova | 07:31 | |
| *** pcaruana has joined #openstack-nova | 07:36 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add method to allow fetch root_volume BDM by instance_uuid https://review.openstack.org/614672 | 07:51 |
| *** adrianc has joined #openstack-nova | 07:53 | |
| *** helenafm has joined #openstack-nova | 08:19 | |
| *** BlackDex has quit IRC | 08:20 | |
| *** lpetrut has joined #openstack-nova | 08:20 | |
| *** ralonsoh has joined #openstack-nova | 08:33 | |
| *** BlackDex has joined #openstack-nova | 08:33 | |
| *** sahid has joined #openstack-nova | 08:36 | |
| *** sahid has quit IRC | 08:37 | |
| *** sahid has joined #openstack-nova | 08:37 | |
| *** sahid has joined #openstack-nova | 08:37 | |
| *** sahid has quit IRC | 08:38 | |
| *** Luzi has quit IRC | 08:40 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Bump compute service to indicate attach/detach root volume is supported https://review.openstack.org/614750 | 08:41 |
| *** sahid has joined #openstack-nova | 08:42 | |
| *** sahid has quit IRC | 08:42 | |
| *** sahid has joined #openstack-nova | 08:44 | |
| *** k_mouza has joined #openstack-nova | 08:48 | |
| *** jpena|off is now known as jpena | 08:50 | |
| *** dims has quit IRC | 08:52 | |
| *** dims has joined #openstack-nova | 08:53 | |
| *** lyarwood has joined #openstack-nova | 08:54 | |
| *** dims has quit IRC | 08:58 | |
| *** jangutter has quit IRC | 08:58 | |
| *** jangutter has joined #openstack-nova | 08:58 | |
| *** dims has joined #openstack-nova | 08:59 | |
| *** kukacz has quit IRC | 09:04 | |
| *** kukacz has joined #openstack-nova | 09:05 | |
| *** alexchadin has quit IRC | 09:06 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add method to allow fetch root_volume BDM by instance_uuid https://review.openstack.org/614672 | 09:08 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Bump compute service to indicate attach/detach root volume is supported https://review.openstack.org/614750 | 09:09 |
| *** ttsiouts has joined #openstack-nova | 09:10 | |
| *** zhanglong has quit IRC | 09:15 | |
| openstackgerrit | Adrian Chiris proposed openstack/nova master: add get_pci_requests_from_vifs to request.py https://review.openstack.org/609166 | 09:20 |
| openstackgerrit | Adrian Chiris proposed openstack/nova master: Allow per-port modification of vnic_type and profile https://review.openstack.org/607365 | 09:20 |
| openstackgerrit | Adrian Chiris proposed openstack/nova master: Add free for claimed, allocated devices https://review.openstack.org/616120 | 09:20 |
| *** k_mouza has quit IRC | 09:21 | |
| *** owalsh_ is now known as owalsh | 09:23 | |
| *** k_mouza has joined #openstack-nova | 09:28 | |
| *** takashin has left #openstack-nova | 09:30 | |
| *** tetsuro has quit IRC | 09:33 | |
| *** ttx has quit IRC | 09:37 | |
| *** dpawlik has quit IRC | 09:38 | |
| *** bhagyashris has quit IRC | 09:41 | |
| *** Luzi has joined #openstack-nova | 09:41 | |
| *** dpawlik has joined #openstack-nova | 09:42 | |
| *** panda|off is now known as panda | 09:43 | |
| *** Dinesh_Bhor has quit IRC | 09:43 | |
| *** ttsiouts has quit IRC | 09:44 | |
| *** derekh has joined #openstack-nova | 09:44 | |
| *** ttsiouts has joined #openstack-nova | 09:44 | |
| *** ttsiouts_ has joined #openstack-nova | 09:48 | |
| *** ttsiouts has quit IRC | 09:49 | |
| *** ttx has joined #openstack-nova | 09:50 | |
| *** cdent has joined #openstack-nova | 09:58 | |
| *** k_mouza has quit IRC | 09:59 | |
| *** k_mouza has joined #openstack-nova | 10:01 | |
| *** ttsiouts_ has quit IRC | 10:07 | |
| *** ttsiouts has joined #openstack-nova | 10:07 | |
| *** sahid has quit IRC | 10:08 | |
| *** sahid has joined #openstack-nova | 10:09 | |
| *** TuanDA has quit IRC | 10:10 | |
| *** ttsiouts_ has joined #openstack-nova | 10:10 | |
| *** ttsiouts has quit IRC | 10:12 | |
| *** ttsiouts_ has quit IRC | 10:13 | |
| *** adrianc has quit IRC | 10:14 | |
| *** ttsiouts has joined #openstack-nova | 10:20 | |
| *** erlon has joined #openstack-nova | 10:27 | |
| *** tssurya has joined #openstack-nova | 10:28 | |
| *** Dinesh_Bhor has joined #openstack-nova | 10:37 | |
| *** dtantsur|afk is now known as dtantsur | 10:47 | |
| *** alexchadin has joined #openstack-nova | 10:50 | |
| *** priteau has joined #openstack-nova | 10:53 | |
| *** davidsha has joined #openstack-nova | 11:06 | |
| *** sapd1 has quit IRC | 11:12 | |
| *** sapd1 has joined #openstack-nova | 11:12 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | 11:25 |
| *** jwongz has quit IRC | 11:29 | |
| *** Dinesh_Bhor has quit IRC | 11:33 | |
| *** ttsiouts has quit IRC | 11:36 | |
| *** ttsiouts has joined #openstack-nova | 11:37 | |
| *** k_mouza has quit IRC | 11:39 | |
| *** cdent has quit IRC | 11:40 | |
| *** ttsiouts has quit IRC | 11:42 | |
| *** brinzh has quit IRC | 11:55 | |
| *** k_mouza has joined #openstack-nova | 12:00 | |
| openstackgerrit | lvxianguo proposed openstack/python-novaclient master: Update the documentation bug tag https://review.openstack.org/616174 | 12:04 |
| *** ttsiouts has joined #openstack-nova | 12:08 | |
| *** dpawlik has quit IRC | 12:11 | |
| *** dpawlik has joined #openstack-nova | 12:12 | |
| *** k_mouza has quit IRC | 12:12 | |
| *** dpawlik has quit IRC | 12:16 | |
| *** alexchadin has quit IRC | 12:17 | |
| *** chudler has joined #openstack-nova | 12:23 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Make _instances_cores_ram_count() be smart about cells https://review.openstack.org/569055 | 12:37 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: [WIP] Add os_compute_api:servers:create:cell_down policy https://review.openstack.org/614783 | 12:37 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: WIP: API microversion bump for handling-down-cell https://review.openstack.org/591657 | 12:37 |
| *** dpawlik has joined #openstack-nova | 12:39 | |
| *** dpawlik has quit IRC | 12:39 | |
| *** dpawlik has joined #openstack-nova | 12:40 | |
| *** cdent has joined #openstack-nova | 12:51 | |
| *** jpena is now known as jpena|lunch | 12:53 | |
| sean-k-mooney | bauzas: cdent gibi for https://review.openstack.org/#/c/610034/10 are we ok with its current form or do we want another revision | 12:56 |
| *** Luzi_ has joined #openstack-nova | 12:57 | |
| sean-k-mooney | bauzas: cdent gibi any change i make to the nova version will also need to be appied to the placement version and i dont really want to kick of ci jobs for all the stuff that depnd on this on the placemnt side un less i need too | 12:57 |
| cdent | sean-k-mooney: I'm not sure. gibi seemed to have some reservations and it might make sense to do his simpler version in nova and the run_once version in placement (where we have longer term needs) | 12:57 |
| sean-k-mooney | that said i also plan to back port this so i what this to be right | 12:57 |
| sean-k-mooney | cdent: nova need to have it done for the nova-api too however | 12:58 |
| sean-k-mooney | do we want two ifs one for each both depending on private data | 12:58 |
| * cdent looks at bauzas and gibi | 12:59 | |
| cdent | I don't have a strong preference, except that we move it forward quickly in some way that satisfies us and recognize that it is okay to fix it more later. | 12:59 |
| sean-k-mooney | cdent: ya same. this has been blocking downstream ci for almost a month at this point. i will have to consider applying a donwstream only fix if we dont merge something with the next week | 13:00 |
| *** Luzi has quit IRC | 13:00 | |
| cdent | Yeah, we seem to have quite the backlog at the moment | 13:01 |
| cdent | sean-k-mooney: I wish I could help to move things a bit more | 13:05 |
| sean-k-mooney | cdent: i need to run to the bank so ill be back in a while. efried spotted a typo so i was going to resping for that and ill add the extra docs that gibi asked for too. unfortunetly that will cause curn on the placement side is that ok with you? | 13:06 |
| sean-k-mooney | cdent: and no worries | 13:06 |
| sean-k-mooney | you have help move it along alot already | 13:06 |
| cdent | no worries on the placement side, I'll watch that and make sure it's properly rebased etc. The test load from placement is currently very light and fast, so no issues there. | 13:07 |
| sean-k-mooney | thanks :) ok got to run brb | 13:07 |
| *** Luzi_ has quit IRC | 13:10 | |
| *** Luzi has joined #openstack-nova | 13:10 | |
| *** k_mouza has joined #openstack-nova | 13:13 | |
| gibi | sean-k-mooney: my -1 is mostly due to the problem with raised exception from the wrapped function | 13:13 |
| gibi | sean-k-mooney: the rest is mostly thinking out loud different alternatives | 13:13 |
| sean-k-mooney | gibi: well we dont want to catch exceptions when we invoke the wrapped function the first time. but we could catch exceptions when logging i guess | 13:14 |
| gibi | sean-k-mooney: my problem is that if the first call raises then we allow a second call | 13:15 |
| gibi | sean-k-mooney: because called only set _after_ the wrapped function returns | 13:15 |
| sean-k-mooney | gibi: yes because this is a general decorator and we should not assume that means another call to the same fucntion will also raise | 13:15 |
| gibi | sean-k-mooney: for me the contract of that decorator is that the decorated function will be called at most once. But the actual implementation allows any number of calls to the wrapped function if the wrapped function raises | 13:16 |
| sean-k-mooney | gibi: yes the order of setting called after was very deliberate | 13:16 |
| gibi | so if my func() has some code in it that can only be run once, but also my func() can raise then I cannot use the new decorator | 13:17 |
| sean-k-mooney | gibi: for me it will be called at most once if it succeeds and the fuctor object will be in its orignal state if not | 13:17 |
| artom | lyarwood, could you hit https://review.openstack.org/#/c/587249/ ? It's the pike version of a backport you +2'ed for Queens - I'll try and get Matt R as well when he comes onlone | 13:17 |
| artom | (Pike is still a thing, right? I know Ocata is EM now) | 13:18 |
| sean-k-mooney | gibi: you can but if you caugt the excption that by definitoin means its recoverable | 13:18 |
| lyarwood | artom: yeah tis and I'll get to it after this call | 13:18 |
| sean-k-mooney | gibi: if its not recoverable you should end the program | 13:18 |
| artom | lyarwood, much thanks! | 13:18 |
| *** jpena|lunch is now known as jpena | 13:18 | |
| sean-k-mooney | gibi: so that means you should be able to call the function again. | 13:18 |
| gibi | sean-k-mooney: but allowing to call the function _again_ is something the run_once decorator should not allow at least that is what the name of the decorator suggests | 13:19 |
| sean-k-mooney | gibi: the contract i wanted to give is this fucntion will run once sucessfully | 13:20 |
| gibi | sean-k-mooney: I don't want to block your CI so I'm fine if you can document the behavior of the decorator precisely | 13:20 |
| sean-k-mooney | sure ill add more documentaion | 13:20 |
| sean-k-mooney | i could may be make the setting of the called value explcit but i was concered that people would complain it was over enginerred. my oringinal solution was a try block | 13:21 |
| sean-k-mooney | then an if and then i was asked for a decorator | 13:22 |
| gibi | sean-k-mooney: I'm one of the guy who feels it is overengineered so please do not add more configurability to the decorator, add just more documentation what is the intention of the decorator. Like that a run only counts if it wasn't raised | 13:23 |
| sean-k-mooney | ok will do thanks ill get this back up in about 2 hours | 13:24 |
| gibi | sean-k-mooney: ping me and I will try to +2 it quickly | 13:24 |
| *** dpawlik has quit IRC | 13:35 | |
| *** dpawlik has joined #openstack-nova | 13:35 | |
| *** cdent has quit IRC | 13:37 | |
| *** ttsiouts has quit IRC | 13:39 | |
| *** dpawlik has quit IRC | 13:40 | |
| artom | lyarwood, there's the one below it as well ;) https://review.openstack.org/#/c/587248/1 | 13:42 |
| artom | Cheers! | 13:43 |
| lyarwood | artom: np LGTM | 13:43 |
| *** pcaruana has quit IRC | 13:45 | |
| *** jmlowe has joined #openstack-nova | 13:47 | |
| *** ttsiouts has joined #openstack-nova | 13:57 | |
| *** mriedem has joined #openstack-nova | 13:58 | |
| *** pcaruana has joined #openstack-nova | 14:00 | |
| *** faridda has joined #openstack-nova | 14:09 | |
| *** tbachman has joined #openstack-nova | 14:17 | |
| *** awaugama has joined #openstack-nova | 14:24 | |
| *** pcaruana has quit IRC | 14:33 | |
| *** pcaruana has joined #openstack-nova | 14:34 | |
| *** cdent has joined #openstack-nova | 14:36 | |
| *** k_mouza has quit IRC | 14:39 | |
| *** jistr is now known as jistr|call | 15:00 | |
| *** priteau has quit IRC | 15:01 | |
| *** cfriesen has joined #openstack-nova | 15:02 | |
| *** mlavalle has joined #openstack-nova | 15:03 | |
| *** priteau has joined #openstack-nova | 15:03 | |
| tomtom001 | lyarwood: hello I was wondering you could say how the encryption on nova works with nova libvirt and ceph... we've noticed really slow performance when using luks encryption... I've turned off xts,and only use aes but it doesn't seem to change anything, what specifically needs to be in place to get a fast encryption for disks? | 15:06 |
| lyarwood | tomtom001: Which version are you using and slow in what sense? Read / Writes / time taken to attach volumes etc? | 15:07 |
| *** faridda has quit IRC | 15:08 | |
| sean-k-mooney | tomtom001: does the host system have aes-ni processor instruction or is this older hardware. there are a log of vairibles that can come into play here | 15:12 |
| *** ttsiouts has quit IRC | 15:15 | |
| *** ttsiouts has joined #openstack-nova | 15:16 | |
| *** ttsiouts has quit IRC | 15:16 | |
| *** ttsiouts has joined #openstack-nova | 15:16 | |
| *** READ10 has joined #openstack-nova | 15:17 | |
| tomtom001 | So they are DellFx2 servers, so recent hardware. We are using Queens as installed from Openstack-ansible and using Ceph luminous. i/o performance is 1/20th the speed of an unencrypted volume. | 15:18 |
| tomtom001 | volume creation and attach time seems normal. | 15:18 |
| sean-k-mooney | tomtom001: that would seam to imply that the perfromace pelenty is coming form the encryption overhead which should be low if its hardware offloaded but would be non trivial if aes-ni was not available | 15:22 |
| *** dpawlik has joined #openstack-nova | 15:22 | |
| *** dpawlik has quit IRC | 15:22 | |
| *** dpawlik has joined #openstack-nova | 15:23 | |
| cdent | efried: do open questions remain on your refresh stack or is it fairly straightforward from here? I'm trying to optimize my time before leaving for berlin (tomorrow) | 15:24 |
| efried | cdent: I'm kind of a mess, need to go back through the emails and the series. But I *think* the only remaining questions are whether to make the timer zero by default for new installs; and/or make it the *only* thing - i.e. remove the option entirely and only refresh the cache when something happens. | 15:25 |
| *** Luzi has quit IRC | 15:26 | |
| efried | cdent: and I've convinced the ptb that, if either of those things comes to pass, they should be done at least separately, and probably later once these have had some field time with CERN/Vexx | 15:26 |
| cdent | makes sense | 15:26 |
| *** jistr|call is now known as jistr | 15:29 | |
| sean-k-mooney | efried: for what its worth im fine with waiting to change the default but in T we proably shoudl change the default to 0 | 15:30 |
| efried | sean-k-mooney: I don't even mind doing it earlier than that if we can demonstrate some traction in a prod env or two. | 15:31 |
| sean-k-mooney | that said cern/vexxhost wont deploy stein right away so it would be late in T before we got any feedback | 15:31 |
| efried | sean-k-mooney: I may be able to backport this code. | 15:32 |
| efried | sean-k-mooney: Not to merge necessarily, but at least so they can pull it into their env. | 15:32 |
| sean-k-mooney | right the catalyst for changing the default would production use showing it works for them | 15:32 |
| tomtom001 | sean-k-mooney - right so hardware offloaded would just be hardware encryption, we are looking at it from the cinder luks ceph side. | 15:39 |
| sean-k-mooney | tomtom001: well hardware offload would acclerate the luks part | 15:40 |
| mriedem | bauzas: do you care about this at all? you added the original lazy-load scheduler report client stuff https://review.openstack.org/#/c/615641/ | 15:40 |
| sean-k-mooney | cinder and ceph should not chagne if you enable encryption | 15:40 |
| bauzas | mriedem: for sure, I'm interested but I have a long customer issue | 15:40 |
| *** ralonsoh has quit IRC | 15:41 | |
| efried | mriedem: bauzas and I discussed it yesterday. I'm not sure whether I succeeded in convincing him that we weren't actually deferring importing the modules until method call. But he also admitted the only reason we were doing it at all was to avoid a circular import, which is clearly not an issue anymore. | 15:41 |
| *** dtantsur is now known as dtantsur|afk | 15:42 | |
| *** david-lyle has quit IRC | 15:42 | |
| efried | btw, at some point when I get breathing room, I'm going to rip out the SchedulerClient entirely. At the top of my series, it only has methods to the query client (all report client passthrough methods are gone). And I want to suck up to jaypipes some more anyway. | 15:43 |
| mriedem | i think that's the nit todo i just left on that change | 15:43 |
| mriedem | eventually remove the proxy | 15:43 |
| cdent | I'm glad we seem to have agreement on that idea | 15:44 |
| efried | yeah, for sure. I've looked into it before and it was going to be nontrivial, which is why I haven't done it in this series. | 15:44 |
| efried | but this series will make it a little bit easier. | 15:44 |
| *** ivve has joined #openstack-nova | 15:44 | |
| bauzas | efried: mriedem: +Wd | 15:44 |
| mriedem | well looky there | 15:45 |
| efried | bauzas: Thank you sir | 15:45 |
| bauzas | sorry for not having a lot of time for upstream :( | 15:45 |
| *** dklyle has joined #openstack-nova | 15:47 | |
| *** munimeha1 has joined #openstack-nova | 15:52 | |
| * mriedem goes to run some errands | 15:53 | |
| *** mriedem has quit IRC | 15:53 | |
| *** ralonsoh has joined #openstack-nova | 15:54 | |
| *** k_mouza has joined #openstack-nova | 15:56 | |
| tomtom001 | sean-k-mooney Sorry i'm not understanding, from what I think is supposed to happen, is that qemu is supposed to handle luks encryption on the fly on a disk correct? | 15:56 |
| *** gyee has joined #openstack-nova | 15:58 | |
| sean-k-mooney | tomtom001: correct but that was only added to openstack in pike see https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/libvirt-qemu-native-luks.html | 15:59 |
| tomtom001 | ok, so I'm running queens so that should be there. | 15:59 |
| tomtom001 | or no it's not implemented yet? | 15:59 |
| sean-k-mooney | tomtom001: no it should be there | 16:00 |
| tomtom001 | so on one cpu host i've removed the xts module to ensure only the aes module is loaded, is there anything else I can check in qemu or libvirt or nova to ensure that encryption is operating normally? | 16:01 |
| tomtom001 | qemu 2.11 is installed | 16:01 |
| tomtom001 | libvirt 4.0.0 and nova compute from stale/queens | 16:02 |
| tomtom001 | *stable | 16:02 |
| sean-k-mooney | there versions of the software you are running should all support it | 16:02 |
| tomtom001 | are there settings I need to check for this? | 16:03 |
| sean-k-mooney | not that i a aware of but lyarwood might be able to provide more info | 16:08 |
| sean-k-mooney | you will see the encyption nebaled in the xml/qemu command line but i dont know of anything you need to do to improve performance | 16:08 |
| *** lpetrut has quit IRC | 16:14 | |
| *** imacdonn has quit IRC | 16:17 | |
| *** imacdonn has joined #openstack-nova | 16:17 | |
| tomtom001 | sean-k-mooney thank you, lyarwood, you have anything I can check or something you'd like to look at to tell you? | 16:19 |
| *** mriedem has joined #openstack-nova | 16:26 | |
| *** pcaruana has quit IRC | 16:38 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send rp_uuid in the port binding https://review.openstack.org/569459 | 16:42 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | 16:42 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Calculate port_id rp_uuid mapping for binding https://review.openstack.org/616239 | 16:42 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Pass allocations to neturonv2 api https://review.openstack.org/616240 | 16:42 |
| efried | mriedem: Update on the nova-next business... | 16:50 |
| efried | https://review.openstack.org/#/c/616033/ see latest comment | 16:50 |
| efried | tldr I don't think the tweak is working, or I'm a dolt and looking in the wrong place. | 16:50 |
| efried | However, having looked at the placement logs for the nova-next runs in the whole series, I see a major drop at the "Reduce calls to placement from _ensure" patch, which is what I would expect; and actually, I wouldn't expect any of the CI jobs to see a significant drop from setting refresh=0 because they're getting refreshed all the time because they're going crazy doing real stuffs, so the refreshing is happening anyway and the | 16:52 |
| efried | The refresh=0 thing is really only going to see a big benefit in steady state, and (afaik) we don't have any CI jobs that just... sit there. | 16:53 |
| mriedem | i've had issues with using post-config and NOVA_CPU_CONF in the nova-live-migration job... | 16:54 |
| efried | mriedem: I found a cells job that was using something similar and it seemed to be working (was able to verify the conf change was made) | 16:56 |
| efried | mriedem: http://logs.openstack.org/33/616033/1/check/nova-cells-v1/fb84e57/logs/etc/nova/nova-cells.conf.txt.gz is getting vif_plugging_timeout = 0 from playbooks/legacy/nova-cells-v1/run.yaml | 16:56 |
| mriedem | devstack is processing the post-config | 16:58 |
| mriedem | http://logs.openstack.org/33/616033/1/check/nova-next/6e893b8/logs/devstacklog.txt.gz#_2018-11-07_00_24_44_044 | 16:58 |
| efried | so... wtf? | 17:00 |
| mriedem | it looks like that post-config happens first, | 17:00 |
| mriedem | and then other stuff is writing to nova-cpu.conf | 17:01 |
| mriedem | so my guess is that devstack is overwriting it... | 17:01 |
| efried | ah http://logs.openstack.org/33/616033/1/check/nova-next/6e893b8/logs/devstacklog.txt.gz#_2018-11-07_00_26_29_702 | 17:01 |
| efried | we blow it away later :( | 17:01 |
| efried | what's the magic env var for the nova.conf? | 17:02 |
| mriedem | yup http://logs.openstack.org/33/616033/1/check/nova-next/6e893b8/logs/devstacklog.txt.gz#_2018-11-07_00_26_29_702 | 17:02 |
| mriedem | $NOVA_CONF | 17:02 |
| mriedem | definitely a bug, but just do $NOVA_CONF for now and comment in the playbook | 17:03 |
| efried | ack | 17:03 |
| mriedem | i can report a devstack back | 17:04 |
| efried | bug? | 17:04 |
| efried | cool, thanks. | 17:04 |
| mriedem | https://bugs.launchpad.net/devstack/+bug/1802143 | 17:06 |
| openstack | Launchpad bug 1802143 in devstack "Changes from [[post-config|$NOVA_CPU_CONF]] are overwritten in "superconductor" mode" [Undecided,New] | 17:06 |
| mriedem | *yeah bug | 17:06 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Turn off rp association refresh in nova-next https://review.openstack.org/616033 | 17:07 |
| efried | mriedem: done ^ | 17:07 |
| efried | mriedem: but as I said, I don't actually expect to see a significant difference, because active state. | 17:08 |
| efried | though making sure it actually works and doesn't blow up the world will be good :) | 17:08 |
| *** efried is now known as efried_rollin | 17:08 | |
| mriedem | hey now that the dems took over the house the world is going to end anyway | 17:10 |
| * mriedem embraces for the caravan | 17:11 | |
| mriedem | heh, *braces | 17:11 |
| mriedem | sorry, trump is doing a news conference talking about the lack of republican losers that didn't accept his warm embrace | 17:11 |
| *** sahid has quit IRC | 17:23 | |
| tssurya | sean-k-mooney: I had a question, would you know the working of virsh capabilities (ca you point me to the code of how this xml is generated)? | 17:25 |
| tssurya | couldn't find the exact source code | 17:25 |
| mriedem | kashyap: ^ | 17:27 |
| dansmith | tssurya: it comes from libvirt, obviously | 17:29 |
| dansmith | tssurya: there is a lot of documentation about what they all mean | 17:29 |
| tssurya | dansmith: yea I tried to go into the libvirt code,, but in vain | 17:29 |
| dansmith | tssurya: https://libvirt.org/formatcaps.html | 17:29 |
| dansmith | tssurya: you want to know how it's determining one thing or another, or just what everything means? | 17:30 |
| tssurya | I already saw that link | 17:30 |
| dansmith | example of what you're trying to figure out? | 17:30 |
| *** k_mouza has quit IRC | 17:30 | |
| tssurya | I want to know how this Core(s) per socket in the topology is calculated | 17:30 |
| tssurya | or the code in libvirt that gets this part | 17:31 |
| dansmith | it should all be straight out of /sys I would think | 17:31 |
| *** helenafm has quit IRC | 17:32 | |
| dansmith | you're talking about host caps in that case right? not how a guest's topo gets determined (by nova) but how libvirt reports the host topo? | 17:32 |
| tssurya | yea the host caps (I think) | 17:32 |
| sean-k-mooney | tssurya: if you talking about the guest topology the answer is dumbly | 17:32 |
| *** mriedem has quit IRC | 17:33 | |
| tssurya | basically we had a live migration failure because the cpu_info didn't match | 17:33 |
| dansmith | there are lots of reasons for that being possible, topo is one and flags being another | 17:33 |
| tssurya | which we started investigating and reached the topo part | 17:34 |
| *** ttsiouts has quit IRC | 17:36 | |
| *** davidsha has quit IRC | 17:37 | |
| sean-k-mooney | tssurya: so ya as dansmith mentioned libvirt gets this info from sysfs which the bios/uefi proved to the kernel based on probing the cpu. the kenrel can re probe this info if it really needs to but libvirt is just taking whatever the kernel provides | 17:37 |
| tssurya | oh okay | 17:38 |
| sean-k-mooney | tssurya: the toplogy info is here ls /sys/bus/cpu/devices/cpu0/topology/ | 17:39 |
| sean-k-mooney | the cpu flags are up one level | 17:39 |
| *** ttsiouts has joined #openstack-nova | 17:39 | |
| dansmith | sean-k-mooney: live migration failing because of topology would only be if you're using pinned cpus and those cpus don't exist on the remote side right | 17:39 |
| dansmith | ? | 17:39 |
| tssurya | thanks, looking | 17:39 |
| sean-k-mooney | dansmith: yes | 17:40 |
| tssurya | yea we have something like 48 cores source out of which 24 are offline and 24 on destination, but the 24 offline is not reflected or something | 17:40 |
| openstackgerrit | Jack Ding proposed openstack/nova master: Add HPET timer support for x86 guests https://review.openstack.org/605902 | 17:40 |
| tssurya | not sure if that makes sense | 17:41 |
| sean-k-mooney | tssurya: have you excluded the offline cpus form the vcpu_pin_set | 17:41 |
| sean-k-mooney | tssurya: use because they are offline does not mean they will not be in sys / virsh capablities | 17:42 |
| sean-k-mooney | tssurya: but ya as dan said topology will only come into it if the cpus you were pinned too on the source are not present on the dest | 17:42 |
| *** erlon_ has joined #openstack-nova | 17:43 | |
| sean-k-mooney | tssurya: if they are present it could still be incorrect to do the migration but that is a sperate issue | 17:43 |
| *** erlon has quit IRC | 17:43 | |
| *** derekh has quit IRC | 17:44 | |
| *** ttsiouts has quit IRC | 17:44 | |
| *** jpena is now known as jpena|off | 17:46 | |
| cfriesen | tssurya: as I'm sure you're aware, live migration with CPU pinning is currently broken. :) | 18:05 |
| tssurya | sean-k-mooney: how do I exclude offline cpus from vcpu_pin_set (sorry I am a totally beginner in this stuff) | 18:06 |
| tssurya | cfriesen: actually I wasn't :) | 18:06 |
| tssurya | thanks for telling me | 18:07 |
| tssurya | sean-k-mooney: because we were looking through the libvirt code and like you said it seems to be looping through all the cores even if they are offline | 18:08 |
| cfriesen | tssurya: technically it's never been reliable. it will try to keep the same topology and host cpus on the dest node, even if those cpus/numa-nodes don't exist or are already in use by other instances. | 18:08 |
| cfriesen | tssurya: there's work in progress to fix it finally by recalculating the resources on the dest as part of the live migration, so that might land in stein | 18:09 |
| tssurya | cfriesen: ah good to know | 18:10 |
| *** panda is now known as panda|off | 18:10 | |
| openstackgerrit | sean mooney proposed openstack/nova master: harden placement init under wsgi https://review.openstack.org/610034 | 18:25 |
| sean-k-mooney | gibi: ^ should have the exception behavior you wanted | 18:25 |
| sean-k-mooney | gibi: the run_once contract is now expcitly stated too. the contract is different then i had planned but it will still work for my usecase and hopefully you find it more intuititve | 18:29 |
| *** Swami has joined #openstack-nova | 18:30 | |
| *** ralonsoh has quit IRC | 18:30 | |
| cdent | edleafe: if you get a chance to cast your eye on https://review.openstack.org/#/c/603955/ (cyborg interaction spec) I think it could do with it. The API bits need some stronger review and I've run out of time and I believe you've got some state on the cyborg stuff? | 18:34 |
| sean-k-mooney | cdent: do you think the spec is close | 18:36 |
| sean-k-mooney | cdent: i spend a lot of time going through it in detail initilly but i have not had time to go through the spec end to end in several interations | 18:37 |
| *** mriedem has joined #openstack-nova | 18:38 | |
| cdent | sean-k-mooney: it's hard to say. a lot of the details are kind of lost on me. When I previously reveiwed it was mostly to look at the shape of the API, which I found a bit off (which is why I've asked edleafe to look as a fellow api-sig person). | 18:40 |
| edleafe | cdent: It's on my plate. I've had the same experience as sean-k-mooney - it's hard to get through the entire thing | 18:42 |
| sean-k-mooney | cdent: i have had similar feeling. i have tried to blance the "is the cyborg api correct" comments with is the workflow and datastuctre used correct in terms of the nova interaction | 18:42 |
| sean-k-mooney | edleafe: there is still alot going on for a single spec but some of that context/complexity is unaviodable unfortunetly | 18:44 |
| edleafe | sean-k-mooney: agreed | 18:44 |
| *** irclogbot_2 has joined #openstack-nova | 18:45 | |
| *** tssurya has quit IRC | 18:47 | |
| artom | mriedem, oh hai, could you hit https://review.openstack.org/#/c/587248/1 and the one on top when you get a chance? It's the pike backport of a thing you +2'ed in queens | 18:48 |
| *** eharney has quit IRC | 18:53 | |
| mriedem | ok | 18:56 |
| mriedem | done | 19:00 |
| *** kmalloc is now known as needscoffee | 19:05 | |
| *** priteau has quit IRC | 19:10 | |
| artom | Cheers! | 19:12 |
| *** priteau has joined #openstack-nova | 19:12 | |
| *** eharney has joined #openstack-nova | 19:26 | |
| mriedem | super critical bug that needs fixing asap https://bugs.launchpad.net/nova/+bug/1802167 | 19:27 |
| openstack | Launchpad bug 1802167 in OpenStack Compute (nova) "nova-manage db purge -h has weird help for --before option" [Low,Triaged] | 19:27 |
| artom | oh shiite im on that | 19:27 |
| sean-k-mooney | artom: feels more like a stephenfin thing :) | 19:27 |
| mriedem | let's not get stephen started on the cliff thing again | 19:29 |
| openstackgerrit | Jack Ding proposed openstack/nova master: Add HPET timer support for x86 guests https://review.openstack.org/605902 | 19:29 |
| mriedem | or clint or clive or whatever manly naming library it is | 19:29 |
| mriedem | efried_rollin: you can drop the -2 on https://review.openstack.org/#/c/605902/ - the hpet blueprint was approved | 19:30 |
| sean-k-mooney | haha didnt he end up porting the stuff he wanted form the one we are not useing to the one we are upstream in the end | 19:30 |
| mriedem | jackding: make sure to throw that in the runways queue when it's ready | 19:30 |
| mriedem | runways reviews are just burning down the house right now | 19:31 |
| *** dpawlik has quit IRC | 19:34 | |
| *** dpawlik_ has joined #openstack-nova | 19:34 | |
| openstackgerrit | sean mooney proposed openstack/os-vif master: always create ovs port during plug https://review.openstack.org/602384 | 19:37 |
| openstackgerrit | sean mooney proposed openstack/os-vif master: add isolate_vif config option https://review.openstack.org/612534 | 19:37 |
| *** READ10 has quit IRC | 19:41 | |
| openstackgerrit | sean mooney proposed openstack/os-vif stable/rocky: always create ovs port during plug https://review.openstack.org/609850 | 19:42 |
| *** betherly has joined #openstack-nova | 19:42 | |
| jackding | mriedem: sure | 19:45 |
| *** betherly has quit IRC | 19:47 | |
| artom | mriedem, keith. I want novaclient to use python-keith | 19:49 |
| mriedem | i had a shift manager at my part time high school job named keith, | 19:50 |
| mriedem | oakleys, jorts, nascar shirts, tight mullet | 19:50 |
| mriedem | that was a man | 19:50 |
| *** mriedem is now known as mriedem_afk | 19:53 | |
| openstackgerrit | sean mooney proposed openstack/os-vif stable/rocky: add isolate_vif config option https://review.openstack.org/616285 | 19:57 |
| openstackgerrit | sean mooney proposed openstack/os-vif stable/queens: always create ovs port during plug https://review.openstack.org/609851 | 20:00 |
| *** erlon_ has quit IRC | 20:05 | |
| *** dpawlik_ has quit IRC | 20:06 | |
| *** dpawlik has joined #openstack-nova | 20:07 | |
| *** dpawlik has quit IRC | 20:09 | |
| *** dpawlik has joined #openstack-nova | 20:09 | |
| efried_rollin | mriedem_afk: done | 20:17 |
| *** efried_rollin is now known as efried | 20:18 | |
| *** needscoffee is now known as kmalloc | 20:19 | |
| *** eharney has quit IRC | 20:25 | |
| *** erlon_ has joined #openstack-nova | 20:27 | |
| efried | mriedem_afk: That nova-next thing worked this time. And the job passed, but as expected, no significant difference in the size of the placement log. | 20:33 |
| *** dpawlik has quit IRC | 20:43 | |
| *** dpawlik has joined #openstack-nova | 20:43 | |
| *** READ10 has joined #openstack-nova | 20:47 | |
| *** whoami-rajat has quit IRC | 20:47 | |
| *** jungleboyj has quit IRC | 20:47 | |
| *** dpawlik has quit IRC | 20:47 | |
| *** jbryce has quit IRC | 20:47 | |
| *** hogepodge has quit IRC | 20:47 | |
| *** auggy has quit IRC | 20:47 | |
| *** whoami-rajat has joined #openstack-nova | 20:48 | |
| *** jbryce has joined #openstack-nova | 20:48 | |
| *** mnaser has quit IRC | 20:48 | |
| *** lamt has quit IRC | 20:48 | |
| *** hogepodge has joined #openstack-nova | 20:48 | |
| *** eharney has joined #openstack-nova | 20:48 | |
| *** mnaser has joined #openstack-nova | 20:49 | |
| *** andreaf has quit IRC | 20:51 | |
| *** andreaf has joined #openstack-nova | 20:52 | |
| *** dpawlik has joined #openstack-nova | 21:14 | |
| *** cdent has quit IRC | 21:19 | |
| *** dpawlik has quit IRC | 21:22 | |
| *** dpawlik has joined #openstack-nova | 21:22 | |
| *** dpawlik has quit IRC | 21:27 | |
| *** dpawlik has joined #openstack-nova | 21:33 | |
| *** mchlumsky has quit IRC | 21:33 | |
| *** READ10 has quit IRC | 21:36 | |
| *** dpawlik has quit IRC | 21:38 | |
| *** dpawlik has joined #openstack-nova | 21:38 | |
| *** dpawlik has quit IRC | 21:39 | |
| *** dpawlik_ has joined #openstack-nova | 21:39 | |
| *** ivve has quit IRC | 21:46 | |
| *** awaugama has quit IRC | 21:54 | |
| *** jungleboyj has joined #openstack-nova | 21:59 | |
| openstackgerrit | Merged openstack/nova stable/pike: Refactor _build_device_metadata https://review.openstack.org/587248 | 22:19 |
| openstackgerrit | Merged openstack/nova stable/pike: Consider hostdev devices when building metadata https://review.openstack.org/587249 | 22:19 |
| *** mriedem_afk is now known as mriedem | 22:22 | |
| *** dpawlik_ has quit IRC | 22:23 | |
| *** dpawlik has joined #openstack-nova | 22:24 | |
| *** dpawlik has quit IRC | 22:28 | |
| openstackgerrit | Merged openstack/nova master: Allow resource_provider_association_refresh=0 https://review.openstack.org/615606 | 22:31 |
| openstackgerrit | Merged openstack/nova master: Remove LazyLoad of Scheduler Clients https://review.openstack.org/615641 | 22:38 |
| *** dpawlik has joined #openstack-nova | 22:40 | |
| *** dpawlik has quit IRC | 22:44 | |
| efried | melwitt: That fixture merged. Are you going to propose a release or you want me to do it? | 22:45 |
| efried | melwitt: Never mind, it seems to have already been done. | 22:46 |
| *** priteau has quit IRC | 22:47 | |
| *** READ10 has joined #openstack-nova | 23:01 | |
| *** slaweq has quit IRC | 23:01 | |
| *** k_mouza has joined #openstack-nova | 23:10 | |
| *** mlavalle has quit IRC | 23:13 | |
| *** eharney has quit IRC | 23:47 | |
| *** vishwana_ has joined #openstack-nova | 23:55 | |
| vishwana_ | Hi, I was wondering if a single VM instance can have multiple interfaces from a single subnet? If yes, how would I enable it? Thanks in advance for your guidance | 23:57 |
| *** hshiina has joined #openstack-nova | 23:57 | |
| *** vishwana_ is now known as vishwanathj | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
