| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-15 | |||
| 12:13:36 | mdbooth | ratailor: Hey, just commented on your patch. Did you respond to bauzas comments oob at any point? I was going to make the same point, but saw he'd already done it. | |
| 12:29:38 | gmann | alex_xu: can you check this - https://review.openstack.org/#/c/522918/ | |
| 12:36:06 | cdent | efried: aren't you supposed to be on holiday? | |
| 12:36:12 | efried | cdent gibi Thanks for the rechecks. Zuul had a really bad week last week; let's hope this one is better. | |
| 12:36:16 | efried | cdent No? | |
| 12:36:32 | efried | I believe mriedem is on holiday this week. | |
| 12:37:01 | cdent | It's mlk day. I guess that's not an enforced US holiday. I think my us employer-colleagues are taking it off. I can never remember. | |
| 12:37:12 | efried | ohhh, right. | |
| 12:37:19 | cdent | zuul probably wants sacrifices | |
| 12:37:20 | efried | I don't think this is one of those, but let me check... | |
| 12:37:44 | efried | nope, doesn't look like it. I've got pto scheduled for tomorrow, though. | |
| 12:37:57 | efried | My kids are off. But not me. | |
| 12:39:38 | kashyap | mdbooth: Hey, when you get a sec, is there a way I can see a Gate job result for an instance boot with an ephemeral disk? | |
| 12:41:26 | mdbooth | kashyap: Almost certainly, but I wouldn't know off the top of my head which tempest tests cover ephemerals. | |
| 12:42:02 | kashyap | mdbooth: No problem, I'll plough through them. | |
| 12:45:53 | Spazmotic | My commit handles ephs, one sec | |
| 12:46:29 | Spazmotic | Ah you wanted gate, not initial check didn't you | |
| 12:48:02 | kashyap | Spazmotic: I want to see the libvirt XML on any of the Gate jobs with an instance boot with ephemeral disk | |
| 12:48:34 | kashyap | Okido, my local DevStack run just finished after 24 minutes (real 24m49.794s), so I'll try locally | |
| 12:49:17 | Spazmotic | Ahhh I see, I cannot help so much with that, hehe. Sorry i'll go back into the shadows | |
| 12:49:27 | kashyap | Heh, np. | |
| 13:03:11 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Make sure that functional test triggered on sample changes https://review.openstack.org/533210 | |
| 13:04:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM: Testing if funct test is triggered https://review.openstack.org/533211 | |
| 13:10:04 | elmaciej | Hi! Maybe someone can point me how to start contributing to openstack. I made some fix to allow live migration on lvm based shared storage and don't know if it's worthly to put it official or not. Where should I get started. | |
| 13:10:23 | zioproto | sdague: ping, is it business hours in your timezone now ? :) | |
| 13:11:24 | zioproto | elmaciej: you need to "git review" your patch. More info here https://docs.openstack.org/infra/manual/developers.html | |
| 13:11:50 | elmaciej | zioproto - so I can create my own branch, push to it and call the review? just like that? | |
| 13:11:58 | zioproto | elmaciej: you need to "Sign the appropriate Individual Contributor License Agreement" | |
| 13:12:05 | zioproto | elmaciej: it is exaplained in that page | |
| 13:12:08 | zioproto | then you can push | |
| 13:12:14 | zioproto | and somebody will review your patch | |
| 13:12:22 | zioproto | you dont need to create a branch | |
| 13:12:28 | zioproto | all the work should be in 1 single commit | |
| 13:12:36 | zioproto | then on your shell | |
| 13:12:38 | zioproto | just type | |
| 13:12:40 | zioproto | git review | |
| 13:12:51 | zioproto | this will automatically create a branch | |
| 13:13:04 | elmaciej | ok,got this, thanks, will take a look at evening then. | |
| 13:13:22 | zioproto | elmaciej: thanks for contributing to openstack | |
| 13:13:34 | Spazmotic | Really no need to create a local branch? have always done it :D | |
| 13:13:45 | elmaciej | it's a pleasure :) I needed live migration on lvm :) buisness requirment | |
| 13:23:05 | cdent | bauzas, gibi: 500 fix, already one +2, about 6 months old, would be nice to have: https://review.openstack.org/#/c/389798/ | |
| 13:28:53 | jaypipes | cdent: done. | |
| 13:29:25 | cdent | danke jaypipes wasn't sure if you were a) still alive, b) here today, c) here yet | |
| 13:29:29 | cdent | feeling better? | |
| 13:29:42 | jaypipes | cdent: much, thank you. I'm back around 75% I think. | |
| 13:30:00 | cdent | flu sucks | |
| 13:30:36 | cdent | (I think it was diana on much of that thing, I just did a rebase towards the end) | |
| 13:32:16 | gibi | jaypipes: welcome back! | |
| 13:32:38 | jaypipes | gibi: thanks mate :) | |
| 13:32:49 | jaypipes | feels good to be able to work again. | |
| 13:33:19 | lyarwood | bauzas: so https://review.openstack.org/#/c/531407/ wasn't really my change FWIW, mdbooth and melwitt rewrote most of it in master so we should be good to +W this now. | |
| 13:35:18 | efried | jaypipes Howdy sir, welcome back to the land of the living. | |
| 13:37:45 | jaypipes | efried: thanks :) | |
| 13:41:47 | mdbooth | lyarwood: IIUC we're expecting failures in https://review.openstack.org/#/c/523958/ because we're assuming c-vol has done its thing, which it hasn't, right? | |
| 13:42:21 | mdbooth | lyarwood: i.e. A lack of tempest failures would indicate a lack of test coverage? | |
| 13:42:40 | mdbooth | And we need a couple of Depends-On: for c-vol patches which don't exist, yet? | |
| 13:43:27 | lyarwood | mdbooth: yes and yes, there's some additional coverage in the barbican plugin but there's also alot missing from standard tempest | |
| 13:43:44 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: create vGPU for instance https://review.openstack.org/528832 | |
| 13:43:45 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Recreate mediated devices on reboot https://review.openstack.org/533642 | |
| 13:43:54 | bauzas | AFAIK, mriedem is not around today, right? | |
| 13:43:58 | mdbooth | lyarwood: Ok, thanks. | |
| 13:48:52 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Recreate mediated devices on reboot https://review.openstack.org/533642 | |
| 13:49:35 | cdent | edleafe, efried: can I assume we are having the usual scheduler meeting? | |
| 13:50:18 | jaypipes | cdent: yes, but edleafe is commuting. | |
| 13:50:29 | jaypipes | cdent: unless he's off for MLK day? | |
| 13:50:32 | jaypipes | not sure.. | |
| 13:51:25 | efried | IBM doesn't have MLK as a site holiday. But edleafe asked me to run it, so either he's off anyway, or he's commuting. | |
| 13:56:10 | efried | cdent You didn't find the first paragraph of the commit message adequately explanatory? https://review.openstack.org/#/c/533244/ | |
| 13:56:39 | cdent | efried: imagine you didn't know what a _Provider is? | |
| 13:56:50 | cdent | That's what I mean by "in isolation" | |
| 13:57:29 | efried | okay. | |
| 13:58:08 | cdent | efried: I know it is tedious, and I know I dont do that to everyone, but a) I should, b) I feel like you'll take it in good intent intended | |
| 13:58:42 | cdent | (also I didn't -1 it, just commented) | |
| 13:59:22 | efried | cdent I'm planning to slot this change into the existing series, so that would give it some context. But I'll add some words. | |
| 13:59:48 | efried | Scheduler meeting NOW in #openstack-meeting-alt | |
| 13:59:49 | Spazmotic | johnthetubaguy I was going to hit you up about a commit the other day but realized I no longer work with you.. too sad. | |
| 14:00:12 | cdent | efried: imagine yourself in the shoes of a drunk, stupid, tired, angry git bisector | |
| 14:00:21 | Spazmotic | I made a commit with some code against the resize_up xenapi stuff if you want to take a look, you're probably one of the best to review it and offer me some suggestions if you find the chance :) https://review.openstack.org/#/c/533168/ | |
| 14:00:36 | Spazmotic | back after a while, stupid.. house.. not self cleaning.. | |
| 14:05:01 | openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | |
| 14:06:45 | ameeda | alex_xu: are you around ? | |
| 14:10:05 | openstackgerrit | Merged openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | |
| 14:10:14 | ameeda | gmann: Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | |
| 14:10:40 | ameeda | gmann: can you please review the code ? | |
| 14:15:20 | openstackgerrit | Merged openstack/nova master: console: Provide an RFB security proxy implementation https://review.openstack.org/345399 | |
| 14:15:31 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162 | |
| 14:16:54 | stephenfin | #success After 3 years and 7 (?) releases, encryption between nova's consoleproxy service and compute nodes is finally possible ✌️ | |
| 14:16:56 | openstackstatus | stephenfin: Added success to Success page | |
| 14:17:10 | cdent | stephenfin: | |
| 14:17:13 | cdent | ++ | |
| 14:19:45 | ameeda | stephenfin: Congratulations | |
| 14:28:23 | jaypipes | it's breakfast time. | |
| 14:31:46 | gibi | stephenfin: nice! | |
| 14:33:42 | ildikov | stephenfin: yaaay! nice job! congrats! :) | |
| 14:34:09 | stephenfin | jaypipes: I'll have you know it's basically supper time, actually ;) | |
| 14:34:41 | stephenfin | johnthetubaguy: You're hardly about, are you? | |
| 14:40:00 | openstackgerrit | Eric Fried proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436 | |
| 14:40:52 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | |
| 14:44:19 | openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.59 - Add trusted_image_certificates https://review.openstack.org/500396 | |