Skip to content
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

flake based on Nixpkgs Python #163

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

roberth
Copy link
Member

@roberth roberth commented Feb 29, 2024

TODO

While I know I've manually tested this functionality, it appears
that I've only confirmed the automatic setting of hostPlatform, but
not the AMI selection. The mistake was that I'd run it on an existing
deployment where the AMI was specified manually. After instance
creation, the AMI is irrelevant, and NixOps does not report anything
about it. That seems to be a desirable behavior, as the original AMI
*should* have no effect on the system state, and updates to the AMIs
could therefore cause unnecessary messages; especially when a new NixOS
comes out (amazon-ec2-amis.nix update) or when you upgrade the machines.
... for basically no other reason than testing the polyfill.
This should keep maintenance a bit simpler, by avoiding poetry2nix.
The Nixpkgs approach needs to be supported anyway.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant