| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-10 | |||
| 20:47:40 | sean-k-mooney | the migration successes as long as the cores used in the souce host exist on the dest | |
| 20:47:45 | sean-k-mooney | but no xml is updated | |
| 20:47:57 | mriedem | so it goes back to the bug behavior | |
| 20:47:58 | mriedem | right? | |
| 20:48:03 | sean-k-mooney | yep | |
| 20:48:15 | sean-k-mooney | it goes back to the current master/stien behavior | |
| 20:48:20 | mriedem | yeah i don't know that we really need to bend over backward to try and detect that from conductor | |
| 20:49:05 | mriedem | if you're computes are fully upgraded and restarted and reporting as train service versions you should have also removed any manual pins | |
| 20:49:11 | sean-k-mooney | if you dont have can_live_migrate_numa set we will block it anyway | |
| 20:49:15 | mriedem | *your | |
| 20:49:44 | mriedem | not with the new checks | |
| 20:49:49 | mriedem | we would totally ignore can_live_migrate_numa | |
| 20:49:59 | mriedem | as long as all computes are reporting train | |
| 20:50:04 | sean-k-mooney | i thik we have a min compute servcie check too before we ignore it | |
| 20:50:20 | sean-k-mooney | oh thats the compute service | |
| 20:50:22 | sean-k-mooney | not the RPC | |
| 20:50:25 | mriedem | if we pass that check we ignore the config | |
| 20:50:30 | mriedem | right | |
| 20:51:09 | sean-k-mooney | well ya i agree we likely dont need to bend over backwards to check this in the conductor | |
| 20:52:10 | sean-k-mooney | anyway my concentration is totally gone so ill call it a day. | |
| 20:52:42 | sean-k-mooney | artom: ill test v 49 tommorow when you have addressed mriedem comments | |
| 21:10:33 | mriedem | melwitt: brinzhang has been asking me to review this api change https://review.opendev.org/#/c/674243/ but i haven't had time with the bw provider migrate and numa lm stuff - maybe you can peruse it? | |
| 21:11:09 | artom | mriedem, ack, thanks! | |
| 21:11:22 | artom | (The func test can wait until after FF if that's what it comes to, right?) | |
| 21:11:39 | artom | I mean, it's not plan A, but... | |
| 21:13:21 | mriedem | my priority is sean's manual testing and the gate job | |
| 21:13:50 | mriedem | brinzhang: a follow up is not the correct answer to all issues https://review.opendev.org/#/c/679413/4 | |
| 21:19:03 | melwitt | mriedem: will do | |
| 21:25:47 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/659691 | |
| 21:25:48 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/670213 | |
| 21:50:02 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/659691 | |
| 21:50:02 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/670213 | |
| 23:34:51 | openstackgerrit | Merged openstack/nova master: api-ref: fix server topology "host_numa_node" field param name https://review.opendev.org/680775 | |
| #openstack-nova - 2019-09-11 | |||
| 00:20:35 | melwitt | sean-k-mooney: fyi I've proposed a stable/stein release of os-vif. I didn't find one already proposed https://review.opendev.org/681358 | |
| 00:31:48 | sean-k-mooney | release or branch. there is a a stable branch but i dont think we have done a stabel release in a while | |
| 00:33:14 | sean-k-mooney | melwitt: in either case a release proably should be done as i dont think we have release since we fixed the cve with the linux bridge flooding | |
| 00:33:15 | alex_xu | sean-k-mooney: so we are still good at current upgrade plan for pcpu? I saw there a looong discussion on that | |
| 00:33:52 | sean-k-mooney | alex_xu: am i think dan wants to make the config outhion not required | |
| 00:34:17 | alex_xu | sean-k-mooney: ok, i see now. | |
| 00:34:26 | sean-k-mooney | by trying to claim with PCPUs and if that fails fall back to claiming with VCPUs and rely on the numa toplogy filter to validate it | |
| 00:34:34 | melwitt | sean-k-mooney: release, for stable branch. and correct, we've not had a release of the stable/stein branch since the cve fix | |
| 00:35:02 | sean-k-mooney | melwitt: ya so i proably should have proposed one when that merged. thanks for doing it | |
| 00:35:23 | alex_xu | sean-k-mooney: thanks, that is interesting... | |
| 00:35:45 | alex_xu | how can I know whether I need to stop fallback after upgrade | |
| 00:35:50 | melwitt | sean-k-mooney: np. I just wanted to make sure I didn't duplicate a release proposal in case I hadn't found it in my gerrit search | |
| 00:37:15 | sean-k-mooney | i think the release are ment to be on a topic branch with the project name but i dont see one looking quickly either | |
| 00:38:25 | sean-k-mooney | ya looking at all the open patches agains the release repo yours is the only os-vif one i see | |
| 00:39:16 | sean-k-mooney | alex_xu: basicailly how it would work is it would ould fallback if the first placmeent query with PCPUs failed then it would try with VCPUs | |
| 00:40:06 | sean-k-mooney | the numa toplogy filter could tell if the compute node was an train node or a pre train node based on the field in the host numa toplogy object | |
| 00:40:36 | sean-k-mooney | so on train nodes all either there ill be no pinned cpus or the will all be used | |
| 00:40:42 | melwitt | sean-k-mooney: lol, looks like my beloved auto topic from git-review removed my good branch name | |
| 00:40:47 | sean-k-mooney | for a stine node we would usee the old logic | |
| 00:41:02 | sean-k-mooney | melwitt: i think that is why it was removed | |
| 00:41:14 | melwitt | really??? | |
| 00:41:15 | melwitt | :P | |
| 00:41:26 | melwitt | I still like it most of the time | |
| 00:41:48 | sean-k-mooney | i think. that if you name you r branches bug/whatever and do git review it will keep it | |
| 00:42:07 | sean-k-mooney | at least i have still been relying on haveing the correct local brnach name | |
| 00:42:22 | alex_xu | sean-k-mooney: ah... | |
| 00:42:59 | sean-k-mooney | alex_xu: im still not 100% sure there are no issue with that but it sound like it might work | |
| 00:43:20 | sean-k-mooney | i think stephen is going to try and code it up tommorow/today wehn he is online in 8 hours | |
| 00:43:25 | alex_xu | yea, I'm trying to build up how that works in my mind also | |
| 00:44:06 | alex_xu | anyway I need moving to office, then continue to look at, see if anything I can help | |
| 00:44:08 | alex_xu | sean-k-mooney: thanks | |
| 00:45:14 | sean-k-mooney | i think it should work in theroy. my concern would be the use of the limit parmater on the second query but in principal i dont imedialy see why it would not work | |
| 00:46:51 | sean-k-mooney | i had assumed somthing like a fallback mechaniums woudl be deamed too complicated. but i guess its for just one cycle | |
| 00:47:30 | sean-k-mooney | well proably more with our tenency not to delete things right away | |
| 00:49:17 | alex_xu | yea, but that fallback should be existing in the whole U release | |
| 00:56:49 | openstackgerrit | weibin proposed openstack/nova master: Add support for using ceph RBD ereasure code https://review.opendev.org/681188 | |
| 01:04:28 | sean-k-mooney | alex_xu: ya we might have to wait till V to drop it but in V it could be removed | |
| 01:06:02 | sean-k-mooney | we want to support people moving from stine to train without needed to modify anything but train to ussuri was when we were thingin of chaning the default to in the config and removing the confiv in V | |
| 01:06:38 | sean-k-mooney | in anycase before you upgrade form U to V we want people to have moved to useing the new way of tracking | |
| 01:10:09 | sean-k-mooney | mriedem: the numa job passed by the way https://review.opendev.org/#/c/680739/ result are here https://zuul.opendev.org/t/openstack/build/2ad3676591e440b6a76d30e966833a49 | |
| 01:12:37 | mriedem | 2am in ireland, go to bed | |
| 01:12:50 | mriedem | but thanks :) | |
| 01:14:22 | sean-k-mooney | i will soon im just finishing playing a game to unwind. irc is on a different monitor | |
| 01:23:39 | alex_xu | ls | |
| 01:23:42 | alex_xu | oops | |
| 01:35:30 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: NUMA live migration support https://review.opendev.org/634606 | |
| 01:35:30 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Deprecate CONF.workarounds.enable_numa_live_migration https://review.opendev.org/640021 | |
| 01:35:31 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595 | |
| 02:02:06 | brinzhang | mriedem:https://review.opendev.org/#/c/679413/4/nova/objects/migration.py@172 | |
| 02:02:36 | brinzhang | in self._context are not contain user_id and project_id | |
| 02:04:27 | brinzhang | mriedem:IMO, when the operator post the request, the user_id (self._context._user_id) need to record, isn't it? | |
| 03:04:51 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add operator user_id/project_id to the migrations https://review.opendev.org/679413 | |
| 03:09:19 | openstackgerrit | Merged openstack/nova master: Fixing broken links https://review.opendev.org/681206 | |
| 04:49:08 | openstackgerrit | Merged openstack/nova master: Fix the race in confirm resize func test https://review.opendev.org/681238 | |
| 05:02:09 | openstackgerrit | Alex Xu proposed openstack/nova master: Counting both of VCPU and PCPU as core quota https://review.opendev.org/681374 | |
| 05:02:58 | alex_xu | stephenfin: efried sean-k-mooney ^ hope this is what you need | |
| 05:04:23 | openstackgerrit | Alex Xu proposed openstack/nova master: Counting both of VCPU and PCPU as core quota https://review.opendev.org/681374 | |
| 05:34:18 | openstackgerrit | Alex Xu proposed openstack/nova master: Counting both of VCPU and PCPU as core quota https://review.opendev.org/681374 | |
| 05:43:13 | alex_xu | stephenfin: https://review.opendev.org/#/c/681374/ according this http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2019-09-10.log.html#t2019-09-10T20:04:42, in case you don't know | |
| 06:25:55 | openstackgerrit | Alex Xu proposed openstack/nova master: DNM: Try to fallback to PCPU request when VCPU failed https://review.opendev.org/681383 | |
| 07:29:00 | openstackgerrit | Alex Xu proposed openstack/nova master: DNM: Try to fallback to PCPU request when VCPU failed https://review.opendev.org/681383 | |
| 07:31:58 | alex_xu | stephenfin: efried sean-k-mooney ^ also tested the dansmith idea | |
| 07:47:03 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Make _revert_allocation nested allocation aware https://review.opendev.org/676138 | |
| 07:49:20 | gibi | bauzas: I needed to recheck anyhow and later part of the series needs a rebase to pick up the race fix. Could you re +W ^^ | |
| 07:49:23 | gibi | ? | |
| 07:49:29 | bauzas | gibi: sure thing | |
| 07:49:32 | gibi | thanks | |