test: Bump e2e test package versions#2218
Conversation
|
Abhijeet Prasad (@AbhiPrasad) Stephen Belanger (@Qard) I know this is a bit ridiculous to review. I honestly did not look at all the outputs myself. This is purely additive and adding/bumping versions, so we have more visibility. We just gotta make sure that the adjustments we did in the scenarios are sane. |
Abhijeet Prasad (AbhiPrasad)
left a comment
There was a problem hiding this comment.
trust the process
Stephen Belanger (Qard)
left a comment
There was a problem hiding this comment.
I have very little trust in any system which results in 200k line PRs. Nobody is going to review that fully, it would take days to do so. 😬
Stephen Belanger (@Qard) what's your suggestion to proceed? |
|
I won't block merging, because being up-to-date is better than not, I guess. But I definitely think we should rethink the value of this e2e set up. It's way too big to be able to update it with any confidence of correctness. At the least we should probably have a conversation about it at a future team sync to think about what we can do to refactor or replace this with something we can actually trust. |
sgtm! This is the best that I could come up with. Feel free to share any thoughts on what you think is optimal. |
No description provided.