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

Add scenario for wrong selectorLabels in services (no endpoints) #1

Closed
thschue opened this issue Mar 22, 2023 · 0 comments · Fixed by #74
Closed

Add scenario for wrong selectorLabels in services (no endpoints) #1

thschue opened this issue Mar 22, 2023 · 0 comments · Fixed by #74
Assignees
@thschue
Labels
enhancement New feature or request

Comments

@thschue
Copy link
Contributor

thschue commented Mar 22, 2023 •
edited

Description

When deploying services, one of the things which are faced very often is that endpoints are not available, caused by pods that are not able to start or incorrect labels on the service object.

Possible solution

  • Find services without endpoints
  • If endpoint is missing, try to find out if pods with matching labels exist
  • Look for problems on the pod
  • If no pod matches labels, try to find pod with similar name
  • Respond with problem solution
@thschue thschue added the enhancement New feature or request label Mar 22, 2023
@thschue thschue mentioned this issue Mar 25, 2023
Merged
4 tasks
@thschue thschue self-assigned this Mar 25, 2023
thschue added a commit that referenced this issue Apr 27, 2023
@TylerGillson @thschue
feat: add helm chart (#318)
5af8178 
* feat: add helm chart

Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>

* Address PR feedback (#1)

* remove K8SGPT_BASEURL

* add default model

* add user-specified Deployment annotations

* remove Values.secret.enabled

---------

Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>

* update chart.appVersion, default labels

Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>

* default image tag to Chart.appVersion

Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>

* release-please annotation for k8sgpt image

Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>

* revert improperly relocated Dockerfile

Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>

* revert changes to docker-build target and actually re-locate Dockerfile

Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>

* Update charts/k8sgpt/Chart.yaml

Co-authored-by: Thomas Schuetz <38893055+thschue@users.noreply.github.com>
Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>

* Update charts/k8sgpt/templates/deployment.yaml

Co-authored-by: Thomas Schuetz <38893055+thschue@users.noreply.github.com>
Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>

---------

Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
Co-authored-by: Thomas Schuetz <38893055+thschue@users.noreply.github.com>
@jkleinlercher jkleinlercher mentioned this issue May 20, 2023
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thschue thschue

Labels
enhancement New feature or request
Projects
Backlog
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: check some service scenarios k8sgpt-ai/k8sgpt
1 participant
@thschue