You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flink.apache.org by Gyula Fóra <gy...@apache.org> on 2022/03/30 14:38:53 UTC

[VOTE] Apache Flink Kubernetes Operator Release 0.1.0, release candidate #3

Hi everyone,

Please review and vote on the release candidate #3 for the version 0.1.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 is staged on github [7]

All artifacts are signed with the key
0B4A34ADDFFA2BB54EB720B221F06303B87DAFF1 [3]

Other links for your review:
* JIRA release notes [4]
* source code tag "release-0.1.0-rc3" [5]
* PR to update the website Downloads page to include Kubernetes Operator
links [6]

**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
<https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release>
.
Note that you don't need to verify everything yourself, but please make
note of what you have tested together with your +- vote.

Thanks,
Gyula

[1]
https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3/
[2] https://repository.apache.org/content/repositories/orgapacheflink-1492/
[3] https://dist.apache.org/repos/dist/release/flink/KEYS
[4]
https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351499
[5]
https://github.com/apache/flink-kubernetes-operator/tree/release-0.1.0-rc3
[6] https://github.com/apache/flink-web/pull/519
[7] ghcr.io/apache/flink-kubernetes-operator:2c166e3

Re: [VOTE] Apache Flink Kubernetes Operator Release 0.1.0, release candidate #3

Posted by Nicholas Jiang <ni...@apache.org>.
+1 (non-binding)

1.Verified maven and helm chart versions for built from source
2.Verified helm chart points to correct docker image and deploys it by
default
3.Verified helm installation and
basic/checkpointing, stateful examples with upgrades and manual savepoints
4.Verified online documents including Quick Start etc.

Best,
Nicholas Jiang

On 2022/03/31 08:53:40 Yang Wang wrote:
> +1 (non-binding)
> 
> Verified via the following steps:
> 
> * Verify checksums and GPG signatures
> * Verify that the source distributions do not contain any binaries
> * Build source distribution successfully
> * Verify all the POM version is 0.1.0
> 
> * License check, the jars bundled in docker image and maven artifacts have
> correct NOTICE and licenses
> 
> # Functionality verification
> * Install flink-kubernetes-operator via helm
>     - helm repo add flink-kubernetes-operator-0.1.0-rc3
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3
>     - helm install flink-kubernetes-operator
> flink-kubernetes-operator-0.1.0-rc3/flink-kubernetes-operator
> 
> * Apply a new FlinkDeployment CR with HA and ingress enabled, Flink webUI
> normal
>     - kubectl apply -f
> https://raw.githubusercontent.com/apache/flink-kubernetes-operator/release-0.1/e2e-tests/data/cr.yaml
> 
> * Upgrade FlinkDeployment, new job parallelism takes effect and recover
> from latest checkpoint
>     - kubectl patch flinkdep flink-example-statemachine --type merge
> --patch '{"spec":{"job": {"parallelism": 1 } } }'
> 
> * Verify manual savepoint trigger
>     - kubectl patch flinkdep flink-example-statemachine --type merge
> --patch '{"spec":{"job": {"savepointTriggerNonce": 1 } } }'
> 
> * Suspend a FlinkDeployment
>     - kubectl patch flinkdep flink-example-statemachine --type merge
> --patch '{"spec":{"job": {"state": "suspended" } } }'
> 
> 
> Best,
> Yang
> 
> Márton Balassi <mb...@apache.org> 于2022年3月31日周四 01:01写道:
> 
> > +1 (binding)
> >
> > Verified the following:
> >
> >    - shasums
> >    - gpg signatures
> >    - source does not contain any binaries
> >    - built from source
> >    - deployed via helm after adding the distribution webserver endpoint as
> >    a helm registry
> >    - all relevant files have license headers
> >
> >
> > On Wed, Mar 30, 2022 at 4:39 PM Gyula Fóra <gy...@apache.org> wrote:
> >
> > > Hi everyone,
> > >
> > > Please review and vote on the release candidate #3 for the version 0.1.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 is staged on github [7]
> > >
> > > All artifacts are signed with the key
> > > 0B4A34ADDFFA2BB54EB720B221F06303B87DAFF1 [3]
> > >
> > > Other links for your review:
> > > * JIRA release notes [4]
> > > * source code tag "release-0.1.0-rc3" [5]
> > > * PR to update the website Downloads page to include Kubernetes Operator
> > > links [6]
> > >
> > > **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
> > > <
> > >
> > https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release
> > > >
> > > .
> > > Note that you don't need to verify everything yourself, but please make
> > > note of what you have tested together with your +- vote.
> > >
> > > Thanks,
> > > Gyula
> > >
> > > [1]
> > >
> > >
> > https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3/
> > > [2]
> > > https://repository.apache.org/content/repositories/orgapacheflink-1492/
> > > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > > [4]
> > >
> > >
> > https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351499
> > > [5]
> > >
> > https://github.com/apache/flink-kubernetes-operator/tree/release-0.1.0-rc3
> > > [6] https://github.com/apache/flink-web/pull/519
> > > [7] ghcr.io/apache/flink-kubernetes-operator:2c166e3
> > >
> >
> 

Re: [VOTE] Apache Flink Kubernetes Operator Release 0.1.0, release candidate #3

Posted by Biao Geng <bi...@gmail.com>.
+1 (non-binding)

1. Verify that the checksums and GPG files
2. Verify that the source distributions do not contain any binaries
3. Build the source distribution to ensure all source files
4. Verify that POM files and the README.md file do not have anything
unexpected.
6. Verify that chart and appVersion matches the target release
7. Verify Helm chart can be installed without overriding any parameters
using helm install flink-kubernetes-operator
flink-kubernetes-operator-0.1.0-rc3/flink-kubernetes-operator
8. Verify docker image building
9. Verify the job using  examples/basic-checkpoint-ha.yaml and check
updated job config(e.g. parallelism) can take effect when using last-state
mode
10. Verified operator logs does not contain unexpected things

For some of the included examples:

Submit yaml
Submit job / spec upgrade
Validate operator logs
Validate deployment status
For stateful examples

Verify stateful upgrade uses the correct checkpoint/savepoint
Trigger manual savepoint
Gyula Fóra <gy...@gmail.com> 于2022年3月31日周四 18:18写道:

> +1 (binding)
>
> Verified the following:
>
>    - Verified source distribution do not contain binaries
>    - Verified LICENSES and NOTICES
>    - Built from source, verified maven and helm chart versions
>    - Verified helm chart points to correct docker image and deploys it by
>    default
>    - Verified maven artifacts correctly deployed to staging repository and
>    contents
>    - Verified staging repo as helm repo, helm installation and
>    basic/checkpointing examples with upgrades and manual savepoints
>    - Verified operator logs and cr status does not contain unexpected
>    things
>
> Cheers,
> Gyula
>
> On Thu, Mar 31, 2022 at 10:54 AM Yang Wang <da...@gmail.com> wrote:
>
> > +1 (non-binding)
> >
> > Verified via the following steps:
> >
> > * Verify checksums and GPG signatures
> > * Verify that the source distributions do not contain any binaries
> > * Build source distribution successfully
> > * Verify all the POM version is 0.1.0
> >
> > * License check, the jars bundled in docker image and maven artifacts
> have
> > correct NOTICE and licenses
> >
> > # Functionality verification
> > * Install flink-kubernetes-operator via helm
> >     - helm repo add flink-kubernetes-operator-0.1.0-rc3
> >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3
> >     - helm install flink-kubernetes-operator
> > flink-kubernetes-operator-0.1.0-rc3/flink-kubernetes-operator
> >
> > * Apply a new FlinkDeployment CR with HA and ingress enabled, Flink webUI
> > normal
> >     - kubectl apply -f
> >
> >
> https://raw.githubusercontent.com/apache/flink-kubernetes-operator/release-0.1/e2e-tests/data/cr.yaml
> >
> > * Upgrade FlinkDeployment, new job parallelism takes effect and recover
> > from latest checkpoint
> >     - kubectl patch flinkdep flink-example-statemachine --type merge
> > --patch '{"spec":{"job": {"parallelism": 1 } } }'
> >
> > * Verify manual savepoint trigger
> >     - kubectl patch flinkdep flink-example-statemachine --type merge
> > --patch '{"spec":{"job": {"savepointTriggerNonce": 1 } } }'
> >
> > * Suspend a FlinkDeployment
> >     - kubectl patch flinkdep flink-example-statemachine --type merge
> > --patch '{"spec":{"job": {"state": "suspended" } } }'
> >
> >
> > Best,
> > Yang
> >
> > Márton Balassi <mb...@apache.org> 于2022年3月31日周四 01:01写道:
> >
> > > +1 (binding)
> > >
> > > Verified the following:
> > >
> > >    - shasums
> > >    - gpg signatures
> > >    - source does not contain any binaries
> > >    - built from source
> > >    - deployed via helm after adding the distribution webserver endpoint
> > as
> > >    a helm registry
> > >    - all relevant files have license headers
> > >
> > >
> > > On Wed, Mar 30, 2022 at 4:39 PM Gyula Fóra <gy...@apache.org> wrote:
> > >
> > > > Hi everyone,
> > > >
> > > > Please review and vote on the release candidate #3 for the version
> > 0.1.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 is staged on github [7]
> > > >
> > > > All artifacts are signed with the key
> > > > 0B4A34ADDFFA2BB54EB720B221F06303B87DAFF1 [3]
> > > >
> > > > Other links for your review:
> > > > * JIRA release notes [4]
> > > > * source code tag "release-0.1.0-rc3" [5]
> > > > * PR to update the website Downloads page to include Kubernetes
> > Operator
> > > > links [6]
> > > >
> > > > **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
> > > > <
> > > >
> > >
> >
> https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release
> > > > >
> > > > .
> > > > Note that you don't need to verify everything yourself, but please
> make
> > > > note of what you have tested together with your +- vote.
> > > >
> > > > Thanks,
> > > > Gyula
> > > >
> > > > [1]
> > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3/
> > > > [2]
> > > >
> > https://repository.apache.org/content/repositories/orgapacheflink-1492/
> > > > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > > > [4]
> > > >
> > > >
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351499
> > > > [5]
> > > >
> > >
> >
> https://github.com/apache/flink-kubernetes-operator/tree/release-0.1.0-rc3
> > > > [6] https://github.com/apache/flink-web/pull/519
> > > > [7] ghcr.io/apache/flink-kubernetes-operator:2c166e3
> > > >
> > >
> >
>

Re: [VOTE] Apache Flink Kubernetes Operator Release 0.1.0, release candidate #3

Posted by Gyula Fóra <gy...@gmail.com>.
+1 (binding)

Verified the following:

   - Verified source distribution do not contain binaries
   - Verified LICENSES and NOTICES
   - Built from source, verified maven and helm chart versions
   - Verified helm chart points to correct docker image and deploys it by
   default
   - Verified maven artifacts correctly deployed to staging repository and
   contents
   - Verified staging repo as helm repo, helm installation and
   basic/checkpointing examples with upgrades and manual savepoints
   - Verified operator logs and cr status does not contain unexpected
   things

Cheers,
Gyula

On Thu, Mar 31, 2022 at 10:54 AM Yang Wang <da...@gmail.com> wrote:

> +1 (non-binding)
>
> Verified via the following steps:
>
> * Verify checksums and GPG signatures
> * Verify that the source distributions do not contain any binaries
> * Build source distribution successfully
> * Verify all the POM version is 0.1.0
>
> * License check, the jars bundled in docker image and maven artifacts have
> correct NOTICE and licenses
>
> # Functionality verification
> * Install flink-kubernetes-operator via helm
>     - helm repo add flink-kubernetes-operator-0.1.0-rc3
>
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3
>     - helm install flink-kubernetes-operator
> flink-kubernetes-operator-0.1.0-rc3/flink-kubernetes-operator
>
> * Apply a new FlinkDeployment CR with HA and ingress enabled, Flink webUI
> normal
>     - kubectl apply -f
>
> https://raw.githubusercontent.com/apache/flink-kubernetes-operator/release-0.1/e2e-tests/data/cr.yaml
>
> * Upgrade FlinkDeployment, new job parallelism takes effect and recover
> from latest checkpoint
>     - kubectl patch flinkdep flink-example-statemachine --type merge
> --patch '{"spec":{"job": {"parallelism": 1 } } }'
>
> * Verify manual savepoint trigger
>     - kubectl patch flinkdep flink-example-statemachine --type merge
> --patch '{"spec":{"job": {"savepointTriggerNonce": 1 } } }'
>
> * Suspend a FlinkDeployment
>     - kubectl patch flinkdep flink-example-statemachine --type merge
> --patch '{"spec":{"job": {"state": "suspended" } } }'
>
>
> Best,
> Yang
>
> Márton Balassi <mb...@apache.org> 于2022年3月31日周四 01:01写道:
>
> > +1 (binding)
> >
> > Verified the following:
> >
> >    - shasums
> >    - gpg signatures
> >    - source does not contain any binaries
> >    - built from source
> >    - deployed via helm after adding the distribution webserver endpoint
> as
> >    a helm registry
> >    - all relevant files have license headers
> >
> >
> > On Wed, Mar 30, 2022 at 4:39 PM Gyula Fóra <gy...@apache.org> wrote:
> >
> > > Hi everyone,
> > >
> > > Please review and vote on the release candidate #3 for the version
> 0.1.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 is staged on github [7]
> > >
> > > All artifacts are signed with the key
> > > 0B4A34ADDFFA2BB54EB720B221F06303B87DAFF1 [3]
> > >
> > > Other links for your review:
> > > * JIRA release notes [4]
> > > * source code tag "release-0.1.0-rc3" [5]
> > > * PR to update the website Downloads page to include Kubernetes
> Operator
> > > links [6]
> > >
> > > **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
> > > <
> > >
> >
> https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release
> > > >
> > > .
> > > Note that you don't need to verify everything yourself, but please make
> > > note of what you have tested together with your +- vote.
> > >
> > > Thanks,
> > > Gyula
> > >
> > > [1]
> > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3/
> > > [2]
> > >
> https://repository.apache.org/content/repositories/orgapacheflink-1492/
> > > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > > [4]
> > >
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351499
> > > [5]
> > >
> >
> https://github.com/apache/flink-kubernetes-operator/tree/release-0.1.0-rc3
> > > [6] https://github.com/apache/flink-web/pull/519
> > > [7] ghcr.io/apache/flink-kubernetes-operator:2c166e3
> > >
> >
>

Re: [VOTE] Apache Flink Kubernetes Operator Release 0.1.0, release candidate #3

Posted by Yang Wang <da...@gmail.com>.
+1 (non-binding)

Verified via the following steps:

* Verify checksums and GPG signatures
* Verify that the source distributions do not contain any binaries
* Build source distribution successfully
* Verify all the POM version is 0.1.0

* License check, the jars bundled in docker image and maven artifacts have
correct NOTICE and licenses

# Functionality verification
* Install flink-kubernetes-operator via helm
    - helm repo add flink-kubernetes-operator-0.1.0-rc3
https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3
    - helm install flink-kubernetes-operator
flink-kubernetes-operator-0.1.0-rc3/flink-kubernetes-operator

* Apply a new FlinkDeployment CR with HA and ingress enabled, Flink webUI
normal
    - kubectl apply -f
https://raw.githubusercontent.com/apache/flink-kubernetes-operator/release-0.1/e2e-tests/data/cr.yaml

* Upgrade FlinkDeployment, new job parallelism takes effect and recover
from latest checkpoint
    - kubectl patch flinkdep flink-example-statemachine --type merge
--patch '{"spec":{"job": {"parallelism": 1 } } }'

* Verify manual savepoint trigger
    - kubectl patch flinkdep flink-example-statemachine --type merge
--patch '{"spec":{"job": {"savepointTriggerNonce": 1 } } }'

* Suspend a FlinkDeployment
    - kubectl patch flinkdep flink-example-statemachine --type merge
--patch '{"spec":{"job": {"state": "suspended" } } }'


Best,
Yang

Márton Balassi <mb...@apache.org> 于2022年3月31日周四 01:01写道:

> +1 (binding)
>
> Verified the following:
>
>    - shasums
>    - gpg signatures
>    - source does not contain any binaries
>    - built from source
>    - deployed via helm after adding the distribution webserver endpoint as
>    a helm registry
>    - all relevant files have license headers
>
>
> On Wed, Mar 30, 2022 at 4:39 PM Gyula Fóra <gy...@apache.org> wrote:
>
> > Hi everyone,
> >
> > Please review and vote on the release candidate #3 for the version 0.1.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 is staged on github [7]
> >
> > All artifacts are signed with the key
> > 0B4A34ADDFFA2BB54EB720B221F06303B87DAFF1 [3]
> >
> > Other links for your review:
> > * JIRA release notes [4]
> > * source code tag "release-0.1.0-rc3" [5]
> > * PR to update the website Downloads page to include Kubernetes Operator
> > links [6]
> >
> > **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
> > <
> >
> https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release
> > >
> > .
> > Note that you don't need to verify everything yourself, but please make
> > note of what you have tested together with your +- vote.
> >
> > Thanks,
> > Gyula
> >
> > [1]
> >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3/
> > [2]
> > https://repository.apache.org/content/repositories/orgapacheflink-1492/
> > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > [4]
> >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351499
> > [5]
> >
> https://github.com/apache/flink-kubernetes-operator/tree/release-0.1.0-rc3
> > [6] https://github.com/apache/flink-web/pull/519
> > [7] ghcr.io/apache/flink-kubernetes-operator:2c166e3
> >
>

Re: [VOTE] Apache Flink Kubernetes Operator Release 0.1.0, release candidate #3

Posted by Márton Balassi <mb...@apache.org>.
+1 (binding)

Verified the following:

   - shasums
   - gpg signatures
   - source does not contain any binaries
   - built from source
   - deployed via helm after adding the distribution webserver endpoint as
   a helm registry
   - all relevant files have license headers


On Wed, Mar 30, 2022 at 4:39 PM Gyula Fóra <gy...@apache.org> wrote:

> Hi everyone,
>
> Please review and vote on the release candidate #3 for the version 0.1.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 is staged on github [7]
>
> All artifacts are signed with the key
> 0B4A34ADDFFA2BB54EB720B221F06303B87DAFF1 [3]
>
> Other links for your review:
> * JIRA release notes [4]
> * source code tag "release-0.1.0-rc3" [5]
> * PR to update the website Downloads page to include Kubernetes Operator
> links [6]
>
> **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
> <
> https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release
> >
> .
> Note that you don't need to verify everything yourself, but please make
> note of what you have tested together with your +- vote.
>
> Thanks,
> Gyula
>
> [1]
>
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3/
> [2]
> https://repository.apache.org/content/repositories/orgapacheflink-1492/
> [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> [4]
>
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351499
> [5]
> https://github.com/apache/flink-kubernetes-operator/tree/release-0.1.0-rc3
> [6] https://github.com/apache/flink-web/pull/519
> [7] ghcr.io/apache/flink-kubernetes-operator:2c166e3
>

Re: [VOTE] Apache Flink Kubernetes Operator Release 0.1.0, release candidate #3

Posted by Aitozi <gj...@gmail.com>.
+1 (non-binding)

- verified building and installing operator
- verified session cluster and application job submit and upgrade

Cheers,
Aitozi.

Xintong Song <to...@gmail.com> 于2022年4月1日周五 19:12写道:

> Thanks Gyula, the doc pr LGTM.
>
> +1 (binding)
>
> - verified checksum & signature
> - installed via helm and tried example CRs
> - viewed online documentations
>
> Thank you~
>
> Xintong Song
>
>
>
> On Fri, Apr 1, 2022 at 4:56 PM Gyula Fóra <gy...@gmail.com> wrote:
>
> > Xintong,
> >
> > You are right, this is indeed missing from the docs :) I will be pushing
> > some quickstart improvements once the release is finalized and will
> include
> > it in there.
> > Please check this doc change and let me know if it clarifies it to a good
> > degree: https://github.com/apache/flink-kubernetes-operator/pull/143
> >
> > As long as we (in addition to 0.1.0)  also publish the commit tag to
> > Dockerhub, switching over would be as simple as: --set
> > image.repository=apache/flink-kubernetes-operator Thanks for the feedback
> > again!
> >
> > Cheers
> > Gyula
> >
> > On Fri, Apr 1, 2022 at 10:30 AM Xintong Song <to...@gmail.com>
> > wrote:
> >
> > > Thanks for the quick response, Gyula.
> > >
> > > I'm fine with having Helm chart pointing to ghcr.io.
> > >
> > > It should be clear from the readme how to use the dockerhub image
> > instead.
> > > >
> > >
> > > Could you point me where to find these instructions? I don't find them
> in
> > > neither the Quick Start nor the Helm page in the documentation. It
> would
> > be
> > > nice to have these instructions somewhere really easy to find. Network
> > > connectivity to ghcr.io from China is super slow. I've been pulling
> the
> > > image for more than 1h and it's still not completed. Shouldn't be a
> > release
> > > blocker though.
> > >
> > > I'll continue verifying this release candidate and vote once I'm done.
> > >
> > > Thank you~
> > >
> > > Xintong Song
> > >
> > >
> > >
> > > On Fri, Apr 1, 2022 at 4:01 PM Gyula Fóra <gy...@gmail.com>
> wrote:
> > >
> > > > Hi Xintong,
> > > > I have been contemplating the same for a while.
> > > > The final docker image that will be uploaded to dockerhub should be
> the
> > > > same image as the one on ghcr.io (re-tagged and reuploaded).
> > > >
> > > > We made the decision to try to keep the integrity of the RC and the
> > final
> > > > release and not make any changes afterwards.
> > > > I think it is fine to have the Helm chart point to ghcr.io image as
> it
> > > is
> > > > also owned by the project. It also makes image verification easier
> > during
> > > > testing.
> > > >
> > > > It should be clear from the readme how to use the dockerhub image
> > > instead.
> > > >
> > > > If we see any problems with this in the future we could try to adapt
> > your
> > > > suggestion.
> > > >
> > > > Cheers,
> > > > Gyula
> > > >
> > > >
> > > > On Fri, Apr 1, 2022 at 9:51 AM Xintong Song <to...@gmail.com>
> > > wrote:
> > > >
> > > > > I have a concern, which I'm not sure whether it should be a release
> > > > > blocker.
> > > > >
> > > > > While trying to install the operator via helm, I noticed that the
> > image
> > > > is
> > > > > pulled from `ghcr.io/apache/flink-kubernetes-operator`
> <http://ghcr.io/apache/flink-kubernetes-operator>
> > <http://ghcr.io/apache/flink-kubernetes-operator>
> > > <http://ghcr.io/apache/flink-kubernetes-operator>
> > > > <http://ghcr.io/apache/flink-kubernetes-operator>
> > > > > <http://ghcr.io/apache/flink-kubernetes-operator>. IIUC, in one of
> > the
> > > > > previous voting threads it is mentioned that we are not using `
> > ghcr.io
> > > `
> > > > > for
> > > > > the final release, but will use DockerHub instead. Then how are we
> > > going
> > > > to
> > > > > change from `ghcr.io` to `DockerHub` without modifying the staged
> > > > release
> > > > > candidate? Maybe we should point to the DockerHub image (which does
> > not
> > > > > exist yet) in `values.yaml` and overwrite it to the temporal `
> > ghcr.io`
> > > > > image only for verifying the release candidate?
> > > > >
> > > > > Thank you~
> > > > >
> > > > > Xintong Song
> > > > >
> > > > >
> > > > >
> > > > > On Fri, Apr 1, 2022 at 1:02 PM Thomas Weise <th...@apache.org>
> wrote:
> > > > >
> > > > > > +1 (binding)
> > > > > >
> > > > > > * installed from staged helm chart and run example
> > > > > > * built from source
> > > > > >
> > > > > >
> > > > > > On Wed, Mar 30, 2022 at 7:39 AM Gyula Fóra <gy...@apache.org>
> > > wrote:
> > > > > >
> > > > > > > Hi everyone,
> > > > > > >
> > > > > > > Please review and vote on the release candidate #3 for the
> > version
> > > > > 0.1.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 is staged on github [7]
> > > > > > >
> > > > > > > All artifacts are signed with the key
> > > > > > > 0B4A34ADDFFA2BB54EB720B221F06303B87DAFF1 [3]
> > > > > > >
> > > > > > > Other links for your review:
> > > > > > > * JIRA release notes [4]
> > > > > > > * source code tag "release-0.1.0-rc3" [5]
> > > > > > > * PR to update the website Downloads page to include Kubernetes
> > > > > Operator
> > > > > > > links [6]
> > > > > > >
> > > > > > > **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
> > > > > > > <
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release
> > > > > > > >
> > > > > > > .
> > > > > > > Note that you don't need to verify everything yourself, but
> > please
> > > > make
> > > > > > > note of what you have tested together with your +- vote.
> > > > > > >
> > > > > > > Thanks,
> > > > > > > Gyula
> > > > > > >
> > > > > > > [1]
> > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3/
> > > > > > > [2]
> > > > > > >
> > > > >
> > >
> https://repository.apache.org/content/repositories/orgapacheflink-1492/
> > > > > > > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > > > > > > [4]
> > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351499
> > > > > > > [5]
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://github.com/apache/flink-kubernetes-operator/tree/release-0.1.0-rc3
> > > > > > > [6] https://github.com/apache/flink-web/pull/519
> > > > > > > [7] ghcr.io/apache/flink-kubernetes-operator:2c166e3
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>

Re: [VOTE] Apache Flink Kubernetes Operator Release 0.1.0, release candidate #3

Posted by Xintong Song <to...@gmail.com>.
Thanks Gyula, the doc pr LGTM.

+1 (binding)

- verified checksum & signature
- installed via helm and tried example CRs
- viewed online documentations

Thank you~

Xintong Song



On Fri, Apr 1, 2022 at 4:56 PM Gyula Fóra <gy...@gmail.com> wrote:

> Xintong,
>
> You are right, this is indeed missing from the docs :) I will be pushing
> some quickstart improvements once the release is finalized and will include
> it in there.
> Please check this doc change and let me know if it clarifies it to a good
> degree: https://github.com/apache/flink-kubernetes-operator/pull/143
>
> As long as we (in addition to 0.1.0)  also publish the commit tag to
> Dockerhub, switching over would be as simple as: --set
> image.repository=apache/flink-kubernetes-operator Thanks for the feedback
> again!
>
> Cheers
> Gyula
>
> On Fri, Apr 1, 2022 at 10:30 AM Xintong Song <to...@gmail.com>
> wrote:
>
> > Thanks for the quick response, Gyula.
> >
> > I'm fine with having Helm chart pointing to ghcr.io.
> >
> > It should be clear from the readme how to use the dockerhub image
> instead.
> > >
> >
> > Could you point me where to find these instructions? I don't find them in
> > neither the Quick Start nor the Helm page in the documentation. It would
> be
> > nice to have these instructions somewhere really easy to find. Network
> > connectivity to ghcr.io from China is super slow. I've been pulling the
> > image for more than 1h and it's still not completed. Shouldn't be a
> release
> > blocker though.
> >
> > I'll continue verifying this release candidate and vote once I'm done.
> >
> > Thank you~
> >
> > Xintong Song
> >
> >
> >
> > On Fri, Apr 1, 2022 at 4:01 PM Gyula Fóra <gy...@gmail.com> wrote:
> >
> > > Hi Xintong,
> > > I have been contemplating the same for a while.
> > > The final docker image that will be uploaded to dockerhub should be the
> > > same image as the one on ghcr.io (re-tagged and reuploaded).
> > >
> > > We made the decision to try to keep the integrity of the RC and the
> final
> > > release and not make any changes afterwards.
> > > I think it is fine to have the Helm chart point to ghcr.io image as it
> > is
> > > also owned by the project. It also makes image verification easier
> during
> > > testing.
> > >
> > > It should be clear from the readme how to use the dockerhub image
> > instead.
> > >
> > > If we see any problems with this in the future we could try to adapt
> your
> > > suggestion.
> > >
> > > Cheers,
> > > Gyula
> > >
> > >
> > > On Fri, Apr 1, 2022 at 9:51 AM Xintong Song <to...@gmail.com>
> > wrote:
> > >
> > > > I have a concern, which I'm not sure whether it should be a release
> > > > blocker.
> > > >
> > > > While trying to install the operator via helm, I noticed that the
> image
> > > is
> > > > pulled from `ghcr.io/apache/flink-kubernetes-operator`
> <http://ghcr.io/apache/flink-kubernetes-operator>
> > <http://ghcr.io/apache/flink-kubernetes-operator>
> > > <http://ghcr.io/apache/flink-kubernetes-operator>
> > > > <http://ghcr.io/apache/flink-kubernetes-operator>. IIUC, in one of
> the
> > > > previous voting threads it is mentioned that we are not using `
> ghcr.io
> > `
> > > > for
> > > > the final release, but will use DockerHub instead. Then how are we
> > going
> > > to
> > > > change from `ghcr.io` to `DockerHub` without modifying the staged
> > > release
> > > > candidate? Maybe we should point to the DockerHub image (which does
> not
> > > > exist yet) in `values.yaml` and overwrite it to the temporal `
> ghcr.io`
> > > > image only for verifying the release candidate?
> > > >
> > > > Thank you~
> > > >
> > > > Xintong Song
> > > >
> > > >
> > > >
> > > > On Fri, Apr 1, 2022 at 1:02 PM Thomas Weise <th...@apache.org> wrote:
> > > >
> > > > > +1 (binding)
> > > > >
> > > > > * installed from staged helm chart and run example
> > > > > * built from source
> > > > >
> > > > >
> > > > > On Wed, Mar 30, 2022 at 7:39 AM Gyula Fóra <gy...@apache.org>
> > wrote:
> > > > >
> > > > > > Hi everyone,
> > > > > >
> > > > > > Please review and vote on the release candidate #3 for the
> version
> > > > 0.1.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 is staged on github [7]
> > > > > >
> > > > > > All artifacts are signed with the key
> > > > > > 0B4A34ADDFFA2BB54EB720B221F06303B87DAFF1 [3]
> > > > > >
> > > > > > Other links for your review:
> > > > > > * JIRA release notes [4]
> > > > > > * source code tag "release-0.1.0-rc3" [5]
> > > > > > * PR to update the website Downloads page to include Kubernetes
> > > > Operator
> > > > > > links [6]
> > > > > >
> > > > > > **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
> > > > > > <
> > > > > >
> > > > >
> > > >
> > >
> >
> https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release
> > > > > > >
> > > > > > .
> > > > > > Note that you don't need to verify everything yourself, but
> please
> > > make
> > > > > > note of what you have tested together with your +- vote.
> > > > > >
> > > > > > Thanks,
> > > > > > Gyula
> > > > > >
> > > > > > [1]
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3/
> > > > > > [2]
> > > > > >
> > > >
> > https://repository.apache.org/content/repositories/orgapacheflink-1492/
> > > > > > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > > > > > [4]
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351499
> > > > > > [5]
> > > > > >
> > > > >
> > > >
> > >
> >
> https://github.com/apache/flink-kubernetes-operator/tree/release-0.1.0-rc3
> > > > > > [6] https://github.com/apache/flink-web/pull/519
> > > > > > [7] ghcr.io/apache/flink-kubernetes-operator:2c166e3
> > > > > >
> > > > >
> > > >
> > >
> >
>

Re: [VOTE] Apache Flink Kubernetes Operator Release 0.1.0, release candidate #3

Posted by Gyula Fóra <gy...@gmail.com>.
Xintong,

You are right, this is indeed missing from the docs :) I will be pushing
some quickstart improvements once the release is finalized and will include
it in there.
Please check this doc change and let me know if it clarifies it to a good
degree: https://github.com/apache/flink-kubernetes-operator/pull/143

As long as we (in addition to 0.1.0)  also publish the commit tag to
Dockerhub, switching over would be as simple as: --set
image.repository=apache/flink-kubernetes-operator Thanks for the feedback
again!

Cheers
Gyula

On Fri, Apr 1, 2022 at 10:30 AM Xintong Song <to...@gmail.com> wrote:

> Thanks for the quick response, Gyula.
>
> I'm fine with having Helm chart pointing to ghcr.io.
>
> It should be clear from the readme how to use the dockerhub image instead.
> >
>
> Could you point me where to find these instructions? I don't find them in
> neither the Quick Start nor the Helm page in the documentation. It would be
> nice to have these instructions somewhere really easy to find. Network
> connectivity to ghcr.io from China is super slow. I've been pulling the
> image for more than 1h and it's still not completed. Shouldn't be a release
> blocker though.
>
> I'll continue verifying this release candidate and vote once I'm done.
>
> Thank you~
>
> Xintong Song
>
>
>
> On Fri, Apr 1, 2022 at 4:01 PM Gyula Fóra <gy...@gmail.com> wrote:
>
> > Hi Xintong,
> > I have been contemplating the same for a while.
> > The final docker image that will be uploaded to dockerhub should be the
> > same image as the one on ghcr.io (re-tagged and reuploaded).
> >
> > We made the decision to try to keep the integrity of the RC and the final
> > release and not make any changes afterwards.
> > I think it is fine to have the Helm chart point to ghcr.io image as it
> is
> > also owned by the project. It also makes image verification easier during
> > testing.
> >
> > It should be clear from the readme how to use the dockerhub image
> instead.
> >
> > If we see any problems with this in the future we could try to adapt your
> > suggestion.
> >
> > Cheers,
> > Gyula
> >
> >
> > On Fri, Apr 1, 2022 at 9:51 AM Xintong Song <to...@gmail.com>
> wrote:
> >
> > > I have a concern, which I'm not sure whether it should be a release
> > > blocker.
> > >
> > > While trying to install the operator via helm, I noticed that the image
> > is
> > > pulled from `ghcr.io/apache/flink-kubernetes-operator`
> <http://ghcr.io/apache/flink-kubernetes-operator>
> > <http://ghcr.io/apache/flink-kubernetes-operator>
> > > <http://ghcr.io/apache/flink-kubernetes-operator>. IIUC, in one of the
> > > previous voting threads it is mentioned that we are not using `ghcr.io
> `
> > > for
> > > the final release, but will use DockerHub instead. Then how are we
> going
> > to
> > > change from `ghcr.io` to `DockerHub` without modifying the staged
> > release
> > > candidate? Maybe we should point to the DockerHub image (which does not
> > > exist yet) in `values.yaml` and overwrite it to the temporal `ghcr.io`
> > > image only for verifying the release candidate?
> > >
> > > Thank you~
> > >
> > > Xintong Song
> > >
> > >
> > >
> > > On Fri, Apr 1, 2022 at 1:02 PM Thomas Weise <th...@apache.org> wrote:
> > >
> > > > +1 (binding)
> > > >
> > > > * installed from staged helm chart and run example
> > > > * built from source
> > > >
> > > >
> > > > On Wed, Mar 30, 2022 at 7:39 AM Gyula Fóra <gy...@apache.org>
> wrote:
> > > >
> > > > > Hi everyone,
> > > > >
> > > > > Please review and vote on the release candidate #3 for the version
> > > 0.1.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 is staged on github [7]
> > > > >
> > > > > All artifacts are signed with the key
> > > > > 0B4A34ADDFFA2BB54EB720B221F06303B87DAFF1 [3]
> > > > >
> > > > > Other links for your review:
> > > > > * JIRA release notes [4]
> > > > > * source code tag "release-0.1.0-rc3" [5]
> > > > > * PR to update the website Downloads page to include Kubernetes
> > > Operator
> > > > > links [6]
> > > > >
> > > > > **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
> > > > > <
> > > > >
> > > >
> > >
> >
> https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release
> > > > > >
> > > > > .
> > > > > Note that you don't need to verify everything yourself, but please
> > make
> > > > > note of what you have tested together with your +- vote.
> > > > >
> > > > > Thanks,
> > > > > Gyula
> > > > >
> > > > > [1]
> > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3/
> > > > > [2]
> > > > >
> > >
> https://repository.apache.org/content/repositories/orgapacheflink-1492/
> > > > > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > > > > [4]
> > > > >
> > > > >
> > > >
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351499
> > > > > [5]
> > > > >
> > > >
> > >
> >
> https://github.com/apache/flink-kubernetes-operator/tree/release-0.1.0-rc3
> > > > > [6] https://github.com/apache/flink-web/pull/519
> > > > > [7] ghcr.io/apache/flink-kubernetes-operator:2c166e3
> > > > >
> > > >
> > >
> >
>

Re: [VOTE] Apache Flink Kubernetes Operator Release 0.1.0, release candidate #3

Posted by Xintong Song <to...@gmail.com>.
Thanks for the quick response, Gyula.

I'm fine with having Helm chart pointing to ghcr.io.

It should be clear from the readme how to use the dockerhub image instead.
>

Could you point me where to find these instructions? I don't find them in
neither the Quick Start nor the Helm page in the documentation. It would be
nice to have these instructions somewhere really easy to find. Network
connectivity to ghcr.io from China is super slow. I've been pulling the
image for more than 1h and it's still not completed. Shouldn't be a release
blocker though.

I'll continue verifying this release candidate and vote once I'm done.

Thank you~

Xintong Song



On Fri, Apr 1, 2022 at 4:01 PM Gyula Fóra <gy...@gmail.com> wrote:

> Hi Xintong,
> I have been contemplating the same for a while.
> The final docker image that will be uploaded to dockerhub should be the
> same image as the one on ghcr.io (re-tagged and reuploaded).
>
> We made the decision to try to keep the integrity of the RC and the final
> release and not make any changes afterwards.
> I think it is fine to have the Helm chart point to ghcr.io image as it is
> also owned by the project. It also makes image verification easier during
> testing.
>
> It should be clear from the readme how to use the dockerhub image instead.
>
> If we see any problems with this in the future we could try to adapt your
> suggestion.
>
> Cheers,
> Gyula
>
>
> On Fri, Apr 1, 2022 at 9:51 AM Xintong Song <to...@gmail.com> wrote:
>
> > I have a concern, which I'm not sure whether it should be a release
> > blocker.
> >
> > While trying to install the operator via helm, I noticed that the image
> is
> > pulled from `ghcr.io/apache/flink-kubernetes-operator`
> <http://ghcr.io/apache/flink-kubernetes-operator>
> > <http://ghcr.io/apache/flink-kubernetes-operator>. IIUC, in one of the
> > previous voting threads it is mentioned that we are not using `ghcr.io`
> > for
> > the final release, but will use DockerHub instead. Then how are we going
> to
> > change from `ghcr.io` to `DockerHub` without modifying the staged
> release
> > candidate? Maybe we should point to the DockerHub image (which does not
> > exist yet) in `values.yaml` and overwrite it to the temporal `ghcr.io`
> > image only for verifying the release candidate?
> >
> > Thank you~
> >
> > Xintong Song
> >
> >
> >
> > On Fri, Apr 1, 2022 at 1:02 PM Thomas Weise <th...@apache.org> wrote:
> >
> > > +1 (binding)
> > >
> > > * installed from staged helm chart and run example
> > > * built from source
> > >
> > >
> > > On Wed, Mar 30, 2022 at 7:39 AM Gyula Fóra <gy...@apache.org> wrote:
> > >
> > > > Hi everyone,
> > > >
> > > > Please review and vote on the release candidate #3 for the version
> > 0.1.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 is staged on github [7]
> > > >
> > > > All artifacts are signed with the key
> > > > 0B4A34ADDFFA2BB54EB720B221F06303B87DAFF1 [3]
> > > >
> > > > Other links for your review:
> > > > * JIRA release notes [4]
> > > > * source code tag "release-0.1.0-rc3" [5]
> > > > * PR to update the website Downloads page to include Kubernetes
> > Operator
> > > > links [6]
> > > >
> > > > **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
> > > > <
> > > >
> > >
> >
> https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release
> > > > >
> > > > .
> > > > Note that you don't need to verify everything yourself, but please
> make
> > > > note of what you have tested together with your +- vote.
> > > >
> > > > Thanks,
> > > > Gyula
> > > >
> > > > [1]
> > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3/
> > > > [2]
> > > >
> > https://repository.apache.org/content/repositories/orgapacheflink-1492/
> > > > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > > > [4]
> > > >
> > > >
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351499
> > > > [5]
> > > >
> > >
> >
> https://github.com/apache/flink-kubernetes-operator/tree/release-0.1.0-rc3
> > > > [6] https://github.com/apache/flink-web/pull/519
> > > > [7] ghcr.io/apache/flink-kubernetes-operator:2c166e3
> > > >
> > >
> >
>

Re: [VOTE] Apache Flink Kubernetes Operator Release 0.1.0, release candidate #3

Posted by Gyula Fóra <gy...@gmail.com>.
Hi Xintong,
I have been contemplating the same for a while.
The final docker image that will be uploaded to dockerhub should be the
same image as the one on ghcr.io (re-tagged and reuploaded).

We made the decision to try to keep the integrity of the RC and the final
release and not make any changes afterwards.
I think it is fine to have the Helm chart point to ghcr.io image as it is
also owned by the project. It also makes image verification easier during
testing.

It should be clear from the readme how to use the dockerhub image instead.

If we see any problems with this in the future we could try to adapt your
suggestion.

Cheers,
Gyula


On Fri, Apr 1, 2022 at 9:51 AM Xintong Song <to...@gmail.com> wrote:

> I have a concern, which I'm not sure whether it should be a release
> blocker.
>
> While trying to install the operator via helm, I noticed that the image is
> pulled from `ghcr.io/apache/flink-kubernetes-operator`
> <http://ghcr.io/apache/flink-kubernetes-operator>. IIUC, in one of the
> previous voting threads it is mentioned that we are not using `ghcr.io`
> for
> the final release, but will use DockerHub instead. Then how are we going to
> change from `ghcr.io` to `DockerHub` without modifying the staged release
> candidate? Maybe we should point to the DockerHub image (which does not
> exist yet) in `values.yaml` and overwrite it to the temporal `ghcr.io`
> image only for verifying the release candidate?
>
> Thank you~
>
> Xintong Song
>
>
>
> On Fri, Apr 1, 2022 at 1:02 PM Thomas Weise <th...@apache.org> wrote:
>
> > +1 (binding)
> >
> > * installed from staged helm chart and run example
> > * built from source
> >
> >
> > On Wed, Mar 30, 2022 at 7:39 AM Gyula Fóra <gy...@apache.org> wrote:
> >
> > > Hi everyone,
> > >
> > > Please review and vote on the release candidate #3 for the version
> 0.1.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 is staged on github [7]
> > >
> > > All artifacts are signed with the key
> > > 0B4A34ADDFFA2BB54EB720B221F06303B87DAFF1 [3]
> > >
> > > Other links for your review:
> > > * JIRA release notes [4]
> > > * source code tag "release-0.1.0-rc3" [5]
> > > * PR to update the website Downloads page to include Kubernetes
> Operator
> > > links [6]
> > >
> > > **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
> > > <
> > >
> >
> https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release
> > > >
> > > .
> > > Note that you don't need to verify everything yourself, but please make
> > > note of what you have tested together with your +- vote.
> > >
> > > Thanks,
> > > Gyula
> > >
> > > [1]
> > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3/
> > > [2]
> > >
> https://repository.apache.org/content/repositories/orgapacheflink-1492/
> > > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > > [4]
> > >
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351499
> > > [5]
> > >
> >
> https://github.com/apache/flink-kubernetes-operator/tree/release-0.1.0-rc3
> > > [6] https://github.com/apache/flink-web/pull/519
> > > [7] ghcr.io/apache/flink-kubernetes-operator:2c166e3
> > >
> >
>

Re: [VOTE] Apache Flink Kubernetes Operator Release 0.1.0, release candidate #3

Posted by Xintong Song <to...@gmail.com>.
I have a concern, which I'm not sure whether it should be a release blocker.

While trying to install the operator via helm, I noticed that the image is
pulled from `ghcr.io/apache/flink-kubernetes-operator`. IIUC, in one of the
previous voting threads it is mentioned that we are not using `ghcr.io` for
the final release, but will use DockerHub instead. Then how are we going to
change from `ghcr.io` to `DockerHub` without modifying the staged release
candidate? Maybe we should point to the DockerHub image (which does not
exist yet) in `values.yaml` and overwrite it to the temporal `ghcr.io`
image only for verifying the release candidate?

Thank you~

Xintong Song



On Fri, Apr 1, 2022 at 1:02 PM Thomas Weise <th...@apache.org> wrote:

> +1 (binding)
>
> * installed from staged helm chart and run example
> * built from source
>
>
> On Wed, Mar 30, 2022 at 7:39 AM Gyula Fóra <gy...@apache.org> wrote:
>
> > Hi everyone,
> >
> > Please review and vote on the release candidate #3 for the version 0.1.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 is staged on github [7]
> >
> > All artifacts are signed with the key
> > 0B4A34ADDFFA2BB54EB720B221F06303B87DAFF1 [3]
> >
> > Other links for your review:
> > * JIRA release notes [4]
> > * source code tag "release-0.1.0-rc3" [5]
> > * PR to update the website Downloads page to include Kubernetes Operator
> > links [6]
> >
> > **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
> > <
> >
> https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release
> > >
> > .
> > Note that you don't need to verify everything yourself, but please make
> > note of what you have tested together with your +- vote.
> >
> > Thanks,
> > Gyula
> >
> > [1]
> >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3/
> > [2]
> > https://repository.apache.org/content/repositories/orgapacheflink-1492/
> > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > [4]
> >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351499
> > [5]
> >
> https://github.com/apache/flink-kubernetes-operator/tree/release-0.1.0-rc3
> > [6] https://github.com/apache/flink-web/pull/519
> > [7] ghcr.io/apache/flink-kubernetes-operator:2c166e3
> >
>

Re: [VOTE] Apache Flink Kubernetes Operator Release 0.1.0, release candidate #3

Posted by Thomas Weise <th...@apache.org>.
+1 (binding)

* installed from staged helm chart and run example
* built from source


On Wed, Mar 30, 2022 at 7:39 AM Gyula Fóra <gy...@apache.org> wrote:

> Hi everyone,
>
> Please review and vote on the release candidate #3 for the version 0.1.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 is staged on github [7]
>
> All artifacts are signed with the key
> 0B4A34ADDFFA2BB54EB720B221F06303B87DAFF1 [3]
>
> Other links for your review:
> * JIRA release notes [4]
> * source code tag "release-0.1.0-rc3" [5]
> * PR to update the website Downloads page to include Kubernetes Operator
> links [6]
>
> **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
> <
> https://cwiki.apache.org/confluence/display/FLINK/Verifying+a+Flink+Kubernetes+Operator+Release
> >
> .
> Note that you don't need to verify everything yourself, but please make
> note of what you have tested together with your +- vote.
>
> Thanks,
> Gyula
>
> [1]
>
> https://dist.apache.org/repos/dist/dev/flink/flink-kubernetes-operator-0.1.0-rc3/
> [2]
> https://repository.apache.org/content/repositories/orgapacheflink-1492/
> [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> [4]
>
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351499
> [5]
> https://github.com/apache/flink-kubernetes-operator/tree/release-0.1.0-rc3
> [6] https://github.com/apache/flink-web/pull/519
> [7] ghcr.io/apache/flink-kubernetes-operator:2c166e3
>