Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI(tests) Use AssertCommand function in every test #1643

Open
PrimalPimmy opened this issue Feb 20, 2024 · 5 comments · May be fixed by #1697
Open

CI(tests) Use AssertCommand function in every test #1643

PrimalPimmy opened this issue Feb 20, 2024 · 5 comments · May be fixed by #1697
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@PrimalPimmy
Copy link
Member

Feature Request

Short Description

The CI tests have been updated to use the Eventually function in the ginkgo tests, however we need to update all the tests, only KSP tests were updated recently.

Is your feature request related to a problem? Please describe the use case.

Today we might get flakey tests and we would need to have a retry logic for executing commands.

in all of our tests
Describe the solution you'd like

A new AssertCommand function was added recently to the utils package. Use this function instead of the K8sExecInPod() in all of our tests:

func AssertCommand(wp string, namespace string, cmd []string, match gomegaTypes.GomegaMatcher, eventual bool) {

NOTE: some the the exec operators do not compare/match with anything and are just meant for executing, in that case you can set Matchregexp(".*") wherever matching is not required.

@PrimalPimmy PrimalPimmy added the enhancement New feature or request label Feb 20, 2024
@PrimalPimmy PrimalPimmy changed the title CI(tests) CI(tests) Use AssertCommand function in every test Feb 20, 2024
@daemon1024 daemon1024 added good first issue Good for newcomers help wanted Extra attention is needed labels Feb 21, 2024
@akshat-kaushik
Copy link

working on it ! pls assign

@rootxrishabh
Copy link
Member

Hey @akshat-kaushik thank you for your interest! Let us know if you have any questions.

@EraKin575
Copy link

Hi!I would like to work on this issue if its not being solved?Can anyone assign me this issue

@EraKin575 EraKin575 linked a pull request Mar 19, 2024 that will close this issue
7 tasks
@bhoopesh369
Copy link

Is this issue resolved?
coz i see the pr that's linked not merged

@bhoopesh369
Copy link

@rootxrishabh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants