-
Notifications
You must be signed in to change notification settings - Fork 21
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
feat: Support Flatcar OS #225
Conversation
Add preKubeadmCommands for node name replacement Add flatcarDriver in poetry plugin list Enable required feature gate
Fix ignition ignoring error. Fix lint. Add comment about PATH overwrite in flatcar.
Remove default os in image build cmd
changed ignition config to match the one used by CAPO
CAPI expects a CA secret of type "cluster.x-k8s.io/secret" instead of "kubernetes.io/tls". Also, the "cluster.x-k8s.io/cluster-name" label has been added, containing the cluster's stack id.
1fe298b
to
dfc1fe8
Compare
sadly it died on me right while i was debugging, but i was able to get into vnc using ngrok and found that it was complaining about an invalid/incorrect pound sign in the ignition config so it just died ( i'm restarting but this time i'll try to pull out the config it generated. |
ok, for some reason, it's getting plain ignition.. hold on... |
Out of curiosity, what does it get? |
5f7c715#diff-b74064dc7fb33b2c4232e9a38c8df0f8c59e996507abeb15478b0973e2e0a2d1R47 cloud-init, but that my bad :) |
success! it's just running sonobuoy now, if it's all good (which i believe it will be), i'll push up a later patch which will add the matrix back and will be able to merge this today. |
Based on: #44
Adapted the aforementioned PR to contain the right ignition config and also modified the secret type used for the CA cert secret.
CAPI expects a CA secret of type "cluster.x-k8s.io/secret" instead of "kubernetes.io/tls". Also, the "cluster.x-k8s.io/cluster-name" label has been added for the secret resource, containing the cluster's stack id.