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

Revisit error status definition #196

Open
yorugac opened this issue Feb 7, 2023 · 1 comment
Open

Revisit error status definition #196

yorugac opened this issue Feb 7, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request PLZ

Comments

@yorugac
Copy link
Collaborator

yorugac commented Feb 7, 2023

In PR #169, error was added to the possible statuses of the CR. Currently, this state happens only on insufficient number of VUs. Consider all the cases when the test run should enter that state and adjust the logic accordingly.

Related issue: #75
Review and close it as needs be.

Docs on k6 error codes:

@yorugac
Copy link
Collaborator Author

yorugac commented Apr 21, 2023

In case of cloud output, failed authentication should cause error state. Right now, error is just logged by the operator while initializer pod exits successfully. K6 resource is stuck in "initialization" and is not being cleaned up.

Consider: missing token case.

@yorugac yorugac added the PLZ label Apr 27, 2023
@yorugac yorugac self-assigned this May 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request PLZ
Projects
None yet
Development

No branches or pull requests

1 participant