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

Remove obsolete OSSMC operator #3533

Merged
merged 1 commit into from
Nov 1, 2023
Merged

Remove obsolete OSSMC operator #3533

merged 1 commit into from
Nov 1, 2023

Conversation

jmazzitelli
Copy link
Contributor

@jmazzitelli jmazzitelli commented Oct 30, 2023 •
edited
Loading

The OSSMC operator (which was only released as a dev preview) is obsolete. It's functionality has been moved to Kiali Operator 1.76.0 which has just been published (see #3532)

To avoid confusion, the dev preview of OSSMC operator is being completely removed here.

fixes: kiali/kiali#6715

@openshift-ci openshift-ci bot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Oct 30, 2023
@openshift-ci
Copy link

openshift-ci bot commented Oct 30, 2023

Hi @jmazzitelli. Thanks for your PR.

I'm waiting for a redhat-openshift-ecosystem member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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 openshift-ci bot requested review from haripate and tomasbakk October 30, 2023 16:32
@jmazzitelli jmazzitelli mentioned this pull request Oct 30, 2023
Closed
@github-actions github-actions bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Oct 30, 2023
@jmazzitelli
Remove obsolete OSSMC operator.
ed76415 
The OSSMC operator (which was only released as a dev preview) is obsolete.
It's functionality has been moved to Kial Operator 1.76.0 which has
just been published.
To avoid confusion, the dev preview of OSSMC operator is being completely
removed here.

Signed-off-by: John Mazzitelli <[email protected]>
@framework-automation
Copy link
Collaborator

/merge possible

@github-actions
Copy link
Contributor

DCO is missing. Please signoff your commits !!!

@framework-automation
Copy link
Collaborator

/merge possible

@framework-automation
Copy link
Collaborator

/merge possible

@github-actions
Copy link
Contributor

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

@framework-automation
Copy link
Collaborator

/merge possible

@framework-automation
Copy link
Collaborator

/merge possible

@github-actions
Copy link
Contributor

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

1 similar comment
@github-actions
Copy link
Contributor

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

@jmazzitelli
Copy link
Contributor Author

jmazzitelli commented Oct 30, 2023 •
edited
Loading

Because I am removing this operator, there is no ci.yaml and thus no authorized users. You can see I am on the original ci.yaml as an authorized user. Can someone approve this?

This is kiali/kiali#6715

@jmazzitelli
Copy link
Contributor Author

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

That "here" link is invalid.

@jmazzitelli
Copy link
Contributor Author

cc @mporrato @Allda

Can you push this PR along?

@jmazzitelli
Copy link
Contributor Author

also cc'ing the reviewers in case they can help move this along: @haripate and @tomasbakk

@jmazzitelli
Copy link
Contributor Author

ping @haripate @tomasbakk @mporrato @Allda

@framework-automation
Copy link
Collaborator

/merge possible

@framework-automation framework-automation merged commit d0ebf6a into redhat-openshift-ecosystem:main Nov 1, 2023
13 checks passed
@jmazzitelli jmazzitelli deleted the remove-ossmc branch November 1, 2023 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@haripate haripate Awaiting requested review from haripate

@tomasbakk tomasbakk Awaiting requested review from tomasbakk

Assignees

@mporrato mporrato

@Allda Allda

Labels
authorized-changes installation-validated installation-validated-4.10 installation-validated-4.11 installation-validated-4.12 installation-validated-4.13 installation-validated-4.14 installation-validated-4.15 ok-to-test Indicates a non-member PR verified by an org member that is safe to test. openshift-operator package-validated
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

purge the OSSMC operator from red hat community catalog once Kiali Operator supports OSSMC
5 participants
@jmazzitelli @framework-automation @mporrato @Allda @haripate