operator-sdk icon indicating copy to clipboard operation
operator-sdk copied to clipboard
verified publisher icon operator-framework

Remove the Remove the Optional(stage: alpha) Community Operator bundle

Open camilamacedo86 opened this issue 4 years ago • 6 comments

Feature Request

Describe the problem you need a feature to resolve.

For SDK release 1.18.0, remove the Optional(stage: alpha) Community Operator bundle since we move its implementation to the external validator.

Describe the solution you'd like.

  • Just perform the changes in: https://github.com/operator-framework/operator-sdk/pull/5412
  • also, remove the warning added in: https://github.com/operator-framework/operator-sdk/pull/5414
  • Then, ensure that we remove that from the API as well: https://github.com/operator-framework/api/pull/196

considered drawbacks

  • Check if its main consumers were not able to move forward and adopted the external validator before remove. (ping @camilamacedo86 and/or @gallettilance )

camilamacedo86 avatar Nov 30 '21 12:11 camilamacedo86

c/c @jmrodri

camilamacedo86 avatar Nov 30 '21 12:11 camilamacedo86

I've added this as a release blocker. This has been deprecated for some time, and I am guessing that this will prevent some more problems for @camilamacedo86 with operatorframework/api version bumps

asmacdo avatar May 23 '22 16:05 asmacdo

After speaking with @camilamacedo86 this is not ready to be done yet. I am moving to the backlog under priority/important-soon so this will be discussed at the next backlog grooming meeting.

asmacdo avatar May 24 '22 17:05 asmacdo

@asmacdo , We might want to add something here that is blocked until we ensure that its consumers no longer use it. I added the block label. Feel free to change if you see that is not good

camilamacedo86 avatar May 24 '22 21:05 camilamacedo86

@camilamacedo86 who are the "consumers" of this?

jmrodri avatar Jul 21 '22 14:07 jmrodri

Hi @jmrodri,

who are the "consumers" of this?

These are the users/places which "consume" the validator. Our concern here is that many places still using this validator so far. We might need to wait a long period to be able to remove the alpha check without impact nobody.

Therefore, I think we could add this one to the 2.x milestone.

camilamacedo86 avatar Jul 21 '22 14:07 camilamacedo86

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close. Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

openshift-bot avatar Nov 30 '22 01:11 openshift-bot