Skip to content

Commit

Permalink
updates per PR review
Browse files Browse the repository at this point in the history
  • Loading branch information
dgkanatsios committed Dec 13, 2021
1 parent 738a3b4 commit 674179a
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 17 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
NS=ghcr.io/playfab
NS ?= ghcr.io/playfab

export IMAGE_NAME_OPERATOR=thundernetes-operator
export IMAGE_NAME_INIT_CONTAINER=thundernetes-initcontainer
Expand Down
5 changes: 3 additions & 2 deletions docs/development.md
Original file line number Diff line number Diff line change
Expand Up @@ -271,9 +271,10 @@ The docker-compose.yml file on the root of this repo was created to facilitate s

To test your changes to thundernetes to a Kubernetes cluster, you can use the following steps:

- Go to the Makefile on the root of the project and change the container registry url to the one you have access to. It is stored in the variable `NS`
- The Makefile on the root of the project contains a variable `NS` that points to the container registry that you use during development. So you'd need to either set the variable in your environment (`export NS=<your-container-registry>`) or set it before calling `make` (like `NS=<your-container-registry> make build push`).
- Login to your container registry (`docker login`)
- Run `make build push` to build the container images and push them to your container registry
- Run `create-install-files-dev` to create the install files for the cluster
- Checkout the `installfilesdev` folder for the generated install files. This file is included in .gitignore so it will never be committed.
- Test your changes and be careful not to commit the container registry URL change
- Test your changes as required.

28 changes: 14 additions & 14 deletions operator/config/manager/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
resources:
- manager.yaml
generatorOptions:
disableNameSuffixHash: true
configMapGenerator:
- files:
- controller_manager_config.yaml
name: manager-config
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
images:
- name: controller
newName: thundernetes-operator
newTag: b460a56
resources:
- manager.yaml
generatorOptions:
disableNameSuffixHash: true
configMapGenerator:
- files:
- controller_manager_config.yaml
name: manager-config
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
images:
- name: controller
newName: lili/lele/thundernetes-operator
newTag: 738a3b4

0 comments on commit 674179a

Please sign in to comment.