operator-framework / community-operators

The canonical source for Kubernetes Operators that appear on OperatorHub.io, OpenShift Container Platform and OKD.
Apache License 2.0
419 stars 561 forks source link

Announcing self-sufficient PR merges to automatically publish Operators #3883

Closed J0zi closed 3 years ago

J0zi commented 3 years ago

Dear OperatorHub.io and OpenShift Operator Community,

we are pleased to announce, that we have invested significantly in the Operator contribution process to allow for your contributions and PRs to be merged faster. This means you can publish your new Operators quickly and without review from the community maintainers. Your PR needs to meet a couple of criteria that are documented here: https://operator-framework.github.io/community-operators/self-merge-updates/

If your PR fits the criteria it will be merged automatically. In this process you have complete control over which GitHub user is able to leverage this feature and by default no auto-merging is allowed until the PR author (including you) is part of the reviewers list.

We hope this change enables you to work more effectively and self-sufficiently in publishing updates to your Operator.

Kind Regards, The community-operators maintainers team

mentions: @2uasimojo, @AdheipSingh, @ArangoGutierrez, @AstroProfundis, @Avni-Sharma, @Carrefour-Group, @CohenAviad, @DTMad, @DhritiShikhar, @EmilyM1, @EnterpriseDB, @Flagsmith, @Fryguy, @HubertStefanski, @J0zi, @JohnStarich, @Kaitou786, @Kong, @LCaparelli, @LaVLaS, @Listson, @LorbusChris, @LouisPlisso, @OchiengEd, @Rajpratik71, @RakhithaRR, @SDBrett, @Simonzhaohui, @SteveMattar, @Tatsinnit, @TheWeatherCompany, @aavarghese, @abaiken, @aceeric, @akhil-rane, @akoserwal, @akram, @alien-ike, @aliok, @andreaskaris, @aneeshkp, @antonlisovenko, @anyasabo, @apahim, @apeschel, @aravindhp, @ashenoy-hedvig, @aslakknutsen, @astefanutti, @atef23, @avalluri, @bart0sh, @bcrochet, @bigkevmcd, @blaqkube, @blublinsky, @brian-avery, @cap1984, @chanwit, @charith-elastic, @chatton, @chbatey, @che-bot, @che-incubator, @che-incubator-bot, @cheesiong-lee, @chetan-rns, @christophd, @clamoriniere, @cliveseldon, @cloudant, @containers-ai, @couchbase-partners, @cschwede, @ctron, @dabeeeenster, @dagrayvid, @danielpacak, @dannyzaken, @darkowlzz, @david-kow, @deeghuge, @deekshahegde86, @devOpsHelm, @dgoodwin, @dinhxuanvu, @djzager, @dlbock, @dmatch01, @dmesser, @dragonly, @dtrawins, @dvropsmx, @dymurray, @ecordell, @eguzki, @eresende-nuodb, @erikerlandson, @esara, @evan-hataishi, @f41gh7, @fbladilo, @fcanovai, @flaper87, @frant-hartm, @gallettilance, @gautambaghel, @gee4vee, @gregsheremeta, @gunjan5, @gurushantj, @guyyakir, @gyliu513, @haibinxie, @hasancelik, @hco-bot, @houshengbo, @husky-parul, @iamabhishek-dubey, @ibuziuk, @ingvagabund, @instana, @irajdeep, @ivanstanev, @jeesmon, @jfchevrette, @jitendar-singh, @jkatz, @jkhelil, @jmazzitelli, @jmeis, @jmesnil, @jmontleon, @joeschram, @jogetworkflow, @jomkz, @jonathanvila, @jpkrohling, @jsenko, @juljog, @k-wall, @kerenlahav, @kerrygunn, @knrc, @ksatchit, @kubemod, @kubemq, @kubemq-io, @kubernetes-projects, @kulkarnicr, @kyguy, @lbroudoux, @leifmadsen, @leochr, @little-guy-lxr, @liuruiyiyang, @lrgar, @lumjjb, @madhukirans, @madorn, @maistra, @maskarb, @matzew, @max3903, @mdonkers, @mflendrich, @microcks, @miguelsorianod, @mkuznyetsov, @mrethers, @mrizzi, @mtyazici, @mvalahtv, @mvalarh, @n1r1, @nickboldt, @nicolaferraro, @nikhil-thomas, @olukas, @open-cluster-management, @operator-framework, @oranichu, @orenc1, @oriyarde, @pavelmaliy, @pb82, @pebrc, @pedjak, @phantomjinx, @piyush-nimbalkar, @portworx, @prft-rh, @pweil-, @radtriste, @raffaelespazzoli, @rainest, @rajivnathan, @rareddy, @raunakkumar, @rayfordj, @redhat-cop, @rensyct, @renuka-fernando, @rgolangh, @rhm-samples, @ricardozanini, @rigazilla, @rodrigovalin, @rohanjayaraj, @rojkov, @rrati, @rubenvp8510, @rvansa, @ryanemerson, @saada, @sabre1041, @sbose78, @scholzj, @seanmalloy, @sebsoto, @secondsun, @selvamt94, @shivanshs9, @shubham-pampattiwar, @slaskawi, @slopezz, @snyk, @snyksec, @spolti, @squids-io, @sunsingerus, @svallala, @sxd, @tahmmee, @tanalam2411, @thbkrkr, @tmckayus, @tolusha, @tomashibm, @tomgeorge, @tphee, @tplavcic, @tumido, @turbonomic, @twiest, @ursais, @vaibhavjainwiz, @vassilvk, @vboulineau, @vkvamsiopsmx, @vmuzikar, @waveywaves, @waynesun09, @weii666, @welshDoug, @wiggzz, @willholley

github-actions[bot] commented 3 years ago

This issue is stale because it has been open for 30 days with no activity.

2uasimojo commented 3 years ago

So hey, this is cool, but per doc:

If author of PR is not in reviewers list, PR will not be merged automatically. For automatic merging PR needs to have authorized-changes label set.

I'm observing (1, 2) that even an /approve or /lgtm by a pre-existing member of the reviewer list (in ci.yaml) doesn't trigger automatic merge. Can it be made so?