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

Ignoring Succeeded Pods when calculating num_of_not_running pods #1330

Merged
merged 152 commits into from
Jul 25, 2024

Conversation

devdattakulkarni
Copy link
Contributor

No description provided.

Also, modified provider-kubeconfig generator to include https
in the provided API server IP only if it is not already present.
In the original design, we were using the output of kubectl connections
to find out the Pods and then running metrics and logs on those Pods.
However, for workloads in which Pods get created at runtime, connections
is not able to find all the newly created workload Pods. This leads to
incomplete results for metrics and logs. A simple way to handle this is
to use all the pods in the namespace in which the application pods are running
for collecting logs and metrics. The runtime Pods are typically
created in the same namespace in which the application is running.
This will give accurate results (atleast more accurate than using the output
of kubect connections for the list of Pods).

Fixes: #1190
Added a field ('error') to the status object.
This is used by helmer to store any errors that
are encountered when performing helm upgrade.
It provides a UI for managing application instances.
But it does not run on the cluster.
The open-source equivalent is consumerUI, which is
deployed as part KubePlus deployment.
- Added link to Contributing guidelines. These guidelines
  contain pointer to setting up development environment
- Extracted Architecture in a separate section
- Updated link to CNCF Application definition section
- Added Getting Started section

Fixes: #1221
@devdattakulkarni devdattakulkarni merged commit 1e05e7f into master Jul 25, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant