Friday, 2019-05-17

imacdonnnot sure what what basic_publish_confirm message really means00:02
melwittI don't either but just the indication that it made an attempt to send the message00:02
melwittI guess that implies it for some reason couldn't get through and results in the timeout?00:03
imacdonnbasic_publish_confirm is the method (that's supposed to get autoretry'ed), I guess00:05
imacdonnI think it comes from python-amqp (?)00:06
*** zer0c00l_ has quit IRC00:06
imacdonnFile "/usr/lib/python2.7/site-packages/amqp/channel.py", line 1787, in basic_publish_confirm00:07
imacdonnit does seem like it calls it the second time (at 23:48:19.113), but  apparently still doesn't get an answer00:10
*** slaweq has joined #openstack-nova00:11
melwittyeah, that's the one I was looking at, the second that times out even after re-establishing the connection00:12
openstackgerritDustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for setting instance id  https://review.opendev.org/65969000:13
openstackgerritDustin Cowles proposed openstack/nova master: WIP: Use SDK instead of ironicclient for add/remove instance info from node  https://review.opendev.org/65969100:15
*** slaweq has quit IRC00:16
*** zer0c00l has joined #openstack-nova00:16
*** zer0c00l has quit IRC00:17
*** zer0c00l has joined #openstack-nova00:17
*** Guest28904 has quit IRC00:34
openstackgerritMerged openstack/nova master: Stop logging traceback when skipping quiesce  https://review.opendev.org/65937400:34
*** nicolasbock has quit IRC00:49
*** tbachman has joined #openstack-nova01:00
imacdonnanother data-point: the retried request DOES actually get sent, because I see the compute node act upon it ... I guess somehow the client (nova-api) has some issue with the response01:08
*** hongbin has joined #openstack-nova01:11
*** slaweq has joined #openstack-nova01:11
*** mmethot has quit IRC01:12
*** guozijn has joined #openstack-nova01:15
*** slaweq has quit IRC01:15
*** ricolin has joined #openstack-nova01:22
*** ricolin has quit IRC01:30
*** brinzhang has joined #openstack-nova01:44
*** mmethot has joined #openstack-nova01:55
*** ricolin has joined #openstack-nova01:56
*** ricolin has quit IRC02:04
*** tinwood has quit IRC02:10
*** slaweq has joined #openstack-nova02:11
*** tinwood has joined #openstack-nova02:12
*** whoami-rajat has joined #openstack-nova02:12
*** slaweq has quit IRC02:15
*** JamesBenson has joined #openstack-nova02:31
openstackgerritchenker proposed openstack/python-novaclient master: Optimize limit option docs string description for novaclient  https://review.opendev.org/64303503:02
openstackgerritGuo Jingyu proposed openstack/nova master: Skip existing VMs when hosts apply forceconfig_drive  https://review.opendev.org/65970303:03
*** guozijn has quit IRC03:04
*** guozijn has joined #openstack-nova03:06
*** ttsiouts has joined #openstack-nova03:08
*** yonglihe has joined #openstack-nova03:12
*** cfriesen has quit IRC03:13
*** awalende has joined #openstack-nova03:13
openstackgerritGuo Jingyu proposed openstack/nova master: Skip existing VMs when hosts apply force_config_drive  https://review.opendev.org/65970303:17
*** awalende has quit IRC03:17
*** hongbin has quit IRC03:24
*** guozijn has quit IRC03:27
*** gaoyan has joined #openstack-nova03:28
*** guozijn has joined #openstack-nova03:30
*** ttsiouts has quit IRC03:41
*** psachin has joined #openstack-nova03:41
*** lbragstad has quit IRC03:53
*** JamesBenson has quit IRC03:59
*** ratailor has joined #openstack-nova04:01
*** slaweq has joined #openstack-nova04:11
*** slaweq has quit IRC04:16
*** gaoyan has quit IRC04:31
*** udesale has joined #openstack-nova04:34
*** ykarel has joined #openstack-nova04:39
ykarelTheJulia, dtantsur|afk can you check comment in https://review.opendev.org/#/c/653279/104:46
ykarelhmm looks like some issue is already there, as i can see https://review.opendev.org/#/c/659612/1, can you check if this ^^ is same issue or something else04:48
*** bhagyashris has joined #openstack-nova04:54
*** slaweq has joined #openstack-nova05:11
*** gaoyan has joined #openstack-nova05:14
*** slaweq has quit IRC05:16
*** pcaruana has joined #openstack-nova05:20
*** brinzhang has quit IRC05:21
*** brinzhang has joined #openstack-nova05:22
*** slaweq has joined #openstack-nova05:24
openstackgerritGuo Jingyu proposed openstack/nova master: Add rfb.VNC support for novncproxy  https://review.opendev.org/62233605:25
openstackgerritGuo Jingyu proposed openstack/nova-specs master: Proposal for a safer noVNC console with password authentication  https://review.opendev.org/62312005:26
*** gaoyan has quit IRC05:29
*** ttsiouts has joined #openstack-nova05:39
*** slaweq has quit IRC05:45
*** guozijn has quit IRC05:50
*** guozijn has joined #openstack-nova05:50
*** ivve has quit IRC05:50
*** Luzi has joined #openstack-nova05:52
*** lpetrut has joined #openstack-nova06:05
*** faizy98 has quit IRC06:07
*** sapd1_x has joined #openstack-nova06:10
*** ttsiouts has quit IRC06:12
openstackgerritGuo Jingyu proposed openstack/nova master: Skip existing VMs when hosts apply force_config_drive  https://review.opendev.org/65970306:21
*** maciejjozefczyk has joined #openstack-nova06:27
*** belmoreira has joined #openstack-nova06:32
*** guozijn has quit IRC06:39
*** guozijn has joined #openstack-nova06:40
*** ykarel is now known as ykarel|afk06:41
*** slaweq has joined #openstack-nova06:52
*** belmoreira has quit IRC06:56
*** ivve has joined #openstack-nova06:58
*** luksky has joined #openstack-nova06:59
*** tesseract has joined #openstack-nova07:08
*** awalende has joined #openstack-nova07:11
*** awalende has quit IRC07:13
*** ykarel|afk is now known as ykarel07:16
*** awalende has joined #openstack-nova07:17
*** rcernin has quit IRC07:19
*** jaosorior has quit IRC07:19
*** rpittau|afk is now known as rpittau07:22
*** ccamacho has joined #openstack-nova07:27
*** dtantsur|afk is now known as dtantsur07:31
*** xek has joined #openstack-nova07:43
*** luksky has quit IRC07:45
*** mcgigglier has joined #openstack-nova07:47
*** belmoreira has joined #openstack-nova07:53
*** belmoreira has quit IRC07:57
*** ykarel is now known as ykarel|lunch08:00
kashyapaspiers: That's in queue, I'm still backlogged :-(  I know it is ready to merge.  If I look at, I would like to spend a 2-hour block on it.  So trying to find that :P08:01
kashyapaspiers: From your comment:08:01
kashyap"I don't quite understand how the fix in https://www.redhat.com/archives/libvir-list/2019-April/msg01418.html would make the Python arguments optional however, since they're still missing defaults."08:01
*** ttsiouts has joined #openstack-nova08:01
kashyapaspiers: Just write to the upstream list (no subscription required; you can ask to be Cced)08:01
kashyapaspiers: But as you noted yourself it's not required, given that you've obtained correct value for @emulatorbin via "Parse <emulator> elements from virConnectGetCapabilities()"08:02
kashyapaspiers: Actually, let me give my review.  You've addressed all my concerns08:03
*** lpetrut has quit IRC08:09
*** ralonsoh has joined #openstack-nova08:12
*** belmoreira has joined #openstack-nova08:13
*** luksky has joined #openstack-nova08:19
*** sapd1_x has quit IRC08:21
aspierskashyap: I'm backlogged too ;-)08:24
*** awalende has quit IRC08:24
aspierskashyap: So I probably owe you some reviews too. Let me know if I can make the review process easier in any way.08:24
kashyapaspiers: Thanks for the offer!  I need to do some work similar to yours -- introducing new CPU-related APIs08:26
kashyapaspiers: For this spec: https://opendev.org/openstack/nova-specs/src/branch/master/specs/train/approved/cpu-selection-with-hypervisor-consideration.rst08:26
kashyapSee the action items at the end.08:27
*** ccamacho has quit IRC08:27
*** wxy-xiyuan has left #openstack-nova08:27
aspiersok08:28
*** lpetrut has joined #openstack-nova08:29
*** derekh has joined #openstack-nova08:32
*** awalende has joined #openstack-nova08:37
*** tssurya has joined #openstack-nova08:38
*** tkajinam has quit IRC08:44
kashyapaspiers: ACKed your change, FWIW08:45
aspierskashyap: yep, thanks!08:45
kashyapaspiers: I think your change is ready to merge.  Let's check with efried (or stephenfin) when they're about.08:45
kashyapaspiers: Hope it won't be delayed, so that you can maintain the momentum.08:46
aspiersWell it's already been delayed a few months ;) but thankfully git makes this much less of an issue. I remember the awful CVS days where the only place for all your pending unmerged work was mushed up together in the work tree, and you had to untangle each new commit-to-be from that08:49
aspiersDrove me crazy08:49
*** ykarel|lunch is now known as ykarel08:50
*** davidsha has joined #openstack-nova08:53
jangutteraspiers: The absolute _best_ thing about git is when you learn about rebasing. And the nice thing about gerrit is it fills in the gaps (tracking iterations of the same "patch queue").08:56
aspiersjangutter: agreed :) git has lots of "best" things though, e.g. I remember when I learnt about the reflog it blew my mind08:56
aspiersditto git-rerere08:56
aspiersso many cool tricks08:57
jangutteraspiers: who knew that a content-addressable file system accidentally solved the "version control" issue :-p08:58
aspiers:D08:58
*** ccamacho has joined #openstack-nova08:59
*** ttsiouts has quit IRC09:04
*** belmoreira has quit IRC09:08
*** belmoreira has joined #openstack-nova09:10
*** panda|rover|off is now known as panda|rover09:10
*** sapd1_x has joined #openstack-nova09:13
*** wwriverrat has quit IRC09:28
*** bhagyashris has quit IRC09:36
*** sapd1_x has quit IRC09:36
*** belmoreira has quit IRC09:47
*** awalende has quit IRC09:49
*** sapd1_x has joined #openstack-nova09:49
*** awalende has joined #openstack-nova09:49
*** rcernin has joined #openstack-nova09:50
*** awalende has quit IRC09:50
*** mdbooth has quit IRC09:50
*** awalende has joined #openstack-nova09:50
*** belmoreira has joined #openstack-nova09:51
*** mdbooth has joined #openstack-nova09:56
*** sapd1_x has quit IRC09:59
*** belmoreira has quit IRC10:00
*** ykarel is now known as ykarel|afk10:03
kashyapaspiers: Yeah, I see what you mean.  I didn't have the misfortune to use CVS, but I did start with SVN, though10:15
aspiersI started with RCS :-o10:16
*** belmoreira has joined #openstack-nova10:21
*** guozijn has quit IRC10:23
*** awalende has quit IRC10:24
*** brinzhang has quit IRC10:28
*** cdent has joined #openstack-nova10:30
kashyapHope you've fully recovered10:34
aspiersLOL10:35
aspiersActually RCS can be surprisingly effective for particular use cases10:35
aspiersbut yeah, it's very limited10:35
aspiersI actually talked a bit about the evolutionary history of version control systems in this interview http://episodes.gitminutes.com/2015/03/gitminutes-32-adam-spiers-on-git-deps.html10:36
aspiersaccording to the index that starts around 07:0810:37
*** whoami-rajat has quit IRC10:42
*** tbachman has quit IRC10:47
*** awalende has joined #openstack-nova10:48
*** sridharg has joined #openstack-nova10:48
*** belmoreira has quit IRC10:49
cdentWhat are the options for a VM after it is evacuated? I ask in relation to this new bug: https://bugs.launchpad.net/nova/+bug/182947910:52
openstackLaunchpad bug 1829479 in OpenStack Compute (nova) "The allocation table has residual records when instance is evacuated and the source physical node is removed" [Undecided,New]10:52
*** jaosorior has joined #openstack-nova10:55
*** ykarel|afk is now known as ykarel10:56
*** jaosorior has quit IRC11:00
*** ttsiouts has joined #openstack-nova11:01
*** guozijn has joined #openstack-nova11:01
*** jaosorior has joined #openstack-nova11:09
*** jcosmao has joined #openstack-nova11:18
sean-k-mooneycdent: in what sense11:20
sean-k-mooneycdent: if it is evacuated sucessfully its as if ti was migrated11:20
sean-k-mooneycdent: assuming it was on shared storage its state was not even lost11:21
sean-k-mooneycdent: althogh if it was on epheraml stoage it wasd effectivly a new instance11:21
cdentsean-k-mooney: So evacuate means "recreate this vm somewhere else"?11:21
sean-k-mooneycdent: yes11:21
sean-k-mooneybut keep the same ips and volumes11:21
cdentSo shouldn't that mean that when the evacuate finishes an allocation is made which replaces the old allocations?11:22
artomRebuild on a different host, basically11:22
artomSo yeah, allocations need to move11:22
sean-k-mooneyso if its boot from volume or you hapeend to have the rbd image backend you keep you root disk11:22
aspiersPLEASE can we use the word "resurrect" instead of evacuate wherever possible11:22
sean-k-mooneyaspiers: no11:22
sean-k-mooney:P11:22
artomaspiers, time to introduce you to this classic: http://www.danplanet.com/blog/2016/03/03/evacuate-in-nova-one-command-to-confuse-us-all/11:23
aspiersthe word "evacuate" implies the VM is still there, but in these cases it is not11:23
sean-k-mooneyyep ^11:23
aspiersartom: I read that years ago ;-)11:23
sean-k-mooneyaspiers: it is for things with ephemeral storage11:23
sean-k-mooney* non ephmeral storage11:23
sean-k-mooneyaspiers: its basically just a cold migration for that case11:23
*** awalende has quit IRC11:23
aspiersthe implementation is like cold migration, but semantically it is not11:24
sean-k-mooneyits when you use the default image backend e.g. qcow2 and its not on shared stoage that we acrully recreate it from glance and loose data11:24
cdentsean-k-mooney: so in that case either the person at bug https://bugs.launchpad.net/nova/+bug/1829479 is on an old version (as I thought we had fixed migration-like allocations bugs) or we've got remaining migration-like allocations bugs11:24
openstackLaunchpad bug 1829479 in OpenStack Compute (nova) "The allocation table has residual records when instance is evacuated and the source physical node is removed" [Undecided,New]11:24
sean-k-mooneycdent: we proably have a remaing bug11:25
aspiersartom: here's another "classic" for you ;-) https://youtu.be/lddtWUP_IKQ?t=75111:26
sean-k-mooneycdent:  im not sure if we actullly do the whole create a migration contex use it for allocation and swap at the end11:26
artomaspiers, oh, hah11:26
aspierssean-k-mooney, cdent: that video will help you understand where I'm coming from too11:26
artomaspiers, this is like the time I naively asked about your experience with Gerrit, isn't it :P11:27
aspiersartom: haha, maybe a bit ;-)11:27
* artom stops talking until coffee11:27
aspiersI've been working on compute HA in nova for about 4-5 years11:27
sean-k-mooneycdent: or it could be that we delete the old allocation on the source node and that does not run in the evacuate case11:27
aspiersartom: but don't feel bad, there's no good reason why you should have known that ;)11:27
cdentthat's what the bug report is saying, yes11:28
sean-k-mooneyaspiers: that was a poor life choice :P11:28
aspierssean-k-mooney: LOL, yeah probably ;-)11:28
artomI see to have a lot of feet in my mouth, maybe wash them out with aforementioned coffee11:28
artom*seem11:28
aspierssean-k-mooney: I mean, not *exclusively* compute HA ... I've done a lot of other stuff too ;)11:28
sean-k-mooneyaspiers: so for https://review.opendev.org/#/c/643578/3 back to the ptg session your prefere is openstack server resuerect11:29
aspierssean-k-mooney: yes exactly11:29
aspiers"evacuate" and "migrate" both imply that everything is currently running (mostly) fine11:30
artomWell11:30
aspiers"resurrect" implies (correctly) "oh shit, something already blew up, so let's clean up after it"11:30
artomThe compute service/control plane has to be down11:30
artomBut the data plane? IIUC the instance can be running, no?11:31
sean-k-mooneyaspiers: ya that why i called the new command recreate but i can update it to resurect11:31
* artom hasn't actually checked whether we make sure to kill the instance first11:31
sean-k-mooneyi spell recreate more consitenly however but i can live with that11:31
aspiersartom: which scenario are you talking about exactly? there are a few here11:31
artomaspiers, evacuate11:31
sean-k-mooneyartom: we dont11:31
sean-k-mooneyartom: but we require that you have marked the compute node as disabled before we allow it11:32
aspiersartom: in this context you're gonna have to use more than one word to describe the scenario sorry ;-)11:32
aspiersthe whole problem is the ambiguity11:32
sean-k-mooneyartom: and we require the operator to confim that it is dead before the evac11:32
aspierssean-k-mooney: actually not always11:32
*** janki has joined #openstack-nova11:32
sean-k-mooneyaspiers: that is the api contract11:32
aspierssean-k-mooney: OOB fencing can *ensure* that the VM is dead11:32
sean-k-mooneysure11:32
sean-k-mooneybut the api contract is only call this if it is dead11:33
aspierssean-k-mooney: i.e. it can be automated. but yes we're on the same page I think11:33
aspiersexactly11:33
aspiersartom: there are multiple failure scenarios here to consider11:33
sean-k-mooneyif you violate that contract and you get 2 instance trying to read/write to the same non multi atach volume some how we just say "have fun" and walk away smiling11:34
*** ttsiouts has quit IRC11:34
aspierssean-k-mooney: exactly, that is the main message of 3 talks I have given on this11:34
openstackgerritMiguel Ángel Herranz Trillo proposed openstack/nova master: Fix type error on call to mount device  https://review.opendev.org/65978011:34
aspiersin Austin, Boston, and OpenStack Day Israel11:35
aspierse.g. https://youtu.be/lddtWUP_IKQ?t=36611:35
artomaspiers, evacuate the nova client command11:35
aspiers(because exploding kittens is fun)11:35
artomAnd my thought was with respect to resurrect, that the *instance* might not be dead, even if it's unmanageable because the host it's on *is* dead11:36
sean-k-mooneyill convert https://review.opendev.org/#/c/643578/3 to resurect and reupload it proably on monday11:36
artomSo "resurrect" might not be the best word11:36
* artom sheds a bike11:36
*** rcernin has quit IRC11:36
aspierssean-k-mooney: ++11:36
sean-k-mooneyartom: well its like if your heart stoped11:36
aspiersartom: as sean-k-mooney said, the API contract is that the instance *must* be dead before calling the API11:36
artomaspiers, ah, didn't know that, ok then11:37
sean-k-mooneyyou were technically dead for a bit and this operation is like a defibulator11:37
aspiersartom: that's why the force_down API was introduced ages ago - to get nova to realise quicker that it's dead, rather than waiting for its own RPC timeout11:37
sean-k-mooneyactully not just for that usecase11:38
aspiersOK, that was one use case at least11:38
sean-k-mooneythere were case where for example the root disk and the disk where the vms images were seperate11:38
sean-k-mooneyso the agent and libvirt could be fine but when the backing disk died for the image and they crashed you wanted to be able to mark the host as down11:39
*** awalende has joined #openstack-nova11:39
sean-k-mooneyor similar for network issues11:39
aspiersright11:39
sean-k-mooneyaspiers: were you working with emma foley on the collectd/doctor/vitriage demoes?11:40
aspiersartom: https://youtu.be/lddtWUP_IKQ?t=648 covers some of the failure modes, but you and sean-k-mooney are both right that there are other failure modes based on different network failures11:40
aspierssean-k-mooney: nope not heard of that, but with my self-healing SIG hat on, I'd really like to learn more about it!11:41
aspierssean-k-mooney: got any URLs?11:41
sean-k-mooneythere have been at least 3 summit presentation on it11:41
sean-k-mooneythe effort was driven via opnfv11:41
aspiersdamn, thought I'd been to all the vitrage presentations11:41
aspiersso I'm just looking through my old decks and found this https://aspiers.github.io/openstack-day-israel-2017-compute-ha/#/comparison11:43
aspiershad forgotten about the "needs improvements in nova" bit11:43
aspierstrying to remember the details of that11:43
aspiersI think it was around the reliability of the evacuate API11:44
*** jaosorior has quit IRC11:44
cdentthat's some can of worms I opened11:44
aspiersbad cdent!11:45
aspierskeep quiet in future ;-)11:45
cdentunpossible!11:45
aspiersj/k, glad to be reminded about this stuff11:45
cdentI hope that somehow this leads to that bug getting more attention. people get squeamish about orphaned providers11:45
cdentespecially because we have that damn uniq constraint on rp names11:46
cdentwhich is the source of a ton of support requests11:46
cdent"I was messing around and one of my compute nodes got rejiggered and it says can't create a resource provider"11:46
cdentturns out the old one with the same name still exists11:46
aspierscdent: sorry to hijack your discussion with a big tangent11:47
* cdent is a fan of tangents11:48
*** yan0s has joined #openstack-nova11:48
aspierssean-k-mooney, artom: I think this 4-year-old blog post is still very relevant http://blog.clusterlabs.org/blog/2015/living-intersection-pets-cattle11:48
cdentI tend to think the reason tangents happen is because there is insufficient shared understanding and that the tangents are necessary to get it11:48
aspiersyeah11:48
cdentbut there is so little shared understanding in nova that is sometimes seems like we are on a tangent all the time11:49
aspiershaha true11:49
cdenttangents on tangets on tangents11:49
aspiersbeekhof and I did a lot of work a few years back considering all the failure cases of nova compute HA11:49
aspiershe was definitely the thought-leader in this space, but he escaped to greener pastures11:49
sean-k-mooneyisnt a tangent to a tangent called a normal11:50
sean-k-mooneynova embrases this :P11:50
aspiersif a tangent is a 90 degree turn, then 4 tangents get you back in the original direction, right?11:50
sean-k-mooneyaspiers: for what its worth a lot of the presentation i was refering to were framed in the context fo service assurance not self healing or instance ha11:51
aspierssean-k-mooney: got it11:52
cdentat the limit, N tangents is a circle, no matter the angle of turn11:52
cdentor something11:52
aspiers:D11:52
aspiershttps://aspiers.github.io/openstack-day-israel-2017-compute-ha/#/nova-host-alerter is the future converged architecture beekhof and I agreed on, in case anyone is interested11:52
sean-k-mooneycdent: are you suggesting we would go around in circles on things, im shocked :)11:53
aspierscoupled with http://blog.clusterlabs.org/blog/2015/reliable-notifications and masakari adding a layer of retries on top of nova's somewhat lossy workflow, the hope was that this would be more robust11:53
aspierss/more/sufficiently/11:53
sean-k-mooneyanyway i better file my expense report for the ptg or finance will get annoyed at me11:54
aspierssean-k-mooney: IIRC one of the concerns was that if scheduler or conductor crashed while handling evacuate API, it would get lost forever - is that still true?11:54
cdentsean-k-mooney: hush11:54
sean-k-mooneyaspiers: ya maybe11:54
*** hamzy has quit IRC11:55
sean-k-mooneyit woudl depend on when it failed i guess11:55
sean-k-mooneyif we have not deleted it form the souce node yet then you should be fine11:55
aspiersI think the idea was maybe that masakari could reinvoke the API in those circumstances11:55
sean-k-mooneyya i think if the db says its still on the source node then you can evac again11:55
aspiersbut is there even a way to query the API for the state of an evacuate workflow?11:55
sean-k-mooneybut depending on the failure mode the vm could be paused/running on another node at that point11:56
aspiersI vaguely remember that being discussed at past summits11:56
aspiersyeah, the devil is in the details I guess11:56
aspiersI don't think there's ever been enough corporate sponsorship from RH or SUSE to spend time really cleaning up all those corner cases11:57
sean-k-mooneyif your contol plane if failing you shoudl really stop what your doing and fix that first then go back to fixing your broken compute nodes11:57
aspiersoh sure, that's a totally different kettle of fish11:57
* artom imagines fish being boiled alive11:58
aspierscompute HA is impossible without a resilient control plane11:58
aspierssame for most workflows11:58
aspierssean-k-mooney: mriedem makes a good point on https://review.opendev.org/#/c/643578/3//COMMIT_MSG that resurrect is an admin-only command, so "openstack server resurrect" could mislead non-admin users into thinking they can call it12:01
aspiersOTOH I don't agree with his statement "evacuate moves the server to another host (actually it rebuilds it on another host)"12:01
sean-k-mooneyaspiers: migrate is an admin command too12:02
sean-k-mooneyand that is openstack server migrate12:02
aspierssince a) you can't "move" a server which has already died, and b) it only rebuilds if it was using ephemeral storage12:02
sean-k-mooneythe only non admin migration is resize12:02
aspierssean-k-mooney: right, that's why he was suggesting nesting under "openstack server migrate"12:03
sean-k-mooneyno12:03
sean-k-mooneymigrate implies it will preserve you data12:03
aspiersbut I really don't like that cos resurrect is not a form of migration12:03
sean-k-mooneyso that would be worse then evac12:03
*** awalende has quit IRC12:03
aspiersyes, I think we are saying the same thing in different ways :)12:03
sean-k-mooneywhat im saying is migrate is not an option because migrate implies we move data and we dont in the most common form for evac12:04
sean-k-mooneymost deployment have an ephermeal root disk12:04
aspiersyes, that's one good reason to avoid migrate12:04
aspiersand it's not the only one12:04
aspiersat a higher level of abstraction, what you are saying could be reframed as "migrate" implies moving something from A to B without significantly changing it12:05
sean-k-mooneywe also discussed having two commands one that guarnetees it preserves your data and one tha that always destroys it12:05
*** awalende has joined #openstack-nova12:05
*** tbachman has joined #openstack-nova12:06
aspiersbut you start with something dead on A, and end with something alive on B, that is a big change in state12:06
sean-k-mooneye.g. recreate woudl always destoy any instance and recreate it loasing data12:06
aspierstherefore does not deserve to be called a form of migration12:06
artomData destruction/preservation also depends on where said data is12:07
sean-k-mooneyopenstack server migrate --evacuate would guarentte that your data was moved or would not try to do dit12:07
aspierssean-k-mooney: yeah that might be a good approach12:07
aspiersI mean, having two different commands12:07
aspiersbut I don't like migrate --evacuate, for reasons just explained12:07
aspiersI don't really like migrate --resurrect either12:08
aspiersalthough it bothers me less than migrate --evacuate12:08
aspiers"openstack server recreate" for the ephemeral case and "openstack server resurrect" for the shared storage case feels OK to me12:09
openstackgerritSurya Seetharaman proposed openstack/python-novaclient master: Allow passing negative values for the locked search_opt in server list  https://review.opendev.org/65978312:09
aspiersbut I can see the concern around non-admin users expecting to be able to use it12:10
sean-k-mooneyaspiers: well wel could allow openstack server recreate to be non admin12:10
sean-k-mooneyits basically a rebuild but optionally to another host12:11
aspierssean-k-mooney: in what scenarios would non-admin users use that? not if the server is already running right? cos it reuses the old name12:12
sean-k-mooneyaspiers: i was implying that openstack server recreate would kill the server if it was running and rescudle it12:13
aspiersoh12:13
sean-k-mooneythey would use it if the image or flavor had been updated and they wanted to create the sever with the updated version12:13
sean-k-mooneybut keep the same ports and data volumes12:14
aspiersyeah I guess that could be useful12:14
aspiersor if the instance got screwed up somehow, e.g. accidental rm -rf ;-)12:14
*** ccamacho has quit IRC12:15
sean-k-mooneyyep if you instacne are vnfs it would be a neat way to do a software update12:15
aspiersyep12:15
sean-k-mooneykindof like in k8s land12:15
sean-k-mooneye.g. just kill the pod and recreate with the latest version of it12:15
aspiersright12:17
aspiersso are you thinking to update https://review.opendev.org/#/c/643578 to split into two separate commands recreate/resurrect?12:18
aspiersthat would work for me, not sure what mriedem will think though12:18
aspierswith both I'd prefer to avoid mentioning the word "migrate" anywhere12:19
aspierssince "migrate" implies avoiding any significant change in state (except the location)12:19
*** nicolasbock has joined #openstack-nova12:23
*** mchlumsky has joined #openstack-nova12:24
aspierssean-k-mooney: BTW which timezone are you in?12:26
aspiersI'm guessing somewhere in Ireland?12:26
sean-k-mooneyyes12:27
sean-k-mooneyim shannon on the west coast of ireland12:27
aspiersnice12:27
sean-k-mooneywhat timezone i am awake in is not alway the same as where im located12:28
aspierstrue for me too ;)12:28
aspiersfor some reason I still expect all nova devs to be in north america12:28
aspiersI'm in London12:28
openstackgerritGorka Eguileor proposed openstack/nova master: Use os-brick locking for volume attach and detach  https://review.opendev.org/61419012:29
aspiersgot a lot of family in Dublin but never been further west than there yet :-(12:29
*** udesale has quit IRC12:30
*** udesale has joined #openstack-nova12:31
*** dikonoor has joined #openstack-nova12:36
*** janki has quit IRC12:40
*** janki has joined #openstack-nova12:40
*** janki has quit IRC12:41
*** dikonoor has quit IRC12:46
*** mdbooth_ has joined #openstack-nova12:47
*** cdent has quit IRC12:48
*** dklyle has joined #openstack-nova12:50
*** mdbooth has quit IRC12:50
*** slaweq has quit IRC12:51
*** slaweq has joined #openstack-nova12:52
*** ratailor has quit IRC12:54
*** ratailor has joined #openstack-nova12:56
*** cdent has joined #openstack-nova13:01
*** jaosorior has joined #openstack-nova13:11
*** jmlowe has quit IRC13:11
*** awalende has quit IRC13:15
efriedkashyap, aspiers: what's up now?13:15
kashyapefried: It's a change of scenery :-)13:16
aspiersefried: ah, you missed the almight bike osc shed ;-)13:16
kashyapefried: SEV-related, uncontroversial SEV-related13:16
kashyapErr13:16
aspiersoh, that too13:16
kashyap"SEV-related, uncontroversial infra patch"13:16
kashyapaspiers: That's what I assumed efried meant, when he pinged both of us together :D13:16
efriedI refer to ==> 3:45:56 AM - kashyap: aspiers: I think your change is ready to merge.  Let's check with efried (or stephenfin) when they're about.13:17
aspiersyeah, that one13:17
kashyapYep, I correctly guessed.13:17
kashyapIt is this one: https://review.opendev.org/#/c/655268/ -- "Add infrastructure for invoking libvirt's getDomainCapabilities API"13:17
efriedOkay. This isn't really in my, ahem, "domain" of expertise13:18
efriedbut I'll take a look.13:18
gansomelwitt, dansmith, tonyb: Hi! could you please review https://review.opendev.org/#/c/659338 when you have some time available? Thanks in advance! =)13:19
aspiersefried: it should be pretty simple and not require special expertise13:19
*** belmoreira has joined #openstack-nova13:20
*** mriedem has joined #openstack-nova13:21
*** tbachman has quit IRC13:21
efriedaspiers, kashyap: Does this also have to do with video models?13:22
efriedhttps://blueprints.launchpad.net/nova/+spec/libvirt-video-device-models13:22
kashyapefried: No, that getDomainCapabilities() isn't related to video models.13:23
efriedight13:23
efriedI see video models in your fake xml, so...13:23
efrieduh, I thought I did...13:23
aspiersI guess it could be a future way of detecting available video models?13:24
efriedyeah, there. (Chrome's text search has been way slow for me lately)13:24
aspiersif anything needed to do that13:24
aspiersbut that's certainly not what's driving the change right now13:24
kashyapefried: You're right -- they _are_ present, though.  But not relevant in this context13:24
kashyapIndeed, what aspiers said.13:24
efriedk13:24
aspiersIt's been driven by SEV and secure boot13:24
kashyapefried: Nothing like jumping right into the bowels of libvirt when you wake up, is it? :D13:24
aspiersI haven't woken up yet13:24
kashyaps/wake up/get started/13:24
kashyapaspiers: Haha13:25
aspiersI'm sure sean-k-mooney could comment on whether this change would be useful for that bp13:25
efriedsomething something bowels when I wake up in general13:25
*** ccamacho has joined #openstack-nova13:25
kashyapaspiers: You got attacked by ZombieLoad13:25
aspiersefried: TMI X-D13:25
efriedno doubt13:25
aspierson the plus side, I actually managed to get out of bed before 9am for the first time since Denver13:26
kashyapaspiers: It's definitely useful -- as that _allows_ querying for the models.13:26
aspiersbut getting out of bed doesn't imply waking up13:26
aspierskashyap: right13:26
kashyapaspiers: Wauw, I thought it was just me, whose sleep cycle got completely botched since Denver.13:26
aspiersnope13:27
efriedmy sleep cycle has been fine, thank you.13:27
aspiersmine's been botched ever since SUSECON :-(13:27
* aspiers -> late lunch13:27
kashyapaspiers: I kept thinking: "No, it can't be Denver, it's been more than 8 days")13:27
efriedShanghai will be a different story13:27
aspiersoh god13:27
kashyapefried: Right, enjoy :D13:27
aspiersfunny thing is, I was fine for Sydney13:27
aspiersbut then I made sleep spreadsheets for that trip, and stuck to them religiously13:27
aspiersand with that teaser hanging, I'm really off for lunch13:28
*** lbragstad has joined #openstack-nova13:28
*** tbachman has joined #openstack-nova13:29
* cdent shakes fist (warily) at efried 13:29
cdentI got the jetlag so hard13:29
sean-k-mooneyaspiers: which change was that?13:30
efriedsean-k-mooney: We're asking whether https://review.opendev.org/#/c/655268/ might be relevant to https://blueprints.launchpad.net/nova/+spec/libvirt-video-device-models13:30
sean-k-mooneyefried: oh not really13:30
efriedk13:30
efriedaslo13:31
efriedalso*13:31
sean-k-mooneythat is not reported in the capablities api13:31
efriedcould you review that patch please?13:31
efried:)13:31
sean-k-mooneyya13:31
efriedthanks13:31
sean-k-mooneyactully it looks like it has been added since i last looked13:32
sean-k-mooneyso i guess i could use it13:32
sean-k-mooneyi had planned to just to version checks13:32
sean-k-mooneybasically reusing the tables we already have in the code13:32
sean-k-mooneylike this one https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L67-L10013:33
sean-k-mooneyso for disk bus and video yes i can use it but vif models are not repored  and it will have to use the table approach13:35
stephenfinkashyap, efried, aspiers: I can take a look at that now. Was Bugzilla wrangling this morning13:35
kashyapstephenfin: Completely understand; aspiers diligently worked through all the feedback.  I can't punch any holes in the change13:36
sean-k-mooneyhum actully i will  have to use both as it look that info is not there for older libvirt13:36
kashyapstephenfin: And it's a _very_ useful infrastructure patch that helps other features as well.  So that can be merged regardless of SEV13:36
kashyap(Although, aspiers might be shaking his fist at me, as I said "regardless of SEV")13:37
*** xek_ has joined #openstack-nova13:37
sean-k-mooneykashyap: aspiers wehre did the fake libvirt capablites xml come form?13:37
sean-k-mooneyaspiers: did you take it form you machine?13:37
stephenfinsean-k-mooney: I was just about to ask the same question :)13:38
kashyapsean-k-mooney: I'd guess so, from his test box13:39
sean-k-mooneythis is what i get on my desktop13:39
sean-k-mooneyhttp://paste.openstack.org/show/751512/13:39
*** xek has quit IRC13:39
stephenfinHmm, Fedora doesn't come with libvirt out-of-the-box13:39
sean-k-mooneyrunning libvirt 5.2.013:39
stephenfinsean-k-mooney: That's capabilities, not domain capabilities13:40
stephenfinThere's a difference in command and output but I'm trying to remember what it is13:40
sean-k-mooneyah i have this for that http://paste.openstack.org/show/751514/13:40
*** tbachman has quit IRC13:41
sean-k-mooneycan we ask them to add interface to that list13:41
sean-k-mooneyi get similar output on 4.0.013:41
stephenfinThose are available under a subcommand, but I don't recall what that's called either13:42
sean-k-mooneyso i guess it been around for a while13:42
* stephenfin had to use it months when trying to figure out if his i350 NIC was dead or not13:42
sean-k-mooneyits sudo virsh domcapabilities13:42
stephenfinno, for the interface capabilities13:42
sean-k-mooneyoh well there are two things13:43
sean-k-mooneythere are the physical nic feature flags13:43
sean-k-mooneyand what virtual nics qemu can emulate13:43
sean-k-mooneythe nic feature flags are under the nodedev-dumpxml command13:44
kashyapsean-k-mooney: It renders the same data without `sudo`, too.  (With the ultra-minor diff being: with `sudo` it links to actual emulator path /usr/bin/qemu-system-x86_64; instead of the wrapper /usr/bin/qemu-kvm)13:44
stephenfinah, I was referring to the former in this case actually, my bad13:44
sean-k-mooneythe models that can be emulated are not available13:44
openstackgerritGuo Jingyu proposed openstack/nova master: Skip existing VMs when hosts apply force_config_drive  https://review.opendev.org/65970313:44
stephenfinRemind me, what's the "is my system set up correctly for KVM" command?13:44
sean-k-mooneykvm-ok13:45
stephenfinta13:45
sean-k-mooneyits part of the cpu-its part of the cpu-checker package if you dont have it installed13:46
stephenfinta x2 :)13:46
sean-k-mooneyits also only available on debian/ubuntu i think :)13:47
sean-k-mooneyso if you use somethin else all its doing is checking for /dev/kvm and checik lsmod for kvm13:47
stephenfinSo it would seem. Damn. I thought libvirt provided some tool for this but maybe I'm mistaking it for something13:47
sean-k-mooneyon and /proc/cpu-info for vmx or svm13:48
sean-k-mooneyyou can tell form virsh capablityes13:48
stephenfinYeah, I went and did it manually instead. All good now13:48
sean-k-mooneyit list the emulator that can be run13:49
sean-k-mooneyif you dont se a hvm section then you cant use kvm13:49
stephenfinI was seeing "error: invalid argument: unable to find any emulator to serve 'x86_64' architecture" because I hadn't installed qemu-kvm yet, assuming libvirt would do it by default13:49
stephenfin*installing libvirt13:49
*** mmethot has quit IRC13:50
sean-k-mooneyor do sudo virsh capabilities | grep  "<domain type='kvm'>"13:50
sean-k-mooneystephenfin: libvirt does not install qemu by defualt13:50
stephenfinIn any case, I'm seeing something similar to you but I've got references to the OVMF stuff13:50
stephenfinYup, I know that now :)13:50
sean-k-mooneyand annoying installing qemu-kvm does not install qemu anymore...13:51
* sean-k-mooney shakes fist at fedora13:51
stephenfinand I'm missing the virtio disk models, oddly13:51
sean-k-mooneystrange13:52
sean-k-mooneyyou have installed qemu and qemu kvm packages and restarted libvirtd after13:52
sean-k-mooneywhat os are you testing by the way13:52
stephenfinYeah. Not the end of the world though13:53
stephenfinFedora 3013:53
stephenfinHold your "popOS!" comments now :P13:53
sean-k-mooneystephenfin: they have been makign the qemu packages modular13:53
sean-k-mooneyhave you tried gentoo :P13:54
stephenfinNo, I figured I should give Slackware a shot first13:54
sean-k-mooneygentoo is the most painful disto i have used even more painful then linux form scratch13:54
sean-k-mooneykashyap: is the virito support for qemu disk part of a sepereate package on fedroa 30?13:55
kashyapsean-k-mooney: What do you mean?13:55
sean-k-mooneykashyap: i woudl guess as that should be part of qemu by default13:55
kashyapsean-k-mooney: Yeah13:55
kashyapThere has been a split of RPM packages for QEMU13:56
kashyapEspecially the block drivers13:56
sean-k-mooneykashyap: stephenfin is not seing vitio in the domcapablities output on f3013:56
stephenfinI'm referring to lines 101-105 from here http://paste.openstack.org/show/751514/13:56
stephenfinIt doesn't matter much. More curiosity than anything13:56
kashyapstephenfin: Can you post your `rpm -qa | grep -i qemu` output?13:56
kashyapsean-k-mooney: Check this one that i did recently: https://github.com/rdo-packages/nova-distgit/commit/4e490d2348cfe99b68d70c4b95408348d0b54492 (" Use granular libvirt and QEMU RPMs for EL-8 Nova")13:57
stephenfinkashyap: http://paste.openstack.org/show/751515/13:57
sean-k-mooneystephenfin: well its novas default for qemu13:57
sean-k-mooneyand kvm13:57
sean-k-mooneyso if it really is not there your vms wont boot13:57
kashyapstephenfin: Can you install 'qemu-kvm-core'?13:58
sean-k-mooneyunless you set hw_disk_bus=sata in the image or something13:58
sean-k-mooneykashyap: waith how is qemu-kvm install without qemu-kvm-core13:58
kashyapstephenfin: The domainCapabilities output you posted -- is that from F30?13:58
sean-k-mooneythat seam like a packageing bug13:58
stephenfinkashyap: Done. No good. That paste is from sean-k-mooney13:58
*** mlavalle has joined #openstack-nova13:59
kashyapsean-k-mooney: Hang on -- I find it strange, too many things flying around :-)13:59
stephenfinHere's mine, fwiw http://paste.openstack.org/show/751516/13:59
stephenfin5.0.16-300.fc30.x86_6413:59
*** roukoswarf has joined #openstack-nova14:00
sean-k-mooneykashyap: :) that ws from manjor/arch with the latest libvirt available.14:00
*** jmlowe has joined #openstack-nova14:00
sean-k-mooneystephenfin: i see it on my popos system too with libvirt 4.0.0 so its not a libvirt issue i dont think14:00
roukoswarfany recommendations on how to get nova scheduler to anti-affinity a bigger unit than a "host"? i have many "hosts" in a single chassis, or sometimes a single rack, and i need the ability to get something like octavia to schedule things to different chassis/racks for HA.14:02
roukoswarfmasses of cells seems like... the wrong solution?14:03
*** JamesBenson has joined #openstack-nova14:04
kashyapsean-k-mooney: It is _not_ a packaging bug: in stephenfin's paste: you _do_ have: qemu-system-x86-core-3.1.0-8.fc30.x86_6414:06
kashyapIt is 'qemu-system-$ARCH-core'14:06
*** lpetrut has quit IRC14:06
kashyap(Also note: 'qemu-kvm-core' is a _dummy_ package that pulls in 'qemu-system-x86' on Fedora)14:08
mriedemefried: remind me, mock spec is a string right? https://review.opendev.org/#/c/659726/2/watcher/tests/datasources/test_ceilometer_helper.py14:11
kashyapstephenfin: You need libvirt-5.2.0 or newer to get the virtio-* enum values reported.14:11
efriedmriedem: no, it is *not* a string. Unless you want to spec a string.14:11
mriedemgdi14:11
efriedhold on, I have a commit that explains it clearly (I hope)14:12
kashyapstephenfin: It's noted "v5.2.0 (2019-04-03)14:12
efriedmriedem: https://review.opendev.org/#/c/650370/14:12
kashyapstephenfin: Err.  I meant: It's noted as part of the release notes of libvirt "v5.2.0 (2019-04-03)" here: https://libvirt.org/news.html14:13
*** mmethot has joined #openstack-nova14:13
mriedemso i just changed to this (added autospec=True):14:13
mriedem@mock.patch.object(ceilometer_helper.CeilometerHelper,14:13
mriedem                       'statistic_aggregation', autospec=True)14:13
mriedemand it blew up as expected14:13
mriedemso why not just autospec?14:13
mriedemi think the answer is "sometimes it doesn't work how you'd expect" is the answer14:15
mriedemlike everything with specs and mock14:15
*** ykarel is now known as ykarel|afk14:16
*** luksky has quit IRC14:19
*** dpawlik has quit IRC14:24
*** Luzi has quit IRC14:25
stephenfinmriedem: claudiub has a series to convert everything to autospec. Would it be worth reviving that?14:25
stephenfinaspiers: Reviewed https://review.opendev.org/#/c/655268/. One concern about logging but otherwise it looks good14:25
openstackgerritMark Goddard proposed openstack/nova master: [ironic] Don't remove instance info twice in destroy  https://review.opendev.org/65980914:30
efriedaspiers, kashyap: reviewed same, soft-ish -1 on some refactoring to make things cleaner.14:30
efriedstephenfin: If those guys aren't around rn, maybe you could look and tell me if I'm way off base with my comments; maybe there's some reason we put in extra fluff.14:31
stephenfinsure14:31
kashyapefried: Just skimmed (in a call); thanks for the detailed review.  I'll pay full attention to your comments on Monday.14:32
efriedthanks both14:33
*** alex_xu has joined #openstack-nova14:33
kashyapThank _you_ for spending a block of time on it.14:35
mriedemstephenfin: this isn't a question for nova really14:35
mriedemit's a generic dev thing14:35
mriedemtrying to apply it to a bug fix in watcher14:36
stephenfinefried: Yup, they all look valid. I was ignoring the added config opts because I saw them used in later patches, but adding them then might make more sense, aye14:37
stephenfin*config classes14:37
stephenfinmriedem: Ah, fair. I didn't catch that14:37
*** jaosorior has quit IRC14:41
*** jmlowe has quit IRC14:41
*** guozijn has quit IRC14:42
*** macza has joined #openstack-nova14:45
efriedstephenfin: I'm less concerned about that than about the unnecessary middleman class14:45
cdentmriedem: are your wips on https://review.opendev.org/#/q/topic:bug/1790204 still a going concern?14:47
*** macza has quit IRC14:47
mriedemcdent: as in vmware needs them now?14:47
cdentno, I'm writing the pupdate14:47
mriedemoh14:47
*** kmalloc is now known as kmalloc_away14:47
mriedemit's very complicated so i'm not working on it regularly no, maybe once per month before i write up my internal monthly report :)14:48
cdenti'll keep them on the list as why not...14:48
mriedemefried: this is why spec baffles me http://paste.openstack.org/show/751523/14:51
mriedemnote expected and actual are identical14:51
mriedemnot sure wtf "b'TypeError: too many positional arguments'" is about14:51
mriedemthis being the test http://paste.openstack.org/show/751524/14:52
*** dtantsur is now known as dtantsur|afk14:52
efriedmriedem: what's the method signature for statistic_aggregation?14:52
*** jamesdenton has quit IRC14:53
mriedemdef statistic_aggregation(self, resource_id=None, meter_name=None,14:53
mriedem                              period=300, granularity=300, dimensions=None,14:53
mriedem                              aggregation='avg', group_by='*'):14:53
mriedemis it complaining that kwargs aren't used for all params?14:54
efriedThis looks to be a buglet in mock itself; it's confusingly triggering a failure on assert_called_once_with14:54
efriedyes14:54
efriedthat's a little weird, but I'm a fan.14:54
*** belmoreira has quit IRC14:54
efriedkwargs should be named14:54
mriedemi've seen the tattoo14:54
*** yan0s has quit IRC14:55
*** shilpasd has quit IRC14:56
aspierssean-k-mooney: yes, I got the sample XML from an SEV-capable test box14:58
*** jmlowe has joined #openstack-nova14:58
* efried needs to chauffeur...14:58
*** belmoreira has joined #openstack-nova14:58
mriedemchanging to use all kwargs results in an even weirder error, so i'll just not try spec14:59
mriedemoh nvm15:00
*** cfriesen has joined #openstack-nova15:04
*** mcgigglier has quit IRC15:05
artommriedem, yey, we can talk to you again!15:13
artomHad some scary legals emails about the Huawei embargo thing15:13
aspiersartom: :-(15:14
aspierswhat a crappy situation15:14
aspiersalso I'm guessing pretty unprecedented and nonsensical in an open source context15:14
jangutterI am so glad South Africa is no longer under sanctions.15:14
*** ykarel|afk is now known as ykarel|away15:15
mriedemartom: i've heard15:16
*** priteau has joined #openstack-nova15:16
mriedemrealize that i already know all of your secrets before you tell me15:16
artomIf you knew all my secrets you'd be in an asylum15:16
*** ratailor has quit IRC15:17
artomaspiers, it's definitely weird... though I suspect if there's on legal department that can make sense of it, it's RH's, given our open source history15:19
artom*one legal15:19
aspiersor SUSE's - a few months older than RH even ;-)15:19
aspiersalthough we're no longer an American company15:20
aspiersso maybe we don't need to care as much15:20
aspiersI'm sure RH has a much more impressive army of lawyers than we do too15:20
*** ivve has quit IRC15:21
*** sapd1_x has joined #openstack-nova15:21
artomTrue, though being German (albeit with US... affiliates?) they may not be subjected to the same laws15:22
aspiersWe're not German15:22
aspiersOwned by EQT which is Swedish15:22
aspiersbut we have companies in many countries around the world, just like RH15:23
* mriedem always thought there were DE now too15:23
aspiersso yeah, there'll be a SUSE Inc. or whatever15:23
aspiersand SUSE GmbH, SUSE UK Ltd. etc.15:23
aspierseach of those subject to the whims of the respective country15:24
openstackgerritMerged openstack/python-novaclient master: Optimize limit option docs string description for novaclient  https://review.opendev.org/64303515:24
aspiersI wonder if it could result in situations where Huawei-related work could get taken off the hands of American members of a team and given to colleagues in other countries15:24
artomaspiers, in our case even our non-US companies are subject to the embargo15:24
aspiersinteresting, why's that?15:25
artomIANAL15:25
aspiersbecause the parent company is US? or some other reason15:25
artomI guess15:25
aspiersif so, SUSE wouldn't have the same issue15:25
artomBut yeah, we got a pretty strongly worded email that we essentially need to stop all dealings with Huawei, all over the world15:25
aspierscrazy stuff15:25
artomApparently open source upstream communities are exempt from that, tho15:26
artomWhich is why mriedem is still my friend15:26
aspiershah15:26
* aspiers quickly backtracks and claims he's not allowed to work on Huawei stuff15:26
artomOh man, you saw your chance and you went for it XD15:26
aspiers;P15:26
* artom wonders off to get a hair cut and lunch15:27
*** gyee has joined #openstack-nova15:34
*** tssurya has quit IRC15:36
aspierskashyap: remind me, will your secure boot code care about <features> at all?15:37
*** jmlowe has quit IRC15:38
*** wwriverrat has joined #openstack-nova15:41
*** ykarel|away has quit IRC15:58
*** rpittau is now known as rpittau|afk16:01
sean-k-mooneyartom: that is tecnically a gray area16:04
sean-k-mooneybut most multintionall play it safe16:04
sean-k-mooneyefried: im going to finish earlish today. i have libvirt domcapabilies review on my todo for today is there anything else you want me to look at before monday16:05
efriedsean-k-mooney: I'm sure nothing is urgent, thanks for asking.16:06
*** panda|rover is now known as panda|rover|off16:06
kashyapaspiers: The code-that-doesn't-exist-yet will care about it16:07
sean-k-mooneycool ya im going to drop off irc soon which is why i asked16:07
kashyapaspiers: I need to be AFK: catch up on Monday.  Sorry16:07
kashyapHave a good weekend, folks (in CEST)16:07
aspierskashyap: cya16:07
aspierssean-k-mooney: I'm just about to push a new patchset16:07
aspiersin response to reviews from efried and stephenfin16:07
aspiersefried: and one of the comments is "please can we discuss in IRC" ;-)16:08
sean-k-mooneycool ill finish updating the downstream but then ill look at it then16:08
sean-k-mooney*bug16:08
efriedaspiers: I'm here, discuss away.16:09
aspiersefried: I don't want to repeat the long essay I've drafted in Gerrit ;)16:09
aspiersefried: give me a few mins to finish the response16:09
efriedack16:09
*** mdbooth_ has quit IRC16:11
*** belmoreira has quit IRC16:11
mriedemgibi_off: i'm guessing we're not going to be able to backport the fix for https://review.opendev.org/#/c/512623/ without dropping the functional test as well because of all the underlying refactor to make the functional test work?16:12
*** tbachman has joined #openstack-nova16:12
*** cdent has quit IRC16:13
*** tesseract has quit IRC16:17
*** psachin has quit IRC16:17
*** tesseract has joined #openstack-nova16:18
*** whoami-rajat has joined #openstack-nova16:20
*** macza has joined #openstack-nova16:21
*** sridharg has quit IRC16:22
*** jangutter has quit IRC16:22
*** macza has quit IRC16:26
stephenfinmriedem: You think a patch to exclude the 'user_data' field for each instance logged in this log sounds reasonable? https://opendev.org/openstack/nova/src/commit/ed5b7c7fc311c2af8c8be09044bb2723141f6f0d/nova/scheduler/host_manager.py#L18016:29
sean-k-mooneystephenfin: we woudl have to modify the __repr__ function in the ovo to not incude that right16:30
stephenfinI'm looking at a log where the user has bulk created some instances and it's just pages and pages of base64 (?) data16:30
stephenfinI guess? I know we do some filtering of objects on security grounds16:30
stephenfinI was hoping to reuse some of that, if that's possible16:30
sean-k-mooney ah then ya it could be simple so16:31
*** jcosmao has left #openstack-nova16:31
sean-k-mooneywell looks like you have the instace info as a dict so you would have to make a copy and just remove the user_data section from teh copy or replace it with a message saying it was remvoed16:32
stephenfinOr that <?> marker that gets inserted by...something16:33
*** luksky has joined #openstack-nova16:33
stephenfino.vo normally, I guess16:33
sean-k-mooneyyes that thing16:33
sean-k-mooneye.g. so you can tell there was user data but not log it16:33
mriedemstephenfin: you're looking at an old log then,16:35
mriedembecause that was fixed to only log instance uuids16:35
mriedemhttps://opendev.org/openstack/nova/commit/4fd7c93726eff4cc0b010741ea1772cf19c314fc16:35
mriedemhttps://review.opendev.org/#/q/I0eda1c58a7eb54121230c880818b4b1d0fdf489316:36
mriedemtake that back to pike and ocata if you want16:36
stephenfinI'm going to do just that. Thanks, mriedem16:36
mriedemnp16:36
openstackgerritStephen Finucane proposed openstack/nova stable/pike: Do not dump all instances in the scheduler  https://review.opendev.org/65983216:36
sean-k-mooney oh the list grabs the keys that are the uuids neat16:37
openstackgerritStephen Finucane proposed openstack/nova stable/ocata: Do not dump all instances in the scheduler  https://review.opendev.org/65983316:37
mriedemit's a dict, the uuids are the keys16:37
sean-k-mooneyyep16:37
sean-k-mooneyso going from  inst_dict to  list(inst_dict) when form full dict content to just keys whic is cool16:38
sean-k-mooneyor we could have dont inst_dict.keys() to be explicit but same effect16:39
stephenfinsean-k-mooney: Python be cool like that16:39
sean-k-mooneyyes and no16:40
stephenfinlist(dict) has the advantage that you'll always get a list back16:40
sean-k-mooneyi know why python iterate over the keys not the items but it always trips me on things like that16:40
stephenfinwhereas with dict.keys(), you'll get an iterator on Python316:40
sean-k-mooneyalso true16:40
openstackgerritAdam Spiers proposed openstack/nova master: Add infrastructure for invoking libvirt's getDomainCapabilities API  https://review.opendev.org/65526816:40
stephenfinand I don't know how oslo.log handles that16:40
sean-k-mooneyi just would have expected list(dict) to be the same as list(dict.itmes()) but i also know why that is not the case16:41
aspiersefried, sean-k-mooney: new patchset 4 and response to patchset 3 ^^^16:41
efriedstephenfin: would you mind taking a quick look at this? https://review.opendev.org/#/c/659682/16:42
efriedThe tox failure is weird, it's claiming it can't find deps, but they're being set in [testenv]16:42
stephenfinlooking16:42
efriedstephenfin: This is a throwaway patch, so if it's fixable by adding explicit deps lines back in, that's fine.16:43
stephenfinefried: They're being set in 'testenv', not 'testenv:functional'16:43
efriedno inheritance there?16:43
efriedokay16:43
stephenfinNope. tox isn't that clever16:43
stephenfinThankfully, tbh. It's already too clever16:44
efriedso just copy them into those envs16:44
efriedalso would have thought having used envdir would be enough16:44
efriedbut apparently something is *requiring* deps?16:45
openstackgerritEric Fried proposed openstack/nova stable/stein: DNM: Revert "Use external placement in functional tests"  https://review.opendev.org/65968216:45
openstackgerritEric Fried proposed openstack/nova stable/stein: Skip _exclude_nested_providers() if not nested  https://review.opendev.org/65920616:45
efriedthanks for the look stephenfin16:45
*** xek_ has quit IRC16:46
*** davidsha has quit IRC16:48
*** sapd1_x has quit IRC16:48
*** mriedem is now known as mriedem_away16:49
*** maciejjozefczyk has quit IRC16:55
*** tesseract has quit IRC16:58
efriedaspiers: thanks for the updates. Convince me that LibvirtConfigDomainCapsFeature needs to exist *ever*.16:58
aspiersefried: just saw your comment on the follow-up change about that16:59
aspiersefried: think I see where you're coming from now, maybe16:59
*** tesseract has joined #openstack-nova16:59
aspiersso your point is that a base class isn't needed here16:59
aspierseven though we expect there to be other feature classes in the future16:59
aspiersor at least, have no reason to believe that there definitely won't be17:00
aspiersright?17:00
efriedRight, because the only thing you're doing with that base class is converting a positional 'name' arg into the 'root_name' kwarg.17:00
aspiersYeah, that's a fair comment17:00
efriedwhich the subclasses can do just as easily.17:00
aspiersbut couldn't the same argument apply to LibvirtConfigGuestDeviceAddress?17:01
aspiersI was trying to follow the precedent which already seemed to exist in config.py17:01
*** derekh has quit IRC17:02
aspiersAnyway I'm not religious about the need for that base class17:03
efriedLooking... but I tend to be in the camp of "follow precedent thoughtfully"17:03
aspiersWe could always reintroduce a base class later if need be17:03
aspiersI'm in the same camp for sure :)17:03
efriedno, **DeviceAddress does some actual things in its overrides.17:03
aspiersI don't see any method in the DeviceAddress base class which adds any value17:04
aspiersthe subclass overrides do add value, naturally17:04
aspiersbut actually there's a much better example which I remember following17:05
aspiersLibvirtConfigGuestFeature17:05
efriedhttps://review.opendev.org/#/c/633855/13/nova/virt/libvirt/config.py@1351 ?17:05
aspiersI was just copying that approach17:05
efriedYes, LibvirtConfigGuestFeature is similarly worthless and could be removed imo.17:06
efried(separately)17:06
aspiersOK, so LibvirtConfigGuestDeviceAddress.__init__() does *one* thing of use: setting self.type17:06
aspiersso not entirely redundant17:07
efriedyeah, but the other methods do real things.17:07
aspiersthey are overridden by both subclasses though17:07
*** igordc has joined #openstack-nova17:07
aspiersoh, super()17:07
aspiersOK17:07
aspiersmissed that17:07
efriedbut for LibvirtConfigGuestFeature, there's no reason e.g. the next one couldn't just be17:07
efriedclass LibvirtConfigGuestFeatureACPI(LibvirtConfigObject):17:07
efried    def __init__(self, **kwargs):17:07
efried        super(LibvirtConfigGuestFeatureACPI, self).__init__(root_name="acpi",17:07
efried                                                            **kwargs)17:07
efriedsimilarly for the subsequent two.17:08
openstackgerritDoug Wiegley proposed openstack/nova master: Improve metadata server performance with large security groups  https://review.opendev.org/65608417:08
aspiersNow that this conversation jogged my memory, I remember I was definitely emulating LibvirtConfigGuestFeature17:08
efriedetc etc17:08
aspiersYeah17:08
aspiersWell like I said, I'm not religious about the base class at all17:08
aspiersI originaly thought you were saying it was pointless for other reasons17:08
efriedyeah, if you were emulating... you could have just used that.17:08
efriedbut I'd rather kill it.17:08
aspiersfine17:09
aspierscoming right up in PS517:09
efriedIn fact, I'm going to propose a patch to get rid of all that other cruft and see if it breaks the world.17:09
sean-k-mooneyim expecting an rm -rf of the nova subdiretory17:10
aspierslol17:10
aspierssean-k-mooney: that would be one way of getting rid of all the bugs ;)17:10
aspiers"Perfection is achieved, not when there is nothing more to add, but when there is nothing left to take away."17:10
aspiersnot sure St Exupery meant it quite like that though17:11
sean-k-mooneywell we have almost reached perfrection of cellsv1 in that case17:12
*** lbragstad has quit IRC17:12
*** lbragstad has joined #openstack-nova17:12
* stephenfin -> 🏡 🍻17:12
aspiersefried: I can remove that cruft if you want17:14
efriedaspiers: I'm already working on it.17:14
aspiersok17:14
*** udesale has quit IRC17:15
aspiersefried: didn't anyone tell you PTLs aren't allowed to code? ;-p17:15
efriedsore point17:15
aspiersstrictly no fun allowed17:15
aspiersjust feeling smug cos I temporarily escaped endless meetings17:15
aspiersI'm sure it won't last17:15
openstackgerritAdam Spiers proposed openstack/nova master: Add infrastructure for invoking libvirt's getDomainCapabilities API  https://review.opendev.org/65526817:16
aspiersand now a rebase for good measure ...17:17
*** ykarel|away has joined #openstack-nova17:17
openstackgerritAdam Spiers proposed openstack/nova master: Add infrastructure for invoking libvirt's getDomainCapabilities API  https://review.opendev.org/65526817:17
*** tesseract has quit IRC17:17
aspiersefried: I'm around for a while in case there are any remaining things you want addressed in that17:21
efriedaspiers: 2 mins and I'll have another look.17:21
aspierskewl17:22
openstackgerritEric Fried proposed openstack/nova master: Remove no-op intermediary libvirt config classes  https://review.opendev.org/65984817:23
efriedaspiers: ^17:23
*** ralonsoh has quit IRC17:24
efriedaspiers: +2. Now you get to rebase the others on top :)17:24
efriedshouldn't be too hard17:24
aspiersefried: already started17:24
aspiersnah it's not17:25
*** ttsiouts has joined #openstack-nova17:26
*** ivve has joined #openstack-nova17:30
*** mrch_ has quit IRC17:40
sean-k-mooneyaspiers: v6 of https://review.opendev.org/#/c/655268 looks good to me. if i want to reuse this for the device model based schduleing i will need to extend the config parsing logic accordingly but tha that shoudl be quite clean to do17:41
aspierssean-k-mooney: cool17:41
openstackgerritAdam Spiers proposed openstack/nova master: Add detection of SEV support from QEMU/AMD-SP/libvirt on AMD hosts  https://review.opendev.org/63385517:42
aspiersefried: there's the first rebase ^^^17:42
aspierssean-k-mooney: ^^^ in case you want to see the first example of a real customer of that API17:43
sean-k-mooneyyep ill try an review the full sev series next week.17:43
sean-k-mooneybut for now time to go get food17:43
sean-k-mooneyo/17:43
aspierssean-k-mooney: thanks, it's still incomplete but working on it17:43
aspierso/17:43
efriedaspiers: Once kashyap and/or sean-k-mooney have +1ed https://review.opendev.org/#/c/633855/ I'll send it.17:51
openstackgerritAdam Spiers proposed openstack/nova master: Add new "supports_amd_sev" capability to libvirt driver  https://review.opendev.org/63868017:51
aspiersefried: thanks, and there's the next one on top of that ^^^17:51
sean-k-mooneyefried: by the way really small change but can you comment on my comment in the commit message regarding bug vs specless blueprint17:51
sean-k-mooneyhttps://review.opendev.org/#/c/658785/417:51
efriedsean-k-mooney: looking17:52
efriedsean-k-mooney: I would need a better explanation of wtf this patch is doing.17:56
*** jdillaman has quit IRC17:56
sean-k-mooneyefried: we support hardware offlwoad ovs with kernel ovs today17:56
efriedkeeping in mind that my eyes glaze over any time I hear ovs or dpdk17:56
sean-k-mooneythis is part fo making it work with ovs-dpdk too17:56
efriedLet me put it this way: what would the reno say?17:57
sean-k-mooneyso this is just passign the ovs datapath type wich we have in teh vif to os-vif17:57
efriedWhat will the user notice?17:57
sean-k-mooneyopereators will now by able to deploy ovs-dpdk with hardwar offloads for nvf/telco deployment that need hi data rates and a fast fallback path when a feature is not supported in hardware17:58
sean-k-mooneywe could skip the reno i guess but it would be nice to call this out in 1 of nova, os-vif or neutron17:59
sean-k-mooneythe patch to enable it in all 3 repos are equally short however17:59
efriedsean-k-mooney: Okay, so a bug definitely doesn't seem appropriate; it sounds like a feature to me.18:00
sean-k-mooneythis prettymuch almost works already18:00
sean-k-mooneyya i was thinking that too but its also a tiny change18:00
sean-k-mooneyhence sepcless blueprint18:00
efriedRight, so if it's truly tiny, a specless blueprint would be fine.18:00
efriedimo18:01
sean-k-mooneywell that patch is the total cahnge needed to nova18:01
efriedwhat would the operator need to do to make things happen?18:01
efried(is the kind of thing I would want the bp to say)18:01
efriedis there an API change?18:01
efriedconfigs that affect?18:02
efrieddocs that need to be updated?18:02
sean-k-mooneyjust install ovs-dpdk on a host with a melonox nic that support hardware offloand and have the right version of nova and neutron18:02
sean-k-mooneythere is not api change or change to iamge or flaovr18:02
sean-k-mooneyyou use the feature the same way as hardware offloaded kernel ovs18:02
openstackgerritAdam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection  https://review.opendev.org/63633418:03
sean-k-mooneyi am not sure if we have docs for that but assuming we do the might want to be updated18:03
efriedneat. So yeah, a specless blueprint sounds appropriate to me. Though I guess the problem is that you can't file a single blueprint against multiple projects?18:03
aspiersALL YOUR REBASE ARE BELONG TO US18:03
sean-k-mooneyefried: https://docs.openstack.org/neutron/rocky/admin/config-ovs-offload.html shoudl be updated18:03
sean-k-mooneybut that is in neutron18:03
aspiersthat's probably the best joke I ever made </tragically_proud_moment>18:04
sean-k-mooneyefried: yep which is why i was questioning should it be an rfe bug18:04
efriedaspiers: Cause you know I'm all about rebase, 'bout rebase...18:04
sean-k-mooneybut specless blueprint for nova and rfe bugs that is shared for os-vif/nueton is fine too18:04
efriedsean-k-mooney: Is that a thing? Then sure.18:04
aspiersdamn, instantly upstaged, just like that18:04
sean-k-mooneyefried: its what neutron uses for all feature requests18:05
efriedheh, okay.18:05
efriedwfm18:05
sean-k-mooneywe dont use rfe bugs in nova18:05
sean-k-mooneyits litrally just a rfe tag on the bug18:05
sean-k-mooneycool anyway really going this time o/18:06
aspiershaha the number of times I've said that18:07
artomaspiers, you have no chance to survive, merge your time18:08
aspiersartom: somebody set up us the branch18:10
artomaspiers, move TAG18:11
aspiersCI was beginning18:11
aspiersWe get review18:12
aspiersYou are on the way to -218:12
aspiersWhat you say !!18:12
artomYou've gone and rewritten the story.18:13
* artom bows.18:13
aspiers:D18:14
artomNot just random puns, a coherent narrative18:14
aspiersI try my best18:14
artomYou definitely know what you doing.18:16
aspiersFor great fun18:17
aspiersOK, I think it's time for the weekend18:17
aspiersMain screen turn off18:17
*** ykarel|away has quit IRC18:20
*** mriedem_away is now known as mriedem18:23
openstackgerritAdam Spiers proposed openstack/nova master: Add <launchSecurity> element to libvirt guest XML for AMD SEV  https://review.opendev.org/63631818:31
efriedaspiers: https://review.opendev.org/#/c/638680/ reviewed, lemme know if we need to Talk.18:37
*** itlinux has joined #openstack-nova18:38
*** whoami-rajat has quit IRC18:39
aspiersefried: looking18:41
aspierstotally fell into the same trap as sean-k-mooney18:41
aspiersMain screen did not turn off :-/18:42
*** macza has joined #openstack-nova18:44
*** maciejjozefczyk has joined #openstack-nova18:45
openstackgerritAdam Spiers proposed openstack/nova master: Add detection of SEV support from QEMU/AMD-SP/libvirt on AMD hosts  https://review.opendev.org/63385518:46
*** macza has quit IRC18:49
*** mvkr has joined #openstack-nova18:49
aspiersefried: thanks, makes sense to move it to u_p_t()18:52
aspiersI'll do that next week18:52
*** maciejjozefczyk has quit IRC18:53
efriedcool, thanks.18:54
aspiersefried: so that would mean it's no longer called from init_host(), right?18:55
openstackgerritEric Fried proposed openstack/nova master: Introduces the openstacksdk to nova  https://review.opendev.org/64366418:57
openstackgerritEric Fried proposed openstack/nova master: WIP: Use OpenStack SDK for placement  https://review.opendev.org/65602318:57
openstackgerritEric Fried proposed openstack/nova master: Introduces the openstacksdk to nova  https://review.opendev.org/64366418:57
openstackgerritEric Fried proposed openstack/nova master: WIP: Use OpenStack SDK for placement  https://review.opendev.org/65602318:57
*** tjgresha has quit IRC18:59
aspiersComputeManager.pre_start_hook() -> .update_available_resource() -> ._update_available_resource_for_node() -> ResourceTracker.update_available_resource() -> ._update_available_resource() -> ._update() -> ._update_to_placement() -> driver.updated_provider_tree()19:00
aspiersefried: but it seems that there are many other code paths which could reach update_provider_tree(), so your previous question about locking of the region modifying _domain_caps now will have a different answer19:01
efriedaspiers: I don't have a problem with the call chain through init_host as you've currently got it setting *some* instance variable on the libvirt driver.19:02
efriedwhich upt can then just go read19:02
aspiersalthough ResourceTracker._update_available_resource has a semaphore, so maybe that's OK anyway?19:02
efriedit just shouldn't be in the capability dict.19:02
aspiersOK, just wondering where's the best place to put it19:02
efriedself._supports_sev19:03
efriedwhatever19:03
aspiersyeah I guess the way you suggest works19:03
*** jmlowe has joined #openstack-nova19:03
efriedyou can put a comment saying we do this during init_host for efficiency and because it doesn't change19:03
aspiersright - if I put it in resource_tracker.py then it would probably get called too often (i.e. more than once)19:03
efriedright. And the singleton-ness you've got set up on it would partially mitigate that - but there's no reason to even have the extra call stacks in the way.19:04
efriedmemoizing, not singleton19:04
aspiersyep19:04
efriedhow's that whole stopping-working-because-it's-eod-friday thing going for you?19:05
aspiersreally great, just fantastic19:05
efriedYou got a taste for it in Denver, now you can't get enough.19:06
aspierssad but true19:07
*** dannins has quit IRC19:08
openstackgerritEric Fried proposed openstack/nova stable/stein: DNM: Revert "Use external placement in functional tests"  https://review.opendev.org/65968219:23
openstackgerritEric Fried proposed openstack/nova stable/stein: Skip _exclude_nested_providers() if not nested  https://review.opendev.org/65920619:23
aspiersefried: last question before I go, you're saying that I don't need a driver capability at all, and u_p_t() would just provide the trait directly?19:25
efriedaspiers: Correct19:25
aspiersOK good, thought so19:25
efriedaspiers: From the operator's point of view:19:25
efriedif they want mem encryption, they request resource MEM_ENCRYPTION_CONTEXT=119:26
efriedif they want SEV specifically, they request required=SEV19:26
aspiersBTW the kernel module parameter file is readable non-root, so I'm assuming privsep not required19:26
aspiersright19:26
efriedokay.19:26
aspiersIIRC, what you just wrote is captured in the spec19:26
aspiersI remember us going over that before19:26
efriedI think so, yes19:26
aspierssweet19:26
efriedbut I guess we never talked about driver capabilities19:27
aspiersright19:27
aspiersOK, *really* going now :)19:27
aspiershave a great w/e!19:27
efriedyeah, it just seems silly for PowerVM, which will never ever ever have AMD SEV support, to have to say no every time.19:27
efriedlater o/19:27
aspiersyes totally, felt wrong to have to add to powervm driver19:27
aspierscya! o/19:27
*** priteau has quit IRC19:31
openstackgerritMatt Riedemann proposed openstack/nova master: api-ref: mention default project filtering when listing servers  https://review.opendev.org/65987319:32
*** hamzy has joined #openstack-nova19:35
*** efried has quit IRC19:43
*** efried has joined #openstack-nova19:43
*** imacdonn has quit IRC19:45
*** raub has joined #openstack-nova19:45
*** raub has left #openstack-nova19:45
*** slaweq has quit IRC19:47
*** mchlumsky has quit IRC19:47
*** ircuser-1 has quit IRC19:50
*** imacdonn has joined #openstack-nova19:52
*** ircuser-1 has joined #openstack-nova19:53
*** ttsiouts has quit IRC19:59
*** pcaruana has quit IRC20:02
*** slaweq has joined #openstack-nova20:04
*** itlinux has quit IRC20:06
*** eandersson has joined #openstack-nova20:14
*** ttsiouts has joined #openstack-nova20:15
*** ttsiouts has quit IRC20:20
*** slaweq has quit IRC20:32
*** ttsiouts has joined #openstack-nova20:51
*** ttsiouts has quit IRC20:56
*** igordc has quit IRC21:02
*** xek_ has joined #openstack-nova21:02
*** igordc has joined #openstack-nova21:03
*** xek_ has quit IRC21:08
*** xek_ has joined #openstack-nova21:08
*** xek_ has quit IRC21:10
*** xek has joined #openstack-nova21:11
*** xek has quit IRC21:13
*** hongbin has joined #openstack-nova21:17
*** ttsiouts has joined #openstack-nova21:28
*** whoami-rajat has joined #openstack-nova21:31
*** xek has joined #openstack-nova21:31
*** igordc has quit IRC21:33
*** igordc has joined #openstack-nova21:33
*** JamesBen_ has joined #openstack-nova21:48
openstackgerritEric Fried proposed openstack/nova stable/stein: Skip _exclude_nested_providers() if not nested  https://review.opendev.org/65920621:48
openstackgerritMerged openstack/nova-specs master: Follow up for I9200c2b18369da8b9194a8ccbbf6f1baf6a9613c  https://review.opendev.org/65930021:49
efriedmriedem, melwitt: ---^ stein & rocky backports are ready for your stable eyes now.21:50
*** JamesBe__ has joined #openstack-nova21:50
melwittack21:50
*** JamesBenson has quit IRC21:51
mriedemmy eyeballs are currently embargoed21:52
mriedemmy sweet dance moves, however, are not21:52
*** JamesBe__ has quit IRC21:54
*** JamesBen_ has quit IRC21:54
*** JamesBenson has joined #openstack-nova21:56
*** itlinux has joined #openstack-nova21:58
*** JamesBenson has quit IRC22:00
*** hongbin has quit IRC22:01
*** rchurch_ has joined #openstack-nova22:01
*** ttsiouts has quit IRC22:01
*** hongbin has joined #openstack-nova22:02
*** rchurch has quit IRC22:02
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Allow searching for hypervisors and getting back details  https://review.opendev.org/65988622:09
*** mriedem has quit IRC22:20
*** jcosmao has joined #openstack-nova22:36
*** jcosmao has left #openstack-nova22:37
*** dave-mccowan has quit IRC22:50
*** luksky has quit IRC22:50
*** itlinux_ has joined #openstack-nova22:51
*** hongbin has quit IRC22:52
*** itlinux has quit IRC22:52
*** tbachman has quit IRC23:08
*** macza has joined #openstack-nova23:11
*** macza has quit IRC23:15
*** mvkr has quit IRC23:21
*** gyee has quit IRC23:25
*** francoisp has quit IRC23:27
*** whoami-rajat has quit IRC23:40
*** mmethot has quit IRC23:53
*** artom has quit IRC23:53
*** ttsiouts has joined #openstack-nova23:59

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!