Skip to content

Telemetry - atmos describe cmd#1342

Merged
goruha merged 9 commits intoDEV-2965/consolidate-cmd-telemetryfrom
DEV-2965/telemetry-describe
Jul 2, 2025
Merged

Telemetry - atmos describe cmd#1342
goruha merged 9 commits intoDEV-2965/consolidate-cmd-telemetryfrom
DEV-2965/telemetry-describe

Conversation

@goruha
Copy link
Member

@goruha goruha commented Jun 24, 2025

what

  • Added telemetry to atmos describe command

why

  • Collect info about usage atmos commands

References

@goruha goruha requested a review from a team as a code owner June 24, 2025 18:41
@goruha goruha added the no-release Do not create a new release (wait for additional code changes) label Jun 24, 2025
@mergify mergify bot added the stacked Stacked label Jun 24, 2025
@codecov
Copy link

codecov bot commented Jun 27, 2025

Codecov Report

Attention: Patch coverage is 60.86957% with 18 lines in your changes missing coverage. Please review.

Please upload report for BASE (DEV-2965/consolidate-cmd-telemetry@86aa42e). Learn more about missing BASE report.

Files with missing lines Patch % Lines
cmd/describe_config.go 0.00% 6 Missing ⚠️
cmd/describe_component.go 66.66% 5 Missing ⚠️
cmd/describe_dependents.go 0.00% 3 Missing ⚠️
cmd/describe_affected.go 66.66% 2 Missing ⚠️
cmd/describe_stacks.go 80.00% 2 Missing ⚠️
Additional details and impacted files
@@                          Coverage Diff                          @@
##             DEV-2965/consolidate-cmd-telemetry    #1342   +/-   ##
=====================================================================
  Coverage                                      ?   51.04%           
=====================================================================
  Files                                         ?      243           
  Lines                                         ?    26109           
  Branches                                      ?        0           
=====================================================================
  Hits                                          ?    13328           
  Misses                                        ?    11132           
  Partials                                      ?     1649           
Flag Coverage Δ
unittests 51.04% <60.86%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@goruha
Copy link
Member Author

goruha commented Jun 30, 2025

@Mergifyio rebase

@mergify
Copy link

mergify bot commented Jun 30, 2025

rebase

✅ Branch has been successfully rebased

@goruha goruha force-pushed the DEV-2965/telemetry-describe branch from 23d4c31 to 770948c Compare June 30, 2025 17:26
@goruha goruha changed the base branch from DEV-2965/implement-atmos-telemetry to DEV-2965/consolidate-cmd-telemetry July 1, 2025 19:15
@mergify
Copy link

mergify bot commented Jul 1, 2025

💥 This pull request now has conflicts. Could you fix it @goruha? 🙏

@mergify mergify bot added the conflict This PR has conflicts label Jul 1, 2025
@mergify mergify bot removed the conflict This PR has conflicts label Jul 2, 2025
@goruha goruha merged commit b26a304 into DEV-2965/consolidate-cmd-telemetry Jul 2, 2025
9 checks passed
@goruha goruha deleted the DEV-2965/telemetry-describe branch July 2, 2025 21:34
goruha added a commit that referenced this pull request Jul 3, 2025
* Consolidate cmd telemetry

* [autofix.ci] apply automated fixes

* Update root.go

* Capture commands

* Telemetry - `atmos describe` cmd (#1342)

* Added telemetry to describe commands

* Update TestCLICommands_atmos_describe_config_imports.stderr.golden

* Update TestCLICommands_atmos_describe_configuration.stderr.golden

* Fix tests

* Update describe_component.go

* Update describe_config.go

* Update describe_stacks.go

* Update describe_workflows.go

* Fix tests

* Fix tests

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
goruha added a commit that referenced this pull request Jul 4, 2025
* Added initial implementation of telemetry

* [autofix.ci] apply automated fixes

* Fix lint

* Fix lint

* Fix lint

* Fix lint

* Fix lint

* Fix lint

* Fix test

* Fix acceptance tests

* Reduce tests verbosity

* Reduce tests verbosity

* Fix tests

* [autofix.ci] apply automated fixes

* Addded tests

* Added tests for utils

* [autofix.ci] apply automated fixes

* Address comments

* [autofix.ci] apply automated fixes

* Fix tests

* [autofix.ci] apply automated fixes

* DRY

* Split full PR to small PRs

* Split full PR to small PRs

* Split full PR to small PRs

* Fix tests

* Added support CI, docker and Atmos pro

* Reduce public package methods

* [autofix.ci] apply automated fixes

* Added interface to be generated by mock

* [autofix.ci] apply automated fixes

* Address coderabbit comments

* Address coderabbit comments

* [autofix.ci] apply automated fixes

* Fix tests

* Remove telemetry timeout schema

* Remove telemetry timeout schema

* Fix tests

* Fix cache dependent tests

* [autofix.ci] apply automated fixes

* Fix cache dependent tests

* Fix read-only dir testcase

* Fix read-only dir testcase

* Fix tests

* Fix tests

* Fix tests bypassing cache on error

* Fix tests bypassing cache on error

* Fix tests bypassing cache on error

* Added comment

* Added comment

* Added comment

* Address comments

* Address comments

* Address comments

* refactor telemetry provider to use pointer config (#1329)

* Added comments

* Revert disclouser message

* Added disclosure messages

* Added disclosure messages

* Fix accptance tests

* Delete test.log

* Move disclosure message to separate PR

* Move disclosure message to separate PR

* Move disclosure message to separate PR

* Move disclosure message to separate PR

* Regenerate README

* Renew token

* [autofix.ci] apply automated fixes

* Address comments

* Add telemetry to commands (#1360)

* Consolidate cmd telemetry

* [autofix.ci] apply automated fixes

* Update root.go

* Capture commands

* Telemetry - `atmos describe` cmd (#1342)

* Added telemetry to describe commands

* Update TestCLICommands_atmos_describe_config_imports.stderr.golden

* Update TestCLICommands_atmos_describe_configuration.stderr.golden

* Fix tests

* Update describe_component.go

* Update describe_config.go

* Update describe_stacks.go

* Update describe_workflows.go

* Fix tests

* Fix tests

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

* Atmos site Telemetry page (#1335)

* Added telemetry site page

* Update website/docs/cli/telemetry.mdx

Co-authored-by: Erik Osterman (CEO @ Cloud Posse) <erik@cloudposse.com>

* Update website/docs/cli/telemetry.mdx

Co-authored-by: Erik Osterman (CEO @ Cloud Posse) <erik@cloudposse.com>

* Update website/docs/cli/telemetry.mdx

Co-authored-by: Erik Osterman (CEO @ Cloud Posse) <erik@cloudposse.com>

* Update website/docs/cli/telemetry.mdx

Co-authored-by: Erik Osterman (CEO @ Cloud Posse) <erik@cloudposse.com>

* Apply suggestions from code review

Co-authored-by: Erik Osterman (CEO @ Cloud Posse) <erik@cloudposse.com>

---------

Co-authored-by: Erik Osterman (CEO @ Cloud Posse) <erik@cloudposse.com>

* Added  telemetry disclosure messages (#1332)

* Added disclosure messages

* Added disclosure messages

* Cherry pick disclosure message

* Fix tests

* Fix tests

* Fix tests

* Improve disclosure messages

* Fix tests

* Fix tests

* [autofix.ci] apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Erik Osterman (CEO @ Cloud Posse) <erik@cloudposse.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-release Do not create a new release (wait for additional code changes) stacked Stacked

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant