-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release - September 2018 - name: 0.18.0 #5963
Comments
@aehlig Can you take care of the release 0.18.0?
I can take care of 0.18.0, provided it is OK that the baseline
will be cut a bit late: I'm on vacation September 1--8. But given
that 0.17.0 is delayed anyway, this should be fine.
…--
Klaus Aehlig
Google Germany GmbH, Erika-Mann-Str. 33, 80636 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Paul Terence Manicle, Halimah DeLaine Prado
|
Sounds good, we can target September 10 or 11. |
It's actually awesome, because I can finish my incompatible flags cls :))) |
Update(2018-09-10): verified that this is not a blocker. |
Looking through the last nighly builds and tests of downstream
projects, the most promising baseline seems to be 2e7ccbe
@laszlocsomor, this baseline also includes the update of protobuf to 3.6.1; do I
understand correctly, that this resolves your potential release blocker?
…--
Klaus Aehlig
Google Germany GmbH, Erika-Mann-Str. 33, 80636 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Paul Terence Manicle, Halimah DeLaine Prado
|
Another potential blocker: #6099 |
Some cherrypicks of 0.17 are not included in your baseline. |
0.18.0rc1 is available at https://releases.bazel.build/0.18.0/rc1/index.html
Will create another RC with the cherry-picks that went into 0.17.1 but are not in
the baseline soon.
…--
Klaus Aehlig
Google Germany GmbH, Erika-Mann-Str. 33, 80636 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Paul Terence Manicle, Halimah DeLaine Prado
|
Some cherrypicks of 0.17 are not included in your baseline.
According to the logs on #5059, we're talking about
9323c57 7eb9ea1 4566a42 1e9f0aa a572c1c 29f1de0
Is this correct?
Will also check if tomorrow's nighly is suitable as a base line. If not,
I will try to add those cherry-picks on the current base line.
…--
Klaus Aehlig
Google Germany GmbH, Erika-Mann-Str. 33, 80636 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Paul Terence Manicle, Halimah DeLaine Prado
|
/cc @cushon |
@mhlopko is informing me that there will be a new baseline |
Looks good. The only thing we have to look at is what's going on with Gerrit and jdk9 as this might be bazel related. So, let's push the baseline forward to c062b1f instead of starting with too many cherry-picks right from the beginning. |
Fantastic, thanks Klaus! |
Can you share the logs? |
> The only thing we have to look at is what's going on with Gerrit and jdk9
Can you share the logs?
The first error message is
```
ERROR: /var/lib/buildkite-agent/builds/buildkite-worker-ubuntu1804-java9-lfqk-1/bazel-downstream-projects/gerrit/java/com/google/gerrit/index/BUILD:8:1: Couldn't build file java/com/google/gerrit/index/libquery_exception-hjar.jar: Compiling Java headers java/com/google/gerrit/index/libquery_exception-hjar.jar (2 files) failed (Exit 1)
An exception has occurred in turbine.
java.lang.IllegalArgumentException: Could not find java.lang on bootclasspath
at com.google.turbine.binder.Binder.bindPackages(Binder.java:170)
at com.google.turbine.binder.Binder.bind(Binder.java:94)
at com.google.turbine.main.Main.compile(Main.java:97)
at com.google.devtools.build.java.turbine.Turbine.compile(Turbine.java:76)
at com.google.devtools.build.java.turbine.Turbine.compile(Turbine.java:69)
at com.google.devtools.build.java.turbine.Turbine.main(Turbine.java:46)
```
This only occurs on JDK9, and Gerrit is the only project we test with JDK9.
A potentially related commit is c1d8776.
@cushon, can you have a look as well? Thanks.
…--
Klaus Aehlig
Google Germany GmbH, Erika-Mann-Str. 33, 80636 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Paul Terence Manicle, Halimah DeLaine Prado
|
rc2 is available at https://releases.bazel.build/0.18.0/rc2/index.html Please test! |
@aehlig Thanks, I see that @meteorcloudy already created extra issue for this breakage. |
This is #6127. |
rc-4-9 pushed to chocolatey. |
Thank you for taking on this release. When can we expect this to be out? |
Thank you for taking on this release. When can we expect this to be out?
My plan was to release on Monday, unless more issues are reported.
…--
Klaus Aehlig
Google Germany GmbH, Erika-Mann-Str. 33, 80636 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Paul Terence Manicle, Halimah DeLaine Prado
|
I'm not sure what the process is, so just to be sure I'll say that I'll perform an incompatible change flip (#6380) in 0.19, so 0.18 should probably mention it as an incoming incompatible change. Or not, if we're not yet doing the new process. |
To suggest changes to the announcement, see: https://docs.google.com/document/d/1wDvulLlj4NAlPZamdlEVFORks3YXJonCjyuQMUQEmB0/edit |
That sounds like something I should have done instead spamming this thread :) Thanks Laurent :) |
As there are no new blockers, releasing now. |
Pushed to chocolatey. |
I think we need a patch release 0.18.1 to fix #6292. |
Unfortunately, this is not a clean cherry-pick for 0.18.0.
Cherry-picking 914b4ce on top of 0.18.0 conflicts with the changes
by aaba6fe which is after the base line for 0.18.0.
@meteorcloudy, please provide guidance on how you want this patch
backported to 0.18.0.
…--
Klaus Aehlig
Google Germany GmbH, Erika-Mann-Str. 33, 80636 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Paul Terence Manicle, Halimah DeLaine Prado
|
@aehlig Thanks for dealing with this. Can you just make sure
is added after |
@aehlig Thanks for dealing with this.
Can you just make sure
```
if (ccToolchain(ruleContext).supportsInterfaceSharedObjects()) {
ccLinkingHelper.emitInterfaceSharedObjects(true);
}
```
is added after `CcLinkingHelper ccLinkingHelper = initializeLinkingHelper(featureConfiguration, deps);`
in CcProtoAspect.java?
The line `CcLinkingHelper ccLinkingHelper = initializeLinkingHelper(featureConfiguration, deps);` does
not exist in `CcProtoAspect.java` of 0.18.0.
Created `0.18.1rc1` as discussed in person with @meteorcloudy.
…--
Klaus Aehlig
Google Germany GmbH, Erika-Mann-Str. 33, 80636 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Paul Terence Manicle, Halimah DeLaine Prado
|
Created `0.18.1rc1` as discussed in person with @meteorcloudy.
After a couple of attempts to backport 914b4ce correctly,
finally arrived at `0.18.1rc3`, see
https://releases.bazel.build/0.18.1/rc3/index.html
Please test carefully!
…--
Klaus Aehlig
Google Germany GmbH, Erika-Mann-Str. 33, 80636 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Paul Terence Manicle, Halimah DeLaine Prado
|
On Wed, Oct 24, 2018 at 01:26:23PM +0200, Klaus Aehlig wrote:
> Created `0.18.1rc1` as discussed in person with @meteorcloudy.
After a couple of attempts to backport 914b4ce correctly,
finally arrived at `0.18.1rc3`, see
https://releases.bazel.build/0.18.1/rc3/index.html
Please test carefully!
This rc fails for the "no JDK" case. The internal CI link
is https://buildkite.com/bazel/bazel-with-downstream-projects-bazel/builds/532#3668026f-a0e0-4d06-ad82-d9d88367347c
@meteorcloudy, is there a way to fix the regression on top of
bazel 0.18.0, or do we have to wait for 0.19.0 with the fix?
…--
Klaus Aehlig
Google Germany GmbH, Erika-Mann-Str. 33, 80636 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Paul Terence Manicle, Halimah DeLaine Prado
|
I think the non-JDK issue is already fixed in 0.19.0 or at least from HEAD. |
Due to some CI change, we cannot test 0.18.0rc3 on our CI. But I ran a test on my local Windows machine that confirmed the manually cherry-pick did fix #6292. |
0.18.1 rc3 pushed to chocolatey. Please could a new release issue be opened for new releases? It's quite easy to miss, otherwise; this ticket is closed. |
On Fri, Oct 26, 2018 at 04:10:42PM +0000, Ittai Zeidman wrote:
Can you please cherry pick e025726 for #6456? Thanks! Cc @cushon
scripts/release/release.sh create --force_rc=4 0.18.1 c062b1f 2834613 2579b79 faaff7f b073a18 bf6a63d c1a7b4c 71926bc 86a8217 914b4ce e025726
…--
Klaus Aehlig
Google Germany GmbH, Erika-Mann-Str. 33, 80636 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Paul Terence Manicle, Halimah DeLaine Prado
|
Thanks!
…On Mon, 29 Oct 2018 at 11:52 Klaus Aehlig ***@***.***> wrote:
On Fri, Oct 26, 2018 at 04:10:42PM +0000, Ittai Zeidman wrote:
> Can you please cherry pick
e025726
for #6456? Thanks! Cc @cushon
scripts/release/release.sh create --force_rc=4 0.18.1 c062b1f 2834613
2579b79 faaff7f b073a18 bf6a63d c1a7b4c 71926bc 86a8217 914b4ce
e025726
--
Klaus Aehlig
Google Germany GmbH, Erika-Mann-Str. 33, 80636 Muenchen
<https://maps.google.com/?q=Erika-Mann-Str.+33,+80636+Muenchen&entry=gmail&source=g>
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Paul Terence Manicle, Halimah DeLaine Prado
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#5963 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABUIF_g0bV8EbCxBk_V-At2gy_g9E1AHks5ups_WgaJpZM4WIVco>
.
|
0.18.1-rc4 is published to chocolatey. |
0.18.1 is available at https://github.com/bazelbuild/bazel/releases/tag/0.18.1
…--
Klaus Aehlig
Google Germany GmbH, Erika-Mann-Str. 33, 80636 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Paul Terence Manicle, Halimah DeLaine Prado
|
0.18.1 is pushed to chocolatey |
|
@aehlig Can you take care of the release 0.18.0?
The text was updated successfully, but these errors were encountered: