-
Notifications
You must be signed in to change notification settings - Fork 9
Make the defaultly skipped integration tests run sequentially #244
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
Conversation
…ntially only after integ job finishes.
|
I've run integration tests to check if those skipped integration tests actually do run sequentially now. They do. Note: These integration tests were run on commit 4adb6e1. Here are the timestamps of each integration test start to make it easier to see the results.
When each of the tests finished, there was a bit of a delay and then the next one started as expected. |
Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
github UI is unreadable Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
OIDC is not fully configured (client id is just "todo") Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
justinc1
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ty.
I added some minor changes. Test certificate module in sequential job. With OIDC configured on VSNSs, we can test there also inventory plugin with OIDC login.
The purpose of this PR is, to make the integration tests that are skipped by default run sequentially after the
integjob.For this, another job was added:
integ-sequentially.