| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-09 | |||
| 16:27:06 | sean-k-mooney | well we could do it without a numa flavor later or i can stack and check it by hand now | |
| 16:27:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add reshaper for PCPU https://review.opendev.org/674895 | |
| 16:27:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: fakelibvirt: Make 'Connection.getHostname' unique https://review.opendev.org/681060 | |
| 16:27:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Mock 'libvirt_utils.file_open' properly https://review.opendev.org/681061 | |
| 16:29:08 | stephenfin | artom: https://review.opendev.org/#/c/681060/ | |
| 16:30:58 | artom | stephenfin, way to steal credit brah ;) | |
| 16:31:15 | stephenfin | desperate times /o\ | |
| 16:31:21 | mriedem | co-authored-by | |
| 16:31:25 | artom | stephenfin, you proll'y want to explain in more details what those lambda scoping issues were | |
| 16:31:35 | stephenfin | I should have set the authorship now that I'm thinking about it | |
| 16:31:49 | cdent | mriedem: I did a quick survey to look for other duplicate placement fixtures and did not find | |
| 16:32:00 | bauzas | melwitt: gibi: I'm +W with the first 3 changes | |
| 16:32:00 | artom | mriedem, to be fair, I didn't write *that* solution | |
| 16:32:05 | artom | I just found the problem | |
| 16:32:06 | cdent | that's me done for today I reckon | |
| 16:32:10 | mriedem | cool | |
| 16:32:12 | mriedem | thanks | |
| 16:32:12 | bauzas | melwitt: gibi: see you tomorrow | |
| 16:32:34 | artom | stephenfin, because they're really not obviousy | |
| 16:32:36 | artom | *obvious | |
| 16:33:41 | sean-k-mooney | i just clicked on it and no ideay lambda were related to this other then th efact i saw lambdas in the code so ya | |
| 16:44:15 | artom | sean-k-mooney, btw, Matt's expecting a vote from you on that NUMA LM patch - just making sure everyone's on the same page :) | |
| 16:44:37 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.79: Add delete_on_termination to volume-attach API https://review.opendev.org/673485 | |
| 16:47:32 | mriedem | ^ i'm +2 on that novaclient change for 2.79 now | |
| 16:47:37 | mriedem | need to get that merged and released by thursday, | |
| 16:47:41 | mriedem | the osc change is also dependent o nit | |
| 16:48:52 | sean-k-mooney | artom: ok i keep getting distracted by jumping between your stuff stephns patach and email. | |
| 16:49:26 | sean-k-mooney | ill kick off a local devstack run with your latest patches then finish my review of stephns code and then review your code | |
| 16:49:35 | artom | sean-k-mooney, I'll stop harassing you - just making sure expectations were clear | |
| 16:49:51 | sean-k-mooney | sorry i ment to finish reviewing stephens code already | |
| 16:50:35 | sean-k-mooney | no worries i just need to focus. i started late today so ill be around for another few hours today | |
| 16:54:04 | mnaser | sean-k-mooney, artom: i know we talked about this but i dont know what that implies on my side | |
| 16:54:46 | mnaser | i can do the custom flavor side if someone wants to volunteer to update the nodepool/infra side of things, i'm a little backed up on things right now | |
| 16:55:39 | artom | mnaser, thanks, but we may have jumped the gun on that | |
| 16:55:54 | mnaser | np :) | |
| 16:55:56 | sean-k-mooney | mnaser: i can do the nodepool/project-config changes | |
| 16:56:09 | sean-k-mooney | mnaser: ideally in U it would be nice to have two providers | |
| 16:56:10 | mnaser | someone will have to let me know how to do/setup the flavor | |
| 16:56:14 | artom | sean-k-mooney, is that... necessary, at this point? | |
| 16:56:27 | sean-k-mooney | artom: nessisary no prefered yes | |
| 16:56:27 | artom | I know FN is... intermittent, but we got the runs we needed, didn't we? | |
| 16:56:59 | artom | sean-k-mooney, agree on that. I guess you're thinking long term? | |
| 16:57:06 | sean-k-mooney | artom: i want to get a perminet job running in U even if its only nightly | |
| 16:57:10 | sean-k-mooney | artom: yes | |
| 16:57:11 | artom | I was concentrating on the short term of getting the series merged | |
| 16:57:25 | sean-k-mooney | we dont need a second provider to merge it correct | |
| 16:57:35 | artom | No | |
| 16:58:04 | sean-k-mooney | mnaser: its relitivly simple change i was going to wait until after feature freeze when you were less busy to bring it up | |
| 16:58:19 | artom | So, mnaser, while I would definitely love to work with you and Sean (or, let's he honest, let Sean do all the work ;) on getting multi-NUMA flavors from vexxhost, I think we can wait until after FF | |
| 16:58:35 | mnaser | no worries, does it require a patch or is it just a flavor with some extra_specs? | |
| 16:58:46 | artom | mnaser, the latter | |
| 16:58:51 | mnaser | ok so thats easy | |
| 16:58:56 | sean-k-mooney | mnaser: but bassicaly jsut need to add hw:numa_nodes=2 an preferably also add hw:cpu_socket=2 and hw:cpu_threads=2 | |
| 16:59:14 | donnyd | mnaser: yea I put in the flavor data and it just worked on FN | |
| 16:59:27 | mnaser | cool, it should be ok for me to do that | |
| 16:59:54 | donnyd | https://www.irccloud.com/pastebin/KXLeMoFH/ | |
| 17:00:02 | donnyd | mnaser: snippet from FN | |
| 17:00:15 | sean-k-mooney | mnaser: donnyd create a multi-numa lable in nodepool for FN so ideally we woudl use the same one for vexhost to allow it to run on either. again thats a long term thing | |
| 17:00:37 | sean-k-mooney | if we had that it would allow us to propaly test all the numa/pinning/hugepage things in the first party ci | |
| 17:00:53 | sean-k-mooney | with multiple providers to make it redundatnt | |
| 17:01:22 | artom | If we have multiple provider we could even make it more than just experimental (eventually) | |
| 17:02:08 | sean-k-mooney | artom: yep, which is why longter i was hoping to reach out to vexhost and maybe packet to be addtional provier of this lable/flavor | |
| 17:02:30 | sean-k-mooney | but that is a U thing not a now thing | |
| 17:05:08 | artom | yep | |
| 17:13:16 | donnyd | sean-k-mooney: I am going to put the compute service back in operation for FN | |
| 17:13:58 | donnyd | Time is a little short and these resources are kinda needed...I don't want to hold up progress on the *hope that the propane guy actaully even makes it here today | |
| 17:14:20 | donnyd | electrical work is all done.. | |
| 17:15:43 | sean-k-mooney | we had good test result form satusrday and im locall redeploying the latest code | |
| 17:19:57 | donnyd | sean-k-mooney: just waiting on https://review.opendev.org/#/c/681075/ | |
| 17:20:22 | donnyd | FN is up.. .just waiting on this to be able to run jobs | |
| 17:20:47 | sean-k-mooney | ack. thanks | |
| 17:48:10 | mriedem | sean-k-mooney: when you're dug out from the current set of tasks, i found this last week and it's a question for you (or adrianc): https://review.opendev.org/#/c/634606/75..80/nova/compute/manager.py@a6504 | |
| 17:51:37 | sean-k-mooney | mriedem: the sriov claim. ill look into that and respond. after im done with the rest of artoms stuff | |
| 17:52:47 | spatel | sean-k-mooney: ping | |
| 17:53:18 | sean-k-mooney | spatel: hi. sorry dont really have much time to talk today. reviewing/testing several things in parallel | |
| 17:53:24 | spatel | how do i rename instance name in command line? ( i can do that in GUI but i need command line) | |
| 17:53:57 | spatel | sean-k-mooney: no worry, you keep going.. | |
| 17:54:36 | sean-k-mooney | i didnt know you could rename an instance after it was booted so maybe. if its supported in the api it should be doable on the cli | |
| 17:54:48 | sean-k-mooney | if not file a bug for the feature gap | |
| 17:56:05 | artom | spatel, "rename"? Are you sure the GUI doesn't just delete it and create a new one? | |
| 17:56:10 | spatel | I think "update" command should do that, rename is depricated | |
| 17:56:21 | artom | It used to do that for "edit" flavor | |
| 17:56:56 | donnyd | https://www.irccloud.com/pastebin/yhNgJLIr/ | |
| 17:56:57 | spatel | In GUI i can just edit Instance name but i wanted to do with command line. | |
| 17:57:05 | artom | spatel, huh, update seems to do it: https://docs.openstack.org/api-ref/compute/?expanded=update-server-detail#update-server | |
| 17:57:25 | donnyd | openstack server set --name new name old-name | |
| 17:58:14 | donnyd | sean-k-mooney: artom just like that | |
| 17:58:42 | spatel | donnyd: it works :) openstack server set --name iptest-11 iptest-1 | |
| 17:58:55 | donnyd | :) glad I could help | |
| 17:59:01 | spatel | Thanks! | |
| 18:00:08 | sean-k-mooney | im getting distracted but does that update the hostname in the metadata service extra or is it just in nova | |
| 18:00:20 | sean-k-mooney | * in the server show | |
| 18:03:43 | spatel | sean-k-mooney: in server show it changed hostname, i didn't check metadata | |
| 18:08:58 | mriedem | artom: some fresh comments in https://review.opendev.org/#/c/634606/ | |
| 18:10:20 | artom | mriedem, much thanks, looking | |
| 18:10:38 | mriedem | sean-k-mooney: yes you're getting distracted, | |
| 18:10:44 | mriedem | it updates the display name on the server | |
| 18:10:49 | mriedem | not the underlying hostname presented to the guest | |
| 18:27:21 | sean-k-mooney | artom: are you expecting your code to handel the non numa instances | |
| 18:27:36 | artom | sean-k-mooney, yes... | |