KubeVirt Community, 30 Mar 2022

Previous Meeting Next Meeting

⏯

youtube image

►

From YouTube: KubeVirt Community Meeting 2022-03-30

Description

Meeting Notes: https://docs.google.com/document/d/1kyhpWlEPzZtQJSjJlAqhPcn3t0Mt_o0amhpuNPGs1Ls/

A

Daniel um hi everyone- this is thecube community meeting at the 30th of march. 2022 um first point is in general introduction, so anyone who wants to introduce themselves to the keyboard community does now help the opportunity to do this. So do we have anyone who is new and would like to introduce himself.

A

Okay, so I think that means no. um So then next point would be the agenda nodes which are quite empty. So I think I guess that um are there any things that someone would like to add on short notice now or should we just go to the open floor.

A

Okay, I think then the lecture is the case. Open floor is also um empty, so yeah I'd like to ask for someone maybe to bring something up. If you want.

A

Okay, so I think there is nothing to discuss then, okay, so then I see that there is a pull request that needs attention.

A

um So I'm going to.

A

Open that real, quick and see and give the word to antonio, hey.

B

Hi hi guys um so yeah I wanted to you- know, discuss a bit about this because I opened this. I think last week, or maybe before I don't remember, but um basically this came out from some uh attempts with uh snow in openshift, not that it it's important, but basically um I noticed that the workload updater doesn't actually check for the live migration feature gate before migrating any vms.

B

So, while adding the check, the discussion came out. So if you don't know you can scroll down.

B

Yeah exactly so lubo uh argued that we should fire an alert when a misconfiguration comes up, and here the misconfiguration would be that you have live migration as a workload update method, but you don't have the feature gate enabled which can easily happen, because we don't have a validating web book that checks the cuber cr at creation.

B

um So yeah I mean the discussion. There was like a small discussion whether we should fire this alert when it should be fired and if it should be, like um you know, rate limited in some way uh right now like. What's in the pr, whenever uh an update to the cubercr would happen, um an alert would probably fire would probably be fired if there is a misconfiguration uh that won't happen.

B

I mean that's the only case when it would happen. It won't happen every time you would try to migrate a vm. If you don't have the feature gate enabled so yeah I mean. Does anyone have any strong opinion? Just any opinion on this?

B

C

Like um I don't know uh a point to make about this like, as far as I understand like the events, are automatically cleaned up after an hour.

C

So if this is just like at editing, time like it'll, probably disappear from the event list and uh the relevant information will be gone unless you have some sort of, uh let's say a third-party service like uh where you're sending the events to which, probably, I guess it's what people use, but nevertheless my point is: wouldn't it make more sense to have like on the migration?

C

uh I don't remember the name of the entities of the crd to put like a status condition on on the migration things saying this like this cannot be done because uh feature gate is not enabled.

B

I think that happens already. I think I'm not 100 sure about it, but this is like this is a little bit different because the workload updater uh tries to migrate like outdated launcher uh when you upgrade kuber. uh So I mean it's a not so rare.

C

B

It's also kind of a rare event and it's a little bit like detached from the vm or vmi itself, because it's a rather like a cluster configuration rather than uh you know. The vm, because I mean vm can be live migratable, but you and you can have the feature. Gate turned off and that's still valid right, because maybe you don't want them to be migrated.

C

I see uh but yeah. My point is only that, like the events will be automatically cleaned up, so I'm not sure if, unless people are using, something else to put the events on uh this information will disappear. So I'm not quite sure how useful it is like. My point is basically that the events do not live forever.

B

Okay, well, actually, I didn't know that, to be honest, because um that's usually I mean uh looking at the code, that's usually what we do uh firing an event when we detect some kind of like warning. In this case, it's not like a straight up error, uh but I guess your suggestion is maybe maybe putting like a condition on the qrcr that would probably go more in the way of what you're suggesting.

C

I am not sure, like I'm, actually not suggesting anything, I'm just saying that uh like if this is something that happens like uh very seldom well yeah, the event will very shortly so probably when you will and check what happened. The event will already be gone and uh like what you're attempting to do will well not happen, or it will happen for a very short for a very brief period of time. So I'm just pointing that out.

C

B

C

An interesting point.

D

Just to I think it what you just said before, if this is, you have a desired state and you cannot it's not applicable right, it's wrong, so you need to represent, in the actual statement, start to stop and say it's migration is not applicable like and then then it's clear.

C

Right yeah, I think so, but I honestly uh and but this is for the vms- it's not for like, uh for let's say pieces of the cluster like uh you're upgrading the launchers, or something like that. These are for users, vms, isn't it.

B

Yeah yeah the workflow updater. Basically, what it does it's uh yeah migrating the the vm. In order to update the you know, the underlying launcher.

C

So I guess so I think uh eddie uh abstracted really well what's happening and I think, like uh a status condition. For this I mean honestly, it seems like the correct place to have it okay, but as you.

E

Said probably all right.

C

Probably we already do it, which means that if we already do it, no there's nothing to be done. I'm not sure yeah.

B

I I'll check that, because.

C

I I think I saw it like.

B

I'm not 100 sure to be honest, but yeah, okay, I'll check that that's a good suggestion.

A

Okay, then, um if that's so concluded, then I don't see any else. Any other pull requests does that need to tension here. So let's have a quick look at the mailing list review whether we probably skip some stuff that might not be handled somehow we have here the design proposal for the memory dump, but I think this is already eight days ago, so I think the only thing that yeah, I think there is nothing to do here. So, um let's move on to the next one.

A

So I'm not exactly sure if we had a box crop last week, but I am going to quickly look at those. So, let's see if we have something here by the way everyone can see my screen. Okay,.

A

Okay, great so just for confirmation, I'm not sure, because I don't really get some validation, whether it shows okay or not, okay, so, um okay, this one is an enhancement proposal. I think that is by boris. I guess I'll just track her. So like issues with the baseballs, probably not.

A

A

What do you think this sounds like? We can accept this right.

F

Yeah I can. I can confirm that that um is the experience funny thing to you as well.

A

Okay, so yeah. I think that we can just um accept this and then find someone who wants to work on it. So, let's see, um maybe I'm not sure if um we can just tag him. Probably.

A

I'm just asking him probably maybe he's interested in working on this so.

A

Okay, so the next one, let's see.

A

By the way, catherine, would you probably be interested in also working on that or um because I think, if it hurt you also, then you might also be interested to improve the experience for other people. I'm not sure.

F

Perhaps but I would be a couple months away from being able to ah okay.

A

Yeah yeah, if you I'm just going to to put the issue link into the chat, if anyone is interested to looking into that. So sorry that was the wrong one. This one then, um when you can might be able to take a look on that.

A

Okay, so next one okay, this one looks looks like there's already enough traffic and it's all really kind of bugged. So I think it should be handled just taking a quick look on that. Okay, finally, already answered okay.

A

So next one is this one: no information about the defensive package continuously linux in the documentation, um okay,.

A

Well, they based on this, such as rocket linux. Oh, I remember something about rocky linux, but I can't recall the exact details on that. I think we had that already.

F

Yeah, I think that they are correct in presuming that it's a limitation of the k zeros documentation, but at the same time it depends on what runtime they're using because, if they're using cryo, oh wait, cryo or container d is wrapped in k. Zeros isn't it.

F

I think that that's actually a k0 spot.

G

Hi everyone um I mean this is more of the issue of the host operating system. um I think what we are missing is the mention or in our dock, that some packages are needed in order to properly work with some settings.

G

And it's similar to the armor issue.

F

A

A

I think, uh can I just just tag you on that lugo um or um should? Should I add something on that? I can just also do that. So so you're saying this is this is just correct so um because we are not exactly saying that we are not directly supporting rocky linux, for example, because there are packages missing right, and maybe I think this the open thing there would be to uh open up a section in the user guide, probably where we would advise what packages are exactly missing or wouldn't that help somehow.

F

If we're trying to address a wider variety of possible common um implementation challenges, we might do a documentation section specific to like common caveats or something I need to find documentation that says that more eloquently but and and emulate that, but that would make sense.

H

I guess we can also suggest installing this, um this policy that we have already compiled just manually, and then this will happen.

A

um Bloody, do we have it? Do we have a link for me what I can just put in there so uh where we describe how that is installed manually, somehow, to be honest,.

H

No, unfortunately not, but I I think yeah documentation is something that we need to yeah some.

A

H

A small writer.

A

Okay um dude, would you be wanting to tackle that somehow or um I guess.

H

I'll try yeah.

A

A

A

I'm just going to attack you on this, so that you know where we need to look so um thanks. Thank you for that.

A

Okay. So, let's see this information, so this one appearance.

A

Oh, I think that didn't we have that.

D

A

I think that is.

A

Just don't get what he's wanting so he would like to have the same ip in the vmi.

F

What is the interface type one on this bug report?

F

Does he say like if it's good question.

A

um I'm having a little bit I'm a bit confused. This looks like chinese no.

A

ah Okay, so he's saying, ipv4 addresses this one, and this is an ip6 ipv6 address right, oh no! This is no.

F

C

A

Of course, oh yeah, so this one is the ip address that is.

C

Like the the mattresses map match,.

A

Sorry, did you get that.

C

Off my my connection is really bad at the moment, but uh the mac addresses match. But apparently I am honestly the big thing is. I have no clue what the the screenshot actually represents. Where did this come from and what are we saying this ipv4 cider actually is I don't know what it says, so I guess that he expects this to match with the info below, but I cannot say that it should match, because I don't know what the above one actually is.

F

That and we would need to do exactly what the ip address.

E

F

Inside the host itself and the guest.

D

Right so one second, if you read the the screenshot with the yemen, so that says that uh the ip address was detected on the pod of the world launcher more or less, and what you see here is I'm guessing it's what he sees in the in the guest itself. This is what so he says, it's not the same.

D

That's that's!.

F

His claim now the question is.

D

No, it's not not depend, it depends on the it depends. uh What binding is he using.

A

Using bridge or bridge or.

D

Okay, he's using bridge ah so if, if he.

C

Is this bridge.

D

Then this should have been correct.

C

Yes, it should match, but, like I don't know what the thing above is like uh you, you're saying this like you're assuming this is the the guest info.

D

C

This is what he wrote.

D

He says this is the guest ip guest toys. Well, one is the.

B

D

B

uh That looks like a windows screenshot of the main interface so.

A

Okay, so let's, let's have a quick look at the uh virtual machine instance, um probably so what we can at least deduce from that is that the network, where is it.

B

Yeah, you can see the uh guest main interface from the yaml as well from the vm definition, because he says that he wants uh an intel e thousand. And I think I read that in the screenshot as well. So.

C

Yeah, that's exactly.

A

The same here right, but I don't see where, where the network configuration is okay, so I think that we shouldn't run up to too much with that. So maybe we could just um triage needs more information.

A

And ask for um what? uh Can you elaborate a little bit on what you're expecting? Because this is not completely clear.

C

A

Yes, of course,.

C

uh Also ask for the configuration for the uh cluster default network because we need to figure out if that is using ipam or not.

A

Oh sorry, I didn't get that completely. Can you repeat that.

C

The cluster default network configuration.

A

Is important right? Yes, thank you.

D

But just if you can just please label it with the sig network, so we'll find it.

A

I think I think he did that already so signed rockets.

D

A

D

A

A

Screenshot or where the screenshot originated from.

A

Okay, I hope hope that makes it a little bit more clear. So I'm going to add this. Okay, uh like I said um eddie, did, was that the would you want to look at that author or should I attack you on the issue, or um should we just wait until he gives us more information.

D

Will I'll guess I or miguel will look at it? Okay,.

C

Yeah for now, let's, let's wait for his reply. Without the cluster default net, configuration there's nothing, we can do.

A

Yeah, I think so: okay, okay, then so, let's see.

D

Let me yell, just one note: is uh I don't I don't I mean if he's using the bridge binding, then whatever ip was said there, it should be faster. I guess.

C

Yeah but I don't know how long.

D

H

C

I don't know how he's putting an ip or not in the in the pod interface, so I just want to look at that.

A

So maybe this should or could also be something what we could probably add on the uh on the issue right. That would help also no.

C

With like with that, we can, we can see it because I mean if the pod interface does not have an ip. I have no clue how this could happen. uh Well, we wouldn't try to pass it along to the guest, but I guess that breaks like kubernetes.

C

I don't know, let's see what he says.

A

Okay, okay, so let's see this one is from or.

C

A

We don't need to look at that, so the next one is by lugo. This one fell to hot clock: sriv dubai interfaces; okay, okay, oh yeah, or is already looking like the info context. Okay,.

A

Just adding the label so that we have it next time.

A

Okay, this one is then the second latest okay, this one is, then the machine, instant status.

C

Fill and let's.

A

Have a quick look at that? Oh.

A

Okay, this is a.

A

Okay, I think we might, at least I can't make anything of that, but.

A

Okay, so let's go down a little bit regardless of this stack trace. Okay, this is a very old coopered version of 35. kubernetes version, 1.12.

A

Okay, so actually can we just can we just um I wouldn't want to just uh just tell him that that we are uh probably not supporting this old version, um but we could at least suggest that he might might be updating to a new version and see if this still occurs somehow or what you would you think I I think yeah, but but if 112, I think he doesn't have any other option right. So.

G

Hi, can you please ask the person to provide uh crd for virtual machine instance.

A

Yeah, that's actually missing here right so.

F

A

Wait a minute isn't, isn't that what we were wanting to look at.

G

This is cr, so this is custom resource, but we want the definition of the custom resource.

A

Oh, you say: okay.

A

So I can change information.

A

Has a chance to upgrade.

A

A

Okay, let's see.

A

This was the last one. Okay, the help cuban run failed an arm. Okay. There are four comments already, so this looks like it's a waste handled.

A

Yeah lots of stuff maya is already looking at that: okay.

A

Okay, so that's done okay, so then I think we should be good with the bug scrub just going to reload reload, the pack we'll stop there. So let me see most of the things have at least a couple of issues or some labels. Okay. So then, let's get back to close this box cup and yeah final call for anyone probably to bring something up before we close the meeting.

A

Oh, I saw that there is another request. I think we have time for that. Probably so um maybe we can have a quick, quick discussion about that. If anyone wants to.

A

So who brought this up, can we can we discuss somehow.

I

Blue, can you hear me fine.

A

Yeah we can hear you.

I

Great okay, hey! This is my pr here.

G

I

uh So effectively it's just uh refactoring some repetitious code in the vmi lifecycle handler added a couple new methods, but mainly just removals of redundant code.

A

Okay, okay, we have already a discussion going on so um so, um okay, so you're just wanting to bring this into to everyone's attention. Or do you have a specific question on this? um How we should proceed or what do you want to ask.

I

I just would like further review and uh you know push it over the hump here. I think it's kind of gone still the past couple weeks.

A

Okay, so yeah. I guess that you've noticed probably that we had some stuff in nci going on and we had a quite a hard time, so maybe uh oh yeah. I see, for example, that yeah, but the job triggers already look good. So, okay, I think there's everything at least regarding from ci side there. It looks that looks good, so yeah. Anyone else wants to want to have a look at that.

A

Besides, I think eddie already looked at that, and I also saw lubo.

A

Can you take us here.

A

Okay, then, so I think I'm I'm going to look at that also, probably so. That helps somehow I'm going to sign myself.

A

G

A

Okay, so let's give this a try.

D

Did you say that I need to do something.

A

What eddie? No! No! I was just saying that that you had already looked at that, and I wanted to ask for other tickets, probably that so that we can bring this pr forward. That was all what I was asking for.

D

Okay, if you, if you need my attention again, then just tag me or something.

A

Yeah sure sure yeah I just saw that this was already reviewed by you. So so we should be good there, okay, um so okay, um then I think that we.

A

Don't have anything more if no one else wants to chime in again and then I can give you a couple of minutes back, which is always great to see and yeah. I wish you everyone a nice rest of the week and yeah. Then we might probably see us next week.

B

Thanks daniel bye, everyone.

A