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.

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

Add "Telemetry E2E" package #288

Merged
merged 8 commits into from
Oct 14, 2021
Merged

Conversation

ctron
Copy link
Contributor

@ctron ctron commented Sep 17, 2021

This is a PR for the new "Telemetry E2" package. As outlined in #274

The current state is:

Additional tasks:

Changes from original plan:

  • Drop the idea of using Raspberry Pi Zero: Support for armv7 is already challenging, the Zero however is an armv6 platform, which is even less supported in the Java world. Thus is may be too much effort this get this working properly, and switching to a Raspberry Pi 3+ model seems a reasonable compromise.

Follow up tasks:

@ctron ctron added the enhancement New feature or request label Sep 17, 2021
@ctron ctron self-assigned this Sep 17, 2021
@ctron ctron mentioned this pull request Oct 4, 2021
@ctron ctron force-pushed the feature/telemetry_e2e_1 branch 2 times, most recently from 1deedc9 to c66e899 Compare October 5, 2021 16:36
@ctron ctron requested a review from calohmn October 5, 2021 16:39
@ctron
Copy link
Contributor Author

ctron commented Oct 5, 2021

@calohmn I think it would be great if you (as project lead) could give a quick thumbs up (or comment otherwise) on the overall idea of adding this chart.

@ctron ctron force-pushed the feature/telemetry_e2e_1 branch 4 times, most recently from 2de514a to fa30bdc Compare October 6, 2021 09:32
@ctron ctron force-pushed the feature/telemetry_e2e_1 branch 7 times, most recently from ad7b704 to ac87d81 Compare October 6, 2021 12:43
@calohmn
Copy link
Contributor

calohmn commented Oct 7, 2021

@ctron +1 from my side on the overall idea here. Good to have another package showcasing the integration of such a comprehensive stack.

@ctron ctron marked this pull request as ready for review October 8, 2021 16:11
@ctron ctron force-pushed the feature/telemetry_e2e_1 branch from 7ac6a10 to 8e43ba2 Compare October 13, 2021 13:43
@ctron ctron force-pushed the feature/telemetry_e2e_1 branch 2 times, most recently from 8e0c5e0 to d3d0899 Compare October 13, 2021 15:26
Citing the Kind documentation:

> You probably don't need this unless you are testing Kubernetes itself.
@ctron ctron force-pushed the feature/telemetry_e2e_1 branch from d3d0899 to a16f9aa Compare October 13, 2021 15:49
ctron added 3 commits October 13, 2021 17:52
This allows charts to create a customized values
file for the chart testing, using e.g. a Kind ingress
IP.
@ctron ctron force-pushed the feature/telemetry_e2e_1 branch from a16f9aa to 001f9b0 Compare October 13, 2021 15:52
@ctron
Copy link
Contributor Author

ctron commented Oct 13, 2021

I squashed the commits and extracted the homepage parts to another PR.

Unless there are any concerns, I would release a first alpha version of this chart in the next days.

@ctron ctron added the Telemetry E2E Improvements to the Telemetry end-to-end package label Oct 14, 2021
@ctron ctron merged commit 5e4700b into eclipse:master Oct 14, 2021
@ctron ctron deleted the feature/telemetry_e2e_1 branch October 14, 2021 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Telemetry E2E Improvements to the Telemetry end-to-end package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants