atmos-dev
653,787
Discuss atmos core development (golang). If you want to help out, reach out to @Erik Osterman (Cloud Posse)
erikabout 10 hours ago
@Andriy Knysh (Cloud Posse) refactoring clean up for store backends.
github.com/cloudposse/atmos/pull/2575
github.com/cloudposse/atmos/pull/2575
erik1 day ago
@Andriy Knysh (Cloud Posse) this is passing https://github.com/cloudposse/atmos/pull/2807
erik1 day ago
@Andriy Knysh (Cloud Posse) github.com/cloudposse/atmos/pull/2842 this updates everything to use
name_templateerik2 days ago
@Andriy Knysh (Cloud Posse) fixes bug reported by @Maksym Vlasov github.com/cloudposse/atmos/pull/2841
erik2 days ago
@Andriy Knysh (Cloud Posse) github.com/cloudposse/atmos/pull/2840 This adds only a regression test to confirm a certain reported behavior is no longer happening
erik3 days ago
@Andriy Knysh (Cloud Posse) github.com/cloudposse/atmos/pull/2831 fixes race condition reported by @Zack
erik3 days ago
@Andriy Knysh (Cloud Posse) this fixes the problem reported by @Juan Aguero where atmos would silently (log debug) ignore invalid atmos configs in
github.com/cloudposse/atmos/pull/2837
atmos.d .atmos.d.github.com/cloudposse/atmos/pull/2837
erik3 days ago
@Andriy Knysh (Cloud Posse) this fixes
github.com/cloudposse/atmos/pull/2812/changes
atmos git clone in CI, so it can clone the current repo.github.com/cloudposse/atmos/pull/2812/changes
erik3 days ago
@Andriy Knysh (Cloud Posse) github.com/cloudposse/atmos/pull/2828 this optimizes the atmos container commands by caching the detected container runtime (e.g. docker or podman)
erik3 days ago
@Andriy Knysh (Cloud Posse) this fixes the constant windows failures github.com/cloudposse/atmos/pull/2834
Zack3 days ago
github.com/cloudposse/atmos/pull/2829Follow up patch for github.com/cloudposse/atmos/pull/2816, then hopefully github.com/cloudposse/atmos/pull/2807 🙏
Zack5 days ago
@Erik Osterman (Cloud Posse) ok - I think this is ready to get into your branch
github.com/cloudposse/atmos/pull/2816
github.com/cloudposse/atmos/pull/2816
Brian5 days ago(edited)
Couple DX feedback:
1. I love
a. Adding the ability to pass state (eg, outputs) that can be used by another step would be a bonus.
2. Support for GitHub Action style
a. Not really needed for
3. Support for dynamic
a. We already discussed this. I just haven't spent time on solving it yet. Probably will start this evening.
1. I love
spin type. It makes it feel like a native atmos command. However, it hide error messages. It needs a way to surface an error on non-zero exits.a. Adding the ability to pass state (eg, outputs) that can be used by another step would be a bonus.
2. Support for GitHub Action style
if: condition for steps would be useful.a. Not really needed for
shell or spin type steps, but it would be useful for choice types. It would allow for similar interactive experience that atmos provides when running commands that required stack or component inputs. eg, custom command that has flag marked optional (but really is required) could prompt user to select a option when flag value is a empty value. 3. Support for dynamic
options for choicea. We already discussed this. I just haven't spent time on solving it yet. Probably will start this evening.
erik5 days ago
@Andriy Knysh (Cloud Posse) this one warrants a closer look and is an area you've spent the most time on over the years.
github.com/cloudposse/atmos/pull/2787
I don't know when it stopped working, but this formalizes and adds regression tests to ensure a "parent stack" can be decomposed into smaller files. It preserves the idea that component instances are unique, while allowing them to be split across multiple files.
github.com/cloudposse/atmos/pull/2787
I don't know when it stopped working, but this formalizes and adds regression tests to ensure a "parent stack" can be decomposed into smaller files. It preserves the idea that component instances are unique, while allowing them to be split across multiple files.
Juan Aguero5 days ago
Hey 👋 ,
Dropping the RDS IAM auth PRs here (full context in discussions #121):
- token —
- connect —
Token's self-contained; connect builds on it and has a couple of open design Qs (embedding the Amazon RDS CA bundle vs fetch-on-demand /
Dropping the RDS IAM auth PRs here (full context in discussions #121):
- token —
atmos aws rds token, ready for review → github.com/cloudposse/atmos/pull/2817- connect —
atmos aws rds connect + aws/rds integration, draft follow-on → github.com/cloudposse/atmos/pull/2818Token's self-contained; connect builds on it and has a couple of open design Qs (embedding the Amazon RDS CA bundle vs fetch-on-demand /
--ca-bundle, and the psql/mysql client-binary prereq) — kept it draft pending direction there. Happy to reshape either.Andriy Knysh (Cloud Posse)8 days ago
@Erik Osterman (Cloud Posse) github.com/cloudposse/atmos/pull/2805
erik9 days ago
@Andriy Knysh (Cloud Posse) we need to cut a patch release tomorrow as soon as these are passing
erik10 days ago
Then lets cut 1.224.0
erik10 days ago
@Andriy Knysh (Cloud Posse) actually one more is ready https://github.com/cloudposse/atmos/pull/2780
erik10 days ago
@Andriy Knysh (Cloud Posse) I was hoping to squeeze in one more PR but I think it's pushing it. Let's cut a release with what we have now. cc @Sean
erik11 days ago
@Andriy Knysh (Cloud Posse) this adds support for
github.com/cloudposse/atmos/pull/2675
tflint as a hook, and atmos terraform lint subcommand; it also includes native support for GitHub annotations, sarif format, and markdown summaries.github.com/cloudposse/atmos/pull/2675
erik11 days ago
Zack11 days ago(edited)
fix for a templating bug that we ran into after upgrading to atmos
github.com/cloudposse/atmos/pull/2779
1.223.0github.com/cloudposse/atmos/pull/2779
Brian12 days ago(edited)
Any opinion on
Related, any opinion on
choice options being determined at runtime? I would like to populate the options for choice in earlier step prior to reaching the step.Related, any opinion on
markdown not being pre-render before any steps: [] are executed? I would like to render .steps.<name>.output in markdown using markdown type.erik13 days ago
@Andriy Knysh (Cloud Posse) this just improves the hint when using
--use-version incorrectly github.com/cloudposse/atmos/pull/2775erik13 days ago
@Andriy Knysh (Cloud Posse) this provides an escape hatch, where if users specify
github.com/cloudposse/atmos/pull/2770
--mask=false, atmos will not attempt to pass sensitive values via ENV vars.github.com/cloudposse/atmos/pull/2770
erik13 days ago
@Andriy Knysh (Cloud Posse) this just adds test coverage; tried to reproduce the issue @Zack was having, but turns out it was already fixed. This just confirms that it's also already fixed 😃
erik13 days ago
@Andriy Knysh (Cloud Posse) this adds missing options to use build caches and mirrors
github.com/cloudposse/atmos/pull/2771
github.com/cloudposse/atmos/pull/2771
erik13 days ago
@Andriy Knysh (Cloud Posse) bug fixes for
github.com/cloudposse/atmos/pull/2774
--help cast output (it was getting recorded twice)github.com/cloudposse/atmos/pull/2774
erik13 days ago
@Andriy Knysh (Cloud Posse) documentation clean up for dependencies
github.com/cloudposse/atmos/pull/2773
github.com/cloudposse/atmos/pull/2773