-
Notifications
You must be signed in to change notification settings - Fork 63
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: CSE-based bootstrapping with bootstrapping client mode (#527)
* chore: update devcontainer go version * chore: refresh toolcain * chore: additional processing on verify (and migration to kube-system) * chore: bump dependencies * chore: refresh Helm charts * chore: update golangci config * chore: remove feature gate for drift * chore: update pre-commit tooling * chore: update the shape of main * chore: update the alt operator * chore: update the API (move kubelet config to AKSNodeClass) * chore: migrate cloud provider to v1 API * chore: migrate operator to v1 API * chore: migrate controllers to v1 API * chore: add nodeclass status controller * chore: migrate providers to v1 API * chore: migrate test pkg to v1 API * chore: update utils * chore: update and migrate E2E tests to v1 API * feat: refresh and relink CRDs * fix: move code generation into subfolders to fix golangci-lint (typecheck detecting multiple main.go) * fix: enable most of govet in golangci * fix(linting): exclude alt operator logger * fix: add nodeclass termination controller * fix(lint): restore linting on verify * feat: add nodeclass hash controller * fix: register additional nodeclass and status controllers * fix(e2e): better selection of karpenter pod for logs * fix(e2e): fix utilization suite * chore(e2e): add events to dump-logs (and simplify) * chore: rename v1 to corev1 * fix: remove extra $ * fix(e2e): add cilium label and taint * fix(e2e): fix labels and disruption for deamonset test * feat: update kubelet configuration * fix: conflicting nodeclaim.garbagecollcation controller name * chore: restore webhooks in alt operator * Clean up commented out webhook code * chore: swagger spec for nodebootstrapping GET and autogenerated client * feat: bootstrapping client provision mode * chore: general code improvements * fix: required VnetCidrs wasn't populated * chore: read error response as text * fix: handle the issue of mismatched error format in response body/header * chore: more accurate overhead subtraction for instancetype * fix: correct VM memory passing * test: some unit tests and improvements * chore: linter changes and small improvements * feat: makefile command for swagger client generation * chore: remove unused parameters in readResponse() * fix: swagger binary not found * chore: small improvements * chore: added missing fullstops in other env descriptions * chore: rename bootstrapping types to scriptless and customscripts to align with AgentBaker * fix(test): fix test for credential provider URL in custom data * Make webhooks work in AKS CCP context (#537) This requires quite a bit of hacking, mostly overriding certain things in the ctx. The major items are: * Copy and modify knative/pkg/webhook/resourcesemantics/conversion to support CRD clientConfig.url in addition to clientConfig.service. * Copy and modify karpenter/pkg/webhooks/webhooks.go to support overriding the informer factory, so that we can point it at the CCP APIServer rather than overlay. * Override Start and supporting methods on the provider specific operator in pkg/operator/operator.go to allow invoking our modified version of karpenter/pkg/webhooks/webhooks.go. * chore: MaxPods is already defaulted * chore: remove failSwapOn from kubelet settings in AKSNodeClass * fix: populate nodeClaim.Status.ImageID * fix: record NodeClass hash and add drift on static fields * chore: rename variabled * fix: remove outdated comment * fix: typo * chore: update CRDs * feat: support custom kubeletconfig per v1 * chore: ignore false positive golint overflow check * test: fix option for provision mode in testing --------- Co-authored-by: tallaxes <[email protected]> Co-authored-by: Matthew Christopher <[email protected]>
- Loading branch information
3 people
authored
Oct 25, 2024
1 parent
46fc0d3
commit 4f922e8
Showing
49 changed files
with
4,380 additions
and
68 deletions.
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
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
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
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
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,15 @@ | ||
| #!/bin/bash | ||
|
|
||
| # Define the file path | ||
| FILE="pkg/provisionclients/client/operations/node_bootstrapping_get_responses.go" | ||
|
|
||
| # Check if the file exists | ||
| if [ ! -f "$FILE" ]; then | ||
| echo "File $FILE does not exist." | ||
| exit 1 | ||
| fi | ||
|
|
||
| # Use sed to delete the readResponse() method if it exists | ||
| sed -i '/func (o \*NodeBootstrappingGetDefault) readResponse/,/^}/d' "$FILE" | ||
|
|
||
| echo "readResponse() method deleted from $FILE if it existed. This is for a temporary fix that is in node_bootstrapping_get_responses_override.go." |
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 |
|---|---|---|
|
|
@@ -24,6 +24,7 @@ tools() { | |
| go install github.com/rhysd/actionlint/cmd/[email protected] | ||
| go install github.com/mattn/[email protected] | ||
| go install github.com/google/go-containerregistry/cmd/[email protected] | ||
| go install github.com/go-swagger/go-swagger/cmd/[email protected] | ||
|
|
||
| if ! echo "$PATH" | grep -q "${GOPATH:-undefined}/bin\|$HOME/go/bin"; then | ||
| echo "Go workspace's \"bin\" directory is not in PATH. Run 'export PATH=\"\$PATH:\${GOPATH:-\$HOME/go}/bin\"'." | ||
|
|
||
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
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
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
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
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
Oops, something went wrong.