| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-14 | |||
| 13:06:20 | openstackgerrit | Merged openstack/nova master: Add test coverage of existing keypairs policies https://review.opendev.org/718604 | |
| 13:06:28 | openstackgerrit | Merged openstack/nova master: Introduce scope_types in keypairs https://review.opendev.org/718609 | |
| 13:06:36 | openstackgerrit | Merged openstack/nova master: Add test coverage of existing quota sets policies https://review.opendev.org/719128 | |
| 13:06:50 | openstackgerrit | Merged openstack/nova master: Add test coverage of existing quota class policies https://review.opendev.org/719095 | |
| 13:06:58 | openstackgerrit | Merged openstack/nova master: Introduce scope_types in quota class Policies https://review.opendev.org/719096 | |
| 13:07:06 | openstackgerrit | Merged openstack/nova master: Add new default roles in keypairs policies https://review.opendev.org/718619 | |
| 13:07:16 | openstackgerrit | Merged openstack/nova master: Pass the actual target in keypairs policy https://review.opendev.org/718621 | |
| 13:07:40 | openstackgerrit | Merged openstack/nova master: api: Allow custom traits https://review.opendev.org/718461 | |
| 13:25:49 | openstackgerrit | Merged openstack/nova master: Use placement stable version for functional job https://review.opendev.org/719119 | |
| 13:51:38 | openstack | Launchpad bug 1872082 in OpenStack Compute (nova) "available disk on compute may be lightly overestimated in some cases" [High,In progress] - Assigned to Alexandre arents (aarents) | |
| 13:51:38 | aarents | Hi there, lyarwood I put few examples on associated bug https://bugs.launchpad.net/nova/+bug/1872082 ( https://review.opendev.org/#/c/719008 ) | |
| 13:59:26 | lyarwood | aarents: yeah sorry missed the RAW example | |
| 14:01:01 | aarents | lyarwood: np | |
| 14:08:15 | lyarwood | aarents: so I think the RAW behaviour is actually a bug due to the way we call fallocate | |
| 14:08:44 | lyarwood | aarents: I think it's writing out a block of zero's at the start | |
| 14:09:01 | lyarwood | aarents: as we aren't using -d | |
| 14:10:08 | gmann | stephenfin: replied for SG scope things- https://review.opendev.org/#/c/717175/9/nova/policies/server_groups.py@36 | |
| 14:10:20 | gmann | stephenfin: please check if that make sense, it is same for POST servers also | |
| 14:11:10 | lyarwood | aarents: https://paste.centos.org/view/90da78f0 for example | |
| 14:11:23 | lyarwood | aarents: I think your change is still good FWIW | |
| 14:11:34 | lyarwood | aarents: but we should also fix our call to fallocate | |
| 14:13:32 | aarents | lyarwood: exact for -d , seems necessary | |
| 14:18:39 | lyarwood | aarents: actually -d is wrong | |
| 14:21:55 | openstackgerrit | Merged openstack/nova master: Add new default roles in tenant tenant usage policies https://review.opendev.org/717587 | |
| 14:22:03 | openstackgerrit | Merged openstack/nova master: Add test coverage of existing server topology policies https://review.opendev.org/717524 | |
| 14:22:33 | nightmare_unreal | how can I get migration ID when migrating an instance ? When I run the live migration cmd from CLI i don't get any output | |
| 14:24:03 | gibi | brinzhang_, sean-k-mooney, dansmith: regarding cyborg-nova integration: can I close the bp and there will be a new bp for Victoria? Does https://review.opendev.org/#/c/716186 needs to be handled as a bug? | |
| 14:24:36 | dansmith | gibi: yes and yes, IMHO | |
| 14:25:11 | gibi | dansmith: OK. I will wait for brinzhang_ to comment too and I will ask him to file the bug too | |
| 14:25:34 | dansmith | ack | |
| 14:26:18 | sean-k-mooney | gibi: yep agree with dansmith | |
| 14:26:24 | gibi | sean-k-mooney: cool | |
| 14:26:59 | sean-k-mooney | speaking of ill go file said bug before i get distracted | |
| 14:27:14 | gibi | sean-k-mooney: thanks | |
| 14:33:33 | sean-k-mooney | gibi: should i mark this as rc potential or shoudl we just treat it as a normal bugfix that we will backport after ussuri is released as normal. | |
| 14:33:58 | stephenfin | gmann: Thanks for the explanation. Would it be possible edit the commit message to include that information there? | |
| 14:34:57 | gibi | sean-k-mooney: what is the worst case issue by that bug? | |
| 14:35:42 | gmann | stephenfin: sure, doing. | |
| 14:36:36 | sean-k-mooney | if we create the ARQ and fail to clean them up becasue binding fail then we leak the arq. since its not bound i think its just leaking a db recored. i need to double check that | |
| 14:37:45 | openstack | Launchpad bug 1872730 in OpenStack Compute (nova) "Delete ARQs for an instance when the instance is deleted only delete bound arqs" [High,Triaged] | |
| 14:37:45 | sean-k-mooney | gibi: https://bugs.launchpad.net/nova/+bug/1872730 | |
| 14:38:10 | gibi | sean-k-mooney: then I think it is not an rc potential bug | |
| 14:38:13 | openstackgerrit | Merged openstack/nova master: Introduce scope_types in server topology https://review.opendev.org/717584 | |
| 14:38:19 | openstackgerrit | Merged openstack/nova master: Add new default roles in server topology policies https://review.opendev.org/717585 | |
| 14:38:27 | openstackgerrit | Merged openstack/nova master: workarounds: Add option to locally attach RBD volumes on compute hosts https://review.opendev.org/708029 | |
| 14:38:42 | gibi | sean-k-mooney: thanks for fileing it | |
| 14:40:02 | sean-k-mooney | gibi: ya i might ping thet cyborg folk to take a look too and confirm its not going to leak devices on there end. but my understanding is they dont allocate or program the device until we bind it as that is when we set the host and pass the allocation to cyborg | |
| 14:40:16 | sean-k-mooney | so if we failed to bind it then they should not need to clean anything up | |
| 14:40:30 | sean-k-mooney | we should be deleted and recreating the arqs when we reschdule | |
| 14:40:34 | gibi | I have a similar understanding, thanks for following it up | |
| 14:45:41 | openstackgerrit | Merged openstack/nova master: fup: Add removal TODOs for disable_native_luksv1 and rbd_volume_local_attach https://review.opendev.org/718099 | |
| 14:45:52 | openstackgerrit | Merged openstack/nova master: Add new default roles in os-evacuate policies https://review.opendev.org/708237 | |
| 14:52:26 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175 | |
| 14:52:47 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix servers policy for admin_or_owner https://review.opendev.org/718501 | |
| 14:55:16 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix servers policy for admin_or_owner https://review.opendev.org/718501 | |
| 14:55:50 | gmann | stephenfin: done. also this one too - https://review.opendev.org/#/c/718501/ | |
| 14:56:09 | gmann | stephenfin: once you are ok with those then i will rebase the series | |
| 14:56:25 | openstackgerrit | Merged openstack/nova master: Pass the target in os-services APIs policy https://review.opendev.org/676688 | |
| 14:56:32 | openstackgerrit | Merged openstack/nova master: Pass the actual target in server group policy https://review.opendev.org/717176 | |
| 14:56:40 | openstackgerrit | Merged openstack/nova master: Add new default roles in server external events policies https://review.opendev.org/717168 | |
| 14:57:55 | stephenfin | gmann: comment on https://review.opendev.org/#/c/717175/10 | |
| 14:58:21 | stephenfin | I could be wrong though | |
| 14:59:38 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175 | |
| 15:00:10 | gmann | stephenfin: i think yes. sorry i update that directly. | |
| 15:07:37 | openstackgerrit | Merged openstack/nova master: Pass the actual target in server external events policy https://review.opendev.org/717169 | |
| 15:41:07 | gmann | johnthetubaguy: one comment, same for extended attr also - https://review.opendev.org/#/c/719730/1/nova/policies/servers.py@133 | |
| 16:09:39 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Update compute rpc version alias for ussuri https://review.opendev.org/719315 | |
| 16:13:56 | gmann | melwitt: lyarwood master patch is merged for functional tox fix, train one is ready - https://review.opendev.org/#/c/719120/ | |
| 16:41:58 | bauzas | can someone tell me why libvirt returns me a former guest that was resized to another host ? | |
| 16:42:15 | bauzas | kashyap: stephenfin: sean-k-mooney: in case you know ? ^ | |
| 16:42:36 | bauzas | tbc, the resize was *confirmed* | |
| 16:42:51 | bauzas | and the allocations diseappeared accordingly | |
| 16:42:57 | bauzas | lyarwood: maybe ? | |
| 16:44:05 | kashyap | bauzas: Hi, need more context. But I'm hungry and just about to cook a meal. | |
| 16:44:13 | bauzas | kashyap: https://github.com/openstack/nova/blob/9001d7e3459d97f507e8ce638d1fc3935401252d/nova/virt/libvirt/driver.py#L7023 | |
| 16:44:16 | kashyap | Trying to maintain a Good Hermit Schedule(tm) :) | |
| 16:46:12 | bauzas | kashyap: anyway, take your time, I'll investigate tonight so we could discuss this tomorrow | |
| 16:46:25 | kashyap | bauzas: Hmm, that requires me to read code and delve deeper into context. I'm just recovering from 3 hours of debugging another libvirt bug | |
| 16:46:30 | kashyap | Yep, thanks! | |
| 17:44:17 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in quota class policies https://review.opendev.org/719100 | |
| 17:49:33 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in quota class policy https://review.opendev.org/720042 | |
| 17:58:08 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in flavor access policy https://review.opendev.org/719607 | |
| 18:06:12 | openstackgerrit | Rafael Weingärtner proposed openstack/nova master: Enhance os-simple-tenant-usage VM hours accounting https://review.opendev.org/711113 | |
| 18:06:12 | openstackgerrit | Rafael Weingärtner proposed openstack/nova master: Enhance os-simple-tenant-usage VM hours accounting https://review.opendev.org/711113 | |
| 18:16:02 | artom | melwitt, I just noticed your have 3 Ts in your Gerrit email | |
| 18:16:22 | artom | Took some head banging to understand why it wasn't accepting my script's query | |
| 18:16:23 | artom | :) | |
| 19:40:17 | bauzas | erm, do folks here have any ideas on how to get a right CONF.host value within a functional test ? | |
| 19:40:51 | bauzas | I mean, I could self.flags it but given it's a global object, all the created computes in the functest will get the same valur | |
| 19:40:53 | bauzas | value* | |
| 20:29:45 | melwitt | bauzas: yes, you do that by passing the 'host' kwarg when you start_service https://github.com/openstack/nova/blob/9001d7e3459d97f507e8ce638d1fc3935401252d/nova/test.py#L381-L391 | |
| 20:30:23 | bauzas | melwitt: sure, I saw it | |
| 20:30:37 | bauzas | melwitt: but for example, I start two computes in a functest | |
| 20:30:42 | bauzas | (using this helper) | |
| 20:31:04 | melwitt | artom: yeah ... the 2 Ts version is owned by someone else and I was uncreative so I just tacked on another t. probably should have picked something else, in hindsight | |
| 20:31:50 | bauzas | melwitt: but then, when I want to confirm resize an instance, the global CONF.host is only having the last host value | |
| 20:31:51 | bauzas | https://github.com/openstack/nova/blob/9001d7e3459d97f507e8ce638d1fc3935401252d/nova/virt/libvirt/driver.py#L1519 | |
| 20:32:47 | melwitt | bauzas: oh, I see. I don't know of a way around that. you would probably need the ghost of mriedem. or maybe gibi might have ideas | |
| 20:33:12 | bauzas | melwitt: so, for example, if I create an instance for host1, resize it to host2 and then confirm the resize, it won't remove the guest in host1 because instance.host (host2) == CONF.host (host2) even I checked that it was calling host1 | |
| 20:33:38 | bauzas | melwitt: no worries, I'll just then modify the CONF before calling confirm_resize | |