-
Notifications
You must be signed in to change notification settings - Fork 430
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate to CRDS and Kube builder #33
Merged
Merged
Changes from 49 commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
befff6d
initial machine actuator update implementation
marwanad 0fbb848
update kubelet and control plane version
marwanad 3ca0378
init kubebuilder project
marwanad 0b4a88d
add API types for machine and cluster provider config
marwanad 5e2c20b
prepare for cluster api vendor upgrade
marwanad 746cf33
vendor: update cluster-api and vendor controller-runtime dependencies
marwanad 2c83deb
move cloud directory under pkg
marwanad 7288998
update cluster actuator to use controller manager
marwanad b245b91
update machine actuator and machine status
marwanad 1023e46
remove unused providerconfig
marwanad 32fa826
update manager to reigster acutators properly
marwanad 75e319e
fix package import for azureconfigv1
marwanad 085d8bd
re-run codegen
marwanad 32bb521
remove unused controller
marwanad 14f5fbe
make it compile
marwanad 16bb34e
move arm templates under resourcemanagement package and update Docker…
marwanad 8906a00
update kustomize config files
marwanad 4539635
use generated cluster name in the template file
marwanad 4d32b96
fix one machine update test case that doesnt utilize cluster api client
marwanad 51de26b
update e2e test client
marwanad de596bb
vendor: update vendored dependencies
marwanad bd40d35
manually add upstream crd configs
marwanad 920853d
set logtostderr flag for manager
marwanad 6212ddf
remove exampleconfigs and duplicate boilerplate.txt
marwanad d2b2e21
move configtemplate to clusterctl/examples
marwanad e9cce81
add vendors target, dev image push and build to makefile
marwanad 68e7d3f
add targets for machine and cluster actuator unit tests
marwanad 6398dc6
update build and test pipeline
marwanad 574d02c
dont prune cluster-api package
marwanad f440c5e
set NODE_NAME env var to avoid self deletion
marwanad 207f55e
fix machine status update
marwanad 86c1d8b
disable e2e pipeline
marwanad 4129ef4
update .gitignore
marwanad dcec2b4
cleanup generate-yaml script and remove unused makefiles
marwanad 675fcd1
fix package import and remove unused test files
marwanad 8884720
remove unused provider components template
marwanad 5d8651e
update kustomize secret generator to read from env variables
marwanad ecb4f7f
replace minikube with kubeadm-dind
marwanad b6aaaab
setup local docker registry and configure the nodes to pull the images
marwanad 4a0e8de
fix the rest of the E2E steps and re-enable the pipeline
marwanad ead34fa
change docker image placeholder
marwanad d914b2c
export service principal values to a file
marwanad 129dfde
update README with the new setup instructions
marwanad fa55115
vendor: update packages
marwanad afe761c
add skipping creating service principal if flag is set
marwanad fe3957b
update e2e tests
marwanad 9b8fd56
fix machine annotation update for first creation
marwanad a5b920d
only update the status once for node recreation
marwanad 9b3a268
install specific control plane version through kubeadm
marwanad aa97db8
upgrade kubectl client version with the control plane
marwanad ab655ce
switch to alpine 3.8
marwanad 06cb84b
use explicit argument indices for formatting strings
marwanad File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,10 @@ | ||
| *.test | ||
| .idea/ | ||
| bin/ | ||
| .env | ||
| exampleconfigs/provider-components.yaml | ||
| minikube.kubeconfig | ||
| generatedconfigs/ | ||
| sshkey* | ||
| kubeconfig | ||
| *.out |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| # Build the manager binary | ||
| FROM golang:1.10.3 as builder | ||
|
|
||
| # Copy in the go src | ||
| WORKDIR /go/src/github.com/platform9/azure-provider | ||
| COPY pkg/ pkg/ | ||
| COPY cmd/ cmd/ | ||
| COPY vendor/ vendor/ | ||
|
|
||
| # Build | ||
| RUN CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -a -o manager github.com/platform9/azure-provider/cmd/manager | ||
|
|
||
| # Copy the controller-manager into a thin image | ||
| FROM ubuntu:latest | ||
| WORKDIR /root/ | ||
| COPY --from=builder /go/src/github.com/platform9/azure-provider/manager . | ||
| COPY --from=builder /go/src/github.com/platform9/azure-provider/pkg/cloud/azure/services/resourcemanagement/template/deployment-template.json . | ||
| ENTRYPOINT ["./manager"] | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is alpine 3.8 possible?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yup - it should be. The file was generated using kube-builder and I think it defaulted to ubuntu.