Thank you, Gyula. +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 Best, Marton On Mon, Feb 20, 2023 at 2:58 PM Gyula Fóra <gyula.f...@gmail.com> wrote: > +1 (binding) > > - Downloaded archives, validated binary content, checksum signatures, > NOTICE files > - Installed to local k8s cluster from Helm repo, verified docker image, > logs > - Ran some basic examples, ran autoscaler example > > Cheers > Gyula > > On Sun, Feb 19, 2023 at 5:09 AM Peter Huang <huangzhenqiu0...@gmail.com> > wrote: > > > Thanks for preparing the release! > > > > +1 (non-binding) > > > > 1. Downloaded the archives, checksums, and signatures > > 2. Extract and inspect the source code for binaries > > 3. Compiled and tested the source code via mvn verify > > 4. Deployed helm chart from flink-kubernetes-operator-1.4.0-helm.tgz to > > local minikube cluster > > 5. Ran example jobs with V1_14, V1_15, V1_16, and verified operator logs > > 6. Built the docker container locally and verified it through repeatting > > step 5 > > > > Best Regards > > Peter Huang > > > > On Fri, Feb 17, 2023 at 8:53 PM Jim Busche <jbus...@us.ibm.com> wrote: > > > > > Thanks Gyula for the release. > > > > > > +1 (non-binding) > > > > > > > > > I tested the following: > > > > > > * Helm repo install from flink-kubernetes-operator-1.4.0-helm.tgz > > > * Podman Dockerfile build from source, looked good. > > > * Twistlock security scans of proposed image looks good. No > > currently > > > known vulnerabilities with the built image or > > > ghcr.io/apache/flink-kubernetes-operator:7fc23a1 > > > * UI, basic sample, basic session jobs look good. Logs look as > > > expected. > > > * Checksums looked good > > > * Tested OLM build/install on OpenShift 4.12 > > > * Verified that sessionjob correctly can write in the operator’s > > > /opt/flink/artifacts filesystem on OpenShift even in a non-default > > > namespace. > > > > > > > > > > > > Thanks, Jim > > > > > >
