*** Liang__ has joined #openstack-nova | 00:04 | |
*** Liang__ has quit IRC | 00:23 | |
*** ivve has quit IRC | 00:33 | |
*** brinzhang has joined #openstack-nova | 01:08 | |
*** ociuhandu has joined #openstack-nova | 01:31 | |
*** chenhaw has joined #openstack-nova | 01:35 | |
*** ociuhandu has quit IRC | 01:41 | |
*** macz has joined #openstack-nova | 02:05 | |
*** alex_xu has joined #openstack-nova | 02:11 | |
*** zhanglong has joined #openstack-nova | 02:24 | |
*** ociuhandu has joined #openstack-nova | 02:26 | |
*** ociuhandu has quit IRC | 02:30 | |
*** brinzhang_ has joined #openstack-nova | 02:47 | |
*** brinzhang has quit IRC | 02:50 | |
*** macz has quit IRC | 02:51 | |
*** ociuhandu has joined #openstack-nova | 03:00 | |
*** ociuhandu has quit IRC | 03:07 | |
*** artom has quit IRC | 03:33 | |
*** artom has joined #openstack-nova | 03:34 | |
*** mkrai has joined #openstack-nova | 03:34 | |
*** mkrai_ has joined #openstack-nova | 03:40 | |
*** mkrai has quit IRC | 03:41 | |
*** udesale has joined #openstack-nova | 04:02 | |
*** davee__ has joined #openstack-nova | 04:03 | |
*** davee_ has quit IRC | 04:04 | |
*** bhagyashris has joined #openstack-nova | 04:20 | |
*** ociuhandu has joined #openstack-nova | 04:27 | |
*** ociuhandu has quit IRC | 04:29 | |
*** ociuhandu has joined #openstack-nova | 04:30 | |
*** ociuhandu has quit IRC | 04:36 | |
*** bhagyashris has quit IRC | 04:45 | |
*** abhishekk has joined #openstack-nova | 04:51 | |
*** ociuhandu has joined #openstack-nova | 04:53 | |
*** ociuhandu has quit IRC | 04:58 | |
*** ociuhandu has joined #openstack-nova | 04:58 | |
*** mkrai_ has quit IRC | 04:59 | |
*** bhagyashris has joined #openstack-nova | 05:03 | |
*** ociuhandu has quit IRC | 05:03 | |
*** mkrai has joined #openstack-nova | 05:11 | |
*** mkrai has quit IRC | 05:14 | |
*** mkrai has joined #openstack-nova | 05:14 | |
*** mkrai_ has joined #openstack-nova | 05:22 | |
*** ratailor has joined #openstack-nova | 05:23 | |
*** links has joined #openstack-nova | 05:24 | |
*** mkrai has quit IRC | 05:24 | |
*** bhagyashris has quit IRC | 05:27 | |
*** ileixe has quit IRC | 05:29 | |
*** ociuhandu has joined #openstack-nova | 05:30 | |
*** bhagyashris has joined #openstack-nova | 05:31 | |
*** ileixe has joined #openstack-nova | 05:32 | |
*** ociuhandu has quit IRC | 05:35 | |
*** ircuser-1 has quit IRC | 05:35 | |
*** cp has joined #openstack-nova | 05:36 | |
*** slaweq has joined #openstack-nova | 05:48 | |
*** huaqiang has quit IRC | 05:49 | |
*** brinzhang has joined #openstack-nova | 05:54 | |
*** slaweq has quit IRC | 05:57 | |
*** brinzhang_ has quit IRC | 05:58 | |
*** abaindur has joined #openstack-nova | 05:58 | |
*** abaindur has quit IRC | 06:05 | |
*** Luzi has joined #openstack-nova | 06:11 | |
*** awalende has joined #openstack-nova | 06:16 | |
*** awalende has quit IRC | 06:20 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717 | 06:21 |
---|---|---|
*** bhagyashris has quit IRC | 06:45 | |
*** ileixe has quit IRC | 06:51 | |
*** ileixe has joined #openstack-nova | 06:52 | |
*** bhagyashris has joined #openstack-nova | 06:52 | |
*** abaindur has joined #openstack-nova | 07:01 | |
*** ociuhandu has joined #openstack-nova | 07:02 | |
*** rcernin has quit IRC | 07:04 | |
*** ociuhandu has quit IRC | 07:06 | |
*** abaindur has quit IRC | 07:07 | |
*** rpittau|afk is now known as rpittau | 07:28 | |
*** dpawlik has joined #openstack-nova | 07:34 | |
*** maciejjozefczyk has joined #openstack-nova | 07:36 | |
*** slaweq has joined #openstack-nova | 07:38 | |
*** damien_r has joined #openstack-nova | 07:57 | |
*** igordc has joined #openstack-nova | 07:58 | |
*** tkajinam has quit IRC | 08:02 | |
*** igordc has quit IRC | 08:02 | |
*** mkrai_ has quit IRC | 08:06 | |
*** luksky has joined #openstack-nova | 08:08 | |
*** tesseract has joined #openstack-nova | 08:15 | |
*** ociuhandu has joined #openstack-nova | 08:22 | |
*** ociuhandu has quit IRC | 08:29 | |
bauzas | good morning Nova | 08:36 |
*** ratailor_ has joined #openstack-nova | 08:40 | |
*** mkrai has joined #openstack-nova | 08:42 | |
*** ratailor has quit IRC | 08:42 | |
*** ivve has joined #openstack-nova | 08:46 | |
gibi | bauzas: good morning | 08:47 |
bauzas | :) | 08:47 |
bauzas | ta, you too | 08:47 |
*** jaosorior has joined #openstack-nova | 08:48 | |
*** ralonsoh has joined #openstack-nova | 08:49 | |
*** mkrai has quit IRC | 08:50 | |
*** mkrai has joined #openstack-nova | 08:51 | |
*** xek has joined #openstack-nova | 08:59 | |
*** ociuhandu has joined #openstack-nova | 09:04 | |
*** martinkennelly has joined #openstack-nova | 09:05 | |
*** nanzha has joined #openstack-nova | 09:10 | |
*** awalende has joined #openstack-nova | 09:16 | |
*** awalende has quit IRC | 09:18 | |
*** awalende has joined #openstack-nova | 09:26 | |
brinzhang | good morning :) | 09:29 |
*** awalende has quit IRC | 09:30 | |
*** awalende has joined #openstack-nova | 09:36 | |
*** awalende has quit IRC | 09:37 | |
*** ileixe has quit IRC | 09:40 | |
*** ratailor__ has joined #openstack-nova | 09:41 | |
*** ileixe has joined #openstack-nova | 09:41 | |
*** mkrai has quit IRC | 09:42 | |
* kashyap goes to update the libvirt distro support matrix ... and then version bump and clean-up ... | 09:42 | |
*** ratailor_ has quit IRC | 09:43 | |
bauzas | brinzhang: heh, good morning ! | 09:49 |
*** brinzhang_ has joined #openstack-nova | 09:52 | |
*** mkrai has joined #openstack-nova | 09:54 | |
*** brinzhang has quit IRC | 09:55 | |
brinzhang_ | bauzas: gibi: I was updated the flavor explosion spec after PTG, and base gibi's comments updated again, please review, Thanks :P | 09:56 |
*** mkrai has quit IRC | 09:56 | |
*** ociuhandu has quit IRC | 09:57 | |
*** mkrai has joined #openstack-nova | 09:57 | |
brinzhang_ | bauzas: gibi: spec https://review.opendev.org/#/c/663563/ | 09:57 |
bauzas | cool, I'll try | 09:57 |
*** ociuhandu has joined #openstack-nova | 09:58 | |
*** ociuhandu has quit IRC | 10:03 | |
*** kaisers has quit IRC | 10:04 | |
*** lennyb has quit IRC | 10:06 | |
*** zhanglong has quit IRC | 10:06 | |
*** mkrai has quit IRC | 10:07 | |
*** bhagyashris has quit IRC | 10:10 | |
*** ileixe has quit IRC | 10:10 | |
*** pcaruana has joined #openstack-nova | 10:10 | |
*** nanzha has quit IRC | 10:11 | |
*** ileixe has joined #openstack-nova | 10:11 | |
*** nanzha has joined #openstack-nova | 10:12 | |
*** owalsh has quit IRC | 10:12 | |
*** abhishekk has quit IRC | 10:12 | |
*** bhagyashris has joined #openstack-nova | 10:12 | |
*** brinzhang_ has quit IRC | 10:13 | |
gibi | ack | 10:13 |
kashyap | aspiers: If you're about, want to update the SLES versions here? -- https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix#Distro_minimum_versions | 10:16 |
*** kaisers has joined #openstack-nova | 10:20 | |
*** owalsh has joined #openstack-nova | 10:22 | |
openstackgerrit | pangliye proposed openstack/nova master: Fix misspell word https://review.opendev.org/694741 | 10:29 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop testing Python 2 https://review.opendev.org/687954 | 10:32 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Make functional job inherit from openstack parents https://review.opendev.org/688425 | 10:32 |
kashyap | Anyone familiar with Debian here: why was "sid" (the 'unstable' version) chcked here? -- https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix | 10:33 |
kashyap | zigo: ^ | 10:34 |
bauzas | kashyap: ask zigo | 10:34 |
bauzas | heh | 10:34 |
kashyap | :-) | 10:36 |
kashyap | "sid" is the unstable edition; normally I'd check the "stable" version to be there | 10:36 |
kashyap | Does Infra gate uses Debian "sid"? Maybe it's for #openstack-infra | 10:36 |
zigo | kashyap: This looks wrong in many ways, for example, the qemu version ... | 10:37 |
kashyap | zigo: Yeah! I'm right now undoing the mess, whoever did that | 10:37 |
*** CeeMac has joined #openstack-nova | 10:37 | |
kashyap | zigo: So, to be clear, these are the versions from 'stable': | 10:37 |
kashyap | - libvirt: 5.0.0-4 | 10:37 |
zigo | kashyap: In buster, we have qemu 1:3.1+dfsg-8+deb10u3 | 10:38 |
zigo | (ie: 3.1) | 10:38 |
kashyap | - QEMU: 3.1 | 10:38 |
kashyap | zigo: Indeed | 10:38 |
zigo | libvirt 5.0.0 | 10:38 |
kashyap | zigo: BTW, the version you see there for QEMU was checked on 2018-APR-05 -- so at _that_ time Buster indeed had 2.11 QEMU | 10:39 |
zigo | kashyap: I believe it's just outdated ... | 10:39 |
kashyap | So that's correct based on what was there at that time | 10:39 |
zigo | Right. | 10:39 |
kashyap | Yes, it is outdated. Updating these MediaWiki pages are a *pain* in the neck | 10:39 |
zigo | Yeah, the syntax is ... hum... suboptimal, to use nice words! :) | 10:40 |
kashyap | zigo: Refresh the page, and check if the Buster ("stable") looks correct now | 10:41 |
zigo | kashyap: Well, it looks ok to me, though instead of Buster ("sid") you may want to write Bullseye ("sid") and write what's there. | 10:42 |
kashyap | zigo: Right, let me do that | 10:43 |
kashyap | I'll update the Buster ("sid") --> Bullseye ("sid") | 10:44 |
kashyap | zigo: Am I correct in saying that what's at top-left (the 4.1.1) indicates the Bullseye ("sid") version? -- https://tracker.debian.org/pkg/qemu | 10:45 |
kashyap | Also what does the "1:" mean? | 10:45 |
zigo | kashyap: That's an "EPOCH".... basically, this is for when we do a mistake with the upstream version, and need to "upgrade" to a lower version. | 10:46 |
zigo | So, that's distro specific ... | 10:46 |
kashyap | Ah, right. The same concept exists in Fedora. | 10:46 |
zigo | 1:2.3.4 is higher that 2.3.5 | 10:46 |
kashyap | Much frowned upon, though. :-) | 10:46 |
kashyap | Right. | 10:47 |
zigo | We try to avoid it as much as possible, because that's very confusing when writting dependency, and a mistake may happen very fast... | 10:47 |
kashyap | Yeah, it's an emergency hammer | 10:47 |
zigo | For example, if you want version >= 2.3.5 and forget the 1:, then you get 1:2.2.4 satisfying your dependency and you're screwed. | 10:47 |
zigo | Does anyone know if Nova fixed the issue of volume live migration in Rocky, and if yes, in what point release? | 10:48 |
zigo | I've tried upgrading os-brick to 2.5.8 in a compute, it didn't help... | 10:49 |
zigo | I'll try nova and cinder next... | 10:49 |
kashyap | zigo: Can you refresh the page to now check if Debian stuff is all kosher? | 10:50 |
zigo | kashyap: LGTM ! :) | 10:50 |
kashyap | stephenfin: Any ideas to transfer this to a better place than MediaWiki? -- can we do proper tables in rST? | 10:51 |
kashyap | https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix | 10:51 |
kashyap | The MediaWiki syntax sucks | 10:51 |
kashyap | We _can_ do tables in rST; I've done it myself. Perhaps will consider moving this page in-tree | 10:51 |
*** rcernin has joined #openstack-nova | 10:52 | |
*** chenhaw has quit IRC | 10:59 | |
*** ociuhandu has joined #openstack-nova | 11:02 | |
openstackgerrit | Merged openstack/nova master: Convert legacy nova-live-migration and nova-multinode-grenade to py3 https://review.opendev.org/692374 | 11:04 |
*** mkrai has joined #openstack-nova | 11:05 | |
*** luksky has quit IRC | 11:06 | |
zigo | Ok, I found it, the issue is in Nova, and fixed by the point release in Rocky. | 11:12 |
*** awalende has joined #openstack-nova | 11:14 | |
*** awalende has quit IRC | 11:14 | |
*** udesale has quit IRC | 11:17 | |
*** awalende has joined #openstack-nova | 11:20 | |
*** awalende has quit IRC | 11:21 | |
*** luksky has joined #openstack-nova | 11:22 | |
*** brinzhang has joined #openstack-nova | 11:23 | |
stephenfin | kashyap: Yeah, that probably makes sense in doc/source/reference | 11:37 |
stephenfin | and you should be able to do it in tables, yup | 11:37 |
kashyap | stephenfin: Thanks! I'll get to it. | 11:40 |
*** mkrai has quit IRC | 11:44 | |
*** brinzhang has quit IRC | 11:47 | |
zigo | Does anyone know which commit has the cinder volume live migration fix in Rocky? | 11:49 |
*** mkrai has joined #openstack-nova | 11:49 | |
lyarwood | zigo: That's pretty generic, do you have a launchpad bug refrence I could check? | 11:50 |
zigo | lyarwood: Basically, in 18.1.0, live migrating a cinder volume never finshes, with 18.2.3 that's fixed... | 11:51 |
zigo | I don't know about any bug number though, I just tested and saw it was fixed. | 11:51 |
zigo | I'm trying to lookup commits now ... | 11:51 |
lyarwood | zigo: ah retyping a volume? | 11:52 |
zigo | lyarwood: Just live-migrating a volume: | 11:53 |
zigo | openstack volume migrate --host <volume-hostname> <volume-id> | 11:53 |
lyarwood | zigo: I landed https://review.opendev.org/#/q/I1bdf3431bda2da98380e0dcaa9f952e6768ca3af to fix that | 11:53 |
zigo | In 18.1.0, the new volume is re-attached ... | 11:53 |
*** mkrai has quit IRC | 11:54 | |
zigo | ... but the old one never deleted (and both new and old are still in the bad migration status) | 11:54 |
*** mkrai_ has joined #openstack-nova | 11:54 | |
lyarwood | ack yeah that came in with 18.2.1 | 11:54 |
zigo | lyarwood: Thanks, this looks like it. | 11:55 |
zigo | lyarwood: FYI, I need to have the exact patch, so I get this fast in Buster (upgrading to 18.2.3 has a lot of chances to be unfortunately rejected by the Debian stable release team...) | 11:55 |
lyarwood | https://review.opendev.org/#/c/657577/ is the change | 11:56 |
zigo | Yeah, got it, thanks a lot. | 11:56 |
*** bhagyashris has quit IRC | 11:56 | |
*** bhagyashris has joined #openstack-nova | 11:57 | |
*** dpawlik has quit IRC | 11:58 | |
zigo | lyarwood: I need that one too, right? | 12:00 |
zigo | https://review.opendev.org/#/c/639331/ | 12:00 |
zigo | Ah no, that's for tempest ... | 12:00 |
*** dpawlik has joined #openstack-nova | 12:01 | |
*** brinzhang has joined #openstack-nova | 12:03 | |
*** zhanglong has joined #openstack-nova | 12:04 | |
lyarwood | zigo: yeah you don't technically need that, it will however ensure things are tested correctly if Debian stable has CI for OpenStack | 12:04 |
zigo | lyarwood: Yeah, I may also patch the tempest package... :) | 12:05 |
*** ociuhandu has quit IRC | 12:13 | |
*** Liang__ has joined #openstack-nova | 12:15 | |
*** brinzhang has quit IRC | 12:19 | |
*** mkrai_ has quit IRC | 12:19 | |
*** brinzhang has joined #openstack-nova | 12:20 | |
*** bhagyashris has quit IRC | 12:21 | |
openstackgerrit | Pavel Gluschak (scsnow) proposed openstack/nova master: Don't delete compute node, when deleting service other that nova-compute https://review.opendev.org/694756 | 12:22 |
openstackgerrit | Pavel Gluschak (scsnow) proposed openstack/nova master: Don't delete compute node, when deleting service other than nova-compute https://review.opendev.org/694756 | 12:27 |
openstackgerrit | Pavel Gluschak (scsnow) proposed openstack/nova master: Don't delete compute node, when deleting service other than nova-compute https://review.opendev.org/694756 | 12:28 |
*** rcernin has quit IRC | 12:31 | |
*** damien_r has quit IRC | 12:41 | |
*** lpetrut has joined #openstack-nova | 12:42 | |
*** ociuhandu has joined #openstack-nova | 12:43 | |
efried | o/ nova | 12:43 |
efried | I'm going to be out today. | 12:43 |
efried | Enjoy. | 12:43 |
*** efried is now known as efried_pto | 12:43 | |
*** luksky has quit IRC | 12:47 | |
*** ratailor__ has quit IRC | 12:51 | |
gibi | efried_pto: enjoy your day too | 13:00 |
*** zhanglong has quit IRC | 13:02 | |
*** ociuhandu has quit IRC | 13:04 | |
*** damien_r has joined #openstack-nova | 13:04 | |
*** zhanglong has joined #openstack-nova | 13:07 | |
*** lennyb has joined #openstack-nova | 13:08 | |
*** zhanglong has quit IRC | 13:13 | |
*** sapd1 has quit IRC | 13:16 | |
*** dtantsur is now known as dtantsur|bbl | 13:21 | |
*** nanzha has quit IRC | 13:24 | |
*** nanzha has joined #openstack-nova | 13:25 | |
*** zhanglong has joined #openstack-nova | 13:29 | |
*** nweinber has joined #openstack-nova | 13:37 | |
*** luksky has joined #openstack-nova | 13:39 | |
*** KeithMnemonic has joined #openstack-nova | 13:40 | |
*** mriedem has joined #openstack-nova | 13:52 | |
*** yan0s has joined #openstack-nova | 13:52 | |
*** awalende has joined #openstack-nova | 13:55 | |
*** zhanglong has quit IRC | 13:55 | |
*** zhanglong has joined #openstack-nova | 13:57 | |
*** slaweq has quit IRC | 14:02 | |
*** slaweq has joined #openstack-nova | 14:04 | |
*** lennyb has quit IRC | 14:14 | |
mriedem | i wonder, maybe we should drop using the neutron-grenade-multinode job which is 2 node grenade + smoke tests. we already have nova-grenade-multinode which is just live migration tests in both a block storage + lvm and shared storage + ceph setup. if we changed nova-grenade-multinode to run smoke tests + live migration for the block storage + lvm case, before changing the env to ceph, then we'd cover what the neutron-grenade-m | 14:23 |
mriedem | node job was doing | 14:23 |
mriedem | oh nova-grenade-multinode also runs a cold migrate test since train | 14:24 |
*** awalende has quit IRC | 14:26 | |
*** jbernard has quit IRC | 14:27 | |
sean-k-mooney | mriedem: that sounds resonable. have you looked at https://review.opendev.org/#/c/548936/ it woudl be good to eventually get that merged to have a native grenade job | 14:27 |
mriedem | nope i haven't. it's huge and it scares me. | 14:27 |
*** davee__ has quit IRC | 14:28 | |
*** beekneemech is now known as bnemec | 14:29 | |
*** jbernard has joined #openstack-nova | 14:29 | |
sean-k-mooney | it is, it does appear to work at least. that said the last ci run was july so it might not anymore | 14:29 |
*** brinzhang has quit IRC | 14:33 | |
*** zhanglong has quit IRC | 14:34 | |
mriedem | grenade-core includes devstack-core and the devstack / qa team have done a lot of the initial zuulv3-ification of stuff, starting with devstack and tempest, so i'll defer to them for reviewing that massive zuulv3 chnage | 14:34 |
mriedem | *change | 14:34 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: FUP to Ie1a0cbd82a617dbcc15729647218ac3e9cd0e5a9 https://review.opendev.org/694787 | 14:38 |
*** jbernard_ has joined #openstack-nova | 14:47 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Drop neutron-grenade-multinode job https://review.opendev.org/694789 | 14:50 |
*** francoisp has joined #openstack-nova | 14:57 | |
artom | If you do a Follow Up Patch to a Follow Up Patch, is it a Follow Up Patch Addendum? | 14:57 |
*** eharney has joined #openstack-nova | 14:57 | |
*** Luzi has quit IRC | 14:59 | |
*** tesseract has quit IRC | 15:01 | |
*** tesseract has joined #openstack-nova | 15:01 | |
*** dtantsur|bbl is now known as dtantsur | 15:01 | |
mriedem | so making nova-grenade-multinode run smoke tests adds about 14-15 minutes to the job unless we prune down the smoke tests that it runs which we might want to consider doing - to just run compute API and scenario smoke tests, though then we lose some of the coverage that neutron-grenade-multinode gave us, | 15:07 |
mriedem | though a nova change likely isn't going to break an identity/image/object/network/volume API change that doesn't use nova at all | 15:07 |
*** dpawlik has quit IRC | 15:14 | |
mriedem | i would need some help on the regexing though.. | 15:19 |
* bauzas is having laptop issues today, sorry folks (upgrading) | 15:19 | |
*** TxGirlGeek has joined #openstack-nova | 15:19 | |
mriedem | artom: you like a good regex test right? | 15:20 |
mriedem | say i have a test string of: tempest.api.compute.flavors.test_flavors.FlavorsV2TestJSON.test_get_flavor[id-1f12046b-753d-40d2-abb6-d8eb8b30cb2f,smoke] | 15:20 |
mriedem | i want to match tempest.api.compute.* and tempest.scenario.* but only for those strings containing 'smoke' | 15:20 |
mriedem | i think (tempest.api.compute.*|tempest.scenario.*)([\bsmoke\b]) might work? | 15:21 |
artom | mriedem, I think aspiers was the regex master, but I'll try | 15:21 |
artom | mriedem, that should work | 15:22 |
*** udesale has joined #openstack-nova | 15:22 | |
artom | I'd have done it as 'tempest\.(api\.compute|scenario)\..*smoke' | 15:23 |
mriedem | yeah that seems to work as long as smoke comes last | 15:25 |
mriedem | well, i guess there is no $ on the regex so i guess it's ok | 15:26 |
artom | So 'tempest\.(api\.compute|scenario)\..*smoke.*' then :) | 15:26 |
mriedem | tempest.(api.compute|scenario)\..*smoke.* | 15:26 |
*** ociuhandu has joined #openstack-nova | 15:26 | |
mriedem | yeah | 15:26 |
*** ociuhandu has quit IRC | 15:28 | |
*** links has quit IRC | 15:36 | |
sean-k-mooney | bauzas: could you take a look at https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1763766 ideally im aiming to get this merged before m1 | 15:36 |
sean-k-mooney | stephenfin: similar for https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/vm-scoped-sriov-numa-affinity if i can get that merged before m1 it would make peopel happy so if you could review it would be nice before i start asking others | 15:37 |
*** Liang__ has quit IRC | 15:48 | |
*** cdent has joined #openstack-nova | 15:55 | |
*** cdent has left #openstack-nova | 15:56 | |
*** dklyle has quit IRC | 15:57 | |
*** macz has joined #openstack-nova | 15:58 | |
*** dklyle has joined #openstack-nova | 15:58 | |
bauzas | sean-k-mooney: ack, will look | 15:59 |
*** luksky has quit IRC | 15:59 | |
*** hamzy has quit IRC | 16:01 | |
*** gyee has joined #openstack-nova | 16:02 | |
*** mlavalle has joined #openstack-nova | 16:04 | |
*** udesale has quit IRC | 16:06 | |
*** jmlowe has joined #openstack-nova | 16:07 | |
openstackgerrit | Mark Goddard proposed openstack/nova master: [ironic] Fix node rebalance race issues https://review.opendev.org/694802 | 16:10 |
mriedem | mgoddard: hi https://review.opendev.org/#/c/684849/ | 16:14 |
kashyap | Can someone update the openSUSE stuff here? -- https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix | 16:14 |
kashyap | (I've updated for Debian, Ubuntu, RHEL and Fedora.) | 16:14 |
*** ociuhandu has joined #openstack-nova | 16:14 | |
kashyap | Who's the contact point for Oracle Linux again? | 16:14 |
mgoddard | mriedem: thanks, I'll check it out | 16:15 |
artom | kashyap, probably Satan | 16:18 |
* mriedem finds the code of conduct link | 16:19 | |
artom | Dammit >< | 16:19 |
kashyap | artom: Yeah, c'mon | 16:19 |
*** ociuhandu has quit IRC | 16:19 | |
* artom stops, apologies | 16:19 | |
mgoddard | mriedem: it could be because I'm using Rocky, but I don't see any errors in RT following rebalance | 16:22 |
mriedem | on either node? | 16:24 |
mriedem | s/node/compute service host/ | 16:24 |
mgoddard | yeah | 16:25 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Add TODOs for remaining nova-network functional tests https://review.opendev.org/684345 | 16:27 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'os-security-group-default-rules' REST API https://review.opendev.org/686807 | 16:27 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove unused '*_default_rules' security group DB APIs https://review.opendev.org/686808 | 16:27 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove (most) '/os-networks' REST APIs https://review.opendev.org/686809 | 16:27 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove '/os-tenant-networks' REST API https://review.opendev.org/686810 | 16:27 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove 'USE_NEUTRON' from functional tests https://review.opendev.org/686811 | 16:27 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove 'networks' quota https://review.opendev.org/686812 | 16:27 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove nova-manage network, floating commands https://review.opendev.org/686813 | 16:27 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove associate, disassociate network APIs https://review.opendev.org/686814 | 16:27 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove 'nova-dhcpbridge' binary https://review.opendev.org/686815 | 16:27 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove 'nova-network' binary https://review.opendev.org/686816 | 16:27 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Blast most references to nova-network https://review.opendev.org/686817 | 16:27 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP https://review.opendev.org/686818 | 16:27 |
mriedem | mgoddard: your commit message says you have sql errors in the logs, likely due to duplicate entry errors? | 16:28 |
mgoddard | mriedem: I never really got to the bottom of that one. http://paste.openstack.org/show/786272/ | 16:29 |
mriedem | do you have this patch? https://review.opendev.org/#/c/676514/ | 16:29 |
mriedem | what version of rocky do you have? | 16:29 |
mgoddard | yes I have that patch | 16:29 |
mgoddard | I have 18.2.0, plus a few backports (including that one) | 16:30 |
*** ociuhandu has joined #openstack-nova | 16:30 | |
mgoddard | mriedem: I have a few things to look at here. I could try my reproducer with your patch, and try your func test with my patch | 16:33 |
mriedem | yeah it would be good to know if they aren't duplicate bugs but something else | 16:35 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle NoSuchOptError in nova-manage gracefully https://review.opendev.org/694806 | 16:40 |
*** mdbooth has joined #openstack-nova | 16:48 | |
*** hamzy has joined #openstack-nova | 16:51 | |
mgoddard | mriedem: my patch passes the func test, with two modifications | 16:53 |
mgoddard | 1. I didn't hit "Skipping removal of allocations for deleted instances" | 16:53 |
mgoddard | 2. Part of my change prevents the orphan deletion happening in that case, so I rolled that part back for testing | 16:54 |
mgoddard | https://review.opendev.org/#/c/694802/1/nova/db/sqlalchemy/api.py | 16:54 |
*** damien_r has quit IRC | 16:54 | |
mgoddard | ^ that prevents host2 from deleting a compute node after host1 has taken ownership of it | 16:54 |
mgoddard | which should prevent us from hitting this issue | 16:55 |
mgoddard | the rest of the change is to help clean up if we have hit it previously | 16:55 |
*** ralonsoh has quit IRC | 16:58 | |
mriedem | mgoddard: ok i guess it's up to king solomon then to figure out which patch to go with | 16:59 |
mriedem | or just -1 my change if you want | 16:59 |
mriedem | either way i think the functional test needs to stay b/c unit tests aren't really sufficient for this kind of multi-service and db interaction | 17:00 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle NoSuchOptError in nova-manage gracefully https://review.opendev.org/694806 | 17:01 |
*** ralonsoh has joined #openstack-nova | 17:01 | |
*** yan0s has quit IRC | 17:02 | |
*** luksky has joined #openstack-nova | 17:02 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Print help if nova-manage subcommand is not specified https://review.opendev.org/694806 | 17:02 |
*** jmlowe has quit IRC | 17:04 | |
*** ociuhandu has quit IRC | 17:17 | |
*** rpittau is now known as rpittau|afk | 17:18 | |
*** nanzha has quit IRC | 17:20 | |
*** hamzy has quit IRC | 17:21 | |
*** hamzy has joined #openstack-nova | 17:22 | |
*** nweinber has quit IRC | 17:26 | |
*** ociuhandu has joined #openstack-nova | 17:28 | |
*** ircuser-1 has joined #openstack-nova | 17:40 | |
*** ociuhandu has quit IRC | 17:40 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Use admin neutron client to see if instance has qos ports https://review.opendev.org/694668 | 17:41 |
artom | sean-k-mooney, left you things on https://review.opendev.org/#/c/687957/5 | 17:44 |
sean-k-mooney | just a couple :) | 17:50 |
sean-k-mooney | ill adress them tomorrow. over all are you ok with the patch | 17:51 |
artom | Yeah, once we've taken image difference out of the picture, I don't see issues with not running the NUMATopologyFilter on rebuilds | 17:56 |
artom | (Famous last words) | 17:56 |
artom | Might be good to get jparker to look at it | 17:57 |
artom | He came up with interesting test cases for NUMA LM, he might find a way to break it :) | 17:57 |
artom | Not necessarily the code itself, just the idea of it | 17:57 |
sean-k-mooney | well the idea is if we assert no numa toplogy constraits change in the api we dont need to use the filter to assert that again since rebuilds cant change host | 17:58 |
sean-k-mooney | so the existing claim should be valid | 17:58 |
artom | Yep :) I'm just paranoid | 17:59 |
sean-k-mooney | you could break the rebuild in other ways | 17:59 |
artom | Right, that's out of scope | 17:59 |
sean-k-mooney | e.g. adding a trait that is not present on the host | 17:59 |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: [RFC] Pick NEXT_MIN libvirt/QEMU versions for "V" release https://review.opendev.org/694821 | 17:59 |
sean-k-mooney | ya im not trying to adress that although i hope we already do | 17:59 |
artom | ... do we check placement for rebuild? | 18:00 |
sean-k-mooney | i hope so but im not sure. its on my todo list to look at in general | 18:00 |
artom | Your ocean-spanning todo list :D | 18:01 |
sean-k-mooney | yes well when i fix things i keep findind other thngs that need to be fixed | 18:01 |
*** tbachman has joined #openstack-nova | 18:02 | |
artom | https://xkcd.com/1739/ | 18:02 |
sean-k-mooney | like i found out that apparently we allow you to attach port that violate numa toplogy and you will only find out if you rebuild or otherwise have to go back to the schduler | 18:02 |
artom | Oh are those your comments on the second patch in that stack? | 18:02 |
sean-k-mooney | they are on a different patch one sec | 18:02 |
*** dtantsur is now known as dtantsur|afk | 18:03 | |
sean-k-mooney | artom: https://review.opendev.org/#/c/674072/7/nova/tests/unit/virt/test_hardware.py@4240 | 18:03 |
sean-k-mooney | no wrong one | 18:04 |
sean-k-mooney | also found that issue... | 18:04 |
sean-k-mooney | https://review.opendev.org/#/c/689861/3/nova/tests/functional/libvirt/test_numa_servers.py@1106 | 18:05 |
sean-k-mooney | i need to make that less wordy but ya that existing test was wrong | 18:06 |
artom | Or make it into its own patch? I haven't dug in yet, is what related to NUMA rebuilds? | 18:10 |
artom | (That last bit) | 18:11 |
sean-k-mooney | fixing the attachment api to reject interace if it would volate the numa aware vstichs feature is a spereate thing | 18:12 |
sean-k-mooney | at present it does not check and then a rebuild or resize to the same host woudl fail | 18:12 |
sean-k-mooney | i need to file a seperate bug and address it seperately | 18:13 |
artom | sean-k-mooney, I mean just putting the comments in a separate patch :) | 18:17 |
artom | Anyways, lemme review that properly | 18:17 |
sean-k-mooney | i needed to chagne the test so i added the comment to explain why | 18:18 |
artom | Ack | 18:18 |
artom | As I said, I hadn't dug in yet | 18:18 |
sean-k-mooney | i proably should change teh NOTE to a FIXME | 18:18 |
openstackgerrit | Archit Modi proposed openstack/nova stable/pike: doc: fix and clarify --block-device usage in user docs https://review.opendev.org/694450 | 18:21 |
*** mvkr has quit IRC | 18:32 | |
*** tbachman has quit IRC | 18:36 | |
*** igordc has joined #openstack-nova | 18:46 | |
*** martinkennelly has quit IRC | 18:48 | |
mriedem | this is a good pike-only fix for something that was regressed with a backport https://review.opendev.org/#/q/topic:bug/1852207+(status:open+OR+status:merged) | 18:50 |
*** gouthamr_ is now known as gouthamr | 18:51 | |
*** tesseract has quit IRC | 18:52 | |
*** tbachman has joined #openstack-nova | 18:52 | |
*** ralonsoh has quit IRC | 18:56 | |
artom | sean-k-mooney, grah, forgot the backporting thing - might have left comments that make no sense in that context | 19:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: FUP to Ie1a0cbd82a617dbcc15729647218ac3e9cd0e5a9 https://review.opendev.org/694787 | 19:24 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Drop neutron-grenade-multinode job https://review.opendev.org/694789 | 19:24 |
*** dpawlik has joined #openstack-nova | 19:26 | |
*** jdillaman has joined #openstack-nova | 19:30 | |
*** abaindur has joined #openstack-nova | 19:34 | |
*** abaindur has quit IRC | 19:35 | |
*** abaindur has joined #openstack-nova | 19:36 | |
*** rafaelweingartne has joined #openstack-nova | 19:43 | |
rafaelweingartne | Hey guys, we are found on Nova that might be a bug | 19:44 |
rafaelweingartne | when a user is updating something in the VM (e.g. its name), this information is not updated in the KVM XML that is used to describe the VM in the compute host | 19:44 |
rafaelweingartne | is that expected? Or, is it a bug? | 19:44 |
dansmith | rafaelweingartne: generally only something like a volume attach will change the libvirt xml at runtime | 19:47 |
rafaelweingartne | hmm | 19:47 |
rafaelweingartne | so, that is by design? | 19:47 |
dansmith | rafaelweingartne: if we store the display name in the xml (do we?) we wouldn't update that until a migration or hard reboot type event | 19:47 |
rafaelweingartne | yes, we do store some data there | 19:47 |
rafaelweingartne | not just the display name, but also project for instance | 19:47 |
rafaelweingartne | and this is troublesome for Ceilometer, as it was trusting that data | 19:48 |
dansmith | I see, in metadata. No, we don't change that on the fly | 19:48 |
rafaelweingartne | hmm | 19:48 |
rafaelweingartne | should we fix it in Ceilometer or in Nova? | 19:48 |
dansmith | not even sure the compute gets notified when that changes.. I suspect not | 19:48 |
rafaelweingartne | I opened a bug report: https://bugs.launchpad.net/ceilometer/+bug/1853048 | 19:48 |
openstack | Launchpad bug 1853048 in OpenStack Compute (nova) "Nova not updating VM's XML in KVM" [Undecided,New] | 19:48 |
dansmith | rafaelweingartne: ceilometer scraping anything out of the xml underneath nova would be at its own peril for sure | 19:49 |
rafaelweingartne | hmm | 19:50 |
*** abaindur has quit IRC | 19:50 | |
dansmith | yeah, confirmed, the compute node won't even be notified that such things have changed via the api, | 19:50 |
rafaelweingartne | So, this is by design in Nova? If someone were to fix it, you guys would prefer to fix on Ceilometer side, right? | 19:51 |
*** abaindur has joined #openstack-nova | 19:51 | |
dansmith | it would just re-read that info from the db the next time it needs to generate the XML, which like I say would be some significant livecycle event | 19:51 |
dansmith | rafaelweingartne: yes, by design | 19:51 |
dansmith | rafaelweingartne: if you mean ceilometer update the xml itself, then no, but if you mean change ceilometer to get that info from somewhere else, then yes | 19:51 |
rafaelweingartne | and, what about changing Nova to update the data when the user is updating something (e.g. display name of a VM)? | 19:52 |
dansmith | rafaelweingartne: I would be -2 on that without some really really strong use-case, and ceilometer scraping that info and expecting it to be reliable is not good enough | 19:54 |
rafaelweingartne | ok | 19:54 |
rafaelweingartne | got it | 19:54 |
*** rafaelweingartne has quit IRC | 19:58 | |
*** eharney has quit IRC | 20:02 | |
*** rafaelweingartne has joined #openstack-nova | 20:03 | |
*** jbernard has quit IRC | 20:14 | |
*** jmlowe has joined #openstack-nova | 20:16 | |
*** jbernard has joined #openstack-nova | 20:17 | |
*** dpawlik has quit IRC | 20:22 | |
*** rafaelweingartne has quit IRC | 20:24 | |
*** dpawlik has joined #openstack-nova | 20:27 | |
*** dpawlik has quit IRC | 20:41 | |
*** dpawlik has joined #openstack-nova | 20:44 | |
*** mgariepy has quit IRC | 20:57 | |
openstackgerrit | Merged openstack/nova master: Remove service_uuids_online_data_migration https://review.opendev.org/693421 | 21:01 |
openstackgerrit | Merged openstack/nova master: functional: Change order of two classes https://review.opendev.org/689178 | 21:01 |
openstackgerrit | Merged openstack/nova master: functional: Rework '_delete_server' https://review.opendev.org/689179 | 21:02 |
*** ozzzo has joined #openstack-nova | 21:08 | |
ozzzo | I created an aggregate with property "allflavors='0'" | 21:08 |
ozzzo | https://paste.fedoraproject.org/paste/X8lzKsU474FD1Ffj2x29uQ | 21:08 |
ozzzo | Then I created VMs and specified "allflavors='1'" | 21:08 |
ozzzo | openstack server create --flavor s1.1cx4g --image NOT-QSC-CentOS7.6-19P1-v4 --network it-network --property allflavors='1' --max 20 alberttest | 21:08 |
ozzzo | but some of the VMs I created went to hv005 which should be restricted by "allflavors" | 21:09 |
ozzzo | https://paste.fedoraproject.org/paste/AhxIfQnQjeWPgMw33xmbKQ | 21:09 |
ozzzo | what am I doing wrong? | 21:09 |
ozzzo | I tried asking in #openstack but nobody is answering questions there | 21:09 |
*** eharney has joined #openstack-nova | 21:12 | |
*** awalende has joined #openstack-nova | 21:16 | |
*** awalende has quit IRC | 21:21 | |
*** awalende has joined #openstack-nova | 21:26 | |
*** dpawlik has quit IRC | 21:29 | |
*** awalende has quit IRC | 21:31 | |
*** awalende has joined #openstack-nova | 21:36 | |
openstackgerrit | Merged openstack/nova master: Stop testing Python 2 https://review.opendev.org/687954 | 21:38 |
*** ociuhandu has joined #openstack-nova | 21:38 | |
*** awalende has quit IRC | 21:40 | |
*** abaindur has quit IRC | 21:42 | |
*** jbernard has quit IRC | 21:42 | |
*** dpawlik has joined #openstack-nova | 21:43 | |
*** ociuhandu has quit IRC | 21:43 | |
*** abaindur has joined #openstack-nova | 21:43 | |
*** jbernard has joined #openstack-nova | 21:45 | |
*** awalende has joined #openstack-nova | 21:46 | |
*** jbernard_ has quit IRC | 21:46 | |
*** abaindur has quit IRC | 21:50 | |
mriedem | that's an exclusion problem with the aggregates filters | 21:51 |
mriedem | and --property is server metadata, not part of the flavor extra spec | 21:52 |
mriedem | see https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html#aggregateinstanceextraspecsfilter | 21:52 |
*** awalende has quit IRC | 21:56 | |
*** awalende has joined #openstack-nova | 21:57 | |
ozzzo | mriedem: does that mean that we can't specify extra-specs on the command line? Does it have to be in a flavor? | 21:59 |
mriedem | correct | 22:00 |
mriedem | admins control extra specs in the flavors, not users | 22:00 |
mriedem | by default policy rules anyway | 22:00 |
ozzzo | so we can only delegate to users by giving them a menu of flavors? | 22:00 |
ozzzo | or by setting specs in image? | 22:01 |
mriedem | correct | 22:01 |
ozzzo | ok that is very helpful, ty! | 22:01 |
mriedem | filtering by image properties is a different filter | 22:01 |
mriedem | https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html#aggregateimagepropertiesisolation | 22:01 |
ozzzo | ok got it | 22:02 |
mriedem | you could of course write your own filter that processes server metadata linked to a host aggregate but that could be semi expensive since the RequestSpec object passed to the filter interface does not have the server metadata in it (you'd have to get that from the db per instance in the request) | 22:03 |
sean-k-mooney | ozzzo: you may be interested in a spec that is currently being proposed in this area | 22:03 |
sean-k-mooney | https://review.opendev.org/#/c/663563/ | 22:04 |
sean-k-mooney | this is still changing alot and is not that popular but if you have usecase that would requrie something other then what flaovrs or images provide today | 22:05 |
ozzzo | that does look interesting, ty! | 22:05 |
sean-k-mooney | it would be good to read that and provide feedback | 22:05 |
sean-k-mooney | changing how this works is problematic as alot of the code and external tooling assumes flavor and image are the way resouce requests and qualitive aspects of an instance are defiened | 22:06 |
*** awalende has quit IRC | 22:07 | |
*** awalende has joined #openstack-nova | 22:07 | |
sean-k-mooney | that does not mean we cant discuss changing it its just hard not to break things in this if we do | 22:07 |
*** dpawlik has quit IRC | 22:08 | |
*** awalende_ has joined #openstack-nova | 22:08 | |
*** awalende has quit IRC | 22:12 | |
*** abaindur has joined #openstack-nova | 22:20 | |
*** abaindur has quit IRC | 22:24 | |
*** pcaruana has quit IRC | 22:26 | |
*** kaisers1 has joined #openstack-nova | 22:28 | |
*** kaisers has quit IRC | 22:29 | |
*** mriedem has quit IRC | 22:31 | |
*** maciejjozefczyk has quit IRC | 22:39 | |
*** maciejjozefczyk has joined #openstack-nova | 22:39 | |
*** mvkr has joined #openstack-nova | 22:40 | |
*** abaindur has joined #openstack-nova | 22:44 | |
*** abaindur has joined #openstack-nova | 22:45 | |
*** awalende_ has quit IRC | 22:46 | |
*** rcernin has joined #openstack-nova | 22:46 | |
*** slaweq has quit IRC | 22:50 | |
*** luksky has quit IRC | 23:02 | |
*** tkajinam has joined #openstack-nova | 23:09 | |
*** aloga_ has quit IRC | 23:10 | |
*** slaweq has joined #openstack-nova | 23:11 | |
*** slaweq has quit IRC | 23:17 | |
*** jaosorior has quit IRC | 23:21 | |
*** mlavalle has quit IRC | 23:24 | |
*** ivve has quit IRC | 23:26 | |
*** Liang__ has joined #openstack-nova | 23:30 | |
*** ociuhandu has joined #openstack-nova | 23:31 | |
*** ociuhandu has quit IRC | 23:36 | |
openstackgerrit | Merged openstack/nova master: Move rng device checks to the appropriate method https://review.opendev.org/693210 | 23:39 |
*** aloga has joined #openstack-nova | 23:40 | |
*** zhanglong has joined #openstack-nova | 23:47 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!