| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-07-03 | |||
| 19:39:57 | mriedem | no | |
| 19:39:59 | dansmith | cdent: rebuild is in place | |
| 19:40:30 | mriedem | rebuild = destroy the guest and re-spawn it with optionally a new image, but re-use the same volumes and ports and uuid and all that jazz | |
| 19:40:42 | zzzeek | mriedem: that will raise yse | |
| 19:40:58 | zzzeek | mriedem: means you did a mistake | |
| 19:41:11 | cdent | thanks dansmith, mriedem | |
| 19:41:15 | mriedem | cdent: don't forget unshelve | |
| 19:41:21 | mriedem | oh you said shelf | |
| 19:41:21 | cdent | see!? | |
| 19:41:40 | zzzeek | mriedem: e.g. one-to-one is not always simple to enforce needs to have correct unique constraints to make sure u dont blow it | |
| 19:41:41 | cdent | efried: so what's going on with your thing? | |
| 19:41:47 | mriedem | dansmith: so are you worried that we're going to blow up someone's db if they have >1 policy record per group https://review.openstack.org/#/c/579113/4/nova/db/sqlalchemy/api_models.py ? | |
| 19:41:55 | efried | cdent: I have several things. | |
| 19:42:20 | zzzeek | mriedem: oh the backref....umm, backref is an independent relationship. depends on if it has uselist=False or not | |
| 19:42:50 | dansmith | mriedem: I think if we don't have the schema to protect us, and we've had interfaces to let us create those, we might as well just do the safe thing and use list[0] everywhere, until we can check for and enforce 1:1 | |
| 19:43:06 | mriedem | zzzeek: tl;dr is we have 2 tables in a 1:1 relationship from a REST API standpoint, but modeled them as 1:M years ago for whatever reason, idk - future requirements or something | |
| 19:43:06 | dansmith | mriedem: just like that instance uuid dupe thing.. shouldn't have ever happened, but there were dupes in people's dbs | |
| 19:43:13 | mriedem | this change is trying to reconcile the 1:1ness | |
| 19:43:16 | zzzeek | mriedem: yeah i am looking | |
| 19:43:51 | mriedem | dansmith: instance uuid dupe thing? you mean when i added that unique constraint? | |
| 19:43:58 | mriedem | those were the days.... | |
| 19:44:00 | dansmith | yeah | |
| 19:44:07 | dansmith | and remember, we added a check, cleanup, and then the enforce patch | |
| 19:45:13 | zzzeek | mriedem: commented | |
| 19:45:21 | zzzeek | mriedem: w/ workaroun | |
| 19:45:22 | mriedem | ok i'm not sure what that does for the rest of his series and the object interaction, maybe his policies property on the InstanceGroup db model just needs to change to return the backref list rather than a list of policy names | |
| 19:45:29 | mriedem | and the object code would have to handle that accordingly | |
| 19:45:44 | openstackgerrit | Andreas Karis proposed openstack/nova master: Add documentation for emulator threads policy https://review.openstack.org/579956 | |
| 19:45:46 | dansmith | just always use _policies[0] | |
| 19:45:57 | dansmith | he already has that on create | |
| 19:46:07 | dansmith | and an @property on the model for policy makes sense | |
| 19:46:39 | mriedem | yeah i think you and zzzeek said the same thing :) | |
| 19:46:41 | mriedem | works for me | |
| 19:46:47 | dansmith | okay | |
| 19:50:03 | mriedem | dansmith: i left a summary comment, can you confirm? | |
| 19:51:30 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Tighten up ReportClient use of generation https://review.openstack.org/556669 | |
| 19:51:49 | efried | mriedem, cdent, dansmith, jroll, sambetts|afk: Let's try ^ that | |
| 19:52:28 | efried | There wound up being no reasonable way to do the retry in its own patch - because we weren't raising the exception to retry on. | |
| 19:54:30 | jmlowe | mnaser: an erasure coded pool is normal'ish, cannot do OMAP so it cannot hold the metadata of an rbd device, only need to reference an erasure coded pool for rbd during create or clone operations | |
| 19:56:41 | jmlowe | my concern is this, you use an erasure coded pool for glance, everything works ok until you go to clone back into glance during image-create then you need to know what pool to put the data in, currently infers the meta data pool just fine | |
| 19:57:24 | mriedem | efried: questions in https://review.openstack.org/#/c/579201/ | |
| 19:58:07 | jmlowe | if you don't specify a data pool all the image data will go back into the metadata pool which may not be sized properly and will probably be a nasty surprise for the operator | |
| 19:59:32 | mnaser | jmlowe: i don't know EC pools enough to help answer your questions | |
| 19:59:33 | mnaser | im sorry | |
| 20:00:34 | jmlowe | mnaser: np, I'll have to play around and see if there is some way to discover from the parent image | |
| 20:21:28 | dansmith | mriedem: yeah I think that's fine | |
| 20:46:13 | efried | mriedem: Replied | |
| 20:50:31 | mriedem | alright +2 | |
| 20:50:40 | mriedem | who wants a +2, i'm in a giving mood | |
| 20:52:01 | efried | mriedem: You could +2/+A the one below it :) | |
| 20:52:13 | mriedem | w/o looking | |
| 20:52:49 | efried | mriedem: I think that lets us close another bp | |
| 20:53:37 | mriedem | yeah yeah i'm looking | |
| 20:53:45 | mriedem | you know what turns me on, process | |
| 20:54:08 | efried | Knew you couldn't resist that carrot | |
| 20:54:42 | Sundar | efried: Please take a look at my unicast message about os-acc plugin design. Thanks. | |
| 20:54:52 | efried | Sundar: Unicast where? | |
| 20:55:07 | Sundar | Your email | |
| 20:55:18 | efried | okay, I see it. | |
| 20:56:11 | efried | Sundar: I plough through like 200-300 emails a day, sometimes with hours of lag. | |
| 21:03:21 | Sundar | efried: NP. | |
| 21:03:28 | efried | Sundar: I'm responding. | |
| 21:03:44 | efried | Sundar: Though this might be a good discussion for the dev mailing list... | |
| 21:07:29 | Sundar | Sure, NP. Let us take it there then | |
| 21:20:12 | mriedem | efried: comments in that cpu features traits patch | |
| 21:20:23 | mriedem | kashyap should also look at that b/c of the extra cpu flags thing | |
| 21:20:29 | mriedem | kashyap: https://review.openstack.org/#/c/560317/ | |
| 21:20:34 | mriedem | also, that patch fails the pkvm ci | |
| 21:23:34 | mriedem | ppc64le?!?! | |
| 21:23:39 | mriedem | where is tonyb when we need him | |
| 21:26:35 | efried | Thanks Matt. | |
| 21:35:31 | mriedem | beyond that it seems ok | |
| 21:36:38 | mriedem | efried: i'm surprised the zkvm ci doesn't blow up from that as well | |
| 21:36:45 | mriedem | since that ci job sets cpu_mode=none | |
| 21:50:34 | openstackgerrit | Andreas Karis proposed openstack/nova master: Add documentation for emulator threads policy https://review.openstack.org/579956 | |
| 22:04:57 | openstackgerrit | Andreas Karis proposed openstack/nova master: Add documentation for emulator threads policy https://review.openstack.org/579956 | |
| 22:41:39 | openstackgerrit | Jay Pipes proposed openstack/nova master: move lookup of provider from _new_allocations() https://review.openstack.org/579920 | |
| 22:41:40 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: delete auto-created consumers on fail https://review.openstack.org/579921 | |
| #openstack-nova - 2018-07-04 | |||
| 00:26:13 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (5) https://review.openstack.org/570842 | |
| 00:36:54 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (6) https://review.openstack.org/571330 | |
| 00:38:33 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992 | |
| 00:49:45 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993 | |
| 01:15:56 | openstack | bug 1779818 in OpenStack Compute (nova) "child's root provider is not updated." [Medium,Confirmed] https://launchpad.net/bugs/1779818 | |
| 01:15:56 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Regression test for bug 1779818 https://review.openstack.org/580011 | |
| 02:31:38 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Tighten up ReportClient use of generation https://review.openstack.org/556669 | |
| 02:47:25 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server create for multiple-create extension https://review.openstack.org/580017 | |
| 03:07:22 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server create for scheduler hint extension https://review.openstack.org/579067 | |
| 03:58:45 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server create for multiple-create extension https://review.openstack.org/580017 | |
| 04:07:41 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add InstanceGroupPolicy object https://review.openstack.org/573628 | |
| 04:07:42 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Refactor policies to policy in InstanceGroup DB model https://review.openstack.org/579113 | |
| 04:15:04 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server create for multiple-create extension https://review.openstack.org/580017 | |
| 05:19:57 | openstackgerrit | Merged openstack/nova master: Add action initiator attribute to the instance payload https://review.openstack.org/536243 | |
| 05:20:03 | openstackgerrit | Merged openstack/nova master: Prevent updating an RP's parent to form a loop https://review.openstack.org/579618 | |
| 05:37:30 | openstackgerrit | Merged openstack/nova master: Remove unused DB API instance_group_member* methods https://review.openstack.org/578488 | |
| 06:00:53 | gmann | nova api office hour time | |
| 06:01:00 | alex_xu | o/ | |
| 06:01:08 | gmann | #startmeeting nova api | |
| 06:01:09 | openstack | Meeting started Wed Jul 4 06:01:08 2018 UTC and is due to finish in 60 minutes. The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 06:01:10 | openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 06:01:13 | openstack | The meeting name has been set to 'nova_api' | |