Commit Graph

704 Commits

Author SHA1 Message Date
Kubernetes Prow Robot
7bc362cfdc Merge pull request #668 from grosser/grosser/systemd
show failed statuses as warning
2022-07-26 19:16:38 -07:00
Kubernetes Prow Robot
341af62275 Merge pull request #646 from notchairmk/notchairmk/custom-skip-initial
Allow skipping condition during customplugin initialization
2022-07-26 19:16:31 -07:00
Kubernetes Prow Robot
2d5de8d0fa Merge pull request #684 from acumino/multi-arch-image
Create multi-arch image
2022-07-26 12:03:10 -07:00
Kubernetes Prow Robot
c9ffa67ec4 Merge pull request #685 from diamondburned/resultChan-rm
Remove unused resultChan field in CPM
2022-07-14 22:34:25 -07:00
diamondburned
6809f445eb Remove unused resultChan field in CPM
This commit removes the resultChan field in ./pkg/custompluginmonitor's
customPluginMonitor struct. This was detected by staticcheck:

    ―❤―▶ staticcheck ./pkg/custompluginmonitor/
    pkg/custompluginmonitor/custom_plugin_monitor.go:50:2: field resultChan is unused (U1000)
2022-07-12 21:43:05 -07:00
Sonu Kumar Singh
04e8d009d4 Use buildx for docker builder 2022-07-05 09:37:52 +05:30
Kubernetes Prow Robot
72f1672634 Merge pull request #675 from mmiranda96/feat/net-monitor-groupings
Add ExcludeInterfaceRegexp to Net Dev monitor
2022-06-29 14:50:06 -07:00
Mike Miranda
1471f74d98 Add ExcludeInterfaceRegexp to Net Dev monitor 2022-06-15 23:22:38 +00:00
Kubernetes Prow Robot
56122ce0dd Merge pull request #678 from 2rs2ts/master
Add condition message to event message
2022-06-15 15:43:11 -07:00
Andrew Garrett
b1bd8e7424 Use %q instead of %s 2022-06-09 17:18:30 +00:00
Andrew Garrett
a39a7c6e0f Add condition message to event message
If you're using some monitoring solution that aggregates events from
your Kubernetes cluster, having the underlying reason why a condition
triggered could be very useful, especially if you are using custom
plugin monitors.

Co-authored-by: Micah Norman <micnorman@paypal.com>
Signed-off-by: Ryan Eschinger <reschinger@paypal.com>
2022-06-08 21:42:40 +00:00
Michael Grosser
011b9e6a46 show failed statuses as warning 2022-04-26 11:50:10 -07:00
Taylor Chaparro
9344c938bb Allow skipping condition during customplugin initialization 2022-04-26 10:12:01 -07:00
Kubernetes Prow Robot
51508603fe Merge pull request #616 from com6056/patch-1
Install systemd in docker image
2022-04-24 13:49:37 -07:00
Kubernetes Prow Robot
c083db10f0 Merge pull request #628 from mx-psi/master
Change to using new dependency name for osreleaser
2022-04-22 11:35:37 -07:00
Kubernetes Prow Robot
a0abe5c667 Merge pull request #653 from matt-cale-do/patch-1
Grammar help
2022-04-21 12:28:18 -07:00
Kubernetes Prow Robot
9c23553e0b Merge pull request #650 from yankay/fix-deprecated-maintainer-in-dockerfile
FIx deprecated "MAINTAINER" in Dockerfile
2022-04-21 12:28:12 -07:00
Kubernetes Prow Robot
8603b5b98b Merge pull request #638 from vteratipally/adfad
Add vteratipally as an approver
2022-04-21 10:58:57 -07:00
Kubernetes Prow Robot
285516dc10 Merge pull request #594 from oif/chore/optimize-netcollector-implementation
optimize netcollector implementation and custom `/proc` mount path
2022-04-11 03:36:06 -07:00
Neo Zhuo
11ddb5e6bf support custom /proc path 2022-04-11 18:15:08 +08:00
Neo Zhuo
78c11c4ceb reimplement net collector metrics register, config check and recording 2022-04-11 18:15:07 +08:00
Pablo Baeyens
5e300846b2 Merge remote-tracking branch 'origin/master' into mx-psi/master 2022-03-29 14:45:27 -04:00
Kubernetes Prow Robot
4412a2b9a4 Merge pull request #657 from vteratipally/remove_ginkgo
remove installing ginkgo using "go get"
2022-03-25 08:33:59 -07:00
varsha teratipally
f508ccea7b remove installing ginkgo using "go get" as it ginkgo version supported
is 2.0
2022-03-25 00:48:12 +00:00
Kubernetes Prow Robot
68314853b8 Merge pull request #656 from vteratipally/ads
update golang, ginkgo and gomega dependencies
2022-03-24 09:58:41 -07:00
varsha teratipally
20c3b6f13c update ginkgo and gomega dependencies 2022-03-24 16:25:03 +00:00
Kubernetes Prow Robot
bdbf6b3df9 Merge pull request #639 from vteratipally/adfa
install dependencies before building the NPD containers.
2022-03-23 13:41:20 -07:00
Matthew Cale
84259052d1 Grammar help
could have been: `"It monitors a specific kind of node problem..."` as well
2022-03-16 15:24:44 -05:00
Varsha Teratipally
c370cfb68a install dependencies before building the NPD containers. 2022-03-10 20:39:56 +00:00
Kay Yan
bc89bbce56 MAINTAINER in Dockerfile is deprecated, change to label
Signed-off-by: Kay Yan <kay.yan@daocloud.io>
2022-03-07 15:27:08 +08:00
Kelvie Wong
363d01392a Create multi-arch image
For linux arm64 and amd64, as per #586.

I moved the builder image into the same dockerfile, and bumped the Go
version on it. It didn't seem like the builder dockerfile worked with
the latest code anyway (the go modules require go 1.15 and higher).

This requires a recent enough docker install with buildx, as well as
an arm64 builder.

BASEIMAGE is changed to not specify an arch, so that the image will
build on its native arch in buildx.

Example image is on docker hub as:

    kelvie/node-problem-detector:v0.8.10-5-gb0fa610
2022-02-13 01:23:31 -08:00
Pablo Baeyens
0b64594d0a Merge remote-tracking branch 'origin/master' into mx-psi/master 2022-01-13 10:42:04 +01:00
vteratipally
3e9834e26d Update OWNERS 2022-01-12 12:10:39 -08:00
Kubernetes Prow Robot
e7fe0b20dc Merge pull request #629 from spiffxp/use-k8s-infra-for-gcb-image
images: use k8s-staging-test-infra/gcb-docker-gcloud
2021-12-15 16:23:18 -08:00
Aaron Crickenberger
0761e11cc4 images: use k8s-staging-test-infra/gcb-docker-gcloud 2021-11-30 13:02:50 -08:00
Pablo Baeyens
a859b5f027 Change to using new dependency name for osreleaser
To do this I
1. changed the name in go.mod and the Go code that used it,
2. ran `go mod tidy -go=1.15` and
3. ran `go mod vendor`.

Step 3 added another vendored dependency unrelated AFAIK to this change.
2021-11-29 16:45:48 +01:00
Jordan Rodgers
760d252808 Only need systemd 2021-09-03 17:11:48 -07:00
Jordan Rodgers
0de6fae1f8 Install curl and systemd in docker image
A few issues have popped up where the provided image doesn't have the required packages for certain health checking operations (like https://github.com/kubernetes/node-problem-detector/issues/584#issuecomment-885832078).

This installs curl and systemd in the container to help alleviate these issues.
2021-09-03 16:46:08 -07:00
Kubernetes Prow Robot
e7d28a3bf1 Merge pull request #615 from mcshooter/updateTimeFormatForUptimeFunc
Ensure time is in Universal Time Zone to properly calculate uptime
2021-09-02 11:41:41 -07:00
michelletandya
3344efd552 ensure time is in Universal Time Zone to properly calculate uptime 2021-09-02 17:41:54 +00:00
Kubernetes Prow Robot
56c592a5d7 Merge pull request #587 from vteratipally/bug_fix
Add a check if the metric is nil so that collector doesn't collect metrics.
v0.8.10
2021-08-31 09:21:37 -07:00
Kubernetes Prow Robot
1123fd22cb Merge pull request #589 from xinydev/fix-build-guide
Update the instructions for build image in the readme
2021-08-30 15:11:13 -07:00
Kubernetes Prow Robot
393a9401b1 Merge pull request #607 from mmiranda96/fix/23202
Create cloudbuild file
2021-08-30 14:41:13 -07:00
Mike Miranda
fd6c80b840 Create cloudbuild file 2021-08-23 23:05:20 +00:00
Kubernetes Prow Robot
3c3609b5fa Merge pull request #612 from mcshooter/updateUptimeCMd
Update powershell command for uptime to help efficiency
2021-08-20 18:42:05 -07:00
Kubernetes Prow Robot
7a33650863 Merge pull request #609 from mcshooter/fixWindowsCPUIssue
Prevent uptimeFunc from being called everytime CheckHealth is called
2021-08-20 18:41:59 -07:00
Kubernetes Prow Robot
a276a05765 Merge pull request #611 from ryuichi1208/master
Fixed dead link
2021-08-20 17:59:59 -07:00
michelletandya
dd0d0d71ab Update powershell command for uptime to help efficiency 2021-08-20 01:16:45 +00:00
michelletandya
26f070bfd4 Prevent uptimeFunc from being called everytime CheckHealth is being called 2021-08-17 19:30:28 +00:00
Ryuichi Watanabe
ca95d61bf8 fix README 2021-08-15 13:58:46 +09:00