-
Notifications
You must be signed in to change notification settings - Fork 14
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
Allow using AMIs built for architectures other than amd64 #202
Comments
@ldx @jrroman @henryprecheur What you guys think of having something really generic, such as: bootImageSpec:
amis:
- familyRegex: "mac.*"
owners: 689494258501
filters: name=elotl-kip-mac*
- familyRegex: "*" # default
owners: 689494258501
filters: name=elotl-kip-* |
@hidalgopl so the issue i see with this is it will break reverse compatibility with previous releases of kip. We could potentially add this for additional ami fields while keeping the base of owner and filter. This is the way i have been testing currently: 16 bootImageSpec:
17 owners: 689494258501
18 filters: name=elotl-kip-*
19 macfilters: name=elotl-kipmac-*
20 armfilters: name=elotl-kiparm-* I do however think what you have described is very clear contextually and intuitive. |
@jrroman You're right, I haven't thought about it. We can keep backwards compatibility by keeping what I proposed under new key: bootImageSpec:
owners: 689494258501
filters: name=elotl-kip-*
overrideAMI:
- familyRegex: "mac.*"
owners: 689494258501
filters: name=elotl-kip-mac*
- familyRegex: "*" # default
owners: 689494258501
filters: name=elotl-kip-* Big question is how flexible this feature needs to be. And whether we want to have different AMI for different arch, or different AMI for different instance type familiy. |
We can keep the current keys (filters, owners, etc) for backward compatibility. Another missing piece: an instance family -> architecture mapping, so we know which architecture we need when starting an instance. |
Right now kip can only use one AMI specified via
bootImageSpec
in provider.yaml.However, AMIs are specific to the OS and CPU architecture. To support other architectures and OSes (ARM, mac1, etc) we need to be able to specify a boot image filter for each.
The text was updated successfully, but these errors were encountered: