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

[RHOAIENG-5033] Odd number of arguments passed as key-value pairs for… #331

Merged
merged 1 commit into from
May 23, 2024

Conversation

jstourac
Copy link
Member

@jstourac jstourac commented May 23, 2024

… logging

Fix the dpanic for log in notebook controller while injecting the trusted CA bundle configuration.

https://issues.redhat.com/browse/RHOAIENG-5033

Description

How Has This Been Tested?

Following steps:

  1. Install latest ODH-nightly build on your cluster
  2. Create the test DS project
  3. Scale down the opendatahub-operator-controller-manager Deployment in openshift-operators namespace to 0
  4. Update the image referenced by odh-notebook-controller-manager Deployment to quay.io/opendatahub/odh-notebook-controller:pr-331
  5. Create a nb-test workbench
  6. See the odh-notebook-controller-manager pod log
{"level":"info","ts":"2024-05-23T15:51:50Z","logger":"controllers.Notebook","msg":"workbench-trusted-ca-bundle ConfigMap is not present, start creating it...","notebook":"nb-test","namespace":"test"}
{"level":"info","ts":"2024-05-23T15:51:50Z","logger":"controllers.Notebook","msg":"Created workbench-trusted-ca-bundle ConfigMap","notebook":"nb-test","namespace":"test"}
{"level":"info","ts":"2024-05-23T15:51:50Z","logger":"controllers.Notebook","msg":"Injecting trusted-ca volume and environment variables","notebook":"nb-test","namespace":"test"}
{"level":"info","ts":"2024-05-23T15:51:50Z","logger":"controllers.Notebook","msg":"Internal registry found. Will pickup the default value from image field.","notebook":"nb-test","namespace":"test"}
{"level":"info","ts":"2024-05-23T15:51:50Z","logger":"controllers.Notebook","msg":"Updating workbench-trusted-ca-bundle ConfigMap","namespace":"test","notebook":"nb-test"}
{"level":"info","ts":"2024-05-23T15:51:50Z","logger":"controllers.Notebook","msg":"Creating Network Policy","name":"nb-test-ctrl-np"}
{"level":"info","ts":"2024-05-23T15:51:51Z","logger":"controllers.Notebook","msg":"Creating Network Policy","name":"nb-test-oauth-np"}
{"level":"info","ts":"2024-05-23T15:51:51Z","logger":"controllers.Notebook","msg":"Creating Service Account","notebook":"nb-test","namespace":"test"}
{"level":"info","ts":"2024-05-23T15:51:51Z","logger":"controllers.Notebook","msg":"Creating OAuth Service","notebook":"nb-test","namespace":"test"}
{"level":"info","ts":"2024-05-23T15:51:51Z","logger":"controllers.Notebook","msg":"Creating OAuth Secret","notebook":"nb-test","namespace":"test"}
{"level":"info","ts":"2024-05-23T15:51:51Z","logger":"controllers.Notebook","msg":"Creating Route","notebook":"nb-test","namespace":"test"}
{"level":"info","ts":"2024-05-23T15:51:51Z","logger":"controllers.Notebook","msg":"Removing reconciliation lock","notebook":"nb-test","namespace":"test"}

Merge criteria:

  • The commits are squashed in a cohesive manner and have meaningful messages.
  • Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
  • The developer has manually tested the changes and verified that the changes work

Copy link

openshift-ci bot commented May 23, 2024

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@jstourac jstourac self-assigned this May 23, 2024
… logging

Fix the dpanic for log in notebook controller while injecting the
trusted CA bundle configuration.

https://issues.redhat.com/browse/RHOAIENG-5033
@jstourac jstourac marked this pull request as ready for review May 23, 2024 15:01
@openshift-ci openshift-ci bot requested review from harshad16 and VaishnaviHire May 23, 2024 15:01
@jstourac jstourac requested review from atheo89 and removed request for VaishnaviHire May 23, 2024 16:10
@harshad16
Copy link
Member

/lgtm
/approve
/ok-to-test

thank you for the work 💯

@harshad16 harshad16 merged commit 417ac03 into opendatahub-io:v1.7-branch May 23, 2024
6 of 7 checks passed
Copy link

openshift-ci bot commented May 23, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: harshad16

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

@harshad16
Copy link
Member

/cherrypick stable

@openshift-cherrypick-robot

@harshad16: new pull request created: #334

In response to this:

/cherrypick stable

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants