| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-08 | |||
| 12:06:11 | openstackgerrit | Alex Szarka proposed openstack/nova master: Reduce code complexity - linux_net.py https://review.openstack.org/359871 | |
| 12:06:18 | openstackgerrit | Alex Szarka proposed openstack/nova master: Reduce code complexity - libvirt/config.py https://review.openstack.org/359879 | |
| 12:07:50 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Fix indentation of database options https://review.openstack.org/443097 | |
| 12:07:50 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Resolve TODOs in 'database' https://review.openstack.org/393695 | |
| 12:12:15 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Short circuit notifiations when not enabled https://review.openstack.org/428260 | |
| 12:50:19 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery https://review.openstack.org/435954 | |
| 12:55:46 | alex_xu | nova api meeting is in 5 mins | |
| 12:58:08 | kaisers | Hi! Can i interested anyone to look into a smalish driver/systemd issue fix @ https://review.openstack.org/#/c/432344/ ? | |
| 13:21:17 | mriedem | o/ | |
| 13:21:47 | bhagyashri_s | johnthetubaguy: Hi, I have some doubt regarding patch: https://review.openstack.org/#/c/409644/ As we are decide to skip the image creation and delete the snapshot if rotation == 0. | |
| 13:22:08 | bhagyashri_s | johnthetubaguy: But As per the API spec: https://developer.openstack.org/api-ref/compute/, if the API is successful, it should return url of the image in the location header. But we are proposing not to create image when rotation parameter passed as 0 then in that case the location header will not exist so it seems like API behavior is changing in that case. | |
| 13:37:16 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Improve assertJsonEqual error reporting https://review.openstack.org/428199 | |
| 13:55:12 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add policy-docs spec https://review.openstack.org/433010 | |
| 14:01:04 | johnthetubaguy | bhagyashri_s: the current API never returns that header if rotation == 0 today, I believe | |
| 14:01:24 | alex_xu | johnthetubaguy: why need a method to get the scope, the scope depend on different instance? | |
| 14:01:41 | alex_xu | johnthetubaguy: ah, i see now | |
| 14:02:04 | alex_xu | still fill scope like {'project_id': instance.project} | |
| 14:02:11 | johnthetubaguy | alex_xu: yeah | |
| 14:02:38 | johnthetubaguy | alex_xu: maybe instance.get_scope() would do it, and keypair.get_scope() | |
| 14:02:49 | johnthetubaguy | alex_xu: its specific to the object passed, not the API | |
| 14:02:53 | alex_xu | johnthetubaguy: so you will have get_flavor_scope and get_global_scope | |
| 14:03:05 | alex_xu | yea, i see | |
| 14:03:33 | johnthetubaguy | alex_xu: well I think a constant for global scope, or something like that | |
| 14:20:11 | sean-k-mooney | hi everyone o/ | |
| 14:20:57 | sean-k-mooney | https://review.openstack.org/#/c/435575/ broke the intel NFV ci which is why it is currently red | |
| 14:27:29 | mriedem | sean-k-mooney: efried was talking about something similar | |
| 14:27:33 | mriedem | sounds like it broke powervm ci too | |
| 14:28:01 | mriedem | sean-k-mooney: how does it break the ci though? new package not in their mirror? | |
| 14:33:43 | gibi | cdent, mriedem: Hi! Sorry that I was not here to help digging the notification code to figure out the topic and the event_type prefix. (I work in UTC+1 so it was a bit late already) Is there anything from the notification side that stil needs clarification? | |
| 14:34:00 | mriedem | gibi: nope, you confirmed everything | |
| 14:34:09 | mriedem | plus i just chided you in the list about not working 20 hours per day :) | |
| 14:34:53 | gibi | yeah, I still try to keep a life outside work ;) | |
| 14:35:10 | cdent | there's an outside work? | |
| 14:35:33 | kaisers | rather: work outside but how is this better? | |
| 14:35:45 | openstackgerrit | Roman Dobosz proposed openstack/nova-specs master: Provide detailed error information for placement API https://review.openstack.org/418393 | |
| 14:36:43 | gibi | :) | |
| 14:41:06 | dansmith | alex_xu: mriedem: so you both still have -1s on that simple_tenant_usage patch, but I think we're good now, am I right? | |
| 14:42:50 | tojuvone | gibi: Thanks for the good comments. | |
| 14:43:00 | mriedem | dansmith: haven't looked | |
| 14:43:33 | tojuvone | Meanwhile have to also do outside work soon; tons of snow coming | |
| 14:43:59 | dansmith | mriedem: okay, talked with alex_xu last night and he agreed that the sorting is only for stable pagination and that we could sort cellmappinglist to be even tighter (which I'm working on now), but otherwise okay | |
| 14:44:31 | dansmith | mriedem: cellmappings should change almost never, and they come out in repeatable order, but I'm going to sort them just to be sure | |
| 14:47:50 | jaypipes | man, I'm tired today :( | |
| 14:47:54 | gibi | tojuvone: I feel a bit bad to proposing that alternative | |
| 14:47:59 | jaypipes | just can't get going, mentally.. | |
| 14:48:01 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add policy-remove-scope-checks spec https://review.openstack.org/433037 | |
| 14:49:21 | tojuvone | gibi, well if we take it Craton that I work for this would be the source of truth for everybody | |
| 14:49:40 | tojuvone | gibi, it would make a lot of sense what you say | |
| 14:50:09 | mriedem | dansmith: alex_xu: what does "I'm ok without sorted by 'project_id' with cells, since if we sort in the memory, that make the pagination meanless." mean? | |
| 14:50:15 | tojuvone | gibi, then again if somebody wants to anynthing else, I do nto know | |
| 14:50:17 | mriedem | oh without | |
| 14:50:21 | mriedem | i thought that said 'with' | |
| 14:50:37 | dansmith | mriedem: I think he means makes the sorting meaningless, like in my example | |
| 14:50:40 | mriedem | jaypipes: shitloads of coffee | |
| 14:50:55 | jaypipes | cdent: oh, I totally need one. as you might have guessed, they didn't last long in the Pipes household after ATL. | |
| 14:51:04 | jaypipes | mriedem: yeah, working on it. | |
| 14:51:17 | cdent | I was given a box of thing mints girl scouts cookies when I got to florida | |
| 14:51:23 | mriedem | i was up from 2-4 and then again at 6 when the neighbors dog was barking | |
| 14:51:24 | mriedem | for no reason | |
| 14:51:25 | cdent | I managed to take them home without opening them | |
| 14:51:28 | cdent | but once open | |
| 14:51:29 | mriedem | because that's just what it does, bark at nothing | |
| 14:51:30 | cdent | gone | |
| 14:51:59 | jaypipes | mriedem: I've got three of those. | |
| 14:52:13 | mriedem | jaypipes: yeah but i assume they sleep and snore most of the night? | |
| 14:52:35 | cdent | until the apnea hits | |
| 14:52:37 | mriedem | lack of oxygen makes one tired | |
| 14:52:41 | jaypipes | mriedem: indeed. | |
| 14:52:54 | mriedem | the neighbor dog has a cocaine supply i'm pretty sure | |
| 14:53:53 | mriedem | dansmith: so net is, you're going to (1) sort cellmapping.get_all separately and (2) do a more efficient marker lookup separately for https://review.openstack.org/#/c/441388/ ? | |
| 14:54:14 | mriedem | the marker is the instance uuid, so we could find the cell that's in and page from there, but then we still potentially have other cells to iterate on after that one | |
| 14:54:22 | cdent | jaypipes: https://www.amazon.com/Crunchie-Milk-Chocolate-Honeycomb-Center/dp/B0007U5J40 that price isn't _that_ bad | |
| 14:55:21 | dansmith | mriedem: yeah.. it's going to make that mess a little more complicated, but seems reasonable | |
| 14:55:35 | gibi | tojuvone: I think it would worth to mention Craton integration as an alternattive | |
| 14:55:36 | mriedem | i left a note with that question | |
| 14:55:40 | jaypipes | cdent: heh, indeed. which reminds me, I owe you money :) | |
| 14:55:42 | mriedem | dansmith: since i don't have it worked out in my head | |
| 14:56:05 | mriedem | alex_xu: i'm +2 on https://review.openstack.org/#/c/441388/ now | |
| 14:56:49 | cdent | jaypipes: you do? if you're talking about for the ones I brought you, those are on the house | |
| 14:57:47 | mriedem | johnthetubaguy: i know you love policy questions, | |
| 14:57:52 | jaypipes | cdent: well, that's mighty fine of you, kind sir. | |
| 14:58:04 | cdent | first one's free! | |
| 14:58:30 | johnthetubaguy | mriedem: you called? | |
| 14:58:45 | mriedem | johnthetubaguy: for lock server, if i'm reading the code correctly, the owner of the instance that locks it is the user, not the project? so that users A and B in project X both have access to a server, but if user A locks the server, user B can't stop/restart it etc? | |
| 14:58:51 | mriedem | but can user B unlock it since they are in the same project? | |
| 14:59:05 | tojuvone | gibi: Craton holds information about any host in cloud, it should have the planned maintenance stuff, just that one should be aware of that somehow in Nova | |
| 14:59:32 | johnthetubaguy | mriedem: thats correct I believe, anyone in the project can do the unlcok | |
| 14:59:35 | johnthetubaguy | unlock | |
| 14:59:37 | tojuvone | gibi, And worked to have servers API to have always consistent state infromatino about host | |
| 14:59:49 | johnthetubaguy | mriedem: the user can't do any operations, unless they first unlock | |
| 15:00:01 | johnthetubaguy | mriedem: the admin can do any action without unlocking (I think...) | |
| 15:00:15 | johnthetubaguy | mriedem: also admin can apply the lock, so no users can remove it | |
| 15:00:22 | gibi | tojuvone: do you mean nova needs information about planned maintenance or you mean that the user looking at the nova REST API needs info about the maintenance? | |
| 15:00:42 | arne_r | Once again, calling attention to get https://review.openstack.org/#/c/436868/ in, after Lee gave it his blessing | |
| 15:00:52 | johnthetubaguy | mriedem: that service level spec of mine builds on top of all that, and adds a third layer in there | |
| 15:01:29 | tojuvone | gibi, user querying his servers from Nova should get consistent state information. Also know that maintenance is coming | |
| 15:01:40 | mriedem | johnthetubaguy: yeah that's all consistent with what i thought, | |
| 15:01:52 | mriedem | i just wanted to make sure that like if user A locks it that user B can still unlock it | |
| 15:02:03 | mriedem | which seems a bit wrong, | |
| 15:02:08 | johnthetubaguy | mriedem: ah, cool, I believe thats true | |