| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-06 | |||
| 19:55:25 | efried | sdague But makes a guy wonder why the client couldn't grab it from the auth, which is made from the context, which contains that request ID... | |
| 19:56:26 | sdague | it could be more magical | |
| 19:56:41 | sdague | but, being explicit here seems fine | |
| 19:57:14 | efried | ight | |
| 19:57:34 | sdague | given that it was a one line change for each service linkage, being explicit didn't seem so bad :) | |
| 19:58:23 | efried | sdague But a zero-line change would have been cooler ;-) | |
| 19:59:21 | mriedem | 0-line change means non-0 amount of debugging in 2 years time | |
| 19:59:24 | mriedem | b/c of magic | |
| 20:00:34 | sdague | mriedem: ++ | |
| 20:00:35 | efried | sdague How/where does global_id get set on a RequestContext? | |
| 20:00:54 | sdague | it's a property which is either global_request_id or request_id | |
| 20:01:07 | sdague | so it prefers global_request_id if that was sent in | |
| 20:01:17 | sdague | otherwise it uses the service request id | |
| 20:01:39 | sdague | request_id is generated as a uuid | |
| 20:01:49 | sdague | ok, TC / Pg thing, so won't be active here | |
| 20:03:41 | efried | sdague Thanks, I found it in the superclass (duh) | |
| 20:04:38 | openstackgerrit | Eric Fried proposed openstack/nova master: Adopts keystoneauth with glance client. https://review.openstack.org/412634 | |
| 20:05:29 | openstackgerrit | Eric Fried proposed openstack/nova master: Add service_token for nova-glance interaction https://review.openstack.org/419662 | |
| 20:11:42 | melwitt | mriedem: forgot to mention last week I wrote a functional test for that local delete allocations thing. looks like they do indeed get cleaned up upon init_host, but they would stay around if the compute host never returned https://review.openstack.org/#/c/470578 | |
| 20:23:12 | openstackgerrit | melanie witt proposed openstack/nova master: placement project_id, user_id in PUT /allocations https://review.openstack.org/469634 | |
| 20:23:13 | openstackgerrit | melanie witt proposed openstack/nova master: placement: Add GET /usages to placement API https://review.openstack.org/469635 | |
| 20:28:55 | openstack | Launchpad bug 1696082 in OpenStack Compute (nova) "'nova.exception.NeutronAdminCredentialConfigurationInvalid" [Undecided,New] | |
| 20:28:55 | bauzas | mriedem: looks like https://bugs.launchpad.net/nova/+bug/1696082 is duplicate of https://bugs.launchpad.net/nova/+bug/1657774 ? | |
| 20:28:56 | openstack | Launchpad bug 1657774 in OpenStack Compute (nova) "Nova does not re-raise 401 Unauthorized received from Neutron for admin users" [Undecided,In progress] - Assigned to Roman Podoliaka (rpodolyaka) | |
| 20:30:33 | mriedem | shrug | |
| 20:30:35 | mriedem | TC meeting | |
| 20:30:44 | mriedem | melwitt: ok | |
| 20:30:58 | mriedem | melwitt: how's that bottom change on the quotas series going? | |
| 20:31:24 | artom | Speaking of bottom changes | |
| 20:31:36 | artom | mriedem, can I get you to put tagged attach on your review list again? | |
| 20:31:47 | bauzas | mriedem: dang. ok | |
| 20:32:13 | melwitt | mriedem: it's next on my list today, I just updated the placement stuff, working on the quotas thing now. when we talked at the summit, dansmith was saying it would be good to gate the "recheck quota" logic on a config option, in case someone doesn't mind quota going over during races | |
| 20:32:46 | melwitt | since a quota recheck is a re-count of resources, not a lightweight thing | |
| 20:33:26 | melwitt | so I was thinking of adding a config option also in the bottom change | |
| 20:34:02 | mriedem | artom: it already is, with 100 other things | |
| 20:34:15 | artom | mriedem, fair enough, thanks :) | |
| 20:35:07 | mriedem | bauzas: shrug == idk and don't have time to look, but maybe | |
| 20:35:32 | bauzas | mriedem: no worries, I see you active on the PG thingy | |
| 20:35:34 | artom | bauzas, don't think so - first is error 500, second one is error 400 | |
| 20:36:17 | openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: Check if domain is persistent before detaching devices https://review.openstack.org/441204 | |
| 20:36:25 | artom | bauzas, but the message definitely is the same... | |
| 20:36:47 | artom | I feel like we should just close the first one with not enough info | |
| 20:36:53 | dansmith | melwitt: I think that's a good plan yeah | |
| 20:36:54 | bauzas | artom: yup, looks like both are related, just the fact that the API for nova list transforms it into 400 | |
| 20:37:00 | artom | You can't just paste a stack trace into launchpad and call it a bug report | |
| 20:37:17 | mriedem | maybe just review https://review.openstack.org/#/c/422696/ | |
| 20:38:06 | melwitt | cool, thanks dansmith | |
| 20:38:31 | mriedem | melwitt: i thought your bottom change was sabout instance groups quota? | |
| 20:38:37 | mriedem | wouldn't a config patch be separate? | |
| 20:38:53 | mriedem | oh wait, | |
| 20:39:01 | mriedem | the recheck quota thing in that patch, now i get it | |
| 20:39:11 | mriedem | and default that to what? | |
| 20:39:13 | mriedem | recheck by default? | |
| 20:39:22 | melwitt | yeah. I could do a separate patch for the config option but it would be by itself with nothing using it | |
| 20:39:32 | mriedem | just mix them together | |
| 20:39:33 | melwitt | yeah I was thinking recheck by default | |
| 20:39:38 | melwitt | ack | |
| 20:39:48 | mriedem | probably with a reno for the config option | |
| 20:40:07 | melwitt | yeah. thanks for reminding bc I probably would have forgotten the reno | |
| 20:44:00 | ckopper | danpawlik: I am finally back from my conference and was curious if you (or anyone else) had thoughts on what to do about the `img_hide_hypervisor_id` issue ( https://review.openstack.org/459753 ). I also hope I did not miss any discussion about this | |
| 20:45:43 | ckopper | (I can certainly wait to discuss this some other time more convenient for you) | |
| 21:03:22 | openstack | Launchpad bug 1696043 in OpenStack Compute (nova) "os-hypervisors api can not list the uuid of the specified VMM" [Undecided,Invalid] - Assigned to mylq (leibnizlq) | |
| 21:03:22 | bauzas | mriedem: heh, fun someone is asking for something you began to write https://bugs.launchpad.net/nova/+bug/1696043 | |
| 21:03:34 | mriedem | melwitt: so on https://review.openstack.org/#/c/470578 i think we both probably think that if you delete the instance, if the compute is down, the local delete flow in the api should cleanup the allocations so they don't count against your quota? | |
| 21:03:48 | bauzas | mriedem: be glad to feel yourself filling a gap :p | |
| 21:04:24 | mriedem | heh, if i'd only start writing that patch | |
| 21:04:33 | melwitt | mriedem: yeah, that's what I was thinking. but that introduces the dependency of nova-api on placement as you mentioned | |
| 21:04:51 | mriedem | yup | |
| 21:05:05 | mriedem | but, i don't know what else we'd do there | |
| 21:05:21 | mriedem | unless we ran a periodic in scheduler, but that seems wrong | |
| 21:05:31 | melwitt | yeah, same. | |
| 21:05:51 | melwitt | I thought about it, noted it in the commit message, and then punted the thought for later :P | |
| 21:06:51 | bauzas | melwitt: mriedem: AFAIK we removed any periodic task in the scheduler service | |
| 21:07:35 | cdent | melwitt: I added yet more commentary on the lifestyle of consumer uuids. Not clear what should be done, but it feels like something. Also: good spot on the dangerous thing. | |
| 21:07:41 | bauzas | and given I'd like to see scheduler code be running within the conductor worker, I tend to pretend it should not exist | |
| 21:10:06 | melwitt | cdent: cool, will check it out. yeah, agreed. I'm hoping once we all chime in there, it'll become more clear what we should do | |
| 21:10:38 | openstack | Launchpad bug 1696001 in OpenStack Compute (nova) "nova-manage cell_v2 map_cell0 can create invalid connection URL" [Undecided,New] | |
| 21:10:38 | bauzas | melwitt: dansmith: fun bug https://bugs.launchpad.net/nova/+bug/1696001 | |
| 21:12:10 | melwitt | dammit, the rstrip() thing bit us. I guess my suggestion to do the equivalent of a "rreplace()" was actually needed | |
| 21:12:30 | bauzas | yup | |
| 21:12:37 | bauzas | I can cook a patch up | |
| 21:15:00 | mriedem | bauzas: we are certainly running periodic tasks in the scheduler | |
| 21:15:02 | mriedem | in the host manager | |
| 21:15:09 | mriedem | the discover hosts thing from ocata is in there | |
| 21:15:14 | dansmith | is this bug melwitt's fault? | |
| 21:15:15 | dansmith | because I really need her to start taking some of the blame | |
| 21:15:50 | melwitt | it could be reasoned as my fault for not pushing my rreplace() agenda | |
| 21:16:10 | mriedem | it's my fault | |
| 21:16:36 | mriedem | at least we have the unit test to recreate it, just add mysql+pymysql://nova:password@db.example.com/nova?charset=utf8 to the ddt list | |
| 21:16:40 | melwitt | I'm still not sure that's really the right thing to do either because we want to make a change to the url.database variable really. it just didn't work right because of encoding or something | |
| 21:17:51 | mriedem | cool that's also in ocata and newton b/c i backported it | |
| 21:18:01 | melwitt | yeah. fun times | |
| 21:18:22 | bauzas | I just feel we need to replace at the index where we found the string | |
| 21:18:50 | mriedem | i was doing sometihng like that in an earlier version | |
| 21:18:58 | mriedem | and then the sqla url thing was pointed out to simplify | |
| 21:19:18 | melwitt | bauzas: yeah, my suggestion in the review was "return cell0_db_name.join(connection.rsplit(url.database, 1))" | |
| 21:19:32 | melwitt | but really we want to just change url.database and then re-make the whole url and return it | |
| 21:19:46 | melwitt | but when mriedem tried it, it blew up because of the str() call | |
| 21:19:54 | bauzas | lemme try it | |
| 21:23:28 | mriedem | str() blows up because sqla tries to do some things on __str__ | |