Wednesday, 2021-02-24

« Tuesday, 2021-02-23 Index Thursday, 2021-02-25 »
*** tosky has quit IRC00:10
*** brinzhang has joined #openstack-nova00:40
openstackgerritBrin Zhang proposed openstack/nova master: Nova supports password encrypted VNC  https://review.opendev.org/c/openstack/nova/+/62233600:40
*** LinPeiWen has joined #openstack-nova00:40
brinzhanggibi, stephenfin: as current implementation of noVNC that we can support open console with password, but not used x509+vnc, we would like to optimize it later, can we let this feature go?00:44
openstackgerritBrin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Update Servers APIs  https://review.opendev.org/c/openstack/nova/+/76429200:46
openstackgerritBrin Zhang proposed openstack/nova master: Replace all_tenants with all_projects in List Server APIs  https://review.opendev.org/c/openstack/nova/+/76531100:47
openstackgerritBrin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Rebuild Server API  https://review.opendev.org/c/openstack/nova/+/76638000:48
openstackgerritBrin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List SG API  https://review.opendev.org/c/openstack/nova/+/76672600:48
openstackgerritBrin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Flavor Access APIs  https://review.opendev.org/c/openstack/nova/+/76770400:49
openstackgerritBrin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Show usage APIs  https://review.opendev.org/c/openstack/nova/+/76850900:50
openstackgerritBrin Zhang proposed openstack/nova master: Replace tenants* with projects* of policies  https://review.opendev.org/c/openstack/nova/+/76531500:50
*** mgariepy has quit IRC00:51
openstackgerritBrin Zhang proposed openstack/nova master: Replace os-simple-tenant-usage with os-simple-project-usage  https://review.opendev.org/c/openstack/nova/+/76885200:51
openstackgerritBrin Zhang proposed openstack/nova master: Replace tenant_id with project_id in os-quota-sets path  https://review.opendev.org/c/openstack/nova/+/76885100:52
openstackgerritBrin Zhang proposed openstack/nova master: Replace tenant_id with project_id in Limits API  https://review.opendev.org/c/openstack/nova/+/76886200:52
*** mgariepy has joined #openstack-nova00:52
openstackgerritBrin Zhang proposed openstack/nova master: Replace tenant* with project* in codes  https://review.opendev.org/c/openstack/nova/+/76932900:52
*** mlavalle has quit IRC00:57
*** tbachman has quit IRC01:02
*** tbachman_ has joined #openstack-nova01:02
*** martinkennelly has quit IRC01:11
*** brinzhang has quit IRC01:13
*** brinzhang has joined #openstack-nova01:15
*** zzzeek has quit IRC01:15
*** brinzhang has quit IRC01:16
*** zzzeek has joined #openstack-nova01:17
*** __ministry has joined #openstack-nova01:27
*** spatel has joined #openstack-nova01:29
*** cz3 is now known as Halabala01:34
*** Halabala is now known as cz301:34
*** hamalq has quit IRC01:53
*** hamalq has joined #openstack-nova01:54
*** macz_ has quit IRC02:04
*** hamalq has quit IRC02:08
*** zul has quit IRC02:08
*** rcernin has quit IRC02:37
*** macz_ has joined #openstack-nova03:05
*** macz_ has quit IRC03:10
*** brinzhang has joined #openstack-nova03:17
*** mkrai has joined #openstack-nova03:22
*** macz_ has joined #openstack-nova03:26
*** macz_ has quit IRC03:31
*** macz_ has joined #openstack-nova04:06
*** macz_ has quit IRC04:10
prometheanfiresean-k-mooney: let me know if https://review.opendev.org/777295 is needed when / if you get a chance  (for reference https://review.opendev.org/777170 is the test for 0.16)04:18
*** macz_ has joined #openstack-nova04:27
*** macz_ has quit IRC04:32
*** rcernin has joined #openstack-nova04:44
*** macz_ has joined #openstack-nova04:48
*** macz_ has quit IRC04:52
*** rcernin has quit IRC04:54
*** rcernin has joined #openstack-nova04:55
*** ratailor has joined #openstack-nova05:01
*** macz_ has joined #openstack-nova05:09
*** macz_ has quit IRC05:13
*** rcernin has quit IRC05:14
*** rcernin has joined #openstack-nova05:21
*** gyee has quit IRC05:34
*** dviroel has quit IRC05:39
*** whoami-rajat has joined #openstack-nova05:41
*** spatel has quit IRC05:42
*** ricolin has quit IRC06:00
*** zoharm has joined #openstack-nova06:12
*** LinPeiWen has quit IRC06:16
*** zenkuro has joined #openstack-nova06:20
*** LinPeiWen79 has joined #openstack-nova06:24
*** ricolin has joined #openstack-nova06:28
*** mkrai has quit IRC06:47
*** macz_ has joined #openstack-nova06:47
*** macz_ has quit IRC06:52
*** LinPeiWen79 has quit IRC06:58
*** LinPeiWen has joined #openstack-nova07:05
*** zenkuro has quit IRC07:08
*** zenkuro has joined #openstack-nova07:08
*** __ministry has quit IRC07:10
*** __ministry has joined #openstack-nova07:15
*** khomesh24 has joined #openstack-nova07:16
*** dklyle has quit IRC07:16
*** khomesh24 has quit IRC07:16
*** khomesh24 has joined #openstack-nova07:18
*** xek has joined #openstack-nova07:24
*** slaweq has joined #openstack-nova07:27
*** xek has quit IRC07:30
*** xek has joined #openstack-nova07:34
*** rcernin has quit IRC07:39
*** ralonsoh has joined #openstack-nova07:48
*** luksky has joined #openstack-nova07:54
*** belmoreira has joined #openstack-nova07:55
*** rpittau|afk is now known as rpittau08:11
*** LinPeiWen has quit IRC08:13
*** rcernin has joined #openstack-nova08:15
*** ociuhandu has joined #openstack-nova08:20
*** tesseract has joined #openstack-nova08:22
*** andrewbonney has joined #openstack-nova08:27
*** ociuhandu has quit IRC08:31
*** rcernin has quit IRC08:31
*** rcernin has joined #openstack-nova08:33
*** ociuhandu has joined #openstack-nova08:36
jkulikfound the answer to my question regarding sync_aggregates: ironic host are not supported with aggregates.08:39
jkulikhow does AZ assignment work there, if they don't support aggregates?08:40
*** macz_ has joined #openstack-nova08:44
*** macz_ has quit IRC08:49
openstackgerritBalazs Gibizer proposed openstack/nova master: Enable mypy on scheduler/report.py  https://review.opendev.org/c/openstack/nova/+/76515208:49
*** martinkennelly has joined #openstack-nova08:51
*** martinkennelly has quit IRC08:52
openstackgerritBalazs Gibizer proposed openstack/nova master: Follow up type hints for a634103  https://review.opendev.org/c/openstack/nova/+/77715909:00
*** lucasagomes has joined #openstack-nova09:07
*** rcernin has quit IRC09:08
*** ociuhandu has quit IRC09:09
*** hoonetorg has joined #openstack-nova09:16
*** tosky has joined #openstack-nova09:18
*** LinPeiWen has joined #openstack-nova09:21
bauzasjkulik: the problem with aggregates is that only service hostnames are used09:21
bauzasso, if you use the same nova-compute service for all the ironic nodes, then you'll have all the nova compute nodes having the same "host" name09:22
bauzasbut...09:22
bauzasyou can have placement aggregates09:22
bauzasusing compute node UUIDs09:22
bauzasso, you could be able to provide flavors for each of them09:23
bauzasbut not AZs09:23
jkulikbauzas: we have different racks of baremetal nodes assigned to different hostnames in nova. they are assigned to aggregates for the AZs in our case. that seemed to work09:25
bauzascool then09:25
jkulikwell yes and no ;)09:26
jkulikwe cannot sync those aggregates automatically to placement, because Nova doesn't support the case.09:26
jkulikbut it looks like that's just a matter of patching it (badly) to allow multiple resource providers/compute nodes for ironic09:26
bauzaswe discussed this a bit in the past but providing nova aggregates support for ironic wasn't pursued09:35
bauzasas there are a ton of implications and some breaking changes09:36
bauzasand then we moved to placement09:36
bauzasthe idea here is maybe to say we could use placement queries with some kind of prefilter that would be smarter09:36
*** derekh has joined #openstack-nova09:36
jkulikbecause the admin can target specific ironic nodes, I guess? makes sense.09:38
jkulikthank you for helping me get a clearer picture on that09:38
*** ociuhandu has joined #openstack-nova09:40
gibilyarwood: I'm OK going with the python based solution in the machine_type series. We can optimize it later if we get reports about slowness09:46
lyarwoodgibi: ack, I'm going to respin the series later today, apologies for not getting to it just yet09:47
lyarwoodFWIW I'm about to admit defeat with the ephemeral encryption series09:48
lyarwoodthere's just too much debt in the libvirt driver to land this in 2 weeks09:48
gibino problem, I also was slow responding09:48
lyarwoodand I've under estimated the required test coverage given the removal of some of that debt in this series09:48
gibilyarwood: ack, thanks for being upfront about that09:48
* lyarwood has qcow2 LUKS working but it's pretty awful09:49
*** ociuhandu has quit IRC09:49
*** ociuhandu has joined #openstack-nova09:52
*** ociuhandu has quit IRC09:53
*** ociuhandu has joined #openstack-nova09:53
*** LinPeiWen has quit IRC09:57
*** ociuhandu has quit IRC09:58
*** ociuhandu has joined #openstack-nova10:00
*** ociuhandu has quit IRC10:00
*** ociuhandu has joined #openstack-nova10:01
*** ociuhandu has quit IRC10:03
*** ociuhandu has joined #openstack-nova10:03
*** brinzhang has quit IRC10:04
*** brinzhang has joined #openstack-nova10:05
*** songwenping_ has quit IRC10:05
*** songwenping_ has joined #openstack-nova10:05
*** dtantsur|afk is now known as dtantsur10:07
*** macz_ has joined #openstack-nova10:07
*** macz_ has quit IRC10:12
stephenfingibi: artom is on PTO so I'm going to address my own nits on https://review.opendev.org/c/openstack/nova/+/772748 to keep it moving10:14
stephenfinis that okay?10:15
stephenfinI can also fix up the next one since you agree with my comments10:15
*** Fl1nt has joined #openstack-nova10:25
Fl1ntHi there!10:25
*** rcernin has joined #openstack-nova10:27
Fl1ntQuick question, let say I want to implement garbage collection/housekeeping and a mechanism to auto-balance my cluster when I had new nodes to a zone/cell/aggregate/whatever, is there already WIP or SIG working on this for nova? If not, would it be better to create a new project that work on nova/placement ressources or is it prefered to create it as a nova/placement feature?10:29
gibistephenfin: sure, it works for me10:34
gibiFl1nt: I'm not sure what is the status of https://wiki.openstack.org/wiki/Watcher but I think it has a scope you are looking for10:35
Fl1ntgibi, perfect, if it already exist then I'll watch (no pun intended) it and contribut to it if it's not a living dead project ^^10:37
gibi:)10:37
*** jangutter_ has joined #openstack-nova10:40
*** jangutte_ has joined #openstack-nova10:41
*** jangutter has quit IRC10:41
*** rcernin has quit IRC10:42
*** lemko has quit IRC10:44
*** jangutter_ has quit IRC10:44
*** k_mouza has joined #openstack-nova10:48
*** lemko has joined #openstack-nova10:48
gibibrinzhang: I need stephenfin and melwitt to agree on the vnc password patch10:49
*** lemko has quit IRC10:50
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add 'socket' PCI NUMA affinity  https://review.opendev.org/c/openstack/nova/+/77274810:51
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Add 'socket' PCI NUMA affinity policy trait  https://review.opendev.org/c/openstack/nova/+/77274910:51
openstackgerritStephen Finucane proposed openstack/nova master: Track host NUMA topology in PCI manager  https://review.opendev.org/c/openstack/nova/+/77414910:51
openstackgerritStephen Finucane proposed openstack/nova master: pci: implement the 'socket' NUMA affinity policy  https://review.opendev.org/c/openstack/nova/+/77277910:51
Fl1ntthanks gibi that's absolutely what I was looking for!10:51
Fl1ntthanks and cheers!10:51
gibiFl1nt: I'm happy that I could help10:52
*** rcernin has joined #openstack-nova10:52
belmoreiraone line patch review: https://review.opendev.org/c/openstack/nova/+/776428 Let me know if this makes sense. Created a lot of confusion some time ago :)10:53
*** Fl1nt has left #openstack-nova10:54
gibibelmoreira: make sense to me10:54
*** LinPeiWen has joined #openstack-nova10:54
*** dviroel has joined #openstack-nova10:55
*** k_mouza has quit IRC10:55
*** k_mouza has joined #openstack-nova10:56
*** k_mouza_ has joined #openstack-nova11:03
*** lamt has quit IRC11:05
*** k_mouza has quit IRC11:06
*** rcernin has quit IRC11:12
*** macz_ has joined #openstack-nova11:13
*** macz_ has quit IRC11:18
*** ociuhandu has quit IRC11:19
*** ociuhandu has joined #openstack-nova11:20
*** ociuhandu has quit IRC11:25
*** ociuhandu has joined #openstack-nova11:26
*** ociuhandu has quit IRC11:30
*** ociuhandu has joined #openstack-nova11:34
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Add tests for insecure UEFI configuration  https://review.opendev.org/c/openstack/nova/+/77736711:39
stephenfinbauzas: ^11:39
bauzasta11:39
bauzaswill look11:39
*** ociuhandu has quit IRC11:41
*** k_mouza_ has quit IRC11:45
*** k_mouza has joined #openstack-nova11:46
gibisean-k-mooney: could you take a look on https://review.opendev.org/c/openstack/nova/+/750552 we have +2s from cores, but you had issues with the ipVersion vs ip_version in the spec. The impl goes with ipVersion. Is it a dealbreakre for you?11:54
sean-k-mooneyno its not11:55
sean-k-mooneyi prefer ip_version11:55
sean-k-mooneybut both work11:55
gibithanks, then I will approve the impl with ipVersion as there was no real preference in the existing attribute names for either ways11:56
sean-k-mooneywe coudl fix it with a follow up to the spec or code to bring them in line11:56
gibias far as I understand there is no definit line11:56
gibihttps://review.opendev.org/c/openstack/nova/+/750552/10/nova/virt/libvirt/config.py#345911:56
sean-k-mooneyi dont think we currently have any 2 word elemnts11:56
sean-k-mooneyat least not nova ones11:57
gibino, not nova ones11:57
gibibut in general11:57
sean-k-mooneywell libvirt uses _11:57
sean-k-mooney <on_reboot>restart</on_reboot>11:58
*** ociuhandu has joined #openstack-nova11:58
sean-k-mooneybut its a minor nit11:58
sean-k-mooneynot a deal breaker i would just prefer to use _ when i extend it with the flavor info next cycle since i never got to it this cycle11:59
sean-k-mooneyacutlly it uses both11:59
sean-k-mooney<currentMemory unit='KiB'>1048576</currentMemory>12:00
sean-k-mooneyso both are fine lets leave it as is12:00
sean-k-mooneyill comment the same in the bug12:00
sean-k-mooney*patch12:00
gibithanks12:02
sean-k-mooneydone12:04
sean-k-mooneyun related to that change but when we use https://www.openstack.org/xmlns/libvirt/nova/1.1 as a namespace url that is actully ment to be a vaild path to the xml schema12:06
sean-k-mooneyi dont know if we care to fix that at some point since we dont actully have a schema for the metadta xml elemnet just said i would note that in passing12:06
*** ratailor has quit IRC12:07
*** k_mouza has quit IRC12:08
gibigood point maybe we can fix that next cycle when we extend the schema further12:11
sean-k-mooneyya not sure how we would fix it unless we put it in the docs and linked to the doc site or to the git repo on opendev.org12:13
sean-k-mooneyxml parsers can parse without a schema file they just cant validate it12:13
sean-k-mooneywell beyond syntax12:13
sean-k-mooneybut this is an internal api fro dubuging any way so its only ment to be read by humans not software12:14
toskyhi nova people! It looks like the devstack-plugin-nfs tempest jobs are failing on ussuri only (see the last jobs on https://review.opendev.org/q/project:openstack/devstack-plugin-nfs+status:open )12:14
toskywhoami-rajat commented on one of the failures, it seems there is something weird on the nova side (please see the last comment here: https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/763516/ )12:15
toskyare you aware of any NFS-related fix which hasn't been backported yet to ussuri?12:15
toskyor at least can you please take a look?12:15
sean-k-mooneyi think lyarwood  was tryign to get that to work again12:15
lyarwoodit's on my TODO list12:15
sean-k-mooneytosky: we know of nfs related bugs12:16
lyarwoodtosky: nothing specific to ussuri btw12:16
toskyright, so this may be another issue?12:16
lyarwoodtosky: yeah there are several with this codepath12:16
lyarwoodtosky: it's unique to NFS volumes12:16
sean-k-mooneymaybe the nfs backend is one of the most problematic we have support for currently12:16
lyarwoodtosky: and not maintained12:16
toskybecause really -ussuri seems to be consistently failing in the last builds12:16
lyarwoodkk well I can't think of anything that has landed since that could cause this tbh12:17
lyarwoodcould someone write up a bug and I'll take a look early next week?12:17
toskysure12:17
lyarwoodthanks12:17
sean-k-mooneylooks like its the volumn snapshot tests that are failing12:17
sean-k-mooneythere are issue with snapshots and the nfs backend right12:18
*** __ministry has quit IRC12:18
sean-k-mooneyit looks like the nfs backend does not supprot snapshots of inuse volumen based on the error12:21
*** k_mouza has joined #openstack-nova12:21
lyarwoodnew test?12:22
sean-k-mooneyhttps://zuul.opendev.org/t/openstack/build/e9e69b9af21c414a83c0437894826178/log/controller/logs/screen-n-cpu.txt#4808212:23
sean-k-mooney Unknown base file: /opt/stack/data/nova/instances/_base/c4c416ed67a8e3a5f1de44029e480806bb6a55f912:23
whoami-rajatsean-k-mooney: nfs driver supports it via blockrebase and blockcommit which is done by nova, but not sure exactly what you mean12:23
*** ociuhandu has quit IRC12:24
sean-k-mooneythere is a libviet error with ehcomand12:24
sean-k-mooneylibvirt.libvirtError: internal error: qemu block name 'json:{"backing": {"driver": "raw", "file": {"driver": "file", "filename": "/opt/stack/data/nova/mnt/896fb15da6036b68a917322e72ebfe57/volume-3c98d95e-a839-4eaf-ba76-325ed3a3a946"}}, "driver": "qcow2", "file": {"driver": "file", "filename":12:24
sean-k-mooney"/opt/stack/data/nova/mnt/896fb15da6036b68a917322e72ebfe57/volume-3c98d95e-a839-4eaf-ba76-325ed3a3a946.adb8bb67-dfb5-4ac7-8e95-3643f8a0e8ed"}}' doesn't match expected '/opt/stack/data/nova/mnt/896fb15da6036b68a917322e72ebfe57/volume-3c98d95e-a839-4eaf-ba76-325ed3a3a946.adb8bb67-dfb5-4ac7-8e95-3643f8a0e8ed'12:24
*** ociuhandu has joined #openstack-nova12:24
sean-k-mooneyso the block commit is failing but i think that path comes form cinder? not sure i know this api is slightly weird.12:27
sean-k-mooneyits the snapshot delete that looks like its failing12:29
sean-k-mooneyhttps://zuul.opendev.org/t/openstack/build/e9e69b9af21c414a83c0437894826178/log/controller/logs/screen-n-cpu.txt#4812512:29
*** ociuhandu has quit IRC12:29
sean-k-mooney: {'file_to_merge': 'volume-3c98d95e-a839-4eaf-ba76-325ed3a3a946.adb8bb67-dfb5-4ac7-8e95-3643f8a0e8ed', 'merge_target_file': 'volume-3c98d95e-a839-4eaf-ba76-325ed3a3a946', 'type': 'qcow2', 'volume_id': '3c98d95e-a839-4eaf-ba76-325ed3a3a946'}12:30
sean-k-mooneyi think file_to_merge and merge_target_file come form cinder12:30
sean-k-mooneytosky: you could poteanally skip that on ussuri temporally i guess if its fine on other brances12:35
sean-k-mooneyit might be due to the libvirt version we are using12:35
sean-k-mooneybut this does look like there is a real bug12:35
openstackgerritMerged openstack/nova master: libvirt: start tracking NUMACell.socket for hosts  https://review.opendev.org/c/openstack/nova/+/76681612:41
whoami-rajatlyarwood: sean-k-mooney  https://bugs.launchpad.net/nova/+bug/191675012:45
openstackLaunchpad bug 1916750 in OpenStack Compute (nova) "[gate] devstack-plugin-nfs-tempest-full-ussuri job failing" [Undecided,New]12:45
sean-k-mooneywhoami-rajat: thanks12:45
*** vishalmanchanda has joined #openstack-nova12:46
*** martinkennelly has joined #openstack-nova12:58
*** tobias-urdin has joined #openstack-nova13:00
*** mgariepy has quit IRC13:00
*** macz_ has joined #openstack-nova13:01
toskywhoami-rajat: thanks13:01
*** macz_ has quit IRC13:06
gibistephenfin, bauzas: I'm +2 through the whole api db compaction series.13:07
whoami-rajatnp13:10
*** LinPeiWen has quit IRC13:17
stephenfingibi: \o/13:17
stephenfinLet me know if I can return the favour. I think I'm on top of most of your work13:17
gibistephenfin: yes, my only remaning sizeable thing is the device detach series but there I have to do fixes in the last patch13:18
sean-k-mooneystephenfin: not sure if you saw the sqlalchemy migrate thread on the mailing list13:18
gibifortunatley I'm running out of code to write as the FF closes in, so I can focus on review13:18
sean-k-mooneystephenfin: is you current db compation work just the comparction13:18
sean-k-mooneyor will you have swapped nova to almenbic this cycle13:18
gibijust compaction13:18
sean-k-mooneyok13:18
sean-k-mooneysince matt left no one has maintained it13:18
* sean-k-mooney has last commit...13:19
sean-k-mooneygibi: so its currenly unmaintained and we need to adress that untill nova nolonger uses it13:19
stephenfindansmith: If you have time this week, your attention would still be appreciated on the API DB compaction series. My testing methodology was identical to the main DB13:19
gibisean-k-mooney: I think stephenfin could propose to move to alembic in the next cycle ;)13:19
sean-k-mooneyyep13:20
stephenfinsean-k-mooney: I have an alembic migration series locally, but I'm not going to push it until after M313:20
stephenfinWe have enough work on our plates :)13:20
sean-k-mooneyhttp://lists.openstack.org/pipermail/openstack-discuss/2021-February/020666.html this is the thread13:20
sean-k-mooneystephenfin: i think the issue is oslo db deprectaed support for it13:20
stephenfinI did that13:20
sean-k-mooneyya so some are concerned that its un maintined and that its not deprecated in oslo13:21
sean-k-mooneybut we still have multipel project using it13:21
sean-k-mooneyas long as we dont remove oslo supprot until they are are all moved then i think its ok13:21
sean-k-mooneymaybe you could respond to the thread and clarify that13:21
stephenfinyup, replying now. Thanks for bringing it to my attention13:22
sean-k-mooneyif there are actully bugs with sqlam i can maybe try and fix some of them. im not really that famialar with it but i can take a look if needed.13:24
*** mgariepy has joined #openstack-nova13:28
*** k_mouza has quit IRC13:31
*** k_mouza has joined #openstack-nova13:32
*** pmannidi has quit IRC13:34
*** dtantsur is now known as dtantsur|brb13:35
*** pmannidi has joined #openstack-nova13:37
bauzasgibi: woah, you're quick13:37
bauzasit takes me a while for reviewing a single DB change13:37
gibibauzas: it is a lot simpler than the db compaction13:37
gibimostly new tables13:37
bauzasand I haven't started the api db ones13:37
gibino shadow complication13:37
bauzasgibi: well, you're probably right13:37
bauzasI was on the uefi secure boot series but I can churn13:38
gibiI haven't looked at it13:38
gibiso it is just load sharing I guess13:38
gibistephenfin: btw, can we remove the placement table creations from the Train db creation? or we removed placement from nova after train?13:39
stephenfingibi: I don't think we can remove anything without an additional migration, otherwise we'll end up with a difference between upgraded deployments and new deployments13:40
bauzasgibi: I'm following the ghostbusters directive called 'Never cross the Beams'13:40
gibistephenfin: you are right, that is a new migration. nvm13:40
gibis/nvm/never mind/13:40
stephenfinThen again, I do know we have some table columns that we don't reflect in the model (something to do with vpmem, iirc) so maybe it's not totally crazy. I should discuss with dansmith next cycle13:41
bauzasdammit s/beams/streams f'ck you French dubbing13:41
*** zul has joined #openstack-nova13:41
stephenfinFor now though, it's easier to keep it. There's a lot of models we need to delete13:41
stephenfin*there are13:41
gibistephenfin: agree. let's do a cleanup next cylce if possible13:41
sean-k-mooneystephenfin: ya the vpmem column was used for like 10 commits then we replaced it with a generic resouces column13:42
*** ociuhandu has joined #openstack-nova13:42
sean-k-mooneywe never released with it in use so you would have only used it if you deployed on master13:42
stephenfinah yes, that was it13:42
stephenfindifferent situation so13:42
sean-k-mooneywe may have others tha were used in the past and are not now13:42
sean-k-mooneywe historically have not dropped those13:43
sean-k-mooneybut we could13:43
sean-k-mooneywe did not drop them incase you were doing FFU13:43
sean-k-mooneybut the oldest ones could likely go13:43
sean-k-mooneyor the placment related ones in the nova db13:43
stephenfinyeah, I have patches up to remove some models but they'll need to be reworked to be alembic things13:44
stephenfinIn Xena though13:44
*** Luzi has joined #openstack-nova13:47
* gibi still not used to call it Xena, as it brings up mental pictures from an old TV series13:48
bauzasI wouldn't say this series helped teenagers to grow13:52
gibi:)13:52
*** jangutter has joined #openstack-nova13:55
dansmithstephenfin: I've had that open for a while now and just haven't gotten to it, I'm sorry... this week has been crazy and I likely won't get to it, but will keep it in my queue13:55
*** belmoreira has quit IRC13:56
stephenfindansmith: no worries :)13:56
dansmithstephenfin: but, my confidence is high based on the last round, so you don't need to wait for me if others have done the due diligence13:56
stephenfinack13:56
*** belmoreira has joined #openstack-nova13:56
*** spatel has joined #openstack-nova13:56
stephenfinbauzas: fwiw ^ (though I rank secure-boot slightly higher rn, in case it's between the two)13:56
*** jangutte_ has quit IRC13:57
bauzasack, reviews reviews reviews13:57
bauzasin the middle I should take opportunity of revisiting my own RPC API change13:58
stephenfinall the reviews!13:58
*** macz_ has joined #openstack-nova13:58
*** macz_ has quit IRC14:02
*** rpittau is now known as rpittau|afk14:04
stephenfinprometheanfire: https://review.opendev.org/c/openstack/nova/+/741351 is approved so mock 4.x should be unblocked from our side now, fwiw14:04
stephenfin*is merged14:04
*** jangutter_ has joined #openstack-nova14:06
*** jangutter has quit IRC14:09
*** janno_ has quit IRC14:12
*** janno has joined #openstack-nova14:13
sean-k-mooneygibi: would you have time to look at this small plamcent change https://review.opendev.org/c/openstack/placement/+/77717414:14
sean-k-mooneygibi: it looks like we skiped a step as part of the rescent os-traits release.14:14
gibilooking14:14
*** zenkuro has quit IRC14:16
*** zenkuro has joined #openstack-nova14:20
*** zenkuro has quit IRC14:25
*** Luzi has quit IRC14:28
*** ociuhandu has quit IRC14:29
*** ociuhandu has joined #openstack-nova14:30
*** ociuhandu has quit IRC14:34
*** spatel has quit IRC14:36
*** luksky has quit IRC14:36
*** Luzi has joined #openstack-nova14:43
*** ociuhandu has joined #openstack-nova14:44
*** lemko has joined #openstack-nova14:46
*** munimeha1 has joined #openstack-nova14:51
*** macz_ has joined #openstack-nova14:53
*** macz_ has quit IRC14:57
jkulikhm ... is there a mechanism to clean up lingering build-requests? We seem to have a couple still around ...15:03
*** zoharm has quit IRC15:08
*** admin0 has quit IRC15:10
*** lpetrut has joined #openstack-nova15:13
sean-k-mooneyjkulik: i belive they shoudl be cleaned up when the instance exits teh build status15:14
sean-k-mooneyso if they are lingering you proably have vms stuck in build or you have deleted them and we did not clean up properly15:14
sean-k-mooneyjkulik: do you know which is the case?15:14
jkulikI cannot find them in any cell DB. I assume they never made it there.15:15
jkulikso probably network problems at the time. if there's no automatic mechanism, we have to clean them manually before upgrading to rocky15:17
jkulikthey're missing the "trusted_certs" attribute and thus make server listing fail with an HTTP 50015:18
*** dtantsur|brb is now known as dtantsur15:18
*** spatel has joined #openstack-nova15:19
*** bhagyashris is now known as bhagyashri|ruck15:20
*** abhishekk is now known as abhishekk|afk15:20
*** lpetrut has quit IRC15:24
gibilyarwood: do you remember why the rpc_response_timeout=1 is needed in https://review.opendev.org/c/openstack/nova/+/768351/2/nova/tests/functional/regressions/test_bug_1909120.py#31 ? That test case sometimes fails to me locally with http://paste.openstack.org/show/802963/ and I try to figure out why15:33
sean-k-mooneygibi: hum maybe it was jus tto have it time out quickly15:34
gibiwondering if what happens is that the timeout is enforced even if we have castAsCall used, and the actual call takes longer than 1 sec15:34
sean-k-mooneylooking at it quickly im not sure. settign it to like 5s or 10 likely is ok15:35
sean-k-mooneywe should not really be relying on that for timing anyway15:36
gibiI don't see where it in the test helps if an rpc times out early15:37
gibihence my question15:37
sean-k-mooneythe closes is proabnly line 5915:38
sean-k-mooneybut we shoudl ahve a wait before that if we were relying on it15:38
* lyarwood reads back15:38
lyarwoodgibi: yeah so that's just to fail the call to detach a volume on the down compute, otherwise we'd wait for the default timeout to be hit.15:39
*** spatel has quit IRC15:39
lyarwoodgibi: and yes it's normally a cast but thanks to castascall here we timeout15:39
gibilyarwood: hm so after the fix https://review.opendev.org/c/openstack/nova/+/768352 the change in the timeout is not used15:40
gibilyarwood: so what I see is that the _create_server fails in that test case sometimes for me locally as the scheduler calls the conductor to build the instance but the conductor takes more than 1 sec to reply15:41
lyarwoodah yeah I see15:41
gibiif I undertand correclty after the fix the shrinked timeout is not needed any more15:41
lyarwoodcorrect15:41
gibicool, then I will remove that to stabilize the func test on a small compute15:41
gibiI mean on a slow node15:42
lyarwoodyup, apologies for missing that, I hadn't seen it fail anywhere like that while working on it15:42
openstackgerritMerged openstack/nova master: apidb: Add manage.py script  https://review.opendev.org/c/openstack/nova/+/77141915:43
gibilyarwood: no worries. I tend to collect a list of unstable func and unit test I see locally and time to time go thought15:44
stephenfinlyarwood: Think you could take a quick look at this stable patch? https://review.opendev.org/c/openstack/nova/+/75124015:45
stephenfinContext is that it'll help a student working on openstacksdk patches for adding placement support15:45
lyarwoodstephenfin: yup looking15:46
*** dklyle has joined #openstack-nova15:46
*** abhishekk|afk is now known as abhishekk15:47
sean-k-mooneystephenfin: gibi lyarwood quick question https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/models.py#L148315:51
sean-k-mooneythe dev_type filed in the db is restrited to a lenth of 815:51
sean-k-mooneytype-vdpa is 915:51
gibi /o\15:51
stephenfindrop type-15:51
sean-k-mooneyso shoudl i extend the column or just sotre vdpa15:51
stephenfinit was silly in the first place15:51
gibiOK, that works for me the type prefix seems redundant15:52
sean-k-mooneyya i really doubt it save any space in reality15:52
*** munimeha1 has quit IRC15:52
sean-k-mooneycool ill do that so15:52
sean-k-mooneyand ya the prefix is not really useful15:52
*** elod has quit IRC15:58
*** elod has joined #openstack-nova16:00
*** ociuhandu has quit IRC16:09
*** ociuhandu has joined #openstack-nova16:10
*** k_mouza_ has joined #openstack-nova16:14
*** ociuhandu has quit IRC16:14
*** macz_ has joined #openstack-nova16:15
prometheanfiresean-k-mooney: yep, thanks :D16:15
prometheanfirestephenfin: meant that for you about the mock thing :D16:16
*** k_mouza has quit IRC16:17
*** tbachman_ has quit IRC16:23
*** tbachman has joined #openstack-nova16:25
*** vishalmanchanda has quit IRC16:36
*** gyee has joined #openstack-nova16:41
*** ociuhandu has joined #openstack-nova16:47
*** belmoreira has quit IRC16:48
*** dtantsur is now known as dtantsur|brb16:50
*** lucasagomes has quit IRC17:01
*** luksky has joined #openstack-nova17:02
*** ociuhandu_ has joined #openstack-nova17:04
*** ociuhandu has quit IRC17:07
*** ociuhandu_ has quit IRC17:08
*** k_mouza_ has quit IRC17:14
*** k_mouza has joined #openstack-nova17:14
*** ociuhandu has joined #openstack-nova17:18
*** ociuhandu has quit IRC17:22
*** derekh has quit IRC17:23
*** derekh has joined #openstack-nova17:23
*** derekh has quit IRC17:26
*** derekh has joined #openstack-nova17:26
*** mgariepy has quit IRC17:33
*** khomesh24 has quit IRC17:42
*** derekh has quit IRC17:46
*** derekh has joined #openstack-nova17:47
*** tesseract has quit IRC17:49
openstackgerritClark Boylan proposed openstack/nova master: DO NOT MERGE testing a thing  https://review.opendev.org/c/openstack/nova/+/77744417:52
*** dklyle has quit IRC17:53
*** Luzi has quit IRC17:56
*** derekh has quit IRC18:00
lyarwoodgibi: https://blueprints.launchpad.net/nova/+spec/ephemeral-storage-encryption - I've moved this to Deferred, not sure if that's the correct state but I'm punting it to at least X, I'll document why upstream next week once I'm back from PTO.18:00
*** ralonsoh has quit IRC18:00
openstackgerritClark Boylan proposed openstack/nova master: DO NOT MERGE testing a thing  https://review.opendev.org/c/openstack/nova/+/77744418:01
gibilyarwood: thanks for that18:01
gibilyarwood: right now Deferred is OK, I will do the cleanup after FF anyhow18:01
lyarwoodgibi: ack cool18:02
*** iurygregory_ has joined #openstack-nova18:04
*** iurygregory has quit IRC18:05
*** iurygregory_ is now known as iurygregory18:05
*** mgariepy has joined #openstack-nova18:11
*** dklyle has joined #openstack-nova18:13
openstackgerritBalazs Gibizer proposed openstack/nova master: Remove short call timeout from func test  https://review.opendev.org/c/openstack/nova/+/77744718:20
gibilyarwood: a small fix for the func test we talked about ^^18:20
*** dtantsur|brb is now known as dtantsur18:31
*** spatel has joined #openstack-nova18:31
*** andrewbonney has quit IRC18:42
*** k_mouza has quit IRC19:00
*** jangutter has joined #openstack-nova19:03
*** gregwork has joined #openstack-nova19:06
*** jangutter_ has quit IRC19:06
gregworkcan nova flavors be created that have multiple ephemeral disks?19:06
gregworki know you can nova boot --ephemeral x --ephemeral y19:06
gregworktrying a nova flavor-create --ephemeral x --ephemeral y doesn't respect this via the CLI19:07
*** rcernin has joined #openstack-nova19:10
*** tbachman_ has joined #openstack-nova19:13
sean-k-mooneygregwork: yes and no19:13
*** tbachman has quit IRC19:13
*** tbachman_ is now known as tbachman19:13
sean-k-mooneyyou can have multiple ephmeral disk19:13
sean-k-mooneyi dont think you can spcify that in the flavor19:13
sean-k-mooneyi think you specify that durign instance creation19:13
sean-k-mooneythere is quite an obsucre way to do this ill see if i can find it but its not commonly used as far as i am aware19:14
sean-k-mooneyok not that obscure19:15
sean-k-mooneyova boot --flavor test-flavor --image cirros --nic net-name=test-network --ephemeral size=1,format=ext4 --ephemeral size=2,format=ext4 test-vm19:15
gregworkso pick whatever flavor and just boot it with whats needed19:15
sean-k-mooneygregwork: but basically the amount of space in the flavor can be subdevied during boot19:15
*** rcernin has quit IRC19:15
gregworkoh19:15
sean-k-mooneyya provided you dont excee the amount of space in the flavor its fine19:15
gregworkdo the value of "disk"19:16
sean-k-mooneydisk is root disk19:16
gregworkyeah so im curious as to what you mean by the space in the flavor19:16
sean-k-mooneytherre are 2 storage vaules in the flavor19:17
sean-k-mooneydisk and OS-FLV-EXT-DATA:ephemeral19:17
sean-k-mooneyhttps://docs.openstack.org/api-ref/compute/?expanded=add-flavor-access-to-tenant-addtenantaccess-action-detail,create-flavor-detail#create-flavor19:18
sean-k-mooneyOS-FLV-EXT-DATA:ephemeral is what is used to limit --ephemeral19:18
gregworki was under the impression that that value is what will be attached as a separate volume to root19:19
gregworki did not know it could be sub divided19:19
sean-k-mooneygregwork: some things to keep in mind are ephemeral disk are not snapshot so that also means they are not preserved on shelve19:19
sean-k-mooneygregwork: most people dont if you dont pass any args that will be prented as a singel addtional disk to the instance by default19:20
gregworkthats kind of cool19:20
sean-k-mooneybut if the flaovr has addtional ephemeral storage you can subdevide it19:20
sean-k-mooneygregwork: the other thing to kep in mind is that resize is undefined19:21
sean-k-mooneyor rather resize where the epmeral size changes is undefiend (read broken as hell)19:21
sean-k-mooneyso its cool be there be dragons if you try to resize down19:21
*** tbachman has quit IRC19:22
sean-k-mooneyup might be ok19:22
sean-k-mooneybut again very rarely used feature19:22
sean-k-mooneyalso very old i think it predate cinder19:22
gregworkthe situation we are trying to work around is this quick poc we are running needs fast disk and cinder is overloaded.  So they are going to stuff some flash in 3 computes dedicated to this project19:23
sean-k-mooneygregwork: if you want to do it with  the openstack client you have to manually build the block device mappings dict and use the local volum type to do this19:23
gregworkso we are going to run off the local ephemeral19:23
*** tbachman has joined #openstack-nova19:23
sean-k-mooneyas long as people are awrae fo the limiations, local scratch space is exactly what that flavor value is for19:25
gregworkfor sure, its a demo environment19:25
*** dtantsur is now known as dtantsur|afk19:56
*** k_mouza has joined #openstack-nova20:14
openstackgerritMerged openstack/nova master: libvirt: Rationalize attachment of USB tablet  https://review.opendev.org/c/openstack/nova/+/75655120:17
*** k_mouza has quit IRC20:19
*** rcernin has joined #openstack-nova20:33
*** zzzeek has quit IRC20:34
*** zzzeek has joined #openstack-nova20:34
*** rcernin has quit IRC20:47
*** spatel has quit IRC20:52
*** zenkuro has joined #openstack-nova20:55
*** zenkuro has quit IRC21:00
*** zenkuro has joined #openstack-nova21:00
*** jdillaman has joined #openstack-nova21:04
*** whoami-rajat has quit IRC21:12
*** rcernin has joined #openstack-nova21:14
*** rcernin has quit IRC21:19
*** zzzeek has quit IRC21:26
*** zzzeek has joined #openstack-nova21:28
*** rcernin has joined #openstack-nova21:32
*** rcernin has quit IRC21:37
*** rcernin has joined #openstack-nova21:40
*** rcernin has quit IRC21:45
*** rcernin has joined #openstack-nova21:46
*** slaweq has quit IRC21:48
*** zzzeek has quit IRC21:49
*** rcernin has quit IRC21:51
*** zzzeek has joined #openstack-nova21:53
*** hamalq has joined #openstack-nova22:11
*** xek has quit IRC22:13
*** rcernin has joined #openstack-nova22:15
openstackgerritMerged openstack/placement master: Update traits in tests and requirements  https://review.opendev.org/c/openstack/placement/+/77717422:22
*** mlavalle has joined #openstack-nova22:36
*** luksky has quit IRC22:39
*** pmannidi has quit IRC23:05
openstackgerritsean mooney proposed openstack/nova master: add constants for vnic type vdpa  https://review.opendev.org/c/openstack/nova/+/77047423:10
openstackgerritsean mooney proposed openstack/nova master: [WIP] add vdpa nodedev parsing and interface config gen  https://review.opendev.org/c/openstack/nova/+/77053223:10
openstackgerritsean mooney proposed openstack/nova master: [WIP] add vdpa min version support  https://review.opendev.org/c/openstack/nova/+/77053323:10
openstackgerritsean mooney proposed openstack/nova master: extend pci dev_type for vdpa  https://review.opendev.org/c/openstack/nova/+/77748123:10
*** pmannidi has joined #openstack-nova23:10
*** zzzeek has quit IRC23:14
*** zzzeek has joined #openstack-nova23:15
*** zenkuro has quit IRC23:30
« Tuesday, 2021-02-23 Index Thursday, 2021-02-25 »

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