| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-12 | |||
| 16:49:11 | stephenfin | hw:cpu_policy is syntactic sugar | |
| 16:49:26 | sean-k-mooney | i dont think it should be jsut syntatic sugar | |
| 16:49:31 | mriedem | efried_afk: in case you didn't see i've created https://etherpad.openstack.org/p/nova-train-release-todo and added it to the meeting agenda | |
| 16:49:48 | sean-k-mooney | i was pretty sure we agree in the reve to explcitly not support resouce:PCPUs | |
| 16:50:21 | sean-k-mooney | this came up in the pmem seriese too | |
| 16:51:00 | sean-k-mooney | we are not infering pmem usage form pmem resocues:... extra specs | |
| 16:53:26 | openstackgerrit | melanie witt proposed openstack/nova master: Add user_id and project_id column to Migration https://review.opendev.org/673990 | |
| 16:53:27 | openstackgerrit | melanie witt proposed openstack/nova master: Set user_id/project_id from context when creating a Migration https://review.opendev.org/679413 | |
| 16:53:28 | openstackgerrit | melanie witt proposed openstack/nova master: Filter migrations by user_id/project_id https://review.opendev.org/674243 | |
| 16:58:05 | sean-k-mooney | anyway i have more or less said my pieice but the final point i want to make is that if you use resouces:* you will either have to do a resize to a flaovr without if if that resouce moves in placmenet before upgrading or we will have to do an online data migration of all instance embeded flavor or the vm will not be able to cold/live migrate after an upgrade from on version of nova to another | |
| 16:58:41 | sean-k-mooney | well or a resize to the new flavor with the updated extra specs | |
| 16:58:51 | sean-k-mooney | reflectign the toplogy change | |
| 16:59:11 | stephenfin | aye, that'll be the same for vGPU too, unfortunately | |
| 16:59:19 | stephenfin | I'd rather like to provide a flavor migration tool in the future | |
| 16:59:36 | stephenfin | Migrate all flavors included embedded ones, as a way of deprecating old extra specs | |
| 16:59:41 | stephenfin | but I haven't really thought it throuhg | |
| 16:59:44 | sean-k-mooney | we could but its still not a good thing to encurage people to use | |
| 16:59:55 | mriedem | melwitt: i'm +2 on those bottom 2 changes now | |
| 16:59:56 | stephenfin | We'll do it in U, eh :) | |
| 17:00:05 | mriedem | if the api change doesn't make train at least the data will start flowing | |
| 17:00:09 | sean-k-mooney | i hope not | |
| 17:00:21 | sean-k-mooney | i dont think we should be deprecating those extrapecs | |
| 17:00:38 | mriedem | stephenfin: heh https://review.opendev.org/#/c/637217/ | |
| 17:01:04 | mriedem | "This code can be removed in Queens" | |
| 17:01:13 | melwitt | mriedem: ack thanks | |
| 17:02:03 | sean-k-mooney | anyway that not an somthing we are doing today | |
| 17:02:07 | stephenfin | \o/ | |
| 17:03:02 | stephenfin | I have the nova-network series ready to go once U opens up | |
| 17:05:35 | sean-k-mooney | stephenfin: what you crurrtly hav is inline with the spec https://specs.openstack.org/openstack/nova-specs/specs/train/approved/cpu-resources.html#example-flavor-configurations so i guess that is what we are stuck with. | |
| 17:06:18 | stephenfin | yeah, I haven't pulled the rug out from anyone here | |
| 17:06:27 | stephenfin | jaypipes wanted the resources-style syntax | |
| 17:06:35 | stephenfin | you wanted the hw:cpu_policy-style | |
| 17:07:02 | sean-k-mooney | yes we have both i would have hoped we could use both together if they dont conflict | |
| 17:08:07 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] legacy vcpu_pin_set pinning with shared emulator threads https://review.opendev.org/681840 | |
| 17:13:30 | sean-k-mooney | https://review.opendev.org/#/c/681773/ is now merged so FN should be back in rotation so ill also kick off the multi numa migration run | |
| 17:13:43 | sean-k-mooney | with stephens changes | |
| 17:13:49 | sean-k-mooney | e.g. https://review.opendev.org/#/c/681771/1 | |
| 17:20:04 | sean-k-mooney | i seam to be getting node failures form the vexxhost lables so if thoese all fail ill swap it over to FN since that seams to be working proably again. | |
| 17:39:47 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] cpu pinning testing https://review.opendev.org/681807 | |
| 17:39:48 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] test with dedicated cpus only https://review.opendev.org/681827 | |
| 17:39:48 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] legacy vcpu_pin_set pinning with shared emulator threads https://review.opendev.org/681840 | |
| 17:41:20 | sean-k-mooney | ok they are all queued to run on FN now | |
| 17:46:01 | efried | stephenfin: I think I'm caught up since nap. Going to look at https://review.opendev.org/#/c/681750/ (additional func tests) now. Anything else I need to hit? | |
| 17:46:31 | stephenfin | efried: I don't think so | |
| 17:46:47 | efried | stephenfin: you still running up the vpmem series? | |
| 17:47:01 | sean-k-mooney | we should get the results fomr FN in about an hour by the way | |
| 17:47:06 | efried | noyce | |
| 17:47:17 | stephenfin | I'm working through manual tests at the moment but once that's done, I'm probably going to string together NUMA-aware live migration, vPMEM, and PCPU into one giant chain | |
| 17:47:47 | sean-k-mooney | the result for combining the numa migration serise with PCU will report back first then teh 3 toplogies for PCPUS series only | |
| 17:47:59 | stephenfin | on account of the few (trivial, but not auto-resolvable) merge conflicts between the three | |
| 17:48:20 | sean-k-mooney | im gong to go eat something. be back in a while | |
| 17:59:51 | sean-k-mooney | stephenfin: can you wait till http://zuul.openstack.org/stream/c1f6ccc551c649bab6f60911fd3485ff?logfile=console.log complete before you push that chain | |
| 18:00:03 | stephenfin | sean-k-mooney: sure, I won't be doing it for a while | |
| 18:00:39 | sean-k-mooney | ok it just started running tempest tests | |
| 18:01:07 | sean-k-mooney | that is numa migration + pcpu in placmenet | |
| 18:01:33 | sean-k-mooney | it would be nice to wait for the 3 other jobs too but if that one pass thats a good start | |
| 18:01:50 | sean-k-mooney | oh the first live migration test passed | |
| 18:04:08 | efried | \o/ | |
| 18:06:19 | sean-k-mooney | one of the tests failed so far but the rest are all passing. any idea what http://paste.openstack.org/show/775434/ is | |
| 18:07:23 | sean-k-mooney | it looks like a delete failed but we will need the full logs to know why | |
| 18:07:31 | sean-k-mooney | it might be unrelated | |
| 18:12:20 | aspiers | efried: I fixed the glance metadata and it makes SEV quite nice to set up from Horizon https://review.opendev.org/#/c/681866/ | |
| 18:13:13 | sean-k-mooney | did you also update the usful image properts. apparenlty that is a thing you are ment too do | |
| 18:13:23 | aspiers | sean-k-mooney: click the link ;-) | |
| 18:13:45 | sean-k-mooney | yep just did and you did + added the release note | |
| 18:13:56 | aspiers | sean-k-mooney: yes I extended your release note | |
| 18:13:57 | sean-k-mooney | so that shoudl be all in order | |
| 18:14:10 | sean-k-mooney | actully i exteded someone elses | |
| 18:14:25 | aspiers | and I used git blame to find a good commit to copy and it was yours ;) | |
| 18:14:25 | sean-k-mooney | they uses 1 release note per release apparely for al the metadefs | |
| 18:14:30 | aspiers | weird | |
| 18:14:36 | aspiers | but better to stay consistent | |
| 18:15:24 | aspiers | efried, sean-k-mooney: look how pretty it is! https://photos.app.goo.gl/MTuTzPnz165bVVaC6 | |
| 18:15:31 | sean-k-mooney | if it works for them. why not. the only issue would be backporting could be weird but you would not backport them anyway | |
| 18:15:56 | sean-k-mooney | ha you with your suse theming | |
| 18:16:03 | aspiers | :-D | |
| 18:16:07 | aspiers | believe it or not that is devstack | |
| 18:16:35 | sean-k-mooney | its says suse openstack cloud | |
| 18:16:41 | aspiers | yup | |
| 18:16:47 | aspiers | it's still devstack ;) | |
| 18:16:50 | aspiers | with our theme | |
| 18:16:50 | sean-k-mooney | oh you have enabled the plugin | |
| 18:16:53 | sean-k-mooney | ya | |
| 18:16:59 | sean-k-mooney | it does look nice | |
| 18:17:21 | sean-k-mooney | aspiers: also its nice to have the docs programatially available | |
| 18:17:39 | aspiers | yes | |
| 18:17:45 | sean-k-mooney | you could in princapal hit the metadef api for the cli or heat too | |
| 18:17:49 | aspiers | I didn't really know it worked this way until today | |
| 18:17:51 | sean-k-mooney | heat does actully use it | |
| 18:17:57 | aspiers | Yeah I guessed it would | |
| 18:18:18 | sean-k-mooney | so the heat ui for flavors will pull that info too | |
| 18:18:23 | aspiers | exactly | |
| 18:19:01 | sean-k-mooney | i have a todo to go through all th eimage porperties we suppport in nova and figure out where the gaps are | |
| 18:19:12 | sean-k-mooney | i would like to get them all up to date | |
| 18:19:13 | stephenfin | so I have never done boot from volume before | |
| 18:19:18 | stephenfin | and our docs don't work | |
| 18:19:21 | stephenfin | hurrah! | |
| 18:19:22 | stephenfin | :D | |
| 18:19:25 | sean-k-mooney | do it from horizon | |
| 18:19:37 | stephenfin | looking at https://docs.openstack.org/nova/latest/user/launch-instance-from-volume.html | |