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

Revert "bug: remove additional operator-backed items in the developer… #171

Merged
merged 2 commits into from
Jul 13, 2021

Conversation

shubhamagarwal19
Copy link
Contributor

@shubhamagarwal19 shubhamagarwal19 commented Jul 13, 2021

… catalog in Dev Console (#165)"

This reverts commit 8bae146.

Need to use Annotations to hide internal APIs ...
operator-framework/operator-lifecycle-manager#1097 (comment)

What type of PR is this?

Uncomment only one /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind bug
/kind cleanup
/kind failing-test
/kind enhancement
/kind documentation
/kind code-refactoring

What does this PR do / why we need it:

Have you updated the necessary documentation?

  • Documentation update is required by this PR.
  • Documentation has been updated.

Which issue(s) this PR fixes:

Fixes #?

Test acceptance criteria:

  • Unit Test
  • E2E Test

How to test changes / Special notes to the reviewer:

@jannfis
Copy link
Member

jannfis commented Jul 13, 2021

/approve

@openshift-ci
Copy link

openshift-ci bot commented Jul 13, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jannfis

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

@iam-veeramalla
Copy link
Collaborator

@shubhamagarwal19 what about marking gitopsservice as an internal object using the annotation operators.operatorframework.io/internal-objects

Are you planning to raise a separate PR for that change ?

@jannfis
Copy link
Member

jannfis commented Jul 13, 2021

Are you planning to raise a separate PR for that change ?

Yes, please make it a separate PR.

@jannfis
Copy link
Member

jannfis commented Jul 13, 2021

/lgtm

@openshift-ci openshift-ci bot removed the lgtm label Jul 13, 2021
@jannfis
Copy link
Member

jannfis commented Jul 13, 2021

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Jul 13, 2021
@jannfis
Copy link
Member

jannfis commented Jul 13, 2021

/kind bug

@openshift-ci openshift-ci bot added the kind/bug Something isn't working label Jul 13, 2021
@openshift-merge-robot openshift-merge-robot merged commit daf76db into redhat-developer:master Jul 13, 2021
trdoyle81 pushed a commit to trdoyle81/gitops-operator that referenced this pull request Aug 13, 2024
… catalog in Dev Console (redhat-developer#165)" (redhat-developer#171)

This reverts commit 8bae146.

Co-authored-by: jannfis <jann@mistrust.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved kind/bug Something isn't working lgtm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants