Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-27
19:01:04 fried_rice jgwentworth: In any case, I think cdent and dhellmann were involved in that work - they would know more.
19:01:27 jgwentworth no, I mean the request-id staying within nova is changing. it used to stay consistent per request
19:02:05 jgwentworth like you used to be able to trace the same request id from nova-api -> nova-conductor -> nova-scheduler -> nova-compute
19:02:30 jgwentworth it's true that request-ids didn't use to span across nova to glance, for example, and that's what the global request-id work was about
19:14:56 openstackgerrit Dan Smith proposed openstack/nova master: Make service all-cells min version helper use scatter-gather https://review.openstack.org/539590
19:14:57 openstackgerrit Dan Smith proposed openstack/nova master: Make host_manager use scatter-gather and ignore down cells https://review.openstack.org/539617
20:06:40 openstackgerrit Brianna Poulos proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
20:06:41 openstackgerrit Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262
20:06:42 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
20:06:43 openstackgerrit Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
20:11:26 jgwentworth bpoulos: hi, just wanted to confirm, is someone from your team going to propose a spec amendment to openstack/nova-specs as described on L80 here? https://etherpad.openstack.org/p/nova-runways-rocky
20:12:06 jgwentworth once we get that reviewed and merged, we can continue to review the remaining patches in the implementation
20:12:25 bpoulos jgwentworth: yes, we have a spec amendment internally, but it has to receive internal review approval before we can post the update. I hope to post the update soon!
20:12:45 jgwentworth bpoulos: perfect, thanks! just wanted to make sure we were on the same page
20:13:05 jgwentworth we'll be on the lookout for the update
20:13:12 bpoulos jgwentworth: no problem, thank you for checking
20:22:50 openstackgerrit Jay Pipes proposed openstack/nova master: add root parent NULL online migration https://review.openstack.org/564876
21:30:53 openstackgerrit sean mooney proposed openstack/nova master: add mtu to libvirt xml for ethernet and bridge types https://review.openstack.org/553072
21:42:22 openstackgerrit Hongbin Lu proposed openstack/nova-specs master: Choose default network on ambiguity https://review.openstack.org/520247
22:08:55 openstackgerrit Hongbin Lu proposed openstack/nova master: Use revision_number on adding/removing security group https://review.openstack.org/535510
22:24:32 openstackgerrit Eric Fried proposed openstack/nova master: Base test module/class for functional placement db https://review.openstack.org/564590
22:24:33 openstackgerrit Eric Fried proposed openstack/nova master: Use test_base symbols directly https://review.openstack.org/564592
22:24:34 openstackgerrit Eric Fried proposed openstack/nova master: Use helpers in test_resource_provider (func) https://review.openstack.org/564638
23:06:25 fried_rice ō/
#openstack-nova - 2018-04-28
00:05:39 openstackgerrit Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596
02:43:53 openstackgerrit Lars Kellogg-Stedman proposed openstack/nova master: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/564964
02:44:46 openstackgerrit Lars Kellogg-Stedman proposed openstack/nova master: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/564964
06:37:25 openstackgerrit jichenjc proposed openstack/nova master: [doc] Add soft_deleted flag https://review.openstack.org/562897
07:20:50 openstackgerrit jichenjc proposed openstack/nova master: Avoid raise InstanceNotRunning exception https://review.openstack.org/541152
07:22:08 openstackgerrit jichenjc proposed openstack/nova master: remove ec2 object definitions https://review.openstack.org/557150
07:22:13 openstackgerrit jichenjc proposed openstack/nova master: remove ec2 db functions https://review.openstack.org/557572
07:34:13 openstackgerrit jichenjc proposed openstack/nova master: Enhance api-guide general info some updates https://review.openstack.org/561773
07:38:29 openstackgerrit jichenjc proposed openstack/nova master: Enhance api-guide general info some updates https://review.openstack.org/561773
10:03:21 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201
10:03:22 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
10:03:23 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077
10:12:22 openstackgerrit sean mooney proposed openstack/os-vif master: convert os-vif docs to follow PTI https://review.openstack.org/559398
11:36:52 openstackgerrit sean mooney proposed openstack/os-vif master: move legacy-tempest-dsvm-nova-os-vif to repo https://review.openstack.org/559401
13:16:43 openstackgerrit sean mooney proposed openstack/os-vif master: move legacy-tempest-dsvm-nova-os-vif to repo https://review.openstack.org/559401
13:17:44 openstackgerrit sean mooney proposed openstack/os-vif master: move legacy-tempest-dsvm-nova-os-vif to repo https://review.openstack.org/559401
18:33:45 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565043
18:33:46 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Lift the restriction of choices for `cpu_model_extra_flags` https://review.openstack.org/565044
19:18:32 openstackgerrit Vladyslav Drok proposed openstack/nova master: Placement: allow to set reserved value equal to total for inventory https://review.openstack.org/564838
19:21:54 openstackgerrit Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757
19:39:33 openstackgerrit Eric Fried proposed openstack/nova master: placement: Object changes for granular https://review.openstack.org/564351
19:39:34 openstackgerrit Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757
20:30:54 openstackgerrit Lars Kellogg-Stedman proposed openstack/nova master: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/564964
20:32:41 openstackgerrit Lars Kellogg-Stedman proposed openstack/nova master: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/564964
#openstack-nova - 2018-04-29
00:04:55 openstackgerrit Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757
01:35:11 pvc hi
01:35:38 pvc may i know what can i do to make my instance performance good?
01:35:40 pvc like tuning
#openstack-nova - 2018-04-30
10:14:42 openstackgerrit fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181
10:20:44 openstackgerrit Radoslav Gerganov proposed openstack/nova master: Handle PortNotFoundClient exception when getting ports https://review.openstack.org/565183
10:30:38 kashyap lyarwood: Et al, we also need this to be backported (all the way to Ocata) -- https://review.openstack.org/#/c/565043/
10:30:59 kashyap (The upstream change is not merged yet; but it's straightfoward)
10:36:20 openstackgerrit Merged openstack/nova master: Make service all-cells min version helper use scatter-gather https://review.openstack.org/539590
10:36:25 lyarwood kashyap: kk, why are you using the bp/ topic for these bug fixes and a bug/ topic for the original change?
10:36:28 openstackgerrit Merged openstack/nova master: Make host_manager use scatter-gather and ignore down cells https://review.openstack.org/539617
10:36:47 kashyap lyarwood: Right, I'll rename it; so that it's all under the same topic
10:36:57 kashyap lyarwood: Do you have a minute more?
10:37:10 kashyap lyarwood: I think I messed up something (unrelated to branch naming) :-(
10:37:18 kashyap See here: https://review.openstack.org/#/q/topic:bp/libvirt-cpu-model-extra-flags+(status:open+OR+status:merged)
10:37:20 lyarwood kashyap: https://review.openstack.org/#/c/563926/ & https://review.openstack.org/#/c/565044/ are duplicates right?
10:37:26 lyarwood kashyap: yeah
10:37:41 kashyap lyarwood: Allow me to explain how I arrived there:
10:38:31 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: Add 'numa-aware-vswitches' spec https://review.openstack.org/541290
10:38:48 kashyap lyarwood: I made the casing patch, and on *top* of it, applied the "lift restrictions" patch.
10:38:58 kashyap lyarwood: Could I have done anything cleaner?
10:39:35 kashyap lyarwood: The first variant https://review.openstack.org/#/c/563926/ had 8 revisions. Now I will "lose" them (or I'll simply link to it from the other)
10:39:53 lyarwood kashyap: AFAICT you changed the change-id causing gerrit to create a new review
10:40:23 lyarwood kashyap: you can close out the new review and just rebase the case-insensitive change on top of the old one
10:40:31 kashyap lyarwood: Right
10:40:33 kashyap I'm doing to do that now
10:40:35 kashyap Thanks!
10:52:28 kashyap lyarwood: Pease remind me again, I think it's okay to include "Closes-Bug" tag on two Gerrit patches, right?
10:52:43 kashyap (I often see "Related-Bug" on several patches, though
10:52:44 kashyap )
10:53:10 lyarwood kashyap: I *think* that's okay yeah
10:53:23 lyarwood kashyap: maybe Related-Bug on the first?
10:53:45 kashyap lyarwood: You mean, add the Related-Bug on the casing patch?
10:54:07 kashyap (Which is the "first" patch in this 2-patch series)
10:54:24 lyarwood kashyap: yeah
10:54:32 kashyap lyarwood: Okido; I'll do that. Gracias
10:55:54 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565043
10:55:55 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Lift the restriction of choices for `cpu_model_extra_flags` https://review.openstack.org/563926
11:01:59 kashyap lyarwood: Done. Mind ACKing this, since you have context too: https://review.openstack.org/#/c/565043/
12:24:42 efried ō/
12:38:14 bauzas efried: you can remove your hat
12:38:26 efried bauzas: It's my crew cut
12:39:27 bauzas well, I'm 37 and my hairs become to leave me, so should I do something like °o/ ?
12:55:10 zzzeek_ jaypipes: mysql question. does SELECT * FROM (SELECT x, y FROM a) AS a_alias JOIN b ON a_alias.x=b.y FOR UPDATE , lock both "a", and "b" ?
12:58:37 jaypipes zzzeek_: it will lock all the rows in a and b that join on x = y, yes
12:58:52 zzzeek_ jaypipes: that's great news got a user claims it is not
12:59:08 jaypipes zzzeek_: what version of MySQL?
12:59:13 zzzeek_ jaypipes: hmm
12:59:31 zzzeek_ jaypipes: would have to ask
12:59:36 jaypipes zzzeek_: not that that should matter... I think that is the behaviour for at least since 4.0
13:00:16 zzzeek_ jaypipes: OK he says "I noticed a different issue when using joinedload and with_for_update in Mysql with a REPEATABLE READ isolation level. In my case, it seems that the with_for_update only locked the joinedtable because the FOR UPDATE statement is only applied to the joinedtable - the root table is not locked because it is queried via the subselect. " but I'm going to ask him to further define how he knows this is the case
13:00:43 zzzeek_ jaypipes: if i had to move the FOR UPDATE around in the query it would really ruin my day :)

Earlier   Later