| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-07 | |||
| 01:34:11 | openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201 | |
| 01:54:17 | openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475 | |
| 01:54:26 | openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077 | |
| 01:56:39 | openstackgerrit | Merged openstack/nova-specs master: Handle rebuild of instance with new image https://review.openstack.org/560718 | |
| 03:01:56 | openstackgerrit | fupingxie proposed openstack/nova master: Remove stale pip-missing-reqs tox test https://review.openstack.org/566498 | |
| 05:25:48 | openstackgerrit | Nguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build https://review.openstack.org/551802 | |
| 08:16:50 | openstackgerrit | Surya Seetharaman proposed openstack/nova stable/queens: Make association_refresh configurable https://review.openstack.org/566288 | |
| 08:44:59 | openstackgerrit | zhangyangyang proposed openstack/nova master: Remove the function get_backdoor_port() https://review.openstack.org/566219 | |
| 08:49:29 | bauzas | so in case you want me reviewing things, ping me directly | |
| 09:03:46 | openstackgerrit | Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703 | |
| 09:05:43 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in tests/unit/api/openstack/compute https://review.openstack.org/566528 | |
| 09:14:12 | mrf | Hi | |
| 09:14:30 | mrf | we reinstalled a compute node queens | |
| 09:14:41 | mrf | and we got error like: | |
| 09:14:47 | mrf | 2018-05-07 07:25:31.266 16682 ERROR nova.scheduler.client.report [req-5c63e054-a5a7-4d53-84b8-6ae24805f83a - - - - -] [req-2cabe222-19a1-46f3-8427-5e297caacec3] Failed to create resource provider record in placement API for UUID 57741047-fb10-4c55-a284-e8d39b84c9f5. Got 409: {"errors": [{"status": 409, "request_id": "req-2cabe222-19a1-46f3-8427-5e297caacec3", "detail": "There was a | |
| 09:14:47 | mrf | conflict when trying to complete your request.\n\n Conflicting resource provider name: compute002.openstackxxxxx already exists. ", "title": "Conflict"}]}. | |
| 09:14:47 | mrf | <mrf> 2018-05-07 07:25:31.268 16682 ERROR nova.compute.manager [req-5c63e054-a5a7-4d53-84b8-6ae24805f83a - - - - -] Error updating resources for node compute002.openstackxxxxxxxx.: ResourceProviderCreationFailed: Failed to create resource provider compute002.openstackxxxxxxxx | |
| 09:48:34 | swamireddy | hi | |
| 09:49:28 | swamireddy | quick Q on libvirt section of nova conf file - can I mention 2 libvirt section in nova conf file..basically one for block storage-1 and next one for 2nd block storage-2 | |
| 10:36:07 | artom | sean-k-mooney[m], sean-k-m1, whichever is the real one, you around? Have some quick questions about the Intel CI | |
| 11:50:51 | openstackgerrit | MultipleCrashes proposed openstack/nova master: NCP-2367-deallocate-network-retry https://review.openstack.org/563418 | |
| 11:51:03 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (1) https://review.openstack.org/566555 | |
| 12:26:03 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add InstanceNUMATopology to LibvirtLiveMigrateData https://review.openstack.org/566398 | |
| 12:51:17 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add InstanceNUMATopology to LibvirtLiveMigrateData https://review.openstack.org/566398 | |
| 12:57:30 | mnaser | morning everyone | |
| 12:57:33 | openstackgerrit | Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.openstack.org/556751 | |
| 12:57:39 | mnaser | it looks liek nova-next job is broken | |
| 12:58:05 | mnaser | http://zuul.openstack.org/builds.html?job_name=nova-next | |
| 12:58:20 | mnaser | oh i guess this is the fix https://review.openstack.org/#/c/566446/ | |
| 13:26:14 | lei-zh | ping efried | |
| 13:26:22 | efried | lei-zh: Howdy. | |
| 13:26:32 | efried | finucannot: Didn't you have a patch for https://review.openstack.org/#/c/566498/ ? | |
| 13:27:05 | lei-zh | hi eric | |
| 13:27:25 | finucannot | efried: https://review.openstack.org/#/c/565535/ | |
| 13:27:39 | efried | finucannot: thanks | |
| 13:28:01 | lei-zh | a question about update provider tree patches, are you going to add that method for every virt driver | |
| 13:28:36 | efried | lei-zh: Not me; individual virt driver maintainers will be doing that. | |
| 13:28:53 | lei-zh | I mean this one, https://blueprints.launchpad.net/nova/+spec/update-provider-tree | |
| 13:29:27 | efried | lei-zh: I know xen is in process; I started a patch for libvirt which bhagyashri_s is going to take over; and I'll be doing the PowerVM one. | |
| 13:33:05 | lei-zh | efried, ok ,thanks, I'm working on reporting cpu features as traits for libvirt, thought it might be helpful to updating provder tree job | |
| 13:34:29 | lei-zh | https://review.openstack.org/#/c/560317/ | |
| 13:35:04 | efried | lei-zh: Sorry, my connection flaked for a couple of minutes there. | |
| 13:35:15 | lei-zh | ok, np | |
| 13:35:23 | lei-zh | efried, ok ,thanks, I'm working on reporting cpu features as traits for libvirt, thought it might be helpful to updating provder tree job | |
| 13:35:24 | efried | lei-zh: The xen series starts here: https://review.openstack.org/#/c/520313/ | |
| 13:36:19 | efried | lei-zh: libvirt is here: https://review.openstack.org/#/c/560444/ | |
| 13:37:52 | efried | lei-zh: If you're working on libvirt traits, you may want to base your series on that patch. | |
| 13:37:55 | lei-zh | efried: I see, I will keep an eye on those patches. | |
| 13:39:37 | lei-zh | efried: I've submitted code for libvirt to expose cpu features, https://review.openstack.org/#/c/560317/, I'm appreciated if you could have a look at it | |
| 13:39:52 | efried | Yeah, I think I remember looking at that already. | |
| 13:40:52 | lei-zh | cool, thanks | |
| 13:51:07 | bhagyashri_s | efried: Hi, I have written functional test case in the patch https://review.openstack.org/#/c/560459 test case code #link: http://paste.openstack.org/show/720489/ but it's giving me the usages that the compute node provider DISK_GGB is taken into account and not shared storage provider DISK_Gb can you plz help me for the same where exactly I am going wrong | |
| 13:53:09 | efried | bhagyashri_s: Can you WIP your change set and post a patch set with the functional test case included? | |
| 13:54:15 | efried | bhagyashri_s: That paste looks like you've duplicated a lot of the framework from some existing functional test suite. | |
| 13:56:05 | bhagyashri_s | efried: yeah for now I have create data separately once it will give correct output I will refactor the code and reuse the code from existing functional test suite | |
| 13:57:40 | efried | bhagyashri_s: At a glance it looks like the test is properly constructed. But if I'm going to debug it I would rather be able to create my environment so it looks like yours. | |
| 13:58:05 | efried | bhagyashri_s: And I'm not sure the utility of separating it out only to have to fold it back in later. | |
| 14:10:53 | dansmith | did mriedem say he was out today? | |
| 14:13:20 | melwitt | not that I recall | |
| 14:14:23 | openstackgerrit | Vabada proposed openstack/nova master: Fix typos in Host aggregates documentation https://review.openstack.org/566591 | |
| 14:15:05 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: Get inventories per vGPU type https://review.openstack.org/564215 | |
| 14:16:07 | dansmith | okay | |
| 14:17:17 | mnaser | melwitt: morning, do you think you could un-wip this to get some reviews to get nova-next to pass? https://review.openstack.org/#/c/566446/ | |
| 14:17:52 | melwitt | mnaser: yeah, going to fix it up soon. just woke up | |
| 14:18:08 | mnaser | melwitt: no worries :D thanks for looking into it in the first place | |
| 14:18:21 | mnaser | i found out that this was the fix by seeing it as the only passing job here, heh: http://zuul.openstack.org/builds.html?job_name=nova-next | |
| 14:18:25 | melwitt | it was caused by one of my console patches, unforunately | |
| 14:18:47 | mnaser | i'm sure we've all had our share of breaking $world | |
| 14:21:00 | efried | bhagyashri_s, bauzas: meet each other | |
| 14:21:43 | bauzas | bhagyashri_s: like I said in #openstack-meeting-4, I'll rebase https://review.openstack.org/#/c/560444/ under my https://review.openstack.org/#/c/564215/ branch | |
| 14:21:51 | bauzas | bhagyashri_s: and provide a new revision | |
| 14:41:33 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306 | |
| 14:41:56 | openstackgerrit | Vabada proposed openstack/nova master: Fix typos in Host aggregates documentation https://review.openstack.org/566591 | |
| 15:01:34 | dansmith | jaypipes: so, looking at your existing set of patches for the consumer thing, I don't see the thing I was suggesting, which was to always create a consumer/user/project with dummy values if the api request was with an old microversion | |
| 15:01:40 | dansmith | jaypipes: is there some reason you're not doing that? | |
| 15:01:49 | dansmith | because I think that would get us to a much more consistent spot | |
| 15:03:18 | jaypipes | dansmith: as opposed to just letting old records have no consumer record? | |
| 15:03:41 | dansmith | jaypipes: yeah | |
| 15:04:11 | dansmith | I was saying that I think we'd be a lot better off if we just did that.. have a config value of "if_no_user_id_then_use_this=$uuid" | |
| 15:04:11 | jaypipes | dansmith: guess I just didn't want to go through a forced data migration. | |
| 15:04:14 | dansmith | well, | |
| 15:04:35 | dansmith | if you did, we could have FKs and not have to conditionally handle "if there is a consumer, else..." anywhere else in the code | |
| 15:06:22 | melwitt | mnaser: just updated the devstack patch, fyi | |
| 15:06:23 | efried | the tradeoff being that we would have to have "if user/project is $dummy then don't display it" | |
| 15:06:30 | efried | etc | |
| 15:06:34 | jaypipes | dansmith: there's only a single place we do that in my code series. | |
| 15:06:37 | dansmith | efried: you must have that anyway | |
| 15:06:59 | dansmith | efried: in case someone creates an allocation with a new one and then displays it with an old one | |
| 15:07:01 | efried | right, just sayin you're not really saving anything. | |
| 15:07:24 | efried | sorry, 'tradeoff' isn't the right word. | |
| 15:07:25 | dansmith | jaypipes: but anyone that uses the api has to have that code as well yeah? | |
| 15:07:57 | dansmith | jaypipes: because if the allocation was created with an older one, then the newer microversion has to show something different and every consumer has to handle that | |
| 15:08:14 | dansmith | which I know they do today, I'm just saying I think this was done wrong from the beginning and we should correct it | |
| 15:08:36 | jaypipes | dansmith: so instead of switching on project_id == None, they would need to switch on project_id == <sentinel>? | |
| 15:08:37 | dansmith | having the ambiguity on both ends makes this more complicated | |
| 15:08:52 | dansmith | jaypipes: they being the api consumer? | |
| 15:09:07 | jaypipes | dansmith: yes | |
| 15:09:34 | dansmith | I'm not sure why they would be "switching" on that value at all | |
| 15:09:37 | jaypipes | dansmith: I thought project == None was as good a representation as any | |
| 15:10:16 | jaypipes | dansmith: lemme back up and make sure I understand your concern | |