| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-07-28 | |||
| 17:30:31 | dansmith | the superconductor will be the same because templated urls are never going to work if you have DBs on different hostnames but try to use {hostname} since that will only ever resolve to one | |
| 17:30:51 | dansmith | so you probably need to just not use {hostname} in the URLs if the DBs are at specific locations | |
| 17:31:06 | dansmith | if you're using DNS names pointing to VIPs like cell1.foo and cell0.foo, then you don't need the template at all right? | |
| 17:31:59 | owalsh_ | full stack doesn't give any more info but I'll post it. It's called from utils.raise_if_old_compute() in service create() AFAICT | |
| 17:32:40 | dansmith | okay, but need to know what's being done to know who is calling that and for why | |
| 17:33:01 | dansmith | but if we want to exclude cell0 from that check (which *is* specific to computes) we should do it there and not in this generalized method | |
| 17:33:19 | dansmith | however, like I say, that's just punting the can down the road, because a superconductor arrangement like this with multiple cells will not work | |
| 17:33:46 | owalsh_ | ack, in tripleo superconductor will use the same DB server as cell0 so it happens to work | |
| 17:34:12 | dansmith | okay | |
| 17:34:31 | dansmith | but if you use per-db hostnames, then no need for the {hostname} in the template at all right? | |
| 17:34:53 | dansmith | (per-db hostnames being far superior, of course) | |
| 17:35:17 | dansmith | the template was really for the credentials and not the other things, I just added every part of the url into the templating engine for flexibility | |
| 17:41:03 | sean-k-mooney[m] | im having some issues with my work laptop so im going to set up my personal one quickly so ill be on matrix for a bit rather then irc | |
| 17:41:11 | sean-k-mooney[m] | i should see either ping | |
| 17:41:15 | sean-k-mooney[m] | but just an fyi | |
| 17:43:20 | gibi_pto | I will be off tomorrow, but back on Monday | |
| 17:53:23 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove double mocking https://review.opendev.org/c/openstack/nova/+/851445 | |
| 17:54:56 | gibi_pto | stephenfin, sean-k-mooney[m]: ^^ it is just a partial fix but I run out of time today and I will be off tomorrow so I pushed it. feel free to take over tomorrow if you have time | |
| 17:59:25 | sean-k-mooney[m] | gibi_pto: ack | |
| 18:00:42 | owalsh_ | dansmith: nova-conductor logs - https://paste.openstack.org/show/b85LsPayPl2QYN6RrDi1/, just noticed that it times out waiting for cell0 after 1 minutes so not fatal | |
| 18:03:11 | owalsh_ | dansmith: even if we just switch to the VIP hostnames might have issues if there is a firewall between the cell conductor and the cell0 db | |
| 18:06:12 | dansmith | owalsh_: yeah, so making that compute-specific check might be good, just needs to be one level up | |
| 18:06:16 | dansmith | I'll look at the logs in just a sec | |
| 18:08:48 | dansmith | owalsh_: yeah it should handle the failure gracefully | |
| 18:09:35 | dansmith | owalsh_: that's not enough to tell what's going on because of how the stack frame gets chopped for a thread, unfortunately | |
| 18:09:44 | dansmith | can you correlate that req-id with the api logs or something? | |
| 18:18:19 | dansmith | owalsh_: nm, I got it.. on service startup | |
| 18:40:12 | owalsh_ | dansmith: hrmm, I guess it's not just the hostname that could be an issue. If cell0 username/password is different to the cell nova db it would also fail | |
| 18:41:04 | dansmith | owalsh_: yep for sure, that's one of the compromises | |
| 18:41:18 | dansmith | but the assertion during design was that credentials are more per-host and less per-database | |
| 18:43:15 | owalsh_ | ack, IIRC for tripleo it's really just the {query} part that mattered as it included the local bind address for each host | |
| 18:43:26 | dansmith | ack | |
| 18:43:28 | owalsh_ | but now that has moved to a local conf file (pymysql version didn't support this at the time) | |
| 18:44:09 | owalsh_ | so could maybe just stop using cell templates, will take a look | |
| 18:44:23 | dansmith | yeah, if possible I'm sure that'd be better | |
| 19:09:11 | opendevreview | Merged openstack/nova master: api: Drop generating a keypair and add special chars to naming https://review.opendev.org/c/openstack/nova/+/849133 | |
| 20:19:13 | owalsh_ | dansmith: re the api db connection currently being set on the cell conductor, I guess we want to keep that? | |
| 20:37:11 | dansmith | owalsh_: "we" being redhat yeah, because we want the affinity check to work | |
| #openstack-nova - 2022-07-29 | |||
| 00:33:40 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273 | |
| 03:39:58 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/851337 | |
| 05:28:28 | whoami-rajat | dansmith, i rebased last night and another nova microversion merged ... can i just rebase it when i start getting reviews? else i will be in the loop of rebasing for new microversion again and again ... https://review.opendev.org/c/openstack/nova/+/830883 | |
| 07:26:32 | auniyal | camera | |
| 07:58:27 | bauzas | whoami-rajat: I was having a plan to contributors to ask for microversions with an etherpad https://etherpad.opendev.org/p/nova-zed-microversions-plan | |
| 07:59:04 | bauzas | whoami-rajat: today, I'll send an email explaining for contributors to propose their changes so we would triage those in our next nova meeting on Tuesday | |
| 07:59:41 | whoami-rajat | bauzas, ack, i see 2.94 being vacant, should i take that? | |
| 07:59:58 | bauzas | whoami-rajat: for the moment, add your proposal in the proposals list | |
| 08:00:07 | bauzas | I'll a bit modify the etherpad | |
| 08:00:23 | whoami-rajat | ack, will add it there | |
| 08:00:24 | whoami-rajat | thanks | |
| 08:01:27 | whoami-rajat | done | |
| 10:02:09 | bauzas | interesting | |
| 10:02:10 | bauzas | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_fb7/851244/2/check/osc-functional-devstack/fb7abbc/testr_results.html | |
| 10:02:19 | bauzas | osc functional tests default to 'latest' | |
| 10:02:29 | bauzas | sean-k-mooney[m]: ^ | |
| 10:03:24 | bauzas | sean-k-mooney[m]: which makes OSC by default asks for latest | |
| 10:04:46 | bauzas | yeah confirmed https://github.com/openstack/python-openstackclient/blob/master/openstackclient/tests/functional/base.py#L62-L65 | |
| 10:53:29 | songwenping | sean-k-mooney: hi, during live migration, DVR floating ip bound the vms on source and dest node, that leads disconnect to this floating ip, do you know this scence? | |
| 11:39:23 | opendevreview | Amit Uniyal proposed openstack/nova master: Updated Suspend definition in server concepts doc https://review.opendev.org/c/openstack/nova/+/851511 | |
| #openstack-nova - 2022-07-30 | |||
| 14:40:09 | opendevreview | Merged openstack/nova stable/yoga: neutron: Unbind remaining ports after PortNotFound https://review.opendev.org/c/openstack/nova/+/842584 | |
| #openstack-nova - 2022-08-01 | |||
| 02:39:19 | redengin | I'm trying to start up a compute node, but when it connects to the controller it fails with "openstack.exceptions.NotSupported: The placement service for 192.168.24.3:regionOne exists but no version was discoverable." - any ideas on steps to debug? | |
| 06:10:05 | tobias-urdin | maybe check that the url for placement in the keystone catalog is correct and goes to the default path which lists versions | |
| 07:09:52 | gibi | o/ | |
| 07:48:12 | tobias-urdin | o/ | |
| 07:55:28 | Uggla | o/ | |
| 08:21:52 | bauzas | \o | |
| 09:29:32 | whoami-rajat | hi bauzas , just checking if there is any update on the MV part? https://etherpad.opendev.org/p/nova-zed-microversions-plan | |
| 09:30:26 | bauzas | whoami-rajat: I'll send the email I promised | |
| 09:30:35 | whoami-rajat | ack, thanks | |
| 09:42:26 | bauzas | done | |
| 09:47:22 | elodilles | bauzas sean-k-mooney fyi, the long waited pike-eol patch for nova: https://review.opendev.org/c/openstack/releases/+/851571 :) | |
| 09:48:55 | bauzas | elodilles: done | |
| 09:49:36 | elodilles | bauzas: thx \o/ | |
| 10:58:06 | opendevreview | Pierre Riteau proposed openstack/nova stable/ussuri: Allow Train compute hosts to register with Ussuri https://review.opendev.org/c/openstack/nova/+/851732 | |
| 11:24:17 | opendevreview | Pierre Riteau proposed openstack/nova stable/ussuri: Allow Train compute hosts to register with Ussuri https://review.opendev.org/c/openstack/nova/+/851732 | |
| 14:14:38 | kashyap | gibi: bauzas: elodilles: Can any of you please put this backport through? - https://review.opendev.org/c/openstack/nova/+/845045 (libvirt: Add a workaround to skip compareCPU() on destination) | |
| 14:14:43 | kashyap | (Or anyone else) | |
| 14:14:55 | kashyap | gibi: You already reviewed it in the past, and helped w/ the UT :) | |
| 14:17:37 | gibi | I did my part :) | |
| 14:17:44 | gibi | now you need a second core | |
| 14:21:15 | kashyap | Thank you! :) | |
| 14:36:52 | elodilles | damn, too, many reviewers :) | |
| 14:37:03 | elodilles | double approved :S | |
| 14:39:11 | bauzas | damn shit. | |
| 14:39:12 | bauzas | https://stackoverflow.com/questions/893333/multiple-variables-in-a-with-statement | |
| 14:39:38 | bauzas | context : https://review.opendev.org/c/openstack/python-openstackclient/+/851244/3/openstackclient/tests/functional/compute/v2/test_keypair.py#140 | |
| 14:42:33 | gibi | bauzas: yepp that is a fairly recent syntax | |
| 14:43:16 | bauzas | wasn't told by pep8 locally | |
| 14:43:20 | bauzas | osc-- | |
| 14:47:26 | bauzas | last time I used nested withs, I was using contextlib stdlib | |
| 14:47:30 | bauzas | and it was working great | |
| 14:47:53 | bauzas | py27, I still love you <3 | |
| 14:48:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove double mocking https://review.opendev.org/c/openstack/nova/+/851445 | |
| 14:48:53 | gibi | stephenfin, sean-k-mooney, efried: ^^ that is the fix for the double mocking in py310, now I going to rebase the whole mock series as we landed things on master that needs care... | |
| 14:50:32 | efried | <3 | |
| 15:03:16 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use unittest.mock instead of third party mock https://review.opendev.org/c/openstack/nova/+/714676 | |
| 15:03:17 | opendevreview | Balazs Gibizer proposed openstack/nova master: hacking: force explicit import of python's mock https://review.opendev.org/c/openstack/nova/+/708768 | |
| 15:03:17 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove double mocking https://review.opendev.org/c/openstack/nova/+/851445 | |
| 15:03:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove the PowerVM driver https://review.opendev.org/c/openstack/nova/+/850346 | |
| 15:12:42 | bauzas | gibi: do you think I should provide a new revision for https://review.opendev.org/c/openstack/nova/+/838976 ? | |
| 15:12:57 | bauzas | I'll be off by tomorrow EOB | |