openshift / console-plugin-template

Minimal template for writing OpenShift console plugins
Apache License 2.0
37 stars 47 forks source link

CONSOLE-3303: Move plugin template to the openshift GitHub org -: Add Dockerfile.ci.test with ENTRYPOINT to failing test in CI #21

Closed cyril-ui-developer closed 1 year ago

cyril-ui-developer commented 1 year ago

Link to CI failing test: https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/33499/rehearse-33499-pull-ci-openshift-console-plugin-template-main-e2e-aws-plugin-template/1632817170343792640

I will have to update the release repo PR with the Dockerfile.ci.test file if this PR changes look good. https://github.com/openshift/release/pull/33499/files#diff-51f0aa858d7d03c2a447beda5b0280f84806b5d830d2d404f564c9005dbdaa81R13

openshift-ci-robot commented 1 year ago

@cyril-ui-developer: This pull request references CONSOLE-3303 which is a valid jira issue.

In response to [this](https://github.com/openshift/console-plugin-template/pull/21): >Link to CI failing test: https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/33499/rehearse-33499-pull-ci-openshift-console-plugin-template-main-e2e-aws-plugin-template/1632817170343792640 > >I will have to update the release repo PR with the Dockerfile.ci.test file if this PR changes look good. >https://github.com/openshift/release/pull/33499/files#diff-51f0aa858d7d03c2a447beda5b0280f84806b5d830d2d404f564c9005dbdaa81R13 > Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
cyril-ui-developer commented 1 year ago

/assign @spadgett

cyril-ui-developer commented 1 year ago

Thanks @cyril-ui-developer. LGTM. Have you been able to build and test the image manually since we need openshift/release#33499 for CI?

@spadgett Yes, it looks good. Waiting on this changes to merge so that I can retest the release repo PR.

jerolimov commented 1 year ago

/cc