Friday, 2020-12-11

« Thursday, 2020-12-10 Index Saturday, 2020-12-12 »
*** k_mouza has joined #openstack-nova00:07
*** _erlon_ has quit IRC00:10
*** macz_ has quit IRC00:14
*** jangutter has quit IRC00:16
*** jangutter has joined #openstack-nova00:17
*** k_mouza has quit IRC00:18
*** k_mouza has joined #openstack-nova00:21
*** ociuhandu has joined #openstack-nova00:21
*** k_mouza has quit IRC00:22
*** ociuhandu has quit IRC00:25
brinzhanggmann: ack, that's what I did, just change the List Security Groups By Server API, dont change the deprecated apis.00:40
brinzhanggmann: thanks00:40
gmannbrinzhang: +1, I will check your patches tomorrow.00:41
brinzhanggmann: yeah, thanks, I have to add the needed unit tests, then submit it, and add you as a reviewer00:42
gmanncool00:42
*** tbachman has joined #openstack-nova01:04
*** mlavalle has quit IRC01:38
*** hamalq_ has quit IRC02:05
*** hemanth_n has joined #openstack-nova02:32
*** spatel has joined #openstack-nova02:38
*** spatel has quit IRC02:42
-openstackstatus- NOTICE: The Gerrit service on review.opendev.org is being restarted quickly to enable support for Git protocol v2, downtime should be less than 5 minutes02:54
*** mkrai has joined #openstack-nova03:01
*** lifeless has quit IRC03:31
*** lifeless has joined #openstack-nova03:33
*** rcernin has quit IRC03:39
*** rcernin has joined #openstack-nova03:40
*** k_mouza has joined #openstack-nova03:48
*** k_mouza has quit IRC03:52
*** evrardjp_ has quit IRC05:33
*** evrardjp has joined #openstack-nova05:33
*** ratailor has joined #openstack-nova05:41
*** lemko0 has joined #openstack-nova05:47
*** vishalmanchanda has joined #openstack-nova05:47
*** ociuhandu has joined #openstack-nova05:50
*** lemko has quit IRC05:50
*** lemko0 is now known as lemko05:50
*** k_mouza has joined #openstack-nova05:52
*** ociuhandu has quit IRC05:54
*** k_mouza has quit IRC05:57
*** xek has joined #openstack-nova06:37
*** adrian-a has joined #openstack-nova06:49
openstackgerritzhulipeng proposed openstack/nova stable/pike: Fixed multipath residue when evacuating the VM  https://review.opendev.org/c/openstack/nova/+/76667506:59
*** mkrai has quit IRC07:04
*** adrian-a has quit IRC07:05
*** adrian-a has joined #openstack-nova07:09
*** zzzeek has quit IRC07:17
*** zzzeek has joined #openstack-nova07:19
*** mkrai has joined #openstack-nova07:22
*** dklyle has quit IRC07:28
*** slaweq has joined #openstack-nova07:45
*** mkrai has quit IRC07:56
*** hamalq has joined #openstack-nova07:56
*** rpittau|afk is now known as rpittau07:57
*** hamalq_ has joined #openstack-nova07:57
*** mkrai has joined #openstack-nova07:59
*** hamalq has quit IRC08:01
*** k_mouza has joined #openstack-nova08:03
gibigood morning08:05
gibimelwitt: thanks for the babysitt08:06
*** k_mouza has quit IRC08:07
*** rcernin has quit IRC08:07
gibiI see we were not lucky so far08:08
*** tesseract has joined #openstack-nova08:10
*** rcernin has joined #openstack-nova08:14
*** evrardjp has quit IRC08:15
*** evrardjp has joined #openstack-nova08:16
*** andrewbonney has joined #openstack-nova08:22
bauzasgibi: good morning08:27
gibibauzas: o/08:29
*** tosky has joined #openstack-nova08:47
*** ociuhandu has joined #openstack-nova09:00
*** ociuhandu has quit IRC09:00
*** ociuhandu has joined #openstack-nova09:00
gibihttps://review.opendev.org/c/openstack/nova/+/765749 is in the gate queue (again), fingers crossed09:02
*** adrian-a has quit IRC09:03
*** derekh has joined #openstack-nova09:07
*** adrian-a has joined #openstack-nova09:12
*** hamalq_ has quit IRC09:15
*** hamalq has joined #openstack-nova09:15
*** martinkennelly has joined #openstack-nova09:23
*** rcernin has quit IRC09:27
elodlyarwood: thanks for those backports \o/ we'll see whether everything is happy with it :) so there are now 3 possible solution for the bandit issue, so hopefully it'll be sorted out soon!09:52
elodlyarwood: btw, I'm kinda @ PTO today and monday, but I'll be here time to time and try to look at things09:53
*** dtantsur|afk is now known as dtantsur09:58
*** zzzeek has quit IRC10:03
*** brinzhang has quit IRC10:04
*** brinzhang has joined #openstack-nova10:04
*** k_mouza has joined #openstack-nova10:05
*** zzzeek has joined #openstack-nova10:05
*** macz_ has joined #openstack-nova10:09
*** k_mouza has quit IRC10:09
*** hamalq has quit IRC10:10
*** ociuhandu has quit IRC10:12
*** macz_ has quit IRC10:14
*** ociuhandu has joined #openstack-nova10:16
*** k_mouza has joined #openstack-nova10:17
*** adrian-a has quit IRC10:21
*** zzzeek has quit IRC10:39
*** zzzeek has joined #openstack-nova10:40
*** adrian-a has joined #openstack-nova10:45
*** ociuhandu has quit IRC10:52
*** mkrai has quit IRC10:53
*** mkrai_ has joined #openstack-nova10:53
*** ociuhandu has joined #openstack-nova10:58
*** mkrai_ has quit IRC11:22
hemanth_nHi Nova team, can i get reviews on ussuri backport https://review.opendev.org/c/openstack/nova/+/761701 (Had initial review from elod with +2)11:22
lyarwoodhemanth_n: ack will look shortly11:25
hemanth_nlyarwood: thanks11:26
*** adrian-a has quit IRC11:28
gibilyarwood: https://review.opendev.org/c/openstack/nova/+/765749 bounced from the gate again11:28
gibiwith the grande job failing on test_live_block_migration_paused11:29
*** adrian-a has joined #openstack-nova11:33
*** k_mouza has quit IRC11:35
*** StevenK has quit IRC11:35
openstackgerritKashyap Chamarthy proposed openstack/nova-specs master: Re-propose "Secure Boot support for KVM & QEMU guests" for Wallaby  https://review.opendev.org/c/openstack/nova-specs/+/75973111:37
kashyapgibi: stephenfin: --^ Addressed the small comments from PTG and review (adding the bit about traits; and adjusting version info, and couple of minor bits)11:38
kashyap(Cc: Sean, Bauzas.  Previous reviewers :-))11:39
lyarwoodgibi: gah, that's still on bionic iirc,11:39
lyarwoodgibi: iirc I had see quite a few failures recently like that on master, I'll write up an ER query shortly and check11:39
*** StevenK has joined #openstack-nova11:44
sean-k-mooneyyep test_live_block_migration_paused has been failing on master11:45
lyarwoodjust in the grenade job however right?11:46
lyarwoodI don't think the focal job is hitting this11:46
sean-k-mooneyyes i think so11:46
*** slaweq_ has joined #openstack-nova11:46
lyarwoodhemanth_n: https://review.opendev.org/c/openstack/nova/+/761701 LGTM but the gate is blocked at the moment, I'm going to +2 and follow up with a +W once it's working again11:46
sean-k-mooneylyarwood: did you see my review comments11:47
lyarwoodsean-k-mooney: nope11:47
* lyarwood refreshes11:47
sean-k-mooneylyarwood: it looks fine but i think it would be cleaner to pull in the two missing trivial refactor patches11:47
lyarwoodsean-k-mooney: ack, I can help with that11:47
hemanth_nsean-k-mooney: do you want them backport separately or as part of this patch11:48
sean-k-mooneyhemanth_n: seperatly11:48
sean-k-mooneyhttps://github.com/openstack/nova/commit/17ba10b75961ef667e45b9dedbac34212a7e7e9b and https://github.com/openstack/nova/commit/0ab8a03c782851d4019c90e6159bd1f83c0658e611:48
sean-k-mooneywell as in as seperate patches11:48
sean-k-mooneybut backprot the 3 together11:48
hemanth_nsean-k-mooney: ack, lyardwood: i am my eod already, if you want me to backport other patches i can do them on monday11:50
sean-k-mooneythe order shoudl be 0ab8a03c782851d4019c90e6159bd1f83c0658e6 then 17ba10b75961ef667e45b9dedbac34212a7e7e9b then your patch11:50
lyarwoodhemanth_n: ack, I can try to do this later today for you11:51
hemanth_nsean-k-mooney, lyardwood: thanks for the quick responses once again11:51
*** tbachman has quit IRC11:55
*** dtantsur is now known as dtantsur|brb11:56
*** ociuhandu has quit IRC11:58
*** ociuhandu has joined #openstack-nova11:58
*** slaweq has quit IRC12:02
*** slaweq_ has quit IRC12:03
*** ociuhandu has quit IRC12:03
openstackgerritLee Yarwood proposed openstack/nova master: nova-grenade-multinode: Skip test_live_block_migration_paused  https://review.opendev.org/c/openstack/nova/+/76672012:06
*** k_mouza has joined #openstack-nova12:06
*** slaweq has joined #openstack-nova12:06
lyarwoodgibi: ^ lets skip that test until multinode grenade moves to zuulv3 and Focal12:07
brinzhanghi team, when do we support GET /allocation_candidates?resources=CUSTOM_PMEM_NAMESPACE_xxGB? in nova and placement?12:08
brinzhangI know support this when we accept the PMEM support patch in Nova, but Icannot find the placement changes, anyone know?12:09
sean-k-mooneybrinzhang: there was no placment change required12:11
brinzhangwe hit an error: GET /allocation_candidates?limit=1000&resources=CUSTOM_PMEM_NAMESPACE_10GB%3A1%2CDISK_GB%3A30%2CMEMORY_MB%3A2048%2CVCPU%3A2 HTTP/1.1" 40012:11
sean-k-mooneybrinzhang: what was the actul error in the placement logs12:12
sean-k-mooney its asking for CUSTOM_PMEM_NAMESPACE_10GB:1,DISK_GB:30,MEMORY_MB:2048,VCPU:2412:13
sean-k-mooneythat looks like a perfectly normal request to me12:14
stephenfinbrinzhang: what's configured in '[libvirt] pmem_namespaces' in nova.conf? You should have an entry like 'pmem_namespaces = 10GB:ns0,...12:14
brinzhangsean-k-mooney: please see: Invalid resource class in resources parameter: No such resource class CUSTOM_PMEM_NAMESPACE_30GB12:14
stephenfinthen nova isn't reporting the inventory12:14
sean-k-mooneyyep nova should create the inventory and register teh resouce class12:15
stephenfineither you don't have it configured, your configuration is wrong, or you're using nova < Train12:15
brinzhangstephenfin: pmem_namaspaces = 10GB:ns10_0|ns10_1|ns10_2|ns10_3,30GB:ns30_0|ns30_1|ns30_2|ns30_312:15
brinzhangthis is our config in nova.conf12:15
brinzhangstephenfin: does the config is right?12:16
stephenfinit looks okay12:17
stephenfinand what does 'openstack resource provider inventory list $COMPUTE_ID' show?12:17
sean-k-mooneybased on the docs yes12:17
sean-k-mooneybrinzhang: you precreated the namespaces right?12:18
sean-k-mooneyyou have to do that manually nova will not do that for you12:18
brinzhangyes, we precreated the namespaces manully12:18
brinzhangs/manully/manually12:19
sean-k-mooneyok then ya can you provide the output of the command stephenfin asked for12:19
brinzhanglet my try12:19
openstackgerritLee Yarwood proposed openstack/nova stable/train: DNM - Testing https://review.opendev.org/c/openstack/devstack/+/766622  https://review.opendev.org/c/openstack/nova/+/76672312:20
*** slaweq has quit IRC12:20
sean-k-mooneyif the inventories are not present you need to check the nova comptue agent to see if the update_avaiable_resoue periodic task is failing and if so why12:21
*** ratailor has quit IRC12:21
*** slaweq has joined #openstack-nova12:21
*** amodi has quit IRC12:22
gibilyarwood: +2 on the skipping the failing live migration test12:22
*** dtantsur|brb is now known as dtantsur12:27
*** ociuhandu has joined #openstack-nova12:33
brinzhangsean-k-mooney, stephenfin: yes, it's empty, Ishould check the update_avaiable_resoue periodic task, thanks.12:37
brinzhangI will check it later ^12:38
*** hemanth_n has quit IRC12:44
openstackgerritBrin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List SG API  https://review.opendev.org/c/openstack/nova/+/76672612:44
openstackgerritBrin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List SG API  https://review.opendev.org/c/openstack/nova/+/76672612:45
openstackgerritAlexandre arents proposed openstack/nova master: libvirt: make cross cell resize spawn from snapshot image  https://review.opendev.org/c/openstack/nova/+/76556112:50
*** k_mouza_ has joined #openstack-nova12:51
*** k_mouza has quit IRC12:51
openstackgerritMerged openstack/nova master: Disable qos resize tempest test for nova-multi-cell job  https://review.opendev.org/c/openstack/nova/+/76647112:53
*** k_mouza_ has quit IRC12:56
*** adrian-a has quit IRC12:58
*** ociuhandu_ has joined #openstack-nova12:58
*** adrian-a has joined #openstack-nova12:59
gibi^^ \o/12:59
*** k_mouza has joined #openstack-nova12:59
*** k_mouza has quit IRC13:00
*** k_mouza_ has joined #openstack-nova13:00
*** ociuhandu has quit IRC13:01
*** k_mouza_ has quit IRC13:02
openstackgerritAlexandre arents proposed openstack/nova master: libvirt: make cross cell resize spawn from snapshot image  https://review.opendev.org/c/openstack/nova/+/76556113:02
*** k_mouza has joined #openstack-nova13:04
*** LinPeiWen has quit IRC13:05
*** efried has joined #openstack-nova13:07
openstackgerritLee Yarwood proposed openstack/nova stable/victoria: nova-grenade-multinode: Skip test_live_block_migration_paused  https://review.opendev.org/c/openstack/nova/+/76574913:08
lyarwoodwhops13:08
*** efried has quit IRC13:14
*** efried has joined #openstack-nova13:14
*** zzzeek has quit IRC13:17
brinzhanggibi, stephenfin, sean-k-mooney: hope you can review cyborg shelve/unshelve support https://review.opendev.org/c/openstack/nova/+/72956313:17
*** zzzeek has joined #openstack-nova13:19
openstackgerritLee Yarwood proposed openstack/nova stable/victoria: lower-constraints: Bump packaging to 20.4  https://review.opendev.org/c/openstack/nova/+/76574913:20
lyarwoodbrb in 3013:25
*** macz_ has joined #openstack-nova13:25
*** k_mouza_ has joined #openstack-nova13:27
*** k_mouza has quit IRC13:29
*** macz_ has quit IRC13:30
*** tbachman has joined #openstack-nova13:31
gibibrinzhang: sorry, I won't get to it this week13:36
openstackgerritAlexandre arents proposed openstack/nova master: Revert "nova-multi-cell: Skip test_cold_migrate_unshelved_instance"  https://review.opendev.org/c/openstack/nova/+/76593313:45
*** songwenping_ has quit IRC13:46
*** songwenping_ has joined #openstack-nova13:47
*** smcginnis has joined #openstack-nova13:56
openstackgerritLee Yarwood proposed openstack/nova stable/train: [stable-only] Cap bandit to 1.6.2  https://review.opendev.org/c/openstack/nova/+/76617113:56
*** mlavalle has joined #openstack-nova13:58
lyarwoodwas anyone working on ERROR: No matching distribution found for hacking<3.1.0,>=3.0.1 in stable/ussuri?13:59
gibilyarwood: I'm not aware of such work14:01
lyarwoodhttps://bugs.launchpad.net/nova/+bug/1907756 kk thanks, we have a bug at least :)14:01
openstackLaunchpad bug 1907756 in OpenStack Compute (nova) "ERROR: No matching distribution found for hacking<3.1.0,>=3.0.1" [Undecided,New]14:01
kashyapsean-k-mooney: Fair point on the upgrade impact and fixing the version history.  Thanks for the quick look14:06
lyarwoodhttp://paste.openstack.org/show/800970/ excellent, pip sorted out the awful error message in the latest version14:10
kashyapsean-k-mooney: Do you have an example of how 'placement-manage' can register a trait?14:10
openstackgerritLee Yarwood proposed openstack/nova stable/ussuri: [stable-only] Bump hacking to 3.0.1 in lower-contraints.txt  https://review.opendev.org/c/openstack/nova/+/76673814:14
bauzaskashyap: not placement-manage14:16
bauzaskashyap: osc-placement rather14:16
bauzasyou ask for a custom trait, right?14:16
kashyapAh, okay; then in the review here, Sean meant 'osc-placement', then: https://review.opendev.org/c/openstack/nova-specs/+/75973114:16
kashyapbauzas: See this part of Sean's comment:14:17
bauzaskashyap: see this example https://docs.openstack.org/nova/latest/admin/virtual-gpu.html#optional-provide-custom-traits-for-multiple-gpu-types14:17
kashyap[quote]14:17
kashyap...  this will require os-traits to be installed14:17
kashyapon the host runing plamcent and the placement-manage executable to be14:17
kashyaprun to register the new standard traits before nova is upgraded."14:17
kashyap[/quote]14:17
kashyapbauzas: /me clicks14:17
bauzasah, this14:17
kashyapI'm placement-unaware -- so is there an example of "placement-manage executable to be run to register the new standard traits"14:18
openstackgerritLee Yarwood proposed openstack/nova stable/ussuri: [stable-only] lower-contraints: Bump hacking to 3.0.1  https://review.opendev.org/c/openstack/nova/+/76673814:19
kashyapbauzas: The only examples of "placement-manage" I see are during data migration here: https://docs.openstack.org/placement/latest/admin/upgrade-to-stein.html14:20
kashyap[Scroll to the bottom to see the 'placement-manage' invocation]14:20
kashyapbauzas: Ah-ha!  I know what Sean meant: to sync the 'os-traits' into Placement: `placement-manage os-traits sync`14:30
bauzascorrect, sorry I forgot to answer14:30
bauzasyou need to tell you support a new os-trait14:31
kashyapNo problem14:31
*** adrian-a has quit IRC14:32
bauzassean-k-mooney: dumb question, but as I copied from https://review.opendev.org/c/openstack/nova/+/656885/7/nova/network/neutron.py, do you know what we should verify the multi-provider extension given we only call the segments API extension ?14:32
*** adrian-a has joined #openstack-nova14:34
openstackgerritKashyap Chamarthy proposed openstack/nova-specs master: Re-propose "Secure Boot support for KVM & QEMU guests" for Wallaby  https://review.opendev.org/c/openstack/nova-specs/+/75973114:34
kashyapgibi: sean-k-mooney: Fixed --^ Hope that looks okay14:34
*** nweinber has joined #openstack-nova14:34
sean-k-mooneyam you repeaded placment manage14:37
sean-k-mooneyservice by ``placement-manage os-traits sync``placement-manage`` .14:37
sean-k-mooneybut ya other wise it looks good to me14:38
*** jangutter_ has joined #openstack-nova14:38
*** jangutter has quit IRC14:42
gibisean-k-mooney: does placement-manager os-traits sync exists?14:43
sean-k-mooneyactully im not sure14:44
sean-k-mooneythere is dbsync14:45
gibiyeah I only see db sync14:45
sean-k-mooneyperhaps placment jsut does the traits import on startu14:45
sean-k-mooney*startup14:45
gibisean-k-mooney: it seems it does it at startup https://github.com/openstack/placement/blob/10146978ea2d7eace1ffdf6552eafd22360e37dd/placement/deploy.py#L13314:46
sean-k-mooneyok cool in this case i guess you just need to install the new os-traits and restart14:47
gibiyepp14:48
kashyapsean-k-mooney: Sigh; fixing :)14:49
kashyapgibi: Pretty sure I saw the "os-traits sync" somewhere -- let me dig it up14:50
kashyaphttps://specs.openstack.org/openstack/nova-specs/specs/pike/implemented/resource-provider-traits.html14:51
gibikashyap: I did not found a corresponding implementation14:52
kashyapgibi: Ah, okay.  Then, what do you suggest here? :-)  Just a 'placement-manage db sync' is okay?14:53
gibikashyap: as sean-k-mooney said, upgrade the os-traits lib and then restat the placement service14:53
gibiplacement service at startup will sync the traits from os-traits into the placement db14:53
kashyapOkay; I'll adjust the text to that effect; thank you!14:53
bauzassean-k-mooney: saw my question about the multi-provider extension ?14:55
bauzasI'm about to remove it14:55
sean-k-mooneybauzas: no i didnt but ill look now14:56
bauzasno worries14:56
kashyap[quote]14:56
kashyap   This requires 'os-traits' to be installed (or upgraded if need be) on14:56
kashyap   the host running ``placement`` service.  Followed by a restart ofthe14:56
kashyap   ``placement`` service -- this will synchronize the traits into the14:56
kashyap   Placement database.14:56
kashyap[/quote]14:56
kashyapgibi: sean-k-mooney: Does that read okay?14:56
sean-k-mooneyya that fine14:57
gibikashyap: OK14:57
bauzassean-k-mooney: tl;dr: why should we verify this extension in https://review.opendev.org/c/openstack/nova/+/656885/7/nova/network/neutron.py@352214:57
bauzas?14:57
kashyapgibi: If you're not fully satisfied, can still tweak it :-)14:57
gibikashyap: I'm satisfied, it is Friday :D14:57
kashyapLOL, noted14:57
* kashyap --> three more meetings to go today; and I'm off for the rest of the year ;-)14:58
sean-k-mooneybauzas: i think this was jsut a mix up and we should jsut eb checkign for the segments exsion14:58
bauzasall good then14:58
gibikashyap: I still have a week to put out fires :D14:58
bauzaswill remove it14:58
bauzaskashyap: wow, lovely14:58
kashyapgibi: Yikes; be sure to disconnect from all e-mail and other firehoses after that14:58
bauzashave a very nice PTO !14:58
sean-k-mooneykashyap: nice hope you enjoy it14:59
kashyapThanks!  Have some non-work) ideas to tinker with :-)14:59
gibikashyap: I want to disconnect my brain from the outside world but then I will get bored :D14:59
kashyapsean-k-mooney: Thanks, and thank for the quick review; appreciate it14:59
bauzaskashyap: go skiing^H^H^H^H^H^H outside?15:00
sean-k-mooneyya im plannign to paint my bedroom to disconnect but i also bought a storage arrawy with 80TB of drives that im going to play with15:00
*** ociuhandu_ has quit IRC15:00
sean-k-mooneyso we will see how disconnecting goes15:00
kashyapgibi: Hehe, nod15:01
sean-k-mooneyi have 4 audio books lined up and proably will add one or two more15:01
*** ociuhandu has joined #openstack-nova15:01
kashyapbauzas: I'm afraid of ice!  My friends mock me when they see me carefully balancing and walking on ice15:01
*** ociuhandu has quit IRC15:01
*** ociuhandu has joined #openstack-nova15:02
kashyapsean-k-mooney: Wow, 80TB!15:02
kashyapsean-k-mooney: Painting also sounds good; hopefully some relaxing colour15:02
sean-k-mooneyi may have bought one of these... https://www.ebay.co.uk/itm/Dell-PowerVault-MD3060e-20-x-4TB-SAS-Dell-Enterprise-Class-Hard-Drives-Rails/28391629322015:04
* kashyap clicks15:05
openstackgerritKashyap Chamarthy proposed openstack/nova-specs master: Re-propose "Secure Boot support for KVM & QEMU guests" for Wallaby  https://review.opendev.org/c/openstack/nova-specs/+/75973115:06
kashyapsean-k-mooney: Wow, that array looks quite sturdy15:06
*** ociuhandu has quit IRC15:06
sean-k-mooneyit can hold 60 drives too so i have space for 40 more if i need them15:06
kashyapHeh; that's quite some storage15:09
*** dtantsur is now known as dtantsur|brb15:15
*** rpittau is now known as rpittau|afk15:15
*** ociuhandu has joined #openstack-nova15:19
sean-k-mooneyi have 3TB of steam games and i want to move my windows gaming pc to a vm on openstack15:19
* stephenfin has a single 4TB drive15:20
sean-k-mooneyso i wanted at least 20TB of addtional storage got 80 after hotspare and raid i should have 64 usable for cinder15:20
stephenfinsean-k-mooney: I assume you used parcel motel for delivery?15:21
sean-k-mooneystephenfin: yep i also have a 4TB external usb hardrive that ehy used to be on but i cant passthough though to a vm15:21
stephenfinTo avoid the £150 postage fee15:21
sean-k-mooneynope its coming on a pallet15:21
sean-k-mooneyit ways 96KG15:21
stephenfinoh15:21
stephenfinwell then15:21
stephenfinthat's more than I weigh15:22
sean-k-mooneyya its over the wight limit15:22
stephenfin150 doesn't sound so bad now, heh15:22
sean-k-mooneyyep also the server the drives cost more then the server i would have been over 3500 for the same capasity in dirves15:23
*** rcernin has joined #openstack-nova15:24
gibisean-k-mooney: you passthrough a physical GPU for the windows gaming VM?15:24
sean-k-mooneythat is my plan yes15:24
sean-k-mooneyi did before but its currenly in my desktop15:24
gibiinteresting15:24
sean-k-mooneyjsut using pci passthoough in the flavor not the vgpu support15:24
sean-k-mooneygpus that support vGPU are still $$$$15:25
gibiyeah, and I guess you don't need multiple gpus in your openstack cluster15:26
sean-k-mooneyit might be nice but ya i dont15:26
*** macz_ has joined #openstack-nova15:26
sean-k-mooneyif amd or intel add it to a consume gpu that would be a huge add value to me15:27
sean-k-mooneybut im not spendign 3000 on a nvida gpu + lisciening15:27
gibiyeah15:28
*** rcernin has quit IRC15:28
* sean-k-mooney my server hardware additcion has limits15:29
*** macz_ has quit IRC15:31
openstackgerritBalazs Gibizer proposed openstack/nova stable/victoria: Disable qos resize tempest test for nova-multi-cell job  https://review.opendev.org/c/openstack/nova/+/76647415:34
gibilyarwood: I've backorted ^^ so we can even chose to redo https://review.opendev.org/c/openstack/nova/+/765749 by squashing ^^ into it and remove making nova-multi-cell non-votin15:36
gibibut I'm OK keeping it as is15:36
lyarwoodgibi: it's just in the check queue at the moment so feel free to if you have time15:36
gibilyarwood: OK, the I will respin it quikly15:37
melwittgibi: np, just checked it (master change) finally merged, geesh15:37
gibimelwitt: yeah, we are making progress.. :)15:37
gibimelwitt: I will drop an update to the ML thread about where we are before I leave for today15:38
melwitt++15:39
openstackgerritKashyap Chamarthy proposed openstack/nova-specs master: Re-propose "Secure Boot support for KVM & QEMU guests" for Wallaby  https://review.opendev.org/c/openstack/nova-specs/+/75973115:40
kashyapgibi: Fixed the nit15:40
gibikashyap: re-applied my badge15:41
kashyapThanks! :)15:41
gibiI thank you15:41
*** adrian-a has quit IRC15:45
openstackgerritBalazs Gibizer proposed openstack/nova stable/victoria: lower-constraints: Bump packaging to 20.4  https://review.opendev.org/c/openstack/nova/+/76574915:47
gibilyarwood: ^^15:47
*** dklyle has joined #openstack-nova15:48
lyarwoodgibi: awesome thanks15:48
gibimelwitt: could you babysitt ^^ during your day if needed?15:48
melwittgibi: yep15:49
gibithanks15:49
sean-k-mooneygibi: fungi raised an interesting point regarding the usefullness of lower constriants. personlly i only found it use full to detect when we use a new feature that is not in the version used by lower constraits15:49
sean-k-mooneyit has some other uses but do we want to discuss if we ant to keep supporting it15:49
gibisean-k-mooney: I don't have to much past knowledge about why we added lower constraints in the first place15:50
sean-k-mooneyin principal i do like the idea of tracking you min requirements and testing it but on the other hand i dont know if distos actully use this15:51
sean-k-mooneygibi: it was mainly for distos to tell them what the min version they could use was15:51
openstackgerritBalazs Gibizer proposed openstack/nova stable/victoria: lower-constraints: Bump packaging to 20.4  https://review.opendev.org/c/openstack/nova/+/76574915:51
gibiOK, then we need to see if distros are depending on this information or not15:51
gmannsean-k-mooney: gibi yeah that is very open thing to discuss on how we should test it and whether we should test it or not15:52
gmannone place it is helpful is while upgrade15:52
sean-k-mooneyonly partly15:52
melwittgibi: were you meaning to write more on this line or is it just a typo? "and bug" at the end https://review.opendev.org/c/openstack/nova/+/765749/7//COMMIT_MSG#1515:52
sean-k-mooneyknowing that there is an overlap is useful yes15:53
gibimelwitt: typo, I've pushed a new ps15:53
sean-k-mooneybut in a none conatienrised env it get complicated bwtween multipel serivces15:53
gibimelwitt: my editor moved the #<bugnumber> ot the beginning of the next line and git ignored that line as a comment :)15:53
melwittgibi: that was nice of it :)15:54
sean-k-mooney:) hehe helpful editors strike again15:54
gibiI'm not even mad, it is soo friday and soo many fires15:54
sean-k-mooneyya if that is the most pressing problem on a friday its not bad. hopefully the rest wont take you too long or can wait till monday15:57
gibisean-k-mooney: master is unblocked and the stable/v unblocking patch is going forward again. So yeah I will sleep well tonight15:59
melwittbauzas: could you take a look at https://review.opendev.org/c/openstack/nova/+/765749 ? gate unblock patch for stable/victoria has gotten a new patch squashed into it16:03
bauzasmelwitt: sure16:03
melwittthanks16:04
bauzasmelwitt: when squashing, should we add the cherry-picked SHA1s for all the squashed backports ?16:04
bauzasgibi: ^16:04
bauzasfortunately, we have the gerrit IDs within the commit msg, so it's more a procedural question16:04
gibibauzas: I can do that16:05
gibigive me 516:05
bauzasit's an open question, I dunno16:05
melwittgood question, I can't remember if we normally do. will do a quick gerrit search16:05
bauzaswe very rarely squash commits when backporting16:05
* bauzas looks at the stable docs16:05
bauzaseyes at https://docs.openstack.org/project-team-guide/stable-branches.html#proposing-fixes16:06
gibiit says cherry-pick with -x16:08
gibiand that does include that hash16:08
melwittyeah. I find examples of both in the gerrit search16:08
gibibauzas: if you feel that the commit hash is helpful then I can add that no problme16:08
bauzasI'm not *that* opiniated16:09
bauzasgibi: but if you feel OK with it, please do16:09
bauzasit doesn't harm16:09
* gibi does it16:09
bauzasthe more the better in this case16:09
gibioops, we need to merge16:12
gibi16:12
gibihttps://review.opendev.org/c/openstack/nova/+/76672016:12
gibito master first16:12
*** adrian-a has joined #openstack-nova16:12
gibias now it is part of the squashy commit on stable/v16:12
openstackgerritBalazs Gibizer proposed openstack/nova stable/victoria: lower-constraints: Bump packaging to 20.4  https://review.opendev.org/c/openstack/nova/+/76574916:13
gibibauzas: could you look at https://review.opendev.org/c/openstack/nova/+/766720  please?16:13
bauzasdone16:13
gibithanks16:13
*** adeberg has joined #openstack-nova16:27
*** ociuhandu_ has joined #openstack-nova16:28
*** ociuhandu has quit IRC16:31
openstackgerritLee Yarwood proposed openstack/nova stable/train: [stable-only] Cap bandit to 1.6.2 and raise hacking to 1.1.0 in LC  https://review.opendev.org/c/openstack/nova/+/76617116:32
*** ociuhandu_ has quit IRC16:32
lyarwoodokaas~.16:33
lyarwoodgah sorry ssh died16:34
openstackgerritLee Yarwood proposed openstack/nova stable/ussuri: [stable-only] lower-contraints: Bump hacking to 3.0.1 and amqp to 2.5.2  https://review.opendev.org/c/openstack/nova/+/76673816:45
lyarwood^ so I can't actually build these lower-constraint venvs locally thanks to virtualenv pulling in weird versions of setuptools16:45
lyarwoodI'm going to go afk for a few hours to deal with some childcare stuff and then attack this again later, with beer.16:46
sean-k-mooneylyarwood: you can contol the version fo setup tools with a flag16:47
fungisean-k-mooney: i did not mean to suggest that jobs testing lower-bounds of our (transitive) dependency sets wouldn't be useful, simply that we don't have (and haven't ever had) them. when they were originally proposed i pointed out the problem is intractable without implementing it in pip or duplicating a large portion of pip's routines, but folks decided to just punt because the (in this case16:50
fungiespecially incorrect) version selections pip failed to particularly enforce were considered "good enough"16:50
*** brinzhang_ has joined #openstack-nova16:50
*** songwenping__ has joined #openstack-nova16:51
*** brinzhang_ has quit IRC16:51
*** brinzhang_ has joined #openstack-nova16:52
sean-k-mooneyfungi: sure but i konw at least form a redhat downstream point of view lower constratis and upper constraits are not really used16:52
sean-k-mooneythey are useful but due to downstream backports and other issues outside of a pip based install they are not directly translateable to distro versions16:53
*** brinzhang_ has quit IRC16:53
*** brinzhang has quit IRC16:53
*** songwenping_ has quit IRC16:54
*** brinzhang_ has joined #openstack-nova16:54
fungithe global upper-constraints.txt wasn't introduced for the benefit of distributors. in was actually introduced to allow us to freeze the exact versions we're testing with so that they don't change in stable branches, but the supposition was that pip would be able to obey that list. it holds true for our normal jobs, but lower-constraints.txt files in projects aren't calculated from a known consistent16:56
fungiset16:56
sean-k-mooneyfungi: oh i know upper-constraits is not for distos16:56
fungiwe rely on pip to calculate our upper-constraints.txt file, and we can do that because we want new packages and pip prefers the latest version of any dependency which can satisfy everyone's requirements16:57
fungithe lower-constraints.txt files in projects are cobbled together by hand, are incomplete, and clearly inconsistent (i always suspected, but the dep solver in pip 2.3 has proven that now)16:57
sean-k-mooneyfungi: im just observing that i have seend requirement lower the lower constirant and higher then uppeor in our product16:57
sean-k-mooneydue to backprots and other reasons16:58
sean-k-mooneye.g. rhel shiping only one verion of a dep16:58
*** dtantsur|brb is now known as dtantsur16:58
sean-k-mooneyfungi: i still stongly belive in co instalablity and the benifit that upper-constrits provides16:59
sean-k-mooneybut without runnign tempest on lower-constriats i dotn really trust that they work17:00
fungiright, i don't think we ever expected to be the proxy for distributors testing their offerings. upper constraints allows us to declare what we tested with at any point in time. we attempt to track what versions of direct deps we think we're compatible with, but that's really just a best effort17:00
sean-k-mooneyfungi: we have seen our functional and unit test pass on py39 for example even thouhg nova dose not run on py 3917:00
sean-k-mooneyfungi: i assume other are aware that eventlet still does not work properly on py39 right17:02
fungii hope they're aware17:03
sean-k-mooneyfungi: slightly differnet topic i know just pointing out that while functional and unit test give me some confidence in the constriat lower or otherwise without tempest it really just a best guess that it shoudl work17:03
fungidebian's going to hit that pretty soon... 3.9 is the planned default for bullseye and release freeze starts first phases in february17:04
fungisean-k-mooney: yeah, and lower-bounds testing with devstack/tempest would also be entirely tractable if someone writes an inverse version sorter for pip17:05
sean-k-mooneyi think 3.9 will be the default for rhel/centos 9 too whenever that happens17:07
sean-k-mooneywell 3.9 or newer17:07
stephenfinIf someone reverts a resize, the expectation is that any data created on the instance between the resize and the revert will be lost, yeah?17:12
stephenfinI mean, I know that will happen given how it's implemented, but I wonder if we ever documented that quirk anywhere?17:12
sean-k-mooneyif it not boot form volume yes17:12
sean-k-mooneyor rbd17:13
sean-k-mooneyif its using a qcow we wont copy it back17:13
sean-k-mooneyfor rbd or bfv the data will be untoched and you wont lose anything17:13
stephenfinOh yeah. Boot from volume would be fine too17:14
* stephenfin closes https://bugs.launchpad.net/nova/+bug/190717917:14
openstackLaunchpad bug 1907179 in OpenStack Compute (nova) "resize revert will let new data lost!" [Undecided,Opinion]17:14
sean-k-mooneyone edgecase of note is what happens if we resize up then revert down for rbd?17:15
sean-k-mooneye.g. if we go form 100G to 200G17:15
sean-k-mooneywill the rbd volume shrink?17:15
sean-k-mooneyfor bfv you the root gb shoudl be ignored( well it shoudl be 0) but for RBD its used17:16
sean-k-mooneyif rbd grows the voluem which it should we cant safely shrink it on revert17:16
stephenfinHmm, that's a fair point. I haven't a clue tbh. Resize revert is a bit of an ugly duckling imo17:16
sean-k-mooneyunless we snapshot it17:16
sean-k-mooneyya17:16
stephenfinI suspect very few people, if any, use it regularly17:16
sean-k-mooneyya, i have suggested removing it in the pastand the confrim step. for example in my recreate api propsosal17:17
sean-k-mooneybut not sure we can really do that17:18
sean-k-mooneyin the existing apis17:18
stephenfinNo really. The config option to auto-confirm after a set time is as close to that as we can get17:18
stephenfinImplementation details leaking out of the API, ftw17:18
openstackgerritMerged openstack/nova-specs master: Re-propose "Secure Boot support for KVM & QEMU guests" for Wallaby  https://review.opendev.org/c/openstack/nova-specs/+/75973117:18
sean-k-mooneyhttps://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.resize_confirm_window17:19
sean-k-mooneyit would be nice if we could add a -1 to that17:20
sean-k-mooneye.g. do it imedietly17:20
sean-k-mooneynormally i would call that 0 but that is taken17:20
sean-k-mooneythe best you can do now is set it to 1 second17:20
stephenfinGood idea17:20
stephenfinOn another bug, you can't change the AZ of the host when there are instances on it, right?17:21
sean-k-mooneycorrect17:21
stephenfinWe should probably have an API check to validate that17:21
sean-k-mooneywe do17:21
stephenfinoh17:22
sean-k-mooneyits not backported to 13/train17:22
stephenfinIs it recent?17:22
stephenfinhttps://bugs.launchpad.net/nova/+bug/190777517:22
openstackLaunchpad bug 1907775 in OpenStack Compute (nova) "After change Host's Availability Zone, the deployed VMs Availability Zone data(in novaapi.request_specs table) can not be updated" [Undecided,New]17:22
sean-k-mooney*queens17:22
sean-k-mooneystephenfin: ill see if i can find it mriadem worte it i think17:22
stephenfinno need, I'm looking17:22
sean-k-mooneyhttps://github.com/openstack/nova/blob/048250a4b76c0b6a85e459f92b436c448d592235/nova/compute/api.py#L606617:25
stephenfinyou're faster than me17:25
sean-k-mooneyi knew roughly where to look17:25
sean-k-mooneyhttps://github.com/openstack/nova/commit/0ad5a64dc9ac4b1cfb8038f9171b6385fdf07f2817:26
stephenfinhmm, I think that's different. That prevents us renaming an aggregates AZ17:26
sean-k-mooneyhttps://github.com/openstack/nova/commit/8e19ef4173906da0b7c761da4de0728a2fd71e2417:27
sean-k-mooneysorry ^17:27
stephenfinbut it doesn't prevent us removing a host from an aggregate (and therefore the AZ)17:27
*** hamalq has joined #openstack-nova17:27
*** hamalq has quit IRC17:28
sean-k-mooneywe do it for add https://github.com/openstack/nova/blob/048250a4b76c0b6a85e459f92b436c448d592235/nova/compute/api.py#L616417:29
*** hamalq has joined #openstack-nova17:29
sean-k-mooneystephenfin: we dont do it for remove17:29
*** k_mouza_ has quit IRC17:30
sean-k-mooneystephenfin: we should here ish https://github.com/openstack/nova/blob/048250a4b76c0b6a85e459f92b436c448d592235/nova/compute/api.py#L621117:30
sean-k-mooneyi guess line 6217 actully to be consitent17:31
sean-k-mooneyand do it after the notificaion17:31
*** dave-mccowan has joined #openstack-nova17:32
sean-k-mooneystephenfin: so that bug is half valid. what they are doing is not supported but we are not blocking the removal17:32
sean-k-mooneyit should be 1 line + tests however17:32
stephenfinagreed17:32
stephenfinI'm just creating a test now17:33
stephenfinEasy win17:33
*** gyee has joined #openstack-nova17:33
sean-k-mooneyi wonder why we did add but not remove in the first place17:33
stephenfinnot sure. It's very strange17:33
stephenfinsurprised no one has noticed before either17:33
sean-k-mooneypeople dont do this often and we have a giant warning in the docs17:35
*** tesseract has quit IRC17:35
stephenfinwe do?17:36
stephenfinabout what? messing with AZs on hosts with instances?17:37
sean-k-mooneyya i think spcificaly about moving host17:37
sean-k-mooneyhttps://docs.openstack.org/nova/latest/admin/availability-zones.html#implications-for-moving-servers17:39
sean-k-mooneyits got its own section17:40
sean-k-mooneyactully that not quite what i was thinking of17:40
sean-k-mooneythat partly covers it but i though we had addtion info specific to move compute nodes17:42
sean-k-mooneyhum i guess we dont17:43
*** derekh has quit IRC17:44
*** tosky has quit IRC17:46
stephenfinsean-k-mooney: should you be allowed to add a host to an aggregate when it has instances on it?17:50
sean-k-mooneyno because if its not in an az it would be in an implcit nova az right17:50
stephenfingood point17:51
sean-k-mooneyso that would result itn the instance moving az17:51
stephenfinI was going to say it won't be in an AZ, but it will. The implicit one, as you note17:51
sean-k-mooneyit might be ok provide they dont have an explict nova request17:51
stephenfinwe tell people not to do that, iirc17:51
sean-k-mooneyyep here https://docs.openstack.org/nova/latest/admin/availability-zones.html17:52
sean-k-mooneythe warning at the top17:52
sean-k-mooneybut thats the edgecase you are hitting if you allow it17:52
*** ralonsoh has quit IRC18:00
JamesBensonhey sean-k-mooney, one follow up question.  Is there a setting (or benefit) to passing all of the CPU flags in the nova.conf in that mixed CPU environment we chatted about yesterday?18:00
sean-k-mooneyin your case i guess you could18:02
sean-k-mooneysince you are enableing flags not on the older host on your newer host you wont be able to migrate form westmer to nehelem anyway18:03
sean-k-mooneyyou can move the other way18:03
sean-k-mooneynehelem to westmer18:03
sean-k-mooneyso you could just use hostmodel or host passthough18:03
sean-k-mooneynormally you dont want ot do that if you have a mixed env18:04
openstackgerritStephen Finucane proposed openstack/nova master: functional: Add reproducer for #1907775  https://review.opendev.org/c/openstack/nova/+/76677118:04
stephenfinsean-k-mooney: ^18:06
sean-k-mooneyhehe you had to reformat things :P18:09
sean-k-mooneyso does that actuly work for both add and remove today18:09
sean-k-mooneyi was expect add to be blocked18:10
sean-k-mooneystephenfin: hum they all pass so ya i guess its not blocked on add currently18:12
*** dave-mccowan has quit IRC18:12
*** dave-mccowan has joined #openstack-nova18:16
*** dtantsur is now known as dtantsur|afk18:17
*** dave-mccowan has quit IRC18:20
*** k_mouza has joined #openstack-nova18:20
*** k_mouza has quit IRC18:21
*** k_mouza_ has joined #openstack-nova18:21
*** dave-mccowan has joined #openstack-nova18:21
*** jangutter has joined #openstack-nova18:23
*** andrewbonney has quit IRC18:24
*** k_mouza_ has quit IRC18:26
*** jangutter_ has quit IRC18:27
*** jangutter has quit IRC18:40
*** jangutter has joined #openstack-nova18:41
*** jangutter has quit IRC18:41
*** adrian-a has quit IRC18:41
*** jangutter has joined #openstack-nova18:42
*** xek has quit IRC19:17
*** smcginnis has left #openstack-nova19:25
*** mgariepy has quit IRC19:58
openstackgerritLance Bragstad proposed openstack/placement master: Implement secure RBAC for resource providers  https://review.opendev.org/c/openstack/placement/+/76024020:02
*** dave-mccowan has quit IRC20:03
*** jangutter has quit IRC20:05
*** jangutter has joined #openstack-nova20:06
*** adrian-a has joined #openstack-nova20:06
openstackgerritLance Bragstad proposed openstack/placement master: Implement secure RBAC for resource providers  https://review.opendev.org/c/openstack/placement/+/76024020:06
openstackgerritLance Bragstad proposed openstack/placement master: Pass context objects to oslo.policy directly  https://review.opendev.org/c/openstack/placement/+/76678520:06
*** dave-mccowan has joined #openstack-nova20:07
*** lemko3 has joined #openstack-nova20:14
*** lemko has quit IRC20:14
*** lemko3 is now known as lemko20:14
openstackgerritBalazs Gibizer proposed openstack/nova master: Reproduce bug 1907522 in functional test  https://review.opendev.org/c/openstack/nova/+/76679120:44
openstackbug 1907522 in OpenStack Compute (nova) "test_*_with_qos_min_bw_allocation fails in the nova-multi-cell job with: nova.exception.MigrationPreCheckError: Migration pre-check error: Failed to create port bindings for host <host>" [High,Triaged] https://launchpad.net/bugs/1907522 - Assigned to Balazs Gibizer (balazs-gibizer)20:44
*** nweinber has quit IRC21:01
*** tosky has joined #openstack-nova21:05
*** martinkennelly has quit IRC21:35
*** martinkennelly has joined #openstack-nova21:36
openstackgerritArtom Lifshitz proposed openstack/nova master: WIP: Add 'socket' to NUMACell and PciDevice  https://review.opendev.org/c/openstack/nova/+/76679521:38
openstackgerritArtom Lifshitz proposed openstack/nova master: WIP: Start populating `socket_id` in host NUMA topology  https://review.opendev.org/c/openstack/nova/+/76681621:38
*** adrian-a has quit IRC21:43
*** adrian-a has joined #openstack-nova21:43
*** martinkennelly has quit IRC21:47
*** martinkennelly has joined #openstack-nova21:48
*** k_mouza has joined #openstack-nova21:48
*** k_mouza has quit IRC21:50
*** martinkennelly has quit IRC21:52
*** vishalmanchanda has quit IRC22:03
*** k_mouza has joined #openstack-nova22:18
*** k_mouza has quit IRC22:22
*** dave-mccowan has quit IRC22:31
*** bbowen has quit IRC22:38
*** bbowen has joined #openstack-nova22:38
*** bbowen has quit IRC22:38
openstackgerritMerged openstack/nova master: nova-evacuate: Remove leftover playbook from standalone job  https://review.opendev.org/c/openstack/nova/+/76461822:42
*** adrian-a has quit IRC22:55
*** k_mouza has joined #openstack-nova23:35
*** k_mouza has quit IRC23:48
openstackgerritMerged openstack/nova master: nova-grenade-multinode: Skip test_live_block_migration_paused  https://review.opendev.org/c/openstack/nova/+/76672023:51
*** k_mouza has joined #openstack-nova23:52
« Thursday, 2020-12-10 Index Saturday, 2020-12-12 »

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