hyperdx icon indicating copy to clipboard operation
hyperdx copied to clipboard

Add Hyperdx helm chart

Open anjiann opened this issue 11 months ago • 5 comments

anjiann avatar Feb 26 '24 04:02 anjiann

⚠️ No Changeset found

Latest commit: ae6c32c678c5be5207f6f43fa3e3e1a4bfa62d73

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar Feb 26 '24 04:02 changeset-bot[bot]

:warning: This PR is too large for Ellipsis to review, but support for larger PRs is coming soon.


Generated with :heart: by ellipsis.dev

ellipsis-dev[bot] avatar Feb 26 '24 20:02 ellipsis-dev[bot]

Sorry, Ellipsis encountered a problem while reviewing this PR at commit 3390dbb0f5bdb11bd8b84f5bd11af83f19e1ae4e. Our team has been alerted and is investigating. (wflow_pQUP06Uo9OdFHFyX) :robot:

ellipsis-dev[bot] avatar Feb 26 '24 20:02 ellipsis-dev[bot]

@anjiann It seems like it hasn't merged for a long time due to something else. you separate the helm chart part and provide this like the https://github.com/pintoinc/hyperdx-helm repository. make a hyperdx-helm? I need a helm chart!

momenthana avatar Apr 24 '24 05:04 momenthana

So, about this. What about this?

lazyoldbear avatar Jun 06 '24 22:06 lazyoldbear

what happened to this PR? If you need someone else to pick it and do a rebase, I happy to help

ttj4 avatar Aug 20 '24 10:08 ttj4

Hey everyone, sorry we've been quiet on this PR. We've made a number of attempts at getting this PR reviewed internally and I've personally been hopeful we could get this PR merged each time. Unfortunately, the PR length/complexity makes it difficult for us to review - and additionally it gives us a lot of concern in our ability to support/maintain it properly, especially as we will not be using this Helm chart ourselves in regular development/testing/deployment.

We're more than happy to assist in a community-maintained version, something like what @momenthana has put together (I assume based off of this PR, which we're really grateful for you putting together @anjiann) at https://github.com/pintoinc/hyperdx-helm.

Apologies for not being able to get this officially supported and taking a while for us to hit this conclusion. In the future, we may be able to figure out a way to incorporate maintaining a Helm chart or an "official community-maintained" repo of resources that can include this chart as well. For now, I'm going to close out this PR, we'll likely start pointing people towards that existing repo, and continue any other relevant discussion at #74.

MikeShi42 avatar Aug 24 '24 04:08 MikeShi42