This repository has been archived by the owner on Jun 29, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 49
Install components using flux helm-controller and source-controller #1557
Closed
Conversation
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
After talking with Suraj offline, it was thought that it would be better to move "converting components to Helm charts" commits in a different PR so that they can be merged to master. |
This was referenced Aug 30, 2021
knrt10
reviewed
Aug 31, 2021
knrt10
suggested changes
Aug 31, 2021
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.
Nice work @surajssd. Some initial review
surajssd
force-pushed
the
surajssd/component-auto-update
branch
4 times, most recently
from
September 9, 2021 12:29
01466d2
to
0a79ba2
Compare
…-driver" This reverts commit 004294c, reversing changes made to 9f67cb3. This is reverted due to flakiness/issue of `aws-ebs-csi-driver` on Kubernetes version 1.21.4. `ebs-plugin` container panics due to not being able to query instance metadata: ``` W0909 10:57:15.342750 1 metadata.go:101] EC2 instance metadata is not available panic: error getting information from metadata service or node object: node providerID empty, cannot parse goroutine 1 [running]: github.com/kubernetes-sigs/aws-ebs-csi-driver/pkg/driver.newNodeService(0xc00003a730, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0) /go/src/github.com/kubernetes-sigs/aws-ebs-csi-driver/pkg/driver/node.go:88 +0x24c github.com/kubernetes-sigs/aws-ebs-csi-driver/pkg/driver.NewDriver(0xc00050ff40, 0x7, 0x7, 0xc000078660, 0xc00032ec30, 0x1fded28) /go/src/github.com/kubernetes-sigs/aws-ebs-csi-driver/pkg/driver/driver.go:94 +0x425 main.main() /go/src/github.com/kubernetes-sigs/aws-ebs-csi-driver/cmd/main.go:46 +0x212 ```
Revert "Merge pull request #1504 from kinvolk/surajssd/update-aws-csi-driver
This commit removes the Dockerfile and iscsiadm files from the repo and updates the README for newer instructions on how to build the kubelet image. Signed-off-by: Imran Pochi <imran@kinvolk.io>
Signed-off-by: Imran Pochi <imran@kinvolk.io>
Signed-off-by: Imran Pochi <imran@kinvolk.io>
Update k8s to 1.21.4
surajssd
force-pushed
the
surajssd/component-auto-update
branch
from
September 13, 2021 09:09
0a79ba2
to
5487d44
Compare
This commit adds additional packages which are required in the `flatcar-install` script. Signed-off-by: Imran Pochi <imran@kinvolk.io>
…-image tinkerbell: Add btrfs-progs and gawk to Dockerfile
Release CHANGELOG and scripts to update controller and worker nodes: etcd and host kubelet. Signed-off-by: Imran Pochi <imran@kinvolk.io> Co-authored-by: Imran Pochi <imran@kinvolk.io>
Signed-off-by: Imran Pochi <imran@kinvolk.io>
Signed-off-by: Imran Pochi <imran@kinvolk.io>
Release v0.9.0
surajssd
force-pushed
the
surajssd/component-auto-update
branch
from
September 24, 2021 05:35
5487d44
to
403785c
Compare
Signed-off-by: Suraj Deshmukh <surajd.service@gmail.com>
- Add flux helm-controller API. - Add flux source-controller API. - Add sigs.k8s.io/controller-runtime. Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
This commit adds a new function in the Component interface that can generate the "HelmRelease" file. Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
This commit collects more git metadata like branch and commit. This will be further used to create GitRepository CR. Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
This commit adds support for `lokoctl experimental component apply`. Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
The CRDs and corresponding CRs are installed from the same template dir, hence the initial validation fails since helm-controller cannot find the schema on api-server. This commit makes the ordering little different by installing the CRDs before CRs using helm hooks. Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
surajssd
force-pushed
the
surajssd/component-auto-update
branch
from
October 4, 2021 12:47
403785c
to
2f5613b
Compare
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Fixes: #1528