Getting Started For Developers

Getting Started For Developers

Download CDI

To download the source directly, simply

$ go get -u kubevirt.io/containerized-data-importer

Lint, Test, Build

GnuMake is used to drive a set of scripts that handle linting, testing, compiling, and containerizing. Executing the scripts directly is not supported at present.

NOTE: Standard builds require a running Docker daemon!

The standard workflow is performed inside a helper container to normalize the build and test environment for all devs. Building in the host environment is supported by the Makefile, but is not recommended.

Docker builds may be disabled by setting DOCKER=0; e.g.
$ make all DOCKER=0

$ make all executes the full workflow. For granular control of the workflow, several Make targets are defined:

Make Targets

all: cleans up previous build artifacts, compiles all CDI packages and builds containers
apidocs: generate client-go code (same as 'make generate') and swagger docs.
build: compile all CDI binary artifacts and generate controller and operator manifests
- build-controller: compile cdi-controller binary
- build-importer: compile cdi-importer binary
- build-apiserver: compile cdi-apiserver binary
- build-uploadproxy: compile cdi-uploadproxy binary
- build-uploadserver: compile cdi-uploadserver binary
- build-operator: compile cdi-operator binary
- No build-cloner target exists as the code is written in bash
build-functest-file-image-init: build the init container for the testing file server. (NOTE: the http and s3 components contain no CDI code, so do no require a build)
build-functest-image-http: build the http container for the testing file server
build-functest-registry-init: build the init container for the testing docker registry server
docker-functest-registry-populate: build the container that popuplates registry server with various container images
docker-functest-registry: build the container that hosts docker registry
clean: cleans up previous build artifacts
cluster-up: start a default Kubernetes or Open Shift cluster. set KUBEVIRT_PROVIDER environment variable to either 'k8s-1.15.1' or 'os-3.11.0' to select the type of cluster. set KUBEVIRT_NUM_NODES to something higher than 1 to have more than one node.
cluster-down: stop the cluster, doing a make cluster-down && make cluster-up will basically restart the cluster into an empty fresh state.
cluster-down-purge: cluster-down and cleanup all cached images from docker registry. Accepts make variables DOCKER_PREFIX. Removes all images of the specified repository. If not specified removes localhost repository of current cluster instance.
cluster-sync: builds the controller/importer/cloner, and pushes it into a running cluster. The cluster must be up before running a cluster sync. Also generates a manifest and applies it to the running cluster after pushing the images to it.
- cluster-sync-controller: builds the controller and pushes it into a running cluster.
- cluster-sync-importer: builds the importer and pushes it into a running cluster.
- cluster-sync-cloner: builds the cloner and pushes it into a running cluster.
- cluster-sync-apiserver: builds the apiserver and pushes it into a running cluster.
- cluster-sync-uploadproxy: builds the uploadproxy and pushes it into a running cluster.
- cluster-sync-uploadserver: builds the uploadserver and pushes it into a running cluster.
- cluster-sync-operator: builds the operator and pushes it into a running cluster.
deps-update: runs 'glide cc' and 'glide update'
docker: compile all binaries and build all containerized
- docker-controller: compile cdi-controller and build cdi-controller image
- docker-importer: compile cdi-importer and build cdi-importer image
- docker-cloner: build the cdi-cloner image (cloner is driven by a shell script, not a binary)
- docker-apiserver: compile cdi-apiserver and build cdi-apiserver image
- docker-uploadproxy: compile cdi-uploadproxy and build cdi-uploadproxy image
- docker-uploadserver: compile cdi-uploadserver and build cdi-uploadserver image
- docker-operator: compile cdi-operator and build cdi-operator image
- docker-olm-catalog: pack OLM bundles in dedicated directories under olm-catalog and build olm-catalog-registry container image. Accepts make variables QUAY_NAMESPACE, QUAY_REPOSITORY, CSV_VERSION.
- docker-registry-cleanup: remove all images of specifed repo from local docker registry. if not specified removes from localhost repo of current cluster instance. Accepts make variables DOCKER_REPO.
- docker-functest-images: compile and build the file host and docker registry images for functional tests
  - docker-functest-image-init: compile and build the file host init image for functional tests
  - docker-functest-image-http: only build the file host http container for functional tests
  - docker-functest-registry-init:compile and build the registry init image for functional tests
  - docker-functest-registry-populate: only build registry-populate container for functional tests
  - docker-functest-registry: only build docker-registry container for functional tests
  - Note: there is no target for the S3 container, an offical Minio container is used instead
format: execute shfmt, goimports, and go vet on all CDI packages. Writes back to the source files.
generate: generate client-go deepcopy functions, clientset, listers and informers.
generate-verify: generate client-go deepcopy functions, clientset, listers and informers and validate codegen.
goveralls: run code coverage tracking system.
manifests: generate a cdi-controller and operator manifests in _out/manifests/. Accepts make variables DOCKER_TAG, DOCKER_PREFIX, VERBOSITY, PULL_POLICY, CSV_VERSION, QUAY_REPOSITORY, QUAY_NAMESPACE
publish: CI ONLY - this recipe is not intended for use by developers
olm-verify: verify generated olm manifests
olm-push: push generated operator bundle to quay.io. Accepts make_variables CSV_VERSION, QUAY_USER, QUAY_PASSWORD, QUAY_REPOSITORY
push: compiles, builds, and pushes to the repo passed in DOCKER_PREFIX=<my repo>
- push-controller: compile, build, and push cdi-controller
- push-importer: compile, build, and push cdi-importer
- push-cloner: compile, build, and push cdi-cloner
- push-apiserver: compile, build, and push cdi-apiserver
- push-uploadproxy: compile, build, and push cdi-uploadproxy
- push-uploadserver: compile, build, and push cdi-uploadserver
- push-operator: compile, build, and push cdi-operator
release-description: generate a release announcement detailing changes between 2 commits (typically tags). Expects RELREF and PREREF to be set
- e.g. $ make release-description RELREF=v1.1.1 PREREF=v1.1.1-alpha.1
test: execute all tests (NOTE: WHAT is expected to match the go cli pattern for paths e.g. ./pkg/.... This differs slightly from rest of the make targets)
- test-unit: execute all tests under ./pkg/...
- test-functional: execute functional tests under ./tests/.... Additional test flags can be passed to the test binary via the TEST_ARGS variable, see below for an example and restrictions.
- test-lint runs gofmt and golint tests against src files
vet: lint all CDI packages

Make Variables

Several variables are provided to alter the targets of the above Makefile recipes.

These may be passed to a target as $ make VARIABLE=value target

WHAT: The path from the repository root to a target directory (e.g. make test WHAT=pkg/importer)
DOCKER_PREFIX: (default: kubevirt) Set repo globally for image and manifest creation
DOCKER_TAG: (default: latest) Set global version tags for image and manifest creation
VERBOSITY: (default: 1) Set global log level verbosity
PULL_POLICY: (default: IfNotPresent) Set global CDI pull policy
CSV_VERSION: (default: v0.0.0) Set CSV version of cdi-operator for OLM manifests
QUAY_USERNAME: (default: N/A) username to quay.io
QUAY_PASSWORD: (default: N/A) password for quay.io
QUAY_NAMESPACE: (default: kubevirt) namespace where cdi application is located
QUAY_REPOSITORY: (default: cdi) application name
TEST_ARGS: A variable containing a list of additional ginkgo flags to be passed to functional tests. The string "--test-args=" must prefix the variable value. For example:
```
       `make TEST_ARGS="--test-args=-ginkgo.noColor=true" test-functional >& foo`.
```
Note: the following extra flags are not supported in TEST_ARGS: -master, -cdi-namespace, -kubeconfig, -kubectl-path since these flags are overridden by the hack/build/run-functional-tests.sh script. To change the default settings for these values the KUBE_MASTER_URL, CDI_NAMESPACE, KUBECONFIG, and KUBECTL variables, respectively, must be set.
RELREF: Required by release-description. Must be a commit or tag. Should be the more recent than PREREF
PREREF: Required by release-description. Must also be a commit or tag. Should be the later than RELREF

Execute Standard Environment Functional Tests

If using a standard bare-metal/local laptop rhel/kvm environment where nested virtualization is supported then the standard kubevirtci framework can be used.

Environment Variables and Supported Values

Env Variable	Default	Additional Values
KUBEVIRT_PROVIDER	k8s-1.15.1	os-3.11.0-crio or okd-4.1
KUBEVIRT_PROVIDER_EXTRA_ARGS		--enable-ceph (for providers that support this argument like k8s-1.15.1
NUM_NODES	1	2-5

To Run Standard cluster-up/kubevirtci Tests

 # make cluster-up
 # make cluster-sync
 # make test-functional

To run specific functional tests, you can leverage ginkgo command line options as follows:

# make TEST_ARGS="--test-args=-ginkgo.focus=<test_suite_name>" test-functional

E.g. to run the tests in transport_test.go:

# make TEST_ARGS="--test-args=-ginkgo.focus=Transport" test-functional

Clean Up

 # make cluster-down

Clean Up with docker container cache cleanup To cleanup all container images from local registry and to free a considerable amount of disk space. Note: caveat - cluser-sync will take longer since will have to fetch all the data again

 # make cluster-down-purge

Execute Alternative Environment Functional Tests

If running in a non-standard environment such as Mac or Cloud where the kubevirtci framework is not supported, then you can use the following example to run Functional Tests.

Stand-up a Kubernetes cluster (local-up-cluster.sh/kubeadm/minikube/etc...)
Clone or get the kubevirt/containerized-data-importer repo

Run the CDI controller manifests

To generate latest manifests

# make manifests

To customize environment variables see make targets

Run the generated latest manifests There are two options to deploy cdi directly via cdi-controller.yaml or to deploy it via operator

Direct deployment

  #kubectl create -f ./_out/manifests/cdi-controller.yaml
  
  namespace/cdi created
  customresourcedefinition.apiextensions.k8s.io/datavolumes.cdi.kubevirt.io created
  customresourcedefinition.apiextensions.k8s.io/cdiconfigs.cdi.kubevirt.io created
  clusterrole.rbac.authorization.k8s.io/cdi created
  clusterrolebinding.rbac.authorization.k8s.io/cdi-sa created
  clusterrole.rbac.authorization.k8s.io/cdi-apiserver created
  clusterrolebinding.rbac.authorization.k8s.io/cdi-apiserver created
  clusterrolebinding.rbac.authorization.k8s.io/cdi-apiserver-auth-delegator created
  serviceaccount/cdi-sa created
  deployment.apps/cdi-deployment created
  configmap/cdi-insecure-registries created
  serviceaccount/cdi-apiserver created
  rolebinding.rbac.authorization.k8s.io/cdi-apiserver created
  role.rbac.authorization.k8s.io/cdi-apiserver created
  rolebinding.rbac.authorization.k8s.io/cdi-extension-apiserver-authentication created
  role.rbac.authorization.k8s.io/cdi-extension-apiserver-authentication created
  service/cdi-api created
  deployment.apps/cdi-apiserver created
  service/cdi-uploadproxy created
  deployment.apps/cdi-uploadproxy created

Deployment via operator

  #./cluster-up/kubectl.sh apply -f "./_out/manifests/release/cdi-operator.yaml" 
  namespace/cdi created
  customresourcedefinition.apiextensions.k8s.io/cdis.cdi.kubevirt.io created
  configmap/cdi-operator-leader-election-helper created
  clusterrole.rbac.authorization.k8s.io/cdi.kubevirt.io:operator created
  serviceaccount/cdi-operator created
  clusterrole.rbac.authorization.k8s.io/cdi-operator-cluster-permissions created
  clusterrolebinding.rbac.authorization.k8s.io/cdi-operator created
  deployment.apps/cdi-operator created

  #./cluster-up/kubectl.sh apply -f "./_out/manifests/release/cdi-cr.yaml"
  cdi.cdi.kubevirt.io/cdi created

Build and run the func test servers In order to run fucntional tests the below servers have to be run
- host-file-server is required by the functional tests and provides an endpoint server for image files and s3 buckets
- registry-server is required by the functional tests and provides an endpoint server for container images. Note: for this server to run the follwoing setting is required in each cluster node systemctl -w user.max_user_namespaces=1024
Build and Push to registry
```
# DOCKER_PREFIX=<repo> DOCKER_TAG=<tag> make docker-functest-images
```
Generate manifests
```
# DOCKER_PREFIX=<repo> DOCKER_TAG=<docker tag> PULL_POLICY=<pull policy> VERBOSITY=<verbosity> CSV_VERSION=<CSV version> QUAY_NAMESPACE=<namespace> QUAY_REPOSITORY=<application name> make manifests 
```
Run servers
```
# ./cluster-up/kubectl.sh apply -f ./_out/manifests/file-host.yaml
# ./cluster-up/kubectl.sh apply -f ./_out/manifests/registry-host.yaml
```
Run the tests

 # make test-functional

If you encounter test errors and are following the above steps try:

 # make clean && make docker

redeploy the manifests above, and re-run the tests.

Submit PRs

All PRs should originate from forks of kubevirt.io/containerized-data-importer. Work should not be done directly in the upstream repository. Open new working branches from master/HEAD of your forked repository and push them to your remote repo. Then submit PRs of the working branch against the upstream master branch.

Releases

Release practices are described in the release doc.

Vendoring Dependencies

This project uses glide as it's dependency manager. At present, all project dependencies are vendored; using glide is unnecessary in the normal work flow.

Install glide:

curl https://glide.sh/get | sh

Then run it from the repo root

glide install -v

glide install scans imports and resolves missing and unused dependencies. -v removes nested vendor and Godeps/_workspace directories.

S3-compatible client setup:

AWS S3 cli

$HOME/.aws/credentials

[default]
aws_access_key_id = <your-access-key>
aws_secret_access_key = <your-secret>

Minio cli

$HOME/.mc/config.json:

{
        "version": "8",
        "hosts": {
                "s3": {
                        "url": "https://s3.amazonaws.com",
                        "accessKey": "<your-access-key>",
                        "secretKey": "<your-secret>",
                        "api": "S3v4"
                }
        }
}

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

README.md

README.md

Getting Started For Developers

Download CDI

Lint, Test, Build

Make Targets

Make Variables

Execute Standard Environment Functional Tests

Execute Alternative Environment Functional Tests

Direct deployment

Deployment via operator

Submit PRs

Releases

Vendoring Dependencies

S3-compatible client setup:

AWS S3 cli

Minio cli

Files

README.md

Latest commit

History

README.md

File metadata and controls

Getting Started For Developers

Download CDI

Lint, Test, Build

Make Targets

Make Variables

Execute Standard Environment Functional Tests

Execute Alternative Environment Functional Tests

Direct deployment

Deployment via operator

Submit PRs

Releases

Vendoring Dependencies

S3-compatible client setup:

AWS S3 cli

Minio cli