+1 (binding) 1. Verified the archives, checksums, and signatures 2. Extracted and inspected the source code for binaries 3. Compiled and tested the source code via mvn verify 4. Verified license files / headers 5. Deployed helm chart to test cluster 6. Ran example job 7. Tested autoscaling without resource requirements API 8. Tested autotuning
-Max On Wed, Oct 23, 2024 at 8:10 PM Márton Balassi <balassi.mar...@gmail.com> wrote: > > +1 (binding) > > - Verified Helm repo works as expected, points to correct image tag, build, > version > - Verified basic examples + checked operator logs everything looks as > expected > - Verified hashes, signatures and source release contains no binaries > - Ran built-in tests, built jars + docker image from source successfully > - Upgraded the operator and the CRD from 1.9.0 to 1.10.0 > > Best, > Marton > > On Mon, Oct 21, 2024 at 8:17 PM Mate Czagany <czmat...@gmail.com> wrote: > > > +1 (non-binding) > > > > - Verified checksums and signatures > > - Verified no binary files > > - Verified all source files contain Apache license header > > - Suspended job with 1.9.0 Operator, upgraded to 1.10.0, restarted job > > - Verified manual/periodic/upgrade FlinkStateSnapshot resources > > > > However with `helm upgrade`, the new FlinkStateSnapshot CRD won't be > > installed, and the docs don't cover this. I have created a PR to fix this > > [1]. > > > > [1] https://github.com/apache/flink-kubernetes-operator/pull/905 > > > > Yuepeng Pan <panyuep...@apache.org> ezt írta (időpont: 2024. okt. 21., H, > > 11:35): > > > > > +1 (non-binding) > > > > > > I verified that: > > > - Build from source on local PC with JDK-17.0.12- Check versions.- Run a > > > wordcount demo on colima + docker + minikube > > > > > > > > > > > > Best, > > > Yuepeng Pan > > > > > > > > > > > > > > > > > > At 2024-10-18 15:47:50, "Gabor Somogyi" <gabor.g.somo...@gmail.com> > > wrote: > > > >+1 (non-binding) > > > > > > > >* Verified versions in the poms > > > >* Built from source > > > >* Verified checksums and signatures > > > >* Started basic workload > > > >* Verified NOTICE and LICENSE files > > > > > > > >G > > > > > > > > > > > >On Fri, Oct 18, 2024 at 4:45 AM Sam Barker <s...@quadrocket.co.uk> > > wrote: > > > > > > > >> +1 (non binding) > > > >> > > > >> I've verified: > > > >> - The src tarball builds and passes mvn verify (with Java 17), > > > >> - the sha512 signatures for both srcs and helm > > > >> - The GPG > > > >> - Checked all poms are version 1.10.0 > > > >> - Checked all poms do not contain -SNAPSHOT dependencies > > > >> - Verified that the chart and appVersions from the helm tarball are > > > 1.10.0 > > > >> - Verified the chart points at the appropriate image > > > >> - Verified that the the RC repo works as a helm Repo > > > >> > > > >> On Wed, 16 Oct 2024 at 05:18, Őrhidi Mátyás <matyas.orh...@gmail.com> > > > >> wrote: > > > >> > > > >> > Hi everyone, > > > >> > > > > >> > Please review and vote on the release candidate #1 for the version > > > 1.10.0 > > > >> > of Apache Flink Kubernetes Operator, > > > >> > as follows: > > > >> > [ ] +1, Approve the release > > > >> > [ ] -1, Do not approve the release (please provide specific > > comments) > > > >> > > > > >> > **Release Overview** > > > >> > > > > >> > As an overview, the release consists of the following: > > > >> > a) Kubernetes Operator canonical source distribution (including the > > > >> > Dockerfile), to be deployed to the release repository at > > > dist.apache.org > > > >> > b) Kubernetes Operator Helm Chart to be deployed to the release > > > >> repository > > > >> > at dist.apache.org > > > >> > c) Maven artifacts to be deployed to the Maven Central Repository > > > >> > d) Docker image to be pushed to dockerhub > > > >> > > > > >> > **Staging Areas to Review** > > > >> > > > > >> > The staging areas containing the above mentioned artifacts are as > > > >> follows, > > > >> > for your review: > > > >> > * All artifacts for a,b) can be found in the corresponding dev > > > repository > > > >> > at dist.apache.org [1] > > > >> > * All artifacts for c) can be found at the Apache Nexus Repository > > [2] > > > >> > * The docker image for d) is staged on github [3] > > > >> > > > > >> > All artifacts are signed with the key 48E78F054AA33CB5 [4] > > > >> > > > > >> > Other links for your review: > > > >> > * JIRA release notes [5] > > > >> > * source code tag "release-1.1.0-rc1" [6] > > > >> > * PR to update the website Downloads page to include Kubernetes > > > Operator > > > >> > links [7] > > > >> > > > > >> > **Vote Duration** > > > >> > > > > >> > The voting time will run for at least 72 hours. > > > >> > It is adopted by majority approval, with at least 3 PMC affirmative > > > >> votes. > > > >> > > > > >> > **Note on Verification** > > > >> > > > > >> > You can follow the basic verification guide here[8]. > > > >> > Note that you don't need to verify everything yourself, but please > > > make > > > >> > note of what you have tested together with your +- vote. > > > >> > > > > >> > Thanks, > > > >> > Matyas > > > >> > > > > >> > [1] > > > >> > > > > >> > > > > >> > > > > > https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-1.10.0-rc1/ > > > >> > [2] > > > >> > > https://repository.apache.org/content/repositories/orgapacheflink-1762 > > > >> > [3] ghcr.io/apache/flink-kubernetes-operator:c703255 > > > >> > [4] https://dist.apache.org/repos/dist/release/flink/KEYS > > > >> > [5] > > > >> > > > > >> > > > > >> > > > > > https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12354833 > > > >> > [6] > > > >> > > > > >> > > > > >> > > > > > https://github.com/apache/flink-kubernetes-operator/releases/tag/release-1.10.0-rc1 > > > >> > [7] https://github.com/apache/flink-web/pull/758 > > > >> > [8] > > > >> > > > > >> > > > > >> > > > > > https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > ghcr.io/apache/flink-kubernetes-operator:c703255 > > > >> > > > > >> > > > > >