2020-09-08 20:16:17 +03:00
|
|
|
---
|
2021-03-09 13:38:09 +02:00
|
|
|
title: "Developer guide"
|
2020-09-08 20:16:17 +03:00
|
|
|
layout: default
|
2022-11-02 14:34:37 +02:00
|
|
|
sort: 5
|
2020-09-08 20:16:17 +03:00
|
|
|
---
|
|
|
|
|
2021-03-09 13:38:09 +02:00
|
|
|
# Developer guide
|
2021-09-27 15:31:49 +03:00
|
|
|
{: .no_toc}
|
2020-09-08 20:16:17 +03:00
|
|
|
|
|
|
|
## Table of contents
|
2021-09-27 15:31:49 +03:00
|
|
|
{: .no_toc .text-delta}
|
2020-09-08 20:16:17 +03:00
|
|
|
|
|
|
|
1. TOC
|
|
|
|
{:toc}
|
|
|
|
|
|
|
|
---
|
|
|
|
|
2021-03-09 13:38:09 +02:00
|
|
|
## Building from source
|
2020-09-08 20:16:17 +03:00
|
|
|
|
2020-10-20 20:50:46 +03:00
|
|
|
### Download the source code
|
2020-09-08 20:16:17 +03:00
|
|
|
|
2020-10-20 20:50:46 +03:00
|
|
|
```bash
|
2020-09-08 20:16:17 +03:00
|
|
|
git clone https://github.com/kubernetes-sigs/node-feature-discovery
|
|
|
|
cd node-feature-discovery
|
|
|
|
```
|
|
|
|
|
2021-03-09 13:38:09 +02:00
|
|
|
### Docker build
|
2020-09-08 20:16:17 +03:00
|
|
|
|
2020-10-20 20:50:46 +03:00
|
|
|
#### Build the container image
|
2020-09-08 20:16:17 +03:00
|
|
|
|
|
|
|
See [customizing the build](#customizing-the-build) below for altering the
|
|
|
|
container image registry, for example.
|
|
|
|
|
2020-10-20 20:50:46 +03:00
|
|
|
```bash
|
2020-09-08 20:16:17 +03:00
|
|
|
make
|
|
|
|
```
|
|
|
|
|
2020-10-20 20:50:46 +03:00
|
|
|
#### Push the container image
|
2021-02-25 13:49:02 +02:00
|
|
|
|
2020-09-08 20:16:17 +03:00
|
|
|
Optional, this example with Docker.
|
|
|
|
|
2020-10-20 20:50:46 +03:00
|
|
|
```bash
|
2020-09-08 20:16:17 +03:00
|
|
|
docker push <IMAGE_TAG>
|
|
|
|
```
|
|
|
|
|
2021-12-10 13:18:31 +01:00
|
|
|
### Docker multi-arch builds with buildx
|
|
|
|
|
|
|
|
The default set of architectures enabled for mulit-arch builds are `linux/amd64`
|
|
|
|
and `linux/arm64`. If more architectures are needed one can override the
|
|
|
|
`IMAGE_ALL_PLATFORMS` variable with a comma separated list of `OS/ARCH` tuples.
|
|
|
|
|
|
|
|
#### Build the manifest-list with a container image per arch
|
|
|
|
|
|
|
|
```bash
|
|
|
|
make image-all
|
|
|
|
```
|
|
|
|
|
|
|
|
Currently `docker` does not support loading of manifest-lists meaning the images
|
|
|
|
are not shown when executing `docker images`, see:
|
|
|
|
[buildx issue #59](https://github.com/docker/buildx/issues/59).
|
|
|
|
|
|
|
|
#### Push the manifest-list with container image per arch
|
|
|
|
|
|
|
|
```bash
|
|
|
|
make push-all
|
|
|
|
```
|
|
|
|
|
|
|
|
The resulting container image can be used in the same way on each arch by pulling
|
2022-03-29 09:47:42 +03:00
|
|
|
e.g. `node-feature-discovery:{{ site.release }}` without specifying the
|
|
|
|
architecture. The manifest-list will take care of providing the right
|
|
|
|
architecture image.
|
2021-12-10 13:18:31 +01:00
|
|
|
|
2020-10-20 20:50:46 +03:00
|
|
|
#### Change the job spec to use your custom image (optional)
|
2020-09-08 20:16:17 +03:00
|
|
|
|
|
|
|
To use your published image from the step above instead of the
|
2022-09-12 11:21:12 +03:00
|
|
|
`registry.k8s.io/nfd/node-feature-discovery` image, edit `image`
|
2020-09-08 20:16:17 +03:00
|
|
|
attribute in the spec template(s) to the new location
|
|
|
|
(`<registry-name>/<image-name>[:<version>]`).
|
|
|
|
|
2020-10-01 15:37:34 +03:00
|
|
|
### Deployment
|
|
|
|
|
2021-08-13 17:35:22 +03:00
|
|
|
The `yamls` makefile generates a `kustomization.yaml` matching your locally
|
|
|
|
built image and using the `deploy/overlays/default` deployment. See
|
|
|
|
[build customization](#customizing-the-build) below for configurability, e.g.
|
|
|
|
changing the deployment namespace.
|
2020-10-01 15:37:34 +03:00
|
|
|
|
|
|
|
```bash
|
|
|
|
K8S_NAMESPACE=my-ns make yamls
|
2021-08-13 17:35:22 +03:00
|
|
|
kubectl apply -k .
|
2020-10-01 15:37:34 +03:00
|
|
|
```
|
|
|
|
|
2021-08-13 17:35:22 +03:00
|
|
|
You can use alternative deployment methods by modifying the auto-generated
|
|
|
|
kustomization file. For example, deploying worker and master in the same pod by
|
|
|
|
pointing to `deployment/overlays/default-combined`.
|
2020-10-01 15:37:34 +03:00
|
|
|
|
2021-03-09 13:38:09 +02:00
|
|
|
### Building locally
|
2020-09-08 20:16:17 +03:00
|
|
|
|
|
|
|
You can also build the binaries locally
|
2020-10-20 20:50:46 +03:00
|
|
|
|
|
|
|
```bash
|
2020-09-08 20:16:17 +03:00
|
|
|
make build
|
|
|
|
```
|
|
|
|
|
|
|
|
This will compile binaries under `bin/`
|
|
|
|
|
2021-03-09 13:38:09 +02:00
|
|
|
### Customizing the build
|
2020-09-08 20:16:17 +03:00
|
|
|
|
|
|
|
There are several Makefile variables that control the build process and the
|
|
|
|
name of the resulting container image. The following are targeted targeted for
|
|
|
|
build customization and they can be specified via environment variables or
|
|
|
|
makefile overrides.
|
|
|
|
|
|
|
|
| Variable | Description | Default value
|
|
|
|
| -------------------------- | ----------------------------------------------------------------- | ----------- |
|
|
|
|
| HOSTMOUNT_PREFIX | Prefix of system directories for feature discovery (local builds) | / (*local builds*) /host- (*container builds*)
|
|
|
|
| IMAGE_BUILD_CMD | Command to build the image | docker build
|
|
|
|
| IMAGE_BUILD_EXTRA_OPTS | Extra options to pass to build command | *empty*
|
2021-12-10 13:18:31 +01:00
|
|
|
| IMAGE_BUILDX_CMD | Command to build and push multi-arch images with buildx | DOCKER_CLI_EXPERIMENTAL=enabled docker buildx build --platform=${IMAGE_ALL_PLATFORMS} --progress=auto --pull
|
|
|
|
| IMAGE_ALL_PLATFORMS | Comma seperated list of OS/ARCH tuples for mulit-arch builds | linux/amd64,linux/arm64
|
2020-09-08 20:16:17 +03:00
|
|
|
| IMAGE_PUSH_CMD | Command to push the image to remote registry | docker push
|
2022-09-12 11:21:12 +03:00
|
|
|
| IMAGE_REGISTRY | Container image registry to use | registry.k8s.io/nfd
|
2020-09-08 20:16:17 +03:00
|
|
|
| IMAGE_TAG_NAME | Container image tag name | <nfd version>
|
|
|
|
| IMAGE_EXTRA_TAG_NAMES | Additional container image tag(s) to create when building image | *empty*
|
2022-02-14 08:46:52 +11:00
|
|
|
| K8S_NAMESPACE | nfd-master and nfd-worker namespace | node-feature-discovery
|
2020-09-08 20:16:17 +03:00
|
|
|
| KUBECONFIG | Kubeconfig for running e2e-tests | *empty*
|
2020-11-02 10:48:17 +02:00
|
|
|
| E2E_TEST_CONFIG | Parameterization file of e2e-tests (see [example][e2e-config-sample]) | *empty*
|
2020-12-01 19:10:38 +01:00
|
|
|
| E2E_PULL_IF_NOT_PRESENT | True-ish value makes the image pull policy IfNotPresent (to be used only in e2e tests) | false
|
2021-02-25 17:43:59 +02:00
|
|
|
| OPENSHIFT | Non-empty value enables OpenShift specific support (currently only effective in e2e tests) | *empty*
|
2020-09-08 20:16:17 +03:00
|
|
|
|
|
|
|
For example, to use a custom registry:
|
|
|
|
|
2020-10-20 20:50:46 +03:00
|
|
|
```bash
|
2020-09-08 20:16:17 +03:00
|
|
|
make IMAGE_REGISTRY=<my custom registry uri>
|
|
|
|
```
|
|
|
|
|
|
|
|
Or to specify a build tool different from Docker, It can be done in 2 ways:
|
2020-10-20 20:50:46 +03:00
|
|
|
|
2020-09-08 20:16:17 +03:00
|
|
|
1. via environment
|
2021-02-25 13:49:02 +02:00
|
|
|
|
|
|
|
```bash
|
|
|
|
IMAGE_BUILD_CMD="buildah bud" make
|
|
|
|
```
|
|
|
|
|
2020-10-20 20:50:46 +03:00
|
|
|
1. by overriding the variable value
|
2021-02-25 13:49:02 +02:00
|
|
|
|
|
|
|
```bash
|
|
|
|
make IMAGE_BUILD_CMD="buildah bud"
|
|
|
|
```
|
2020-09-08 20:16:17 +03:00
|
|
|
|
|
|
|
### Testing
|
|
|
|
|
|
|
|
Unit tests are automatically run as part of the container image build. You can
|
|
|
|
also run them manually in the source code tree by simply running:
|
2020-10-20 20:50:46 +03:00
|
|
|
|
|
|
|
```bash
|
2020-09-08 20:16:17 +03:00
|
|
|
make test
|
|
|
|
```
|
|
|
|
|
|
|
|
End-to-end tests are built on top of the e2e test framework of Kubernetes, and,
|
|
|
|
they required a cluster to run them on. For running the tests on your test
|
|
|
|
cluster you need to specify the kubeconfig to be used:
|
2020-10-20 20:50:46 +03:00
|
|
|
|
|
|
|
```bash
|
2020-09-08 20:16:17 +03:00
|
|
|
make e2e-test KUBECONFIG=$HOME/.kube/config
|
|
|
|
```
|
|
|
|
|
2021-03-09 13:38:09 +02:00
|
|
|
## Running locally
|
2020-09-08 20:16:17 +03:00
|
|
|
|
|
|
|
You can run NFD locally, either directly on your host OS or in containers for
|
|
|
|
testing and development purposes. This may be useful e.g. for checking
|
|
|
|
features-detection.
|
|
|
|
|
|
|
|
### NFD-Master
|
|
|
|
|
|
|
|
When running as a standalone container labeling is expected to fail because
|
2021-02-24 14:29:07 +02:00
|
|
|
Kubernetes API is not available. Thus, it is recommended to use `-no-publish`
|
2020-09-08 20:16:17 +03:00
|
|
|
command line flag. E.g.
|
2020-10-20 20:50:46 +03:00
|
|
|
|
|
|
|
```bash
|
2020-11-20 16:48:57 +02:00
|
|
|
$ export NFD_CONTAINER_IMAGE={{ site.container_image }}
|
2021-02-24 14:29:07 +02:00
|
|
|
$ docker run --rm --name=nfd-test ${NFD_CONTAINER_IMAGE} nfd-master -no-publish
|
2020-09-08 20:16:17 +03:00
|
|
|
2019/02/01 14:48:21 Node Feature Discovery Master <NFD_VERSION>
|
|
|
|
2019/02/01 14:48:21 gRPC server serving on port: 8080
|
|
|
|
```
|
|
|
|
|
|
|
|
### NFD-Worker
|
|
|
|
|
|
|
|
In order to run nfd-worker as a "stand-alone" container against your
|
|
|
|
standalone nfd-master you need to run them in the same network namespace:
|
2020-10-20 20:50:46 +03:00
|
|
|
|
|
|
|
```bash
|
|
|
|
$ docker run --rm --network=container:nfd-test ${NFD_CONTAINER_IMAGE} nfd-worker
|
2020-09-08 20:16:17 +03:00
|
|
|
2019/02/01 14:48:56 Node Feature Discovery Worker <NFD_VERSION>
|
|
|
|
...
|
|
|
|
```
|
2020-10-20 20:50:46 +03:00
|
|
|
|
2020-09-08 20:16:17 +03:00
|
|
|
If you just want to try out feature discovery without connecting to nfd-master,
|
2021-02-24 14:29:07 +02:00
|
|
|
pass the `-no-publish` flag to nfd-worker.
|
2020-09-08 20:16:17 +03:00
|
|
|
|
|
|
|
**NOTE** Some feature sources need certain directories and/or files from the
|
|
|
|
host mounted inside the NFD container. Thus, you need to provide Docker with the
|
|
|
|
correct `--volume` options in order for them to work correctly when run
|
|
|
|
stand-alone directly with `docker run`. See the
|
2021-08-13 17:35:22 +03:00
|
|
|
[default deployment](https://github.com/kubernetes-sigs/node-feature-discovery/blob/{{site.release}}/deployment/components/common/worker-mounts.yaml)
|
2020-09-08 20:16:17 +03:00
|
|
|
for up-to-date information about the required volume mounts.
|
|
|
|
|
2021-01-20 21:49:41 +00:00
|
|
|
### NFD-Topology-Updater
|
|
|
|
|
|
|
|
In order to run nfd-topology-updater as a "stand-alone" container against your
|
|
|
|
standalone nfd-master you need to run them in the same network namespace:
|
|
|
|
|
|
|
|
```bash
|
|
|
|
$ docker run --rm --network=container:nfd-test ${NFD_CONTAINER_IMAGE} nfd-topology-updater
|
|
|
|
2019/02/01 14:48:56 Node Feature Discovery Topology Updater <NFD_VERSION>
|
|
|
|
...
|
|
|
|
```
|
|
|
|
|
|
|
|
If you just want to try out feature discovery without connecting to nfd-master,
|
|
|
|
pass the `-no-publish` flag to nfd-topology-updater.
|
|
|
|
|
|
|
|
NOTE:
|
|
|
|
|
|
|
|
NFD topology updater needs certain directories and/or files from the
|
|
|
|
host mounted inside the NFD container. Thus, you need to provide Docker with the
|
|
|
|
correct `--volume` options in order for them to work correctly when run
|
|
|
|
stand-alone directly with `docker run`. See the
|
|
|
|
[template spec](https://github.com/kubernetes-sigs/node-feature-discovery/blob/{{site.release}}/deployment/components/topology-updater/topologyupdater-mounts.yaml)
|
|
|
|
for up-to-date information about the required volume mounts.
|
|
|
|
|
|
|
|
[PodResource API][podresource-api] is a prerequisite for nfd-topology-updater.
|
|
|
|
Preceding Kubernetes v1.23, the `kubelet` must be started with the following flag:
|
|
|
|
`--feature-gates=KubeletPodResourcesGetAllocatable=true`.
|
|
|
|
Starting Kubernetes v1.23, the `GetAllocatableResources` is enabled by default
|
|
|
|
through `KubeletPodResourcesGetAllocatable` [feature gate][feature-gate].
|
|
|
|
|
2022-08-30 11:24:08 +03:00
|
|
|
## Running with Tilt
|
|
|
|
|
|
|
|
Another option for building NFD locally is via Tilt tool, which can build container
|
|
|
|
images, push them to a local registry and reload your Kubernetes pods automatically.
|
|
|
|
When using Tilt, you don't have to build container images and re-deploy your pods
|
|
|
|
manually but instead let the Tilt take care of it. Tiltfile is a configuration file
|
|
|
|
for the Tilt and is located at the root direcotry. To develop NFD with Tilt, follow
|
|
|
|
the steps below.
|
|
|
|
|
|
|
|
### Prerequisites
|
|
|
|
|
|
|
|
1. Install [Docker](https://docs.docker.com/engine/install/)
|
|
|
|
1. Setup Docker as a non-root user.
|
|
|
|
1. Install [kubectl](https://kubernetes.io/docs/tasks/tools/)
|
|
|
|
1. Install [kustomize](https://github.com/kubernetes-sigs/kustomize)
|
|
|
|
1. Install [tilt](https://docs.tilt.dev/install.html)
|
|
|
|
1. Create a local Kubernetes cluster
|
|
|
|
|
|
|
|
To start up your Tilt development environment, just run
|
|
|
|
|
|
|
|
```shell
|
|
|
|
tilt up
|
|
|
|
```
|
|
|
|
|
|
|
|
at the root of your local NFD codebase. Tilt will start a web interface in the
|
|
|
|
localhost and port 10350. From the web interface, you are able to see how NFD worker
|
|
|
|
and master are progressing, watch their build and runtime logs. Once your code changes
|
|
|
|
are saved locally, Tilt will notice it and re-build the container image from the
|
|
|
|
current code, push the image to the registry and re-deploy NFD pods with the latest
|
|
|
|
container image.
|
|
|
|
|
|
|
|
### Environment variables
|
|
|
|
|
|
|
|
To override environment variables used in the Tiltfile during image build,
|
|
|
|
export them in your current terminal before starting Tilt.
|
|
|
|
|
|
|
|
```shell
|
|
|
|
export IMAGE_TAG_NAME="v1"
|
|
|
|
tilt up
|
|
|
|
```
|
|
|
|
|
|
|
|
This will override the default value(`master`) of `IMAGE_TAG_NAME` variable defined
|
|
|
|
in the Tiltfile.
|
|
|
|
|
2020-09-08 20:16:17 +03:00
|
|
|
## Documentation
|
|
|
|
|
2020-11-02 10:48:17 +02:00
|
|
|
All documentation resides under the
|
2021-02-25 13:49:02 +02:00
|
|
|
[docs](https://github.com/kubernetes-sigs/node-feature-discovery/tree/{{site.release}}/docs)
|
2020-11-02 10:48:17 +02:00
|
|
|
directory in the source tree. It is designed to be served as a html site by
|
|
|
|
[GitHub Pages](https://pages.github.com/).
|
2020-10-01 09:25:52 +03:00
|
|
|
|
|
|
|
Building the documentation is containerized in order to fix the build
|
|
|
|
environment. The recommended way for developing documentation is to run:
|
|
|
|
|
|
|
|
```bash
|
|
|
|
make site-serve
|
|
|
|
```
|
|
|
|
|
|
|
|
This will build the documentation in a container and serve it under
|
|
|
|
[localhost:4000/](http://localhost:4000/) making it easy to verify the results.
|
|
|
|
Any changes made to the `docs/` will automatically re-trigger a rebuild and are
|
|
|
|
reflected in the served content and can be inspected with a simple browser
|
|
|
|
refresh.
|
|
|
|
|
|
|
|
In order to just build the html documentation run:
|
|
|
|
|
|
|
|
```bash
|
|
|
|
make site-build
|
|
|
|
```
|
|
|
|
|
|
|
|
This will generate html documentation under `docs/_site/`.
|
|
|
|
|
2020-11-02 10:48:17 +02:00
|
|
|
<!-- Links -->
|
2021-01-20 21:49:41 +00:00
|
|
|
[e2e-config-sample]: https://github.com/kubernetes-sigs/node-feature-discovery/blob/{{site.release}}/test/e2e/e2e-test-config.exapmle.yaml
|
|
|
|
[podresource-api]: https://kubernetes.io/docs/concepts/extend-kubernetes/compute-storage-net/device-plugins/#monitoring-device-plugin-resources
|
|
|
|
[feature-gate]: https://kubernetes.io/docs/reference/command-line-tools-reference/feature-gates
|