If we had closed the release today, this would still have broken all our users, correct?
Kenn On Mon, Oct 9, 2023 at 3:37 PM Anand Inguva via dev <[email protected]> wrote: > There was a regression[1] on fastavro latest release 1.8.4. Fix was merged > at https://github.com/apache/beam/pull/28896. The RC1 includes that > version in the range for fastavro[2]. I think we need to CP > https://github.com/apache/beam/pull/28896 to solve the fastavro > regression. > > [1] https://github.com/apache/beam/issues/28811 > [2] > https://github.com/apache/beam/blob/cd653e33b342bd09c76c2bbaca12597fec5b4a2c/sdks/python/setup.py#L245 > > > On Mon, Oct 9, 2023 at 3:15 PM Kenneth Knowles <[email protected]> wrote: > >> Ran a couple of Java pipelines "as a newb user" to make sure our >> instructions weren't out of date. There are some errors in the instructions >> but they don't have to do with this release. >> >> Re-ran mass_comment.py on https://github.com/apache/beam/pull/28663. >> There are enough red signals there that some triage is needed. Any help >> triaging would be appreciated. >> >> I'll close the vote once everything is run and examined. >> >> Kenn >> >> On Sat, Oct 7, 2023 at 9:58 AM Yi Hu via dev <[email protected]> wrote: >> >>> +1 (non-binding) Tested on Java IO load tests ( >>> https://github.com/bvolpato/DataflowTemplates/tree/56d18a31c1c95e58543d7a1656bd83d7e859b482/it) >>> BigQueryIO, TextIO, BigtableIO, SpannerIO on Dataflow legacy runner and >>> runner v2 >>> >>> >>> On Fri, Oct 6, 2023 at 3:23 PM Kenneth Knowles <[email protected]> wrote: >>> >>>> Additionally we need https://github.com/apache/beam/pull/28665/files >>>> in order to run GHA tests. >>>> >>>> On Fri, Oct 6, 2023 at 3:19 PM Kenneth Knowles <[email protected]> wrote: >>>> >>>>> That PR was prior to many cherry-picks so it is not the signal we >>>>> need. I have updated it to the tip of the release-2.51.0 branch. >>>>> >>>>> There were some post-commit tests involving JPMS that I believe need >>>>> https://github.com/apache/beam/pull/28726 to pass. >>>>> >>>>> Kenn >>>>> >>>>> On Fri, Oct 6, 2023 at 2:53 PM Valentyn Tymofieiev via dev < >>>>> [email protected]> wrote: >>>>> >>>>>> > PR to run tests against release branch [12]. >>>>>> >>>>>> https://github.com/apache/beam/pull/28663 is closed and test signal >>>>>> is no longer available. did all the tests pass? >>>>>> >>>>>> On Fri, Oct 6, 2023 at 5:32 AM Alexey Romanenko < >>>>>> [email protected]> wrote: >>>>>> >>>>>>> +1 (binding) >>>>>>> >>>>>>> — >>>>>>> Alexey >>>>>>> >>>>>>> > On 5 Oct 2023, at 18:38, Jean-Baptiste Onofré <[email protected]> >>>>>>> wrote: >>>>>>> > >>>>>>> > +1 (binding) >>>>>>> > >>>>>>> > Thanks ! >>>>>>> > Regards >>>>>>> > JB >>>>>>> > >>>>>>> > On Tue, Oct 3, 2023 at 7:58 PM Kenneth Knowles <[email protected]> >>>>>>> wrote: >>>>>>> >> >>>>>>> >> Hi everyone, >>>>>>> >> >>>>>>> >> Please review and vote on the release candidate #1 for the >>>>>>> version 2.51.0, as follows: >>>>>>> >> >>>>>>> >> [ ] +1, Approve the release >>>>>>> >> [ ] -1, Do not approve the release (please provide specific >>>>>>> comments) >>>>>>> >> >>>>>>> >> Reviewers are encouraged to test their own use cases with the >>>>>>> release candidate, and vote +1 if no issues are found. Only PMC member >>>>>>> votes will count towards the final vote, but votes from all community >>>>>>> members is encouraged and helpful for finding regressions; you can >>>>>>> either >>>>>>> test your own use cases or use cases from the validation sheet [10]. >>>>>>> >> >>>>>>> >> The complete staging area is available for your review, which >>>>>>> includes: >>>>>>> >> >>>>>>> >> GitHub Release notes [1], >>>>>>> >> the official Apache source release to be deployed to >>>>>>> dist.apache.org [2], which is signed with the key with fingerprint >>>>>>> FFFFFFFF [3], >>>>>>> >> all artifacts to be deployed to the Maven Central Repository [4], >>>>>>> >> source code tag "v1.2.3-RC3" [5], >>>>>>> >> website pull request listing the release [6], the blog post [6], >>>>>>> and publishing the API reference manual [7]. >>>>>>> >> Java artifacts were built with Gradle GRADLE_VERSION and >>>>>>> OpenJDK/Oracle JDK JDK_VERSION. >>>>>>> >> Python artifacts are deployed along with the source release to >>>>>>> the dist.apache.org [2] and PyPI[8]. >>>>>>> >> Go artifacts and documentation are available at pkg.go.dev [9] >>>>>>> >> Validation sheet with a tab for 1.2.3 release to help with >>>>>>> validation [10]. >>>>>>> >> Docker images published to Docker Hub [11]. >>>>>>> >> PR to run tests against release branch [12]. >>>>>>> >> >>>>>>> >> The vote will be open for at least 72 hours. It is adopted by >>>>>>> majority approval, with at least 3 PMC affirmative votes. >>>>>>> >> >>>>>>> >> For guidelines on how to try the release in your projects, check >>>>>>> out our blog post at >>>>>>> https://beam.apache.org/blog/validate-beam-release/. >>>>>>> >> >>>>>>> >> Thanks, >>>>>>> >> Kenn >>>>>>> >> >>>>>>> >> [1] https://github.com/apache/beam/milestone/15 >>>>>>> >> [2] https://dist.apache.org/repos/dist/dev/beam/2.51.0 >>>>>>> >> [3] https://dist.apache.org/repos/dist/release/beam/KEYS >>>>>>> >> [4] >>>>>>> https://repository.apache.org/content/repositories/orgapachebeam-1356/ >>>>>>> >> [5] https://github.com/apache/beam/tree/v2.51.0-RC1 >>>>>>> >> [6] https://github.com/apache/beam/pull/28800 >>>>>>> >> [7] https://github.com/apache/beam-site/pull/649 >>>>>>> >> [8] https://pypi.org/project/apache-beam/2.51.0rc1/ >>>>>>> >> [9] >>>>>>> https://pkg.go.dev/github.com/apache/beam/sdks/[email protected]/go/pkg/beam >>>>>>> >> [10] >>>>>>> https://docs.google.com/spreadsheets/d/1qk-N5vjXvbcEk68GjbkSZTR8AGqyNUM-oLFo_ZXBpJw/edit#gid=437054928 >>>>>>> >> [11] https://hub.docker.com/search?q=apache%2Fbeam&type=image >>>>>>> >> [12] https://github.com/apache/beam/pull/28663 >>>>>>> >>>>>>>
