43 messages
Discuss atmos core development (golang). If you want to help out, reach out to @Erik Osterman (Cloud Posse)
E
E
E
E
E
E
M
Michael3 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 inputE
erik3 months 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
shirkevich2 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
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
erik2 months ago
@Billy Atkins shared that atmos is now live in the Google Code Wiki https://codewiki.google/github.com/cloudposse/atmos
R
RB2 months ago
E
erik2 months ago
E
erik2 months ago
@RB thanks for the PRs. As soon as you're ready, nudge us in here.
E
E
E
erik2 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
https://github.com/cloudposse/atmos/pull/2179
E
erik2 months ago
@RB may need your help to finish this https://github.com/cloudposse/atmos/pull/2174#issuecomment-4052196897
E
erik2 months 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
erik2 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
https://github.com/cloudposse/atmos/pull/2191
R
R
RB2 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
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
RB2 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
https://github.com/cloudposse/atmos/pull/2199/changes
--explainhttps://github.com/cloudposse/atmos/pull/2199/changes
R
R
RB2 months 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
RB2 months 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
erik2 months 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
RB2 months 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
erik2 months ago
R
RB2 months 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
RB2 months 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
RB2 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
Related PRs
• https://github.com/cloudposse/atmos/pull/2226
• https://github.com/cloudposse/atmos/pull/2225
• https://github.com/cloudposse/atmos/pull/2204
E
erik2 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.
This fixes the problem where users try to log into the console with multiple identities and get an error from AWS.
R
RB2 months ago
E
R
RBabout 2 months ago
do we have any tests that check for “speed” so an SLA/SLO can be created for various atmosy things?
E
erikabout 2 months 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
erikabout 2 months 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
erikabout 2 months ago
@Andriy Knysh (Cloud Posse): @Zack confirmed this fix is working: https://github.com/cloudposse/atmos/pull/2253
E
erikabout 2 months ago
@Andriy Knysh (Cloud Posse) https://github.com/cloudposse/atmos/pull/2264 just improved the error messaging from Atmos uploads.
R
RBabout 2 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
fix: clear-text logging of sensitive information (CodeQL #5157) #2163
J
Jonathan Roseabout 2 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
E
erikabout 2 months 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
erikabout 2 months 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