| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-07-03 | |||
| 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 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Regression test for bug 1779818 https://review.openstack.org/580011 | |
| 01:15:56 | openstack | bug 1779818 in OpenStack Compute (nova) "child's root provider is not updated." [Medium,Confirmed] https://launchpad.net/bugs/1779818 | |
| 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' | |
| 06:01:15 | gmann | alex_xu: hi | |
| 06:01:27 | alex_xu | gmann: o/ waves again :) | |
| 06:02:04 | gmann | we will start the API discussion after long time :) | |
| 06:02:11 | alex_xu | hah | |
| 06:02:29 | gmann | anyone want to get notification please add your irc name on wiki's PING_LIST | |
| 06:02:32 | gmann | #link https://wiki.openstack.org/wiki/Meetings/NovaAPI#Agenda_for_next_Office_hours | |
| 06:02:39 | gmann | let's start | |
| 06:02:54 | gmann | i have updated the wiki page for new timing and channel info | |
| 06:03:26 | alex_xu | cool | |
| 06:03:28 | gmann | #link https://wiki.openstack.org/wiki/Meetings/NovaAPI#Agenda_for_next_Office_hours | |
| 06:04:10 | gmann | ^^ this will be agenda for this office hour which is generic one and people can add discussion item under open discussion if those are not covered under priority section | |
| 06:05:28 | gmann | #topic Priorities | |
| 06:05:40 | gmann | #link https://etherpad.openstack.org/p/rocky-nova-priorities-tracking | |
| 06:06:12 | alex_xu | gmann: is it updated list for the API? | |
| 06:06:20 | gmann | alex_xu: there is not set priority for APIs BP but i have listed all the ongoing API change work/BP in the API section of above etherpad | |
| 06:06:50 | alex_xu | gmann: cool, good to know what is ongoing | |
| 06:06:52 | gmann | alex_xu: i updated last week and i can keep updating this before office hour every week | |
| 06:07:00 | alex_xu | gmann: cool | |
| 06:07:52 | alex_xu | hah | |
| 06:08:38 | gmann | done | |
| 06:09:33 | gmann | we are getting good review from johnthetubaguy and stephenfinon also on that, thanks to both of them :) | |
| 06:09:56 | alex_xu | good progress | |
| 06:10:17 | gmann | idea is to keep review those items and then discuss here if anything needs to be. | |
| 06:10:43 | gmann | i have not reviewed much last week but i will make sure to review before next Wed. | |
| 06:11:20 | gmann | alex_xu: for extensions merge, i will try to push all the code by next week. | |