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

k8s-monitoring 2.0 alpha #757

Merged
merged 42 commits into from
Oct 3, 2024
Merged

k8s-monitoring 2.0 alpha #757

merged 42 commits into from
Oct 3, 2024

Conversation

petewall
Copy link
Collaborator

@petewall petewall commented Sep 30, 2024 •
edited
Loading

Add k8s-monitoring Helm chart 2.0 to this repository. This will not start releasing the chart, yet. It's making it available for visibility and feedback.

Steps:

  1. Add the feature charts.
    1. Add source
    2. Make GitHub actions to test changed feature charts.
    3. Make Code owners sections to give feature developers access to write to their configurations
  2. Add the main chart.
    1. Add source
    2. Make GitHub actions to test main chart

@petewall petewall marked this pull request as draft September 30, 2024 20:10
@petewall petewall self-assigned this Sep 30, 2024
@petewall petewall force-pushed the chore/prep-for-v2 branch 11 times, most recently from 85a628d to a739f35 Compare October 3, 2024 02:14
@petewall petewall marked this pull request as ready for review October 3, 2024 14:48
@petewall
Add cluster events feature chart and a new github action to test feat…
618b5c4 
…ure charts

Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Update test action
c3aa695 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Add a gitkeep to make the snapshot directory exist
ada7950 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Add annotation feature chart
f2c55ca 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Add cluster metrics feature
0d23025 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Fix yaml lint
8f9ce23 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Update linters and top-level makefile
0099f17 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
More lint fixes
554a0c9 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Re-enable linting for test yamls
d286900 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Make markdownlint happier
744aa6a 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Adding pod logs, profiling, and prom operator object feature charts
0ba4c08 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
remove -it from docker run
f07049f 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Update checkout action
4fda5c4 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Simplify diff check
2667b85 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Add app o11y and frontend o11y feature charts
4884450 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Move codeowners into .github and break things down by directory
3c75fa9 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
ct lint in every chart
a32484d 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
legacy test should only work on the the v1 chart
4d75415 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Add ct to the github action
53d9420 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
add ct configs for charts that use dependenties
d31b8cd 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Include chart v2 in the chart test workflow
5d29033 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Add clean target
9329421 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Add file to make helm unittest work
c614935 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Add a test chart and use that to parallelize integration tests
e32f0c4 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
inline the helm repo
b50278e 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Update generated files
3794691 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Need alloy for some unittests
9ea8c1c 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Fix v1 test and output helm commands in new integration tests
3e1c530 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Fix integration test script and don't check chart bundles for modific…
0f4ea3d 
…ations for now

Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Move function up
211a839 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Use the pre-existing cluster
34c1e42 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
More prerequisites before deploying prometheus
3dd00cc 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Update integration test scripts to work this time
1f7565b 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
further work on the test chart
569339e 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Gotta get that chart dir
17bba77 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall
Remove ec lint, which we don't really need and also fix some lint issues
00a9d0a 
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall petewall requested review from a team, skl, nevermind89x and jewbetcha and removed request for a team October 3, 2024 15:19
jewbetcha
jewbetcha approved these changes Oct 3, 2024
View reviewed changes
Copy link
Collaborator

@jewbetcha jewbetcha left a comment

Choose a reason for hiding this comment

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

🎉

skl
skl approved these changes Oct 3, 2024
View reviewed changes
Copy link
Collaborator

@skl skl left a comment

Choose a reason for hiding this comment

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

🚢

@petewall petewall merged commit 3b4ffa4 into main Oct 3, 2024
41 checks passed
@petewall petewall deleted the chore/prep-for-v2 branch October 3, 2024 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jewbetcha jewbetcha jewbetcha approved these changes

@skl skl skl approved these changes

@nevermind89x nevermind89x Awaiting requested review from nevermind89x nevermind89x is a code owner automatically assigned from grafana/k8s-monitoring-dev

Assignees

@petewall petewall

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@petewall @skl @jewbetcha