Friday, 2019-09-27

*** sapd1_x has joined #openstack-nova00:05
*** gyee has quit IRC00:05
openstackgerritMerged openstack/nova master: Reset forced_destination before migration at a proper time  https://review.opendev.org/68454600:07
*** sapd1_x has quit IRC00:14
*** BjoernT has joined #openstack-nova00:14
*** KeithMnemonic has quit IRC00:18
*** markvoelker has joined #openstack-nova00:29
*** markvoelker has quit IRC00:33
*** gbarros has joined #openstack-nova00:38
*** gbarros has quit IRC00:42
*** mkrai has joined #openstack-nova00:49
*** markvoelker has joined #openstack-nova01:06
*** Liang__ has joined #openstack-nova01:13
*** francoisp has quit IRC01:28
*** mkrai has quit IRC01:37
*** HagunKim has joined #openstack-nova01:39
HagunKimHello, I changed compute node's disk for /var/lib/nova and mount new disk to /var/lib/nova successfully. All vm instances work well.01:45
HagunKimBut 'openstack hypervisor show [Compute node]' shows local_gb as old disk capacity.01:45
HagunKimHow can I update it?01:45
HagunKimIs there problem when I modify DB(nova.compute_nodes.local_gb) directly?01:45
*** mkrai has joined #openstack-nova02:13
*** markvoelker has quit IRC02:25
openstackgerritmelanie witt proposed openstack/nova master: Reduce scope of 'path' query parameter to noVNC consoles  https://review.opendev.org/68519402:25
*** BjoernT has quit IRC02:38
*** artom has quit IRC02:42
*** ricolin has joined #openstack-nova02:49
*** mkrai has quit IRC02:51
*** dave-mccowan has quit IRC03:24
*** brinzhang_ has joined #openstack-nova03:47
*** brinzhang has quit IRC03:50
*** boxiang has quit IRC04:06
*** boxiang has joined #openstack-nova04:06
*** igordc has quit IRC04:08
*** ratailor has joined #openstack-nova04:14
*** mkrai has joined #openstack-nova04:17
*** markvoelker has joined #openstack-nova04:26
*** janki has joined #openstack-nova04:26
*** markvoelker has quit IRC04:30
*** ratailor has quit IRC04:42
*** ratailor has joined #openstack-nova04:42
*** tkajinam has quit IRC05:00
*** udesale has joined #openstack-nova05:00
*** tkajinam has joined #openstack-nova05:02
*** dpawlik has joined #openstack-nova05:16
*** brinzhang has joined #openstack-nova05:29
*** ociuhandu has joined #openstack-nova05:31
*** brinzhang_ has quit IRC05:32
*** ociuhandu has quit IRC05:35
*** tkajinam_ has joined #openstack-nova05:36
*** tkajinam has quit IRC05:38
*** Luzi has joined #openstack-nova05:42
*** ratailor has quit IRC05:43
*** ratailor has joined #openstack-nova05:44
*** maciejjozefczyk has joined #openstack-nova06:03
*** ratailor_ has joined #openstack-nova06:09
*** ratailor has quit IRC06:11
*** Guest29634 has quit IRC06:16
*** slaweq has joined #openstack-nova06:27
*** rcernin has quit IRC06:29
*** benj has joined #openstack-nova06:32
*** benj is now known as Guest5786106:32
*** Luzi has quit IRC06:40
*** larainema has joined #openstack-nova06:49
*** Luzi has joined #openstack-nova06:56
*** damien_r has joined #openstack-nova06:58
*** damien_r has quit IRC06:59
*** damien_r has joined #openstack-nova07:00
*** rpittau|afk is now known as rpittau07:05
*** ratailor_ has quit IRC07:05
*** ratailor has joined #openstack-nova07:08
*** tesseract has joined #openstack-nova07:20
*** pcaruana has joined #openstack-nova07:21
*** ociuhandu has joined #openstack-nova07:27
*** ociuhandu has quit IRC07:28
*** luksky has joined #openstack-nova07:35
*** yan0s has joined #openstack-nova07:37
*** priteau has joined #openstack-nova07:43
*** eharney has joined #openstack-nova07:46
*** jaosorior has quit IRC07:48
*** ralonsoh has joined #openstack-nova07:57
*** ociuhandu has joined #openstack-nova08:02
*** ttsiouts has joined #openstack-nova08:04
*** ociuhandu has quit IRC08:07
*** priteau has quit IRC08:09
*** lpetrut has joined #openstack-nova08:13
*** tkajinam_ has quit IRC08:16
openstackgerritSeyeong Kim proposed openstack/nova master: block_device: Optionally recreate attachments when refreshing connection_info  https://review.opendev.org/57900408:21
openstackgerritYongli He proposed openstack/nova master: clean up orphan instances  https://review.opendev.org/62776508:22
*** counterpart has joined #openstack-nova08:30
*** counterpart has left #openstack-nova08:30
*** ivve has joined #openstack-nova08:30
*** derekh has joined #openstack-nova08:33
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Disable neutron explicitly for 'ServersSampleBase' subclasses  https://review.opendev.org/68433008:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove *even* more unnecessary 'USE_NEUTRON = False'  https://review.opendev.org/68433108:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_availability_zone' functional tests  https://review.opendev.org/68433208:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ip_pools' functional tests  https://review.opendev.org/68433308:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_quota_sets' functional tests  https://review.opendev.org/68433408:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_server_tags' functional tests  https://review.opendev.org/68433508:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_servers' functional tests  https://review.opendev.org/68433608:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hosts' functional tests  https://review.opendev.org/68433708:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_networks_associate' functional tests  https://review.opendev.org/68433808:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_rescue' functional tests  https://review.opendev.org/68433908:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hypervisors' functional tests  https://review.opendev.org/68434008:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_attach_interfaces' functional tests  https://review.opendev.org/68434108:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_simple_tenant_usage' functional tests  https://review.opendev.org/68434208:42
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Neutron fixture cleanup  https://review.opendev.org/68434308:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ips' functional tests  https://review.opendev.org/68434408:42
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Add TODOs for remaining nova-network functional tests  https://review.opendev.org/68434508:42
*** ratailor has quit IRC08:48
*** ratailor has joined #openstack-nova08:51
*** pcaruana has quit IRC08:52
gibistephenfin: hi! do you have oppinion about his oslo.messaging bugfix ? https://review.opendev.org/#/c/683600/08:53
*** rcernin has joined #openstack-nova08:53
*** pcaruana has joined #openstack-nova09:03
openstackgerritOpenStack Release Bot proposed openstack/nova stable/train: Update .gitreview for stable/train  https://review.opendev.org/68525609:06
openstackgerritOpenStack Release Bot proposed openstack/nova stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68525709:06
openstackgerritOpenStack Release Bot proposed openstack/nova master: Update master for stable/train  https://review.opendev.org/68525809:06
openstackgerritLuyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory  https://review.opendev.org/67847009:14
*** dtantsur|afk is now known as dtantsur09:18
openstackgerritMerged openstack/nova master: docs: Note use of 'nova-manage db sync --config-file'  https://review.opendev.org/67129809:18
*** rpittau is now known as rpittau|bbl09:24
*** ricolin has quit IRC09:24
*** dtantsur is now known as dtantsur|lunch09:24
openstackgerritTakashi NATSUME proposed openstack/nova master: conf: Remove deprecated 'project_id_regex' opt  https://review.opendev.org/50105709:26
openstackgerritMerged openstack/nova master: docs: Correct 'nova-manage db sync' documentation  https://review.opendev.org/67750809:28
openstackgerritMerged openstack/nova master: docs: Rework the PCI passthrough guides  https://review.opendev.org/63524309:28
*** mkrai has quit IRC09:32
*** Liang__ has quit IRC09:39
openstackgerritTakashi NATSUME proposed openstack/nova master: conf: Remove deprecated 'project_id_regex' opt  https://review.opendev.org/50105709:41
*** ttsiouts has quit IRC09:41
*** ttsiouts has joined #openstack-nova09:42
stephenfingibi: I was hoping someone more knowledgeable in oslo.messaging stuff would take a look first, but adding oslo-messaging-core a few days ago hasn't helped. +2d09:44
gibistephenfin: thanks. It is a pretty old bug09:44
*** Guest57861 has quit IRC09:44
gibistephenfin: with lost history09:44
*** ttsiouts has quit IRC09:46
*** benj has joined #openstack-nova09:47
*** benj is now known as Guest9408309:48
stephenfinShouldn't we have a stable/train branch now?09:48
stephenfinOh, wait, we do09:48
* stephenfin should fetch first09:48
* gibi 's mirror seems to be lagging09:49
*** mkrai has joined #openstack-nova09:53
stephenfinThis means I can merge all the mox removal patches now, I assume?09:58
openstackgerritTakashi NATSUME proposed openstack/nova master: conf: Remove deprecated 'project_id_regex' opt  https://review.opendev.org/50105709:59
takashinstephenfin: The stable/train branch has been created in nova. So would you remove -2 in https://review.opendev.org/#/c/576020/ and vote workflow +1 for the patch and subsequent patches?10:03
stephenfintakashin: look two lines up :)10:04
stephenfinI'm trying to figure out if there's any reason not to do it now. It shouldn't cause too many conflicts but I'd like someone else's ack first10:04
takashinOkay.10:04
*** ratailor has quit IRC10:11
gibistephenfin: does these patches makes backporting potential rc bugs harder?10:11
stephenfingibi: I would say no because it only touches the one file (nova/tests/unit/network/test_neutronv2.py) and there have been very few non-mox removal changes against that file this cycle10:14
stephenfinSo the chances of there being bugs introduced that we'd have to fix in the file are low10:14
openstackgerritStephen Finucane proposed openstack/nova master: tox: Use common 'command' definition for unit tests  https://review.opendev.org/68477410:21
openstackgerritStephen Finucane proposed openstack/nova master: tox: Stop overriding the 'install_command'  https://review.opendev.org/68477510:21
openstackgerritStephen Finucane proposed openstack/nova master: tests: Nuke OS_STDOUT_CAPTURE, OS_STDERR_CAPTURE  https://review.opendev.org/68477310:21
openstackgerritStephen Finucane proposed openstack/nova master: tests: Start using global stdout, stderr buffers  https://review.opendev.org/68531010:21
*** brinzhang has quit IRC10:25
*** xek has quit IRC10:31
*** ratailor has joined #openstack-nova10:31
*** xek has joined #openstack-nova10:31
*** cdent has joined #openstack-nova10:31
*** takashin has left #openstack-nova10:33
*** tesseract has quit IRC10:35
*** tesseract has joined #openstack-nova10:35
*** HagunKim has quit IRC10:38
gibistephenfin: cool. if you need a second core to push them through then ping me10:44
*** ratailor_ has joined #openstack-nova10:45
*** ratailor has quit IRC10:47
stephenfingibi: I think oomichi has reviewed most of them too so we should be good. Think you could look at https://review.opendev.org/#/c/684762/ though?10:48
*** luksky has quit IRC10:49
*** boxiang has quit IRC10:59
*** zhubx has joined #openstack-nova10:59
*** dtantsur|lunch is now known as dtantsur11:03
*** mkrai has quit IRC11:13
*** mkrai has joined #openstack-nova11:13
*** luksky has joined #openstack-nova11:15
*** jaosorior has joined #openstack-nova11:19
*** ttsiouts has joined #openstack-nova11:23
*** jaosorior has quit IRC11:33
*** cdent has quit IRC11:34
*** jhesketh has quit IRC11:34
*** jhesketh has joined #openstack-nova11:35
*** mkrai has quit IRC11:46
*** ociuhandu has joined #openstack-nova11:47
*** ratailor__ has joined #openstack-nova11:49
*** ratailor_ has quit IRC11:51
*** Luzi has quit IRC11:52
efriedstephenfin: re mox, go for it.11:53
* stephenfin goes for it11:53
efriedand yes, having looked briefly, the only delta since +A on those was updating the bp name in the commit messages. If that's true, feel free to fast approve.11:53
efriedIs the bp already approved for ussuri? I think I did that. /me looks...11:54
efriednope, lemme hit that...11:54
efriedoh, I did but forgot to no-op update the series goal11:55
efrieddone now11:55
*** mkrai has joined #openstack-nova11:55
efriedkashyap: Could http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009743.html be https://review.opendev.org/#/c/684375/ ?11:58
* kashyap clicks11:59
kashyapefried: Indeed, that's it -- the "kvm_init_vcpu failed" error gives it away (it's _not_ main memory allocation, but a prob w/ NUMA interaction)12:00
efriedkashyap: sweet. Can you respond?12:00
kashyapefried: I'll reply there; first to try with kernel 4.1912:00
efriedbeaut12:00
efriedthank you12:00
kashyapefried: Will do; seems like it _just_ arrived?12:01
efriedyeah, <5h ago12:01
efriedserendipity12:01
kashyapAh, missed it; as I was reading only the [nova] tag12:01
kashyap(And this didn't have the tag)12:02
efriedCouldn't tell you why I actually read this one. Maybe because it said "numa affinity"?12:02
kashyapLikely12:02
*** markvoelker has joined #openstack-nova12:02
kashyapI don't know if we should make it into yet-another tunable; I'd think we should do the _right_ thing there.  Still mulling12:02
*** markvoelker has quit IRC12:03
efriedbut someone running into this can resolve by upgrading kernel?12:03
kashyap(Where just use the conservative, 'preferred'.  I'll let Sean come back to triple-check on me)12:03
kashyapefried: That's my theory.  Let's see if this reporter confirms.12:03
efriedis there not a case where you would truly *want* to use strict?12:03
kashyapAlso the kernel fix is extremely small (yes, yes, that doesn't say _anything_ about risk) and isoloated; so I'd suggest the Linux distros backport it.12:03
efriedseems like something something edge HPC realtime something12:03
kashyapefried: There are cases, but I'll double-check w/ the libvirt dev who meditates on this12:04
*** cdent has joined #openstack-nova12:04
efriedcool. /me watches from a safe distance12:05
* kashyap is only NUMA-surface-level-aware12:05
kashyapHeh12:05
openstackgerritStephen Finucane proposed openstack/python-novaclient master: Stop silently ignoring invalid 'nova boot --hint' options  https://review.opendev.org/68476212:06
*** zul has quit IRC12:06
*** maciejjozefczyk has quit IRC12:08
*** rchurch has quit IRC12:08
*** manjeets has quit IRC12:08
*** maciejjozefczyk has joined #openstack-nova12:08
kashyapefried: Completely unrelated - I wonder if all the 'no-reply' emails can be directed to another 'openstack-releases' list12:08
kashyapOtherwise the noise feels just too much.12:08
kashyapMaybe I should ask it on the list12:08
*** rchurch has joined #openstack-nova12:09
kashyapI know that the point of it is to give a heads-up to devs that a new release came out.12:09
*** dpawlik has quit IRC12:09
efriedkashyap:  respond to http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009744.html12:09
*** dpawlik has joined #openstack-nova12:09
kashyapAh, will do12:09
kashyapThanks for the pointer12:10
*** bnemec has quit IRC12:10
efriedyahyoubetcha. I didn't care enough to respond; I'm happy to hit [Delete] a couple dozen times twice a year.12:10
*** mvkr has quit IRC12:11
*** bnemec has joined #openstack-nova12:11
kashyapMy gripe is, the mailing list archives turn into a toilet12:11
kashyapIt just becomes so polluted that the signal is drowned in that noise.12:12
kashyapAnd it shoos away those who're reading the list to ignore it.  I'll respond on that thread. :D12:12
*** mvkr has joined #openstack-nova12:13
*** markvoelker has joined #openstack-nova12:15
*** dpawlik has quit IRC12:16
zigoIs Nova really broken with cryptography 2.6 ?12:22
zigoThat's kind of annoying me to require 2.7, because the Debian package isn't realy.12:22
zigo(and I'm not the maintainer of that one...)12:22
*** tbachman has joined #openstack-nova12:27
*** ratailor__ has quit IRC12:30
*** rpittau|bbl is now known as rpittau12:36
*** ociuhandu has quit IRC12:37
*** ociuhandu has joined #openstack-nova12:41
*** dpawlik has joined #openstack-nova12:42
zigobauzas: ^ Do you know ?12:45
openstackgerritMerged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13)  https://review.opendev.org/57602012:46
openstackgerritMerged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14)  https://review.opendev.org/57602712:46
openstackgerritMerged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15)  https://review.opendev.org/57603112:46
openstackgerritMerged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16)  https://review.opendev.org/57629912:46
openstackgerritMerged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17)  https://review.opendev.org/57634412:46
openstackgerritMerged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18)  https://review.opendev.org/57667312:47
openstackgerritMerged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19)  https://review.opendev.org/57667612:47
openstackgerritMerged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20)  https://review.opendev.org/57668912:47
openstackgerritMerged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21)  https://review.opendev.org/57670912:47
*** jangutter has quit IRC12:47
openstackgerritMerged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22)  https://review.opendev.org/57671212:47
*** nweinber_ has joined #openstack-nova12:51
*** dpawlik has quit IRC12:53
*** nweinber_ has quit IRC12:53
*** nweinber_ has joined #openstack-nova12:54
*** mkrai has quit IRC12:55
*** dpawlik has joined #openstack-nova12:56
*** larainema has quit IRC12:57
*** udesale has quit IRC12:58
*** udesale has joined #openstack-nova12:58
*** dpawlik has quit IRC13:00
*** mkrai has joined #openstack-nova13:02
*** francoisp has joined #openstack-nova13:03
gibistephenfin: regarding https://bugs.launchpad.net/python-novaclient/+bug/1845322 I think the same bug exists in openstackclient as well13:04
openstackLaunchpad bug 1845322 in python-novaclient "'--hint' argument to 'nova boot ' should be a key-value pair" [Low,In progress] - Assigned to Stephen Finucane (stephenfinucane)13:04
*** tkajinam has joined #openstack-nova13:05
*** eharney has quit IRC13:07
*** pcaruana has quit IRC13:09
gibistephenfin: opened a story for it https://storyboard.openstack.org/#!/story/200662813:09
*** mriedem has joined #openstack-nova13:10
openstackgerritTakashi NATSUME proposed openstack/nova master: Add TODO note for mox removal  https://review.opendev.org/57675813:18
*** rcernin has quit IRC13:18
*** BjoernT has joined #openstack-nova13:21
openstackgerritWalter A. Boring IV (hemna) proposed openstack/nova stable/pike: Avoid redundant initialize_connection on source post live migration  https://review.opendev.org/68300813:22
*** ociuhandu has quit IRC13:25
efriedstephenfin: Is your nova-net removal series13:27
efried... "done"? Like, at least proposed all the way up?13:27
efried(I haven't looked)13:27
efriedcurious whether we could just do away with https://review.opendev.org/#/c/576758/ and/or do the actual removal on top of your series.13:28
*** eharney has joined #openstack-nova13:29
*** luksky has quit IRC13:29
efried...looks like just the test bits are proposed at this point.13:30
*** ricolin has joined #openstack-nova13:30
*** dave-mccowan has joined #openstack-nova13:32
*** yan0s has quit IRC13:37
*** cdent has quit IRC13:41
*** mkrai has quit IRC13:41
stephenfingibi: Yeah, I pushed patches for that13:45
stephenfinWas trying to figure out why I couldn't file against openstackclient on launchpad. That would be why,heh13:46
*** artom has joined #openstack-nova13:46
stephenfinefried: Yeah, just the functional test piece pushed so far. I'm working on the rest of it slowly13:46
stephenfinTrying to avoid a mega "remove all the things" patch13:47
efriedstephenfin: as for cellsv1, good approach.13:47
stephenfinefried: And the ec2 cleanup...which isn't actually merged yet (I thought it was). Would you mind looking at them if/when you have a chance? https://review.opendev.org/#/q/topic:ec2-cleanup+(status:open+OR+status:merged)13:48
gibistephenfin: cool reading the openstackclient patch now13:48
*** yan0s has joined #openstack-nova13:49
dansmithmriedem: I think I'm +2 on all the +2able things in stein13:51
dansmithmriedem: (re: your mention from the meeting yesterday)13:51
openstackgerritmelanie witt proposed openstack/nova master: Reduce scope of 'path' query parameter to noVNC consoles  https://review.opendev.org/68519413:51
*** yan0s has quit IRC13:52
mriedemdansmith: ack13:52
mriedemlyarwood: if you're about can you hit these? https://review.opendev.org/#/q/project:openstack/nova+branch:stable/stein+status:open+label:Code-Review=213:53
mriedemdansmith: can you hit https://review.opendev.org/#/c/680871/ and https://review.opendev.org/#/c/676498/ for rocky?13:55
artommriedem, you saw ya.wang's review on that NUMA fix?13:55
dansmithI've got the rocky set up right now, are those two more important?13:55
mriedemmore important?13:56
mriedemthe former is a long-standing issue for baremetal instances, the latter is an lxc thing which is low priority13:56
mriedemlxc+py313:56
*** udesale has quit IRC13:56
mriedemartom: not yet13:56
dansmithmriedem: meaning you pointed me at two rocky reviews, just wondering if I can hit those when I get to them in the rocky queue or if I should do those first for some reason13:56
mriedemartom: does it work?13:56
artommriedem, yeah, tl;dr, he's +113:56
mriedemdansmith: oh, they're just ready13:56
dansmithmriedem: or were you just spoon-feeding me?13:56
mriedemmeaning already merged in stein13:56
mriedemyes, pureed stable food13:57
mriedemartom: +213:57
mriedemyaawang: thanks for testing https://review.opendev.org/#/c/684409/13:57
artomyaawang, yep, much appreciated :)13:57
*** dave-mccowan has quit IRC13:58
*** mlavalle has joined #openstack-nova13:59
*** ratailor has joined #openstack-nova14:02
*** eharney has quit IRC14:04
*** pcaruana has joined #openstack-nova14:04
efriedstephenfin: done. Do you have any paste savvy?14:05
efrieddansmith: or you?14:05
stephenfinvery little, unfortunately14:05
dansmithnot really, no14:05
dansmithI'm always mystified by that stuff14:05
efrieddamn and blast14:06
dansmithwhat is the question?14:07
efriedgoogle seems clueless as well14:07
efrieddansmith: this https://review.opendev.org/#/c/685181/14:07
dansmithis this a thing we want to do?14:08
efriedtldr version discovery endpoints shouldn't require auth, but I don't know how to make /v2.1 use a different paste composite from /v2.1/$everything_else without enumerating $everything_else14:08
dansmithlike, should I be able to probe vexxhost to figure out what version it's running with no credentials?14:08
efriedabsolutely14:08
efriedthe root discovery documents (everywhere) are already unauth'd14:08
efriedbut at least nova, cinder, and neutron are broken in this same way, requiring auth for the versioned discovery docs (which are simply subsets of the root one)14:09
dansmithsure, but the microversion available indicates what level of code is running14:09
efriedThe version discovery process that ksa uses (currently) assumes you can probe both the root and the versioned discovery URIs without auth.14:10
dansmithack, it just seems like getting into actual details of the system should require auth, but I'm not an expert obviously14:11
efriedyeah, it's not really details of the system14:11
efriedIn theory I agree it would seem as though that process could just rely on the root discovery doc, since it's just a superset of the versioned ones.14:11
efriedbut I think perhaps the problem is that the service catalog is set up with the versioned endpoint14:12
dansmithavailable microversion is totally details of the system14:12
*** luksky has joined #openstack-nova14:12
efriedokay, point is we've already decided that's okay to expose, because it's in the root discovery doc (I think, /me confirms...)14:12
efriedyeah:14:13
efriedcurl http://192.168.218.28/compute/14:13
efried<snip> "version": "2.79" <snip>14:13
*** TxGirlGeek has joined #openstack-nova14:13
efried"perhaps the problem is that the service catalog is set up with the versioned endpoint" shouldn't actually be a problem, which is the point.14:13
efriedmordred is working on a hacked up workaround for this in ksa and/or sdk to bridge the gap, but we (nova and the other services running afoul of this) really need to fix it in our APIs.14:14
*** luksky has quit IRC14:17
*** gbarros has joined #openstack-nova14:19
*** munimeha1 has joined #openstack-nova14:23
*** tkajinam has quit IRC14:24
openstackgerritStephen Finucane proposed openstack/nova master: ec2: Move ec2utils functions to their callers  https://review.opendev.org/66250514:27
openstackgerritStephen Finucane proposed openstack/nova master: api: Remove 'Debug' middleware  https://review.opendev.org/66250614:27
*** lpetrut has quit IRC14:33
*** jaosorior has joined #openstack-nova14:34
*** pcaruana has quit IRC14:35
efriedkashyap: thank you for the thorough response on the ML ++14:38
*** lpetrut has joined #openstack-nova14:39
kashyapefried: No problem.  I'll let Manuel file the bug14:39
*** _mmethot_ has quit IRC14:50
*** ratailor has quit IRC14:52
*** BjoernT has quit IRC14:53
*** mrch_ has quit IRC14:56
*** BjoernT has joined #openstack-nova14:56
*** cdent has joined #openstack-nova14:58
*** ivve has quit IRC14:58
*** lpetrut has quit IRC15:08
openstackgerritEric Fried proposed openstack/nova master: test cleanup: Remove skipIf test decorator  https://review.opendev.org/68537515:10
*** mmethot has joined #openstack-nova15:11
*** dtantsur is now known as dtantsur|afk15:20
*** arxcruz|rover is now known as arxcruz|zzz15:22
*** damien_r has quit IRC15:28
*** artom has quit IRC15:28
*** gyee has joined #openstack-nova15:30
*** mriedem has quit IRC15:32
efrieddansmith: are you aware of a reason we don't use oslotest.base.BaseTestCase in nova?15:33
efriedor stephenfin ^15:34
stephenfinI'm not sure15:34
stephenfinI'd guess just legacy15:34
stephenfinnova is olllld15:34
dansmithnova pre-dates oslo, so yeah something like that15:34
efriedcool, Ima go clean that up.15:36
*** pcaruana has joined #openstack-nova15:36
dansmithis there a reason?15:37
efriedtech debt reduction?15:37
dansmithwhat's the tech debt here?15:38
efriedWe have copies of half a dozen fixtures and are duplicating the setup steps for them.15:38
*** rchurch has quit IRC15:38
efriedOutput capture, logging, timeout, tempdir...15:38
dansmithwe have copies of those in our fixtures?15:38
*** rchurch has joined #openstack-nova15:38
efriedyes15:38
dansmithokay, the actual oslotest base doesn't seem to have much in it, but copies fo the fixtures make sense to squash15:39
*** mriedem has joined #openstack-nova15:40
mriedemdansmith: this is a fix for a race in the numa live migration series (that came up in the ML), probably good to get in for an rc2 if we have one https://review.opendev.org/#/c/684409/15:41
dansmithmriedem: hrm, do we call that method from anywhere else other than the periodic?15:43
dansmith"accepted" could be considered to be non-terminal and thus "in progress"15:43
dansmithoh, I see it's removing accepted15:44
mriedemright, the ~ is confusing,15:44
mriedemit's only used by the RT,15:44
mriedemthere is another method below that is used by the API15:44
mriedemmigration_get_in_progress_by_instance15:44
mriedemthat will still filter out 'accepted' state migrations15:44
dansmithokay, so this is querying for finished migrations we need to clean up15:44
mriedemwhich?15:45
mriedemmigration_get_in_progress_by_instance was written for the 2.23 change in the api to list in-progress live migrations that can be cancelled for force-completed15:45
dansmiththe one we're changing15:45
dansmithyeah, I mean the host-n-node one here15:46
mriedemthe one we're changing is used by the RT to track incoming/outgoing migrations so we claim/don't drop resource claims on the source/dest host15:46
dansmithright15:46
dansmithso we need to backport that to stable/train to be ready for rc2?15:47
*** rpittau is now known as rpittau|afk15:48
mriedemyeah, i haven't checked yet if we have a stable/train set of patches up, but we probably do15:48
* mriedem looks15:48
dansmiththe branch is there15:49
mriedemyar https://review.opendev.org/#/q/project:openstack/nova+status:open+branch:stable/train15:49
*** ttsiouts has quit IRC15:49
openstackgerritDan Smith proposed openstack/nova stable/train: Stop filtering out 'accepted' for in-progress migrations  https://review.opendev.org/68538715:50
dansmithmriedem: ^15:50
dansmithbiab15:52
openstackgerritEric Fried proposed openstack/nova master: test cleanup: Use oslotest's Timeout fixture  https://review.opendev.org/68538915:56
efrieddansmith: there's one for e.g. ^15:56
*** artom has joined #openstack-nova15:57
dansmithack15:58
mriedemhuh, interestingly enough, we don't set a migration.status during _post_live_migration before just marking the migration 'completed', meaning a user could attempt to abort/force-complete a live migration after the guest is transferred and we're in post-processing15:58
mriedemit will probably fail, but we shouldn't even make that an option15:58
artomdansmith, thanks for the train backport (sorry, laptop was rebooting/installing updates)15:58
* artom -> lunch15:59
openstackgerritMerged openstack/python-novaclient master: Stop silently ignoring invalid 'nova boot --hint' options  https://review.opendev.org/68476216:00
dansmithartom: np16:01
mriedemefried: to make launchpad sane, i've updated https://launchpad.net/nova/train to be in pre-release freeze status and ussuri is active development16:01
efriedthanks16:01
mriedeme.g. technically https://launchpad.net/bugs/1845146 will be fixed in ussuri but not yet fixed in train16:01
openstackLaunchpad bug 1845146 in OpenStack Compute (nova) train "NUMA aware live migration failed when vCPU pin set" [High,In progress] - Assigned to Artom Lifshitz (notartom)16:01
mriedemartom: when you're back you might sanity check my comments on https://review.opendev.org/#/c/684409/1/nova/db/sqlalchemy/api.py to see if we have a (latent) bug to follow up on16:04
openstackgerritMatthew Booth proposed openstack/nova master: libvirt: Ignore DiskNotFound during update_available_resource  https://review.opendev.org/68539116:05
openstackgerritMatthew Booth proposed openstack/nova master: libvirt: Ignore DiskNotFound during update_available_resource  https://review.opendev.org/68539116:06
openstackgerritEric Fried proposed openstack/nova master: test cleanup: Use oslotest's CaptureOutput fixture  https://review.opendev.org/68539216:07
openstackgerritMatt Riedemann proposed openstack/nova stable/train: Stop filtering out 'accepted' for in-progress migrations  https://review.opendev.org/68538716:17
openstackgerritMerged openstack/nova master: ec2: Remove ec2.CloudController  https://review.opendev.org/66250316:18
* stephenfin goes to airport. Have a good weekend o/16:18
*** jaosorior has quit IRC16:21
openstackgerritMerged openstack/nova master: ec2: Pre-move cleanup of utils  https://review.opendev.org/66250416:22
*** tesseract has quit IRC16:32
openstackgerritEric Fried proposed openstack/nova master: test cleanup: Make base TestCase subclass oslotest  https://review.opendev.org/68539916:33
efrieddansmith, stephenfin: I think that's the last one ^16:34
efriedstephenfin: note that this conflicts (both git-wise and spiritually) with your patch to remove those output-capturing env vars.16:34
*** zbr|ruck is now known as zbr16:41
*** cdent has quit IRC16:42
*** dpawlik has joined #openstack-nova16:42
*** derekh has quit IRC17:01
*** mrjk_ has quit IRC17:08
*** igordc has joined #openstack-nova17:09
*** pcaruana has quit IRC17:23
*** itlinux has joined #openstack-nova17:39
openstackgerritMerged openstack/nova stable/rocky: Fix rebuild of baremetal instance when vm_state is ERROR  https://review.opendev.org/68087117:48
openstackgerritMerged openstack/nova stable/rocky: Fix 'has_calls' method calls in unit tests  https://review.opendev.org/67733017:48
openstackgerritMerged openstack/nova stable/rocky: Fix wrong assertions in unit tests  https://review.opendev.org/67738917:48
*** janki has quit IRC17:49
*** markvoelker has quit IRC18:03
*** dpawlik has quit IRC18:06
openstackgerritMerged openstack/nova stable/rocky: Retrun 400 if invalid query parameters are specified  https://review.opendev.org/67602718:08
openstackgerritMerged openstack/nova stable/rocky: Fix non-existent method of Mock  https://review.opendev.org/67684018:08
openstackgerritMerged openstack/nova master: Update master for stable/train  https://review.opendev.org/68525818:08
openstackgerritMerged openstack/nova master: Add TODO note for mox removal  https://review.opendev.org/57675818:08
*** markvoelker has joined #openstack-nova18:08
*** mriedem has quit IRC18:12
*** mriedem has joined #openstack-nova18:14
*** tbachman has quit IRC18:37
*** KeithMnemonic has joined #openstack-nova18:41
*** maciejjozefczyk has quit IRC18:42
*** gbarros has quit IRC18:48
*** tbachman has joined #openstack-nova18:52
*** davee__ has quit IRC18:55
*** davee__ has joined #openstack-nova18:55
*** itlinux has quit IRC19:01
*** ricolin has quit IRC19:03
openstackgerritMerged openstack/nova master: Add functional tests for virtual persistent memory  https://review.opendev.org/67847019:11
*** tbachman has quit IRC19:11
*** ralonsoh has quit IRC19:16
*** KeithMnemonic has quit IRC19:23
*** jamesdenton has joined #openstack-nova19:25
*** tbachman has joined #openstack-nova19:43
*** igordc has quit IRC19:52
*** TxGirlGeek has quit IRC19:52
*** nweinber_ has quit IRC19:54
*** yonglihe has quit IRC19:54
openstackgerritEric Fried proposed openstack/nova master: Allow versioned discovery unauthenticated  https://review.opendev.org/68518120:07
efriedI think I did it dansmith stephenfin ^20:07
*** tbachman has quit IRC20:08
*** tbachman has joined #openstack-nova20:08
*** mvkr has quit IRC20:11
*** tbachman has quit IRC20:12
*** maciejjozefczyk has joined #openstack-nova20:22
*** igordc has joined #openstack-nova20:23
*** mvkr has joined #openstack-nova20:25
*** maciejjozefczyk has quit IRC20:29
*** luksky has joined #openstack-nova20:40
openstackgerritMerged openstack/nova master: nova-net: Migrate some API sample tests off of nova-net  https://review.opendev.org/68432520:45
openstackgerritMerged openstack/nova stable/train: Update .gitreview for stable/train  https://review.opendev.org/68525620:45
*** TxGirlGeek has joined #openstack-nova20:46
openstackgerritMerged openstack/nova stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68525720:48
mriedemBLARG20:55
mriedemmelwitt: fun fact,20:55
mriedemcinder's db api code to get a volume filters on project_id https://github.com/openstack/cinder/blob/33b32d982094a9d85dabd737cb1d634b3164c187/cinder/db/sqlalchemy/api.py#L189320:55
mriedemso if you're an admin with a different project trying to get the volume it's filtered out if that volume is in another project20:56
mriedemi know you're familiar with that20:56
mriedemidk how the hell volume show as admin works then...20:59
mriedemah because in devstack the admin and service projects have different roles, and the cinder user doesn't have the admin role21:21
mriedemhttps://review.opendev.org/68548821:21
efriedmriedem: I see a route for /versions/{id} but it's not documented. Does that mean I can rip it tf out?21:25
mriedemprobably not21:26
mriedemlack of documentation or wrong documentation doesn't mean we can just remove things from the api since people in the wild might be using them,21:26
mriedemand historically our api docs have sucked21:27
*** ociuhandu has joined #openstack-nova21:27
efriedFINE21:29
mriedemsorry broheem!21:29
efriedmriedem: I think I figured out that paste thing I was asking about yesterday btw21:29
efriedhttps://review.opendev.org/68518121:30
mriedemi'm not really your guy for that, but alex_xu and/or johnthetubaguy might be21:30
mriedemif either ever show up again21:30
efriedyah21:30
mriedemjust tell stephen that removing paste and using flask is on the table for ussuri21:31
efriedFunny, I was talking to lbragstad about this earlier and he showed me how they did that for keystone.21:31
mriedemor whatever wsgi middleware thing is cool these days21:31
mriedemwhat is neutron using now? pecan?21:31
efriedHe said it took them two releases. I said I can't imagine how long that means it would take nova21:31
mriedemoh neutron is still using paste21:32
efriedum, I thought I saw the same paste.ini snafu in neutron as we've got in nova...21:32
mriedemi thought there was an effort there to move to pecan years ago21:32
efriedyeah, I was gonna say that may mean they're off of it and just haven't removed paste.ini yet.21:32
efriedthat was part of keystone's process too.21:32
mriedemidk when glance got off paste21:32
mriedemremoving paste form nova likely means breaking out of tree extension stuff and making a lot of people unhappy21:32
mriedemi was just joking about doing it21:32
*** ociuhandu has quit IRC21:33
efriedI'm not laughing, but also realistic that it's never gonna happen.21:33
mriedemROI21:34
efriedroight.21:34
mriedemdude, just submit it as a cycle goal21:34
mriedemto de-paste openstack21:34
efriedooo21:34
mriedempeople will just sign up to do it21:34
*** munimeha1 has quit IRC21:35
lbragstadlol21:35
efriedsee ^21:36
efriedlbragstad is just itching to rip paste out of nova21:36
mriedemno that's just a rash21:36
mriedemhe's rashy21:37
mriedemdiaper wise21:37
efriedrazor burn21:37
mriedem"rashy lance" we used to call him at ibm21:37
mriedemwith all of that helpfulness, i'm signing off, o/21:37
efriedo/21:37
*** mriedem has quit IRC21:37
*** dave-mccowan has joined #openstack-nova21:39
*** BjoernT has quit IRC21:40
openstackgerritEric Fried proposed openstack/nova-specs master: Re-proposed Nova Cyborg interaction specification.  https://review.opendev.org/68263721:43
*** zhubx has quit IRC21:57
*** igordc has quit IRC22:01
*** igordc has joined #openstack-nova22:01
*** mmethot has quit IRC22:04
*** bbowen has quit IRC22:06
*** bbowen has joined #openstack-nova22:06
*** mtreinish has quit IRC22:07
openstackgerritMerged openstack/nova-specs master: Re-proposed Nova Cyborg interaction specification.  https://review.opendev.org/68263722:08
*** mtreinish has joined #openstack-nova22:10
*** igordc has quit IRC22:17
*** rcernin has joined #openstack-nova22:24
*** igordc has joined #openstack-nova22:29
*** xek has quit IRC22:43
*** rcernin has quit IRC22:48
*** slaweq has quit IRC22:50
openstackgerritMerged openstack/nova stable/rocky: lxc: make use of filter python3 compatible  https://review.opendev.org/67649823:00
*** igordc has quit IRC23:32
*** igordc has joined #openstack-nova23:34
*** igordc has quit IRC23:38
*** mmethot has joined #openstack-nova23:47
*** mmethot has quit IRC23:57

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!