43 messages
Discuss atmos core development (golang). If you want to help out, reach out to @Erik Osterman (Cloud Posse)
E
E
E
erikabout 1 month ago
@Andriy Knysh (Cloud Posse) docs only https://github.com/cloudposse/atmos/pull/2132
E
E
E
M
Michaelabout 1 month 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 inputE
erikabout 1 month ago
@Andriy Knysh (Cloud Posse) this was confirmed by @MB https://sweetops.slack.com/archives/C031919U8A0/p1772642542133729?thread_ts=1772637524.256159&channel=C031919U8A0&message_ts=1772642542.133729
S
shirkevichabout 1 month 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
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
E
erikabout 1 month ago
@Billy Atkins shared that atmos is now live in the Google Code Wiki https://codewiki.google/github.com/cloudposse/atmos
R
RBabout 1 month ago
E
erikabout 1 month ago
E
erikabout 1 month ago
@RB thanks for the PRs. As soon as you're ready, nudge us in here.
E
E
E
erikabout 1 month ago
I loosed the requirements on issue templates, and added an AI prompt template for feature requests.
https://github.com/cloudposse/atmos/pull/2179
https://github.com/cloudposse/atmos/pull/2179
E
erikabout 1 month ago
@RB may need your help to finish this https://github.com/cloudposse/atmos/pull/2174#issuecomment-4052196897
E
erik30 days ago
@Andriy Knysh (Cloud Posse) this was hand validated by me (against
• 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
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
E
erik29 days 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
https://github.com/cloudposse/atmos/pull/2191
R
R
RB28 days 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
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
etcR
RB28 days 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
https://github.com/cloudposse/atmos/pull/2199/changes
--explainhttps://github.com/cloudposse/atmos/pull/2199/changes
R
R
RB28 days ago
A possible way to improve the stack deep merge by 3.5 times
https://github.com/cloudposse/atmos/pull/2201/changes
https://github.com/cloudposse/atmos/pull/2201/changes
R
RB28 days ago
Last one, thoughts on adding a description key for humans and for ai context
https://github.com/cloudposse/atmos/pull/2200/changes
https://github.com/cloudposse/atmos/pull/2200/changes
E
erik28 days ago
@RB 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.R
RB28 days ago
This PR fixes the last HIGH severity security issue in atmos
https://github.com/cloudposse/atmos/pull/2163/changes
https://github.com/cloudposse/atmos/pull/2163/changes
E
erik26 days ago
R
RB26 days ago
reduce ExecuteDescribeStacks cyclomatic complexity 247→10 + near-100% unit test coverage
https://github.com/cloudposse/atmos/pull/2204/changes
https://github.com/cloudposse/atmos/pull/2204/changes
R
RB26 days ago
Can we include the openssf scorecard in the atmos repo to improve security score ?
https://github.com/cloudposse/atmos/pull/2176/changes
https://github.com/cloudposse/atmos/pull/2176/changes
R
RB25 days 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
Related PRs
• https://github.com/cloudposse/atmos/pull/2226
• https://github.com/cloudposse/atmos/pull/2225
• https://github.com/cloudposse/atmos/pull/2204
E
erik24 days 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.
This fixes the problem where users try to log into the console with multiple identities and get an error from AWS.
R
RB20 days ago
E
R
RB18 days ago
do we have any tests that check for “speed” so an SLA/SLO can be created for various atmosy things?
E
erik17 days ago
@Andriy Knysh (Cloud Posse) this was validated by @Alexander Matveev
https://github.com/cloudposse/atmos/pull/2143
cc @Sean
https://github.com/cloudposse/atmos/pull/2143
cc @Sean
E
erik17 days ago
@Charles Smith I looked into what you reported on Office Hours.
https://github.com/cloudposse/atmos/pull/2254
https://github.com/cloudposse/atmos/pull/2254
E
erik16 days ago
@Andriy Knysh (Cloud Posse): @Zack confirmed this fix is working: https://github.com/cloudposse/atmos/pull/2253
E
erik15 days ago
@Andriy Knysh (Cloud Posse) https://github.com/cloudposse/atmos/pull/2264 just improved the error messaging from Atmos uploads.
R
RB15 days 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
fix: clear-text logging of sensitive information (CodeQL #5157) #2163
J
Jonathan Rose15 days 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
E
erik12 days ago
@Andriy Knysh (Cloud Posse) this PR from @Igor Rodionov LGTM
https://github.com/cloudposse/atmos/pull/2263
https://github.com/cloudposse/atmos/pull/2263
E
erik12 days ago
@Andriy Knysh (Cloud Posse) this PR from @Ben Smith (Benbentwo) LGTM
https://github.com/cloudposse/atmos/pull/2144
https://github.com/cloudposse/atmos/pull/2144