| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-09 | |||
| 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 | artom | I know FN is... intermittent, but we got the runs we needed, didn't we? | |
| 16:56:27 | sean-k-mooney | artom: nessisary no prefered yes | |
| 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... | |
| 18:27:44 | sean-k-mooney | you code is not running in that case | |
| 18:28:05 | artom | Well, only when vcpu_pin_set is configured | |
| 18:28:09 | artom | Otherwise there's nothing to do | |
| 18:28:10 | sean-k-mooney | i dont see the migration objecte being logged and the xml is not updated | |
| 18:28:20 | sean-k-mooney | i have it configured | |
| 18:28:34 | sean-k-mooney | and set to different values on each host | |
| 18:28:50 | sean-k-mooney | oh wait | |
| 18:29:12 | artom | Oh, made me realize a thing | |
| 18:29:18 | artom | If the source has it, but dest doesn | |
| 18:29:22 | artom | doesn't | |
| 18:29:28 | artom | We need to remove cpuset from <vcpus> | |
| 18:29:33 | sean-k-mooney | i still have the the versions pinned | |
| 18:29:35 | artom | Which I don't think I'm doing | |
| 18:29:37 | sean-k-mooney | for the rpc | |
| 18:29:42 | sean-k-mooney | that is why its disabled | |
| 18:30:24 | sean-k-mooney | ill unping the rpc and test again | |
| 18:31:49 | sean-k-mooney | shoudl i resterat all nova service for it to pick up the new rpc version | |
| 18:31:57 | sean-k-mooney | oh both hosts | |
| 18:31:58 | mriedem | artom: dansmith: comments in the big patch https://review.opendev.org/#/c/634606/80 | |
| 18:32:01 | sean-k-mooney | *on | |
| 18:32:18 | mriedem | -1 since changes need to be made, but we can decide if they can just be in a follow up after dansmith goes through it | |
| 18:32:38 | artom | mriedem, dansmith, I need to fix https://review.opendev.org/#/c/635229/61 anyways | |
| 18:32:43 | mriedem | i have to run to an appointment, back in a couple of hours | |
| 18:33:17 | dansmith | I think we can just integrate the fixes into the main patch, if artom is okay with it | |
| 18:33:18 | dansmith | and still have time to get it all landed tomorrow | |
| 18:33:19 | mriedem | artom: ok, well try to coordinate that with sean's local testing and if dan is actively doing a review | |
| 18:33:57 | dansmith | I'm still mulling this pcpu series, so I'm cool waiting until artom has put all your comments into the latest version | |
| 18:34:15 | artom | dansmith, you're suggesting we start +W'ing the bottom patches and fix my own -1 later? | |
| 18:34:19 | dansmith | I can plan to hit that one first thing tomorrow and hopefully be all good for mriedem to be happy with it | |
| 18:34:37 | dansmith | artom: no I'm saying go ahead and fix those things | |
| 18:34:58 | artom | OK rihgt, for a second there I didn't recognize you ;) | |
| 18:35:28 | dansmith | glad we cleared that up :) | |
| 18:36:04 | dansmith | I mean, we could land the bottom patch since its just setup, but I don't think we're down to that level of panic just yet | |