| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-05-22 | |||
| 09:02:15 | stephenfin | lyarwood: Ta. It reads rather oddly though. Can you take one more swing at it (comments left)? | |
| 09:04:51 | lyarwood | stephenfin: yeah of course | |
| 09:07:16 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/572790 | |
| 09:07:21 | lyarwood | stephenfin: words-- ^ there you go. | |
| 09:09:04 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/572790 | |
| 09:09:17 | stephenfin | Thanks | |
| 09:09:38 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Keep attach_mode as top-level field in _translate_attachment_ref https://review.opendev.org/574413 | |
| 09:10:44 | lyarwood | haha fs, sorry | |
| 09:50:11 | openstackgerrit | Boxiang Zhu proposed openstack/nova-specs master: Update "add host and hypervisor_hostname flag to create server" spec https://review.opendev.org/660560 | |
| 09:50:48 | stephenfin | bauzas: Wanna help remove nearly 13,000 lines of dead code? :) https://review.opendev.org/#/q/topic:bp/remove-cells-v1+status:open (ignore the ones in merge conflict, obviously) | |
| 10:01:53 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop handling 'InstanceUnknownCell' exception https://review.opendev.org/651307 | |
| 10:01:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove unnecessary wrapper https://review.opendev.org/651308 | |
| 10:01:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Remove cell APIs https://review.opendev.org/651309 | |
| 10:01:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove cells v1 options, group https://review.opendev.org/651310 | |
| 10:08:48 | openstackgerrit | Merged openstack/nova stable/stein: Add regression test for bug 1825034 https://review.opendev.org/657420 | |
| 10:08:49 | openstack | bug 1825034 in OpenStack Compute (nova) stein "listing deleted servers from the API fails after running fill_virtual_interface_list online data migration" [High,In progress] https://launchpad.net/bugs/1825034 - Assigned to Matt Riedemann (mriedem) | |
| 10:09:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove cells v1 parameter from 'ComputeTaskAPI.resize_instance' https://review.opendev.org/651311 | |
| 11:13:06 | cdent | welcome back gibi | |
| 11:13:14 | gibi | cdent: thanks | |
| 12:01:32 | openstackgerrit | Hamdy Khader proposed openstack/os-vif master: OVS DPDK port representors support https://review.opendev.org/658786 | |
| 12:12:39 | openstackgerrit | Jean-Philippe Evrard proposed openstack/nova master: [Docs] Fix minor typo https://review.opendev.org/660696 | |
| 12:25:25 | sean-k-mooney | stephenfin: bauzas can ye add https://review.opendev.org/#/c/653506/ to your review queue | |
| 12:25:34 | bauzas | mmm ok | |
| 12:31:54 | sean-k-mooney | alex_xu: not sure if you usually review networking/libvirt livemigrtaion stuff but https://review.opendev.org/#/c/653506/ might be in your area too. | |
| 12:46:19 | abhi89 | HI.. i am getting this error in my logs.. | |
| 12:46:20 | abhi89 | The [cinder] section of your nova configuration file must be configured for authentication with the block-storage service endpoint.: InstanceNotFound: Instance 50C28C76-6E6B-4BF0-8291-FBD8C176B9B2 could not be found. | |
| 12:46:43 | abhi89 | it is coming from | |
| 12:46:44 | abhi89 | File "/usr/lib/python2.7/dist-packages/nova/volume/cinder.py", line 78, in _load_auth_plugin | |
| 12:46:44 | abhi89 | 2019-05-06 08:58:32.792 123932 ERROR nova.compute.manager [instance: 50c28c76-6e6b-4bf0-8291-fbd8c176b9b2] raise cinder_exception.Unauthorized(401, message=err_msg) | |
| 12:47:18 | openstackgerrit | Silvan Kaiser proposed openstack/nova stable/stein: Exec systemd-run without --user flag in Quobyte driver https://review.opendev.org/660705 | |
| 12:48:09 | openstackgerrit | Silvan Kaiser proposed openstack/nova stable/stein: Added mount fstype based validation of Quobyte mounts https://review.opendev.org/660706 | |
| 12:48:53 | abhi89 | this is very rare.. can anyone help me here to understand do we really need to add 'auth' in [cinder] section of nova.conf..? because without this also deploys work fine | |
| 12:54:36 | efried | abhi89: What release are you on? | |
| 12:54:54 | abhi89 | efried: stein | |
| 12:55:51 | efried | abhi89: Do you have a message about "Unknown auth type"? | |
| 12:56:01 | efried | (from line 77) | |
| 12:56:15 | alex_xu | sean-k-mooney: not too much, but have involve lm for the lm with numa patche, anyway I will put that to my review list, give a try | |
| 12:56:50 | abhi89 | efried: yes.. Unauthorized: Unknown auth type: None (HTTP 401) | |
| 12:56:59 | sean-k-mooney | alex_xu: no worries | |
| 12:57:19 | efried | abhi89: Do you have service_auth set up? | |
| 12:58:30 | abhi89 | efried: service_auth? where? | |
| 13:00:15 | efried | abhi89: https://docs.openstack.org/nova/stein/configuration/config.html#service-user | |
| 13:01:21 | efried | You're saying the symptom is rare, which leads me to suspect it's happening over long-running operations when your user token is expiring. That's what service auth was designed for. If you configure this, it wraps your user token in a service token so that when your user token expires, subsequent operations in that thread will still succeed (using the service token). | |
| 13:02:37 | efried | The way it's manifesting is odd, though. That code path should only be happening under admin context. | |
| 13:05:25 | abhi89 | efried: hmm.. let me try adding service_auth section to nova.conf.. thanks for the info.. | |
| 13:05:37 | efried | abhi89: That's probably not it. | |
| 13:05:44 | efried | it's probably I3c35bba43fee81baebe8261f546c1424ce3a3383 | |
| 13:06:04 | efried | abhi89: https://review.opendev.org/#/c/522112/ | |
| 13:06:19 | efried | read the commit message and the bugs tagged there... | |
| 13:06:38 | efried | if you've got a situation similar to what's described, then you *do* need to set up admin auth for cinder to resolve it. | |
| 13:09:05 | efried | stephenfin, cdent: What's the thing that has to be in place for Depends-On to work? Something tox something sibling? Specifically, is that currently set up to work from nova to os-traits? | |
| 13:09:39 | cdent | it is not set up for os-traits unless add a required-project of openstack/os-traits | |
| 13:09:53 | cdent | (to the zuul job config) | |
| 13:10:22 | openstackgerrit | Eric Fried proposed openstack/nova master: Support old & new versions of svm and avx traits https://review.opendev.org/660515 | |
| 13:10:34 | efried | o | |
| 13:10:53 | abhi89 | efried: ok.. let me check on that.. | |
| 13:11:02 | efried | cdent: for the base job? | |
| 13:11:03 | sean-k-mooney | cadding os-traits to the required projects will add it to LIB_FROM_GIT and cause the zuul clone to prepar it in the workspace | |
| 13:11:15 | sean-k-mooney | but that will result in the job running with master of os-tratis | |
| 13:11:20 | sean-k-mooney | not the releaded version | |
| 13:11:30 | sean-k-mooney | so we normally dont want that to be the case | |
| 13:11:42 | efried | mm | |
| 13:11:45 | cdent | sean-k-mooney: why not? | |
| 13:11:53 | efried | because we would get false positives | |
| 13:12:09 | cdent | meh | |
| 13:12:09 | sean-k-mooney | cdent: because we wont be testing with the released version or the version listed in upperconstriats | |
| 13:12:26 | sean-k-mooney | so we wont be testing with what people actullly will be deploying | |
| 13:12:30 | efried | right | |
| 13:12:34 | efried | and that would be bad. | |
| 13:12:47 | sean-k-mooney | we could do it in nova-next or a specific job | |
| 13:13:01 | sean-k-mooney | we just dont wnat to do that in all jobs | |
| 13:13:05 | cdent | or constraints in general | |
| 13:13:17 | sean-k-mooney | cdent: do you use arch | |
| 13:13:19 | cdent | hell, I'm going the whole way: I don't like | |
| 13:14:11 | aarents | efried: Hi, about https://review.opendev.org/#/c/659054 I had to rebase, it is ok now, can you have a look on it please? | |
| 13:14:23 | sean-k-mooney | hehe look im fine with deploying with os-traits from master as long as we have at least one job that is testing the released version | |
| 13:15:02 | sean-k-mooney | maybe there is a feature request here to zuul for a way to add peoject dynamiclaly per patch | |
| 13:15:36 | efried | sean-k-mooney: ^ this | |
| 13:15:38 | sean-k-mooney | e.g. Depends-on: xyw Require: openstack/os-traits | |
| 13:15:45 | efried | ooo | |
| 13:16:16 | sean-k-mooney | that said we still would not want to merge the patch without the release so not sure how useful that is | |
| 13:16:22 | efried | As it stands, I have to hand-edit .zuul.yaml. And for inherited jobs/templates, I would have to write whole derived jobs for them. | |
| 13:16:32 | efried | sean-k-mooney: It's useful for me to validate that my patch is correct. | |
| 13:16:56 | efried | Because as always I don't want to run all of the nova unit test and functional suites locally. | |
| 13:17:04 | sean-k-mooney | efried: right but outside of that not sure. | |
| 13:17:43 | sean-k-mooney | efried: i recently set up rtox to run my unit tests over ssh on my desktop. they now take ~ 2 mins to run | |
| 13:17:44 | efried | Sure, outside of that it's not very helpful, and needs to be negated once the dep merges, so I can't accidentally merge that patch. | |
| 13:18:14 | efried | aarents: looking... | |
| 13:18:15 | sean-k-mooney | efried: it woudl be useful for test only patch on top of the actul change | |
| 13:18:26 | sean-k-mooney | *test only -> DNM | |
| 13:19:59 | cdent | efried: clone yourself and fix the core problem: tests are too slow | |
| 13:20:24 | mriedem | this contributes to ^ https://bugs.launchpad.net/nova/+bug/1829896 | |
| 13:20:24 | openstack | Launchpad bug 1829896 in OpenStack Compute (nova) "libvirt: "Instance failed to shutdown in 60 seconds." in the gate" [Undecided,New] | |
| 13:20:28 | sean-k-mooney | cdent: not if you throw 48 cores at them | |
| 13:21:20 | cdent | 48!? luxury | |
| 13:21:57 | sean-k-mooney | cdent: the full nova py37 tox env takes 72 seconds to run on my desktop or about 2 minutes totoal when you account for the tox startup time and report generation | |
| 13:22:16 | cdent | try the placement tests, I suspect you will go backwards in time | |
| 13:22:44 | sean-k-mooney | cdent: i decomisined the tower server i baught do some ci | |
| 13:24:21 | sean-k-mooney | ====== | |
| 13:24:22 | sean-k-mooney | Totals | |