console icon indicating copy to clipboard operation
console copied to clipboard

CONSOLE-3077: Promote ConsolePlugins API version to v1 in console repository

Open jhadvig opened this issue 3 years ago • 10 comments

Story: https://issues.redhat.com/browse/CONSOLE-3077

Updating ConsolePlugin manifest, model and type in console and also Readme docs. Link to the v1 api for ConsolePlugin CRD - https://github.com/openshift/api/blob/master/console/v1/types_console_plugin.go

/assign @TheRealJon

jhadvig avatar Sep 27 '22 10:09 jhadvig

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

jhadvig avatar Sep 30 '22 13:09 jhadvig

/label px-approved

RickJWagner avatar Sep 30 '22 21:09 RickJWagner

@opayne1 thanks for review. Comment addressed :)

jhadvig avatar Oct 05 '22 11:10 jhadvig

/label docs-approved

opayne1 avatar Oct 05 '22 11:10 opayne1

/lgtm

TheRealJon avatar Oct 05 '22 13:10 TheRealJon

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jhadvig, TheRealJon

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 Oct 05 '22 14:10 openshift-ci[bot]

/retest

jhadvig avatar Oct 07 '22 21:10 jhadvig

@jhadvig it looks like we have an issue here, user has no way to set loadType to Lazy

  1. First I create a ConsolePlugin with following YAML(no i18n defined)
apiVersion: console.openshift.io/v1alpha1
kind: ConsolePlugin
metadata:
  name: console-customization
spec:
  displayName: 'OpenShift Console Customization Plugin'
  service:
    name: console-customization-plugin
    namespace: console-customization-plugin
    port: 9443
    basePath: '/'
  1. Check consoleplugin metadata generated, i18n.loadType was set to empty ""
$ oc get consoleplugin console-customization -o json | jq .spec.i18n.loadType
""
  1. then I tried to update i18n.loadType to Lazy with command , it shows patch completed
$ oc patch consoleplugin console-customization -p '{"spec":{"i18n":{"loadType": "Lazy"}}}' --type merge
consoleplugin.console.openshift.io/console-customization patched
  1. finally when I check consoleplugin metadata, i18n.loadType is still empty "" while it is expected to be Lazy since I updated it in step 3
$ oc get consoleplugin console-customization -o json | jq .spec.i18n.loadType
""

yapei avatar Oct 11 '22 08:10 yapei

@yapei issue should be fixed by https://github.com/openshift/console-operator/pull/684

jhadvig avatar Oct 18 '22 13:10 jhadvig

above reported issue has been fixed in openshift/console-operator#684, no other issues found /label qe-approved

yapei avatar Oct 19 '22 08:10 yapei

/retest-required

Remaining retests: 0 against base HEAD 738cb1a2a87b4b97f875517799cba8e91427d240 and 2 for PR HEAD f7167de6bc6c2adef7f32832786b3207436ab102 in total

openshift-ci-robot avatar Oct 19 '22 08:10 openshift-ci-robot

/retest

jhadvig avatar Oct 19 '22 11:10 jhadvig

Screenshot 2022-10-20 at 13 37 55

/retest

jhadvig avatar Oct 20 '22 11:10 jhadvig

/retest

jhadvig avatar Oct 20 '22 22:10 jhadvig

/test e2e-gcp-console

jhadvig avatar Oct 21 '22 12:10 jhadvig

/retest

jhadvig avatar Oct 24 '22 15:10 jhadvig

/retest

jhadvig avatar Oct 25 '22 13:10 jhadvig

/retest

jhadvig avatar Oct 25 '22 20:10 jhadvig

Screenshot 2022-10-25 at 22 55 45

jhadvig avatar Oct 25 '22 20:10 jhadvig

/test e2e-gcp-console

jhadvig avatar Oct 26 '22 07:10 jhadvig

/test e2e-gcp-console

jhadvig avatar Oct 26 '22 13:10 jhadvig

/test e2e-gcp-console

jhadvig avatar Oct 26 '22 20:10 jhadvig

/test e2e-gcp-console

jhadvig avatar Oct 27 '22 12:10 jhadvig

/test e2e-gcp-console

jhadvig avatar Oct 27 '22 19:10 jhadvig

/test e2e-gcp-console

jhadvig avatar Oct 27 '22 22:10 jhadvig

/test e2e-gcp-console

jhadvig avatar Oct 28 '22 10:10 jhadvig

/retest

jhadvig avatar Oct 31 '22 14:10 jhadvig

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

openshift-ci[bot] avatar Oct 31 '22 19:10 openshift-ci[bot]