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.

Sign up for GitHub

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

Jump to bottom

E2E creation test should measure creation time #90

Open
m1kola opened this issue Jan 20, 2020 · 2 comments
Open

E2E creation test should measure creation time #90

m1kola opened this issue Jan 20, 2020 · 2 comments
Labels
priority-medium Medium priority issue or pull request size-medium Size medium

Comments

@m1kola
Copy link
Contributor

m1kola commented Jan 20, 2020

I would like to have an e2e test which apart from validating the create operation itself also makes sure that we do not accidentally make cluster creation longer by making changes in the install process or by updating dependecies (installer, for example).

This might require a discussion to define time limit which we should not exceed.
@m1kola m1kola added priority-medium Medium priority issue or pull request size-medium Size medium labels Jan 20, 2020
@m1kola m1kola mentioned this issue Jan 20, 2020
Open
@m1kola m1kola changed the title E2E creation test should mesure creation time E2E creation test should measure creation time Jan 20, 2020
@jim-minter
Copy link
Member

It might be that this is solveable/solved by reading RP metrics when running under test rather than adding specific e2e test metrics, but the request is a valid one

@sjkingo
Copy link

sjkingo commented Apr 17, 2020

@jim-minter would you like me to look into this given my work on #91?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority-medium Medium priority issue or pull request size-medium Size medium
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@sjkingo @m1kola @jim-minter