| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-07-27 | |||
| 10:30:16 | lyarwood | cool thanks | |
| 10:31:02 | brinzhang | sean-k-mooney: cool, thanks | |
| 10:31:25 | tosky | lyarwood: iirc zuul_copy_output is a dictionary so you can extend it, but it makes sense to add that to devstack-plugin-ceph jobs | |
| 10:32:04 | sean-k-mooney | tosky: right you can add more keys but not remove keys set by parent jobs | |
| 10:32:26 | lyarwood | ack cool I'll add it and remove it from the original with a depends-on now | |
| 11:12:40 | jsuchome | lyarwood: hi, once you have time, please take a look at https://review.opendev.org/#/c/574301/ again, it's rebased on top of recently merged change | |
| 11:34:52 | openstackgerrit | Lee Yarwood proposed openstack/nova master: nova-live-migration: Add encrypted volume to BFV evacuation test https://review.opendev.org/743067 | |
| 11:35:32 | lyarwood | jsuchome: ack will do | |
| 11:36:54 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: libvirt: Handle VIR_ERR_DEVICE_MISSING when detaching devices https://review.opendev.org/742415 | |
| 11:46:30 | sean-k-mooney | brinzhang_: i thinik the current iteration of https://review.opendev.org/#/c/716186/17 is good | |
| 11:47:09 | sean-k-mooney | bauzas: dansmith can ye take a look if ye have time ^ | |
| 12:09:55 | openstackgerrit | Merged openstack/nova master: scheduler: Default request group to None https://review.opendev.org/742651 | |
| 12:12:24 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Further usage of new server helpers https://review.opendev.org/743204 | |
| 12:16:17 | bauzas | sean-k-mooney: ack, will try | |
| 12:16:24 | openstackgerrit | Stephen Finucane proposed openstack/nova master: api: Add type hints for 'nova.compute.api' https://review.opendev.org/741623 | |
| 12:16:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: api: Add 'nova.compute.API.migrate' https://review.opendev.org/741624 | |
| 12:16:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Duplicate '_test_resize' to '_test_migrate' https://review.opendev.org/743206 | |
| 12:16:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Remove unnecessary logic from resize, migrate tests https://review.opendev.org/743207 | |
| 12:16:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: api: Split 'nova.compute.API._resize' https://review.opendev.org/743208 | |
| 12:22:27 | stephenfin | sean-k-mooney: did you see https://review.opendev.org/#/c/738432/ ? | |
| 12:22:37 | stephenfin | full disclosure: I haven't reviewed it yet | |
| 12:26:41 | bauzas | lyarwood: simple question : when we FFU, does the compute still has libvirtd running ? | |
| 12:26:48 | bauzas | when migrating ? | |
| 12:26:56 | bauzas | I don't think so tho | |
| 12:27:24 | lyarwood | bauzas: yeah if the compute is up then libvirtd is running | |
| 12:27:44 | bauzas | lyarwood: ahah, good to know for my offline reshape tool | |
| 12:27:54 | bauzas | so, say you want to upgrade from Stein | |
| 12:28:25 | bauzas | you need to FFU to Victoria, but then you will only have computes that are Stein | |
| 12:28:42 | bauzas | and then you need to upgrade directly before restarting the services | |
| 12:29:14 | bauzas | once you upgrade to Victoria, then you need to upgrade the DB by nova-manage | |
| 12:29:30 | lyarwood | bauzas: no, you upgrade through each release | |
| 12:29:30 | bauzas | and once you're done, then you restart all the compute services | |
| 12:29:40 | lyarwood | bauzas: for the db migrations | |
| 12:29:48 | bauzas | oh yeah of course you're right | |
| 12:29:58 | lyarwood | bauzas: the classic FFU approach is to stop all openstack services on S | |
| 12:30:05 | lyarwood | bauzas: roll through the db migrations to V | |
| 12:30:09 | bauzas | but during this time, do you have libvirtd running ? | |
| 12:30:10 | lyarwood | bauzas: bring the control plane up on V | |
| 12:30:15 | lyarwood | bauzas: and then computes on V | |
| 12:30:23 | bauzas | on computes | |
| 12:30:26 | lyarwood | bauzas: and yeah assuming there's no underlying host upgrade libvirtd will continue to run | |
| 12:30:59 | bauzas | lyarwood: okay, so, we *could* expect libvirtd to be around when we want to nova-manage a specific compute for reshapes | |
| 12:31:06 | bauzas | that's then acceptable | |
| 12:31:20 | sean-k-mooney | stephenfin: not yes no | |
| 12:31:25 | bauzas | if so, that's simplier | |
| 12:31:25 | lyarwood | bauzas: are you running nova-manage on the compute itself | |
| 12:31:36 | bauzas | lyarwood: I'm about to provide a new spec for it | |
| 12:31:41 | bauzas | for reshapes | |
| 12:32:03 | bauzas | lyarwood: anyway, I'll upload my new revision and you could review it | |
| 12:32:15 | lyarwood | bauzas: yeah happy to | |
| 12:35:05 | sean-k-mooney | * not yet | |
| 12:38:32 | lyarwood | jsuchome: the change LGTM now, did we talk about getting this tested somewhere in the gate previously? | |
| 12:38:54 | lyarwood | jsuchome: would you be able to wire it up in the ceph job? | |
| 12:39:07 | sean-k-mooney | stephenfin: im not sure how there patch and that bug are related | |
| 12:40:00 | sean-k-mooney | i need to check but i did not think we used the devname | |
| 12:40:15 | sean-k-mooney | we should be using just the pci address in nova | |
| 12:41:02 | openstackgerrit | Sylvain Bauza proposed openstack/nova-specs master: Offline Reshape tool spec https://review.opendev.org/742908 | |
| 12:42:02 | jsuchome | lyarwood: I think that was that test job you created previously ... I lost the link | |
| 12:43:21 | jsuchome | lyarwood: I think this one https://review.opendev.org/#/c/727225/ ? | |
| 12:43:59 | sean-k-mooney | hum i guess we do use the device name | |
| 12:44:27 | jsuchome | lyarwood: for release nodes we have this I think https://review.opendev.org/#/c/728095/ ... | |
| 12:50:15 | lyarwood | jsuchome: yeah I wonder if we want to enable this all of the time in the ceph job? | |
| 12:51:11 | jsuchome | lyarwood: unfortunatelly I do not really know how this test infrustructure works ... I remember the job always failed but it does not seem to be related to the change | |
| 12:51:31 | lyarwood | jsuchome: k, let me just wire it up quickly | |
| 12:51:48 | lyarwood | jsuchome: ack on the other change having a releasenote for this | |
| 12:51:54 | lyarwood | jsuchome: as long as it's somewhere I'm okay | |
| 12:52:57 | jsuchome | you even commented back in May about the possible reason for the job failure (https://review.opendev.org/#/c/727225/) | |
| 12:54:51 | bauzas | lyarwood: the spec I raised to you https://review.opendev.org/742908 | |
| 12:55:01 | bauzas | (the offline reshape one) | |
| 12:56:44 | lyarwood | bauzas: ack thanks queued | |
| 12:59:56 | artom | What's the reverse of Depends-On again? | |
| 12:59:59 | artom | Used-by? | |
| 13:00:58 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add ability to download Glance images into the libvirt image cache via RBD https://review.opendev.org/574301 | |
| 13:00:59 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Enable [glance]/allowed_direct_url_schemes within nova-ceph-multistore https://review.opendev.org/743220 | |
| 13:01:00 | lyarwood | jsuchome: ^ I had to rebase that change to pick up nova-ceph-multistore | |
| 13:01:46 | lyarwood | hmm acually that job uses ceph for everything so I'm not sure if this is going to exercise it correctly | |
| 13:01:56 | lyarwood | now I remember why I had to create that other job | |
| 13:02:15 | lyarwood | well at least we can show it doesn't break nova-ceph-multistore now | |
| 13:02:30 | artom | Ah, Needed-by | |
| 13:04:54 | jsuchome | lyarwood: ok, let's see how it's gonna fail :-) | |
| 13:12:33 | mnaser | has anyone seen this before? | |
| 13:12:38 | mnaser | select count(*) from key_pairs where user_id='b1807b548e8a422ab74a05fe52f42ea1'; => 100 | |
| 13:12:46 | mnaser | nova quota-show --user b1807b548e8a422ab74a05fe52f42ea1 --detail => in_use = 0 | |
| 13:12:56 | mnaser | i'm going to go start digging into this now but in case someone has just.. ran into it | |
| 13:13:35 | artom | mnaser, stab in the dark, but I assume you've checked soft-deleted stuff? | |
| 13:14:15 | mnaser | artom: looks like keypairs are not soft deleted | |
| 13:14:17 | mnaser | no deleted_at column | |
| 13:14:30 | artom | mnaser, ack - as I said, stab in the dark | |
| 13:14:35 | artom | Sorry I can't be of more help :( | |
| 13:14:48 | mnaser | yeah i'm going to dig into where the api gets those numbers from | |
| 13:14:53 | mnaser | i dont think the api counts | |
| 13:15:24 | artom | mnaser, melwitt might have an idea, when she comes online in a few hours | |
| 13:16:34 | mnaser | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/quota_sets.py#L81-L82 | |
| 13:18:23 | mnaser | ok i think i see the issue here | |
| 13:18:30 | mnaser | i need to provide _both_ tenant id and user_id | |
| 13:18:38 | mnaser | because it tries to get quota of user in context of a specific tenant | |
| 13:19:12 | mnaser | and i assume this was trying to check user_id in scope of admin tenant | |
| 13:23:41 | mnaser | https://github.com/openstack/nova/blob/eeeb964a5f65e6ac31dfb34b1256aaf95db5ba3a/nova/quota.py#L178-L182 | |
| 13:23:42 | mnaser | oh | |
| 13:25:04 | sean-k-mooney | didnt we stop applying quotas for keypairs as part of unified limits? | |