Skip to content

Conversation

lance5890
Copy link
Contributor

@lance5890 lance5890 commented Sep 12, 2025

As we use kubectl logs -n openshift-operator-lifecycle-manager package-server-manager-5f47cc476f-vlwhw , we see the default container logs is kube-rbac-proxy, like :

kubectl logs -n openshift-operator-lifecycle-manager package-server-manager-5f47cc476f-vlwhw 
Defaulted container "kube-rbac-proxy" out of: kube-rbac-proxy, package-server-manager
W0707 04:28:36.737827       1 deprecated.go:66] 
==== Removed Flag Warning ======================

logtostderr is removed in the k8s upstream and has no effect any more.

===============================================

I0707 04:28:36.738391       1 kube-rbac-proxy.go:233] Valid token audiences: 
I0707 04:28:36.738440       1 kube-rbac-proxy.go:347] Reading certificate files
I0707 04:28:36.738810       1 kube-rbac-proxy.go:395] Starting TCP socket on 0.0.0.0:8443
I0707 04:28:36.739099       1 kube-rbac-proxy.go:402] Listening securely on 0.0.0.0:8443

However, most of the time we just care about the container package-server-manager logs , not the kube-rbac-proxy

@openshift-ci openshift-ci bot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Sep 12, 2025
Copy link
Contributor

openshift-ci bot commented Sep 12, 2025

Hi @lance5890. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@lance5890
Copy link
Contributor Author

/cc @perdasilva

@openshift-ci openshift-ci bot requested a review from perdasilva September 12, 2025 01:37
@jianzhangbjz
Copy link
Contributor

/ok-to-test

@openshift-ci openshift-ci bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Sep 12, 2025
@perdasilva
Copy link
Contributor

/retest

Copy link
Contributor

openshift-ci bot commented Sep 12, 2025

@lance5890: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@perdasilva
Copy link
Contributor

/approve

@perdasilva
Copy link
Contributor

/retitle NO-ISSUE: set package-server-manager as default container

@openshift-ci openshift-ci bot changed the title set package-server-manager as default container NO-ISSUE: set package-server-manager as default container Sep 12, 2025
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Sep 12, 2025
@openshift-ci-robot
Copy link

@lance5890: This pull request explicitly references no jira issue.

In response to this:

As we use kubectl logs -n openshift-operator-lifecycle-manager package-server-manager-5f47cc476f-vlwhw , we see the default container logs is kube-rbac-proxy, like :

kubectl logs -n openshift-operator-lifecycle-manager package-server-manager-5f47cc476f-vlwhw 
Defaulted container "kube-rbac-proxy" out of: kube-rbac-proxy, package-server-manager
W0707 04:28:36.737827       1 deprecated.go:66] 
==== Removed Flag Warning ======================

logtostderr is removed in the k8s upstream and has no effect any more.

===============================================

I0707 04:28:36.738391       1 kube-rbac-proxy.go:233] Valid token audiences: 
I0707 04:28:36.738440       1 kube-rbac-proxy.go:347] Reading certificate files
I0707 04:28:36.738810       1 kube-rbac-proxy.go:395] Starting TCP socket on 0.0.0.0:8443
I0707 04:28:36.739099       1 kube-rbac-proxy.go:402] Listening securely on 0.0.0.0:8443

However, most of the time we just care about the container package-server-manager logs , not the kube-rbac-proxy

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 12, 2025
Copy link
Contributor

openshift-ci bot commented Sep 12, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lance5890, perdasilva

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 12, 2025
Copy link
Contributor

openshift-ci bot commented Sep 12, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lance5890, perdasilva

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. ok-to-test Indicates a non-member PR verified by an org member that is safe to test.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants