Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feat: Introducing Grafana(7.0.2) and Prometheus(25.4.0) Addon #44

Merged
merged 36 commits into from
Jan 5, 2024
Merged

Feat: Introducing Grafana(7.0.2) and Prometheus(25.4.0) Addon #44

merged 36 commits into from
Jan 5, 2024

Conversation

nileshgadgi
Copy link
Member

@nileshgadgi nileshgadgi commented Nov 1, 2023 •
edited by h1manshu98

what

  • Added helm chart of jaeger,Grafana and prometheus.
  • Updated workflow for dynamic checkout reference.
  • added Jaeger dependent helm chart with jaeger deployment. (by default false)

why

  • Updated workflow to have dynamic solution for branches if action triggered by pull request or anything.
  • Used jaeger dependency if user want to use the jaeger with it's dependency to reduce manual work.

references

@nileshgadgi nileshgadgi added enhancement New feature or request Work In Progress labels Nov 1, 2023
@nileshgadgi nileshgadgi self-assigned this Nov 1, 2023
@nileshgadgi nileshgadgi changed the title Feat: introducing Jaeger and Prometheus addon Feat: Introducing Jaeger(0.71.18) and Prometheus(25.4.0) Addon Nov 2, 2023
@nileshgadgi nileshgadgi changed the title Feat: Introducing Jaeger(0.71.18) and Prometheus(25.4.0) Addon Feat: Introducing Jaeger(0.71.18), Grafana(7.0.2) and Prometheus(25.4.0) Addon Nov 29, 2023
anmolnagpal and others added 14 commits December 13, 2023 01:24
@anmolnagpal
fix- update trigger point for readme workflow
8cbeba9
@anmolnagpal
fix: added output in the root structure to get addons details in example
464813d
@anmolnagpal
fix- terraform code format command ran
673e974
@anmolnagpal
fix: removed jaeger manifest deployment
0901f6a
@anmolnagpal
Merge branch 'master' of github.com:clouddrove/terraform-aws-eks-addo…
e72e2fb 
…ns into feat/prom_jaeger
@anmolnagpal
fix- terraform code format command ran
80c3b94
@anmolnagpal
Merge branch 'master' of github.com:clouddrove/terraform-aws-eks-addo…
5bb32b5 
…ns into feat/prom_jaeger
@anmolnagpal
Merge branch 'feat/prom_jaeger' of github.com:clouddrove/terraform-aw…
d725de1 
…s-eks-addons into feat/prom_jaeger
@anmolnagpal
fix: added jaeger helm config file for jaeger deploy
94c05aa
@anmolnagpal
fix- terraform code format command ran
c893237
@anmolnagpal
feat- added vs for grafana service
7d7f8c6
@anmolnagpal
feat- added grafana virtual service and update prometheus readme
2808067
@anmolnagpal
feat- added grafana virtual service and update prometheus readme
4092d20
@h1manshu98
fix: remove jaeger addon
1b7c688
@h1manshu98 h1manshu98 changed the title Feat: Introducing Jaeger(0.71.18), Grafana(7.0.2) and Prometheus(25.4.0) Addon Feat: Introducing Grafana(7.0.2) and Prometheus(25.4.0) Addon Jan 5, 2024
h1manshu98
h1manshu98 approved these changes Jan 5, 2024
View reviewed changes
Copy link
Member

@h1manshu98 h1manshu98 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@anmolnagpal anmolnagpal enabled auto-merge (squash) January 5, 2024 16:26
@anmolnagpal anmolnagpal enabled auto-merge (squash) January 5, 2024 16:26
@anmolnagpal anmolnagpal merged commit 7ad9b97 into master Jan 5, 2024
14 checks passed
@anmolnagpal anmolnagpal deleted the feat/prom_jaeger branch January 5, 2024 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anmolnagpal anmolnagpal anmolnagpal approved these changes

@h1manshu98 h1manshu98 h1manshu98 approved these changes

@themaniskshah themaniskshah themaniskshah left review comments

@clouddrove-ci clouddrove-ci Awaiting requested review from clouddrove-ci clouddrove-ci is a code owner

Assignees

@nileshgadgi nileshgadgi

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@nileshgadgi @anmolnagpal @themaniskshah @h1manshu98 @clouddrove-ci