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.

Sign up for GitHub

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

Jump to bottom

Interop UI topology helmrelease #7845

Closed
wants to merge 1 commit into from
Closed

Interop UI topology helmrelease #7845

wants to merge 1 commit into from

Conversation

shvasude
Copy link
Contributor

@shvasude shvasude commented Jan 15, 2021 •
edited

Jira Id: https://issues.redhat.com/browse/HELM-15, https://issues.redhat.com/browse/HELM-67
Add scenarios implementation for feature - topology on helm release

  • Open Side Bar for the Helm release
  • Deployment Configs link on the sidebar for the Helm Release
  • Build Configs link on the sidebar for the Helm Release
  • Services link on the sidebar for the Helm Release
  • Image Streams link on the sidebar for the Helm Release
  • Routes link on the sidebar for the Helm Release
  • Actions drop down on the side bar

@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 15, 2021
@openshift-ci-robot openshift-ci-robot added component/backend Related to backend component/core Related to console core functionality component/olm Related to OLM kind/i18n Indicates issue or PR relates to internationalization or has content that needs to be translated labels Jan 17, 2021
@shvasude shvasude force-pushed the interop-ui-topology-helmrelease branch from ed42fde to 697b575 Compare January 17, 2021 02:23
@shvasude shvasude changed the title [WIP]: Interop UI topology helmrelease Interop UI topology helmrelease Jan 17, 2021
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 17, 2021
@shvasude
Copy link
Contributor Author

/assign @makambalaji

@openshift-ci-robot openshift-ci-robot added component/ceph Related to ceph-storage-plugin component/dashboard Related to dashboard component/knative Related to knative-plugin component/kubevirt Related to kubevirt-plugin component/lso Related to local-storage-operator-plugin component/monitoring Related to monitoring component/shared Related to console-shared needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Jan 29, 2021
@makambalaji
Copy link
Contributor

Please fix the conflicts @shvasude

@makambalaji
Copy link
Contributor

@shvasude - It's rebase issue, please fix this issues
@sanketpathak - Please review this pr after fixes

@openshift-ci-robot openshift-ci-robot added component/insights Related to insights plugin component/metal3 Related to metal3-plugin component/network-attachment-definition Related to network-attachment-definition component/noobaa Related to noobaa-storage-plugin component/sdk Related to console-plugin-sdk and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Feb 3, 2021
@shvasude shvasude force-pushed the interop-ui-topology-helmrelease branch from 17f8bf4 to 336ab1e Compare February 3, 2021 20:25
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: shvasude
To complete the pull request process, please assign makambalaji after the PR has been reviewed.
You can assign the PR to them by writing /assign @makambalaji in a comment when ready.

The full list of commands accepted by this bot can be found 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-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 4, 2021
add implementation for feature: install-HelmChart for scenarios
6a6e29d 
* The Helm Chart option on the +Add Page
* Install Helm Chart from Developer Catalog Page using YAML View
* Install Helm Chart from +Add Page using Form View
* Chart versions drop down menu
@shvasude shvasude force-pushed the interop-ui-topology-helmrelease branch from 08ca779 to 6a6e29d Compare February 4, 2021 03:37
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 4, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 4, 2021 •
edited

@shvasude: The following test failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/e2e-gcp-console 6a6e29d link /test e2e-gcp-console

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-robot
Copy link
Contributor

@shvasude: PR needs rebase.

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.

@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 7, 2021
@shvasude shvasude closed this Feb 8, 2021
@shvasude
Copy link
Contributor Author

shvasude commented Feb 8, 2021

There has been multiple labels added automatically by Openshift-CI robot which is causing issues. Hence closing this and opening
#8093

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gajanan-more gajanan-more Awaiting requested review from gajanan-more

@sanketpathak sanketpathak Awaiting requested review from sanketpathak

Assignees

@makambalaji makambalaji

Labels
component/backend Related to backend component/ceph Related to ceph-storage-plugin component/core Related to console core functionality component/dashboard Related to dashboard component/dev-console Related to dev-console component/insights Related to insights plugin component/knative Related to knative-plugin component/kubevirt Related to kubevirt-plugin component/lso Related to local-storage-operator-plugin component/metal3 Related to metal3-plugin component/monitoring Related to monitoring component/network-attachment-definition Related to network-attachment-definition component/noobaa Related to noobaa-storage-plugin component/olm Related to OLM component/sdk Related to console-plugin-sdk component/shared Related to console-shared kind/i18n Indicates issue or PR relates to internationalization or has content that needs to be translated needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@shvasude @makambalaji @openshift-ci-robot