1
0
Fork 0
mirror of https://github.com/kubernetes-sigs/node-feature-discovery.git synced 2024-12-14 11:57:51 +00:00
Commit graph

2409 commits

Author SHA1 Message Date
Markus Lehtonen
2382c34697 nfd-master: fix node status patching
Correctly patch the "status" subresource. This got broken when
refactoring the code in 7a050e7cf9 and
wasn't even catched by the unit tests as the fake kubernetes client
doesn't handle subresources as the real apiserver does.
2024-01-26 22:00:13 +02:00
Kubernetes Prow Robot
a66e1d3004
Merge pull request #1561 from marquiz/devel/apihelper-refactor
Drop pkg/apihelper
2024-01-26 18:14:30 +01:00
Markus Lehtonen
8a6a731eb0 Drop pkg/apihelper
The code is now unused.
2024-01-26 18:50:31 +02:00
Kubernetes Prow Robot
33858b7502
Merge pull request #1567 from marquiz/devel/apihelper-refactor-3
topology-updater: ditch apihelper
2024-01-26 17:07:13 +01:00
Kubernetes Prow Robot
8e53bb3ad4
Merge pull request #1570 from marquiz/devel/apihelper-refactor-5
nfd-master: ditch apihelper
2024-01-26 15:30:06 +01:00
Markus Lehtonen
7a050e7cf9 nfd-master: ditch apihelper
Implement some of frequently used helper functions inpackage.

This patch also contains big changes to the nfd-master unit tests. Much
of this is about migrating from the mocked apihelper interface to fake
kubernetes client that provides a bit more apiserver'ish functionality.
At the same time there is quite a bit of renaming in the tests,
shortening and unifying naming and getting rid of the extensive usage of
"mock" everywhere.
2024-01-26 16:09:22 +02:00
Kubernetes Prow Robot
43a9239962
Merge pull request #1568 from marquiz/devel/apihelper-refactor-4
pkg/utils: move JsonPatch from pkg/apihelper
2024-01-26 11:48:24 +01:00
leemingeer
b6d8ce7a5a nfd-topology-updater add pods fingerprint by default 2024-01-26 17:55:34 +08:00
Markus Lehtonen
c581a25a39 topology-updater: ditch apihelper
Stop using pkg/apihelper for accessing the Kubernetes API. Modify unit
tests to use the fake kubernetes client instead of mocked apihelper
interface.
2024-01-25 22:15:20 +02:00
Markus Lehtonen
53003cbf69 pkg/utils: move JsonPatch from pkg/apihelper 2024-01-25 17:23:14 +02:00
Kubernetes Prow Robot
5d30be1013
Merge pull request #1566 from marquiz/devel/apihelper-refactor-2
topology-updater: get topology api client directly
2024-01-25 16:21:08 +01:00
Kubernetes Prow Robot
238105cf16
Merge pull request #1565 from marquiz/devel/helm
deployment/helm: don't deploy topology-updater conf unnecessarily
2024-01-25 15:47:27 +01:00
Markus Lehtonen
2326459d05 topology-updater: get topology api client directly
Stop using apihelper for getting the noderesourcetopology-api client.
2024-01-25 16:33:34 +02:00
Markus Lehtonen
8adb4b38da deployment/helm: don't deploy topology-updater conf unnecessarily
Only deploy the topology-updater config if topology-updater itself (the
daemon) is deployed.
2024-01-25 16:15:58 +02:00
Kubernetes Prow Robot
f3e07aeb91
Merge pull request #1564 from marquiz/devel/owners
OWNERS: add AhmedGrati as a reviewer
2024-01-25 12:19:29 +01:00
Markus Lehtonen
d6ee3b2778 OWNERS: add AhmedGrati as a reviewer
Also, sort the names alphabetically.
2024-01-25 13:12:51 +02:00
Kubernetes Prow Robot
3ef10fecd0
Merge pull request #1562 from marquiz/devel/apihelper-refactor-1
pkg/utils: move GetKubeconfig from pkg/apihelper here
2024-01-25 10:24:39 +01:00
Markus Lehtonen
acf815fb10 pkg/utils: move GetKubeconfig from pkg/apihelper here
This change is part of an effort to remove the pkg/apihelper package.
GetKubeconfig is useful helper functionality shared accross the codebase
so move it into a "safe" location.
2024-01-24 16:10:02 +02:00
mantramantra12
ea4504a916
Prevent nfd-worker erroring when reading attributes from paravirtual devices (#1557)
* prevent erroring on paravirtual devices

* Use `errors.Is()` for error checking

Co-authored-by: Markus Lehtonen <markus.lehtonen@intel.com>

* add imports

---------

Co-authored-by: Markus Lehtonen <markus.lehtonen@intel.com>
2024-01-23 22:20:43 +01:00
Kubernetes Prow Robot
c677736e4c
Merge pull request #1558 from marquiz/devel/error-wrap
Wrap nested errors
2024-01-23 19:37:18 +01:00
Markus Lehtonen
57b7a3c6a8 Wrap nested errors 2024-01-22 22:45:15 +02:00
Kubernetes Prow Robot
42b125107b
Merge pull request #1555 from marquiz/devel/topology-updater-no-publish-docs
topology-updater: document the -no-publish flag correctly
2024-01-22 17:19:41 +01:00
Kubernetes Prow Robot
9be4f04218
Merge pull request #1554 from marquiz/devel/topology-updater-no-publish
topology-updater: initialize properly with -no-publish
2024-01-22 13:34:18 +01:00
Markus Lehtonen
d7ec0bf674 topology-updater: document the -no-publish flag correctly 2024-01-22 14:21:02 +02:00
Markus Lehtonen
b452ab6a5c topology-updater: initialize properly with -no-publish
We need to parse kubeconfig (and initialize the apihelper) even with
-no-publish as the PodResourcesScanner accesses the k8s API even if
we're not publishing/updating NRTs.
2024-01-22 14:15:12 +02:00
Kubernetes Prow Robot
d80e4f7411
Merge pull request #1543 from marquiz/devel/docs-deps
docs: update docs build dependencies
2024-01-19 15:25:48 +01:00
Kubernetes Prow Robot
3667a4d073
Merge pull request #1537 from ozhuraki/apis-nfd-test
apis/nfd: Trivial typo fix in tests
2024-01-19 15:25:41 +01:00
Kubernetes Prow Robot
0cbab3743e
Merge pull request #1553 from marquiz/devel/hack
hack/generate: patch auto-generated deepcopy functions
2024-01-18 16:35:44 +01:00
Markus Lehtonen
ef4ac7c5d3 hack/generate: patch auto-generated deepcopy functions
Temporary (hopefully) fix for deepcopy functions that code-generator
doesn't get quite right.
2024-01-18 17:15:09 +02:00
Kubernetes Prow Robot
22cd9d3d18
Merge pull request #1552 from marquiz/devel/readme
Update readme to v0.15.1 release
2024-01-18 15:40:41 +01:00
Markus Lehtonen
4774e447b9 Update readme to v0.15.1 release 2024-01-18 16:26:51 +02:00
AhmedGrati
5798a7d54e docs: add kep of spiffe integration
Signed-off-by: AhmedGrati <ahmedgrati1999@gmail.com>
2024-01-18 15:09:10 +01:00
Markus Lehtonen
7fa5fcf22a docs: update docs build dependencies 2024-01-16 16:05:04 +02:00
Kubernetes Prow Robot
2694448a7f
Merge pull request #1530 from marquiz/devel/rdt
source/cpu: drop deprecated cpu-rdt labels
2024-01-16 11:08:18 +01:00
Kubernetes Prow Robot
b72000b933
Merge pull request #1505 from AhmedGrati/chore-refactor-topology-discover
chore: combine cpu count and thread_siblings functions into discover topology function
2024-01-16 11:08:09 +01:00
Kubernetes Prow Robot
7cfcd3b1a7
Merge pull request #1539 from marquiz/devel/deps
go.mod: update dependencies
2024-01-15 20:50:27 +01:00
Markus Lehtonen
58ae81804c go.mod: update dependencies 2024-01-15 21:29:32 +02:00
Oleg Zhurakivskyy
eec05e1c7a apis/nfd: Trivial typo fix in tests
Signed-off-by: Oleg Zhurakivskyy <oleg.zhurakivskyy@intel.com>
2024-01-15 18:06:58 +02:00
Kubernetes Prow Robot
bc80071c0b
Merge pull request #1536 from marquiz/devel/network
source/network: discover speed of virtual network interfaces
2024-01-15 15:12:31 +01:00
Kubernetes Prow Robot
4501bedd61
Merge pull request #1535 from marquiz/devel/grpc-probe
nfd-master: run a separate gRPC health server
2024-01-05 15:24:28 +01:00
Markus Lehtonen
887f208f5d source/network: discover speed of virtual network interfaces 2024-01-04 14:14:56 +02:00
Markus Lehtonen
a053efda64 nfd-master: run a separate gRPC health server
This patch separates the gRPC health server from the deprecated gRPC
server (disabled by default, replaced by the NodeFeature CRD API) used
for node labeling requests. The new health server runs on hardcoded TCP
port number 8082.

The main motivation for this change is to make the Kubernetes' built-in
gRPC liveness probes to function if TLS is enabled (as they don't
support TLS).

The health server itself is a naive implementation (as it was before),
basically only checking that nfd-master has started and hasn't crashed.
The patch adds a TODO note to improve the functionality.
2024-01-04 13:58:26 +02:00
Kubernetes Prow Robot
64eba874db
Merge pull request #1534 from marquiz/devel/kustomize-cleanup
deployment/kustomize: drop the sample cert-manager overlay
2024-01-03 20:49:05 +01:00
Markus Lehtonen
09b5af74de deployment/kustomize: drop the sample cert-manager overlay
Drop the deprecated and broken sample overlay. This was an example for
enabling TLS with cert-manager. However, the overlay has been broken
(and useless) since NodeFeature API was enabled by default - and gRPC
disabled - in v0.14.
2024-01-03 21:13:15 +02:00
Kubernetes Prow Robot
b3919f3e0d
Merge pull request #1532 from marquiz/devel/helm-hook
helm: add post-delete hook that cleans up the node
2024-01-02 11:34:12 +01:00
Markus Lehtonen
889fffd7d4 helm: add post-delete hook that cleans up the node
This patch adds a post-delete hook to the Helm chart that runs
"nfd-master --prune" in the cluster. This cleans up the node of labels,
annotations, taints and extended resources that were created by NFD.
2023-12-29 15:36:41 +02:00
Markus Lehtonen
cd18fe8970 source/cpu: drop deprecated cpu-rdt labels
Drop RDT labels that were deprecated in NFD v0.13. The RDT features
remain available for NodeFeatureRules to serve custom labeling.
2023-12-22 17:29:00 +02:00
Kubernetes Prow Robot
456a77b31a
Merge pull request #1526 from marquiz/devel/e2e-timeout
Makefile: add -timeout argument to e2e-tests
2023-12-22 15:46:43 +01:00
Kubernetes Prow Robot
f8ba32af8d
Merge pull request #1528 from marquiz/devel/makefile
makefile: fix build: target
2023-12-22 15:20:06 +01:00
Markus Lehtonen
4d2feca004 makefile: fix build: target
Create binaries under bin/ rather than a binary named bin.
2023-12-22 10:48:15 +02:00