profile
viewpoint
Eric Greer integrii @Comcast Gig Harbor, WA https://ericgreer.info When your biggest problems aren't big problems, you start fixing things that don't need fixing.

Comcast/kuberhealthy 794

A Kubernetes operator for running synthetic checks as pods. Works great with Prometheus!

integrii/flaggy 679

Idiomatic Go input parsing with subcommands, positional values, and flags at any position. No required project or package layout and no external dependencies.

integrii/go-redis-migrator 28

A cluster aware Redis key migrator. Moves keys from one cluster or host to another cluster or host.

integrii/A858DE45F56D9BC9 9

Attempting to decode the mysterious /r/A858DE45F56D9BC9 subreddit data

integrii/bashgame 8

A command line game written in bash. Great for learning to use github and bash!

integrii/cadvisor-daemonset 5

A kubernetes daemonset for cadvisor which exposes a web interface on every node.

integrii/FreeTakServerContainer 4

A docker container for FreeTakServer

integrii/.vim 3

A simple but useful vim setup that has few conflicts, overrides, or complicated configs.

integrii/define 3

Defines the supplied word on the command line. Golang.

push eventComcast/kuberhealthy

Mike Tougeron

commit sha 230c4f1b74cead4e16765b8df9b4206086206311

allow setting a priority class name for daemonsets

view details

Mike Tougeron

commit sha 68792ad8332d85bb5b930dcd07c421fb72191beb

variable naming consistency

view details

Eric Greer

commit sha a71205eda64589a99f38160c2f61216ac2d0320a

Merge pull request #646 from mtougeron/ds-pod-priority Allow setting a priorityClassName for DaemonSet check

view details

push time in 5 days

PR merged Comcast/kuberhealthy

Reviewers
Allow setting a priorityClassName for DaemonSet check

This PR allows for you to optionally set a priorityClassName for the DaemonSet that is created in the check. This allows for preemption when some nodes are fully utilized.

While preemption isn't ideal, knowing the state of the synthetic checks & overall cluster health are a priority workload.

+20 -3

1 comment

6 changed files

mtougeron

pr closed time in 5 days

PullRequestReviewEvent

pull request commentComcast/kuberhealthy

feat: add external dns status check to helm templates

That was exactly it. We have to be very careful about what defaults we enable. I would rather see people have to opt-in to several things (and make that easy) than to risk Kuberhealthy being broken on some clusters.

kuzm1ch

comment created time in 6 days

delete branch Comcast/kuberhealthy

delete branch : ssl-check-minor-fix

delete time in 6 days

push eventComcast/kuberhealthy

zhanso101

commit sha 0a4c49aed00fd923e7b7fbc2dbfdfce0ee04bc7a

Fixes check name typos and comment variable name typo.

view details

zhanso101

commit sha 7ef2317531ca420b761af613419e6bb609192838

Fixes check name typos and README typos/incorrect check name.

view details

Zach H

commit sha cbff61b5e793cc10b25c5f5d01ed93e48c4590c7

Update ssl-secure-expiry-check.yaml Ups time out time for SSL checks

view details

Zach H

commit sha 1764e3fd259ae9d8bcaec19e6ef8cbafa3412ba2

Update ssl-handshake-check.yaml

view details

Zach H

commit sha 05c4a2f49099ff8a00d984dd7fe36b7a67b9f102

Update ssl-selfsign-handshake-check.yaml

view details

Zach H

commit sha 7ebb7644e821592a16182d66f7340a4d1aab8abc

Update ssl-selfsign-expiry-check.yaml

view details

Zach H

commit sha b530136c0a3e08685cecb45bdc94c7a421cd9257

Update main.go

view details

Zach H

commit sha 2050b839de7b5c470f40bb966deeb7a3eb9b99d4

Update ssl-handshake-check.yaml

view details

Eric Greer

commit sha 5e4ac3346af6fcbedfc38d9c88834ff0b765de8b

Merge pull request #651 from Comcast/ssl-check-minor-fix Ssl check minor fix

view details

push time in 6 days

PR merged Comcast/kuberhealthy

Ssl check minor fix

Fixes to check versions/names in spec sheets and readme files, comment typo.

+19 -19

0 comment

8 changed files

zjhans

pr closed time in 6 days

PullRequestReviewEvent

startedmtougeron/kuberhealthy-ami-exists-check

started time in 6 days

Pull request review commentComcast/kuberhealthy

Ssl check minor fix

 spec:           # If not using default SSL port of 443, port env variable must be updated             - name: PORT             value: "443"-        image: kuberhealthy/ssl-selfsign-handshake:v1.2.0

If you're changing image names, be sure that the github action (under .github/workflows at the root) has the right info in it. I would also modify the Makefile for the check so that it builds with the right image name and tag.

zjhans

comment created time in 7 days

Pull request review commentComcast/kuberhealthy

Ssl check minor fix

 func init() { 	} 	insecureCheck = os.Getenv("INSECURE") 	if len(daysToExpire) == 0 {
	if len(insecureCheck) == 0 {
zjhans

comment created time in 7 days

PullRequestReviewEvent
PullRequestReviewEvent

issue commentComcast/kuberhealthy

Test Kuberhealthy 2.3 in lab clusters

It looks like #648 may be new in this 2.3 release. We should fix that before a release.

integrii

comment created time in 7 days

issue openedComcast/kuberhealthy

Improve automation for releases

Relevant to #652

We should ensure the triggers for github actions cause the following workflow:

  • Whenever a tag is created, the helm chart Chart.yaml should be updated to the matching appVersion and the version tag for the chart should be updated. This will be in the form of an automatic PR.
  • When the helm chart is modified, a new action should kick off that causes the chart repository to be regenerated (adding the new chart version). This will cause an automated PR.

created time in 7 days

issue openedComcast/kuberhealthy

Re-enable unit tests in Kuberhealthy dockerfile

Kuberhealthy's container build currently is missing unit tests. The below line is commented out:

#RUN go test -v -short -- --debug --forceMaster

We should re-enable this and ensure all tests pass. Bonus points for fixing existing tests instead of disabling them.

created time in 7 days

push eventComcast/kuberhealthy

cbitze7387k

commit sha a080dcc59a0b91a3ef990b30d3047f271ecf3182

change trigger event to fix triggering on branch creation

view details

cbitze7387k

commit sha 149858ddb8d92caad12404fa7dda518c69f0a052

modify release>published syntax

view details

cbitze7387k

commit sha 67ed4f85a02b54b5a5bcede0940b2014c6b8e7fb

Edit workflow trigger to eliminate error and multiple trigger events

view details

cbitze7387k

commit sha 08899ab0b51b159c2bb4556ec2a11ab37a73adc6

Remove one space between sentences in description strings of existing index.yaml as workaround for bug that bumps last capitalized word, Prometheus, to an indented yaml newline.

view details

cbitze7387k

commit sha 78040b4b4dca38e9cc42933f96c3267645277eca

edit helm-repo-update.sh and dir structure to eliminate overwriting of existing index.yaml contents (specifically time stamps) during helm repo index --merge

view details

Eric Greer

commit sha 8b2cf1e59d2407ec234edebdbe65de8691e44b22

Merge pull request #647 from Comcast/HelmRepo_GhActionsAutoUpdate Helm Repo - Fix Github Actions Triggers

view details

push time in 8 days

PR merged Comcast/kuberhealthy

Reviewers
Helm Repo - Fix Github Actions Triggers

See issue #638 for full description and resolution, along with caveats for releases that must be adhered to manually to preserve semantic versioning and avoid triggering of "pre-release" package/index.

Sample fork can be viewed and tested at: https://github.com/cdbitz/kuberhealthy

+24 -15

0 comment

4 changed files

chadbitzer

pr closed time in 8 days

issue closedComcast/kuberhealthy

"Update Helm Repo" Action- faulty triggering of index and package generation

"Update Helm Repo" Action triggered on a branch merge, "Eric Stuff".

Steps To Reproduce

Expected behavior "Update Helm Repo" should only trigger on a tag creation of format "v#.#.#"

Versions

  • Kuberhealthy v2.3 release candidate

closed time in 8 days

chadbitzer
PullRequestReviewEvent

issue openedComcast/kuberhealthy

Create new cronjob run check

Cronjobs can sometimes get stuck and miss 100 runs. When they do this, Kubernetes will stop scheduling them entirely. We should be alerted when cronjobs have stopped.

  • catch when a cron job has skipped some number of runs
    • either by looking at last run time, or
    • by looking for an event that says 100 events have been missed
  • allow an optional whitelist environment variable that restricts which cronjobs in a namespace are analyzed

created time in 11 days

issue openedComcast/kuberhealthy

Add concurrencyPolicy: Forbid to check-reaper cronjob

We should not allow the check-reaper cronjob to build up extra pods if it can't complete a run. This can cause a breakage in the cluster by spawning thousands of pods.

created time in 11 days

issue commentComcast/kuberhealthy

Kuberhealthy status page is missing `CurrentMaster` field.

This may be connected to a bug we are seeing in the v2.3 image where we get a false check error that says something like check deleted expectedly.

jonnydawg

comment created time in 11 days

issue closedComcast/kuberhealthy

Support for newer versions of Kubernetes

Hello,

Currently, the README.md for the repo says "Kuberhealthy is currently tested on Kubernetes 1.9.x to 1.15.x."

Is Kuberhealthy consistently tested and maintained with new versions of Kubernetes as they are released?

closed time in 13 days

ewohltman

Pull request review commentComcast/kuberhealthy

Release process doc

+## Kuberhealthy Release Process++To release a new version of Kuberhealthy:++1. Verify that the master branch has all the latest changes++2. Tag the new release image tag and build the image+    - To tag and build the new release:+    - Update the `TAG` in the [Makefile](../cmd/kuberhealthy/Makefile)+    - In the Kuberhealthy project directory, run `make build` to build the image+    +3. Test the new image locally:+    - Install Kuberhealthy locally running the installation command found [here](../README.md#installation)+    - Make sure to supply your own [values.yaml](../deploy/helm/kuberhealthy/values.yaml), updating the `image.tag` to the new release imag tag and and `deployment.imagePullPolicy` to `Never`+    - Wait for all the kuberhealthy checks to come up and run as expected+    +4. Push image to [Dockerhub](https://hub.docker.com/r/kuberhealthy/kuberhealthy)+    - In the Kuberhealthy project directory, run `make push` to push the image++5. Soak test in lab and prod for several days++6. Create a PR updating the project with the latest Kuberhealthy tag:+    - [Makefile](../cmd/kuberhealthy/Makefile)+    - Helm [values.yaml](../deploy/helm/kuberhealthy/values.yaml)+    - Integration test values [values.yaml](../.ci/values.yaml)+    - Once the PR is merged, make sure the Flat File Generation PR is also merged -- this updates the deployment flat spec files with the latest Kuberhealthy release image tag++7. Write up and publish release notes, giving an update / latest changes from the last release: new *Kuberhealthy Checks* (if any), new *Features*, and *Bug Fixes*
7. Write up release notes, giving an update / latest changes from the last release: new *Kuberhealthy Checks* (if any), new *Features*, and *Bug Fixes*.  Go into Github and create a new release with the appropriate version and change notes in the body.
joshulyne

comment created time in 13 days

Pull request review commentComcast/kuberhealthy

Release process doc

+## Kuberhealthy Release Process++To release a new version of Kuberhealthy:++1. Verify that the master branch has all the latest changes++2. Tag the new release image tag and build the image+    - To tag and build the new release:+    - Update the `TAG` in the [Makefile](../cmd/kuberhealthy/Makefile)+    - In the Kuberhealthy project directory, run `make build` to build the image+    +3. Test the new image locally:+    - Install Kuberhealthy locally running the installation command found [here](../README.md#installation)+    - Make sure to supply your own [values.yaml](../deploy/helm/kuberhealthy/values.yaml), updating the `image.tag` to the new release imag tag and and `deployment.imagePullPolicy` to `Never`+    - Wait for all the kuberhealthy checks to come up and run as expected+    +4. Push image to [Dockerhub](https://hub.docker.com/r/kuberhealthy/kuberhealthy)+    - In the Kuberhealthy project directory, run `make push` to push the image++5. Soak test in lab and prod for several days++6. Create a PR updating the project with the latest Kuberhealthy tag:+    - [Makefile](../cmd/kuberhealthy/Makefile)+    - Helm [values.yaml](../deploy/helm/kuberhealthy/values.yaml)+    - Integration test values [values.yaml](../.ci/values.yaml)+    - Once the PR is merged, make sure the Flat File Generation PR is also merged -- this updates the deployment flat spec files with the latest Kuberhealthy release image tag++7. Write up and publish release notes, giving an update / latest changes from the last release: new *Kuberhealthy Checks* (if any), new *Features*, and *Bug Fixes*+    - Tag the issue and pr as well as the contributor to the release notes+ 

What about creating a tag in github? This says to publish notes, but does not talk about the tag that gets created.

joshulyne

comment created time in 13 days

PullRequestReviewEvent
PullRequestReviewEvent

pull request commentComcast/kuberhealthy

Fix send report logic on Kubernetes 1.16 above

A few things here...

A) I am not sure that Kubernetes now reflects the node's IP from all pods. Are you sure this wasn't just a part of your environment? Sounds like a CNI configuration option... We actually have a test that builds Kuberhelathy on Kubernetes 1.18 in a container and it passes with the code as-is.

B) There are several changes in this PR that are conflated. I would prefer to do one thing at a time, even if that means opening several individual PRs.
- Changing default helm chart options in values.yaml - Adding some nodecheck package calls in the DNS check

C) The Report struct should not be modified to accomplish our goals here.

Bo0km4n

comment created time in 13 days

PullRequestReviewEvent

push eventComcast/kuberhealthy

Mike Tougeron

commit sha 7508d68cc5b3393e7a0a6a2e4db9fd42467a31a6

Add AWS AMI Exists check to External Checks Registry

view details

Eric Greer

commit sha b0d0e4f9847022a3a113ca3eab2c8d225299e1c3

Merge pull request #643 from mtougeron/ami-exists-check-listing Add AWS AMI Exists check to External Checks Registry

view details

push time in 14 days

pull request commentComcast/kuberhealthy

Add AWS AMI Exists check to External Checks Registry

This rocks!

mtougeron

comment created time in 14 days

delete branch Comcast/kuberhealthy

delete branch : flat-file-auto-regen

delete time in 14 days

push eventComcast/kuberhealthy

Github Actions

commit sha 3d8a1da4ad6a132aec6412b725cf6f4b922c9afb

flat file spec regeneration

view details

Eric Greer

commit sha 306de2e37b950d5f40363a6e8ab94d614376345e

Merge pull request #645 from Comcast/flat-file-auto-regen Flat Spec File Regeneration

view details

push time in 14 days

PR merged Comcast/kuberhealthy

Flat Spec File Regeneration

An automatic PR for the regeneration of flat deployment files.

+51 -51

0 comment

3 changed files

github-actions[bot]

pr closed time in 14 days

PullRequestReviewEvent

push eventComcast/kuberhealthy

zhanso101

commit sha 105d3bd19220e87199cc6204583351781de5bde7

Updates check names to match established naming convention, and ups version number of checks

view details

Eric Greer

commit sha f9e68027c2e3354df3306239093781406e0c0ff2

Merge pull request #644 from Comcast/ssl-check-minor-fix Updates check names to match established naming convention, and ups v…

view details

push time in 14 days

PullRequestReviewEvent

push eventComcast/kuberhealthy

Isa Aguilar

commit sha 3615352f22d6159e79d0743e8fb041ac12fc7c0b

add more options for check-reaper cj

view details

Eric Greer

commit sha 9d762e2eaaa2abc6b50df19761bfb064107f8e6d

Merge pull request #635 from isaaguilar/check-reaper-startingDeadlineSeconds add more options for check-reaper cj

view details

push time in 14 days

PR merged Comcast/kuberhealthy

add more options for check-reaper cj
  • added startingDeadlineSeconds to check-reaper's cronjob template

I've been running into issues where the check-reaper has too many missed schedules for whatever reason. I figured adding this is harmless and gives the user an option to configure this without having to patch manually.

+5 -1

1 comment

3 changed files

isaaguilar

pr closed time in 14 days

PullRequestReviewEvent

push eventComcast/kuberhealthy

zhanso101

commit sha 07e27fd8d7f3c0414c7754984c5e53043bd2692b

Removes extraneous configmap file, fixes SSL expiry run intervals, adds clarifying comments to expiry checks, and fixes SSL self sign expiry check name field.

view details

Eric Greer

commit sha fde633f67059cf17eb3e313d0cebe11907497a9f

Merge pull request #642 from Comcast/ssl-check-minor-fix Removes extraneous configmap file, fixes SSL expiry run intervals, ad…

view details

push time in 15 days

PR merged Comcast/kuberhealthy

Removes extraneous configmap file, fixes SSL expiry run intervals, ad…

…ds clarifying comments to expiry checks, and fixes SSL self sign expiry check name field.

+6 -45

0 comment

4 changed files

zjhans

pr closed time in 15 days

PullRequestReviewEvent

push eventComcast/kuberhealthy

Adrian Aneci

commit sha 19b496f029e79b269ff4f651104cddd89ce83a3e

update k8s client-go to v1.19.0

view details

Adrian Aneci

commit sha 0cc537cd7b6b250b1db7b136bc2a71603cabed28

update k8s client-go to v1.19.0

view details

Adrian Aneci

commit sha 5876b57f26a5d73f3d36f98bc675575f46fe5d29

update CONTRIBUTORS.md

view details

Eric Greer

commit sha b9f397d20f9af68937f12ec663a23d3b6e143d01

Merge pull request #636 from adriananeci/update_k8sclient Upgrade k8s.io packages to v1.19.0

view details

push time in 21 days

PR merged Comcast/kuberhealthy

Upgrade k8s.io packages to v1.19.0

Required k8s.io packages seem to be a bit old. This PR updates k8s.io/api, k8s.io/apimachinery and k8s.io/client-go packages to v0.19.0 and adjust the code accordingly. I've ran go test ./... and got few failures, but I'm not sure if they are related to this PR changes. Is there a better way trigger the some tests?

+277 -232

4 comments

22 changed files

adriananeci

pr closed time in 21 days

pull request commentComcast/kuberhealthy

Upgrade k8s.io packages to v1.19.0

I made an issue to fill in the missing contexts: #640

adriananeci

comment created time in 21 days

issue openedComcast/kuberhealthy

Implement context.TODO() entries

During an upgrade of the Kubernetes package versions in PR #636 a bunch of context.TODO() function call parameters were added. We should go through and plug these into any existing contexts we have and implement new contexts for the others.

created time in 21 days

Pull request review commentComcast/kuberhealthy

Package and Index for Helm Repo

 apiVersion: v1 entries:   kuberhealthy:+  - apiVersion: v2+    appVersion: eric-stuff

This was the name of the branch being merged. This should have been the tag that was created.

github-actions[bot]

comment created time in 21 days

PullRequestReviewEvent

PR closed Comcast/kuberhealthy

Package and Index for Helm Repo

An automatic PR for updating of the Helm Repo based on releases tag.

+44 -6

1 comment

2 changed files

github-actions[bot]

pr closed time in 21 days

PR closed Comcast/kuberhealthy

Flat Spec File Regeneration

An automatic PR for the regeneration of flat deployment files.

+38 -38

0 comment

3 changed files

github-actions[bot]

pr closed time in 21 days

Pull request review commentComcast/kuberhealthy

add more options for check-reaper cj

 spec:           restartPolicy: OnFailure           serviceAccountName: check-reaper   concurrencyPolicy: Forbid+  startingDeadlineSeconds: {{ .Values.checkReaper.startingDeadlineSeconds }}

Can we add .Values.checkReaper.startingDeadlineSeconds to the values.yaml file here? That way we don't end up templating a blank property in the spec.

isaaguilar

comment created time in 21 days

PullRequestReviewEvent
PullRequestReviewEvent

pull request commentComcast/kuberhealthy

Upgrade k8s.io packages to v1.19.0

This looks great! I think we're going to do some extra validation on our end before merging this one. We also might be able to implement some of those context.TODO() items. Thank you!

adriananeci

comment created time in 21 days

push eventComcast/kuberhealthy

Mykola Mogylenko

commit sha 351dbc07fdb2687b8af83285744fb8371e81f89f

Adding AWS IAM Role check to External Checks Registry

view details

Eric Greer

commit sha aff54e2822d860c5b974d3291edb076e31166e82

Merge pull request #637 from mmogylenko/external-check-registry-add-aws-iam-role-check Adding AWS IAM Role check to External Checks Registry

view details

push time in 21 days

PR merged Comcast/kuberhealthy

Adding AWS IAM Role check to External Checks Registry

Adding documentation for the AWS IAM Role External Check

+2 -1

2 comments

1 changed file

mmogylenko

pr closed time in 21 days

PullRequestReviewEvent

pull request commentComcast/kuberhealthy

Adding AWS IAM Role check to External Checks Registry

This looks awesome. Your repo even has github actions integrated! Thanks for your contribution!

mmogylenko

comment created time in 21 days

startedmmogylenko/kuberhealthy-aws-iam-role-check

started time in 21 days

issue commentComcast/kuberhealthy

kuberhealthy/deployment error from /metrics have a wrong string format

Looks like our error metadata needs sanitized so that the prometheus metrics don't become unparsable. Thanks for bringing this up! We will look at sanitizing the error messages in our Prometheus metric data.

darleilopes

comment created time in 21 days

push eventComcast/kuberhealthy

Eric Greer

commit sha 84510508fbd71bff68c1cf941b88fea32ed14ce5

workload type panic

view details

push time in a month

IssuesEvent

issue commentComcast/kuberhealthy

Support for newer versions of Kubernetes

i just saw this run today with kube 1.18... we should update the readme to indicate the new version support. reopening

ewohltman

comment created time in a month

delete branch Comcast/kuberhealthy

delete branch : eric-stuff

delete time in a month

push eventComcast/kuberhealthy

Eric Greer

commit sha f38157c6d033ca92b19d77267eebccf0ce5c97d6

fixed scratch kuberhealthy build and prevented NPE when creating dynamic client

view details

Eric Greer

commit sha a1df6b7068f8ff17df4e61ee73fd5502d0f9476e

Merge pull request #632 from Comcast/eric-stuff fixed scratch kuberhealthy build and prevented NPE when creating dynamic client

view details

push time in a month

PR merged Comcast/kuberhealthy

fixed scratch kuberhealthy build and prevented NPE when creating dynamic client bug

This fixes the scratch image for Kuberhealthy so that it properly starts without complaining that /bin/sh is missing. I also found a null pointer exception to fix as well.

+30 -25

0 comment

2 changed files

integrii

pr closed time in a month

issue commentComcast/kuberhealthy

Kuberhealthy Vulnerabilities found

A new patched image with an important fix is available at kuberhealthy/kuberhealthy:v2.3.1-scratch

naikajah

comment created time in a month

PR opened Comcast/kuberhealthy

fixed scratch kuberhealthy build and prevented NPE when creating dynamic client

This fixes the scratch image for Kuberhealthy so that it properly starts without complaining that /bin/sh is missing. I also found a null pointer exception to fix as well.

+30 -25

0 comment

2 changed files

pr created time in a month

create barnchComcast/kuberhealthy

branch : eric-stuff

created branch time in a month

pull request commentComcast/kuberhealthy

Fix send report logic on Kubernetes 1.16 above

Check this code here and possibly open a fresh PR?

Bo0km4n

comment created time in a month

pull request commentComcast/kuberhealthy

Fix send report logic on Kubernetes 1.16 above

Thanks for bringing this to the project's attention! I would rather implement this fix in a slightly different way. We have code that overrides environment variables on checker pods before they are created. Adding the pod IP there would cause it to get added to all existing and future checks without any spec modifications.

It also looks like you may have accidentally enabled some checks in the helm chart by accident in this PR.

Bo0km4n

comment created time in a month

delete branch Comcast/kuberhealthy

delete branch : KinD_update

delete time in a month

push eventComcast/kuberhealthy

cbitze7387k

commit sha ff58324883bbd9aaf07a3025418f0ff83f5e1df1

modify version references for kind in e2e workflow to latest; setup-kind action v0.4.0 and kind v.0.8.1

view details

cbitze7387k

commit sha ddf9a484008ba0f97c0dddb04bb13e879462c380

modify version tested-on in README.md

view details

Eric Greer

commit sha 5e1d164e5616847b5ea65a5d8e581672d1abe560

Merge pull request #626 from Comcast/KinD_update modify version references for kind in e2e workflow to latest

view details

push time in a month

PR merged Comcast/kuberhealthy

modify version references for kind in e2e workflow to latest

setup-kind action v0.4.0 and kind v.0.8.1.

e2e workflow tested successfully at https://github.com/cdbitz/kuberhealthy/runs/1043631973?check_suite_focus=true , using node v1.18.2.

+3 -3

0 comment

2 changed files

cdbitz

pr closed time in a month

issue closedComcast/kuberhealthy

Support for newer versions of Kubernetes

Hello,

Currently, the README.md for the repo says "Kuberhealthy is currently tested on Kubernetes 1.9.x to 1.15.x."

Is Kuberhealthy consistently tested and maintained with new versions of Kubernetes as they are released?

closed time in a month

ewohltman
PullRequestReviewEvent

push eventComcast/kuberhealthy

bkatta317

commit sha 0081e2c3fe24d6e0ce748d41c10f404aa04455be

integrating nodecheck pkg into daemonset check

view details

bkatta317

commit sha f674f946f28ca6f12574469f5142d5af2b12b4e2

adding the error messages

view details

bkatta317

commit sha 8b85d5b291880da86aa7d92bce3e2fcdaf994da9

added the code to handle the errors

view details

bkatta317

commit sha b2cd21d80f050e593f5eef2f087199f617193e72

updated the image tag in values.yaml for daemonsetcheck

view details

bkatta317

commit sha 038ac8b7332a63f9d7d6b7df7301cdd87761d17f

updated the image tag in values.yaml for daemonsetcheck

view details

bkatta317

commit sha ca8c6694ba388375a3bb1af7729d2b3ad0d96787

Merge branch 'nodecheck-daemonsetcheck' of https://github.com/Comcast/kuberhealthy into nodecheck-daemonsetcheck

view details

Eric Greer

commit sha 92fba4d9468beb6a642aee26829cf071c5cf37fa

Merge pull request #618 from Comcast/nodecheck-daemonsetcheck Nodecheck daemonsetcheck

view details

push time in a month

PR merged Comcast/kuberhealthy

Nodecheck daemonsetcheck

#587, integrated the nodecheck package into the daemonset check

+48 -14

1 comment

4 changed files

brahminikatta

pr closed time in a month

issue closedComcast/kuberhealthy

Impement new `nodeCheck` package in `daemonsetcheck`

The new nodeCheck package allows a check to gracefully wait for a node to be fully operational before doing operations. We should implement this on all our checks. Without this check, there are circumstances where checker pods start quickly and instantly fail to resolve DNS, reach the API, or access public nets.

closed time in a month

integrii
PullRequestReviewEvent

issue commentComcast/kuberhealthy

Add jobs to check-reaper

Waiting on issue #528

integrii

comment created time in a month

issue openedComcast/kuberhealthy

Test Kuberhealthy 2.3 in lab clusters

created time in a month

PR opened helm/charts

[stable/kuberhealthy] kuberhealthy chart no longer in use

Signed-off-by: Eric Greer eric.greer@comcast.com

this is a simple deprecation change to our kuberhealthy chart in favor of our own chart repository.

+2 -45

0 comment

1 changed file

pr created time in a month

create barnchintegrii/charts

branch : kuberhealthy

created branch time in a month

issue commentComcast/kuberhealthy

Support for newer versions of Kubernetes

We definitely should bump the supported version. To do that, we just need to update the KinD tests to the newest label. @chadbitzer should be able to take a look at this.

ewohltman

comment created time in a month

PR closed helm/charts

add new kuberhealthy approvers approved needs-ok-to-test size/XS

Just adding a new owner to the Kuberhealthy chart

+2 -0

2 comments

1 changed file

integrii

pr closed time in a month

PR opened helm/charts

add new kuberhealthy approvers

Just adding a new owner to the Kuberhealthy chart

+2 -0

0 comment

1 changed file

pr created time in a month

push eventintegrii/charts

Eric Greer

commit sha c2e76a3e118fe588ce33b8f25e5e5d32dbad14c0

add new kuberhealthy approvers

view details

push time in a month

push eventComcast/kuberhealthy

Zach Hanson

commit sha 0683a56c6c45956009012b743fdc1c90b458388b

Adds self-signed TLS handshake check, minor verbiage updates to ssl-handshake and ssl-expiry README files.

view details

Zach Hanson

commit sha 362ae5711914fc8e53e7c6059542e873f3646ece

Corrects check version number in ssl-expiry-checl README file

view details

Eric Greer

commit sha 57240209ec7665a1b9dd8e72deb251234499d184

Merge pull request #615 from Comcast/self-signed-ssl-handshake Self signed ssl handshake

view details

push time in a month

PullRequestReviewEvent

push eventComcast/kuberhealthy

Github Actions

commit sha 04f1b3edae34ed03cf0781b20af4e502be0fc7ee

flat file spec regeneration

view details

Eric Greer

commit sha 08bce72dd2e53e17ea6a68b30643cb95817d3dba

Merge pull request #622 from Comcast/flat-file-auto-regen Flat Spec File Regeneration

view details

push time in a month

PR merged Comcast/kuberhealthy

Flat Spec File Regeneration

An automatic PR for the regeneration of flat deployment files.

+32 -32

0 comment

3 changed files

github-actions[bot]

pr closed time in a month

PullRequestReviewEvent

pull request commentComcast/kuberhealthy

Reaper khjob

I think this one is blocked waiting on #584

jdowni000

comment created time in a month

push eventComcast/kuberhealthy

cbitze7387k

commit sha a44f753ec258c96d4b4ad2b656567cb321b8a8fd

broke out v.2.2.0 from ./helm-repos/index.yaml into ./deploy/helm/kuberhealthy/index.yaml for merge test, created new 2.2.0.tgz

view details

cbitze7387k

commit sha d94e846d1b4d0840d5500c34d2b29b929e33552d

Merge remote-tracking branch 'origin' into HelmRepo_GhActionsAutoUpdate

view details

cbitze7387k

commit sha dbaa36a53d62246f3c9e6af49bfbd97bbdf79fb7

Merge branch 'master' into HelmRepo_GhActionsAutoUpdate

view details

cbitze7387k

commit sha 05f299ad7f61bcfa171f60856f03678e70daab5a

shell script & PR commands added

view details

cbitze7387k

commit sha 2eeb7c2afb9fae386ae4e19aff06d992ff7d5758

add release type released and kubernetes tools step for cmd to include Helm version

view details

cbitze7387k

commit sha 932185b499e65e8f44f67c89cb2e83cb8108d987

added .github/workflows/publish-latest-to-gh-pages.yml.disabled for master > gh-pages merge

view details

cbitze7387k

commit sha 1985e994b36758550ad0f048cd65ba26cba1924d

clean up unused lines in .github/workflows/helm-repo-update.yml.disabled

view details

Eric Greer

commit sha 033f3850184636c9fc46276808e95ef3577bc15c

Update helm-repo-update.yml.disabled

view details

Eric Greer

commit sha abf27740df5ff69ef0115059c050774087a11eb3

Update helm-repo-update.yml.disabled

view details

Eric Greer

commit sha aa6544fcec8659690fd23fe98b1e3472fcfa5144

Update helm-repo-update.yml.disabled

view details

Eric Greer

commit sha 3a89ef634714a5b4ec74e9002dc35be5d584d236

Update publish-latest-to-gh-pages.yml.disabled

view details

Eric Greer

commit sha 7ccd3c3cdd3f519162f48767425dd5d7ada91758

Update publish-latest-to-gh-pages.yml.disabled

view details

Eric Greer

commit sha 71a78ce5b35dc203318bee82df1a680d7b9be4da

Update helm-repo-update.yml.disabled

view details

Eric Greer

commit sha 5abe1bd0c48cdf1549b59941a1f4aea722950197

Update helm-repo-update.yml.disabled

view details

Eric Greer

commit sha c8fac40b8d48b441e1752a379231118bf65300ff

Update publish-latest-to-gh-pages.yml.disabled

view details

Eric Greer

commit sha 8ce57403e5d1e8dd5121f098bc4e5c15e6471b6f

Merge branch 'master' of github.com:Comcast/kuberhealthy into HelmRepo_GhActionsAutoUpdate

view details

Eric Greer

commit sha 3d4e687fa8ecb4a31097e4fe5964a9d8b6239538

Merge branch 'HelmRepo_GhActionsAutoUpdate' of github.com:Comcast/kuberhealthy into HelmRepo_GhActionsAutoUpdate

view details

Eric Greer

commit sha 107b39c742a3716547cf1d8aa2d678f669bd84f4

enable gh pages and helm repo update automation

view details

Eric Greer

commit sha 9f179872f3c9c20f6e1b4041fba7245717d7d6d3

re-add missing workflow files

view details

joshulyne

commit sha 8bff76675c4828bb133a729437288b7a1b322613

Merge branch 'master' into HelmRepo_GhActionsAutoUpdate

view details

push time in a month

more