Skip to content

add resource request and limit for server pods #2462

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

Merged
merged 1 commit into from
Jul 19, 2021
Merged

add resource request and limit for server pods #2462

merged 1 commit into from
Jul 19, 2021

Conversation

xiancao
Copy link
Member

@xiancao xiancao commented Jul 16, 2021

In the nightly parallel runs, ItParameterizedDomain failed intermittently. In the event log, some managed servers got killed. In the kublet.log, it shows:

Jul 15 09:00:15 kind-worker kubelet[312]: W0715 09:00:15.832773 312 empty_dir.go:421] Warning: Failed to clear quota on /var/lib/kubelet/pods/cc48f103-2b9b-49f0-a9a5-b7c11489d9dd/volumes/kubernetes.io~configmap/mii-weblogic-domain-introspect-cm: ClearQuota called, but quotas disabled
Jul 15 09:00:39 kind-worker kubelet[312]: E0715 09:00:39.973754 312 kuberuntime_container.go:476] preStop hook for container "weblogic-server" failed: command '/weblogic-operator/scripts/stopServer.sh' exited with 137:

Adding the resource request and limit for the server pods to prevent the pods get evicted by the k8s cluster.

Running the Jenkins job now.
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/5714/
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/5715/
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/5716/

Copy link
Member

@anpanigr anpanigr left a comment

Choose a reason for hiding this comment

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

LGTM

@rjeberhard rjeberhard merged commit 9462ab5 into main Jul 19, 2021
@xiancao xiancao deleted the xc-90899 branch August 10, 2021 21:43
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.

3 participants