{"push_id":1709119023,"size":27,"distinct_size":27,"ref":"refs/heads/bind-1.7","head":"291687f3eaa7873e31b83c31ffc7a7efea3e4b4e","before":"38efb955ebda1a57bce4932bca4c6bb2138bb619","commits":[{"sha":"533d4bfd546a7a11fb6b358265e0c6facc4e24ae","author":{"name":"Konstantinos Tsakalozos","email":"005079433f6c96f3479b89bce7c5aef5144a7cb0@canonical.com"},"message":"Fixes: 1. Get certs for a dead leader. 2. Append tokens.","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/533d4bfd546a7a11fb6b358265e0c6facc4e24ae"},{"sha":"12a5c3a2f31a758d85c1cb9bce2d999d73b00f1d","author":{"name":"Konstantinos Tsakalozos","email":"005079433f6c96f3479b89bce7c5aef5144a7cb0@canonical.com"},"message":"Fixing bug: non-leaders should retry fetching auth keys","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/12a5c3a2f31a758d85c1cb9bce2d999d73b00f1d"},{"sha":"9daa13af3eeb4603255493b4b08c03c513607234","author":{"name":"Derek McQuay","email":"d0630b92905ee748db3032d174b6bbb8f5bda4d1@apprenda.com"},"message":"kubeadm: kubeadm join test cmds for new flags","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/9daa13af3eeb4603255493b4b08c03c513607234"},{"sha":"cc5a309fd7b3091d6674a1385068c9c1c2f9424a","author":{"name":"Cezar Sa Espinola","email":"3bd45eca7064b89e5a0486f5bacd9eb977501023@gmail.com"},"message":"prevent corrupted spdy stream after hijacking connection\n\nReading directly from a hijacked connection isn't safe because some\ndata may have already been read by the server before Hijack was called.\nTo ensure all data will be received it's safer to read from the returned\nbufio.Reader.","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/cc5a309fd7b3091d6674a1385068c9c1c2f9424a"},{"sha":"a7dd554ce3bebf4f4eb392fb7edf3203de094e77","author":{"name":"weizhanxu","email":"3351a38d814b9b4b0994ef50ed5ced1983322bb7@zte.com.cn"},"message":"no need check is nil, because has check before here","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/a7dd554ce3bebf4f4eb392fb7edf3203de094e77"},{"sha":"ab4029c85b037bbb590bd114d057cf3144d00277","author":{"name":"bruceauyeung","email":"1a2be6bf2d0f9557cdf7c9e7ea2970a9bf893607@zte.com.cn"},"message":"Signed-off-by: bruceauyeung <ouyang.qinhua@zte.com.cn>\n\nrename variables to make sure that they conform to golang variable name convention","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/ab4029c85b037bbb590bd114d057cf3144d00277"},{"sha":"e02db69826c2028e9c1ab0af58e4e1641c1378b7","author":{"name":"Jacek Nykis","email":"1f79d749e587d5de149ba1fd9bc96d4b1afe620a@canonical.com"},"message":"Use correct option name in the kubernetes-worker registry action. Fixes #44920","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/e02db69826c2028e9c1ab0af58e4e1641c1378b7"},{"sha":"f5fa115536c7851181e1862e86f1ae1353f0a572","author":{"name":"Eric Chiang","email":"3038b30a67389788b7adfe6aecdffba1dc77ea3d@gmail.com"},"message":"kube-apiserver: improve bootstrap token authentication error messages","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/f5fa115536c7851181e1862e86f1ae1353f0a572"},{"sha":"a5d86270d3ce2a956a21e83bb200f51431adb52c","author":{"name":"Shayne Wang","email":"55a7c2b51aa34b2876d2e9f67bc9c60acfce9df8@gmail.com"},"message":"added coverage for scheduler/algorithm/types_test.go","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/a5d86270d3ce2a956a21e83bb200f51431adb52c"},{"sha":"f2aa330a38a412320aa32dec6d248956de1b1649","author":{"name":"Hemant Kumar","email":"d37ca1f57b0cd9ec1cdf94bef11b4b475ecc8013@redhat.com"},"message":"Start recording cloud provider metrics for AWS\n\nLets start recording storage metrics for AWS.","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/f2aa330a38a412320aa32dec6d248956de1b1649"},{"sha":"9afeabb642e03c9c50b8dea1d43785eade7f6088","author":{"name":"Kubernetes Submit Queue","email":"260061d766fe633e5411cb223defd984f1ece7b6@users.noreply.github.com"},"message":"Merge pull request #43477 from gnufied/cloudprovider-aws-metrics\n\nAutomatic merge from submit-queue\n\nStart recording cloud provider metrics for AWS\n\n**What this PR does / why we need it**:\r\n\r\nThis PR implements support for emitting metrics from AWS about storage operations.\r\n\r\n**Which issue this PR fixes** \r\n\r\nFixes https://github.com/kubernetes/features/issues/182\r\n\r\n**Release note**:\r\n```\r\nAdd support for emitting metrics from AWS cloudprovider about storage operations.\r\n```","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/9afeabb642e03c9c50b8dea1d43785eade7f6088"},{"sha":"fb3e30729ecf90f4f11af10f852545f07217b69b","author":{"name":"Mik Vyatskov","email":"714823502c74551132d79cd4f17a3753961f101b@google.com"},"message":"Add metrics exporter to the fluentd-gcp deployment","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/fb3e30729ecf90f4f11af10f852545f07217b69b"},{"sha":"f4f95be4e7663cb3ab81270a59dd06fb79e947bc","author":{"name":"Davanum Srinivas","email":"2ddd2dda6bd370eec0ab604a69d844da6cc90a39@gmail.com"},"message":"Mark PersistentVolumes as [Feature:Volumes]\n\nJust so that we know that we need a cloud provider that\nsupports volumes to run this test. This is similar to\nthe change in 63bc42c87241bcfc2ff4ef071aaca5dac1be4049.\n\nRan into this when i was trying to run e2e tests with\nlocal-up-cluster locally and figured out this test will\nnot work since we don't support local storage persistent\nvolumes.","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/f4f95be4e7663cb3ab81270a59dd06fb79e947bc"},{"sha":"8787b13d756c0078156b179bb22c567cf6b22ae9","author":{"name":"Kubernetes Submit Queue","email":"260061d766fe633e5411cb223defd984f1ece7b6@users.noreply.github.com"},"message":"Merge pull request #43922 from cezarsa/spdy-fix\n\nAutomatic merge from submit-queue\n\nprevent corrupted spdy stream after hijacking connection\n\nThis PR fixes corner case in spdy stream code where some bytes would never arrive at the server.\r\n\r\nReading directly from a hijacked connection isn't safe because some data may have already been read by the server before `Hijack` was called. To ensure all data will be received it's safer to read from the returned `bufio.Reader`. This problem seem to happen more frequently when using Go 1.8.\r\nThis is described in https://golang.org/pkg/net/http/#Hijacker:\r\n\r\n> // The returned bufio.Reader may contain unprocessed buffered\r\n   // data from the client.\r\n\r\nI came across this while debugging a flaky test that used code from the `k8s.io/apimachinery/pkg/util/httpstream/spdy` package. After filling the code with debug logs and long hours running the tests in loop in the hope of catching the error I finally caught something weird.\r\n\r\nThe first word on the first spdy frame [read by the server here](https://github.com/kubernetes/kubernetes/blob/b625085230f384fa7f2ac161a5ec15cd8dcbc896/vendor/github.com/docker/spdystream/spdy/read.go#L148) had the value `0x03000100`. See, the first frame to arrive on the server was supposed to be a control frame indicating the creation of a new stream, but all control frames need the high-order bit set to 1, which was not the case here, so the saver mistakenly assumed this was a data frame and the stream would never be created. The correct value for the first word of a SYN_STREAM frame was supposed to be `0x80030001` and this lead me on the path of finding who had consumed the first 1 byte prior to the frame reader being called and finally finding the problem with the Hijack call.\r\n\r\nI added a new test to try stressing this condition and ensuring that this bug doesn't happen anymore. However, it's quite ugly as it loops 1000 times creating streams on servers to increase the chances of this bug happening. So, I'm not sure whether it's worth it to keep this test or if I should remove it from the PR. Please let me know what you guys think and I'll be happy to update this.\r\n\r\nFixes #45093 #45089 #45078 #45075 #45072 #45066 #45023","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/8787b13d756c0078156b179bb22c567cf6b22ae9"},{"sha":"e99d6062774bbadd7aac4c3cef28574652d3c018","author":{"name":"Konstantinos Tsakalozos","email":"005079433f6c96f3479b89bce7c5aef5144a7cb0@canonical.com"},"message":"Merge branch 'master' into bug/juju-master","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/e99d6062774bbadd7aac4c3cef28574652d3c018"},{"sha":"021f542f241bf388c419491013482e5e9a0c6a05","author":{"name":"Kubernetes Submit Queue","email":"260061d766fe633e5411cb223defd984f1ece7b6@users.noreply.github.com"},"message":"Merge pull request #42432 from shaynewang/test_types\n\nAutomatic merge from submit-queue\n\nImproved code coverage for plugin/pkg/scheduler/algorithm\n\n**What this PR does / why we need it**:\r\nPart of #39559 , code coverage improved from 0% to 100%\r\n**Special notes for your reviewer**:\r\nImproved coverage for scheduler/algorithm to 100%\r\nTest cover output:\r\n```\r\nmake test WHAT=./plugin/pkg/scheduler/algorithm KUBE_COVER=y\r\nRunning tests for APIVersion: v1,apps/v1beta1,authentication.k8s.io/v1,authentication.k8s.io/v1beta1,authorization.k8s.io/v1,authorization.k8s.io/v1beta1,autoscaling/v1,autoscaling/v2alpha1,batch/v1,batch/v2alpha1,certificates.k8s.io/v1beta1,extensions/v1beta1,imagepolicy.k8s.io/v1alpha1,policy/v1beta1,rbac.authorization.k8s.io/v1beta1,rbac.authorization.k8s.io/v1alpha1,storage.k8s.io/v1beta1,federation/v1beta1\r\n+++ [0302 10:43:05] Saving coverage output in '/tmp/k8s_coverage/v1,apps/v1beta1,authentication.k8s.io/v1,authentication.k8s.io/v1beta1,authorization.k8s.io/v1,authorization.k8s.io/v1beta1,autoscaling/v1,autoscaling/v2alpha1,batch/v1,batch/v2alpha1,certificates.k8s.io/v1beta1,extensions/v1beta1,imagepolicy.k8s.io/v1alpha1,policy/v1beta1,rbac.authorization.k8s.io/v1beta1,rbac.authorization.k8s.io/v1alpha1,storage.k8s.io/v1beta1,federation/v1beta1/20170302-104305'\r\nskipped\tk8s.io/kubernetes/cmd/libs/go2idl/generator\r\nskipped\tk8s.io/kubernetes/vendor/k8s.io/client-go/1.4/rest\r\nok  \tk8s.io/kubernetes/plugin/pkg/scheduler/algorithm\t0.061s\tcoverage: 100.0% of statements\r\n+++ [0302 10:43:07] Combined coverage report: /tmp/k8s_coverage/v1,apps/v1beta1,authentication.k8s.io/v1,authentication.k8s.io/v1beta1,authorization.k8s.io/v1,authorization.k8s.io/v1beta1,autoscaling/v1,autoscaling/v2alpha1,batch/v1,batch/v2alpha1,certificates.k8s.io/v1beta1,extensions/v1beta1,imagepolicy.k8s.io/v1alpha1,policy/v1beta1,rbac.authorization.k8s.io/v1beta1,rbac.authorization.k8s.io/v1alpha1,storage.k8s.io/v1beta1,federation/v1beta1/20170302-104305/combined-coverage.html\r\n```","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/021f542f241bf388c419491013482e5e9a0c6a05"},{"sha":"228219b0ce3865da528a224a844b04f6be8c45e5","author":{"name":"Kubernetes Submit Queue","email":"260061d766fe633e5411cb223defd984f1ece7b6@users.noreply.github.com"},"message":"Merge pull request #44628 from dmmcquay/kubeadm_join_tests\n\nAutomatic merge from submit-queue (batch tested with PRs 42432, 44628, 45101, 44921)\n\nkubeadm: join test cmds for new flags\n\n**What this PR does / why we need it**: Adding test-cmds for new kubeadm join flags. \r\n\r\nAdding tests is a WIP from #34136\r\n\r\nThis is a continuation from https://github.com/kubernetes/kubernetes/pull/42812 since it had to be closed.\r\n\r\n**Special notes for your reviewer**: /cc @luxas \r\n\r\n**Release note**:\r\n```release-note\r\nNONE\r\n```","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/228219b0ce3865da528a224a844b04f6be8c45e5"},{"sha":"774bb58e02736a4c86d952892d16d714b483e65f","author":{"name":"Kubernetes Submit Queue","email":"260061d766fe633e5411cb223defd984f1ece7b6@users.noreply.github.com"},"message":"Merge pull request #45101 from dims/mark-test-with-feature-volumes\n\nAutomatic merge from submit-queue (batch tested with PRs 42432, 44628, 45101, 44921)\n\nMark PersistentVolumes as [Feature:Volumes]\n\n**What this PR does / why we need it**:\r\nJust so that we know that we need a cloud provider that\r\nsupports volumes to run this test. This is similar to\r\nthe change in 63bc42c87241bcfc2ff4ef071aaca5dac1be4049.\r\n\r\nRan into this when i was trying to run e2e tests with\r\nlocal-up-cluster locally and figured out this test will\r\nnot work since we don't support local storage persistent\r\nvolumes.\r\n\r\n**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #\r\n\r\n**Special notes for your reviewer**:\r\n\r\n**Release note**:\r\n\r\n```release-note\r\nNONE\r\n```","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/774bb58e02736a4c86d952892d16d714b483e65f"},{"sha":"bec2c604acdf08aae46093552f02011dfc018fde","author":{"name":"Kubernetes Submit Queue","email":"260061d766fe633e5411cb223defd984f1ece7b6@users.noreply.github.com"},"message":"Merge pull request #44921 from jacekn/registry-fix\n\nAutomatic merge from submit-queue (batch tested with PRs 42432, 44628, 45101, 44921)\n\nUse correct option name in the kubernetes-worker layer registry action\n\n**What this PR does / why we need it**: It fixes #44920 \r\n\r\n**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #44920 \r\n\r\n**Special notes for your reviewer**:\r\n\r\n**Release note**:\r\n\r\n```\r\nEnsure kubernetes-worker juju layer registry action uses correct ingress controller option name\r\n```","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/bec2c604acdf08aae46093552f02011dfc018fde"},{"sha":"94567f64e1fb6f73b8926ef7a1edbd9293d7748e","author":{"name":"Kubernetes Submit Queue","email":"260061d766fe633e5411cb223defd984f1ece7b6@users.noreply.github.com"},"message":"Merge pull request #45096 from crassirostris/fluentd-metrics-exporter\n\nAutomatic merge from submit-queue\n\nAdd metrics exporter to the fluentd-gcp deployment\n\nMetrics exporter container reads metrics from the `/metrics` endpoint in fluentd and exports them directly to the Stackdriver. It assumes that Stackdriver Monitoring API is enabled.\r\n\r\n/cc @fgrzadkowski","distinct":true,"url":"https://api.github.com/repos/ravigadde/kubernetes/commits/94567f64e1fb6f73b8926ef7a1edbd9293d7748e"}]}
{"actor":{"display_login":"ravigadde"}}