| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-16 | |||
| 20:05:29 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: Use tempfile for powervm config drive https://review.openstack.org/610174 | |
| 20:11:27 | mriedem | dansmith: np i got it, it was 1 line | |
| 20:11:39 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix formatting non-templated cell URLs with no config https://review.openstack.org/611094 | |
| 20:12:27 | mriedem | easy fix for another core ^ | |
| 20:17:22 | melwitt | +W | |
| 20:27:14 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Ignore uuid if already set in ComputeNode.update_from_virt_driver https://review.openstack.org/611162 | |
| 20:27:15 | mriedem | efried: jroll: ^ | |
| 20:30:07 | openstackgerrit | melanie witt proposed openstack/nova master: Bump os-brick version to 2.6.1 https://review.openstack.org/611109 | |
| 20:30:52 | efried | I don't get it. | |
| 20:32:25 | openstackgerrit | Sundar Nadathur proposed openstack/nova-specs master: Nova Cyborg interaction specification. https://review.openstack.org/603955 | |
| 20:33:58 | spatel | Folks! i have 64G compute node | |
| 20:34:09 | dansmith | spatel: it's not nice to brag | |
| 20:34:14 | dansmith | mriedem: thanks home skillet | |
| 20:34:16 | spatel | Should i go with 1G hugepage or 2M | |
| 20:34:46 | spatel | dansmith: i was going to write question but hit enter middle of i | |
| 20:34:49 | spatel | dansmith: i was going to write question but hit enter middle of it | |
| 20:34:55 | dansmith | spatel: I know, I'm just joking :P | |
| 20:35:00 | spatel | :) | |
| 20:35:49 | spatel | what do you recommend if that is the case | |
| 20:36:08 | spatel | Problem is if i launch application then it will be hard to adjust those value | |
| 20:36:36 | spatel | currently i have "hugepagesz=2M hugepages=27000 transparent_hugepage=never" | |
| 20:36:50 | dansmith | spatel: you probably want cfriesen | |
| 20:37:09 | spatel | cfriesen: ^^ | |
| 20:38:36 | spatel | He may be not around | |
| 20:38:42 | mriedem | efried: you were on the original regression patch of mine so figured you'd have context | |
| 20:38:51 | mriedem | this https://review.openstack.org/#/c/571535/ | |
| 20:38:56 | efried | mriedem: Yeah, I think I get it now. | |
| 20:39:07 | efried | See if my review comment makes sense. | |
| 20:40:12 | mriedem | efried: yup | |
| 20:40:32 | efried | mriedem: ...and another update | |
| 20:40:33 | cfriesen | spatel: in our testing 2M gave a noticeable benefit. 1G gave some additional benefit but only for specific testcases | |
| 20:40:50 | spatel | There you go!! thanks you | |
| 20:41:15 | mriedem | efried: yup | |
| 20:41:21 | cfriesen | spatel: are you using dedicated CPUs? | |
| 20:41:21 | efried | cool | |
| 20:41:22 | spatel | Do you think 27000 is good number on 64G compute node? | |
| 20:41:30 | spatel | yes I am pinning CPU | |
| 20:41:56 | spatel | i am going to run riak cluster application on this compute node | |
| 20:42:11 | spatel | riak love memory | |
| 20:42:16 | cfriesen | you don't need to allocate hugepages at boot. you can allocate them at runtime | |
| 20:42:33 | spatel | i heard sometime it cause issue during runtime | |
| 20:42:58 | cfriesen | spatel: If you allocate them early during startup the memory hasn't gotten fragmented yet | |
| 20:43:14 | spatel | hmm! | |
| 20:43:39 | spatel | i will keep that in mind then.. | |
| 20:43:48 | cfriesen | it's a bit of a tradeoff, since any memory you reserve for hugepages can't be allocated to small-page instances. Also, you need to keep some 4K memory around for the host itself. | |
| 20:43:59 | spatel | but you have to reboot your flavor also right after adjust hugepage | |
| 20:44:35 | cfriesen | not sure what you mean by "reboot your flavor" :) | |
| 20:44:39 | spatel | I kept 8G memory for host that is why i pick 27000 pages | |
| 20:44:50 | spatel | i meant reboot your VM | |
| 20:44:59 | cfriesen | spatel: that should be fine as long as most of your guests are using hugepages | |
| 20:45:27 | spatel | cfriesen: thanks! in that case i will go with 27000 | |
| 20:46:31 | cfriesen | spatel: changing a flavor and then rebooting your vm won't do anything. the flavor information was cached in the VM at creation time. | |
| 20:47:33 | cfriesen | in the instance object, rather | |
| 20:47:38 | spatel | oh!! | |
| 20:47:41 | spatel | cool | |
| 20:56:01 | mriedem | i still don't understand how this forbidden aggregate placement API thing is going to be used via nova for blazar which is use case in this spec https://review.openstack.org/#/c/603352/ | |
| 20:56:19 | efried | mriedem: You kind of had to be in the room, unfortunately. | |
| 20:56:21 | mriedem | it seems we're gung ho about adding a placement api without any details on how to ues it | |
| 20:56:22 | mriedem | *use | |
| 20:56:43 | mriedem | well, the place to document that is in the spec for those not in the room right? | |
| 20:56:58 | mriedem | if the answer is, "we're going to fork nova and make pre-request filters an extension point" then say that | |
| 20:57:00 | efried | Yes, I agree. If it's not clear to someone who wasn't in the room, it needs a rewrite. | |
| 20:58:34 | mriedem | i particularly want Kevin_Zheng on board with this b/c he had a use case for the dedicated host stuff as well | |
| 20:58:41 | mriedem | but he needs to speak up on the spec review too | |
| 21:02:22 | mriedem_away | time for parent/teacher conferences | |
| 21:18:42 | efried | mnaser, dansmith: Would DISTINCT have done the same thing? And possibly be more efficient? | |
| 21:18:59 | efried | sorry, I'm talking about https://review.openstack.org/#/c/611115/3 | |
| 21:19:07 | mnaser | efried: i dunno, not an sql expert, i didnt try it and it seemed like the.. easier way | |
| 21:42:06 | openstackgerrit | Merged openstack/nova master: Transform volume.usage notification https://review.openstack.org/580345 | |
| #openstack-nova - 2018-10-17 | |||
| 00:14:59 | openstackgerrit | Artom Lifshitz proposed openstack/nova stable/rocky: Handle volume API failure in _post_live_migration https://review.openstack.org/611083 | |
| 00:42:06 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix best_match() deprecation warning https://review.openstack.org/611204 | |
| 01:14:52 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628 | |
| 02:35:02 | openstackgerrit | melanie witt proposed openstack/nova master: Add nova-status upgrade check for consoles https://review.openstack.org/611214 | |
| 02:48:49 | openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: Handle volume API failure in _post_live_migration https://review.openstack.org/611084 | |
| 02:57:32 | openstackgerrit | melanie witt proposed openstack/nova master: Add rolling upgrade info to enable_consoleauth workaround reno https://review.openstack.org/611218 | |
| 03:52:30 | openstackgerrit | Merged openstack/nova master: Handle volume API failure in _post_live_migration https://review.openstack.org/609517 | |
| 03:52:36 | openstackgerrit | Merged openstack/nova master: Increment versioning with pbr instruction https://review.openstack.org/610534 | |
| 06:02:02 | gmann | API office hour time | |
| 06:02:09 | gmann | alex_xu: ping | |
| 06:02:14 | alex_xu | gmann: pong | |
| 06:02:30 | gmann | #startmeeting nova api | |
| 06:02:31 | openstack | Meeting started Wed Oct 17 06:02:30 2018 UTC and is due to finish in 60 minutes. The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 06:02:32 | openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 06:02:34 | openstack | The meeting name has been set to 'nova_api' | |
| 06:02:57 | gmann | who else here today ? | |
| 06:04:01 | gmann | #link https://wiki.openstack.org/wiki/Meetings/NovaAPI#Agenda_for_next_Office_hours | |
| 06:04:04 | gmann | agenda ^^ | |
| 06:04:11 | gmann | #topic API Subteam Tracking | |
| 06:04:23 | gmann | #link https://etherpad.openstack.org/p/stein-nova-subteam-tracking | |
| 06:06:05 | gmann | 1. Boot instance specific storage backend | |
| 06:06:10 | gmann | this is merged which was in runway | |
| 06:06:20 | alex_xu | yup, we done last week | |
| 06:06:47 | gmann | yeah | |
| 06:06:54 | gmann | https://review.openstack.org/#/q/topic:bp/api-extensions-merge-stein+status:open | |
| 06:07:06 | gmann | this is also next in runway but i saw you already reviewed most of them | |
| 06:07:39 | alex_xu | yea, part of review for the last patch | |
| 06:08:04 | alex_xu | https://review.openstack.org/#/c/607088/5/nova/api/openstack/compute/views/servers.py | |
| 06:08:13 | alex_xu | gmann: I guess you just forget to remove one line code | |
| 06:09:19 | gmann | yeah, i am sure i did but got restore from rebse form PS4 to 5 | |
| 06:10:35 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove more code related to extensions and testing https://review.openstack.org/607088 | |