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

Update Operator bundle for release 1.6 #228

Merged

Conversation

VaishnaviHire
Copy link
Member

@VaishnaviHire VaishnaviHire commented May 1, 2023

Description

How Has This Been Tested?

  1. Run Test Bundle:
 operator-sdk run bundle quay.io/vhire/opendatahub-operator-bundle:v1.6.0 --namespace openshift-operators
  1. Create KfDef using the given alm example

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

@VaishnaviHire VaishnaviHire requested a review from LaVLaS May 1, 2023 18:14
@openshift-ci openshift-ci bot requested a review from anishasthana May 1, 2023 18:14
@VaishnaviHire VaishnaviHire changed the title Update Operator bundle for release 1.6 [WIP] Update Operator bundle for release 1.6 May 1, 2023
@VaishnaviHire
Copy link
Member Author

Note : Operator e2e tests will fail unless following PR is merged
openshift/release#38899

@VaishnaviHire VaishnaviHire changed the title [WIP] Update Operator bundle for release 1.6 Update Operator bundle for release 1.6 May 1, 2023
@LaVLaS
Copy link
Contributor

LaVLaS commented May 3, 2023

We need to update the CSV description to remove the references to the stable channel. We included instructions in the 1.5 release on how to convert from stable --> rolling and drop stable from the available channels

@VaishnaviHire
Copy link
Member Author

@LaVLaS Do you think we need to update description to include addition of distributed-workloads component?

@VaishnaviHire
Copy link
Member Author

We need to update the CSV description to remove the references to the stable channel. We included instructions in the 1.5 release on how to convert from stable --> rolling and drop stable from the available channels

Updated

@VaishnaviHire
Copy link
Member Author

/retest

1 similar comment
@VaishnaviHire
Copy link
Member Author

/retest

@VaishnaviHire
Copy link
Member Author

/retest-required

@VaishnaviHire
Copy link
Member Author

Updated the testing steps with test operator-bundle .

 oc get po -n opendatahub
NAME                                                              READY   STATUS    RESTARTS   AGE
data-science-pipelines-operator-controller-manager-576bbbb5hfxm   1/1     Running   0          25m
etcd-85c59bc4d6-4b5zr                                             1/1     Running   0          25m
grafana-deployment-6cf577dbb6-f6qzz                               1/1     Running   0          24m
grafana-operator-controller-manager-54fbd5b876-4dgg9              2/2     Running   0          25m
jupyter-nb-kube-3aadmin-0                                         2/2     Running   0          11m
modelmesh-controller-5588b58d79-4tkgj                             1/1     Running   0          25m
modelmesh-controller-5588b58d79-jklg7                             1/1     Running   0          25m
modelmesh-controller-5588b58d79-rzt4d                             1/1     Running   0          25m
notebook-controller-deployment-5c565c4c75-gm75k                   1/1     Running   0          25m
odh-dashboard-559f9899f5-tl6qv                                    2/2     Running   0          25m
odh-dashboard-559f9899f5-zjgwr                                    2/2     Running   0          25m
odh-model-controller-79c67bc689-6vvb8                             1/1     Running   0          25m
odh-model-controller-79c67bc689-d49sm                             1/1     Running   0          25m
odh-model-controller-79c67bc689-pw8ph                             1/1     Running   0          25m
odh-notebook-controller-manager-5cf77fdc56-rws2r                  1/1     Running   0          25m
prometheus-odh-model-monitoring-0                                 3/3     Running   0          25m
prometheus-odh-model-monitoring-1                                 3/3     Running   0          25m
prometheus-odh-model-monitoring-2                                 3/3     Running   0          25m
prometheus-odh-monitoring-0                                       2/2     Running   0          25m
prometheus-odh-monitoring-1                                       2/2     Running   0          25m
prometheus-operator-779f765944-6pxd8                              1/1     Running   0          25m

@VaishnaviHire
Copy link
Member Author

@LaVLaS Updated alm example and test bundle to include trustyai-service

@openshift-ci
Copy link

openshift-ci bot commented May 16, 2023

@VaishnaviHire: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/opendatahub-manifests-e2e 104ce52 link false /test opendatahub-manifests-e2e

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/test-infra repository. I understand the commands that are listed here.

Copy link
Contributor

@LaVLaS LaVLaS left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci
Copy link

openshift-ci bot commented May 17, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: LaVLaS

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-merge-robot openshift-merge-robot merged commit 274ffc2 into opendatahub-io:master May 17, 2023
VaishnaviHire added a commit to VaishnaviHire/opendatahub-operator that referenced this pull request Jun 21, 2023
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