#atmos-dev - March 2026

43 messages

Discuss atmos core development (golang). If you want to help out, reach out to @Erik Osterman (Cloud Posse)

erik
erik8 months ago
erik
erik8 months ago
erik
erik8 months ago
@Andriy Knysh (Cloud Posse) docs only https://github.com/cloudposse/atmos/pull/2132
erik
erik8 months ago
Michael
Michael8 months ago
Just testing out some of the new Atmos agent skills and ran into this install issue. Am I doing something wrong?
❯ /plugin marketplace add cloudposse/atmos
  ⎿  Error: Invalid schema:
     /Users/rosesecurity/.claude/plugins/marketplaces/cloudposse-atmos/.claude-plugin/marketplace.json
     plugins.0.source: Invalid input
shirkevich
shirkevich8 months ago
I would like to contribute concurrent component provisioning, here is a proposal for discussion:
https://github.com/cloudposse/atmos/pull/2159

Currently we are using our own github actions workflows that are patched to implement dependencies, but it is really ugly to do it via GH and matrix,

Possibly it should be implemented after this one: https://github.com/cloudposse/atmos/pull/1891
erik
erik8 months ago
@Billy Atkins shared that atmos is now live in the Google Code Wiki https://codewiki.google/github.com/cloudposse/atmos
RB (pat leave again)
RB (pat leave again)8 months ago
Please review some gha fixes in the atmos repo

https://github.com/cloudposse/atmos/pull/2160/changes
erik
erik8 months ago
@RB (pat leave again) https://github.com/cloudposse/atmos/pull/2158

Did you personally validate this is working?
erik
erik8 months ago
@RB (pat leave again) thanks for the PRs. As soon as you're ready, nudge us in here.
erik
erik8 months ago
erik
erik8 months ago
I loosed the requirements on issue templates, and added an AI prompt template for feature requests.

https://github.com/cloudposse/atmos/pull/2179
erik
erik8 months ago
@RB (pat leave again) may need your help to finish this https://github.com/cloudposse/atmos/pull/2174#issuecomment-4052196897
erik
erik8 months ago
@Andriy Knysh (Cloud Posse) this was hand validated by me (against examples/toolchain) and resolves a few of the recurring problems we've been having.
• Package dependency is not accessible after install.
• Uninstall command not removing package dependencies.
After this change, finally one-shotting, toolchain installation should be possible or working with atmos terraform commands.

https://github.com/cloudposse/atmos/pull/2172
erik
erik8 months ago
@Andriy Knysh (Cloud Posse) I ran into this defining some custom commands. Manually verified this works for me.

https://github.com/cloudposse/atmos/pull/2191
RB (pat leave again)
RB (pat leave again)8 months ago
RB (pat leave again)
RB (pat leave again)8 months ago
What do you guys think of having atmos plugins so each subcommand (validate/describe/terraform/helmfile) can be its own separate repo with its own versioned binary?

This way atmos can be a "shell" of a program and can be extendable e.g. you can do something like this

atmos plugin add validate@1.0.0
atmos plugin add terraform@1.2.3
atmos plugin add opentofu@2.0.0
etc
RB (pat leave again)
RB (pat leave again)8 months ago
I recall we talked about a switch that would explain where every key came from post deep merge. This should give us that via --explain

https://github.com/cloudposse/atmos/pull/2199/changes
RB (pat leave again)
RB (pat leave again)8 months ago
RB (pat leave again)
RB (pat leave again)8 months ago
A possible way to improve the stack deep merge by 3.5 times

https://github.com/cloudposse/atmos/pull/2201/changes
RB (pat leave again)
RB (pat leave again)8 months ago
Last one, thoughts on adding a description key for humans and for ai context

https://github.com/cloudposse/atmos/pull/2200/changes
erik
erik8 months ago
@RB (pat leave again) please start with PRDs for all the features. see docs/prds; without the PRD, cannot assess the scope easily and implementations tend to not follow conventions.
RB (pat leave again)
RB (pat leave again)8 months ago
This PR fixes the last HIGH severity security issue in atmos

https://github.com/cloudposse/atmos/pull/2163/changes
erik
erik8 months ago
@Andriy Knysh (Cloud Posse) I think this is ready
https://github.com/cloudposse/atmos/pull/2181
RB (pat leave again)
RB (pat leave again)8 months ago
reduce ExecuteDescribeStacks cyclomatic complexity 247→10 + near-100% unit test coverage

https://github.com/cloudposse/atmos/pull/2204/changes
RB (pat leave again)
RB (pat leave again)8 months ago
Can we include the openssf scorecard in the atmos repo to improve security score ?

https://github.com/cloudposse/atmos/pull/2176/changes
RB (pat leave again)
RB (pat leave again)8 months ago
prd to track and decrease cyclomatic complexity in order to improve test coverage and thus reduce future bugs

Related PRs

• https://github.com/cloudposse/atmos/pull/2226
• https://github.com/cloudposse/atmos/pull/2225
• https://github.com/cloudposse/atmos/pull/2204
erik
erik8 months ago
@Andriy Knysh (Cloud Posse) https://github.com/cloudposse/atmos/pull/2229
This fixes the problem where users try to log into the console with multiple identities and get an error from AWS.
RB (pat leave again)
RB (pat leave again)8 months ago
Improve test coverage, please review

https://github.com/cloudposse/atmos/pull/2173/changes
RB (pat leave again)
RB (pat leave again)7 months ago
do we have any tests that check for “speed” so an SLA/SLO can be created for various atmosy things?
erik
erik7 months ago
@Andriy Knysh (Cloud Posse) this was validated by @Alexander Matveev
https://github.com/cloudposse/atmos/pull/2143

cc @Sean
erik
erik7 months ago
@Charles Smith I looked into what you reported on Office Hours.
https://github.com/cloudposse/atmos/pull/2254
erik
erik7 months ago
@Andriy Knysh (Cloud Posse): @Zack confirmed this fix is working: https://github.com/cloudposse/atmos/pull/2253
erik
erik7 months ago
@Andriy Knysh (Cloud Posse) https://github.com/cloudposse/atmos/pull/2264 just improved the error messaging from Atmos uploads.
RB (pat leave again)
RB (pat leave again)7 months ago
Could i get a review to close this small security finding from github codeql?

fix: clear-text logging of sensitive information (CodeQL #5157) #2163
Jonathan Rose
Jonathan Rose7 months ago
Dumb question. I was told to fork the project and make a PR. I did that with https://github.com/thejrose1984/atmos/pull/1 but I'm assuming that means nothing because it won't actually do anything to make a PR against the source
erik
erik7 months ago
@Andriy Knysh (Cloud Posse) this PR from @Igor Rodionov LGTM
https://github.com/cloudposse/atmos/pull/2263
erik
erik7 months ago
@Andriy Knysh (Cloud Posse) this PR from @Ben Smith (Benbentwo) LGTM
https://github.com/cloudposse/atmos/pull/2144