Monday, 2021-03-01

« Sunday, 2021-02-28 Index Tuesday, 2021-03-02 »
*** tosky has quit IRC00:11
*** zzzeek has quit IRC00:35
*** zzzeek has joined #openstack-nova00:36
*** swp20 has joined #openstack-nova00:38
*** zenkuro has quit IRC01:34
*** rcernin has quit IRC02:19
*** rcernin has joined #openstack-nova02:35
openstackgerritXing Zhang proposed openstack/nova stable/train: replace the "hide_hypervisor_id" to "hw:hide_hypervisor_id"  https://review.opendev.org/c/openstack/nova/+/76873602:58
*** brtknr has quit IRC03:00
*** brtknr has joined #openstack-nova03:00
*** tinwood has quit IRC03:10
*** tinwood has joined #openstack-nova03:13
*** psachin has joined #openstack-nova03:45
*** mkrai has joined #openstack-nova04:03
*** rcernin has quit IRC04:14
*** rcernin has joined #openstack-nova04:21
*** rcernin has quit IRC05:01
yongliheatmark: the pci alias is not a list.05:02
*** rcernin has joined #openstack-nova05:08
*** ratailor has joined #openstack-nova05:38
*** ralonsoh has joined #openstack-nova06:32
*** etp has quit IRC06:54
*** mkrai has quit IRC06:59
*** mkrai has joined #openstack-nova06:59
*** etp has joined #openstack-nova07:01
*** rcernin has quit IRC07:04
*** slaweq has joined #openstack-nova07:08
*** whoami-rajat has joined #openstack-nova07:11
*** etp has quit IRC07:17
*** mkrai has quit IRC07:18
*** etp has joined #openstack-nova07:19
*** k_mouza has joined #openstack-nova07:23
*** khomesh24 has joined #openstack-nova07:24
*** k_mouza has quit IRC07:27
*** aarents has joined #openstack-nova07:35
*** brinzhang has joined #openstack-nova07:37
brinzhanggmann: are you around?07:37
brinzhanggmann: I dont understand what are you point *'groups' data* in https://review.opendev.org/c/openstack/nova/+/766726/17/nova/tests/unit/api/openstack/compute/test_security_groups.py#100507:38
*** grandchild has joined #openstack-nova07:39
*** rcernin has joined #openstack-nova07:48
*** rcernin has quit IRC07:53
*** tobias-urdin has joined #openstack-nova07:55
openstackgerritBrin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Update Servers APIs  https://review.opendev.org/c/openstack/nova/+/76429208:01
openstackgerritBrin Zhang proposed openstack/nova master: Replace all_tenants with all_projects in List Server APIs  https://review.opendev.org/c/openstack/nova/+/76531108:01
openstackgerritBrin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Rebuild Server API  https://review.opendev.org/c/openstack/nova/+/76638008:01
openstackgerritBrin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List SG API  https://review.opendev.org/c/openstack/nova/+/76672608:01
*** rpittau|afk is now known as rpittau08:04
*** k_mouza has joined #openstack-nova08:10
*** k_mouza has quit IRC08:14
*** luksky has joined #openstack-nova08:17
*** tosky has joined #openstack-nova08:20
*** rcernin has joined #openstack-nova08:21
*** mkrai has joined #openstack-nova08:24
*** rcernin has quit IRC08:26
*** hemanth_n has joined #openstack-nova08:34
*** jangutter_ has joined #openstack-nova08:43
*** jangutter has quit IRC08:47
*** lucasagomes has joined #openstack-nova08:50
*** kashyap has joined #openstack-nova08:50
*** rcernin has joined #openstack-nova08:55
*** ociuhandu has joined #openstack-nova08:56
*** rcernin has quit IRC09:00
*** zoharm has joined #openstack-nova09:07
*** ociuhandu has quit IRC09:10
*** rcernin has joined #openstack-nova09:11
*** brinzhang_ has joined #openstack-nova09:15
*** ociuhandu has joined #openstack-nova09:15
*** rcernin has quit IRC09:16
*** brinzhang has quit IRC09:18
*** ttx has quit IRC09:20
*** ttx has joined #openstack-nova09:21
gibistephenfin: can we land https://review.opendev.org/c/openstack/python-novaclient/+/770573 before the client freze?09:29
*** derekh has joined #openstack-nova09:31
brinzhang_gibi, stephenfin, gmann: we add none type to the auth_schemes when we configing the devstack env, it's fixing the noVNC nova-next task failed, pls review if you have free time09:39
brinzhang_gibi, stephenfin, gmann: https://review.opendev.org/c/openstack/devstack/+/777923 thanks.09:39
brinzhang_gmann: I think this is not necessary to register a bug for devstack, it belongs to the bp/nova-support-webvnc-with-password-anthentication09:41
openstackgerritSylvain Bauza proposed openstack/nova master: WIP: Bump the Compute RPC API to version 6.0  https://review.opendev.org/c/openstack/nova/+/76145209:51
stephenfingibi: Sure, I'll respin that now09:52
*** k_mouza has joined #openstack-nova09:53
*** martinkennelly has joined #openstack-nova09:56
gibithanks09:57
*** khomesh24 has quit IRC10:00
openstackgerritStephen Finucane proposed openstack/python-novaclient master: Add support for microversion v2.88  https://review.opendev.org/c/openstack/python-novaclient/+/77057310:06
stephenfingibi ^10:06
gibistephenfin: I'm on it10:07
gibibrinzhang_: left a question in the tempest review10:11
gibisean-k-mooney: is there anything open in os-vif that needed for Wallaby? This week is nonclient library freeze. I looked tht the open patches and nothing popped out for me as important.10:16
*** jawad_axd has joined #openstack-nova10:21
gibistephenfin: +2 thanks for the client patch10:24
openstackgerritMerged openstack/nova master: objects: Add 'socket' PCI NUMA affinity  https://review.opendev.org/c/openstack/nova/+/77274810:26
*** ociuhandu has quit IRC10:36
*** ociuhandu has joined #openstack-nova10:38
*** jangutter has joined #openstack-nova10:41
*** jangutter has quit IRC10:42
*** jangutter has joined #openstack-nova10:43
*** ociuhandu has quit IRC10:43
*** jangutter_ has quit IRC10:44
*** rcernin has joined #openstack-nova10:51
*** xek has joined #openstack-nova10:54
*** mkrai has quit IRC10:56
*** sapd1 has joined #openstack-nova10:56
stephenfinbrinzhang_: gibi: Big comment left on https://review.opendev.org/c/openstack/nova/+/622336. I think we've all gotten confused /o\10:59
openstackgerritMerged openstack/nova master: Remove short call timeout from func test  https://review.opendev.org/c/openstack/nova/+/77744710:59
*** jangutter_ has joined #openstack-nova11:02
*** jangutter has quit IRC11:05
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Remove dead code  https://review.opendev.org/c/openstack/nova/+/77292811:07
openstackgerritStephen Finucane proposed openstack/nova master: rpc: Rework 'get_notifier', 'wrap_exception'  https://review.opendev.org/c/openstack/nova/+/74166311:07
openstackgerritStephen Finucane proposed openstack/nova master: mypy: Add type annotations to 'nova.pci'  https://review.opendev.org/c/openstack/nova/+/67620911:07
openstackgerritStephen Finucane proposed openstack/nova master: mypy: Add nova.cmd, nova.conf, nova.console  https://review.opendev.org/c/openstack/nova/+/70565711:07
openstackgerritStephen Finucane proposed openstack/nova master: mypy: Add type annotations to top-level modules  https://review.opendev.org/c/openstack/nova/+/70565811:07
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Clean manager.Manager, service.Service signatures  https://review.opendev.org/c/openstack/nova/+/76480611:07
*** ociuhandu has joined #openstack-nova11:15
*** ociuhandu has quit IRC11:19
*** k_mouza has quit IRC11:27
*** k_mouza has joined #openstack-nova11:28
*** ociuhandu has joined #openstack-nova11:31
*** rcernin has quit IRC11:31
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Allow disabling CPU flags via `cpu_model_extra_flags`  https://review.opendev.org/c/openstack/nova/+/77424011:35
*** rcernin has joined #openstack-nova11:38
*** ociuhandu has quit IRC11:44
*** zenkuro has joined #openstack-nova11:47
kashyapGah, needs a rebase11:49
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Allow disabling CPU flags via `cpu_model_extra_flags`  https://review.opendev.org/c/openstack/nova/+/77424011:49
*** rcernin has quit IRC11:55
*** ociuhandu has joined #openstack-nova11:56
*** rcernin has joined #openstack-nova12:00
*** ociuhandu has quit IRC12:01
*** ociuhandu has joined #openstack-nova12:12
stephenfinlyarwood: I saw that before. I think a second attempt to commit fixed the issue12:12
stephenfinotherwise just pass the -n flag temporarily12:12
lyarwoodhuh12:13
lyarwoodTIL about the -n flag12:13
lyarwoodthanks12:13
*** yoctozepto has quit IRC12:17
*** yoctozepto has joined #openstack-nova12:17
*** ociuhandu has quit IRC12:18
*** rcernin has quit IRC12:21
*** derekh has quit IRC12:23
*** derekh has joined #openstack-nova12:25
*** lpetrut has joined #openstack-nova12:30
sean-k-mooneygibi: thanks for the reminder. no not on master. there is one patch i want to be backported to train but ill ping lyarwood for that seperatly12:30
gibi sean-k-mooney: thanks for the confirmation12:30
sean-k-mooneylyarwood: https://review.opendev.org/c/openstack/os-vif/+/765970 is the patch i was refering too above12:31
sean-k-mooneylyarwood: we will need it downstream for 16.2 because the linux kernel patch was added in 8.412:32
sean-k-mooneygibi: ill try and figure out a unit test for https://review.opendev.org/c/openstack/nova/+/777679 when i get time12:34
sean-k-mooneyraising the excption via a mock and adding the assertions is not hard but its in an inner funciton so its a littele more work then normal to test.12:35
openstackgerritStephen Finucane proposed openstack/python-novaclient master: Add support for microversion v2.88  https://review.opendev.org/c/openstack/python-novaclient/+/77057312:43
*** hemanth_n has quit IRC12:44
*** ociuhandu has joined #openstack-nova12:45
*** ociuhandu_ has joined #openstack-nova12:47
*** ociuhandu has quit IRC12:50
*** tbarron|out is now known as tbarron12:51
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Add support for virtio-based input devices  https://review.opendev.org/c/openstack/nova/+/75655212:55
*** Luzi has joined #openstack-nova12:55
legochenhey sean-k-mooney, do you know anyone familiar with "Launch an instance from a volume” practcie… some questions posted above… would like to discuss13:00
sean-k-mooneyhi i have not read them back but i can take a look in a second13:02
gibisean-k-mooney: ack13:03
*** hoonetorg has joined #openstack-nova13:03
legochenthank you13:03
sean-k-mooneyok so your asking about osc and block device mappings13:04
sean-k-mooneystephenfin: has a patch up to rework how osc supports this currently13:05
sean-k-mooneyhttps://review.opendev.org/c/openstack/python-openstackclient/+/77169613:06
sean-k-mooneyso there are several ways to create a vm as im sure your aware13:07
sean-k-mooneyyou can create a vm with local root disk and have seperate data volumes13:08
sean-k-mooneyor you can replace the root disk with a cinder volume instaed of having it stored locally on the compute node.13:08
*** zul has joined #openstack-nova13:08
sean-k-mooneythe later can be done several ways and is what we refer to as boot form volume13:09
sean-k-mooneynova support eighter using a prexisting bootable volume, or a new volumn created by nova form either an image of volume snapshot13:09
sean-k-mooneythe efficncy fo each depend on 1 how you judge efficncy and 2 whatn nova, glance and cinder backends you have configured13:10
sean-k-mooneyfor example if you are using ceph for cinder nova and glance then boot form volume and non boot form volume are basically the same13:11
sean-k-mooneyin all cases teh root disk will just be a thin snapshot over the orignial image13:11
sean-k-mooneysimilarly you use cider as the glance backend and your cinder driver support it then bfv volumes created from images can simialrly be a thin provisioned volumn13:12
sean-k-mooneythe presence of addtional data volumne in any configuration is largely irrelvent to effincincy or perfroamce since in bfv or non bfv cases you should keep all data you care about in a cinder data volumn anyway so you can tree the root disk as ephemeral13:13
sean-k-mooneylegochen: not sure if that help but if you have a specific question i can try and answer that13:14
sean-k-mooneyim not really sure how you are gaugeing effiency (storage space? perfroamce(troughtput,latency,boot time)? network bandwith utilisation?)  or what your usecase is13:15
legochenhey sean-k-mooney, sorry just back. I think my major questions are about OSC stuff.13:18
legochen1) let “—boot-from-volume <volume-size>” option can specify volume-type13:18
legochen2) support —block-device option as what ocata version has.13:19
sean-k-mooney2 is done in https://review.opendev.org/c/openstack/python-openstackclient/+/771699/3/openstackclient/compute/v2/server.py13:20
legochenotherwise, the current OSC doesn’t have the ability to use an available & existing block-storage volume as root partition to boot an instance.13:20
sean-k-mooneystill unmerged13:20
legochenokay, understood, do we have timeline to merge it :D13:21
sean-k-mooneywe are deprecating --block-device-mapping in fravor of --block-device13:21
sean-k-mooneyam not sure13:21
sean-k-mooneystephenfin: ^ any idea when you expect the osc bfv change to merge13:22
sean-k-mooneywill it happen this cycle13:22
legochendo you mean you recommend --block-device instead of “ --block-device-mapping” ?13:22
sean-k-mooneylegochen: osc does not curently have --block-device but we are adding it and it will replace teh use fo teh existing --block-device-mapping command in osc13:23
lyarwoodsean-k-mooney: ack sorry was out at lunch13:23
legochenokay, thanks. clear now.13:23
legochenabout 1) … I feel if it cannot specify “volume-type”, it will be not that useful.13:24
sean-k-mooneylegochen: the imporant point form your point of view will be the abllit to set volume_type13:24
sean-k-mooneylegochen: that could be added13:24
sean-k-mooneywell looking at it its proably not possible actuly13:25
sean-k-mooneycurrently --boot-from-volume is defiend as an int13:26
sean-k-mooneywe could proably add a --volumn-type however13:26
sean-k-mooneyso you would use both13:26
*** ociuhandu_ has quit IRC13:26
sean-k-mooneyso somthign like --boot-from-volume 100G --image <my-image> --volume-type=<my-volume-type>13:27
legochenyes, that would be better !13:27
*** ociuhandu has joined #openstack-nova13:27
*** ociuhandu has quit IRC13:27
*** ratailor has quit IRC13:27
sean-k-mooneyyou should file a bug with osc or if you have time a patch?13:28
sean-k-mooneyotherwise see if stephenfin has time to adress it in a follow up.13:28
sean-k-mooneylegochen: currently he is trying to close the remaining gaps between nova clinet and osc13:28
sean-k-mooneyso that has priorty over adding syntatic sugar. this would be nice to have but less of a priorty then his other patches13:29
legochenthanks, about 2) is not that require to me beacuse I feel 1) is much better in my use cases.13:29
sean-k-mooney2 is needed for other usecase and what stephen has already implmented13:29
sean-k-mooney1 i would guess he wont have time to work on in the near future13:30
*** ociuhandu has joined #openstack-nova13:30
legochenunderstood….btw, if you don’t mind, I’d like to go back to talk the performance questions with you.13:30
legochenhttps://docs.openstack.org/nova/latest/user/launch-instance-from-volume.html13:31
sean-k-mooneysure13:31
legochenbasically, one OpenStack cluster could be split into two parts 1) Control Plane 2) Hypervisors .13:32
legochenin that document13:32
legochenThese two options “Create a volume from an image and boot an instance from that volume.” and “Boot from an existing source image, volume, or snapshot.”13:32
legochenLooks like the image download stuff happens on the Control Plane side.13:33
legochenOnly this otion “Boot an instance from an image and attach a non-bootable volume.” is happen on the hypervisor side13:33
*** jangutter has joined #openstack-nova13:33
sean-k-mooneythe first option https://docs.openstack.org/nova/latest/user/launch-instance-from-volume.html#boot-instance-from-image-and-attach-non-bootable-volume13:34
sean-k-mooneyis stanard boot. e.g. not boot form volumn13:34
openstackgerritLee Yarwood proposed openstack/nova master: nova-manage: Add libvirt get_machine_type command  https://review.opendev.org/c/openstack/nova/+/76954813:34
openstackgerritLee Yarwood proposed openstack/nova master: nova-manage: Add libvirt update_machine_type command  https://review.opendev.org/c/openstack/nova/+/77489613:34
openstackgerritLee Yarwood proposed openstack/nova master: nova-manage: Add libvirt list_unset_machine_type command  https://review.opendev.org/c/openstack/nova/+/77489713:34
openstackgerritLee Yarwood proposed openstack/nova master: nova-status: Add hw_machine_type check for libvirt instances  https://review.opendev.org/c/openstack/nova/+/77064313:34
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Add a config update workflow test for [libvirt]hw_machine_type  https://review.opendev.org/c/openstack/nova/+/77489813:34
openstackgerritLee Yarwood proposed openstack/nova master: docs: Add admin docs for configuring and updating machine types  https://review.opendev.org/c/openstack/nova/+/77489913:34
sean-k-mooneyso the image donwload happens on the compute node and we create a raw or qcow 2 file by defualt13:34
sean-k-mooneythe download only happens if we dont have the image in teh compute nodes image cache already13:35
*** eharney has joined #openstack-nova13:35
*** ociuhandu has quit IRC13:35
legochennova boot --flavor a5.8 --image rhel-7.9.11  --block-device source=volume,id=33ecc63a-ccc1-496b-ae9a-61babe615568,dest=volume,shutdown=preserve --availability-zone my-az --nic13:35
legochen net-name=my-network  boot-volume-instance-0113:35
sean-k-mooneyin the other cases the image may or may not be downloaded to the compte13:35
legochensean-k-mooney, okay, I mean this command13:35
legochenthis command will use an existing block volume attach to instance’s root partition to install the OS.13:36
legochenSo, that’s why I say the image download behavior happen on the hypervisor side.13:37
*** jangutter_ has quit IRC13:37
sean-k-mooneyno this will create a root disk form the image and attach a second data volumn13:37
sean-k-mooneylegochen: that command is the same as doing "nova boot --flavor a5.8 --image rhel-7.9.11 --availability-zone my-az --nic et-name=my-network  non-boot-volume-instance-0113:39
sean-k-mooneythen attaching a volumn13:39
sean-k-mooneyyou are not create a boot form volun instance13:39
sean-k-mooneyyou creatting a non boot form volumn instance with an addtion cinder volumn that is not used for the root disk13:39
sean-k-mooneyhttps://docs.openstack.org/nova/latest/user/launch-instance-from-volume.html#create-volume-from-image-and-boot-instance create a boot form volumn image using an image as a source13:40
sean-k-mooneynova boot --flavor FLAVOR --block-device \13:40
sean-k-mooney  source=SOURCE,id=ID,dest=DEST,size=SIZE,shutdown=PRESERVE,bootindex=INDEX \13:40
sean-k-mooney  NAME13:40
legochenokay, thanks. that’s my misunderstanding13:41
sean-k-mooneye.g.   source=image,id=<glance image uuid>,dest=volumn,size=100G,shutdown=PRESERVE,bootindex=0 \13:41
sean-k-mooneylegochen: basically the first workflow is used in 2 cases13:42
sean-k-mooney1 you do not want boot form volume13:42
legochenthen, seems like the follow would always require to create an image volume first and then use this volume to boot an instance.13:42
sean-k-mooney2 the image is an iso and you want to install it to a volumn which you will use later for a different vm13:42
legochenfollow = flow (typo)13:43
*** mgariepy has joined #openstack-nova13:44
*** jangutter_ has joined #openstack-nova13:44
sean-k-mooneylegochen: well you have 4 options 1 dont use boot form volumn, 2 boot form a new volumn created form an iamge, 3 boot form an existing volumne, 4 boot form a new volumne created as a volume snapshot form an existing volumne13:44
sean-k-mooneythe last 3 all use https://docs.openstack.org/nova/latest/user/launch-instance-from-volume.html#create-volume-from-image-and-boot-instance13:45
sean-k-mooneybut the source changes e.g. image,volumn or snapshot13:45
legochenour requirement is just wanted to let VM instance can use our block storage as root partition to boot.13:46
sean-k-mooneythat is the default if you dont use --block-device13:46
sean-k-mooneyunless you mean cinder13:46
sean-k-mooneyif you want to use cinder as teh root disk then you can use https://docs.openstack.org/nova/latest/user/launch-instance-from-volume.html#create-volume-from-image-and-boot-instance13:47
legochenblock storage - yes, managed by cinder13:47
sean-k-mooneythen yes follow ^ and it will create a bfv guest13:47
*** jangutter has quit IRC13:47
legochenwhat’s bfv?13:47
sean-k-mooneyboot form volumn13:47
sean-k-mooney*boot from volume13:48
*** luksky has quit IRC13:48
*** hemna has quit IRC13:48
sean-k-mooneyit gets tiresome to type so we usually use bfv as a shorthand13:48
legochenok, thanks. I just feel this flow is not that efficient. because it needs to13:48
legochen1) copy the image from glance to cinder image conversation folder13:49
legochen2)  mount cinder block storage on control plance13:49
legochen3) create image volume13:49
legochen4) create VM by this volume13:49
*** hemna has joined #openstack-nova13:49
sean-k-mooneyit wont do that in all cases13:49
sean-k-mooneyas i said if cinder and glance share teh same sotrage backend or if glance uses cinder volumns to store the image then cinder can do a more effect process13:50
legochenfeel the first two steps take time and cause disk-io loading during copy images.13:50
sean-k-mooneycinder does not need to do a copy in all cases13:50
sean-k-mooneyif you you ceph for glance and cidner then no copy happens provided they both use the same ceph cluster13:51
sean-k-mooneycinder will insted create a new volume as a thin snapshot of the image13:51
*** psachin has quit IRC13:51
sean-k-mooneyit will also do this in most cases when cinder is used as the glance image store backend13:51
legochendo I need to configure something on both cinder and glance config? let them can identify they don’t need to do copy stuff13:51
gibistephenfin: thanks for the comment on the vnc series. I'm glad we had the tempest test to show that we have a more complicated situation than what I thought13:52
sean-k-mooneyit need driver support but the copy should be elided.13:52
sean-k-mooneylegochen: am for ceph i dont think so but you would be better asking that question in the cinder channel13:52
legochendo you have example … I’d like to look into more about this.13:53
sean-k-mooneythey can explaine how to configure ciner/glace to avoid the copy13:53
legochenour glance is using NetApp filer to store images, but cinder is using EMC VxFlex block storage13:53
legochenwe don’t use ceph :)13:54
sean-k-mooneylegochen: in that case i dont think there is any way to fuly elid the copy although cinder also has an image cache.13:54
sean-k-mooneylegochen: in general its not a great idea to have multiple sotrage solution deployed in a single openstack cloud it prevent some optimnisation form being done13:55
sean-k-mooneyok i have to go do some work downstream for a while so ill be semi away from irc for a while o/13:57
legochengot it. thank you for your knowledge13:57
legochensean-k-mooney :)13:57
sean-k-mooneylegochen: in your spcific case createing vms form volumne snapshots might be the most effecinct13:59
sean-k-mooneybut do follow up with the cinder folks they may be able to help more. i belvie cinder has some config option to optimise this somewhat but im not super familar with all the options they have14:00
*** luksky has joined #openstack-nova14:01
legochensean-k-mooney, stephenfin - as my misunderstanding about —block-device option, I thought it can be used to root partition. So, I think to support —volume-type in OSC become a more useful feature … we need to have. otherwise, we’ll need to create image volume first and use that volume to boot an instance.14:01
legochensean-k-mooney, got it. will check with cinder channel about that. thank you.    “volume snapshots” could be a good idea, let me try it more.14:02
sean-k-mooneylegochen: it can be used for the root disk but --block-device is not only used for cinder it is used for advance block device configuration in general14:08
*** beekneemech is now known as bnemec14:08
legochenso, sounds like —block-device is also an ideal option?14:08
sean-k-mooneyi think that is what you missed if the dest type is local for exaple its used to configure epmermal storage14:08
legochenI guess I can use nova command to test that, right14:10
legochen  --block-device source=volume,id=33ecc63a-ccc1-496b-ae9a-61babe615568,dest=volume,shutdown=preserve14:10
legochen,bootindex=014:11
legochenneed to add this14:11
sean-k-mooneywell that will bot with an exsiting volumn14:13
sean-k-mooneyit wont create a new one14:13
sean-k-mooneybut ya anyway sorry got to go.14:13
legochenERROR (BadRequest): Block Device Mapping is Invalid: Boot sequence for the instance and image/block device mapping combination is not valid. (HTTP 400) (Request-ID: req-c004edcf-0deb-4354-8358-c78cdf7f6c08)14:14
legochenokay, I got this error when add “,bootindex=0"14:14
legochensure, you busy first.14:14
*** jangutter has joined #openstack-nova14:16
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Deprecate disable_native_luksv1 and rbd_volume_local_attach  https://review.opendev.org/c/openstack/nova/+/77800414:17
*** luksky has quit IRC14:17
sean-k-mooneylyarwood: i thought ^ was alredy dperecated when we added them14:19
sean-k-mooneyik guess you did not formally do it but that was the intent of your comment14:20
*** jangutter_ has quit IRC14:20
sean-k-mooneyto so we could actully remove them in W14:20
*** jangutte_ has joined #openstack-nova14:20
lyarwoodsean-k-mooney: yeah I didn't formally do it at the time, was about to do it now and noticed so thought I'd follow normal procedure and remove them early in Xena14:21
sean-k-mooneyok i guess it does not matter too much14:21
sean-k-mooneyit proably makes upgrades slightly simpler for 1714:21
lyarwoodyeah, would be nice to remove them now for my sanity but that's about it14:22
sean-k-mooneysince they can disable this after upgrade instead14:22
lyarwoodright14:22
lyarwoodwell they can disable it after they move to 16.2 downstream14:22
lyarwoodwe've backported the perf fixes to 8.314:22
sean-k-mooneyah ya good point14:23
*** jangutter has quit IRC14:23
sean-k-mooneylyarwood: also no worries about lunch, if we can move the os-vif backport forward that would be good as we need it before 16.2 is release but we have a few months to work on that14:24
*** mjturek has joined #openstack-nova14:25
lyarwoodkk14:26
*** ralonsoh_ has joined #openstack-nova14:30
*** ralonsoh has quit IRC14:30
*** luksky has joined #openstack-nova14:32
*** ralonsoh_ is now known as ralonsoh14:37
*** derekh has quit IRC14:38
*** derekh has joined #openstack-nova14:38
*** Luzi has quit IRC14:44
openstackgerritJessie Lass proposed openstack/nova master: Add emulation support if host arch != guest arch.  https://review.opendev.org/c/openstack/nova/+/77215615:00
*** amodi has joined #openstack-nova15:03
kashyapStrange, before rebasing, I didn't see this exception when running unit tests; but after rebase, I see:15:09
kashyap    "Exception: This test uses methods that set internal oslo_db state, but it does not claim to use the database [...]"15:09
kashyap[https://zuul.opendev.org/t/openstack/build/0347425ea0d74725a73f41ee4eaf6dd1]15:10
* kashyap taps on the table and thinks...15:10
sean-k-mooneylikely not mocking something you should15:17
lyarwoodyeah it should be in the trace15:18
lyarwoodah it's my stuff that's causing this for you15:19
lyarwood_register_instance_machine_type15:19
lyarwoodjust mock that out15:19
lyarwoodkashyap: ^15:19
kashyaplyarwood: Oh, I see.  That's the change the rebase brought15:20
kashyapsean-k-mooney: It came only after the rebase; it was not there before.15:20
sean-k-mooneylyarwood: that might want to be mocked in the setup funciton15:20
*** artom has joined #openstack-nova15:20
lyarwoodsean-k-mooney: I assume kashyap's tests are calling init_host directly15:22
kashyaplyarwood: Indeed, I noticed that removing the call to init_host "fixes" it - drvr.init_host("dummyhost")15:23
kashyaplyarwood: I don't even think it's required to call here.  Because, I checked in the PDB --15:23
kashyapMy test asserts expected results w/o the init_host() call15:23
kashyapSo I'm actually going to remove it; it's not required in this case, near as I can tell15:24
kashyapThanks for looking, folks!15:24
lyarwoodkashyap: https://review.opendev.org/c/openstack/nova/+/774240/10/nova/tests/unit/virt/libvirt/test_driver.py@147815:25
lyarwoodkashyap: looks like you removed the mock from test__check_cpu_compatibility_start_ok15:25
*** __ministry1 has joined #openstack-nova15:25
kashyaplyarwood: Urgh; that was accidental damage during rebase.  Sloppy me.15:25
kashyapI thought I double-checked, and still I failed15:25
*** __ministry1 has quit IRC15:26
*** mkrai has joined #openstack-nova15:26
kashyapThat's it; it's the missing mock15:28
*** lpetrut has quit IRC15:29
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Allow disabling CPU flags via `cpu_model_extra_flags`  https://review.opendev.org/c/openstack/nova/+/77424015:34
*** amodi has left #openstack-nova15:38
*** amodi has joined #openstack-nova15:38
openstackgerritBalazs Gibizer proposed openstack/nova stable/pike: rt: Make resource tracker always invoking get_inventory()  https://review.opendev.org/c/openstack/nova/+/77801415:43
*** eharney_ has joined #openstack-nova15:45
openstackgerritBalazs Gibizer proposed openstack/nova stable/pike: Update resources once in update_available_resource  https://review.opendev.org/c/openstack/nova/+/61229515:45
*** eharney has quit IRC15:46
*** spatel has joined #openstack-nova15:48
*** zoharm has quit IRC15:50
*** eharney_ is now known as eharney15:58
*** dklyle has joined #openstack-nova15:59
*** k_mouza has quit IRC15:59
*** k_mouza has joined #openstack-nova15:59
*** macz_ has joined #openstack-nova16:13
*** macz_ has quit IRC16:13
*** macz_ has joined #openstack-nova16:14
openstackgerritMerged openstack/nova master: apidb: Compact Mitaka database migrations  https://review.opendev.org/c/openstack/nova/+/75940016:23
stephenfinsean-k-mooney: Can you live with the bit of duplication in this? https://review.opendev.org/c/openstack/nova/+/775415/ I'd like to keep that series moving forward /o\16:24
*** macz_ has quit IRC16:27
*** macz_ has joined #openstack-nova16:28
*** mkrai has quit IRC16:32
*** mkrai has joined #openstack-nova16:32
openstackgerritBalazs Gibizer proposed openstack/nova stable/pike: rt: Make resource tracker always invoking get_inventory()  https://review.opendev.org/c/openstack/nova/+/77801416:33
openstackgerritBalazs Gibizer proposed openstack/nova stable/pike: Update resources once in update_available_resource  https://review.opendev.org/c/openstack/nova/+/61229516:34
sean-k-mooneystephenfin: ill take a look after the meeting16:37
*** mkrai has quit IRC16:51
dansmithbauzas: the compute 6.0 patch is failing almost everything because of versioning stuff. You have a TODO to fix that in the commit message, but are you looking for help via review or are we still waiting for you to work those issues out first?16:53
*** gyee has joined #openstack-nova16:53
*** jdillaman has joined #openstack-nova16:55
bauzasdansmith: I'm just fixing a lot of them ;)16:59
bauzasI saw a lot of stuff I forgot to modify16:59
*** lucasagomes has quit IRC16:59
bauzashopefully, will upload this in 1 hour16:59
*** efried has quit IRC16:59
dansmithbauzas: ah cool16:59
*** rpittau is now known as rpittau|afk17:00
bauzasdansmith: thanks for having looked at it btw. ;)17:02
dansmithbauzas: np, I said I would help, so definitely hold me to it.. just didn't want to duplicate effort17:02
bauzasdansmith: thanks17:03
*** efried has joined #openstack-nova17:09
openstackgerritBalazs Gibizer proposed openstack/nova stable/ussuri: Fallback to same-cell resize with qos ports  https://review.opendev.org/c/openstack/nova/+/77393217:23
gibimelwitt: fixed the functional test in the backport ^^17:23
sean-k-mooneystephenfin: oh it was the os.name poison change17:26
stephenfinyarp17:26
sean-k-mooneystephenfin: since other dont think its worth effort to remove the duplication fair ennough i guess.17:27
sean-k-mooneyi would still like it to be done but i can live with it so ill change to +1 after i quickly re review17:27
sean-k-mooneyi dont think you have changed anything since i last looked have you17:27
stephenfinnope, same as previously17:28
sean-k-mooneyya just rebases ok +1'd17:29
stephenfinta17:29
sean-k-mooneystephenfin: bauzas  suggested thinking about putting it in the integreated_helpers module as a followup17:30
sean-k-mooneynot sure if you saw that17:30
sean-k-mooneyi also think there is a tone of stuff in the chain after it and its not worth the gate time to respin now17:31
stephenfinI did. Given this thing is only used in unit tests, I don't think we want to be dragging in stuff from the functional test module17:31
stephenfinI can reply now17:31
sean-k-mooneyfair we import things form unti in fucntional17:32
sean-k-mooneyso better to avoid possible curcular deps between the too17:32
stephenfinyeah, if I had my way we wouldn't even import from unit inside functional tests17:32
stephenfinfixtures should be a separate submodule17:33
stephenfinbut that's a lot of work17:33
lyarwoodstephenfin: so I'm stuck on https://review.opendev.org/c/openstack/nova/+/673790/14/nova/virt/libvirt/host.py#123517:44
lyarwoodstephenfin: what's the reasoning behind flagging support based *any* machine type indicating support and not the default or requested machine type?17:45
lyarwoodstephenfin: I guess they all support UEFI and secure boot for x86_6417:45
lyarwoodstephenfin: but couldn't that cause issues on other arches if it isn't the case17:46
openstackgerritSylvain Bauza proposed openstack/nova master: WIP: Bump the Compute RPC API to version 6.0  https://review.opendev.org/c/openstack/nova/+/76145217:50
bauzasdansmith: hopefully this latest rev should fix all the func/unit tests $17:50
bauzas^17:50
bauzasnow, I need to see what tempest and grenade tell me17:51
*** songwenping_ has joined #openstack-nova17:52
*** MrClayPole_ has quit IRC17:53
*** MrClayPole has joined #openstack-nova17:53
*** swp20 has quit IRC17:54
*** mlavalle has joined #openstack-nova17:58
*** derekh has quit IRC18:01
*** ralonsoh has quit IRC18:08
dansmithbauzas: ack cool, will circle back later to see how zuul has judged you :P18:11
sean-k-mooneyfyi ptg dates are  April 19-23, 202118:11
sean-k-mooneygibi: i assume you will create a poll for times and such at some point but  the foundation just annouched that and registrations aree now open18:12
bauzasa bit sad to have again a virtual DFG instead of a face-to-face :(18:15
sean-k-mooneywell in that time frame realsiticlaly vactiontion wont be to the point that people could attend18:16
sean-k-mooneymaybe in november18:16
sean-k-mooneyor whenever the next one would be.18:16
*** k_mouza has quit IRC18:18
*** k_mouza_ has joined #openstack-nova18:18
*** k_mouza_ has quit IRC18:19
sean-k-mooneystephenfin: by the way yes it would be nice if the fixtures were in a sperate module not under the unit or functional test but also yes that is a lot of work18:26
atmarkHello, I have 2x NVMes that has same vendor and product id and I want to setup PCI passthrough. How can I separate the two NVMes in the alias? I currently have the following config18:30
atmarkpassthrough_whitelist = {"vendor_id":"1344", "product_id":"51b2"}18:30
atmarkalias = {"vendor_id":"1344", "product_id":"51b2", "device_type":"type-PCI", "name":"nvme"}18:30
sean-k-mooneyatmark: technially that is not supported. or rather nova pci passtough is only supported when passing through stateless devices18:32
sean-k-mooneyNVME ssd are state full18:32
sean-k-mooneyits technically a security risk to sue them with nvme devices18:32
sean-k-mooneysince we will not ease the data when the vm is deleted18:32
sean-k-mooneyatmark: with that said that whitelist and alias should work18:33
sean-k-mooneycan you explain what you mean by seperate the two in teh whitelist18:34
sean-k-mooneyare you trying to tag the devices in some way and match the alias to a specific device on the host via that tag18:34
sean-k-mooneythere is no supported way to do that im affraid18:36
sean-k-mooneythe schema for the alias is https://github.com/openstack/nova/blob/master/nova/pci/request.py#L64-L9718:36
sean-k-mooneyit does not have a way to match on the extra tags you can add in the pci whitelist18:36
sean-k-mooneynor can it match on a pci address or similar and that is by design18:36
sean-k-mooneythe alias is not allowed to be host specific18:37
atmarkYup. I want to tag devices that match alias to a specific device on a host. I have these NVMes which has same product and vendor ID18:37
atmarkIOMMU Group 30 41:00.0 Non-Volatile memory controller [0108]: Micron Technology Inc Device [1344:51b2] (rev 02)18:38
atmarkIOMMU Group 30 42:00.0 Non-Volatile memory controller [0108]: Micron Technology Inc Device [1344:51b2] (rev 02)18:38
sean-k-mooneyatmark: ya so we do not supprot that. i can see aded support for matching on the extra tags in teh pci whitelist18:38
sean-k-mooneybut not specificly based on address18:38
sean-k-mooneyatmark: the alais has to be the same on all compute nodes and the contolers so we need a layer of indriection18:39
sean-k-mooneythis would be a new feature unfortunetly and  it still would not adress the fact that stateful devices are unsupported both upstream and downstream18:39
sean-k-mooneystateful devices can work but there is no tenant isolation18:40
atmarkThanks for the info. I will consider other solution then.19:00
*** stand has joined #openstack-nova19:12
*** hamalq has joined #openstack-nova19:24
*** belmoreira has joined #openstack-nova19:26
openstackgerritMerged openstack/nova master: tests: Add 'nova.virt.libvirt.utils.get_arch' stub to fixture  https://review.opendev.org/c/openstack/nova/+/77490419:44
*** LinPeiWen has quit IRC19:44
openstackgerritMerged openstack/nova master: tests: Remove unnecessary mock  https://review.opendev.org/c/openstack/nova/+/77490519:44
*** jawad_axd has quit IRC20:01
*** haleyb has joined #openstack-nova20:04
*** jawad_axd has joined #openstack-nova20:07
*** jawad_axd has quit IRC20:10
*** psycocyst has joined #openstack-nova20:15
*** k_mouza has joined #openstack-nova20:20
*** k_mouza has quit IRC20:25
*** mgagne has joined #openstack-nova20:41
*** luksky has quit IRC20:46
openstackgerritsean mooney proposed openstack/nova master: [WIP] add vdpa nodedev parsing and interface config gen  https://review.opendev.org/c/openstack/nova/+/77053220:48
openstackgerritsean mooney proposed openstack/nova master: extend pci dev_type for vdpa  https://review.opendev.org/c/openstack/nova/+/77748120:48
openstackgerritsean mooney proposed openstack/nova master: [WIP] add vdpa device detection  https://review.opendev.org/c/openstack/nova/+/77053320:48
*** luksky has joined #openstack-nova21:02
*** ihti[m] has joined #openstack-nova21:07
*** belmoreira has quit IRC21:20
openstackgerritMerged openstack/nova master: scheduler: Add 'socket' PCI NUMA affinity policy trait  https://review.opendev.org/c/openstack/nova/+/77274921:29
openstackgerritSylvain Bauza proposed openstack/nova master: Bump the Compute RPC API to version 6.0  https://review.opendev.org/c/openstack/nova/+/76145221:30
bauzasdansmith: zuul gave me a -1 but I think i fixed the latest issues ^21:32
*** spatel has quit IRC21:33
*** rcernin has joined #openstack-nova21:33
bauzasjust a concern about a grenade multihost job giving me a -1 but when looking at the logs, it's about a timeout21:34
*** spatel has joined #openstack-nova21:34
*** tbachman has quit IRC21:36
*** rcernin has quit IRC21:54
*** rcernin has joined #openstack-nova21:55
*** rcernin has quit IRC21:55
*** rcernin has joined #openstack-nova21:55
*** psycocyst has quit IRC22:03
*** whoami-rajat has quit IRC22:07
*** tbachman has joined #openstack-nova22:10
openstackgerritLee Yarwood proposed openstack/nova master: nova-manage: Add libvirt update_machine_type command  https://review.opendev.org/c/openstack/nova/+/77489622:11
openstackgerritLee Yarwood proposed openstack/nova master: nova-manage: Add libvirt list_unset_machine_type command  https://review.opendev.org/c/openstack/nova/+/77489722:11
openstackgerritLee Yarwood proposed openstack/nova master: nova-status: Add hw_machine_type check for libvirt instances  https://review.opendev.org/c/openstack/nova/+/77064322:11
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Add a config update workflow test for [libvirt]hw_machine_type  https://review.opendev.org/c/openstack/nova/+/77489822:11
openstackgerritLee Yarwood proposed openstack/nova master: docs: Add admin docs for configuring and updating machine types  https://review.opendev.org/c/openstack/nova/+/77489922:11
*** xek has quit IRC22:19
*** grandchild has quit IRC22:22
*** tbachman has quit IRC22:33
*** rcernin has quit IRC22:39
*** luksky has quit IRC22:41
*** rcernin has joined #openstack-nova22:42
lyarwoodsean-k-mooney: https://zuul.opendev.org/t/openstack/build/ba023003dd21428b95f27a200437dfd8/log/job-output.txt  / https://review.opendev.org/c/openstack/os-vif/+/765419/6 ; had a quick look at the os-vif backports, did you plan on fixing the LC job?22:42
lyarwoodsean-k-mooney: if not I could take a look after M322:42
*** grami[m] has left #openstack-nova22:46
*** luksky has joined #openstack-nova22:54
*** spatel has quit IRC23:00
*** slaweq has quit IRC23:07
*** gibi has quit IRC23:58
« Sunday, 2021-02-28 Index Tuesday, 2021-03-02 »

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!