| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-10-08 | |||
| 14:07:59 | sean-k-mooney | at least i have irc again | |
| 14:16:01 | bauzas | sean-k-mooney: glad to see you | |
| 14:18:21 | sean-k-mooney | i now have fedora 32. goggle calandar is still broken in thruderbird and evoltion and now im having issue with email too but i have a gui again so i guess that is progress | |
| 14:21:43 | openstackgerrit | Navid Pustchi proposed openstack/nova-specs master: add availability-zone-affinity-anti-affinity-filter review. https://review.opendev.org/756380 | |
| 14:30:21 | openstackgerrit | Dan Smith proposed openstack/nova master: Allow excluding image type capabilities https://review.opendev.org/756534 | |
| 14:43:00 | ygk_12345 | hi all | |
| 14:43:02 | ygk_12345 | i have created a availability zone and added a compute node into it. but the zone status is showing as "not available". how can I change its status ? | |
| 14:43:27 | sean-k-mooney | i did not thnk the AZ had a staus | |
| 14:43:31 | sean-k-mooney | the host do | |
| 14:44:28 | ygk_12345 | sean-k-mooney check this outpout http://paste.openstack.org/show/798870/ | |
| 14:45:00 | ygk_12345 | sean-k-mooney the "test-aggregate-az" is not available | |
| 14:45:22 | ygk_12345 | i am not able to launch any vms in it | |
| 14:46:09 | sean-k-mooney | and if you check the host in test-aggreate | |
| 14:46:30 | sean-k-mooney | you see the host is listed and its avaiable/active | |
| 14:46:33 | ygk_12345 | the host is present in the aggregate | |
| 14:46:55 | ygk_12345 | but the host is not getting listed in the output | |
| 14:47:29 | ygk_12345 | http://paste.openstack.org/show/798871/ | |
| 14:48:35 | stephenfin | lyarwood: What, if anything, do you know about image-defined BDMs? | |
| 14:48:45 | stephenfin | Are they a thing we still use/support? | |
| 14:48:48 | ygk_12345 | sean-k-mooney did u see that. it hs one compute | |
| 14:48:54 | ygk_12345 | *has | |
| 14:49:08 | stephenfin | I'd trying to figure out what the e.g. 'img_mappings' image metadata property is/was used for | |
| 14:49:12 | stephenfin | *I'm | |
| 14:54:16 | sean-k-mooney | stephenfin: it might be realted to ephemeral disk | |
| 14:55:12 | sean-k-mooney | when you are using qcow the bdms have a type of local but i think there is a way or there was a way to split up the flavor.ephmerial storage using dbms to have multipel disks | |
| 14:55:28 | sean-k-mooney | no idea if that is what you are trying to figure out | |
| 14:55:50 | sean-k-mooney | e.g. i dont know if we support specfiying that in the image | |
| 14:56:02 | ygk_12345 | sean-k-mooney any idea abt the availability-zone issue ? | |
| 15:01:07 | sean-k-mooney | the docs dont really say much ill have to look at the code | |
| 15:01:09 | sean-k-mooney | https://docs.openstack.org/api-ref/compute/?expanded=get-detailed-availability-zone-information-detail,get-availability-zone-information-detail#get-availability-zone-information | |
| 15:01:46 | ygk_12345 | sean-k-mooney strange behavior here. its status is "not available" | |
| 15:03:00 | sean-k-mooney | https://github.com/openstack/nova/blob/2745e685376abbc4c32516837f6074a3de23aa24/nova/availability_zones.py#L108-L140 | |
| 15:03:07 | sean-k-mooney | so its what i tought | |
| 15:03:26 | sean-k-mooney | we list it as unavaiable if non of the compute service in the aggreate are enabled | |
| 15:03:44 | sean-k-mooney | meaning the compute service on the host you have added to the aggrate is down or disabled | |
| 15:03:59 | ygk_12345 | oh ok let me check that compute node then | |
| 15:06:30 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add image metadata property reference guide https://review.opendev.org/756867 | |
| 15:27:06 | stephenfin | bauzas: tox config isn't a API we need to protect :) We change things in there (and rightly so) all the time https://review.opendev.org/#/c/756135/ | |
| 15:27:37 | bauzas | stephenfin: sure, but I want to make sure that scripts still work and be a gentleman | |
| 15:27:49 | bauzas | I'm currently looking at how we could do it | |
| 15:27:52 | stephenfin | the only scripts using that are in zuul, which uses both | |
| 15:27:58 | stephenfin | but prefers the new style | |
| 15:28:30 | bauzas | honestly, I'm not that opiniated | |
| 15:28:55 | bauzas | stephenfin: are we sure that all CI (meaning 3rd party) won't use it too ? | |
| 15:30:10 | stephenfin | bauzas: I very much doubt it. The only reason to do so is if you wanted to use a different set of upper constraints than master. Also, they've probably hit it elsewhere http://codesearch.openstack.org/?q=TOX_CONSTRAINTS_FILE&i=nope&files=&repos= | |
| 15:31:03 | stephenfin | *different set of upper constraints than upstream/opendev.org | |
| 15:32:25 | bauzas | stephenfin: well, surely, but we can't assure we won't break others | |
| 15:32:41 | bauzas | stephenfin: that's the reason why I do care to be soft :) | |
| 15:32:41 | stephenfin | bauzas: actually, we can | |
| 15:32:55 | stephenfin | look at the zuul checks | |
| 15:32:59 | stephenfin | no failing third party CIs | |
| 15:33:17 | stephenfin | (in fact, no third-party CI results whatsoever) | |
| 15:34:12 | sean-k-mooney | if its overly complicated im fine with just changing it i like haveing the url in only one place however in the file | |
| 15:34:21 | sean-k-mooney | that the main thing i didnt like | |
| 15:34:37 | sean-k-mooney | we we can support both and prefer teh new form then that fine too | |
| 15:34:42 | bauzas | okay you know what ? | |
| 15:34:50 | sean-k-mooney | but if its overly complex lets just keep it simple | |
| 15:35:03 | bauzas | this change is taking too much of my time, I'll just quit the discussion with a +W | |
| 15:35:17 | bauzas | let's over bikeshed it, it's not worth it | |
| 15:35:25 | bauzas | not* bikeshed | |
| 15:37:34 | sean-k-mooney | sure i +1ed becasue i was not going to block over it i just did not want to merge it unless other agreeed as i prefered it teh way it was before | |
| 15:46:03 | gibi | nova meeting starts in 15 minutes on #openstack-meeting-3 | |
| 16:11:15 | openstack | Launchpad bug 1898715 in OpenStack Compute (nova) "Live migration fails despite matching CPUs" [Undecided,New] | |
| 16:11:15 | stephenfin | kashyap: You'd probably be interested in this https://bugs.launchpad.net/nova/+bug/1898715 | |
| 16:11:17 | stephenfin | Seems an easy fix | |
| 16:25:28 | kashyap | stephenfin: Will look; but from a quick first look, great sleuthing there by the reporter | |
| 16:25:51 | kashyap | The CPU model stuff...never ending joy | |
| 16:54:53 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move query param generation to RequestGroup o.vo https://review.opendev.org/756894 | |
| 16:54:59 | kashyap | stephenfin: Responded to Andrew there on the bug, and also added a note about what on earth does "require" supposed to mean. Thanks for bringing it to my attention :-) | |
| 16:55:19 | kashyap | (The "require" being an libvirt XML bit) | |
| 16:56:17 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor _claim_pci_device_for_interface_attach to prepare for qos https://review.opendev.org/756895 | |
| 16:57:38 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support interface attach with qos ports https://review.opendev.org/756530 | |
| 17:05:47 | openstackgerrit | Merged openstack/os-vif master: tox: Rename 'UPPER_CONSTRAINTS_FILE' -> 'TOX_CONSTRAINTS_FILE' https://review.opendev.org/756135 | |
| #openstack-nova - 2020-10-09 | |||
| 08:26:49 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ignore PCI devices with 32bit domain https://review.opendev.org/756697 | |
| 09:06:32 | lyarwood | stephenfin: image defined bdms | |
| 09:06:46 | lyarwood | stephenfin: block_device_mappings in the image metadata or something else? | |
| 09:06:55 | lyarwood | stephenfin: oh img_mappings | |
| 09:09:09 | lyarwood | stephenfin: I didn't think we used it anymore tbh | |
| 09:09:21 | lyarwood | stephenfin: I'll take a better look later today | |
| 09:41:14 | openstackgerrit | wangzhh proposed openstack/nova master: Ignore InstanceNotFound exception when host rejoin cluster https://review.opendev.org/757053 | |
| 10:21:12 | sean-k-mooney | lyarwood: we use it here https://github.com/openstack/nova/blob/9fa563666e8500510def19269f740d99cab0e994/nova/compute/api.py#L833-L856 | |
| 10:23:44 | sean-k-mooney | which is used here https://github.com/openstack/nova/blob/9fa563666e8500510def19269f740d99cab0e994/nova/compute/api.py#L907 | |
| 10:25:33 | lyarwood | sean-k-mooney: right we use block_device_mapping, I don't think we use img_mappings anywhere | |
| 10:25:42 | sean-k-mooney | we do | |
| 10:25:52 | sean-k-mooney | https://github.com/openstack/nova/blob/9fa563666e8500510def19269f740d99cab0e994/nova/compute/api.py#L933-L934 | |
| 10:26:04 | sean-k-mooney | we merge the image mappings with the block device mappings | |
| 10:26:21 | sean-k-mooney | before we append the flavor mappings | |
| 10:26:48 | sean-k-mooney | so right now we get the mappigns form the commandline the falvaor and iamge and merge them all together | |
| 10:27:06 | lyarwood | right but I still don't see any code looking up img_mappings | |
| 10:27:22 | sean-k-mooney | this does it https://github.com/openstack/nova/blob/9fa563666e8500510def19269f740d99cab0e994/nova/compute/api.py#L907-L908 | |
| 10:27:59 | sean-k-mooney | https://github.com/openstack/nova/blob/9fa563666e8500510def19269f740d99cab0e994/nova/compute/api.py#L841 | |
| 10:28:27 | lyarwood | sean-k-mooney: mappings != img_mappings | |
| 10:28:31 | sean-k-mooney | in the ovo we store "img_mappings" in the "mappings" field | |
| 10:28:34 | lyarwood | sean-k-mooney: that's my point | |
| 10:28:35 | sean-k-mooney | lyarwood:yes it is | |
| 10:29:22 | sean-k-mooney | https://github.com/openstack/nova/blob/f24e054f5e41dcf45b7ecd20e6d41f4a293a91d2/nova/objects/image_meta.py#L555 | |
| 10:29:40 | sean-k-mooney | img_mappings is the legacy name for mappings | |
| 10:29:54 | sean-k-mooney | we coalese it and store it in mappings | |
| 10:30:19 | lyarwood | mappings is the legacy name | |
| 10:30:21 | sean-k-mooney | actully i guess its the other way around | |