*** gyee has quit IRC | 00:04 | |
*** ttsiouts has joined #openstack-nova | 00:10 | |
*** luksky has quit IRC | 00:14 | |
*** ttsiouts has quit IRC | 00:16 | |
*** Sundar has quit IRC | 00:22 | |
openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Set the lower limit of api_version for volume_type https://review.opendev.org/660861 | 00:25 |
---|---|---|
*** brinzhang has joined #openstack-nova | 00:29 | |
*** frank_wang has quit IRC | 00:39 | |
*** itlinux has joined #openstack-nova | 00:44 | |
*** ttsiouts has joined #openstack-nova | 00:45 | |
*** frankwang has joined #openstack-nova | 00:47 | |
*** itlinux has quit IRC | 00:52 | |
*** frankwang has quit IRC | 00:52 | |
*** frankwang has joined #openstack-nova | 00:53 | |
*** slaweq has joined #openstack-nova | 00:57 | |
*** slaweq has quit IRC | 01:01 | |
openstackgerrit | Jake Yip proposed openstack/nova master: refactor nova-manage archive_deleted_rows https://review.opendev.org/643779 | 01:07 |
openstackgerrit | Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.opendev.org/556751 | 01:08 |
*** itlinux has joined #openstack-nova | 01:10 | |
*** itlinux has quit IRC | 01:10 | |
*** ttsiouts has quit IRC | 01:18 | |
*** bbowen has quit IRC | 01:19 | |
*** guozijn has joined #openstack-nova | 01:22 | |
*** guozijn has quit IRC | 01:27 | |
*** guozijn has joined #openstack-nova | 01:27 | |
*** itlinux has joined #openstack-nova | 01:30 | |
*** whoami-rajat has joined #openstack-nova | 01:31 | |
*** guozijn_ has joined #openstack-nova | 01:34 | |
*** ricolin has joined #openstack-nova | 01:36 | |
*** guozijn has quit IRC | 01:37 | |
*** guozijn_ is now known as guozijn | 01:37 | |
*** tbachman has quit IRC | 01:40 | |
*** bbowen has joined #openstack-nova | 01:50 | |
openstackgerrit | Merged openstack/nova stable/stein: Restore connection_info after live migration rollback https://review.opendev.org/660370 | 02:06 |
*** slaweq has joined #openstack-nova | 02:11 | |
*** slaweq has quit IRC | 02:15 | |
*** ykarel|away has joined #openstack-nova | 02:20 | |
*** edleafe has quit IRC | 02:26 | |
*** irclogbot_0 has quit IRC | 02:26 | |
*** irclogbot_3 has joined #openstack-nova | 02:29 | |
*** ttsiouts has joined #openstack-nova | 02:30 | |
*** itlinux has quit IRC | 02:39 | |
openstackgerrit | Merged openstack/nova stable/rocky: xenapi/agent: Change openssl error handling https://review.opendev.org/656307 | 02:48 |
*** nicolasbock has quit IRC | 02:51 | |
*** _erlon_ has quit IRC | 02:52 | |
*** itlinux has joined #openstack-nova | 02:53 | |
*** ttsiouts has quit IRC | 03:04 | |
*** itlinux has quit IRC | 03:08 | |
openstackgerrit | Merged openstack/python-novaclient master: Allow passing negative values for the locked search_opt in cs.servers.list https://review.opendev.org/659783 | 03:13 |
*** itlinux has joined #openstack-nova | 03:35 | |
*** itlinux has quit IRC | 03:39 | |
*** itlinux has joined #openstack-nova | 03:42 | |
*** itlinux has quit IRC | 03:43 | |
*** samueldmq has quit IRC | 03:50 | |
*** yikun_ has quit IRC | 03:50 | |
*** frankwang has quit IRC | 03:59 | |
*** itlinux has joined #openstack-nova | 04:00 | |
*** itlinux has quit IRC | 04:05 | |
*** ttsiouts has joined #openstack-nova | 04:09 | |
*** itlinux has joined #openstack-nova | 04:10 | |
*** slaweq has joined #openstack-nova | 04:11 | |
*** slaweq has quit IRC | 04:16 | |
*** ykarel|away has quit IRC | 04:17 | |
openstackgerrit | Merged openstack/nova stable/stein: libvirt: Do not reraise DiskNotFound exceptions during resize https://review.opendev.org/660361 | 04:28 |
*** udesale has joined #openstack-nova | 04:28 | |
*** ykarel|away has joined #openstack-nova | 04:31 | |
*** luksky has joined #openstack-nova | 04:35 | |
*** luksky has quit IRC | 04:41 | |
*** luksky has joined #openstack-nova | 04:42 | |
*** ttsiouts has quit IRC | 04:43 | |
*** itlinux has quit IRC | 04:43 | |
*** luksky has quit IRC | 04:56 | |
*** lpetrut has joined #openstack-nova | 05:11 | |
*** ttsiouts has joined #openstack-nova | 05:12 | |
*** ratailor has joined #openstack-nova | 05:16 | |
*** ttsiouts has quit IRC | 05:16 | |
*** ykarel|away is now known as ykarel | 05:20 | |
*** ccamacho has quit IRC | 05:29 | |
*** itlinux has joined #openstack-nova | 05:30 | |
*** janki has joined #openstack-nova | 05:31 | |
*** zbr has joined #openstack-nova | 05:34 | |
*** imacdonn has quit IRC | 05:41 | |
*** imacdonn has joined #openstack-nova | 05:42 | |
*** frankwang has joined #openstack-nova | 05:43 | |
openstackgerrit | Arthur Dayne proposed openstack/nova stable/rocky: Add the missing ' in routes.py https://review.opendev.org/660885 | 05:49 |
*** ttsiouts has joined #openstack-nova | 05:49 | |
*** slaweq has joined #openstack-nova | 05:59 | |
*** Luzi has joined #openstack-nova | 06:02 | |
*** slaweq has quit IRC | 06:03 | |
*** slaweq has joined #openstack-nova | 06:11 | |
*** cfriesen has quit IRC | 06:14 | |
*** dpawlik has joined #openstack-nova | 06:16 | |
*** jchhatbar has joined #openstack-nova | 06:19 | |
*** janki has quit IRC | 06:20 | |
*** itlinux has quit IRC | 06:20 | |
*** ttsiouts has quit IRC | 06:23 | |
*** ileixe has joined #openstack-nova | 06:38 | |
ileixe | Hi forks. | 06:38 |
ileixe | I got an simple test using cpu pinning lately, and the result is somewhat unexpected. So if you are experince in the same environment, please let me know what's wrong. | 06:40 |
ileixe | My test is that VMs with cpu pinning host can interfere affect to each other. | 06:41 |
*** tesseract has joined #openstack-nova | 06:41 | |
ileixe | I assume that VM with CPU pinning can achieve their own workload though, but the fact is If nosiy VM (which is also CPU pinned) is there they affect to each other. | 06:42 |
*** tbachman has joined #openstack-nova | 06:49 | |
*** rcernin has quit IRC | 07:00 | |
*** markvoelker has joined #openstack-nova | 07:03 | |
*** tbachman has quit IRC | 07:03 | |
*** pcaruana has joined #openstack-nova | 07:07 | |
*** markvoelker has quit IRC | 07:08 | |
openstackgerrit | Merged openstack/nova stable/stein: Log when port resource is leaked during port delete https://review.opendev.org/657581 | 07:09 |
*** helenafm has joined #openstack-nova | 07:12 | |
*** rpittau|afk is now known as rpittau | 07:20 | |
*** awalende has joined #openstack-nova | 07:20 | |
*** ircuser-1 has quit IRC | 07:26 | |
*** jchhatbar has quit IRC | 07:27 | |
*** ircuser-1 has joined #openstack-nova | 07:28 | |
*** ttsiouts has joined #openstack-nova | 07:29 | |
openstackgerrit | Merged openstack/nova stable/stein: Include all network devices in nova diagnostics https://review.opendev.org/657125 | 07:30 |
openstackgerrit | Merged openstack/nova master: Update description of valid whitelist for non-admin user https://review.opendev.org/642403 | 07:30 |
openstackgerrit | Hamdy Khader proposed openstack/os-vif master: OVS DPDK port representors support https://review.opendev.org/658786 | 07:34 |
*** ccamacho has joined #openstack-nova | 07:35 | |
*** ircuser-1 has quit IRC | 07:36 | |
*** ykarel is now known as ykarel|afk | 07:37 | |
openstackgerrit | Alexandre arents proposed openstack/nova master: Fix live-migration when glance image deleted https://review.opendev.org/659054 | 07:37 |
*** ircuser-1 has joined #openstack-nova | 07:38 | |
*** ivve has joined #openstack-nova | 07:42 | |
openstackgerrit | Merged openstack/nova master: Skip existing VMs when hosts apply force_config_drive https://review.opendev.org/659703 | 07:43 |
*** janki has joined #openstack-nova | 07:44 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/stein: Skip existing VMs when hosts apply force_config_drive https://review.opendev.org/660914 | 07:44 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: Skip existing VMs when hosts apply force_config_drive https://review.opendev.org/660915 | 07:45 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: Skip existing VMs when hosts apply force_config_drive https://review.opendev.org/660917 | 07:45 |
*** tssurya has joined #openstack-nova | 07:46 | |
*** ykarel|afk is now known as ykarel | 08:03 | |
*** trident has quit IRC | 08:04 | |
*** ralonsoh has joined #openstack-nova | 08:04 | |
alex_xu | ileixe: what kind of nosiy you pointed to? | 08:05 |
*** tssurya has quit IRC | 08:05 | |
ileixe | CPU load | 08:05 |
ileixe | alex_xu: I used stress and ssl encryption | 08:05 |
ileixe | tool called 'stress' | 08:05 |
*** trident has joined #openstack-nova | 08:05 | |
alex_xu | ileixe: the vcpu and pcpu is 1:1 pinning, but in the same cpu socket, they still share cpu cache | 08:08 |
ileixe | I made them to be separated by using cpu_thread_policy | 08:08 |
ileixe | to 'require' | 08:08 |
ileixe | so I think no VMs share physical core for the test environment. | 08:09 |
ileixe | alex_xu: Have you ever expereince cpu pinning? Is it assure that VMs could not interfere each other? | 08:11 |
ileixe | (if there are only cpu loads) | 08:12 |
lyarwood | # git review | 08:13 |
lyarwood | Could not connect to gerrit. | 08:13 |
lyarwood | hmm anyone seeing this or have I just borked something locally? | 08:13 |
*** tkajinam has quit IRC | 08:14 | |
lyarwood | `RSA host key for review.openstack.org has changed and you have requested strict checking.` - nope it's remote | 08:15 |
*** ykarel is now known as ykarel|lunch | 08:16 | |
*** tobias-urdin has joined #openstack-nova | 08:19 | |
*** tssurya has joined #openstack-nova | 08:20 | |
alex_xu | ileixe: i only can suggestion you to check whether is system process running on the those pinned pcpu. and setting the emulator thread policy ensure the emulator thread not running on those pcpu. | 08:21 |
*** janki has quit IRC | 08:23 | |
ileixe | alex_xu: Hm.. I double checked libvirt configuration and PID/stats at proc interface, and it was expected. Thanks anyway. | 08:23 |
alex_xu | ileixe: np, good luck | 08:25 |
*** frankwang has quit IRC | 08:30 | |
*** frankwang has joined #openstack-nova | 08:31 | |
*** ttsiouts has quit IRC | 08:32 | |
*** panda|rover has quit IRC | 08:40 | |
*** panda has joined #openstack-nova | 08:42 | |
*** derekh has joined #openstack-nova | 08:46 | |
*** jangutter has joined #openstack-nova | 08:54 | |
*** ttsiouts has joined #openstack-nova | 09:03 | |
*** markvoelker has joined #openstack-nova | 09:04 | |
*** panda is now known as panda|rover | 09:11 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'ComputeManager._reschedule' https://review.opendev.org/660774 | 09:15 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove cells v1 parameter from 'ComputeTaskAPI.resize_instance' https://review.opendev.org/651311 | 09:15 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop passing 'kwargs' to 'rebuild_instance' https://review.opendev.org/651312 | 09:15 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop passing 'delete_type' to 'terminate_instance' https://review.opendev.org/651313 | 09:15 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: filters: Stop handling cells v1 https://review.opendev.org/651314 | 09:15 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove nova.compute.*API() shims https://review.opendev.org/660527 | 09:15 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Ensure controllers all call super https://review.opendev.org/660950 | 09:15 |
*** ricolin has quit IRC | 09:15 | |
*** luksky has joined #openstack-nova | 09:29 | |
*** ykarel|lunch is now known as ykarel | 09:34 | |
*** ttsiouts has quit IRC | 09:37 | |
*** markvoelker has quit IRC | 09:38 | |
kashyap | stephenfin: Hi | 09:49 |
kashyap | stephenfin: I want to submit this doc to one of the upstream Nova admin guides -- which guide do you suggest: https://kashyapc.fedorapeople.org/Nova-Mitigation-for-Intel-MDS-Flaws.txt | 09:50 |
stephenfin | kashyap: probably the reference guide (doc/source/reference) | 09:52 |
*** derekh has quit IRC | 09:54 | |
*** derekh has joined #openstack-nova | 09:54 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove global state from the FakeDriver https://review.opendev.org/656709 | 09:58 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enhance service restart in functional env https://review.opendev.org/512552 | 09:59 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reproduce bug 1724172 in the functional test env https://review.opendev.org/512553 | 10:02 |
openstack | bug 1724172 in OpenStack Compute (nova) "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Medium,In progress] https://launchpad.net/bugs/1724172 - Assigned to Balazs Gibizer (balazs-gibizer) | 10:02 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: cleanup evacuated instances not on hypervisor https://review.opendev.org/512623 | 10:02 |
openstackgerrit | Arnaud Morin proposed openstack/nova master: Force refresh instance network info on deletion https://review.opendev.org/660761 | 10:02 |
*** cdent has joined #openstack-nova | 10:03 | |
*** sridharg has joined #openstack-nova | 10:14 | |
*** frankwang has quit IRC | 10:15 | |
*** guozijn has quit IRC | 10:23 | |
*** guozijn has joined #openstack-nova | 10:26 | |
*** guozijn has quit IRC | 10:32 | |
*** markvoelker has joined #openstack-nova | 10:35 | |
*** sapd1_x has joined #openstack-nova | 10:37 | |
*** ttsiouts has joined #openstack-nova | 10:40 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove global state from the FakeDriver https://review.opendev.org/656709 | 10:44 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enhance service restart in functional env https://review.opendev.org/512552 | 10:44 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reproduce bug 1724172 in the functional test env https://review.opendev.org/512553 | 10:44 |
openstack | bug 1724172 in OpenStack Compute (nova) "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Medium,In progress] https://launchpad.net/bugs/1724172 - Assigned to Balazs Gibizer (balazs-gibizer) | 10:44 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: cleanup evacuated instances not on hypervisor https://review.opendev.org/512623 | 10:44 |
*** bbowen has quit IRC | 10:50 | |
*** ttsiouts has quit IRC | 10:59 | |
*** ttsiouts has joined #openstack-nova | 11:00 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Do not start nova-network in the notification func test https://review.opendev.org/660975 | 11:06 |
*** markvoelker has quit IRC | 11:08 | |
*** davidsha has joined #openstack-nova | 11:09 | |
*** nicolasbock has joined #openstack-nova | 11:11 | |
*** janki has joined #openstack-nova | 11:11 | |
*** udesale has quit IRC | 11:13 | |
*** udesale has joined #openstack-nova | 11:13 | |
*** ttsiouts has quit IRC | 11:14 | |
*** hrw has joined #openstack-nova | 11:22 | |
hrw | morning | 11:22 |
*** sridharg has quit IRC | 11:25 | |
*** dave-mccowan has joined #openstack-nova | 11:31 | |
*** eharney has quit IRC | 11:34 | |
*** jaosorior has joined #openstack-nova | 11:38 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: Use os-brick locking for volume attach and detach https://review.opendev.org/614190 | 11:38 |
openstackgerrit | Hamdy Khader proposed openstack/os-vif master: OVS DPDK port representors support https://review.opendev.org/658786 | 11:43 |
*** jaosorior has quit IRC | 11:49 | |
*** _erlon_ has joined #openstack-nova | 11:54 | |
*** bbowen has joined #openstack-nova | 11:57 | |
*** hrw has left #openstack-nova | 11:59 | |
*** panda|rover is now known as panda|rover|eat | 11:59 | |
*** markvoelker has joined #openstack-nova | 11:59 | |
openstackgerrit | Arnaud Morin proposed openstack/nova master: Use force_refresh=True on network info update https://review.opendev.org/660982 | 11:59 |
*** Alon_KS has joined #openstack-nova | 12:00 | |
*** bbowen_ has joined #openstack-nova | 12:03 | |
*** hemna has quit IRC | 12:04 | |
*** janki has quit IRC | 12:04 | |
*** janki has joined #openstack-nova | 12:05 | |
*** bbowen has quit IRC | 12:05 | |
*** dpawlik has quit IRC | 12:06 | |
*** dpawlik has joined #openstack-nova | 12:10 | |
*** dpawlik has quit IRC | 12:12 | |
*** dpawlik has joined #openstack-nova | 12:13 | |
*** edleafe has joined #openstack-nova | 12:16 | |
*** Sundar has joined #openstack-nova | 12:17 | |
*** Sundar has quit IRC | 12:17 | |
*** eharney has joined #openstack-nova | 12:18 | |
*** ttsiouts has joined #openstack-nova | 12:28 | |
*** luksky has quit IRC | 12:33 | |
*** brinzhang has quit IRC | 12:36 | |
*** brinzhang has joined #openstack-nova | 12:36 | |
*** ratailor has quit IRC | 12:47 | |
*** ykarel is now known as ykarel|afk | 12:48 | |
openstackgerrit | Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500 | 12:50 |
*** Luzi has quit IRC | 12:54 | |
*** ttsiouts has quit IRC | 12:58 | |
*** panda|rover|eat is now known as panda|rover | 13:00 | |
*** mriedem has joined #openstack-nova | 13:03 | |
*** brinzhang has quit IRC | 13:05 | |
*** dave-mccowan has quit IRC | 13:12 | |
*** ykarel|afk is now known as ykarel | 13:12 | |
*** dave-mccowan has joined #openstack-nova | 13:13 | |
kashyap | stephenfin: Thx | 13:22 |
*** luksky has joined #openstack-nova | 13:25 | |
efried | kashyap: Did you see my patch answering the "how do we support renamed arch-specific cpu flag traits" question? | 13:27 |
kashyap | efried: Afraid, not yet. I'm at a one-day conference in the Netherlands, so my connectivity is a bit spotty today. | 13:28 |
kashyap | efried: Got the URL handy? | 13:28 |
efried | kashyap: https://review.opendev.org/#/c/660515/ | 13:28 |
*** burt has joined #openstack-nova | 13:29 | |
efried | kashyap: it's all red in the CI because it depends on your os-traits patch, but the dep chain doesn't work easily | 13:29 |
efried | kashyap: But once your patch is in an os-traits release, we put that in requirements.txt/lower-constraints.txt in this patch and it should go green. | 13:29 |
efried | anyway, point is it's not super hard to make it work, and we're probably going to have to do it eventually anyway, so might as well. | 13:30 |
efried | what this means to you is: no reason to hold off "moving" those traits in your patch. | 13:30 |
*** janki has quit IRC | 13:31 | |
openstackgerrit | Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500 | 13:31 |
*** tbachman has joined #openstack-nova | 13:32 | |
aspiers | efried: ^^^ this feels like the cleanest approach to me | 13:32 |
kashyap | efried: Thank you ... "For backward compatibility reasons, we have to continue exposing those versions of those traits" :-) | 13:33 |
aspiers | efried: I'm working on something similar for open() | 13:33 |
aspiers | but for now I need to go vote and get lunch. biab | 13:33 |
kashyap | efried: So my patch, shall I go back to making it a new PS of the older PS to preserve history? (Sorry for not doing that to begin with) | 13:33 |
kashyap | Or shall I just mention it in this new one, saying: "please read here for history"? | 13:33 |
efried | I would like someone libvirt-y, like stephenfin and aspiers, to have a look at that solution and make sure it's not insane. | 13:34 |
efried | kashyap: I would like it all kept in the original patch please. | 13:34 |
* aspiers wonders if he merits being called libvirt-y | 13:34 | |
efried | aspiers: Well, you were the last one to touch the CPU_TRAITS_MAPPING anyway :) | 13:34 |
aspiers | true | 13:34 |
aspiers | reminder to self: voting and lunching means not being here :-/ | 13:35 |
kashyap | efried: Will keep it in the original (then you need to update your PS, too) | 13:35 |
efried | kashyap: Thanks. Yeah, at this point I've proven the concept; I'll just wait until your patch is merged and released and then update mine properely. | 13:35 |
efried | properly | 13:35 |
efried | (sean-k-mooney moment) | 13:35 |
kashyap | efried: Yeah, noted. Tomm I'm off the afternoon. Can get to it only next week. Since there's no urgency here, hope that's OK | 13:36 |
*** mfeder has joined #openstack-nova | 13:36 | |
*** samueldmq has joined #openstack-nova | 13:36 | |
* efried caffeinates más | 13:36 | |
efried | kashyap: Sure, that's fine, no hurry from my end | 13:36 |
kashyap | efried: Also you might want to have a gander at this (which I'll submit as a doc to Nova): https://kashyapc.fedorapeople.org/Nova-Mitigation-for-Intel-MDS-Flaws.txt | 13:36 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make nova-next archive using --before https://review.opendev.org/661002 | 13:37 |
efried | kashyap: lgtm | 13:39 |
efried | I'll have a few comments when it's up in gerrit | 13:39 |
efried | but overall clear and concise | 13:39 |
kashyap | efried: Thx; sure. Posting a variant of it now...before I forget. | 13:40 |
kashyap | Anything that improves clarity. | 13:40 |
mfeder | Hi all, | 13:42 |
mfeder | Is there any option how to execute command as another user (linux way : "sudo -u user `command`") ? | 13:42 |
mfeder | I would like to access regular-user's key pair using admin user. | 13:42 |
mfeder | E.g. | 13:42 |
mfeder | # as regular user : | 13:42 |
mfeder | `openstack keypair create kp1` | 13:42 |
mfeder | # as admin user: | 13:42 |
mfeder | `openstack keypair show kp1 ` | 13:42 |
mfeder | Thanks for any hint ! | 13:42 |
*** roukoswarf has quit IRC | 13:43 | |
mfeder | There seems to be an option in the nova API to specify a user ID for keypair requests: https://developer.openstack.org/api-ref/compute/?expanded=list-keypairs-detail,create-or-import-keypair-detail#keypairs-keypairs But I don't know how to do this via `openstack` command | 13:44 |
*** Sundar has joined #openstack-nova | 13:45 | |
*** ricolin has joined #openstack-nova | 13:49 | |
*** tbachman has quit IRC | 13:49 | |
*** awalende_ has joined #openstack-nova | 13:52 | |
frickler | mfeder: there's a patch proposed for that, so currently it isn't possible yet, see https://review.opendev.org/576507 . you may want to ask dtroyer or mordred in #openstack-sdks for help with getting this merged and released | 13:55 |
mfeder | frickler: Thank you ! | 13:56 |
*** awalende has quit IRC | 13:57 | |
*** awalende_ has quit IRC | 13:57 | |
*** ttsiouts has joined #openstack-nova | 14:02 | |
*** mmethot has joined #openstack-nova | 14:05 | |
*** mfeder has left #openstack-nova | 14:05 | |
openstackgerrit | Merged openstack/nova stable/stein: Skip _exclude_nested_providers() if not nested https://review.opendev.org/659206 | 14:09 |
*** pcaruana has quit IRC | 14:10 | |
*** sridharg has joined #openstack-nova | 14:12 | |
*** mlavalle has joined #openstack-nova | 14:12 | |
*** tbachman has joined #openstack-nova | 14:16 | |
*** cmart has joined #openstack-nova | 14:21 | |
*** dpawlik has quit IRC | 14:27 | |
*** pcaruana has joined #openstack-nova | 14:29 | |
*** yankcrime has quit IRC | 14:30 | |
*** hongbin has joined #openstack-nova | 14:33 | |
*** ttsiouts has quit IRC | 14:36 | |
mordred | frickler: wow. that's 10 months old | 14:36 |
*** READ10 has joined #openstack-nova | 14:36 | |
frickler | mordred: yeah, progress in osc land is ... not fast | 14:37 |
efried | aspiers: +2, some tweaks if you feel so inclined. | 14:37 |
openstackgerrit | Merged openstack/python-novaclient master: Set the lower limit of api_version for volume_type https://review.opendev.org/660861 | 14:38 |
mriedem | get'ya simple perf optimizations hey! https://review.opendev.org/#/c/651647/ | 14:43 |
*** dave-mccowan has quit IRC | 14:52 | |
*** dave-mccowan has joined #openstack-nova | 14:52 | |
openstackgerrit | Brin Zhang proposed openstack/python-novaclient stable/stein: Set the lower limit of api_version for volume_type https://review.opendev.org/661024 | 14:54 |
*** sridharg has quit IRC | 14:56 | |
openstackgerrit | Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500 | 14:57 |
aspiers | efried: ^^^ | 14:57 |
aspiers | doh | 14:58 |
aspiers | docstring error | 14:58 |
openstackgerrit | Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500 | 14:58 |
efried | yeah, you could even remove the word 'static' entirely to convey the meaning. | 14:59 |
efried | as the call doesn't change regardless | 14:59 |
aspiers | true | 14:59 |
aspiers | well, it says "static method" now | 14:59 |
aspiers | that should be good enough | 14:59 |
efried | yup | 14:59 |
efried | stephenfin: easy and fun if you're so inclined --^ | 15:00 |
aspiers | I hadn't previously realised that @contextlib.contextmanager gives you something which can be both a decorator and a context manager. Pretty nice | 15:00 |
*** cfriesen has joined #openstack-nova | 15:02 | |
efried | I didn't know that either. TIL | 15:05 |
*** ivve has quit IRC | 15:06 | |
*** lpetrut has quit IRC | 15:08 | |
*** luksky has quit IRC | 15:12 | |
aspiers | this is comically wrong https://www.stackalytics.com/?metric=commits&release=train&project_type=all&module=nova | 15:13 |
aspiers | efried: if there were only 16 commits so far this release, I guess you'd be feeling a lot more relaxed :) | 15:13 |
aspiers | I've already reported the issue, BTW | 15:14 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Avoid unnecessary joins in InstanceGroup.get_hosts https://review.opendev.org/661032 | 15:20 |
mriedem | another easy optimization ^ | 15:20 |
*** hongbin has quit IRC | 15:20 | |
*** hongbin has joined #openstack-nova | 15:21 | |
*** ccamacho has quit IRC | 15:22 | |
efried | aspiers: Yup, saw the thread, thanks for reporting. | 15:26 |
*** itlinux has joined #openstack-nova | 15:26 | |
mriedem | mlavalle: sean-k-mooney: btw i fixed the revert cross-cell cold migration port binding thing from the other day - activating the source host binding before deleting the dest host binding did the trick as expected | 15:26 |
mlavalle | mriedem: good to know :-) | 15:29 |
*** priteau has joined #openstack-nova | 15:31 | |
*** helenafm has quit IRC | 15:31 | |
*** tssurya has quit IRC | 15:35 | |
sean-k-mooney | mriedem: cool good to know. i need to add that review serise back to my review queue | 15:35 |
sean-k-mooney | mriedem:are you are only using the mulitple port binding flow for cross cell by the way or will it now be ussed for all resizes/cold migrations too | 15:36 |
mriedem | the former | 15:36 |
sean-k-mooney | ok | 15:36 |
sean-k-mooney | i guess we can expand it after in U to non cross cell migration too if it makes sense | 15:37 |
*** macza has joined #openstack-nova | 15:38 | |
*** ykarel is now known as ykarel|away | 15:39 | |
*** wwriverrat has quit IRC | 15:40 | |
*** boxiang has joined #openstack-nova | 15:43 | |
*** ttsiouts has joined #openstack-nova | 15:43 | |
*** hamzy has quit IRC | 15:44 | |
*** ttsiouts has quit IRC | 15:45 | |
*** ttsiouts has joined #openstack-nova | 15:46 | |
openstackgerrit | Merged openstack/nova-specs master: Train Cycle Themes https://review.opendev.org/657171 | 15:46 |
*** ykarel|away has quit IRC | 15:46 | |
*** whoami-rajat has quit IRC | 15:50 | |
*** rpittau is now known as rpittau|afk | 15:51 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: check job status for VIR_DOMAIN_EVENT_SUSPENDED_MIGRATED event https://review.opendev.org/594527 | 15:58 |
*** itlinux has quit IRC | 16:00 | |
*** boxiang has quit IRC | 16:01 | |
*** davidsha has quit IRC | 16:05 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Enable cross-cell resize in the nova-multi-cell job https://review.opendev.org/656656 | 16:06 |
*** ykarel|away has joined #openstack-nova | 16:08 | |
*** pcaruana has quit IRC | 16:09 | |
*** cdent has quit IRC | 16:15 | |
*** gyee has joined #openstack-nova | 16:16 | |
*** ttsiouts has quit IRC | 16:16 | |
*** ivve has joined #openstack-nova | 16:23 | |
*** dpawlik has joined #openstack-nova | 16:25 | |
stephenfin | efried: I have a "remove-nova-consoleauth specless blueprint" topic on the agenda for the team meeting today but I won't be around (it's 10pm my time). If you've any questions, stick my nick in there and I'll revisit tomorrow | 16:34 |
stephenfin | (*remove-consoleauth, fwiw) | 16:36 |
*** mgoddard has quit IRC | 16:38 | |
*** mgoddard has joined #openstack-nova | 16:39 | |
efried | stephenfin: roger that. | 16:43 |
efried | main question was, have we gotten the all-clear from... whoever we thought might care? | 16:43 |
mriedem | consoleauth != console | 16:44 |
mriedem | the latter is xen specific | 16:44 |
mriedem | the former is not | 16:44 |
efried | o | 16:44 |
stephenfin | what mriedem said | 16:45 |
*** zbr has quit IRC | 16:45 | |
stephenfin | the only reason to keep consoleauth around was cells v1 so that should deem it no longer necessary | 16:46 |
stephenfin | nova-console and nova-xvpvncproxy and on my chopping block too though but later, after nove-network | 16:46 |
stephenfin | remove allll the code | 16:46 |
* stephenfin -> 🏡 | 16:46 | |
mriedem | there is also a migration involved in the consoleauth stuff, need to determine if that is done before we remove it, using a blocker migration or something perhaps, | 16:47 |
mriedem | i know we have an upgrade check but not sure if it fails or warns | 16:47 |
*** ttsiouts has joined #openstack-nova | 16:48 | |
melwitt | it warns. and I don't think there's a migration about it, just an upgrade status check to advise whether to enable the [workarounds] config option or not | 16:49 |
*** whoami-rajat has joined #openstack-nova | 16:51 | |
mriedem | ok so workarounds.enable_consoleauth is already deprecated, that's good at least | 16:52 |
*** ttsiouts has quit IRC | 16:53 | |
mriedem | would we change the upgrade check from warning to failure? | 16:53 |
mriedem | it only warns if there are non-deleted, enabled nova-consoleauth service records in any cell and there are computes < rocky | 16:54 |
*** dtantsur is now known as dtantsur|afk | 16:54 | |
*** sapd1_x has quit IRC | 16:54 | |
mriedem | the latter shouldn't be the case by the time someone upgrades to train | 16:55 |
*** ricolin has quit IRC | 16:55 | |
*** tjgresha has quit IRC | 16:55 | |
*** tjgresha has joined #openstack-nova | 16:55 | |
*** derekh has quit IRC | 17:00 | |
*** pcaruana has joined #openstack-nova | 17:00 | |
*** ykarel|away has quit IRC | 17:02 | |
*** udesale has quit IRC | 17:03 | |
melwitt | I guess if it would fail for the combo of both service records and computes < rocky, that seems like it would be ok. I wouldn't be sure about failing it for the service records alone, because I'm not sure operators definitely deleted the services after they stopped running nova-consoleauth | 17:07 |
aspiers | efried: I've got the patch_open() helper ready now - should I submit that separately to switching stuff over to it? | 17:13 |
efried | aspiers: I don't see why | 17:13 |
aspiers | so, altogether then? | 17:14 |
efried | If you've identified instances where it can be used, yeah, fo sho. | 17:14 |
aspiers | there's a whole bunch of other stuff which could be converted to use patch_exists() too | 17:14 |
efried | okay | 17:14 |
efried | up to you, I'm fine either way. | 17:14 |
aspiers | hah, just found some other reimplementations | 17:15 |
aspiers | https://opendev.org/openstack/nova/src/branch/master/nova/tests/unit/virt/disk/mount/test_nbd.py | 17:17 |
*** dpawlik has quit IRC | 17:17 | |
aspiers | but those conditions are more specific | 17:17 |
aspiers | hmm | 17:17 |
efried | one of them is. the other could be swapped out as it stands. | 17:19 |
efried | You could juice up patch_exists so the path arg can be a str or a regex :) | 17:20 |
efried | that would satisfy the other one. | 17:21 |
efried | oh, no it wouldn't. | 17:22 |
efried | aspiers: You could make it a str or a callable... | 17:22 |
efried | starts to smell a lot like side_effect at that point | 17:22 |
*** ykarel has joined #openstack-nova | 17:24 | |
*** ttsiouts has joined #openstack-nova | 17:26 | |
*** dpawlik has joined #openstack-nova | 17:26 | |
*** hemna has joined #openstack-nova | 17:26 | |
aspiers | efried: yeah, I was thinking of supporting callables, but ... scope creep, I dunno | 17:27 |
*** hamzy has joined #openstack-nova | 17:28 | |
efried | aspiers: Feel free to push it to the wishlist for later. Add a `TODO(aspiers): figure out how to use patch_exists here` and move on. | 17:31 |
openstackgerrit | Merged openstack/nova stable/stein: Handle placement error during re-schedule https://review.opendev.org/657602 | 17:35 |
openstackgerrit | Merged openstack/nova stable/stein: Only call _fill_provider_mapping if claim succeeds https://review.opendev.org/657603 | 17:35 |
aspiers | efried: good idea | 17:36 |
*** hamzy has quit IRC | 17:40 | |
*** hamzy has joined #openstack-nova | 17:40 | |
openstackgerrit | Merged openstack/nova stable/stein: Stop logging traceback when skipping quiesce https://review.opendev.org/660325 | 17:42 |
*** hamzy_ has joined #openstack-nova | 17:46 | |
*** itlinux has joined #openstack-nova | 17:46 | |
*** hamzy has quit IRC | 17:49 | |
*** ralonsoh has quit IRC | 17:51 | |
*** itlinux has quit IRC | 17:56 | |
*** ttsiouts has quit IRC | 17:59 | |
*** lpetrut has joined #openstack-nova | 18:03 | |
*** lpetrut has quit IRC | 18:07 | |
*** itlinux has joined #openstack-nova | 18:09 | |
*** itlinux has quit IRC | 18:11 | |
*** READ10 has quit IRC | 18:13 | |
*** klindgren has quit IRC | 18:13 | |
*** klindgren has joined #openstack-nova | 18:14 | |
*** luksky has joined #openstack-nova | 18:20 | |
*** hamzy__ has joined #openstack-nova | 18:26 | |
*** hamzy_ has quit IRC | 18:26 | |
ganso | lyarwood: Hi! Could you please take a look at this patch? https://review.opendev.org/#/c/659338 Thanks in advance! | 18:32 |
*** ykarel has quit IRC | 18:32 | |
*** hamzy__ has quit IRC | 18:34 | |
*** mriedem has quit IRC | 18:37 | |
*** hamzy has joined #openstack-nova | 18:43 | |
*** mriedem has joined #openstack-nova | 18:43 | |
openstackgerrit | melanie witt proposed openstack/nova master: Count instances from mappings and cores/ram from placement https://review.opendev.org/638073 | 18:43 |
openstackgerrit | melanie witt proposed openstack/nova master: Set [quota]count_usage_from_placement = True in nova-next https://review.opendev.org/653146 | 18:43 |
openstackgerrit | melanie witt proposed openstack/nova master: Use instance mappings to count server group members https://review.opendev.org/638324 | 18:43 |
openstackgerrit | melanie witt proposed openstack/nova master: Add documentation for counting quota usage from placement https://review.opendev.org/653845 | 18:43 |
*** priteau has quit IRC | 18:44 | |
*** hamzy has quit IRC | 18:47 | |
*** hamzy has joined #openstack-nova | 18:47 | |
*** tasker has joined #openstack-nova | 19:02 | |
tasker | I'm running Pike and trying to turn the logging level down to WARNING ( instead of the default INFO ), and "verbose = False" no longer works. The documentation at pike/admin/manage-logs.html and pike/configuration/config.html provide non-matching information. | 19:04 |
tasker | one ( admin ) says to use "log-config" while the other ( configuration ) says "log-config" is deprecated and that all other log items will be ignored if the new "log_config_append" is used. | 19:04 |
tasker | what's the easiest way to turn Nova's output down to WARNING? | 19:04 |
*** hamzy has quit IRC | 19:05 | |
*** tesseract has quit IRC | 19:05 | |
*** ttsiouts has joined #openstack-nova | 19:09 | |
*** aram1s has joined #openstack-nova | 19:10 | |
*** hemna has quit IRC | 19:10 | |
*** hamzy has joined #openstack-nova | 19:26 | |
*** bbowen_ has quit IRC | 19:27 | |
sean-k-mooney | efried: do you know what the policy on deprecating parts of config vaules is | 19:34 |
sean-k-mooney | efried: i want to deprecate and remove one of the ways of whitelisting device in the pci_white_list this cycle | 19:34 |
sean-k-mooney | efried: specificly the devicename parmater | 19:34 |
efried | that's interesting. | 19:35 |
sean-k-mooney | would it need a blueprint and or spec? or jsut a bug ? | 19:35 |
efried | For something like that, you would just have to note the deprecation in the help text of the opt and in a reno; and add a warning log to the code that parses it. | 19:35 |
efried | oh, you mean *paperwork* | 19:35 |
efried | depends, is there a bug? :) | 19:36 |
sean-k-mooney | ya so we woudl need to keep it around for train and log a warning if its used | 19:36 |
sean-k-mooney | not for this but there are bugs caused by that parmaater | 19:36 |
efried | I would think so, yes. | 19:36 |
sean-k-mooney | the issue is if your kernel is configured not to bind back the PFs/VFs to the nic driver e.g. they stay boudn to vfio | 19:37 |
sean-k-mooney | then if you restart the compute agent it cant dicover the VFs or PFs again | 19:37 |
sean-k-mooney | because the PF/VF does not have a netdev and therfore you cant find it by name | 19:38 |
efried | And this is the best way to "fix" the bug? Deprecating/removing the ability to specify a device by name? | 19:38 |
sean-k-mooney | but also the name can somethime change after it rebound | 19:38 |
sean-k-mooney | efried: ya basically | 19:38 |
efried | are there not scenarios where identifying a device by name *doesn't* break? | 19:38 |
efried | (s/not// for same meaning with fewer negatives) | 19:39 |
*** tssurya has joined #openstack-nova | 19:39 | |
sean-k-mooney | it only works if the device is reliable bound back to the nic driver when nolonger used by the guest and the name does not change whne it rebound to the host | 19:39 |
sean-k-mooney | so it can work but its really fragile | 19:39 |
sean-k-mooney | using vendor id and product id or useing the pci address are both more robost as they dont depned on teh driver or the semanitc of the nic vendor | 19:40 |
efried | sean-k-mooney: I guess what I would do in this case is write the patch and start a thread on the ML with your intent, soliciting opinions on whether this is okay to kill. Link the patch and ML thread from each other. IMO that's sufficient paperwork. And if someone thinks otherwise, you can always write a bug/bp later. | 19:40 |
efried | though it sounds like what you're talking about actually *is* a bug, and this is just one possible remedy. | 19:41 |
sean-k-mooney | efried: ok sound good. | 19:41 |
efried | In my ignorance, another remedy could be "blow up if devname is used for devices it has the potential to break on" | 19:41 |
sean-k-mooney | it depend on how you configre your driver at the kernel levle | 19:42 |
sean-k-mooney | and to some extent the nic vendor | 19:42 |
efried | no way to detect it, other than going all the way through until you hit the bug?? | 19:42 |
*** ttsiouts has quit IRC | 19:42 | |
sean-k-mooney | i belive intel nics prefer the networkign driver over vfio-pci but i think it depend on the model to some extend and your udev rules | 19:42 |
sean-k-mooney | efried: the main issue comes down to will udev/systemd chose to bind the driver to i40e or leave it bound to vfio-pci | 19:43 |
sean-k-mooney | so its not really a nova issue | 19:44 |
sean-k-mooney | its more a distro/packaging/install/confituion thing on the host | 19:44 |
sean-k-mooney | which is what make it hard for nova to dectet and or warn about | 19:45 |
efried | tasker: Does | 19:45 |
efried | [DEFAULT] | 19:45 |
efried | default_log_levels = nova=WARNING | 19:45 |
efried | not work? | 19:45 |
efried | sean-k-mooney: ack, and shrug. I hope we can soon deprecate *all* of [pci]passthrough_whitelist and [pci]alias. | 19:46 |
sean-k-mooney | efried: maybe in U | 19:46 |
sean-k-mooney | of course then we woudl have a cyborg whitelist and device profile | 19:46 |
sean-k-mooney | but then its there problem :) | 19:46 |
efried | heh | 19:46 |
sean-k-mooney | if we get the cyborg integration done fully this cycle that might be a nice thing to port over early next cycle | 19:47 |
* sean-k-mooney that is partly a comment to my self but i also dont want to sign up to it yet :) | 19:48 | |
efried | sean-k-mooney: cyborg/fpga + providers.yaml we might have all we need. | 19:48 |
sean-k-mooney | well i was thinig more generic pci passhtough rather then smart nic | 19:48 |
sean-k-mooney | e.g. replaceing all the uses of the pci tracker in nova with a very basic generic pci driver in cyborg | 19:49 |
sean-k-mooney | but making that work with hardare offload ovs and all the other legacy things we have would be a lot of work | 19:50 |
sean-k-mooney | manyly on the nova side to make it transparent | 19:50 |
sean-k-mooney | efried: anyway thanks ill send an email to the list | 19:51 |
efried | coo | 19:51 |
tasker | efried: did not. I still see a bunch of "HEAD / => generated 0 bytes" | 20:02 |
efried | tasker: Coming from nova? | 20:02 |
tasker | yes. specifically the "nova-api-os-compute" container ( if that distinction matters. ) | 20:03 |
efried | tasker: Can you paste me the entire log line please? | 20:04 |
efried | I'm thinking that might be coming from the wsgi layer | 20:04 |
tasker | http://dpaste.com/2C5P2PY | 20:04 |
tasker | I concur. to complicate things, I don't know how to turn that thing down, either. | 20:05 |
efried | in which case you may wish to try adding ,eventlet.wsgi.server=WARNING | 20:05 |
efried | I think the left-hand-side of that '=' is a substring match. So if you say foo.bar=WARNING then log lines coming from foo.bar and foo.bar.baz and foo.bar.etc will all get filtered at a WARNING level. | 20:06 |
efried | hmph, no indication in that line where it's coming from. Is there a log line template that will add that? Looking... | 20:08 |
efried | tasker: Looks like you might be able to add %(pathname)s to your logging_default_format_string to get a clue where that line is being generated from; then use the prefix trick above to suppress it. | 20:09 |
tasker | efried: did not work. "%(pathname)s" was applied to another set of messages, but not the one I'm trying to filter out. | 20:17 |
efried | interesting | 20:18 |
tasker | wait .. scratch that. it wasn't applied at all. | 20:18 |
*** hemna has joined #openstack-nova | 20:18 | |
efried | assume you're restarting services each time | 20:19 |
tasker | yup. | 20:19 |
tasker | http://dpaste.com/1ZSB78D -- what the "wsgi" logs look like from "nova-api-wsgi.log" when all testing options are removed. this is the original behaviour. | 20:19 |
efried | guess you could try adding the pathname to logging_debug_format_suffix | 20:20 |
tasker | when I set anything to the config file, the lines in nova-api-wsgi.log stop, but not to nova-api-os-compute.log | 20:20 |
tasker | the latter file contains lines of the first paste I sent you. | 20:20 |
efried | hmph. I'm afraid I'm out of ideas. | 20:21 |
efried | Course, I'm not what you'd call an "operator". | 20:22 |
tasker | hehe. can we bring back "verbose = False"? | 20:22 |
tasker | efried: I appreciate you taking the time to help me out. You've shed some light on some things I did not know and given me a lot of good information to play around with. | 20:23 |
efried | you're welcome, sorry I couldn't be more help. | 20:23 |
tasker | I did match the "pid" in the log file to the PIDs of the uWSGI helpers that spawn whenever the nova service starts. | 20:30 |
tasker | that lends weight to it being the WSGI module(s). | 20:30 |
tasker | are the modules using the same file independent of Nova? | 20:31 |
*** whoami-rajat has quit IRC | 20:31 | |
*** aram1s has quit IRC | 20:31 | |
efried | tasker: You're out of my wheelhouse at this point. Throwing it around earlier like I knew what I was talking about, but I can barely spell WSGI. | 20:32 |
tasker | it's alright. those were mostly musings. I tend to "stream of conciousness" debug and use any open chat window to get ideas out. | 20:33 |
efried | heh, wfm. | 20:33 |
efried | tasker: I've got a guy who might know a bit more. He's on his way into the channel, if you're going to hang around for a bit. | 20:34 |
*** donnyd has joined #openstack-nova | 20:34 | |
tasker | yeah. thanks! | 20:34 |
*** dpawlik has quit IRC | 20:34 | |
efried | o/ donnyd | 20:34 |
efried | donnyd: meet tasker. tasker: likewise | 20:35 |
donnyd | Hi | 20:35 |
efried | donnyd has actual experience deploying openstack | 20:35 |
tasker | donnyd: helo! | 20:35 |
donnyd | I'm just getting caught up on what the issue is | 20:35 |
tasker | trying to turn down all logging in Nova to WARNING because I'm getting these messages that indicate the API endpoints are hit at ( what I guess is INFO ) level. efried was helping and about all we can do is determine that they're coming from WSGI. | 20:36 |
donnyd | Do you have an example of the log entry you don't want to see? | 20:37 |
*** itlinux has joined #openstack-nova | 20:37 | |
tasker | http://dpaste.com/2C5P2PY | 20:37 |
tasker | this is Pike, and prior to it, I could set "verbose = False" in nova.conf and call it a day. | 20:39 |
*** bbowen has joined #openstack-nova | 20:39 | |
donnyd | Next question is what deployment method did you use? | 20:39 |
tasker | OSA | 20:39 |
*** slaweq has quit IRC | 20:41 | |
donnyd | That surely looks like a wsgi log to me. (I see the same log entry type in my ister server). | 20:42 |
tasker | know if there's a comparable "verbose = False" flag for uWSGI? | 20:43 |
tasker | their docs don't hint at such. | 20:43 |
*** pcaruana has quit IRC | 20:47 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Link versioned notification talk into docs https://review.opendev.org/661115 | 20:50 |
tasker | appending "--disable-logging" to the uwsgi ExecStart string in the service file killed the messages, but that's too heavy-handed. | 20:53 |
donnyd | Still searching | 20:55 |
*** ttsiouts has joined #openstack-nova | 20:56 | |
tasker | and having "--disable-logging" and having Nova at "debug_log_levels = nova=INFO | 20:56 |
tasker | " I can still get Nova's notices of API hits. | 20:56 |
openstackgerrit | Merged openstack/nova master: refactor nova-manage archive_deleted_rows https://review.opendev.org/643779 | 20:57 |
tasker | I'm stepping out for some "fresh air". back in a few. | 20:58 |
efried | Nova meeting in 2 minutes in #openstack-meeting | 20:58 |
*** nicolasbock has quit IRC | 20:58 | |
*** nicolasbock has joined #openstack-nova | 20:59 | |
efried | pawing through uwsgi docs, I'm not actually finding any hints that they *have* different log levels. | 21:00 |
efried | I think it's just on or off. | 21:00 |
efried | in which case your heavy-handed solution might be the only solution, beyond `grep` :P | 21:01 |
donnyd | Thats what it looks like to me @efried | 21:01 |
efried | tasker: Is there anything in nova-api-os-compute.log that you actually care about? | 21:02 |
openstackgerrit | melanie witt proposed openstack/nova stable/rocky: [stable-only] Improve the reno for consoleauth upgrade check https://review.opendev.org/661118 | 21:03 |
*** Sundar has quit IRC | 21:06 | |
tasker | efried: errors and warnings. . ) what are you thinking? | 21:07 |
efried | tasker: I was just wondering whether you could filter the whole thing out :) | 21:09 |
openstackgerrit | Merged openstack/nova master: Use InstanceList.get_count_by_hosts when deleting a compute service https://review.opendev.org/651647 | 21:10 |
donnyd | https://github.com/unbit/uwsgi-docs/blob/master/ManagementFlag.rst | 21:10 |
*** itlinux has quit IRC | 21:10 | |
donnyd | 0loggingenable/disable logging | 21:10 |
donnyd | Looks like its mostly an off or on kinda thing | 21:10 |
*** itlinux has joined #openstack-nova | 21:11 | |
tasker | efried: donnyd: thank you both very much for your assistance. with your help, I think I have a solution that will work. I'll be leaving it overnight and trying to trigger various warnings and events to verify it does what I need. | 21:15 |
efried | good luck tasker | 21:15 |
*** Sundar has joined #openstack-nova | 21:18 | |
*** eharney has quit IRC | 21:20 | |
*** ttsiouts has quit IRC | 21:26 | |
sean-k-mooney | on the nova console auth topic kolla-ansibel still defualt to deploying it on master | 21:34 |
efried | whee | 21:35 |
openstackgerrit | melanie witt proposed openstack/nova stable/stein: [stable-only] Improve the reno for consoleauth upgrade check https://review.opendev.org/661123 | 21:37 |
sean-k-mooney | osa looks like id does not but i only quickly looked | 21:38 |
openstackgerrit | melanie witt proposed openstack/nova stable/rocky: [stable-only] Improve the reno for consoleauth upgrade check https://review.opendev.org/661118 | 21:39 |
sean-k-mooney | and tripleo remvoed it 14 days ago https://github.com/openstack/tripleo-heat-templates/blob/c33f8cb1174146e3cbdca6241914a78a623ea505/releasenotes/notes/nova_remove_nova-consoleauth-227cbeb44e9ab5ef.yaml | 21:39 |
melwitt | nice detective work :) | 21:41 |
*** ttsiouts has joined #openstack-nova | 21:42 | |
sean-k-mooney | melwitt: what was the console auth service used for with cells v1 again | 21:42 |
melwitt | all console token auth. for cells v1 the database backend stuff was _not_ implemented | 21:43 |
sean-k-mooney | ah right so its jsust issueing the tokens to connect to the websockets? | 21:44 |
mriedem | osa is done https://github.com/openstack/openstack-ansible-os_nova/commit/1df0bd51ee5031039d3852cefbb63162e29b22a5 | 21:44 |
melwitt | sean-k-mooney: console auth service? yeah it stores the token auths locally and responds yea/nay when a request to connect to console proxy comes in | 21:45 |
melwitt | and the change we did for non-cells v1 was store the token auths in the cell databases and have console proxies (websockets) run per cell so they can query the database to validate token auths | 21:46 |
sean-k-mooney | melwitt ok so looking at the patches to remove it in osa and triplo if we just dont deploy it will it work with nova today? | 21:46 |
sean-k-mooney | i dont see any config cahnge in those patches so it seam like nova just does the right thing now? | 21:47 |
melwitt | sean-k-mooney: yeah, the default behavior is to use the database as the console token auth storage. so running it is not necessary. unless you're running cells v1 | 21:47 |
sean-k-mooney | which kolla never supported | 21:47 |
sean-k-mooney | so i can write a quick patch to delete the container form kolla and remove it form the kolla-ansibel playbook if that is all thats needed | 21:48 |
sean-k-mooney | well the upgade playbook would also need to stop any running containers | 21:48 |
sean-k-mooney | but that seams trivial too | 21:49 |
melwitt | I don't know if this is relevant at this point but during a live, rolling upgrade from queens to rocky, you would need to keep nova-consoleauth running until everything is rocky | 21:50 |
melwitt | if you were not OK with disrupting existing console sessions | 21:50 |
sean-k-mooney | looks like https://bugs.launchpad.net/kolla-ansible/+bug/1804474 is open but i am gessing its not been worked on | 21:50 |
openstack | Launchpad bug 1804474 in kolla-ansible "nova-consoleauth is deprecated" [Undecided,In progress] - Assigned to caoyuan (cao-yuan) | 21:50 |
sean-k-mooney | melwitt: well if i just remove it in the train branch that should not be an issue right | 21:51 |
melwitt | yeah should not be | 21:51 |
sean-k-mooney | e.g. going form stien to train it wont be used anyway | 21:51 |
sean-k-mooney | and it woudl be safe to backprot technically to rock->stien too | 21:51 |
sean-k-mooney | ... that said im not ment to be starting new things until i finish some of my curent in flight tasks. | 21:53 |
* sean-k-mooney shakes fist at kanban board :) | 21:53 | |
melwitt | hah | 21:53 |
*** tasker has quit IRC | 21:57 | |
*** ivve has quit IRC | 21:59 | |
*** Sundar has quit IRC | 22:12 | |
*** mriedem has quit IRC | 22:13 | |
openstackgerrit | Merged openstack/nova master: Do not start nova-network in the notification func test https://review.opendev.org/660975 | 22:30 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Link versioned notification talk into docs https://review.opendev.org/661115 | 22:34 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Enable cross-cell resize in the nova-multi-cell job https://review.opendev.org/656656 | 22:36 |
*** mlavalle has quit IRC | 22:47 | |
*** tkajinam has joined #openstack-nova | 22:57 | |
*** panda|rover has quit IRC | 22:59 | |
*** panda has joined #openstack-nova | 22:59 | |
*** macza has quit IRC | 23:03 | |
*** itlinux has quit IRC | 23:04 | |
*** sapd1_x has joined #openstack-nova | 23:06 | |
*** rcernin has joined #openstack-nova | 23:18 | |
*** _erlon_ has quit IRC | 23:24 | |
*** itlinux has joined #openstack-nova | 23:27 | |
*** tssurya has quit IRC | 23:38 | |
*** ttsiouts has quit IRC | 23:42 | |
*** trident has quit IRC | 23:51 | |
*** hongbin has quit IRC | 23:52 | |
*** trident has joined #openstack-nova | 23:53 | |
openstackgerrit | Merged openstack/python-novaclient stable/stein: Set the lower limit of api_version for volume_type https://review.opendev.org/661024 | 23:57 |
*** ttsiouts has joined #openstack-nova | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!