tetragon icon indicating copy to clipboard operation
tetragon copied to clipboard

Migrate e2e labels test from `isovalent/jobs-app`

Open mtardy opened this issue 1 year ago • 4 comments
trafficstars

The e2e labels test is suffering from flakes because the jobs-app sometimes fails to deploy, even though we provide enough resources, see https://github.com/cilium/tetragon/issues/1954. The idea would be to migrate from these deployments to something more lightweight and maintained to make this faster and more reliable (thus fix https://github.com/cilium/tetragon/issues/1954).

Our initial idea is to use Google's microservices demo "Online Boutique" https://github.com/GoogleCloudPlatform/microservices-demo with the minimal footprint possible.

The whole e2e labels test (https://github.com/cilium/tetragon/blob/main/tests/e2e/tests/labels/labels_test.go) will need to be adapted first by changing the Helm chart to be deployed: https://github.com/cilium/tetragon/blob/004d9cfe1982614fa4e83a2e55edea2a20f37b56/tests/e2e/tests/labels/labels_test.go#L45-L49

And especially, the checkers, to suit the new deployments' labels: https://github.com/cilium/tetragon/blob/004d9cfe1982614fa4e83a2e55edea2a20f37b56/tests/e2e/tests/labels/labels_test.go#L121-L184

mtardy avatar Jan 15 '24 10:01 mtardy

Hey @mtardy what is the estimated deadline to resolve this issue ?

prateek041 avatar Jan 16 '24 15:01 prateek041

Hey, there is no deadline to resolve this issue.

On 16 Jan 2024, at 16:17, prateek singh @.***> wrote:

Hey @mtardy https://github.com/mtardy what is the estimated deadline to resolve this issue ?

— Reply to this email directly, view it on GitHub https://github.com/cilium/tetragon/issues/1976#issuecomment-1893954573, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACV4B474B3HMWYRJXYORCDLYO2KYRAVCNFSM6AAAAABB3CF2T6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJTHE2TINJXGM. You are receiving this because you were mentioned.

mtardy avatar Jan 16 '24 15:01 mtardy

Ok then, I'm gonna give this a shot. Assigning it to myself.

prateek041 avatar Jan 16 '24 18:01 prateek041

This is the detailed issue @jrfastab

mtardy avatar Feb 08 '24 11:02 mtardy

Hi @mtardy , looks like this issue is still open?. If yes, may I pick up this issue?, please let me know

rajesamp avatar Mar 02 '24 04:03 rajesamp

working on resolving undefined: unix.CBitFieldMaskBit34 -> even after upgrading sys/unix package to .18 and mod vendor, issue still exists.

rajesamp avatar Mar 18 '24 06:03 rajesamp