Jenkins 2.188 release is stuck

classic Classic list List threaded Threaded
17 messages Options
Reply | Threaded
Open this post in threaded view
|

Jenkins 2.188 release is stuck

Kohsuke Kawaguchi
Administrator
Currently the release of 2.188 is stuck due to a failing jenkins.triggers.TriggerTest. The log is https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4

I'll look into this further later, but if anyone has any thoughts on what might be wrong, that'd be greatly appreciated.

--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAN4CQ4yZGKt8W1Vk8eVZgsSvNLG9N07j24dERj7JX632WP-P-Q%40mail.gmail.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Oleg Nenashev
I was unable to reproduce it locally on Maven 3.6.0 and Windows.
Would it be possible to provide a crashdump?

Thanks in advance,
Oleg


On Tuesday, August 6, 2019 at 6:53:57 AM UTC+2, Kohsuke Kawaguchi wrote:
Currently the release of 2.188 is stuck due to a failing jenkins.triggers.TriggerTest. The log is <a href="https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4" target="_blank" rel="nofollow" onmousedown="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fgist.github.com%2Fkohsuke%2F7ba26b2d9f4a2a49d832610d069d0ab4\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEfJ-GXb-TlbZL6opsqippgMxQsoA&#39;;return true;" onclick="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fgist.github.com%2Fkohsuke%2F7ba26b2d9f4a2a49d832610d069d0ab4\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEfJ-GXb-TlbZL6opsqippgMxQsoA&#39;;return true;">https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4

I'll look into this further later, but if anyone has any thoughts on what might be wrong, that'd be greatly appreciated.

--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/155971a0-1c75-43f1-95db-a8b056529c34%40googlegroups.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Lloyd Chang
> We have it fixed in 2.20.1.
> https://issues.apache.org/jira/browse/SUREFIRE-1408

On Monday, August 5, 2019 at 11:54:30 PM UTC-7, Oleg Nenashev wrote:
I was unable to reproduce it locally on Maven 3.6.0 and Windows.
Would it be possible to provide a crashdump?

Thanks in advance,
Oleg


On Tuesday, August 6, 2019 at 6:53:57 AM UTC+2, Kohsuke Kawaguchi wrote:
Currently the release of 2.188 is stuck due to a failing jenkins.triggers.TriggerTest. The log is <a href="https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4" rel="nofollow" target="_blank" onmousedown="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fgist.github.com%2Fkohsuke%2F7ba26b2d9f4a2a49d832610d069d0ab4\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEfJ-GXb-TlbZL6opsqippgMxQsoA&#39;;return true;" onclick="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fgist.github.com%2Fkohsuke%2F7ba26b2d9f4a2a49d832610d069d0ab4\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEfJ-GXb-TlbZL6opsqippgMxQsoA&#39;;return true;">https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4

I'll look into this further later, but if anyone has any thoughts on what might be wrong, that'd be greatly appreciated.

--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/793e1b80-2efc-41f1-ac89-480661917483%40googlegroups.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Lloyd Chang
clarification: the quoted version number "2.20.1" was Surefire, and not Jenkins.

On Tuesday, August 6, 2019 at 12:36:21 AM UTC-7, Lloyd Chang wrote:
> We have it fixed in 2.20.1.
> <a href="https://issues.apache.org/jira/browse/SUREFIRE-1408" target="_blank" rel="nofollow" onmousedown="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fissues.apache.org%2Fjira%2Fbrowse%2FSUREFIRE-1408\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEXcbmzfupEzO-Rs6-yMS2uBNkRCQ&#39;;return true;" onclick="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fissues.apache.org%2Fjira%2Fbrowse%2FSUREFIRE-1408\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEXcbmzfupEzO-Rs6-yMS2uBNkRCQ&#39;;return true;">https://issues.apache.org/jira/browse/SUREFIRE-1408

On Monday, August 5, 2019 at 11:54:30 PM UTC-7, Oleg Nenashev wrote:
I was unable to reproduce it locally on Maven 3.6.0 and Windows.
Would it be possible to provide a crashdump?

Thanks in advance,
Oleg


On Tuesday, August 6, 2019 at 6:53:57 AM UTC+2, Kohsuke Kawaguchi wrote:
Currently the release of 2.188 is stuck due to a failing jenkins.triggers.TriggerTest. The log is <a href="https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4" rel="nofollow" target="_blank" onmousedown="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fgist.github.com%2Fkohsuke%2F7ba26b2d9f4a2a49d832610d069d0ab4\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEfJ-GXb-TlbZL6opsqippgMxQsoA&#39;;return true;" onclick="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fgist.github.com%2Fkohsuke%2F7ba26b2d9f4a2a49d832610d069d0ab4\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEfJ-GXb-TlbZL6opsqippgMxQsoA&#39;;return true;">https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4

I'll look into this further later, but if anyone has any thoughts on what might be wrong, that'd be greatly appreciated.

--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/1bd559a1-7c52-4a60-91ad-9c2f97ebe8d5%40googlegroups.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Kohsuke Kawaguchi
Administrator
Thanks Lloyd. With Surefire 2.20.1, the error message changes more explicitly to timeout: https://gist.github.com/kohsuke/4d31b8dbcbfe1926e71ccba08f7f9555

The test case in question includes an invocation of 'interactiveBreak', which is surely wrong? When I remove that line, the test passes.

Oleg, how does this test pass for you (and presumably the CI system that gated this change) with the interactiveBreak invocation intact?



On Tue, Aug 6, 2019 at 12:57 AM Lloyd Chang <[hidden email]> wrote:
clarification: the quoted version number "2.20.1" was Surefire, and not Jenkins.

On Tuesday, August 6, 2019 at 12:36:21 AM UTC-7, Lloyd Chang wrote:
> We have it fixed in 2.20.1.

On Monday, August 5, 2019 at 11:54:30 PM UTC-7, Oleg Nenashev wrote:
I was unable to reproduce it locally on Maven 3.6.0 and Windows.
Would it be possible to provide a crashdump?

Thanks in advance,
Oleg


On Tuesday, August 6, 2019 at 6:53:57 AM UTC+2, Kohsuke Kawaguchi wrote:
Currently the release of 2.188 is stuck due to a failing jenkins.triggers.TriggerTest. The log is https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4

I'll look into this further later, but if anyone has any thoughts on what might be wrong, that'd be greatly appreciated.

--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/1bd559a1-7c52-4a60-91ad-9c2f97ebe8d5%40googlegroups.com.


--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAN4CQ4xqPDTq3wvo_5s%2B9Qhx%3DUH6-fvJtPVBf7N-RauNDsMpjw%40mail.gmail.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Kohsuke Kawaguchi
Administrator
I created https://github.com/jenkinsci/jenkins/pull/4142 to get us past this blockage.

On Tue, Aug 6, 2019 at 8:42 AM Kohsuke Kawaguchi <[hidden email]> wrote:
Thanks Lloyd. With Surefire 2.20.1, the error message changes more explicitly to timeout: https://gist.github.com/kohsuke/4d31b8dbcbfe1926e71ccba08f7f9555

The test case in question includes an invocation of 'interactiveBreak', which is surely wrong? When I remove that line, the test passes.

Oleg, how does this test pass for you (and presumably the CI system that gated this change) with the interactiveBreak invocation intact?



On Tue, Aug 6, 2019 at 12:57 AM Lloyd Chang <[hidden email]> wrote:
clarification: the quoted version number "2.20.1" was Surefire, and not Jenkins.

On Tuesday, August 6, 2019 at 12:36:21 AM UTC-7, Lloyd Chang wrote:
> We have it fixed in 2.20.1.

On Monday, August 5, 2019 at 11:54:30 PM UTC-7, Oleg Nenashev wrote:
I was unable to reproduce it locally on Maven 3.6.0 and Windows.
Would it be possible to provide a crashdump?

Thanks in advance,
Oleg


On Tuesday, August 6, 2019 at 6:53:57 AM UTC+2, Kohsuke Kawaguchi wrote:
Currently the release of 2.188 is stuck due to a failing jenkins.triggers.TriggerTest. The log is https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4

I'll look into this further later, but if anyone has any thoughts on what might be wrong, that'd be greatly appreciated.

--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/1bd559a1-7c52-4a60-91ad-9c2f97ebe8d5%40googlegroups.com.


--
Kohsuke Kawaguchi


--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAN4CQ4zrE%3DybktmRMf187nd-EtbWnt6x_OqrmoC85sY1%3D-FHaw%40mail.gmail.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Oleg Nenashev
Oleg, how does this test pass for you (and presumably the CI system that gated this change) with the interactiveBreak invocation intact?

I do not run tests locally when integrating changes, I rely on CI.
And CI was green, not sure why.

The issue itself came from https://github.com/jenkinsci/jenkins/pull/3802 , and it definitely did not get noticed during code reviews.



 

On Tuesday, August 6, 2019 at 5:49:24 PM UTC+2, Kohsuke Kawaguchi wrote:
I created <a href="https://github.com/jenkinsci/jenkins/pull/4142" target="_blank" rel="nofollow" onmousedown="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fgithub.com%2Fjenkinsci%2Fjenkins%2Fpull%2F4142\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNGREBAkLac8tZLVxe9l3MJIHOKQMA&#39;;return true;" onclick="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fgithub.com%2Fjenkinsci%2Fjenkins%2Fpull%2F4142\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNGREBAkLac8tZLVxe9l3MJIHOKQMA&#39;;return true;">https://github.com/jenkinsci/jenkins/pull/4142 to get us past this blockage.

On Tue, Aug 6, 2019 at 8:42 AM Kohsuke Kawaguchi <<a href="javascript:" target="_blank" gdf-obfuscated-mailto="ZJpe8M5bEAAJ" rel="nofollow" onmousedown="this.href=&#39;javascript:&#39;;return true;" onclick="this.href=&#39;javascript:&#39;;return true;">k...@...> wrote:
Thanks Lloyd. With Surefire 2.20.1, the error message changes more explicitly to timeout: <a href="https://gist.github.com/kohsuke/4d31b8dbcbfe1926e71ccba08f7f9555" target="_blank" rel="nofollow" onmousedown="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fgist.github.com%2Fkohsuke%2F4d31b8dbcbfe1926e71ccba08f7f9555\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNGRFM61b1bc_C0B3oQXtbJyXeBCyw&#39;;return true;" onclick="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fgist.github.com%2Fkohsuke%2F4d31b8dbcbfe1926e71ccba08f7f9555\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNGRFM61b1bc_C0B3oQXtbJyXeBCyw&#39;;return true;">https://gist.github.com/kohsuke/4d31b8dbcbfe1926e71ccba08f7f9555

The test case in question includes an invocation of 'interactiveBreak', which is surely wrong? When I remove that line, the test passes.

Oleg, how does this test pass for you (and presumably the CI system that gated this change) with the interactiveBreak invocation intact?



On Tue, Aug 6, 2019 at 12:57 AM Lloyd Chang <<a href="javascript:" target="_blank" gdf-obfuscated-mailto="ZJpe8M5bEAAJ" rel="nofollow" onmousedown="this.href=&#39;javascript:&#39;;return true;" onclick="this.href=&#39;javascript:&#39;;return true;">lloyd...@...> wrote:
clarification: the quoted version number "2.20.1" was Surefire, and not Jenkins.

On Tuesday, August 6, 2019 at 12:36:21 AM UTC-7, Lloyd Chang wrote:
> We have it fixed in 2.20.1.
> <a href="https://issues.apache.org/jira/browse/SUREFIRE-1408" rel="nofollow" target="_blank" onmousedown="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fissues.apache.org%2Fjira%2Fbrowse%2FSUREFIRE-1408\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEXcbmzfupEzO-Rs6-yMS2uBNkRCQ&#39;;return true;" onclick="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fissues.apache.org%2Fjira%2Fbrowse%2FSUREFIRE-1408\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEXcbmzfupEzO-Rs6-yMS2uBNkRCQ&#39;;return true;">https://issues.apache.org/jira/browse/SUREFIRE-1408

On Monday, August 5, 2019 at 11:54:30 PM UTC-7, Oleg Nenashev wrote:
I was unable to reproduce it locally on Maven 3.6.0 and Windows.
Would it be possible to provide a crashdump?

Thanks in advance,
Oleg


On Tuesday, August 6, 2019 at 6:53:57 AM UTC+2, Kohsuke Kawaguchi wrote:
Currently the release of 2.188 is stuck due to a failing jenkins.triggers.TriggerTest. The log is <a href="https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4" rel="nofollow" target="_blank" onmousedown="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fgist.github.com%2Fkohsuke%2F7ba26b2d9f4a2a49d832610d069d0ab4\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEfJ-GXb-TlbZL6opsqippgMxQsoA&#39;;return true;" onclick="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fgist.github.com%2Fkohsuke%2F7ba26b2d9f4a2a49d832610d069d0ab4\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEfJ-GXb-TlbZL6opsqippgMxQsoA&#39;;return true;">https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4

I'll look into this further later, but if anyone has any thoughts on what might be wrong, that'd be greatly appreciated.

--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to <a href="javascript:" target="_blank" gdf-obfuscated-mailto="ZJpe8M5bEAAJ" rel="nofollow" onmousedown="this.href=&#39;javascript:&#39;;return true;" onclick="this.href=&#39;javascript:&#39;;return true;">jenkin...@googlegroups.com.
To view this discussion on the web visit <a href="https://groups.google.com/d/msgid/jenkinsci-dev/1bd559a1-7c52-4a60-91ad-9c2f97ebe8d5%40googlegroups.com?utm_medium=email&amp;utm_source=footer" target="_blank" rel="nofollow" onmousedown="this.href=&#39;https://groups.google.com/d/msgid/jenkinsci-dev/1bd559a1-7c52-4a60-91ad-9c2f97ebe8d5%40googlegroups.com?utm_medium\x3demail\x26utm_source\x3dfooter&#39;;return true;" onclick="this.href=&#39;https://groups.google.com/d/msgid/jenkinsci-dev/1bd559a1-7c52-4a60-91ad-9c2f97ebe8d5%40googlegroups.com?utm_medium\x3demail\x26utm_source\x3dfooter&#39;;return true;">https://groups.google.com/d/msgid/jenkinsci-dev/1bd559a1-7c52-4a60-91ad-9c2f97ebe8d5%40googlegroups.com.


--
Kohsuke Kawaguchi


--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/5bae9334-db08-4753-b893-c8fb8187ccc2%40googlegroups.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Oleg Nenashev
Should we just mark the method as `@Restricted(NoExternalUse.class)` in JTH so that all production build fail by default?
This method is IMO dangerous by default

On Tue, Aug 6, 2019 at 6:04 PM Oleg Nenashev <[hidden email]> wrote:
Oleg, how does this test pass for you (and presumably the CI system that gated this change) with the interactiveBreak invocation intact?

I do not run tests locally when integrating changes, I rely on CI.
And CI was green, not sure why.

The issue itself came from https://github.com/jenkinsci/jenkins/pull/3802 , and it definitely did not get noticed during code reviews.



 

On Tuesday, August 6, 2019 at 5:49:24 PM UTC+2, Kohsuke Kawaguchi wrote:
I created https://github.com/jenkinsci/jenkins/pull/4142 to get us past this blockage.

On Tue, Aug 6, 2019 at 8:42 AM Kohsuke Kawaguchi <[hidden email]> wrote:
Thanks Lloyd. With Surefire 2.20.1, the error message changes more explicitly to timeout: https://gist.github.com/kohsuke/4d31b8dbcbfe1926e71ccba08f7f9555

The test case in question includes an invocation of 'interactiveBreak', which is surely wrong? When I remove that line, the test passes.

Oleg, how does this test pass for you (and presumably the CI system that gated this change) with the interactiveBreak invocation intact?



On Tue, Aug 6, 2019 at 12:57 AM Lloyd Chang <[hidden email]> wrote:
clarification: the quoted version number "2.20.1" was Surefire, and not Jenkins.

On Tuesday, August 6, 2019 at 12:36:21 AM UTC-7, Lloyd Chang wrote:
> We have it fixed in 2.20.1.

On Monday, August 5, 2019 at 11:54:30 PM UTC-7, Oleg Nenashev wrote:
I was unable to reproduce it locally on Maven 3.6.0 and Windows.
Would it be possible to provide a crashdump?

Thanks in advance,
Oleg


On Tuesday, August 6, 2019 at 6:53:57 AM UTC+2, Kohsuke Kawaguchi wrote:
Currently the release of 2.188 is stuck due to a failing jenkins.triggers.TriggerTest. The log is https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4

I'll look into this further later, but if anyone has any thoughts on what might be wrong, that'd be greatly appreciated.

--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/1bd559a1-7c52-4a60-91ad-9c2f97ebe8d5%40googlegroups.com.


--
Kohsuke Kawaguchi


--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/l2vGNx289vE/unsubscribe.
To unsubscribe from this group and all its topics, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/5bae9334-db08-4753-b893-c8fb8187ccc2%40googlegroups.com.

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAPfivLB5R0y3MYOUYHpSzZMsJCOMmUY1XiwtnAWY_b5pHaib0w%40mail.gmail.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Kohsuke Kawaguchi
Administrator
In reply to this post by Oleg Nenashev
I was referring to your attempt to reproduce this earlier this thread.

My guess on why this passes the CI is that it runs in an environment where stdin is /dev/null, which I suspect will make interactiveBreak() a no-op. When I run a release or "mvn test" from a terminal, it actually does block.

If my guess is right, it's probably possible to harden interactiveBreak() to detect that it's reading from /dev/null and fail the test.

On Tue, Aug 6, 2019 at 9:04 AM Oleg Nenashev <[hidden email]> wrote:
Oleg, how does this test pass for you (and presumably the CI system that gated this change) with the interactiveBreak invocation intact?

I do not run tests locally when integrating changes, I rely on CI.
And CI was green, not sure why.

The issue itself came from https://github.com/jenkinsci/jenkins/pull/3802 , and it definitely did not get noticed during code reviews.



 

On Tuesday, August 6, 2019 at 5:49:24 PM UTC+2, Kohsuke Kawaguchi wrote:
I created https://github.com/jenkinsci/jenkins/pull/4142 to get us past this blockage.

On Tue, Aug 6, 2019 at 8:42 AM Kohsuke Kawaguchi <[hidden email]> wrote:
Thanks Lloyd. With Surefire 2.20.1, the error message changes more explicitly to timeout: https://gist.github.com/kohsuke/4d31b8dbcbfe1926e71ccba08f7f9555

The test case in question includes an invocation of 'interactiveBreak', which is surely wrong? When I remove that line, the test passes.

Oleg, how does this test pass for you (and presumably the CI system that gated this change) with the interactiveBreak invocation intact?



On Tue, Aug 6, 2019 at 12:57 AM Lloyd Chang <[hidden email]> wrote:
clarification: the quoted version number "2.20.1" was Surefire, and not Jenkins.

On Tuesday, August 6, 2019 at 12:36:21 AM UTC-7, Lloyd Chang wrote:
> We have it fixed in 2.20.1.

On Monday, August 5, 2019 at 11:54:30 PM UTC-7, Oleg Nenashev wrote:
I was unable to reproduce it locally on Maven 3.6.0 and Windows.
Would it be possible to provide a crashdump?

Thanks in advance,
Oleg


On Tuesday, August 6, 2019 at 6:53:57 AM UTC+2, Kohsuke Kawaguchi wrote:
Currently the release of 2.188 is stuck due to a failing jenkins.triggers.TriggerTest. The log is https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4

I'll look into this further later, but if anyone has any thoughts on what might be wrong, that'd be greatly appreciated.

--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/1bd559a1-7c52-4a60-91ad-9c2f97ebe8d5%40googlegroups.com.


--
Kohsuke Kawaguchi


--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/5bae9334-db08-4753-b893-c8fb8187ccc2%40googlegroups.com.


--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAN4CQ4yFpWt6XsnuYeh1cHqiKVHXV_suT6CeVPrfmM5AzuoFEg%40mail.gmail.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Kohsuke Kawaguchi
Administrator
In reply to this post by Oleg Nenashev
Up to you --- I just came up with another suggestion, but whatever works for you is fine by me.

On Tue, Aug 6, 2019 at 9:12 AM Oleg Nenashev <[hidden email]> wrote:
Should we just mark the method as `@Restricted(NoExternalUse.class)` in JTH so that all production build fail by default?
This method is IMO dangerous by default

On Tue, Aug 6, 2019 at 6:04 PM Oleg Nenashev <[hidden email]> wrote:
Oleg, how does this test pass for you (and presumably the CI system that gated this change) with the interactiveBreak invocation intact?

I do not run tests locally when integrating changes, I rely on CI.
And CI was green, not sure why.

The issue itself came from https://github.com/jenkinsci/jenkins/pull/3802 , and it definitely did not get noticed during code reviews.



 

On Tuesday, August 6, 2019 at 5:49:24 PM UTC+2, Kohsuke Kawaguchi wrote:
I created https://github.com/jenkinsci/jenkins/pull/4142 to get us past this blockage.

On Tue, Aug 6, 2019 at 8:42 AM Kohsuke Kawaguchi <[hidden email]> wrote:
Thanks Lloyd. With Surefire 2.20.1, the error message changes more explicitly to timeout: https://gist.github.com/kohsuke/4d31b8dbcbfe1926e71ccba08f7f9555

The test case in question includes an invocation of 'interactiveBreak', which is surely wrong? When I remove that line, the test passes.

Oleg, how does this test pass for you (and presumably the CI system that gated this change) with the interactiveBreak invocation intact?



On Tue, Aug 6, 2019 at 12:57 AM Lloyd Chang <[hidden email]> wrote:
clarification: the quoted version number "2.20.1" was Surefire, and not Jenkins.

On Tuesday, August 6, 2019 at 12:36:21 AM UTC-7, Lloyd Chang wrote:
> We have it fixed in 2.20.1.

On Monday, August 5, 2019 at 11:54:30 PM UTC-7, Oleg Nenashev wrote:
I was unable to reproduce it locally on Maven 3.6.0 and Windows.
Would it be possible to provide a crashdump?

Thanks in advance,
Oleg


On Tuesday, August 6, 2019 at 6:53:57 AM UTC+2, Kohsuke Kawaguchi wrote:
Currently the release of 2.188 is stuck due to a failing jenkins.triggers.TriggerTest. The log is https://gist.github.com/kohsuke/7ba26b2d9f4a2a49d832610d069d0ab4

I'll look into this further later, but if anyone has any thoughts on what might be wrong, that'd be greatly appreciated.

--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/1bd559a1-7c52-4a60-91ad-9c2f97ebe8d5%40googlegroups.com.


--
Kohsuke Kawaguchi


--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/l2vGNx289vE/unsubscribe.
To unsubscribe from this group and all its topics, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/5bae9334-db08-4753-b893-c8fb8187ccc2%40googlegroups.com.

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAPfivLB5R0y3MYOUYHpSzZMsJCOMmUY1XiwtnAWY_b5pHaib0w%40mail.gmail.com.


--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAN4CQ4wRpd__yNOTb-VTqPr7b0ZMD0ko242NRDWQoR%3Dgy6Dfrw%40mail.gmail.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Jesse Glick-4
In reply to this post by Oleg Nenashev
On Tue, Aug 6, 2019 at 12:12 PM Oleg Nenashev <[hidden email]> wrote:
> Should we just mark the method as `@Restricted(NoExternalUse.class)` in JTH so that all production build fail by default?

Would have no effect—access modifiers are ignored in test sources.

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CANfRfr3DvcBrL4tG7nN6y4y3_LqoWUJpADLB7CKd%3D%2BN2UPWoFQ%40mail.gmail.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Oleg Nenashev
It looks like the new release attempt did not go well
There was https://repo.jenkins-ci.org/snapshots/org/jenkins-ci/main/jenkins-war/2.188-SNAPSHOT/ deployed this night, and the release is not there so far.


On Tuesday, August 6, 2019 at 8:24:32 PM UTC+2, Jesse Glick wrote:
On Tue, Aug 6, 2019 at 12:12 PM Oleg Nenashev <<a href="javascript:" target="_blank" gdf-obfuscated-mailto="G6YCTPOrDgAJ" rel="nofollow" onmousedown="this.href=&#39;javascript:&#39;;return true;" onclick="this.href=&#39;javascript:&#39;;return true;">o.v.n...@...> wrote:
> Should we just mark the method as `@Restricted(NoExternalUse.class)` in JTH so that all production build fail by default?

Would have no effect—access modifiers are ignored in test sources.

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/1053517a-513b-4ba6-92ab-ca5c3abed392%40googlegroups.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Kohsuke Kawaguchi
Administrator
Yep, still investigating. I'm burning 2.188 and moving to 2.189.

On Wed, Aug 7, 2019 at 5:41 AM Oleg Nenashev <[hidden email]> wrote:
It looks like the new release attempt did not go well
There was https://repo.jenkins-ci.org/snapshots/org/jenkins-ci/main/jenkins-war/2.188-SNAPSHOT/ deployed this night, and the release is not there so far.


On Tuesday, August 6, 2019 at 8:24:32 PM UTC+2, Jesse Glick wrote:
On Tue, Aug 6, 2019 at 12:12 PM Oleg Nenashev <[hidden email]> wrote:
> Should we just mark the method as `@Restricted(NoExternalUse.class)` in JTH so that all production build fail by default?

Would have no effect—access modifiers are ignored in test sources.

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/1053517a-513b-4ba6-92ab-ca5c3abed392%40googlegroups.com.


--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAN4CQ4w1MmrvywEDZ1_7_Ni-LrMKahN8omtqnUwvjC8QBomGEg%40mail.gmail.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Kohsuke Kawaguchi
Administrator
I think 2.189 finally went through.

On Wed, Aug 7, 2019 at 7:57 AM Kohsuke Kawaguchi <[hidden email]> wrote:
Yep, still investigating. I'm burning 2.188 and moving to 2.189.

On Wed, Aug 7, 2019 at 5:41 AM Oleg Nenashev <[hidden email]> wrote:
It looks like the new release attempt did not go well
There was https://repo.jenkins-ci.org/snapshots/org/jenkins-ci/main/jenkins-war/2.188-SNAPSHOT/ deployed this night, and the release is not there so far.


On Tuesday, August 6, 2019 at 8:24:32 PM UTC+2, Jesse Glick wrote:
On Tue, Aug 6, 2019 at 12:12 PM Oleg Nenashev <[hidden email]> wrote:
> Should we just mark the method as `@Restricted(NoExternalUse.class)` in JTH so that all production build fail by default?

Would have no effect—access modifiers are ignored in test sources.

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/1053517a-513b-4ba6-92ab-ca5c3abed392%40googlegroups.com.


--
Kohsuke Kawaguchi


--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAN4CQ4we6GswUsX03gAkE%2BinabvJFjHXL2YbsxT5r1qpw1uw7w%40mail.gmail.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Oleg Nenashev
Thanks for the release Kohsuke!
2.188 looks OK according to JIRA and community stats.

Regarding the today's weekly, I think we can skip it. ci.jenkins.io has been unstable since Monday, and I do not feel confident about integrating changes while the maintainers bandwidth is saturated by DW-JW. So I have not merged anything and do not plan to do so until the CI instance stabilizes, sorry to all contributors.

Best regards,
Oleg

On Wednesday, August 7, 2019 at 8:46:28 PM UTC+2, Kohsuke Kawaguchi wrote:
I think 2.189 finally went through.

On Wed, Aug 7, 2019 at 7:57 AM Kohsuke Kawaguchi <<a href="javascript:" target="_blank" gdf-obfuscated-mailto="UiaqG7r7DgAJ" rel="nofollow" onmousedown="this.href=&#39;javascript:&#39;;return true;" onclick="this.href=&#39;javascript:&#39;;return true;">k...@...> wrote:
Yep, still investigating. I'm burning 2.188 and moving to 2.189.

On Wed, Aug 7, 2019 at 5:41 AM Oleg Nenashev <<a href="javascript:" target="_blank" gdf-obfuscated-mailto="UiaqG7r7DgAJ" rel="nofollow" onmousedown="this.href=&#39;javascript:&#39;;return true;" onclick="this.href=&#39;javascript:&#39;;return true;">o.v.n...@...> wrote:
It looks like the new release attempt did not go well
There was <a href="https://repo.jenkins-ci.org/snapshots/org/jenkins-ci/main/jenkins-war/2.188-SNAPSHOT/" target="_blank" rel="nofollow" onmousedown="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Frepo.jenkins-ci.org%2Fsnapshots%2Forg%2Fjenkins-ci%2Fmain%2Fjenkins-war%2F2.188-SNAPSHOT%2F\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEoIxjgbf2ul1QOYHE59sdZdqXKgA&#39;;return true;" onclick="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Frepo.jenkins-ci.org%2Fsnapshots%2Forg%2Fjenkins-ci%2Fmain%2Fjenkins-war%2F2.188-SNAPSHOT%2F\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEoIxjgbf2ul1QOYHE59sdZdqXKgA&#39;;return true;">https://repo.jenkins-ci.org/snapshots/org/jenkins-ci/main/jenkins-war/2.188-SNAPSHOT/ deployed this night, and the release is not there so far.


On Tuesday, August 6, 2019 at 8:24:32 PM UTC+2, Jesse Glick wrote:
On Tue, Aug 6, 2019 at 12:12 PM Oleg Nenashev <[hidden email]> wrote:
> Should we just mark the method as `@Restricted(NoExternalUse.class)` in JTH so that all production build fail by default?

Would have no effect—access modifiers are ignored in test sources.

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to <a href="javascript:" target="_blank" gdf-obfuscated-mailto="UiaqG7r7DgAJ" rel="nofollow" onmousedown="this.href=&#39;javascript:&#39;;return true;" onclick="this.href=&#39;javascript:&#39;;return true;">jenkin...@googlegroups.com.
To view this discussion on the web visit <a href="https://groups.google.com/d/msgid/jenkinsci-dev/1053517a-513b-4ba6-92ab-ca5c3abed392%40googlegroups.com?utm_medium=email&amp;utm_source=footer" target="_blank" rel="nofollow" onmousedown="this.href=&#39;https://groups.google.com/d/msgid/jenkinsci-dev/1053517a-513b-4ba6-92ab-ca5c3abed392%40googlegroups.com?utm_medium\x3demail\x26utm_source\x3dfooter&#39;;return true;" onclick="this.href=&#39;https://groups.google.com/d/msgid/jenkinsci-dev/1053517a-513b-4ba6-92ab-ca5c3abed392%40googlegroups.com?utm_medium\x3demail\x26utm_source\x3dfooter&#39;;return true;">https://groups.google.com/d/msgid/jenkinsci-dev/1053517a-513b-4ba6-92ab-ca5c3abed392%40googlegroups.com.


--
Kohsuke Kawaguchi


--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/98e5b0ea-9462-4cd7-bc65-bf91f7cdf87e%40googlegroups.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Kohsuke Kawaguchi
Administrator
OK. Thanks for the heads-up 

On Sat, Aug 10, 2019 at 23:09 Oleg Nenashev <[hidden email]> wrote:
Thanks for the release Kohsuke!
2.188 looks OK according to JIRA and community stats.

Regarding the today's weekly, I think we can skip it. ci.jenkins.io has been unstable since Monday, and I do not feel confident about integrating changes while the maintainers bandwidth is saturated by DW-JW. So I have not merged anything and do not plan to do so until the CI instance stabilizes, sorry to all contributors.

Best regards,
Oleg

On Wednesday, August 7, 2019 at 8:46:28 PM UTC+2, Kohsuke Kawaguchi wrote:
I think 2.189 finally went through.

On Wed, Aug 7, 2019 at 7:57 AM Kohsuke Kawaguchi <[hidden email]> wrote:
Yep, still investigating. I'm burning 2.188 and moving to 2.189.

On Wed, Aug 7, 2019 at 5:41 AM Oleg Nenashev <[hidden email]> wrote:
It looks like the new release attempt did not go well
There was https://repo.jenkins-ci.org/snapshots/org/jenkins-ci/main/jenkins-war/2.188-SNAPSHOT/ deployed this night, and the release is not there so far.


On Tuesday, August 6, 2019 at 8:24:32 PM UTC+2, Jesse Glick wrote:
On Tue, Aug 6, 2019 at 12:12 PM Oleg Nenashev <[hidden email]> wrote:
> Should we just mark the method as `@Restricted(NoExternalUse.class)` in JTH so that all production build fail by default?

Would have no effect—access modifiers are ignored in test sources.

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/1053517a-513b-4ba6-92ab-ca5c3abed392%40googlegroups.com.


--
Kohsuke Kawaguchi


--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/98e5b0ea-9462-4cd7-bc65-bf91f7cdf87e%40googlegroups.com.
--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAN4CQ4wb6zqDU59thK91E5YZWBQKOoeZ7H4RhZvJWjbSC5edYQ%40mail.gmail.com.
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins 2.188 release is stuck

Oleg Nenashev
I think we can continue weekly releases on Sunday/Monday as usual. We have restored the CI (see https://groups.google.com/forum/#!topic/jenkinsci-dev/v9d-XosOp2s) and integrated a bunch of changes, so we have something for the next weekly releases.


On Sunday, August 11, 2019 at 8:18:24 AM UTC+2, Kohsuke Kawaguchi wrote:
OK. Thanks for the heads-up 

On Sat, Aug 10, 2019 at 23:09 Oleg Nenashev <<a href="javascript:" target="_blank" gdf-obfuscated-mailto="A4eAEjoNEAAJ" rel="nofollow" onmousedown="this.href=&#39;javascript:&#39;;return true;" onclick="this.href=&#39;javascript:&#39;;return true;">o.v.n...@...> wrote:
Thanks for the release Kohsuke!
2.188 looks OK according to JIRA and community stats.

Regarding the today's weekly, I think we can skip it. <a href="http://ci.jenkins.io" target="_blank" rel="nofollow" onmousedown="this.href=&#39;http://www.google.com/url?q\x3dhttp%3A%2F%2Fci.jenkins.io\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNFOz7dbDkaa430NrqJYgXhfPnQC2Q&#39;;return true;" onclick="this.href=&#39;http://www.google.com/url?q\x3dhttp%3A%2F%2Fci.jenkins.io\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNFOz7dbDkaa430NrqJYgXhfPnQC2Q&#39;;return true;">ci.jenkins.io has been unstable since Monday, and I do not feel confident about integrating changes while the maintainers bandwidth is saturated by DW-JW. So I have not merged anything and do not plan to do so until the CI instance stabilizes, sorry to all contributors.

Best regards,
Oleg

On Wednesday, August 7, 2019 at 8:46:28 PM UTC+2, Kohsuke Kawaguchi wrote:
I think 2.189 finally went through.

On Wed, Aug 7, 2019 at 7:57 AM Kohsuke Kawaguchi <[hidden email]> wrote:
Yep, still investigating. I'm burning 2.188 and moving to 2.189.

On Wed, Aug 7, 2019 at 5:41 AM Oleg Nenashev <[hidden email]> wrote:
It looks like the new release attempt did not go well
There was <a href="https://repo.jenkins-ci.org/snapshots/org/jenkins-ci/main/jenkins-war/2.188-SNAPSHOT/" rel="nofollow" target="_blank" onmousedown="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Frepo.jenkins-ci.org%2Fsnapshots%2Forg%2Fjenkins-ci%2Fmain%2Fjenkins-war%2F2.188-SNAPSHOT%2F\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEoIxjgbf2ul1QOYHE59sdZdqXKgA&#39;;return true;" onclick="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Frepo.jenkins-ci.org%2Fsnapshots%2Forg%2Fjenkins-ci%2Fmain%2Fjenkins-war%2F2.188-SNAPSHOT%2F\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEoIxjgbf2ul1QOYHE59sdZdqXKgA&#39;;return true;">https://repo.jenkins-ci.org/snapshots/org/jenkins-ci/main/jenkins-war/2.188-SNAPSHOT/ deployed this night, and the release is not there so far.


On Tuesday, August 6, 2019 at 8:24:32 PM UTC+2, Jesse Glick wrote:
On Tue, Aug 6, 2019 at 12:12 PM Oleg Nenashev <[hidden email]> wrote:
> Should we just mark the method as `@Restricted(NoExternalUse.class)` in JTH so that all production build fail by default?

Would have no effect—access modifiers are ignored in test sources.

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit <a href="https://groups.google.com/d/msgid/jenkinsci-dev/1053517a-513b-4ba6-92ab-ca5c3abed392%40googlegroups.com?utm_medium=email&amp;utm_source=footer" rel="nofollow" target="_blank" onmousedown="this.href=&#39;https://groups.google.com/d/msgid/jenkinsci-dev/1053517a-513b-4ba6-92ab-ca5c3abed392%40googlegroups.com?utm_medium\x3demail\x26utm_source\x3dfooter&#39;;return true;" onclick="this.href=&#39;https://groups.google.com/d/msgid/jenkinsci-dev/1053517a-513b-4ba6-92ab-ca5c3abed392%40googlegroups.com?utm_medium\x3demail\x26utm_source\x3dfooter&#39;;return true;">https://groups.google.com/d/msgid/jenkinsci-dev/1053517a-513b-4ba6-92ab-ca5c3abed392%40googlegroups.com.


--
Kohsuke Kawaguchi


--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to <a href="javascript:" target="_blank" gdf-obfuscated-mailto="A4eAEjoNEAAJ" rel="nofollow" onmousedown="this.href=&#39;javascript:&#39;;return true;" onclick="this.href=&#39;javascript:&#39;;return true;">jenkin...@googlegroups.com.
To view this discussion on the web visit <a href="https://groups.google.com/d/msgid/jenkinsci-dev/98e5b0ea-9462-4cd7-bc65-bf91f7cdf87e%40googlegroups.com?utm_medium=email&amp;utm_source=footer" target="_blank" rel="nofollow" onmousedown="this.href=&#39;https://groups.google.com/d/msgid/jenkinsci-dev/98e5b0ea-9462-4cd7-bc65-bf91f7cdf87e%40googlegroups.com?utm_medium\x3demail\x26utm_source\x3dfooter&#39;;return true;" onclick="this.href=&#39;https://groups.google.com/d/msgid/jenkinsci-dev/98e5b0ea-9462-4cd7-bc65-bf91f7cdf87e%40googlegroups.com?utm_medium\x3demail\x26utm_source\x3dfooter&#39;;return true;">https://groups.google.com/d/msgid/jenkinsci-dev/98e5b0ea-9462-4cd7-bc65-bf91f7cdf87e%40googlegroups.com.
--
Kohsuke Kawaguchi

--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/e667299c-e025-4068-a812-6707be578baf%40googlegroups.com.