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

Delete AlloyDB Omni operator #5925

Merged
merged 1 commit into from
Feb 11, 2025

Conversation

jahnavi2k
Copy link
Contributor

Signed-off-by: Jahnavi Malhotra [email protected]
Co-authored-by: Manvi Gupta [email protected]

Thanks for submitting your Operator. Please check the below list before you create your Pull Request.

New Submissions

Your submission should not

  • Add more than one operator bundle per PR
  • Modify any operator
  • Rename an operator
  • Modify any files outside the above-mentioned folders
  • Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

  1. Description of the managed Application and where to find more information
  2. Features and capabilities of your Operator and how to use it
  3. Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

  • Human-readable name and 1-liner description about your Operator
  • Valid category name 1
  • One of the pre-defined capability levels2
  • Links to the maintainer, source code and documentation
  • Example templates for all Custom Resource Definitions intended to be used
  • A quadratic logo

Remember that you can preview your CSV here.

--

1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need

2 For more information see here

Signed-off-by: Jahnavi Malhotra [email protected]
Co-authored-by: Manvi Gupta [email protected]

Signed-off-by: Jahnavi Malhotra <[email protected]>
@rh-operator-bundle-bot
Copy link
Contributor

Pipeline Summary

Pipeline: operator-hosted-pipeline
PipelineRun: operator-hosted-pipeline-runsrfvv
Start Time: 2025-02-08 04:39:56+00:00

Tasks

Status Task Start Time Duration
✔️ get-pr-number 2025-02-08 04:39:58+00:00 8 seconds
✔️ acquire-lease 2025-02-08 04:40:09+00:00 7 seconds
✔️ set-github-started-label 2025-02-08 04:40:21+00:00 8 seconds
✔️ set-github-status-pending 2025-02-08 04:40:31+00:00 9 seconds
✔️ set-env 2025-02-08 04:40:45+00:00 8 seconds
✔️ clone-repository 2025-02-08 04:40:56+00:00 51 seconds
✔️ clone-repository-base 2025-02-08 04:40:56+00:00 52 seconds
✔️ detect-changes 2025-02-08 04:41:51+00:00 a minute
✔️ check-permissions 2025-02-08 04:42:56+00:00 47 seconds
✔️ set-github-pr-title 2025-02-08 04:43:45+00:00 7 seconds
✔️ read-config 2025-02-08 04:43:56+00:00 43 seconds
✔️ resolve-pr-type 2025-02-08 04:44:42+00:00 6 seconds
✔️ content-hash 2025-02-08 04:44:51+00:00 42 seconds
✔️ certification-project-check 2025-02-08 04:45:35+00:00 47 seconds
✔️ get-organization 2025-02-08 04:46:24+00:00 39 seconds
✔️ get-pyxis-certification-data 2025-02-08 04:47:07+00:00 38 seconds
✔️ static-tests 2025-02-08 04:47:47+00:00 48 seconds
✔️ static-tests-results 2025-02-08 04:48:38+00:00 23 seconds
✔️ merge-registry-credentials 2025-02-08 04:49:04+00:00 24 seconds
✔️ digest-pinning 2025-02-08 04:49:31+00:00 46 seconds
✔️ verify-pinned-digest 2025-02-08 04:50:19+00:00 7 seconds
✔️ dockerfile-creation 2025-02-08 04:50:28+00:00 46 seconds
✔️ build-bundle 2025-02-08 04:51:16+00:00 53 seconds
✔️ make-bundle-repo-public 2025-02-08 04:52:10+00:00 7 seconds
✔️ get-supported-versions 2025-02-08 04:52:19+00:00 45 seconds
✔️ make-index-repo-public 2025-02-08 04:53:05+00:00 7 seconds
✔️ get-ci-results-attempt 2025-02-08 04:53:14+00:00 16 seconds
✔️ preflight-trigger 2025-02-08 04:53:31+00:00 20 seconds
✔️ get-ci-results 2025-02-08 04:53:51+00:00 23 seconds
✔️ link-pull-request-with-open-status 2025-02-08 04:54:14+00:00 8 seconds
✔️ merge-pr 2025-02-08 04:54:22+00:00 42 seconds

Pipeline logs: https://gist.github.com/rh-operator-bundle-bot/3f01af0c3494baf3bfdf587326e852b0

Troubleshooting

Please refer to the troubleshooting guide.

Run /pipeline restart operator-hosted-pipeline in case of pipeline failure to restart a pipeline.

@jahnavi2k
Copy link
Contributor Author

jahnavi2k commented Feb 11, 2025

Hi @Allda , @mporrato
Could you please help merge this or point to the best contact for PR approval?

Thanks and Regards
Jahnavi

@mporrato mporrato merged commit 94b7ec5 into redhat-openshift-ecosystem:main Feb 11, 2025
2 checks passed
@mporrato
Copy link
Collaborator

mporrato commented Feb 11, 2025

@jahnavi2k I see this is the second time this operator is being removed. Are you facing an issue with the workflow?
You can enable auto-merge of pull requests by setting merge: true in your ci.yaml.

@rh-operator-bundle-bot
Copy link
Contributor

Pipeline Summary

Pipeline: operator-release-pipeline
PipelineRun: operator-release-pipeline-rungz9nr
Start Time: 2025-02-11 09:15:20+00:00

Tasks

Status Task Start Time Duration
✔️ set-github-started-label 2025-02-11 09:15:23+00:00 11 seconds
✔️ set-github-status-pending 2025-02-11 09:15:35+00:00 8 seconds
✔️ set-env 2025-02-11 09:15:45+00:00 14 seconds
✔️ clone-repository 2025-02-11 09:16:00+00:00 a minute
✔️ clone-repository-base 2025-02-11 09:16:01+00:00 a minute
✔️ detect-changes 2025-02-11 09:17:08+00:00 a minute
✔️ read-config 2025-02-11 09:18:08+00:00 14 seconds
✔️ resolve-pr-type 2025-02-11 09:18:22+00:00 7 seconds
✔️ content-hash 2025-02-11 09:18:30+00:00 49 seconds
✔️ certification-project-check 2025-02-11 09:19:19+00:00 14 seconds
✔️ get-organization 2025-02-11 09:19:33+00:00 12 seconds
✔️ get-pyxis-certification-data 2025-02-11 09:19:46+00:00 17 seconds
✔️ copy-bundle-image-to-released-registry 2025-02-11 09:20:06+00:00 12 seconds
✔️ get-supported-versions 2025-02-11 09:20:22+00:00 49 seconds
✔️ acquire-lease 2025-02-11 09:21:13+00:00 7 seconds

Pipeline logs: https://gist.github.com/rh-operator-bundle-bot/b1f72511fdb4b7cad0f8cea0def62dc4

Release info

Nothing was released.

Troubleshooting

Please refer to the troubleshooting guide.

Run /pipeline restart operator-release-pipeline in case of pipeline failure to restart a pipeline.

@jahnavi2k
Copy link
Contributor Author

jahnavi2k commented Feb 11, 2025

Hi @mporrato
The first time we deleted the operator was because it was an accidental merge and we had a release scheduled. This time we had to delete it because there was a slight issue with the description which came to our notice later. We will be raising a new PR shortly with the correct description.
Oh, we will set the field in the ci.yaml. Thank you for letting us know!

Also, needed a small clarification: Any updates to an existing bundle except rollback isn't possible, even with FBC- enabled, right? Our change was failing the 'detect changes' test in the pipeline, that is why we had to completely rollback.
Regards
Jahnavi

@mporrato
Copy link
Collaborator

What you can do with fbc enabled is remove the incorrect bundle and then add it back with the fix applied. It will involve two PRs, but at least it can be done in a self service fashion.

@jahnavi2k
Copy link
Contributor Author

So what we tried the last time was to remove the bundle from the catalogs as well as from the operators directory in a single PR. But the pipeline failed due to which we had to first remove it from the catalogs and then delete it with manual approval. Is there a better way that could have been handled or even now? Because without removing it from the catalogs, the pipeline fails.
Also in our new PR, we added merge:true (#5936) but I suppose for new operators approval is needed?
@mporrato

@mporrato
Copy link
Collaborator

You're right. It will actually be 4 PRs: remove the bundle from the indices, remove the actual bundle, add it back and update the indices again. We are working on making it possible to change indices and a bundle in a single PRs but this is not ready yet.
Ad yes, adding a new operator always requires manual approval from our side: auto-merge will take effect starting from the next PR.

@jahnavi2k
Copy link
Contributor Author

Got it, thank you so much for the insights and help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants