| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-02-18 | |||
| 23:05:26 | spatel | taking* | |
| 23:46:12 | spatel | johnsom all my problem solved :) | |
| 23:46:25 | johnsom | spatel Oh good | |
| 23:46:38 | spatel | after setting driver = noop in nova-api and nova-compute* nodes | |
| 23:47:20 | spatel | my issue is nova still looking for designate.info queue which doesn't exist and that was causing all chaos | |
| 23:48:02 | spatel | I need to work on designate to find out what is going on but i believe interim solution is to tell nova don't look for notification queue | |
| 23:49:19 | spatel | anyway! time for dinner! see you tomorrow.. thank you for your support. | |
| #openstack-nova - 2021-02-19 | |||
| 00:47:27 | openstackgerrit | Jorhson Deng proposed openstack/nova master: Reject to attach and detach volume while task_state is not None https://review.opendev.org/c/openstack/nova/+/758918 | |
| 02:18:05 | openstackgerrit | Merged openstack/nova master: Add regression test for bug 1914777 https://review.opendev.org/c/openstack/nova/+/775307 | |
| 02:18:07 | openstack | bug 1914777 in OpenStack Compute (nova) "Possible race condition between n-cpu and n-api when deleting a building instance" [High,In progress] https://launchpad.net/bugs/1914777 - Assigned to melanie witt (melwitt) | |
| 02:18:25 | openstackgerrit | Merged openstack/nova stable/ussuri: tools: Allow check-cherry-picks.sh to be disabled by an env var https://review.opendev.org/c/openstack/nova/+/766024 | |
| 04:47:35 | openstackgerrit | melanie witt proposed openstack/nova master: Handle instance = None in _local_delete_cleanup https://review.opendev.org/c/openstack/nova/+/775308 | |
| 07:02:05 | openstackgerrit | Merged openstack/nova stable/ussuri: libvirt: Only ask tpool.Proxy to autowrap vir* classes https://review.opendev.org/c/openstack/nova/+/761223 | |
| 07:41:04 | openstackgerrit | Yongli He proposed openstack/nova master: smartnic support https://review.opendev.org/c/openstack/nova/+/758944 | |
| 07:57:26 | yonglihe | sean-k-mooney: review invite, thanks: https://review.opendev.org/q/topic:%22bp%252Fsriov-smartnic-support%22+(status:open%20OR%20status:merged) | |
| 08:02:13 | yonglihe | gibi: review invite, thanks. https://review.opendev.org/q/topic:%22bp%252Fsriov-smartnic-support%22+(status:open%20OR%20status:merged) | |
| 08:05:11 | gibi | yonglihe: I will try | |
| 08:06:38 | yonglihe | thanks, it's kind late for this release . lots of depend-on items, good news is that's all got merged(neutron and cyborg). | |
| 08:10:22 | gibi | yonglihe: ack, honestly I have a lot of thing to review so I might not be able to get to it this week | |
| 08:12:25 | yonglihe | ack -:) | |
| 09:07:51 | lyarwood | elod: https://review.opendev.org/c/openstack/nova/+/761224 - would you mind hitting this today, downstream finally ran into it so I'm getting chased around with a pitchfork. ^_^ | |
| 09:14:56 | elod | lyarwood: sure, looking :) | |
| 09:23:58 | openstackgerrit | Jorhson Deng proposed openstack/nova master: Reject to attach and detach volume while task_state is not None https://review.opendev.org/c/openstack/nova/+/758918 | |
| 09:24:07 | bauzas | gibi: stephenfin: eek, now I added nova/network/neutron.py in the mypy files, looks to me I need to add *all* the type hints for all the methods /o\ | |
| 09:24:21 | bauzas | gibi: stephenfin: context, Zuul is -1 for pep8 https://review.opendev.org/c/openstack/nova/+/773976 | |
| 09:26:32 | elod | lyarwood: patch looks good, +2+W'd | |
| 09:27:29 | melwitt | bauzas: mypy is now your life | |
| 09:28:02 | bauzas | :( | |
| 09:28:13 | bauzas | I was already loud about mypy | |
| 09:28:18 | bauzas | so i won't tell more | |
| 09:28:39 | melwitt | it's ok, I was just trying to be funny | |
| 09:28:40 | bauzas | but next time I'm asked to provide type hints for a module, I'll lookup whether the file is already checked | |
| 09:29:05 | bauzas | melwitt: no worries, I prefer american pies | |
| 09:29:55 | lyarwood | bauzas: -1 nit did you mean bauzas(self) -> 'SadPanda': ? | |
| 09:30:04 | melwitt | lol | |
| 09:30:09 | bauzas | excellent | |
| 09:30:28 | bauzas | AttibutError : object 'SadPanda' does not exist | |
| 09:31:56 | bauzas | but yeah, mypy is like a cold sea, you just step into it by the toe first | |
| 09:32:15 | bauzas | or you'll get cold | |
| 09:35:31 | bauzas | but at least I can reproduce it locally \o/ | |
| 09:41:18 | swp20 | stephenfin: morning | |
| 09:42:30 | swp20 | for the noVNC patch, can we add TLS in the follow change? as the bp's goal is add the password to get vnc url. | |
| 09:51:48 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Add net & utils methods for routed nets & segments https://review.opendev.org/c/openstack/nova/+/773976 | |
| 09:51:49 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Add a routed networks scheduler pre-filter https://review.opendev.org/c/openstack/nova/+/749068 | |
| 09:52:04 | bauzas | stephenfin: gibi: eventually gave up on fixing neutron issues | |
| 09:52:07 | bauzas | (for mypy) | |
| 09:56:46 | openstackgerrit | Wenping Song proposed openstack/nova master: Nova supports password encrypted VNC https://review.opendev.org/c/openstack/nova/+/622336 | |
| 10:00:17 | openstackgerrit | Lucian Petrut proposed openstack/nova master: hyper-v rbd volume support https://review.opendev.org/c/openstack/nova/+/763550 | |
| 10:06:02 | brinzhang | by password, if we want to add the x509vnc support, we can do that later, rather than in this spec | |
| 10:06:02 | brinzhang | stephenfin, swp20: yes, this spec we just want to add the vnc authenticate | |
| 10:08:00 | brinzhang | stephenfin, swp20: as your point in https://review.opendev.org/c/openstack/nova/+/622336/30/nova/console/rfb/auths.py#30 | |
| 10:14:51 | stephenfin | brinzhang: swp20: I need to think about it more and read melwitt's replies. I don't understand why someone would opt for basic VNC auth when X509+VNC is an option. Using just basic VNC auth makes this feature effectively useless for deployments with encryption, which is virtually all TripleO deployments at least | |
| 10:15:16 | stephenfin | brinzhang: Surely you folks aren't going to recommend people use VNC and drop encryption? | |
| 10:22:51 | lyarwood | stephenfin / gibi ; https://review.opendev.org/c/openstack/nova/+/774897/ - Would you mind looking at this again today and highlighting which approach you think I should take here (o.vo vs sqla)? I'm not sure how I can test this on an actual env at scale before we merge tbh. | |
| 10:28:01 | lyarwood | urgh lc-- pip-- how does it pass locally and then fail in the gate | |
| 10:29:05 | stephenfin | sure | |
| 10:32:51 | brinzhang | stephenfin: I know you want to encrypt identities and files, but this is not my original intention of designing this feature. For the users I have seen, vnc password authentication is sufficient. | |
| 10:33:25 | brinzhang | stephenfin: but as you said, using x509+vnc is an optional, we can try | |
| 10:37:05 | brinzhang | s/optional/option/ | |
| 11:01:14 | gibi | bauzas: have you tried the routed net patches in a devstack? I think it is cannot be activated as the extension name is wrong in the code https://review.opendev.org/c/openstack/nova/+/773976/9/nova/network/constants.py#32 | |
| 11:02:01 | gibi | bauzas: I can give a try to adding mypy needs to neutron.py | |
| 11:59:01 | openstackgerrit | Merged openstack/nova stable/train: tools: Allow check-cherry-picks.sh to be disabled by an env var https://review.opendev.org/c/openstack/nova/+/766025 | |
| 12:39:47 | openstackgerrit | Lee Yarwood proposed openstack/nova master: block_device: Use initialize APIs to refresh when reported as idempotent https://review.opendev.org/c/openstack/nova/+/720769 | |
| 13:14:30 | bauzas | gibi: unfortunately no, I don't have an env with multiple nodes at hand | |
| 13:14:33 | bauzas | ... | |
| 13:42:03 | openstackgerrit | Lucian Petrut proposed openstack/nova master: hyper-v rbd volume support https://review.opendev.org/c/openstack/nova/+/763550 | |
| 13:57:00 | lyarwood | dansmith / gmann / gibi ; https://review.opendev.org/c/openstack/tempest/+/771499 - This came up during the meeting yesterday, IMHO we should go ahead with this now if multinode provides the same coverage | |
| 14:09:36 | stephenfin | bauzas: It should be pretty easy to set one up with DevStack and VMs, if you have something with 32 GB of RAM | |
| 14:10:39 | stephenfin | If not, sean-k-mooney might be able to lend you a hand? | |
| 14:16:20 | gibi | bauzas: ack, I will try to validate as much of the assumption the functional test cases makes as possible in a devstack | |
| 14:16:30 | gibi | right now I saw the problem with the extension name | |
| 14:17:05 | gibi | and strugling with neutron to produce the assumed placement config (PRs, inventories, aggregates) and nova config (aggregates) for the networks | |
| 14:19:03 | gibi | lyarwood: I'm OK to drop the base grenade | |
| 14:20:44 | sean-k-mooney | i see my name | |
| 14:20:46 | sean-k-mooney | reading | |
| 14:21:34 | sean-k-mooney | bauzas: i have got 3-4 nodes on my p50 laptop before but if you need nodes i think i still have space on my home cloud | |
| 14:21:47 | sean-k-mooney | bauzas: i can give you a log in and you can install away | |
| 14:22:10 | bauzas | stephenfin: gibi: sean-k-mooney: my laptop (a t470s) only has 16GB of RAM | |
| 14:22:17 | dansmith | lyarwood: cool with me! | |
| 14:22:22 | sean-k-mooney | bauzas: i have found the min ram size for contolers is about 6G and comptue is about 2-4 | |
| 14:22:30 | bauzas | I also have two mini optiplexs but they also have 16GB | |
| 14:23:03 | bauzas | and I'm waiting for my new laptop since beginning of december, but the provider is running out of stock till' March :) | |
| 14:23:15 | bauzas | things are going crazy those days with CPUs and GPUs | |
| 14:23:25 | bauzas | same with bikes fwiw | |
| 14:24:34 | gibi__ | hm, my raspberry hosting my irc client just died | |
| 14:25:29 | sean-k-mooney | bauzas: looking at my cloud i currently have about 35GB free if you need to spin up a couple of 8G vms | |
| 14:26:07 | bauzas | sean-k-mooney: well, spinning up a devstack env with multiple nodes would take me a couple of days | |
| 14:26:26 | bauzas | if someone can verify the segments extension for me, this should be enough IMHO | |
| 14:26:44 | gibi__ | bauzas: I'm on that | |
| 14:26:51 | bauzas | gibi__: <3 | |
| 14:27:18 | bauzas | I'm enough confident in my functional testing | |
| 14:27:48 | bauzas | but yeah, this (the segments extension name) wasn't verified | |
| 14:27:56 | gibi__ | bauzas: yes, I also think that the funct test are plenty, I just need to verify the assumptions the func test makes about neutron | |
| 14:28:01 | sean-k-mooney | bauzas: ok i can get you a 2 node deployment in my cloud in a hour or so. | |
| 14:28:03 | gibi__ | like the name of the extension | |
| 14:28:21 | sean-k-mooney | bauzas: oh you just want that | |
| 14:28:30 | sean-k-mooney | i cans show you were its defiend | |
| 14:28:55 | sean-k-mooney | https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/segment.py#L32 | |
| 14:29:28 | bauzas | https://docs.openstack.org/neutron/latest/contributor/internals/segments.html | |
| 14:29:37 | bauzas | it says SEGMENT with capitals | |