console icon indicating copy to clipboard operation
console copied to clipboard

WIP: CONSOLE-3824: Remove kubevirt-plugin code

Open pcbailey opened this issue 10 months ago • 5 comments

This PR does the following:

  • Removes the kubevirt-plugin package and references in configuration files
  • Copies code upon which the topology plugin is dependent from the kubevirt-plugin package to the topology package
  • Changed the i18n prefix in the network-attachment-definition-plugin package from kubevirt-plugin to network-attachment-definition-plugin

Jira: https://issues.redhat.com/browse/CONSOLE-3824

pcbailey avatar Apr 17 '24 12:04 pcbailey

@pcbailey: This pull request references CONSOLE-3824 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.16.0" version, but no target version was set.

In response to this:

This PR does the following:

  • Removes the kubevirt-plugin package and references in configuration files
  • Copies code upon which the topology plugin is dependent from the kubevirt-plugin package to the topology package
  • Changed the i18n prefix in the network-attachment-definition-plugin package from kubevirt-plugin to network-attachment-definition-plugin

Jira: https://issues.redhat.com/browse/CONSOLE-3824

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-robot avatar Apr 17 '24 12:04 openshift-ci-robot

This PR will also solve: https://issues.redhat.com//browse/CNV-37607

avivtur avatar Apr 18 '24 15:04 avivtur

/retest

pcbailey avatar May 20 '24 19:05 pcbailey

/retest

pcbailey avatar May 21 '24 13:05 pcbailey

Excellent work @pcbailey :1st_place_medal: /lgtm

avivtur avatar May 22 '24 15:05 avivtur

QE Approver: /assign @yapei Docs Approver: /assign @opayne1 PX Approver: /assign @RickJWagner

jhadvig avatar May 28 '24 11:05 jhadvig

/label px-approved

RickJWagner avatar May 28 '24 18:05 RickJWagner

  1. launched a cluster, subscribed OpenShift Virtualization operator successfully
  2. Verify Virtualization menu exists
  3. Verify if secondary menu Network Attachment Definitions exist in Networking menu -> No Network Attachment Definitions menu, see screen recording

https://github.com/openshift/console/assets/12692381/0c1af56c-3e3b-4af8-808a-d5ed7f6618e9

yapei avatar May 29 '24 07:05 yapei

also due to https://issues.redhat.com/browse/OCPBUGS-22853 I can't verify if VM topology codes are working as expected or not(these codes moved away from frontend/packages/kubevirt-plugin/src/topology to frontend/packages/topology ) since VirtualMachines page is not accessible

yapei avatar May 29 '24 08:05 yapei

  1. launched a cluster, subscribed OpenShift Virtualization operator successfully
  2. Verify Virtualization menu exists
  3. Verify if secondary menu Network Attachment Definitions exist in Networking menu -> No Network Attachment Definitions menu, see screen recording

I fixed this issue. It was looking for the KUBEVIRT flag, so I updated it to look for the KUBEVIRT_DYNAMIC flag.

pcbailey avatar May 31 '24 19:05 pcbailey

also due to https://issues.redhat.com/browse/OCPBUGS-22853 I can't verify if VM topology codes are working as expected or not(these codes moved away from frontend/packages/kubevirt-plugin/src/topology to frontend/packages/topology ) since VirtualMachines page is not accessible

I'm not sure I understand what you mean. That bug pertains to OCP 4.14/CNV 4.13. I also confirmed that the root cause of that bug is no longer present in our code from 4.14 onward. I was able to test the Topology page and Virtual Machines pages. Could you please clarify the issue you mention here? Thanks!

pcbailey avatar May 31 '24 19:05 pcbailey

/retest

pcbailey avatar May 31 '24 21:05 pcbailey

/retest

pcbailey avatar Jun 01 '24 14:06 pcbailey

/retest

pcbailey avatar Jun 03 '24 13:06 pcbailey

/lgtm

avivtur avatar Jun 03 '24 14:06 avivtur

/retest

pcbailey avatar Jun 03 '24 19:06 pcbailey

/retest

pcbailey avatar Jun 04 '24 02:06 pcbailey

/retest

pcbailey avatar Jun 04 '24 11:06 pcbailey

/retest

pcbailey avatar Jun 05 '24 02:06 pcbailey

I still can't visit VirtualMachines page and other pages(OpenShift Virtualization 4.14.0 installed), here is the recording

https://github.com/openshift/console/assets/12692381/8088a280-81bf-46fd-8be6-22e935d30231

yapei avatar Jun 05 '24 07:06 yapei

I still can't visit VirtualMachines page and other pages(OpenShift Virtualization 4.14.0 installed), here is the recording

Ah, I understand now. This PR should be tested on the latest versions of OCP and OpenShift Virtualization.

pcbailey avatar Jun 05 '24 09:06 pcbailey

generally the changes look good from QE's point of view /label qe-approved

yapei avatar Jun 12 '24 02:06 yapei

@pcbailey: This pull request references CONSOLE-3824 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.17.0" version, but no target version was set.

In response to this:

This PR does the following:

  • Removes the kubevirt-plugin package and references in configuration files
  • Copies code upon which the topology plugin is dependent from the kubevirt-plugin package to the topology package
  • Changed the i18n prefix in the network-attachment-definition-plugin package from kubevirt-plugin to network-attachment-definition-plugin

Jira: https://issues.redhat.com/browse/CONSOLE-3824

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-robot avatar Jun 12 '24 02:06 openshift-ci-robot

/retest

pcbailey avatar Jun 17 '24 13:06 pcbailey

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: avivtur, jhadvig, pcbailey

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[bot] avatar Jun 24 '24 09:06 openshift-ci[bot]