-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[wip] Break e2e tracking into its own service #1210
[wip] Break e2e tracking into its own service #1210
Conversation
# limitations under the License. | ||
|
||
FROM google/debian:jessie | ||
MAINTAINER Brendan Burns <bburns@google.com> |
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.
really? :)
I'll look more later-- only comment so far is don't check the e2e-tests binary in :) |
fa916ae
to
cccce21
Compare
cccce21
to
93eb498
Compare
So I don't know if I understand the intention here-- do you want the submit queue to be looking at this second service? Or will they both be checking the tests? (I want to make this start checking presubmit runs to file issues. I'll try to not make your rebase too hard. Alternatively if you want to split off something that does the rename & moves I could rebase on top of that?) |
I intend to move all of the e2e stuff to its own daemon which the Everything that runs against all open PRs in one daemon. Everything that deals with the e2e tests in its own daemon. Completely decoupling the logic. More micro-servicy :) I'd say just keep doing what you want to do. Because I don't know when I'll have time to finish that separation... |
Sounds good. Thanks! On Mon, Jun 27, 2016 at 11:46 AM, Eric Paris notifications@github.com
|
@eparis PR needs rebase |
recomputing cla status... |
[CLA-PING] @eparis Thanks for your pull request. It looks like this may be your first contribution to a CNCF open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://identity.linuxfoundation.org/projects/cncf to sign. Once you've signed, please reply here (e.g. "I signed it!") and we'll verify. Thanks.
|
@foxish who owns setting the |
@eparis, could you explain on kubernetes/kubernetes#27796? @caniszczyk should be able to help out. |
Bump (signed it?). (edit: worked, cla/linuxfoundation no->yes) |
The e2e stuff has a life of its own and is too tightly coupled with the merge queue. This starts the coding to break all of the e2e stuff into its own service (which the merge queue can make use of)
This change is