| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-30 | |||
| 02:48:30 | mriedem | on nova-net | |
| 02:52:42 | mriedem | Kevin_Zheng: https://bugs.launchpad.net/nova/+bug/1713895 | |
| 02:52:43 | openstack | Launchpad bug 1713895 in OpenStack Compute (nova) "block_device_mapping_v2.bus_type is missing from POST /servers API reference" [Medium,Confirmed] | |
| 02:56:41 | Kevin_Zheng | ^ THX | |
| 03:57:14 | abhi89 | Hi all.. can someone please review https://review.openstack.org/#/c/485121/ | |
| 05:15:53 | openstackgerrit | Merged openstack/nova master: Fix nova assisted volume snapshots https://review.openstack.org/498845 | |
| 05:19:57 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954 | |
| 06:10:15 | openstackgerrit | Rajat Sharma proposed openstack/nova master: Typo fix in admin doc ssh-configuration.html https://review.openstack.org/499034 | |
| 08:07:20 | openstackgerrit | zhangbailin proposed openstack/nova-specs master: Modify spelling error in nova-specs document https://review.openstack.org/494094 | |
| 09:18:42 | openstackgerrit | jichenjc proposed openstack/nova master: Remove qpid description in doc https://review.openstack.org/499087 | |
| 09:26:07 | openstackgerrit | jichenjc proposed openstack/nova master: check query param for used_limits function https://review.openstack.org/499091 | |
| 09:38:23 | openstackgerrit | Ahmed Zaid proposed openstack/nova master: Modife Some comments. https://review.openstack.org/499095 | |
| 09:39:48 | openstackgerrit | jichenjc proposed openstack/nova master: Remove qpid description in doc https://review.openstack.org/499087 | |
| 09:41:09 | openstackgerrit | jichenjc proposed openstack/nova master: check query param for used_limits function https://review.openstack.org/499091 | |
| 09:47:56 | openstackgerrit | Merged openstack/nova master: Typo fix in admin doc ssh-configuration.html https://review.openstack.org/499034 | |
| 10:04:17 | openstackgerrit | Merged openstack/nova master: Clarify the field usage guidelines https://review.openstack.org/498581 | |
| 10:05:26 | sean-k-mooney | anyone know if sergey is planning to continue this work https://review.openstack.org/#/c/361140/ https://review.openstack.org/#/c/390520/14 if not i or someone on my team might take it over if no one objects. | |
| 10:47:30 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rename cellsv2_layout -> cellsv2-layout https://review.openstack.org/498821 | |
| 10:47:31 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Cleanup of existing index pages https://review.openstack.org/498819 | |
| 10:47:31 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add contents page https://review.openstack.org/498820 | |
| 10:47:32 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add user index page https://review.openstack.org/498817 | |
| 10:47:32 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add configuration index page https://review.openstack.org/498818 | |
| 10:51:52 | ioggstream | hi@all | |
| 10:54:53 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform servergroup.delete notification https://review.openstack.org/483902 | |
| 10:59:53 | ioggstream | can I use scheduler_hints to spread hosts on different AZ ? | |
| 11:00:38 | ioggstream | actually I use server_groups with anit-affinity | |
| 11:07:56 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-evacuate notification https://review.openstack.org/482148 | |
| 11:08:30 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-evacuate notification https://review.openstack.org/482148 | |
| 11:09:49 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-live_migration_post notification https://review.openstack.org/480119 | |
| 11:10:48 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | |
| 11:13:44 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | |
| 11:14:46 | openstackgerrit | Alex Szarka proposed openstack/nova master: Refactor create_delete_server_with_instance_update https://review.openstack.org/466296 | |
| 11:15:12 | openstackgerrit | Alex Szarka proposed openstack/nova master: Refactor create_delete_server_with_instance_update https://review.openstack.org/466296 | |
| 11:51:38 | openstackgerrit | Lucian Petrut proposed openstack/nova master: Hyper-V: Perform proper cleanp after cold migration https://review.openstack.org/486955 | |
| 12:51:46 | alex_xu | nova api sub-team meeting is in 9 mins at #openstack-meeting-4 | |
| 13:39:48 | openstackgerrit | Ed Leafe proposed openstack/nova-specs master: Return Destination Objects https://review.openstack.org/498830 | |
| 13:55:45 | dansmith | cdent: hey, so, we need another placement api | |
| 13:55:58 | dansmith | cdent: we don't *have* to have it, but we *should* have it somehow | |
| 13:56:00 | cdent | dansmith: oh yeah, do tell? | |
| 13:56:08 | dansmith | we need to be able to PUT multiple allocations in a single go | |
| 13:56:45 | dansmith | for the migration thing, I need to be able to PUT a new allocation for an instance and a new allocation for the migration simultaneously | |
| 13:56:51 | cdent | so being able to say “put some allocations for both this migration uuid and this consumer uuid in one go”? | |
| 13:56:54 | cdent | jinx | |
| 13:57:11 | dansmith | without this, I could delete the instance allocation and create the migration one, but there's a race, or reverse that, but then I need twice the space available for a second | |
| 13:57:16 | dansmith | cdent: yes, that | |
| 13:57:59 | dansmith | cdent: so right now I'm doing the safer option which will fail if computes are full, but I think we should go for the atomic swap before we call this done | |
| 13:58:14 | cdent | one way to do that woudl be POST /allocations | |
| 13:58:31 | cdent | in fact I would say that’s probably the only way to do that | |
| 13:59:03 | dansmith | https://review.openstack.org/#/c/498949/1/nova/compute/manager.py | |
| 13:59:09 | dansmith | L3801 here is the first example | |
| 13:59:22 | cdent | yeah, I was just through there a few minutes ago | |
| 13:59:32 | dansmith | okay cool, so could you cook that up for us? | |
| 14:00:08 | cdent | yeah, shouldn’t be too hard, I’m trying to think what the body should look like | |
| 14:00:28 | cdent | we’ve already complained about this mismatch between GET and PUT on /allocations/{c_u} | |
| 14:00:51 | cdent | so if we’re considering changing the PUT format, then would be best to make POST similar to that | |
| 14:01:05 | dansmith | well, | |
| 14:01:26 | dansmith | POST of multiple things being slightly different than a PUT of one is less terrible than PUT/GET of a single thing being different | |
| 14:01:33 | cdent | we’ll also want to decide, sooner than later, if this POST is only for one project_id, user_id pair, or can be mixed? | |
| 14:01:38 | dansmith | but yeah, whatever we need to do.. | |
| 14:01:56 | cdent | dansmith: yeah, sure, but it’s a question of list or dict being the fundamental structure | |
| 14:01:56 | dansmith | well, for this we only need it to apply to a single project/user | |
| 14:02:04 | dansmith | ack | |
| 14:02:27 | cdent | I’ve got a nine hour flight tomorrow, I’ll see if I can make it happen then | |
| 14:02:50 | dansmith | theoretically you wouldn't (or shouldn't) need atomic operations across tenants/users, but I'm also not sure there's a good reason to specifically design it not to work | |
| 14:03:18 | dansmith | cdent: cool, decide on these things, put it in a spec calling out the potential questions and we can go from there | |
| 14:03:29 | cdent | ✔ | |
| 14:03:42 | dansmith | thanks! | |
| 14:04:26 | cdent | a pleasure. it feels like it’s been a while since I’ve written much code | |
| 14:05:45 | cdent | edleafe, jaypipes : if you have any ideas/concerns with what dan and I just discussed ^ please let me know before tomorrow morning (I’ll see about cooking a quick spec sooner) | |
| 14:10:42 | mriedem | o/ | |
| 14:11:36 | edleafe | cdent: this would be a PUT against /allocations (no {consumer_uuid}), right? | |
| 14:12:05 | cdent | edleafe: I’m thinking POST as PUT would imply a replace of _all_ allocations | |
| 14:13:12 | dansmith | cdent: all allocations for any consumer specified in the body you mean | |
| 14:13:20 | mriedem | efried: can you redo this with the -x option for git cherry-pick? https://review.openstack.org/#/c/498463/ | |
| 14:13:33 | efried | mriedem ... | |
| 14:13:38 | cdent | dansmith: a PUT to /uri means replace resource /uri | |
| 14:14:00 | dansmith | oh sorry I misread, | |
| 14:14:06 | dansmith | you were saying a PUT would imply replacing all | |
| 14:14:08 | dansmith | gotcha | |
| 14:14:08 | efried | mriedem Hmph, I cherry-picked it from gerrit; isn't that supposed to do the -x thing? Will redo. | |
| 14:14:11 | edleafe | cdent: yeah, what dansmith just said. POST is fortunately flexible. | |
| 14:14:12 | cdent | but POST to /uri means create some stuff using whatever semantics you want, and the semantics we want are “create some allocations for the multiple consumers ids we speak here" | |
| 14:14:28 | mriedem | efried: not if the change you're cherry picking from in gerrit isn't merged yet | |
| 14:14:37 | efried | mmkay | |
| 14:14:51 | cdent | efried: I got caught by that the other day and shook my tiny fist | |
| 14:15:22 | efried | mriedem Do you care if it's a new change set? | |
| 14:16:18 | efried | Actually, is the only difference the "(cherry picked from commit 9c7d73195e4fd1c890228e3223106ccf4f13e22b)" line? | |
| 14:17:06 | efried | howzat: https://review.openstack.org/#/c/498463/ | |
| 14:18:29 | efried | (No bot post for stable? (Have I asked that before? (Seems like a reasonable thing to have; should be relatively few, and certainly noteworthy.))) | |
| 14:19:39 | mriedem | efried: patch bot post for stable would be fine, but you'd have to find out where that is configured; it's either in project-config or system-config | |
| 14:20:02 | efried | will dig | |
| 14:22:19 | beagles | moshele, I was going over https://bugs.launchpad.net/os-vif/+bug/1713590 with sean-k-mooney the other day. | |
| 14:22:20 | openstack | Launchpad bug 1713590 in os-vif "Plugging VFs no longer works without a readable phys_switch_id" [Medium,Triaged] | |
| 14:23:58 | beagles | moshele, it seems that ordering of the mechanism_drivers is a new backwards-compatibility-breaking requirement. Is there a use pattern outside of reordering configurations that could workaround the issue of direct ports getting "caught" by the ovs plugging? | |
| 14:25:01 | beagles | moshele, I was also expecting a failed bind to go to the next driver that supported that port type, but this apparently does not happen | |
| 14:25:56 | beagles | moshele, in short... how is this expected to work :) | |
| 14:29:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Move 'ipv6' opts to 'network' https://review.openstack.org/499168 | |
| 14:29:29 | efried | mriedem https://review.openstack.org/499167 | |
| 14:31:02 | beagles | sean-k-mooney, same question ^ :) | |
| 14:32:11 | mriedem | oh gantt | |