openstackgerrit | Merged openstack/nova master: Functional reproduction for bug 1844993 https://review.opendev.org/683947 | 00:22 |
---|---|---|
openstack | bug 1844993 in OpenStack Compute (nova) "migrate a server with qos port with compute RPC pinned to 5.1 fails and leaves the qos port in an inconsistent state" [High,In progress] https://launchpad.net/bugs/1844993 - Assigned to Balazs Gibizer (balazs-gibizer) | 00:22 |
openstackgerrit | Sundar Nadathur proposed openstack/nova-specs master: Re-proposed Nova Cyborg interaction specification. https://review.opendev.org/682637 | 00:28 |
openstackgerrit | Sundar Nadathur proposed openstack/nova-specs master: Updated Nova-Cyborg interaction spec. https://review.opendev.org/684151 | 00:28 |
*** gbarros has joined #openstack-nova | 01:09 | |
*** brinzhang has joined #openstack-nova | 01:12 | |
*** Kevin_Zheng has joined #openstack-nova | 01:21 | |
*** hongbin has joined #openstack-nova | 01:39 | |
*** hongbin_ has joined #openstack-nova | 01:44 | |
*** hongbin has quit IRC | 01:46 | |
*** yedongcan has joined #openstack-nova | 01:48 | |
*** larainema has joined #openstack-nova | 02:28 | |
openstackgerrit | Merged openstack/nova master: Add librsvg2* to bindep https://review.opendev.org/683003 | 02:36 |
*** BjoernT has joined #openstack-nova | 02:40 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova stable/rocky: Replace glance command with openstack command https://review.opendev.org/637060 | 02:41 |
*** markvoelker has joined #openstack-nova | 02:41 | |
*** BjoernT has quit IRC | 02:43 | |
*** BjoernT has joined #openstack-nova | 02:45 | |
*** ricolin has joined #openstack-nova | 02:48 | |
*** boxiang has joined #openstack-nova | 02:53 | |
*** BjoernT has quit IRC | 02:53 | |
*** zhubx has quit IRC | 02:55 | |
*** BjoernT has joined #openstack-nova | 03:02 | |
*** BjoernT has quit IRC | 03:17 | |
*** BjoernT has joined #openstack-nova | 03:22 | |
*** hongbin has joined #openstack-nova | 03:28 | |
*** hongbin_ has quit IRC | 03:30 | |
*** hongbin has quit IRC | 03:38 | |
*** hongbin has joined #openstack-nova | 03:39 | |
*** BjoernT_ has joined #openstack-nova | 03:43 | |
*** BjoernT has quit IRC | 03:45 | |
*** hongbin has quit IRC | 03:48 | |
*** hongbin_ has joined #openstack-nova | 03:49 | |
*** hongbin_ has quit IRC | 03:49 | |
*** artom has quit IRC | 03:56 | |
*** jamesdenton has quit IRC | 04:07 | |
*** takashin has quit IRC | 04:16 | |
*** BjoernT_ has quit IRC | 04:17 | |
*** eharney has joined #openstack-nova | 04:21 | |
*** mkrai has joined #openstack-nova | 04:22 | |
*** gbarros has quit IRC | 04:29 | |
*** eharney has quit IRC | 04:33 | |
*** jamesden_ has joined #openstack-nova | 04:43 | |
*** ratailor has joined #openstack-nova | 04:43 | |
*** takashin has joined #openstack-nova | 04:47 | |
*** ratailor has quit IRC | 05:00 | |
*** udesale has joined #openstack-nova | 05:00 | |
*** bnemec has quit IRC | 05:09 | |
*** igordc has quit IRC | 05:12 | |
*** brault has quit IRC | 05:12 | |
*** Luzi has joined #openstack-nova | 05:13 | |
*** markvoelker has quit IRC | 05:14 | |
*** brault has joined #openstack-nova | 05:15 | |
*** bnemec has joined #openstack-nova | 05:15 | |
*** ratailor has joined #openstack-nova | 05:26 | |
*** davee__ has joined #openstack-nova | 05:38 | |
*** davee_ has quit IRC | 05:39 | |
*** dpawlik has joined #openstack-nova | 05:44 | |
*** slaweq has joined #openstack-nova | 06:09 | |
*** brault has quit IRC | 06:17 | |
*** osmanlicilegi has quit IRC | 06:20 | |
*** osmanlicilegi has joined #openstack-nova | 06:21 | |
*** cfriesen has quit IRC | 06:23 | |
*** rpittau|afk is now known as rpittau | 06:43 | |
*** abhishekk has joined #openstack-nova | 06:46 | |
*** lpetrut has joined #openstack-nova | 06:47 | |
*** eharney has joined #openstack-nova | 06:52 | |
*** pcaruana has joined #openstack-nova | 06:56 | |
*** damien_r has joined #openstack-nova | 07:01 | |
*** awalende has joined #openstack-nova | 07:04 | |
*** donnyd has quit IRC | 07:07 | |
*** tesseract has joined #openstack-nova | 07:07 | |
*** donnyd has joined #openstack-nova | 07:08 | |
*** markvoelker has joined #openstack-nova | 07:15 | |
*** xek has joined #openstack-nova | 07:16 | |
*** Luzi has quit IRC | 07:18 | |
*** markvoelker has quit IRC | 07:19 | |
*** osmanlicilegi has left #openstack-nova | 07:26 | |
*** ralonsoh has joined #openstack-nova | 07:34 | |
*** brault has joined #openstack-nova | 07:38 | |
*** maciejjozefczyk has joined #openstack-nova | 07:38 | |
*** brault has quit IRC | 07:43 | |
*** ivve has joined #openstack-nova | 07:52 | |
*** priteau has joined #openstack-nova | 07:58 | |
*** psachin has joined #openstack-nova | 07:59 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject migration with QoS port from conductor if RPC pinned https://review.opendev.org/683948 | 08:13 |
*** tkajinam has quit IRC | 08:14 | |
*** derekh has joined #openstack-nova | 08:28 | |
*** takashin has left #openstack-nova | 08:30 | |
*** liuyulong_ has joined #openstack-nova | 08:33 | |
*** mkrai has quit IRC | 08:41 | |
*** mkrai has joined #openstack-nova | 08:42 | |
*** mtreinish has quit IRC | 08:43 | |
*** mtreinish has joined #openstack-nova | 08:44 | |
*** amorin has quit IRC | 08:44 | |
*** stephenfin has quit IRC | 08:44 | |
*** dosaboy has quit IRC | 08:45 | |
*** frickler has quit IRC | 08:45 | |
*** lyarwood has quit IRC | 08:45 | |
*** stephenfin has joined #openstack-nova | 08:45 | |
*** amorin has joined #openstack-nova | 08:45 | |
*** dosaboy has joined #openstack-nova | 08:45 | |
*** frickler has joined #openstack-nova | 08:46 | |
*** lyarwood has joined #openstack-nova | 08:46 | |
openstackgerrit | Arthur Dayne proposed openstack/nova master: Fix block disk attachment failure https://review.opendev.org/682772 | 08:49 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove upgrade specific info from user facing exception text https://review.opendev.org/684253 | 08:49 |
*** ccamacho has joined #openstack-nova | 09:01 | |
*** rcernin has quit IRC | 09:04 | |
*** brault has joined #openstack-nova | 09:05 | |
openstackgerrit | Arthur Dayne proposed openstack/nova master: Fix block disk attachment failure https://review.opendev.org/682772 | 09:05 |
*** ricolin_ has joined #openstack-nova | 09:05 | |
*** ricolin has quit IRC | 09:08 | |
*** brault has quit IRC | 09:09 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move HostNameWeigher to a common fixture https://review.opendev.org/683874 | 09:16 |
*** yikun has quit IRC | 09:22 | |
*** ricolin_ is now known as ricolin | 09:23 | |
*** ratailor_ has joined #openstack-nova | 09:26 | |
*** ratailor has quit IRC | 09:28 | |
*** ociuhandu has joined #openstack-nova | 09:30 | |
*** cdent has joined #openstack-nova | 09:37 | |
*** ttsiouts has joined #openstack-nova | 09:39 | |
*** brault has joined #openstack-nova | 09:42 | |
*** brault has quit IRC | 09:42 | |
*** brault has joined #openstack-nova | 09:42 | |
*** tellesnobrega has joined #openstack-nova | 09:58 | |
tellesnobrega | hi folks, is there a way from nova to check state of instances and update db if it detects that an instance is not running anymore? | 09:59 |
tellesnobrega | I saw the reset-state command but wanted to make sure it does what I'm looking for | 10:00 |
*** macz has quit IRC | 10:10 | |
*** brault has quit IRC | 10:11 | |
*** openstackstatus has quit IRC | 10:12 | |
*** openstack has joined #openstack-nova | 10:15 | |
*** ChanServ sets mode: +o openstack | 10:15 | |
*** dtantsur|afk is now known as dtantsur | 10:17 | |
*** mkrai_ has joined #openstack-nova | 10:22 | |
*** mkrai_ has quit IRC | 10:22 | |
*** mkrai__ has joined #openstack-nova | 10:22 | |
*** pcaruana has quit IRC | 10:24 | |
*** mkrai has quit IRC | 10:25 | |
*** mkrai_ has joined #openstack-nova | 10:25 | |
*** mkrai__ has quit IRC | 10:27 | |
*** ttsiouts has quit IRC | 10:28 | |
*** ttsiouts has joined #openstack-nova | 10:28 | |
*** mkrai_ has quit IRC | 10:30 | |
*** ttsiouts has quit IRC | 10:33 | |
*** markvoelker has joined #openstack-nova | 10:40 | |
*** tbachman has quit IRC | 10:44 | |
*** sapd1_x has quit IRC | 10:46 | |
*** yedongcan has left #openstack-nova | 10:47 | |
*** pcaruana has joined #openstack-nova | 10:48 | |
*** ratailor_ has quit IRC | 10:58 | |
*** ociuhandu has quit IRC | 11:00 | |
*** bbowen has quit IRC | 11:01 | |
*** ociuhandu has joined #openstack-nova | 11:01 | |
*** liuyulong_ has quit IRC | 11:06 | |
*** ttsiouts has joined #openstack-nova | 11:07 | |
*** udesale has quit IRC | 11:08 | |
*** udesale has joined #openstack-nova | 11:09 | |
*** ociuhandu has quit IRC | 11:09 | |
*** ociuhandu has joined #openstack-nova | 11:09 | |
*** sapd1_x has joined #openstack-nova | 11:15 | |
*** artom has joined #openstack-nova | 11:17 | |
*** sapd1_x has quit IRC | 11:51 | |
openstackgerrit | Jacek Tomasiak proposed openstack/nova master: Ensure volume filesystems are mounted on resume https://review.opendev.org/684290 | 11:54 |
*** derekh has quit IRC | 12:00 | |
*** larainema has quit IRC | 12:03 | |
gibi | bauzas: left comments and a bug report in https://review.opendev.org/#/c/670112 | 12:07 |
*** awalende_ has joined #openstack-nova | 12:09 | |
*** awalende has quit IRC | 12:09 | |
cdent | gibi: you have a theory about what's going wrong there? | 12:11 |
gibi | cdent: not yet but I can dig in if needed | 12:11 |
cdent | no, I was just curious if anything obvious stuck out. and was very happy to see you testing it with real stuff | 12:12 |
cdent | presumably better functional tests is needed | 12:12 |
*** tbachman has joined #openstack-nova | 12:16 | |
*** bbowen has joined #openstack-nova | 12:20 | |
*** mrch_ has quit IRC | 12:21 | |
*** mkrai has joined #openstack-nova | 12:21 | |
gibi | cdent: I have one commment about context targeting when querying instances from the db that could be the reason | 12:21 |
*** tbachman has quit IRC | 12:21 | |
*** mrch_ has joined #openstack-nova | 12:22 | |
*** mkrai has quit IRC | 12:22 | |
*** mkrai has joined #openstack-nova | 12:22 | |
*** belmoreira has joined #openstack-nova | 12:27 | |
*** tbachman has joined #openstack-nova | 12:27 | |
gibi | cdent: yeah something with the db access is problematic as the script does not even found the compute node. I guess it querying from the wrong cell db | 12:28 |
*** jamesden_ has quit IRC | 12:34 | |
*** jamesden_ has joined #openstack-nova | 12:34 | |
*** brault has joined #openstack-nova | 12:37 | |
*** brault has quit IRC | 12:40 | |
*** brault has joined #openstack-nova | 12:40 | |
bauzas | gibi: thanks for the comments, as great as usual ;) | 12:42 |
bauzas | gibi: I'll first write the last functional test and then I'll look at your comments | 12:42 |
gibi | bauzas: sure. If you need me to re-run the test with more logs or a differnet patch set just let me know | 12:43 |
gibi | bauzas: or I can provide my devstack conf to test with bandwidth allocation | 12:43 |
*** mkrai has quit IRC | 12:43 | |
bauzas | gibi: well, why not testing it for bandwith allocations | 12:44 |
bauzas | gibi: the main issue is that we need also to check child allocations | 12:44 |
*** jamesden_ is now known as jamesdenton | 12:44 | |
*** psachin has quit IRC | 12:49 | |
*** brault has quit IRC | 12:49 | |
*** brault has joined #openstack-nova | 12:50 | |
*** jaosorior has quit IRC | 12:59 | |
*** jdillaman has joined #openstack-nova | 13:00 | |
*** ociuhandu has quit IRC | 13:03 | |
*** brault has quit IRC | 13:03 | |
*** mkrai has joined #openstack-nova | 13:03 | |
*** henriqueof1 has joined #openstack-nova | 13:04 | |
*** henriqueof has quit IRC | 13:05 | |
*** brault has joined #openstack-nova | 13:05 | |
*** brault has quit IRC | 13:10 | |
*** liuyulong has joined #openstack-nova | 13:16 | |
*** psachin has joined #openstack-nova | 13:19 | |
*** mdbooth has joined #openstack-nova | 13:20 | |
*** mriedem has joined #openstack-nova | 13:24 | |
*** BjoernT has joined #openstack-nova | 13:26 | |
mriedem | artom: https://bugs.launchpad.net/nova/+bug/1845146 this is a known issue right? | 13:27 |
openstack | Launchpad bug 1845146 in OpenStack Compute (nova) "NUMA aware live migration failed when vCPU pin set" [Undecided,New] | 13:27 |
*** gbarros has joined #openstack-nova | 13:30 | |
*** belmoreira has quit IRC | 13:30 | |
artom | mriedem, no, that's the thing that Ya was testing | 13:30 |
artom | mriedem, http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009608.html | 13:31 |
*** belmoreira has joined #openstack-nova | 13:31 | |
artom | I'm gonna look at his logs today, but initial instinct is there's a race somewhere | 13:31 |
*** belmoreira has quit IRC | 13:32 | |
*** nweinber_ has joined #openstack-nova | 13:32 | |
*** mkrai has quit IRC | 13:34 | |
*** dpawlik has quit IRC | 13:34 | |
*** ociuhandu has joined #openstack-nova | 13:36 | |
efried | dansmith: Please review this RC candidate with RPC in the title and one +2 from mriedem: https://review.opendev.org/#/c/683948/ | 13:36 |
*** ociuhandu has quit IRC | 13:36 | |
dansmith | ah cool, we're on to approving actual bugs during this bug phase? | 13:37 |
* dansmith looks | 13:37 | |
*** ociuhandu has joined #openstack-nova | 13:37 | |
*** ociuhandu has quit IRC | 13:40 | |
*** ociuhandu has joined #openstack-nova | 13:41 | |
*** lpetrut has quit IRC | 13:41 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Add reno about nova-api eventlet monkey-patching and rabbitmq https://review.opendev.org/662095 | 13:44 |
dansmith | mriedem: so on that gibi rpc version patch... | 13:45 |
dansmith | mriedem: we often gate features on service version, especially when they have API impacts, which is cool, and this is kinda like that | 13:45 |
dansmith | however, one could take a more RPC-centric world view and say that "if the RPC version allows it I should send it", meaning if they're not set =auto or =stein, they're asking us to send the newer thing | 13:46 |
dansmith | which could be useful if you're trying to get migrated off the last few computes, or something like that, | 13:46 |
dansmith | although it could cause problems for *other* calls that might be too new, and/or if you scheduled to an old one but don't have your rpc version pinned properly | 13:47 |
mriedem | you mean if upgrade_levels=train or =5.3? | 13:47 |
dansmith | yeah | 13:47 |
*** boxiang has quit IRC | 13:49 | |
mriedem | out of curiosity, does tripleo actually pin to specific versions or release names during an upgrade or just use 'auto'? | 13:49 |
*** boxiang has joined #openstack-nova | 13:49 | |
mriedem | because osa uses auto and doesn't allow you to override it https://github.com/openstack/openstack-ansible-os_nova/blob/3d99b2bd80287a3dd7cd35eb7075b80535f9ce42/templates/nova.conf.j2#L273 | 13:49 |
dansmith | I dunno, I both suspect they default to auto and also let you override it to something else | 13:49 |
dansmith | however,I think that you *can* do a gradual upgrade with tripleo, but it's not the default IIRC | 13:50 |
dansmith | so it likely doesn't matter as much to them | 13:50 |
dansmith | at least in the kinda default case | 13:50 |
dansmith | like, I think that if you're doing a piecemeal upgrade, you have to do a stack update each time you roll any compute forward, which touches everything, so it's not worth it, or something like that | 13:50 |
mriedem | do most tripleo (private cloud) deployments just do a full down maintenane upgrade rather than rolling then? | 13:52 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate some API sample tests off of nova-net https://review.opendev.org/684325 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Use nova-net explicitly in '_IntegratedTestBase' subclasses https://review.opendev.org/684326 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove unnecessary 'USE_NEUTRON = False' https://review.opendev.org/684327 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Use nova-net explicitly in 'ApiSampleTestBase' subclasses https://review.opendev.org/684328 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove yet more unnecessary 'USE_NEUTRON = False' https://review.opendev.org/684329 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Disable neutron explicitly for 'ServersSampleBase' subclasses https://review.opendev.org/684330 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove *even* more unnecessary 'USE_NEUTRON = False' https://review.opendev.org/684331 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_availability_zone' functional tests https://review.opendev.org/684332 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ip_pools' functional tests https://review.opendev.org/684333 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_quota_sets' functional tests https://review.opendev.org/684334 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_server_tags' functional tests https://review.opendev.org/684335 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_servers' functional tests https://review.opendev.org/684336 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hosts' functional tests https://review.opendev.org/684337 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_networks_associate' functional tests https://review.opendev.org/684338 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_rescue' functional tests https://review.opendev.org/684339 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hypervisors' functional tests https://review.opendev.org/684340 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_attach_interfaces' functional tests https://review.opendev.org/684341 | 13:53 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_simple_tenant_usage' functional tests https://review.opendev.org/684342 | 13:53 |
*** ociuhandu has quit IRC | 13:53 | |
mriedem | efried: it's normally not tracked as a blueprint, but one could argue that there should be a blueprint tracking a compute and compute task api (conductor) rpc api major version bump in ussuri since i think in the last few weeks we've identified a lot of cruft that needs to be cleaned out | 13:53 |
dansmith | mriedem: afaik, pretty much | 13:55 |
efried | mriedem: Why have a blueprint if there's not normally a blueprint? | 13:56 |
mriedem | efried: to track it and make it a priority | 13:56 |
mriedem | last time we had a major compute rpc api bump was queens | 13:56 |
*** mlavalle has joined #openstack-nova | 13:56 | |
mriedem | and never for the conductor compute task api | 13:57 |
mriedem | for example, there is cells v1 stuff in conductor that we can't remove without that | 13:57 |
mriedem | well, we can, but not in good conscience | 13:57 |
mriedem | anyway, just an idea | 13:58 |
*** BjoernT has quit IRC | 13:58 | |
mriedem | i'm not sure i trust myself to do that properly so i'm not necessarily signing up for the work either | 13:58 |
efried | I'm still stuck on the part where having a blueprint does anything to make something a priority. | 13:59 |
mriedem | blueprints are historically how we herd cats in nova since we're not using storyboard | 14:00 |
dansmith | so we can track the work against a milestone? | 14:00 |
mriedem | if there is another better way, sure | 14:00 |
mriedem | random etherpad o wishlist is an option, but those generally don't go well and aren't indexable | 14:00 |
mriedem | or a bug, "nova's rpc interfaces for compute related stuff are crusty" | 14:01 |
* stephenfin likes them just to have semi-official topic in Gerrit | 14:01 | |
efried | stephenfin: speaking of, do we have a bp for nova-net removal? | 14:03 |
stephenfin | I've just grabbed remove-nova-network | 14:03 |
stephenfin | ...which already exists. Damn you, Riedemann | 14:03 |
bauzas | efried: do you want to wait until tomorrow for +Wing https://review.opendev.org/#/c/683327/ ? | 14:04 |
efried | remove-nova-network-freal? | 14:04 |
bauzas | mriedem: thanks for the PS3 | 14:04 |
sean-k-mooney | related to that i was wondering if we should also move the neutron related code in nova/network to os-vif and load via the exsitsing driver mechanis | 14:04 |
efried | bauzas: I really just want more people to look at it | 14:04 |
stephenfin | remove-nova-network-redux ? | 14:04 |
bauzas | efried: ack | 14:05 |
bauzas | dansmith: stephenfin: https://review.opendev.org/#/c/683327/ if you want to look at the prelude | 14:05 |
sean-k-mooney | but thats just something im toying with. | 14:05 |
*** ociuhandu has joined #openstack-nova | 14:05 | |
efried | alex_xu, luyao: since it mentions vpmem | 14:05 |
efried | aspiers: since it mentions SEV | 14:05 |
efried | gmann: to make sure we don't need to call out anything specific about API updates | 14:05 |
efried | More cores. | 14:05 |
bauzas | melwitt: if you want to also look at the prelude https://review.opendev.org/#/c/683327/ (once you're there) | 14:05 |
mriedem | stephenfin: remove-nova-network-ussuri | 14:06 |
mriedem | is the pattern when we have a blueprint that spans releases | 14:06 |
mriedem | see the mox removal bp | 14:06 |
*** belmoreira has joined #openstack-nova | 14:06 | |
*** belmoreira has quit IRC | 14:06 | |
aspiers | efried: struggling to regain contact here, what mentions SEV? | 14:07 |
aspiers | s/contact/context/ | 14:07 |
mriedem | efried: for https://review.opendev.org/#/c/680300/ who reviewed the vpmem series besides you and alex_xu that can approve that? stephenfin? | 14:07 |
efried | yes | 14:07 |
mriedem | aspiers: both, highlights and release notes | 14:07 |
stephenfin | mriedem: how come we do that? purely so we don't have blueprints that span multiple releases? | 14:07 |
efried | aspiers: https://review.opendev.org/#/c/683327/ release prelude | 14:08 |
mriedem | stephenfin: how come we have a -<release> suffix pattern for bp naming as a convention? | 14:08 |
*** belmoreira has joined #openstack-nova | 14:08 | |
stephenfin | yup | 14:08 |
efried | I think highlights already merged and you looked at 'em. | 14:08 |
mriedem | stephenfin: because we sometimes have blueprints that...span releases | 14:08 |
mriedem | and a naming convention is nice | 14:08 |
sean-k-mooney | mriedem: we dont alway do that. we only do it if some of the blueprint was merged in the cycle right | 14:08 |
efried | I'm going to guess it's not documented anywhere, just grew out organically over time. | 14:08 |
gmann | efried: you mean in this - https://review.opendev.org/#/c/683327 | 14:08 |
stephenfin | yeah, but I'm saying we can't just have that blueprint span multiple cycles? | 14:09 |
efried | gmann: yes | 14:09 |
sean-k-mooney | if it was just defereed we keep the same blueprint | 14:09 |
aspiers | efried: IIRC I wrote that text | 14:09 |
mriedem | stephenfin: b/c we've marked some as partially complete when they've had some non-trivial amount of work merged | 14:09 |
mriedem | stephenfin: generally blueprints that are more mechanical in nature | 14:09 |
mriedem | house cleaning and such | 14:09 |
dansmith | stephenfin: launchpad doesn't allow a blueprint to be targeted twice | 14:09 |
efried | aspiers: cool, so just make sure you were quoted appropriately, and that the text is still appropriate for a reno prelude (as opposed to market-y cycle highlights) and we're good | 14:09 |
mriedem | not something like cross-cell-resize am i going to say "partially complete in train b/c i landed some data migrations" | 14:09 |
dansmith | stephenfin: so pointing it at a release lets us mark that some work was done there, for writing the highlights, and then more gets done in that the next cycle | 14:10 |
aspiers | efried: I guess it was copied there from some other review I was involved in? | 14:10 |
aspiers | ah yes, https://review.opendev.org/#/c/681943/ | 14:10 |
*** lennyb has quit IRC | 14:11 | |
stephenfin | dansmith: aha, fair | 14:12 |
gmann | efried: lgtm. 'API Improvements' section is enough for API updates. there is no specific API things need separate highlight. | 14:15 |
efried | thanks for the look gmann | 14:16 |
mriedem | aspiers: that's already noted in the commit message | 14:16 |
*** gbarros has quit IRC | 14:18 | |
*** BjoernT has joined #openstack-nova | 14:20 | |
*** ttsiouts has quit IRC | 14:21 | |
*** BjoernT_ has joined #openstack-nova | 14:22 | |
*** liuyulong has quit IRC | 14:22 | |
*** ttsiouts has joined #openstack-nova | 14:22 | |
*** ttsiouts has quit IRC | 14:22 | |
*** mrch_ has quit IRC | 14:22 | |
*** ttsiouts has joined #openstack-nova | 14:23 | |
*** lennyb has joined #openstack-nova | 14:23 | |
KeithMnemonic | good day mriedem. quick question on all of those patches related to https://bugs.launchpad.net/nova/+bug/1469179 it seems everything made it down to rocky other than https://review.opendev.org/#/c/551026/ is there any issue in me trying to also backport this to Rocky? | 14:24 |
openstack | Launchpad bug 1469179 in OpenStack Compute (nova) "instance.root_gb should be 0 for volume-backed instances" [Medium,Fix released] - Assigned to Dan Smith (danms) | 14:24 |
kashyap | sean-k-mooney: Hey, on that 'strict' vs. 'preferred' thing for NUMA allocation from the downstream bug - if we change to 'preferred', we'd be "deviating" from libvirt's default of 'strict' | 14:25 |
kashyap | sean-k-mooney: Not that it's some "blasphemy"; but I'd like to understand why libvirt defaults to it | 14:25 |
*** BjoernT has quit IRC | 14:25 | |
mriedem | KeithMnemonic: heh yes | 14:25 |
sean-k-mooney | it defualt to stict in numatune because if you said you want to tune the numa memory it makes sense to default to strict | 14:26 |
sean-k-mooney | given by default numa tune is not used and no tuning is used | 14:26 |
*** brault has joined #openstack-nova | 14:26 | |
sean-k-mooney | so you are already opting into affinty by generating the numatune elements | 14:26 |
KeithMnemonic | mriedem yes there is an issue? or yes i can try and backport it ? | 14:26 |
mriedem | KeithMnemonic: you basically asked if it's ok to backport something from train that removed something that was deprecated in stein, and has nothing to do with rocky | 14:27 |
mriedem | i know suse is super in love with rocky | 14:27 |
mriedem | but... | 14:27 |
kashyap | sean-k-mooney: I'm NUMA-unaware, so afraid, I still can't parse _why_ libvirt defaults to 'strict' | 14:28 |
kashyap | (Or weakly-aware) | 14:28 |
KeithMnemonic | oh i see, filters were deprecated in stein | 14:28 |
kashyap | sean-k-mooney: Please rephrase, if you can... | 14:28 |
KeithMnemonic | https://review.opendev.org/#/c/596502/ | 14:28 |
mriedem | correct | 14:28 |
sean-k-mooney | kashyap: by default you do not specify numatune elements. so by defualt libvirt does not enforce strict affinity | 14:28 |
KeithMnemonic | so at a minimum it would make sense to put this to stein as well if someone needed it? | 14:29 |
kashyap | sean-k-mooney: Aah, like that! | 14:29 |
mriedem | KeithMnemonic: put what? https://review.opendev.org/#/c/551026/ in stein? | 14:29 |
sean-k-mooney | kashyap: when you therefor add the numatune element it makes sense for it to default to strcit since you ar opting out of the default behavior of no tuneing | 14:29 |
sean-k-mooney | kashyap: does that make sense? | 14:30 |
mriedem | KeithMnemonic: https://review.opendev.org/#/c/551026/ depends on https://review.opendev.org/#/c/672065/ which you can't backport to stein | 14:30 |
kashyap | sean-k-mooney: Yes, that's clearer. Thank you | 14:30 |
KeithMnemonic | got it | 14:30 |
mriedem | you can't say "x is deprecated and also removed in the same release" | 14:30 |
mriedem | maybe suse can | 14:30 |
KeithMnemonic | thanks for cleating it up | 14:30 |
mriedem | but that's a pretty bad precedent | 14:30 |
KeithMnemonic | no we dont | 14:30 |
KeithMnemonic | just a customer complaining about the root issue https://bugs.launchpad.net/nova/+bug/1469179 and see what options there are | 14:31 |
openstack | Launchpad bug 1469179 in OpenStack Compute (nova) "instance.root_gb should be 0 for volume-backed instances" [Medium,Fix released] - Assigned to Dan Smith (danms) | 14:31 |
sean-k-mooney | KeithMnemonic: thy can change there flavor to have 0 root disk | 14:32 |
mriedem | KeithMnemonic: the main fix was in rocky https://review.opendev.org/#/c/580720/ | 14:32 |
KeithMnemonic | yup, that is the workaround they have in place | 14:32 |
mriedem | the DiskFilter should be disabled (since pike i think) | 14:33 |
sean-k-mooney | mriedem: yes it should | 14:33 |
KeithMnemonic | but some sales guy saw the launchpad bug and is asking why it was not fully fixed in rocky. and thanks to mriedem i can answer now ;-) | 14:33 |
mriedem | it is fixed in rocky | 14:34 |
sean-k-mooney | did we add the policy rule in rocky too? i think we did but its deisable by defualt? | 14:35 |
mriedem | yes https://docs.openstack.org/nova/rocky/configuration/policy.html | 14:35 |
mriedem | os_compute_api:servers:create:zero_disk_flavor | 14:35 |
*** TxGirlGeek has joined #openstack-nova | 14:37 | |
sean-k-mooney | so the rocky fix is 1.) disable ram|cpu|disk filters 2.) create bfv 0 disk flavors, 3.) chage the os_compute_api:servers:create:zero_disk_flavor rule to rule:admin_api | 14:39 |
mriedem | it depends on what you care about, | 14:40 |
mriedem | the immediate issue is failing scheduling b/c of the DiskFilter or claim (by having DiskFilter enabled) and booting a volume-backed server - that is fixed with dan's fix above and disabling the DiskFilter since placement won't filter on DISK_GB | 14:40 |
mriedem | the resource tracker should also not be counting volume-backed server usage against the compute node (hypervisors API) since rocky | 14:41 |
mriedem | so i'm not sure what is *not* fixed in rocky | 14:41 |
sean-k-mooney | right. the only thing i can think of is that we dont default to safe regarding os_compute_api:servers:create:zero_disk_flavor | 14:42 |
sean-k-mooney | but that is for backwards comaptablity in rocky | 14:42 |
*** awalende_ has quit IRC | 14:43 | |
*** belmoreira has quit IRC | 14:50 | |
*** belmoreira has joined #openstack-nova | 14:51 | |
*** tellesnobrega has quit IRC | 14:52 | |
openstackgerrit | Merged openstack/os-resource-classes master: Update bug link in docs to point to storyboard https://review.opendev.org/681464 | 14:56 |
*** abhishekk has quit IRC | 14:58 | |
*** eharney has quit IRC | 14:58 | |
*** hemna has quit IRC | 14:59 | |
*** ratailor has joined #openstack-nova | 15:00 | |
*** pcaruana has quit IRC | 15:01 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Move pre-3.44 Cinder post live migration test to test_compute_mgr https://review.opendev.org/683597 | 15:02 |
openstackgerrit | Merged openstack/os-resource-classes master: Build pdf docs https://review.opendev.org/681463 | 15:02 |
*** cfriesen has joined #openstack-nova | 15:03 | |
*** ratailor has quit IRC | 15:03 | |
*** ccamacho has quit IRC | 15:05 | |
*** gbarros has joined #openstack-nova | 15:08 | |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Use the "memory" mode 'preferred' for a guest NUMA node https://review.opendev.org/684375 | 15:13 |
kashyap | sean-k-mooney: ^ In your "copious spare time" | 15:14 |
kashyap | sean-k-mooney: Shouldn't we consider migration here: if an instance from a Compute node with 'strict' is being migrated to a host with 'preferred' - possible explosions? | 15:14 |
kashyap | sean-k-mooney: Err, that main commit summary is nonsense, let me fix that | 15:15 |
sean-k-mooney | yes this would need to be taken into account | 15:15 |
sean-k-mooney | but if we hardcore preferred then it not an issue excapt on upgrde | 15:15 |
sean-k-mooney | it would be fixed by a hard reboot | 15:15 |
sean-k-mooney | i would not nessisarly require this to be handeled in live migration given that numa live migration was not really correct until train | 15:16 |
*** brault has quit IRC | 15:18 | |
*** brault has joined #openstack-nova | 15:19 | |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Use the `preferred` memory allocation mode for NUMA https://review.opendev.org/684375 | 15:19 |
aspiers | kashyap: "pfalsh" typo in https://libvirt.org/formatdomaincaps.html#elementsOSBIOS | 15:19 |
*** brault has quit IRC | 15:19 | |
kashyap | aspiers: Ah, you saw the libvirt patch I Cced you on? | 15:19 |
aspiers | yes | 15:19 |
kashyap | aspiers: Will fix; it's merged already :D | 15:20 |
kashyap | Thanks for spotting; feel free to reply there | 15:20 |
aspiers | kashyap: So I saw. Thanks - don't think I'm subscribed to that list | 15:20 |
kashyap | sean-k-mooney: Ah-ha; good point on "NUMA + live migration not really being a 'thing' until Train" | 15:20 |
aspiers | kashyap: While you're at it, you could add some helpful hyperlinks | 15:20 |
sean-k-mooney | kashyap: well it was a thing but it had a load of edgecases | 15:21 |
kashyap | aspiers: You don't need to be subscribed; just reply they moderators will (usually, quickly) take care | 15:21 |
sean-k-mooney | so i dont really considre this edgecase to be important | 15:21 |
sean-k-mooney | at least not relitive to the other ones that were not handeled | 15:21 |
kashyap | sean-k-mooney: So, an upgrade note is required, too, along with the 'fixes' note I added -- isn't it? | 15:21 |
aspiers | kashyap: I'd rather not get involved - too much other stuff to deal with :) | 15:21 |
kashyap | aspiers: Mind pointing out which ones? | 15:21 |
kashyap | aspiers: Heh, I hear ya | 15:22 |
aspiers | kashyap: Here are some suggested edits | 15:22 |
aspiers | "For the loader element, the following can occur:" | 15:22 |
aspiers | "the following elements can occur:" | 15:22 |
kashyap | (Oh, I wanted to fix that, too! Realized it too late) | 15:22 |
aspiers | but then you list "value" alongside "type" etc. which is incorrect | 15:22 |
aspiers | type/readonly/secure are values of the name attribute of <enum> | 15:22 |
sean-k-mooney | we could update the numa migration code in train to also update it to prefered | 15:22 |
aspiers | but you don't even mention <enum> | 15:22 |
sean-k-mooney | but i would do that as a seperate patch | 15:23 |
*** damien_r has quit IRC | 15:23 | |
aspiers | and the confusion over the point of <enum> was the whole reason we started looking at this in the first place | 15:23 |
aspiers | kashyap: ^^^ | 15:24 |
kashyap | aspiers: Okay, my brain is completely exhausted for the moment; will make a note to get back to this and submit a patch | 15:25 |
kashyap | Thanks! | 15:25 |
aspiers | kashyap: "For cases like that, os element exposes ..." is ambiguous about whether it's <os> in domCaps or caps | 15:25 |
aspiers | which again was the crux of the confusion | 15:25 |
aspiers | These docs need to be crystal clear about when they are referring to elements of domCaps vs. caps | 15:25 |
aspiers | so that the parallel between the two is understood | 15:25 |
*** lennyb has quit IRC | 15:25 | |
kashyap | aspiers: I didn't touch those bits; but you're right, they _do_ need "less careless" documentation. I'll get back to this tomm / this week | 15:26 |
aspiers | kashyap: thanks! one more pointer | 15:26 |
kashyap | Non-null? | 15:26 |
kashyap | :D | 15:26 |
aspiers | kashyap: I'm suggesting to hyperlink text like "the <loader/> element in the domain XML" | 15:27 |
aspiers | should link to https://libvirt.org/formatdomain.html#elementLoader | 15:27 |
aspiers | in both directions, ideally | 15:27 |
*** BjoernT_ is now known as BjoernT | 15:27 | |
aspiers | kashyap: feel free to send me something to review | 15:27 |
aspiers | when you have a chance | 15:27 |
kashyap | aspiers: Will definitely do. | 15:29 |
aspiers | thanks again! | 15:29 |
*** lennyb has joined #openstack-nova | 15:30 | |
*** ivve has quit IRC | 15:30 | |
kashyap | You're welcome again | 15:30 |
*** belmoreira has quit IRC | 15:32 | |
kashyap | sean-k-mooney: Yeah, indeed. | 15:33 |
kashyap | Sean, sorry, disregard that; was replying to a stale message, as had the IRC buffer scrolled up. | 15:34 |
*** maciejjozefczyk has quit IRC | 15:34 | |
*** hemna has joined #openstack-nova | 15:34 | |
*** gyee has joined #openstack-nova | 15:44 | |
*** shilpasd has joined #openstack-nova | 15:45 | |
mlavalle | gmann: hi, I have some questions regarding http://specs.openstack.org/openstack/nova-specs/specs/train/approved/policy-default-refresh.html. when you have some time, please ping me | 15:48 |
*** ttsiouts has quit IRC | 15:49 | |
*** panda is now known as panda|bbl | 16:02 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Add missing parameter https://review.opendev.org/682886 | 16:02 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Use compression by default for 'SshDriver' https://review.opendev.org/684393 | 16:02 |
stephenfin | mriedem: Seeing as we're on a docs roll, could you take a look at https://review.opendev.org/#/c/671298/, please? Maybe efried or gibi too | 16:03 |
*** macz has joined #openstack-nova | 16:04 | |
gibi | stephenfin: done | 16:05 |
mriedem | hmm | 16:05 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document global options for nova-manage https://review.opendev.org/677443 | 16:07 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: config: Explicitly register 'remote_debug' CLI opts https://review.opendev.org/677444 | 16:07 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: docs: Rewrite nova-manage docs to use proper directives https://review.opendev.org/677509 | 16:07 |
*** rpittau is now known as rpittau|afk | 16:08 | |
*** udesale has quit IRC | 16:10 | |
shilpasd | mriedem: hi, to verify performance for changes https://review.opendev.org/#/c/612626/ Ignore root_gb for BFV in simple tenant usage API, i have written test program http://paste.openstack.org/show/779196/, please go through once and share any suggestions/corrections if any | 16:12 |
mriedem | shilpasd: you should paste that into the patch and/or the bug report | 16:12 |
shilpasd | ok, will do that | 16:13 |
shilpasd | mriedem: thanks, will paste along with test results | 16:13 |
dansmith | stephenfin: comment on that db sync one | 16:15 |
dansmith | stephenfin: If you just did what you said in the commit message, I'd be good, but I feel like you're regressing the verbiage in there | 16:15 |
*** dtantsur is now known as dtantsur|afk | 16:15 | |
dansmith | maybe mriedem has an opinion | 16:15 |
dansmith | I mean, I bet he does, but maybe he'll tell us what it is | 16:16 |
stephenfin | makes sense. I can respin | 16:17 |
mriedem | dansmith: i just said the same thing :) | 16:17 |
mriedem | and said "maybe dan has an opinion on this" | 16:17 |
dansmith | heh nice | 16:17 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Note use of 'nova-manage db sync --config-file' https://review.opendev.org/671298 | 16:18 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Correct 'nova-manage db sync' documentation https://review.opendev.org/677508 | 16:18 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document global options for nova-manage https://review.opendev.org/677443 | 16:18 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: config: Explicitly register 'remote_debug' CLI opts https://review.opendev.org/677444 | 16:18 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: docs: Rewrite nova-manage docs to use proper directives https://review.opendev.org/677509 | 16:18 |
*** priteau has quit IRC | 16:19 | |
*** maciejjozefczyk has joined #openstack-nova | 16:20 | |
*** brault has joined #openstack-nova | 16:20 | |
stephenfin | dansmith, mriedem: okay, took what dansmith said and made it a thing ^ | 16:22 |
dansmith | looks okay to me, but mriedem was more specific about details, so I'll let him ack it first | 16:22 |
*** maciejjozefczyk has quit IRC | 16:24 | |
*** brault has quit IRC | 16:27 | |
mriedem | you and gibi can approve | 16:27 |
*** brault has joined #openstack-nova | 16:28 | |
*** brault has quit IRC | 16:28 | |
*** brault has joined #openstack-nova | 16:28 | |
*** jaosorior has joined #openstack-nova | 16:29 | |
*** brault has quit IRC | 16:29 | |
*** brault has joined #openstack-nova | 16:32 | |
*** markvoelker has quit IRC | 16:40 | |
*** tesseract has quit IRC | 16:46 | |
ozzzo | after rebuilding controllers in my dev cluster, is is possible to re-register the existing HV, or do I need to rebuild them? | 16:50 |
*** jaosorior has quit IRC | 16:51 | |
openstackgerrit | Merged openstack/nova master: objects: use all_things_equal from objects.base https://review.opendev.org/681397 | 16:51 |
openstackgerrit | Merged openstack/nova master: Func test for migrate reschedule with pinned compute rpc https://review.opendev.org/683385 | 16:51 |
openstackgerrit | Merged openstack/nova master: Add evacuate vs rebuild contributor doc https://review.opendev.org/683245 | 16:51 |
artom | mriedem, so, I think I figured out the NUMA LM thing | 16:51 |
artom | We filter out 'accepted' migrations in https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L4422 | 16:52 |
artom | (That's called from the update resources periodic task | 16:53 |
artom | ) | 16:53 |
artom | But we only set the 'preparing' status here: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L7020 | 16:54 |
artom | (It's created with 'accepted' by the conductor) | 16:54 |
mriedem | and we set accepted in the live migration task | 16:54 |
artom | So there's a window between doing the claim and setting the status when the resource tracker periodic can kick in and miss the migration | 16:54 |
artom | And free the incoming instance's resources because it's still on the source according to the DB | 16:55 |
mriedem | compared to the cold migrate / resize of https://www.health.harvard.edu/blog/common-anticholinergic-drugs-like-benadryl-linked-increased-dementia-risk-201501287667 | 16:55 |
mriedem | oops | 16:55 |
mriedem | ha | 16:55 |
artom | Dementia indeed | 16:55 |
mriedem | 'pre-migrating' | 16:55 |
artom | I guess now we know why pre-migrating exists :) | 16:55 |
mriedem | https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L4408-L4412 | 16:55 |
artom | Yep, saw that | 16:56 |
sean-k-mooney | pre-migration is the default status no? | 16:56 |
sean-k-mooney | at least for cold migraitons | 16:56 |
*** panda|bbl is now known as panda | 16:56 | |
artom | sean-k-mooney, it's one giant mess | 16:56 |
mriedem | artom: ok so i guess just remove 'accepted' from that db api query and see if ya.wang can reproduce with his test script right? | 16:56 |
artom | mriedem, as an initial smoke test, yeah, but I'm not sure that's the correct permanent solution | 16:57 |
mriedem | why? if you haven't realized yet, the migration status stuff and the rt is all very tightly coupled | 16:57 |
artom | Well... | 16:57 |
sean-k-mooney | mriedem: i like that comment :) not the reason why its needed but i like the phrasing | 16:57 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rework the PCI passthrough guides https://review.opendev.org/635243 | 16:58 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Remove 'adv-config', 'system-admin' subdocs https://review.opendev.org/684402 | 16:58 |
artom | The migration status is technically part of our API, no? | 16:58 |
artom | And... even it isn't - as you said, it's all tightly coupled, I'd be afraid to break something else | 16:58 |
artom | Ideally we would implement your TODO | 16:58 |
artom | Though I suppose one doens't hurt the other | 16:59 |
sean-k-mooney | if we removed accepted we would need to add it to migration_get_in_progress_by_instance right | 16:59 |
mriedem | artom: "accepted" is only set while a live migration is in-progress | 16:59 |
mriedem | and that api is for getting in-progress migrations | 16:59 |
mriedem | the only reason it didn't matter before was because the RT filtered out anything that wasn't evacuate/resize/cold migrate | 16:59 |
*** markvoelker has joined #openstack-nova | 16:59 | |
artom | mriedem, that's a good point | 16:59 |
* artom h4x | 17:00 | |
mriedem | artom: specifically this v | 17:00 |
mriedem | https://review.opendev.org/#/c/634606/86/nova/compute/resource_tracker.py | 17:00 |
mriedem | _is_trackable_migration | 17:00 |
*** KeithMnemonic has quit IRC | 17:00 | |
*** KeithMnemonic has joined #openstack-nova | 17:00 | |
mriedem | so accepted is definitely an in-progress live migration and something we track now | 17:00 |
artom | Yeah, I remember that | 17:00 |
artom | Yep, fair enough | 17:00 |
artom | If we removed that method entirely we don't need to filter out accepted anymore | 17:01 |
sean-k-mooney | should we be adding accepted here too https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L4438 | 17:01 |
sean-k-mooney | it would be kind of nice if we had 1 list | 17:01 |
* artom is a proponent of minimal blast radius | 17:01 | |
sean-k-mooney | that was shared between those two | 17:01 |
mriedem | sean-k-mooney: idk, pre-migrating isn't in that list either | 17:02 |
sean-k-mooney | its not clear that pre-migration means its in progress | 17:03 |
ozzzo | is this not the correct channel to ask for help with Nova? | 17:03 |
artom | sean-k-mooney, so, I tracked it up the rabbit hole, that one's only used in the server migrations API | 17:03 |
sean-k-mooney | you can end up in pre-migration if you do an evac to the same host | 17:03 |
*** ricolin has quit IRC | 17:03 | |
mriedem | a lot of that crap was added for the /servers/{server_id}/migration apis added awhile back so you could list in-progress live migrations specifically and cancel/force-complete them | 17:03 |
mriedem | so i wouldn't be surprised if a lot of this is biased to live migratoin only | 17:03 |
mriedem | artom: yup ^ | 17:04 |
*** macz has quit IRC | 17:04 | |
artom | ozzzo, if you're looking for help running/deploying Nova, #openstack in the place | 17:04 |
artom | ozzzo, this channel is for development (see /topic) | 17:04 |
ozzzo | no, nobody ever answers a question in #openstack | 17:05 |
ozzzo | I tried that a few weeks ago | 17:05 |
ozzzo | I've been idling in there and I see noobs come in asking questions every day, and nobody ever answers | 17:05 |
*** tbachman has quit IRC | 17:05 | |
sean-k-mooney | ozzzo: if we are not too busy we sometime help if its not a specifc deployment issue | 17:05 |
ozzzo | they don't even recommend another channel, just silence | 17:05 |
sean-k-mooney | ozzzo: what is the question? | 17:05 |
mriedem | ozzzo: sounds like an opportunity for you to be the change in that channel :) | 17:06 |
ozzzo | after rebuilding controllers in my dev cluster, is is possible to re-register the existing HV, or do I need to rebuild them? | 17:06 |
dansmith | ozzzo: see topic | 17:06 |
sean-k-mooney | if you start and stop the compute agents it shoudl recreate the compute service recorred but it wont rebuild any instance info | 17:07 |
sean-k-mooney | for a dev environment its normally simpler to redeploy as its only a hand full of nodes | 17:07 |
sean-k-mooney | ozzzo: if you are using devstack you can often just do a git pull and restart the service instead of unstacking and stacking again | 17:08 |
ozzzo | we | 17:08 |
ozzzo | re using ansible-openstack | 17:08 |
ozzzo | I tried stop/starting services, and even rebooting, but they don't re-register | 17:08 |
dansmith | ozzzo: maybe you and sean-k-mooney can go to #openstack to discuss further | 17:08 |
sean-k-mooney | i was going to suggest the openstack ansibel channel | 17:09 |
dansmith | or that | 17:09 |
ozzzo | Failed to create resource provider <hostname> | 17:09 |
ozzzo | ok I'll try asking there, thank you for the advice | 17:09 |
kashyap | Occasionally having a user question discussed here is not really some blasphemy. | 17:09 |
sean-k-mooney | ya i think i know what the issue. is. the compute service uuid has change but you are using the same hostnames | 17:09 |
sean-k-mooney | so the placmet resouce provide exists with the hostname and old uuid | 17:10 |
sean-k-mooney | and when nova tries to create it again it gets that failure | 17:10 |
sean-k-mooney | thats a known failure mode. you need to just delete the old resouce providers | 17:11 |
*** boxiang has quit IRC | 17:11 | |
dansmith | sean-k-mooney: he just asked in #openstack | 17:11 |
dansmith | sean-k-mooney: maybe you could answer him there | 17:11 |
dansmith | you know, generate some traffic over there, help wake people up | 17:11 |
*** boxiang has joined #openstack-nova | 17:11 | |
sean-k-mooney | ok ya im not there but i can | 17:11 |
sean-k-mooney | also im on PTO for the next 3 days ill only be online for another 40 mins or so | 17:12 |
sean-k-mooney | am is there anything i should look at before i finish up today? | 17:12 |
ozzzo | where can I see the old resources; are they openstack services? | 17:13 |
*** ociuhandu has quit IRC | 17:15 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Make nova.compute.rpcapi.ComputeAPI.router a singleton https://review.opendev.org/684405 | 17:20 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Func test for migrate reschedule with pinned compute rpc https://review.opendev.org/684406 | 17:20 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Handle legacy request spec dict in ComputeTaskManager._cold_migrate https://review.opendev.org/684407 | 17:20 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional tests for [cinder]/cross_az_attach=False https://review.opendev.org/684141 | 17:23 |
*** brault has quit IRC | 17:31 | |
*** shilpasd has quit IRC | 17:38 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Stop filtering out 'accepted' for in-progress migrations https://review.opendev.org/684409 | 17:39 |
*** ralonsoh has quit IRC | 17:42 | |
*** igordc has joined #openstack-nova | 17:48 | |
*** psachin has quit IRC | 17:59 | |
*** cdent has quit IRC | 18:08 | |
KeithMnemonic | mriedem,sean-k-mooney: What was not fixed is perception. The 3 items needed are not something we do by default in our release | 18:14 |
KeithMnemonic | so for the customer since they do they after install, it was their view it was not fixed | 18:14 |
mriedem | ack | 18:15 |
*** belmoreira has joined #openstack-nova | 18:17 | |
*** ociuhandu has joined #openstack-nova | 18:18 | |
KeithMnemonic | and ideally I think their "pie in the sky" is for the amount of storage/usage on their backend to be used in anything that shows consumption or checks free space | 18:20 |
*** ociuhandu has quit IRC | 18:23 | |
openstackgerrit | Merged openstack/nova master: Reject migration with QoS port from conductor if RPC pinned https://review.opendev.org/683948 | 18:24 |
mriedem | KeithMnemonic: let me guess, they are using ceph? | 18:24 |
mriedem | and confused about nova saying they have $num_computes x total ceph pool size of storage? | 18:25 |
mriedem | KeithMnemonic: or you mean to use cinder for all storage rather than local disk on the compute hosts, e.g. volume-backed everywhere? | 18:25 |
mriedem | the latter has details in https://review.opendev.org/#/c/511965/ | 18:26 |
mriedem | the former https://review.opendev.org/#/c/650188/ | 18:26 |
KeithMnemonic | second, they needed large root volumes, so until they applied the 3 steps, they did not understand why a backend with 50 TB was complaining about no space ;-) | 18:26 |
mriedem | tbc i think the only thing you really need in rocky to properly track disk usage for volume-backed servers in nova is to disable the DiskFilter | 18:27 |
mriedem | that removes the root disk filtering and claims during scheduling/build and what the resource tracker reports out of the hypervisors API | 18:28 |
mriedem | the zero_disk_flavor policy thing is more tied to a security issue than anything | 18:28 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle legacy request spec dict in ComputeTaskManager._cold_migrate https://review.opendev.org/680762 | 18:32 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Isolate request spec handling from _cold_migrate https://review.opendev.org/680763 | 18:32 |
mriedem | gibi: stephenfin: ^ was approved but needed to be rebased if one of you can re-approve | 18:32 |
mriedem | ah crap i missed something gibi changed, sec | 18:33 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle legacy request spec dict in ComputeTaskManager._cold_migrate https://review.opendev.org/680762 | 18:37 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Isolate request spec handling from _cold_migrate https://review.opendev.org/680763 | 18:37 |
*** luksky has joined #openstack-nova | 18:43 | |
*** mriedem has quit IRC | 18:48 | |
*** mriedem has joined #openstack-nova | 18:49 | |
*** bbowen has quit IRC | 18:54 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Make nova-next multinode and drop tempest-slow-py3 https://review.opendev.org/683988 | 18:59 |
*** mrch_ has joined #openstack-nova | 19:07 | |
*** tbachman has joined #openstack-nova | 19:29 | |
openstackgerrit | Akihiro Motoki proposed openstack/nova master: doc: Improve PDF document structure https://review.opendev.org/682746 | 19:41 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add heal_instance_mappings command https://review.opendev.org/655908 | 19:50 |
*** bbowen has joined #openstack-nova | 19:54 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova stable/pike: WIP: Avoid redundant initialize_connection on source post live migration https://review.opendev.org/683008 | 19:54 |
*** nweinber_ has quit IRC | 19:56 | |
mriedem | ♪ jesus just left chicago ♪ | 19:59 |
dansmith | I approve. | 20:01 |
openstackgerrit | François Palin proposed openstack/nova master: Add retry when attachment_delete or terminate_connection fails https://review.opendev.org/669674 | 20:22 |
*** belmoreira has quit IRC | 20:27 | |
*** tbachman has quit IRC | 20:31 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add heal_instance_mappings command https://review.opendev.org/655908 | 20:34 |
*** tbachman has joined #openstack-nova | 20:43 | |
*** tbachman has quit IRC | 21:01 | |
*** xek has quit IRC | 21:11 | |
*** artom has quit IRC | 21:17 | |
*** artom has joined #openstack-nova | 21:18 | |
*** tellesnobrega has joined #openstack-nova | 21:25 | |
*** luksky has quit IRC | 21:33 | |
*** tellesnobrega has quit IRC | 21:48 | |
*** markvoelker has quit IRC | 21:52 | |
*** slaweq has quit IRC | 22:00 | |
*** rcernin has joined #openstack-nova | 22:07 | |
*** tellesnobrega has joined #openstack-nova | 22:15 | |
*** tbachman has joined #openstack-nova | 22:20 | |
*** tellesnobrega has quit IRC | 22:22 | |
*** tellesnobrega has joined #openstack-nova | 22:24 | |
*** aloga has quit IRC | 22:27 | |
*** aloga has joined #openstack-nova | 22:34 | |
*** dtantsur|afk has quit IRC | 22:51 | |
*** tkajinam has joined #openstack-nova | 22:51 | |
*** BjoernT has quit IRC | 23:15 | |
openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Merge provider configs to provider tree https://review.opendev.org/676522 | 23:26 |
*** gbarros has quit IRC | 23:31 | |
*** mlavalle has quit IRC | 23:33 | |
*** BjoernT has joined #openstack-nova | 23:45 | |
*** BjoernT_ has joined #openstack-nova | 23:49 | |
*** BjoernT has quit IRC | 23:50 | |
*** gbarros has joined #openstack-nova | 23:52 | |
*** markvoelker has joined #openstack-nova | 23:54 | |
*** markvoelker has quit IRC | 23:58 | |
*** gbarros_ has joined #openstack-nova | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!