Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

testgrid results not appearing for (new) federated test results #4362

Closed
justinsb opened this issue Sep 4, 2017 · 11 comments
Closed

testgrid results not appearing for (new) federated test results #4362

justinsb opened this issue Sep 4, 2017 · 11 comments
Assignees

Comments

@justinsb
Copy link
Member

justinsb commented Sep 4, 2017

I am populating this bucket with federated test results:
https://console.cloud.google.com/storage/browser/kopeio-kubernetes-e2e/logs/kubernetes-e2e-aws

I believe they should now appear here: https://k8s-testgrid.appspot.com/kopeio#kopeio-aws and the gubernator link goes to the right place. But while the job is updating, results are not appearing.

Can someone check the logs from testgrid?

@justinsb
Copy link
Member Author

justinsb commented Sep 4, 2017

The build ids follow this pattern, BTW: 20170904-021830-ad26a7c7-7722

I think that is this pattern: https://github.com/kubernetes/test-infra/blob/master/jenkins/bootstrap.py#L651-L655

But I may have got the wrong pattern or maybe this pattern isn't yet implemented in testgrid?

@luxas
Copy link
Member

luxas commented Sep 4, 2017

Why not 1, 2, 3, 4, ...?

@justinsb
Copy link
Member Author

justinsb commented Sep 4, 2017 via email

@luxas
Copy link
Member

luxas commented Sep 4, 2017

Heh, indeed 😜

@justinsb
Copy link
Member Author

justinsb commented Sep 4, 2017

Inspired by @luxas I sent through a test with build id = 12345 and that did show up.

What are the valid values for a build id? Can we publish the testgrid source, even if it is read-only?

@krzyzacy
Copy link
Member

krzyzacy commented Sep 5, 2017

build number is the subdir name and need to be numerically increasing

@michelle192837
Copy link
Contributor

To the source-publishing question; to my understanding, we can't, no.

@krzyzacy
Copy link
Member

krzyzacy commented Sep 5, 2017

But hopefully we will rewrite an open-sourced version of client in the near future.

@justinsb
Copy link
Member Author

justinsb commented Sep 5, 2017

@krzyzacy can I use YYYYMMDDHHMMSS or will that be too big?

I mentioned it in the meeting, but even if it isn't true open source (i.e. you can't accept contributions), and even if it is a partial snapshot, it would have a lot of value to people in figuring out this stuff or in helping to build testgrid v2.

@krzyzacy
Copy link
Member

krzyzacy commented Sep 5, 2017

@justinsb I think it need to fit into an int (?) - so basically testgrid caches the largest processed build id, and next time it will skip once it hits a build id smaller/equal to the largest processed build id.

For second part, I cannot make the call, <----- @fejta

@krzyzacy
Copy link
Member

krzyzacy commented Sep 6, 2017

I'll close this issue, we can move the opensource discussion to #3323

@krzyzacy krzyzacy closed this as completed Sep 6, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants