| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-16 | |||
| 19:42:59 | mriedem | so controller can be queens and then computes can be pike+ | |
| 19:43:05 | tmcm | understood, thanks | |
| 19:43:17 | mriedem | at certain points we randomly delete compat code that's older than n-1 | |
| 19:44:10 | mriedem | tmcm: this should be a bit more clear https://docs.openstack.org/nova/latest/user/upgrade.html#rolling-upgrade-process | |
| 19:45:26 | cfriesen | mriedem: for a recreate test could we add a new DB entry with a deleted service that has the same name as the existing service? | |
| 19:46:16 | tmcm | mriedem: got it. i'd actually worked through that document already and my queens controller was happy from a db standpoint, i was just stuck at: | |
| 19:46:18 | tmcm | | Placement service and need to be upgraded and/or fixed. | | |
| 19:46:18 | tmcm | | This means no compute nodes are reporting into the | | |
| 19:46:18 | tmcm | | service but there are 80 compute nodes in the deployment. | | |
| 19:46:18 | tmcm | | Details: There are no compute resource providers in the Placement | | |
| 19:46:18 | tmcm | | Result: Warning | | |
| 19:46:18 | tmcm | | Check: Resource Providers | | |
| 19:46:18 | tmcm | +-------------------------------------------------------------------+ | |
| 19:46:19 | tmcm | | https://docs.openstack.org/nova/latest/user/placement.html | | |
| 19:46:19 | tmcm | | See | | |
| 19:46:20 | tmcm | +-------------------------------------------------------------------+ | |
| 19:46:20 | tmcm | | for more details. | | |
| 19:46:25 | tmcm | on the compute side | |
| 19:46:31 | openstackgerrit | Merged openstack/nova master: List instances performace optimization https://review.openstack.org/507854 | |
| 19:46:33 | tmcm | well, from the compute standpoint, i mean | |
| 19:47:27 | tmcm | so if there was any way to force the mitaka compute nodes to use the placement service, i might be OK | |
| 19:47:39 | artom | tmcm, there isn't, the code didn't exist in mitaka :) | |
| 19:47:46 | tmcm | got it | |
| 19:47:50 | tmcm | thanks to both of you | |
| 19:47:50 | artom | (Err, right?) | |
| 19:47:55 | tmcm | i didn't see it in there | |
| 19:48:12 | artom | This self-doubt thing is cripling | |
| 19:49:06 | mriedem | artom: yes, new in newton | |
| 19:49:13 | mriedem | optional in newton, required starting in ocata | |
| 19:49:16 | cfriesen | mriedem: apparently "nova show" on the instance will result in adding the uuid onto the service, but "nova list" hits the ServiceNotFound error | |
| 19:49:20 | artom | mriedem, thank you :) | |
| 19:49:33 | melwitt | you'd have to upgrade your compute nodes to at least newton to make them report in to placement. might even be ocata, not sure | |
| 19:49:44 | mriedem | cfriesen: those should both go down the same code path as far as i know, but maybe not | |
| 19:50:48 | melwitt | I think it would be random depending on the ordering of the returned dupe services | |
| 19:54:33 | melwitt | nevermind, it looks like it should try to generate a UUID for both services that match the instance via host, so it should fail reliably | |
| 19:54:35 | sean-k-mooney | dansmith: just finished looking at the patch. the broad strokes look fine. i left some nits inline. main concern is the name attribute of the driver field. i think we should just not set it for safty across livemirate in the future. anyway i left comments in the patch but over all it looks ok | |
| 19:56:10 | sean-k-mooney | dansmith: i might try an apply it locally tomorow and see if i have any issues. anyway im going to head for the evening. | |
| 19:57:12 | dansmith | sean-k-mooney: okay just to be sure, you know we're already setting it to vhost in almost all cases now right? | |
| 19:57:24 | dansmith | sean-k-mooney: it's done in _get_virtio_mq_settings() | |
| 19:57:43 | sean-k-mooney | dansmith: am i dont think we do that for vhost-user ports however | |
| 19:57:51 | dansmith | sean-k-mooney: we do | |
| 19:57:56 | dansmith | sean-k-mooney: even though it is wrong | |
| 19:57:59 | dansmith | that's the point :) | |
| 19:58:09 | sean-k-mooney | when we request vhost multiqueue only right | |
| 19:58:16 | dansmith | no | |
| 19:58:33 | dansmith | if we're in kvm, and MODEL_VIRTIO, we call that method and it sets it to vhost always I think | |
| 19:58:56 | TheJulia | efried: your python-ironicclient patch, Would you be okay if I updated the commit message? | |
| 19:59:08 | dansmith | sean-k-mooney: oh, actually, I guess you're right | |
| 19:59:16 | dansmith | sean-k-mooney: sahid said the opposite and I looked but didn't see | |
| 19:59:19 | sean-k-mooney | dansmith: oh we did not used too | |
| 19:59:32 | dansmith | sean-k-mooney: so yeah, this does change it to be more often, which is what I was trying to avoid when we talked last week | |
| 19:59:36 | sean-k-mooney | dansmith: ya this is a normal vhost-user xml http://paste.openstack.org/show/719324/ | |
| 19:59:39 | dansmith | but I missed that we only do it for multiqueue | |
| 19:59:43 | efried | TheJulia: Sure, you can do whatever you like with it. I would be happy to keep working on it too if you prefer. (I saw you made a comment, but I haven't gotten to reading it yet.) | |
| 19:59:45 | dansmith | sean-k-mooney: so good call on that, thanks | |
| 20:00:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add wrinkle to FailedEvacuateStateTests https://review.openstack.org/561707 | |
| 20:00:31 | sean-k-mooney | dansmith: well i dont think it would break anything today since its ignored but if they ever stop ignoring it i dont know if that would break us | |
| 20:00:47 | dansmith | sean-k-mooney: yeah | |
| 20:01:03 | dansmith | sean-k-mooney: I figured since I don't know the details, it'd be better to just avoid changing it | |
| 20:01:16 | efried | TheJulia: Oh, it was rloo's note. Anyway, just let me know whatever you want me to do there. | |
| 20:01:20 | dansmith | sean-k-mooney: I was asking him to just set it explicitly where he knows it should be vhost or vhostuser | |
| 20:01:33 | dansmith | sean-k-mooney: did you see the cleanup patch below? | |
| 20:01:38 | efried | TheJulia: Caveat: Never used storyboard before... | |
| 20:01:43 | dansmith | that was also based on ignoring, but it's vhostuser and you said vhost-user, so is it wrong? | |
| 20:01:58 | TheJulia | efried: I can create it, its not a big deal | |
| 20:02:07 | TheJulia | efried: but your welcome to if you want | |
| 20:02:31 | TheJulia | Story clean-up is taking... many braincells. | |
| 20:02:39 | efried | TheJulia: Go for it, co-author away. | |
| 20:02:56 | sean-k-mooney | dansmith: well neighter vhostuser or vhost-user is documented explictly as an allowed value but they used vhost-user when refing to it. | |
| 20:03:13 | TheJulia | efried: don't need that :) But thanks. | |
| 20:03:16 | dansmith | sean-k-mooney: okay you wanna comment on the patch below with that info then? | |
| 20:03:18 | sean-k-mooney | dansmith: ill try and take a look at the libvirt source code tomorrow and see whent there parse has defiend | |
| 20:03:56 | dansmith | sean-k-mooney: okay | |
| 20:11:17 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add (Same|Different)HostFilter to defaults https://review.openstack.org/561651 | |
| 20:13:32 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | |
| 20:13:33 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | |
| 20:18:03 | openstackgerrit | Lance Bragstad proposed openstack/nova master: Set scope_types for admin password policies https://review.openstack.org/553613 | |
| 20:18:05 | openstackgerrit | Lance Bragstad proposed openstack/nova master: Add scope_types to server policies https://review.openstack.org/525772 | |
| 20:19:54 | lbragstad | johnthetubaguy_: jaypipes ^ look another shot at working a rough test structure into those | |
| 20:20:36 | lbragstad | they will fail, but i'm curious about the approach you'd like me to take with the testing | |
| 20:20:52 | lbragstad | or if what's proposed even makes sense | |
| 20:22:04 | jaypipes | lbragstad: cool, thanks | |
| 20:22:12 | jaypipes | lbragstad: will re-review shortly. | |
| 20:22:30 | lbragstad | awesome - ty | |
| 20:54:35 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: List instances performace optimization https://review.openstack.org/561715 | |
| 20:54:49 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | |
| 20:54:50 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | |
| 20:56:23 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Support forced separation of providers in granular https://review.openstack.org/561717 | |
| 21:03:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove vestigial system_metadata param from info_from_instance() https://review.openstack.org/561724 | |
| 21:27:00 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | |
| 21:27:01 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | |
| 21:32:42 | jackie-truong | mriedem dansmith jaypipes melwitt johnthetubaguy_: Thanks for reviewing the certificate validation patches. 479949 and 561262 should just need one last look. The API patch is going through Zuul now. | |
| 21:33:00 | jackie-truong | https://review.openstack.org/#/c/479949/ | |
| 21:33:05 | jackie-truong | https://review.openstack.org/#/c/561262/ | |
| 21:39:39 | melwitt | thanks jackie-truong. I've rechecked the tempest tests so we can look at that too for the latest revision of the series https://review.openstack.org/#/c/515210 | |
| 21:42:50 | jackie-truong | melwitt: Awesome, thanks | |
| 22:15:02 | mriedem | artom: gotta say, i'm not a fan of this https://review.openstack.org/#/c/561651/ | |
| 22:15:31 | cfriesen | mriedem: I reproduced the Service UUID issue with vanilla stable/pike devstack. Writing up a bug now. | |
| 22:15:31 | mriedem | if there is a problem with tempest, let's fix tempest | |
| 22:15:50 | mriedem | artom: as noted inline, those tempest tests for these filters probably shouldn't even live in tempest | |