*** sapd1_x has joined #openstack-nova | 00:05 | |
*** gyee has quit IRC | 00:05 | |
openstackgerrit | Merged openstack/nova master: Reset forced_destination before migration at a proper time https://review.opendev.org/684546 | 00:07 |
---|---|---|
*** sapd1_x has quit IRC | 00:14 | |
*** BjoernT has joined #openstack-nova | 00:14 | |
*** KeithMnemonic has quit IRC | 00:18 | |
*** markvoelker has joined #openstack-nova | 00:29 | |
*** markvoelker has quit IRC | 00:33 | |
*** gbarros has joined #openstack-nova | 00:38 | |
*** gbarros has quit IRC | 00:42 | |
*** mkrai has joined #openstack-nova | 00:49 | |
*** markvoelker has joined #openstack-nova | 01:06 | |
*** Liang__ has joined #openstack-nova | 01:13 | |
*** francoisp has quit IRC | 01:28 | |
*** mkrai has quit IRC | 01:37 | |
*** HagunKim has joined #openstack-nova | 01:39 | |
HagunKim | Hello, 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 |
HagunKim | But 'openstack hypervisor show [Compute node]' shows local_gb as old disk capacity. | 01:45 |
HagunKim | How can I update it? | 01:45 |
HagunKim | Is there problem when I modify DB(nova.compute_nodes.local_gb) directly? | 01:45 |
*** mkrai has joined #openstack-nova | 02:13 | |
*** markvoelker has quit IRC | 02:25 | |
openstackgerrit | melanie witt proposed openstack/nova master: Reduce scope of 'path' query parameter to noVNC consoles https://review.opendev.org/685194 | 02:25 |
*** BjoernT has quit IRC | 02:38 | |
*** artom has quit IRC | 02:42 | |
*** ricolin has joined #openstack-nova | 02:49 | |
*** mkrai has quit IRC | 02:51 | |
*** dave-mccowan has quit IRC | 03:24 | |
*** brinzhang_ has joined #openstack-nova | 03:47 | |
*** brinzhang has quit IRC | 03:50 | |
*** boxiang has quit IRC | 04:06 | |
*** boxiang has joined #openstack-nova | 04:06 | |
*** igordc has quit IRC | 04:08 | |
*** ratailor has joined #openstack-nova | 04:14 | |
*** mkrai has joined #openstack-nova | 04:17 | |
*** markvoelker has joined #openstack-nova | 04:26 | |
*** janki has joined #openstack-nova | 04:26 | |
*** markvoelker has quit IRC | 04:30 | |
*** ratailor has quit IRC | 04:42 | |
*** ratailor has joined #openstack-nova | 04:42 | |
*** tkajinam has quit IRC | 05:00 | |
*** udesale has joined #openstack-nova | 05:00 | |
*** tkajinam has joined #openstack-nova | 05:02 | |
*** dpawlik has joined #openstack-nova | 05:16 | |
*** brinzhang has joined #openstack-nova | 05:29 | |
*** ociuhandu has joined #openstack-nova | 05:31 | |
*** brinzhang_ has quit IRC | 05:32 | |
*** ociuhandu has quit IRC | 05:35 | |
*** tkajinam_ has joined #openstack-nova | 05:36 | |
*** tkajinam has quit IRC | 05:38 | |
*** Luzi has joined #openstack-nova | 05:42 | |
*** ratailor has quit IRC | 05:43 | |
*** ratailor has joined #openstack-nova | 05:44 | |
*** maciejjozefczyk has joined #openstack-nova | 06:03 | |
*** ratailor_ has joined #openstack-nova | 06:09 | |
*** ratailor has quit IRC | 06:11 | |
*** Guest29634 has quit IRC | 06:16 | |
*** slaweq has joined #openstack-nova | 06:27 | |
*** rcernin has quit IRC | 06:29 | |
*** benj has joined #openstack-nova | 06:32 | |
*** benj is now known as Guest57861 | 06:32 | |
*** Luzi has quit IRC | 06:40 | |
*** larainema has joined #openstack-nova | 06:49 | |
*** Luzi has joined #openstack-nova | 06:56 | |
*** damien_r has joined #openstack-nova | 06:58 | |
*** damien_r has quit IRC | 06:59 | |
*** damien_r has joined #openstack-nova | 07:00 | |
*** rpittau|afk is now known as rpittau | 07:05 | |
*** ratailor_ has quit IRC | 07:05 | |
*** ratailor has joined #openstack-nova | 07:08 | |
*** tesseract has joined #openstack-nova | 07:20 | |
*** pcaruana has joined #openstack-nova | 07:21 | |
*** ociuhandu has joined #openstack-nova | 07:27 | |
*** ociuhandu has quit IRC | 07:28 | |
*** luksky has joined #openstack-nova | 07:35 | |
*** yan0s has joined #openstack-nova | 07:37 | |
*** priteau has joined #openstack-nova | 07:43 | |
*** eharney has joined #openstack-nova | 07:46 | |
*** jaosorior has quit IRC | 07:48 | |
*** ralonsoh has joined #openstack-nova | 07:57 | |
*** ociuhandu has joined #openstack-nova | 08:02 | |
*** ttsiouts has joined #openstack-nova | 08:04 | |
*** ociuhandu has quit IRC | 08:07 | |
*** priteau has quit IRC | 08:09 | |
*** lpetrut has joined #openstack-nova | 08:13 | |
*** tkajinam_ has quit IRC | 08:16 | |
openstackgerrit | Seyeong Kim proposed openstack/nova master: block_device: Optionally recreate attachments when refreshing connection_info https://review.opendev.org/579004 | 08:21 |
openstackgerrit | Yongli He proposed openstack/nova master: clean up orphan instances https://review.opendev.org/627765 | 08:22 |
*** counterpart has joined #openstack-nova | 08:30 | |
*** counterpart has left #openstack-nova | 08:30 | |
*** ivve has joined #openstack-nova | 08:30 | |
*** derekh has joined #openstack-nova | 08:33 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Disable neutron explicitly for 'ServersSampleBase' subclasses https://review.opendev.org/684330 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove *even* more unnecessary 'USE_NEUTRON = False' https://review.opendev.org/684331 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_availability_zone' functional tests https://review.opendev.org/684332 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ip_pools' functional tests https://review.opendev.org/684333 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_quota_sets' functional tests https://review.opendev.org/684334 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_server_tags' functional tests https://review.opendev.org/684335 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_servers' functional tests https://review.opendev.org/684336 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hosts' functional tests https://review.opendev.org/684337 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_networks_associate' functional tests https://review.opendev.org/684338 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_rescue' functional tests https://review.opendev.org/684339 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hypervisors' functional tests https://review.opendev.org/684340 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_attach_interfaces' functional tests https://review.opendev.org/684341 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_simple_tenant_usage' functional tests https://review.opendev.org/684342 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Neutron fixture cleanup https://review.opendev.org/684343 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ips' functional tests https://review.opendev.org/684344 | 08:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Add TODOs for remaining nova-network functional tests https://review.opendev.org/684345 | 08:42 |
*** ratailor has quit IRC | 08:48 | |
*** ratailor has joined #openstack-nova | 08:51 | |
*** pcaruana has quit IRC | 08:52 | |
gibi | stephenfin: hi! do you have oppinion about his oslo.messaging bugfix ? https://review.opendev.org/#/c/683600/ | 08:53 |
*** rcernin has joined #openstack-nova | 08:53 | |
*** pcaruana has joined #openstack-nova | 09:03 | |
openstackgerrit | OpenStack Release Bot proposed openstack/nova stable/train: Update .gitreview for stable/train https://review.opendev.org/685256 | 09:06 |
openstackgerrit | OpenStack Release Bot proposed openstack/nova stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train https://review.opendev.org/685257 | 09:06 |
openstackgerrit | OpenStack Release Bot proposed openstack/nova master: Update master for stable/train https://review.opendev.org/685258 | 09:06 |
openstackgerrit | Luyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470 | 09:14 |
*** dtantsur|afk is now known as dtantsur | 09:18 | |
openstackgerrit | Merged openstack/nova master: docs: Note use of 'nova-manage db sync --config-file' https://review.opendev.org/671298 | 09:18 |
*** rpittau is now known as rpittau|bbl | 09:24 | |
*** ricolin has quit IRC | 09:24 | |
*** dtantsur is now known as dtantsur|lunch | 09:24 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: conf: Remove deprecated 'project_id_regex' opt https://review.opendev.org/501057 | 09:26 |
openstackgerrit | Merged openstack/nova master: docs: Correct 'nova-manage db sync' documentation https://review.opendev.org/677508 | 09:28 |
openstackgerrit | Merged openstack/nova master: docs: Rework the PCI passthrough guides https://review.opendev.org/635243 | 09:28 |
*** mkrai has quit IRC | 09:32 | |
*** Liang__ has quit IRC | 09:39 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: conf: Remove deprecated 'project_id_regex' opt https://review.opendev.org/501057 | 09:41 |
*** ttsiouts has quit IRC | 09:41 | |
*** ttsiouts has joined #openstack-nova | 09:42 | |
stephenfin | gibi: 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. +2d | 09:44 |
gibi | stephenfin: thanks. It is a pretty old bug | 09:44 |
*** Guest57861 has quit IRC | 09:44 | |
gibi | stephenfin: with lost history | 09:44 |
*** ttsiouts has quit IRC | 09:46 | |
*** benj has joined #openstack-nova | 09:47 | |
*** benj is now known as Guest94083 | 09:48 | |
stephenfin | Shouldn't we have a stable/train branch now? | 09:48 |
stephenfin | Oh, wait, we do | 09:48 |
* stephenfin should fetch first | 09:48 | |
* gibi 's mirror seems to be lagging | 09:49 | |
*** mkrai has joined #openstack-nova | 09:53 | |
stephenfin | This means I can merge all the mox removal patches now, I assume? | 09:58 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: conf: Remove deprecated 'project_id_regex' opt https://review.opendev.org/501057 | 09:59 |
takashin | stephenfin: 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 |
stephenfin | takashin: look two lines up :) | 10:04 |
stephenfin | I'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 first | 10:04 |
takashin | Okay. | 10:04 |
*** ratailor has quit IRC | 10:11 | |
gibi | stephenfin: does these patches makes backporting potential rc bugs harder? | 10:11 |
stephenfin | gibi: 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 cycle | 10:14 |
stephenfin | So the chances of there being bugs introduced that we'd have to fix in the file are low | 10:14 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Use common 'command' definition for unit tests https://review.opendev.org/684774 | 10:21 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Stop overriding the 'install_command' https://review.opendev.org/684775 | 10:21 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Nuke OS_STDOUT_CAPTURE, OS_STDERR_CAPTURE https://review.opendev.org/684773 | 10:21 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Start using global stdout, stderr buffers https://review.opendev.org/685310 | 10:21 |
*** brinzhang has quit IRC | 10:25 | |
*** xek has quit IRC | 10:31 | |
*** ratailor has joined #openstack-nova | 10:31 | |
*** xek has joined #openstack-nova | 10:31 | |
*** cdent has joined #openstack-nova | 10:31 | |
*** takashin has left #openstack-nova | 10:33 | |
*** tesseract has quit IRC | 10:35 | |
*** tesseract has joined #openstack-nova | 10:35 | |
*** HagunKim has quit IRC | 10:38 | |
gibi | stephenfin: cool. if you need a second core to push them through then ping me | 10:44 |
*** ratailor_ has joined #openstack-nova | 10:45 | |
*** ratailor has quit IRC | 10:47 | |
stephenfin | gibi: 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 IRC | 10:49 | |
*** boxiang has quit IRC | 10:59 | |
*** zhubx has joined #openstack-nova | 10:59 | |
*** dtantsur|lunch is now known as dtantsur | 11:03 | |
*** mkrai has quit IRC | 11:13 | |
*** mkrai has joined #openstack-nova | 11:13 | |
*** luksky has joined #openstack-nova | 11:15 | |
*** jaosorior has joined #openstack-nova | 11:19 | |
*** ttsiouts has joined #openstack-nova | 11:23 | |
*** jaosorior has quit IRC | 11:33 | |
*** cdent has quit IRC | 11:34 | |
*** jhesketh has quit IRC | 11:34 | |
*** jhesketh has joined #openstack-nova | 11:35 | |
*** mkrai has quit IRC | 11:46 | |
*** ociuhandu has joined #openstack-nova | 11:47 | |
*** ratailor__ has joined #openstack-nova | 11:49 | |
*** ratailor_ has quit IRC | 11:51 | |
*** Luzi has quit IRC | 11:52 | |
efried | stephenfin: re mox, go for it. | 11:53 |
* stephenfin goes for it | 11:53 | |
efried | and 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 |
efried | Is the bp already approved for ussuri? I think I did that. /me looks... | 11:54 |
efried | nope, lemme hit that... | 11:54 |
efried | oh, I did but forgot to no-op update the series goal | 11:55 |
efried | done now | 11:55 |
*** mkrai has joined #openstack-nova | 11:55 | |
efried | kashyap: Could http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009743.html be https://review.opendev.org/#/c/684375/ ? | 11:58 |
* kashyap clicks | 11:59 | |
kashyap | efried: 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 |
efried | kashyap: sweet. Can you respond? | 12:00 |
kashyap | efried: I'll reply there; first to try with kernel 4.19 | 12:00 |
efried | beaut | 12:00 |
efried | thank you | 12:00 |
kashyap | efried: Will do; seems like it _just_ arrived? | 12:01 |
efried | yeah, <5h ago | 12:01 |
efried | serendipity | 12:01 |
kashyap | Ah, missed it; as I was reading only the [nova] tag | 12:01 |
kashyap | (And this didn't have the tag) | 12:02 |
efried | Couldn't tell you why I actually read this one. Maybe because it said "numa affinity"? | 12:02 |
kashyap | Likely | 12:02 |
*** markvoelker has joined #openstack-nova | 12:02 | |
kashyap | I don't know if we should make it into yet-another tunable; I'd think we should do the _right_ thing there. Still mulling | 12:02 |
*** markvoelker has quit IRC | 12:03 | |
efried | but 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 |
kashyap | efried: That's my theory. Let's see if this reporter confirms. | 12:03 |
efried | is there not a case where you would truly *want* to use strict? | 12:03 |
kashyap | Also 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 |
efried | seems like something something edge HPC realtime something | 12:03 |
kashyap | efried: There are cases, but I'll double-check w/ the libvirt dev who meditates on this | 12:04 |
*** cdent has joined #openstack-nova | 12:04 | |
efried | cool. /me watches from a safe distance | 12:05 |
* kashyap is only NUMA-surface-level-aware | 12:05 | |
kashyap | Heh | 12:05 |
openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: Stop silently ignoring invalid 'nova boot --hint' options https://review.opendev.org/684762 | 12:06 |
*** zul has quit IRC | 12:06 | |
*** maciejjozefczyk has quit IRC | 12:08 | |
*** rchurch has quit IRC | 12:08 | |
*** manjeets has quit IRC | 12:08 | |
*** maciejjozefczyk has joined #openstack-nova | 12:08 | |
kashyap | efried: Completely unrelated - I wonder if all the 'no-reply' emails can be directed to another 'openstack-releases' list | 12:08 |
kashyap | Otherwise the noise feels just too much. | 12:08 |
kashyap | Maybe I should ask it on the list | 12:08 |
*** rchurch has joined #openstack-nova | 12:09 | |
kashyap | I 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 IRC | 12:09 | |
efried | kashyap: respond to http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009744.html | 12:09 |
*** dpawlik has joined #openstack-nova | 12:09 | |
kashyap | Ah, will do | 12:09 |
kashyap | Thanks for the pointer | 12:10 |
*** bnemec has quit IRC | 12:10 | |
efried | yahyoubetcha. 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 IRC | 12:11 | |
*** bnemec has joined #openstack-nova | 12:11 | |
kashyap | My gripe is, the mailing list archives turn into a toilet | 12:11 |
kashyap | It just becomes so polluted that the signal is drowned in that noise. | 12:12 |
kashyap | And it shoos away those who're reading the list to ignore it. I'll respond on that thread. :D | 12:12 |
*** mvkr has joined #openstack-nova | 12:13 | |
*** markvoelker has joined #openstack-nova | 12:15 | |
*** dpawlik has quit IRC | 12:16 | |
zigo | Is Nova really broken with cryptography 2.6 ? | 12:22 |
zigo | That'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-nova | 12:27 | |
*** ratailor__ has quit IRC | 12:30 | |
*** rpittau|bbl is now known as rpittau | 12:36 | |
*** ociuhandu has quit IRC | 12:37 | |
*** ociuhandu has joined #openstack-nova | 12:41 | |
*** dpawlik has joined #openstack-nova | 12:42 | |
zigo | bauzas: ^ Do you know ? | 12:45 |
openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13) https://review.opendev.org/576020 | 12:46 |
openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14) https://review.opendev.org/576027 | 12:46 |
openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.opendev.org/576031 | 12:46 |
openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.opendev.org/576299 | 12:46 |
openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.opendev.org/576344 | 12:46 |
openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.opendev.org/576673 | 12:47 |
openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.opendev.org/576676 | 12:47 |
openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.opendev.org/576689 | 12:47 |
openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.opendev.org/576709 | 12:47 |
*** jangutter has quit IRC | 12:47 | |
openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.opendev.org/576712 | 12:47 |
*** nweinber_ has joined #openstack-nova | 12:51 | |
*** dpawlik has quit IRC | 12:53 | |
*** nweinber_ has quit IRC | 12:53 | |
*** nweinber_ has joined #openstack-nova | 12:54 | |
*** mkrai has quit IRC | 12:55 | |
*** dpawlik has joined #openstack-nova | 12:56 | |
*** larainema has quit IRC | 12:57 | |
*** udesale has quit IRC | 12:58 | |
*** udesale has joined #openstack-nova | 12:58 | |
*** dpawlik has quit IRC | 13:00 | |
*** mkrai has joined #openstack-nova | 13:02 | |
*** francoisp has joined #openstack-nova | 13:03 | |
gibi | stephenfin: regarding https://bugs.launchpad.net/python-novaclient/+bug/1845322 I think the same bug exists in openstackclient as well | 13:04 |
openstack | Launchpad 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-nova | 13:05 | |
*** eharney has quit IRC | 13:07 | |
*** pcaruana has quit IRC | 13:09 | |
gibi | stephenfin: opened a story for it https://storyboard.openstack.org/#!/story/2006628 | 13:09 |
*** mriedem has joined #openstack-nova | 13:10 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add TODO note for mox removal https://review.opendev.org/576758 | 13:18 |
*** rcernin has quit IRC | 13:18 | |
*** BjoernT has joined #openstack-nova | 13:21 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova stable/pike: Avoid redundant initialize_connection on source post live migration https://review.opendev.org/683008 | 13:22 |
*** ociuhandu has quit IRC | 13:25 | |
efried | stephenfin: Is your nova-net removal series | 13:27 |
efried | ... "done"? Like, at least proposed all the way up? | 13:27 |
efried | (I haven't looked) | 13:27 |
efried | curious 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-nova | 13:29 | |
*** luksky has quit IRC | 13:29 | |
efried | ...looks like just the test bits are proposed at this point. | 13:30 |
*** ricolin has joined #openstack-nova | 13:30 | |
*** dave-mccowan has joined #openstack-nova | 13:32 | |
*** yan0s has quit IRC | 13:37 | |
*** cdent has quit IRC | 13:41 | |
*** mkrai has quit IRC | 13:41 | |
stephenfin | gibi: Yeah, I pushed patches for that | 13:45 |
stephenfin | Was trying to figure out why I couldn't file against openstackclient on launchpad. That would be why,heh | 13:46 |
*** artom has joined #openstack-nova | 13:46 | |
stephenfin | efried: Yeah, just the functional test piece pushed so far. I'm working on the rest of it slowly | 13:46 |
stephenfin | Trying to avoid a mega "remove all the things" patch | 13:47 |
efried | stephenfin: as for cellsv1, good approach. | 13:47 |
stephenfin | efried: 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 |
gibi | stephenfin: cool reading the openstackclient patch now | 13:48 |
*** yan0s has joined #openstack-nova | 13:49 | |
dansmith | mriedem: I think I'm +2 on all the +2able things in stein | 13:51 |
dansmith | mriedem: (re: your mention from the meeting yesterday) | 13:51 |
openstackgerrit | melanie witt proposed openstack/nova master: Reduce scope of 'path' query parameter to noVNC consoles https://review.opendev.org/685194 | 13:51 |
*** yan0s has quit IRC | 13:52 | |
mriedem | dansmith: ack | 13:52 |
mriedem | lyarwood: if you're about can you hit these? https://review.opendev.org/#/q/project:openstack/nova+branch:stable/stein+status:open+label:Code-Review=2 | 13:53 |
mriedem | dansmith: can you hit https://review.opendev.org/#/c/680871/ and https://review.opendev.org/#/c/676498/ for rocky? | 13:55 |
artom | mriedem, you saw ya.wang's review on that NUMA fix? | 13:55 |
dansmith | I've got the rocky set up right now, are those two more important? | 13:55 |
mriedem | more important? | 13:56 |
mriedem | the former is a long-standing issue for baremetal instances, the latter is an lxc thing which is low priority | 13:56 |
mriedem | lxc+py3 | 13:56 |
*** udesale has quit IRC | 13:56 | |
mriedem | artom: not yet | 13:56 |
dansmith | mriedem: 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 reason | 13:56 |
mriedem | artom: does it work? | 13:56 |
artom | mriedem, yeah, tl;dr, he's +1 | 13:56 |
mriedem | dansmith: oh, they're just ready | 13:56 |
dansmith | mriedem: or were you just spoon-feeding me? | 13:56 |
mriedem | meaning already merged in stein | 13:56 |
mriedem | yes, pureed stable food | 13:57 |
mriedem | artom: +2 | 13:57 |
mriedem | yaawang: thanks for testing https://review.opendev.org/#/c/684409/ | 13:57 |
artom | yaawang, yep, much appreciated :) | 13:57 |
*** dave-mccowan has quit IRC | 13:58 | |
*** mlavalle has joined #openstack-nova | 13:59 | |
*** ratailor has joined #openstack-nova | 14:02 | |
*** eharney has quit IRC | 14:04 | |
*** pcaruana has joined #openstack-nova | 14:04 | |
efried | stephenfin: done. Do you have any paste savvy? | 14:05 |
efried | dansmith: or you? | 14:05 |
stephenfin | very little, unfortunately | 14:05 |
dansmith | not really, no | 14:05 |
dansmith | I'm always mystified by that stuff | 14:05 |
efried | damn and blast | 14:06 |
dansmith | what is the question? | 14:07 |
efried | google seems clueless as well | 14:07 |
efried | dansmith: this https://review.opendev.org/#/c/685181/ | 14:07 |
dansmith | is this a thing we want to do? | 14:08 |
efried | tldr 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_else | 14:08 |
dansmith | like, should I be able to probe vexxhost to figure out what version it's running with no credentials? | 14:08 |
efried | absolutely | 14:08 |
efried | the root discovery documents (everywhere) are already unauth'd | 14:08 |
efried | but 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 |
dansmith | sure, but the microversion available indicates what level of code is running | 14:09 |
efried | The version discovery process that ksa uses (currently) assumes you can probe both the root and the versioned discovery URIs without auth. | 14:10 |
dansmith | ack, it just seems like getting into actual details of the system should require auth, but I'm not an expert obviously | 14:11 |
efried | yeah, it's not really details of the system | 14:11 |
efried | In 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 |
efried | but I think perhaps the problem is that the service catalog is set up with the versioned endpoint | 14:12 |
dansmith | available microversion is totally details of the system | 14:12 |
*** luksky has joined #openstack-nova | 14:12 | |
efried | okay, 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 |
efried | yeah: | 14:13 |
efried | curl http://192.168.218.28/compute/ | 14:13 |
efried | <snip> "version": "2.79" <snip> | 14:13 |
*** TxGirlGeek has joined #openstack-nova | 14: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 |
efried | mordred 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 IRC | 14:17 | |
*** gbarros has joined #openstack-nova | 14:19 | |
*** munimeha1 has joined #openstack-nova | 14:23 | |
*** tkajinam has quit IRC | 14:24 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: ec2: Move ec2utils functions to their callers https://review.opendev.org/662505 | 14:27 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: api: Remove 'Debug' middleware https://review.opendev.org/662506 | 14:27 |
*** lpetrut has quit IRC | 14:33 | |
*** jaosorior has joined #openstack-nova | 14:34 | |
*** pcaruana has quit IRC | 14:35 | |
efried | kashyap: thank you for the thorough response on the ML ++ | 14:38 |
*** lpetrut has joined #openstack-nova | 14:39 | |
kashyap | efried: No problem. I'll let Manuel file the bug | 14:39 |
*** _mmethot_ has quit IRC | 14:50 | |
*** ratailor has quit IRC | 14:52 | |
*** BjoernT has quit IRC | 14:53 | |
*** mrch_ has quit IRC | 14:56 | |
*** BjoernT has joined #openstack-nova | 14:56 | |
*** cdent has joined #openstack-nova | 14:58 | |
*** ivve has quit IRC | 14:58 | |
*** lpetrut has quit IRC | 15:08 | |
openstackgerrit | Eric Fried proposed openstack/nova master: test cleanup: Remove skipIf test decorator https://review.opendev.org/685375 | 15:10 |
*** mmethot has joined #openstack-nova | 15:11 | |
*** dtantsur is now known as dtantsur|afk | 15:20 | |
*** arxcruz|rover is now known as arxcruz|zzz | 15:22 | |
*** damien_r has quit IRC | 15:28 | |
*** artom has quit IRC | 15:28 | |
*** gyee has joined #openstack-nova | 15:30 | |
*** mriedem has quit IRC | 15:32 | |
efried | dansmith: are you aware of a reason we don't use oslotest.base.BaseTestCase in nova? | 15:33 |
efried | or stephenfin ^ | 15:34 |
stephenfin | I'm not sure | 15:34 |
stephenfin | I'd guess just legacy | 15:34 |
stephenfin | nova is olllld | 15:34 |
dansmith | nova pre-dates oslo, so yeah something like that | 15:34 |
efried | cool, Ima go clean that up. | 15:36 |
*** pcaruana has joined #openstack-nova | 15:36 | |
dansmith | is there a reason? | 15:37 |
efried | tech debt reduction? | 15:37 |
dansmith | what's the tech debt here? | 15:38 |
efried | We have copies of half a dozen fixtures and are duplicating the setup steps for them. | 15:38 |
*** rchurch has quit IRC | 15:38 | |
efried | Output capture, logging, timeout, tempdir... | 15:38 |
dansmith | we have copies of those in our fixtures? | 15:38 |
*** rchurch has joined #openstack-nova | 15:38 | |
efried | yes | 15:38 |
dansmith | okay, the actual oslotest base doesn't seem to have much in it, but copies fo the fixtures make sense to squash | 15:39 |
*** mriedem has joined #openstack-nova | 15:40 | |
mriedem | dansmith: 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 |
dansmith | mriedem: 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 |
dansmith | oh, I see it's removing accepted | 15:44 |
mriedem | right, the ~ is confusing, | 15:44 |
mriedem | it's only used by the RT, | 15:44 |
mriedem | there is another method below that is used by the API | 15:44 |
mriedem | migration_get_in_progress_by_instance | 15:44 |
mriedem | that will still filter out 'accepted' state migrations | 15:44 |
dansmith | okay, so this is querying for finished migrations we need to clean up | 15:44 |
mriedem | which? | 15:45 |
mriedem | migration_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-completed | 15:45 |
dansmith | the one we're changing | 15:45 |
dansmith | yeah, I mean the host-n-node one here | 15:46 |
mriedem | the 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 host | 15:46 |
dansmith | right | 15:46 |
dansmith | so we need to backport that to stable/train to be ready for rc2? | 15:47 |
*** rpittau is now known as rpittau|afk | 15:48 | |
mriedem | yeah, i haven't checked yet if we have a stable/train set of patches up, but we probably do | 15:48 |
* mriedem looks | 15:48 | |
dansmith | the branch is there | 15:49 |
mriedem | yar https://review.opendev.org/#/q/project:openstack/nova+status:open+branch:stable/train | 15:49 |
*** ttsiouts has quit IRC | 15:49 | |
openstackgerrit | Dan Smith proposed openstack/nova stable/train: Stop filtering out 'accepted' for in-progress migrations https://review.opendev.org/685387 | 15:50 |
dansmith | mriedem: ^ | 15:50 |
dansmith | biab | 15:52 |
openstackgerrit | Eric Fried proposed openstack/nova master: test cleanup: Use oslotest's Timeout fixture https://review.opendev.org/685389 | 15:56 |
efried | dansmith: there's one for e.g. ^ | 15:56 |
*** artom has joined #openstack-nova | 15:57 | |
dansmith | ack | 15:58 |
mriedem | huh, 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-processing | 15:58 |
mriedem | it will probably fail, but we shouldn't even make that an option | 15:58 |
artom | dansmith, thanks for the train backport (sorry, laptop was rebooting/installing updates) | 15:58 |
* artom -> lunch | 15:59 | |
openstackgerrit | Merged openstack/python-novaclient master: Stop silently ignoring invalid 'nova boot --hint' options https://review.opendev.org/684762 | 16:00 |
dansmith | artom: np | 16:01 |
mriedem | efried: to make launchpad sane, i've updated https://launchpad.net/nova/train to be in pre-release freeze status and ussuri is active development | 16:01 |
efried | thanks | 16:01 |
mriedem | e.g. technically https://launchpad.net/bugs/1845146 will be fixed in ussuri but not yet fixed in train | 16:01 |
openstack | Launchpad 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 |
mriedem | artom: 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 on | 16:04 |
openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Ignore DiskNotFound during update_available_resource https://review.opendev.org/685391 | 16:05 |
openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Ignore DiskNotFound during update_available_resource https://review.opendev.org/685391 | 16:06 |
openstackgerrit | Eric Fried proposed openstack/nova master: test cleanup: Use oslotest's CaptureOutput fixture https://review.opendev.org/685392 | 16:07 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/train: Stop filtering out 'accepted' for in-progress migrations https://review.opendev.org/685387 | 16:17 |
openstackgerrit | Merged openstack/nova master: ec2: Remove ec2.CloudController https://review.opendev.org/662503 | 16:18 |
* stephenfin goes to airport. Have a good weekend o/ | 16:18 | |
*** jaosorior has quit IRC | 16:21 | |
openstackgerrit | Merged openstack/nova master: ec2: Pre-move cleanup of utils https://review.opendev.org/662504 | 16:22 |
*** tesseract has quit IRC | 16:32 | |
openstackgerrit | Eric Fried proposed openstack/nova master: test cleanup: Make base TestCase subclass oslotest https://review.opendev.org/685399 | 16:33 |
efried | dansmith, stephenfin: I think that's the last one ^ | 16:34 |
efried | stephenfin: 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 zbr | 16:41 | |
*** cdent has quit IRC | 16:42 | |
*** dpawlik has joined #openstack-nova | 16:42 | |
*** derekh has quit IRC | 17:01 | |
*** mrjk_ has quit IRC | 17:08 | |
*** igordc has joined #openstack-nova | 17:09 | |
*** pcaruana has quit IRC | 17:23 | |
*** itlinux has joined #openstack-nova | 17:39 | |
openstackgerrit | Merged openstack/nova stable/rocky: Fix rebuild of baremetal instance when vm_state is ERROR https://review.opendev.org/680871 | 17:48 |
openstackgerrit | Merged openstack/nova stable/rocky: Fix 'has_calls' method calls in unit tests https://review.opendev.org/677330 | 17:48 |
openstackgerrit | Merged openstack/nova stable/rocky: Fix wrong assertions in unit tests https://review.opendev.org/677389 | 17:48 |
*** janki has quit IRC | 17:49 | |
*** markvoelker has quit IRC | 18:03 | |
*** dpawlik has quit IRC | 18:06 | |
openstackgerrit | Merged openstack/nova stable/rocky: Retrun 400 if invalid query parameters are specified https://review.opendev.org/676027 | 18:08 |
openstackgerrit | Merged openstack/nova stable/rocky: Fix non-existent method of Mock https://review.opendev.org/676840 | 18:08 |
openstackgerrit | Merged openstack/nova master: Update master for stable/train https://review.opendev.org/685258 | 18:08 |
openstackgerrit | Merged openstack/nova master: Add TODO note for mox removal https://review.opendev.org/576758 | 18:08 |
*** markvoelker has joined #openstack-nova | 18:08 | |
*** mriedem has quit IRC | 18:12 | |
*** mriedem has joined #openstack-nova | 18:14 | |
*** tbachman has quit IRC | 18:37 | |
*** KeithMnemonic has joined #openstack-nova | 18:41 | |
*** maciejjozefczyk has quit IRC | 18:42 | |
*** gbarros has quit IRC | 18:48 | |
*** tbachman has joined #openstack-nova | 18:52 | |
*** davee__ has quit IRC | 18:55 | |
*** davee__ has joined #openstack-nova | 18:55 | |
*** itlinux has quit IRC | 19:01 | |
*** ricolin has quit IRC | 19:03 | |
openstackgerrit | Merged openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470 | 19:11 |
*** tbachman has quit IRC | 19:11 | |
*** ralonsoh has quit IRC | 19:16 | |
*** KeithMnemonic has quit IRC | 19:23 | |
*** jamesdenton has joined #openstack-nova | 19:25 | |
*** tbachman has joined #openstack-nova | 19:43 | |
*** igordc has quit IRC | 19:52 | |
*** TxGirlGeek has quit IRC | 19:52 | |
*** nweinber_ has quit IRC | 19:54 | |
*** yonglihe has quit IRC | 19:54 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Allow versioned discovery unauthenticated https://review.opendev.org/685181 | 20:07 |
efried | I think I did it dansmith stephenfin ^ | 20:07 |
*** tbachman has quit IRC | 20:08 | |
*** tbachman has joined #openstack-nova | 20:08 | |
*** mvkr has quit IRC | 20:11 | |
*** tbachman has quit IRC | 20:12 | |
*** maciejjozefczyk has joined #openstack-nova | 20:22 | |
*** igordc has joined #openstack-nova | 20:23 | |
*** mvkr has joined #openstack-nova | 20:25 | |
*** maciejjozefczyk has quit IRC | 20:29 | |
*** luksky has joined #openstack-nova | 20:40 | |
openstackgerrit | Merged openstack/nova master: nova-net: Migrate some API sample tests off of nova-net https://review.opendev.org/684325 | 20:45 |
openstackgerrit | Merged openstack/nova stable/train: Update .gitreview for stable/train https://review.opendev.org/685256 | 20:45 |
*** TxGirlGeek has joined #openstack-nova | 20:46 | |
openstackgerrit | Merged openstack/nova stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train https://review.opendev.org/685257 | 20:48 |
mriedem | BLARG | 20:55 |
mriedem | melwitt: fun fact, | 20:55 |
mriedem | cinder's db api code to get a volume filters on project_id https://github.com/openstack/cinder/blob/33b32d982094a9d85dabd737cb1d634b3164c187/cinder/db/sqlalchemy/api.py#L1893 | 20:55 |
mriedem | so if you're an admin with a different project trying to get the volume it's filtered out if that volume is in another project | 20:56 |
mriedem | i know you're familiar with that | 20:56 |
mriedem | idk how the hell volume show as admin works then... | 20:59 |
mriedem | ah because in devstack the admin and service projects have different roles, and the cinder user doesn't have the admin role | 21:21 |
mriedem | https://review.opendev.org/685488 | 21:21 |
efried | mriedem: I see a route for /versions/{id} but it's not documented. Does that mean I can rip it tf out? | 21:25 |
mriedem | probably not | 21:26 |
mriedem | lack 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 |
mriedem | and historically our api docs have sucked | 21:27 |
*** ociuhandu has joined #openstack-nova | 21:27 | |
efried | FINE | 21:29 |
mriedem | sorry broheem! | 21:29 |
efried | mriedem: I think I figured out that paste thing I was asking about yesterday btw | 21:29 |
efried | https://review.opendev.org/685181 | 21:30 |
mriedem | i'm not really your guy for that, but alex_xu and/or johnthetubaguy might be | 21:30 |
mriedem | if either ever show up again | 21:30 |
efried | yah | 21:30 |
mriedem | just tell stephen that removing paste and using flask is on the table for ussuri | 21:31 |
efried | Funny, I was talking to lbragstad about this earlier and he showed me how they did that for keystone. | 21:31 |
mriedem | or whatever wsgi middleware thing is cool these days | 21:31 |
mriedem | what is neutron using now? pecan? | 21:31 |
efried | He said it took them two releases. I said I can't imagine how long that means it would take nova | 21:31 |
mriedem | oh neutron is still using paste | 21:32 |
efried | um, I thought I saw the same paste.ini snafu in neutron as we've got in nova... | 21:32 |
mriedem | i thought there was an effort there to move to pecan years ago | 21:32 |
efried | yeah, I was gonna say that may mean they're off of it and just haven't removed paste.ini yet. | 21:32 |
efried | that was part of keystone's process too. | 21:32 |
mriedem | idk when glance got off paste | 21:32 |
mriedem | removing paste form nova likely means breaking out of tree extension stuff and making a lot of people unhappy | 21:32 |
mriedem | i was just joking about doing it | 21:32 |
*** ociuhandu has quit IRC | 21:33 | |
efried | I'm not laughing, but also realistic that it's never gonna happen. | 21:33 |
mriedem | ROI | 21:34 |
efried | roight. | 21:34 |
mriedem | dude, just submit it as a cycle goal | 21:34 |
mriedem | to de-paste openstack | 21:34 |
efried | ooo | 21:34 |
mriedem | people will just sign up to do it | 21:34 |
*** munimeha1 has quit IRC | 21:35 | |
lbragstad | lol | 21:35 |
efried | see ^ | 21:36 |
efried | lbragstad is just itching to rip paste out of nova | 21:36 |
mriedem | no that's just a rash | 21:36 |
mriedem | he's rashy | 21:37 |
mriedem | diaper wise | 21:37 |
efried | razor burn | 21:37 |
mriedem | "rashy lance" we used to call him at ibm | 21:37 |
mriedem | with all of that helpfulness, i'm signing off, o/ | 21:37 |
efried | o/ | 21:37 |
*** mriedem has quit IRC | 21:37 | |
*** dave-mccowan has joined #openstack-nova | 21:39 | |
*** BjoernT has quit IRC | 21:40 | |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Re-proposed Nova Cyborg interaction specification. https://review.opendev.org/682637 | 21:43 |
*** zhubx has quit IRC | 21:57 | |
*** igordc has quit IRC | 22:01 | |
*** igordc has joined #openstack-nova | 22:01 | |
*** mmethot has quit IRC | 22:04 | |
*** bbowen has quit IRC | 22:06 | |
*** bbowen has joined #openstack-nova | 22:06 | |
*** mtreinish has quit IRC | 22:07 | |
openstackgerrit | Merged openstack/nova-specs master: Re-proposed Nova Cyborg interaction specification. https://review.opendev.org/682637 | 22:08 |
*** mtreinish has joined #openstack-nova | 22:10 | |
*** igordc has quit IRC | 22:17 | |
*** rcernin has joined #openstack-nova | 22:24 | |
*** igordc has joined #openstack-nova | 22:29 | |
*** xek has quit IRC | 22:43 | |
*** rcernin has quit IRC | 22:48 | |
*** slaweq has quit IRC | 22:50 | |
openstackgerrit | Merged openstack/nova stable/rocky: lxc: make use of filter python3 compatible https://review.opendev.org/676498 | 23:00 |
*** igordc has quit IRC | 23:32 | |
*** igordc has joined #openstack-nova | 23:34 | |
*** igordc has quit IRC | 23:38 | |
*** mmethot has joined #openstack-nova | 23:47 | |
*** mmethot has quit IRC | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!