Skip to content

Conversation

@baude
Copy link
Member

@baude baude commented Jul 31, 2018

Third round of speed improvements to the integration tests.

Signed-off-by: baude bbaude@redhat.com

@mheon
Copy link
Member

mheon commented Jul 31, 2018

LGTM

@baude baude force-pushed the integrationtestspeedups3 branch from d4fbd25 to 9c53e31 Compare July 31, 2018 14:48
Third round of speed improvements to the integration tests.

Signed-off-by: baude <bbaude@redhat.com>
@baude baude force-pushed the integrationtestspeedups3 branch from 9c53e31 to 83bd273 Compare July 31, 2018 17:49
@rhatdan
Copy link
Member

rhatdan commented Aug 1, 2018

I am fine with this, but I think we have enough speed up. I still see us removing dirversity in the the testing, which makes me worried.
I think at this point we are reaching the point of deminishing returns. And potentially harming our ability to find regressions.
@rh-atomic-bot r+

@rh-atomic-bot
Copy link
Collaborator

📌 Commit 83bd273 has been approved by rhatdan

@rh-atomic-bot
Copy link
Collaborator

⚡ Test exempted: merge already tested.

@baude baude deleted the integrationtestspeedups3 branch December 22, 2019 18:55
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 26, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants