| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-07 | |||
| 16:09:12 | jaypipes | dansmith: sorry for the misunderstanding | |
| 16:09:32 | dansmith | I guess I have to update this test now and fix the nova-manage too | |
| 16:14:42 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use Selection object to fill request group mapping https://review.openstack.org/638711 | |
| 16:20:36 | mriedem | efried: +1 thanks | |
| 16:21:25 | mriedem | stephenfin: this has you written all over it https://review.openstack.org/#/c/640181/ | |
| 16:21:56 | stephenfin | sure thing | |
| 16:25:46 | efried | mriedem: updated with more information (mostly copy/paste) in the commit message. | |
| 16:27:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Allow utime call to fail on qcow2 image base file https://review.openstack.org/641315 | |
| 16:27:41 | mriedem | mdbooth: do you fancy addressing review comments on https://review.openstack.org/#/c/641315/ ? | |
| 16:27:51 | mriedem | i'd like to make sure people are on board with that before approving your revert | |
| 16:28:45 | mriedem | efried: beautiful | |
| 16:28:58 | openstackgerrit | Dan Smith proposed openstack/nova master: Add user_id column to the instance_mappings table https://review.openstack.org/633349 | |
| 16:28:59 | openstackgerrit | Dan Smith proposed openstack/nova master: Add user_id field to InstanceMapping https://review.openstack.org/633350 | |
| 16:29:00 | openstackgerrit | Dan Smith proposed openstack/nova master: Populate InstanceMapping.user_id during migrations and schedules https://review.openstack.org/638574 | |
| 16:29:01 | openstackgerrit | Dan Smith proposed openstack/nova master: Add online data migration for populating user_id https://review.openstack.org/633351 | |
| 16:29:19 | mriedem | cdent: regarding the placement extraction forum session proposal in https://etherpad.openstack.org/p/DEN-train-nova-brainstorming | |
| 16:29:38 | mriedem | cdent: i'm going to submit that and also propose you as co-presentor or whatever, ok? | |
| 16:33:08 | mdbooth | mriedem: Yep | |
| 16:34:06 | mriedem | mdbooth: thanks | |
| 16:34:06 | mdbooth | mriedem: I did wonder actually if we should confine it further to EPERM | |
| 16:34:34 | mriedem | maybe ask neil since he had the recreate | |
| 16:34:44 | mdbooth | ack | |
| 16:36:50 | cdent | mriedem: yeah, that's fine | |
| 16:36:59 | mdbooth | EACCES, not EPERM | |
| 16:44:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Fix nits in remote console guide https://review.openstack.org/641098 | |
| 16:44:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Rename 'configuration drive' to 'config drive' https://review.openstack.org/641723 | |
| 16:44:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rework all things metadata'y https://review.openstack.org/640730 | |
| 16:52:17 | openstackgerrit | Dan Smith proposed openstack/nova master: Add online data migration for populating user_id https://review.openstack.org/633351 | |
| 16:53:05 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add online data migration for populating user_id https://review.openstack.org/633351 | |
| 16:53:10 | mriedem | +2 | |
| 16:53:16 | mriedem | dansmith: thanks for reving those | |
| 16:53:24 | mriedem | jaypipes: i'm +2 on the first 4 in that series now | |
| 16:53:30 | dansmith | aye | |
| 16:54:06 | jaypipes | aye aye. just wrapping up a meeting | |
| 17:10:09 | openstackgerrit | Dan Smith proposed openstack/nova master: Add online data migration for populating user_id https://review.openstack.org/633351 | |
| 17:15:30 | mriedem | johnthetubaguy: are you going to be in denver? i was considering proposing a forum session to get people together about your unified limits spec https://review.openstack.org/#/c/602201/ | |
| 17:15:51 | mriedem | even if you're not, i assume melwitt and/or jaypipes and/or lbragstad can help run that since i'm not qualified to run that session | |
| 17:16:02 | mriedem | but i don't know if jaypipes will be at the forum... | |
| 17:16:12 | mriedem | given his too cool for schoolness | |
| 17:16:52 | lbragstad | i haven't put together my schedule yet, but i'll try and be there for that | |
| 17:17:21 | jaypipes | mriedem: I will | |
| 17:17:35 | jaypipes | mriedem: can one be too uncool for school? | |
| 17:17:43 | jaypipes | mriedem: cuz I would be that. | |
| 17:18:13 | mriedem | that question is too philosophical for me to think about right now | |
| 17:18:43 | mriedem | dansmith: i'm sure you'd like to propose a "nova at the edge?!" forum session | |
| 17:18:50 | mriedem | you could even dress up like the edge | |
| 17:18:53 | dansmith | mriedem: I'm sure you're wrong | |
| 17:19:21 | mriedem | what if, we do a reenactment of the movie 'the edge' from the 90s though? | |
| 17:19:28 | mriedem | i'll be alec baldwin | |
| 17:19:53 | dansmith | :| | |
| 17:20:04 | mriedem | we need an anthony hopkins and a bear | |
| 17:20:24 | mriedem | bauzas: ... | |
| 17:20:43 | dansmith | https://www.youtube.com/watch?v=Dn8-4tjPxD8 | |
| 17:20:55 | bauzas | hum, I can wait for shaving | |
| 17:21:04 | mriedem | https://www.dailymotion.com/video/x3s9vt2 | |
| 17:21:12 | dansmith | keep thinking I need to be working that in .. the U2 edge thing is already played | |
| 17:21:34 | mriedem | if we're going to talk about stevie nicks i can't let the fajita roundup go unnoticed | |
| 17:26:42 | aspiers | kashyap (or anyone): need advice on how to reuse some code from libvirt.driver._get_machine_type() inside libvirt.host | |
| 17:27:13 | kashyap | aspiers: Afraid, I'm totally spent for the day :-( And about to go for some air and food | |
| 17:27:24 | aspiers | kashyap: OK maybe someone else can help | |
| 17:27:40 | kashyap | If it's on a particular review, think out loud. I'll see tomm. | |
| 17:27:52 | aspiers | no, it's on how to write my code | |
| 17:28:17 | aspiers | I want to reuse these two lines https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L4262 | |
| 17:28:32 | aspiers | but from Host class, which doesn't have access to the driver AFAICS | |
| 17:28:48 | aspiers | maybe there's a utils class or something | |
| 17:28:57 | aspiers | oh look at that, there is :) | |
| 17:29:21 | aspiers | ah no, because it needs access to _machine_type_mappings() also | |
| 17:29:40 | aspiers | hrm, maybe that can also be moved to utils | |
| 17:29:42 | aspiers | I'll try that | |
| 17:36:57 | openstackgerrit | Matthew Booth proposed openstack/nova master: Allow utime call to fail on qcow2 image base file https://review.openstack.org/641315 | |
| 17:41:37 | sean-k-mooney | mriedem: just reading you reminder tread. regarding melwitt's change to add the user_id to the instance mappings in the api db i assume we shoudl add a nova-status check for that right? | |
| 17:42:18 | sean-k-mooney | mriedem: we said we would add a blocker migration but this is the kindof thing nova-status should be checking before an op does an upgrade correct? | |
| 17:42:29 | dansmith | sean-k-mooney: for train you mean right? | |
| 17:42:33 | sean-k-mooney | ya | |
| 17:44:02 | sean-k-mooney | we wanted to make the colume non nullable after train so on an upgrade to U we should have a nova-status check to make sure you have run the data migration to populate the colume so that the schame migration can work | |
| 17:44:31 | sean-k-mooney | no one memtioned nova-status the other day so i just tought i would bring it up | |
| 17:45:00 | dansmith | I did in IRC actually, but .. we need to land that in train nova-status, not right now | |
| 17:45:26 | sean-k-mooney | oh ok i missed that then. cool | |
| 17:46:06 | dansmith | adding it in stein would tell them they need to do something they can't do until they deploy stein code | |
| 17:46:45 | sean-k-mooney | right because we are only adding the column in stien | |
| 17:49:10 | jaypipes | dansmith, mriedem: bottom 3 +W. still plowing through the nova-manage migration in https://review.openstack.org/#/c/633351... | |
| 17:51:17 | openstackgerrit | Matthew Booth proposed openstack/nova master: Allow utime call to fail on qcow2 image base file https://review.openstack.org/641315 | |
| 17:51:42 | mriedem | jaypipes: don't make me get you a mr plow jacket | |
| 17:51:46 | mriedem | so you can strut around | |
| 17:52:07 | mriedem | https://www.youtube.com/watch?v=fhW3OyifFBE | |
| 17:54:24 | dansmith | mriedem: how would you communicate with the other members of the tribe if you didn't have the simpsons and every snippet documented on youtube? | |
| 17:54:34 | mriedem | grunts | |
| 17:54:43 | mriedem | flute sounds | |
| 17:55:05 | jaypipes | heh | |
| 17:55:44 | mriedem | jaypipes: so i think i'll abstain from proposing a unified limits session, it's out of my ability to lead that | |
| 17:57:28 | jaypipes | mriedem: I can participate in such a session. | |
| 17:58:04 | mriedem | i'm going to go participate in a bowl of chili | |
| 17:58:45 | jaypipes | :) | |
| 17:59:04 | cdent | a forum session with a chili bakeoff would be pretty cool | |
| 18:00:37 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Fix nits in remote console guide https://review.openstack.org/641098 | |
| 18:00:38 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Rename 'configuration drive' to 'config drive' https://review.openstack.org/641723 | |
| 18:00:38 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rework all things metadata'y https://review.openstack.org/640730 | |
| 18:03:59 | jaypipes | mriedem_afk, dansmith: please see my question on https://review.openstack.org/#/c/633351/. it's the only part of that migration I don't get... | |
| 18:06:06 | dansmith | jaypipes: replied | |
| 18:08:34 | jaypipes | dansmith: thx | |