*** openstack has joined #openstack-nova | 00:22 | |
*** ChanServ sets mode: +o openstack | 00:22 | |
*** swp20 has quit IRC | 00:47 | |
*** mgoddard has quit IRC | 00:56 | |
*** sapd1 has joined #openstack-nova | 00:57 | |
*** sapd1 has quit IRC | 01:02 | |
*** mgoddard has joined #openstack-nova | 01:04 | |
*** MrClayPole_ has quit IRC | 01:07 | |
openstackgerrit | liuzhuangzhuang proposed openstack/nova master: Fix RBD timeout https://review.opendev.org/c/openstack/nova/+/786588 | 01:11 |
---|---|---|
*** MrClayPole has joined #openstack-nova | 01:18 | |
*** bbowen has joined #openstack-nova | 01:23 | |
*** hamalq has quit IRC | 01:30 | |
*** iurygregory has quit IRC | 01:48 | |
*** sapd1 has joined #openstack-nova | 01:55 | |
*** ricolin_ is now known as ricolin | 02:05 | |
*** sapd1_x has joined #openstack-nova | 02:12 | |
*** ricolin has quit IRC | 02:27 | |
*** ricolin_ has joined #openstack-nova | 02:31 | |
*** gyee has quit IRC | 02:31 | |
*** ricolin_ is now known as ricolin | 02:32 | |
*** ricolin has quit IRC | 02:39 | |
*** swp20 has joined #openstack-nova | 02:49 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Update Servers APIs https://review.opendev.org/c/openstack/nova/+/764292 | 03:11 |
openstackgerrit | liuzhuangzhuang proposed openstack/nova master: Fix RBD timeout https://review.opendev.org/c/openstack/nova/+/786588 | 03:16 |
*** pmannidi has joined #openstack-nova | 03:29 | |
*** pmannidi has quit IRC | 03:34 | |
*** ociuhandu has joined #openstack-nova | 03:47 | |
*** ociuhandu has quit IRC | 03:51 | |
*** ricolin has joined #openstack-nova | 03:56 | |
*** Alon_KS has quit IRC | 04:10 | |
*** Alon_KS has joined #openstack-nova | 04:14 | |
*** k_mouza has joined #openstack-nova | 04:16 | |
*** k_mouza has quit IRC | 04:21 | |
*** ratailor has joined #openstack-nova | 04:29 | |
*** aarents has quit IRC | 04:42 | |
*** swp20 has quit IRC | 04:42 | |
*** mkrai has joined #openstack-nova | 04:46 | |
*** jawad_axd has joined #openstack-nova | 05:01 | |
*** ralonsoh has joined #openstack-nova | 05:14 | |
*** sapd1 has quit IRC | 05:21 | |
*** mkrai_ has joined #openstack-nova | 05:22 | |
*** ftarasenko has joined #openstack-nova | 05:24 | |
*** mkrai has quit IRC | 05:25 | |
gibi | sean-k-mooney: regarding provider.yaml; I think technically it is possible to report bw and pps inventories via that. As a downside of it the neutron agents already have the bridge_mapping and physical_device_mappings in their config file as we use that to figure out which physnet traits needs to be reported | 06:07 |
gibi | sean-k-mooney: if you do the physnet trait reporting in provider.yaml then you manually need to make sure that it is in sync with the bridge mappings | 06:08 |
gibi | sean-k-mooney: I don't know how ovn does the bridge mapping so there it might not be a problem | 06:08 |
gibi | the vnic_type trait has a similar consequence | 06:09 |
gibi | lyarwood: hi! I | 06:22 |
gibi | lyarwood: hi! I'm looking at the next min libvirt and qemu versions, does that mean we will switch to those in Xena? | 06:23 |
gibi | lyarwood: as far as I see 20.04 still on libvirt 6.0.0 and our next minimum is 7.0.0 | 06:24 |
*** slaweq has joined #openstack-nova | 06:26 | |
*** iurygregory has joined #openstack-nova | 06:28 | |
openstackgerrit | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | 06:32 |
openstackgerrit | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | 06:32 |
openstackgerrit | Yongli He proposed openstack/nova master: smartnic support https://review.opendev.org/c/openstack/nova/+/758944 | 06:32 |
openstackgerrit | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | 06:32 |
openstackgerrit | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | 06:32 |
*** ratailor_ has joined #openstack-nova | 06:42 | |
*** ratailor has quit IRC | 06:44 | |
*** aarents has joined #openstack-nova | 06:52 | |
*** ircuser-1 has joined #openstack-nova | 06:52 | |
openstackgerrit | liuzhuangzhuang proposed openstack/nova master: Clean up instance allocations if the host is not in any cell https://review.opendev.org/c/openstack/nova/+/792322 | 06:56 |
*** jkulik is now known as jkulik_ | 07:01 | |
*** jkulik_ is now known as jkulik | 07:02 | |
*** lucasagomes has joined #openstack-nova | 07:05 | |
*** jkulik has quit IRC | 07:06 | |
*** jkulik has joined #openstack-nova | 07:06 | |
*** mkrai_ has quit IRC | 07:06 | |
*** jkulik has quit IRC | 07:12 | |
*** jkulik has joined #openstack-nova | 07:12 | |
*** dpawlik8 is now known as dpawlik | 07:20 | |
*** rpittau|afk is now known as rpittau | 07:24 | |
*** avolkov has joined #openstack-nova | 07:26 | |
*** fyx has quit IRC | 07:34 | |
*** donnyd has quit IRC | 07:34 | |
*** gouthamr has quit IRC | 07:34 | |
slaweq | gibi: hi, can You check https://review.opendev.org/c/openstack/nova/+/787252 when You will have some time? | 07:35 |
slaweq | thx in advance | 07:35 |
*** erbarr has quit IRC | 07:35 | |
*** cz3 has quit IRC | 07:35 | |
gibi | slaweq: sure | 07:36 |
*** mnaser has quit IRC | 07:36 | |
*** guilhermesp has quit IRC | 07:36 | |
slaweq | thx | 07:36 |
*** gmann has quit IRC | 07:36 | |
*** nicolasbock has quit IRC | 07:36 | |
*** bbezak has quit IRC | 07:36 | |
gibi | slaweq: sorry I have no +2 right on stable branches I will leave my +1 | 07:36 |
slaweq | gibi: ok, I though You have | 07:37 |
*** bbezak has joined #openstack-nova | 07:37 | |
*** mnaser has joined #openstack-nova | 07:37 | |
*** nicolasbock has joined #openstack-nova | 07:37 | |
*** fyx has joined #openstack-nova | 07:37 | |
*** gouthamr has joined #openstack-nova | 07:37 | |
gibi | I never needed one :) I trust the stable cores in the team | 07:37 |
*** cz3 has joined #openstack-nova | 07:37 | |
slaweq | :) | 07:37 |
*** erbarr has joined #openstack-nova | 07:38 | |
*** donnyd has joined #openstack-nova | 07:38 | |
*** guilhermesp has joined #openstack-nova | 07:38 | |
*** gmann has joined #openstack-nova | 07:38 | |
yonglihe | Hi, gibi, alex_xu, sean, smartnic patch set is revised per approved spec, and patch set link: https://review.opendev.org/q/topic:%22bp%252Fsriov-smartnic-support%22+(status:open%20OR%20status:merged) | 07:38 |
yonglihe | To make your day easy, here is a change list since last review: | 07:38 |
yonglihe | https://www.irccloud.com/pastebin/9jnZwn6w/ | 07:38 |
gibi | yonglihe: thanks. I will try to get to it | 07:39 |
yonglihe | many thanks | 07:40 |
*** tosky has joined #openstack-nova | 07:46 | |
*** rcernin has quit IRC | 07:52 | |
*** ociuhandu has joined #openstack-nova | 07:53 | |
*** dtantsur|afk is now known as dtantsur | 08:03 | |
*** derekh has joined #openstack-nova | 08:07 | |
bauzas | I just get one half-day off and Freenode gets on fire... | 08:07 |
bauzas | man, I'm tempted to not take my usual 3 weeks PTO | 08:08 |
*** sapd1 has joined #openstack-nova | 08:09 | |
*** macz_ has joined #openstack-nova | 08:10 | |
kashyap | bauzas: Don't be silly; are you gonna cancel your PTO because of some IRC server nonsense? :) | 08:12 |
kashyap | We just move to a different server; plain and simple, and keep going. | 08:12 |
*** macz_ has quit IRC | 08:14 | |
bauzas | kashyap: tbc, agreed on the consensus to move to OFTC | 08:15 |
bauzas | this is absolutely not the right time for evaluating other products | 08:15 |
kashyap | bauzas: Yep; that makes sense. | 08:15 |
bauzas | the lower change, the better. | 08:15 |
kashyap | bauzas: Sure. I've said as much. I don't know about the "absolute"; but it's always the time to change one's mind ;-) | 08:16 |
bauzas | the question, tho, is the absolute need to the move | 08:16 |
bauzas | that's unclear and I wouldn't argue for urgency | 08:16 |
bauzas | the operations still continue and we should just make a reasonable transition plan to OFTC but not under pressure | 08:16 |
bauzas | nothing justifies us to move ASAP | 08:17 |
kashyap | Yep; I don't see any crazy urgency. | 08:17 |
bauzas | and I'd recommend us to keep freenode chat rooms for a while | 08:17 |
bauzas | in order to reroute people coming by | 08:17 |
bauzas | not everyone comes everyday on the rooms | 08:17 |
bauzas | and not everyone reads openinfra mailing lists | 08:18 |
bauzas | actually, I'll reply by this | 08:18 |
* bauzas usually hates to jump on those kind of large and opiniated threads | 08:18 | |
*** pmannidi has joined #openstack-nova | 08:19 | |
kashyap | Sure; what you suggest should be "obvious" for the Infra folks; and is part of clear communication to the communities. | 08:25 |
*** sapd1 has quit IRC | 08:29 | |
*** andrewbonney has joined #openstack-nova | 08:34 | |
*** ociuhandu has quit IRC | 08:34 | |
aarents | Morning nova! | 08:36 |
aarents | in this change "libvirt: Provide the backing file format when creating qcow2 disks" https://review.opendev.org/c/openstack/nova/+/708745/ , | 08:37 |
aarents | we ensure to add the "backing_fmt" field at qcow2 image creation due to newer libvirt constraints.(if not present newer libvirt does not start instance) | 08:37 |
aarents | there is the Lee following quote: "Nova needs to [...] potentially handle the upgrade case where we may need to rebase existing disks in order to update the metadata." | 08:37 |
aarents | I would like to know if there is an ongoing work with the "upgrade case" ? | 08:38 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/nova master: DO NOT REVIEW: Testing new OVN as default https://review.opendev.org/c/openstack/nova/+/792328 | 08:38 |
lyarwood | aarents: no as the check was removed in the next minor release of libvirt | 08:50 |
lyarwood | aarents: AFAIK UCA is still shipping the original version with the check so if you're running into this you need to raise a bug against UCA and ask them to bump their version of libvirt | 08:50 |
aarents | lyarwood: intersting I will double check, thank you | 08:54 |
lyarwood | aarents: np, fixing this by rebasing every single instance disk as a result just isn't worth the effort IMHO | 08:57 |
*** k_mouza has joined #openstack-nova | 08:59 | |
*** k_mouza has quit IRC | 09:00 | |
*** k_mouza has joined #openstack-nova | 09:00 | |
*** rcernin has joined #openstack-nova | 09:04 | |
*** ociuhandu has joined #openstack-nova | 09:05 | |
*** ociuhandu has quit IRC | 09:14 | |
artom | Huh, http://status.openstack.org/reviews/ is broken | 09:16 |
*** ociuhandu has joined #openstack-nova | 09:29 | |
*** ociuhandu has quit IRC | 09:30 | |
*** ociuhandu has joined #openstack-nova | 09:31 | |
*** avolkov has quit IRC | 09:36 | |
*** rcernin has quit IRC | 09:38 | |
*** mkrai has joined #openstack-nova | 09:44 | |
*** rcernin has joined #openstack-nova | 09:53 | |
admin0 | hi all.. how do I rename an existing host to a new one ? | 09:56 |
stephenfin | admin0: afaik, you can't | 09:56 |
admin0 | the issue i have is that i started with hostname say h7 .. then at some point, it registered as h7.openstack.local and at some point its registered at h7. ( only a . with no domain at the end) | 09:56 |
admin0 | so now my associations are broken .. i cannot migrate stuff .. i cannot use security groups | 09:57 |
admin0 | because the instances are running, how can i fix this | 09:57 |
stephenfin | oh, so you had one name and you've inadvertently changed it? | 09:57 |
stephenfin | artom can hopefully sanity check what I'm saying, but I think you'll need to manually correct the ComputeNode entries for the host in the DB and ensure your RabbitMQ queues are correctly named | 09:58 |
admin0 | stephenfin, i did not knowingly changed it .. i think one admin accidently added openstack_domain: "" because he did not wanted to see openstack.local and ran the openstack ansible nova playbooks .. and that caused hosts calling h7 to be re-added as h7. | 09:59 |
admin0 | stephenfin, here is what i see now: openstack_domain: "" | 09:59 |
admin0 | stephenfin, sorry .. https://gist.github.com/a1git/c22ec0c17aaa9dcf95fd7485eb76af2f | 09:59 |
artom | stephenfin, that sounds about right | 10:00 |
artom | There may also be placement and/or cinder fallout | 10:00 |
artom | And Neutron? | 10:00 |
admin0 | its in rocky release .. there is no placement database or api as far as i know | 10:01 |
artom | Wait, when did we move placement out of tree? | 10:02 |
artom | Before rocky, surely | 10:02 |
admin0 | then maybe i am wrong | 10:02 |
artom | admin0, but the point, the hostname is used in placement resource providers, in neutron port bindings, and in cinder connection_info (sometimes? depending on the volume backend configuration?) I'm less familiar with how that works) | 10:03 |
admin0 | i do not see a database called "placemenet" like i see in newer deployments | 10:03 |
admin0 | fair point .. so right now, i have around 16 instances that are running in a compute node called h7 (ceph backend) .. which if I want to migrate from h7 -> h9, it says h7- host not found | 10:05 |
admin0 | openstack server show $uuid shows it be on h7 ( without . and without .openstack.local) | 10:05 |
admin0 | if there is a way to know what exactly hostname an instance is supposed to be at, then maybe at least i can migrate by matching the existing hostname to what the instance is expecting | 10:06 |
admin0 | artom, stephenfin this shows the issue and the command I tried: https://gist.github.com/a1git/ce400f1084a184fa5c805ee72554a6e3 | 10:06 |
admin0 | because i was 100% sure the hostname was h7 and not h7. before, ( the . got added farily recently ) , i changed the hostname, hostname -f, virsh hostname back to h7 .. but it did not solved the issue | 10:10 |
stephenfin | admin0: so like I said above, fixing this will likely involve manual database fixes | 10:11 |
artom | s/likely/definitely/ | 10:11 |
stephenfin | if you have a backup (you have a backup, right), you should use that to see what values were there previously | 10:11 |
admin0 | stephenfin, that is fine .. but which database/tables have this associations | 10:11 |
*** macz_ has joined #openstack-nova | 10:11 | |
stephenfin | admin0: compute_nodes has a host column and hypervisor_hostname column, the migrations table also has source_node and dest_node columns | 10:13 |
stephenfin | admin0: tbh, you should look at 'nova/db/sqlalchemy/models.py' for anything hostname-related (i.e. the column name has "host" or "node" in it) and then check your DB to see if entries matching those hostnames appears | 10:14 |
stephenfin | admin0: and for the love of God, back up your DB before you start messing around with it | 10:14 |
stephenfin | :) | 10:14 |
*** macz_ has quit IRC | 10:16 | |
admin0 | stephenfin, host says h7 and hypervisor_hostname says h7. | 10:16 |
artom | I'd also do stuff like check the volume attachments and port bindinds, and the resource provider, see if there are host names in there anywhere. Sorry to not be more specific | 10:16 |
admin0 | so if i put the hostname, hostname-f and virsh hostname to be h7, i think i can safely update hypervisor_hostname to h7 ( removing the .) | 10:16 |
stephenfin | admin0: that sounds correct, but you'll need to experiment | 10:17 |
stephenfin | admin0: also, what artom said | 10:17 |
admin0 | yeah | 10:17 |
aarents | lyarwood: agree that rebasing thousands of instance isn't worth the effort, and indeed as you said UCA did not backport the "relaxing patch" in focal. I will do downstream, thanks again for the hint | 10:20 |
lyarwood | aarents: np anytime | 10:22 |
admin0 | stephenfin, i think this provides a picture --> https://gist.githubusercontent.com/a1git/0af2d9fbd93787ba34723c9c7bdbba4a/raw/1eb6c6f93ed5495f5ac4c484c97fc53a41ca6f2e/gistfile1.txt | 10:28 |
admin0 | looking into the original ids at the top . | 10:28 |
admin0 | and i think .openstack.local was added by openstack as i did not had any domain in the hostname -f part before | 10:28 |
admin0 | so question is, do i set the hypervisor_hostname to just h7 or h7.openstack.local ? | 10:30 |
sean-k-mooney | admin0: it will depend on when the vms were cereated | 10:31 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: Use jinja2 to generate fake libvirt data https://review.opendev.org/c/openstack/nova/+/790971 | 10:31 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Rename 'ImageBackendFixture' to 'LibvirtImageBackendFixture' https://review.opendev.org/c/openstack/nova/+/792353 | 10:31 |
sean-k-mooney | if you have had vms created with different values over the lifetime of the host then you wil not be able to avoid having to modify the nova, nueton and cinder dbs to correcct this | 10:32 |
stephenfin | lyarwood: Addressed your comments for last week ^ | 10:32 |
sean-k-mooney | admin0: this is why we dont actullly supprot changing the host name | 10:32 |
*** pmannidi has quit IRC | 10:32 | |
sean-k-mooney | admin0: it will also affect the placment resouce provider name too | 10:32 |
sean-k-mooney | admin0: i would proably go with h7 but both are valid | 10:33 |
*** pmannidi has joined #openstack-nova | 10:33 | |
*** ociuhandu has quit IRC | 10:34 | |
*** ociuhandu has joined #openstack-nova | 10:34 | |
lyarwood | stephenfin: ack thanks I'll take a look shortly | 10:34 |
sean-k-mooney | stephenfin: artom this has come up enough times at this poitn that i think we need to put a check in init host to block starting the agent if the hypervior_hostname has change | 10:34 |
sean-k-mooney | what do you think ? | 10:35 |
artom | sean-k-mooney, ah, interesting... | 10:35 |
stephenfin | how would you detect that. I was under the impression that changing the hostname meant the compute node was re-registering itself or something | 10:36 |
sean-k-mooney | not in all cases | 10:36 |
sean-k-mooney | it will but the compute service record wont always change | 10:37 |
artom | stephenfin, compare the hostname from the OS to the existing DB record? | 10:37 |
admin0 | in my case, i cannot migrate, spawn new instance, work on security groups, so i think its no harm kind of | 10:37 |
sean-k-mooney | artom: there is a chiken and egg problem | 10:37 |
*** mkrai has quit IRC | 10:37 | |
sean-k-mooney | artom: i was thinkign we look up the compute service recored using conf.host then look up the compute node records | 10:38 |
sean-k-mooney | if its the libvirt dirver their shoudl be at most one | 10:38 |
sean-k-mooney | we then check the compute node recored and compare the hypervior hostname | 10:38 |
sean-k-mooney | for ironic this wont work but for libvirt we can do that | 10:39 |
*** ociuhandu has quit IRC | 10:39 | |
artom | sean-k-mooney, so it would have to be a driver-level check, not compute service | 10:39 |
sean-k-mooney | yes | 10:39 |
sean-k-mooney | init_host in the driver not in the compute manager | 10:39 |
sean-k-mooney | admin0: you cant perform api action because the compute agent create a queue with the vauels of CONF.host | 10:40 |
sean-k-mooney | admin0: but the conductor send rpc calls to the queue named in instance.host | 10:41 |
artom | If we can make it work I think it'd be worthwhile | 10:41 |
sean-k-mooney | so if the instance.host does not match the agent queu name it gets lost | 10:41 |
admin0 | is there a quick way to check what is the value of CONF.host and what is the value of instance.host ? | 10:41 |
sean-k-mooney | you will also have the schduelr issue you hit | 10:41 |
artom | But I suspect it would have to be overridable, with a conf option or something | 10:41 |
sean-k-mooney | artom: im not sure it would | 10:42 |
sean-k-mooney | but we can discuss that in the implemantion | 10:42 |
sean-k-mooney | artom: the only time it would be allowable but ill advised to change the hostname today is if there are no instnaces on the host | 10:42 |
artom | sean-k-mooney, oh, I agree with you in principle | 10:43 |
artom | But you know how operators be ;) | 10:43 |
openstackgerrit | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | 10:45 |
admin0 | mine was an oversight.. unintentional | 10:45 |
admin0 | midway we decided to not have .openstack.local but the playbooks updated every hostname ( including existing ones) | 10:45 |
admin0 | what creates a new queue in rabbitmq? is it the compute node ? | 10:46 |
artom | admin0, they usually are, and this is what sean-k-mooney is trying to prevent. But I can easily imagine someone saying "no no, we know what we're doing, we're totally going to rename this host" | 10:46 |
admin0 | i also think its a lack of documentation or warning . i don't recall reading this otherwise we would have put it in the policy | 10:47 |
artom | Fair point. I guess we thought it was implicit? | 10:47 |
admin0 | you guys know the code and for you all.. its implicit .. we ( operators) read the deployment guide and make policies based on the guides .. so this point got overlooked | 10:48 |
artom | "you guys know the code" Should... should we tell them? ;) | 10:49 |
admin0 | :D | 10:49 |
*** mgariepy has quit IRC | 10:49 | |
sean-k-mooney | artom: operators tend to get annoy wehn we tell them we explictly dont support this so your on your own to fix your mess | 10:49 |
sean-k-mooney | admin0: we should have documenation for this as far as i know we dissuced adding it a few times | 10:51 |
sean-k-mooney | admin0: i still cant finx the warning we said shold be added like 2 years ago :( | 10:55 |
sean-k-mooney | admin0: not to dismay you but this is one of the harder thing to fix to the point that the only supporte way to actully change the host name is generally to scale in and redeploy | 10:56 |
*** ociuhandu has joined #openstack-nova | 11:04 | |
admin0 | sean-k-mooney, if i have a way to migraete ceph backed instances off this hypervisor to another one, i would happilfy redeploy | 11:09 |
admin0 | i don't understand nova :) .. https://gist.githubusercontent.com/a1git/9d4f3115529a53793703ef08bdfd752b/raw/ceec98a6145170143146d9dbd276156c5a52278f/gistfile1.txt | 11:09 |
admin0 | i changed all to h3 .. then it still adds as h3. | 11:10 |
sean-k-mooney | is h3. in the instance.host | 11:11 |
sean-k-mooney | by the way libvirt also caches info | 11:12 |
sean-k-mooney | so i think you will need to restart libvirt when you make these change before you restart the compute agent | 11:13 |
sean-k-mooney | have you restarted the nova-compute agent after the hostname change | 11:13 |
admin0 | i have restarted the nova-compute agent only | 11:15 |
admin0 | and not libvirt | 11:15 |
admin0 | will restarting libvirt cause a downtime to the guests ? | 11:15 |
sean-k-mooney | no | 11:15 |
sean-k-mooney | there lifetime is governe by that of the qemu process that is runnign them | 11:16 |
*** ociuhandu has quit IRC | 11:16 | |
admin0 | virsh hostname returns h3 and not h3. | 11:16 |
sean-k-mooney | libvirt and the nova comptue agent can be restarted without affectign the vms | 11:16 |
sean-k-mooney | well thats what its returing to you not nessialry what its returing to nova | 11:17 |
admin0 | "is h3. in the instance.host" -- where do I see this | 11:17 |
sean-k-mooney | for reasons they can be different | 11:17 |
sean-k-mooney | in the cell db | 11:17 |
sean-k-mooney | select * form nova_cell1.instance where host == 'h3.'; | 11:18 |
*** sapd1 has joined #openstack-nova | 11:18 | |
sean-k-mooney | you can also get this in an instnace show | 11:18 |
admin0 | [nova_cell0]> select * from compute_nodes | 11:18 |
admin0 | i only have cell0 | 11:18 |
sean-k-mooney | as an admin | 11:18 |
admin0 | this is an old ( rocky) platform | 11:18 |
sean-k-mooney | admin0: do you also have a db call just nova | 11:18 |
sean-k-mooney | you should have 3 nova dbs | 11:19 |
sean-k-mooney | nova_cell0 nova_api and a third one? | 11:19 |
admin0 | https://gist.github.com/a1git/9bc0c00aaacb3e8757b40e678a3be0fc are the dbs i ahve | 11:20 |
sean-k-mooney | yep so the nova db is the cell1 db | 11:20 |
admin0 | select count(*) from nova.instances where host='h3.' = > 0 ; select count(*) from nova.instances where host='h3' => 71 | 11:23 |
admin0 | nova_cell0.instances has zero rows | 11:23 |
sean-k-mooney | ok | 11:23 |
sean-k-mooney | so you have restart both libvirt and nova compute and confirm the hypervior hostname in the in the db is h3 | 11:24 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Add 'hw:vif_multiqueue_enabled' flavor extra spec https://review.opendev.org/c/openstack/nova/+/792356 | 11:25 |
*** links has joined #openstack-nova | 11:25 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Describe the use of Review-Priority label https://review.opendev.org/c/openstack/nova/+/792357 | 11:25 |
stephenfin | sean-k-mooney: ^ | 11:25 |
sean-k-mooney | that would need at least a specless blueprint by the way | 11:26 |
*** sapd1 has quit IRC | 11:27 | |
*** ociuhandu has joined #openstack-nova | 11:28 | |
admin0 | i have restarted libvirtd and nova-compute .. " confirm the hypervior hostname in the in the db is h3" | 11:29 |
admin0 | i see it added itself ( the last row in the db) is wtih h3 | 11:29 |
admin0 | without the . | 11:29 |
admin0 | Final resource view: name=h3 phys_ram=257915MB used_ram=2048MB phys_disk=136958GB used_disk=0GB total_vcpus=40 used_vcpus=0 pci_stats=[] -- in horizon, if i go to admin >> hypervisors .. it does not update the resource view | 11:31 |
admin0 | instances, ram used, disk used all is shown as zero | 11:31 |
admin0 | do i also need ot check for all active instances and set their association to h3 as hostname as opposed to h3. | 11:32 |
sean-k-mooney | you need to check placment and see if there are multipel resouce providers for the host | 11:32 |
sean-k-mooney | if so then yes you neeed to fix the allocations | 11:32 |
sean-k-mooney | artom: do you know if the placment audit or heal allocations commands willl do that | 11:33 |
admin0 | sorry for this question, but how exactly do i chck placement ? | 11:34 |
*** ociuhandu has quit IRC | 11:35 | |
sean-k-mooney | do you have the osc-placment plugin installed | 11:36 |
sean-k-mooney | if you do you can do openstack resource provider list | 11:36 |
sean-k-mooney | if not you can isntall it form pip | 11:36 |
admin0 | so i changed h3 and h7 from with a . to without a . .. and select id, hostname, launched_on, host from instances where deleted_at is null and host like '%h7%' gives me 17 insances with host set to h7 (without the .) .. but if i enter the command openstack server migrate 7e6e9758-0d62-4870-861e-34682e8a3580 --wait --live h10 .. it says Compute host h7 could not be found. (HTTP 400) | 11:36 |
admin0 | where do I install that plugin ? i mean in the hypervsior, or in one of the controllers ? | 11:37 |
sean-k-mooney | where ever you run the openstack command | 11:37 |
sean-k-mooney | so your laptop or the server you ssh into | 11:37 |
admin0 | pip install osc-placement ? | 11:38 |
sean-k-mooney | preferally pip install --user ... | 11:38 |
sean-k-mooney | but yes | 11:38 |
sean-k-mooney | or pip3 in somecases if your useing python 3 | 11:38 |
sean-k-mooney | you can also check in the db too if you prefer | 11:39 |
sean-k-mooney | since you are not using a sperate placment db i belive placmeent records will be in the nova_api db | 11:39 |
*** ociuhandu has joined #openstack-nova | 11:39 | |
sean-k-mooney | so select * form nova.resouce_providers | 11:39 |
admin0 | i would prefer the db | 11:40 |
admin0 | its empty | 11:41 |
admin0 | 0 rows | 11:41 |
admin0 | sorry | 11:41 |
admin0 | i was checking in nova | 11:42 |
admin0 | my bad | 11:42 |
admin0 | sean-k-mooney, this is what it has: https://gist.githubusercontent.com/a1git/9eeab38dd606ebd05cbe5995b67254e5/raw/2aa55a648165b17032c2b42e35d81be7bfb6dbc7/gistfile1.txt | 11:43 |
admin0 | for h7, from where i am trying to migrate off instance, i see only 1 line ther | 11:43 |
admin0 | there* | 11:43 |
sean-k-mooney | ok good there are no duplicates it correctly renamed the placemetn RP | 11:44 |
sean-k-mooney | so your allocations shoudl also be correct | 11:44 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document virtio-net multiqueue https://review.opendev.org/c/openstack/nova/+/792362 | 11:45 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Address TODO https://review.opendev.org/c/openstack/nova/+/792363 | 11:45 |
stephenfin | sean-k-mooney: now with docs ^ | 11:45 |
sean-k-mooney | stephenfin: you could not wait for my -1 | 11:45 |
sean-k-mooney | stephenfin: you forgot to add a check in _validate_flavor_image_nostatus | 11:45 |
stephenfin | you must -1 faster :P | 11:45 |
sean-k-mooney | that invokes the new fuction you put in hardware.py | 11:46 |
stephenfin | wdym? I added tests in test_hardware.py if that's what you mean | 11:46 |
sean-k-mooney | nope | 11:46 |
sean-k-mooney | you are ment to check for flavor and image conflcit in the api | 11:46 |
stephenfin | ohh, gotcha | 11:46 |
stephenfin | yes | 11:46 |
sean-k-mooney | https://github.com/openstack/nova/blob/82141b12c356fe374822038bdf0a6f0aaa32047b/nova/compute/api.py#L643 | 11:46 |
stephenfin | I'll add that. Thanks | 11:46 |
stephenfin | and then, lunch | 11:47 |
sean-k-mooney | ohter then that and the need for a specless blueprint and docs which you have just added looked good | 11:47 |
sean-k-mooney | stephenfin: can you add this to the meeting adgenda later | 11:48 |
admin0 | i have checked that there are no instances in nova.instance where hostname = $hostname(.) .. so all instances are set to $hostname (without .) only | 11:48 |
admin0 | placement has only hosts without . | 11:48 |
admin0 | so this all looks good | 11:48 |
stephenfin | sean-k-mooney: sure | 11:48 |
admin0 | but openstack migrate $uuid --wait --live $dest_host returns => compute host $source not found (400) | 11:49 |
*** dklyle has quit IRC | 11:52 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Add 'hw:vif_multiqueue_enabled' flavor extra spec https://review.opendev.org/c/openstack/nova/+/792356 | 11:58 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document virtio-net multiqueue https://review.opendev.org/c/openstack/nova/+/792362 | 11:58 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Address TODO https://review.opendev.org/c/openstack/nova/+/792363 | 11:58 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Move 'hw:pmu', 'hw_pmu' parsing to nova.virt.hardware https://review.opendev.org/c/openstack/nova/+/792364 | 11:58 |
artom | sean-k-mooney, IIRC no, the audit command checks for allocations, not RPs themselves | 11:58 |
*** k_mouza_ has joined #openstack-nova | 11:59 | |
admin0 | how to check how servers are selected for migration ? | 12:00 |
admin0 | i mean what would cause an openstack migrate to say $source not found | 12:00 |
admin0 | nova hypervisor-servers h7 .. now shows all servers with hostname as h7 (without the .) | 12:01 |
*** k_mouza has quit IRC | 12:03 | |
sean-k-mooney | you might need to restart the nova schduler | 12:13 |
sean-k-mooney | i know we send som data taht is cached in the schduler for the hoststate objects | 12:13 |
sean-k-mooney | but i did not think it would cause that type of error | 12:14 |
*** rcernin has quit IRC | 12:14 | |
sean-k-mooney | although its possibel the host mappigns in the api db need to be updated too | 12:14 |
sean-k-mooney | admin0: check nova_api.host_mappings | 12:14 |
*** mgariepy has joined #openstack-nova | 12:16 | |
admin0 | sean-k-mooney, this is what i see: https://gist.githubusercontent.com/a1git/aeba25d6e7a37241ce2ed8ef1547953e/raw/df42c1309a7ad88f393980a7654992fb93bd365a/gistfile1.txt | 12:21 |
*** lbragstad_ has quit IRC | 12:24 | |
sean-k-mooney | ok so they all look correct | 12:29 |
sean-k-mooney | you are still getting the same error after restarting the scheduler? | 12:29 |
admin0 | wait .. that is something i did not restarted | 12:30 |
admin0 | one moment | 12:30 |
*** lbragstad has joined #openstack-nova | 12:31 | |
*** lbragstad has quit IRC | 12:32 | |
*** lbragstad has joined #openstack-nova | 12:33 | |
admin0 | sean-k-mooney, i restared the whole nova containers .. but still get Compute host h7 could not be found. ( | 12:36 |
gibi | stephenfin, sean-k-mooney, bauzas: I responded in the pps spec https://review.opendev.org/c/openstack/nova-specs/+/785014 Thanks for the reviews. Would be good to make some decision about the additional OVS RP question then I will respin the spec fixing up the things you asked for | 12:39 |
gibi | stephenfin: could you check back to https://review.opendev.org/c/openstack/nova-specs/+/785014/7/specs/xena/approved/qos-minimum-guaranteed-packet-rate.rst#82 as I might missed the point of your comment | 12:40 |
sean-k-mooney | gibi: im hoping we just will never support multiple openvswitch on the same host | 12:42 |
sean-k-mooney | gibi: it comes up every few months | 12:42 |
gibi | :) | 12:42 |
sean-k-mooney | for those that want to support ovs-dpdk and kernel ovs at the same time | 12:42 |
sean-k-mooney | but it woudl be a fairly invaise change | 12:43 |
sean-k-mooney | we would need to modify the ml2 driver to include the ovs connection detail in the port bindings | 12:43 |
sean-k-mooney | and modify os-vif to then use those | 12:43 |
gibi | it would be almost free to add the extra RP now except that we would need to reparent the bridge RPs which needs the new placement version | 12:44 |
sean-k-mooney | yep but what value does it add | 12:44 |
gibi | it models the reality a bit better as the agent and the OVS instance is two different software service i | 12:45 |
sean-k-mooney | so woudl modling the pps on the br-int | 12:45 |
gibi | no | 12:45 |
sean-k-mooney | and the reality is today you can only have one ovs agent per host | 12:46 |
gibi | we would have Agent RP->OVS RP->Bridge RP | 12:46 |
sean-k-mooney | if we had 2 ovs instance we woudl have 2 ovs agents | 12:46 |
gibi | sean-k-mooney: true, so we _can_ merge the agent with the OVS RP, but we dont _have to_ | 12:46 |
gibi | honestly I tend to agree with YAGNI because of the extra placement dependency :) | 12:47 |
admin0 | i launched a new instance 2 vcpu and now h3 shows 2vcpu being used.. but it does not update stats from the instances already running there .. i see no diffence in nova.insances table | 12:47 |
admin0 | is there another table where hosts vs instances are mapped ? | 12:47 |
sean-k-mooney | admin0: not off the top of my head but maybe | 12:48 |
sean-k-mooney | gibi: where is there suggestion/comment | 12:49 |
admin0 | sean-k-mooney, how does the nova-compute process work ? it boots up, gathers a list of insances it has .. dumps to rabbitmq and then it gets populated to the db ? | 12:49 |
sean-k-mooney | gibi: i need to step away for 10-20 mins but i can talk about htis when i get back or on the review | 12:49 |
gibi | sean-k-mooney: this is the comment thread https://review.opendev.org/c/openstack/nova-specs/+/785014/7/specs/xena/approved/qos-minimum-guaranteed-packet-rate.rst#98 | 12:49 |
gibi | sean-k-mooney: ack | 12:50 |
gibi | thanks | 12:50 |
sean-k-mooney | admin0: not quite the db is the souce of truth for what instance should be on the host | 12:50 |
*** __ministry has quit IRC | 12:50 | |
admin0 | isn't there any nova command that can force update these ? | 12:50 |
sean-k-mooney | gibi lol ok i forgot YAGNI was not an irc name hehe i havent heard that term use since i last spoke to cdent | 12:51 |
*** sapd1_x has quit IRC | 12:51 | |
gibi | sean-k-mooney: ohh, sorry, stephenfin used that now in the review | 12:51 |
gibi | but true it was cdent's terminology :) | 12:52 |
sean-k-mooney | im familar with it just havent heard it in a while but yes stephen did use it | 12:52 |
*** tbachman has quit IRC | 12:53 | |
sean-k-mooney | gibi: ok ill be back soon but did you read may question regardign provder.yaml last night? | 12:53 |
gibi | sean-k-mooney: responded it this morning | 12:53 |
gibi | in IRC | 12:53 |
*** ratailor_ has quit IRC | 12:53 | |
sean-k-mooney | ah ok i have not check scroll back ill do that when i get back | 12:53 |
gibi | 08:07 < gibi> sean-k-mooney: regarding provider.yaml; I think technically it is possible to | 12:53 |
gibi | report bw and pps inventories via that. As a downside of it the neutron agents | 12:53 |
gibi | already have the bridge_mapping and physical_device_mappings in their config | 12:53 |
gibi | file as we use that to figure out which physnet traits needs to be reported | 12:54 |
gibi | 08:08 < gibi> sean-k-mooney: if you do the physnet trait reporting in provider.yaml then you | 12:54 |
gibi | manually need to make sure that it is in sync with the bridge mappings | 12:54 |
gibi | 08:08 < gibi> sean-k-mooney: I don't know how ovn does the bridge mapping so there it might | 12:54 |
gibi | not be a problem | 12:54 |
gibi | 08:09 < gibi> the vnic_type trait has a similar consequence | 12:54 |
sean-k-mooney | ack. bridge mapping are stored in the ovn south db casisi table in the external-ids field | 12:55 |
*** tbachman has joined #openstack-nova | 12:56 | |
sean-k-mooney | although in general with ovn you just have one bridge br-int | 12:56 |
sean-k-mooney | it might be something to test out i need to redeploy openstack soon and ill likely try doing it with ovn | 12:57 |
sean-k-mooney | ok brb | 12:57 |
openstackgerrit | Merged openstack/osc-placement master: Note env OS_PLACEMENT_API_VERSION support https://review.opendev.org/c/openstack/osc-placement/+/791356 | 12:57 |
admin0 | "admin0: not quite the db is the souce of truth for what instance should be on the host" -- will deleteing the rabbitmq and re-creating it help ? | 12:59 |
openstackgerrit | Merged openstack/osc-placement master: Add openstackclient-plugin-jobs https://review.opendev.org/c/openstack/osc-placement/+/780598 | 13:00 |
stephenfin | gibi: done | 13:02 |
gibi | stephenfin: thanks for the clarification, It is clear now | 13:05 |
*** ociuhandu has quit IRC | 13:11 | |
*** eharney has quit IRC | 13:13 | |
*** eharney has joined #openstack-nova | 13:19 | |
*** __ministry has joined #openstack-nova | 13:21 | |
*** sapd1 has joined #openstack-nova | 13:22 | |
*** ociuhandu has joined #openstack-nova | 13:23 | |
*** ralonsoh has quit IRC | 13:26 | |
*** ralonsoh has joined #openstack-nova | 13:27 | |
*** ociuhandu has quit IRC | 13:28 | |
sean-k-mooney | gibi: too bad we cant rename the bandwith resouce classes | 13:29 |
sean-k-mooney | although the orginal approved specs used ingress and egress | 13:33 |
sean-k-mooney | the were retoactivly updated to the short names https://github.com/openstack/neutron-specs/commit/2af53625e7a2e39e53db394f97328e6427cd9fb9 | 13:34 |
*** ociuhandu has joined #openstack-nova | 13:34 | |
sean-k-mooney | which is why i was surprised we were using them | 13:34 |
sean-k-mooney | if it was not for the fact that we cant remove standard resouce class i woudl be very tempeted to treat this a a bug and change them | 13:36 |
sean-k-mooney | but reshaps are way to expensive to waste it on something as trivial as this | 13:36 |
sean-k-mooney | sigh ... https://review.opendev.org/c/openstack/nova/+/570847/10/nova/rc_fields.py#45 | 13:39 |
sean-k-mooney | so that how that happend | 13:39 |
sean-k-mooney | gibi: unless you 100% agaisnt the long names i would prefer to use ingress and egress expanded but its not like im going to block on that point | 13:40 |
*** jawad_axd has quit IRC | 13:43 | |
*** ociuhandu has quit IRC | 13:45 | |
*** ociuhandu has joined #openstack-nova | 13:45 | |
*** sapd1_x has joined #openstack-nova | 13:49 | |
gibi | sean-k-mooney: I really don't want to change the bandwidth naming, lets accept that as history. a well discussed history as we see from the original review | 13:50 |
gibi | sean-k-mooney: I'm not against long names | 13:50 |
*** ociuhandu has quit IRC | 13:50 | |
gibi | it will be weird when the bw and the pps resources next to each outher in a resource_request has different spelling | 13:50 |
sean-k-mooney | i dont agree that it was well disucsed since it did not happen in the spec. but yes i accetp we cant change it for bandwith | 13:51 |
gibi | fine, I can change the name :) maybe we wil re-discuss it in the os-resource-classes review :) | 13:54 |
gibi | I will spell out egress and ingress | 13:54 |
gibi | I don't want to get hung up on that. | 13:55 |
gibi | it is just a name | 13:55 |
sean-k-mooney | well i was actully going to abstain form the name | 13:55 |
sean-k-mooney | pass my last comment | 13:55 |
sean-k-mooney | if we can agree with the rest of the spec we could keep it as written | 13:56 |
sean-k-mooney | just wanted to exprss my frustration at yet another case wehre part of the agreee api were changed at the implation time | 13:56 |
*** sapd1_x has quit IRC | 13:56 | |
*** ociuhandu has joined #openstack-nova | 13:56 | |
sean-k-mooney | im ok with implation detail chagne during code review | 13:56 |
sean-k-mooney | i stongly dislike change any public aspect of the feature in code review | 13:56 |
sean-k-mooney | which this way | 13:57 |
sean-k-mooney | *was | 13:57 |
gibi | sean-k-mooney: based on the os-resource-classes review if that would happen in a spec it would have been approved by the same set of cores anyhow | 13:57 |
* sean-k-mooney regrest that i had to proxy my inovlvment in this through others and was not directly invovle din the review a the time | 13:58 | |
gibi | sean-k-mooney: don't feel bad about it, it just a name, we have soo many names to decide on in the future | 13:59 |
sean-k-mooney | lol :) | 13:59 |
gibi | :) | 13:59 |
*** __ministry has quit IRC | 14:00 | |
sean-k-mooney | gibi: well min bandwidth just feels a little more personal since i first discussed it with you after the briton mid cycle when you visited intel shannon | 14:00 |
*** jawad_axd has joined #openstack-nova | 14:00 | |
sean-k-mooney | and i tired to follow the implemation later via other at a high level but i tought we were still using the name in the orginal spec | 14:01 |
sean-k-mooney | anyway are you going to push an updates spec soon | 14:01 |
gibi | old times | 14:02 |
sean-k-mooney | with the other poitn address or should i re review the current version | 14:02 |
gibi | just read the current version | 14:02 |
sean-k-mooney | ok will do | 14:02 |
gibi | I will wait bauzas to check back about the OVS RP question | 14:02 |
gibi | sean-k-mooney: thanks | 14:02 |
*** sapd1_x has joined #openstack-nova | 14:03 | |
openstackgerrit | Merged openstack/nova master: Use the new type HostDomainOpt. https://review.opendev.org/c/openstack/nova/+/788240 | 14:06 |
*** jawad_axd has quit IRC | 14:06 | |
openstackgerrit | Merged openstack/nova master: tests: Move remaining non-libvirt fixtures https://review.opendev.org/c/openstack/nova/+/790967 | 14:06 |
openstackgerrit | Merged openstack/nova master: tests: Split external service fixtures out https://review.opendev.org/c/openstack/nova/+/790968 | 14:07 |
gibi | sean-k-mooney: I do remember the bristol mid cycle, but I haven't visited Intel Ireland. :) | 14:10 |
*** sapd1_x has quit IRC | 14:11 | |
gibi | I hope I'm not that old that I already forget about these things :) | 14:11 |
sean-k-mooney | really? i could have swore you were the person that acompanied jay pipes | 14:11 |
gibi | that was somebody else | 14:12 |
gibi | we should ask jay :) | 14:12 |
sean-k-mooney | really. i wonder who it was then. it was me jay rodolfo adrian and mistry person in a room | 14:13 |
*** macz_ has joined #openstack-nova | 14:13 | |
sean-k-mooney | well and malini and adrian hoban | 14:13 |
sean-k-mooney | i tough it was you since i assumed you were going to visit errison in athlone | 14:13 |
*** k_mouza_ has quit IRC | 14:13 | |
*** k_mouza has joined #openstack-nova | 14:14 | |
sean-k-mooney | maybe it was someone form his company | 14:14 |
sean-k-mooney | i think he woudl have been at mirantis at the time | 14:14 |
sean-k-mooney | anyway not imporant | 14:14 |
*** sapd1_x has joined #openstack-nova | 14:14 | |
sean-k-mooney | excpt im clearly imanging things. | 14:14 |
gibi | our mind fills up the gapes | 14:15 |
gibi | gaps | 14:15 |
sean-k-mooney | ya they do which what we precivie as the logical answer | 14:15 |
gibi | yepp | 14:16 |
*** macz_ has quit IRC | 14:17 | |
*** jawad_axd has joined #openstack-nova | 14:20 | |
gibi | now I got in a nostalgic mood. I miss jaypipes, cdent, mriedem, efried, and others :/ | 14:25 |
*** jangutter has joined #openstack-nova | 14:26 | |
lyarwood | elod / melwitt / bauzas ; https://review.opendev.org/c/openstack/nova/+/788467 - stable reviews on this would be great if anyone has time | 14:29 |
*** tbachman_ has joined #openstack-nova | 14:30 | |
*** tbachman has quit IRC | 14:30 | |
*** tbachman_ is now known as tbachman | 14:30 | |
lyarwood | gibi: https://review.opendev.org/q/topic:bug/1928063 - if you have time today, it's blocking some downstream SEV verification | 14:31 |
*** sapd1_x has quit IRC | 14:31 | |
gibi | looking... | 14:31 |
*** mgariepy has quit IRC | 14:31 | |
*** sapd1_x has joined #openstack-nova | 14:32 | |
gibi | lyarwood: This morning I was looking at the next libvirt version we declare in the libvirt/driver.py. It is 7.0.0. Do we actually promise to bump to it in Xena? I'm asking this as focal still only has 6.0.0 | 14:32 |
lyarwood | gibi: I think the comments suggest we will but it has never been enforced | 14:33 |
lyarwood | gibi: we even missed it for a few cycles after all | 14:34 |
* lyarwood checks the docs again | 14:34 | |
gibi | yeah I saw on the wiki that we kept the same versions for multiple releases | 14:34 |
lyarwood | yup, let me updated the wiki page with the latest | 14:35 |
lyarwood | but yeah if we are stuck on focal anyway for Xena we can't bump | 14:35 |
gibi | I agree, we have no test coverage at the moment with 7.0.0 so we should not bump | 14:36 |
gibi | thanks for updating the wiki! | 14:36 |
*** ociuhandu has quit IRC | 14:40 | |
*** ociuhandu has joined #openstack-nova | 14:41 | |
gibi | lyarwood: regarding the https://review.opendev.org/c/openstack/nova/+/790659/3/nova/objects/image_meta.py Do we need to persist the image_ref in the sysmeta? Can we simply fill image_meta.id from instance.image_ref when an ImageMeta is constructed? | 14:44 |
*** jawad_axd has quit IRC | 14:44 | |
*** jawad_axd has joined #openstack-nova | 14:44 | |
*** jawad_axd has quit IRC | 14:44 | |
gibi | we anyhow filling ImageMeta from instance.system_metadata so we always need to have a ref for an instance when we create the ImageMeta object | 14:45 |
*** ociuhandu has quit IRC | 14:46 | |
lyarwood | gibi: yeah true | 14:46 |
*** sapd1_x has quit IRC | 14:46 | |
lyarwood | gibi: I was sure there was a reason I left this in | 14:46 |
lyarwood | gibi: maybe test fallout? | 14:46 |
* lyarwood checks | 14:46 | |
*** ociuhandu has joined #openstack-nova | 14:46 | |
gibi | now it feels like 1) data duplication 2) you need a special handling for initialize_instance_snapshot_metadata to drop the field coming from the sysmeta | 14:47 |
lyarwood | hmm true | 14:49 |
lyarwood | let me respin and drop it | 14:49 |
gibi | lyarwood: thanks | 14:50 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support the new port resource_request format https://review.opendev.org/c/openstack/nova/+/787208 | 14:56 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | 14:57 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | 15:00 |
*** dklyle has joined #openstack-nova | 15:00 | |
*** macz_ has joined #openstack-nova | 15:02 | |
*** sapd1_x has joined #openstack-nova | 15:05 | |
*** damien_r has quit IRC | 15:05 | |
*** supamatt has quit IRC | 15:07 | |
*** lucasagomes has quit IRC | 15:08 | |
*** mgariepy has joined #openstack-nova | 15:17 | |
admin0 | in which table or how does nova migrate check source ? | 15:18 |
admin0 | i am still stuck .. the hostnames seems to be fixed though | 15:18 |
lyarwood | admin0: the host actually running the instance? | 15:20 |
admin0 | yes | 15:20 |
lyarwood | admin0: instance.host and instance.node | 15:20 |
admin0 | lyarwood, how do I validate that | 15:20 |
admin0 | my issue is this.. nova migrate $uuid $destination => compute host $source not found | 15:21 |
admin0 | openstack server show $uuid => lists $source as host | 15:21 |
admin0 | databases lists $source as host | 15:21 |
*** belmoreira has joined #openstack-nova | 15:22 | |
*** mlavalle has joined #openstack-nova | 15:22 | |
lyarwood | admin0: oh you changed the hostname with instances running on $source? | 15:23 |
lyarwood | admin0: I assume it doesn't show up in `openstack compute service list` anymore right | 15:24 |
admin0 | lyarwood, it appears: https://gist.githubusercontent.com/a1git/032e6a0a69fcafba6e40d1b436ee40cc/raw/9df1a213ca44c6b0965a1cacdfdf24ec4b186065/gistfile1.txt | 15:25 |
admin0 | host is h7 | 15:25 |
admin0 | so first it was set as h7 . and h7.opestack.local .. then at some point, we made a mistake and put OPENSTACK_DOMAIN = "", which caused it to rename itself to h7. ( with a . at the end) .. | 15:25 |
admin0 | then we realized that and set it back to h7 (without the .) | 15:25 |
lyarwood | admin0: and the instance.host is set to h7.? | 15:26 |
admin0 | let me do a dump on one uuid | 15:26 |
admin0 | one moment | 15:26 |
admin0 | noewhere in the database is instance.host set to h7. .. its only set to h7 | 15:26 |
admin0 | (without the .) | 15:26 |
admin0 | hostname, hostname -f, virsh hostname all report h7 ( without the .) | 15:27 |
lyarwood | admin0: can you pastebin the error you're getting? | 15:27 |
admin0 | yes one moment please | 15:27 |
lyarwood | admin0: thanks, a full trace of the associated request-id would be ace | 15:28 |
lyarwood | admin0: openstack server event list $instance | 15:28 |
lyarwood | admin0: and use the request-id associated with the migration | 15:28 |
admin0 | lyarwood, this sums up the error: https://gist.githubusercontent.com/a1git/8a5fc5efbff91e9a7707815107dd7590/raw/62c46fce22be769fc1abc7e913b5102dd78cbe5f/gistfile1.txt | 15:29 |
admin0 | lyarwood, event list is here: https://gist.githubusercontent.com/a1git/bd821ac42072aa1db8e6697b5765a545/raw/f776b95d086673d25d2ff6cb734912c8c97042e2/gistfile1.txt | 15:30 |
lyarwood | admin0: thanks, could you take a look in the api and conductor logs for req-c50dfe07-b126-4ff9-9932-04e24af1d33f | 15:30 |
lyarwood | admin0: just trying to work out what is throwing the exception that leads to the 400 | 15:31 |
lyarwood | gibi: I've respun the change btw, just waiting for the func test to land so I can rebase without pulling it out of the gate | 15:31 |
gibi | lyarwood: ack | 15:31 |
admin0 | lyarwood, log: https://gist.githubusercontent.com/a1git/92688f89eab9fad7e2b30b9e3d2fd998/raw/353e37d292ae1ce4f4d6de8d4c0f20cd3920871d/gistfile1.txt | 15:32 |
*** jangutter has quit IRC | 15:33 | |
admin0 | "Unable to find record for source node h7.openstack.local on h7: ComputeHostNotFound: Compute host h7 could not be found." -- not sure how to set this up or configure it | 15:33 |
lyarwood | admin0: that's coming from https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L510-L520 but I thought that was using instance.host | 15:36 |
admin0 | lyarwood, what do i need to do to fix it | 15:36 |
*** damien_r has joined #openstack-nova | 15:38 | |
lyarwood | admin0: oh you need to update instance.node to h7 | 15:40 |
lyarwood | admin0: it's currently h7.openstack.local | 15:40 |
admin0 | is it on a file or a database that i need to update this ? | 15:40 |
lyarwood | admin0: database, the 'node' field of the instance table for this instance | 15:41 |
lyarwood | admin0: sorry the instances table | 15:43 |
*** damien_r has quit IRC | 15:43 | |
lyarwood | brb kid just got home | 15:44 |
*** damien_r has joined #openstack-nova | 15:44 | |
*** damien_r has quit IRC | 15:44 | |
*** ociuhandu has quit IRC | 15:46 | |
gibi | nova weekly meeting starts in 10 minutes on #openstack-meeting-3 | 15:50 |
*** rpittau is now known as rpittau|afk | 15:50 | |
openstackgerrit | Merged openstack/nova master: Add regression test for bug #1928063 https://review.opendev.org/c/openstack/nova/+/790658 | 15:50 |
openstack | bug 1928063 in OpenStack Compute (nova) "SEV enabled instance unable to hard reboot" [Medium,In progress] https://launchpad.net/bugs/1928063 - Assigned to Lee Yarwood (lyarwood) | 15:50 |
admin0 | i wait for you lyarwood .. i will be back in 40 mins | 16:00 |
lyarwood | admin0: I'm back, did you want the actual query? | 16:00 |
amodi | stephenfin: hi, can u merge https://review.opendev.org/c/openstack/nova/+/790724? | 16:01 |
stephenfin | amodi: nope - I'm not stable core, sadly. lyarwood could help though | 16:01 |
amodi | oh didn't know | 16:01 |
lyarwood | that reminds me... | 16:01 |
amodi | lyarwood: ^^ merge pls :) | 16:01 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: image_meta: Provide image_ref as the id when fetching from instance https://review.opendev.org/c/openstack/nova/+/790659 | 16:02 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: hardware: Use image_meta.id within get_mem_encryption_constraint https://review.opendev.org/c/openstack/nova/+/790660 | 16:02 |
*** sapd1_y has joined #openstack-nova | 16:02 | |
admin0 | lyarwood, yes please | 16:02 |
*** sapd1_x has quit IRC | 16:03 | |
admin0 | or if changing h7 to h7.openstack.local is faster/easier .. just need something that works in my usecase | 16:03 |
* admin0 is gone for 40 mins .. will try after coming back | 16:03 | |
lyarwood | admin0: update instances set node = 'h7' where uuid = '8733db85-23c6-4e10-8465-994eccdf549a'; | 16:06 |
*** raildo has quit IRC | 16:06 | |
stephenfin | gmann: if you've a chance later today, lyarwood is +2 on https://review.opendev.org/c/openstack/nova/+/790969/ now since I posted a follow-up https://review.opendev.org/c/openstack/nova/+/792353/ | 16:07 |
gmann | stephenfin: checking | 16:08 |
gmann | lyarwood: +2 on this https://review.opendev.org/c/openstack/devstack/+/792108 but we devstack wallaby gate is blocked until this merge https://review.opendev.org/c/openstack/devstack/+/792393 | 16:14 |
lyarwood | gmann: ack thanks | 16:14 |
*** dtantsur is now known as dtantsur|afk | 16:23 | |
*** yonglihe has quit IRC | 16:25 | |
*** ociuhandu has joined #openstack-nova | 16:30 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: docs: Add releases to libvirt distro support matrix https://review.opendev.org/c/openstack/nova/+/792415 | 16:35 |
*** ociuhandu has quit IRC | 16:36 | |
*** sapd1_y has quit IRC | 16:36 | |
*** sapd1_y has joined #openstack-nova | 16:37 | |
sean-k-mooney | lyarwood: well we could use the cloud archive right | 16:40 |
sean-k-mooney | althgoh that may noe have 7.0.0 | 16:41 |
sean-k-mooney | also i was ondering why the channel was so quite i had scolled up | 16:41 |
sean-k-mooney | lol and meeting just ended | 16:42 |
gibi | sean-k-mooney: yepp | 16:42 |
gibi | sean-k-mooney: and to confuse you we agreed to move the meeting to Tuesday 16:00 UTC :) | 16:42 |
sean-k-mooney | haha ok | 16:42 |
sean-k-mooney | sorry i had planned on attending but lost track of time | 16:43 |
gibi | sean-k-mooney: and 20.04 wallaby UCA only has libvirt 6.0.0 at the moment | 16:46 |
sean-k-mooney | ah ok | 16:46 |
sean-k-mooney | we dont actuly have to bump each cycle | 16:47 |
gibi | it seems at the moment we cannot bump even :) | 16:47 |
sean-k-mooney | as long as we do the review of what is avaiable | 16:47 |
sean-k-mooney | yep not sure that has hppened before we have usually been movign rahter slow | 16:47 |
sean-k-mooney | looks like we finally got to a relivtly modern min version | 16:48 |
gibi | yepp it seems so | 16:48 |
gibi | cores, the patch to move the weekly meeting: https://review.opendev.org/c/opendev/irc-meetings/+/792454 | 16:55 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | 17:04 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | 17:04 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | 17:04 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | 17:04 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: fixup for 1eca1c8b0cde23e701500655cdf2e3ead58a00e2 https://review.opendev.org/c/openstack/nova/+/792459 | 17:04 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support the new port resource_request format https://review.opendev.org/c/openstack/nova/+/787208 | 17:07 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | 17:07 |
*** derekh has quit IRC | 17:08 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | 17:11 |
openstackgerrit | Merged openstack/nova stable/wallaby: Replace blind retry with libvirt event waiting in detach https://review.opendev.org/c/openstack/nova/+/788720 | 17:27 |
*** ralonsoh has quit IRC | 17:37 | |
admin0 | lyarwood, thanks .. it worked :) | 17:40 |
admin0 | means now the status is shown in openstack horizon dashboard | 17:40 |
admin0 | trying migration | 17:41 |
*** links has quit IRC | 17:48 | |
*** andrewbonney has quit IRC | 17:48 | |
*** tbachman has quit IRC | 18:42 | |
sean-k-mooney | stephenfin: by the way im still not seeign a bluepirnt for https://review.opendev.org/q/topic:%22bp%252Fmultiqueue-flavor-extra-spec%22+(status:open%20OR%20status:merged) | 18:55 |
sean-k-mooney | ah its here https://blueprints.launchpad.net/nova/+spec/multiqueue-flavor-extra-spec | 18:56 |
sean-k-mooney | didnt come up in my sarch for some reason cool | 18:56 |
sean-k-mooney | $title lol | 18:57 |
*** tbachman has joined #openstack-nova | 19:12 | |
*** tbachman has quit IRC | 19:14 | |
*** tbachman has joined #openstack-nova | 19:22 | |
*** tbachman_ has joined #openstack-nova | 19:23 | |
*** tbachman has quit IRC | 19:26 | |
*** dustinc has joined #openstack-nova | 19:28 | |
*** tbachman_ has quit IRC | 19:29 | |
*** jmlowe has quit IRC | 19:33 | |
*** jmlowe has joined #openstack-nova | 19:36 | |
*** ociuhandu has joined #openstack-nova | 19:55 | |
*** ociuhandu has quit IRC | 20:00 | |
*** jawad_axd has joined #openstack-nova | 20:01 | |
*** jawad_axd has quit IRC | 20:06 | |
*** rcernin has joined #openstack-nova | 20:07 | |
*** belmoreira has quit IRC | 20:11 | |
*** rcernin has quit IRC | 20:13 | |
*** slaweq has quit IRC | 20:20 | |
*** slaweq has joined #openstack-nova | 20:22 | |
*** tbachman has joined #openstack-nova | 20:26 | |
*** slaweq has quit IRC | 20:29 | |
*** rcernin has joined #openstack-nova | 20:50 | |
*** sean-k-mooney[m] has joined #openstack-nova | 20:53 | |
openstackgerrit | Merged openstack/nova stable/wallaby: Move the guest.get_disk test to test_guest https://review.opendev.org/c/openstack/nova/+/788721 | 21:29 |
*** dustinc has quit IRC | 21:38 | |
*** rcernin has quit IRC | 21:49 | |
*** dansmith has quit IRC | 22:02 | |
*** dansmith has joined #openstack-nova | 22:06 | |
*** dansmith has quit IRC | 22:25 | |
*** dansmith has joined #openstack-nova | 22:25 | |
*** rcernin has joined #openstack-nova | 22:42 | |
openstackgerrit | Merged openstack/nova stable/wallaby: Enable mypy on libvirt/guest.py https://review.opendev.org/c/openstack/nova/+/788722 | 22:48 |
*** ociuhandu has joined #openstack-nova | 23:14 | |
*** macz_ has quit IRC | 23:17 | |
*** ociuhandu has quit IRC | 23:18 | |
openstackgerrit | Merged openstack/nova stable/wallaby: Follow up type hints for a634103 https://review.opendev.org/c/openstack/nova/+/788723 | 23:25 |
*** k_mouza has quit IRC | 23:26 | |
*** tosky has quit IRC | 23:52 | |
*** pmannidi has quit IRC | 23:54 | |
*** pmannidi has joined #openstack-nova | 23:56 | |
*** pmannidi has quit IRC | 23:56 | |
*** pmannidi has joined #openstack-nova | 23:57 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!