{"action":"created","issue":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/30894","repository_url":"https://api.github.com/repos/kubernetes/kubernetes","labels_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/30894/labels{/name}","comments_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/30894/comments","events_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/30894/events","html_url":"https://github.com/kubernetes/kubernetes/pull/30894","id":171940769,"number":30894,"title":"Fix overlong junit filename prefixes","user":{"login":"ivan4th","id":190448,"avatar_url":"https://avatars.githubusercontent.com/u/190448?v=3","gravatar_id":"","url":"https://api.github.com/users/ivan4th","html_url":"https://github.com/ivan4th","followers_url":"https://api.github.com/users/ivan4th/followers","following_url":"https://api.github.com/users/ivan4th/following{/other_user}","gists_url":"https://api.github.com/users/ivan4th/gists{/gist_id}","starred_url":"https://api.github.com/users/ivan4th/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ivan4th/subscriptions","organizations_url":"https://api.github.com/users/ivan4th/orgs","repos_url":"https://api.github.com/users/ivan4th/repos","events_url":"https://api.github.com/users/ivan4th/events{/privacy}","received_events_url":"https://api.github.com/users/ivan4th/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/cla:%20yes","name":"cla: yes","color":"bfe5bf"},{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/release-note-none","name":"release-note-none","color":"c2e0c6"},{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/retest-not-required","name":"retest-not-required","color":"eb6420"},{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/size/XS","name":"size/XS","color":"009900"}],"state":"open","locked":false,"assignee":{"login":"lavalamp","id":647318,"avatar_url":"https://avatars.githubusercontent.com/u/647318?v=3","gravatar_id":"","url":"https://api.github.com/users/lavalamp","html_url":"https://github.com/lavalamp","followers_url":"https://api.github.com/users/lavalamp/followers","following_url":"https://api.github.com/users/lavalamp/following{/other_user}","gists_url":"https://api.github.com/users/lavalamp/gists{/gist_id}","starred_url":"https://api.github.com/users/lavalamp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lavalamp/subscriptions","organizations_url":"https://api.github.com/users/lavalamp/orgs","repos_url":"https://api.github.com/users/lavalamp/repos","events_url":"https://api.github.com/users/lavalamp/events{/privacy}","received_events_url":"https://api.github.com/users/lavalamp/received_events","type":"User","site_admin":false},"assignees":[{"login":"lavalamp","id":647318,"avatar_url":"https://avatars.githubusercontent.com/u/647318?v=3","gravatar_id":"","url":"https://api.github.com/users/lavalamp","html_url":"https://github.com/lavalamp","followers_url":"https://api.github.com/users/lavalamp/followers","following_url":"https://api.github.com/users/lavalamp/following{/other_user}","gists_url":"https://api.github.com/users/lavalamp/gists{/gist_id}","starred_url":"https://api.github.com/users/lavalamp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lavalamp/subscriptions","organizations_url":"https://api.github.com/users/lavalamp/orgs","repos_url":"https://api.github.com/users/lavalamp/repos","events_url":"https://api.github.com/users/lavalamp/events{/privacy}","received_events_url":"https://api.github.com/users/lavalamp/received_events","type":"User","site_admin":false}],"milestone":null,"comments":4,"created_at":"2016-08-18T16:13:00Z","updated_at":"2016-08-18T21:11:38Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/kubernetes/kubernetes/pulls/30894","html_url":"https://github.com/kubernetes/kubernetes/pull/30894","diff_url":"https://github.com/kubernetes/kubernetes/pull/30894.diff","patch_url":"https://github.com/kubernetes/kubernetes/pull/30894.patch"},"body":"Build scripts construct filenames that are longer than 255 chars when producing JUnit output for tests and `KUBE_COVER=y`:\r\n\r\n```\r\nvagrant@devbox:~/work/kubernetes/src/k8s.io/kubernetes (test %) $ KUBE_JUNIT_REPORT_DIR=/tmp/art KUBE_COVER=y make test\r\nE0818 15:33:29.312549    9426 conversion.go:589] Warning: could not generate autoConvert functions for k8s.io/kubernetes/pkg/apis/extensions/v1beta1.HorizontalPodAutoscalerSpec <-> k8s.io/kubernetes/pkg/apis/autoscaling.HorizontalPodAutoscalerSpec\r\nE0818 15:33:29.323685    9426 conversion.go:589] Warning: could not generate autoConvert functions for k8s.io/kubernetes/pkg/apis/extensions/v1beta1.JobSpec <-> k8s.io/kubernetes/pkg/apis/batch.JobSpec\r\nE0818 15:33:29.344775    9426 conversion.go:589] Warning: could not generate autoConvert functions for k8s.io/kubernetes/pkg/apis/extensions/v1beta1.RollingUpdateDeployment <-> k8s.io/kubernetes/pkg/apis/extensions.RollingUpdateDeployment\r\nE0818 15:33:29.349260    9426 conversion.go:589] Warning: could not generate autoConvert functions for k8s.io/kubernetes/pkg/apis/extensions/v1beta1.ScaleStatus <-> k8s.io/kubernetes/pkg/apis/extensions.ScaleStatus\r\nRunning tests for APIVersion: v1,apps/v1alpha1,authentication.k8s.io/v1beta1,authorization.k8s.io/v1beta1,autoscaling/v1,batch/v1,batch/v2alpha1,certificates/v1alpha1,extensions/v1beta1,federation/v1beta1,policy/v1alpha1,rbac.authorization.k8s.io/v1alpha1,imagepolicy.k8s.io/v1alpha1\r\n+++ [0818 15:33:30] Saving coverage output in '/tmp/k8s_coverage/v1,apps/v1alpha1,authentication.k8s.io/v1beta1,authorization.k8s.io/v1beta1,autoscaling/v1,batch/v1,batch/v2alpha1,certificates/v1alpha1,extensions/v1beta1,federation/v1beta1,policy/v1alpha1,rbac.authorization.k8s.io/v1alpha1,imagepolicy.k8s.io/v1alpha1/20160818-153330'\r\ntee: /tmp/art/junit_v1,apps-v1alpha1,authentication.v1beta1,authorization.v1beta1,autoscaling-v1,batch-v1,batch-v2alpha1,certificates-v1alpha1,extensions-v1beta1,federation-v1beta1,policy-v1alpha1,rbac.authorization.v1alpha1,imagepolicy.v1alpha1_20160818-153330-cmd_genutils.stdout: File name too long\r\ntee: /tmp/art/junit_v1,apps-v1alpha1,authentication.v1beta1,authorization.v1beta1,autoscaling-v1,batch-v1,batch-v2alpha1,certificates-v1alpha1,extensions-v1beta1,federation-v1beta1,policy-v1alpha1,rbac.authorization.v1alpha1,imagepolicy.v1alpha1_20160818-153330-cmd_kube-apiserver_app.stdout: File name too long\r\ntee: /tmp/art/junit_v1,apps-v1alpha1,authentication.v1beta1,authorization.v1beta1,autoscaling-v1,batch-v1,batch-v2alpha1,certificates-v1alpha1,extensions-v1beta1,federation-v1beta1,policy-v1alpha1,rbac.authorization.v1alpha1,imagepolicy.v1alpha1_20160818-153330-cmd_hyperkube.stdout: File name too long\r\ntee: /tmp/art/junit_v1,apps-v1alpha1,authentication.v1beta1,authorization.v1beta1,autoscaling-v1,batch-v1,batch-v2alpha1,certificates-v1alpha1,extensions-v1beta1,federation-v1beta1,policy-v1alpha1,rbac.authorization.v1alpha1,imagepolicy.v1alpha1_20160818-153330-cmd_kube-apiserver_app_options.stdout: File name too long\r\nok  \tk8s.io/kubernetes/cmd/genutils\t0.002s\r\ntee: /tmp/art/junit_v1,apps-v1alpha1,authentication.v1beta1,authorization.v1beta1,autoscaling-v1,batch-v1,batch-v2alpha1,certificates-v1alpha1,extensions-v1beta1,federation-v1beta1,policy-v1alpha1,rbac.authorization.v1alpha1,imagepolicy.v1alpha1_20160818-153330-cmd_kubelet_app.stdout: File name too long\r\nqok  \tk8s.io/kubernetes/cmd/kube-apiserver/app/options\t0.159s\r\ntee: /tmp/art/junit_v1,apps-v1alpha1,authentication.v1beta1,authorization.v1beta1,autoscaling-v1,batch-v1,batch-v2alpha1,certificates-v1alpha1,extensions-v1beta1,federation-v1beta1,policy-v1alpha1,rbac.authorization.v1alpha1,imagepolicy.v1alpha1_20160818-153330-cmd_kube-proxy_app.stdout: File name too long\r\n```\r\n\r\nLet's shorten filenames a bit more to make them fit in 255 chars. This is probably temporary solution too, the bulletproof one being (e.g.) using `sha1sum` of `KUBE_TEST_API` but that would produce files with meaningless filenames so for now maybe it may wait.\r\n\r\n(there was recent attempt to shorten the filenames by removing `k8s.io`s, but that was not enough for `KUBE_COVER=y`)\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.kubernetes.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.kubernetes.io/reviews/kubernetes/kubernetes/30894)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/comments/240858236","html_url":"https://github.com/kubernetes/kubernetes/pull/30894#issuecomment-240858236","issue_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/30894","id":240858236,"user":{"login":"k8s-bot","id":12551979,"avatar_url":"https://avatars.githubusercontent.com/u/12551979?v=3","gravatar_id":"","url":"https://api.github.com/users/k8s-bot","html_url":"https://github.com/k8s-bot","followers_url":"https://api.github.com/users/k8s-bot/followers","following_url":"https://api.github.com/users/k8s-bot/following{/other_user}","gists_url":"https://api.github.com/users/k8s-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/k8s-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/k8s-bot/subscriptions","organizations_url":"https://api.github.com/users/k8s-bot/orgs","repos_url":"https://api.github.com/users/k8s-bot/repos","events_url":"https://api.github.com/users/k8s-bot/events{/privacy}","received_events_url":"https://api.github.com/users/k8s-bot/received_events","type":"User","site_admin":false},"created_at":"2016-08-18T21:11:37Z","updated_at":"2016-08-18T21:11:37Z","body":"GCE e2e build/test **passed** for commit 98ea7b944143aa854b0101def2273e0f8ef12b52.\n* [Test Results](https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/pr-logs/pull/30894/kubernetes-pull-build-test-e2e-gce/54492)\n* [Build Log](http://pr-test.k8s.io/30894/kubernetes-pull-build-test-e2e-gce/54492/build-log.txt)\n* [Test Artifacts](https://console.developers.google.com/storage/browser/kubernetes-jenkins/pr-logs/pull/30894/kubernetes-pull-build-test-e2e-gce/54492/artifacts/)\n* [Internal Jenkins Results](http://goto.google.com/prkubekins/job/kubernetes-pull-build-test-e2e-gce//54492)\n\n"}}
{"actor":{"display_login":"k8s-bot"}}