| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-10-28 | |||
| 18:39:28 | sean-k-mooney | rm_work: are you using a containerised install | |
| 18:39:34 | sean-k-mooney | or package install | |
| 18:39:35 | rm_work | mixed :P | |
| 18:39:45 | rm_work | we create our own portable venvs for most services | |
| 18:39:52 | rm_work | but placement will be our first actual container | |
| 18:39:54 | sean-k-mooney | ah perfect | |
| 18:40:16 | melwitt | fast-forward is ok but skip-level is not, you could get screwed if data migration code was removed in the middle of what you're skipping | |
| 18:40:17 | sean-k-mooney | i was going to point out the bigest obstical to different version will be oslo requirements | |
| 18:40:26 | sean-k-mooney | venvs or container solve that | |
| 18:40:51 | rm_work | yeah, the thought of installing multiple openstack services in the SAME PYTHON ENV is just ... i know it's halloween, but that's TOO scary | |
| 18:41:07 | sean-k-mooney | rm_work: well we test that in the gate | |
| 18:41:14 | sean-k-mooney | but only on a singel version | |
| 18:41:34 | sean-k-mooney | at a minium you would want 1 python env per release | |
| 18:41:45 | rm_work | yeah honestly I think a lot of stuff will become a lot simpler once people are willing to accept that it just isn't good practice to do IMO T_T | |
| 18:41:49 | sean-k-mooney | but if you are already doing that per servicce then cool | |
| 18:42:13 | rm_work | or rather, if we were willing to make the decision to say "no, one env per service", think of how much simpler G-R management would be, heh | |
| 18:42:34 | sean-k-mooney | well co installablity is currenlty a goal/requirement | |
| 18:42:42 | rm_work | right, but imagine if it weren't! :P | |
| 18:43:00 | sean-k-mooney | packaging would be a pain and you would have to use containers | |
| 18:43:11 | sean-k-mooney | well or venvs | |
| 18:43:21 | sean-k-mooney | so it would make distro lives harder | |
| 18:43:27 | rm_work | well, I guess to be fair packaging venvs is ... not STRICTLY recommended by the pip/venv folks | |
| 18:43:41 | rm_work | seeing as how they keep trying to break it / make it harder to create portable venvs | |
| 18:43:51 | rm_work | and saying "this was never actually intended to work" | |
| 18:44:17 | sean-k-mooney | :) | |
| 18:44:44 | sean-k-mooney | but ya do think about shareing your experince with multiple versions at some point | |
| 21:44:48 | openstack | bug 1731668 in OpenStack Compute (nova) queens "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [Low,In progress] https://launchpad.net/bugs/1731668 - Assigned to melanie witt (melwitt) | |
| 21:44:48 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: [stable-only] Add functional test for bug 1731668 https://review.opendev.org/756636 | |
| 21:44:49 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: [stable-only] Use a separate transaction for reading after race https://review.opendev.org/756637 | |
| #openstack-nova - 2020-10-29 | |||
| 00:47:40 | openstackgerrit | norman shen proposed openstack/nova master: DNM: Cpu and memory live resize https://review.opendev.org/760264 | |
| 08:20:42 | bauzas | good morning Nova | |
| 08:24:45 | gibi | bauzas: o/ | |
| 09:01:55 | lyarwood | Morning all | |
| 09:27:15 | brinzhang_ | lyarwood: morning^ | |
| 09:27:47 | brinzhang_ | lyarwood: hope you can review Add instance_state to ensure volume attachment successful https://review.opendev.org/#/c/758920/ while you have free time | |
| 09:28:20 | openstack | bug 1897424 in OpenStack Compute (nova) "attach volume consume too long time, and if attaching another volume at the same time, create the bdm will be timeout" [Wishlist,Incomplete] https://launchpad.net/bugs/1897424 - Assigned to Lee Yarwood (lyarwood) | |
| 09:28:20 | brinzhang_ | it comes from bug 1897424 | |
| 09:32:16 | lyarwood | brinzhang_: ack yeah it's in my queue | |
| 09:32:33 | lyarwood | brinzhang_: thanks for reminding me :) | |
| 09:33:52 | brinzhang_ | larywood: np, thanks for your concern :D | |
| 11:18:42 | openstackgerrit | Shi Yan proposed openstack/nova master: Remove unused bindir config option https://review.opendev.org/760331 | |
| 11:49:41 | gibi | melwitt, lyarwood, elod: reviewed https://review.opendev.org/#/q/topic:bug/1731668+(status:open+OR+status:merged) and I think this is OK from db/placement perspective. | |
| 12:10:09 | elod | thx gibi, I was just about to comment that 'it looks OK to me, but I would ask for other core members to review it' :) | |
| 12:48:57 | elod | sean-k-mooney: a minor thing: I don't want to merge this with your -1 on it o:) -- https://review.opendev.org/#/c/750925/ | |
| 12:50:18 | sean-k-mooney | elod: oh ill change that we talked about it on the master version | |
| 12:51:23 | sean-k-mooney | i dont really like the change sicne we have never used it for lower constraitns and apparently that is why it was renamed but its was not enought for me to -2 on master and other were happy so its fine | |
| 12:52:01 | sean-k-mooney | elod: +1 now | |
| 12:53:24 | sean-k-mooney | elod: this was the master version https://review.opendev.org/#/c/756135/ | |
| 12:53:48 | sean-k-mooney | it looks like the review bot version is not a cherry pick but a stable only patch | |
| 12:54:21 | sean-k-mooney | but for something this trivail im not sure we care too much that its stable only | |
| 12:59:54 | openstackgerrit | Andrey Volkov proposed openstack/nova master: Functional test test_boot_reschedule_with_proper_pci_device_count https://review.opendev.org/760354 | |
| 13:08:19 | openstack | Launchpad bug 1901739 in OpenStack Compute (nova) " libvirt.libvirtError: internal error: missing block job data for disk 'vda'" [High,Confirmed] | |
| 13:08:19 | kashyap | lyarwood: The libvirt/QEMU folks are rushing hither and thither w/ KVM Forum thing in flight; I'm still tryin to get the right eyes on this: https://bugs.launchpad.net/nova/+bug/1901739 | |
| 13:08:52 | kashyap | lyarwood: Peter Krempa would be the right one, based on my Git analysis; but it's probly a holiday in Czech | |
| 13:10:25 | tosky | kashyap: it was yesterday | |
| 13:10:48 | kashyap | tosky: Oh, sorry. Let me check w/ him again, then :) | |
| 13:10:51 | kashyap | tosky: Thank you | |
| 13:11:27 | tosky | but then people may have used that for longer holidays :) | |
| 13:12:01 | lyarwood | kashyap: ack thanks | |
| 13:58:48 | lyarwood | anyone having issues pushing new patchsets to nova-specs? | |
| 14:27:24 | elod | sean-k-mooney: thanks, I'll +2 then :) | |
| 15:03:11 | melwitt | gibi, elod: thank you both for the reviews! so much appreciated, I know that was a tough one. please lmk if there is anything I can review for you. I will fix the nits in the bottom patch | |
| 15:08:41 | melwitt | lyarwood: easy review for the cherry pick check fix https://review.opendev.org/759118 (it's blocking my stable-only patch, need it back to queens) | |
| 15:13:01 | openstack | bug 1731668 in OpenStack Compute (nova) queens "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [Low,In progress] https://launchpad.net/bugs/1731668 - Assigned to melanie witt (melwitt) | |
| 15:13:01 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: [stable-only] Add functional test for bug 1731668 https://review.opendev.org/756636 | |
| 15:13:02 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: [stable-only] Use a separate transaction for reading after race https://review.opendev.org/756637 | |
| 15:17:05 | lyarwood | melwitt: ack I'll look shortly | |
| 15:18:17 | melwitt | thanks | |
| 15:39:56 | sean-k-mooney | johnthetubaguy: so i would need to find it again but there is a fuse module that allows you to mount http directoes as a local file system | |
| 15:40:25 | sean-k-mooney | so the config drive alternitve was basically mounting metadata on the host and exposing it via cephfs | |
| 15:40:51 | sean-k-mooney | allow a config driver style approch without a config drive for guests without networking or whatever | |
| 15:42:06 | sean-k-mooney | the other cyborg like storage usecause weas using cyborg or something else to manage local storage on the host that can be exposed as fast epmeral storage withoug using the falvor epmeral option | |
| 15:42:37 | sean-k-mooney | that would only be useful if we had the ablity to have multiple device profiles or have a --device attach/detach | |
| 15:43:07 | sean-k-mooney | so right now im not proposing the cyborg approch since you cant contole it outside the flavor | |
| 15:43:17 | sean-k-mooney | also i have other more important things | |
| 15:51:01 | sean-k-mooney | but ya i have wanted to have a cyborg lvm driver for a while that we could use for local storage and testing in the gate since we can test programablity by just downloading the glance image to the volume. | |
| 16:03:11 | openstackgerrit | Merged openstack/os-vif stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/750925 | |
| 16:21:19 | bauzas | lyarwood: (14:58:48) lyarwood: anyone having issues pushing new patchsets to nova-specs? | |
| 16:21:37 | bauzas | lyarwood: oh, are you sure you use SSH over gerrit and not HTTPS ? | |
| 17:01:00 | legochen | hi nova experts, I’d like to understand why nova doesn’t provide provide access control for AZ like what network RBAC does :) | |
| 17:01:34 | lyarwood | bauzas: ssh://lyarwood@review.opendev.org:29418/openstack/nova-specs.git that's the remote it's trying to push to | |
| 17:01:38 | sean-k-mooney | legochen: AZ are just tags on aggreates | |
| 17:01:52 | sean-k-mooney | legochen: you can restrict aggeates to tenatns seperatly | |
| 17:02:23 | sean-k-mooney | legochen: so you kindo of can do this but no one has really asked for it before | |
| 17:02:25 | bauzas | legochen: you have policies on AZs.... | |
| 17:02:39 | sean-k-mooney | bauzas: not to filter the list of azs | |
| 17:02:56 | sean-k-mooney | bauzas: e.g. you cant just retrun the list of azs you can boot too | |
| 17:03:24 | bauzas | sean-k-mooney: really? https://docs.openstack.org/api-ref/compute/?expanded=get-availability-zone-information-detail#availability-zones-os-availability-zone | |
| 17:03:24 | sean-k-mooney | you can restict who can list them or pass them but that it with policy | |
| 17:03:28 | legochen | For example, we use AZ to group hypervisors for multiple different environments. And we only allow A users to use environment A, B users to use environment B. | |
| 17:03:45 | bauzas | legochen: then use aggregate metadata | |
| 17:03:54 | bauzas | lyarwood: looks legit | |
| 17:04:10 | bauzas | lyarwood: I was planning to bump on F33 tonight, will let you know | |
| 17:04:40 | sean-k-mooney | legochen you can do that with https://docs.openstack.org/nova/latest/configuration/config.html#scheduler.limit_tenants_to_placement_aggregate | |
| 17:04:56 | legochen | yes, but the user experience is not so good for that case as they still can see that AZ listed. And when they specify that AZ to create VM, no errors will promot immediately. | |
| 17:04:56 | sean-k-mooney | and https://docs.openstack.org/nova/latest/configuration/config.html#scheduler.placement_aggregate_required_for_tenants | |
| 17:05:05 | legochen | the error happens in scheduling step. | |
| 17:05:24 | legochen | users hard to aware of that. | |
| 17:05:37 | bauzas | anyway, bailing out, wanting to live the 3 last hours before the lockdown out of home | |
| 17:05:59 | sean-k-mooney | legochen: yes that is true we could maybe filter based on the other meatadata | |