Friday, 2021-01-08

*** spatel has joined #openstack-nova00:18
*** spatel has quit IRC00:23
*** sapd1 has quit IRC00:39
*** macz_ has quit IRC00:41
*** rcernin has quit IRC00:44
*** ircuser-1 has quit IRC01:12
*** rcernin has joined #openstack-nova01:45
*** Yumeng has quit IRC02:01
*** mlavalle has quit IRC02:06
*** _mlavalle_1 has joined #openstack-nova02:07
*** zzzeek has quit IRC02:09
*** zzzeek has joined #openstack-nova02:10
*** zzzeek has quit IRC02:15
*** zzzeek has joined #openstack-nova02:16
*** sapd1 has joined #openstack-nova02:20
*** hemanth_n has joined #openstack-nova02:23
*** sapd1 has quit IRC02:30
*** macz_ has joined #openstack-nova02:42
*** zzzeek has quit IRC02:43
*** sapd1 has joined #openstack-nova02:43
*** zzzeek has joined #openstack-nova02:45
*** macz_ has quit IRC02:47
*** sapd1 has quit IRC03:01
*** hemanth_n has quit IRC03:02
*** sapd1 has joined #openstack-nova03:10
*** sapd1 has quit IRC03:13
*** hemanth_n has joined #openstack-nova03:14
*** rcernin has quit IRC03:39
*** rcernin has joined #openstack-nova03:52
*** rcernin has quit IRC03:52
*** rcernin has joined #openstack-nova03:52
*** hamalq has quit IRC04:01
*** bbowen_ has joined #openstack-nova04:06
*** ircuser-1 has joined #openstack-nova04:07
*** bbowen has quit IRC04:08
*** mkrai has joined #openstack-nova04:21
*** zzzeek has quit IRC04:36
*** zzzeek has joined #openstack-nova04:38
*** macz_ has joined #openstack-nova04:43
*** macz_ has quit IRC04:47
*** zzzeek has quit IRC04:49
*** zzzeek has joined #openstack-nova04:51
*** zzzeek has quit IRC05:05
*** zzzeek has joined #openstack-nova05:07
*** whoami-rajat__ has joined #openstack-nova05:25
*** zzzeek has quit IRC05:28
*** zzzeek has joined #openstack-nova05:31
openstackgerritBrin Zhang proposed openstack/python-novaclient master: Microversion 2.88: Replace tenant* with project* in quota* commands  https://review.opendev.org/c/openstack/python-novaclient/+/76986905:33
*** rcernin has quit IRC05:36
*** zzzeek has quit IRC05:38
*** zzzeek has joined #openstack-nova05:40
*** rcernin has joined #openstack-nova05:44
*** sapd1 has joined #openstack-nova05:47
*** zzzeek has quit IRC05:54
*** rcernin has quit IRC05:56
*** zzzeek has joined #openstack-nova05:56
*** sapd1 has quit IRC06:10
*** gyee has quit IRC06:11
*** hoonetorg has quit IRC06:19
*** rcernin has joined #openstack-nova06:22
*** rcernin has quit IRC06:23
*** rcernin has joined #openstack-nova06:24
*** sapd1 has joined #openstack-nova06:25
openstackgerritBrin Zhang proposed openstack/python-novaclient master: DNM/TEST: nova live migration  https://review.opendev.org/c/openstack/python-novaclient/+/76987206:28
*** hoonetorg has joined #openstack-nova06:32
*** ratailor has joined #openstack-nova06:32
*** rcernin has quit IRC06:39
*** belmoreira has joined #openstack-nova06:40
*** rcernin has joined #openstack-nova06:44
*** zzzeek has quit IRC06:51
*** zzzeek has joined #openstack-nova06:53
*** sapd1 has quit IRC06:59
*** ralonsoh has joined #openstack-nova07:03
*** spatel has joined #openstack-nova07:05
*** spatel has quit IRC07:05
*** admin0 has quit IRC07:09
*** rcernin has quit IRC07:14
*** rcernin has joined #openstack-nova07:28
*** zzzeek has quit IRC07:31
*** zzzeek has joined #openstack-nova07:32
*** rcernin has quit IRC07:43
*** zenkuro has joined #openstack-nova07:47
*** dklyle has quit IRC07:51
*** swp20 has quit IRC07:55
*** swp20 has joined #openstack-nova07:55
*** bbowen has joined #openstack-nova08:01
*** bbowen_ has quit IRC08:02
*** mkrai has quit IRC08:10
*** tesseract has joined #openstack-nova08:16
*** zzzeek has quit IRC08:21
openstackgerritWenping Song proposed openstack/nova master: Nova supports password encrypted VNC  https://review.opendev.org/c/openstack/nova/+/62233608:22
*** zzzeek has joined #openstack-nova08:23
*** andrewbonney has joined #openstack-nova08:24
*** zzzeek has quit IRC08:27
*** zzzeek has joined #openstack-nova08:28
*** mkrai has joined #openstack-nova08:29
*** rpittau|afk is now known as rpittau08:32
*** tesseract has quit IRC08:41
*** tesseract has joined #openstack-nova08:42
*** tkajinam has quit IRC08:55
*** songwenping_ has joined #openstack-nova08:56
*** tosky has joined #openstack-nova08:56
*** zzzeek has quit IRC08:57
*** zzzeek has joined #openstack-nova08:58
*** swp20 has quit IRC08:59
openstackgerritWenping Song proposed openstack/nova master: Nova supports password encrypted VNC  https://review.opendev.org/c/openstack/nova/+/62233609:15
openstackgerritzhangboye proposed openstack/nova master: Remove __unicode__() from nova unit test Exception  https://review.opendev.org/c/openstack/nova/+/76989409:21
*** tesseract has quit IRC09:36
*** tesseract has joined #openstack-nova09:36
*** derekh has joined #openstack-nova09:37
*** brinzhang has quit IRC09:38
*** tesseract has quit IRC09:38
*** tesseract has joined #openstack-nova09:38
*** zzzeek has quit IRC09:44
*** zzzeek has joined #openstack-nova09:46
*** martinkennelly has joined #openstack-nova09:46
*** zzzeek has quit IRC09:56
*** zzzeek has joined #openstack-nova09:57
*** dtantsur|afk is now known as dtantsur09:57
*** sapd1 has joined #openstack-nova10:09
openstackgerritStephen Finucane proposed openstack/nova master: api: Drop statistics-style fields from os-hypervisors  https://review.opendev.org/c/openstack/nova/+/76404010:12
openstackgerritStephen Finucane proposed openstack/nova master: tests: Clean up 'test_hypervisors'  https://review.opendev.org/c/openstack/nova/+/76990310:12
*** whoami-rajat__ has quit IRC10:13
*** sapd1 has quit IRC10:14
*** sapd1 has joined #openstack-nova10:15
*** ttx has quit IRC10:16
*** ttx has joined #openstack-nova10:17
*** zzzeek has quit IRC10:18
*** sapd1 has quit IRC10:20
*** zzzeek has joined #openstack-nova10:20
openstackgerritElod Illes proposed openstack/nova stable/stein: [stable-only] Cap bandit and make lower-constraints job non-voting  https://review.opendev.org/c/openstack/nova/+/76648710:20
*** songwenping_ has quit IRC10:21
gibisean-k-mooney: hi! There is a SRIOV NIC, couple of VF is created on the PF and then both PF and VFs are whitelisted in nova compute. Then booting a VM with a direct-physical port nova will allocated the PF and makes the VFs unavailable. But which module (nova, neutron, libvirt) makes sure that the VFs are deleted?10:21
gibiDoes it work like the following? libvirt detaches the PF from the host and that makes the VFs deleted?10:22
openstackgerritElod Illes proposed openstack/nova stable/stein: WIP [stable-only] Fix lower-constraints job  https://review.opendev.org/c/openstack/nova/+/76980910:22
*** zzzeek has quit IRC10:27
*** zzzeek has joined #openstack-nova10:29
*** ociuhandu has joined #openstack-nova10:31
*** ociuhandu has quit IRC10:39
*** ociuhandu has joined #openstack-nova10:41
*** sapd1 has joined #openstack-nova11:07
*** sapd1 has quit IRC11:15
*** zzzeek has quit IRC11:18
*** zzzeek has joined #openstack-nova11:20
*** sapd1 has joined #openstack-nova11:21
*** ociuhandu has quit IRC11:22
*** ratailor has quit IRC11:28
*** sapd1 has quit IRC11:30
*** zzzeek has quit IRC11:38
*** zzzeek has joined #openstack-nova11:43
openstackgerritElod Illes proposed openstack/nova stable/rocky: [stable-only] Cap bandit and make lower-constraints job non-voting  https://review.opendev.org/c/openstack/nova/+/76649211:43
openstackgerritElod Illes proposed openstack/nova stable/rocky: WIP [stable-only] Fix lower-constraints job  https://review.opendev.org/c/openstack/nova/+/76991011:44
yonglihegibi, look libvirt detach it from host if use managed mode.11:46
yonglihe<hostdev mode='subsystem' type='pci' managed='yes'>, if this is what you looking for.11:47
*** xinranwang has joined #openstack-nova11:47
gibiyonglihe: thanks that is what I was looking for11:47
xinranwanggibi: Hi gibi, as we discussed in nova spec, a new vnic type is needed for device managed by cyborg. I create a neutron RFE for this, and there is some question posed. Could you please check it ? https://bugs.launchpad.net/neutron/+bug/190910011:51
openstackLaunchpad bug 1909100 in neutron "[RFE]add new vnic type "cyborg"" [Wishlist,Confirmed] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)11:52
gibixinranwang: I deferr to sean-k-mooney and the neutron folks here. From my perspective what nova needs is a way to know that the port needs a special handling (talking to cyborg) during boot.11:54
xinranwanggibi:  yes, nova can check the vnic type and trigger the interaction with cyborg then.11:56
gibicommented on the RFE11:56
gibixinranwang: yes11:56
gibia new vnic_type is OK for me11:56
gibibut maybe the neturon folks has other idea11:57
xinranwangNeutron will have irc meeting at 14:00 UTC today. We will discuss this topic. Do you and sean-k-mooney  have time to join the discussion?11:58
stephenfinlyarwood, gibi, sean-k-mooney: If you've a chance, could you take a look at https://review.opendev.org/c/openstack/python-openstackclient/+/762396/ again at some point today?11:58
stephenfin(OSC change to "Add 'server shelve --offload', 'server shelve --wait' options")11:59
gibixinranwang: I will join, please ping me if I did not show up11:59
gibistephenfin: added to my review list11:59
xinranwanggibi:  cool, thanks12:00
lyarwoodstephenfin: queued12:02
stephenfingibi: I've closed this bug as WONTFIX. I don't think we have access to PPC nodes in zuul, which means we can't say we support anything other than x86/x86_64. Lemme know if you disagree https://bugs.launchpad.net/nova/+bug/190997212:07
openstackLaunchpad bug 1909972 in OpenStack Compute (nova) "a number of tests fail under ppc64el arch" [Low,Won't fix]12:07
gibistephenfin: I saw that and I have the same problem, I cannot start fixing those tests as I have no way to reproduce the problem12:08
gibi(no I did not tried spinning up a ppc VM with qemu)12:08
stephenfinYeah, I've suggested that zigo simply mark the package as arch-specific. If it's not tested, it's broken12:08
gibistephenfin: did we declare support for ppc somewhere in nova doc?12:08
stephenfinI don't think so12:08
stephenfinWe don't block it but https://github.com/openstack/nova/blob/46899968619e4ea0ff2ab380977619bb29578d43/nova/virt/libvirt/driver.py#L572-L58112:09
lyarwoododd why don't we list aarch64 there?12:10
lyarwoodI thought we had jobs now12:10
lyarwoodah third party ignore me12:10
stephenfinsomeone's working on it but it's not merged yet, no12:10
gibistephenfin: good point12:10
*** bbowen_ has joined #openstack-nova12:10
gibistephenfin: if we even log a warning then that should be enough to close the bug12:10
*** bbowen has quit IRC12:10
gibilet's see what zigo's oppinion about this12:12
*** ociuhandu has joined #openstack-nova12:13
*** ociuhandu has quit IRC12:14
*** ociuhandu has joined #openstack-nova12:15
sean-k-mooneyxinranwang: i should be able to join too12:19
zigostephenfin: gibi: My opinion is: if you can't test with PPC, and that the unit test is reported as broken on non-x86_64, then skip it on non-x86_64 rather than just closing the bug ...12:20
sean-k-mooneystephenfin: yes i can take a look at that shortly12:20
stephenfinzigo: what about when someone adds the next test that doesn't work on x86_6412:20
sean-k-mooneygibi: the vf are deleted by the kernel when the PF is passhtough to the vm12:21
zigostephenfin: These issues are *very* rare.12:21
zigoSo we can act on a case-by-case ...12:21
gibisean-k-mooney: thanks12:21
stephenfinI disagree. You skip tests because they're not applicable. These tests are applicable. They simply don't work on PPC12:22
sean-k-mooneythe managed=yes part of <hostdev mode='subsystem' type='pci' managed='yes'> i blelive tells libvirt to manage bining the pf to vfio-pci form its current nic driver and to restore it when the vm is done12:22
stephenfinwho knows what else doesn't work12:22
sean-k-mooneyby which i mean its stop and the pf returns to the host12:23
sean-k-mooneytechnically its udev coupled with the kernel config that decide what driver it goes back too12:23
zigostephenfin: As much as I understand, they would work if s/ide/scsi/ ...12:23
stephenfinPerhaps. I genuinely don't know12:24
sean-k-mooneyif you used udev rules to allocate the vfs then that will recreate them12:24
stephenfinwhich is kind of the point12:24
sean-k-mooneyon the ppc front12:25
sean-k-mooneywe have support power kvm for years12:25
sean-k-mooneywhich is libvirt/kvm on ppc12:25
sean-k-mooneyibm has a ci for it12:25
sean-k-mooneythat runs against nova12:26
stephenfinsean-k-mooney: clearly they don't, or these tests would be failing there12:26
stephenfinor their CI job matrix isn't complete12:26
*** ociuhandu has quit IRC12:26
sean-k-mooneyit only runs integration tests ie tempest12:26
sean-k-mooneyhttps://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/68/767368/1/check/tempest-dsvm-full-focal-py3/ef10362/12:27
sean-k-mooneythat a run from the IBM PowerKVM CI12:27
sean-k-mooneyppc64el12:27
sean-k-mooneystephenfin: gibi zigo  we have no unit or func test coverage12:28
*** zzzeek has quit IRC12:28
sean-k-mooneybut we cant mark the package as not supported on ppc or it will break the ablity to install nova on ppc which has been supported for 4+ years12:29
sean-k-mooneypowervm support started in ocata and was finished in queens. powerkvm i think predates powervm which is there own hypervior12:30
*** zzzeek has joined #openstack-nova12:30
sean-k-mooneystephenfin: i think we should reopen https://bugs.launchpad.net/nova/+bug/190997212:34
openstackLaunchpad bug 1909972 in OpenStack Compute (nova) "a number of tests fail under ppc64el arch" [Low,Won't fix]12:34
sean-k-mooneystephenfin: we cant just declare ppc64el as unsupported when it has been supported for years12:35
sean-k-mooneystephenfin: we even have a mulit arch dfg downstream that is workign to get support for power into osp12:35
sean-k-mooneystephenfin: we event ship ppc version of osp in osp 16 https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/16.0/html/release_notes/chap-introduction12:38
sean-k-mooneyppc64le to be specific12:38
*** zzzeek has quit IRC12:42
*** hemanth_n has quit IRC12:44
*** zzzeek has joined #openstack-nova12:45
zigoAt least, LibvirtBlockInfoTest.test_get_disk_mapping_stable_rescue_ide_cdrom can be fixed...12:46
zigoLibvirtBlockInfoTest.test_get_disk_mapping_rescue_with_config too.12:46
zigoTest result is {'bus': 'scsi', 'dev': 'sda', 'type': 'cdrom'} while it's expecting {'bus': 'ide', 'dev': 'hda', 'type': 'cdrom'}12:46
sean-k-mooneyyes12:47
lyarwoodyeah that's my bad, smells like I didn't mock out the arch check somewhere12:47
gibisean-k-mooney: could we ask IBM to include unit and functional test in their CI? then we could have coverage and properly fix the test12:47
sean-k-mooneygibi: we can always ask but not sure if it will happen :)12:48
zigoThis is also a problem on arm, FYI.12:48
zigo(the above 2 unit tests failing...)12:48
sean-k-mooneyi might reach out to our multi arch folks downstream and see if they can help12:48
sean-k-mooneywe do package nova for ppc64el downstream and we have limited downstream ci for it12:49
sean-k-mooneypresumably that shoudl included unit and functional test but they may still be priamlary working with 16 which is train or relying on x86 runs12:49
sean-k-mooneyfor unit and functional12:50
sean-k-mooneyzigo: for arm we can test that in the gate12:50
sean-k-mooneyzigo: we have limited ablity to run thigns in the arm check pipeline12:51
sean-k-mooneywell we can run anything we want in that pipeline just limited capsity12:51
sean-k-mooneyrunning unit and funcitonal there i think would be ok12:51
sean-k-mooneyill ask infra and can submit a patch to do that12:51
sean-k-mooneyactully openstack-qa woudl be better12:52
*** ociuhandu has joined #openstack-nova12:53
zigoGreat !12:55
lyarwoodwhat was the bug for that again?12:56
lyarwoodnvm12:57
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Mock get_arch during some stable rescue unit tests  https://review.opendev.org/c/openstack/nova/+/76991612:57
*** ociuhandu has quit IRC12:58
*** ociuhandu has joined #openstack-nova12:58
gibisean-k-mooney, stephenfin, zigo: I've reached out to the IBM CI contact person and asked about running unit and functional tests in there system on ppc http://lists.openstack.org/pipermail/openstack-discuss/2021-January/019726.html13:00
zigoNice ! Thanks.13:00
xinranwangsean-k-mooney:  thanks!13:07
sean-k-mooneygibi: ack. ill be afk for 30 mins or so, if they dont respond or cant add them in a week or so ill follow up with our multi arch team and see if there is anything we can do. we normally have issue ruinng upstream ci on our hardware due to it being conencted to our internal network but i could ask if they have any disconnected pool that could be used for public unit/func test runs. im not13:09
sean-k-mooneyaware of one but there might be13:09
sean-k-mooneygibi: they might be providing testign for rdo for example on power13:09
gibisean-k-mooney: ack, thanks13:10
openstackgerritLee Yarwood proposed openstack/nova master: tests: Posion nova.objects.fields.Architecture.from_host  https://review.opendev.org/c/openstack/nova/+/76992013:16
lyarwoodgibi: ^ nice idea, forgot we could do this. I'm going to let this fail in the gate and then clean things up afterwards.13:17
* lyarwood local machine is busy with some other tasks at the moment13:17
gibilyarwood: cool, thanks13:17
*** whoami-rajat___ has joined #openstack-nova13:18
openstackgerritMerged openstack/placement master: Bump oslo.log version to 4.3.0  https://review.opendev.org/c/openstack/placement/+/76022913:36
*** sapd1 has joined #openstack-nova13:46
*** zenkuro has quit IRC13:51
lyarwoodFailed: 249 with the Architecture.from_host poison in place /o\13:52
lyarwoodI'll clean that up next week13:52
*** zenkuro has joined #openstack-nova13:52
gibilyarwood: ack13:55
*** _mlavalle_1 has quit IRC13:58
*** mlavalle has joined #openstack-nova13:58
*** zzzeek has quit IRC13:59
*** dtantsur is now known as dtantsur|brb14:00
*** zzzeek has joined #openstack-nova14:01
*** jawad_axd has joined #openstack-nova14:03
gibisean-k-mooney: if you are around, smartnic discussion happening on openstack-meeting (neutron ddrivers)14:05
*** spatel has joined #openstack-nova14:07
stephenfinartom: When you're about, can you say if you're happy with my response at https://review.opendev.org/c/openstack/nova/+/75815514:09
stephenfinAlso, if you've bandwidth, reviews will be appreciated. I'd love dansmith to look at them but I don't know if he'll have the time/inclination so I need to find people != gibi :)14:10
* gibi likes these type of pings :D14:10
jawad_axdHi nova folks, I have done passthrough nvidia k20c with window 10 instance. Installed latest drivers from nvidia site. I can see " This device is working properly" in device manager.Apparently, gpu is not being used for any application. I can see only developer task in nvidia control panel.No 3D settings where I can set to use GPU for windows. Why 3D settings are missing in nvidia control panel? Is it14:17
jawad_axdsomething with nvidia driver or windows or openstack..Anybody had same problem or any suggestions? Thanks14:17
dansmithstephenfin: oof, yeah that probably needs several sets of eyes. I won't get to it today, but maybe next week14:24
stephenfinAgreed. Any time in the near future is good14:25
artomstephenfin, I suppose if every reviewer runs your tool manually it's as good as CI14:36
artomWe'd also need to make sure your tool has no bugs, but then that's the same as with a CI job14:36
*** rpittau is now known as rpittau|afk14:46
bauzasjawad_axd: is the PCI device provided in the instance ?14:53
jawad_axd@bauzas Yes14:53
bauzasso, it's either a windows or a nivdia issue, sorry14:54
sean-k-mooneyjawad_axd: i assume if you look in the device manager the device is there with the nvida drivae but error 4314:54
sean-k-mooneyjawad_axd: it looks like nvidia has updated there driver detection logic recently and broken our ablity to do pci passhtouhg again14:54
sean-k-mooneybauzas: i did pci passhtough of my gtx-780 last week and found our hypervior hiding code nolonger is sufficent14:55
sean-k-mooneyerror 43 is back14:55
jawad_axd@sean-k-mooney There is no error 43. Device manager shows device working properly.14:55
bauzaswhat ?14:55
jawad_axdI had error 43 with genforce GTX 780. But with this Tesla gpu, driver installed successfully.14:56
sean-k-mooneyjawad_axd: ok then in that case have you disabel the video device so its the only gpu14:56
sean-k-mooneyjawad_axd: ya the tesla driver stack allows virutalistation14:56
sean-k-mooneyit does not have the same test14:57
sean-k-mooney*check14:57
sean-k-mooneyjawad_axd: you could add hw_video_model=none to the image as a test14:57
sean-k-mooneyjawad_axd: that would disable the software gpu emulation in qemu so the only gpu would be the nvidia one14:57
jawad_axd@sean-k-mooney Thanks. I try this hw_video_model=none .14:58
sean-k-mooneybefore you do if you install gpuz in the vm does it detect support for cuda/driectx ectra14:58
jawad_axd@sean-k-mooney gpuz detects gpu.14:59
sean-k-mooneyjawad_axd: and it has all the apis available?14:59
sean-k-mooneyif so i would guess the other application are just selecting the first gpu which will be the cirros/virtio/qxl one form qemu14:59
sean-k-mooneyjawad_axd: tensorflow used to do this for example which is why hw_video_model=none was added15:00
jawad_axd@sean-k-mooney hw_video_model=none  would stop using qxl/virtio, right ?15:00
sean-k-mooneyyep15:01
sean-k-mooneyso the only gpu the guest will see it the pci passthough one15:01
jawad_axd@sean-k-mooney What do you mean by apis in gpuz?15:07
sean-k-mooneygpuz has a bunch of checkboxs on the overview page if i remmeber15:08
sean-k-mooneyfor cuda directx opengl15:08
sean-k-mooneyand some others15:08
sean-k-mooneyjust wanted to confim the expected api were avaiable15:08
sean-k-mooneyand that it was usign the nvida driver not the generic vga windows direver15:08
jawad_axdIt has OpenCL, CUDA and OpenGL1.1 checked.15:09
sean-k-mooneyyep so the nvidia direr is really working then15:09
sean-k-mooneyif they were not check then it might be using the fallback driver instead of the nvidia one15:10
sean-k-mooneyso i would guess its just a gpu selection issue cause by both of them being there15:10
jawad_axdAlright. I try hw_video_model=none . Will get back if it works :)15:11
*** nweinber has joined #openstack-nova15:14
openstackgerritBalazs Gibizer proposed openstack/nova master: Refactor ResourceRequest constructor  https://review.opendev.org/c/openstack/nova/+/76972015:25
openstackgerritBalazs Gibizer proposed openstack/nova master: Enable mypy on scheduler/utils.py  https://review.opendev.org/c/openstack/nova/+/76994215:25
openstackgerritBalazs Gibizer proposed openstack/nova master: Support interface attach with QoS ports  https://review.opendev.org/c/openstack/nova/+/75653015:27
*** zzzeek has quit IRC15:38
*** nweinber has quit IRC15:39
*** zzzeek has joined #openstack-nova15:41
*** mkrai has quit IRC15:41
dansmithsean-k-mooney: do you know of any instances where we use the raw keystoneauth stuff to just make a quick call to another service, but with all the request info we got (i.e user token etc)?15:41
dansmithlooking for something to copy15:41
sean-k-mooneymaybe in the ironic code15:43
dansmithokay I think we used to talk to placement like that didn't we?15:43
sean-k-mooneymaybe but i think palcment now uses the sdk15:43
dansmithalthough not with the user's token I guess15:43
dansmithyeah I know it does now15:43
*** dtantsur|brb is now known as dtantsur15:45
sean-k-mooneyhttps://github.com/openstack/nova/blob/master/nova/virt/ironic/client_wrapper.py#L66-L7315:45
dansmithironic lazy-imports ironic client, but it's also not using the user's token right?15:45
sean-k-mooneyya that im not sure about it look like that is using the config options15:45
*** macz_ has joined #openstack-nova15:46
*** macz_ has quit IRC15:46
sean-k-mooneydansmith: i guess for ironic and placment we just use admin tokens since they are normally admin only apis right15:47
*** macz_ has joined #openstack-nova15:47
sean-k-mooneyso we just use the configs15:47
dansmithright15:47
sean-k-mooneydo you need ksa or could you just use the sdk15:47
dansmithwell, I'm trying to PoC something where glance calls to itself,15:48
*** spatel has quit IRC15:48
sean-k-mooneytoo answer you question though i dont know off the top of my head15:48
dansmithso I could just import glanceclient, but I really just need to replay an http call so it seems a little heavy15:48
dansmithack15:49
sean-k-mooneyah ok15:49
*** dklyle has joined #openstack-nova15:49
openstackgerritElod Illes proposed openstack/nova stable/stein: [stable-only] Fix lower-constraints job  https://review.opendev.org/c/openstack/nova/+/76980915:53
stephenfindansmith: https://review.opendev.org/c/openstack/nova/+/706295/7/nova/network/neutron.py15:55
stephenfinIf I understood you correctly, I think that's what we do for port bindings in neutron15:55
dansmithstephenfin: ah, yeah that might be it, thanks15:57
dansmithI figured even though neutron was the right pattern, that everything would be through the client and not helpful,15:57
dansmithbut that chunk is like "in case of no client support, break glass" :)15:57
jawad_axd@sean-k-mooney Apparently , hw_video_model doest work with none' anymore? I am getting invalid metadata.. I guess valid options are just graphics adapter model with this property..16:05
openstackgerritBalazs Gibizer proposed openstack/nova master: Enable mypy on scheduler/report.py  https://review.opendev.org/c/openstack/nova/+/76515216:06
openstackgerritBalazs Gibizer proposed openstack/nova master: Remove unused Instance method  https://review.opendev.org/c/openstack/nova/+/75761516:06
gibistephenfin: I've resolved you comments in the qos interace attach series16:06
stephenfingibi: ack, I'll take a look straight after this meeting16:06
gibicool, thanks16:07
*** zzzeek has quit IRC16:10
*** zzzeek has joined #openstack-nova16:12
*** ociuhandu_ has joined #openstack-nova16:28
*** ircuser-1 has quit IRC16:29
*** ociuhandu has quit IRC16:31
*** ociuhandu_ has quit IRC16:33
*** belmoreira has quit IRC16:35
*** zzzeek has quit IRC16:35
gibistephenfin: went through once again the db migration series, all my questions are answered or fixed so I'm positive all along the series16:36
stephenfingreat, thank you16:37
gibigood job!16:37
*** zzzeek has joined #openstack-nova16:37
sean-k-mooneyjawad_axd: what release are you on?16:37
sean-k-mooneyjawad_axd:https://github.com/openstack/nova/blob/46899968619e4ea0ff2ab380977619bb29578d43/nova/objects/fields.py#L562-L57316:38
*** ociuhandu has joined #openstack-nova16:38
sean-k-mooneyjawad_axd: i added none in train https://github.com/openstack/nova/commit/35a591d33d8b1a6c30bf40ddc48a07715fd8733916:39
jawad_axdI am using stein16:39
*** whoami-rajat___ is now known as whoami-rajat__16:39
sean-k-mooneyah you one release too old16:39
sean-k-mooneyin your case i think your only really choice other then upgrading is to force the applciations to use the gpu via there config or runtime options16:40
sean-k-mooneyjawad_axd: the image properties are defiend in oslo versions objects so there is no way to backport support for none with out upgrade issues16:41
sean-k-mooneyform a nova poing of view the phsyical gpu passthough is working correctly but the applciaiton is just not selceting the correct gpu of the two avaiable16:42
*** ociuhandu has quit IRC16:44
jawad_axdAlright..16:45
sean-k-mooneysorry cant help more but once nova has succfully booted the vm and made the gpu avaiable to it its jobs is basically done16:46
jawad_axd@sean-k-mooney Atleast I got to know, passthrough is working fine. Now I need to figure out how to use it as default. Thanks16:56
*** nweinber has joined #openstack-nova16:57
sean-k-mooneydepending on your application it will differ16:57
*** gyee has joined #openstack-nova16:58
sean-k-mooneygames/rendering applcaition like blender usally have an option you can set in there menu or config files16:58
sean-k-mooneywith tensorflow for example you can do https://www.tensorflow.org/api_docs/python/tf/config/set_visible_devices16:58
*** tbachman has quit IRC17:01
sean-k-mooneyhuh did people know you can click on the files and expand them in the top review windows17:16
sean-k-mooneyhttps://review.opendev.org/c/openstack/python-openstackclient/+/76239617:16
sean-k-mooneyso if you click on any of the boxes beside the file links not on the links it will render the files in a dropdown17:17
sean-k-mooneythat is what thos little arrows on earch row on the right are signaling17:18
*** tesseract has quit IRC17:18
sean-k-mooneystephenfin: im 50:50 on that reviw by the way. it looks liek ti will work correctly but lee basically made the same point about doing it in parralel in as i did in my first review17:20
sean-k-mooneystephenfin: i think we should do that optimisation but im kind of with gibi that it could be a follow up patch17:21
sean-k-mooneyi know you are also looking a provideing a generic way to do the waits in https://review.opendev.org/c/openstack/osc-lib/+/76250317:22
sean-k-mooneyso i can see why you would prefer not to do it direcly in https://review.opendev.org/c/openstack/python-openstackclient/+/762396 and wait to reuse that17:23
*** nweinber has quit IRC17:42
sean-k-mooneymelwitt: just responed to your questions in https://review.opendev.org/c/openstack/nova/+/769614 i proably wont respin that today but let me know if you still have questions17:43
*** jangutter_ has joined #openstack-nova17:43
sean-k-mooneymelwitt: how urgent is the downstream issue by the way. this is not a escalation or anything is it17:44
sean-k-mooneye.g. it can wait until next week while i update a spec17:45
*** jangutter has quit IRC17:46
*** tbachman has joined #openstack-nova17:54
*** andrewbonney has quit IRC18:01
*** jawad_axd has quit IRC18:05
sean-k-mooneyartom: responded to your questions in https://review.opendev.org/c/openstack/nova-specs/+/76499918:16
*** xinranwang has quit IRC18:16
*** ralonsoh has quit IRC18:17
dansmithstephenfin: that wasn't exactly what I wanted, but definitely led me down the path to success, so thanks18:20
stephenfingood stuff18:25
gmannstephenfin: thanks for update, +2 on https://review.opendev.org/c/openstack/nova/+/764040/818:30
gmanngibi: ^^18:30
gmannstephenfin: I added comment for policy updates also in case you missed it - https://review.opendev.org/c/openstack/nova/+/765798/318:31
*** itsjg has quit IRC18:32
*** tbachman has quit IRC18:36
stephenfingmann: ack, will take a look first thing monday. Thanks!18:37
gmannsure. just pinged in case it was missed.  have a happy weekend!18:38
*** dtantsur is now known as dtantsur|afk18:43
sean-k-mooneygmann: when you have time can you confirm i am uderstanding what we are doing for cyborg correctly and that this is fine for vdpa too https://review.opendev.org/c/openstack/nova-specs/+/764999/2/specs/wallaby/approved/libvirt-vdpa-support.rst#4118:47
sean-k-mooneyyou welcom to review the whole spec obviously but this spcificly would be good for you to weigh in on18:47
gmannsean-k-mooney: sure18:47
sean-k-mooneybasiclaly blocking live migation with vdpa using a 403 until we support it to avoid a micorverion bump on the api side18:48
*** ociuhandu has joined #openstack-nova18:48
sean-k-mooneywhen ever qemu actully add suport we would still need to check the compute capabliteis and may fail if both the source and destionation nodes are not updated18:49
sean-k-mooneyso even if we had a microversion saying nova supports this we would no know if your qemu supported it so i dont think a microverion is the way to go since there are no api changes involved18:50
artomsean-k-mooney, yep, got the email. I'll be honest, understand your changes and replying will go to next week :)18:53
*** ociuhandu has quit IRC18:53
artom*understanding18:53
*** tbachman has joined #openstack-nova18:57
*** whoami-rajat__ has quit IRC18:58
*** hamalq has joined #openstack-nova19:10
*** nweinber has joined #openstack-nova19:19
*** hamalq has quit IRC19:33
*** hamalq has joined #openstack-nova19:33
*** jawad_axd has joined #openstack-nova19:38
*** itsjg has joined #openstack-nova20:17
sean-k-mooneyartom: no worries20:21
melwittsean-k-mooney: thanks! it's not an escalation and I hope it doesn't become one21:04
*** slaweq has quit IRC21:13
*** derekh has quit IRC21:19
*** jawad_axd has quit IRC21:20
*** martinkennelly has quit IRC21:37
*** nweinber has quit IRC21:37
openstackgerritGhanshyam proposed openstack/nova master: Fix ForbiddenWithAccelerators to HTTPForbidden for shelve API  https://review.opendev.org/c/openstack/nova/+/77000722:55
*** artom has quit IRC23:07
*** spatel has joined #openstack-nova23:52
*** spatel has quit IRC23:56

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